Replica information missing in GoneException

Describe the bug
Below shows null in resourceAddress and requesUri .  It should not be null
2020-08-17 11:14:51,538       [cosmos-rntbd-nio-2-4] INFO  com.azure.cosmos.implementation.directconnectivity.GoneAndRetryWithRetryPolicy - Received gone exception, will retry, GoneException{userAgent=azsdk-java-cosmos/4.3.2-beta.2 Linux/5.3.0-1034-azure JRE/11.0.8, error={"Errors":["The requested resource is no longer available at the server."]}, resourceAddress='null', requestUri='null', statusCode=410, message=["The requested resource is no longer available at the server."], causeInfo=null, responseHeaders={x-ms-transport-request-id=0, x-ms-last-state-change-utc=Mon, 17 Aug 2020 11:13:09.677 GMT, x-ms-serviceversion= version=<IP_ADDRESS>, x-ms-activity-id=deaef0ae-e07a-11ea-98f9-b54cde315be6, x-ms-schemaversion=1.9}, requestHeaders=null}

@kushagraThapar  fyi

Hitting this exact issue in a production system today. Any leads towards solving this issue yet?

@Olatunji-Longe - This issue has been solved, and will be released once we ship out next release: https://github.com/Azure/azure-sdk-for-java/pull/17279


"A simple connected animation" sample does not explains the process of creating it.

Is this a totally new sample, an addition to an existing sample page, or a modification of an existing sample page?

[ ] New sample
[x] Addition to existing sample
[ ] Modification to existing sample

If this is an addition/modification to an existing sample page, which one?
Addition to Motion -> Connected Animation page
Describe the sample

This page has section A simple connected animation which shows us different connected animation transitions configurations, but the following code under that sample does not explains how to it at all. Please, add some code samples to explain how exactly animation configuration is set.
Is your sample request related to a problem? Please describe it

Docs says that there is an ability to change transition from Gravity (default) to Direct or Basic, but not explaining how to do that. Same goes for this application: it's demonstrated, but never explained.
Additional context

This is great feedback, thank you for filing this issue @LilyStilson! I'll update the code sample to be more relevant.
In the meantime, the code for this sample can be found on GitHub: https://github.com/microsoft/Xaml-Controls-Gallery/blob/265c5979c6e1ff65a579a25ea151456db9ed6ab7/XamlControlsGallery/SamplePages/SamplePage1.xaml.cs#L21

We also have XAML based helpers for this in the Toolkit, did we want to cross-promote?


LUIS does not work when I integrate bot with webchat.js in any browser

Hello, I created a chatbot in C#  with my virtual assistant template and the web chat works fine with the iframe

but LUIS doesn't work when I integrate it with js in any browser

What can I do for this to work? I already tried to make it work by adding the secrets of webchat and directline.
This is the location of the error it send me:
public class DefaultAdapter : BotFrameworkHttpAdapter`
   {
        public DefaultAdapter(
            BotSettings settings,
            ICredentialProvider credentialProvider,
            IBotTelemetryClient telemetryClient,
            BotStateSet botStateSet)
            : base(credentialProvider)
        {
            OnTurnError = async (turnContext, exception) =>
            {
                await turnContext.SendActivityAsync(new Activity(type: ActivityTypes.Trace, text: $"{exception.Message}"));
                await turnContext.SendActivityAsync(new Activity(type: ActivityTypes.Trace, text: $"{exception.StackTrace}"));
                await turnContext.SendActivityAsync(MainStrings.ERROR);
                telemetryClient.TrackException(exception);
            };`

            Use(new TranscriptLoggerMiddleware(new AzureBlobTranscriptStore(settings.BlobStorage.ConnectionString, settings.BlobStorage.Container)));
            Use(new TelemetryLoggerMiddleware(telemetryClient, logPersonalInformation: true));
            Use(new ShowTypingMiddleware());
            Use(new SetLocaleMiddleware(settings.DefaultLocale ?? "en-us"));
            Use(new EventDebuggerMiddleware());
            Use(new AutoSaveStateMiddleware(botStateSet));
        }
    }

which is in the startup.cs file:
services.AddSingleton<IBotFrameworkHttpAdapter, DefaultAdapter>();

the bot responds to me well when it enters the user data request flow (which does not need LUIS nor QnA) But it sends me the error when I send a message that needs to use LUIS and QnA
(Edited by @compulim for code formatting)

@guadalupejaime Does your bot work in the Test in Web Chat blade in the Azure portal?

@tdurnford yes, the bot work in Test in Web Chat

What happened when it is not working? Do you see something strange/red in the developer tool console?
FYI, "Test in Web Chat" blade should be using Web Chat 4.3.0.

This also looks like a duplicate of this Stack Overflow question.  @guadalupejaime  can you try @stevkan's recommendations and let us know if his answer resolves your issue?

Stevkan's suggestion didn't work for me.
This is the error it send me:
public class DefaultAdapter : BotFrameworkHttpAdapter
 {         public DefaultAdapter(             BotSettings settings,             ICredentialProvider credentialProvider,             IBotTelemetryClient telemetryClient,             BotStateSet botStateSet)             : base(credentialProvider)         {             OnTurnError = async (turnContext, exception) =>             {                 await turnContext.SendActivityAsync(new Activity(type: ActivityTypes.Trace, text: $"{exception.Message}"));                 await turnContext.SendActivityAsync(new Activity(type: ActivityTypes.Trace, text: $"{exception.StackTrace}"));                 await turnContext.SendActivityAsync(MainStrings.ERROR);                 telemetryClient.TrackException(exception);             };
        Use(new TranscriptLoggerMiddleware(new AzureBlobTranscriptStore(settings.BlobStorage.ConnectionString, settings.BlobStorage.Container)));
        Use(new TelemetryLoggerMiddleware(telemetryClient, logPersonalInformation: true));
        Use(new ShowTypingMiddleware());
        Use(new SetLocaleMiddleware(settings.DefaultLocale ?? "en-us"));
        Use(new EventDebuggerMiddleware());
        Use(new AutoSaveStateMiddleware(botStateSet));
    }
}`

the bot responds to me well when it enters the user data request flow (which does not need LUIS nor QnA) But it sends me the error when I send a message that needs to use LUIS and QnA

@guadalupejaime
Could you send us the error message? Do you get the error in C#, or in a browser?

@compulim in all browsers

locally and in the test in webchat it work fine

and I created another chatbot with the virtual assistant template from scratch and it doesn't work with webchat.js

Can you send us the error to look at?
Also, I am quite confused with your last 3 messages.
You said you received the error "in all browsers". And Web Chat "works locally and in the test in webchat". And you created another VA bot and "it doesn't work with Web Chat". Is that correct?

it not work when I integrate bot with webchat.js, only in js

I don't know how to debug it from the browser but the error code that it sends is the one I put above

I didn't see any error or error code above.

the bot works in https://webchat.botframework.com/embed/NAME_OF_BOT?s=YOUR_SECRET_HERE but not with

every time I try to send a message the bot enters the following code:
public class DefaultAdapter : BotFrameworkHttpAdapter
   {
        public DefaultAdapter(
            BotSettings settings,
            ICredentialProvider credentialProvider,
            IBotTelemetryClient telemetryClient,
            BotStateSet botStateSet)
            : base(credentialProvider)
        {
            OnTurnError = async (turnContext, exception) =>
            {
                await turnContext.SendActivityAsync(new Activity(type: ActivityTypes.Trace, text: $"{exception.Message}"));
                await turnContext.SendActivityAsync(new Activity(type: ActivityTypes.Trace, text: $"{exception.StackTrace}"));
                await turnContext.SendActivityAsync(MainStrings.ERROR);
                telemetryClient.TrackException(exception);
            };

            Use(new TranscriptLoggerMiddleware(new AzureBlobTranscriptStore(settings.BlobStorage.ConnectionString, settings.BlobStorage.Container)));
            Use(new TelemetryLoggerMiddleware(telemetryClient, logPersonalInformation: true));
            Use(new ShowTypingMiddleware());
            Use(new SetLocaleMiddleware(settings.DefaultLocale ?? "en-us"));
            Use(new EventDebuggerMiddleware());
            Use(new AutoSaveStateMiddleware(botStateSet));
        }
    }

which is in the startup.cs file:
 services.AddSingleton<IBotFrameworkHttpAdapter, DefaultAdapter>();

What is the error?

await turnContext.SendActivityAsync (MainStrings.ERROR); I know that the bot is entering this line because I changed the "ERROR" message to another one that I edited

Now I understand.
Could you send us the content of the exception object you got in OnTurnError? I.e. the exception object on this line.
OnTurnError = async (turnContext, exception) =>

Appication Insights send me this Error: The given key 'es' was not present in the dictionary.

Do you have the full exception object? The stack trace inside the exception object would be helpful to find out which line asking for the key "es".

Could it be the language? when I use the bot at webchat.botframework.com/embed / ... I receive the message "" write your message ... and when I integrate it with js it appears in Spanish "Escribe tu mensaje..."
How can i change it to english?

Could it be "Es" of "Español"?

I changed the code to this:

```
to change the language and the bot already work well in webchat.js

Thank you very much everyone, how kind

If you found out the offending code (finding key "es") is actually on Web Chat, Bot Builder SDK or VA. Please feel free to reopen this bug and point us to it. Happy to fix. 😉


Add Microsoft Teams

This basically adds the manifest provided by @FriendlyNeighborhoodShane in #3595 + Autoupdate.
I tested the install and it works on my machine ™

Closes #3595

Neat. Also I would like to point to you a suggestion I have about the post_install here https://github.com/ScoopInstaller/Extras/issues/7157#issuecomment-957349926, ICYMI

Yeah, it's the first time I commit something to scoop. Can you point me to a documentation section documenting how to use a script within the repo instead of the inlined scripts in the manifest? Or even just an example with another manifest.

I think the post_install script is short enough, it's fine here.

/verify

Rename microsoft-teams.json -> teams.json

@rashil2000  i think the original name of the product should be retained.(micrsoft-teams)
see:
https://www.microsoft.com/en-us/microsoft-teams/download-app

I recommended that because of slack.json manifest


[CHALLENGE SUBMISSION] Day 6

Please provide the following information for your submission:
What is your name? (First, Last)
Rahul Ruikar
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/rahulruikar/25DaysOfServerless/tree/master/Day6
What Challenge is this for? (Challenge number between 1 and 25)
6
(Optional) Anything else we should know? e.g., language used, location, blog post?

C#/.net
Azure Durable Functions
https://github.com/CXuesong/nChronic (Natural language processing for Time and Date)

THANK YOU FOR YOUR #25DaysOfServerless SUBMISSION
We checked out your repository. We're thrilled to say your solution is now listed in our Hall of Fame
Thank you for your contributions!
This issue is now resolved! 🎉


Memory caching for CRD navigation entries

During performance testing, I noticed that we are rebuilding a list of CRD navigation entries twice per second, for cluster scope and current namespace. That is very costly operation taking ~500ms for larger CRD sets. To address the issue, I added a memory cache holding CRD navigation entries. Cache is invalidated every time new CRD appears or goes away.
Performance improvement is considerable but it depends on number of CRDs in cluster/namespace - I am seeing 15-20% improvement on larger sets on my mac.
Signed-off-by: Milan Klanjsek<EMAIL_ADDRESS>

Good start, this will need to be tied in to the Add/Update/Delete events of crd_watcher.go so that it removes old CRDS and adds new CRDs as they are applied. This will only cache the CRDs that are installed at the time you startup Octant.

@wwitzel3 I already handle Add/Delete in crd_watcher, invalidating the cache there so it's recreated during next call.

@wwitzel3 I already handle Add/Delete in crd_watcher, invalidating the cache there so it's recreated during next call.

I see it now, probably was just GitHub not showing me the whole diff.

Current:

This change alone:

This change combined with the #2102 branch

Good catch Wayne, missed the race condition.  I eliminated the redundant sort on cached object and as a result the race condition is gone now.
Discouraged by negligible performance gain in your tests, makes me wonder if it's worth it...

Negligible? Avg. 50% improvement once cached, that's a big improvement.

Ooops my bad, didn't read that well 🤦  Definitely not negligible!

I want to think about this a bit more, the idea is solid and effective. I'm not sure I like the current implementation of just slapping a global in to navigation and calling it from the watcher callbacks and accessing it in CRDEntires and CustomResourceDefinitions.
Also there will need to be at least a simple assertion test: non-cache call, cache call, clear cache, non-cache call, cache call.

@mklanjsek and I talked about this, we will implement a cache layer and then overview and custom overview will maintain their own cache of the CRDEntries call. This is because the modules actually care about different results and the CRDEntires function is bi-modal depending of if you want namespace scoped CRDs or not.

Waiting until #2144 is merged before finishing this.

@wwitzel3 PTAL

Please rebase, there is a failing test when merged with master because we no longer use RegisterOnUpdate methods.

Rebased and fixed test failure.


AttributeError: 'Exp' object has no attribute 'pre_no_aug'

2024-11-05 09:04:41 | INFO     | yolox.core.vid_trainer:240 - ---> start train epoch1
2024-11-05 09:04:41 | INFO     | yolox.core.vid_trainer:235 - Training of experiment is done and the best AP is 0.00
2024-11-05 09:04:41 | ERROR    | yolox.core.launch:98 - An error has been caught in function 'launch', process 'MainProcess' (12747), thread 'MainThread'<PHONE_NUMBER>86752):
Traceback (most recent call last):
File "tools/vid_train.py", line 131, in
launch(
└ <function launch at 0x7ff2bfac99d0>

File "/22085400415/leilanfang/smoke_fusion_theraml/YOLOV-master/yolox/core/launch.py", line 98, in launch
main_func(*args)
│          └ (╒════════════════════╤══════════════════════════════════════════════════════════════════════════════════════════════════════...
└ <function main at 0x7ff2b69a0ca0>

File "tools/vid_train.py", line 115, in main
trainer.train()
│       └ <function Trainer.train at 0x7ff2b69a0430>
└ <yolox.core.vid_trainer.Trainer object at 0x7ff2b6998a30>
File "/22085400415/leilanfang/smoke_fusion_theraml/YOLOV-master/yolox/core/vid_trainer.py", line 122, in train
self.train_in_epoch()
│    └ <function Trainer.train_in_epoch at 0x7ff2b69a04c0>
└ <yolox.core.vid_trainer.Trainer object at 0x7ff2b6998a30>
File "/22085400415/leilanfang/smoke_fusion_theraml/YOLOV-master/yolox/core/vid_trainer.py", line 130, in train_in_epoch
self.before_epoch()
│    └ <function Trainer.before_epoch at 0x7ff2b69a0790>
└ <yolox.core.vid_trainer.Trainer object at 0x7ff2b6998a30>
File "/22085400415/leilanfang/smoke_fusion_theraml/YOLOV-master/yolox/core/vid_trainer.py", line 262, in before_epoch
elif 0< self.epoch + 1 <= self.exp.pre_no_aug + self.exp.warmup_epochs:
│    │            │    │                │    │   └ 0
│    │            │    │                │    └ ╒════════════════════╤═══════════════════════════════════════════════════════════════════════════════════════════════════════...
│    │            │    │                └ <yolox.core.vid_trainer.Trainer object at 0x7ff2b6998a30>
│    │            │    └ ╒════════════════════╤═══════════════════════════════════════════════════════════════════════════════════════════════════════...
│    │            └ <yolox.core.vid_trainer.Trainer object at 0x7ff2b6998a30>
│    └ 0
└ <yolox.core.vid_trainer.Trainer object at 0x7ff2b6998a30>
AttributeError: 'Exp' object has no attribute 'pre_no_aug'
Exception in thread Thread-1:
Traceback (most recent call last):
File "/opt/conda/lib/python3.8/threading.py", line 932, in _bootstrap_inner
self.run()
File "/opt/conda/lib/python3.8/threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "/opt/conda/lib/python3.8/site-packages/torch/utils/data/_utils/pin_memory.py", line 28, in _pin_memory_loop

i had the same problem when i was training yolov_s.
i change the file "exps/yolov/yolov_s.py" and add code line  self.pre_no_aug = 0 and it works.
hope my method works for you too

I have solved this problem. Can you add my contact information?

i had the same problem when i was training yolov_s. i change the file "exps/yolov/yolov_s.py" and add code line self.pre_no_aug = 0 and it works. hope my method works for you too

I have solved this problem. Can you add my contact information?

ok  XD


Web worker import fails for iOS 14 on dev server

Describe the bug
Cannot import web worker file on iOS 14 (tried with iPhone6S).
Import fails with message
SyntaxError: Unexpected string literal<EMAIL_ADDRESS>import call expects exactly one argument
Reproduction
Reproduction repo
System Info
System:
    OS: Linux 5.4 Ubuntu 18.04.6 LTS (Bionic Beaver)
    CPU: (8) x64 Intel(R) Core(TM) i7-10510U CPU @ 1.80GHz
    Memory: 208.87 MB / 15.34 GB
    Container: Yes
    Shell: 4.4.20 - /bin/bash
  Binaries:
    Node: 16.6.1 - /usr/local/bin/node
    Yarn: 1.22.5 - /usr/bin/yarn
    npm: 8.1.3 - /usr/local/bin/npm
  Browsers:
    Chrome: 95.0.4638.69
    Firefox: 94.0.2

Used Package Manager
npm
Logs
SyntaxError: Unexpected string literal<EMAIL_ADDRESS>
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Can confirm this issue on iOS 13.6.1. The behavior is the same as described above. I'm looking for a workaround if there is one. We really need that worker support.
Btw, everything seem to work fine on Safari 15 (tested on MacOS Monterey and iOS 15.2 simulator).


TwoHandsManipulation acts weird when One Hand Movement disabled.

Does this affect the legacy HoloToolkit (master) or the Mixed Reality Toolkit (mrtk_release)?
HoloToolkit
Describe the bug
Per my discovery I shared on Slack. If it hasn't been brought yet (that would surprise me as it's easy to reproduce) I think  there's a bug in TwoHand manipulation script. It happens when one disables One Hand Movement for the script, leaving the object being only manipulatable by two hands. Following is the video I've just recorded:
https://www.dropbox.com/s/oi1q7jy3u7f4idr/two_hands_manipulation_bug.mp4?dl=0
To Reproduce
Steps to reproduce the behavior:

Run the example from MRTH (Two Hands Manipulation scene)
Disable "One Hand Movement" for every game object
Play with them for a while..
See error

Expected behavior
Expected is two hands manipulation working properly over time.
Actual behavior
Two hands manipulation is broken after a while.
Additional context
In the referenced video, problem starts occurring at 1:07.

I've encountered the same bug and I really wanted this to work, so I fixed it myself. When I troubleshooted the bug, I noticed the issue wasn't with playing around with the game objects. It happened when you tapped and held your hand (both or one) and then moved your gaze away or into another game object.
This should trigger OnSourceLost() method, but it never did as the GameObject wasn't added to the InputManager when One Hand Movement was disabled. One Hand Movement relied completely on OnInputUp() to work in the older version.
Which in turn caused one extra hand location to stay in memory, which in turn allowed you to do Two Hand Manipulations with only one hand (as the application thought there was two of them present). And if you did this with two hands, then the application thought you had three hands active. Because the GameObject was added to the InputManager when two hands were used, the count never went higher than three.
So I implemented a way to add the GameObject to InputManager when One Hand Movement is disabled. That was the simplest way fix this (at least to me).
I have tested the code mainly with Scale manipulation enabled and One Hand Movement disabled, as that is the scenario I wanted it to work in. Thus, I am not sure if I have broken something when it comes to other modes.
I uploaded my version as a public gist TwoHandManipulatable, so it can just be copied and pasted over the old code.
What I added/edited

Two additional methods at the end of file: CheckIfInputHandlerIsRequired and CheckIfInputHanderShouldBeRemoved.
Modified methods OnManipulationStarted and OnManipulationEnded to not add anything to InputManager when One Hand Movement is disabled.
Modified InvokeStateUpdateFunctions() =>  CheckIfInputHandlerIsRequired() is called when ManipulationMode is None (Line 327 in my version).
Modified methods OnInputUp and OnSourceLost to call CheckIfInputHanderShouldBeRemoved() at the end.
I left the Debug.Logs that I used to debug my code, however they are commented out so you don't have to do it yourself.
There is a private bool variable addedToInputManager to determine if the actions inside CheckIfInputHandlerIsRequired() or CheckIfInputHanderShouldBeRemoved() should be called.

Closing issues older than 180 days. If this is still an issue, please reactivate with recent information.


fix(types): preserve input stream type

Fix type definition to communicate that the stream type doesn't change.

@stephenplusplus would you mind merging and releasing this when you get a chance? thxlotsbye.

Gentle ping 🛎

Done as 1.0.5. Sorry for the lag!


Add "Share" wording and remove tool tip text on Save/Export button

Remove tooltip content and add Share Label on Export button
Resolves #122

Hi @chrisirhc , I renamed the PR to "Add "Share" wording and remove tool tip text on Save/Export button" , I get this errors, I will solve tonight and I think tomorrow, everything is ok. Thanks for your time, sorry for this problems.

@chrisirhc Sorry for disturb you again, but the problem was simpler than I thinked, and I solved on the last commit. Jenkins build passed, so I think is done to be reviewed again.
If something more was need, please, let me know

New version: Bold

New version: Non Bold

It looks like the bold version doesn't match completely the requirement from this one:

I wonder if we need a different font weight. @heshan0131 what do you think?

Os there something more to be change on this PR?

Thank you for making the changes. One last comment and we are good to go. It seems your prettier settings added space between the bracket. Can you remove those?

Space removed, sorry. I forgot to change my editor configs. If something more is wrong, please, let me know.

Please select Squash and merge, change the title of this PR to [Enhancement] Add "Share" wording and remove tooltip text on Save/Export button

@heshan0131 where Can I select Squash and merge ?

nvm, you might not have the option. I will do it :)

@heshan0131 Thanks a lot! It's a  great honor contribute to this project! Thanks for all learn! And if you think I can solve more issues, please, let me know


Use common arm definitions for dns

use common arm definitions
extract golang config

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java
A PR has been created for you:
https://github.com/Azure/azure-sdk-for-java/pull/2267

Automation for azure-sdk-for-node


Directory finallinks not found.

Hi,
I tried scraping urls with archive_scrapper, it worked smoothly.
But when trying to scrapping the content out of those url i tried running scrape with url. it is looking in the directory "links/finallinks" but there is no folder as such as finallinks created by archive_scrapper.

@bhangaboy Hi, you need to run merger.py before running scrape_with_bs4.py. It will create separate final data file for each of the company. This data is merged from the google search results as well as all the web archives together. This is required to get rid of duplicate entries from different sources. scrape_with_bs4.py scrapes the data sequentially. To scrape it parallely, use quick_scraper.py .

Hi gyanesh,
Thanks for replying. I tried running merger.py file for every company a new file was created for each company even after that i was getting that same directory error. It would be great help if you provide steps in good detail.


cx_wrappers.c: Fix missing const attribute on cx_ccitt32

Description
cx_wrappers.c: Fix missing const attribute on cx_ccitt32
Changes include

[x] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Tests
[ ] Documentation
[ ] Other (for changes that might not fit in any category)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (7bb31ca) 60.42% compared to head (5567b62) 60.42%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #518   +/-   ##
=======================================
  Coverage   60.42%   60.42%
=======================================
  Files          12       12
  Lines        1660     1660
=======================================
  Hits         1003     1003
  Misses        657      657

Flag
Coverage Δ

unittests
60.42% <ø> (ø)

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.


Make ockam tcp-inlet delete (no args) interactive by asking the user to choose from a list of tcp-inlet aliases to delete (tuify)

Current behavior
When a user runs ockam tcp-inlet delete without any arguments, currently they are shown an error and some help text. Here's a screenshot of this.

Desired behavior
Instead of showing help text, change the behavior of this command so that it becomes interactive with the user. Note that this can only be allowed to happen if:

the terminal is interactive, and,
the --no-input or --quiet flags have not been passed, and,
no arguments have been passed in to the command.

This what the user flow should look like in interactive mode.

Use the tuify crate's select_from_list() function to ask the user to select from a list of tcp-inlet aliases, at the default node. They should be able to make multiple selections. If there are none, then simply display a message saying that none exist (not an error) and exit.

Once the user has made their selection(s), display a confirmation prompt letting the user know that that a destructive operation is about to take place with the tcp-inlet aliases they have selected. Use select_from_list() to display this prompt. They should be able to make a single selection.

If they have made no selections then exit by displaying a message to the user letting them know they did not make any selection.

If they choose "YES", then delete each selected tcp-inlet, and display a success / failure message for each tcp-inlet that got deleted.

If an operation succeeds then use the ✅ emoji as a prefix for the message.
And if it fails, then use the ⚠ emoji as a prefix for the message.

The following is a video showing a generic user flow that shows the user experience when a user is asked to choose a from a list of "things". Then an action is performed on each "thing". Finally status information is reported for each action that is performed on each "thing". Please note that this flow is not specific to this issue.

https://asciinema.org/a/612547

Implementation details

Please use the functions select_multiple, and confirm_interactively in this file.

Here's a similar PR to get some ideas from: https://github.com/build-trust/ockam/pull/6480

You can learn more about how to use the select_from_list() in the tuify crate here. Here is an example that is very similar to the steps you might have to take for this issue.

Here's a file you can look at to detect when a terminal is interactive or not.

Here's a file you can look at to see where the ockam tcp-inlet delete command is implemented.

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

Hi, I think that I can work on this, please assign this issue to me if there's no one working on

@0x61nas awesome, all yours.
Please let us know if your have any questions as you explore. You can also ask questions on the contributors discord https://discord.gg/RAbjRr3kds

@0x61nas Since you started working on this issue, here's some new information that just became available to help you w/ your PR:

Please use the functions select_multiple, and confirm_interactively in this file.

Here's a similar PR to get some ideas from: https://github.com/build-trust/ockam/pull/6480

I've updated the PR, thanks for notifying me.


Build unsuccessful when I run build_desktop_examples.sh

Unable to build mediapipe in linux (ubuntu)
I get error while running build_desktop_examples.sh file
/build_desktop_examples.sh
app_dir: mediapipe/examples/desktop
out_dir: .
=== Target: mediapipe/examples/desktop/face_detection:face_detection_cpu
Starting local Bazel server and connecting to it...
... still trying to connect to local Bazel server after 10 seconds ...
DEBUG: /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/external/org_tensorflow/third_party/repo.bzl:109:14:
Warning: skipping import of repository 'com_google_absl' because it already exists.
DEBUG: /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/external/org_tensorflow/third_party/repo.bzl:109:14:
Warning: skipping import of repository 'com_google_protobuf' because it already exists.
DEBUG: /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/external/org_tensorflow/third_party/repo.bzl:109:14:
Warning: skipping import of repository 'com_google_googletest' because it already exists.
DEBUG: /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/external/org_tensorflow/third_party/repo.bzl:109:14:
Warning: skipping import of repository 'com_github_gflags_gflags' because it already exists.
DEBUG: /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/external/org_tensorflow/third_party/repo.bzl:109:14:
Warning: skipping import of repository 'build_bazel_rules_apple' because it already exists.
DEBUG: /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/external/org_tensorflow/third_party/repo.bzl:109:14:
Warning: skipping import of repository 'build_bazel_rules_swift' because it already exists.
DEBUG: /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/external/org_tensorflow/third_party/repo.bzl:109:14:
Warning: skipping import of repository 'build_bazel_apple_support' because it already exists.
DEBUG: /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/external/org_tensorflow/third_party/repo.bzl:109:14:
Warning: skipping import of repository 'bazel_skylib' because it already exists.
DEBUG: /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/external/org_tensorflow/third_party/repo.bzl:109:14:
Warning: skipping import of repository 'pybind11' because it already exists.
INFO: Analyzed target //mediapipe/examples/desktop/face_detection:face_detection_cpu (118 packages loaded, 6504 targets configured).
INFO: Found 1 target...
INFO: Deleting stale sandbox base /home/thushar/.cache/bazel/_bazel_thushar/fea302e8c8b93ec8c2ae770cdccdcb8b/sandbox
ERROR: /home/thushar/mediapipe/mediapipe/framework/formats/BUILD:190:11: C++ compilation of rule '//mediapipe/framework/formats:location' failed (Exit 1): gcc failed: error executing command /usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections ... (remaining 56 argument(s) skipped)
Use --sandbox_debug to see verbose messages from the sandbox gcc failed: error executing command /usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections ... (remaining 56 argument(s) skipped)
Use --sandbox_debug to see verbose messages from the sandbox
In file included from ./mediapipe/framework/formats/location.h:44,
from mediapipe/framework/formats/location.cc:15:
./mediapipe/framework/port/opencv_core_inc.h:18:10: fatal error: opencv2/core/version.hpp: No such file or directory
18 | #include <opencv2/core/version.hpp>
|          ^~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
Target //mediapipe/examples/desktop/face_detection:face_detection_cpu failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 1879.314s, Critical Path: 1734.01s
INFO: 262 processes: 126 internal, 136 linux-sandbox.
FAILED: Build did NOT complete successfully

Hi @Thushar-marvel, Could you please confirm this is duplicate of https://github.com/google/mediapipe/issues/496. If not can you share the system information details to investigate further. Thanks!

@sgowroji
I tried to the issue #496 .Even after altering ./mediapipe/third_party/opencv_linux.BUILD it didn't work for me.

Device name	LAPTOP-4P7IBL6V
Processor	Intel(R) Core(TM) i5-9300H CPU @ 2.40GHz   2.40 GHz
Installed RAM	8.00 GB (7.85 GB usable)
Device ID	E593A1D8-AE75-4009-BF6F-9E437572701F
Product ID	00327-35852-62270-AAOEM
System type	64-bit operating system, x64-based processor
Pen and touch	No pen or touch input is available for this display

I have used Oracle VM virtual box manager to build virtual machine with ubuntu 20.04

Well I had met some errors like you...
I think the key to solve is you should make sure your opencv should be installed successfully AND your environment variables should be set properly.
And the build_desktop_examples.sh is too much complicated and I suggest you can build single example(face_detection ,eg) to begin, if ok, then go to the other examples.


making it so it requires all the webpack config

Similar to #46
Basically allows for any webpack config but will always override with the one needed by this plugin.

This seems pretty cool to me.
I'll take this in.

Released with v1.5.0.


Latex Entities Missing Night Vision Toggle

What version are you playing on?

What version are you playing on? The toggle ability isn't added in 0.14

0.14b... I think this answers my question.


kpcli: 4.1 -> 4.1.2 & KDBX 4 support

Fixes #330177
cc @Schweber
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

LGTM, pending eval

@ofborg build perlPackages.IteratorSimple perlPackages.FileKDBX

z88dk has its own IteratorSimple, it's a bit unexpected that this triggers a rebuild
2 packages updated:
kpcli (4.1 → 4.1.2) z88dk

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 348984

x86_64-linux

:white_check_mark: 10 packages built:

kpcli
perl538Packages.FileKDBX
perl538Packages.FileKDBX.devdoc
perl538Packages.IteratorSimple
perl538Packages.IteratorSimple.devdoc
perl540Packages.FileKDBX
perl540Packages.FileKDBX.devdoc
perl540Packages.IteratorSimple
perl540Packages.IteratorSimple.devdoc
z88dk

z88dk has its own IteratorSimple, it's a bit unexpected that this triggers a rebuild

I moved that IteratorSimple to the perl package set. It probably rebuilds because the order of packages used in z88dk changed.

Rebased (due to #348678)


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1356

Questions ?
Please refer to the Backport tool documentation


[Azure Bot][AdaptiveCards in Other Channels] [Adaptive Card not displayed in Directline Channel]

Hello , i am using  "Adaptive Cards Designer" to create new adaptive card for different bot channels , when i use webchat channel payload the adaptive card rendered successfully , but when i tried to render the same card on Directline channel using "Bot Framework Other Channels (Image render)" payload  the card was not displayed at all . i tried to use .Net render image using wpf but i t did not work with me too .
I am using Bot Framework v4 and .net core 2.1
May i know reasons or  how to solve this issue , please ?
Many Thanks

@HebatallaTarek can you pls share your card payload here? @dandriscoll since i am still having issues transferring issues to the botframework repositories, could you pls help route this over to the right folks? Thanks again

Closing due to age and lack of response


Median of two sorted arrays of different size

Hi, I am GSSoC participant,I would like to add this problem with Time and Space Complexity.

Hello,
I would like to contribute to this issue in java as a GSSoC 2021 participant
kindly assign me the issue

Hi,
I am GSSOC'21 participant
My code will be in JAVA with proper comments and test cases.
@rudrakshi99 @nakul-19 @plazzy99 please assign me this issue

present at #631


additional metadata on service parameters

We have a use case where we want to indicate that for a particular parameter, if the user does not provide a value, a value will be generated for them (e.g. a random password).
I don't necessarily expect something like this to be added to the spec, but i am curious if other broker authors see a use for this and whether we can agree on a convention for this bit of metadata.
/cc @jwforres @jim-minter

@bparees afew questions:

where do you see the random password being generated?
why do you think it is important for the user to know that a random password will be generated?

@avade we have a broker which, among other services it offers, can provision databases for users.  The password is generated(if necessary), effectively, by the broker (really it happens a bit downstream from the broker).
As part of that, we have a password parameter that will be used as the password for the created database.
We want to allow users to supply their own password if they want, but we also want to generate one for them if they don't care what the password is.  So it's useful for our service catalog UI to be able to present to the user "here's a password parameter, you can supply a value but if you don't, one will be generated for you."  It's important for the user to know because it allows them to understand the implications of leaving the password field blank (otherwise they might assume leaving it blank means their DB will be unprotected).

Hey Ben,
You could just add the password field to the schema, do not set required to
be true, and then use the "description" key to explain that?
Descriptions are a valid part of the JSON schema and UIs could display
those to explain things like this to users. What do you think?
On Tue, 6 Jun 2017 at 23:48, Ben Parees<EMAIL_ADDRESS>wrote:

@avade https://github.com/avade we have a broker which, among other
services it offers, can provision databases for users. The password is
generated(if necessary), effectively, by the broker (really it happens a
bit downstream from the broker).
As part of that, we have a password parameter that will be used as the
password for the created database.
We want to allow users to supply their own password if they want, but we
also want to generate one for them if they don't care what the password is.
So it's useful for our service catalog UI to be able to present to the user
"here's a password parameter, you can supply a value but if you don't, one
will be generated for you." It's important for the user to know because it
allows them to understand the implications of leaving the password field
blank (otherwise they might assume leaving it blank means their DB will be
unprotected).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openservicebrokerapi/servicebroker/issues/219#issuecomment-306639162,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG7UzPhTIInXpRdCBglprV7Fp4-gaG--ks5sBdcogaJpZM4NxpVx
.

You could just add the password field to the schema, do not set required to be true, and then use the "description" key to explain that?

There are nicer things the UI can do w/ the field(in terms of displaying+validating it) if it knows the parameter is generated, rather than relying on the parameter description to mention that the field is generated.  It also requires template authors(our serviceclasses are defined by openshift templates) to explicitly mention that, vs automatically getting the behavior when they write templates.  Here's an example of what is feeding the parameters in our serviceclasses:
https://github.com/openshift/library/blob/master/official/django/templates/django-psql-example.json#L459-L464
so everyone who writes one of these templates w/ a generated parameter would have to mention in the description that the parameter is/can be generated.  Or we'd have to tack our own message onto the author provided description which may or may not end up being redundant to what the template author already put in the description.
So yeah, it's an option, but realistically we're going to do something better in terms of what our broker sends back, so our UI can consume it (since the format of the broker parameter definition is just jsonschema so my understanding is we can extend it in whatever way we choose).  This is more a question of whether that "something better" is interesting enough to the community to try to agree on a common mechanism.

We have definitely heard of multiple use cases for adding additional keywords to the spec (for example, adding "sensitive": true for password fields).
JSON schema allows for these arbitrary keywords, but we would definitely like to add any common and consistent use cases to a future version of the specification. In CF, we will be using JSON schema core to generate UIs for now and using the description fields to detail any important contextual information.
It would be great if we could keep this thread open and if everyone in the community could add their use cases to this, and +1 any use cases that have already been listed that you have come across. I'd like to find out how many people are encountering these issues so we can prioritise accordingly.
Outstanding issues:

Sensitive fields: UIs will want to show that fields containing sensitive values (ie. passwords) may be set, although brokers will never return the values for these (see #159).
Auto-generated fields: UIs will want to show that some fields, if left blank, will be automatically generated by the platform.

Nobody added use cases to this, so closing due to inactivity. Please reopen if this is still a problem for anyone!


also create request context based on partition path

[x] Fix destruction of job factory

closing in favour of https://github.com/atom/brightray/pull/136


Other sites adding

please try to add UVa,timus
another thing will be better if hightail downloads the system test cases also of finished contests , not only sample test cases along with problem statement in html or pdf format. so that We can practice problems in offline also. I mean a tool like gettc for topcoder.

In which judge systems can you download systests? Not CodeForces and not CodeChef, for sure.


Refactor(YoloSimulation): Run Simulation as separate process

This pull request changes YoloSimulation behaviour by creating a new YoloSimulation subprocess.
This allows us to run the regular YoloDarknet code even in simulation, as opposed to replacing it with a mock YoloDarknet object as before.
PR Notes
This PR is based on https://github.com/vsaw/opendatacam which includes the following pending PRs

#284
#287
#289
#291
#293
#299
https://github.com/opendatacam/opendatacam/pull/301

It is therefore recommended to merge this after merging the PRs above.

Waiting for #301 to be merged. Then we can look at this one.

Now that all dependencies are merged in develop this is good for review :-)

I quickly had a look, looks good but when I try it by doing this:

pulling this PR
Setting "simulation" in config.json "VIDEO_INPUT"
npm run dev

I get this error:
Projects/opendatacam/server/processes/YoloDarknet.js:50
    var initialCommand = [this.config.darknetCmd, 'detector', 'demo', this.config.yoloParams.data, this.config.yoloParams.cfg, this.config.yoloParams.weights];
                                                                                ^

TypeError: Cannot read property 'data' of undefined

Didn't try to investigate, can you reproduce ?
Also we should had a note in the documentation about the simmulation mode .. maybe here: https://github.com/opendatacam/opendatacam/blob/master/documentation/DEVELOPMENT_NOTES.md ?

@tdurand Your error is expected as you did not select a neural network, just like with "real Darknet" :-)
Regarding the documentation. Sure, I'll try to write a few lines. If you want to beat me to it, the PR is now open for contributions from you :-)

ok, could we easily make this optional ? just needing to put VIDEO_INPUT: simulation ?  A good point to do this would be here: https://github.com/opendatacam/opendatacam/pull/302/files#diff-a4c65ede64197e1a112899a68bf994485b889c4b143198bac4af53425b38406fR57-R60
As for your error. I'm a bit baffled. Can you provide more detiails? And does npm test work for you?

npm test result, get the same but the test pass
.YoloDarknet: Empty configuration passed, most likely because you are in Simulation mode.
Process YOLO initialized
YoloSimulation {
  _events: [Object: null prototype] {},
  _eventsCount: 0,
  _maxListeners: undefined,
  isStarting: false,
  isStarted: false,
  isInitialized: true,
  process: null,
  videoResolution: { w: 0, h: 0 },
  config: {
    videoParams: {
      yolo_json: '/home/tdurand/Documents/Projects/opendatacam/public/static/placeholder/alexeydetections30FPS.json',
      video_file_or_folder: '/home/tdurand/Documents/Projects/opendatacam/public/static/placeholder/frames',
      isLive: false,
      jsonFps: 20,
      mjpgFps: 0.2
    },
    jsonStreamPort: 8070,
    mjpegStreamPort: 8090,
    simulationStartupDelayMs: 0,
    darknetStdout: false
  },
  isVideoDirectory: true,
  videoFileOrFolderExists: true,
  videoFileFps: null,
  detections: [
    { frame_id: 16, objects: [Array] },
    ... 337 more items
  ],
  simulationMJPEGServe: null,
  simulationJSONHTTPStreamServer: null,
  [Symbol(kCapture)]: false
}
Missing required argument: yolo_json

Options:
  --help                  Show help                                    [boolean]
  --version               Show version number                          [boolean]
  --yolo_json                                                         [required]
  --video_file_or_folder                                           [default: ""]
  --isLive                                             [boolean] [default: true]
  --jsonFps                                               [number] [default: 20]
  --mjpgFps                                               [number] [default: 20]
  --darknetStdout                                      [boolean] [default: true]
  --json_port                                           [number] [default: 8070]
  --mjpeg_port                                          [number] [default: 8090]
.Process YOLO initialized
.Process YOLO initialized
Process YOLO started
Video stream: 1280x720
.

8 specs, 0 failures
Finished in 0.301 seconds

For the bug, to try to reproduce, try checkout this PR and put this in config :
"OPENDATACAM_VERSION": "3.0.1",
  "PATH_TO_YOLO_DARKNET" : "TO_REPLACE_PATH_TO_DARKNET",
  "CMD_TO_YOLO_DARKNET" : "TO_REPLACE_PATH_TO_DARKNET/darknet",
  "VIDEO_UPLOAD_FOLDER": "TO_REPLACE_PATH_TO_DARKNET/opendatacam_videos_uploaded",
  "VIDEO_INPUT": "simulation",
  "NEURAL_NETWORK": "yolov4",
  "VIDEO_INPUTS_PARAMS": {
    "file": "opendatacam_videos/demo.mp4",
    "usbcam": "v4l2src device=/dev/video0 ! video/x-raw, framerate=30/1, width=640, height=360 ! videoconvert ! appsink",
    "raspberrycam": "nvarguscamerasrc ! video/x-raw(memory:NVMM),width=1280, height=720, framerate=30/1, format=NV12 ! nvvidconv ! video/x-raw, format=BGRx, width=640, height=360 ! videoconvert ! video/x-raw, format=BGR ! appsink",
    "remote_cam": "YOUR IP CAM STREAM (can be .m3u8, MJPEG ...), anything supported by opencv",
    "remote_hls_gstreamer": "souphttpsrc location=http://YOUR_HLSSTREAM_URL_HERE.m3u8 ! hlsdemux ! decodebin ! videoconvert ! videoscale ! appsink",
    "simulation": "--yolo_json public/static/placeholder/alexeydetections30FPS.json --video_file_or_folder public/static/placeholder/frames --isLive true --jsonFps 20 --mjpgFps 0.2"
  }

Then do
npm i
npm run build
npm run start

// or npm run dev

And then when you open the localhost:8080 you get infinite message
Missing required argument: yolo_json

Options:
  --help                  Show help                                    [boolean]
  --version               Show version number                          [boolean]
  --yolo_json                                                         [required]
  --video_file_or_folder                                           [default: ""]
  --isLive                                             [boolean] [default: true]
  --jsonFps                                               [number] [default: 20]
  --mjpgFps                                               [number] [default: 20]
  --darknetStdout                                      [boolean] [default: true]
  --json_port                                           [number] [default: 8070]
  --mjpeg_port                                          [number] [default: 8090]
Restart YOLO
Missing required argument: yolo_json

Thanks! The output of npm test is fine as I do not intercept stdout.
I really don't understand why this does not work. Can you try the following in your repository root:
node scripts/YoloSimulation.js --yolo_json public/static/placeholder/alexeydetections30FPS.json --video_file_or_folder public/static/placeholder/frames --isLive true --jsonFps 20 --mjpgFps 0.2

This should tell us if there's something wrong with the script or the way how YoloDarknet invokes it...
Thanks again for your debugging!

no problem.. sorry I didn't look into the code still ;-)
node scripts/YoloSimulation.js --yolo_json public/static/placeholder/alexeydetections30FPS.json --video_file_or_folder public/static/placeholder/frames --isLive true --jsonFps 20 --mjpgFps 0.2

YoloSimulation Start with Arguments
{
  videoParams: {
    yolo_json: 'public/static/placeholder/alexeydetections30FPS.json',
    video_file_or_folder: 'public/static/placeholder/frames',
    isLive: true,
    jsonFps: 20,
    mjpgFps: 0.2
  },
  jsonStreamPort: 8070,
  mjpegStreamPort: 8090,
  darknetStdout: true
}
YoloDarknet: Empty configuration passed, most likely because you are in Simulation mode.
Process YOLO initialized
YoloSimulation {
  _events: [Object: null prototype] {},
  _eventsCount: 0,
  _maxListeners: undefined,
  isStarting: false,
  isStarted: false,
  isInitialized: true,
  process: null,
  videoResolution: { w: 0, h: 0 },
  config: {
    videoParams: {
      yolo_json: '/home/tdurand/Documents/Projects/opendatacam/public/static/placeholder/alexeydetections30FPS.json',
      video_file_or_folder: '/home/tdurand/Documents/Projects/opendatacam/public/static/placeholder/frames',
      isLive: true,
      jsonFps: 20,
      mjpgFps: 0.2
    },
    jsonStreamPort: 8070,
    mjpegStreamPort: 8090,
    simulationStartupDelayMs: 5000,
    darknetStdout: true
  },
  isVideoDirectory: true,
  videoFileOrFolderExists: true,
  videoFileFps: null,
  detections: [
    { frame_id: 16, objects: [Array] },
    ... 337 more items
  ],
  simulationMJPEGServe: null,
  simulationJSONHTTPStreamServer: null,
  [Symbol(kCapture)]: false
}
Video stream: 1280x720
Start HTTP JSON Stream server
undefined
Start MJPEG server

There is an undefined
Also running with node v12

Can't you reproduce on your end ?

Yeah I can reproduce now. Don't really know why this broke now but at least I found where it broke. Will update the PR.

@tdurand it should be fixed now. Please test again, and be aware that running npm run dev will most likely fail until https://github.com/opendatacam/opendatacam/issues/313 is fixed

works great, and thanks for the documentation !


feat: Same bottom bar accross screens

Hi everyone!
We now use the same bottom bar across:

Edit mode
Product creation
Country picker

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

Project coverage is 6.92%. Comparing base (4d9c7fc) to head (cb2dfa0).
Report is 469 commits behind head on develop.

Files with missing lines
Patch %
Lines

...th_app/lib/pages/product/add_new_product_page.dart
0.00%
34 Missing :warning:

.../smooth_app/lib/widgets/v2/smooth_buttons_bar.dart
0.00%
17 Missing :warning:

..._app/lib/pages/product/common/product_buttons.dart
0.00%
7 Missing :warning:

..._app/lib/pages/product/edit_ocr/edit_ocr_page.dart
0.00%
5 Missing :warning:

...mooth_app/lib/pages/product/simple_input_page.dart
0.00%
4 Missing :warning:

...preferences/country_selector/country_selector.dart
0.00%
2 Missing :warning:

..._app/lib/pages/product/add_basic_details_page.dart
0.00%
1 Missing :warning:

...h_app/lib/pages/product/nutrition_page_loaded.dart
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5844      +/-   ##
==========================================
- Coverage     9.54%   6.92%   -2.63%
==========================================
  Files          325     422      +97
  Lines        16411   23247    +6836
==========================================
+ Hits          1567    1609      +42
- Misses       14844   21638    +6794

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

Hi @g123k! Not a big fan of naming just 2 alternate versions of widgets, but beyond that, looks OK.

The 2 is just to distinguish the current app vs the POC.
But at the end, we will be able to remove that "2".


Type stability of eigen

It's a well-known issue that eigen return a union of complex matrix and real matrix. It seems that your code assumes that the return value is always a real matrix. To prevent type instability, consider replacing all the eigen with following function:
function realeigen(x::Matrix{T})::Pair{Vector{T}, Matrix{T}} where T <: AbstractFloat
    (v, M) = eigen(x)
    return v=>M
end

This type instability is detected by SimpleTypeChecker.jl, a type checker to enforce Fortran-style programming.

Thanks for the tip! I'll look into this shortly, but it may not be an issue. The eigen decomposition is always called on a real symmetric matrix, so the eigenvalues and vectors will also always be real.
See https://discourse.julialang.org/t/eigen-warntype/21239/4

The eigen decomposition is always called on a real symmetric matrix

Then you should use eigen(LinearAlgebra.Symmetric(x)) to ensure type stable result.

I'll look into this shortly, but it may not be an issue. The eigen decomposition is always called on a real symmetric matrix, so the eigenvalues and vectors will also always be real.

Unfortunately, this is not how Julia's type inferencer works. Because it's impossible for it to know whether a matrix is a real symmertric one (such reasoning might be too hard), so it always returns a union. (It's common that inferred type is larger than actual runtime). This union can pollute all the codes following eigen.

Calling Symmetric(x) seems to have negligible overhead, so I will replace calls to eigen(x) with eigen(Symmetric(x)). I'll mark this issue as closed, and you should see the change in the next patch release.


Sidecar can't be added back to store list (sd via peers) after restart?

Thanos, Prometheus and Golang version used

Thanos 0.3.1
Prometheus 2.7.1

What happened

Upgrade all components to 0.3.1, include all sidecars, it's ok. The sd is gossip peers.
Restart 3 of the sidecars, 1 of the sidecars in the store list: status=Down, error_message=removing store because it's unhealthy or does not exist, another 2 sidecars are ok.
But in the Query-Graph, the query can query and get the recent data of the Down sidecar
If I restart all query compoents, all the sidecars are Up
The sidecar peers is in the metric thanos_store_node_info.

What you expected to happen

All the sidecars are Up after restart.

How to reproduce it (as minimally and precisely as possible):

Use thanos v0.3.1, sd via gossip peers
Restart sidecar

Full logs to relevant components
I grepped all logs of the query components.
There got no log update of store node failed of the sidecar which status is Down

Anything else we need to know
No warn log update of store node failed, maybe is the page in Query did not get the right data? any clues?
Thanks:)

when Unhealthy, delete from s.stores
https://github.com/improbable-eng/thanos/blob/master/pkg/query/storeset.go#L225
when Connected and checked healthy,  should it be added back to s.stores?
https://github.com/improbable-eng/thanos/blob/master/pkg/query/storeset.go#L312
It should be like this ?
s.stores[addr] = store

I'm not sure is the len(s.stores) is always equals to len(s.storeSpecs()), will read the code later

I think there were some fixes regarding this in the last few versions. Could you check out 0.5.0 and see if this still happens?

Sorry, all the sd of my environments are  gossip peers, so I can't upgrade to 0.5.0.  Close the issue:)


ImportError: /usr/lib/x86_64-linux-gnu/libcuda.so.1: file too short

I use Docker to run this project, but meet some problems.
The progress is below:
docker image build -t ppdemo .
docker run --name my_mongo -p 27017:27017 -d mongo
docker run -t --link my_mongo:mongo -v $PWD:/app/Personae ppdemo spider/stock_spider.py
It is fine now. But some errors happened when python import tensorflow.
$ docker run -t --link my_mongo:mongo -v $PWD:/app/Personae ppdemo algorithm/SL/DualAttnRNN.py
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
  File "/usr/lib/python3.5/imp.py", line 242, in load_module
    return load_dynamic(name, filename, file)
  File "/usr/lib/python3.5/imp.py", line 342, in load_dynamic
    return _load(spec)
ImportError: /usr/lib/x86_64-linux-gnu/libcuda.so.1: file too short

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "algorithm/SL/DualAttnRNN.py", line 3, in <module>
    import tensorflow as tf
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/__init__.py", line 24, in <module>
    from tensorflow.python import *
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/__init__.py", line 49, in <module>
    from tensorflow.python import pywrap_tensorflow
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/pywrap_tensorflow.py", line 72, in <module>
    raise ImportError(msg)
ImportError: Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
  File "/usr/lib/python3.5/imp.py", line 242, in load_module
    return load_dynamic(name, filename, file)
  File "/usr/lib/python3.5/imp.py", line 342, in load_dynamic
    return _load(spec)
ImportError: /usr/lib/x86_64-linux-gnu/libcuda.so.1: file too short

Failed to load the native TensorFlow runtime.

See https://www.tensorflow.org/install/install_sources#common_installation_problems

for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

I am a noob in using docker. May you tell me what's wrong with the progress?

It seems like your CUDA version or cuDNN version is not correct, can you tell me what version are they?

I did nothing with CUDA or cuDNN in Docker. Just build the docker image using this project's Dockerfile.
Outside the Docker, I installed CUDA8.0 && CUDA9.1 in macOS, and CUDA9.1 is linked.
$ ls /Developer/NVIDIA
CUDA-8.0 CUDA-9.1

$ nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Tue_Dec_19_21:36:29_CST_2017
Cuda compilation tools, release 9.1, V9.1.128

And my cuDNN's version seems like is 5, but it is not work with CUDA9.1 now.
$ ls /usr/local/cuda/lib | grep cudnn
libcudnn.5.dylib
libcudnn.dylib
libcudnn_static.a

$ cat /usr/local/cuda/include/cudnn.h
cat: /usr/local/cuda/include/cudnn.h: No such file or directory

Should I install the right version of CUDA and cuDNN outside the Docker? Or install the CUDA8.0 and cuDNN6 in Docker manually?  Thank you very much for explaining my confusion.

macOS

If the GPU of your mac is provided by NVIDIA, then you can use CUDA, and the version should be CUDA8, cuDNN6. Then you could run this project by the docker image mentioned in README.
If the GPU of your mac is not provided by NVDIA, then you cannot use CUDA and tensorflow-gpu and the Docker image of this project, I did not mention it in README, sorry for that, for this case, you could only use tensorflow-cpu and conda to build the environment and run this project.

It's recommended to use docker on Ubuntu with NVIDIA GPU.

@Ceruleanacg You should use Nvidia-docker2.

nvidia-docker run -dit -p 8888:8888 -p 6006:6006 tensorflow/newbuild:2.0 /bin/bash
like me .you should use nvidia-docker to run


EHPR-272 schedule data extract

Add Lambda function to extract submissions and send them as a spreadsheet to the registered recipients
Schedule extraction at 2PM UTC(7AM PST) on every Monday
List of recipients' email needs to be stored AWS Parameter Store.

@dbayly-freshworks
The Sonar error is a false positive. Can we ignore it?
https://github.com/SonarSource/SonarJS/issues/3351


Why some translations starts with <<<'TEXT'  and ends with TEXT in the language file?

Some of my translations starts with <<<'TEXT'  and ends with TEXT, I didn't put that, it's being done by the package after publishing the translations from the webUI.
<?php

return [
    'Frequent-asked-questions'  => 'Preguntas Frecuentes',
    'transactions'              => [
        'applicable-legislation'    => [
            'content'   => <<<'TEXT'

El Reglamento de la Organización y régimen del Notariado (Decreto de 2 junio de 1944) y sus posteriores modificaciones, en su artículo 144 y en los artículos 198 a 220, ambos inclusive respectivamente.

TEXT
,
        ],
    ],
];

These translations works, but I would like to know what's that syntax and if it should be there.

@gerardreches, HERE_DOC format for strings is used when a translation contains multi-line text. i.e. an end of line.

Thanks for the explanation, I couldn't find anything about it on google.


Standard MVC Template with folder Home\About gives 404

The Piranha PermalinkHandler registration
"Handlers.Add("home", "PERMALINK", new PermalinkHandler());"
clashes with the standard MVC template which creates a Home\About view. All references to the Home views are intercepted by the Permalink Handler and dont get routed to the MVC Controller creating a 404.
Request to make this re-routing optional (perhaps web.config driven, or by dynamic examination of the MVC routing table)

You can either run you application with prefixlessPermalinks disabling the permalink keyword, or you can remove the permalinkhandler and then add it with another keyword of your choice, for example cms.

Works great, thankyou. Probably would trip up anyone creating a new MVC application.


tap: gRPC push API

Define a tap xDS API that streams taps from Envoy to server

Thanks @yuval-k for putting in the work to make this API and sink happen. Are you working on the  GRPC sink implementation? Is there anything I can do to help?

I'm interested in the gRPC sink implementation. Has there been any progress?

Bump on this, interested in having this feature!

Yes, GRPC streaming of taps is highly desirable. Is this on the radar?

Bump: how can I help contribute to this feature?

Note that (1) is already handled using ECDS. We just need a gRPC streaming output (2). I will update the title.

This is something that would be very useful for my org! Is anyone still looking in to this? Anything I can do to help?

I don't think anyone is working on this, but I would love to see it implemented!

@abasitt No, nothing on this. I've moved on from the team that needed this.

@leosunmo : https://groups.google.com/g/envoy-dev/c/ItHvQvODaRk/m/pyBtlR-tAAAJ?utm_medium=email&utm_source=footer

cc @devannair77

Hi,
The design doc for StreamingGrpcSink   implementation , Let me know your comments

Hi, The design doc for StreamingGrpcSink  implementation , Let me know your comments

Thanks @devannair77 , cc @mattklein123 @soulxu

Can you please make the document into a gdoc that can be commented on? Thank you.

Can you please make the document into a gdoc that can be commented on? Thank you.

@mattklein123 I think the doc is already enabled for comments? I do see comments from other people and I am able to add comments myself too.

Sorry for some reason when I opened it before it opened just the docx file without editing. I see the right link now.

I left a few comments, thanks.

@devannair77 Is there any progress on this work? I am also very interested in this work.

@kfaseela I notice that the StreamingGrpcSink is not yet implemented, although the proto is defined.  Is this on the roadmap?

@kfaseela I notice that the StreamingGrpcSink is not yet implemented, although the proto is defined.  Is this on the roadmap?

@devannair77 did propose the RFC, but then he got pulled into other priorities that the implementation was not done

@kfaseela I notice that the StreamingGrpcSink is not yet implemented, although the proto is defined.  Is this on the roadmap?

@devannair77 did propose the RFC, but then he got pulled into other priorities that the implementation was not done

Is there any alternative solution that uses the socket transport that can post to an endpoint, or is that only for streaming results to file/admin endpoint.


Merge user provided plugin configs with default supported configs

Currently, we allow users to config the plugins as per their requirement (which is great), but newly provided config replaces the existing default config instead of merging with it.
For example: If i decided to disable certain plugin or change its config, I have to update that plugin and add all other plugins to activate them.
const tracer = new NodeTracer({
  plugins: {
    dns: { enabled: false },
    http: { enabled: true, path: ... },
    mysql: { enabled: true, path: ... },
    ...
   }
});

Expected behavior: Update only specific plugin and others should work seamlessly.
const tracer = new NodeTracer({
  plugins: {
    dns: { path: '<new path>' },
   }
});

Code reference at:
https://github.com/open-telemetry/opentelemetry-js/blob/d32565acf8f1cd7b6e0d887705123b2e0f1916a2/packages/opentelemetry-node/src/NodeTracer.ts#L39

I think at the same time we should define a configuration interface for configuring behavior of plugins.
Right now this is our plugin configuration interface
export interface PluginConfig {
    /**
     * Whether to enable the plugin.
     * @default true
     */
    enabled?: boolean;
    /**
     * Path of the trace plugin to load.
     * @default '@opentelemetry/plugin-http' in case of http.
     */
    path?: string;
    /**
     * Request methods that match any string in ignoreMethods will not be traced.
     */
    ignoreMethods?: string[];
    /**
     * URLs that partially match any regex in ignoreUrls will not be traced.
     * In addition, URLs that are _exact matches_ of strings in ignoreUrls will
     * also not be traced.
     */
    ignoreUrls?: Array<string | RegExp>;
    /**
     * List of internal files that need patch and are not exported by
     * default.
     */
    internalFilesExports?: PluginInternalFiles;
    /**
     * If true, additional information about query parameters and
     * results will be attached (as `attributes`) to spans representing
     * database operations.
     */
    enhancedDatabaseReporting?: boolean;
}

ignoreMethods

unused
definition of "method" is not clear

ignoreUrls

used by xhr
other plugins that could benefit from this but don't

internalFilesExports

used by grpc, but not configured by user. this set by the plugin loader
should not be a user-visible configuration

enhancedDatabaseReporting

unused
not well defined behavior

I am proposing to separate user-configurable options and internal configuration similar to the following:
const tracer = new NodeTracer({
  plugins: {
    dns: { options: { someconfig: true } },
   },
   sharedPluginOptions: {
      someotherconfig: true,
   }
});

Additionally, I think we should better define and document the configuration options we have that could be useful for other plugins.

I like the idea of
sharedPluginOptions: {
      someotherconfig: true,
   }

We should look at how other sigs are configuring plugins. Ruby has some way to provide a hash of options directly in the gemfile (similar to package.json), and I know some js packages allow configuration inside the package.json. Maybe this is a good way for us to go.

For now can we dial back the scope and just merge user defined config with the default config? as far as having default values for enabled and path remain untouched so one could simply:
  plugins: {
    http: {
      ignoreIncomingPaths: [/\/healthz/],
    },
  },

rather than:
  plugins: {
    http: {
      enabled: true,
      ignoreIncomingPaths: [/\/healthz/],
      path: '@opentelemetry/plugin-http',
    },
    mysql: { enabled: true, path: '@opentelemetry/plugin-mysql' },
    redis: { enabled: true, path: '@opentelemetry/plugin-redis' },
  },

Yes I think this is a good idea. If you want to make a PR that would be great, or else I can do it when I have time.

Yes I think this is a good idea. If you want to make a PR that would be great, or else I can do it when I have time.

Sure thing! I'll submit a PR.


[FEAT]: Job Listings - View Job Listing Page

Description
Develop the frontend functionality to retrieve and send available job listings to users. The listings should support sorting and filtering based on various criteria to enhance the user experience.
Acceptance Criteria

Integrate with a CMS to display job listings.
Enable users to search for jobs by name.
Allow users to sort job listings by tags.
Provide filtering options for remote and non-remote jobs.
Allow users to click "View/Apply Now" to see detailed job postings.

Purpose
To facilitate the retrieval and navigation of job listings for users.
Requirements

Handle API failures and network issues with appropriate recovery mechanisms.
Handle pagination
Limit the amount of data fetched from the API at a time.
Show a 404 page and provide a way to reset search parameters if a user searches for a non-existent job.
Display detailed job information when users click "Apply Now".

Expected Outcome

Users should see job listings in real-time.
Users should be able to search for jobs.
Users should be able to sort and filter job listings.

Tasks

[ ] Fetch job listings from the CMS via secure API call.
[ ] Limit the data fetched to 5 jobs per page.
[ ] Implement pagination and display meta information.
[ ] Enable sorting by tags (e.g., Marketing, Finance, Healthcare, Tech, Communication).
[ ] Ensure "Apply Now" links lead to detailed job pages.
[ ] Display a 404 page for non-existent job searches.
[ ] Handle network errors and API response errors gracefully.
[ ] Handle Responsiveness across all screens

Testing

[ ] Write unit tests to verify API call success.
[ ] Write integration tests for the job listing page and the detailed job page.
[ ] Write end-to-end tests to ensure smooth navigation from the public page to the job listing page to the detailed job page.

Poorly written ticket

@cobidex
@Am0du
@trevorjob

@trevorjob The salary column should be optional. Also, Follow the API route versioning rule.

@NdubuisiJr i can't edit the issue any more


Shortcuts to type unicode

In some programming languages and some lines of work, I am used to typing a lot of unicode characters. In the Julia REPL, the workflow is:
\mu<TAB> => µ
\sigma<TAB> => σ
etc...

This might slow down an editor too much to constantly watch for signals, but would this be reasonable as a plugin?
For reference, an equivalent in Atom is developed here: https://github.com/JunoLab/atom-latex-completions

The jlabbrev plugin from https://micro-editor.github.io/plugins.html is our friend.


Emotes

Added functionality to detect contributors emojis. Reactions to these messages will occur, and respective contributor will be DM'd with a link to the message their emoji has been mentioned in.
Contributors will NOT receive a DM if it is outside their UTC working hours. Timer is presently set to every 4 hours.

Lets not do the every 4h okey once is okey but i like the enthusiasm there haha
non persistent data on restart consciously?

No worries I will remove the timer.
I'll create a JSON file to store the contributor data

Addressed change suggestions raised. Contributor data now stored in contributors.json.

I did not see where you made the change to use persistent data but  i did not suggest that you do that i just asked but all good any way

I did not see where you made the change to use persistent data but i did not suggest that you do that i just asked but all good any way

I added contributors.json to keep track of contributor information. This includes the name UID and emoji => uid mapping.
Your mention of persisting the data was a good idea and something. I was aware statically defining it in the code wasn't ideal but hadn't put much thought in it before your comment :)

This is not what i meant by persistent data
The bot essentially keeps everything in memory which will therefore be bigger and if the memory goes all data in it goes
one example would be:
You get mentioned on friday evening right after your official work hours
now the bot essentially needs to hold everything necessary in memory until monday and with all the time zones etc.. this could be 3days or so in which it has to stay alive otherwise it will lose data and won't perfrom its intended functionality
But like i said it was just a question

This is not what i meant by persistent data
The bot essentially keeps everything in memory which will therefore be bigger and if the memory goes all data in it goes
one example would be:
You get mentioned on friday evening right after your official work hours now the bot essentially needs to hold everything necessary in memory until monday and with all the time zones etc.. this could be 3days or so in which it has to stay alive otherwise it will lose data and won't perfrom its intended functionality
But like i said it was just a question

Thanks for the clarification :). For the time being I have removed the UTC hours thing and the bot simply sends a DM to the user once. I maybe misinterpreted what you initially said about not pinging the contributor every 4 hours and removed that part completely. can add this back in though if we want it easily.

@Balugrizzly , have separated this from the combined branch and should now be ready to review

There are open points from other branches and from our discussions

Yeah just a few :P I will work on these today

Okey like i said in the other PR up to you how to proceed

This still has merge conflicts

This still has merge conflicts

Conflicts now resolved :)


forwardedHeaders.insecure to true

Environmental Info:
K3s Version: k3s version v1.21.2+k3s1 (5a67e8dc)

Describe the bug:
Hi, I have nginx terminating SSL and forwarding to traefik in a k3s cluster. Traefik is overwriting the X-Forwarded-* headers and passing on X-Forwarded-Proto: http instead of passing this through from nginx.
Steps To Reproduce:
How do I enable forwardedHeaders.insecure to true using the K3s helm chart thats installed by default.
I have created a trafik-config.yaml file in /var/lib/rancher/k3s/server/manifests/ with
apiVersion: helm.cattle.io/v1
kind: HelmChartConfig
metadata:
  name: traefik
  namespace: kube-system
spec:
  valuesContent: |-
    entryPoints:
      web:
        address: ":80"
        forwardedHeaders:
          insecure: true

This appears to be picked up and the helm-install-traefik pod is detecting the change in config and restarting but I never get the forwarded headers from nginx.
Any help would be appreciated.

Are you using Traefik v1 or v2? Either way, I'd probably defer to the upstream documentation, as we just package their helm chart.


Adds ZERO_PADDING variable for the default padding value instead of None. (fixes #578)

I added a new variable to padding.py named ZERO_PADDING for the default value instead of None.
For margin can be similar to this PR. If you want to apply for the margin, I can create a new RP. I hope, I have could help with this project.

Oh, unfortunately, it doesn't work like that.
Setting padding to None leaves it to be default padding determined by control of a theme.
For example, default View's, Container's, ListViews, AlertDialogs padding is 10.
This PR, on the other hand, forces all paddings to be 0. Merging this PR would practically "shrink" many UI elements.

Can I ask why?
Did you encounter an error with None?

Oh, So for that issue (#578), I will change content_padding on textfield.py if you agree.

With content_padding=0 TextField looks like:


Adds ability to add static headers to config client request.

This commit provides an alternative implementation for creating
config client request using 'ConfigurationProperties' instead
of having to create custom 'RestTemplate' when adding header to
the request.
See discusions at gh-650
Not covering gh-613

Codecov Report

Merging #656 into master will increase coverage by 0.32%.
The diff coverage is 56.52%.

@@            Coverage Diff             @@
##           master     #656      +/-   ##
==========================================
+ Coverage    71.9%   72.22%   +0.32%
==========================================
  Files          70       70
  Lines        2317     2315       -2
  Branches      322      321       -1
==========================================
+ Hits         1666     1672       +6
+ Misses        503      493      -10
- Partials      148      150       +2

Impacted Files
Coverage Δ

...rk/cloud/config/client/ConfigClientProperties.java
87.61% <100%> (+0.49%)
:white_check_mark:

...fig/client/ConfigServicePropertySourceLocator.java
64.28% <47.36%> (+5.63%)
:white_check_mark:

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 a51412e...681aeb1. Read the comment docs.

Should I also mark the spring.cloud.config.authorization property as deprecated (with @DeprecatedConfigurationProperty)?

Squashed and merged via b559735377c48c137d347d74b0151c0ffb61a86d


WebDAV: Use the proper sendRequest mechanism to append messages

This should fix #1500
I say should because I don't have a WebDAV set-up any more but the code does roughly the same as it did, only using the proper sendRequest code to handle auth errors - so I think it's fairly safe.

Still now working in K-9 Mail v5.204:

The last version I could SEND emails with is 5.010.
All later Versions broke WEBDAV-Exchange Email sending.
Can't you just check what has been changed since 5.010 in that part of code and roll it back ?
Is it possible to use SMTP for sending emails?
The (sending) Server Settings don't open in the DEVDAV account.


Invalid special_id calling Array prototype methods on TAs

const u8 = new Uint8Array(1);

Array.prototype.forEach.call(u8, () => {}); // special_id != TA
u8.forEach(() => {});  // special_id == TA (expected)

This causes https://github.com/tc39/test262/blob/c1281dba453b4d97f2bc04b8dfe722c5d5f90b89/test/built-ins/Array/prototype/forEach/callbackfn-resize-arraybuffer.js#L26 to fail in #85

What specifically needs to change and what's special_id? The Array methods work properly on arraylikes, AFAICT.

Currently these methods have predefined IDs that "hint" the this_val type to the method:
static JSValue js_array_every(JSContext *ctx, JSValueConst this_val,
                              int argc, JSValueConst *argv, int special)
{
    // ...
    if (special & special_TA) {
        obj = JS_DupValue(ctx, this_val);
        len = js_typed_array_get_length_internal(ctx, obj);
        if (len < 0)
            goto exception;

Array prototype method def:
static const JSCFunctionListEntry js_array_proto_funcs[] = {
    // ...
    JS_CFUNC_MAGIC_DEF("every", 1, js_array_every, special_every ), // <-- special_TA path not taken.

We need to change the Array prototype methods to typecheck argument and get the length (for variable-length TA) when special_id is special_every.
I opened this issue because it's probably a big change.

Ah okay, I follow you now. I believe it should Just Work(TM), just slower than otherwise because it takes the general array-like path instead of the typed array fast path.
For something like Array.prototype.map.call(new Uint8Array(1), v => v) you'll get back an Array, not a TypedArray, but that's per spec.
Can you be specific about the issues you hit?

My bad, got confused about resizable arraybuffer behaviour with Array methods. https://github.com/quickjs-ng/quickjs/pull/85#issuecomment-1824268026


Revert "Updating add-in documentation to account for add-ins on Android"

Reverts OfficeDev/outlook-dev-docs#78
These changes aren't for immediate publish, so removing from master (for now) and adding to wamaina-android branch.

@jasonjoh,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/add-ins/add-mobile-support.md
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-addin-interaction-android.gif
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-addin-taskpane-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-action-cells-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-action-dos-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-branding-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-buttons-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-cell-donts-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-cell-dos-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-cell-input-1-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-cell-input-2-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-cell-type-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-crm-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-giphy-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-icons-android.jpg
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-loading-android.jpg
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-nimble-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-overview-android.jpg
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-signin-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-tabs-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-trello-1-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-trello-2-android.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-typography-android.png
:white_check_mark:Succeeded

docs/add-ins/outlook-addin-design.md
:white_check_mark:Succeeded

docs/add-ins/outlook-mobile-addins.md
:white_check_mark:Succeeded

docs/add-ins/use-rest-api.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


added doc for image resizeMode

based on https://github.com/facebook/react-native/blob/62e8ddc20561a39c3c839ab9f83c95493df117c0/ReactAndroid/src/main/java/com/facebook/react/views/image/ImageResizeMode.java  and http://developer.android.com/reference/android/widget/ImageView.ScaleType.html

@facebook-github-bot shipit


Fix mini typos in the docs

As part of the JuliaCon hackathon, I have run all the docs through Grammarly. In case of ambiguity, I have picked the more common spelling on the site or googled for the more variant.
I'm happy to withdraw this PR if it's not helpful.

woah this is great, thank you very much!


test_runner: improve wait_until

Improves wait_until by:

Use timeout instead of iterations. This allows changing the timeout/interval parameters independently.
Make timeout and interval optional (default 20s and 0.5s). Most callers don't care.
Only output status every 1s by default, and add optional status_interval parameter.
Remove show_intermediate_error, this was always emitted anyway.

Most callers have been updated to use the defaults, except where they had good reason otherwise.

One note of caution: the use of defaults in so many places means that these default values are going to become somewhat sacred, as ever changing them will likely destabilise some tests.

Yeah, that's always a risk here. I figure 20 seconds is enough to cover practically all common cases and still low enough not to be annoying. Many of the explicit timeouts were also either 20 or 30 seconds, and this seemed to have been mostly arbitrary.

Will hold off on this until Monday, to avoid any weekend CI breakage.


Fixes a few items not being colored, and Allows for Dark and High Contrast Themes

@HuanhuanSunMSFT
Fixes #29.

Looks good.


Bump testcontainer version to support docker compose v2

It would be good to support docker compose v2 and bump testcontainers version to v0.18.0 or newer to use the native compose integration instead of invoking a shell command.
This would reduce the random compatibility issues that make the tests sometimes fail and also reduce the boilerplate implemented in order to have the same control over the running containers.

Depends on https://github.com/testcontainers/testcontainers-go/issues/1276


Build Components and Pipelines

https://www.kubeflow.org/docs/pipelines/sdk/build-component/
Under "Create a container image for each component" we are linked to a Docker file for XGBoost that is located in a "deprecated" folder on Github.

/priority p2


Adding 00_bashling theme, and sort the authors names in the README

Is putting '00_' in the name just an attempt to get to the top of the sort list?

Yes.
Is that okay?

Le 17 sept. 2015 à 20:07, Bo Lopker<EMAIL_ADDRESS>a écrit :
Is putting '00_' in the name just an attempt to get to the top of the sort list?
—
Reply to this email directly or view it on GitHub https://github.com/Colorsublime/Colorsublime-Themes/pull/202#issuecomment-141169302.

I say 'no', but I'll let @GianlucaGuarini make the call.

Sure.

Le 17 sept. 2015 à 20:11, Bo Lopker<EMAIL_ADDRESS>a écrit :
I say 'no', but I'll let @GianlucaGuarini https://github.com/GianlucaGuarini make the call.
—
Reply to this email directly or view it on GitHub https://github.com/Colorsublime/Colorsublime-Themes/pull/202#issuecomment-141170196.

@lingtalfi thanks for your pull request. But I would prefer a new one where you just submit your theme without '00_' and without changing the order of the authors in the readme file ( let us doing it ).

You've got it.


Following semver

Are you willing to follow semver more closely? :) I know redux-form is very very young, but I feel that it might be a good thing now to at least signal breaking changes by bumping the minor version, instead of just the patch version. Thoughts on this?

Excellent suggestion. I'm an OSS n00b, and just learned about semver yesterday.

Ah ok cool, no worries :) Usually you'd bump the major version, if there is a breaking change, the minor version if you add new features which do not break old behavior and the patch version for bugfixes etc.
Some projects choose to add a disclaimer that pre-1.0 a bump in the minor version can contain breaking changes.

I mean, she's only 13 days old! :-)
It certainly feels like the API is stabilizing, but who knows...
Thank you for your suggestion. I'm going to close this ticket for now, as it's not so much of an actionable item that can be completed, but a suggestion for going forward. I might even start a changelog!


Remove . from phpstan files in .gitattributes

I'm not sure if you actually want this, or if you want to move the phpstan files to the . prefixed names as per .gitattributes?

nice catch


Coverity issues with Coverity 2024.6.0

Description
Hi,
I am using the library at a place where Coverity scans the source code.
Recently the scanner was upgraded to 2024.6.0
Some new issues are popping up:
For example:

There are around 17 reported issues as far as I can see. I can add the full list in the ticket if interested.
Thanks for this amazing library in any case!
Reproduction steps
Run Coverity 2024.6.0 on json.hpp
Expected vs. actual results
Expect: no errors
Results: some errors are reported
Minimal code example
No response
Error messages
No response
Compiler and operating system
Linux/ unknown
Library version
960b763ecd144f156d05ec61f577b04107290137
Validation

[X] The bug also occurs if the latest version from the develop branch is used.
[X] I can successfully compile and run the unit tests.

@alainsanguinetti Can you provide an updated list of the results?

2024_11_20_coverity_scan.pdf
Hi @nlohmann, here's an "export", sorry it's the only way I could get the information out efficiently..

it's based on a version from July

Thanks! I will see what's still relevant from this.

The only valid issue I see here is parser_callback_t, which is an alias for std::function, being taken by value and not being moved.
The ones about the resource leak are incomplete analysis, Coverity just can't see how they're being deleted.
It is assuming that get() and get_token() return the same thing every time they're called, but they do not.
It is complaining about asserts.
It is complaining that the move constructor is forwarding the argument to the base class constructor.  This is how it needs to be done (it could actually just be std::move instead of std::forward, but that doesn't change the behavior, just the amount of text).
It is complaining that the noexcept constructor for nullptr_t calls a function that can throw, but doesn't do the analysis to see that it can't reach any of the throwing code because of the parameter value passed in.
It is complaining about the use of 32 bit time_t instead of 64 bit in a templated function, so the time_t is selected by the caller and not the library code.
It is complaining about using '0' instead of 0 in a memset call, but the function is creating the string form of a number and filling the string with zeros, not nulls.

Thanks @gregmarr for the analysis!

The issue with not moving a parser_callback_t should be fixed with #4489
I agree with your analysis on resource leaks and the misunderstanding of Coverity about get() and get_token().
Asserts are used to avoid undefined behavior in debug builds as well as to document invariants. Great to see some invariants "validated" by a tool.

Maybe @alainsanguinetti can create another check on the latest develop branch.

Hi, I'll give a try to the update and will report later, could take a few weeks


ace-jump split between files?

There's ace-jump stuff in both behavior/text.el and behavior/ace-jump.el. We need to consolidate. Personally, I think we should go for text.el.

Sounds good.


Sending email throws error

Hello,
Sending email (s/mime) from sender to receiver throws error:
Error, unable to find recipient key "sender adress"
Option "Encrypt mail to Self" is disabled.
For information, the error threw is with the address in uppercase whereas in the certificate I use, this very adress is lowercase.
Versions:
Outlook : Pro + 2010 (14.0.7163.5000)
OPP : BETA56
GnuPG4Win : 2.3.0
Here is the log collected in %AppData%
-------- 12/16/2015 3:41:24 PM --------
EncryptEmail
Deja.Crypto.BcPgp.PublicKeyNotFoundException: Error, unable to find recipient key "sender adress".
at Deja.Crypto.BcPgp.PgpCrypto.Encrypt(Byte[] data, IList1 recipients, Dictionary2 headers)
at OutlookPrivacyPlugin.OutlookPrivacyPlugin.EncryptEmail(Byte[] data, IList`1 recipients)

Are you actually using S/MIME or are you using OPP to encrypt the outbound email?  OPP doesn't perform S/MIME which is a different encryption method from PGP/GPG.

Oh, mybad, i misread the description, so OPP is not what i am looking for? It's not going to answer my need? If so, close this ticket, sorry for the inconvenience.

What's your need?
Cheers,
Mike

On Dec 16, 2015, at 11:45 PM, rmarsollier<EMAIL_ADDRESS>wrote:
Oh, mybad, i misread the description, so OPP is not what i am looking for? It's not going to answer my need? If so, close this ticket, sorry for the inconvenience.
—
Reply to this email directly or view it on GitHub.

Well, I talked to my managers, and we will use plain PGP/mime instead of S/MIME with certs of our CA, so I simplified the problem :)
BTW : in the main description : "Decrypting in preview pain" should be "Decrypting in preview pane".

Closing as duplicate of #123