LSOpenURLsWithRole() failed with error -600

I try to refresh the plugin by：
open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=plugin.py'
And I got:
LSOpenURLsWithRole() failed with error -600 for the URL xbar://app.xbarapp.com/refreshPlugin?path=plugin.py.
I guess some settings of my computer may cause this problem. Does anyone have similar problems?
System version: MACOS 10.15.7

@xfangfang The internet tells me this is usually a permissions error. Are you able to run xbar? Did you install it by dragging it into the Applications folder?

@matryer
I can use xbax. After download it, I drag it to the application folder. When first time I run it, shows that because of untrusted developers I can't open, and I go to settings to allow xbar to be opened.
Also tried: sudo open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=plugin.py'
Shows the same to before.
Thank you very much for your reply.

@xfangfang You're welcome.
Does plugin.py definitely exist in the ~/Library/Application Support/xbar/plugins folder?

(jupyter369) xfangfangs-MBP:plugins fang$ pwd
/Users/fang/Library/Application Support/xbar/plugins

(jupyter369) xfangfangs-MBP:plugins fang$ ls -lah
total 96
drwxr-xr-x  8 fang  staff   256B  4  6 19:00 .
drwxr-xr-x  5 fang  staff   160B  4  5 21:23 ..
-rwxr-xr-x@ 1 fang  staff   893B  4  6 19:00 001-age-ticker.1s.py

(jupyter369) xfangfangs-MBP:Contents fang$ ls -lah /Applications/xbar.app
total 0
drwxr-xr-x   3 fang  admin    96B  4  6 06:30 .
drwxrwxr-x+ 88 root  admin   2.8K  4  6 06:30 ..
drwxr-xr-x   5 fang  admin   160B  4  6 06:30 Contents

Nice report @xfangfang. Which URL are you trying to refresh that age-ticker plugin?

@matryer
(jupyter369) xfangfangs-MBP:~ fang$ open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=001-age-ticker.1s.py'
LSOpenURLsWithRole() failed with error -600 for the URL xbar://app.xbarapp.com/refreshPlugin?path=001-age-ticker.1s.py.
(jupyter369) xfangfangs-MBP:~ fang$ open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=001-age-ticker'
LSOpenURLsWithRole() failed with error -600 for the URL xbar://app.xbarapp.com/refreshPlugin?path=001-age-ticker.
(jupyter369) xfangfangs-MBP:~ fang$ open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=age-ticker.1s.py'
LSOpenURLsWithRole() failed with error -600 for the URL xbar://app.xbarapp.com/refreshPlugin?path=age-ticker.1s.py.
(jupyter369) xfangfangs-MBP:~ fang$ open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=age-ticker.py'
LSOpenURLsWithRole() failed with error -600 for the URL xbar://app.xbarapp.com/refreshPlugin?path=age-ticker.py.

I can open Safari by open -a safari, but open -a xbar doesn't works.
Try this on another Mac (OS11.0.1), got the same error, but this time shows more information.
FANGdeAir:679b9982232b318b fang$ open 'xbar://app.xbarapp.com/refreshPlugin?path=001-age-ticker.1s.py'
The application cannot be opened for an unexpected reason, error=Error Domain=NSOSStatusErrorDomain Code=-600 "procNotFound: no eligible process with specified descriptor" UserInfo={_LSLine=379, _LSFunction=_LSAnnotateAndSendAppleEventWithOptions}

FANGdeAir:679b9982232b318b fang$ open -a xbar
The application /Applications/xbar.app cannot be opened for an unexpected reason, error=Error Domain=NSOSStatusErrorDomain Code=-600 "procNotFound: no eligible process with specified descriptor" UserInfo={_LSLine=379, _LSFunction=_LSAnnotateAndSendAppleEventWithOptions}

xbar v2.0.42.-beta

@matryer I found an interesting thing. If I close the software first, and then open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=001-age-ticker.1s.py', there will be no error. However, the Xbar icon will only flash in the dock, and nothing happened.
If the software has been opened, and then refresh the plugin through the terminal, it will report an error.
I get more error information on another computer(MACOS 11.0.1)
FANGdeAir:679b9982232b318b fang$ open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=001-age-ticker.1s.py'
The application cannot be opened for an unexpected reason, error=Error Domain=NSOSStatusErrorDomain Code=-600 "procNotFound: no eligible process with specified descriptor" UserInfo={_LSLine=379, _LSFunction=_LSAnnotateAndSendAppleEventWithOptions}

I have the exact same behaviour:
~/Library/Application Support/xbar/plugins on  master
⚡23% ❯ open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=autogit_status.15m.sh'
LSOpenURLsWithRole() failed with error -600 for the URL xbar://app.xbarapp.com/refreshPlugin?path=autogit_status.15m.sh.`

For every plugin:
💥~/Library/Application Support/xbar/plugins on  master
⚡23% ❯ ll
total 40
-rwxr-xr-x  1    2.7K 29 Mar 17:23 clip-hist.3s.sh*
-rwxr-xr-x@ 1    400B  4 Apr 20:53 tugbanner.1m.sh*
drwxr-xr-x  4    128B  4 Apr 20:56 purgatory/
-rwxr-xr-x@ 1    1.6K 10 Apr 07:33 weather.15m.sh*
lrwxr-xr-x  1     17B 11 Apr 06:46 autogit_status.15m.sh@ -> ../autogit_status
-rwxr-xr-x  1    1.2K 11 Apr 08:03 ansi.sh*
drwxr-xr-x  3     96B 11 Apr 09:38 hold/
-rwxr-xr-x@ 1    1.3K 11 Apr 12:30 batt-mins.10s.sh*

💥~/Library/Application Support/xbar/plugins on  master
⚡23% ❯ open -jg 'xbar://app.xbarapp.com/refreshPlugin?path=batt-mins.10s.sh'
LSOpenURLsWithRole() failed with error -600 for the URL xbar://app.xbarapp.com/refreshPlugin?path=batt-mins.10s.sh.

Without seeming to update the relevant plugin.
interestingly ...
💥~/Library/Application Support/xbar/plugins on  master
⚡23% ❯ open -a xbar
LSOpenURLsWithRole() failed for the application /Applications/xbar.app with error -600.

💥~/Library/Application Support/xbar/plugins on  master
⚡23% ❯ open /Applications/xbar.app/
LSOpenURLsWithRole() failed with error -600 for the file /Applications/xbar.app.

But in this latter case it does seem to open the application and give it focus
Now my confession:
💥~/Library/Application Support/xbar/plugins on  master
⚡21% ❯ cd ..

💥~/Library/Application Support/xbar
⚡21% ❯ ll
total 0
drwxr-xr-x  4    128B 29 Mar 17:20 cache/
lrwxr-xr-x  1     28B 29 Mar 17:22 plugins@ -> /Users/tadhg.obrien/bin/xbar

💥~/Library/Application Support/xbar
⚡21% ❯

The plugin directory is symlinked to a subdir of my custom bin folder, for the purpose of keeping it under source control and I don't like keeping my code in random hidden folders that Apple has created. Though Apple didn't create this one, in all fairness :)
I thought this might do the trick:

But no luck, same results

Update ... I thought I was coming back to make a correction that the full disk access had done the trick, but then, ...
💥~/Library/Application Support/xbar
⚡21% ❯ open-jg xbar://app.xbarapp.com/refreshPlugin\?path=batt-mins.10s.sh

💥~/Library/Application Support/xbar

crashed xbar
After starting it again manually, we're back to -600 errors

@tigger04 I don't think it's related, but I noticed an extra \ in your URL:
xbar://app.xbarapp.com/refreshPlugin\?path=batt-mins.10s.sh
                                    ^

Oh - I was just escaping it. You'll see in the previous examples it's within quotes and no escaping with \

This problem stems from the fact that xbar is from an untrusted source. To confirm whether this is the case for you, you can run the following:
$ xattr -l /Applications/xbar.app/Contents/MacOS/xbar
com.apple.quarantine: 0000;00000000;Source;00000000-0000-0000-0000-000000000000

If there's output with com.apple.quarantine then that's the issue. You can manually remove the executable from quarantine, following which you should get no output when running the above again.
$ xattr -dr com.apple.quarantine /Applications/xbar.app/Contents/MacOS/xbar
$ xattr -l /Applications/xbar.app/Contents/MacOS/xbar
# No output

Following the above, the open call works as expected. The problem should go away when the executable is properly signed.

@Etheryte
It doesn't work for me.
On my MacBook(MacOS 10.15.7), running xattr -l ... output is empty, and running open -jg ... will still report an error.
What's interesting is that I had this problem on another device(MacOS 11.0.1). After updating to MacOS 11.3, the problem disappeared, but when I ran:
$ xattr -l /Applications/xbar.app/Contents/MacOS/xbar

I can get a string of output starting with com.apple.quarantine

This problem might stem from the fact that xbar is from an untrusted source. To confirm whether this is the case for you, you can run the following:

What's to confirm? All of the builds on the releases page, and via Homebrew, are unsigned and therefore untrusted on MacOS. The only way around this is to build it yourself from source in Xcode.

I don't mean to sound ungrateful to the creator and maintainers who have put so much time and effort in to this wonderful tool, but the rebrand to xbar and decision to rewrite it from scratch has obviouly been a failure since there are a plethora of regression defects that make this pretty much un-usable now. This in itself would not be a problem if it was maintained, but as it's now 3 months since the last release I think it's safe to assume it is not.

Anyone ever solve this problem as I am running into it as well.
I will note that the problem only occurs if I start xbar by double clicking the icon and then trying to run 'open open -jg 'xbar://app.xbarapp.com/refreshAllPlugins' from a terminal window. But, from a terminal window if I start xbar by running /Applications/xbar.app/Contents/MacOs/xbar it runs successfully and then in a separate terminal window the above open command works as expected with no errors.

Can confirm for me as well that starting it through terminal is the only way to get the URL-scheme functionality to work.


Add MPI_ prefix to internal status field

We're just being extra careful here.

I don't understand how this matters. No one should ever touch this. How can the name be significant?

The calling code might do a #define internal 42. If we stick with using an MPI_ prefix for everything then we're safe.
See e.g. struct rusage where the fields all have an ru_ prefix.

Wouldn't it be better to use something that starts with __ then since that's reserved by the C language?

As C reserves identifiers starting with _ and continuing with an uppercase char, we would be breaking the rules and a picky compiler could potentially generate a warning/error, and we do not want that.
MPI has already reserved both the MPI_ and mpi_ prefixes (page 24), therefore I don't have too many arguments to not use one of those here and pick something different. Perhaps lower case mpi_  would be more appropriate here?

Caveat: MPI_internal does not convey properly the "private" intention for the field. Using _mpi_private would work better in this case as  _<lowercase...> is not reserved by C, but MPI has not so far reserved lowercase _mpi_ starting with underscore.


[TFLite] Support Operator MUL,RESIZE_BILLINEAR not implemented

when I try a pretrained tflite model, get a error:
tvm.error.OpNotImplemented: The following operators are not supported in frontend TFLite: 'MUL','RESIZE_BILINEAR'

Added MUL https://github.com/dmlc/tvm/pull/3304

PR to support RESIZE_BILLINEAR https://github.com/dmlc/tvm/pull/3370


Add AVA dataset

🚀 The feature
Add the AVA dataset (AVA: A Large-Scale Database for Aesthetic Visual Analysis) in TorchVision.
Motivation, pitch
It currently has +600 citations. It's a commonly used dataset for image Aesthetics score prediction which is used for building image ranking models. It might be worth including it so that we can provide some Learn-to-rank examples using TorchVision.
cc @pmeier

Can do.

@datumbox @pmeier I'll wait with the implementation until the new datapipes as that would require a rework anyways.


Clarify what sort of biomedical data ARAs will be required to obtain …

…from KPs or other translator resources.
In particular, while obtaining Message nodes and edges from Translator resources makes complete sense imho, ARAs frequently require additional information to perform reasoning. Examples include:

General use case: utilizing KS (and perhaps KP) data dumps to speed performance.

Specific use case: NCBIeUtils is a slow API endpoint. When an ARA receives/creates a Message KG with many thousands of nodes and edges, the ARA may wish to annotate all pairs of nodes or all edges with co-occurrence frequency in PubMed literature. A locally cached version of PubMed/baseline stored in a local database causes these (tens of) thousands of queries to be executed in a performant fashion. Even though technically a Translator KP could be used to provide a faster API endpoint than NCBIeUtils for this purpose, the performance penalty would still be experienced. i.e. keep these latency numbers in mind.

General use case: requiring locally aggregated graphs or other non-Translator resources for machine learning/reasoning purposes.

Specific use case: An ARA may need access to a specifically formatted graph (not associated with any particular Message) in order to utilize graph convolutional neural network methods (or node embedding methods for downstream traditional ML techniques). The ARA would store and use this information for link prediction, answer scoring, etc. purposes. It seems unreasonable to require a KP to provide this resource for a particular ARA.

@dkoslicki - these are great points, and I'd like to ask a few questions to help clarify my understanding.
So for use case 1, is it fair to say that this is mostly a concern about caching? If so, then I think it's mostly orthogonal to whether ARAs can get non-KP information.  So, if caching of KPs were allowed, then I think we would want to make a KP for that information, such that any of the ARAs could use it.  I guess one problem could be if the data being pulled is not amenable to being served via a ReasonerAPI, but I'm not sure if that's the problem here...
For use case 2, I agree that we need to be able to instantiate those aggregate graphs for all kinds of big-graph analysis.  The question in my mind is whether they can be created from KPs.  My hope is that they could be - the KGX tools developed by @cmungall and @deepakunni3 among others should allow this to happen pretty easily?  Or do you see a need to bring in non-KP information into such a graph?

@cbizon
For use case 1, yes this is “mostly” a concern about caching. Hitting an API is significantly slower than hitting a database stored in memory. While KP’s could provide endpoints that allow bulk download of the data:

You point of ReasonerAPI perhaps not being amenable to the task is well taken (and apparently proposed on the agenda for the current “relay” breakout groups on Thursday).
Some data sources are not provided by any KP (eg. We (Team Expander Agent) have used the Veteran’s Association National Drug File as a cached sources of information previously). Seems onerous to ask a KP team to roll out such a data source when we’re using it for a specific task (eg. use case 2).

For use case 2: I think the aggregated graphs (from tools such as KGX) would be a great starting point, but additional (non-KP) information might be required. For example, some GNN’s require graphs to have a specific format (easily done by post-processing a, say, KGX aggregated graph), but also require additional information decorating node/edge properties (specifically, numerical values related to non-KP derived (or only partially KP derived) training data, etc). It would be nice to clarify if it’s “ok” for ARA’s to keep/control these modified graphs (originally derived from some Translator KP-aggregated graph(s)) as local sources of information to informal machine learning models.

@dkoslicki - did the Relay discussion help with the clarification here?  I think that the outcome of that discussion was that:

caching is fine, as long as the things being cached were from KPs
If an ARA needs non KP-served data it should try to get a KP team to stand them up, but at least in the short term, standing up their own KP is ok.

@dkoslicki - did the Relay discussion help with the clarification here? I think that the outcome of that discussion was that:

caching is fine, as long as the things being cached were from KPs
If an ARA needs non KP-served data it should try to get a KP team to stand them up, but at least in the short term, standing up their own KP is ok.

Yup! I think that accurately summarizes the discussion. Though on point 2, "in the short term" may end up being longer than we think (as KP teams have their own milestones to prioritize). Might be wise to make the architecture doc just not say anything/much on this point (as I think is currently the case) until a KP/KS registry is settled on, a request/ticketing system is set up, etc.


added idle time in vad metrics

example
    @agent.on("metrics_collected")
    def _on_metrics_collected(mtrcs: metrics.AgentMetrics):
        if isinstance(mtrcs, metrics.PipelineVADMetrics):
            logger.info(f"VAD metrics: idle_time={mtrcs.idle_time:.2f}")
        metrics.log_metrics(mtrcs)
        usage_collector.collect(mtrcs)

2024-11-10 07:00:08,775 - DEBUG livekit.agents.pipeline - speech playout finished {"speech_id": "5e2f1364bd6d", "interrupted": false}
2024-11-10 07:00:08,775 - DEBUG livekit.agents.pipeline - committed agent speech {"agent_transcript": " Hey, how can I help you today?", "interrupted": false, "speech_id": "5e2f1364bd6d"}
2024-11-10 07:00:09,195 - INFO voice-assistant - VAD metrics: idle_time=3.06
2024-11-10 07:00:10,215 - INFO voice-assistant - VAD metrics: idle_time=4.08
2024-11-10 07:00:11,234 - INFO voice-assistant - VAD metrics: idle_time=5.10
2024-11-10 07:00:12,265 - INFO voice-assistant - VAD metrics: idle_time=6.13
2024-11-10 07:00:13,285 - INFO voice-assistant - VAD metrics: idle_time=7.15
2024-11-10 07:00:14,314 - INFO voice-assistant - VAD metrics: idle_time=8.18
2024-11-10 07:00:15,335 - INFO voice-assistant - VAD metrics: idle_time=0.48
2024-11-10 07:00:15,685 - DEBUG livekit.agents.pipeline - received user transcript {"user_transcript": "Hi."}
2024-11-10 07:00:16,143 - DEBUG livekit.agents.pipeline - validated agent reply {"speech_id": "930dc55c54bc"}
2024-11-10 07:00:16,144 - INFO livekit.agents - Pipeline EOU metrics: sequence_id=930dc55c54bc, end_of_utterance_delay=0.91, transcription_delay=0.45
2024-11-10 07:00:16,354 - INFO voice-assistant - VAD metrics: idle_time=0.57
2024-11-10 07:00:16,814 - INFO livekit.agents - Pipeline STT metrics: duration=10.68, audio_duration=2.10
2024-11-10 07:00:16,935 - INFO livekit.agents - Pipeline LLM metrics: sequence_id=930dc55c54bc, ttft=0.65, input_tokens=63, output_tokens=10, tokens_per_second=12.67
2024-11-10 07:00:17,384 - INFO voice-assistant - VAD metrics: idle_time=1.60
2024-11-10 07:00:18,403 - INFO voice-assistant - VAD metrics: idle_time=2.62
2024-11-10 07:00:18,535 - DEBUG livekit.agents.pipeline - speech playout started {"speech_id": "930dc55c54bc"}
2024-11-10 07:00:18,774 - INFO livekit.agents - Pipeline TTS metrics: sequence_id=930dc55c54bc, ttfb=1.5990837999997893, audio_duration=2.26
2024-11-10 07:00:19,436 - INFO voice-assistant - VAD metrics: idle_time=3.65
2024-11-10 07:00:20,454 - INFO voice-assistant - VAD metrics: idle_time=4.67
2024-11-10 07:00:20,796 - DEBUG livekit.agents.pipeline - speech playout finished {"speech_id": "930dc55c54bc", "interrupted": false}
2024-11-10 07:00:20,797 - DEBUG livekit.agents.pipeline - committed agent speech {"agent_transcript": " Hello! What can I help you with today?", "interrupted": false, "speech_id": "930dc55c54bc"}
2024-11-10 07:00:21,475 - INFO voice-assistant - VAD metrics: idle_time=5.69
2024-11-10 07:00:22,505 - INFO voice-assistant - VAD metrics: idle_time=6.72
2024-11-10 07:00:23,525 - INFO voice-assistant - VAD metrics: idle_time=7.74
2024-11-10 07:00:24,557 - INFO voice-assistant - VAD metrics: idle_time=8.77
2024-11-10 07:00:25,575 - INFO voice-assistant - VAD metrics: idle_time=9.79
2024-11-10 07:00:26,596 - INFO voice-assistant - VAD metrics: idle_time=10.81

PTAL @davidzhao @theomonnom

this gets emitted when the next non-silence is detected?

Basically PipelineVADMetrics emits events on every seconds when vad event type is VADEventType.INFERENCE_DONE,
I've added condition to reset idle time, whenever event type is VADEventType.START_OF_SPEECH or VADEventType.END_OF_SPEECH

I don't think this is the right place to put this metric. This is being called each 1 second. I think we already have something similar inside the INFERENCE_DONE of VAD

nit: use time.perf_counter instead of time.time()

@theomonnom Yes, that's correct! the metric is meant to track the time since the last user speech.

Nice lgtm!


README/help output have unclear information about CLI options.

(to be fleshed out further)
Currently, the help output in the README demonstrates the CLI flags as being global options, regardless of subcommand. This is not the case, and trying to pass them as global options results in error messages about the options being unknown.
We should, ideally:

clean up the help output itself to show the command line usage as something closer to datadog-sync [global options] [command] [options] [args]
remove the help text in the README itself, since it's a lot of extra noise and is at risk of falling out of sync with the actual help output.

Closed by: https://github.com/DataDog/datadog-sync-cli/pull/216


Attempting login to Pokemon Go Failed

Expected Behavior
starts normally
Actual Behavior
[11:43:58] PokemonGO Bot v1.0
[11:43:58] Configuration initialized
[11:43:58] [x] Coordinates found in passed in location, not geocoding.
[11:43:58]
[11:43:58] Location Found: 48.893666,2.390437
[11:43:58] GeoPosition: (48.893666, 2.390437, 0.0)
[11:43:58]
[11:43:58] Attempting login to Pokemon Go.
Traceback (most recent call last):
  File "pokecli.py", line 223, in <module>
    main()
  File "pokecli.py", line 209, in main
    bot.start()
  File "/root/PokemonGo-Bot/pokemongo_bot/__init__.py", line 32, in start
    self._setup_api()
  File "/root/PokemonGo-Bot/pokemongo_bot/__init__.py", line 257, in _setup_api
    self.login()
  File "/root/PokemonGo-Bot/pokemongo_bot/__init__.py", line 242, in login
    str(self.config.password)):
  File "/root/PokemonGo-Bot/src/pgoapi/pgoapi/pgoapi.py", line 158, in login
    response = self.call()
  File "/root/PokemonGo-Bot/src/pgoapi/pgoapi/pgoapi.py", line 81, in call
    response = request.request(api_endpoint, self._req_method_list, player_position)
  File "/root/PokemonGo-Bot/src/pgoapi/pgoapi/rpc_api.py", line 98, in request
    raise NotLoggedInException()
pgoapi.exceptions.NotLoggedInException

Steps to Reproduce
launch with PTC credential
Other Information
OS: Debian x64_86
Git Commit: eaef8fcb9eb8d7006fcdda8ae289ca87e825773d

things sometime result into an :
[11:49:07] Attempting login to Pokemon Go.
2016-07-25 11:49:47,627 [    pgoapi] [ERROR] Login failed - unexpected server response!
[11:49:47] [X] Login Error, server busy
[11:49:47] [X] Waiting 10 seconds to try again

i cant even login with the official app, think it is a server problem from PTC, google works without issues...

http://ispokemongodownornot.com/

I think it is a PTC problem, as their web interface is also down: https://club.pokemon.com/us/pokemon-trainer-club/login

ok closed


Weird default username output

coming through as thejameskyle\n,

ping @megawac. any cluez?

trim all the things?

It could be that it is an array:
_.trim(['thejameskyle\n', '']) >> "thejameskyle\n,"

Weird - the same thing is happening to me. This is a new bug!

Not happening to me anymore. @thejameskyle, are you still experiencing this? In other werdz, what does npm whoami return for you?

Notes for future James: npm adduser sets the username config property that npm whoami returns. This couldn't possible be an Array, right? So I'm not too sure why this would happen.

ping @thejameskyle. Still having this issue?

Just got this issue...it may be a problem with npm3? Will investigate further soon...

I haven't looked at it in awhile

This is the most bizarre thing. On npm 2.14.7 and npm 3.3.12, the boilerplate's code returns:
[ [ 'jmeas\n', '' ] ]
That is, a nested array with the first value being jmeas\n.
Running that same exact code outside of the boilerplate results in [ 'jmeas\n' ].
Pretty crazy stuff.

Upgrading to Bluebird v3 fixes this. Pushing a bug fix now.


Your site is down

5 mins ago your side goes down with 502 nginx error.

https://vuejs.org/ seem to be up for me. Is it still down for you?

No, l still get 502 error((

It start to work. Thank you @simplesmiler for quick response


CASMINST-6228 Add diagrams for IUf installs and upgrades

Add diagrams for IUf installs and upgrades that correspond to the instructions already provided.
Description
CASMINST-6228
Checklist

[x] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[x] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[x] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.4

/backport release/1.4

/backport release/1.4


fix:(Navisworks): progress bar should have a maxium value of 1.0

closes: #2762
An occasional rounding error was causing some updates to the progress bar of greater than 100% (or over 1.0). This throws an error from the host application causing a failed conversion)
Changes:
For this hotfix, the Invoker method updating the progress UI has a Math.Max(fractiondone,1) shield.
TODO: Will revisit this as there are three different methodologies currently used to make traversal progress reporting - should adopt a single pattern

bound into https://github.com/specklesystems/speckle-sharp/pull/2766


Implementing getLayers

Uses the layers that are being passed into requestPresent without any additional validation

Sweet!
Shouldn't currentLayers be a member of VRDisplay?

Can make it so :-)

Excellent! Thank you! 👍🏽
I'm going to check if there's something else that can be added for a new release to the Chrome Store.


Implement certifying and withdrawing Plutus script tests

bors r+

bors r+

bors r+

bors r+

bors r+


Allow a custom alphabet to be used by the encoder

this is great. is it possible to support a character map?
I'm trying to use kai notation
https://github.com/elixireumorg/base32-alphabets#kai
https://github.com/elixireumorg/base32-alphabets/blob/master/lib/base32-alphabets/kai.rb
kai notation maps certain characters like l to 0,  and uses letters in place of numbers after 9, etc.


does dockerspawner work on windows?

I don't believe JupyterHub is supported on Windows, so I would not expect dockerspawner to work. I don't know if anybody has actually tried it, though.

No, dockerspawner won't work on Windows because JupyterHub itself won't run on Windows. I suppose it could work if you ran JupyterHub itself in a docker container on the Windows machine, though.

@minrk docker containers still share the kernel of their operating system host, so running JupyterHub in a docker container on Windows would also require running that container in a VM...at which point you're not really running on Window anyway.

Supposedly docker containers are coming to Windows Server
On Thu, Apr 23, 2015, 2:21 PM Scott Sanderson<EMAIL_ADDRESS>wrote:

@minrk https://github.com/minrk docker containers still share the
kernel of their operating system host, so running JupyterHub in a docker
container on Windows would also require running that container in a VM...at
which point you're not really running on Window anyway.
—
Reply to this email directly or view it on GitHub
https://github.com/jupyter/dockerspawner/issues/28#issuecomment-95690906
.

Docker on Windows generally means boot2docker, so the 'host' is actually a Linux VM already. The same is true on OS X.

Supposedly docker containers are coming to Windows Server

It's certainly on their agenda.
For the purposes of Dockerspawner, it interacts with the Docker API. As long as it supports the API properly, that's the path.
The Docker Engine for Windows Server doesn't exist yet, but if I had to guess they'll be running their own stripped down version of Linux to make it possible. It does have to be POSIX compatible after all.

Jupyterhub works on Windows from within a docker container if you use Oauth and either simplespawner (for testing only) or dockerspawner (binding to the docker socket on the host and creating the single notebook containers on the same network).


gunicorn worker and master will exit and http response with "maximum recursion depth exceeded while calling a Python object" accidentally

Describe the bug
My new created flow app will stopped accidentally, worker and master will exit automatically, meanwhile, the http request will response with: "{"error":{"code":"SystemError","message":"maximum recursion depth exceeded while calling a Python object"}}". After a restart of gunicorn, the service will work again. I have experiences this problem couple of times.
How To Reproduce the bug
Steps to reproduce the behavior, how frequent can you experience the bug:

pf flow init --flow to create a flow app with --type chat
impl some logic
start the service with: gunicorn --reload -w 8 --threads 1 -b "<IP_ADDRESS>:8080" --timeout 300 "promptflow._sdk._serving.app:create_app()"
test the api, and working well
after sometime, may coupls of hours
access the api again, it will respond with error
check the command output, it shows:
[2024-01-25 11:28:04 +0800] [10585] [INFO] Worker exiting (pid: 10585)
[2024-01-25 11:28:04 +0800] [10587] [INFO] Worker exiting (pid: 10587)
[2024-01-25 11:28:04 +0800] [9734] [INFO] Handling signal: term
[2024-01-25 11:28:04 +0800] [10589] [INFO] Worker exiting (pid: 10589)
[2024-01-25 11:28:04 +0800] [10588] [INFO] Worker exiting (pid: 10588)
[2024-01-25 11:28:04 +0800] [10590] [INFO] Worker exiting (pid: 10590)
[2024-01-25 11:28:04 +0800] [10584] [INFO] Worker exiting (pid: 10584)
[2024-01-25 11:28:04 +0800] [10591] [INFO] Worker exiting (pid: 10591)
[2024-01-25 11:28:04 +0800] [10586] [INFO] Worker exiting (pid: 10586)
[2024-01-25 11:28:05 +0800] [9734] [INFO] Shutting down: Master

Expected behavior
The flow app should respose with correct result
Screenshots
Running Information(please complete the following information):

Promptflow Package Version using pf -v: 1.3.0
Operating System: CentOS
Python Version: python3.10

Hi, @kevindany.  Thanks for reporting the issue. Could you please share your flow folder if possible?
And may I ask whether you deploy your flow by Dockerfile (pf flow deploy) or directly start the service with gunicorn command


I wish for a  component for next-routes

Since the it's so common to use next exclusively for ssr and we normally handle authentication from another server other than the next server, I think it would be cool to have a React component that lets one redirect to a page of their choice without bothering the next-server. Something like:
<Redirect to='/dashboard?usr=${id}'/>
Will be glad to here from the community soon.

We typically use an util function for that because Redirect is typically not something to render & it would make the DOM tree looks weird to us. Please find the attached function and see if it can fits your need:
// lib/utils/redirect.js
import Router from "next/router";

// redirect helper function to be used in getInitialProps
export default (res, url) => {
  // res is only available if server-side
  if (res) {
    res.writeHead(302, { location: url });
    res.end();
  } else {
    Router.push(url);
  }
};

With that file in utils, you can then call it with isomorphic support, i.e. In the case you'd like to redirect an user back to login if he is not logged in:
// _document.js
const loginRedirect = (req, res) => {
  const uri = new Url(req ? req.url : window.location, true)
  const redirectUri = uri.query.redirect || '/'
  redirect(res, redirectUri)
}

// withAuthRouter is a simple HOC that check if user is logged in via GraphQL
const withLoginRedirect = withAuthRouter(({ error, user, ctx }) => {
  if (!error && user) {
    loginRedirect(ctx.req, ctx.res)
  }
})

const DocumentPage = props => props.children
export default withLoginRedirect(DocumentPage);

Hope this helps.

Thank @WNemencha for this feedback.


Cant access dapr helm-charts

Hi,
When deploying dapr helm charts to kubernetes cluster, It refers to the url : https://dapr.github.io/helm-charts/
It is blocked for my company and I need to install it manually. Where can I get the helm charts ?
Regards,
Samiksha Jadhav

Hi @samikshaJ, please check https://github.com/dapr/helm-charts for the helm charts.


Rework Controller tests

Short Description
Add missing controller tests that are needed after the refactoring and adjust some existing ones.
This PR also reduces the log output, to ensure our log files do not exceed the travis limit of 10 MB.
To do this, stack traces are filtered out of the log output.

Current State of coverage, excluding lombok:


When will codes release ?

Thank you for your attention. The code is currently being organized, and we will release it as soon as possible.


chck in wlc doesn't seem to compile in Debian Stetch

I think it might be because Debian uses ver 5 by default, and ver 5 tries to use CPP-2014... ...I think
I'm able to get it to build by running
sed -i 's/-Wsuggest-attribute=const)/-Wsuggest-attribute=const -std=c11)/g' lib/chck/CMakeLists.txt
[  8%] Building C object lib/chck/chck/buffer/CMakeFiles/chck_buffer.dir/buffer.c.o
In file included from /srcbuild/wlc/lib/chck/chck/buffer/buffer.h:9:0,
from /srcbuild/wlc/lib/chck/chck/buffer/buffer.c:1:
/srcbuild/wlc/lib/chck/chck/buffer/endianess.h: In function ‘chck_bswap’:
/srcbuild/wlc/lib/chck/chck/buffer/endianess.h:108:4: error: ‘for’ loop initial declarations are only allowed in C99 or C11 mode
for (uint8_t p = v; p < (uint8_t)v + (memb * size); p += size)
^
/srcbuild/wlc/lib/chck/chck/buffer/endianess.h:108:4: note: use option -std=c99, -std=gnu99, -std=c11 or -std=gnu11 to compile your code
/srcbuild/wlc/lib/chck/chck/buffer/buffer.c: In function ‘chck_buffer_compress_zlib’:
/srcbuild/wlc/lib/chck/chck/buffer/buffer.c:409:39: warning: passing argument 2 of ‘compress’ from incompatible pointer type
while ((ret = compress(compressed, &dsize, buf->buffer, buf->size)) == Z_BUF_ERROR) {
^
In file included from /srcbuild/wlc/lib/chck/chck/buffer/buffer.c:8:0:
/usr/include/zlib.h:1160:21: note: expected ‘uLongf *’ but argument is of type ‘size_t *’
ZEXTERN int ZEXPORT compress OF((Bytef *dest,   uLongf *destLen,
^
/srcbuild/wlc/lib/chck/chck/buffer/buffer.c: In function ‘chck_buffer_decompress_zlib’:
/srcbuild/wlc/lib/chck/chck/buffer/buffer.c:461:43: warning: passing argument 2 of ‘uncompress’ from incompatible pointer type
while ((ret = uncompress(decompressed, &dsize, buf->buffer, buf->size)) == Z_BUF_ERROR) {
^
In file included from /srcbuild/wlc/lib/chck/chck/buffer/buffer.c:8:0:
/usr/include/zlib.h:1197:21: note: expected ‘uLongf *’ but argument is of type ‘size_t *’
ZEXTERN int ZEXPORT uncompress OF((Bytef *dest,   uLongf *destLen,
^
lib/chck/chck/buffer/CMakeFiles/chck_buffer.dir/build.make:54: recipe for target 'lib/chck/chck/buffer/CMakeFiles/chck_buffer.dir/buffer.c.o' failed
make[2]: *** [lib/chck/chck/buffer/CMakeFiles/chck_buffer.dir/buffer.c.o] Error 1
CMakeFiles/Makefile2:1090: recipe for target 'lib/chck/chck/buffer/CMakeFiles/chck_buffer.dir/all' failed
make[1]: *** [lib/chck/chck/buffer/CMakeFiles/chck_buffer.dir/all] Error 2
Makefile:147: recipe for target 'all' failed
make: *** [all] Error 2

Please post output without sedding the CMakeLists.txt, also related https://github.com/Cloudef/wlc/issues/62

You most likely have too old cmake. I've bumped the minimum to 3.1 which is needed for CMAKE_C_STANDARD. If you want to compile on older cmake, you need explicitly  -DCMAKE_C_FLAGS=-std=c99 when invoking cmake.


Added support for installing pushgateway

This role is too big for now.  I suggest that you do it in another separated role.
Thank you.


HOG detect face is more accurate than CNN

when I  use a pic from this link test face detector both of HOG and CNN
The result is CNN detect the hand as a face; but HOG is correct。
The img which detected by CNN lack the chin。
https://timgsa.baidu.com/timg?image&quality=80&size=b9999_10000&sec=1507896274082&di=824f7f59943a71e2e9904d22175ce92c&imgtype=0&src=http%3A%2F%2Fwww.moontalk.com.tw%2Fupload%2Fimages%2F20160606angelina-03.jpg

When I run the CNN detector on that image it works perfectly.  What CNN are you talking about?  I ran this example program on that image http://dlib.net/dnn_mmod_face_detection_ex.cpp.html.

I run the detector in Python environment. Install dlib via pip and the version is 19.7.0 .
The same problem occured when call the method cnn_face_detector.
I call dlib via this project
https://github.com/ageitgey/face_recognition/blob/master/examples/find_faces_in_picture_cnn.py

Use the python examples that come with dlib.

OK，I will try.

OK，I will try

I run cnn_face_detector example.The result is correct.
http://dlib.net/cnn_face_detector.py.html
Thank you

@ageitgey you might want to look at this.  There might be something wrong with your face detection wrapper.

@davisking Thanks. I'll take a look over at https://github.com/ageitgey/face_recognition/issues/209 and update it there.

@davisking If you change the dlib example http://dlib.net/cnn_face_detector.py.html to upsample 0 times (instead of the default 1), it detects the hand as an additional face with a very low confidence:
    # Pass in 0 instead of 1 for upsampling
    dets = cnn_face_detector(img, 0)

Result:
Processing file: testface.jpg
Number of faces detected: 2
Detection 0: Left: 258 Top: 211 Right: 494 Bottom: 446 Confidence: 1.072353720664978
Detection 1: Left: 357 Top: 558 Right: 521 Bottom: 722 Confidence: 0.042258620262145996

The demo file at https://github.com/ageitgey/face_recognition/blob/master/examples/find_faces_in_picture_cnn.py he was using just defaulted to zero upsampling instead of 1 - explaining the difference result you were seeing.
So dlib really is detecting the hand as a face with low confidence with this specific image and upsampling setting. I don't think that's a 'bug' per se, but just sharing the result.

Yeah, in that case then it's all good.  Thanks for checking.

Sure, no prob.

does dlib return the actual number of landmarks detected? For example, if a face is slightly turned, it will probably not be able to detect all 68, and will detect say 50. will it tell me how many landmarks found?
I have in code to ignore any detections of 45 or less, but no matter what dlib always returns 69 landmarks even on sideways faces. Is this a dlib issue? has anyone else faced this?

The shape_predictor always outputs the same number of landmarks, no matter what. It isn't detecting if they are there or not, it's finding the best way to deform the shape to the object present in the image.  If you want to know if a particular landmark is occluded or not you need to make some additional model to do that test.
It should be noted that you can easily do this by training a linear svm on the same features the shape_predictor uses.  They are available in the feats output of the shape_predictor.  That is, you train a binary classifier for each landmark to predict if it is occluded or not.  I've done this and it works fine, which is why the feats output is available in the API.


Add role assignment deployment tests

Overview/Summary
Replace this with a brief description of what this Pull Request fixes, changes, etc.
This PR fixes/adds/changes/removes

Fixes #51
Comment tidy
Role assignments now depends on other modules

Testing Evidence
Please provide testing evidence to show that your Pull Request works/fixes as described and documented above.

As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x] Run make docs and make fmt to update the documentation and format your code.
[x] Created unit and deployment tests and provided evidence.
[x] Updated relevant and associated documentation.
[ ] (Optionally) Updated the "What's New?" wiki page (located in the Enterprise-Scale repo in the directory: /docs/wiki/whats-new.md)

RoleAssignment deployment tests run in: https://github.com/Azure/terraform-azurerm-lz-vending/actions/runs/2662092198


fix(onboarding): show recover wallet passphrase form

Description:
When recovering a wallet wallet ensure that we show the passphrase form in the case that that there is an error due to a missing or invalid passphrase. Show an error dialog incase of all other errors.
Motivation and Context:
Unable to recover encrypted wallet
How Has This Been Tested?
Try recovering this encrypted wallet:
1. abstract   2. solution   3. ramp     4. repeat
5. sentence   6. soup       7. quick    8. trend
9. nation    10. cute      11. bridge  12. note
13. ritual    14. century   15. game    16. sponsor
17. sea       18. impact    19. game    20. wrist
21. follow    22. rail      23. lemon   24. curious

password: password
passphrase: passphrase

Types of changes:
Bug fix
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage remained the same at 20.147% when pulling 7ae13a5ca793ec9199dce4dcd17c16319f4adc95 on mrfelton:fix/promise-timeout into f57f918fc3be277498cd890fbe4189961131ca94 on LN-Zap:master.


Added larger set for org and user repo - fixes 1158

Allows for up to 1000 org images + user images to be displayed
Signed-off-by: French Ben<EMAIL_ADDRESS>

LGTM!


Handover of package ownership.

@fengsp
Let's get talking about where to have this package owned and maintained. You've done great work over the years but we need to find a future for this package.
@christopherpickering Would you consider merging your fork or something similar and taking over? is https://github.com/christopherpickering/flask-session2 the best current fork?
@davidism perhaps this might belong as part of the pallets family?

I have directly emailed Feng and hopefully he replies but are there any options if he doesn't?

@Lxstr I try to keep the flask-session2 fork updated. I would be happy to merge back or give to another developer who is interested in continuing its use.

We have a https://github.com/pallets-eco for community maintenance of extensions. You're welcome to transfer it there, you'll retain ownership but others will be able to commit, and Pallets maintainers will be able to push to PyPI.

@davidism thank you that sounds quite good. Is there also an ownership strategy if for whatever reason they go MIA?
This seems the appropriate time to do something as latest flask release has broken Feng's package and the issues will be streaming in now.
Also, I noticed that there has been some pollution of the flask-session name. A few years ago this was unmaintained for a period and a fork was run at mcrowson/flask-sessionstore-fork which is now archived but is still connected to the highest SEO ranking documentation as flasksession.readthedocs.io vs flask-session.readthedocs.io, to the great confusion of anyone new. That dev is also unreachable: https://github.com/fengsp/flask-session/issues/138.
Over at pyrebase a similar thing happened (pyrebase4, pirebase, etc) but we have recently moved forwards with firebase-rest-api, which is a more accurate description anyway. It has just required telling people in the new issues and some blogging.
Long story in short, I am floating the idea of @christopherpickering doing a move to pallets-eco and a name change to say flask-server-side-session for the long term sake. Is this too much or what are the thoughts?
I'm not 100% on the best way to do this but possibly archiving flask-sessions2 and putting a deprecation warning in the short term. Ideally, the same on flask-session if we ever hear back.
import warnings
warnings.warn("Flask-session2 deprecated, please use flask-server-side-session", DeprecationWarning)

I'd be willing to help with readthedocs and awareness of the new home and/or name if there is support for that.

@Lxstr Sorry for the late reply, It would be great if pallets-eco could help.

Some updates:

I would have a look at some breaking change required prs and merge them
In the same time, it would be great that pallets-eco could help so I won't be the bottleneck for future maintenance

@fengsp I've sent you an invite to the github org, then you'll be able to transfer the repo. I'll also need full permission on PyPI, Test PyPI, and Read the Docs, my username is davidism on all of them.
@Lxstr were you planning to contribute to help get the next version out? Just transferring it to pallets-eco isn't enough, we need to find the community to continue contributions.

@fengsp Thank you for replying to the email about transferring this repo to the pallets-eco. Thank you to @davidism for the quick reply to get things moving. I look forward to helping with this project where I can.

@davidism  Repo transfer completed. Full permission on PyPI and Read the Docs added. We do not use Test PyPI for now.

Thank you. For community members who are interested in contributing, please join us at https://discord.gg/pallets in the #pallets-eco channel.

If you'd like I can do a pr from flask-session2 which will close out many of the outstanding issues.

@christopherpickering @rayluo @Lxstr I've given you all write access. Together, we're still figuring out how pallets-eco will work. I've posted some ideas in Discord about first steps to making newly-added projects more maintainable, ping me there if you have any questions or suggestions.

Flask-Session is now part of Pallets-Eco, an open source organization managed by the Pallets team to facilitate community maintenance of Flask extensions. Anyone interested in helping maintain this project can ask for commit access in the #flask-session channel on https://discord.gg/pallets.


[5.3] Fail tests that don't pass on PHP 7.1

Now that PHP 7.1 is stable, we should allow its tests to fail.

@JosephSilber Would we need to fix the tests that fail on PHP 7.1?

@srmklive if we had failing tests, then yes. We currently don't, and it should stay that way.


FREE VPS =) // From 10Bots To 80 :D

Guys i'm making a video about free VPS :)
if u want to try it without tutorial
http://c9.io/
Creat an acc / use NodeJS / upload FEEDER / wait my video :package:
After That i will do how to update to PRO for free That mean u will get +150 without losing anyone
#PEACE

Create*
:p

How can.


refactor: fix transformer versions

Transformers 4.0.0 broke something

Some requirements conflicts

Rebase

Done! Thx!

This should be closed, because our torch encoder seems to work with 4.0.1

This should be closed, because our torch encoder seems to work with 4.0.1


Claude Haiku replaces Instant in builder

Description
Move to Haiku instead of Claude Instant 1.2 in Builder
Risk
N/A
Deploy Plan

deploy front

PMRW


⚠ ClamAV has degraded performance

In 2d0339b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in 61450ca.


Add installer for all different platforms

As title, especially for Windows user, we need to make an installer to let users easily install this application to their own storage easily.
https://github.com/maxogden/electron-packager/issues/33
https://github.com/loopline-systems/electron-builder
https://github.com/maxogden/electron-packager/issues/21

@EragonJ
Hi,
I guess it's related. When we launch the Kaku shortcut (with mouseover, I see the name "Electron") in the taskbar, another Kaku icon is launched
We don't have this duplicate issue with other apps.

What is kaku shortcut ?? I didn't get it because I am not windows user. Can you attach more screenshots about each step ? it would be helpful !

I used to be a threesome user because I was using OS X - Windows - GNU/Linux. But yes, for specific reasons, Windows 10 is now my main OS.

oh i see thanks @HLFH ! this looks like Electron's bug ! I'll find sometime to write a patch for it and will ask you to verify the bug ! thanks !!

Not in a high priority now, so close first.


requireBlocksOnNewline: Add object option to handle comments

Add object option { includeComments: true } and { includeComments: true,
minStatements: 1} that allows the rule to turn on or off the inclusion of
comments as part of the block being checked using exisitng rules. A new
config option is introduced to maintain compatability with existing code like
the crockford preset which ignores comments.
Fixes #1417

@markelog how about minLinesOfCode?

@markelog Had some free time today so I updated the PR to use minLinesOfCode instead of minStatements.

Would like to hear another opinion for the new name properties, @hzoo, @mrjoelkemp, @zxqfox?

I think statements are better here than lines of code, but guess entities (or something similar and abstract) are better.

@zxqfox we can't use "statement", since as i said, a lot of constructions that would be counted by this rule are not "statements" in terms of ECMAScript specification.

I will fix the line comments to become block comments before the closing braces and also restructure the options to follow the referenced documentation. As for the name I will wait for more feedback

If we also counter multilines inside multiline comment blocks then it could be just minLines (without OfCode). Good naming is always a hard part ;-)

Bikeshedding always a "hard part" as said in that popular proverb ;).
minLines sounds good to me

I am good with minLines as well but any suggestions the word statements in the documentation as well
specifies a minimum number of statements in the block before validating
Should I change the

constructions? entities?

elements?

nodes?
anyway, @oredi, choose you like best :-)

I will try elements, I am away from my PC. Will get to updating the PR in my Day. :)

Updated with changes from the comments

Thank you!


tests-ci : net: socket: tls.preempt test Failed

**Describe the bug**
tls.preempt test is Failed on zephyr-v3.2.0-818-g938b9533c727 on frdm_k64f

see logs for details

**To Reproduce**
1.
```
scripts/twister --device-testing --device-serial /dev/ttyACM0 -p frdm_k64f      --sub-test net.socket
```
2. See error

**Expected behavior**
test pass

**Impact**

**Logs and console output**

```
START - test_v4_msg_trunc

Assertion failed at WEST_TOPDIR/zephyr/tests/net/socket/tls/src/main.c:421: test_msg_trunc: (rv not equal to 0)
server bind failed
FAIL - test_v4_msg_trunc in 0.013 seconds
```

**Environment (please complete the following information):**
 - OS: (e.g. Linux )
 - Toolchain (e.g Zephyr SDK)
 - Commit SHA or Version used: zephyr-v3.2.0-818-g938b9533c727

@hakehuang, I can confirm that this test case passes with the current version of main. Via git bisect, the failure was introduced with #51240 and subsequently fixed with #51613


⚠ Bofrost.de has degraded performance

In 35729fa, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1567 ms

Resolved: Bofrost.de performance has improved in 6f51f6d after 8 minutes.


implement export registers feature

As discussed in https://github.com/eclipse-cdt-cloud/vscode-peripheral-inspector/issues/28, I do a PR to support the export registers feature.

Many file changes are just removing the last new line, could you revert those, please?

Looks like it's stopped building :(

Many file changes are just removing the last new line, could you revert those, please?

I finished rebasing after some pull requests were merged. Could you let me know about reviewing process?

If you could just delete the view/item/context block, I think we are good to merge 👍

All works as expected and all feedback has been resolved, I'm merging it.
Thanks for your work @QuocTrung76


dma: hda: dma channel status check

To avoid enabling power gaiting on HST power domain by mistake we need to be shore that DMA stop function is called only once per channel. PM device context is counting usage for entire HDA controller.
For example, in scenario with usage of two DMA channels, disabling twice the same channel will result with removing PG prevent on entire power domain, despite that the second channel will be still in use.

Closing, this https://github.com/zephyrproject-rtos/zephyr/pull/53504 resolve the same problem.


Move typechecker

What this PR does / why we need it:

Removes the explicit TypeChecker instantiation in the validator: It simply uses the type-checker on the validator.
Removes the cache from the TypeChecker, making it fully stateless.
Moves extern definitions from the type-checker/expr internals to the il/runtime, thus fully decoupling intrinsics from externs.
Breaks a dependency cycle caused by config/handler depending on type-checker and IL depending on config/handler by introducing a testing implementation of the type-checker to be used in the config/handler tests. This enables the next change below.
Moves the bulk of the TypeChecker code to Il/evaluator, creating a single set of entry methods for external code to consume, co-located together.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

NONE

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
53f92b7f1c06e6033ea12bb8ac83dc35ddc01cd2
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e

/lgtm

Your commit failed Itpostsubmit. All test failed, so it is ulikely to be a test infrastructure issue. @guptasu Could you help
https://k8s-gubernator.appspot.com/build/istio-prow/e2e-cluster_wide-auth/28/

It looks like it is a problem with my PR itself. I'll take a look.
Sunny, Feel free to roll it back in the meantime.
I1117 14:58:39.710772       1 registry.go:109] Registered quotas / noop builder
I1117 14:58:39.710759       1 server.go:96]
handler.prometheus.istio-system:serving prometheus metrics on :42422
proto: no coders for interface {}
proto: no encoder for Params interface {} [GetProperties]
proto: no coders for interface {}
proto: no encoder for Params interface {} [GetProperties]
Zipkin traces being sent to http://zipkin:9411/api/v1/spans
W1117 14:58:39.715313       1 descriptor.go:183] quotas missing
W1117 14:58:39.715334       1 descriptor.go:183] attributes missing
W1117 14:58:39.715342       1 descriptor.go:183] logs missing
W1117 14:58:39.715348       1 descriptor.go:183] metrics missing
W1117 14:58:39.715361       1 descriptor.go:183] monitoredResources missing
W1117 14:58:39.715368       1 descriptor.go:183] principals missing
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x14bcc2a]
goroutine 1 [running]:istio.io/istio/mixer/pkg/il/evaluator.(*attrContext).getOrCreateCacheEntry(0x0,
0xc4201db580, 0x1e, 0xc4203d4150, 0xc420470ca8, 0x40b130,
0xc420470e00, 0x0)
mixer/pkg/il/evaluator/evaluator.go:180
+0x5aistio.io/istio/mixer/pkg/il/evaluator.(*IL).EvalType(0xc420386140,
0xc4201db580, 0x1e, 0x7fde71ddb640, 0xc420cfda70, 0x1, 0x1, 0x0)
mixer/pkg/il/evaluator/evaluator.go:99
+0x79istio.io/istio/mixer/pkg/aspect.attrGenMgr.ValidateConfig(0x21fd700,
0xc4201c3770, 0x7fde71dda370, 0xc420386140, 0x2209ce0, 0xc420cfda70,
0x21fd6c0)
mixer/pkg/aspect/attrgenmgr.go:59
+0x163istio.io/istio/mixer/pkg/aspect.(*attrGenMgr).ValidateConfig(0x2295318,
0x21fd700, 0xc4201c3770, 0x7fde71dda370, 0xc420386140, 0x2209ce0,
0xc420cfda70, 0x1873b4e)
:1
+0x70istio.io/istio/mixer/pkg/config.convertAspectParams(0xc4202ca1e0,
0xc420c1de90, 0xa, 0x1672e80, 0xc420b86660, 0x1, 0x2209ce0,
0xc420cfda70, 0x7fde71dda370, 0xc420386140, ...)
mixer/pkg/config/validator.go:814
+0x23cistio.io/istio/mixer/pkg/config.(*validator).validateAspectRules(0xc4204713b0,
0xc420b65a40, 0x1, 0x4, 0x1873b4e, 0x1, 0x1, 0x1437450, 0xc420b654d0)
mixer/pkg/config/validator.go:377
+0x12fistio.io/istio/mixer/pkg/config.(*validator).validateServiceConfig(0xc4204713b0,
0xc420436ca1, 0x6, 0xc420436cb1, 0x6, 0xc42009b200, 0x44f, 0x1,
0x179f180)
mixer/pkg/config/validator.go:670
+0x1a5istio.io/istio/mixer/pkg/config.(*validator).validate(0xc4204713b0,
0xc420ac7b90, 0xc420ac7e90, 0x0)
mixer/pkg/config/validator.go:551
+0x651istio.io/istio/mixer/pkg/config.NewManager.func1(0xc420ac7b90,
0xc420b1e500, 0x1873b4e, 0x1, 0xc420ac7b90)
mixer/pkg/config/manager.go:104
+0x509istio.io/istio/mixer/pkg/config.(*Manager).fetch(0xc4203206c0,
0x7fde71d93a18, 0x0, 0x7fde71e7de50, 0xc420068000, 0x7fde71ed3d90,
0x0)
mixer/pkg/config/manager.go:175
+0x109istio.io/istio/mixer/pkg/config.(*Manager).fetchAndNotify(0xc4203206c0)
mixer/pkg/config/manager.go:192
+0x40istio.io/istio/mixer/pkg/config.(*Manager).Start(0xc4203206c0)
mixer/pkg/config/manager.go:231
+0x2fistio.io/istio/mixer/cmd/server/cmd.setupServer(0xc4201b22d0,
0xc420423350, 0xc42007f4f0, 0x9, 0x9, 0xc4204eb910, 0x2, 0x2,
0x1904f40, 0x1904f38, ...)
mixer/cmd/server/cmd/server.go:384
+0x1235istio.io/istio/mixer/cmd/server/cmd.runServer(0xc4201b22d0,
0xc420423350, 0xc42007f4f0, 0x9, 0x9, 0xc4204eb910, 0x2, 0x2,
0x1904f40, 0x1904f38)
mixer/cmd/server/cmd/server.go:423
+0x12cistio.io/istio/mixer/cmd/server/cmd.serverCmd.func2(0xc4201d3200,
0xc42038e460, 0x0, 0x7)
mixer/cmd/server/cmd/server.go:143
+0x7bgithub.com/spf13/cobra.(*Command).execute(0xc4201d3200,
0xc42038e3f0, 0x7, 0x7, 0xc4201d3200, 0xc42038e3f0)
external/com_github_spf13_cobra/command.go:654
+0x2a2github.com/spf13/cobra.(*Command).ExecuteC(0xc4201d2480,
0xc4201d3680, 0xc4201d3200, 0xc4201d2900)
external/com_github_spf13_cobra/command.go:729
+0x2fegithub.com/spf13/cobra.(*Command).Execute(0xc4201d2480, 0x8,
0x8)
external/com_github_spf13_cobra/command.go:688 +0x2b
On Fri, Nov 17, 2017 at 7:34 AM, Sebastien Vas<EMAIL_ADDRESS>wrote:

Your commit failed Itpostsubmit. All test failed, so it is ulikely to be a
test infrastructure issue. @guptasu https://github.com/guptasu Could
you help
https://k8s-gubernator.appspot.com/build/istio-prow/
e2e-cluster_wide-auth/28/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/1760#issuecomment-345276219, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQR8I03W84KSB_HZeGq8u7HgdkgGVKwhks5s3adzgaJpZM4QhOZW
.

Fix is here:
https://github.com/istio/istio/pull/1765
Trying to validate locally.
On Fri, Nov 17, 2017 at 8:40 AM, Oz Evren<EMAIL_ADDRESS>wrote:

It looks like it is a problem with my PR itself. I'll take a look.
Sunny, Feel free to roll it back in the meantime.
I1117 14:58:39.710772       1 registry.go:109] Registered quotas / noop builder
I1117 14:58:39.710759       1 server.go:96] handler.prometheus.istio-system:serving prometheus metrics on :42422
proto: no coders for interface {}
proto: no encoder for Params interface {} [GetProperties]
proto: no coders for interface {}
proto: no encoder for Params interface {} [GetProperties]
Zipkin traces being sent to http://zipkin:9411/api/v1/spans
W1117 http://zipkin:9411/api/v1/spansW1117 14:58:39.715313       1 descriptor.go:183] quotas missing
W1117 14:58:39.715334       1 descriptor.go:183] attributes missing
W1117 14:58:39.715342       1 descriptor.go:183] logs missing
W1117 14:58:39.715348       1 descriptor.go:183] metrics missing
W1117 14:58:39.715361       1 descriptor.go:183] monitoredResources missing
W1117 14:58:39.715368       1 descriptor.go:183] principals missing
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x14bcc2a]
goroutine 1 [running]:istio.io/istio/mixer/pkg/il/evaluator.(*attrContext).getOrCreateCacheEntry(0x0, 0xc4201db580, 0x1e, 0xc4203d4150, 0xc420470ca8, 0x40b130, 0xc420470e00, 0x0)
mixer/pkg/il/evaluator/evaluator.go:180 +0x5aistio.io/istio/mixer/pkg/il/evaluator.(*IL).EvalType(0xc420386140, 0xc4201db580, 0x1e, 0x7fde71ddb640, 0xc420cfda70, 0x1, 0x1, 0x0)
mixer/pkg/il/evaluator/evaluator.go:99 +0x79istio.io/istio/mixer/pkg/aspect.attrGenMgr.ValidateConfig(0x21fd700, 0xc4201c3770, 0x7fde71dda370, 0xc420386140, 0x2209ce0, 0xc420cfda70, 0x21fd6c0)
mixer/pkg/aspect/attrgenmgr.go:59 +0x163istio.io/istio/mixer/pkg/aspect.(*attrGenMgr).ValidateConfig(0x2295318, 0x21fd700, 0xc4201c3770, 0x7fde71dda370, 0xc420386140, 0x2209ce0, 0xc420cfda70, 0x1873b4e)
:1 +0x70istio.io/istio/mixer/pkg/config.convertAspectParams(0xc4202ca1e0, 0xc420c1de90, 0xa, 0x1672e80, 0xc420b86660, 0x1, 0x2209ce0, 0xc420cfda70, 0x7fde71dda370, 0xc420386140, ...)
mixer/pkg/config/validator.go:814 +0x23cistio.io/istio/mixer/pkg/config.(*validator).validateAspectRules(0xc4204713b0, 0xc420b65a40, 0x1, 0x4, 0x1873b4e, 0x1, 0x1, 0x1437450, 0xc420b654d0)
mixer/pkg/config/validator.go:377 +0x12fistio.io/istio/mixer/pkg/config.(*validator).validateServiceConfig(0xc4204713b0, 0xc420436ca1, 0x6, 0xc420436cb1, 0x6, 0xc42009b200, 0x44f, 0x1, 0x179f180)
mixer/pkg/config/validator.go:670 +0x1a5istio.io/istio/mixer/pkg/config.(*validator).validate(0xc4204713b0, 0xc420ac7b90, 0xc420ac7e90, 0x0)
mixer/pkg/config/validator.go:551 +0x651istio.io/istio/mixer/pkg/config.NewManager.func1(0xc420ac7b90, 0xc420b1e500, 0x1873b4e, 0x1, 0xc420ac7b90)
mixer/pkg/config/manager.go:104 +0x509istio.io/istio/mixer/pkg/config.(*Manager).fetch(0xc4203206c0, 0x7fde71d93a18, 0x0, 0x7fde71e7de50, 0xc420068000, 0x7fde71ed3d90, 0x0)
mixer/pkg/config/manager.go:175 +0x109istio.io/istio/mixer/pkg/config.(*Manager).fetchAndNotify(0xc4203206c0)
mixer/pkg/config/manager.go:192 +0x40istio.io/istio/mixer/pkg/config.(*Manager).Start(0xc4203206c0)
mixer/pkg/config/manager.go:231 +0x2fistio.io/istio/mixer/cmd/server/cmd.setupServer(0xc4201b22d0, 0xc420423350, 0xc42007f4f0, 0x9, 0x9, 0xc4204eb910, 0x2, 0x2, 0x1904f40, 0x1904f38, ...)
mixer/cmd/server/cmd/server.go:384 +0x1235istio.io/istio/mixer/cmd/server/cmd.runServer(0xc4201b22d0, 0xc420423350, 0xc42007f4f0, 0x9, 0x9, 0xc4204eb910, 0x2, 0x2, 0x1904f40, 0x1904f38)
mixer/cmd/server/cmd/server.go:423 +0x12cistio.io/istio/mixer/cmd/server/cmd.serverCmd.func2(0xc4201d3200, 0xc42038e460, 0x0, 0x7)
mixer/cmd/server/cmd/server.go:143 +0x7bgithub.com/spf13/cobra.(*Command).execute(0xc4201d3200, 0xc42038e3f0, 0x7, 0x7, 0xc4201d3200, 0xc42038e3f0)
external/com_github_spf13_cobra/command.go:654 +0x2a2github.com/spf13/cobra.(*Command).ExecuteC(0xc4201d2480, 0xc4201d3680, 0xc4201d3200, 0xc4201d2900)
external/com_github_spf13_cobra/command.go:729 +0x2fegithub.com/spf13/cobra.(*Command).Execute(0xc4201d2480, 0x8, 0x8)
external/com_github_spf13_cobra/command.go:688 +0x2b
On Fri, Nov 17, 2017 at 7:34 AM, Sebastien Vas<EMAIL_ADDRESS>wrote:

Your commit failed Itpostsubmit. All test failed, so it is ulikely to be
a test infrastructure issue. @guptasu https://github.com/guptasu Could
you help
https://k8s-gubernator.appspot.com/build/istio-prow/e2e-
cluster_wide-auth/28/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/1760#issuecomment-345276219, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQR8I03W84KSB_HZeGq8u7HgdkgGVKwhks5s3adzgaJpZM4QhOZW
.

I ll revert it

There is a fix I already sent out. Can we try it first
On Nov 17, 2017 10:02 AM, "Sebastien Vas"<EMAIL_ADDRESS>wrote:

I ll revert it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/1760#issuecomment-345317707, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQR8I66DgsVKMrut6eETlajqbpJSt5r4ks5s3coHgaJpZM4QhOZW
.


[Oreilly] update the web url to match the new location

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[ ] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[ ] Checked the code with flake8

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Explanation of your pull request in arbitrary form goes here. Please make sure the description explains the purpose and effect of your pull request and is worded well enough to be understood. Provide as much context and examples as possible.

Tested this with my credentials and it's working both for individual video pages and course page to download all videos.

Tested this with my credentials and it's working both for individual video pages and course page to download all videos.
Can you confirm it works using either cookies text or imported from a browser? thanks

Can you confirm it works using either cookies text or the import from browser option? thanks

Passed the credentials as --username and --password arguments.


avalibility-zone-for-vm-in-openstack

Добавлена возможность через конфиг выбирать, в какой зоне vmmaster будет создавать виртуалки(т.е. на какой группе нод)
Если по дефолту задан пустой строкой, то будет создавать в зоне по умолчанию, которую сам openstack проставит.

!test

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://jenkins.test//job/vmmaster.staging.regression/139/

!test

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://jenkins.test//job/vmmaster.staging.regression/141/

!test

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://jenkins.test//job/vmmaster.staging.regression/142/

!test

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://jenkins.test//job/vmmaster.staging.regression/143/

!test

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://jenkins.test//job/vmmaster.staging.regression/144/

:+1:


Add build branch to version information

Enhance the information displayed by the version command, to minimize any confusion about built code.

Example output:
$ ./influxd version
InfluxDB v1.0 (git: master 1234)

@toddboom

I wonder if we should add that to the http headers as well, or maybe that's getting to noisy.  Currently we send down the version header with each request, but not the build:
https://github.com/influxdb/influxdb/blob/master/services/httpd/handler.go#L674
Thoughts?

@corylanou -- I think that would be too noisy. I think just the version is the right thing to do for HTTP headers, but I don't feel that strongly about it.

I don't feel that strongly either.  If they really need it, it will be in the log from when they start.

@otoolep @corylanou I could see adding the full information into the HTTP headers when hitting the /ping endpoint, otherwise it seems unnecessary. If it's not trivial to do it only for the ping endpoint then I say leave it out for now.

@corylanou -- does this get a +1 from you?

+1


rpc: implement getstorage RPC

Closes #343 .

Also, documentation should be updated.

Care to add some tests for it?

@roman-khimov The easiest way to do it (just put in storage and test RPC) requires exporting makeStorageItemKey or newDao which is not the right way to do it IMO.
I will postpone it until #675 will be merged and there will be some storage in test blocks already.

Updated PR with new testchain and tests.


Broken link on left hand navigation - Choose between MDM and Intune

The link to Choose between MDM and Intune directs to an incorrect URL.
Currently it links to https://docs.microsoft.com/en-us/SecurityCompliance/choose-between-mdm-and-intune.md
I assume it's meant to link to this page.
https://docs.microsoft.com/en-us/office365/securitycompliance/choose-between-mdm-and-intune

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

ID: dcdf56d0-5b7a-f0df-86c2-14ce1c067514
Version Independent ID: 76a8737d-8f21-56ae-93f5-5d7942813484
Content: Subscriptions, licenses, accounts, and tenants for Microsoft's cloud offerings
Content Source: Enterprise/subscriptions-licenses-accounts-and-tenants-for-microsoft-cloud-offerings.md
Service: o365-solutions
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@markaldridge and @AndreaBarr I believe this is fixed. Thank you so much for your feedback and help.


Make kubefed webhook replica count configurable for High Availability…

… (HA) and made default as 2 replicas
Added in description to the README.md
Bumped up the chart version of the controller manager
Bumped up the dependency version
What this PR does / why we need it: This PR gives the users an option to set the number of replicas for the Kubefed Admission Webhook
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1456
Special notes for your reviewer:
Tested it out as follows in a local kind cluster
$ kubectl get pods -A
NAMESPACE                NAME                                             READY   STATUS    RESTARTS   AGE
...
kube-federation-system   kubefed-admission-webhook-67fb48d5f7-kw4ps       1/1     Running   0          4d17h
...

helm upgrade dry-run
$ helm --namespace kube-federation-system upgrade -i kubefed ./kubefed --create-namespace --dry-run
...
# Source: kubefed/charts/controllermanager/templates/deployments.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  namespace: kube-federation-system
  name: kubefed-admission-webhook
  labels:
    kubefed-admission-webhook: "true"
spec:
  replicas: 2
  selector:
    matchLabels:
...

helm upgrade
$ helm --namespace kube-federation-system upgrade -i kubefed ./kubefed --create-namespace
Release "kubefed" has been upgraded. Happy Helming!
NAME: kubefed
LAST DEPLOYED: Tue Sep 14 10:33:35 2021
NAMESPACE: kube-federation-system
STATUS: deployed
REVISION: 13
TEST SUITE: None

$ kubectl get pods -A
NAMESPACE                NAME                                             READY   STATUS    RESTARTS   AGE
...
kube-federation-system   kubefed-admission-webhook-67fb48d5f7-kw4ps       1/1     Running   0          4d17h
kube-federation-system   kubefed-admission-webhook-67fb48d5f7-r4vf5       1/1     Running   0          93s
...

Unfortunately, I'm not the reviewer of kubefed, but the change looks good to me.

Unfortunately, I'm not the reviewer of kubefed, but the change looks good to me.

Ok not a problem, thanks though.
Guess i'll have to wait then.
Nitin

Closing this as the branch doesn't exist and is now replaced by https://github.com/kubernetes-sigs/kubefed/pull/1469
Sorry


Find a mistake

In detector.py,  your possitive anchors are defined as is_positive_anchor = tf.greater_equal(anchor_matches, 0)Line 138, so don't include negetive sample. You use it in function positive_negative_subsample, in this function  mistake appears here,in fact after the logical_and operation, is_positive  becomes indicator, so error happens. but is_negative works correctly, to delete line 53 can fix the error.

Find another mistake, in your model.py Line 38, you use features['images_size'], however i can't find the feature images_size in the features defined in pipeline.py, so i believe it is a mistake.

In detector.py, your possitive anchors are defined as is_positive_anchor = tf.greater_equal(anchor_matches, 0)Line 138, so don't include negetive sample. You use it in function positive_negative_subsample, in this function mistake appears here,in fact after the logical_and operation, is_positive becomes indicator, so error happens. but is_negative works correctly, to delete line 53 can fix the error.

Have you tried it ? Did it improve the performance?


[BUG] epicli upgrade fails when upgrading Kibana

Describe the bug
There is a fatal error when upgrading Kibana:
TASK [kibana : Create log directory for Kibana]
FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'kibana_log_dir'\n\nThe error appears to be in '/shared/build/06todevawrhcalico/ansible/roles/kibana/tasks/setup-logging.yml': line 2, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n---\n- name: Create log directory for Kibana\n  ^ here\n"}

To Reproduce
Steps to reproduce the behavior:

create cluster using epicli 0.6
run epicli upgrade using a build from develop branch

Additional context
It looks like defaults from core\src\epicli\data\common\ansible\playbooks\roles\kibana\defaults\main.yml are not loaded and specification.versions[ansible_os_family].opendistro is loaded from roles/opendistro_for_elasticsearch/defaults/main.yml.

Unable to test - getting error during upgrade:
ERROR epicli - No such attribute: cloud

Similar issue what we have with task: #1647
Bug already created: #1741

Fix merged into develop #1750

Tested on Ubuntu / Azure.

Blocked on RedHat (see blockers)

Upgrading Kibana also fails for v0.4.5 since there is no opendistroforelasticsearch-kibana package but legacy kibana-oss.
2020-10-13T17:03:00.7317205Z 17:03:00 INFO cli.engine.ansible.AnsibleCommand - TASK [upgrade : Open Distro for Elasticsearch - Kibana | Assert that opendistroforelasticsearch-kibana package is installed] ***
2020-10-13T17:03:00.7671398Z 17:03:00 ERROR cli.engine.ansible.AnsibleCommand - fatal: [ec2-18-202-18-39.eu-west-1.compute.amazonaws.com]: FAILED! => {"assertion": "ansible_facts.packages['opendistroforelasticsearch-kibana'] is defined", "changed": false, "evaluated_to": false, "msg": "opendistroforelasticsearch-kibana package not found, nothing to upgrade"}

Tested on RedHat / Azure - OK
Before upgrade:

After upgrade:


upgrade(package): Update udif 0.9.0 -> 0.10.0

This updates udif from 0.9.0 to 0.10.0, fixing two bugs;

57ddf788 fix(readstream): Avoid read-before-open situations
6333a183 fix(image): try/catch parsing the resource fork plist

Change-Type: patch
Connects To: #1572

@jhermsmeier Apologies in advance for sounding like a broken record, but is there any way that the bugs these fix could be added to the unit-tests? (either on Etcher and/or udif, whichever is most appropriate)

@lurch haha, no worries – the funny thing is that there are tests in both, Etcher and udif, that make use of the readStream (all those dmg image tests, i.e.) in the same way – but none of them experience this particular case.

I'll see if I can somehow reproduce this race-condition reliably outside of Etcher, and then add that to udif, but my hopes of managing that are quite low (due to how the fs read requests work).


Incorrect locked balance

Here it says the balance is locked when I believe it is not:
https://westmint.statescan.io/account/5DvoL2BNoSm7wRt2tfZ6WW5QFrxm68GLv5SCrPQ4JBLjbvpL?tab=extrinsics
but I don't believe it is. The collator deposit of 10 WND gets unlocked when the collator gets kicked (This happens on a new session) here:
https://westmint.statescan.io/event/1722600-1
I suspect this is a bit of an edge case that statescan is possibly not expecting?
Thank you btw for the https://westmint.statescan.io/events?page=1&module=collatorSelection page - this was most helpful for diagnostics.

Hey @gilescope , thanks a lot for reporting this issue. Sure you're right, I checked the address balance and there is no lock.

Actually we haven't handle the cases with collators, but will add the logics ASAP. Thanks again.

We fixed westmint with the latest version, but it seems this account is added as curator again. So there is still 10 WND reserved.


added WIP for pattern matching

Still need to add tests but I think I got the core functionality working now


adds faceplusplus

a face detection tool

Can you squash this into a single commit without the added whitespace?


Stack deploy of windows containers get "mount error"

Bug description
We have set a swarm cluster with 4 linux managers and 4 windows nodes (2 with 1709 version and 2 with 1803). The linux machines are running docker version 18.03.1-ce while windows 17.06.2-ee-12.
The point is, when i use the stack option of portainer to deploy a stack it works perfectly for linux containers, but when I try to create a stack deploy for a windows node I get error on mount volumes: "invalid bind mount source, must be an absolute path..."
Expected behavior
We expected that the stack start working and all mount volumes be assign correctly like when we run the command "docker service create".
Steps to reproduce the issue:
Steps to reproduce the behavior:

Go to Portainer dashboard (running on a linux manager), click on stack menu, add stack button, fill with the stack name and paste the stack deploy yml on the filed and hit "deploy the stack".

Click on the new created stack
The command:

version: "3"
services:
  centralservicos:
    image: ourinternalimage-windows1803
    volumes:
      - E:\Docker\docker-data\Central\CentralServicos\Anexos:C:\Websites\Prod_CentralServicos\CentralServicos\Anexos
      - E:\Docker\docker-data\Central\GestaoRH\Arquivos:C:\Websites\Prod_CentralServicos\GestaoRH\Arquivos
      - E:\Docker\docker-data\Central\CentralServicos\Log:C:\Websites\Prod_CentralServicos\CentralServicos\Log
    deploy:
      labels:
      - traefik.enable=true
      - traefik.docker.network=traefik_traefik
      - traefik.port=80
      - traefik.backend=centralservicos
      - traefik.backend.loadbalancer.sticky=true
      - traefik.frontend.passHostHeader=true
      - traefik.frontend.rule=Host:ourinternalurl.teste
      - traefik.frontend.entryPoints=http
      placement:
        constraints:
        - node.labels.so==windows
        - node.labels.no==1
    networks:
      - traefik
networks:
  traefik:
    external: true

Scroll down to the container status and you will see only rejected status. Click in the last one to see the logs

See error
ID | kr441b92qb06pq3qonomqq3d2
-- | --
State | rejected
State Message | resolving controller failed
Error message | invalid bind mount source, must be an absolute path: /data/compose/centralservicos_6iszgratbstqts1eeccr27tdl/E:\Docker\docker-data\Central\CentralServicos\Anexos
Image | ourinternalimage-windows1803
Slot | 1
Created | 2018-06-14 11:00:05

Technical details:

Portainer version: 1.17.1
Docker version (managed by Portainer): linux 18.03.1-ce, windows 17.06.2-ee-12
Platform (windows/linux): both
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):

version: "3"
services:
  portainer:
    image: portainer/portainer
    environment:
      - "constraint:node.role==manager"
      - "secret portainer-pass"
      - "admin-password-file '/run/secrets/portainer-pass'"
      - "-H unix:///var/run/docker.sock"
    volumes:
      - /mnt/portainer:/data
      - /var/run/docker.sock:/var/run/docker.sock
    deploy:
      labels:
      - traefik.enable=true
      - traefik.docker.network=traefik_traefik
      - traefik.port=9000
      - traefik.backend=portainer
      - traefik.backend.loadbalancer.sticky=true
      - traefik.frontend.passHostHeader=true
      - traefik.frontend.rule=Host:url-to-our-portainer
    networks:
      - traefik
networks:
  traefik:
    external: true

Additional context
Add any other context about the problem here.

Hi @danielfrancora
Have you tried it on the Docker CLI ? I believe that this is an issue in your compose file. Could you try with the extended volume syntax?
Example:
    volumes:
      - type: bind
        source: E:\Docker\docker-data\Central\CentralServicos\Anexos
        target: C:\Websites\Prod_CentralServicos\CentralServicos\Anexos
      - type: bind
        source: E:\Docker\docker-data\Central\GestaoRH\Arquivos
        target: C:\Websites\Prod_CentralServicos\GestaoRH\Arquivos
      - type: bind
        source: E:\Docker\docker-data\Central\CentralServicos\Log
        target: C:\Websites\Prod_CentralServicos\CentralServicos\Log

Hi @deviantony
I have tried the extended syntax, but portainer shows me an "floating" error message (on the right corner, as a red ballon): "volumes must be a string".

Hey @danielfrancora
The extended (or long) syntax requires to use Compose format version v3.2: https://docs.docker.com/compose/compose-file/#long-syntax-3

Hey again @deviantony
I've made the changed you comment (now the stack is deployed sucefully), but we get the same error:

The stack:
version: "3.2"
services:
centralservicos:
image: private
volumes:
- type: bind
source: E:\Docker\docker-data\Central\CentralServicos\Anexos
target: C:\Websites\Prod_CentralServicos\CentralServicos\Anexos
- type: bind
source: E:\Docker\docker-data\Central\GestaoRH\Arquivos
target: C:\Websites\Prod_CentralServicos\GestaoRH\Arquivos
- type: bind
source: E:\Docker\docker-data\Central\CentralServicos\Log
target: C:\Websites\Prod_CentralServicos\CentralServicos\Log
deploy:
labels:
- traefik.enable=true
- traefik.docker.network=traefik_traefik
- traefik.port=80
- traefik.backend=centralservicos
- traefik.backend.loadbalancer.sticky=true
- traefik.frontend.passHostHeader=true
- traefik.frontend.rule=Host:teste.com
- traefik.frontend.entryPoints=http
placement:
constraints:
- node.labels.so==windows
- node.labels.no==1
networks:
- traefik
networks:
traefik:
external: true

Something is wrong with your Compose file... I'd suggest you to have a look at the compose docs or ask for help in an official Docker support as this is more related to Docker than Portainer.

The point is tat this compose files refers to a windows docker host, as the node.labels.so refer. The compose files created in the same way for linux hosts works perfectly. I'm expecting this kind of problem when Portainer trys to create a stack that is not in a linux host, but in a windows host.
The constraints its been respected, because the container is being assigned to sw-per01, who is a windows host.
But I'll follow your sugestion.
Thx in advance.

Portainer uses docker stack deploy behind the scenes to deploy the stack. So I think that this is related to an incorrect usage of stacks on Windows, you might want to dig more information about this.
Also do no hesitate to report your findings here !

I think I found the problem, but I don't know why it happen.
When I deploy the stack posted bellow, with you sugestion, look what happen with the mount types in Portainer:

When I correct them manually by the portainer interface the container go up normally.
AND when I create the container by the command line  E:\Docker\docker-data\Central> docker stack deploy -c .\docker-compose.yml centralservicos it works perfectly.
So, something is wrong in portainer when working with windows stuff.

Hmm, thanks for the report.
What is the version of the Docker CLI that you're using?

On linux distro: Docker version 18.03.1-ce, build 9ee9f40
On Windows distro: Docker version 17.06.2-ee-12, build fb5da67

AND when I create the container by the command line E:\Docker\docker-data\Central> docker stack deploy -c .\docker-compose.yml centralservicos it works perfectly.

So when you use docker stack on the Windows host, it's working. What happens when you use it on a Linux host?
E.g.:
$ docker -H windowshost.domain:2375 stack deploy -c .\docker-compose.yml centralservicos

@deviantony
Good morning.
Sorry for the delay.
I've tryd many forms to try your suggestion and I can't get it to work.
When I run the command docker -H sw-per01:2375 stack deploy -c E:\\Docker\\docker-data\\Central\\docker-compose.yml centralservicos against one linux manager (sw-per01 is a windows manager) I get the following error:
open \E:\Docker\docker-data\Central\docker-compose.yml: no such file or directory
I also tryed many different ways to specify the path with no luck.

Hi @danielfrancora
The compose file must be located on the host where you are executing the docker -H ... command.

Hi @deviantony
Yes, the path e:\Docker\docker-data\Central\docker-compose.yml is locates in sw-per01 that is a windows manager.
I'm running the command docker -H sw-per01:2375 stack deploy -c E:\\Docker\\docker-data\\Central\\docker-compose.yml centralservicos from sw-per06 that is a linux manager.
So, I'm running the command from a linux host against a windows host and I get that file not found.... but, If I copy the comand and remove the -H part and run it from the windows host it works perfectly.

Yes, the path e:\Docker\docker-data\Central\docker-compose.yml is locates in sw-per01 that is a windows manager.

The compose file must be located on the same host where you execute the command (sw-per06), try to copy the file on this host and execute the command again?

Sorry for the delay, but I have some problems and coudn't put ir to work.
I placed the compose-file on linux node and tryed to execute again the command, but I'm expecting the error:
`[root@sw-per06 centralservicos]# docker -H fd://<IP_ADDRESS>:2377 stack deploy -c docker-compose.yml centralservicos
Get http://<IP_ADDRESS>:2377/v1.37/info: net/http: HTTP/1.x transport connection broken: malformed HTTP response "\x15\x03\x01\x00\x02\x02".

Are you trying to connect to a TLS-enabled daemon without TLS?
Even all my swarm machines (4 windows and 4 linux) are in the enterprise domain I'm getting this message and I can't figure out how to disable it or put it to work with TLS. Portainer works fine because use the-H unix:///var/run/docker.sock` statement.

Sorry, I trying to put it to work.

Hey @danielfrancora
Standard TLS port should be 2376 not 2377 (which is usually used by Swarm for internal communication).
Anyway, if you're sure about this, you'll need the TLS certificates that you used when creating the endpoint inside portainer to use the CLI like this: docker --tlsverify --tlscacert=ca.pem --tlscert=cert.pem --tlskey=key.pem -H ....
See https://docs.docker.com/engine/security/https/ for more info.

Hello @deviantony
Sorry for the newbie answer but I created the portainer like the stack I posted here at the beginning, and I didnt use any certificate when I created the endpoint inside portainer, I use the -H unix:///var/run/docker.sock to put it to work.
I've tryed using my cert from godaddy and didn't work. I'll try with my AD CA.

All right, thanks for trying.
I'll tag this as a bug although I do not have the means to reproduce it at the moment. I'll keep it open as the investigation is not over.

@danielfrancora I just tested of combination of Win Srv 1803 + Docker 18.03.1-ee-1 and Portainer 1.18.1 with this stack and I can tell that it works just fine:
version: '3.3'

networks:
  test:
    driver: overlay

services:
  win:
    image: microsoft/nanoserver:1803
    volumes:
      - C:\test:C:\test
    networks:
      - test
    deploy:
      endpoint_mode: dnsrr
      placement:
        constraints:
          - node.platform.os==windows
    command: ping -t <IP_ADDRESS>

Side note: Both Win 1803 and  Docker 18.03.1-ee-1 contained lot of fixes to multiple issues so it is highly recommended to update to these versions.

@olljanat I tried your stack at Docker 18.09.5, Portainer 1.20.2 on swarm with 3 Ubuntu managers and 3 Windows 2019 workers. Still mount error with "absolute path". Do you have any idea? I came to a property COMPOSE_CONVERT_WINDOWS_PATHS but haven't found any docs about it...

@mashbrno do you have that docker version on all the nodes?
What I quickly tested on single node swarm on Win Srv 2019 it looks to be that stack my earlier message is valid as long you just update image to image: mcr.microsoft.com/windows/nanoserver:1809

@olljanat Yes all my nodes have that docker version. Wasn't yesterday, so I did a little upgrade and destroyed whole swarm. There is a warning missing when demoting Leader!
The problem persists /data/compose/9/C:\test might be that my swarm is hybrid contains linux managers and Windows workers.  Could you please help me with it via gitter?

@mashbrno I don't use gitter but you can reach me from Portainer Slack.

@danielfrancora @deviantony FYI
We investigated this one with @mashbrno and find out that this looks to be bug on Docker side which is already reported to https://github.com/moby/moby/issues/34810
So deploying above stack will works if you have Windows node as swarm manager and you use it to deploy stack.

Closing as this looks to be a bug in Docker as reported above.


Adding orchestral/testbench 3.2/3.3/3.4/3.5 versions

Hello,
In the objective to support laravel 5.5 (https://github.com/Maatwebsite/Laravel-Excel/pull/1259)
i'm adding 3.2/3.3/3.4/3.5 versions of orchestral/testbench in require-dev
thanks.

Please be aware that the tests are not passing.

Yep I'm aware. Probably because travis doesn't use the phpunit from the vendor folder. Will fix that myself. Thanks for your PR.


add fund_pledged_at and textable to archived patients

I rule and have completed some work on Case Manager that's ready for review!
These fields aren't populating - I think it's because archived patient is missing a few fields. This adds two of them that I noticed, textable and fund_pledged_at.
This is blocking a deploy so I'm going to smash it.
This pull request makes the following changes:

add some fields that patient has to archivedpatient

no view changes
no issues

smashing this in to get the build green


http: clear hop by hop TE header

Changes Envoy to clear TE header by default, as it is a hop by hop header and should not be forwarded on.
Risk Level: medium (behavioral change)
Testing: new e2e test
Docs Changes: n/a
Release Notes: inline.
[Optional Runtime guard:] yes.
Fixes https://github.com/envoyproxy/envoy/issues/30362

Since Envoy 1.29 I get Missing :te header when proxying to a Python-gRPC API because of this.
Removing the TE isn't the solution. We should replace it by trailers.

So thankfully this is runtime guarded so you should be able to reinstate prior behavior with a config change.
I think if you want the envoy-upstream hop to support trailers you should just add request headers "te: trailers" to the outgoing request.

I think if you want the envoy-upstream hop to support trailers you should just add request headers "te: trailers" to the outgoing request.

If you are meaning unconditionally, that seems a bit simplistic. That ignores whether the originating client supports trailers. From RFC 7230 §4.1.2:

The presence of the keyword "trailers" indicates that the client is
willing to accept trailer fields in a chunked transfer coding, as
defined in Section 4.1.2, on behalf of itself and any downstream
clients.

"any downstream clients" being the important part. So you'd want to make it conditional based on the originating request.

I think if you want the envoy-upstream hop to support trailers you should just add request headers "te: trailers" to the outgoing request.

If you are meaning unconditionally, that seems a bit simplistic. That ignores whether the originating client supports trailers. From RFC 7230 §4.1.2:

The presence of the keyword "trailers" indicates that the client is
willing to accept trailer fields in a chunked transfer coding, as
defined in Section 4.1.2, on behalf of itself and any downstream
clients.

"any downstream clients" being the important part. So you'd want to make it conditional based on the originating request.

Oh, interesting. Yes, I think that's a good point. Envoy itself is able to process trailers and pass them along, but it's only "safe" to do that in the case where Envoy knows that the downstream client is able to process trailers. The client, of course, indicates this by sending "trailers" in te.
Does that sound right?


Increase max overview amount

Summary
This increases the max number of Safes returned by the overview endpoint (/v1/safes) from 7 to 10. Although this can be achieved by an env. var., this is the desired value by the interface and as this is still being tested, it has been adjusted directly.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 93.033%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
6724

Relevant Lines:
6973

💛  - Coveralls


[stable/minecraft] Add ability to use an Ingress

Allow minecraft to use an ingress controller for it's public traffic.  Right now you have a lot of flexibility around the LB settings, but it would be great to use Ingress.  Using an Ingress allows frugal users to keep costs down for all their public traffic on K8s implementations.
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Similar to other HELM charts, provide an Ingress section in the values.yaml file for me to fill in.
Describe alternatives you've considered
I found another HELM chart on github that does this, but have not tested.  I'm thinking for forking this one to try out my idea.

What was the finding here? We also don't want to use a load balancer for this.


New command to replace deprecated ones

https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Thanks! Sorry, I somehow completely missed this


Allow changing amounts on existing subscriptions

Easiest way to do this is to cancel the old subscriptions and create a new one with updated info.
Note that when we say "subscriptions" below, that's represented in our database as an order and a subscription row.
There are three types of subscriptions. For each one, here's what we'll do:

Active subscription

Cancel current subscription (mark in Subscriptions table)
Create new subscription with updated order.totalAmount, subscription.amount and subscription.activatedAt date

Past_due

Behavior-wise exactly same as above. Worth pointing out that we'll keep nextChargeDate and nextPeriodCycle the same. So, they will get charged new amount whenever the next charge date is set to.

Canceled

Not supported. I think only action allowed for canceled subscriptions should be 'Restart' subscription. Otherwise, it can get confusing.

This could be a good starter feature for @znarf next week

Design at: https://github.com/opencollective/opencollective/issues/861#issuecomment-379466178. Ignore the Monthly dropdown in the design. We don't want to allow changing frequency through this flow.


Fix find handles to deleted file test for Windows 10 1903

When BuildXL fails to delete a file due to access denied issues or similar, it attempts to find all open handles to the file for logging.
Previously, if the file was in the Windows "pending deletion" state (file deleted, but there is an open handle with FileShare.Delete permissions to it somewhere), Windows would thrown an access exception if any attempt to create a new file handle to the file was created. BuildXL's attempts to find open handles would also fail to detect any existing file handles open.
The test FileUtilitiesUnsafeTests.FailToFindAllOpenHandlesPendingDeletion pins this behavior.
With Windows 10, Version 1903, Windows no longer throws an access exception in the above scenario and the File is considered deleted by C# File.Exists as long as all existing handles have FileShare.Delete permissions. but BuildXL is still unable to detect any existing file handles open.
This adds an or condition to fix the assumptions of the test to handle Version 1903.
Fixes #1496544

    /// </remarks>

nit: should we nuke this now?

Refers to: Public/Src/Utilities/UnitTests/Storage.Untracked/FileUtilitiesTests.Unsafe.cs:299 in e50905f. [](commit_id = e50905fb1b402909a0cd1a4a4f94e5f57653b7d8, deletion_comment = False)


More of a question

I can't figure out what mod is adding step assist. I have auto jump set to off but I am still getting step assist from something. Thanks for your help.

Quark gives you step-assist if you have Jump Boost II.

I don't have jump boost II currently

Then it must be some piece of equipment you're using. Come to the discord so we can discuss this properly, this tracker isn't the place for this.

I would have done that if I could have found a discord link

I ended up figuring it out anyways, it was the random stats that items get, my boots have step height 0.5.

I would have done that if I could have found a discord link

The discord link is on the pack title screen.


Please use "git switch" instead of "git checkout"

see: https://github.com/coderefinery/git-intro/issues/326
git switch instead of git checkout (when switching branches)
git switch --create instead of git checkout -b

@johanhellsvik @dhanyapushpadas for info

I can send the PR if you want but I wanted you to be aware of it to not create confusion and to align with days 1 and 2.

Thanks @bast. I will do it as part of the preparation.


Infrequent build timeout on beam7

Found this issue when running Python ValidatesContainer test multiple times. Always timeout on beam7 when building docker image but pass on other nodes. see https://builds.apache.org/job/beam_PostCommit_Py_ValCont_PR/
From history of beam7 (https://builds.apache.org/computer/beam7/builds), builds that're in state aborted because of timeout and belong to different tests. Console output shows Gradle process was stuck without any error and job reach to Jenkins timeout at the end.
cc [~yifanzou]
Imported from Jira BEAM-6570. Original Jira may contain additional context.
Reported by: markflyhigh.

obsolete


Better widget/notebook detection

Fixes a bug that causes pyccc to crash outside of the notebook with recent ipython/ipywidgets libraries

This functionality was merged elsewhere