Enhancement suggestion : add darkmode on reader

Nice piece of work so far ;)
My issue is all in the title.

I've added a basic dark theme in b38a6bdd40ae8cb72b9ac24137acc627423e467c. It automatically turns on between 10pm and 6am.
There should probably be a settings panel for this, so it can be used at other times of day.


Fix FindReplaceBar focus problems

After #81128 and then #82914 the _unhandled_input in FindReplaceBar did not work correctly when it was focused.
Fixes #83303

Thanks!


Teku fuzzing does not work

A little birdy suggested I do the following:
make teku
docker run -it -v $(pwd)/workspace:/eth2fuzz/workspace eth2fuzz_teku target teku_attester_slashing

The second command is not working:
Justins-MBP:eth2fuzz justin$ docker run -it -v $(pwd)/workspace:/eth2fuzz/workspace eth2fuzz_teku target teku_attester_slashing
[eth2fuzz] Testing FuzzerJavaJQFAfl is available
Performing pilot run....  Pilot run success! Launching AFL now...
[+] Loaded environment variable AFL_SKIP_BIN_CHECK with value 1
[+] Loaded environment variable AFL_AUTORESUME with value 1
[+] Loaded environment variable AFL_SKIP_CPUFREQ with value 1
[+] Loaded environment variable AFL_SKIP_CRASHES with value 1
[+] Loaded environment variable AFL_NO_AFFINITY with value 1
[+] Loaded environment variable AFL_I_DONT_CARE_ABOUT_MISSING_CRASHES with value 1
afl-fuzz++2.65d based on afl by Michal Zalewski and a big online community
[+] afl++ is maintained by Marc "van Hauser" Heuse, Heiko "hexcoder" Eißfeldt, Andrea Fioraldi and Dominik Maier
[+] afl++ is open source, get it at https://github.com/AFLplusplus/AFLplusplus
[+] Power schedules from github.com/mboehme/aflfast
[+] Python Mutator and llvm_mode whitelisting from github.com/choller/afl
[+] afl-tmin fork server patch from github.com/nccgroup/TriforceAFL
[+] MOpt Mutator from github.com/puppet-meteor/MOpt-AFL
[*] Getting to work...
[+] Using exploration-based constant power schedule (EXPLORE, default)
[+] You have 4 CPU cores and 4 runnable tasks (utilization: 100%).
[!] WARNING: Not binding to a CPU core (AFL_NO_AFFINITY set).
[*] Checking core_pattern...
[*] Setting up output directories...
[*] Scanning '/eth2fuzz/workspace/corpora/attester_slashing'...
[+] No auto-generated dictionary tokens to reuse.
[*] Creating hard links for all input files...
[*] Validating target binary...
[*] Attempting dry run with 'id:000000,time:0,orig:00000000000000000000000000000000.00000002.honggfuzz.cov'...
[*] Spinning up the fork server...

[-] PROGRAM ABORT : Timeout while initializing fork server (adjusting -t may help)
         Location : afl_fsrv_start(), src/afl-forkserver.c:682

[eth2fuzz] Fuzzer quit

Look like a default timeout issue, I just published a fix https://github.com/sigp/beacon-fuzz/commit/1b5cfd978afc35285a60ec8c365663d071cc2ee8
Please, do the following:
$ git pull
$ make teku

Let me know if it is not working.

Nope, still not working.

Can you give me your spec, because I suppose the fuzzer is not running fast enough. Teku fuzzing is really slow for now because it is running inside the JVM...

Yep, Teku's fuzzers are the slowest we have. Here's @JustinDrake 's laptop specs:

MacBook Pro (2016), macOS 10.14.6
2 GHz Intel Core i5
8 GB RAM

Hey @JustinDrake - we're pretty sure this is a timeout related issue, most likely due to the slowness of Teku's fuzzer.
I've created a dedicated branch that should hopefully address this problem. Could you please follow these steps (from the beacon-fuzz folder):

git pull
git checkout teku-timeout-fix
cd eth2fuzz && make teku CACHE=--no-cache
docker run -it -v $(pwd)/workspace:/eth2fuzz/workspace eth2fuzz_teku target teku_attester_slashing

Let us know if this fixes the problem for you. Cheers!

Nope, still not working :) I performed all the steps above.
Justins-MBP:eth2fuzz justin$ docker run -it -v $(pwd)/workspace:/eth2fuzz/workspace eth2fuzz_teku target teku_attester_slashing
[eth2fuzz] Testing FuzzerJavaJQFAfl is available
Performing pilot run....  Pilot run success! Launching AFL now...
[+] Loaded environment variable AFL_SKIP_BIN_CHECK with value 1
[+] Loaded environment variable AFL_AUTORESUME with value 1
[+] Loaded environment variable AFL_SKIP_CPUFREQ with value 1
[+] Loaded environment variable AFL_SKIP_CRASHES with value 1
[+] Loaded environment variable AFL_NO_AFFINITY with value 1
[+] Loaded environment variable AFL_I_DONT_CARE_ABOUT_MISSING_CRASHES with value 1
afl-fuzz++2.65d based on afl by Michal Zalewski and a big online community
[+] afl++ is maintained by Marc "van Hauser" Heuse, Heiko "hexcoder" Eißfeldt, Andrea Fioraldi and Dominik Maier
[+] afl++ is open source, get it at https://github.com/AFLplusplus/AFLplusplus
[+] Power schedules from github.com/mboehme/aflfast
[+] Python Mutator and llvm_mode whitelisting from github.com/choller/afl
[+] afl-tmin fork server patch from github.com/nccgroup/TriforceAFL
[+] MOpt Mutator from github.com/puppet-meteor/MOpt-AFL
[*] Getting to work...
[+] Using exploration-based constant power schedule (EXPLORE, default)
[+] You have 4 CPU cores and 2 runnable tasks (utilization: 50%).
[+] Try parallel jobs - see /usr/local/share/doc/afl/parallel_fuzzing.md.
[!] WARNING: Not binding to a CPU core (AFL_NO_AFFINITY set).
[*] Checking core_pattern...
[*] Setting up output directories...
[+] Output directory exists but deemed OK to reuse.
[*] Deleting old session data...
[+] Output dir cleanup successful.
[*] Scanning '/eth2fuzz/workspace/corpora/attester_slashing'...
[+] No auto-generated dictionary tokens to reuse.
[*] Creating hard links for all input files...
[*] Validating target binary...
[*] Attempting dry run with 'id:000000,time:0,orig:00000000000000000000000000000000.00000002.honggfuzz.cov'...
[*] Spinning up the fork server...

[-] PROGRAM ABORT : Timeout while initializing fork server (adjusting -t may help)
         Location : afl_fsrv_start(), src/afl-forkserver.c:682

[eth2fuzz] Fuzzer quit

Interesting, you seem to be the only person with this issue (we confirmed with other MacOS users that they're able to run this fuzzer).
After [*] Spinning up the fork server..., did it hang for 5 minutes before the next error message? Asking since I changed the timeout value to 5 minutes. Thanks!

After [*] Spinning up the fork server..., did it hang for 5 minutes before the next error message?

Nope, it does not. It fails immediately after this message.

I'm on the correct branch and I did recompile teku as above:
Justins-MBP:eth2fuzz justin$ git branch
  master
* teku-timeout-fix

Ok, turns out an environment variable used within AFL might be overriding the timeout value. We just (hopefully) fixed this in the latest commit on teku-timeout-fix (commit 753b00a4). If this doesn't work, I suspect it'll most likely be related to something else (perhaps an OS limitation on Mojave).
Please follow these steps, assuming you're on the teku-timeout-fix branch  (it should take you at least 30+ minutes to rebuild the docker):

git pull
cd eth2fuzz && make teku CACHE=--no-cache
docker run -it -v $(pwd)/workspace:/eth2fuzz/workspace eth2fuzz_teku target teku_attester_slashing

If this doesn't work, we'll have to investigate further.


DONOTMERGE langserver: Publish empty diagnostics

We publish empty diagnostics if we had a typecheck cache miss (ie we actually
did typechecking). This change works since at this point diags is nil
https://sourcegraph.com/github.com/sourcegraph/go-langserver@39e41a9f7071d047fc97dfe52d51dca11d6ac04b/-/blob/langserver/loader.go#L197:1-198:1
but is only updated to the diags returned by typecheck if we have a cache
miss. typecheck always returns a non-nil (but possibly empty) diags after this
change.
I'm not 100% sure if this solves the underlying issue, since the list of diagnostics we are publishing here are specific to a file and some extra dependencies.
Fixes #23

I'm not 100% sure if this solves the underlying issue, since the list of diagnostics we are publishing here are specific to a file and some extra dependencies.

What do you mean by that? publishDiagnostics is always called for a single file.

yeah I realise this isn't explained very well. So the more I think about it, the more I think this probably doesn't fix the bug. So if we report an error for a file and it is later fixed, that means we need to publish empty diagnostics for that file. Currently this change is actually a no-op, without me realising. An empty diagnostics does nothing. When I wrote this I thought publishDiagnostics would be a single notify call for the whole workspace. https://sourcegraph.com/github.com/sourcegraph/go-langserver@39e41a9f7071d047fc97dfe52d51dca11d6ac04b/-/blob/langserver/diagnostics.go#L20:23-20:41
I will update.

After more thought this is a bit tricky for us to do, because we don't really keep good track of files we have or have not typechecked. I am going to abandon this PR and instead disable diagnostics by default since no one actually consumes them (yet)


Set IFS only locally for read in deploy_documentation_tags.sh

Summary
Previously IFS was set for the whole bash setting, which later affected
the word splitting when the secrets were read from the variables and
split into the openssl command.
This also tidies up the array reading - previously, the VERSION
variable was an array (so echo $VERSION would be equivalent to
echo ${VERSION[0]}), and it was passed to cut, which isn't
necessary, since it was already split.

Details and comments
@mtreinish: I think this is the problem on the docs tag deploy action.

I don't think we can manually trigger a re-run of the tags build with the new file until another release goes out, but that should go out with Aer 0.10 next week anyway.


BZ1264339: Guided Rule Editor: List contains condition missing bound value on re-opening rule

See https://bugzilla.redhat.com/show_bug.cgi?id=1264339
(cherry picked from commit a59b3c4dfd5ec114b9af0f204186c6d688750989)

+1

+1 if the tests finish ;)
I trust that they do, since master worked.


remove unused template provider

what

remove unused template provider

why

template provider does not have darwin/arm64 binary, modules including it don't work on m1 mac.

/test all

sorry @Gowiem, I didn't actually remove the dependency on that provider. looking into that now. It's a little complicated because of the way it was being used.

@kevcube Ah yeah, I see what you mean. Give it a shot and ping me when it's ready for review 👍

@Gowiem I'm not super happy with the solution but it looks like it might be what's necessary. please kick off tests and let me know if the replace(replace(replace(...))) is acceptable

/test all

@kevcube haha yeah, I wouldn't be happy with this solution either, but your comment with the link to @nitrocode's workaround / the actual terraform issue is a good one and provides the context that needs to be shared.
This is good from my end, but I also do want to pull @nitrocode in so he can weigh in if there is another way that he has been solving this recently.

/test all

@kevcube got another one for you:
TestExamplesComplete 2022-02-16T23:00:02Z command.go:121: Error: Invalid function argument
TestExamplesComplete 2022-02-16T23:00:02Z command.go:121:
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:   on ../../main.tf line 24, in resource "aws_route53_zone" "default":
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:   18:   name = replace(replace(replace(replace(replace(replace(replace(replace(var.zone_name,
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:   19:     "$$$namespace", module.this.namespace),
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:   20:     "$$$environment", module.this.environment),
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:   21:     "$$$name", module.this.name),
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:   22:     "$$$stage", module.this.stage),
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:   23:     "$$$id", module.this.id),
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:   24:     "$$$attributes", module.this.attributes),
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:     |----------------
TestExamplesComplete 2022-02-16T23:00:03Z command.go:121:     | module.this.attributes is empty list of string

oops! @Gowiem fixed that one too.

/test all

@Gowiem last error is kinda obscure, I pushed up what might be a fix, can you re-test?

/test all

Thanks for the help so far @Gowiem I'm going to set up a local terratest environment so I can iterate before asking you again 😝

/test all

@kevcube Thanks for the contribution! Released as https://github.com/cloudposse/terraform-aws-route53-cluster-zone/releases/tag/0.14.0


Add completeness counts to subject sets

For Engaging Crowds, we'd like to display subject set completeness when giving volunteers a choice of subject sets to work on for a particular workflow. It would be useful to have completeness, or number of completed subjects, recorded per subject set, if that's possible.

One possible complication here: a subject set can be linked to multiple workflows, and will have different completeness counts for each workflow that uses it.

There's also conversation on https://github.com/zooniverse/front-end-monorepo/issues/1884, around how we'll want to list individual subjects from a set and show workflow status for each one, for the current user.

Linking https://github.com/zooniverse/front-end-monorepo/issues/1884#issuecomment-726006195
One naive approach could be to do this when doing the project completion counts on the hourly schedule? https://github.com/zooniverse/panoptes/blob/e4dcb068ad39bddd960423ef6e794f9917e940ca/app/workers/calculate_completeness_worker.rb
I'm not sure what kind of load the above would add to the system, it might be more prudent to modify the set counts on each retirement event? https://github.com/zooniverse/panoptes/blob/e4dcb068ad39bddd960423ef6e794f9917e940ca/app/workers/retirement_worker.rb#L12
Once those counts are on the SubjectSet resource we can serialize them via the API on the subject set resource.


User Authentication in the app

We would want to have login functionality but the app features shouldn't depend on that. So, if a user logs in, he gets a personalised experience with his custom settings but if he doesn't login he gets to see all the basic information and features like talks, bookmark sessions etc.

[ ] Google+ sign in
[ ] Facebook Oauth
[ ] Twitter Oauth
[ ] Server only login ( This is more inclined for the f droid version where we can't be using the above three login mechanisms )
[ ] No login at all ( Skip login )

@mariobehling @the-dagger I'd like to work on this issue so as to get a basic understanding of codebase of Android client.

@brainbreaker Please feel free to make a proposal. Integrating this feature should be done later, when there are use cases implemented as well though.

@mariobehling @mananwason Can i work on this??

Not required yet because server doesn't support it. Cheers!
On Mon 16 Jan, 2017, 7:41 PM Saket Kumar<EMAIL_ADDRESS>wrote:

@mariobehling https://github.com/mariobehling @mananwason
https://github.com/mananwason Can i work on this??
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/issues/363#issuecomment-272870851,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKFKCT4DG8daxxfZWhSErxHNtkqCbUksks5rS3p2gaJpZM4IgtuF
.

--
Regards,
Manan Wason
4th year, B.tech - CSE
Indraprastha Institute of Inf. Tech. (IIIT) Delhi
Okhla, Phase-III, New Delhi-110020, INDIA
Website: www.mananwason.me http://www.mananwason.me/

can this be done using FireBase?

@Dipendra, I don't think firebase is the way to go here. All our data comes
from the JSON fetched from open-event server. It is not gonna be useful
unless we host our JSON database on firebase as well. The server has to
have the API to support G+/FB authentication.
On Tue, Feb 28, 2017 at 6:59 AM, Dipendra Singh<EMAIL_ADDRESS>wrote:

can this be done using FireBase?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fossasia/open-event-android/issues/363#issuecomment-282913041,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJE8R8JBR3lh6H-48se5vDejO-DQTFc_ks5rg3iPgaJpZM4IgtuF
.

@mananwason Please reopen this issue :smile:

@Shailesh351 As there is quite a bit of work about storing of the token and revalidating it to check if it is expired or not, you can check the Orga App implementation and reuse code from it. It is quite modular :smile:

@Shailesh351 also ensure all user authentication related features are toggle-able. That is we should be able to generation a version of the app that doesn't have any user authentication related features or features that need user authentication. We want give an option of a completely offline app too to the users.
It would be best if this requirement is taken into consideration from the very start of this.

@niranjan94 Sure!!


Find thick dual carriageway roads that overlap, and cut their width in

half. Only rolled out to one map to start.
How can we deal with situations like this:

where thickened roads smush into each other. This is near https://www.openstreetmap.org/node/53179301, and the root problem is bad guesses for the width of the road, or the center-line of each piece of the road not really being the thing that's mapped in OSM.
Let's just apply a very blunt hammer and shrink both roads in half:

A 1.2 meter lane for cars is absurd, of course, but I'm going to vote that this is better than the smushed geometry and all of the other problems that come with it.
Only rolling out to one map for now; this is in service of fixing blockfinding and needing to see what's even happening near here. Much more testing needed before wider rollout

Aaargh actually this is a binary format change for all RawMaps and I don't want to push this yet. I'll just hold off.

Reviving this, rolling out to all maps. Some more places that look improved... Before:

and before:

after:

Regenerating everything now, will merge if nothing regresses terribly. Lane markings look too thick, and agents on the lanes will of course visibly overlap. But better than endless downstream geometry issues.

Geometry improved a bit everywhere I checked. The simulation in arboretum breaks somehow, removing prebaked data there -- my priorities are clear. :)


[Feature] Limit intellisense within the classes

Greetings,
Does the extension work so that its intellisense is limited to html tags? If I press ctrl+space outside html tags, I still get all of the alpine directives and stuff.
Thanks.

It is not. I'm not quite sure how to go about doing that (I'm pretty sure I'm limited based on my current implementation). Feel free to put in a PR if that's something you want to give a shot at!


WCAG 1.3.1 Info and Relationships G57 needs test

G57: Ordering the content in a meaningful sequence LTS Note:

In progress... seems like everything is pretty straightforward in terms of structure as-is but continuing to explore.


Rename generate to iterateUntil and introduce new generateM

I'm planning an 0.3.0 release and decided that I'd like to rename generate and generateM to iterateUntil and iterateUntilM (to avoid the confusing collision with Play's naming). I've also introduced a new generateM that works like Play's.
/cc @Crafter6432

Shouldn't f in generateM be a function instead of a value ?

@Crafter6432 For most of the contexts this library is likely to be used with (cats.Eval, Scalaz's Task, my new Rerunnable wrapper for Twitter futures), computations are (or at least may be) rerunnable. Most things will work with contexts that don't support this (like the standard library's Future), but nothing in the API is specifically designed to accommodate them.
I'd prefer to keep generateM as is because of things like Eval.always vs. Eval.later, which will behave differently if we have a by-name parameter here.

Current coverage is 99.14%

Merging #61 into master will increase coverage by +0.01% as of c4d6e80

@@            master     #61   diff @@
======================================
  Files           17      17
  Stmts          461     466     +5
  Branches        37      38     +1
  Methods          0       0
======================================
+ Hit            457     462     +5
  Partial          0       0
  Missed           4       4

Review entire Coverage Diff as of c4d6e80

Powered by Codecov. Updated on successful CI builds.


[Bug]: Passing multi line of text to API

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
WHEN a user enters in a multi line of text into the input widget (that has been expanded in height)
AND there is an action OnClick of a button
THAT uses the api.run() function to pass through some parameters into an API
THEN the api fails to parse any values
Following the same steps but ONLY replacing the multi line of text with a single line of text in the same widget the API runs just fine.
Interestingly however, if you enter a single line of text with the following string 'Multi\nLine' it also works just fine.
API Note: Smart JSON Substitution was disabled
API POST body Example
{
  'id': {{this.params.id}},
  'value': '{{this.params.value}}'
}

JS Example's
please be aware that I am using a custom object inside the store (appsmith.store.param) that is an object with an id parameter.
api.run(null,null,
{
  'id': appsmith.store.param.id,
  'value': inputWidget.text
})

api.run(null,null,
{
  'id': appsmith.store.param.id,
  'value': `${inputWidget.text}'
})

api.run(null,null,
{
  'id': appsmith.store.param.id,
  'value': inputWidget.text.toString()
})

Steps To Reproduce
1 - Add input widget for text
2 - Add button
3 - Add POST API that takes a body param of type string
4 - Add js to button to submit API with input text param
5 - Enter multi line of text in input widget
6 - Click button

Environment
Release
Version
Cloud

Hey @Nikhil-Nandagopal looks like this kinda works 😆.
With smart substitution off and the following JS Input1.text.replaceAll('\\n','\\\\n') this works.
Looks like there is an issue with whatever parser you are using for the JS in the widgets because I am unable to write the \n otherwise I get a syntax error.


graphql: Log details about subscription execution

The logging is done in a way that qlog will ignore these log messages for now. But we need some indication of how slow subscription queries are in the logs

This isn't needed anymore.

This isn't needed anymore.


nixos/freeradius: fix group assignment for radius user

Description of changes
Fixes a regression caused by https://github.com/NixOS/nixpkgs/pull/265762#issuecomment-2383251728

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Sorry, this is my negligence. I had group = "radius"; in my config but I think a mistake was made when transferring to my nixpkgs PR.


lld: dropped package

No point in keeping this here since Zig was dropped -- no other package needs it.
Existing package

[x] I am the maintainer of this package.

I can adopt it on the next update to lld. It's useful as a general linker too (faster) and needed for chromium LTO/CFI

If you want it to stay here I'm happy to continue to maintain it :)


Materialize virtual arrays in to_arrow

Was this simply missing? If so, thanks for noticing this!
Whereas ak.to_buffers might have an experts-only persist_virtual option, ak.to_arrow can't (no virtual Arrow).
And, of course, let me know if you're done with this branch and I'll merge it after tests pass.

Was this simply missing? If so, thanks for noticing this!
Whereas ak.to_buffers might have an experts-only persist_virtual option, ak.to_arrow can't (no virtual Arrow).
And, of course, let me know if you're done with this branch and I'll merge it after tests pass.

I'm done

I'm done


Incomparable variant attribute

[x] implement for PartialOrd
[x] does it make sense on structs?
[x] don't allow item-level and variant-level incomparable at the same time
[x] documentation
tests

Unit tests

[x] Enum with an empty variant and an incomparable empty variant.
[x] Enum with an empty variant and an incomparable non-empty variant.
[x] Enum with multiple incomparable variants.
[x] Enum with an empty variant and multiple incomparable non-empty and empty variants.
[x] Enum with a skipped variant and an incomparable variant.
[x] Enum with skipped, and incomparable on the same variant
[x] Item

[x] Integration tests
UI tests

[x] Incomparable on item + variant
[x] Incomparable without PartialEq/PartialOrd
[x] Incomparable with Ord/Eq

We should get our CI working again :D

We have some funny file permission changes going on there, but it shouldn't have been executable to begin with ... so looks okay to me.

Yeah, I noticed they were x so I removed that.

There are probably more variations that we should unit test:

Enum with an empty variant and an incomparable empty variant.

Enum with an empty variant and an incomparable non-empty variant.

Enum with multiple incomparable variants.

Enum with an empty variant and multiple incomparable non-empty and empty variants.

Enum with a skipped variant and an incomparable variant.

We should also not forget integrations tests, error message tests and documentation.

Definitely, I just wanted to lay out an initial implementation

Open questions:

Should we even allow an enum having all variants incomparable?

I'd say that is valid, while probably seldom useful. If for some reason something has an PartialEq trait bound, but you're fine with it never succeeding, you could make every comparison false. If we support this use case, it could make sense to support the attribute on item level as well.

Whats the interaction with PartialOrd here? Was this a mistake or do you intend to add an implementation?

It has to match, i.e. PartialOrd should always return None.

Almost forgot, we need a Changelog entry!


NTVS 1.3 Beta

:runner: in progress and preliminary planning notes for next release.

This release will ship for:

VS 2015 (Dev14)
VS "15" Preview (Dev15)

For the most part, we will be targeting Dev15 because we want to take advantage of new platform improvements, but we intend to maintain our Dev14 bits in parallel
Emoji legend

:hotsprings: red-hots (like the candy!): should happen this week, otherwise may impede future efforts
:busts_in_silhouette: external dependency: requires coordination
:no_entry_sign: blocked
:crown: nice to have

"Big Rocks"
Product
Setup authoring improvements (VS "15")
For this release, we'll continue to test and improve VS "15" setup / acquisition experience
Regressed functionality

[x] #1222 Azure deployment doesn't work
[ ] #1180 Cannot Discover Unit Tests
[ ] #1179 Cannot Find RemoteDebugProxy folder in-progress

Too much functionality

[ ] #1218 UGH C#/VB get out of my NODE workload! Is that why you're so big??
[ ] #1223 Default workload install size is 1.2 GB
[ ] #1220 Profiling component is huge (>3GB)
[ ] #1224 Unnecessary context menu clutter in Node.js workload

Node.js Component / Workload definition

[x] #1221 Node.js v6 x86 should be recommended in default install

Folder-based projects

[ ] Improve File > Open Folder IntelliSense experience  (:busts_in_silhouette: VS Platform - wibu)
[x] Take ownership of debug adapter code
[ ] Provide "asks" for task runner experience w/ folder-based projects

Improved ES6 / IntelliSense experiences

[x] #1219 F12 on routes.index doesn't work in Dev15 (:busts_in_silhouette: TypeScript)
[ ] #1090 Enable ES6 IntelliSense in Node.js Interactive Window ( :busts_in_silhouette: Roslyn, :busts_in_silhouette: TypeScript)
[ ] #1225 shouldn't generate typings.json in Dev15 project templates

Improved Stability and Performance

[ ] #443 Interactive Window package should be loaded lazily
[ ] Investigate and fix miscellaneous stability issues

User Feedback and engagement

[x] #965 Agree upon and add any additional telemetry points

(Mostly) Code Happiness

[ ] CTI runs
[ ] driving dev build usage :bulb:

Targets (subject to change)

9/12: "Big Rocks" complete
9/15-9/19: Stabilization and release activities
TBD: Release

Closing since this is no longer reflects our current plans.
We currently are planning to ship NTVS 1.3 as part of VS 2017. Before VS 2017 is released in its final state, we'll continue to ship preview NTVS 1.3 builds as part of the Node workload. These are essentially our beta/preview releases.
For VS 2015, we're still discussing how we want to release 1.3. The bulk of 1.3 work has been to make NTVS a more stable, performant, and modern Node tool, with a special focus on VS 2017 support. I believe 1.3 has some real benefits, even for VS 2015, but it also brings a few regressions in functionality (such as no IntelliSense in the interactive window and removal of many of our old templates).
Until our plans are better known, I'll keep posting dev builds for VS2015 every week or so. You can also pick up NTVS 1.3 preview builds in VS 2017 ("VS15") by selecting the Node workload in the installer.


Feature: podman support

With Fedora30 the cockpit podman plugin is supported and should be included as a new role.

not supported in redhat


Update token type definition and arg description in hf_file_system.py

Refactor token argument in hf_file_system.py for consistency across the hub.

Standardize the type definition of the token parameter to Union[bool, str, None] = None
Unify docstring descriptions for the token parameter to enhance clarity and usage guidance

See #2251 and #2252 for related discussions.
Thought i just go on and open a PR for this.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


empty transaltion file

The main translation csv file has been deleted,

We have fixed it. You can re-install the language pack and check it again.


cmd/compile: panic: dvarint: bad offset for funcdata

What version of Go are you using (go version)?

$ go version
go version go1.18.1 windows/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=on
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\ddaa\AppData\Local\go-build
set GOENV=C:\Users\ddaa\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\ddaa\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\ddaa\go
set GOPRIVATE=
set GOPROXY=https://goproxy.cn,direct
set GOROOT=C:\Program Files\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\Program Files\Go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.18.1
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=D:\programing\go\bugReview\go.mod
set GOWORK=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\ddaa\AppData\Local\Temp\go-build2453048728=/tmp/go-build -gno-rec
ord-gcc-switches

What did you do?
https://go.dev/play/p/SJUm7jbaIjh

What did you expect to see?
The compiler don't panic and compile successfully or report "stack frame too large (>1GB)"
What did you see instead?
panic: dvarint: bad offset for funcdata -<PHONE_NUMBER>
goroutine 20 [running]:
cmd/compile/internal/ssagen.dvarint(0xc00031c200?, 0xc0000e8a08?, 0x11?)
/usr/local/go/src/cmd/compile/internal/ssagen/ssa.go:287 +0x205
cmd/compile/internal/ssagen.(*state).emitOpenDeferInfo(0xc0002ead00)
/usr/local/go/src/cmd/compile/internal/ssagen/ssa.go:326 +0xfe
cmd/compile/internal/ssagen.buildssa(0xc0003bef20, 0x1)
/usr/local/go/src/cmd/compile/internal/ssagen/ssa.go:605 +0x1ffd
cmd/compile/internal/ssagen.Compile(0xc0003bef20, 0xc00003cf90?)
/usr/local/go/src/cmd/compile/internal/ssagen/pgen.go:183 +0x4c
cmd/compile/internal/gc.compileFunctions.func4.1(0x0?)
/usr/local/go/src/cmd/compile/internal/gc/compile.go:153 +0x3a
cmd/compile/internal/gc.compileFunctions.func3.1()
/usr/local/go/src/cmd/compile/internal/gc/compile.go:140 +0x4d
created by cmd/compile/internal/gc.compileFunctions.func3
/usr/local/go/src/cmd/compile/internal/gc/compile.go:138 +0x78
Go build failed.

@cherrymui @dr2chase  This looks like one of ours, someone has been fuzzing the compiler.  I don't know that it actually requires generics.

This is failure to correctly complain about a ginormous stack frame, not generics -- generics just allow us to say "ginormous stack frame" in fewer words.  This can be seen by replacing the array dimensions "10" and "6" with constants, and varying them between larger and smaller sizes and watch the error come and go.
That said, we should complain more gracefully.

As a simpler reproduction case, this short program fails to compile with panic: dvarint: bad offset for funcdata -<PHONE_NUMBER>.

The problem comes from emitting open defer info. It's even worse because it could prevent valid code to be compiled. For example, this program should compile ok, but panic during runtime due to stack overflow:
package main

func g() {
	xs := [32537630 / 8][33]int{}
	for _, x := range xs {
		if len(x) > 50 {

		}
	}
}

func main() {
	defer f()
	g()
}

func g() {}

but currently panic with open defer.


Profile fix merge time of runtime filter

Proposed changes
fix merge time of runtime filter

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Missing CSQlite on Jenkins Server.

Hello,
I'm currently integrating your Framework, and everything works on my local machine, but as soon as i check out the project on an other machine or when Jenkins is building the app, the "CSQlite" is missing.
I do have XCode 8.2 and added the Framework to the XCode project.
Can you help me?
Greetings

Please try the current master branch, should be fixed there

Hey,
I tried it, but I do get the error:
error: missing required module 'CSQLite' import SQLite
again.
I pulled, and build the framework locally via XCode 8.2.1.
After that I inserted the .framework file into the project and added as library. Then I build the project.

Sorry, I can't reproduce your problem. Maybe try with a simple project from scratch?

create new project
git clone git@github.com:stephencelis/SQLite.swift.git
drag SQLite.xcodeproj into project
Add SQLite.framework under "Linked Frameworks and Libraries"
Run

It works now.
I followed your instructions and after that I had to import my source code from the old project.
Simple drag and drop the SQLite.xcodeproj in the old project didn't worked.
Thanks for your help.


Existing users login issues

As we know, some existing users are experiencing login issues after the migration.  Relevant slack thread.
Users in the thread have mentioned

that they can log in via different browser (or incognito)
clearing cookies / cache did not help

I imagine the issue has something to do with a pre-existing roamjs:oauth-google:{graph} local storage object.
Noting my superficial exploration so far for future me:

index.ts > settings.panel.create (Log in) > roamjs-components OauthPanel

Probably related: https://github.com/RoamJS/google/issues/9
Tagging @dvargas92495 for visibility as he was working on this issue recently

if it has to do with local storage, logging out and back in should fix. I'm seeing some undefined errors now

Actually that id issue is Dropbox related and should now be fixed. I'll keep an eye out for any google-related emails

it's probably disappears bc it's crashing after the error - it's existence is not dependent on whether there's a value set there or not.
That 500 should be a clue - let's see if there are any errors sent

Here's what I meant about the value being set:
If I remove the value, I see Login with Google.
If I set a fake value, I see Add Another Google Account as if I'm logged in.
https://github.com/RoamJS/google/assets/3792666/8571f7c1-3492-4a1b-81f4-9c90fdf1e602
RE: 500 error
Nice catch, I wasn't looking for that.  More often than not it doesn't show up.  And it doesn't seem to affect my ability to login.
https://github.com/RoamJS/google/assets/3792666/252af9f6-1689-4208-a0ad-18a1458c5881
Continued auth request
Another think I noticed is that sometimes it will continue requesting. @ 45 seconds in
https://github.com/RoamJS/google/assets/3792666/5484bd0e-ab13-4e9f-9f2c-e43601e4e950


Manual cast: entry not working

Home Assistant release with the issue: 0.73.1

Last working Home Assistant release (if known): N/A
Operating environment (Hass.io/Docker/Windows/etc.):
Hassbian.
Home Assistant installed in a python virtual environment on a Raspberry Pi 3.
Component/platform:

https://www.home-assistant.io/components/cast/
Description of problem:
When addin a google cast device manually to the configuration, it does not get loaded/added to Home Assistant.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
Tried this from the docs:
cast:
  media_player:
  - <IP_ADDRESS>

and:
cast:
  media_player:
    - <IP_ADDRESS>

Traceback (if applicable):
2018-07-10 06:26:15 INFO (MainThread) [homeassistant.loader] Loaded cast from homeassistant.components.cast
2018-07-10 06:26:17 INFO (MainThread) [homeassistant.setup] Setting up cast
2018-07-10 06:26:17 INFO (MainThread) [homeassistant.setup] Setup of domain cast took 0.0 seconds.
2018-07-10 06:26:17 INFO (MainThread) [homeassistant.core] Bus:Handling <Event component_loaded[L]: component=cast>

Additional information:
If I use the old way of adding this devices:
media_player:
  - platform: cast

It is available in Home Assistant, but gives this warning on startup:
WARNING (MainThread) [homeassistant.components.media_player.cast] Setting configuration for Cast via platform is deprecated. Configure via Cast component instead.

I have the same problem with cast.
Running Home Assistant in a container on an Ubuntu Server 18.04 host.
It worked on previous versions of Home Assistant but stopped, unless I use the deprecated method mentioned above.
The hardware device is a Google Home.
I've tried going through Configuration - Google Cast, but it doesn't find the device (and as I'm running the container bridged, not home/host, I'm not surprised).

Having the same issue.  Looks like this was also brought up under Hass.io as an issue.  Seems to mention the cause as well.
https://github.com/home-assistant/home-assistant.io/issues/5673

Did you enable the Cast component by going to the Integrations page?

Sure, I will get it up later tonight.  Only thing that might impact this, is that I did a downgrade from 73.1 to 72.1 after finding the DHT issue.  Not sure if it retained any of that.
On the flip side, I am actually going out in a bit to get a new RPI3 B+ to replace my 2B.  Going to do from scratch install to correct some mistakes I made when first setting it up.  See what happens then.

Anybody know what's the difference between the cast component and the old media-player component? with the old media-player component as mentioned above I can with no problems manually define and add the devices with the old media-player component, but  when trying to do the same with the cast component I can't find my devices. also tested to find them with the Integrations feature.
also as mentioned above, I can find my devices on the network. but not in home-assistant.
Running with docker and no network_mode specified.

Doesn't work for me either.
My config:
#discovery:

cast:
  media_player:
  - host: <IP_ADDRESS>

I'm not sure if I'm supposed to click CONFIGURE in Integrations (why should I?), but if I do I get a window with
Aborted

No Google Cast devices found on the network.


added tests that work in Windows CI

Windows CI wasn't testing our ensurePosix fix, because the input paths were posix only. This injects a normalized input path in all our walkSync calls, mainly for cleanliness so we don't have to update all the references in the tests.

Excellent, thanks @kellyselden!


[mandatory-inlining] Add more sil tests that use non-trivial parameters.

This makes sure that when we inline a partial apply, we put the release from the
@callee_owned call in the appropriate place.
rdar://31521023

@swift-ci smoke test and merge


Caching a header from one route to another

Hi,
I am trying to cache a certain header (CamelCaffeineKey) to use it in another route so I have tried caffeine-cache but I dont seem to be able to make it work, could you please explain to me how to use it / load the cache from the first route to the other route?

@davsclaus is Caffeine Cache component scope is Route or CamelContext?

You can bind the cache to registry and re-use the same instance in multiple routes of the context https://camel.apache.org/components/4.0.x/caffeine-cache-component.html#_examples

its context scoped (in reality per component), also it creates a default in-memory cache if not specially created as the link from Andrea above.

Thanks @davsclaus and @oscerd
@samar-elsayed you can not put header directly to Caffeine. Set the data you want to cache to exchange body and then put into caffeine cache. In the second route get data from caffeine (it will be in exchange body) and then setHeader from the body.
- route:
    id: route-cbf4
    nodePrefixId: route-1f7
    from:
      id: from-7856
      uri: kamelet:timer-source
      parameters:
        message: Hello
      steps:
        - setBody:
            id: setBody-eaa0
            expression:
              simple:
                id: simple-3dc1
                expression: ${date:now}
        - to:
            id: to-604c
            uri: caffeine-cache
            parameters:
              cacheName: demo1
              action: PUT
              key: key1
- route:
    id: route-778e
    nodePrefixId: route-289
    from:
      id: from-aedc
      uri: kamelet:timer-source
      parameters:
        message: world
      steps:
        - to:
            id: to-191e
            uri: caffeine-cache
            parameters:
              cacheName: demo1
              action: GET
              key: key1
        - log:
            id: log-d474
            message: ${body}


⚠ Bofrost.be has degraded performance

In 2481344, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2235 ms

Resolved: Bofrost.be performance has improved in 46620be.


Problem with navigateByUrl in Angular

Hi,
Thanks for the excellent plugin. I was trying it in my Angular project with NativeScript 4.0.0. Everything was working great except when I was trying to navigate to another page from tap event of the listed item, that time I am getting only white page.
private _showPopup(source, view) {
        this.popup = new Popup({
            height: 50,
            width: 40,
            unit: '%',
            elevation: 10,
            borderRadius: 25
        });
        this.popup.showPopup(source, view).then(data => {
            //console.log(data);
        });
    }
    private hidePopup(index?) {
        let t = this;
        this.popup.hidePopup(index)
    }
public openPopUp() {
        let t = this;
        const stack: StackLayout = new StackLayout();
        stack.paddingTop = 30;
        stack.paddingLeft = 30;
        stack.height = "auto";

const lbl4: Label = new Label();
        lbl4.text = 'Profile';
        lbl4.marginBottom = 5;
        lbl4.marginLeft = 8;
        lbl4.height = 40;
        lbl4.on("tap", function () {
            t.routerExt.navigateByUrl("/login");

        })
        stack.addChild(lbl4);

        const sv = new ScrollView();
        sv.content = stack;
        this._showPopup(this.page.getViewById('profile'), sv);
}

Any hints in where I am doing mistake?
Thanks

Have you tried closing the popup when you navigate

@triniwiz Thanks for your quick response. Yes, I tried that too like this:
t.hidePopup();
setTimeout(() => {
       t.routerExt.navigateByUrl("/login");
}, 500);

But no luck :(

Any error?
On Wednesday, April 18, 2018, 1:38 PM, Jibon Lawrence Costa<EMAIL_ADDRESS>wrote:
@triniwiz Thanks for your quick response. Yes, I tried that too like this:
t.hidePopup();
setTimeout(() => {
t.routerExt.navigateByUrl("/login");
}, 500);
But no luck :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Actually not error but blank page like this

Try subscribing to the router and see what page you are on
On Wednesday, April 18, 2018, 1:49 PM, Jibon Lawrence Costa<EMAIL_ADDRESS>wrote:
Actually not error but blank page like this
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@triniwiz will you please give some hints to do so?
Thanks

@triniwiz I tried to check where it's going to that login page or not. I saw it isn't displaying any contents if I tried to display any data. So, I ain't sure if that is really going to appropriate page or not.

@triniwiz Any hints to solve this problem?


Fix object property shorthand

Fixes #49. This will transform instances of:
{
  name
}

into
{
  name: it.name
}

thank you! 🎉


send .to_i to datacenter arg before using to select from the available list

This makes knife linode server create work for me.

What values could be returned otherwise? Did you run into an exception that triggered this change?

The plugin wasn't creating servers before (there was an exception, but I don't remember what it was) because the linode_datacenter arg/config value was a string but the values in connection.data_centers were integers. They are always datacenter id numbers, so .to_i should be safe to call if the value has passed validation.

Alright, I was just interested why Lamont hadn't seen this issue and being sad that we don't have unit tests.


[FR] enable hasStyle= for MessageDialog/BuiltInDialog

I'm using MessageDialog as error message dialog for FileChooserDialog error. Would like to be able to style the text. It's currently pretty hard to even see the MessageDialog over the normal app window, at least with dark DE theme. Not super readable either.
thanks

Should be fixed here: https://github.com/can-lehmann/owlkettle/commit/bec3124214faccd223defeb44515893d7c72c4e9

yes, it works now! thanks


[c++] [draft] Support Arrow ordered enumeration data on read

Issue and/or context:
This PR adds support for return Arrow tables with dictionaries that can include ordered enumerations.
Changes:
Given #1559 which it depends upon, a very small change to just three files in libtiledbsoma.
This should become clearer once the dependent PR is merged and can be rebased.
Notes for Reviewer:
SC 34073

Codecov Report
Patch has no changes to coverable lines.

:exclamation: Current head c21c853 differs from pull request most recent head 8571ae0. Consider uploading reports for the commit 8571ae0 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:loudspeaker: Thoughts on this report? Let us know!.

@ihnorton has stacked this atop #1559

Kinda sad to see once-clean branches die.  The price of progress.


Unexpected Amplitude snipped included along with Google Analytics

Issue:
When using Google Analytics through astro-analytics, an unexpected Amplitude Analytics snippet is also being included
Repo with reproduction:
Included Google Analytics component: https://github.com/iron-cherep/astro-analytics-repro/blob/23314b745c3af5c326851746441d03e2b353ef47/src/pages/index.astro#L16
Amplitude snippet in resulting build: https://github.com/iron-cherep/astro-analytics-repro/blob/23314b745c3af5c326851746441d03e2b353ef47/dist/index.html#L9

I tried to check the code, but I cannot see why this snippet is being included. But this is kind of breaking the Google Pagespeed score as well 😅
I will try to dig a little deeper tonight see if I can fix it.

Same issue for me. I am trying to use the Umami component but I see Amplitude errors in console and network tab on website. Not sure why it is being included by default.

I checked it a while ago, it seems like the original JavaScript code from Amplitude auto injects itself in some way. But this would mean every time an update comes out, that we need to manually edit the code that they use if we try to change it our-self.
Eventually I switched away from astro-analytics, as this cannot be solved without modifying the code from Amplitude itself.

I tried importing the analytics directly from src/foo.astro but it wouldnt let me.
The problem arises because the index.ts imports / exports all astro components.
If we could import the component we need directly, it would work

same issue , when i want to use only google analytics and i had an error on the console that i miss an api key for amplitude


Search gets cropped in compact size

Searchbar in the Settings gets cropped in compact size (4k Screen)

Can confirm, same issue on 1920x1080. The marging between the menu left of the search box and the box itself is too large, or minium window size should be increase slightly.

or it should minimize to a button , which expands, when you click it

or it should minimize to a button , which expands, when you click it

The search box doesn't even work yet so that will be done after implementing search.

This issue has been fixed with v0.11.


Add missing comment

I've noticed that there is a comment missing for exactly one (WM_DEVICECHANGE) event in Window implementation for Win32.
This change fixes that.
Since only comment was added, there is no testing was done.

SFMLBuildMaster: Build this please

SFMLBuildMaster: Build this please.

Thanks for this small change 🙂


Release pipeline needs to support more then the MSShared Key

Currently the release pipeline expect the only key that will be release is 31bf3856ad364e35 but there are some other keys that are being used that we need to support.
For example https://github.com/Azure/azure-sdk-for-net/blob/master/src/SDKs/ServiceBus/data-plane/assets/azure-messaging.snk which is a full public/private key so we don't need to SN sign but we do need to allow the release pipeline to accept this as currently it fails. https://dev.azure.com/azure-sdk/internal/_releaseProgress?_a=release-environment-logs&releaseId=834&environmentId=904.

I've added support to our release pipelines to support the existing 31bf3856ad364e35 key as well as any libraries that are already fully signed.


Add meta object documentation for all rules. Closes #48

This pull request is an alternative from https://github.com/nodesecurity/eslint-plugin-security/pull/61 and closes issue #48  Difference being:

This one has no merge conflicts.
This one attempts to do minimal changes to the code beyond adding the meta property for rules.
This one uses the current markdown documentation rather than the inline comments as source for the descriptions.
It still has a lot of indention changes, due to the fact that adding the meta property required converting the export into a one level higher object.
Corrects gramatical errors in the documentation.
Adds documentation for detect-new-buffer to the readme aswell, as it was missing.

@nzakas When are you planning to release a new version?
This is a great improvement for everyone who uses ESLint programmatically and reads the meta objects.
It also adds the recommended patterns which are very useful.

@lolgab I'd still like to get through the rest of the PRs and issues before releasing a new version.


Add a basic unit test coverage for codesets

https://github.com/fuseml/fuseml-core/pull/10

PR was merged


only output, no real running of a test

   ./vendor/phpunit/phpunit/phpunit --configuration ./tests/phpunit_all.xml
   c:\Data\git\ANGEL\tests\Unit\Modules\Frontend\Controllers\CountryControllerTest.php

is shown in the output, but it won't run.
When I copy and paste it into the terminal window, it still won't run.
I have to add php before it to make it run.
So I did that in the configuration, and now I see that in the output window, but it still doesn't fire anything in the terminal window. How can I solve it?

Hi, please have a look at https://github.com/elonmallin/vscode-phpunit/issues/7#issuecomment-299284668.
I think you have to use php as your execPath and the rest as args, including the path to phpunit as the first argument.

Ok, so this is what I have now:
    // phpunit
    "phpunit.execPath": "c:/Development/Laragon_lite/bin/php/php-7.2.9-nts-Win32-VC15-x64/php.exe",
    "phpunit.args": [
        "./vendor/phpunit/phpunit/phpunit --configuration ./tests/phpunit_all.xml"
    ],
    "phpunit.preferRunClassTestOverQuickPickWindow": false // Default false

and in my output window, this shows:
c:/Development/Laragon_lite/bin/php/php-7.2.9-nts-Win32-VC15-x64/php.exe ./vendor/phpunit/phpunit/phpunit --configuration ./tests/phpunit_all.xml c:\Data\git\ANGEL\tests\Unit\Modules\Api\Controllers\ApiControllerTest.php
Could not open input file: ./vendor/phpunit/phpunit/phpunit --configuration ./tests/phpunit_all.xml

I even changed the config in full path's like this:
c:/Development/Laragon_lite/bin/php/php-7.2.9-nts-Win32-VC15-x64/php.exe c:/Data/git/ANGEL/vendor/phpunit/phpunit/phpunit --configuration c:/Data/git/ANGEL/tests/phpunit_all.xml c:\Data\git\ANGEL\tests\Unit\Modules\Api\Controllers\ApiControllerTest.php

Still the same error:
Could not open input file: c:/Data/git/ANGEL/vendor/phpunit/phpunit/phpunit --configuration c:/Data/git/ANGEL/tests/phpunit_all.xml

I try the same command from the terminal in VSC:
c:/Development/Laragon_lite/bin/php/php-7.2.9-nts-Win32-VC15-x64/php.exe ./vendor/phpunit/phpunit/phpunit --configuration ./tests/phpunit_all.xml c:\Data\git\ANGEL\tests\Unit\Modules\Api\Controllers\ApiControllerTest.php

and then I get the following message:
OK (6 tests, 8 assertions)

I don't know wjhat I can do or try to make this work... :-(

Sorry for the trouble, Windows is a little tricky. I have created a .bat file as my execPath.
phpunit.bat
php C:/Projects/testproj/vendor/phpunit/phpunit/phpunit %*

So I have this in my config:
"phpunit.execPath": "C:/Projects/testproj/vendor/bin/phpunit.bat"

Please see if that works. I have some pending changes to make it just find phpunit in the vendor directory automatically but It's not ready for release yet.

This should now work in the latest release v3.0.0 also published to the marketplace.
Now you don't need any config at all if you have php globally and phpunit installed with composer or as a .phar file in the project directory.
Please try and with this settings:
"phpunit.php": "c:/Development/Laragon_lite/bin/php/php-7.2.9-nts-Win32-VC15-x64/php.exe",
 "phpunit.args": [
     "--configuration ./tests/phpunit_all.xml"
 ],

Note that phpunit.execPath is now deprecated and replaced with phpunit.php. However it will still work but will be removed at some point.

Please close this issue if it works for you 👍

Doing it the other way around xP Closing this issue, please reopen if you have problems again.


Use simple physics engine

Move player in all directions with arrow keys.
Use simple physics engine for movement and collisions with walls.
Remove arguments to Player init restricting x position

We should be able to move both vertically and horisontally simultaneously.

Diagonal movement for the player is now possible. Let me know what you think.

This looks good. But currently we move much faster when moving diagonally - because both our change_x and change_y is equal to PLAYER_SPEED.
I personally think that the sum of the two, should equal PLAYER_SPEED


[Bug]: ValueError: The checkpoint you are trying to load has model type cohere2 but Transformers does not recognize this architecture.

Your current environment

The output of `python collect_env.py`
PyTorch version: 2.5.1+cu124
Is debug build: False
CUDA used to build PyTorch: 12.4
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.3 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.35

Python version: 3.12.8 (main, Dec  4 2024, 08:54:12) [GCC 11.4.0] (64-bit runtime)
Python platform: Linux-6.8.0-49-generic-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA GeForce RTX 4090
Nvidia driver version: 565.57.01
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                         x86_64
CPU op-mode(s):                       32-bit, 64-bit
Address sizes:                        48 bits physical, 48 bits virtual
Byte Order:                           Little Endian
CPU(s):                               24
On-line CPU(s) list:                  0-23
Vendor ID:                            AuthenticAMD
Model name:                           AMD Ryzen 9 5900X 12-Core Processor
CPU family:                           25
Model:                                33
Thread(s) per core:                   2
Core(s) per socket:                   12
Socket(s):                            1
Stepping:                             2
Frequency boost:                      enabled
CPU max MHz:                          4950.1948
CPU min MHz:                          2200.0000
BogoMIPS:                             7386.11
Flags:                                fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf rapl pni pclmulqdq monitor ssse3 fma cx16 sse4_1 sse4_2 x2apic movbe popcnt aes xsave avx f16c rdrand lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt tce topoext perfctr_core perfctr_nb bpext perfctr_llc mwaitx cpb cat_l3 cdp_l3 hw_pstate ssbd mba ibrs ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 erms invpcid cqm rdt_a rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local user_shstk clzero irperf xsaveerptr rdpru wbnoinvd arat npt lbrv svm_lock nrip_save tsc_scale vmcb_clean flushbyasid decodeassists pausefilter pfthreshold avic v_vmsave_vmload vgif v_spec_ctrl umip pku ospke vaes vpclmulqdq rdpid overflow_recov succor smca fsrm debug_swap
Virtualization:                       AMD-V
L1d cache:                            384 KiB (12 instances)
L1i cache:                            384 KiB (12 instances)
L2 cache:                             6 MiB (12 instances)
L3 cache:                             64 MiB (2 instances)
NUMA node(s):                         1
NUMA node0 CPU(s):                    0-23
Vulnerability Gather data sampling:   Not affected
Vulnerability Itlb multihit:          Not affected
Vulnerability L1tf:                   Not affected
Vulnerability Mds:                    Not affected
Vulnerability Meltdown:               Not affected
Vulnerability Mmio stale data:        Not affected
Vulnerability Reg file data sampling: Not affected
Vulnerability Retbleed:               Not affected
Vulnerability Spec rstack overflow:   Vulnerable: Safe RET, no microcode
Vulnerability Spec store bypass:      Mitigation; Speculative Store Bypass disabled via prctl
Vulnerability Spectre v1:             Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:             Mitigation; Retpolines; IBPB conditional; IBRS_FW; STIBP always-on; RSB filling; PBRSB-eIBRS Not affected; BHI Not affected
Vulnerability Srbds:                  Not affected
Vulnerability Tsx async abort:        Not affected

Versions of relevant libraries:
[pip3] flashinfer==0.1.6+cu121torch2.4
[pip3] numpy==1.26.4
[pip3] nvidia-cublas-cu12==<IP_ADDRESS>
[pip3] nvidia-cuda-cupti-cu12==12.4.127
[pip3] nvidia-cuda-nvrtc-cu12==12.4.127
[pip3] nvidia-cuda-runtime-cu12==12.4.127
[pip3] nvidia-cudnn-cu12==<IP_ADDRESS>
[pip3] nvidia-cufft-cu12==<IP_ADDRESS>
[pip3] nvidia-curand-cu12==<IP_ADDRESS>
[pip3] nvidia-cusolver-cu12==<IP_ADDRESS>
[pip3] nvidia-cusparse-cu12==<IP_ADDRESS>
[pip3] nvidia-ml-py==12.560.30
[pip3] nvidia-nccl-cu12==2.21.5
[pip3] nvidia-nvjitlink-cu12==12.4.127
[pip3] nvidia-nvtx-cu12==12.4.127
[pip3] pyzmq==26.2.0
[pip3] torch==2.5.1
[pip3] torchvision==0.20.1
[pip3] transformers==4.47.1
[pip3] triton==3.1.0
[conda] Could not collect
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.6.5
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0	CPU Affinity	NUMA Affinity	GPU NUMA ID
GPU0	 X 	0-23	0		N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

NVIDIA_VISIBLE_DEVICES=all
NVIDIA_REQUIRE_CUDA=cuda>=12.1 brand=tesla,driver>=470,driver<471 brand=unknown,driver>=470,driver<471 brand=nvidia,driver>=470,driver<471 brand=nvidiartx,driver>=470,driver<471 brand=geforce,driver>=470,driver<471 brand=geforcertx,driver>=470,driver<471 brand=quadro,driver>=470,driver<471 brand=quadrortx,driver>=470,driver<471 brand=titan,driver>=470,driver<471 brand=titanrtx,driver>=470,driver<471 brand=tesla,driver>=525,driver<526 brand=unknown,driver>=525,driver<526 brand=nvidia,driver>=525,driver<526 brand=nvidiartx,driver>=525,driver<526 brand=geforce,driver>=525,driver<526 brand=geforcertx,driver>=525,driver<526 brand=quadro,driver>=525,driver<526 brand=quadrortx,driver>=525,driver<526 brand=titan,driver>=525,driver<526 brand=titanrtx,driver>=525,driver<526
NVIDIA_DRIVER_CAPABILITIES=compute,utility
VLLM_USAGE_SOURCE=production-docker-image
CUDA_VERSION=12.1.0
VLLM_ATTENTION_BACKEND=FLASHINFER
LD_LIBRARY_PATH=/usr/local/lib/python3.12/dist-packages/cv2/../../lib64:/usr/local/nvidia/lib:/usr/local/nvidia/lib64
VLLM_PORT=8101
CUDA_MODULE_LOADING=LAZY

Model Input Dumps
No response
🐛 Describe the bug
I'm unable to run the model CohereForAI/c4ai-command-r7b-12-2024 with latest container version of vLLM (vllm/vllm-openai:v0.6.5). I get an ValueError:
The checkpoint you are trying to load has model type cohere2 but Transformers does not recognize this architecture. This could be because of an issue with the checkpoint, or because your version of Transformers is out of date.
During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.12/multiprocessing/process.py", line 314, in _bootstrap
    self.run()
  File "/usr/lib/python3.12/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/local/lib/python3.12/dist-packages/vllm/engine/multiprocessing/engine.py", line 368, in run_mp_engine
    raise e
  File "/usr/local/lib/python3.12/dist-packages/vllm/engine/multiprocessing/engine.py", line 357, in run_mp_engine
    engine = MQLLMEngine.from_engine_args(engine_args=engine_args,
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/vllm/engine/multiprocessing/engine.py", line 114, in from_engine_args
    engine_config = engine_args.create_engine_config(usage_context)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/vllm/engine/arg_utils.py", line 1027, in create_engine_config
    model_config = self.create_model_config()
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/vllm/engine/arg_utils.py", line 954, in create_model_config
    return ModelConfig(
           ^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/vllm/config.py", line 270, in __init__
    hf_config = get_config(self.model, trust_remote_code, revision,
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/vllm/transformers_utils/config.py", line 239, in get_config
    raise e
  File "/usr/local/lib/python3.12/dist-packages/vllm/transformers_utils/config.py", line 219, in get_config
    config = AutoConfig.from_pretrained(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/dist-packages/transformers/models/auto/configuration_auto.py", line 1040, in from_pretrained
    raise ValueError(
ValueError: The checkpoint you are trying to load has model type `cohere2` but Transformers does not recognize this architecture. This could be because of an issue with the checkpoint, or because your version of Transformers is out of date.

In https://github.com/vllm-project/vllm/pull/11203 a newer version of the Transformers library is mentioned:  >= 4.48.0, but in the container has version 4.47.1 (see report).
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

You can create your own Dockerfile that installs the dev version of transformers on top of our Docker image.

For stability reasons, our Docker images don't use development versions of external dependencies.

Ok. That makes sense. I didn't noticed that the 4.48.0 is still in dev.
A note in the release notes or model list would be nice.
Thanks for your feedback!

pip install 'git+https://github.com/huggingface/transformers.git'

Solved my problem.

https://github.com/vllm-project/vllm/pull/11583 should also fix it.


New devices cant be found - but old ones updating

I cant seem to get alexa to find new UDP devices im adding into HABRIDGE.  If I delete an existing device from alexa app, it picks it up again through discovery and re adds it in.
But adding in new devices Alexa just keeps saying no new devices found.
Ive updated HaBridge to 5.4.1RC1 and this hasnt fixed it.
Also weirdly my new devices start with 100, or 101, or 102 - which is odd as I dont have 100 devices
Any ideas?

Fixed it - updated to 5.4.1RC! - enabled Oct 9 in settings, renumbered devices and now its working,


Runner not working properly . My program gets stopped in between while being executed before it is finished entirely .

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

/needsMoreInfo

/gifPlease


feat(Combinatorics/SimpleGraph): define extremalNumber

Define the extremal number extremalNumber β A of a finite type β and a simple graph A: extremalNumber β A is the maximum number of edges in a A-free simple graph on the vertex type β, if A is contained in all simple graphs on the vertex type β, then this is 0.
Also define

the type of subgraph isomorphisms SubgraphIso as the subtype of injective homomorphisms,
the predicate that a simple graph contains another simple graph IsIsoSubgraph,
the predicate that a simple graph is free of another simple graph Free for convenience ,
the predicate that a simple graph is an extremal graph IsExtremal satisfying some predicate,

and prove lemmas/theorems about these definitions.

This is the first of a few PRs for a formalisation of the Erdős-Stone-Simonovits theorem. I created the new directory Combinatorics/SimpleGraph/Extremal as I intend to put a couple files here that really only pertain to extremal graph theory, e.g., Turán density.

The definitions in this PR do generalise a couple existing definitions, e.g., IsTuranMaximal r iff IsExtremal (·.CliqueFree (r+1)) in Turan.lean. If this PR is successful it might be worth rewriting bits of Turan.lean using these common definitions, e.g.,
theorem extremalNumber_completeGraph
    (β α : Type*) [Fintype β] [DecidableEq β] [Fintype α] [Nontrivial α] :
    extremalNumber β (completeGraph α) =
      (turanGraph (Fintype.card β) (Fintype.card α-1)).edgeFinset.card := sorry

theorem card_edgeFinset_eq_extremalNumber_completeGraph_iff_iso_turanGraph
    {β: Type*} [Fintype β] (G : SimpleGraph β) [DecidableRel G.Adj]
    (α : Type*) [Fintype α] [Nontrivial α] :
    (completeGraph α).Free G ∧
        G.edgeFinset.card = extremalNumber β (completeGraph α)
      ↔ Nonempty (G ≃g turanGraph (Fintype.card β) (Fintype.card α-1)) := sorry


Align the PUC for mobile and AMP

There's confusion and misalignment over how publishers are supposed to use the PUC across different channels. e.g. mobile app developers are supposed to use mobile.js or creative.js instead of the newest  HB_FORMAT.js innovation.
Also it's unclear if native rendering works for AMP or app. Probably doesn't.
Here's a cut at the requirements.
High Level Requirements

Publishers should be able to utilize the PUC across web, app, and AMP
Publishers should be able to utilize the PUC within safeframes for all of the following mediatypes: banner, outstream video, native, interstitial banner and interstitial outstream
Non-safeframe support is required for web and app. (AMP always requires safeframes)
Publishers should be able to set up one set of line items that works across all channels and all mediatypes listed above. Note that instream video is expected to be a separate set of line items.
For scenarios where the publisher wants to mix safeframes and non-safeframes, it should be possible to employ creative-level targeting for ad servers that support it. i.e. there should be a key-value-pair (e.g. hb_channel) that allows them to know what channel this bid is coming from: web/amp/app.

#5 is a newly proposed requirement that might go a way towards solving some of the edge cases straining us. By utilizing creative-level targeting against hb_channel, publishers could create one set of line items with different sets of creatives:

if hb_channel=amp, make sure the creative is safeframes
if hb_channel=app, make sure the creative refers to moble.js
etc

Web Safeframe Banners
For Web Safeframe Banners, the PUC must:

Call PostMessage to get the winning ad from Prebid.js
Create an iframe of the appropriate size and displays the winning ad within it

AMP and Mobile
For AMP and Mobile, the PUC must:

Update the size of the iframe to the size of the winning ad.
Retrieve the body of the creative from Prebid Cache based on the UUID
If the ‘burl’ parameter is present, create a tracking pixel. Includes special support for triggering the viewable billing url for mobile MRAID creatives.
If the ‘nurl’ parameter is present, create the appropriate HTML to fire the notice URL.
If the ‘wurl’ parameter is present, create a tracking pixel. This is needed for Programmatic Guaranteed support.
Resolve any ${AUCTION_PRICE} macro in the creative body.

Native
For Native, the PUC must:

Retrieve the native attributes from the winning ad.
Coordinate the rendering of the native ad using the template method specified by the publisher.

+1****

+1

@bretg
going to write what I know about the puc currently.
Files mobile.js and amp.jsare now identical. In the previous version of the PUC too, those two were following the same paths in the code. The only difference is that, when mobile environment is detected, mraid is called and a markup is built. Currently, the same files (either mobile.js and amp.js) are able to run both. I can explain better why previous and current versions are equivalent, if needed.
Regarding the section "Web Safeframe Banners", the point:

Create an iframe of the appropriate size and displays the winning ad within it

I don't think this applies here. the PUC cannot build iframes. It is already in an iframe, created by the adserver. the PUC is loaded inside the given iframe. Maybe there's something I didn't understand correctly here?


Allow placing layers on subpixels if a specific keyword is used in symbol artboard

Icons tend not to place layers on 0.0 and 0.5 pixel increments.
It gets tedious clicking warnings one by one (a single icon might have 10 such layers) for standard fontawesome icons etc.
Imagine how wonderful it would be if the assistant didn't complain when the symbol artboard contained a keyword like #allowsubpixels

Good one.
We could definitely use the Assistant configuration to define keywords to skip checks on certain layers.

Closed without any explanation?


Document unsafe blocks

We should document all unsafe blocks in the codebase, explaining the rationale for why using the keyword is safe (i.e. // SAFETY: comments). This will allow us to enable the undocumented_unsafe_blocks clippy lint (e.g. cargo clippy -- -D clippy::undocumented_unsafe_blocks), and emit an error on new undocumented unsafe blocks.

I started to look into that, but we have a looooot of unsafe blocks without any comment.
It will take time, but in the meantime we should enforce new PRs to carry SAFETY comments. Not sure if we should add some scripts in the CI for now, till we can enable undocumented_unsafe_blocks clippy lint.

I think we have to rely on reviews for making sure all new unsafe sections get a SAFETY comment.

I think we have to rely on reviews for making sure all new unsafe sections get a SAFETY comment.

Yes, but we are human, and it can be missed, so I added a new pipeline for that, see https://github.com/coconut-svsm/svsm/pull/535
For example, we have recently added several with the work for user space (I will send another PR tomorrow to fix some of them, but I need help with some that I don't know how to document).

Hi @stefano-garzarella, I am able to repro missing unsafe block comment with recently introduced userspace PRs using below changes. Please let me know if you want me to cook up a patch for fixing it.
diff --git a/Makefile b/Makefile
index 0916f1e1..26874350 100644
--- a/Makefile
+++ b/Makefile
@@ -175,10 +175,10 @@ bin/svsm-test.bin: bin/stage1-test
        objcopy -O binary $< $@

 clippy:
-       cargo clippy --workspace --all-features --exclude packit --exclude svsm-fuzz --exclude igvmbuilder --exclude igvmmeasure -- -D warnings
-       cargo clippy --workspace --all-features --exclude packit --exclude svsm-fuzz --exclude svsm --target=x86_64-unknown-linux-gnu -- -D warnings
-       RUSTFLAGS="--cfg fuzzing" cargo clippy --package svsm-fuzz --all-features --target=x86_64-unknown-linux-gnu -- -D warnings
-       cargo clippy --workspace --all-features --exclude packit --tests --target=x86_64-unknown-linux-gnu -- -D warnings
+       cargo clippy --workspace --all-features --exclude packit --exclude svsm-fuzz --exclude igvmbuilder --exclude igvmmeasure -- -D warnings -D clippy::undocumented_unsafe_blocks
+       cargo clippy --workspace --all-features --exclude packit --exclude svsm-fuzz --exclude svsm --target=x86_64-unknown-linux-gnu -- -D warnings -D clippy::undocumented_unsafe_blocks
+       RUSTFLAGS="--cfg fuzzing" cargo clippy --package svsm-fuzz --all-features --target=x86_64-unknown-linux-gnu -- -D warnings -D clippy::undocumented_unsafe_blocks
+       cargo clippy --workspace --all-features --exclude packit --tests --target=x86_64-unknown-linux-gnu -- -D warnings -D clippy::undocumented_unsafe_blocks

 clean:
        cargo clean

@vijaydhanraj I already have a patch here https://github.com/stefano-garzarella/svsm/commit/5c2fb547b18e08ce46cb2ddc45da38c7418b8920
I'm going to post a PR with multiple SAFETY comments (not all, because they are too much for a single PR).
BTW if you think it's incomplete or incorrect, feel free to go head and post your version. Any help on this effort is more than welcome :-)
Just a note, instead of modifying the Makefile, you can just uncomment this line in Cargo.toml:
diff --git a/Cargo.toml b/Cargo.toml
index 1d3a7bf2..a1fe07a3 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -91,7 +91,7 @@ rest_pat_in_fully_bound_structs = "warn"
 string_to_string = "warn"
 suboptimal_flops = "warn"
 # TODO: fires many times, fix then enable.
-# undocumented_unsafe_blocks = "warn"
+undocumented_unsafe_blocks = "warn"
 unnecessary_box_returns = "warn"

 [workspace.metadata.scripts]

Thanks @stefano-garzarella for the suggestion and the patch! The changes here https://github.com/stefano-garzarella/svsm/commit/5c2fb547b18e08ce46cb2ddc45da38c7418b8920 look good to me.

Just reopen this, not sure why GH decided to close it.


Autocomplete with dash

VSCode Version: Code - Insiders 1.8.0-insider (0d65ced0fa7398dc9d50c39a23a2a8cc8406ff18, 2016-12-07T07:05:09.124Z)
OS Version: Darwin x64 16.1.0

Steps to Reproduce:

TypeScript

Hit Enter

That's a really annoying bug! Not just in typescript.

@mjbvz The TypeScript extension tells the editor to only delete the 2 preceding characters, not the entire word. That might be a problem with the work definition you are using.

I have extrapolated about snippets to the following piece of code

type abc = 'string-multiline' | 'string-singleline';

let _a: abc;

_a = 'string-mu'

I think that Typescript needs to be explicit in this case, and provide a range for the completion items. We currently automatically pick up the range from TextDocument.getWordRangeAtPosition

@mjbvz do you know if this is tracked on TS's side? I want to make sure I have it tracked in the OneNote table appropriately.

@waderyan I don't think this is an issue on the TypeScript side. The bug was that we were relying on the default VSCode logic for determining what word to replace, and it breaks words that contain - or any other non letter or digit characters.
The result TypeScript provides to us is correct as far as I know


Overlays

Is it possible to get the "like" and "dislike" overlays as you swipe left and right, so that the user knows what action they are taking when swiping?

Yes, you can , just put an overlay on top and listen to swipe event. It is not part of component functionality though.

is it possible to disable the background color?

How would you go about putting an overlay on top?


Update run_black_on_git_diff_adds.sh

Can now handle if a file is deleted. If the PR only contains deletion of files black is not applied. Should fix #1 .

@rgasper Bumping this back into your notifications list. I'm sure you are probably busy with other things these days, but if you have the time we would appreciate you getting this fix in.

Oh hey sorry about such a long delay @DueViktor and @msmcconnell, thanks for submitting the fix. I don't check all my github notifications regularly...


genre section added

I have added the genre section again with a little change in the background color for each card.
i have tried to resolve  the conflicts.

@anuragverma108 okay


feature/twcc record packet status

Record packet status in SendRecord
Account for packet status for bitrate estimates

It's getting a bit state heavy. But I can clean that up later.

Do you mean SendRecord or the return type on the iterator(I was thinking that might be due to become a struct)?

More the interaction between all the fields. The status vs instant. They both encode an implicit state that we received feedback. Would be nice to make that explicit.

More the interaction between all the fields. The status vs instant. They both encode an implicit state that we received feedback. Would be nice to make that explicit.

Ah yes I was thinking that too, something like
struct SendRecord {
  seq_no: SeqNo,
  local_send_time: Instant,
  size: u16,
  recv_report: Option<RecvReport>
}

Yeah. That's nice.

can any of this go out in the stats ?
No I don't think anything here is needed for stats

Yeah. That's nice.
I can move it to something like that next time I touch the code


get_rule("'") returns nil: attempt to call method 'with_pair' (a nil value)

Description
After update the plugin after this commit https://github.com/windwp/nvim-autopairs/commit/ff985c71417bceb168c5415813e40e656e3ea9ed, I got an error from get_rule("'"):with_pair.
E5113: Error while calling lua chunk: /Users/name/.config/nvim/lua/plugins/autopairs.lua:5: attempt to call method 'with_pair' (a nil value)

This is my usage
npairs.get_rule("'"):with_pair(function()
  if vim.bo.filetype == 'reason' then
    return false
  end
end)

Mapping  bug
1.If you report a bug about indent. Please remember that plugin doesn't do anything about indent.
It just trigger the indent of your vim config so if you have wrong indent config then it will do wrong indent.
You can check by select a block of code and press ==
2. provide result of command :verbose imap <cr>.
Steps to reproduce
No response
Minimal config
local npairs = require('nvim-autopairs')

npairs.setup()

npairs.get_rule("'"):with_pair(function()
  if vim.bo.filetype == 'reason' then
    return false
  end
end)

now there have 2 rules use single quote 1 rule specify for rust
so that function is a return to a table  of rules not a single rule
npairs.get_rule("'")[1]:with_pair(function()
  if vim.bo.filetype == 'reason' then
    return false
  end
end)

Thank you, I will close this.


Adding vjjmiras as a reviewer

Signed-off-by: Vicente J. Jiménez Miras<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog
/area documentation

/area videos

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: vjjmiras
Once this PR has been reviewed and has the lgtm label, please assign issif for approval by writing /assign @issif in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the falcosecurity org. You can then trigger verification by writing /verify-owners in a comment.

vjjmiras

User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.

+1 for me! What do other maintainers think? cc @falcosecurity/falco-website-maintainers

+1 for me! What do other maintainers think? cc @falcosecurity/falco-website-maintainers

+1 for me :+1:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, vjjmiras
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

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

LGTM label has been added.  Git tree hash: 5b21290e50b280be9c122eb7f16ab2e3f14d41a0

/hold

+1

/verify-owners

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the falcosecurity org. You can then trigger verification by writing /verify-owners in a comment.

vjjmiras

User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.

/verify-owners


pv: 1.8.5 -> 1.8.9

Description of changes
Update pv to its latest released version, 1.8.9. The project license changed to GPL-3+.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 306463 run on x86_64-linux 1

1 package marked as broken and skipped:

radicle-upstream

1 package blacklisted:

appimage-run-tests

2 packages failed to build:

badlion-client
lunatask

114 packages built:

Sylk
altair
alvr
anytype
appimage-run
arduino-ide
bazecor
beekeeper-studio
beeper
betterdiscord-installer
bloomrpc
bootstrap-studio
buttercup-desktop
caido
caprine-bin
chrysalis
cider
codux
cozy-drive
crypto-org-wallet
davinci-resolve
davinci-resolve-studio
deskreen
devdocs-desktop
electron-mail
electronplayer
firefly-desktop
flexoptix-app
fluent-reader
framesh
freetube
hamsket
hifile
hoppscotch
immersed-vr
irccloud
jan
jbrowse
jetbrains-toolbox
joplin-desktop
kchat
keet
kmeet
lbry
ldtk
ledger-live-desktop
lens
listen1
lmstudio
logseq
losslesscut-bin
lunar-client
marktext
mathpix-snipping-tool
mendeley
miru
mobilecoin-wallet
mockoon
molotov
motrix
mqttx
museeks
mycrypto
neo4j-desktop
nosql-workbench
notable
notesnook
notion-app-enhanced
nuclear
openlens
osu-lazer-bin
p3x-onenote
parsify
pcloud
plexamp
polypane
protonup-qt
pureref
pv
rambox
raven-reader
remnote
requestly
revolt-desktop
ripcord
rshim-user-space
saleae-logic-2
sanoid
session-desktop
simplex-chat-desktop
sonixd
soulseekqt
spacedrive
ssb-patchwork
station
steam-rom-manager
super-productivity
timeular
todoist-electron
trezor-suite
tusk
tutanota-desktop
uhk-agent
uhk-udev-rules
upscayl
via
vial
wal_e
wal_e.dist
wootility
xlights
zecwallet-lite
zettlr
zulip

Lunatask maintainer @henkery - do you want to investigate here?
badlion-client has no maintainer, so I'd merge this if @henkery doesn't mind.

For now I agree to merge, will try to resolve failing build of lunatask after rebuild

The badlion-client failure is a mismatched source hash, so it's totally unrelated here

Okay, as @henkery approved, lets merge this earlier than I said :laughing:


Improve filename regex - recognize filenames with multiple dots in it.

If you say this case is found often then I'm ok with the change. This PR looks straightforward and clear. I would love if you add unit-test too but we can do that later.
Somehow the repo owner has gone inactive for months :(

If we could add a couple of unit tests to cover this problem, I would be more than happy to merge this PR.


pifi: remove

Motivation for this change
This upstream repository has been archived.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Please also add a throw in pkgs/top-level/aliases.nix :-)


ci: remove the iframe with the logs

This doesn't work at all on small and very large screens.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "iframe" git@github.com:samoht/datakit.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

If you rebase on master then the tests should pass (need to pin an older version of opam2).

Rebased


ZigBee TRV current temperature not updating unless opened in SmartLife

I've got several TeslaSmart ZigBee TRVs paired to standalone ZigBee hub. I'm able to control their target temperature through both SmartLife app and Home Assistant integration, however, value of current temperature being shown in Home Assistant will keep the same until I open the status page of each TRV in SmartLife app - only then it is updated in Home Assistant as well.
Somewhere on the web I found that current value update can be triggered by using Home Assistant "Home Assistant Core Integration: Reload config entry" automation, however, even with doing that way, current temperature values keep the same.
I've already tried both old and v2 Tuya integrations and both behave in the same way.
Expected behavior
I need the values of current temperature of each TRV be updated without triggering it using SmartLife app.
Diagnostics for integrations and devices
config_entry-tuya-1ad1ca8f55d628c0763353230ba35b19.json.txt
Home Assistant Version
2022.11.2

Currenly I've partially solved my issue by using simple Python script scheduled to execute by cron every 10 minutes:
refresh-trv-status.py.txt
from time import sleep
from tuya_iot import TuyaOpenAPI
ACCESS_ID = 'xxx'
ACCESS_KEY = 'yyy'
ENDPOINT = "https://openapi.tuyaeu.com"
USERNAME =<EMAIL_ADDRESS>PASSWORD = 'zzz'
DEVICE_IDS = [
'aaa',
'bbb',
'ccc',
]
openapi = TuyaOpenAPI(ENDPOINT, ACCESS_ID, ACCESS_KEY)
openapi.connect(USERNAME, PASSWORD, "420", "smartlife")
for DEVICE_ID in DEVICE_IDS:
print(DEVICE_ID)
openapi.post(
"/v1.0/devices/{}/commands".format(DEVICE_ID),
{"commands": [{"code": "online", "value": False}]}
)
sleep(1)
openapi.post(
"/v1.0/devices/{}/commands".format(DEVICE_ID),
{"commands": [{"code": "online", "value": True}]}
)
openapi.get("/v1.0/devices/{}".format(DEVICE_ID))
sleep(1)


v4 has still same scroll glitches as in v3 in iOS

Bug Report
Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.
cli packages: (C:\Users\pak43\AppData\Roaming\npm\node_modules)

   @ionic/cli-utils  : 2.0.0-rc.6
   ionic (Ionic CLI) : 4.0.0-rc.6

local packages:

   @angular-devkit/core       : 0.6.0
   @angular-devkit/schematics : 0.6.0
   @angular/cli               : 6.0.7
   @ionic/schematics-angular  : 1.0.0-rc.6
   Ionic Framework            : @ionic/angular 4.0.0-alpha.7

System:

   NodeJS : v8.11.2
   npm    : 5.3.0
   OS     : Windows 10

Describe the Bug
When you scroll in iOS it has some glitches as in v3 too.
Additional Context
Check this:

Hello! What am I looking at here? Is this a page that you want to be scrollable? Also, are you using wkwebview?

@jgw96 i don't wanna this page to be scrollable but i need scroll in other pages with same styling but long form such as registration page. You see the glitches ? it has flickering in bouncing effect which also there in v3 as shown in this video. I am using ionic serve on windows and accessing the page in safari browser as PWA app as you can see in the app URL.

And just to be clear, you see the same glitches in the pages that need to be scrollable right?

@jgw96 yes. You can keep focus on that white background. you will see the glitches.

Here is my view:
<ion-header>
  <ion-toolbar>
      <ion-buttons slot="start">
          <ion-back-button></ion-back-button>
      </ion-buttons>
    <ion-buttons slot="start">
      <ion-menu-button></ion-menu-button>
    </ion-buttons>

  </ion-toolbar>
</ion-header>

<ion-content scrollEnabled="true">

      <ion-icon name="contact"></ion-icon>

    <form [formGroup]="loginForm" (ngSubmit)="onSubmit()">
      <ion-list>
        <ion-item>
          <ion-label position="floating">Email</ion-label>
          <ion-input type="email" formControlName="username"></ion-input>
        </ion-item>
        <ion-item>
            <ion-label position="floating">Password</ion-label>
            <ion-input type="password" formControlName="password"></ion-input>
          </ion-item>
      </ion-list>
      <ion-button expand="block" type="submit" [disabled]="!loginForm.valid || isLoading">
          {{'label.customer.login' | localizedresource | async}}
      </ion-button>
      <ion-button expand="block" type="button" (click)="onRegisterButtonClicked()">
          {{'label.customer.register' | localizedresource | async}}
      </ion-button>
    </form>

      {{'label.customer.forgotpassword' | localizedresource | async}}

</ion-content>

@jgw96  here is the new video. Check it:
https://vimeo.com/user87232084/review/279683820/953710ae0b

Is this still an issue in beta.17? I also have diffrent apps on iOS and Android and cannot confirm it

@paulstelzer i will check this out today :)


HHH-11864: release JDBC-connection on cleanup().

Release JDBC connection so that auto-commit mode can be reset in JdbcConnectionAccessProvidedConnectionImpl. (as is performed in ExtractionContextImpl).

Could you also include a test case for the bug fix?

I don't have any source checked-out, so I can't supply a test case I'm
afraid.
Kind regards,
Jeroen van der Vegt
2017-07-12 15:14 GMT+02:00 Chris Cranford<EMAIL_ADDRESS>
Could you also include a test case for the bug fix?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/1949#issuecomment-314765250,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACevI96sIlBfjBoFY3TCnwukaSHWBEoEks5sNMbMgaJpZM4OSz2x
.

Thanks for the PR. It's superseded by this PR.