fix(server): Remove optional params count check

Checking actual params count towards expected optionals seems to be broken - handling request with optional param omitted fails.
Description

This PR relates to closes #39.

This PR aims to fix the situation about checking the amount of provided params towards expected optional inputs count.
Comments
Pre-submit checklist

Branch

[ ] Tests are provided (if possible)
[x] No compilation warning is added (if possible)
[x] No deprecated methods or objects are added
[x] Commits have useful messages

PR

[x] Self-reviewed the diff
[x] Create a useful and clear PR description

Documentation

[ ] Update README.md file (if relevant)
[ ] Update documentation files in docs folder (if relevant)

First of all thank you for the PR.
Second, we try to maintain quite comprehensive test suite in https://github.com/input-output-hk/armadillo/tree/main/server/test so please try to always cover your changes with some tests.
Regarding the change itself. You are right that the bug is in this line however I don't think that the provided solution is correct. The idea of this check was to verify if number of provided inputs is greater or equal to the number required inputs and lower or equal to the number of total inputs of the endpoint.
The fix should look more like:
 if (jsonAsMap.size >= in.count(i=> !i.codec.schema.isOptional) && jsonAsMap.size <= in.size)


Requesting for guidance to help me understand the project

Welcome! Please use this template for reporting documentation issues on https://developers.google.com/web/.
For questions about Chrome DevTools, checkout the
google-chrome-devtools tag on StackOverflow.
For questions about Chrome, check the google-chrome
tag. If you think you've found a Chrome bug, check out our
feedback page for more details on how to file it.
Page Affected: https://developers.google.com/web/TODO
What needs to be done?
Please provide description of what's needs to be done here. Please be sure to include the link to the original page
and any supporting links/data.

[ ] Appropriate type-something label has been added.
[ ] Appropriate section-something label has been added.

Hello everyone ,can anyone please guide me to understand the project , where can i begin my contributions.thanks in advance


How do I handle 404s?

Hi, I'm pretty new to Adonis and I am having some issues regarding 404 errors in production. Unfortunately, when you visit a non existent route Adonis exhibits weird (?) behavior and crashes immediately.
I am wondering how I can handle this error.
Thanks.

Inside bootstrap/start.js there is a block to listen to error event and same is used to handle error. https://github.com/adonisjs/adonis-app/blob/develop/bootstrap/start.js#L51.
If you have better suggestions, i am all open to improve http error handling

I think if we could have some sort of error handling within the routes file that would be great. Something along the lines of how ExpressJS does it with Middleware perhaps?

I have added the Event provider to the core of the framework, it will make it possible to handle Http errors , in a better way.
Teaser
const Event = use('Event')
Event.on('Http.404', 'Http.handle404')

Here Http.handle404 comes from app/Listeners/Http.js file.
Http.handle404 = function * (error, request, response) {
 yield response.sendView('errors/404')
}

Hi Aman, thanks.
Also For now, line 55 of bootstrap/start.js should be removed in any serious app.
process.exit(1)
In the current state of this project, 404s cause the entire server to crash. Calling that an over-sight is a massive over-simplification.
Being a singular thread, The node process should never be ended trivially.
Nothing should kill it of it's own volition.
That design pattern of ending process on a simple sneeze only makes sense if you are writing PHP, not server-side Javascript.

Nope, you should always let your application crash and get re-started by deamons like forever or pm2. http://shapeshed.com/uncaught-exceptions-in-node/
For http 404 error, you can manually add a check on status to see if it's 404 or not. For any other errors you should make the process die, failing fast and first is always a good practice

That depends on the type of the exception, i.e. an uncaught exception and not every exception.
404's and infact **all HTTP errors, should never qualify as uncaught exceptions **on a framework level, not an app level, and especially not a framework that prides itself on convention-over-config).
UncaughtExceptions, by their very nature of being uncaught, kill the app on their own, in which case yes the best thing to do is to restart it.
Theory context mismatch aside, rebooting an app takes time.
The realities of running a server mean you have to account for attacks.
If it becomes clear that a certain url on your app makes your entire server-side app reboot, what black-hat won't have a field day messing up your system with a loop attack. That would be the easiest site to bring down and KEEP down.

Awesome stuff here, I forward to 3.0/

Hello Everyone!!!
The fact that Adonisjs handle various errors is great, but it is always a good practice to make an app prepared for all errors and not let a framework to show an error itself, that errors are actually for developers, and it fill be embarrassing to have a live web application which is accidentally can show an system error.
So, even that from bootstrap/events.js this Event.when('Http.error.*', 'Http.handleError') navigates to /app/Listeners/Http.js it is not a event solution, but it is awesome cause by 2 lines of simple code we can handle an important case of 404's simply adding
if (status === 404 )
    yield response.sendView('front')

Hey @DBadalyan! 👋
The error you are currently seeing is only for development purpose, on production, these will not be displayed.

@RomainLanz I'm new to adonisjs, and would like to know what would be then on production, I started looking and playing Adonis from yesterday. Thank you!

@RomainLanz Oh, I'm new to Adonis, playing with it from yesterday, now I see it acts wonderful. Happy to learn that. Actually, I really appreciate the great job of Adonis developers.

Yes!


module 'torch.cuda' has no attribute 'amp'

when training DPR, this error occurs.
Some says that attribute "cuda.amp" needs a torch version>=1.50.
In the guide the recommanded torch version is 1.4.0, so I wonder if it is possible to solve the problem without installing other versions of cuda and packages again.

The same error,have you fix it?


Image Text Recognition

MediaPipe Solution (you are using)
None
Programming language
Javascript
Are you willing to contribute it
None
Describe the feature and the current behaviour/state
I need feature that can analyze text in an images
Will this change the current API? How?
No response
Who will benefit with this feature?
People that need to detect text and object in an images
Please specify the use cases for this feature
User want to detect text in an images or video for simplify the process input the special character of numbers and alphabet
Any Other info
No response

This is currently not one of our offerings. Will let you know if this changes.


[Question] How do we go about using sceneview in jetpack compose app with multiple screens?

I am trying to incorporate 3d in my android application built with jetpack compose. I am able to get Sceneview to render model but as I navigate from that particular screen. It crashes with error:
java.lang.IllegalStateException: LayoutNode should be attached to an owner
Code :
@Composable
fun IslandScreen(){
    val nodes = remember { mutableStateListOf<Node>() }

    Box(modifier = Modifier.fillMaxSize()) {
        Scene(
            modifier = Modifier.fillMaxSize(),
            nodes = nodes,
            onCreate = { sceneView ->
                // Apply your configuration
                sceneView.apply {
                    setLifecycle(lifecycle)
                    // Apply your configuration
                    sceneView.lifecycleScope.launch {
                        val hdrFile = "environments/kloofendal_48d_partly_cloudy_puresky_4k.hdr"
                        sceneView.loadHdrIndirectLight(hdrFile, specularFilter = true) {
                            intensity(30_000f)
                        }
                        sceneView.loadHdrSkybox(hdrFile) {
                            intensity(50_000f)
                        }

                        val model = sceneView.modelLoader.loadModel("models/mouse 2.glb")!!
                        val modelNode = ModelNode(sceneView, model).apply {
                            transform(
                                position = Position(z = -4.0f),
                                rotation = Rotation(x = 15.0f)
                            )
                            scaleToUnitsCube(2.0f)
                        }
                        sceneView.addChildNode(modelNode)

                    }
                }
            }
        )
    }
}

I suspect that this error arises as the lifecycle of Scene is same as that of View.lifecycle so when I change screen, box is no longer in the screen thus Scene will have no parent.
I don't know if I am right about it. I'll be grateful for the help on how to resolve this issue.

Just in case anyone wants to know, I ended up changing my strategy and used filament directly with custom model viewer implementation (mostly same as sample-gltf viewer from filament repository samples) alongside surface view.


Changed strand_.post to strand_.dispatch

asio already has a built-in function for this operation.
You can check source code: https://github.com/chriskohlhoff/asio/blob/master/asio/include/asio/detail/impl/strand_service.hpp#L58-L63
Referance: http://www.boost.org/doc/libs/1_65_1/doc/html/boost_asio/reference/io_service/dispatch.html

@tanisman, do you plan on updating this PR to address the feedback by David Schwartz? If not, should we consider closing this for now?


unsupported ext tag detected: EXT-X-INDEPENDENT-SEGMENTS

Hi.
I've run across a new issue. Maybe just evolution of m3u8 playlists, but a new tag that's not supported.
com.iheartradio.m3u8.ParseException: unsupported ext tag detected: EXT-X-INDEPENDENT-SEGMENTS - #EXT-X-INDEPENDENT-SEGMENTS
at com.iheartradio.m3u8.ParseException.create(ParseException.java:35)
at com.iheartradio.m3u8.ExtendedM3uParser.parse(ExtendedM3uParser.java:66)
at com.iheartradio.m3u8.PlaylistParser.parse(PlaylistParser.java:115)
You can get the playlist that has these tags here.
https://ngcstatpack-a.akamaihd.net/Nat_Geo_Channels/896/599/1509829223445/Long_Road_101_HD_Clean_AUTH_movie.m3u8
I will probably work on integrating this is my clone so if you don't have anything by the time I do I'll post my edits for you.

Adding ParsingMode.LENIENT bypasses this error:
Playlist playlist = new PlaylistParser(someinputstream, Format.EXT_M3U, Encoding.UTF_8, ParsingMode.LENIENT).parse();


question: Why use pop-launcher?

Describe the bug
Hi, what is the purpose of using pop-launcher versus systemctl/dbus for launching?
Thanks!

You can take a look at #38 and #19.
I admit the name can be confusing but pop-launcher is not used only for launching only.
It has the free-desktop entry specification sorted out already, i.e searching, launching, sorting desktop entry.
It also provides a set of builtin plugin and can be easily extended with user defined plugin.


Use v2 api for spatial_transformer_*

The functions spatial_transformer_grid and spatial_transformer_sampler do not follow v2 API with respect to cuDNN use settings.

@yuyu2172 Thank you for sending this PR, but I'm afraid it's dubplicated to this PR https://github.com/chainer/chainer/pull/2955 (it came 40 minutes earlier...)

@mitmul
Oh. Thank you for fixing. Sorry for the bug.
I will close this PR


Update README.md

Corrected typo's.
Changed words for proper readability.
Corrected colon to semi-colon .

Thanks! You will also have to signoff your commits for the merge to be possible, these are linux foundation requirements ;-) https://github.com/AcademySoftwareFoundation/aswf-docker/pull/31/checks?check_run_id=485970481

After executing the last command, it gives an error as "failed to push some reference to 'URL' ".
Can you please help?

The command is just to force push your changes isn't it?

Ya, I am just executing the command. It says, you are not allowed to push changes. I don't know why is it showing it.
Can you please help?

What's the command that is failing?

The command is as follows:
git push --force-with-lease origin patch-1

You must have configured your origin remote to point to the wrong repo then... What are your git remotes?

I first cloned the repo. Then connected my github to git. Pointed to this downloaded repo's pull request #31(Mine one) and then force pushing it with sign off comments as in the command.

you need to replace "origin" with the name of the remote that points to your repo, not the official one that can't be pushed to...

I think I am rebasing it with the command:
git rebase HEAD~2 --signoff

Please paste the content of your .git/config file so I can tell you what to put in your push command instead of origin.

That's a nice idea. I will paste it then.:)

@aloysbaillet Here's the link for the information:
https://pastebin.com/PKiy6Nh6
I have tried to reach my local branch and then push it using the following command
git push --force-with-lease origin patch-1

but still it shows the same error.

Looks like your git repo is in a bad state, I'd recommend you restart from scratch.


Old crossbeam-channel could return a crazy length because of integer overflow

In crossbeam-channel 0.3.x and 0.4.x (also happen to crossbeam_queue::SegQueue), when computing the length of the channel, if the indices fall onto the block end, they will be fixed up by:
https://github.com/crossbeam-rs/crossbeam/blob/48b13dc43cf19e0544d03d16f4b3da3d8b23f076/crossbeam-channel/src/flavors/list.rs#L511-L518
Although the indices have been rotated with wrapping_sub previously, if both the head and the tail index are equal to BLOCK_CAP here, then tail -= LAP will overflow and wrap into usize::MAX when build in release mode (or panic when build in debug mode).
https://github.com/crossbeam-rs/crossbeam/commit/1df3ee7b09feabf71d85bac46e36f1eef6257bca has been changed the way to fix up indices, so this problem won't happen on 0.5.x. We may need to cherry-pick this commit to 0.3.x and 0.4.x branches.

Closing -- Given that the first version of 0.5 that fixed this issue was released over a year ago, the major version is 0, and that there have been few breaking changes between 0.4 and 0.5, there is no motivation to continue maintaining 0.4.


not compatible with networkx 2.x module

Description
pyannote-algorithms requires networkx version 1.x Current is 2.x but due to API changes are they not compatible.
Steps/Code to Reproduce
pip list
to show the networkx version
To make it compatible with the networkx 2.x API I suggest the following changes to  pyannote-algorithms/pyannote/algorithms/clustering/hac/history.py:

clusters = topological_sort(g, nbunch=clusters, reverse=True)

clusters = list(reversed(list(topological_sort(g.subgraph(clusters)))))

Expected Results

Actual Results

Versions

pyannote.algorithms==0.7.3
pyannote.core==1.3.1
pyannote.parser==0.7.1
pyannote.video==1.5.1

Thanks for your feedback. I'd be happy to merge a pull request fixing the issue!


Parse panic doesn't seem to be caught?

This showed up as a part of RLS test failure. The string in question is
/// Bar variant\nBar,\n/// Baz variant\nBaz\n}\n\n/// Bar struct\npub struct Bar<T>{}

Repro: https://github.com/Xanewok/simple_fmt/tree/master
I'd expect the parser to catch possible panics:
https://github.com/rust-lang/rustfmt/blob/fbeabe9547954195a65077a54d41956257086ffb/src/formatting.rs#L622
however it seems to be an aborting panic? (no backtrace)
Is this completely on rustc side or can we do something here to not abort on parse error?

It seems that creating parser
https://github.com/rust-lang/rustfmt/blob/fbeabe9547954195a65077a54d41956257086ffb/src/formatting.rs#L609
can panic as well (new_parser_from_source_str). I guess we should also catch the error here or use maybe_new_parser_from_source_str here instead.


Update Gemfile.lock

@adrianhall: Any commands I can run to verify this change?  What process(es) is Gemfile.lock used by?

This isn't the way to update Gemfile.lock - it's similar to the package-json.lock in npm-land.  You don't edit it by hand.  I'll fix up appropriately now I know the appropriate package, and re-submit.

@adrianhall: For the record, I didn't update gemfile.lock by hand, I ran the ruby bundle update command.


Consider removing link ink skipping

In #281 we added text-decoration-skip: ink; to skip underlines near descenders - in theory because it looks better.
I wonder if we should reconsider this decision.
There are many examples where it makes single links look like multiple links. Are we happy with this?
NB - it generally looks better in Safari which doesn't skip as much.
Discuss.
Chrome

Safari

I would be in favour of reverting it again. Evidence from a single user: It actually has happened to me that I opened the same page twice because I didn't realise that it was the same link.

Agree.
Additionally, there are also instances within single words where the letter combinations just look strange.

To the contrary in 4 rounds of research on GOV.UK since this was deployed we haven't seen any issues with people getting confused.

I wonder if we might be able to test anything in analytics related to this - any ideas?
Perhaps: number of people opening the same link twice from one source page - ie clicking both parts because they thought they were two links.

Will discuss with Vin when he is back from leave as we are already measuring this stuff for navigation. Not sure if we will have any significant findings however.

On the other hand, have there ever been users in user research who said anything ever about underlining? ;-)

Just to clarify the original change was to make links (in lists) more legible.
I'd be 👍  to revert since Chrome doesnt seem to render these as well as Safari 😢

@alextea I think it's less about getting confused and more about cognitive load and readability.
I'm in favour of reverting until Chrome has better typography.
As discussed with @nickcolley, words containing g and p appear easier to read, while other characters like y, commas , and brackets ) leave strange artefacts that look like punctuation or a break in the text. I think the latter is more damaging than the former is beneficial – but that's opinion and reckons.

@selfthinker it's not about what they say, but we have observed that in some circumstances users don't recognise text as links when it is not underlined.

I'm also in favour of reverting btw.

To be clear, are we considering actively disabling it in Safari (where it's on by default) or are we just talking about removing the 'opt in' for Chrome?

I think Safari is marginally better. Don't know if it's good enough to keep though.
Chrome:

Safari:

In safari that is the default behaviour, not something we introduced just fyi

Fixed by #316.

Did anyone file a bug with Chrome about this issue? Would be good to reinstate in future if they fix their rendering.


Disables Sidebar

When I use the Brackets Compare extension it closes the sidebar. I cannot enable the sidebar again until I remove the Brackets Compare extension from Brackets.

Will fix this with the new update to the app

This is fixed with the overhaul of the app.


Iron Source adapter isn't initialized

I followed the AdMob instructions, yet the Iron Source adapter's output is:
I/flutter (13965): Adapter status for com.google.ads.mediation.ironsource.IronSourceMediationAdapter: 102: IronSource SDK requires an Activity context to initialize.
Is it needed to even include the adapter even though I only use it in the AdMob waterfall?

Hi @mikron123,  Thanks for filing the issue. can you please share the version of the ads plugin you are using and also the complete error logs, Additionally Please refer these docs https://developers.google.com/admob/android/mediation/ironsource
Thanks.

Using 1.0.0 for googleads
and copied the relevant plugin line from the AdMob IronSource adapter docs

Thanks for the info, Keeping this issue open for further investigation.

I think we have the same issue with UnityAds.
The mediation adapter worked fine until 0.13.5
Starting with 0.13.6 the mediation adapter can not load ads anymore. This behaviour did not change with google_mobile_ads 1.0.0. So we rolled back to 0.13.5 for now.

Closing this issue as fixed if you disagree feel free to write in the comments and I will reopen it.


When line is too big, it surpass the next line and mixes both end of current line and begins of next line.

Workaround is set hasLineNumbers to false, this just occour when lineNumbers is enabled

This is still happening when hasLineNumbers is set to false on 4.87.100.

Confirmed still a problem on react-log-viewer version 5.1.0

@nicolethoen Is there any motion on this? We are running into it in OpenShift AI as well

Glad to see it, thanks! I'll test soon

Issue was reported

Issue was still being seen on OCP when there is data coming back over stderr & stdout.  Reopening for further investigations.

Messages are being sent over stdout and stderr due to the script that is run with cockpit.spawn
The code is doing a CURL to retrieve the data.

I have tried out react-log-viewer version 5.2.0 and I'm still seeing this bug.


Import-Module AzureRM.NetCore should be replaced?

It appears Import-Module AzureRM.NetCore and Connect-AzureRmAccount don't work after spining up a container. The Docker Hub repo for the container indicates the command to connect to an Azure account is Add-AzureRmAccount, which does successfully work.
Alternatively, perhaps the documentation should read "Install-Module AzureRM.NetCore" and not "Import-Module AzureRM.NetCore".
Am I missing somthing?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d93f3194-ff59-02b1-8e3a-22a5647fe107
Version Independent ID: 51584194-e101-e4d7-a033-fdf63e2d1d1e
Content: Run Azure PowerShell in a Docker container
Content Source: docs-conceptual/azurermps-6.8.1/azurerm-ps-in-docker.md
Product: powershell
GitHub Login: @sptramer
Microsoft Alias: sttramer

Can you tell me which container you're trying to run? It may not have AzureRM.NetCore installed.

docker run -it azuresdk/azure-powershell-core:latest

Issuing an "Install-Module" command does work...that differs from what the doc indicates.

References to AzureRM.Netcore have been removed and an issue has been filed to add the Connect-AzureRmAccount cmdlet to Az. Going to close this issue.


All the examples on mobile devices cannot interactive

When I click on all the examples on my phone, there is no response at all. I don't know why this is

It seems that the mobile end is not adapted, and only displayed as pictures.


Update repositories.txt

https://github.com/dnzayan/ESP32_Pinoo
Library will be added to the Arduino Library Manager.

@ArduinoBot

@ArduinoBot


Redirect Loop

After auth with any providers, can't return. It's stuck between providers to return url.
Our Wordpress has Multisite feature and server runs Nginx 1.7.12 and PHPNG. We already configurated for;
location /wp-content/plugins/wordpress-social-login/hybridauth/ {
allow all;
}
I couldn't found the same issue on your documents & wordpress support forum

Your document has;
I'm having an infinite loop of redirection when using the popup for authentication
This can be caused when "PHP Register Globals" are On.
The solution is to make a trouble ticket with your web host to disable it, or, if you have a dedicated server and you know what are you doing then edit php.ini file and turn it Off.
that information.
But, PHP says;
As of » PHP 4.2.0, this directive defaults to off.
http://us2.php.net/manual/en/ini.core.php#ini.register-globals


feedback

your project is very good, the whitepaper is also very clear, I hope your project can be successful and successful in the future, I also hope the community can grow even bigger than now

@030499 Thank you for your testing and we hope to continue delivering high quality features and improvements!


cargo master fails to build with latest nightly

Problem

cargo +nightly test fails
Steps

cargo +nightly test

This errors starting with:
$ cargo +nightly test
    Updating crates.io index
   Compiling cargo-test-macro v0.1.0 (/home/r/src/rust/cargo/crates/cargo-test-macro)
   Compiling syn v1.0.8
   Compiling clap v2.32.0
   Compiling serde_derive v1.0.70
   Compiling libnghttp2-sys v0.1.2
   Compiling backtrace-sys v0.1.23
   Compiling libssh2-sys v0.2.13
   Compiling flate2 v1.0.12
   Compiling openssl v0.10.11
error[E0432]: unresolved imports `crate::ffi::Deflate`, `crate::ffi::Inflate`
 --> /home/r/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.12/src/mem.rs:6:33
  |
6 | use crate::ffi::{self, Backend, Deflate, DeflateBackend, Inflate, InflateBackend};
  |                                 ^^^^^^^                  ^^^^^^^ no `Inflate` in `ffi`
  |                                 |
  |                                 no `Deflate` in `ffi`

error: cannot find macro `__cfg_if_items` in this scope
  --> /home/r/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.12/src/ffi/mod.rs:40:1
   |
40 | / cfg_if::cfg_if! {
41 | |     if #[cfg(target_arch = "wasm32")] {
42 | |         mod rust;
43 | |         pub use self::rust::*;
...  |
50 | |     }
51 | | }
   | |_^
   |
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

error[E0433]: failed to resolve: could not find `uInt` in `ffi`
   --> /home/r/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.12/src/mem.rs:242:45
    |
242 |             assert!(dictionary.len() < ffi::uInt::max_value() as usize);
    |                                             ^^^^ could not find `uInt` in `ffi`

Notes
$ cargo +nightly version
cargo 1.40.0-nightly (5da4b4d47 2019-10-28)
$ rustc +nightly --version
rustc 1.41.0-nightly (82cf3a448 2019-11-14)

Hm, I can't seem to find a way to reproduce this.
This: cannot find macro __cfg_if_items in this scope  seems really strange.
Can you maybe try to update your Cargo.lock and see if that helps?  It's almost like cfg_if/flate2/miniz-sys are maybe using a too-old version?

Oh, I thought Cargo.lock would be version controlled... deleting that file fixes the problem.
Wouldn't it be better to have that file in git so that things are more reproducible?

Wouldn't it be better to have that file in git so that things are more reproducible?

It is in a sense on rust-lang/rust.  That is the official lock file that Cargo is built with.  I wouldn't want to maintain and sync two different lock files.
A solution might be to implement minimal versions for immediate dependencies, which can help catch situations like this.

I'm going to close this since it looks like it has been resolved, and I don't think we want to manage Cargo.lock in repo.

and I don't think we want to manage Cargo.lock in repo.

Fair. I'll just throw in the remark that Miri and Clippy both do have a Cargo.toml in their repo, and for Miri this has saved us some pain and so far not caused pain. The file is indeed not synced with the one in the rustc repo, but just having a stable baseline for local development and CI at all is tremendously useful for us.
YMMV.


Added login/registration feature

https://github.com/Suchitra-Sahoo/AgriLearnNetwork/assets/145560431/7b3b5d76-f1e8-4e2e-b54c-f6cd940fe1ab
Solved Issue #595
Added login/registration feature
Description
I have added a login/register button which will direct the user to a login/registration page.
##Feature Enhancement

[ ] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[X] I have performed a self-review of my code
[X] I have read and followed the Contribution Guidelines.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

Can you add a video to demonstrate your changes, also show the login page too.

Can you add a video to demonstrate your changes, also show the login page too.

@Suchitra-Sahoo I have added the video.

Once the form is filled it should show up a message about suuggesful login/registeration. Also there no way to go back to home so add the navbar and footer as in main page.


Fix: MakeGate error

📑 Description
Fix MakeGate return nil
✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

I've just checked this PR resolves the Gate panic. Thanks @hwbrzzl


Generated GPX file contains both  and  tags

Hi,
as discussed separately with Peter, I am encountering issues with the GPX graphhopper generates.
Indeed, as they contain both the rte and trk tags, the softwares I use are computing both thus generating a longer route than created.
As an example, I created the following mountain bike route:
https://graphhopper.com/maps/?point=43.87287%2C7.006069&point=43.866868%2C7.005318&point=43.851581%2C6.99415&point=43.844123%2C7.026186&point=43.839665%2C7.087083&point=43.843999%2C6.993184&point=43.849879%2C6.967607&point=43.872792%2C7.006075&vehicle=foot&elevation=true&layer=TF Cycle
Here is a recap of the behavior I am getting with different tools:

josm display both the route and the track when opening the GPX file.
Oruxmaps on Android considers both rte and trk, thus a 35km track becomes a 60km route+track overall. The elevation profile is pretty clear: you see more than 20kms at elevation 0 and then the real track starts.
this website also has the same symptoms : http://www.visugpx.com/index.php?i=N4SHU8rKaV (the URL is valid for 24hours). Here is a screenshot (which is more or less what Oruxmaps does as well):

http://www.imagehangar.com/view.php?pic=small/29703700_1422363331.jpg
I'm not sure what should exactly be the content of the gpx and I understand both route and track have their use but it would be nice to have the choice when downloading the gpx.
For example, http://www.openrunner.com/ has 2 different buttons to generate the gpx: one for route and one for track.
Any thoughts about this?
Regards,
Florent.

IMO this is a tooling problem but yes we could offer such a functionality to separate the download, although I don't like the idea of a complex interface. And GPX export is an expert-option by itself and also not really used that often.

josm display both the route and the track when opening the GPX file.

As separate routes?
When I e.g. use http://www.gpsvisualizer.com/ then it correctly displays them separately.

gpsvisualizer indeed displays both separately on the map, but when it comes to display the elevation profile, it appends the route to the track (as opposed as visugpx and Oruxmaps) and we again end up with a 60km+ total:
http://www.gpsvisualizer.com/display/20150128073940-48028-profile.png

Having added GPX support in Cruiser / Atlas lately, I started playing with the GraphHopper exported GPX file.
I'd vote too for some differentation between the turn instructions and the track in the gpx file.
They could be included as gpx waypoints instead of gpx route, so we don't confuse them with the detailed gpx track ?

As current systems properly import the data I would just provide 3 options: both, only route, only track ... would that serve your needs?

Yes the gpx data are valid.
I'm talking more of them being in the same gpx file and having a route with connected the turn instructions points with straight lines, which appears a bit strange with the detailed track.
But your suggestion is fine.

Indeed, having the 3 options you mentioned (both, only route, only track) would match most usages I guess.

Another interesting option would be to split the tracks into track segments:
<trk>
        <trkseg>
            <trkpt lat="x" lon="y"></trkpt>  //Depature
            ......
            <trkpt lat="" lon=""></trkpt>
        </trkseg>
        <trkseg>
            <trkpt lat="" lon=""></trkpt> // Via-point 1
            .........
            <trkpt lat="" lon=""></trkpt>
        </trkseg>
            .........

I think GPX export has 2 aspects:

Make optional the gpx info to be included (only one type, more than one etc.)
What tags are to used for that info

e.g.
I'd propose the route points (start, via, end) to be "waypoints"
and the calculated route to be "track" (single or split)

I have implemented a popup window where the user may select via three checkboxes (route, track,waypoints) what data should be put into the GPX file. Additinally I adopted the GPX generation such that only the checked parts get written. Now I have a small problem with the export of the waypoints. I'm exporting only the start point, via points and the endpoint as suggested. The problem is that the translation string for "finish" is not a valid name for a finish point. A waypoint called "Ziel erreicht" on a GPS device sounds strange. Additionally I'm missing a translation string for the start point.
For this reason I would like to do the following:

Add a translation string start_point into https://t.co/f086oJXAEI
Add a translation string finish_point into https://t.co/f086oJXAEI. I plan to copy the content of the current finish translation to it, but leave out the "!" character. For German the translation changes from "Ziel erreicht" just into "Ziel".
Rename the translation string finish to 'finish_reached', change the string name in https://t.co/f086oJXAEI and in the code

With some minor changes to ./core/files/update-translations.sh I was able to perform the generation of the translations text files. Therefore I would run it and afterwards prepare a pull request.
Is this OK?

Can you elaborate what route, track and waypoint option would include? I only see the following:

rte represents route - an ordered list of waypoints representing a series of turn points leading to a destination.

trk represents a track - an ordered list of points describing a path.

If 'waypoint' is something new I would create a separate issue for this, including the proposed translation keys. Here I would just make 3 options via Java API and web UI available: rte OR trk OR rte and trk

A GPX waypoint is something other on the level of "route" or "track". See
https://de.m.wikipedia.org/wiki/GPS_Exchange_Format
I would like to keep the support for it, because it is implemented now and I only have a separated commit for the java server java, but not for the java script part.  The change on the java side is very small. I'll leave the translation to a separate pull request. If really needed, I could separate it. But as this  justs costs time and I don't see any benefit, I would ask you to first take a look on it.
I'm using a Garmin GPS. Waypoints are places, which are displayed on the map with an icon. Until now, I always had to remove the route part by hand. The turn instructions messed up the way points on the device, as it also generates a separate place point for every turn. As a consequnce you see nearly nothing except icons. And  manual deletion of them on the device is a real pain. Therefore, I think that by putting only the start, the via and the endpoint to the waypoints allows the biggest flexibility for the end users, which we should reasonably support. If further flxibility is needed, the user always can use separate tools as e.g. gpsbabel.

Ok, this makes sense.  Is wpt used normally for start/via/end points?
Please go ahead with the PR including the 4th option 'wpt-only' besides 'all', 'trk-only', 'rte-only'. Also make sure (e.g. with existing or new test) that 'all' does not break a currently consuming client, i.e. only wpt is added, the other entries stay.

I'm in favor too for better handling the export gpx functionality.
A more simple gpx structure would be more user convenient.

I'm in favor too for better handling the gpx functionality.
A simpler gpx structure would be more user convenient.

What do you mean? This change here is only about optionally separating trk and rte, no changes to the structure are done.

Current situation has the routing as trk and the turn points as rte, which is a bit strange in rendering and processing in external devices (issue start comments).
That's where I'm referring too, that a better or optional selection would be beneficial.

Merged now to master


Cannot Build and Push solution with ARM64v8

Hi,
I am trying to do this tutorial and upload it to my Jetson Nano device. It uses ARM64. I have read the tutorial and it says that VSCode only supports AMD64 and ARM32 at this point. However, when I create a new solution and select python module, I get an option for ARM64v8.
I have read this article: https://devblogs.microsoft.com/iotdev/develop-and-debug-arm64-iot-edge-modules-in-visual-studio-code-preview/
that explains how to develop ARM64 modules in VScode. However when I click Build and Push IOT Edge Solution I get the following error:
Step 4/6 : RUN pip install -r requirements.txt
---> Running in adbac47976fe
standard_init_linux.go:211: exec user process caused "exec format error"
The command '/bin/sh -c pip install -r requirements.txt' returned a non-zero code: 1
Any help will be greatly appreciated. I want to be able to write a python file so that I can read some data from the Jetson Nano and send it to IOT Central to provide analytics.
Kind Regards,

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

ID: b251621c-bcb0-b9ce-bd87-90499766137b
Version Independent ID: b23ad340-bdfa-0597-9a0a-51c5e7b18b48
Content: Create custom Python module - Azure IoT Edge
Content Source: articles/iot-edge/tutorial-python-module.md
Service: iot-edge
GitHub Login: @shizn
Microsoft Alias: xshi

You should build ARM64 image on ARM64 devices. You can refer following blog to build the image on a remote ARM64 device: https://devblogs.microsoft.com/iotdev/easily-build-and-debug-iot-edge-modules-on-your-remote-device-with-azure-iot-edge-for-vs-code-1-9-0/

@Y07yoyo Thanks for the feedback! please check the suggestions provided by @blackchoey and let us know if you have further queries.

Hi,
Thanks for the link. I was able to compile and build the solution through the suggestions provided above. I just have a question related to the messages being sent to IoT Hub though.
This is the message that gets sent to IoT Hub:
[IoTHubMonitor] [10:41:27 AM] Message received from [JetsonNanoDemo02/PythonModule]:
{
"body": {
"machine": {
"temperature": 27.413572905894174,
"pressure": 1.7306602044689565
},
"ambient": {
"temperature": 21.459578114542914,
"humidity": 26
},
"timeCreated": "2020-04-28T00:41:27.3861936Z"
},
"applicationProperties": {
"$.cdid": "JetsonNanoDemo02",
"$.cmid": "SimulatedTemperatureSensor",
"MessageType": "Alert",
"batchId": "9fee87f5-04fd-4e65-8d1c-ac77ca9dffd1",
"sequenceNumber": "13"
}
}
But in Main.py, it shows several print statements with information such as "total Messages received" that do not appear on the message going to IoT Hub. How do I view these messages?
Kind Regards,

Do you mean the messages outputted to console using print() were not sent to IoT Hub? You need to use Azure IoT SDK to send messages. print() just outputted message to console.

We hope that answers your query.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


The Black Bars

What is your question?
Hey can I ask something? How did u do on the Black Bars Event?


Use pkg-config for dependencies in Makefile when possible

If we can use pkg-config in the Makefile, it will simplify dependency management and reduce potential mistakes. We should leave the existing code for systems that does not have it.

This is part of #2231.


fix(material/datepicker): fix date picker shortcuts

Fixes a bug in the Angular Material datepicker component where the datepicker should open/close only on alt+down and alt+up respectively but using shift + ctrl + alt + down or ctrl + alt + down the datepicker was opening, and same happens for closing the date picker added a condition to check if ctrl, shift or meta keys are present and if it is present restricting datepicker to not to open/close.
Fixes #25868

@sasidharansd can you run yarn bazel run //tools/public_api_guard:cdk/keycodes.md_api.accept to approve the API change?

@sasidharansd can you run yarn bazel run //tools/public_api_guard:cdk/keycodes.md_api.accept to approve the API change?

Done, I have ran the cmd and updated the PR.

@mmalerba  This is my first time contributing. Could you please help me to know. is there anything that I should be doing more or is everything done from my end and I should be wait for the pr to be reviewed ?

There is nothing to be done on your end for the moment. We need to run this against Google's internal tests and make sure everything passes. We'll reach out if there are issues


Eliminate the manual process of creating CI tests

We need to use the following setting keys to perform CI tests: ciTargetJavaVersions, ciTargetMinJavaVersions, and ciTargetScalaVersions.
However, these are implementation details and we should not expect the user to fulfill these settings. Instead, we should read all submodules and identify which projects are to be tested (test/skip := true). Then, we can read the crossScalaVersions and javaPlatform setting keys to generate CI tests automatically, without any manual intervention.

@guizmaii provided additional feedback suggesting that it would be more efficient to modify the default behavior of test generation to use the ++ test command instead. This command would reduce the need for many GitHub action pods to be used. However, we could still offer a setting for developers to customize this behavior as needed.


Class not found in edit mode

Tip: Use View.isInEditMode() in your custom views to skip code when shown in Eclipse

I have similar  problem


findComponent not working with stubbed component

I have a component I'm trying to test that uses router-link, and I'm trying to stub it out with my own component.
My test looks like this:
test('Displays a normal router link when the route is not hidden and behavior is set to hide', () => {
  const wrapper = mount(HidableRouterLink, {
    global: {
      stubs: { 'router-link': MyRouterLinkStub },
      provide: { store: reactive(new Store()) },
    },
    props: {
      behavior: 'hide',
      to: currentRoute,
    },
    slots: {
      default: 'hello',
    },
  })
  expect(wrapper.findComponent(MyRouterLinkStub).text()).toBe('hello')
})

My expected outcome is that the test passes, but instead it fails with "Cannot call text on an empty VueWrapper."
If I use console.log(wrapper.html()) it contains the contents of the MyRouterLinkStub component, so I know it's being inserted

@dospunk could you please show implementation of MyRouterLinkStub ?
Also could you try adding a name to your MyRouterLinkStub?

Here is my implementation,
<template>
  <slot v-bind="slotProps" />
</template>

Adding the name property makes it work

@dospunk Can you give it a try with VTU v2.0.0-rc.20, please? There have been a lot of changes since you opened this issue, so I'm wondering if this is still a problem or if we can close it.

I think this should no longer be an issue with the latest VTU releases, I'll close this. Feel free to reopen with a repro if that's not the case


Website: Update all quotes on website

Update quotes, names, companies and head shots
https://www.figma.com/file/buzGz2v44DfkUQf112KlcO/Landing-Page-Complete-Design?node-id=1591%3A98
Replace quote at top (next to Betting today is broken & exploitative.)

Replace next two quotes

Remove quote from bottom section
Centre the text block with image

@bconfortin  I forgot to add quotation marks to the Balaji quote, could you add these in.
The headshots look too compressed. You should be able to export these at a higher quality. Let me know if there's an issue getting these from the figma file

headshots look better. closing


ocaml-lsp update

Description of changes
tested as:
nix build -f. ocaml-ng.ocamlPackages_4_{06,07,08,09,10,11,12,13}.ocaml-lsp --keep-going
and on a real project with ocaml 4.08, 12, and 13
4.14 ppxlib does not build so untested.
Things done

Built on platform(s)

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

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

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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
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.

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

1 package built:

stog

@symphorien thanks for upstreaming this! I think you just missed dune-rpc-lwt

@anmonteiro As it is already merged. If you know the proper solution, feel free to open a new PR.

What do you mean by missed? Honestly I won't have time to fix it soon, anyway.


Add diagflat op

@fchollet

Codecov Report
Attention: Patch coverage is 85.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 60.12%. Comparing base (e0369f6) to head (fd8d6fd).

Files with missing lines
Patch %
Lines

keras/src/backend/jax/numpy.py
33.33%
2 Missing :warning:

keras/src/backend/tensorflow/numpy.py
33.33%
2 Missing :warning:

keras/src/backend/torch/numpy.py
33.33%
2 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (e0369f6) and HEAD (fd8d6fd). Click for more details.
HEAD has 6 uploads less than BASE

Flag
BASE (e0369f6)
HEAD (fd8d6fd)

keras
4
1

keras-torch
1
0

keras-tensorflow
1
0

keras-jax
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #20547       +/-   ##
===========================================
- Coverage   82.17%   60.12%   -22.05%
===========================================
  Files         515      515
  Lines       47982    48022       +40
  Branches     7509     7516        +7
===========================================
- Hits        39427    28872    -10555
- Misses       6738    17391    +10653
+ Partials     1817     1759       -58

Flag
Coverage Δ

keras
60.12% <85.00%> (-21.90%)
:arrow_down:

keras-jax
?

keras-numpy
60.12% <85.00%> (+0.01%)
:arrow_up:

keras-tensorflow
?

keras-torch
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

LGTM -- thanks for the addition. It's useful for Keras to increase its coverage of the NumPy API. Question for you: how are you prioritizing which ops to add?

I’ve compiled a list of all operations that are not yet in Keras and am gradually adding them one by one. To create this list, I systematically compared the operations available across different frameworks.


FeedBack - Some extended problems

I again thank the Cvat community for creating this amazing open source annotation tool, it has saved us so much time and effort.
While working with it I have observed certain things, if these could be resolved it will improve for better.
As we are trying to add some attributes to label too that include Pose and some other details, while doing so,
I  realized, that when I change a detail of a box (E.g. in one frame you see a whole person, in the next frame just the hand of the person and I want to change it from front to body part) it changes this detail also for the boxes before that (so in the frame before, where you could see the whole person it also says bodypart). How should I deal with that?
I tried Keyframe as well as outisde and then merge property but it doesn't help.
Secondly, for the settings of workspace i.e. Frame step, Save automatically etc, there is no SAVE CHANGES option thus you have to change it every time you start any task. Also there is no way to APPLY CHANGES, its just try and try until its saved.
It would really be great if you can throw light on these issues and could add them in the new version. I believe its not individual problem and would help many.

[... an attribute that] changes this detail also for the boxes before that [...]

There are 2 kinds of attributes - per-track and per-frame. When registering an attribute, there is a Mutable checkbox to switch between 2 variants.

Right! Can you explain the utilization a little more in detail please

Please, read the documentation https://github.com/openvinotoolkit/cvat/blob/develop/cvat/apps/documentation/user_guide.md#vocabulary

Just wondering if there could be some way to handle noisy frames, just in case if some frame is noisy and has nothing to be labelled and can be tagged noise.

@ZahraAnam , it looks like the question was answered. I will close the issue.

Yes sure!
Thank you for the help.
On Wed, Nov 24, 2021 at 8:45 AM Nikita Manovich @.***>
wrote:

@ZahraAnam https://github.com/ZahraAnam , it looks like the question
was answered. I will close the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openvinotoolkit/cvat/issues/2759#issuecomment-977612967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIIJTCSVWQ6PFMTQR4WHXITUNSJZLANCNFSM4W6LV55A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Best Regards,
Anam Zahra,
PhD Fellow MPI,
Email- @.***


add au/countrywide accuracy field

GNAF provides a few fields measuring geocode:

reliability a measure of how accurate the geometry of the geocode is, did the geometry come from the site boundary, road, locality?
confidence metric based on how many different address databases it appears in
and type.
geocode_type what is this address/geometry representing. eg. is it the building centroid or the front door to the building? or is it the property parcel centroid, or does it come from offsetting from the road centerline at a distance along the road? etc.

OA only provides accuracy.
Looking through which fields accually appear in the GNAF data and comparing the field definitions to the OA accuracy field, this is my suggested mapping. Comments welcome.

These changes are now reflected in http://data.openaddresses.io/cache/au-nov2016.zip.

These changes are now reflected in http://data.openaddresses.io/cache/au-nov2016.zip.

Awesome. thanks.


Add post for Lissted

:+1:

Please change the link to the homepage and change the date to publish.

@gcollazo What do you mean by publish? Should I change to today's date?

@ynnoj I just pushed a change that I think fixes what @gcollazo was talking about... @gcollazo can you let me know if you need any other changes

Looks good. I will publish on Monday. Thanks :)


Adding throttled to param

Codecov Report

Merging #3 (213e999) into master (d67c09d) will increase coverage by 0.31%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   84.81%   85.13%   +0.31%
==========================================
  Files         151      151
  Lines       17578    17957     +379
==========================================
+ Hits        14909    15287     +378
- Misses       2669     2670       +1

Impacted Files
Coverage Δ

panel/param.py
90.08% <66.66%> (-0.01%)
:arrow_down:

panel/pane/alert.py
100.00% <0.00%> (ø)

panel/tests/test_docs.py
100.00% <0.00%> (ø)

panel/tests/test_viewable.py
100.00% <0.00%> (ø)

panel/tests/pane/test_plot.py
100.00% <0.00%> (ø)

panel/tests/pane/test_image.py
100.00% <0.00%> (ø)

panel/tests/io/test_location.py
100.00% <0.00%> (ø)

panel/tests/io/test_notebook.py
100.00% <0.00%> (ø)

panel/tests/layout/test_base.py
100.00% <0.00%> (ø)

panel/tests/layout/test_grid.py
100.00% <0.00%> (ø)

... and 70 more

Continue to review full report at Codecov.

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


React Project Hosting issue on AWS.

Hello Everyone.
Hope you are doing well.
I was going to host my website using AWS.
But I got below issue.
2022-09-16T06:44:30.354Z [INFO]: Creating an optimized production build...
2022-09-16T06:45:47.774Z [INFO]: Compiled successfully.
2022-09-16T06:45:47.776Z [INFO]: File sizes after gzip:
2022-09-16T06:45:47.910Z [INFO]: 724.63 KB  build/static/js/13.3884d1db.chunk.js
2022-09-16T06:45:47.910Z [INFO]: 132.68 KB  build/static/js/main.1066e3a2.chunk.js
                                 6.55 KB    build/static/css/13.b9513a47.chunk.css
                                 4.27 KB    build/static/css/main.4c686009.chunk.css
                                 1.33 KB    build/static/js/runtime-main.c5c7d5bb.js
                                 172 B      build/static/js/tr-messages.0c205b2b.chunk.js
                                 172 B      build/static/js/vi-messages.98c0ba48.chunk.js
                                 172 B      build/static/js/zh-messages.b3fd10dc.chunk.js
                                 171 B      build/static/js/ar-messages.7611b85e.chunk.js
                                 171 B      build/static/js/es-messages.2397f56d.chunk.js
                                 171 B      build/static/js/fr-messages.1e109c62.chunk.js
                                 171 B      build/static/js/ko-messages.352b9aa2.chunk.js
                                 171 B      build/static/js/pl-messages.1d59d245.chunk.js
                                 171 B      build/static/js/pt-messages.050456a7.chunk.js
                                 170 B      build/static/js/de-messages.7e88e19a.chunk.js
2022-09-16T06:45:47.910Z [INFO]: 170 B      build/static/js/en-messages.621fe8bd.chunk.js
2022-09-16T06:45:47.911Z [INFO]: The project was built assuming it is hosted at ./.
                                 You can control this with the homepage field in your package.json.
                                 The build folder is ready to be deployed.
                                 Find out more about deployment here:
                                 https://cra.link/deployment
2022-09-16T06:45:48.015Z [INFO]: **Executing command: node ./node_modules/gulp/bin/gulp.js**
2022-09-16T06:45:48.781Z [INFO]: [06:45:48]
2022-09-16T06:45:48.781Z [INFO]: _Using gulpfile /codebuild/output/src088257269/src/SpozzAirdrop-v1/gulpfile.js_
2022-09-16T06:45:48.782Z [WARNING]: [06:45:48]
2022-09-16T06:45:48.782Z [WARNING]: **Task never defined: default**
                                    [06:45:48] To list available tasks, try running: gulp --tasks
2022-09-16T06:45:48.790Z [ERROR]: !!! Build failed
2022-09-16T06:45:48.790Z [ERROR]: !!! Non-Zero Exit Code detected
2022-09-16T06:45:48.790Z [INFO]: # Starting environment caching...
2022-09-16T06:45:48.791Z [INFO]: # Environment caching completed
Terminating logging.

If anyone know this issue, please help me.
Thanks.

Hi @teradev203 👋🏽 thanks for opening this issue. We need more details such as reproduction steps, your app ID, and your build settings. Please fill out the Bug report template so that we can assist you with this issue. https://github.com/aws-amplify/amplify-hosting/issues/new?assignees=&labels=&template=1.bug_report.yaml


foo

bar

Oops, wanted to create a blank issue in my fork and didn't realize it redirected me to the upstream repository.


Update to 6.6.4-SNAPSHOT

The version should be updated after the release of v6.6.3.

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


Cannot generate the example image

How can I generate the netflix_05.png?

I've been trying distinct parameter configurations. However, it is not clear what ControlNet (w/ Canny) + Generation configurations you used.
Also is it expected the error: Lora not found: test_task_model?
I have ControlNet extension installed and identified.
I fed ControlNet with nextflix_01.png

I managed to solve th eLora not found: test_task_modelerror. However, the generated image stilll doesn't look like the one generated in the example you provided (i.e., netflix_5.png).

Can you tell me how you did that? Mine seems weird as well

Did I make any mistakes in finetuning? I should have followed the instruction strictly

like this


Update doxygen version to 1.10.0 to match MRC

Description

Closes
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

Note: Due to the large number of errors that are occurring with the latest doxygen version, we are exploring reverting the MRC version back down to match Morpheus instead of the other way around

Need to downgrade MRC's version of doxygen, as we can't upgrade Morpheus's without updating some of the documentation, and we don't have bandwidth for that at the moment.


simple user agent that only prints body

Adds a simple user agent that prints just the text within the body tag, all tags within that are treated the same.
The main component of this is a function get_node_by_path which accepts a document and vector of tag names and returns the node at that path if it exists. It might make sense to move this function to the library. Though if we add css selectors that might replace this function.

@ynewmark Nice pr!  Could you try and see if you can fix the CI issues (clippy / fmt)

Can you rerun the workflow?

All green. @ynewmark Thanks for the PR!


train_meshtransformer.py not available

The mentioned "train_meshtransformer.py" is not present. Could you please upload it? Thank you!

Hi !
Thank you for your message ! There was indeed a mistake in the readme, the correct file is "train_graphvit.py". It has been fixed.


Using uv, how do I implement functionality similar to cmdclass in setup.py?

from setuptools.command.install import install

class CustomInstallCommand(install):

    def run(self):
        install.run(self)

        script_path = os.path.join(os.path.dirname(__file__), "install.sh")
        if os.path.exists(script_path):
            print("Running install_script.sh...")
            try:
                subprocess.check_call(["/bin/bash", script_path])
            except subprocess.CalledProcessError as e:
                print(f"Error executing script {script_path}: {e}")
            sys.exit(1)
        else:
            print(f"Script {script_path} not found!")

setup(
    ...
    cmdclass={
        "install": CustomInstallCommand,
    },
)

Using uv, how do I implement functionality similar to cmdclass in setup.py?

Not familiar with that particular setuptools functionality, but it looks like it's for overriding setuptools behavior at build time? You should just be able to use setuptools for that still with uv. uv is just a build frontend, we have to invoke a build backend (i.e., setuptools) to install packages.


⚠ Bofrost.at has degraded performance

In 77eb641, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1058 ms

Resolved: Bofrost.at performance has improved in 45650f0.


make access_token optional to allow usage of offline models

Some offline models are OpenAI compatible (ollama for instance : https://ollama.com/blog/openai-compatibility)
Offline models do not require an access token.
This PRs makes access token optional in order to be able to use self hosted models
All Submissions:

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

@alexrudall can this be merged?

I'm open to this but would like some instructions on how to actually do this in the README. Like, a link to how to run Ollama locally and then exactly what config is needed. Also, that error is I think helpful for people who just want to use OpenAI API - might be better to pass an option like api_type: :ollama to skip the error

I've added a section in the readme to explain how to use this gem with ollama
I believe we should not add a new option in the configuration, keeping the same API allow gem using this gem to work without any change.
Here is the behavior of this branch when we try to connect to OpenAPI without an acess_token :
client = OpenAI::Client.new
response = client.chat(
  parameters: {
    model: "mistral:latest", # Required.
    messages: [{ role: "user", content: "Hello!"}], # Required.
    temperature: 0.7,
})
# => OpenAI HTTP Error (spotted in ruby-openai 6.3.1): {"error"=>{"message"=>"You didn't provide an API key. You need to provide your API key in an Authorization header using Bearer auth (i.e. Authorization: Bearer YOUR_KEY), or as the password field (with blank username) if you're accessing the API from your browser and are prompted for a username and password. You can obtain an API key from https://platform.openai.com/account/api-keys.", "type"=>"invalid_request_error", "param"=>nil, "code"=>nil}}

Thanks @ThomasSevestre - after testing I went with your original idea and removed the access_token check - appreciate your work on this!


Add Ollama embeddings support

Based on the recent PR by herrjemand in the Langchain python repository I have implemented a class for embeddings using Ollama.
I have only added a little bit of documentation and two simple test cases. This PR might very well not be production ready yet, feedback is much appreciated.
Current problems I see with the implementation:

The default optional constructor params "maxConcurrency" and "maxRetries" have to exist in order to extend the base Embeddings class, but are unused by the OllamaEmbeddings class.

In order to be consistent with the llms/ollama and chatModels/ollama classes, the api request params are supplied in camelCase ("useMMap") and are then converted to the snake_case format used by the ollama API ("use_mmap").
This is rather awkward, imo.

Thank you for this!
Agree on the mapping being awkward but it's the general style of the repo/JS generally at the moment - it's nice to keep people from guessing which style to use in general.

Overall looks great!

@jacoblee93 I have implemented your suggestions and added some basic documentation for the embedding model.

Thanks very much!


Selfhosting directs to opensprites.org ._.

How is this not a bug.

He's talking about hosting a development copy locally, by the way.
No OS developer here uses a local copy because the whole website is specifically built around our setup and getting every link on the website to point to its root location instead of the OS domain isn't worthwhile.

It still is worthwhile what if you change domain? Find and replace could
fix this.
On 21 Mar 2016 9:58 p.m., "James"<EMAIL_ADDRESS>wrote:

He's talking about hosting a development copy locally, by the way.
No OS developer here uses a local copy because the whole website is
specifically built around our setup and getting every link on the website
to point to its root location instead of the OS domain isn't worthwhile.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/OpenSprites/OpenSprites/issues/207#issuecomment-199504814

Why would we ever change domain names?

You never know.

Honestly, I agree with Quillz, if I'm reading this correctly - are you saying to not develop locally?

That would sorta work, l am suggesting a way to instantly test your code.
On 21 Mar 2016 10:14 p.m., "Matthew"<EMAIL_ADDRESS>wrote:

Honestly, I agree with Quillz, if I'm reading this correctly - are you
saying to not develop locally?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/OpenSprites/OpenSprites/issues/207#issuecomment-199512509

unittestsunittestsunittestsunittests

Wut the hell
On 21 Mar 2016 10:18 p.m., "Matthew"<EMAIL_ADDRESS>wrote:

unittestsunittestsunittestsunittests
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/OpenSprites/OpenSprites/issues/207#issuecomment-199513848

https://lmddgtfy.net/?q=unittest

Oh yeah you guys are using straight PHP aren't you lol
This is a great example of technical debt :P

A python library? I didn't think open sprites used python?
On 21 Mar 2016 10:33 p.m., "Firedrake969"<EMAIL_ADDRESS>wrote:

Oh yeah you guys are using straight PHP aren't you lol
This is a great example of technical debt :P
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/OpenSprites/OpenSprites/issues/207#issuecomment-199518869

Where did Python come into this?  o.o

Unittest

That's not exclusive to Python :P

To be honest the best way to fix these bugs is a rewrite.
/controversial

^ pls rewrite in node k tnx bye

that wouldn't be that bad :P

Pls rewrite in English merci

@QuillzLouis mais qui etait telephone?


Compatibility issue with android 8.1

Hi,
Thanks for your tool, i'm using it on project with android 8.0.
But i started to work on a project with android 8.1
Will StyleableToast be compatible with android 8.1 ?
Thank you.
Regards, Sam.

Hi. Yes it should. There is support for the new Toast in Android Oreo. If
it's not working then please just make a new issue
On Fri, Jul 13, 2018, 19:48 SamMcfly<EMAIL_ADDRESS>wrote:

Hi,
Thanks for your tool, i'm using it on project with android 8.0.
But i started to work on a project with android 8.1
Will StyleableToast be compatible with android 8.1 ?
Thank you.
Regards, Sam.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Muddz/StyleableToast/issues/34, or mute the thread
https://github.com/notifications/unsubscribe-auth/AOIZaAAPxyJ-aD2E_tUIbmk7rmNzG8hkks5uGN2CgaJpZM4VPRfu
.


FlexibleSslListener secure remember me not working

As far as I can tell this feature is not working.  The remember me cookie remains unsecure.

Confirmed, looking into it. Thanks!

This can be configured at the firewall level as follows:
security:
    firewalls:
        main:
            remember_me:
                key:      aSecretKey
                lifetime: 3600
                path:     /
                domain:   ~
                secure: true

So I'm not sure we even have to take care of it. Jordi?

@igorw There is also this. Although, I'm not sure if requires_channel is only used in matching or if it will do the switch.

@dustin10: As far as I know it only matches (or does not match).

@igorw @kbond I still think we should fix it, this basically should enforce the core config to do secure remember-me cookies.


Disconnecting wallets doesn't work properly

Describe the bug
If you use the Disconnect button inside the Template's wallet, this seems to only disconnect it internally from the app.  If you hit 'connect' again it will reconnect to the same account.
This is perfectly normal behavior.
To connect to a different account you need to disconnect using Metamask's > Connected Sites > disconnect.
Which works.
As soon as you do this, however the Metamask connect in the App pops up.  It feels a bit intrusive and I think this should be prevented from happening.
The bug is that after disconnecting either way (App or MM), the EventChannel on listenAccountChange() in Wallet.ts (ethers 6 - I haven't tried 5) is not firing. Thus your app is not aware of the account disconnection.
I understand that you would most likely isProtected your page.  But I don't think you should rely on that and I think the listener should be consistent and complete.
To Reproduce
Steps to reproduce the behavior:

Connect to account
Go to metamask in browser and disconnect account from site
In a Page that uses EventChannel on listenAccountChange(), the event doesnt fire so your page is unaware of the change.

Expected behavior
Your pages that use  EventChannel on listenAccountChange() should fire when an account is disconnected either in App or via Metamask.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS] MacOS 13.6.3
Browser [e.g. chrome, safari] Brave
Version [e.g. 22] v1.62.153

Smartphone (please complete the following information):

Device: [e.g. iPhone6] NA
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

hi and thanks for the detailed description. I think I know this problem. Like you said, Metamask's "disconnect from site" action doesn't trigger the event that listenAccountChange method listening. I can't remember now, but most probably I couldn't find a way to handle this case at that time while using ethersV5. I'll check this when I have time.

Hi @huseyindeniz , this is biting me.  Can you tell me where you think the problem lies and I'll have a look?

hi @oehm-smith , this UX issue is also bothering me, but I'm not sure whether it's a "web3 wallet's UX issue" or "this template's login flow issue". Let me explain.
When a user connects to a dApp with their Web3 wallet:

Web3 wallet first asks the user to connect their wallet.
Once this is done by the user, it doesn't change until the user opens the web3 wallet and clicks "disconnect" inside the Web3 wallet.

So, in this template, or in other dApps, when you click disconnect in the dApp, your account never disconnected from the dApp in the Web3 wallet. Only the dapp's current internal connection cleared.
If you look at the recordings below, you will see that when users click disconnect inside the dApp, it only clears the dApp connection state, not web3 wallet's. But in the second example, there is a problem. It doesn't listen to account changes correctly. And because of this, users could think they connected with their account A but dApp connected to account B. This problem is exists in almost every dApp, because almost all of them use those external web3 modal packages. This is actually the reason I wrote my own. Normally, I prefer delegating these kinds of things to the dedicated packages, but current Web3 login packages have lots of problems.
As a summary, could you record a session so I can see exactly what part is confusing users in my implementation. I agree that there is a scenario that confuses web3 users, but I can't identify the exact problem. I mean, this issue was a bug at the beginning but I solved that bug in this issue: https://github.com/huseyindeniz/vite-react-dapp-template/issues/24 . Now it's more like a design issue.
Note: All the business logic of this flow is under

src/features/wallet/models
src/services/ethersV6/wallet/WalletAPI.

src/features/wallet/components is UI logic of this flow.
https://github.com/huseyindeniz/vite-react-dapp-template/assets/1695352/58ec0e28-b35e-455c-aeac-6a496b276cbf
https://github.com/huseyindeniz/vite-react-dapp-template/assets/1695352/663071b3-e858-45c5-a539-9712bc52fcc9

I think my problem was related to MM.  It somehow became stuck on one account and I couldn't disconnect it.  Hence my having to go in to application settings to remove data.  But i can't reproduce that now, however I will keep an eye on it and update this or a new issue if I see again.
When it is working properly, as it is now, it makes sense.
What's confusing is - and this is not a problem with this app, but with dapps and wallets in general - connecting to multiple accounts.  If you disconnect in the app, I feel that it should give you the choice of which of the connected wallets to connect to.  It is not a big problem - it is just that you need to manage the connections in MM.  It feels unintuitive.  Again, a wallet problem and not your dapp.
Now that I worked out my state got messed up and I've played around with it more, I'm happy with what I'm seeing :).  I saw one glitch when I disconnected in MM and it didn't disconnect in the app, but like the other example I can't reproduce this either.
I will close this issue and reopen if I have any solid evidence of a problem.

Actually, I think there is still a problem in the flow, and I can't reproduce it either. But sometimes login flow can fail exactly because of this issue. I mean, sometimes I need to correct things inside the Web3 wallet manually to fix the problem. But I agree; we can close this for now. When the issue is more clean and consistently reproducible, it can be opened again. Thanks for your effort on this one.


make hill pass through the origin

Closes https://github.com/pymc-labs/pymc-marketing/issues/917 by adding a constant

📚 Documentation preview 📚: https://pymc-marketing--920.org.readthedocs.build/en/920/

The plot of the current priors as reference: https://pymc-marketing--920.org.readthedocs.build/en/920/api/generated/pymc_marketing.mmm.components.saturation.HillSaturation.html

The plot of the current priors as reference: https://pymc-marketing--920.org.readthedocs.build/en/920/api/generated/pymc_marketing.mmm.components.saturation.HillSaturation.html

Looks good as they all pass through the origin as expected. I constrained the priors a bit.


Parse error should be clearer to users

          Parse error should be clearer to users, use another PR to resolve this problem

Originally posted by @h-a-n-a in https://github.com/modern-js-dev/rspack/issues/1665#issuecomment-1386449431

I guess this was solved, if not please create a new issue.


TX Disable  since 16.3

Why, starting from firmware 16.3 and up to 17.1, when trying to TX, even a PMR band writes TX DISABLE. Everything works in version 16.2. Perhaps it has something to do with the menu item 10 - RxOffs:? I don't understand why the value 0.00262 Mhz is entered there. I didn't put it there. in new firmware, should this value be =0 ?
I will be very grateful for the answer!

Just read the online manual....
and report if you understand.... (or not.)
Than item may be closed.
Tip... read last line... starting with IMPORTANT:

Just read the online manual....
and report if you understand.... (or not.)
Than item may be closed.
Tip... read last line... starting with IMPORTANT:

Thank you very much, I couldn't understand this point. I entered the value 0 there and it worked, thank you!


built-in generate-completion subcommand: can't get it working

I can't get the built-in generate-completion subcommand to work:
Example: 2 source files, one for main command and one for subcommand
Main command
package test;
import picocli.CommandLine;

public class AutoCompleteDemo {
    public static void main(String[] args) {
        CommandLine cmd = new CommandLine(new TestCommand());
        cmd.execute("generate-completion");
    }
}

Subcommand
package test;

import picocli.CommandLine;
import picocli.CommandLine.Command;
import picocli.CommandLine.Option;

@Command(name = "testcommand", subcommands = { CommandLine.HelpCommand.class, picocli.AutoComplete.GenerateCompletion.class },
mixinStandardHelpOptions = true, description = "test")
public class TestCommand implements Runnable {

    @Option(names="-n")
    int n;

    @Override
    public void run() { }
}

I can produce the completion script by invoking picocli.AutoComplete from the command line:
$ java -cp "AutoCompleteDemo.jar;picocli-4.1.0.jar"  picocli.AutoComplete -f  test.TestCommand
$ echo $?
0
$ ls testcommand_completion
testcommand_completion*

However, I cannot produce the completion script by simply using the subcommand generate-completion:
$ java -cp "AutoCompleteDemo.jar;picocli-4.1.0.jar"  test.AutoCompleteDemo generate-completion
$ echo $?
0

The exit code is 0, however, after running the subcommand, I could not find any file inside the directory (debugging output is given below).
What am I doing wrong?
By the way: is there any way (--verbose switch or so) to get an idea what is running behind the scenes when generating the completion script? Would be really helpful!
Debugging output
$ java -cp "S.jar;picocli-4.1.0.jar"  test.AutoCompleteDemo
[picocli DEBUG] Creating CommandSpec for object of class test.TestCommand with factory picocli.CommandLine$DefaultFactory
[picocli DEBUG] Creating CommandSpec for object of class picocli.CommandLine$HelpCommand with factory picocli.CommandLine$DefaultFactory
[picocli DEBUG] Adding subcommand 'help' to 'testcommand'
[picocli DEBUG] Created Messages from resourceBundle[base=null] for command 'help' (picocli.CommandLine$HelpCommand)
[picocli DEBUG] Creating CommandSpec for object of class picocli.AutoComplete$GenerateCompletion with factory picocli.CommandLine$DefaultFactory
[picocli DEBUG] Creating CommandSpec for object of class picocli.CommandLine$AutoHelpMixin with factory picocli.CommandLine$DefaultFactory
[picocli DEBUG] Adding subcommand 'generate-completion' to 'testcommand'
[picocli DEBUG] Created Messages from resourceBundle[base=null] for command 'generate-completion' (picocli.AutoComplete$GenerateCompletion)
[picocli DEBUG] Creating CommandSpec for object of class picocli.CommandLine$AutoHelpMixin with factory picocli.CommandLine$DefaultFactory
[picocli INFO] Picocli version: 4.1.0, JVM: 11.0.1 (Oracle Corporation Java HotSpot(TM) 64-Bit Server VM 11.0.1+13-LTS), OS: Windows 10 10.0 amd64
[picocli INFO] Parsing 1 command line args [generate-completion]
[picocli DEBUG] Parser configuration: posixClusteredShortOptionsAllowed=true, stopAtPositional=false, stopAtUnmatched=false, separator=null, overwrittenOptionsAllowed=false, unmatchedArgumentsAllowed=false, expandAtFiles=true, atFileCommentChar=#, useSimplifiedAtFiles=false, endOfOptionsDelimiter=--, limitSplit=false, aritySatisfiedByAttachedOptionParam=false, toggleBooleanFlags=false, unmatchedOptionsArePositionalParams=false, collectErrors=false,caseInsensitiveEnumValuesAllowed=false, trimQuotes=false, splitQuotedStrings=false
[picocli DEBUG] (ANSI is enabled by default: systemproperty[picocli.ansi]=null, isatty=false, TERM=xterm, OSTYPE=null, isWindows=true, JansiConsoleInstalled=false, ANSICON=null, ConEmuANSI=null, NO_COLOR=null, CLICOLOR=null, CLICOLOR_FORCE=null)
[picocli DEBUG] Set initial value for field int test.TestCommand.n of type int to 0.
[picocli DEBUG] Set initial value for field boolean picocli.CommandLine$AutoHelpMixin.helpRequested of type boolean to false.
[picocli DEBUG] Set initial value for field boolean picocli.CommandLine$AutoHelpMixin.versionRequested of type boolean to false.
[picocli DEBUG] Initializing test.TestCommand: 3 options, 0 positional parameters, 0 required, 0 groups, 2 subcommands.
[picocli DEBUG] Processing argument 'generate-completion'. Remainder=[]
[picocli DEBUG] Found subcommand 'generate-completion' (picocli.AutoComplete$GenerateCompletion)
[picocli DEBUG] Set initial value for field boolean picocli.CommandLine$AutoHelpMixin.helpRequested of type boolean to false.
[picocli DEBUG] Set initial value for field boolean picocli.CommandLine$AutoHelpMixin.versionRequested of type boolean to false.
[picocli DEBUG] Initializing picocli.AutoComplete$GenerateCompletion: 2 options, 0 positional parameters, 0 required, 0 groups, 0 subcommands.

This is a bug. Thank you for finding this! My tests did not show it because I was setting the destination stream to a StringWriter. If the stream is not set, then the library will create a new PrintWriter (wrapped around System.out) for every call to CommandLine.getOut. This internally has a buffer. The autocompletion command calls getOut three times: once for the script, once for the closing \n, and once to call flush(). It used to be simply getOut().println(script), but I replaced that with the current code to ensure that the command never prints a Windows line separator \r\n to the script (which can cause weird errors). However, I failed to test this with the standard out stream...
This exposes an existing problem with getOut. I will fix this ASAP.

I released picocli 4.1.1 with the fix for this issue.
Thanks again for finding this!


Important Information about NAT Gateway in your Account

Hello! AWS emailed me that our EKS configuration (done with terraform-aws-modules/eks/aws)
is not a good.
There is their email text:
Hello,
We are reaching out because your AWS resources in one or more VPCs are sharing one NAT Gateway. Each NAT gateway is created in a specific Availability Zone and implemented with redundancy in that zone only. Thus, if the NAT gateway's Availability Zone is down, resources in the other Availability Zones that are using that NAT gateway lose internet access too.
To maintain availability across Availability Zones, we recommend that you create a NAT gateway in each Availability Zone in your VPC(s) and configure your routing to ensure that resources use the NAT gateway in the same Availability Zone.
The following is a list of your VPC ID(s) with a single NAT Gateway:
vpc-123
vpc-456
Should you have any questions or concerns, please reach the AWS Support team [1].
[1] http://aws.amazon.com/support
Sincerely,
Amazon Web Services
I agree with them, is it possible to have option to create NAT Gateway per AZ, but not a single one for all subnets ?

oh sorry there is doc about it:
One NAT Gateway per availability zone
If one_nat_gateway_per_az = true and single_nat_gateway = false, then the module will place one NAT gateway in each availability zone you specify in var.azs. There are some requirements around using this feature flag:

The variable var.azs must be specified.
The number of public subnet CIDR blocks specified in public_subnets must be greater than or equal to the number of availability zones specified in var.azs. This is to ensure that each NAT Gateway has a dedicated public subnet to deploy to.

oh sorry there is doc about it:
One NAT Gateway per availability zone
If one_nat_gateway_per_az = true and single_nat_gateway = false, then the module will place one NAT gateway in each availability zone you specify in var.azs. There are some requirements around using this feature flag:

The variable var.azs must be specified.
The number of public subnet CIDR blocks specified in public_subnets must be greater than or equal to the number of availability zones specified in var.azs. This is to ensure that each NAT Gateway has a dedicated public subnet to deploy to.


runtime: chanrecv funcs don't actually use the *chantype parameter

func chanrecv(t *chantype, c *hchan, ep unsafe.Pointer, block bool) (selected, received bool)

The t *chantype parameter isn't actually used. I assume it could be removed, but the entry point magic scares me a bit and I don't dare do a CL myself.
Unless the parameter is actually used/needed for a reason that's not obvious given the implementation?
Found with https://github.com/mvdan/unparam.

Also worth noting that chansend does use its t *chantype parameter, but only if either raceenabled or msanenabled are true. Would either of those need the type parameter at some point in the future?

The chantype is stored as a field in the hchan anyway. I think we can get rid of the parameters.
Do you want to send a CAL?

Sure, I'll get on that.

The chantype is stored as a field in the hchan anyway.

So you mean get rid of it in chansend too?

So you mean get rid of it in chansend too?

Correct.

Having some issues with panics, I'm likely missing some change to make the compiler and/or runtime fully aware of the new func signature. I'll post the CL on monday in any case, asking for pointers if I'm still stuck.

Ah, found it - a missing mkcall1 tweak in gc.


Update version on pub.dartlang.org

There are have been a few bug fixes recently. Can we get a new version pushed to https://pub.dartlang.org/packages/google_maps_webservice?

done :)


Compilation error

Hello,
I don't know if it's a problem for everyone but there is a compilation error on Android with the updated Google Play services.
Prevent Error: more than one library with package name 'com.google.android.gms.license' with Google Play services 12.0.0

Hi see https://github.com/idehub/react-native-google-analytics-bridge#use-environments-google-service-version-due-to-crash-on-google-service-version-12

Thanks for the info!


EL8 uses rng-tools for entropy, not haveged

Description
This PR changes the EL8 packaging to depend on rng-tools instead of the deprecated haveged that was inadvertently added previously. There is no haveged provided in EL8.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[x] Trivial

Screenshots (if appropriate):
How Has This Been Tested?
cloudstack-management EL8 RPM previously would not install as no dependency was found in standard repos. Now it does. Also confirmed rngd running after install.

looks good but @rohityadavcloud question seems valid for newer distros (debian and suse) I would not change el7 in this perspect.

That's a good observation @DaanHoogland. I'm not in a position to actually test Suse or others, I only made the change on EL8 because it was something I ran into and was able to validate a fix for.

@mlsorensen can you check the other spec files in el7 and suse directories under packaging, and apply the same pattern of changes? If you git grep on haveged in the debian directory you can apply the same pattern of changes there as well. For testing we can use blueorangutan to kick packaging and run smoketest on all those distros.

@blueorangutan package

@rohityadavcloud a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan help

@rohityadavcloud I understand these words: "help", "hello", "thanks", "package", "test"
Test command usage: test [mgmt os] [hypervisor] [keepEnv] [qemuEv]
Mgmt OS options: ['centos7', 'centos6', 'suse15', 'alma8', 'ubuntu18', 'ubuntu22', 'ubuntu20', 'rocky8']
Hypervisor options: ['kvm-centos6', 'kvm-centos7', 'kvm-rocky8', 'kvm-alma8', 'kvm-ubuntu18', 'kvm-ubuntu20', 'kvm-ubuntu22', 'kvm-suse15', 'vmware-55u3', 'vmware-60u2', 'vmware-65u2', 'vmware-67u3', 'vmware-70u1', 'vmware-70u2', 'vmware-70u3', 'xenserver-65sp1', 'xenserver-71', 'xenserver-74', 'xcpng74', 'xcpng76', 'xcpng80', 'xcpng81', 'xcpng82']
Note: when keepEnv is passed, you need to specify mgmt server os and hypervisor or use the matrix command.
when qemuEv is passed, it will deploy KVM hyperviosr hosts with qemu-kvm-ev, else it will default to stock qemu.
Package command usage: package [all(default value),kvm,xen,vmware,hyperv,ovm] - a comma separated list can be passed with package command to bundle the required hypervisor's systemVM templates. Not passing any argument will bundle all - kvm,xen and vmware templates.
Blessed contributors for kicking Trillian test jobs: ['rohityadavcloud', 'nvazquez', 'borisstoyanov', 'DaanHoogland', 'shwstppr', 'andrijapanicsb', 'rahulbcn27', 'davidjumani', 'harikrishna-patnala', 'vladimirpetrov', 'weizhouapache', 'NuxRo', 'mlsorensen']

@blueorangutan test matrix

@blueorangutan test

@rohityadavcloud a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@blueorangutan test rocky8 kvm-rocky8

@rohityadavcloud a Trillian-Jenkins test job (rocky8 mgmt + kvm-rocky8) has been kicked to run smoke tests

@blueorangutan test ubuntu20 kvm-ubuntu20

@rohityadavcloud a Trillian-Jenkins test job (ubuntu20 mgmt + kvm-ubuntu20) has been kicked to run smoke tests

@blueorangutan test suse15 vmware-70u3

@rohityadavcloud a Trillian-Jenkins test job (suse15 mgmt + vmware-70u3) has been kicked to run smoke tests

Trillian Build Failed (tid-5110)

Trillian test result (tid-5108)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 44471 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6775-t5108-kvm-centos7.zip
Smoke tests completed. 103 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers
Failure
471.94
test_vpc_redundant.py

Trillian test result (tid-5111)
Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server s15
Total time taken: 48814 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6775-t5111-vmware-70u3.zip
Smoke tests completed. 103 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_list_management_server_metrics
Failure
0.02
test_metrics_api.py

Trillian test result (tid-5109)
Environment: kvm-rocky8 (x2), Advanced Networking with Mgmt server r8
Total time taken: 50686 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6775-t5109-kvm-rocky8.zip
Smoke tests completed. 103 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_08_upgrade_kubernetes_ha_cluster
Failure
764.04
test_kubernetes_clusters.py

Trillian test result (tid-5113)
Environment: kvm-ubuntu20 (x2), Advanced Networking with Mgmt server u20
Total time taken: 40668 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6775-t5113-kvm-ubuntu20.zip
Smoke tests completed. 104 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

I see mgmt server pkg change is working, we've tested Ubuntu 20, Suse15, CentOS7 and Rocky Linux8 as mgmt server in smoketests. Merging based on this.


Tweak .gitignore to avoid checking in .map files

it'd be good to avoid checking in generated .css files as well (but keep the non-generated CSS files)

done.


I can't set a cursor

After last update (0.13.0) I can't set a cursor (I want i-beam type).

The settings have changed quite a bit, to allow for more flexibility. I apologize for the inconvenience. For an i-beam cursor you'll have to set the background color to transparent and the border style to i-beam and set a border color.

Having the same issue. I'm not seeing the options you specify in the config interface. Should we be editing the config.cson or am I missing something?

Ok, I edited my config.cson with some success to the following:
First find the old "block-cursor" config.
Replaced it with this:
"block-cursor":
global:
blinkOn:
borderStyle: "i-beam"
borderColor: "#F92"
backgroundColor: "transparent"
blinkOff:
borderStyle: "i-beam"
borderColor: "#666"
pulseDuration: 3000
Pulse duration doesn't seem to be working, however. Not sure if this is just for I-beam type cursors.

Forgive my ignorance @njoannidi, where is config.cson file?

@njoannidi Yeah, there's a bug in Atom that causes some settings to not appear in the settings view. It's been reported a few weeks back, but nothing has happened since, and I just wanted to get this new version out. I apologize for the inconvenience.
I'll take a look at the pulseDuration setting in the weekend.
@timhecker You can find the config.cson in ~/.atom.

@olmokramer No problem. Thanks for the awesome plugin!  Do you want me to make a separate issue for pulseDuration?

@njoannidi Yes please, create a separate issue. Easier to keep track of :) Thanks!


[Question] Alpine 3.4?

Have you thought about bumping the FROM to alpine:3.4? This way you can run apk --update add supervisor and it'll install 3.2.0-r0.

Oh, I haven't actually touched this in a while, wasn't expecting anyone to use it.
I'll see if I can find some time to take a look at it again, but you're of course welcome to submit a PR if you want.

Closed by fbafb2ec38e4deafd9aea731f27f4a217290401b.


ci workflow: trigger on pull_request too

GitHub should be clever enough to not expose secrets for external pull
request runs.
Good idea?

Following discussion, probably not for now.


Batch

10: completed. --> completed successfully
I think we shall mention that a batch may never reach COMPLETED or PARTIAL_COMPLETED if order e.g. get stuck.
Which state will an API request get if prior operation on same subscr fails?

I added some more info about what happens when order fails.
If an API request fails and there are more API requests to the same subscription, the subsequent API requests will remain in state APPROVED


First Capstone Project Team Review

This is the team review of this project.

High Lights 😁:
✅ Good commit messages.
✅ Descriptive READme file in a professional way.
✅ No linters errors.
✅ Good usage in correcting GitHub flow

Optional Suggestions:

I think you should review the original design again and try to make your project look more similar changing colors, fonts, margins and positions.

I recomment to use the monserrat font for the header. I have seen some projects and they use this font.

Good job so far, but you still have to do some work.

Please adjust the media to each resolution, to make it readable for every version.
Stick more to the original design, fonts, colors and margins are very off.


Compatible camera and Mic

Here is the error I am getting upon running the line
npm install --only=prod
npm WARN deprecated<EMAIL_ADDRESS>The major version is no longer supported. Please update to 4.x or newer
npm notice created a lockfile as package-lock.json. You should commit this file.
+<EMAIL_ADDRESS>added 114 packages in 114.632s

><EMAIL_ADDRESS>install /home/pi/MagicMirror/modules/MMM-awesome-alexa/node_modules/snowboy
> node-pre-gyp install --fallback-to-build

node-pre-gyp ERR! Tried to download(403): https://snowboy-release-node.s3-us-west-2.amazonaws.com/snowboy/v1.2.0/Release/snowboy-v1.2.0-node-v59-linux-arm.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v59 ABI, glibc) (falling back to source compile with node-gyp)

I apologize if this such an easy problem to solve, I am relatively new to this and I would be deeply greatful for your help!
UPDATE: This got solved already. Nevermind.
I am having trouble with a camera I just bought because the camera seems to be working fine but the microphone that comes with it seem to be not working with my RPi3 B and thus, my project smart mirror. Any recommendations on what camera and mic I should use? thanks

Mic like this is what I use, seems to be good:
https://www.amazon.co.uk/Hosaire-Mini-Microphone-Laptop-Desktop/dp/B01IN197B0/ref=sr_1_19?ie=UTF8&qid=1536082587&sr=8-19&keywords=microphone+raspberry+pi
Camera, no idea. My RP3 barely manages to run MagicMirror, let alone camera software on top.

Completely unrelated but I cant seem to find the log when i run npm start dev, ive read somewhere that it might be because of using python3/3.5 which I currently have?

Solved. Closing

Completely unrelated but I cant seem to find the log when i run npm start dev/ pm2 log... , ive read somewhere that it might be because of using python3/3.5 which I currently have?

No idea bro


Update README to reflect referral fee

If we merge https://github.com/solidusio-contrib/solidus_paypal_commerce_platform/pull/92 the CI should go back to green 🚥


autobuyer stopped Time elapsed

not sure what is happening, i have updated to the latest version and it wont let the buyer run

not sure what is happening, i have updated to the latest version and it wont let the buyer run

Same here

Reinstall the latest version, should work now

Not working with fresh instalation
El vie., 5 de noviembre de 2021 3:41 p. m., Ck Algos <
@.***> escribió:

Reinstall the latest version, should work now
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chithakumar13/FUT-Auto-Buyer/issues/507#issuecomment-962238745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWLTU5M3I5AZJ2U2ZNWB6ITUKRFR5ANCNFSM5HOWCPTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


chore(release): 26.4.0

Release 26.4.0

Pull Request Test Coverage Report for Build 5947

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.214%

Totals

Change from base Build 5946:
0.0%

Covered Lines:
1137

Relevant Lines:
1171

💛  - Coveralls