sqlserver datetime

Laravel Version: 5.4
PHP Version: 7+
Database Driver & Version:
sql server 2016  php_pdo_sqlsrv_71_ts.dll

Description:
sql server datetime format
Steps To Reproduce:
in migrations : error convert nvarchar to datetime relate fields created_at updated_at
tested patch's
in Illuminate\Database\Schema\Grammars\SqlServerGrammar class
methods  typeDateTime / typeTimestamp  return 'datetime2'
in Illuminate\Database\Query\Grammars\SqlServerGrammar class
public function getDateFormat()
{
return 'Y-m-d H:i:s.0000000';
}

Could you please describe the issue and the steps to reproduce it? It's hard to understand, from what you've posted:

What the issue actually is;
How to reproduce said issue.

on launch db:seed user
[Illuminate\Database\QueryException]
SQLSTATE[22007]: [Microsoft][ODBC Driver 13 for SQL Server][SQL Server]La conversione di un tipo di dati nvarchar in datetime ha generato un valore non com
preso nell'intervallo dei valori consentiti. (SQL: insert into [users] ([name], [email], [password], [remember_token], [updated_at], [created_at]) values (
Ms. Jeanette Lind<EMAIL_ADDRESS>$2y$10$W2YmpsJKE2w6QXg4Z97WYuiy1ZMHBoAdznUPMpP7bFyAkWm3MN9IG, XqwisFf3Em, 2017-05-30 13:24:31.000, 2017-05-30 13:2
4:31.000))

Possible duplicate of: https://github.com/laravel/framework/issues/18991

sql server 2016
setting results at audit login :
set quoted_identifier on
set arithabort off
set numeric_roundabort off
set ansi_warnings on
set ansi_padding on
set ansi_nulls on
set concat_null_yields_null on
set cursor_close_on_commit off
set implicit_transactions off
set language Italiano
set dateformat dmy
set datefirst 1
set transaction isolation level read committed
the dateformat [dmy] conflict with
public function getDateFormat()
{
return 'Y-m-d H:i:s.000';
}
How to not reconfiguring db ?

Closing since it's a duplicate of an old issue.

Hello there, I just came across this problem on my laravel (passport) - sqlserver configuration and I noticed the problem was that when running "php artisan migrate:refresh --seed" or "php artisan passport:install" the error "SQLSTATE[22007]:" appeared for the reason that laravel sends this date format "Y-m-d H:m:s.v" but sqlserver recevies "Y-d-m H:m:s.v" so I change the order of the format from 'Y-m-d H:m:s.v' to 'Y-d-m H:m:s.v' on SqlServerGrammar.php file
public function getDateFormat()
{
return 'Y-d-m H:m:s.v';
}
Worked from me!
Im commeting here too just in case

Hello there, I just came across this problem on my laravel (passport) - sqlserver configuration and I noticed the problem was that when running "php artisan migrate:refresh --seed" or "php artisan passport:install" the error "SQLSTATE[22007]:" appeared for the reason that laravel sends this date format "Y-m-d HⓂs.v" but sqlserver recevies "Y-d-m HⓂs.v" so I change the order of the format from 'Y-m-d HⓂs.v' to 'Y-d-m HⓂs.v' on SqlServerGrammar.php file
public function getDateFormat() { return 'Y-d-m HⓂs.v'; }
Worked from me! Im commeting here too just in case

This Work Well, thanks so much 👍


When launches a saved game

When I launch in a saved game the progress goes down to zero  and i have to redo the selecting of the mining sites. I also have other mods that could affect it like Resource overhaul or rso. i goes down and says that 1320000m/s on the tacker so I don't know how to fix it

Sorry, Factorio 0.12.13 and 0.12.14 both have this issue: http://www.factorioforums.com/forum/viewtopic.php?f=7&t=17275.
Not a YARM bug, I'm afraid.

thank you for helping me i will wait for the update


Helm - bump version

what

use helm 2.9.1

why

#155

Move to https://github.com/cloudposse/geodesic/blob/master/Dockerfile#L44


176 migration guide and data table example

There are currently no links to the publications table but going to /publications will display them. need to have your server running to get publications.
This does handle pagination, sorting and searching.
Unfortunately there isn't great test coverage ~63% of the AdminTable (still missing sorting/pagination tests).
I figured getting this merged will allow others to at least start on building the admin table views.

Thank you. I've created a PR for the updates to the migration guide:
https://github.com/Call-for-Code-for-Racial-Justice/Truth-Loop/pull/220
On Wed, Oct 13, 2021 at 11:07 AM Mark Sturdevant @.***>
wrote:

@.**** approved this pull request.
This looks great. Contributing doc still says Carbon and has a todo, but
I'd like to get this code in place so I'll merge it now. Thanks, Ed!
In contributing/ADMIN-COMPONENT-MIGRATION.md
https://github.com/Call-for-Code-for-Racial-Justice/Truth-Loop/pull/214#discussion_r728188948
:

+## Vue to React
+
+The <template></template> portion of the vue component is the basically what will make up your JSX.
+- Tags will be different
+- The class attribute will be className
+- There will be other tweaks necessary as JSX does not like hyphens (-) in attributes
+- The CarbonComponent design system will be used: https://react.carbondesignsystem.com/?path=/story/getting-started--welcome
+
+The data() function will primarily be the props that you pass in or the state values that you set.
+
+In the mounted() function you'll find what needs to be executed immediately.
+
+The methods object contains the functions that may be executed within this component or passed to a sub-component.
+
+(TODO: Add the code excerpts of turning the Vue Publication* files into the corresponding React components. To include tests.)
TODO: there is a TODO
In contributing/ADMIN-COMPONENT-MIGRATION.md
https://github.com/Call-for-Code-for-Racial-Justice/Truth-Loop/pull/214#discussion_r728190072
:

@@ -0,0 +1,45 @@
+# Admin UI component migration from Vue to React

+## Vue to React
+
+The <template></template> portion of the vue component is the basically what will make up your JSX.
+- Tags will be different
+- The class attribute will be className
+- There will be other tweaks necessary as JSX does not like hyphens (-) in attributes
+- The CarbonComponent design system will be used: https://react.carbondesignsystem.com/?path=/story/getting-started--welcome
you removed Carbon
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Call-for-Code-for-Racial-Justice/Truth-Loop/pull/214#pullrequestreview-778750327,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASQF7KDN5GCWRGB2VTMYG3UGWVCXANCNFSM5F4HLJHQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Pin hash for GitHub actions in workflow yml files

Fixes: #81 #82
Pin hash for GitHub actions in workflow yml files.
I checked the latest stable versions for each of our action dependencies and got the commit hashes.
I have done a check for security advisories for each of the actions, but haven't found any.
The versions I have used for each of the actions are as follows:

actions/checkout v3.1.0
actions/setup-python v4.3.0
docker/setup-qemu-action v2.1.0
docker/setup-buildx-action v2.2.0
docker/login-action v2.1.0
docker/build-push-action v3.2.0

Signed-off-by: Martin Vrachev<EMAIL_ADDRESS>

@kairoaraujo sorry that I have taken issue #81 as you have assigned yourself, but when I did the work for #82 it just made sense to do the same for #81 as well.

I updated the changes to include the github/workflows/ci.yml and .github/workflows/cd.yml workflow files as well as they use almost the same third-party dependencies as the other two.

Released on top of latest dependabot changes


Ability to choose the installation folder

Feature request
It seems that the current installation folder is C:\Users[user]\AppData\Local\lxss according to #402, while one might want to install it on another drive or folder. (to save space or whatever)
A command switch or dialog to specify the installation folder would be nice.

Duplicate of #449.

Oh right sorry, I didnt find it when searching


Allow PHP 8

The upstream package https://packagist.org/packages/ramsey/uuid#4.0.0 allows php 8, but it need not bother, if it's dependencies don't allow it. So, either we should remove php 8 from the uuid package, or add it in here. ;)

Thanks!


Some key bindings conflict with those of the window manager

VSCode Version: 1.0.0
OS Version: Ubuntu 14.04 LTS, Cinnamon desktop environment

Steps to Reproduce:

Open more than one files on the side (so that the editor ends up with multiple columns)
Try using Ctrl+Alt+Shift+<Left/Right> to switch between editors

Expected behavior:
Editors are switched
Observed behavior:
Cinnamon intercepts the shortcut and switches workspaces instead
Is it possible to configure the editor's key bindings manually?

I do not think we can overwrite any of the OS taken keys. @alexandrudima ?

Each desktop environment "eats" different keybindings, it is almost impossible for us to have defaults that work in all Linux desktop environments.
@10098 I recommend customizing the keybindings via File > Preferences > Keyboard Shortcuts


Question: Is zort's timSort() implementation correct?

Hi Ali,
I have just read this article: http://www.envisage-project.eu/proving-android-java-and-python-sorting-algorithm-is-broken-and-how-to-fix-it/
Is Your implementation affected or You were aware of this issue?

It seems like the timsort in this repository is incorrect close to 100% of the time, which is probably due to a different bug.

yes current tim sort is completely wrong and have to be rewritten like main Java implementation

I would like to give it a go. Assigned to myself, if somebody already working on this speak up!


Resolve snapshot indices on cluster state update

Relates to https://github.com/crate/crate/issues/12598
Avoid sending GetSnapshotsRequest in the planner to avoid race condition.
Also, we operate now with table/partition in RestoreService, will do all renaming, including schema renaming in the restore cluster state update.
Old code is kept in planner for BWC. Index resolving code is not reused but somehow replicated in the RestoreService since it's not 100% same and BWC code in planner is anyway subject of removal

or does the logical replication hit the BWC paths anyway?

Yep, we now operate everywhere with resolved indices/templates we resolve. And in case of LogicalReplication we don't actually resolve indices but fallback to the request values.


[SPARK-8455][ML] Implement n-gram feature transformer

Implementation of n-gram feature transformer for ML.

Merged build triggered.

Merged build started.

Test build #35173 has started for   PR 6887 at commit fe93873.

Test build #35173 has finished for   PR 6887 at commit fe93873.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class NGram(override val uid: String)

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #35214 has started for   PR 6887 at commit 9fadd36.

Test build #35214 timed out     for PR 6887 at commit 9fadd36     after a configured wait of 175m.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #35288 has started for   PR 6887 at commit d2c839f.

Test build #35288 has finished for   PR 6887 at commit d2c839f.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class NGram(override val uid: String)

Merged build finished. Test FAILed.

Jenkins test this please

Merged build triggered.

Merged build started.

Test build #35404 has started for   PR 6887 at commit d2c839f.

Test build #35404 has finished for   PR 6887 at commit d2c839f.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class NGram(override val uid: String)

Merged build finished. Test PASSed.

2 thoughts:

@mengxr suggested that for a sequence of length < n, we return nothing.  That does not seem ideal since it throws out information.  (I would be surprised if I applied a transformer and got back empty sequences.)  Using the default behavior of Scala's grouped seems better to me.
(future) In general, people will want to apply: Tokenizer, NGrams, HashingTF.  Later on, we should provide something which handles this directly, rather than creating a bunch of intermediate objects.

I actually think @mengxr suggestion makes sense; it guarantees that the output is an actual n-gram rather than being dependent on whether the input length > n or not.
Another possibility could be to pad with some null word. The use cases I am imagining is a n-gram HMM language model, in which case partial sentences with k words (k < n) should be represented by padding null words on the right.

Yeah, I guess I'm OK either way.  I figure the normal user won't care if it's a strict n-gram but might be upset if a non-empty document produces an all-zero feature vector.

LGTM  Merging with master
Thanks!


Ferrous holopad updates

cc @PursuitInAshes
Ferrous needs holopads and a possibly AI? We have not played it in a while and people keep saying it is out of date. If you know what needs to be done with it let me know and I can edit this.

Ferrous is out of date but not substantially, the following changes need to be made for Ferrous to be brought in line again:
AI Core (Make not broken and add AI)
This means making sure the cameras are well placed around the map too.
Holopads for the AI and In General
Roundstart Ripley
Station Anchor
Event Shuttle Dock
Jobslot Console


Change/delete motion support

Are there any plans to add support for ct<CHAR>/dt<CHAR> for this plugin?

It works already :)

Oh... I've had it configured only for visual mode apparently... 🤦
I will gladly close this! Just another question, is there any plans for repeat support?
E.g.
|Plug('ggandor/flit.nvim')
-- enter: "ctg"
|   t  sf

-- enter: "t123"
123|g('ggandor/flit.nvim')

-- enter "."
12123|g('ggandor/flit.nvim')

I mean, I am not sure what should happen if you do a dot-repeat, if multiple matches exist on the same line.

It's a bit weird to me, maybe showing the labels on repeat would be better (at least in some cases). Not sure if it's possible to implement with repeat.vim, tho. :)


No ESLint configuration (e.g .eslintrc) found for file... -  Set eslint config in file comment instead of jsonrc file

JSHint extension settings enable us to add
`/*jshint esversion: 6 */
In the js file. Could we do the same with eslint instead of having to create a separate file?`

Such a support must come from the ESLint npm module. The ESlint extension doesn't add any features on top of the normal ESLint. It only integrates ESLint into VS Code. Having the feature in ESLint npm will also ensure that it works correctly when executing eslint on the command line.
Can you please file the request here: https://github.com/eslint/eslint
Please ping if I misunderstood your request.

Thx. Discussion with ESLint opened https://github.com/eslint/eslint/issues/10877

ESLint maintainer here :smile:
Hi @giorgio79, I took a look at the documentation here and it looks like there is an eslint.options configuration setting which takes options that go to CLIEngine.
Try adding the following option there:
{
    "useEslintrc": false
}

That should allow you not to use a config file, and then you can just use the configuration comments (as we discussed in the ESLint issue).
I don't think anything needs to be done either on the ESLint or vscode-eslint side.

@platinumazure thanks for clarifying this.


Amend minor errors in UG and java docs/comments

Codecov Report

Merging #261 (b814eb7) into master (b11cb19) will increase coverage by 0.44%.
The diff coverage is 100.00%.

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


Sampling from MCMCPosterior fails with num_workers > 1

Hello,
I am trying to sample from an MCMCPosterior using multiple cpu cores because I would like to make the sampling faster, however, I get an error when I try to set num_workers > 1. I've taken the example from the sampler interface tutorial on the sbi website and added an argument for num_workers=2 and reproduced my error.
# Setup from sbi sampler interface tutorial
num_dim = 2
prior = torch.distributions.MultivariateNormal(torch.zeros(num_dim), torch.eye(num_dim))
theta = prior.sample((1000,))
x = theta + torch.randn((1000, num_dim))
x_o = torch.randn((1, num_dim))

inference = SNLE(show_progress_bars=False)
likelihood_estimator = inference.append_simulations(theta, x).train()

potential_fn, parameter_transform = likelihood_estimator_based_potential(likelihood_estimator, prior, x_o)
posterior = MCMCPosterior(potential_fn, proposal=prior, theta_transform=parameter_transform, num_workers=2)
samples = posterior.sample((100,), x=x_o)

Here is the error that I get:
BrokenProcessPool: A task has failed to un-serialize. Please ensure that the arguments of the function are all picklable.
Please let me know if you have a fix or if you have other advice on how to speed up the sampling process.
Thank you!
Lindsey

Hi Lindsey,
the code you sent works for me on Linux (sbi on the bleeding edge github versionm, torch 1.12.0). I guess you are using sbi v0.18.0? What operating system are you using? What version of torch?
Best wishes
Michael

and what are your version of pickle and joblib?

Aside: I'm not so sure that using multiple workers gives you a big speedup as compared to using the vectorized slice_np: method=slice_np_vectorized and num_chains=100.

Thanks for the quick response. Super sorry, I realized my error was because I had forgotten to import pickle. It is working for me now. Thanks for the advice on using a different method and number of chains instead!


Add granular reactivity that matches Svelte's reactivity model

We don't have to use the same reactivity model as React, as we have deep-reactivity supported as part of Svelte's runes.
We can deviate from jazz-react here.

Looking at the Svelte docs related to the runes: https://svelte.dev/docs/svelte/$state#Deep-state
Seems to me that to handle fine-grained reactivity we need:

To get rid of our proxies (should come with the next API iteration)
To build a subscribe system that mutates the nested values instead of regenerating them from scratch. We can start by aiming to object-level reactivity by hacking with the subscription scope. We actually already do nested subscriptions there so achieving this should be fairly easy.
In the future we can also aim to property level reactivity for specific classes (e.g. CoMap) we just need a way to notify which keys have changed in each update.

This highlights that Jazz can be actually very good at this, making fine-grained reactivity quite effortless for the end-users.


Support for scale = 0.003?

I'm using this tool (with https://github.com/SolidLabResearch/css-populate) to generate data for benchmarking CSS. The default setting works correctly for me.
But during testing, I often found the data too large. I then found https://github.com/ldbc/ldbc_snb_datagen_spark/pull/146 which added another scale factor (scale) value 0.003.
But when trying to use 0.003 on this tool, it fails. After manually modifying CommandGenerate to accept this value as argument, it still fails.
Is there a way to make it work?

LDBC SNB unfortunately stopped supporting Turtle as output (https://github.com/ldbc/ldbc_snb_datagen_spark/issues/228). So we're using an older version of their generator, which does not seem to include that scale factor yet.
So there's no way to make this work I'm afraid, for this we'd need Turtle support in LDBC SNB.

I see. That's unfortunate...
Do you have any suggestions on where I should look to find information on (controlling) the output format for LDBC generator?

This is the place I think: https://github.com/ldbc/ldbc_snb_datagen_spark
One option is to create an issue on that issue tracker to ask for help.


Added PermissionsBoundary and FunctionRole optional parameters

PermissionsBoundary allows additional security control when the new role is created for this Lambda
FunctionRole parameter allows customer to provide a predefined role with required privileges

Thanks for the clarification @haihongren, I've made the changes and will be merging and publishing this in a few.


dev-cmd/pr-publish: pass --autosquash by default

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

It's too easy to forget to, and the consequences of forgetting to pass
--autosquash are worse (i.e. almost always permanent) than forgetting
to pass --no-autosquash (i.e. just do pr-publish --no-autosquash
when your command fails).
See the discussion at Homebrew/homebrew-core#80717.

Review period will end on 2021-07-20 at 00:00:00 UTC.

We should have the Homebrew/core PR ready to go for when this is merged. Do you mind taking care of that?

Yep. One thing: we still have pr-automerge with this option too.

Hmm, yes, I think we want to make --autosquash the default for pr-automerge too. I don't think we want that for pr-pull; one common use case for pr-pull is to have finer control than pr-publish over how things get squashed. You also get the chance to examine what you're pushing to master with pr-pull unlike with pr-publish.

But we would still have a finer control. Just with --no-autosquash option. Autosquash will fail anyway if there are multiple commits modifying multiple formulae and I believe that would be the root factor of one running it locally and trying to fixup some commits manually.

I dunno, I just think I almost always want --no-autosquash when doing pr-pull, but almost always want --autosquash when doing pr-publish or pr-automerge.
That said, I already have aliases that include --autosquash for me when doing pr-publish, so I guess I can do the same with pr-pull, so if you feel strongly about making the same changes to pr-pull then I don't mind.

Review period ended.

I wonder if this is still a breaking change that we need to worry about.

I don't know; I'm not too worried about it. Our tap-new templates only make use of pr-pull, which I haven't touched (though I think @dawidd6 has a different opinion here), so most third-party taps should be unaffected by these changes. The ones that will be are probably managed by users who were adventurous enough to go digging in our codebase (or were mindlessly copying Homebrew/core workflows...), so I think they'll manage. (Except for the mindless copiers. But I have to say I'm not too sympathetic here.)

Okay. Last thing: we usually wait to add the odeprecated until the next major/minor version is going to be released. Should we do that here, too? If so, that would probably mean waiting on this PR, though

No opinion here. I don't think this is urgent, so we can wait till the next release tag. I'm happy to defer to your or @MikeMcQuaid's opinion on this one, though.

Given this is "developer command that I'd be surprised if it's even used by Not Us": I think it's fine to merge as-is, personally.

Ok, great. Merging this and Homebrew/homebrew-core#81464 then.


aclpub2/generate hangs up

Hi,
Following Danilo's advice, I updated the aclpub2 version and changed the conference_details.yml from
name: Workshop on Computational Approaches to Subjectivity, Sentiment \& Social Media Analysis
volume: Proceedings of the Twelfth Workshop
abbreviation: WASSA
start_date: 2022-05-26
end_date: 2022-05-26
isbn:
location: Dublin, Ireland
publisher: Association for Computational Linguistics

to
book_title: "Proceedings of the 12th Workshop on Computational Approaches to Subjectivity, Sentiment & Social Media Analysis (WASSA 2022)"
event_name: "The 12th Workshop on Computational Approaches to Subjectivity, Sentiment & Social Media Analysis"
cover_subtitle: Proceedings of the Workshop
anthology_venue_id: WASSA
start_date: 2022-05-26
end_date: 2022-05-26
isbn: XXX-X-XXXXXX-XX-X (you should replace this with the real ISBN)
location: Dublin, Ireland
editors:
  - first_name: Jeremy
    last_name: Barnes
  - first_name: Orphée
    last_name: De Clercq
  - first_name: Valentin
    last_name: Barriere
  - first_name: Shabnam
    last_name: Tafreshi
  - first_name: Sawsan
    last_name: Alqahtani
  - first_name: João
    last_name: Sedoc
  - first_name: Roman
    last_name: Klinger
  - first_name: Alexandra
    last_name: Balahur
publisher: Association for Computational Linguistics

However, when I run ./bin/generate ../Wassa --proceedings --overwrite to generate the proceedings, the script hangs up after processing 3 papers. It doesn't give any error that I can find. Any advice?

I can dowload the scripts from the shared repository.
Can you share with me the PDFs with a private message?

shared via email

It was caused by the presence of "&" in the book_titlethat was not correctly escaped.
Now I fixed it and it should work.
Please update up to the last commit of the main branch.
Two final comments:

We added the possibility to provide a customized watermark. It can be useful (as in your case) for long book titles. You just need to add the optional field in the conference_details.ymllike

watermark_book_title: Proceedings of the 12th Workshop on Computational Approaches to\\ Subjectivity, Sentiment & Social Media Analysis (WASSA 2022)

Your program.yml is malformed. The program will skip it when loading that file as it would not be able to compile.

Does it fix your problem?

Thank you, Danilo. It now compiles correctly. I haven't completed the program.yml yet.


TypeError: Cannot read property 'properties' of undefined

Environment infos
node -v -> v10.14.2
npm -v -> 6.4.1
OS: Ubuntu 18.04 LTS
What I did
I tried to start the dev-server by doing:
npm i
npm start

can you reproduce this?
What I get
Here is the complete log, where I also used npm audit fix. After that I could reproduce without the audit fix though
potatoUser@potatoComputer riot-webpack-starter-master $ npm i
npm WARN<EMAIL_ADDRESS>No repository field.

added 992 packages from 529 contributors and audited 15563 packages in 22.575s
found 32 moderate severity vulnerabilities
  run `npm audit fix` to fix them, or `npm audit` for details

potatoUser@potatoComputer riot-webpack-starter-master $ npm audit fix
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

+<EMAIL_ADDRESS>updated 2 packages in 13.277s
fixed 26 of 32 vulnerabilities in 15563 scanned packages
  1 package update for 6 vulns involved breaking changes
  (use `npm audit fix --force` to install breaking changes; or refer to `npm audit` for steps to fix these manually)

potatoUser@potatoComputer riot-webpack-starter-master $ npm start

><EMAIL_ADDRESS>start /path/to/Repos/riot-webpack-starter-master
> webpack-dev-server --inline --watch --hot --colors -d --port 3000

/path/to/Repos/riot-webpack-starter-master/node_modules/webpack-cli/bin/config-yargs.js:89
                                describe: optionsSchema.definitions.output.properties.path.description,
                                                                           ^

TypeError: Cannot read property 'properties' of undefined
    at module.exports (/path/to/Repos/riot-webpack-starter-master/node_modules/webpack-cli/bin/config-yargs.js:89:48)
    at Object.<anonymous> (/path/to/Repos/riot-webpack-starter-master/node_modules/webpack-dev-server/bin/webpack-dev-server.js:84:40)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
    at Function.Module._load (internal/modules/cjs/loader.js:530:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:742:12)
    at startup (internal/bootstrap/node.js:282:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:743:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `webpack-dev-server --inline --watch --hot --colors -d --port 3000`
npm ERR! Exit status 1

ok I could get it running by reinstalling the latest version of webpack-cli
This is the answer on so that helped me figure it out: https://stackoverflow.com/a/52795175/3313410
This is now my dependency list:
{
  "name": "riot-plus-webpack",
  ...
  "devDependencies": {
    "babel-core": "^6.26.0",
    "babel-loader": "^7.1.3",
    "babel-preset-es2015-riot": "^1.1.0",
    "riot-hot-reload": "^1.0.0",
    "riot-tag-loader": "^2.0.2",
    "webpack": "^4.28.0",
    "webpack-cli": "^3.1.2",
    "webpack-dev-server": "^3.1.0"
  },
  "dependencies": {
    "riot": "^3.9.0"
  }
}


Update Python version from 3.6 to 3.9

We should update Python from 3.6 to 3.9 (which is supported by Rocky8).

Fully in agreement

Ansible 2.9 is compatible up to Python 3.8. If we go with Python 3.9 we'll have to drop support for Ansible 2.9 (which is ok if we upgrade Ansible to 2.15 #774).

From my testing, Ansible 2.9 work with an Ansible controller with python 3.9. There is nothing to do for me.

Shouldn't we at least update the README?


Egress hidden participant sometimes appears in LiveKitRoom component

Describe the bug
We are using the react components and some egresses in the backend. Some times (let’s say 1/3 times), we have the egress showing up in the react LiveKitRoom component with name like EG_xxxxx. This seems to happen often when an ingress (ogg/ivf via livekit-cli) leaves the call.
Server

Version: see below
Environment: Kubernetes/RKE2

livekit/egress:v1.5.1
livekit/livekit-server:v1.3.1
livekit/livekit-cli:v1.2
Client

SDK: js
Version: see below

"@livekit/react-components”: “^1.0.3"
“livekit-client”: “^1.5.0"
“livekit-server-sdk”: “^1.1.0".
To Reproduce
Not always happening, but often in this scenario:

web client (only 1) connect the room
start egress
start ingress
ingress stops
EG_xxxx shows up

Expected behavior
No egress participant visible.
Screenshots

Additional context
https://livekit-users.slack.com/archives/C01KVTJH6BX/p1668580708737279

If you are using a custom template, ensure you are not rendering the local participant itself. See this example as to how we are handling it.


"Recipient time zone" based push notifications not working

I followed these steps:
a. cloned the Messaging example project
b. changed the bundle id to my own (that's already set to a Firebase console app)
c. ran the app on a real device and accepted the permissions request
I then created a couple of notifications similar to this one yesterday:

And I didn't receive any of them.
I successfully received "Single device" and "Send Now" push notifications.

Could you confirm that you did not uninstall the app after sending the notification. The scheduled notifications are scheduled locally and if the application is uninstalled the local notification would go with it.

So the data notifications are sent immediately to the existing devices, even if I schedule it to be sent the next week?

So, a new user that installs the app after a push notification is scheduled, will receive it or not?

The message is immediately sent to the device, but not displayed to the user until the scheduled time.

@revolter correct, only users at the time of the scheduling will receive the notification.

@kroikie, but, if the users at the time of the scheduling don't open the app at all, will they receive it?

Yes, once the users are able to receive "now" notifications then they will be able to receive scheduled ones.

But this documentation page states that:

On iOS, FCM stores the message and delivers it only when the app is in the foreground and has established a FCM connection.

That is referring to data messages. Scheduled messages (as all messages from the console) are notification messages sent through APNs.

But doesn't that mean that the users will receive a notification immediately after we set it in the console, and then the scheduled local notification?

Yes that is correct. The device will receive the notification (not displayed) and then a local notification would be scheduled.

How can you receive a notification which is not displayed? Resty from the support told me that "You can create a notification message and a notification + data message only, not data message alone".

The message will be sent with the content_available field set to true, this will make a silent notification. Once received silently it is then scheduled.

Ooh, I see. Is the content_available field supported on iOS 7+?

Yes.

@kroikie, Ran some tests this weekend, and it looks like indeed it's not working well.
I did this:

cloned this project

changed the Messaging project's bundle it to my own

added my own GoogleServices.plist file to it

ran the app on one of my devices

accepted the notifications permission request

closed the app

ran my own app (that has the same bundle id) on another device

accepted the notifications permission request

closed the app

So I had 2 apps with the same bundle id, so they'll be treated as the same, one being the sample (with swizzling enabled, of course) and the other one being my code with my own Firebase integration (with swizzling disabled).
I then went to the console and scheduled 3 pushes for Saturday, but none was received.
I opened the apps then backgrounded them and scheduled 9 pushes for Sunday. This time, the Quickstart app received all but the first 2 pushes, and my app received all of them.
I then closed the apps and scheduled 11 pushes for Monday. Up until now, none was received.
The Quickstart app was run on an iOS 10.1.1 iPhone 5S and my app was run on an iOS 10.2 iPhone 6S.

Is the Background fetch mode needed for this to work? Or only the Remote notifications mode?

I have the same issue in my Unity3D project.

Still face the same issue, any ideas on solving it?

Is this STILL an issue? Can someone please help? Its 2024! My Flutter iOS app does not receive these scheduled "recipient time zone" notifications.
This thread seems to suggest some fixes back in 2021 when this thread was still active - https://github.com/firebase/firebase-ios-sdk/issues/8172
But I still don't receive these notifications on iOS.


Spherical finite volume

Description
Fixes a bug in finite_volumes.py for spherical diffusion.
(r_edge[i+1]**3 - r_edge[i]**3)/3 is now used instead of r_node**2. This is what it should be according to Finite Volume theory e.g. https://journals.aps.org/prapplied/abstract/10.1103/PhysRevApplied.6.064010

Previous convergence:

New convergence:

Code:
import pybamm
import numpy as np

model = pybamm.lithium_ion.DFN()

chemistry = pybamm.parameter_sets.Ecker2015
parameter_values = pybamm.ParameterValues(chemistry=chemistry)

var = pybamm.standard_spatial_vars
npts = [10, 20, 25, 30, 40, 50]

solver = pybamm.CasadiSolver(mode="safe")
t_eval = np.linspace(0, 3800, 200)

sols = []
for N in npts:
    var_pts = {var.x_n: 20, var.x_s: 20, var.x_p: 20, var.r_n: N, var.r_p: N}
    sim = pybamm.Simulation(
        model,
        solver=solver,
        parameter_values=parameter_values,
        var_pts=var_pts,
        # C_rate=5,
    )
    sol = sim.solve(t_eval=t_eval)
    print("N={}: {} s".format(N, sol.solve_time))
    sols.append(sol)

pybamm.dynamic_plot(sols, ["Terminal voltage [V]"], time_unit="seconds", labels=npts)

Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[x] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Since the convergence is better, we could reduce the default number of r points to 20 (10 still gives inaccurate results at higher C-rates). See #1075

How did you come up with 150 points for the ORegan parameter set
before:

now:

?

There were some issues with the convergence with the low diffusion areas, but maybe with the final parameter set is not needed any more. If it converges, happy to reduce the number of points 😄


Refactoring the Management endpoints

The discussion on #503 has drifted from the original topic. This issue has been created for the general discussion on management endpoints so that the original issue can resume it's original duty.
My thoughts on improving the management endpoint features can be broken down into the following phases.
Phase 1 is to get #543 merged since that makes testing easier.
Phase 2 is to update the tests so that DI is the primary mechanism for creating objects. This allows us to refactor the internals without breaking too many tests. A lot of tests are currently using TestHost and could be updated to use WebApplicationFactory that was created by Microsoft for testing purposes; wraps TestHost and sets up other things.
Since we are updating tests, it allows us to fix #542. I have an approach that doesn't depend on external libraries that works for most cases; depends on how much you want to analyze the logging output during the test.
Phase 3 is to update the internals of the management endpoint code. I have some ideas on how this can be achieved. One idea I'm toying around with is using an EndpointDataSource for the actuator and CloudFoundry endpoints.

I'm looking at this from a .NET Core perspective. Great that there was support for .NET Framework, but long term, Core will be the only game in town. Based on that, there are things that should be done to align it with how .NET Core does things. The big challenge is how do we get there, and how can we incrementally make changes that don't break everything. At some point in the future, the code will just have to accept that .NET Framework is dead and buried. As long as EndpointBase is constrained by history, it restricts what type of changes can be made. When that time comes, I don't know.
I have played around with several ideas on what can be changed, but usually hit a dead end that would cause too many changes that are hard to track or become a binary breaking change. I'm trying to find small changes that don't break things completely and can be used to aid the long term goals of the project; makes it easier to merge between branches. This to delay the split from .NET Framework as long as possible.
So what types of changes would I like to make?
The code contains a lot of Option objects. For .NET Core developers, that comes with a lot assumptions since Microsoft has the IOptions pattern. It would not be unreasonable for .NET Core developers to think that they could leverage all the possibilities of the IOptions pattern. i.e. Configure() and PostConfigure().
Since Steeltoe isn't using this pattern, it may become a developer surprise. I couldn't find anything in the documentation stating that IOptions is not supported. The options pattern is to expose configuration values consumers can change. The Options classes in Steeltoe contain options that can't be changed by consumers; AllowedVerbs and ExactMatch. Another small surprise.
Another problem, depending on your point of view, with the codebase is that everything is public. A lot of the classes will most likely not be used by consumers, so some changes probabaly wouldn't result in broken builds. But since they are public, it's not hard to make a breaking change. Long term things should be made internal to make it easier to change implementation details without impacting the public API surface.
Lets assume some endpoint is currently depending on A, B, and C. Then sometime in the future, B should be changed to D and a new dependency on E is added. This is a breaking change for consumers, but it also results in a lot of tests changing just to get things to build.
This is where IoC containers come into play. Rather than newing up every object explicitly, use the container to create them.
Over time the dependencies for each service can change without impacting the tests.
In the tests, something like this can be achieved
[Fact]
public async Task SomeTest()
{
	// the TestContext is used to set up the IServiceCollection
	// some things like Logging and Options infrastructure is added by default
	using(var tc = new TestContext())
	{
		// each test can add additional dependencies specific for the test
		tc.Services.AddEnvAcutatorServices();

		var endpoint = tc.GetService<IEnvService>();

		var response = await endpoint.Invoke();

		Assert.NotNull(response);
	}
}

Will the dependency problem be solved by PR #553? No, but it allows us to start creating tests that don't explicitly new up every single object. At some point in the future, the code behind AddEnvAcutatorServices will be radically different from today. What we do achieve with the PR is to push back further the date when a break with .NET Framework happens.
In this comment, https://github.com/SteeltoeOSS/Steeltoe/issues/503#issuecomment-737501108, the idea has been to support exposing actuators on other things than HTTP. That doesn't match very well with the current codebase. Things are called Endpoints, that doesn't technically need to be HTTP, but often will assumed to be. The Options classes contain AllowedVerbs and Path, and that smells very of HTTP.
If we are going really crazy with changes, then I would propose having an IEnvService that exposes the core functionality.This could be exposed through an Endpoint/Actuator, or someone could trigger it via a message on a Kafka topic.

I had a quick look at Nuget, and all the latest versions of Microsoft.Extensions.* are still supported on .NET Framework 4.6.1.
From a technical point of view, it's possible to create code that runs on both Framework and Core that uses IServiceCollection.
This would be a breaking change, but if .NET Framework support is really important, it might be something to consider.
Rather than writing
var endpoint = new EnvEndpoint(options, configuration, hostEnvironment, logger);

you can write
var services = new ServiceCollection();
services.AddEnvAcutatorServices();

var provider = services.BuildServiceProvider();
var endpoint = provider.GetService<EnvEndpoint>();

With some helper methods, consumers can wrap this to make it easier to use. Since they are on Framework, chances are that they are using static member variables for singletons, so that can be used to hold services.
The earliest nuget release for EndpointBase was published in December 2017. I'm guessing that the majority of .NET Framework projects using Steeltoe are of a newer vintage; those created between 2002 and 2016 may not be prime candidates.
This might be a topic for a survey? How important is .NET Framework support?

Regarding .NET framework support, Steeltoe will continue maintaining framework since it is important to our customers. I don't think there is any question about deprecating it. Regarding maintainability, I think its best to leave Endpoint base where it is because changes to the core functionality are isolated from infrastructure concerns. I am not super familiar with WebApplicationFactory, but it sounds like it is there to test complete applications rather than middleware. From Andrew Lock's blog: "Using the TestServer directly like this is very handy for testing "infrastructural" components like middleware, but it's less convenient for integration testing of actual apps. For those situations, the Microsoft.AspNetCore.Mvc.Testing package takes care of some tricky details like setting the ContentRoot path, copying the .deps file to the test project's bin folder, and streamlining TestServer creation with the WebApplicationFactory<> class." What we are adding and testing here is literally middleware.  So I don't think it is a step up from what we have with test host. What is needed is more test coverage, not a different way of testing - which would in the short term achieve the opposite effect.

WebApplicationFactory uses TestServer internally, so it's possible to build up your own wrapper around it. I've used both and find WebApplicationFactory easier to build around, it also gives me the closest thing to a real running environment. I've used this for both application testing and library testing that may or may not add middleware. Since it does more than TestServer, it may cost more in terms of memory and execution time. For my scenarios this cost has been something I've been willing to pay since it reduces the chances of weird platform issues cropping up when the code runs in an actual application.
WebApplicationFactory can use a Startup class; not something the TestServer knows anything about. Since the EndpointCore tests contains more than 40 references to Startup classes, using the WebApplicationFactory seems like something that can be considered.

We are reopening this.
Hi @kimbell.  Thanks again for the contributions to the project.  We would like to discuss this further with you and how we can do this work in smaller slices.  The team is actively working on other priority features, but we think you are addressing areas that need to be considered and updated.  Since some of the work touches a lot of the code, we want to make sure we can prioritize the work you are doing.   We will reach out to discuss.

We are reopening this.
Hi @kimbell.  Thanks again for the contributions to the project.  We would like to discuss this further with you and how we can do this work in smaller slices.  The team is actively working on other priority features, but we think you are addressing areas that need to be considered and updated.  Since some of the work touches a lot of the code, we want to make sure we can prioritize the work you are doing.   We will reach out to discuss.

This should all be resolved, or at least very different now that a large chunk of refactoring work has been completed


[Request] Auto-cover application for Pf2e

It would be amazing if the option to auto-apply cover worked for Pathfinder 2e.

Moved to Alt Token Cover.


Form Filter is needed.

The current filtering function is on the header, but these take effect in real time. Is there a possibility to make a filter form interface to pass filter items set in batches? Currently, we need to use the parent component to coordinate the filtering form and table components, which is a bit troublesome. Can we have a built-in filter form like pagination?

I'm sorry, but this is way of the scope of this library. There are infinite ways to implement this and there's no "one solution to fit all". Even in my own projects I use the parent component for that.


Allow stopping server

Hello,
this can be useful to either let the server only run in contexts where it is actually required
and also to serve different pages for different parts of an application.

+1


Fix for ZFS enumeration

This should fix #22
Proposed change
As explained in the linked issue, ZFS filesystems don't properly enumerate since the OMV api returns a null uuid for zfs file systems and this plugin effectively skips anything with a null uuid.  I made a change where devicefile would be used in place of uuid as the key for these zfs cases only.  Please let me know if this approach is not preferred!
I have tested this change on my local instance of home assistant (see screenshot of the entity below).
Type of change

[x] Bugfix
[ ] New feature
[ ] Code quality improvements to existing code or addition of tests
[ ] Documentation

Additional information
Here's a screenshot of the newly created entity in my local HA instance:

Checklist

[x] The code change is tested and works locally.
[ ] The code has been formatted using Black.
[ ] Tests have been added to verify that the new code works.
[x] Documentation added/updated if required.

Just noticed that the code formatter check failed on a file I didn't modify.  You want me to run through and format that for this PR also?

Just noticed that the code formatter check failed on a file I didn't modify.  You want me to run through and format that for this PR also?

Looks good to me.
No need for you to fix that formatting issue. Its really small thing that will fix itself next time I save that file in editor anyway.

Looks good to me.
No need for you to fix that formatting issue. Its really small thing that will fix itself next time I save that file in editor anyway.


How to include another theme

Hey @joshmanders, do you plan to add other themes into this distribution?
Is there a way to achieve that, if I need to use another theme with this SASS package?

Hey @teohm. Right now there's no clean way to provide themes as we don't port from LESS directly to SCSS, the SCSS code is executed and converted to the CSS equivalent and then we use a little find and replace to make it customizable a bit.


Make a participant win if the opponent is a BYE

Currently, a match has no result if one opponent is a BYE. It would be better if the participant against a BYE had a win.

Actually, ParticipantSlot doesn't contain the result property. A slot shouldn't be able to "win".
So, not having a green 'W' indicating that the opponent "won" isn't a bad behavior. It's just that the participant didn't have to compete.


refactor: extract AESCipher to crypto.py

Summary
AESCipher class is used for encrypting/decrypting messages.
Initially it was required for macaroons, but now the auth logic also uses it. Therefore it has been moved to its own utils.crypto.py file and the imports updated.

Great refactor!
The cashu extension has its own copyof AESCipher here: https://github.com/lnbits/cashu/blob/main/lib/cashu/core/crypto/aes.py
We can refactor it to use lnbits.utils.crypto.AESCipher and bump it min_lnbits_version to 0.12.0 once released.


Add dataset consitency checks

At the moment we do not valdiate input data before using it in training. This
can cause confusion, since resulting error-messages can be not descriptive.
Thus, I think we should add an option to allows models to validate the input
data.
There are different places, where we could add such a functionality. For example, it could be part of the transformation chain or we could add a new validation interface.
Thougts?

I think this would be great to have. Not sure how generic we can make this, though.

Potentially doable together with #384?


events.S3Event range

I was looking at the S3 Lambda example and I noticed that the records is a range: for _, record := range s3Event.Records {...}
I thought the Lambda events spawned a new process per event so I was wondering why there is a loop at all?
When can the handler receive multiple records? Is that possible? If so what would be the use case?

The data model was designed to be future proof to support multiple records per event. https://forums.aws.amazon.com/thread.jspa?messageID=592264#592264, and also for structural consistency with similar event sources that do support multiple records, like Kinesis.


Blacklist second device of the same person

Current Implementation
App does not distinguish between possible contacts/phones.
Suggested Enhancement
Blacklist devices shown in a list of surrounding bluetooth counterparts.
Expected Benefits
I usually have two phones with me during the day. Hence, most communication will be between my devices (which are more than 15 minutes near to each other, of course).

Chain more than one device of a single user together (UI dialog?)
Not sure about the consequences but it seems reasonable to me
Maybe also useful for other issues

This is impossible if you respect privacy.

You can only report on one device. You can receive on two devices. So you're not harming yourself by having the app on two devices, but it's not really beneficial either.
Just have the app on the main device you're using.
Other requests re changing are out of scope for the time being.

Thanks for your answers @uschindler and @corneliusroemer
This issue was also discussed in, e.g., corona-warn-app/cwa-documentation#82
We will not blacklist second devices as there is really now way of knowing that they are second devices without compromising on privacy questions.

@uschindler and @tkowark, I assumed because the bluetooth identifier is derived of an App/Device identifier, you could exchange the latter key (QR code?) to synchronize two apps such that they recognize each other in a secure way.


TypeError: unsupported operand type(s) for +: 'NoneType' and 'int'

Hi there,
I got an error
File "/home/venv/lib/python3.7/site-packages/fastprogress/fastprogress.py", line 264, in add_child
    self.child.prefix = f'Epoch {self.main_bar.last_v+1}/{self.main_bar.total} :'
TypeError: unsupported operand type(s) for +: 'NoneType' and 'int'

I solved it by commenting that particular line. But, is there any better workaround for this? Because the workaround I did means that I have to change the script in the installation.
Thanks!

        self.last_v = 0
        if parent is None: self.leave,self.display = leave,display
        else:
            self.leave,self.display=False,False
            parent.add_child(self)
       #  self.last_v,self.comment = None,''
        self.last_v,self.comment = 0,''

This works, the author should have mistakenly written it. @imayachita

As mentioned in #57, please reopen with a clear reproducer if needed.


Support all package install options

Critical Changes

We now support the activate_remote_site_settings, security_type, name_conflict_resolution, and password options on both update_dependencies and install_managed. We also fixed a minor inconsistency in defaulting the activate_remote_site_settings (or formerly activateRSS) setting. Projects that define custom tasks based on the InstallPackageVersion class should ensure they explicitly set the activate_remote_site_settings option, or accept the new default of `True.

Changes
Issues Closed

Note for discussion: these changes result in MetaDeploy plans freezing without package install options anywhere they're not explicitly specified (that is, allowing the defaults to be used). This is a change from the previous behavior, which was inconsistent: previous versions froze a default of True for activateRSS on install_managed and install_managed_beta, and update_dependencies froze a default of FULL for security_type.
I could argue either way whether or not we should have the tasks freeze the current defaults at the time of freezing if no explicit configuration is present.


Update README.md

Description of changes:
Updating README of this client-side library to add information about server-side Transactions
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Update README.md with reference to server-side Transactions


White color in between border and background

Have a look at the following screenshot:

For some colors, there's a very small white border. This is there whether I use setImageDrawable() with a ColorDrawable or use setFillColor() on a transparent background.

That's strange. Could you provide details (code extracts) about the listitem-layout and the way you set the drawable?

I came up with a minimal example that should illustrate the problem:

<LinearLayout
	xmlns:android="http://schemas.android.com/apk/res/android"
	xmlns:app="http://schemas.android.com/apk/res-auto"
	android:layout_width="match_parent"
	android:layout_height="match_parent">

	<de.hdodenhof.circleimageview.CircleImageView
		android:id="@+id/circleView"
		android:layout_width="48dp"
		android:layout_height="48dp"
		android:layout_gravity="center"
		android:src="@android:color/transparent"
		app:civ_border_color="#000000"
		app:civ_border_width="2dp"
		app:civ_fill_color="#000000"/>

</LinearLayout>

Please note that the same problem exists when filling the CircleImageView differently:
<LinearLayout
	xmlns:android="http://schemas.android.com/apk/res/android"
	xmlns:app="http://schemas.android.com/apk/res-auto"
	android:layout_width="match_parent"
	android:layout_height="match_parent">

	<de.hdodenhof.circleimageview.CircleImageView
		android:id="@+id/circleView"
		android:layout_width="48dp"
		android:layout_height="48dp"
		android:layout_gravity="center"
		android:src="@android:color/black"
		app:civ_border_color="#000000"
		app:civ_border_width="2dp"/>

</LinearLayout>

Thanks, I'm able to reproduce the issue and I'll look into it. I guess it's related to both the border and the drawable being antialiased. As a workaround you could just add app:civ_border_overlay="true"

Thanks! An initial test indicates that adding app:civ_border_overlay="true" does indeed solve the problem.

Fix will be release as part of 2.1.0.

Awesome, thanks! Do I understand correctly that with 2.1.0, it will not be necessary to add app:civ_border_overlay="true" ?

Exactly!


Use top-level statements in the C# implementation

This is more of a stylistic change, but you actually didn't need to have a Program class with a Main method, it's redundant code and adds two unnecessary levels of indentation — C# 10 introduced top-level statements, eliminating the need for explicitly declaring these. The only thing is that the types need to come after the statements.

Thanks


fix: migration script for user/invitations sid

Description
We have null values in prod and cannot run the migration
Risk

Deploy Plan

I guess we can run this from front-edge and not commit it, right?

Why wouldn't we ?

We likely never want this migration (or any migration) to overwrite "real" values with backfill values.


Checkbox / Radio Button: Improve HTML whitespace sensitivity

Related: https://github.com/uswds/uswds/pull/4286
Context: https://github.com/18F/identity-idp/pull/5279#discussion_r686146591
This pull request seeks to maintain the existing visual appearance of the Checkbox and Radio Button components, while improving developer ergonomics regarding HTML whitespace sensitivity.
See additional details at https://github.com/uswds/uswds/pull/4286 .
The expectation is that there should be no user-facing visual effect of these changes, hopefully demonstrated by a passing visual regression test.
This does not incorporate other design revisions proposed for the checkbox and radio component, which will be implemented separately.

The expectation is that there should be no user-facing visual effect of these changes, hopefully demonstrated by a passing visual regression test.

I lied... the horizontal offset of the checkboxes changes ever-so-slightly. Checking out the artifacts for the build (specifically -diff.png) gives a good overview of the changes. I might be able to get this back to its original styling, but in retrospect I feel this is a bit more faithful to the original Figma designs, as far as the checkboxes and radios appearing at the leftmost edge, with a consistent right margin of 0.5rem. It gives a slightly imbalanced appearance in the bordered and tile variations, but this exists in the Figma design as well. Curious @nickttng @anniehirshman-gsa @cdelivuk if you'd have thoughts here.
Live demo: https://federalist-2f194a10-945e-4413-be01-46ca6dae5358.app.cloud.gov/preview/18f/identity-style-guide/aduth-tile-checkbox-label-size/components/form-fields/#radio-buttons

Noting that the upstream https://github.com/uswds/uswds/pull/4286 is now merged, we'll need to come back and revisit this in the future to reconcile redundancies.


Update Playground with the latest version of the library

This pull request builds the Playground with the latest version of the library

coverage: 100.0%. remained the same
when pulling 6545ebf546514df22ffbf5d4f20f5729578c72e4 on update_playground_bundle
into 0013dd0e9e1c2fd1e16e72d1d81aee4ecc1d6682 on master.


ROCANA-9536: Shutdown hangs with several channel subscriptions

This change avoids blocking on shutdown when errors are available on the channel.
Also an improvement to error handling: according to the documentation, when the first argument provided to the subscription callback is EvtSubscribeActionError, the third argument is a Win32 error code, not an event handle. Further, GetLastError() does not describe the error code, but instead the error code can be looked up in the Microsoft System Error Code table (link provided in the PR).

+1, LGTM


Canary Insider Preview 27744.rs_prerelease.241101-1329 is not compatible (was MIDI ports are not displayed on midi.exe.)

Describe the bug
MIDI ports are not displayed on midi.exe.
To Reproduce
1.Update to the following versions of the Windows Insider Program (latest version of Canary channel)
Windows 11 Pro Insider Preview Build 27744.rs_prerelease.241101-1329
2.Reboot the OS
3.Start the monitor with midi.exe

midi endpoint monitor --verbose

4.No endpoints are displayed.

Expected behavior
Endpoints are displayed.
Installer Name or Version
・Windows.MIDI.Services.In-Box.Service.-.1.0.1-preview.7.24305.1438-x64.exe
・Windows.MIDI.Services.Tools.and.SDKs.1.0.1-preview.7.24305.1438-x64.exe
Desktop (please complete the following information):
Windows 11 Pro Insider Preview Build 27744.rs_prerelease.241101-1329

This is a known issue. I forget that not everyone is on Discord, where I posted a not about this build.
https://discord.com/channels/980245825202552942/980246206779383848/1305589346891731065
That canary build is not compatible with Windows MIDI Services DP releases. It contains an early test deployment of the MIDI Service/API and plugins. Build numbers below that, including those in the dev channel, will work.
I'll leave this issue open in case others run into it. I don't have a date/build # yet for the version that is compatible with the SDK.

I checked the other versions.
27749.rs_prerelease_241108-1425 is no good as well.
27729.rs_prerelease.241011-1428 is OK.

For me, everything seems fine on 27749.1000 with DP7 services installed from git release
Midi ep monitor --verbose. Getting UMP packed form UMP device.

It looks like you will need at least 27751.1000.rs_prerelease.241112-1359
I'm working on instructions to de-register the in-box components and allow the install to complete.

 Not sure where is 27751...

It's working in your case because somehow the midisrv abstraction/transport registered for the DP, and the service is running. Not sure what order of operations permits that.
A couple things to check

From the command line issue

sc qc midisrv
If the results point to the midisrv.exe in Program Files, you are running the DP version. If it points to one in System32, you are running the older one from the Insider build.

Then open regedit and look here

Computer\HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows MIDI Services\Transport Plugins
Anything with "Abstraction" in the name came from the Insider build. Anything with "Transport" came from the DP. There are probably duplicates there. The one that wins is the one that is loaded first.
Right now, mididiag doesn't produce a list of file locations for which COM components are being loaded, which makes it difficult for us to know what someone is running when on an insider build. I'm looking into what I can do there and also in the service.

The problem:
Recent Canary builds ship with a version of Windows MIDI Services in-box. This version is a month or so behind the developer previews. In this case, we had a breaking change where all the component and interface names that say "Abstraction" had to be changed to "Transport" per the internal API review feedback. There were a few other changes related to that, all of which are implemented in the DP7 bits, but not the early Canary bits.
The steps below can also be used to install DP bits on a version of Windows that ships with Windows MIDI Services (retail, canary, etc.), whenever there's a new developer preview in the future.
To uninstall the existing version of midisrv that ships in-box:

Open up the Services snap-in (search services. It's the app with two gears).
Scroll down and verify "Windows MIDI Service" is listed
Open an Administrator command prompt
Run sc stop midisrv in case it has been started
Run sc delete midisrv
In the services app, go to Action > Refresh and scroll down. Verify Windows MIDI Service is no longer listed

We won't delete/replace the COM registrations for the existing service plugins that are in System32, because the installer will overwrite those registrations and point to the DP versions in Program Files.
There will be some duplicate entries in the midisrv plugin load list at
Computer\HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows MIDI Services\Transport Plugins
If you have the rights to do so, it's a good idea to remove any entries in there which have "Abstraction" in the name. Just remove the entire key such as Midi2KSAbstraction. If you don't have rights to delete those keys, you can right-click "Windows MIDI Services" in the registry and grant yourself the permissions / take ownership.

Next, take ownership of some key files in System32. At the administrator command prompt, navigate to \Windows\System32 and then
takeown /F wdmaud2.drv

And then it will be able to be replaced by the installer.
Finally, install the DP version. You will likely need to reboot to replace wdmaud2.drv as it is often loaded into other processes using audio.
The above steps will need to be done any time you install a new version of Windows (Canary or other).
TIP: Before updating to a new Insider build, always uninstall Windows MIDI Services first, so as not to leave bits lying around and unable to be cleanly uninstalled.

latest version 27754.rs_prerelease.241115-1409 is OK.

27754.rs_prerelease.241115-1409 is OK.

I've added some powershell scripts under src\dev-tools to make installing a dev preview over the in-box components possible. I'll provide more detailed instructions with DP8.


Add SSH key to Flatcar Linux on AWS Pulumi program

The Pulumi program found at flatcar/flatcar-pulumi-aws currently has a required configuration value that specifies the name of the AWS keypair. By modifying the program to create an SSH key and an AWS keypair, that required configuration value can be eliminated.
The ubuntu/ubuntu-pulumi-aws program already includes these resources and can be used as a model for the changes needed here.

Fixed via #152


Error while importing cards data for the first time

While importing card data via php app/console nrdb:import:std ./cards-json I've received the following error. I was following the instruction step by step and trying to run everything inside a docker container.
Importing Cards...
Changing the code of
Changing the position of 11001
Changing the title of 11001

In ImportStdCommand.php line 551:

  Missing key [image_url] in {"code":"11001","cost":1,"deck_limit":3,"faction
  _code":"anarch","faction_cost":2,"illustrator":"Meg Owenson","keywords":"Cu
  rrent","pack_code":"23s","position":1,"quantity":3,"side_code":"runner","te
  xt":"This card is not trashed until another <strong>current<\/strong> is pl
  ayed or an agenda is scored.\nWhenever the Corp draws 1 or more cards, he o
  r she loses 1[credit] if it is not the first time he or she has drawn cards
   this turn.","title":"System Outage","type_code":"event","uniqueness":false
  }

nrdb:import:std [-f|--force] [-h|--help] [-q|--quiet] [-v|vv|vvv|--verbose] [-V|--version] [--ansi] [--no-ansi] [-n|--no-interaction] [-e|--env ENV] [--no-debug] [--] <command> <path>

For full reproduction, just clone my repo: https://github.com/dotintegral/munchkindb then enter the docker dir and run init.sh. Obviously, docker is need to be installed on the host.

Try switching to the munchkin-conversion branch: https://github.com/CallidusAsinus/munchkin-cards-json/tree/munchkin-conversion. This might fix the import error.

Thanks! I was able to move a bit further. But now I'm stuck on the following:
Changing the pack_code of 216 Zorch!
Changing the side_code of 216 Zorch!
Changing the type_code of 216 Zorch!

 Thanks for posting.

Just pushed commit https://github.com/CallidusAsinus/munchkin-cards-json/commit/f110f0c9e20b67d389fb8ebcef201c3d82351c41 which should solve the issue. I created ticket https://github.com/CallidusAsinus/munchkindb/issues/9 to support the underlying data structure causing this issue.


Package not found error

I installed pika globally and then the standard-pkg as npm install @pika/plugin-standard-pkg --save-dev
However when running the pack build command I get the following error.
@pika/pack build v0.3.1
[1/4] Validating source...
[2/4] Preparing pipeline...
      ❇  pkg/
[3/4] Running undefined...
Error: Command failed.
Exit code: 127
Command: @pika/plugin-standard-pkg
Arguments:
Directory: /home/<user>/Development/<project>/component-library
Output:
/bin/sh: 1: @pika/plugin-standard-pkg: not found

This is the pika section of my package.json
"@pika/pack": {
    "pipeline": [
      "@pika/plugin-standard-pkg",
      {
        "exclude": [
          "**/*.test.js",
          "**/*.stories.js",
          "__mocks__/*"
        ]
      }
    ]
  },

If anyone can point me to a fix I would appreciate it.

Make sure you put each plugin in an array, your pipeline would instead look like:
"@pika/pack": {
    "pipeline": [
      ["@pika/plugin-standard-pkg", {
        "exclude": [
          "**/*.test.js",
          "**/*.stories.js",
          "__mocks__/*"
      ]}]
    ]
},

Absolutely that was my error. Thank you. Now to figure out how to get styles to not crash it with:
SyntaxError: /home/travis/Development/fitplan/component-library/src/config/variables.css: Unexpected token (1:0)

> 1 | :root {
    | ^
  2 |   --black: #111213;
  3 |   --flat-green-bright-bg: #12b587;
  4 |   --flat-green-dark-bg: #00ffb3;

Hmm, does this repo exist anywhere I could check it out? I'm surprised it's outright failing, that could be happening inside of Rollup.
In general, importing CSS from inside of JS isn't allowed on any JS platform. Bundlers allow it as a convenient shortcut (I've done this in plenty of apps I've built, and it's great). But since you're publishing to a shared NPM registry where you don't know who your consumer is, I'd recommend keeping it outside of your JS if you can.
We recommend an assets/ directory for non-JS assets that would be shared across different distributions of your package. My # 1 todo right now is to document all of this, which didn't make it into that first blog post :)

@FredKSchott, can you provide some examples of importing css with pika build?

I will try to add one to https://github.com/pikapkg/examples, if no one else beats me to it :)

I was never able to find the time for this, but here's the basic steps:

put your CSS in a top-level assets/ dir
make sure you have this plugin in your build pipeline https://github.com/pikapkg/builders/tree/master/packages/plugin-copy-assets
If you only care about your package working when consumed with a bundler, just import '../assets/my.css' inside of your src directory
If you want to target direct usage, you'll want to do some special logic to add a <style>/<link> tag to the page via JavaScript that will load your CSS file.


latest version can't use 0/1 as bool

So messages properly parsed before are not anymore.
I understand that this is reasonable option... so fill free to close if it has been an intended change.

This is similar to #39. Do you have any comments on my proposed fix over there? I'm not quite sure what the right thing to do is, that's why I haven't pushed it yet. Maybe I should also allow ints for bool (which would fix this issue)? Or maybe I shouldn't allow floats for the integer types over there?

Upon reflection, I think it's better to force the programmer to explicitly convert their ints to bools, if that's ok for you. I'm going to close it for now. Feel free to reopen if there's more.

Sure, sure;  is strict but reasonable. I already adapted my code to convert. Thanks!


CORS configuration

This may be a documentation request more than a feature request or issue.
Is there a way to use rack-cors with message_bus? In testing, when I place:
use Rack::Cors do
  allow do
    origins '*'
    resource '*', :headers => :any, :methods => :any
  end
end

in my config.ru, my routes accept cross origin, but the /message-bus/:id/poll route does not seem to inherit this rack config. I see preflight errors in the browser console, whereas the routes I have defined in my application code are returning properly.
Am I missing something right in front of me? Or am I trying to achieve something not intended to be supported?
Great project. Love using it.

Thanks,
message bus is usually earlier in the middleware stack and handles it this way:
https://github.com/discourse/discourse/blob/master/config/initializers/004-message_bus.rb#L44-L47

@bkeane closing this, is that ok? perhaps send through a PR to document CORS?


Supermium 124 R2 produces component location error

Title says it all. I am running Windows XP Professional SP3 (x86).

Install the Visual C++ 2015 runtime (or 2015-19 runtime). Or remove dwrite.dll. I will release an updated version dwrite.dll that will remove the VC++ dependency in the future, only depending on msvcrt + some replacements in my p_vcrt wrapper.

I only encountered that error when VC++ 2015-2019 was installed. If I just installed VC++ 2015 (its an earlier version), it doesn't come up with the error on XP.

@SleakaJ Best is to install VC++ Repack. https://github.com/abbodi1406/vcredist

To be clear, the error message is not correct. If you click the OK button, Supermium will still start.


only static widgets can put to place?

the code below not works as expected, after the add file button get clicked and choosen a file, the label do not shows.
int main()
{
	using namespace nana;
	form fm;
	//It's unnecessary to specify a rectangle if useing
	//layout management.
	label lb1{ fm, u8"l1" }, lb2{ fm,u8"l2" }, lb3{ fm, u8"l2" }, lb4{ fm, u8"l3" };
	lb1.text_align(nana::align::center, nana::align_v::center);
	lb2.text_align(nana::align::center, nana::align_v::center);
	lb3.text_align(nana::align::center, nana::align_v::top);
	lb4.text_align(nana::align::center, nana::align_v::center);
	place layout(fm);
	button bt2{ fm ,u8"add file" };
	layout.div("<l4_1 vert gap=2>");
	std::set<std::string> attachments;
	layout["l4_1"] << bt2;
	bt2.events().click([&] {
		nana::filebox fb(true);
		if (fb()) {
			//attachments.insert(fb.file());
			nana::label tmp{ fm, fb.file() };
                       //static nana::label tmp{ fm, fb.file() };   // this line works,  but can only add one file
			layout["l4_1"] << tmp;
			layout.collocate();
		}
	});
	layout.collocate();
	fm.show();
	exec();
}

I thought the code above should work due to the right value reference of cpp 11, is this a bug, or i was wrong on the usage of right value reference?

Hi,
I don't like very much your use of a local object tmp that will be destroyed immediately. I would better find a way to keep it alive all the time it is espected to be on the screen (defining it in main, for example). Nana is not exactly like Qt that create objects with new and keep the pointer with the parent object... I think. We have more c++ -like rules about that.

That said, I'd like to help you more specifically. What do you mean with

wrong on the usage of right value reference ?

just I think that whatever kind of reference the << tmp keep into the place, it is going to be a dangling one, because tmp go out of scope. Temporary objects are kept alive by references when they are returned from a function, not when they are passed as an argument to a function, and saved as a reference somewhere else. Dangling references are still a problem in any c++.

I know the tmp will be out of scope after the if block, but nana overides the operator method <<  to  put widgets into fields of a place, naturally it should works as same as std::cout.  The std::cout only use the value of tmp, no matter how the tmp value changes after <<.

std::cout dumps a copy of content of a object, place refers to a specified widget. They have different sematics. Though the field of place overrides the operator<<, it doesn't mean that it has to work as same as std::cout. For example, operator<<(ostream& os, T t) doesn't work as same as  int<<int.

so when dealing with dynamic contents, it's the callers' reponsibilty to allocate/free a widgets in head.

I'am trying to use smart ptrs to help manages dynamic widgets now.

Example:
https://github.com/qPCR4vir/nana-demo/blob/master/Examples/group.cpp#L45

The point is that normaly 'place' don't owns the widgets (not even a copy of then), it only dinamically layouts the widgets of the parent windows following the rules of the div-str. But if you are willing to experiment with advanced functions, then you can tell the 'place' to own the widget using an 'agent': https://github.com/cnjinhao/nana/blob/master/include/nana/gui/place.hpp#L56
But if what you really want is to put a very simple label owned by the place, then place support that directly:
bt2.events().click([&] {
	nana::filebox fb(true);
	if (fb()) {
		layout["l4_1"] << fb.file();
		layout.collocate();
	}
});

See: https://github.com/cnjinhao/nana/blob/master/include/nana/gui/place.hpp#L92


dt,bt or crithack problem (major fps drops)

Fedoraware Version Date:
Config used:
https://files.catbox.moe/yrjm3b.fw -- my config
Description:
i am pretty sure this is caused either by dt, bt or crit hack but im very certain this is crithack. when i use crithack for a decent period like 10 mins or 20 with dt or without when i use crithack key and use aim key my fps drops to like 20 and then back to my usual amount ( like 150 ig )
Steps to reproduce:

[x] I am able to consistently reproduce this error

Steps to reproduce the behavior:

Enable DT and CritHack
Use both of them for like 10 to 20 minutes
Game starts to lag when I try to use both of them

Additional context
this bug is kinda annoying it might be config issue but im guessing not since crithack is already buggy

[x] I have included the steps to reproduce to the best I am able to share.

probably crithack issue, just don't crit as much

yeah prob but still i wanna 1 shot heavies or just idk use crit hack in general

why did my reply not work

Never had issues with fps, prob render issues or game

Never had issues with fps, prob render issues or game

i went in third person and the fps problem basically fixed itself (i am confused??)

I will see in my build if it happens if not then look into it with pub build.. since I have removed glow and other stuff that kill fps and crithack is a bit changed for me so.. yea let me look into it

H

I will see in my build if it happens if not then look into it with pub build.. since I have removed glow and other stuff that kill fps and crithack is a bit changed for me so.. yea let me look into it

ok

Crithack should've been fixed now, I think. My dumbass self will look at the DT and Backtrack soon.

Crithack should've been fixed now, I think. My dumbass self will look at the DT and Backtrack soon.

only crithack made your game lag
backtrack is supposed to be laggy (you cant fix that)
and dt isnt lagging at all


No text on taskbar icon when right clicked - win7

screenshot is easier than explaining.
When hovering (no problem):

on right click:

Can't reproduce this on Windows 10. I'll make a virtual machine with Windows 7 to reproduce this... going to take some time.

not major issue.  Is there anything I can do to help, save you sorting out a VM?

Oh no, don't worry, installing Windows is one of my hobbies... and I've got an MSDN licence from work so I can legally install Windows 7 in a virtual machine. I mean to test on different Windows versions anyway.

This issue is resolved... although the game doesn't have an icon at all at the moment, I will implement better OS integration at a later time.


Crashing on startup

I recently installed a library for another mod I wanted to download, but it crashed my game whenever I started it up. I opened an issue with the mod author and he said "the reach-entity-attributes library [you] are using is broken" and that what was causing problems. Can you look into this? I've attached my most recent crash report if it helps.
crash-2024-02-15_22.32.26-client.txt

An update for reach entity attributes just got released today for MC 1.20.4, so I will update enchantments-plus soon and hopefully that will fix your crash

Thankfully your library is no longer causing problems, although now another one is. Thanks for the help tho.


⚠ eaglestorage.com.au has degraded performance

In d2893c2, eaglestorage.com.au (https://eaglestorage.com.au) experienced degraded performance:

HTTP code: 200
Response time: 6337 ms

Resolved: eaglestorage.com.au performance has improved in 7c16b7c after 12 minutes.


Create Embed component

Here is the original one: https://semantic-ui.com/modules/embed.html.
Please have a look at contributing.md for instructions on how to contribute and submit a PR.

Let me take a stab on this

@mariolamacchia PR is now ready for review

Released in 0.0.28


[BUG] usePort hook message rendering data not capturing all event

What happened?
In my chrome extension project with react, some code like:
// chat.tsx
const chatPort = usePort('fetch/chat-stream');
useEffect(() => {
    console.log('chatPort.data', chatPort.data?.data);
}, [chatPort.data])
useEffect(() => {
    (chatPort as any)?.listen(e => {
      console.log('chatPort.listen', e?.data);
    })
  }, [])

function handleSend() {
    chatPort.send({data: "xxxxx"})
}

const handler: PlasmoMessaging.PortHandler = async (req, res) => {
...

res.send({ id: "xxx", data: "xxxxx" })
...
}

Sometimes chatPort.data will lose message, while chatPort.listen will not be lost.

Version
Latest
What OS are you seeing the problem on?
No response
What browsers are you seeing the problem on?
No response
Relevant log output
No response
(OPTIONAL) Contribution

[ ] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

@yaxiaoliu Can you try debugging why the last few message is not propagated to the state variable, the data is done like this:
https://github.com/PlasmoHQ/plasmo/blob/3c73df64d57479865b0c870e62890a4e56b34608/packages/messaging/src/hook.ts#L40-L49
I'm not sure how it would... lose message xd.... it's pretty much identical to your listener :d....

I tried to debug it, but the issue appears randomly and I couldn't find any specific reason in the usePort source code.
Currently, I have resolved the lost message issue by using (chatPort as any).listen, but the listen method is not defined in the function signature, which is not elegant.
My useEffect only depends on chatData.data, but it also prints out lost messages. I will try to debug it again and see if I can find the cause.

Currently, I have resolved the lost message issue by using (chatPort as any).listen, but the listen method is not defined in the function signature, which is not elegant.

@yaxiaoliu Oh whoops that's a but with the typing interface of the hook! Fix coming up

The typing issue should be fixed in latest messaging package

Currently, the latest version of data still has this problem, which has not been fixed.


Skip floating windows in nvim

Plugins like asyncomplete.vim make use of NeoVim's floating windows but lightline doesn't skip those so status line becomes disabled the first time such window appears.
Source: This neovim issue, :help api-window

Describe the issue this PR fixes with reproducible steps.

Correction: it was related to asyncomplete.vim, I don't know why I wrote about vim-lsp.
Here are reproduction steps:

Install asyncomplete: Plug 'github.com/prabirshrestha/asyncomplete.vim' and set basic configuration (described in README)
Try to trigger completion
lightline becomes disabled

I cannot still reproduce the issue with Neovim v0.5.0-dev+a06c256. Please describe the entire configuration and precise steps to invoke completions. Also attach the Neovim version.

NeoVim version: v0.4.4 release
Minimal vimrc:
Plug 'itchyny/lightline.vim'
Plug 'prabirshrestha/asyncomplete.vim'
Plug 'prabirshrestha/vim-lsp'
Plug 'prabirshrestha/asyncomplete-lsp.vim'

" You can use anything there
if executable('gopls')
    au User lsp_setup call lsp#register_server({
        \ 'name': 'gopls',
        \ 'cmd': {server_info->['gopls']},
        \ 'allowlist': ['go'],
        \ })
endif

inoremap <expr> <Tab>   pumvisible() ? "\<C-n>" : "\<Tab>"
inoremap <expr> <S-Tab> pumvisible() ? "\<C-p>" : "\<S-Tab>"
inoremap <expr> <cr>    pumvisible() ? asyncomplete#close_popup() : "\<cr>"

Then open any file for registered language server and trigger the completion (either start typing or hit Tab)

Can you test with the latest Neovim? It still works for me.

Hm, I built NeoVim from source and cannot reproduce this bug anymore. But I still think that the fix is necessary because not everyone runs git master

Sorry but I don't accept workaround patches for older versions. Using win_gettype() is a clean solution and works in both Vim and Neovim.

Ok, I'll just switch to git version then


Add CentOS 8 support

Prerequisites

[X ] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filled: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aidealista
Checked that there is not already provided the described functionality

Description
Add CentOS 8 support
Additional Information
I'll open a new PR for this over the weekend.

CentOS is now supported using v5.1.0


macOS CI failure: "brew update" doesn't work in CI image; workaround causes openssl to be missing

These occur in the Install native dependencies step in our macOS CI:
fatal: 'origin' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights

Error: Fetching /usr/local/Homebrew/Library/Taps/local/homebrew-openssl failed!
Fetching /usr/local/Homebrew/Library/Taps/local/homebrew-python2 failed!
fatal: invalid upstream 'origin/master'
fatal: invalid upstream 'origin/master'

##[error]Bash exited with code '1'.

This is caused by a macOS CI image bug: https://github.com/actions/virtual-environments/issues/1811

There is a workaround, but applying it causes the build to be unable to find openssl. More info in https://github.com/dotnet/source-build/pull/1817 (master) and https://github.com/dotnet/source-build/pull/1818 (release/3.1).

The CI at https://github.com/dotnet/source-build/pull/1925 failed later into the build than brew configuration. Maybe we can look at turning this back on.


Error while using record with --mfcc

Hi, @timediv. I changed in the recording.py
print('Recording audio')
raw_audio, sample_width = recorder.record()
raw_audio = np.array(raw_audio)

to
import soundfile as sf
raw_audio, sample_rate = sf.read(path_wav_file)
raw_audio = np.array(raw_audio)

and try to run sudo python3 speecht-cli record --mfcc --train-dir train --run-name best_run --language-model kenlm-english/, but have:
Generate MFCCs or power spectrogram
Running speech recognition
Traceback (most recent call last):
  File "speecht-cli", line 221, in <module>
    cli.run()
  File "speecht-cli", line 207, in run
    self.command_executor.run()
  File "/home/ubuntu/s2t/speecht/recording.py", line 71, in run
    [decoded] = model.step(sess, loss=False, update=False, decode=True)
  File "/home/ubuntu/s2t/speecht/speech_model.py", line 231, in step
    input_feed_dict = self.input_loader.get_feed_dict() or {}
  File "/home/ubuntu/s2t/speecht/speech_input.py", line 108, in get_feed_dict
    input_tensor, sequence_lengths, max_time = self._get_inputs_feed_item([self.speech_input])
  File "/home/ubuntu/s2t/speecht/speech_input.py", line 43, in _get_inputs_feed_item
    input_tensor[idx, :inp.shape[0], :] = inp
ValueError: could not broadcast input array from shape (493,39) into shape (493,128)

Why does it happen and how can I fix it?
P.S. with --power work OK

Have you trained your model using mfccs?

Oh! I hadn`t thought about it, but I have used your pretrained one from here


Open editors list lost middle click to close file

https://github.com/Microsoft/vscode/issues/56

True. @isidorn you can see how this used to work in https://github.com/Microsoft/vscode/blob/1.2.0/src/vs/workbench/parts/files/browser/views/workingFilesViewer.ts#L349


Manage Groups with UI (Adding users and owners)

User Stories
As a User, I want to be part of different groups, in Order for my Permissions to be manageable.
Dependencies

#104
#106

UI Mockup
missing
Acceptance Criteria

[ ] if one is owner of the group, in the list view there is a plus in the top right to add members to the group
[ ] #259

Other Issues will take this Issues place


Enable remote video track publications on first attachment

When adaptiveStream is enabled, video tracks should be enabled only when they are visible. The current implementation presumes that new tracks are attached as soon as they are subscribed and are therefore enabled by default.
As one approach to allow for use cases where this is not the case (e.g. a user starts sharing their screen but decides to actually present it at a later point), this behaviour could be inverted:
The initial value of RemoteTrackPublication.disabled should be set to true if kind === Track.Kind.Video and this value should be then set to false as soon as the corresponding  RemoteVideoTrack has been attach()'ed for the first time (only if it's actually visible at this point, of course).

that's a great point, this will be handled in the future on the server side via an additional parameter for adaptiveStream to signal to the server that the publications can start off in a paused state:
client implementation is here: https://github.com/livekit/client-sdk-js/commit/2bff21fc5c3070ba4d150d52cb0c043373bb9dd4
(one of the relevant) server PRs here: https://github.com/livekit/livekit-server/pull/623


docs: add guide for how to send a user operation

Pull Request Checklist

[ ] Did you add new tests and confirm existing tests pass? (yarn test)
[ ] Did you update relevant docs? (docs are found in the site folder, see guidleines below)
[ ] Do your commits follow the Conventional Commits standard?
[ ] Does your PR title also follow the Conventional Commits standard?
[ ] If you have a breaking change, is it [correctly reflected in your commit message](https://www.conventionalcommits.org/en/v1.0.0/#examples? (e.g. feat!: breaking change)
[ ] Did you run lint (yarn lint:check) and fix any issues? (yarn lint:fix)
[ ] Is the base branch you're merging into development and not main?

PR-Codex overview
Detailed summary

Added a new section "Guides" to the website navigation menu.
Created a guide on how to send a User Operation using Alchemy's Account Kit.
Added code snippets and explanations for creating a provider, connecting a smart account, and sending a User Operation.
Added a full example code snippet for reference.
Added metadata and content for the new guide page.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Current dependencies on/for this PR:

development

PR #201   👈

This stack of pull requests is managed by Graphite.


documentation: plugin implementation descriptions

I think that we should have a documentation for the implemented plugins in this repository. I'd imagine to write about (in)compatibilities, specific example configurations and also use-case suggestions.

+1


[Docs]: Adding why to open source in main README.MD

Description
Adding a section on what is an open source program and why to take part in it.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the [Contributing Guidelines]
[X] I have read the discussion tab thoroughly and got the project idea
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

@deepshikha2708 I don't think this Issue is relevant.


Tiny blog

Thought: a small blog, on oversight.io itself, where we can put the launch post, and potentially permalink versions of anything we send out on Tinyletter. It'd be nice to help the site accrue PageRank and overall awareness.
This could easily get more annoying and complicated than it's worth, and I'm even up for just hardcoding a single page to get it started and not building a whole blogging engine.

I'm interested in taking this issue.

Go for it!

Okay, so I've created a very basic system using Wintersmith which was the most embeddable static content generator I could find.
So basically it works like this:

Globally install wintersmith using npm
Install the node module for wintersmith-ejs
Create a post in ./blog/, the name of the file on disk can be separated from the URL filename using the filename param in the frontmatter
Use $ wintersmith build --config=blog-config.js
The files end up in /public/blog, where they are already served statically by express

Also, because of using wintersmith-ejs, the blog-post.ejs template uses header.html and footer.html from the main site.
Additional work could be done to create a blog index with pagination and such things. But for your bare bones, let's throw up some static content that we author using Markdown, this does it.

This assumes that blog posts will be committed and part of the repo, of course, which seems like a good assumption and good practice.

:+1: on keeping blog posts in the repo

Blogging works!

It sure does!


Doesn't work with jruby x64

https://gist.github.com/1503754

What Ruby runtime are you using? Tests pass with therubyrhino and jruby 1.6.5 (Travis CI, other libraries use C-extensions which are known to have problems on JRuby.

$ jruby -v
jruby 1.6.5 (ruby-1.9.2-p136) (2011-10-25 9dcd388) (Java HotSpot(TM) 64-Bit Server VM 1.6.0_29) [darwin-x86_64-java]

gives
$ jruby -e "require 'uglifier'; Uglifier.compile File.read 'jquery-1.7.1.js'"
java(25487,0x10f18a000) malloc: *** error for object 0x7f9604a05300: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
Abort trap: 6

What does jruby -e "require 'uglifier'; puts ExecJS.runtime say? You might also want to try EXECJS_RUNTIME =RubyRhino jruby -e "require 'uglifier' …"

#ExecJS::RubyRhinoRuntime:0x3710b205
On Wed, Dec 21, 2011 at 14:01, Ville Lautanala
<EMAIL_ADDRESS>wrote:

What does jruby -e "require 'uglifier'; puts ExecJS.runtime say? You might also want to try EXECJS_RUNTIME =RubyRhino jruby -e "require 'uglifier' …"

Reply to this email directly or view it on GitHub:
https://github.com/lautis/uglifier/issues/22#issuecomment-3233372

I can't replicate this with jruby 1.6.5 (running in 1.9 mode on same 64bit jvm).
Can you run anything with ExecJS? E.g. `ExecJS.eval '1+1'?

indeed and uglifier works for very small files as well, but not with
for example jquery (http://code.jquery.com/jquery-1.7.1.js)
On Wed, Dec 21, 2011 at 14:21, Ville Lautanala
<EMAIL_ADDRESS>wrote:

I can't replicate this with jruby 1.6.5 (running in 1.9 mode on same 64bit jvm).
Can you run anything with ExecJS? E.g. `ExecJS.eval '1+1'?

Reply to this email directly or view it on GitHub:
https://github.com/lautis/uglifier/issues/22#issuecomment-3233547

I managed to replicate the segfault with yajl-ruby gem. It uses C extensions, which AFAIK are still experimental in JRuby.
Uglifier uses multi_json, which tries to load the best available JSON parser. You can manually change the used engine to something which works on your platform with MultiJson.engine = :json_gem or just remove yajl-ruby from your Gemfile/system.


Basic schedule clock offset wrong

Seems the offset of the clock is wrong by one hour from what is reported in the easee app and the easee integration, it seems the same offset is for both start and stop

@prilly-dev What versions of Homeassistant and Easee integration are you using?
Times are identical in my setup.
Are the settings for timezone and DST the same in phone/app and homeassistant?

iam using Europe/oslo on the server, app and easee platform

iam running docker.
root@ha:~# timedatectl                Local time: Tue 2021-10-05 21:46:54 CEST            Universal time: Tue 2021-10-05 19:46:54 UTC                  RTC time: Tue 2021-10-05 19:46:55                 Time zone: Europe/Oslo (CEST, +0200) System clock synchronized: yes               NTP service: n/a           RTC in local TZ: no root@ha:~# ls -l /etc/localtime lrwxrwxrwx 1 root root 31 Sep 29 09:11 /etc/localtime -> /usr/share/zoneinfo/Eur                                 ope/Oslo
From server, i am then passing the etc/localtime to docker
-v /etc/localtime:/etc/localtime:ro
and home assistant is set in to europe/oslo, it seems that GMT +1 is used for this region in HA, and currently is should be gmt + 2 with summer / winter time

@prilly-dev Good catch!


Audio Streaming

Feature Request
Is your feature request related to a problem? Please describe.
Yes, i want to reinforce the need for streaming support.  I am encountering the mentioned instabilities in Stream Text to Speech
Describe the solution you'd like
Unity support for Stream Text to Speech
Describe alternatives you've considered
currently i am loading the full audio and its a very long delay for multi-page text.
Additional context
I realize this may be out of your control, but alternative solutions would be appreciated, is Unity aware of and tracking this bug?

Never got an update after submitting repro examples
https://forum.unity.com/threads/downloadhandleraudioclip-streamaudio-is-ignored.699908/#post-9127066

I also discussed the same problem to the eleven labs team and proposed changes to the API to enable streaming of the raw pcm data but they didn't want to make the requred changes.

My only suggestion is to maybe break up the text into smaller segments and submit in chunks

thanks for the reply, and the suggestion!