Expose programmatic equivalent of CLI commands

Implements feature requested in https://github.com/AgnostiqHQ/covalent/issues/1853
edit: UX updated to
import covalent as ct

# start covalent
ct.covalent_start()

# stop covalent
ct.covalent_stop()

[x] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

@sriranjanivenkatesan can we pick this up and add it in the docs website please in API section?

Quick question @araghukas  -- with this approach, does  import covalent  pull in covalent_dispatcher?

Looks good @araghukas @kessler-frost

@santoshkumarradha @kessler-frost After lots of frustration on my end trying to get the API docs for covalent_start() etc. to renger, I noticed that we're getting some dependency issues when building the docs.
Surely this must be the reason why nothing is rendering for the new functions in this PR  (among other things).
See build logs here, this happens many times... examples below.
e.g.
WARNING: autodoc: failed to import function 'covalent_start' from module 'covalent'; the following exception was raised:
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/sphinx/ext/autodoc/importer.py", line 70, in import_module
    return importlib.import_module(modname)
  File "/home/docs/.asdf/installs/python/3.8.18/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 843, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/__init__.py", line 22, in <module>
    from . import executor, leptons  # nopycln: import
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/executor/__init__.py", line 372, in <module>
    _executor_manager = _ExecutorManager()
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/executor/__init__.py", line 58, in __init__
    self.generate_plugins_list()
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/executor/__init__.py", line 104, in generate_plugins_list
    self._load_installed_plugins()
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/executor/__init__.py", line 220, in _load_installed_plugins
    the_module = entry.load()
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2515, in load
    self.require(*args, **kwargs)
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2538, in require
    items = working_set.resolve(reqs, env, installer, extras=self.extras)
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/pkg_resources/__init__.py", line 829, in resolve
    dist = self._resolve_dist(
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/pkg_resources/__init__.py", line 875, in _resolve_dist
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (pydantic 2.5.1 (/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages), Requirement.parse('pydantic<2.0,>=1.9'), {'amazon-braket-default-simulator'})

e.g.
WARNING: autodoc: failed to import decorator 'electron' from module 'covalent'; the following exception was raised:
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/sphinx/ext/autodoc/importer.py", line 70, in import_module
    return importlib.import_module(modname)
  File "/home/docs/.asdf/installs/python/3.8.18/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 843, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/__init__.py", line 22, in <module>
    from . import executor, leptons  # nopycln: import
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/executor/__init__.py", line 372, in <module>
    _executor_manager = _ExecutorManager()
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/executor/__init__.py", line 58, in __init__
    self.generate_plugins_list()
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/executor/__init__.py", line 104, in generate_plugins_list
    self._load_installed_plugins()
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/covalent/executor/__init__.py", line 220, in _load_installed_plugins
    the_module = entry.load()
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2515, in load
    self.require(*args, **kwargs)
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2538, in require
    items = working_set.resolve(reqs, env, installer, extras=self.extras)
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/pkg_resources/__init__.py", line 829, in resolve
    dist = self._resolve_dist(
  File "/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages/pkg_resources/__init__.py", line 875, in _resolve_dist
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (pydantic 2.5.1 (/home/docs/checkouts/readthedocs.org/user_builds/covalent/envs/1854/lib/python3.8/site-packages), Requirement.parse('pydantic<2.0,>=1.9'), {'amazon-braket-default-simulator'})

@araghukas we can follow up the doc with next PR. But could you let @sriranjanivenkatesan know where to add these function in main docs ?
Meanwhile did directly referencing it from main covalent namespace work? Like covalent.start etc... instead of the actual module path?
Unblocking

@araghukas we can follow up the doc with next PR. But could you let @sriranjanivenkatesan know where to add these function in main docs ?
Meanwhile did directly referencing it from main covalent namespace work? Like covalent.start etc... instead of the actual module path?
Unblocking

@santoshkumarradha No, it does not seem to matter how we reference the function. I believe it finds the code just fine in either case, but fails to import and therefore analyze it because of the dependency conflict.
@sriranjanivenkatesan The goal is to have the covalent_start(), covalent_is_running(), and covalent_stop() functions exposed in the API docs. Like how we have them here in this file.

Add the "Covalent Server" section at the top of the API docs (above "Workflow Components")

@sriranjanivenkatesan btw, on the same API docs page, we have "Workflow Contents" in the index versus "Workflow Components" in the actual API below. Can we please fix it to have "Components" for both. 🙏
P.S. @santoshkumarradha We might want to add a couple sentences about the Python functions for start/stop on this how-to page as well.

Marking ready for review since the added tests are indeed passing (tests/covalent_tests/programmatic/commands_test.py)
Test failures now are due to UI backend tests...
Is this expected @kessler-frost?

Yep, I have fixed them in this PR: https://github.com/AgnostiqHQ/covalent/pull/1856, @AlejandroEsquivel will merge it soon, then we can merge this as well.

@araghukas just checked this in google colab with latest pre-release, does not seem to be working 👀


Detect assume-initial-arn from OCM URL

What type of PR is this?
feature
What this PR does / Why we need it?
It can still be supplied in the config file as an override, but with this change it is now optional for individuals interacting with standard OCM URLs.
Which Jira/Github issue(s) does this PR fix?
OSD-20606
Pre-checks (if applicable)

[x] Ran unit tests locally
[x] Validated the changes in a cluster
[ ] Included documentation changes with PR

Codecov Report
Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (8f6281c) 45.97% compared to head (43ed1d0) 45.97%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #325      +/-   ##
==========================================
- Coverage   45.97%   45.97%   -0.01%
==========================================
  Files          53       53
  Lines        3754     3772      +18
==========================================
+ Hits         1726     1734       +8
- Misses       1754     1764      +10
  Partials      274      274

Files
Coverage Δ

cmd/ocm-backplane/cloud/common.go
20.11% <42.10%> (+2.72%)
:arrow_up:


Execute files using fzf

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[ ] Linux
[ ] Mac OS X
[x] Windows
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish
[x] cmd (I use Windows, but I have access to bash sometimes)

Problem / Steps to reproduce
Is there an easy way to use fzf to execute the selected file? I think that would be a nice additional feature, a very useful one.

What do you mean by execute? Because preview window generated its preview by executing the file and displaying the output and there’s the —bind=“execute:command {}” you can also do.
Choice=“$(fd -t f . ./ | fzf)”
Eval”$Choice”

I meant for example a video file. I search for it and then I want to play it.

Here you go wrote this up and tested (works on mine) uses fd-find (rust rewrite of find) and ripgrep (rust rewrite of grep) but you can sub those out for find and grep if you prefer or ag or any search / filter command)
 FILE: vid.sh
#
#         USAGE: ./vid.sh
#
#         DESCRIPTION:
#
#         OPTIONS: ---
#         REQUIREMENTS: ---
#         BUGS: ---
#         NOTES: ---
#         AUTHOR: Rayiik<EMAIL_ADDRESS>#         ORGANIZATION:
#         CREATED: 10/07/2020 10:53:50 AM
#         REVISION:  ---
#===============================================================================
vidvar="$1"
export vidvar
_TF='true'
echo "$_TF"
export _TF
truefalse () {
if [[ "$_TF" == 'true' ]];
then
    _TF='false';
        else
    _TF='true'
fi
echo "$_TF"
export _TF
   }

vidmain () {
    local choice
choice="$(fd -uu -t f -d 5 "${vidvar=-'.'}" | fzf)"
    echo "$_TF"
while [[ "$_TF" == 'true' ]]; do
    vlc "$choice" && disown -a &
    truefalse
done
unset _TF
unset choice
}
vidmain

first argument is your search query and runs from your current working directory i tested with mkv and mp4 as an example but when i have time ill update to add more video types in a single search and targeted directory search

Thanks that's awesome!


Release 2.4.0

Can't see anything that would make this a breaking change. We did relax a rule in deprecated-3.3 (2b2d373ba08d619e914fa289a9fa0ff0b373c7e5) but at worst that would create a new unused directive warning.
Obviously if you are using 'deprecated' without a version that would be a breaking change, but I think we should just warn against doing that instead doing a breaking change release every time there's a jQuery release.

Sounds good to me.


[Update] Conditionally hide favorites category

Description
This PR updates the "Favorites" category in the Samples App:

Hides the "Favorites" category when there are no favorited samples (see issue for why).
Replaces some of the boilerplate code surrounding the favorites with an @AppFavorites type.

I will be adding additional ways to favorite in a follow up PR.
Linked Issue(s)

#407

How To Test

Ensure the favorites category hides/shows depending on whether there are favorited samples.
Verify favorite/unfavorite still work as expected (long press on sample row).

Screenshots

Before
After

Thank you!


Long compile times

I'm using Argo in production app, and I can see that the Swift compile time takes a lot of time in my project. Wondering if this is because of Argo & Argo based models which use a lot of generic code ? Please let me know if you have done any bench marking around this.
Argo Version
Argo 3.1
Dependency Manager
Carthage

Most likely this is due to a large model that has some complex or large quantity of Argo decoding going on. Argo usually speeds up with smaller models or simpler decoding setups. If you paste some of your bigger models here we could help advise where a slowdown maybe occurring.

@tonyd256 Thanks for replying. I don't have single large model in the app, however I've around 20 smaller models with max 15 properties. So I hope this won't hurt the compile time ?
I tried to measure the compile time with "-Xfrontend -debug-time-function-bodies" and I don't see much compilation times reported Argo based models. I hope the above flag would have exposed if argo based models are at fault ?

15 properties is a large enough model to slow down compile time for sure. I'd suggest trying to break up anything around 10-12 properties into smaller models. You could also try making it easier for the compiler by adding an explicit type to each line like so: <*> j <| "name" as Decoded<String> (might need parens)

@tonyd256, thanks for the suggestions. Will do that for the larger models!

FYI, @tonyd256 solution works quite fine as temporary fix. One of our project's model was compiling in 11568.x ms and now, with the explicit type, it takes only 26.x ms

@dalu93 Great to hear! Unfortunately, I think this is a limitation of the Swift compiler and there isn't much Argo can do to lighten the load without sacrificing what it is. Closing this for now, but feel free to continue commenting or create a new issue for anything new!

Hi @tonyd256, the workaround is not working. The error I get is:
No '<|' candidates produce the expected contextual result type 'Decoded<_>'

The configuration I have is:
Argo 4.1.0
Curry 3.0.0
Runes 4.0.1

Is there a way to apply the same behavior? I'm getting again long compile times unfortunately

@dalu93 you might need to add some parens around the j <| "foo" part of the expression. This is due to precedence changes we needed to make alongside Swift 3

I started trying Argo today. Unfortunately with a very small model object User, the compiler either spent painfully long time or gave me "Expression was too complex to be solved in reasonable time". If I commented out some fields and left 4 or 5 in User struct, the compiling time is a few seconds, which is OK. So I have big concern whether Argo is ready for production.

There are a few different ways to mitigate the Swift compiler's performance issues with Argo, which you can read about here. And it's best to open an issue and show your code if you have performance issues you can't solve yourself. I've shipped two apps now that use Argo as their JSON decoding framework.

@zhijunsheng Like @jshier said, if you could open a new issue with more sample code that'd be helpful for us. We're shipping Argo with dozens of client applications, and it's also included in other large scale apps without seeing the compiler issues you're describing.

Thanks for your quick responses! Now I'm upgrading Xcode from 8.3.1 to Xcode 8.3.2 since it said "improves performance". I'll try more and may report Argo performance issues if I do find some. BTW I already read the page jshier suggested and, what I just tried is very simple. A User struct with 10 fields of type String or Bool, the compiler took about 1 minute to finish; if I commented out 5 or 6 fields, the time was about a few seconds.

I just tested on another computer with Xcode 8.3.2 (Curry at "v3.0.0", Runes at "v4.0.1", Argo at "v4.1.2"). The result is basically same. If I comment out the last 3 fields, the compiling time is less than 10 seconds; otherwise it is more than 20 seconds.
struct User {
    let id: Int
    let firstName: String?
    let lastName: String?
    let jobTitle: String?
    let neighbourhood: String?
    let avatarURL: String?
    let useFacebookPicture: Bool?
    let isLead: Bool?
    let isFullyVerified: Bool?
}

extension User: Decodable {
    static func decode(_ j: JSON) -> Decoded<User> {
        return curry(User.init)
            <^> j <| "id"
            <*> j <|? "first_name"
            <*> j <|? "last_name"
            <*> j <|? "job_title"
            <*> j <|? "neighbourhood"
            <*> j <|? "avatar_url"
            <*> j <|? "use_facebook_picture"
            <*> j <|? "is_lead"
            <*> j <|? "is_fully_verified"
    }
}

@donaldsheng  One can try to solve the following:
extension User: Decodable {
    static func decode(_ j: JSON) -> Decoded<User> {
        let tmp1 = curry(User.init)
            <^> j <| "id"
            <*> j <|? "first_name"
            <*> j <|? "last_name"
        let tmp2 = tmp1
            <*> j <|? "job_title"
            <*> j <|? "neighbourhood"
            <*> j <|? "avatar_url"
        return tmp2
            <*> j <|? "use_facebook_picture"
            <*> j <|? "is_lead"
            <*> j <|? "is_fully_verified"
    }
}

This writing can improve the speed of Xcode compilation

@Arcovv That did the trick! Now it's super fast. Thanks a lot.

Just separating out the curry call should be a substantial improvement by itself.

This should be put together with "Hello Argo" sample, obviously. To be frank, I almost left Argo but it turns out I'm so lucky :-)  Thanks for all the hard work and help 👍

Thanks for this hack! I decrease the project compiler time from 5 mins to 15 seconds!. Life saver. 🎉


Preferences in Firefox (macOS) aren't loading

Preferences aren't working...
I want to add TinEye & others to search results, (as only Google images is finding me results,)
and there's no way to add/change searches

Which version of Firefox do you use?
Which lanuage?
Do you use any script blockers?
Do you use first party isolation?

Firefox 77.0.1 (64 bit)
English/US english
no blockers, no
don’t know what iso is?

On Jun 23, 2020, at 8:41 AM, Andreas<EMAIL_ADDRESS>wrote:

Which version of Firefox do you use?
Which lanuage?
Do you use any script blockers?
Do you use first party isolation?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

don’t know what iso is?

It's just for isolating site data, if you don't know it you probably don't have it activated (must be activated in about:config via privacy.firstparty.isolate, but it breaks a lot of sites).
Hm, can you please open the options page, press F12, navigate to the console and show me if there are any warnings or errors?

Everything is blank: no errors, warnings, messages, etc
On Thu, Jun 25, 2020 at 3:52 PM Andreas<EMAIL_ADDRESS>wrote:

don’t know what iso is?
It's just for isolating site data, if you don't know it you probably don't
have it activated (must be activated in about:config via
privacy.firstparty.isolate, but it breaks a lot of sites).
Hm, can you please open the options page, press F12, navigate to the
console and show me if there are any warnings or errors?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Brawl345/Image-Reverse-Search-WebExtension/issues/48#issuecomment-649835979,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBOCZOXQAD24A7W2JKT5CLRYPBILANCNFSM4OFEZQGA
.

still the same thing in Prefs today, & no, i didn't change anything in
about:config
On Thu, Jun 25, 2020 at 3:52 PM Andreas<EMAIL_ADDRESS>wrote:

don’t know what iso is?
It's just for isolating site data, if you don't know it you probably don't
have it activated (must be activated in about:config via
privacy.firstparty.isolate, but it breaks a lot of sites).
Hm, can you please open the options page, press F12, navigate to the
console and show me if there are any warnings or errors?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Brawl345/Image-Reverse-Search-WebExtension/issues/48#issuecomment-649835979,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQBOCZOXQAD24A7W2JKT5CLRYPBILANCNFSM4OFEZQGA
.

I honestly don't know why it doesn't work for yoiu, I sadly can't reproduce this. Have you tried removing the addon, restarting Firefox and reinstalling it?

yes i have tried all those
=====

On Jun 26, 2020, at 1:40 AM, Andreas<EMAIL_ADDRESS>wrote:
I honestly don't know why it doesn't work for yoiu, I sadly can't reproduce this. Have you tried removing the addon, restarting Firefox and reinstalling it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I was able to recreate this error.
Steps:

Open options in a private (incognito) window

If you check the console for the options page, you get
Uncaught TypeError: backgroundPage is null

Probably related to this: https://discourse.mozilla.org/t/getbackgroundpage-in-private-mode/14046/5


[Suggestion] Add loader while waiting for data to be fetched

While testing app on slower connection screen was totally blank for a long few seconds.
My suggestion is for now to add loader while waiting for response.
Example:

In a later stage it would be cool to implement something like this:
Example:

Please note that these examples are just rough sketches and are not supposed to represent final design, i'll leave that to people that are far more skilled at this than my self :)
Edit: Fixed typo.

Is anybody working actively on this issue/suggestion ? I can take the responsibility of it.


hypothesis tweaks

Run hypothesis more on CI and less by default locally, add a URL parsing case to hypothesis tests.

@Zac-HD just a heads up in case you're interested, as per fe29f19 and its CI run, hypothesis here is getting flaky with more than 1000 examples - some of the runs tests timed-out, but others errored with
>       raise the_error_hypothesis_found
E       hypothesis.errors.Flaky: Unreliable assumption: An example which satisfied assumptions on the first run now fails it.
E
E       You can reproduce this example by temporarily adding @reproduce_failure('6.56.4', b'AAABAAEADAEAAwEBBgEGAQEGAQYAAQYBAgBjBQEBGf13BqO+AwEHBwIBEaGIDVFKAQMAUgEEANMCAQQAoAYBBwUBAot5AA==') as a decorator on your test case

When I tried using the @reproduce_failure, the tests didn't fail locally. Or rather they errored complaining they hadn't failed as expected.
Not a problem for me, I've just set the limit to 1000, but I thought you might be interested.

Totally possible, I'm afraid I didn't dig too far.


Can't locate path: <.../vendor/binarytorch/larecipe/publishable/config/LaRecipe.php>

After running php artisan larecipe:install you get this error. It seams that either the filename is wrong or the code in the ServiceProvider has a typo:
$publishable = [
    'larecipe_config' => [
        **"{$publishablePath}/config/LaRecipe.php"** => config_path('larecipe.php'),
    ],
    'larecipe_assets' => [
        "{$publishablePath}/assets/" => public_path('vendor/binarytorch/larecipe/assets'),
        "{$publishablePath}/fonts/"  => public_path('fonts'),
    ],
];

Hi @andresgcarmona
Could you please provide the error message?

Thank you @andresgcarmona
I remembered now that this issue was already solved by one PR #19, the fix will be delivered in the next version.


Fix the return as an object, not a list, in the /validator

I called the address below to get the following results.
http://localhost:3836/validator/GBUVRIIBMHKC4PE6BK7MO2O26U2NJLW4WGGWKLAVLAA2DLFZTBHHKOEK

{"address":"GBUVRIIBMHKC4PE6BK7MO2O26U2NJLW4WGGWKLAVLAA2DLFZTBHHKOEK","enrolled_at":0,"stake":"0x86f1a6dff3b1f2256d2417b71ecc5511293b224894da5fd75c192965aa1874824ca777ecac678c871e717ad38c295046f4f64130f31750aa967c30c35529944a","preimage":{"distance":0,"hash":"0"}}

What about the previous comment ?


OpenJDK7 images?

You have a great set of Java images, but unfortunately, my app is stuck on Java7. Would you consider creating a set of Docker images for OpenJDK 7?

Although our upstream images have OpenJDK 7 the decision for this repo was not to build OpenJDK 7 images because OpenJDK 7/Java 1.7 has been EOL'd and we don't want to build EOL'd software.
@iynere @ryanwohara @marcomorain Do you all have any thoughts on reversing our decision on this? Personally, I agree with not building EOL software. I don't think it should be done. The only reason for consideration for me is that upstream builds it so it's little work for us (longer build times though).

@iynere Unless you disagree, I'm closing this.


Simplify tracing onboarding experience

Reference #1317 , goal 3.
Changes to simplify tracing onboarding by removing "enabled option" to turn off trace, user can set samplingRate to 0. With samplingRate=0, Start Trace Span operation in opencensus will not hurt performance.
apiVersion: dapr.io/v1alpha1
kind: Configuration
metadata:
name: myappconfig
namespace: app_namespace
spec:
tracing:
samplingRate: 0.001

/area runtime

Parent #1317
Changes done as part of https://github.com/dapr/dapr/issues/1395

@shalabhms is this done? Can we close this?


Doesn't work with PDFBox 3.0.0-alpha3

It return this error: java.lang.NoClassDefFoundError: Could not initialize class org.vandeseer.easytable.structure.Table.
The dependency is imported correctly in the pom file.

Hey @erirapce,
Thanks for reporting. Didn't yet try it with the new version of PDFBox. Also not quite sure whether I will find enough time to support it ...

Thanks for letting me know! :)

Hello, I was also a bit interested in this as pdfbox has now officially released 3.0.0 in August.
I've been using the library a bit and had it working initially with 3.0.0-RC1. However, the official 3.0.0 version no longer works.
Table classes are imported fine, but they fail during compile time which is caused by removal of static font instances by pdfbox
https://github.com/apache/pdfbox/commit/2d529f94b17a3f23aa8b2f566bc33947a91c245a
The static initializers are used heavily currently in easytable, e.g. PDType1Font.TIMES_ROMAN.
My current experience has been that the library could be fairly easily migrated to pdfbox v3`` while keeping support for v2with just the font changes, i.e. PDType1Font.TIMES_ROMAN=>new PDType1Font("Times-Roman")```
Is the library maintained still in any form or would it be possible to create pull requests for 3.0.0 support?

problem is org.vandeseer.easytable.structure.Table line 19:
private static final PDFont DEFAULT_FONT = PDType1Font.HELVETICA;
should be replaced with:
private static final PDFont DEFAULT_FONT = PDType1Font(FontName.HELVETICA);
PDType1Font.HELVETICA doesn't exist anymore
tested with pdfbox v3.0.0

Hey yes, I've been having this issue as well as the static initializers were removed
https://github.com/apache/pdfbox/commit/2d529f94b17a3f23aa8b2f566bc33947a91c245a
I was initially thinking this could work for both v2.x.x and v3.0.0, but it seems that Standard14Fonts.FontName is not yet available in v2.x.x, meaning that I've not found a way to support both pdfbox versions v2 and v3 simultaneously
https://github.com/apache/pdfbox/blob/2.0.29/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java
https://github.com/apache/pdfbox/blob/3.0.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/Standard14Fonts.java
(note I am not contributor to the repo, but I just use pdfbox actively)

I created a PR that should fix this issue as PDFBox v3 has been released

Thanks for your PR once again @jsimomaa! :+1:
I upgraded the develop branch. It supports PDFBox 3 now. A release will hopefully follow shortly.
However, I decided to only support PDFBox 3 with the next release. Supporting both at the same time wouldn't make much sense given the complexity involved.

Just released version 1.0.0 which is basically the same code base but upgraded to support PDFBox 3 and Java 11.


An error message "Invalid path mode '' in: No newline at end of file" pops up when for formatting document.

Environment

Steps to Reproduce

Check Formatter (Tools/Options/Text Editor/Python/Formatting) is black (default)
Type

	#####this is a comment
	import os,sys;
	spam( ham [ 1 ], { eggs: 2 } )
	def foo ():pass
	x=1;y     =2;
	y = 2

Go to Edit > Advanced >Format Document

Expected behavior
The package is installed successfully and the code is formatted automatically.
Actual result
An error message pops up. This issue reproduced when set Tools/Options/Text Editor/Python/Formatting to black and autopep8, do not reproduce when set yapf.

On today's build, this issue is not reproduce.


Cant Refine Objects as Documented

Try Flow
Disjoint Unions do not work when using exact object types as defined
https://flow.org/en/docs/types/unions/#toc-disjoint-unions-with-exact-types
/* @flow */

type Foo = {|
             +foo: 'foo',
            |}

type Bar = {|
            +foo: 'bar'
            |}

type Baz = {|
             +payload: string
            |}

function example(obj: Foo | Bar | Baz) {
	if (typeof obj.payload !== 'string') {
    	(obj: Foo | Bar)
    }
}

in this case it errors because obj could be Baz... but there is actually ZERO possibility in this case it could possibly be Baz.  In fact it would appear that it is 100% impossible to refine this object even with it being as strict as can be.
function example(obj: Foo | Bar | Baz) {
	if (obj.foo === 'foo' || obj.payload === 'bar') {
    	(obj: Foo | Bar)
    }
}

also does not work . here
nor does trying to take it a step further
function example<+O: Foo | Bar | Baz>(obj: O) {
	if (typeof obj.payload !== 'string') {
    	(obj: Foo | Bar)
    }
}

Union types work with string literals on one property that all members have in common. You reference "Disjoint unions with exact types", but then you go ahead and show an example that has a "member" (for Flow it isn't) that has a different property and even property type.

It actually specifically mentions that if you want to compare different objects it’s fine if you use exact types.  Also this is absolutdly something that can easily be inferred.

“With exact object types, we cannot have additional properties, so the objects conflict with one another and we are able to distinguish which is which.”

It doesn't work because it isn't implemented, like 10,000+ other things that should work. As I said... by mixing the disjoint union with something different you leave the space covered by the current Flow implementation. As I said, there are many, many many such limitations. It's certainly okay to write issues, just don't expect anything. The incompleteness (of Flow) is a given. Just use the parts that work. Like, disjoint unions (without mixing them with something else, unless you are lucky and it works).

This is the union that works, that Flow happens to have implemented.

Well then it should be documented that it doesn’t work or removed from the docs as it clearly states that it does work

You can also try this.
That's Flow, it understands some constructs but not others. The capability to understand the code has to be written into the OCaml code of Flow, and since there are an eternity of options they will never capture everything, you will always find holes. Avoid the holes, experiment. Flow and TS are hacks, full of compromises.

Yeah i knew the filter thing


fix(container-queries): add additional ogImage

SVGs in OG images don't work, added a second JPG-version. This way the SVG is used on the website and the JPG will be picked up by the platforms.

This does not seem to resolve the issue.
@sanderdejong88 Can we create some a sort of additional frontmatter key for the og:image


weed-fs on Linode

Hi, Im having issues with the master server system on my server in Linode. I have posted a detailed question on StackOverflow(http://stackoverflow.com/questions/30185696/weed-fs-in-linode-running-master).
Perhaps, somebody can help me out and look into it.

Use ip=your_ip_address instead of ip.bind

if i replace it, it will crash after it finds the Master Server Leader.
But if i add that extra parameter, it is seen properly.
But some commands seems to still not working.
curl "http://{IP-ADDRESS}.166:9333/cluster/status?pretty=y"
shows {
"Peers": [
"localhost:9333"
]
While curl "http://{IP-ADDRESS}:9333/dir/status?pretty=y"
show nothing. Any idea

Very likely on the linode server itself, it can not access your "http://{IP-ADDRESS}:9333/cluster/status?pretty=y".


Add jwt claims

Describe the pull request:

[x] New feature
[x] Code cleanup

Pull request long description:

Changes made:

remove fixed versions from python packages
added sub claim to jwt token per AAI specification

Related issues:

Additional information:
necessary for ega-data-api testing

This seems to be usable for getting user and email https://github.com/neicnordic/ega-data-api/pull/13


1st adopt a drain email

the email says, 'thank you for adopting one of the Lower Grand River's 40,000+ stormdrains...' I thought I had deleted the 40,000 in the email so that we would not have to update the number as we get more communities participating?

@cara-lgrow Could you provide the link to the current welcome email so we can check? I'm having trouble finding it in our other issues.

https://docs.google.com/document/d/1en7smcuMtIj7P-VY-JJT3_vkA2e7QFHzRLFAsLCnLeA/edit


chore: simplify pr test

Describe your changes
update runner to ubuntu
add timeout
remove init step

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Test Plan
After merge verify actions running on proper device with timeout
Checklist before requesting a review

[x] A self-review of my code has been completed
[ ] Tests have been added / updated if required
[ ] Documentation has been updated to reflect these changes

I can put it back in and increase the timeout


Remove sitemap. Keep dependencies minimized.

I just now pulled the master branch and ran jekyll serve locally.
I get this:
in `require': cannot load such file -- jekyll-sitemap (LoadError)

When commenting out the requirement in the config everything works directly.
Unless Pixyll is designed specifically for GitHub pages I think you should remove
jekyll-sitemap (I am guessing this gem is installed on github pages).
If you want to add more features that are designed to work out of the box with github pages why not do it on another branch and keep the Pixyll core clean and as simple as possible on master branch.
I rather we keep it as simple as possible to use Pixyll, minimizing install requirements. That is why I am using Jekyll in the first place, to get rid of the complexity with databases etc. To be able to just download the latest Pixyll to an empty folder and run Jekyll Serve is great. No gemfile or anything.
Pixyll is great now that it is so simple. I hope that we can keep this in mind as development continues and hopefully not add anymore dependencies.

Hey @rassol,
I definitely see where you are coming from here. Pixyll is intended to be simple. Super simple. The introduction of jekyll-sitemap into the _config.yml makes it a bit less so. I'm thinking a way to solve this problem is to comment out the gems section into something like:
# For sitemaps, which are compatible with Github Pages, uncomment out the following two lines.
# Note that you will need to then install the jekyll-sitemap gem to run this project locally with `gem install jekyll-sitemap`
# gems:
#  - jekyll-sitemap

That way, it's still pretty simple to integrate sitemaps, while still only requiring the following steps to get up and running with Pixyll:
gem install jekyll
jekyll serve

I'd initially merged in the PR for this because I assumed that jekyll-sitemap was an included dependency for the jekyll gem. Unbeknownst to me, I just had the gem installed already from another project. The current dependencies for Jekyll:
fast-stemmer, classifier-reborn, ffi, rb-inotify, rb-fsevent, hitimes, timers, celluloid, listen, jekyll-watch, sass, jekyll-sass-converter, coffee-script-source, execjs, coffee-script, jekyll-coffeescript, jekyll-gist, jekyll-paginate, blankslate, parslet, toml, redcarpet, yajl-ruby, posix-spawn, pygments.rb, colorator, safe_yaml, mercenary, kramdown, liquid, jekyll

Thanks for pointing out this inconvenience. When I get a moment I will submit a PR with an update to require an opt-in for sitemaps.

Your solution is not scalable as more gems find their way in. Messy config.
The jekyll build and serve options accept a --config parameter that takes a comma separated list of config files, where later ones override former. In this case, another solution would be to have a config _config-github.yml and put extra stuff in there.
Oh wait, that probably doesn't work with GitHub pages since you are not building yourself. Nevermind :)
Maybe there's some other solutions. Having another branch for github version is messy too but if additional files are added to support some new dependency that are not needed in the base configuration of Pixyll, I do not think it should be in the same branch.
I only know Ruby to the extent that I can install and use Jekyll. So I think that all gems that come with Jekyll are fine. Just running the 2 lines you mentioned to get it running is super great. If the sitemap gem were to be included in Jekyll I would have no problem with this since there are no extra steps.
Anyway, I think your solution works for now but not in the long run.

I don't really foresee more than another gem or two being added down the road (if that), so I don't think a few commented out lines will be too disastrous. I think this is preferable to maintaining a separate branch, and since a new user needs to modify the config anyway, they can either: remove the commented out gems list, uncomment it and install the gem, or just leave it. Seems simple enough to me.
Yeah, the problem with the config option is, as part of the ease of use, I'd like for Pixyll to be 100% Github Pages compatible. At this current time, Github Pages doesn't allow you to pass any build options.
For the time being we can roll with the commented out list, and consider new options if that list grows to more than one or two.
Thanks for your thoughts, and I appreciate you taking the time to open this issue. I think this proposed solution will definitely be an improvement to what was there before!

Really, ok.
2 or 3 is no problem, but yes, if it grows to more than that...
Thank you too!

FYI, this still happens when serving from the current master branch. I'm not sure if the intention was to leave it as-is or to go with the opt-in approach for the list of gems, but from reading this discussion it sounds like the issue might have been forgotten once the issue was closed.
Thanks!

Good catch, I'll get that fixed right now. Thanks!


🍒[cxx-interop] Enable CxxShim headers for armv7k watchOS

This fixes the no such module 'CxxStdlibShim' error when building CxxStdlib for armv7k.
For watchOS, armv7k is included in ${SWIFT_SDK_${sdk}_MODULE_ARCHITECTURES} but not in ${SWIFT_SDK_${sdk}_ARCHITECTURES}.
rdar://108206099
(cherry picked from commit ce2b116dabbe21a1203ca6fbc5694fc8396b276d)

@swift-ci please test


Update alter-index-transact-sql.md

Fixed various small issues with ] and } terminators, spacing and spelling issues

@microsoft-github-policy-service agree

@rwestMSFT
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Looks good, thanks @JohnKNess!
#sign-off

Due to the upcoming Ignite conference, all merging and publishing is on hold until the morning of 11/15.
This PR is ready for merge on 11/15.
#label: "event-triaged-to-10AM"
#label: "missed-content-complete"


tcl-tk: Tiger patch does not apply

@mistydemeo See https://github.com/Homebrew/homebrew-dupes/blob/master/tcl-tk.rb#L35-L37
  # sqlite won't compile on Tiger due to missing function;
  # patch submitted upstream: http://thread.gmane.org/gmane.comp.db.sqlite.general/83257
  patch :DATA if MacOS.version < :leopard

==> Upgrading tcl-tk
==> Downloading https://downloads.sourceforge.net/project/tcl/Tcl/8.6.4/tcl8.6.4
######################################################################## 100.0%
==> Patching
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/pkgs/sqlite<IP_ADDRESS>/generic/sqlite3.c b/pkgs/sqlite<IP_ADDRESS>/generic/sqlite3.c
|index 5c8e1c1..bb3fdfc 100644
|--- a/pkgs/sqlite<IP_ADDRESS>/generic/sqlite3.c
|+++ b/pkgs/sqlite<IP_ADDRESS>/generic/sqlite3.c
--------------------------
No file to patch.  Skipping patch.
3 out of 3 hunks ignored
Error: Failure while executing: patch -g 0 -f -p1

Thanks, Misty.

Sorry for the delay!

No worries at all.


Conditionally Onboarded IIS workload with Application Insights

Issue #, if available:
Description of changes:
Integrating QuickStart with Application Insights. This allows customers to setup AppInsights monitoring by setting "SetupAppInsightsMonitoring" to true
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/do-e2e-tests

/do-e2e-tests

/do-e2e-tests


Very simple samples for DefaultAzureCredential

Truth in advertising.
Disclaimer 1: There was some surprise expressed by others that we'd even be spinning samples for DefaultAzureCredential so I kept this rather terse.  Shout out if more (or less?) detailed comments or further samples (other services) would be desired.
Disclaimer 2: Storage bits have been going into preview5, if we want to keep that distinct I can split this into two (eh vs storage)

Can one of the admins verify this patch?

Splitting this into two PRs so that blob samples can go under new folder structure into distinct branch, and eventhubs samples can go into master.


Climber timers in ClimberFullClimb able to be changed through the con…

…fig, deleted unused EXTEND_L3_HEIGHT enum and references to it

Compiled successfully, needs to be tested on a robot


Finalise documentation

Partially addressed by https://github.com/alan-turing-institute/sqlsynthgen/pull/155
Could always use more work though.

I've done a pass on the documentation. I made mid-sized edits to the introductory tutorial, and small edits everywhere else. The one major thing I neglected to do is I didn't try running the code in the loan tutorial to see that it still works, that would require setting up a MariaDB instance. The edits I did were merged in #159 and #160, without review because we want the docs improvements visible ASAP for any potential handover needs.


Sending an email caches a view with the wrong group (ownership)

Laravel Version: 5.7
PHP Version: 7.1.3
Database Driver & Version: MySQL / 5.6.43

Description:
When my app sends an email with a Job, it caches the view of the email.
However, the ownership of the cached files is incorrect.
When I run ls -al storage/framework/views, I can see that each file looks something like...
-rw-rw-r--  1 user user   192 Apr 27 09:46 4251.....php
This app is running on Nginx, and all the files in storage (and bootstrap) has its owner user and the group nginx.
i.e.
drwxrwxr-x.   5 user nginx   46 Feb 23 02:50 storage
If there is a file the group of which isn't nginx, there will be the "Permission denied" error.
Therefore, each cached file should have its ownership be like...
-rw-rw-r--  1 user nginx   192 Apr 27 09:46 4251.....php
If I change the ownership of cached views with chown $USER:nginx -R storage/, the "Permission denied" error won't be caused. This is why I assume that this is a bug.
Steps To Reproduce:
First of all, I'm using CentOS 7, which (unlike Ubuntu) isn't using www-data user/group.
To reproduce the bug, it'll be necessary to create a job with your DB.
In other words, run the following commands.
php artisan queue:table
php artisan migrate
Then, send an email from a controller with a job in the following way.
class ExampleController extends Controller
{
    // __construct

    public function store()
    {
        //other lines here...
        SendEmail::dispatch($order);
    }
}

class SendEmail implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    //__construct

    public function handle()
    {
        Mail::to($this->order->email)->send(new OrderEmail($this->order));
    }
}

Are you running your webserver and your workers under different users?

My web server (Nginx) is running under nginx.
i.e.
When I ran ps aux|grep nginx, I can see....
root      6456  0.0  0.0 126504  5468 ?        Ss   Apr26   0:00 nginx: master process /usr/sbin/nginx
nginx     6509  0.0  0.2<PHONE_NUMBER>8 ?        S    Apr26   0:01 php-fpm: pool www
//simlar lines here
user    8268  0.0  0.0 112708   980 pts/0    S+   08:53   0:00 grep --color=auto nginx
nginx    12138  0.0  0.0 138756  4444 ?        S    Apr27   0:00 nginx: worker process
//again, other similar lines
nginx    18688  0.0  0.2<PHONE_NUMBER>0 ?        S    Apr26   0:01 php-fpm: pool www

As far as I understand from these lines, the webserver is run by the nginx user.
However, I'm not sure what you mean by "your workers".
In the snippet above, I can see "nginx: worker process".
Is this what you're asking about?

The job is executed by a queue worker. It isn't visible in the ps data you've sent us. It's not the same as "nginx: worker process". Try ps aux | grep queue and you'll probably find something executing "php artisan queue:work" or "php artisan queue:listen". That process seems to be running with another user account than what nginx is using.
Why do you two separate accounts; a user account, and a nginx account? Configure the fpm pool to use your user account, and the php code will execute as your user, have the same permissions as your user, and be able to do the exact same things as your user.

Thank you for your reply.
user   32212  0.0  0.0 112708   980 pts/0    S+   09:59   0:00 grep --color=auto queue

This is the result of ps aux | grep queue, and I can see that the queue is owned by user.
"Why do you two separate accounts; a user account, and a nginx account? "
The reason why I was using nginx account was because the server, Nginx, was run by the nginx user account. The server had be able to access the Laravel app.
So, I changed the ownership status for files in storage (and bootstrap), so that the user is $USER and the group is nginx
but now I've changed the user of Nginx into user, so that Nginx can access storage/framework/view
I'll see if this is the right approach.

There's two different components here.

nginx handles incoming http requests and tries to figure out what to do with it. It may need readonly access to source files to determine if they exist at all.
php-fpm handles actually executing the code. This needs readonly access to the code, and write-access to places you need to write.

You should probably change the user of the php-fpm pool, not nginx.

@sisve
I understand your explanations on Nginx and php-fpm, but I'm not really sure why php-fpm's user should be changed (instead of Nginx's user).
On Ubuntu, I used to have Nginx and Laravel files' owned by www-data.
(When I google something like "setup Nginx on Ubuntu for Laravel", I see many examples which use www-data as the owner of Nginx and Laravel files, like this post and this post.
This means, in my opinion, that Nginx and Laravel apps are supposed to be owned by the same user, not php-fpm.
Sorry this is a bit off-topic, but could you elaborate on why it is php-fpm that should change its user?

The process that executes the code needs to be using the proper user account. It's the fpm that executes the code, so by that logic it's fpm that should be using the user account.
Keep in mind that the fpm supports several pools, and you can setup on for every user account. That way you can have several different php applications executing with different php permissions. However, you can only have one nginx process running.
I believe that nginx doesn't even need permissions to read the files if it is configured to send everything to the fpm. That way nginx will blindly accept anything, and forward it to the fpm, which is executing as the proper user and doing the actual work. This setup will however have downsides with static files which is no longer optimized in nginx but served via the fpm.

Going to close this because we don't offer support for this version anymore. Please try to ask this on a support channel next time (see the issue templates).


Update js2coffee dependency

Fixes installation issue (cyclic.js not found)

@Chandler Please get this merged!


Operation sequence and file naming

Hi, thanks for sharing this, but I have a few questions:

Could you clarify how the order of operations (OP1, OP2, OP3) is executed on the workpieces? Specifically, does each machine perform OP1, then OP2, and finally OP3 on the same workpiece in sequence? Or does the machine complete OP1 on all workpieces before moving on to OP2, and so on?

The data is divided by machine, operation, and status, with file names formatted as {Machine}(Month){Year}{Operation Code}{Code}. However, the Code does not indicate the sequence of operations.

Is there additional information available that could be added to the dataset to provide this sequence? This information is crucial for working with continuous models to detect pattern changes before the first fault occurs. Currently, all the "anomalous" data is in sequence, causing a loss of the "time information" necessary for time series analysis.

Another issue concerns the window size. You recommend using 4096, but when I calculate the window size using the formula
ws =Sampling rate/Rotational speed with a sampling rate of 2 kHz and a rotational speed of 250 Hz (which applies to most of our operations), I get a different value for the window size.

Hi @mariaclaraav ,
thank you for your interest in our dataset.

As explained in Section 3.1 of the paper, a machining process consists of a sequential combination of different operations (OPs). In Figure 4, an example of the machining process is provided, where the process is segmented, and each segment corresponds to an OP. This indicates that each machine performs OP1, OP2, and OP3 on the same workpiece in sequence; however, the operations do not necessarily follow a specific order. For confidentiality reasons, the order of the tool operations has been shuffled and anonymized. This means that OP2 does not necessarily follow OP1, and the operations can occur in any sequence.
The samples are named using the following format: {Machine}(Month){Year}{Operation Code}{Number of sample}. We collected multiple samples from different process runs, designated as Mx_OPy (where Mx represents machine x and OPy represents process ID y). These samples have been numbered sequentially using the {Number of sample}.
you are right. The window size is set to cover at least one full revolution of the motor. So using the formula we get that we can capture a full revolution with 8 samples. However, in our work the ws parameter has been tuned in a big range and the best result was given by ws=4096.


Implement TCGC's Model Types

Language emitters should work to move entirely to TCGC's new ecosystem. I've created a separate issue if people want to first move to the SdkModelType and SdkEnumType. To access these types, you can access  and  on the  type

fixed in #2476


Treat CMake warnings as errors

We should consider treating CMake's warnings as errors.
Can you guys think of reasons not to do so?
Originally posted by @dalg24 in https://github.com/arborx/ArborX/pull/102#issuecomment-532197489

While I don't have a good overview of the types of CMake warnings, my knee jerk reaction is to say yes.

To be clear, I meant treating them as error in the Jenkins CI builds.  We had no warnings before and we got with #99
CMake Warning:
  Manually-specified variables were not used by the project:

    ArborX_ENABLE_MPI
    MPIEXEC_PREFLAGS

We currently have
kokkos-install/lib/cmake/Kokkos/KokkosConfig.cmake:113 (MESSAGE):
  The installed Kokkos configuration does not support CXX extensions.
  Forcing -DCMAKE_CXX_EXTENSIONS=Off
Call Stack (most recent call first):
  CMakeLists.txt:4 (find_package)

and
CMake Warning at cmake-3.12.0-Linux-x86_64/share/cmake-3.12/Modules/FindBoost.cmake:847 (message):
  New Boost version may have incorrect or missing dependencies and imported
  targets
Call Stack (most recent call first):
cmake-3.12.0-Linux-x86_64/share/cmake-3.12/Modules/FindBoost.cmake:959 (_Boost_COMPONENT_DEPENDENCIES)
cmake-3.12.0-Linux-x86_64/share/cmake-3.12/Modules/FindBoost.cmake:1618 (_Boost_MISSING_DEPENDENCIES)
  test/CMakeLists.txt:16 (find_package)

Treeating them as errors might be pretty annoying.

Can we close this now that we have quality gates in Jenkins?


希望能有 Mac 版和 iOS 版的客户端

Up for 1!


How to use

Hello, this project looks nice.
How do I use it? Do you have a tutorial or some hints?
Thanks!

I came up with this when coding on another project that maintained a bespoke set of scripts via actions/github-script to accomplish similar things. Those scripts were a bit hard to maintain, given that there was no real IDE support, not to mention unit tests. I thought it would be nice to have it as a reusable action, as a real project. I never came along to reintegrate it back, and now this action is basically just used by itself, and I never wrote a README snippet on how to use it.
That being said, considering the limitations mentioned in the README, I think it is functional. Have a look at the "Re-run workflows for incoming PRs" workflow defined in trigger.yaml. This is how it invokes itself, and triggers the "CI" workflow defined in ci.yaml. The workflow to be triggered is to be specified via the workflow key in the action's with block.
So, without having tested this, something like the following should be working for other projects:
name: Re-run workflows for incoming PRs

on:
  push:
    branches:
    - '**'

jobs:
  trigger_some_other_workflow:
    name: Trigger some other workflow
    runs-on: ubuntu-20.04
    steps:
    - name: Trigger some other workflow
      uses<EMAIL_ADDRESS>      with:
        workflow: Some other workflow

Would be nice to add this to the README! Interested in doing a PR? :smiley:

Opened #94


death messages: player names disappear

Hello,
when viewing any demo with HLAE, the player names in the death messages disappear.
Only the weapon and headshot icons show up ("killed with"):

However, when starting CS without HLAE the names appear normally:

My settings:

I didn't use any "mirv" commands or anything, all standard. No demo tools used for fixes.
Any hints on this?

Hello.
This is very strange.
Which HLAE Version are you using?
Is it possible to upload a demo with the problem?
Do you have any moviemaking commands in autoexec.cfg or s.th. like that?

Also try this with HLAE
playdemo
disconnect
viewdemo
Chances are nothing is broken, but you joined a server beforehand or played a different demo (especially when using COL DemoPlayer in default settings).
Meaning it could be a viewdemo bug, when you view the demo the first time.

Hello,
I'm using the latest HLAE version. Also all demos are effected.
I only specified an alias in my autoexec.cfg which I didn't use when viewing the demo, so I doubt this could be the problem.
I'll try your solution later, thanks for the feedback and this awesome tool :)

Would be great if you could provide me with the demo, in case the problem is HLAE specific.

Also are you using juGGaKNot's new or old MovieMaking pack by any Chance?
I talked to him and we just noticed there seems to be a problem when using the pack.
Or are you using any other moviemaking pack?

Using blank CS without any moviemaking related stuff.
Demo: http://www.file-upload.net/download-10778993/thisismydemomydemoisamazing.zip.html
Also not fixed after a complete computer restart. But I noticed that it worked when I first used HLAE... and then it suddenly broke.

Hello.
I had a look at the demo and it got a known problem whether or not HLAE is used:
How to reproduce it:

Freshly start the game (doesn't matter if with HLAE or not)
viewdemo yourDemoNameHerre

Symptom:
Before the next round start, the player names are missing like on your screenshot.
Workaround:
Let the demo play until the first recorded round start, then rewind it (preferred) or restart it. This should fix the problem in most cases.
If no round start is recorded in the demo:
Sometimes it also helps to start a listenserver beforehand in the game, but I usually don't recommend that:

Listenservers should not be used with HLAE (though listenservers usually don't have VAC and I use them for testing with HLAE sometimes, but still don't recommend it - if you join a real server accidentally with HLAE, then you will get most likely VAC banned, that is why HLAE issues a warning before allowing to join any server)
It will still be bugged (i.e. grey names instead of T/CT colour), the workaround above is much better (if applicable)

As said this is a known problem with the way the game records and reads data from demos. There is not much I can do about that (I don't have enough demo format knowledge to write s.th. that fixes the demo, i.e. by adding a fake round start right at the start).
Please check if you can confirm that it is not an HLAE related problem and if the workaround above works for you.

Thanks again!
But if it's not HLAE related, why can I see all names normally when starting CS without HLAE and viewing the demo?
I'll try your workaround later, thanks for your investigations :)

So, the workaround isn't working. Still no names showing up, tried it multiple times.
As I said, without HLAE it works as it should with all demos, so it must be something related to HLAE.
I hope I can help you to further investigate this. Seems like I'm having a different problem here than the one you described ;(

Hm that's really odd.
Well the problem is HLAE should do nothing to the death messages unless you execute mirv_deathmsg the first time.
Is the path your are launching CS with the same as the path your Steam installation uses? Maybe you have two installations of CS (one on C: and one on D:) and are using an modified / old one (on D:)?
I can only offer to have a look at your Installation / PC through TeamViewer, you can contact me in #hlae, check http://www.advancedfx.org/contact/ for webchat.

You an also add me on Steam if you want in order to contact me: http://steamcommunity.com/id/ripieces/
I tried to search you there but only found an account that was online over an year ago :/

Okay, the workaround works.
There was a different problem:
The height was bellow 480 pixels, for this case there is no EngineFont defined in C:\Program Files (x86)\Steam\steamapps\common\Half-Life\platform\resource\TracherScheme.res, that is why the workaround did not work for @CREEATION
I will add a warning box for users that set bellow 480 pixels in the launcher with one of the next updates and when it's released I'll close the issue :-)

Thanks for resolving this issue :)
Isn't the width also affected?

Thank you.
This issue has been addressed in HLAE <IP_ADDRESS> (2015-07-23T16:00Z).
Thus closing.


Windows 2008 R2 BCDEdit

Hi , While trying to create the windows 2008 R2 image using the script it gives the BCDEdit warning message : WARNING: BCDEdit failed: bootmgr device locate. It does create an image file though, however once we boot the image it is giving error while installing the cloudbaseinit packages.

using the -installupdate parameter the instance checks the WSUS server for available updates and accepts it. however it gets stuck at the downloading phase and proceeds with next part of the script which is to install cloudbase init. Please find attached screenshot of the same.

Got the issue fixed in another post by @tomcsanyid  hence closing the issue.


Broken link "Announcing larger, higher scale storage accounts"

On this page, this link is broken:

see Announcing larger, higher scale storage accounts.

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

ID: 6cca4823-69a3-0b1a-059a-3a0605ed74ce
Version Independent ID: 8e8299f1-e618-f1d1-bed2-0e3f4e05829e
Content: Performance and scalability checklist for Blob storage - Azure Storage
Content Source: articles/storage/blobs/storage-performance-checklist.md
Service: storage
Sub-service: blobs
GitHub Login: @tamram
Microsoft Alias: tamram

Hi @arphox I have found no broken URL, could you please help with broken URL.

Hi @arphox I have found no broken URL, could you please help with broken URL.

Hi @SaibabaBalapur-MSFT,
Yes, I can.
As you can see, there is a part like this on the linked page:

For more information, see Announcing larger, higher scale storage accounts.

Let's check the URL:
https://azure.microsoft.com/blog/announcing-larger-higher-scale-storage-accounts/.
When I open this URL, with a 301 Moved Permanently HTTP status code, my browser gets redirected to
https://azure.microsoft.com/en-us/blog/announcing-larger-higher-scale-storage-accounts (same as before, but en-us appended after the domain).
However, this URL also gets a 301 Moved Permanently HTTP status code, and redirects my browser to https://azure.microsoft.com/en-us/blog/category/storage/. This loads.
This is why I arrive to the https://azure.microsoft.com/en-us/blog/category/storage/ URL when I click the link.
So, the issue you should have seen is this and I am quite disappointed you wrote you have found no broken URL. This is clearly a content that has been deleted or moved as it can be seen not just from the redirects, but from the link itself: in the text it says "for more information" read that link, but that link just sends you to a general blog website which is suspicious already.

Since I got annoyed by this situation, I did some digging on the internet and found:

an archived version of the linked webpage on web.archive
which seems to be the same as what I found from a quick google search to the article's title: https://azure.microsoft.com/es-es/blog/announcing-larger-higher-scale-storage-accounts/ (notice the es-es part). So fortunately the content is still there, but for some reason has been moved to some Spanish area.

At this point, of course you could just fix the link so it points directly to the Spanish localization subpage, but I'm not sure this is the best solution as I think there is a reason that URLs on documentation pages refer to non-localized URLs.

@arphox
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.
@tamram
Can you please check and add your comments on this doc update request as applicable.

@arphox - Thanks for your feedback. I've removed the link, and the update should be published shortly.
That's a link to a very old blog post. You can find up to date information about storage account scalability targets in the documentation at https://learn.microsoft.com/azure/storage/common/scalability-targets-standard-account.
#please-close


Excalidraw can’t decrypt data

I have an issue with the whiteboard when the user doesn't moderator  role open the white board its show error can’t decrypt data

Can you please provide more details? How are you using this repo, is it with a custom excalidraw implementation?

I add a socket to the nginx following the instruction https://community.jitsi.org/t/installing-excalidraw-backend-for-whiteboard-feature-on-self-hosted-jitsi-mee/118883/10
however, when the moderator opens the whiteboard it does not show and show the error can’t decrypt data


Mobile version of Rank'em?

Not sure if this is needed, but a quick preview on mobile makes me think the orientation of our fig and scaling will be hard to see/read in mobile portrait mode.
By having a mobile version of the scaleX and scaleY and text placement should allow us to shift this fig around and keep all of the other interactions the same.
Maybe worth it?

That does look pretty slick in that arrangement....I'm not opposed, depending on time/people availability.


Fix 404 errors

remove ellipsis syntax from action creator (ellipsis syntax
expands iterables into argument lists)
modifiy repo-select action creator to read the payload key

I looked at the console output and it was saying stuff like changeRepo request received with url https://github.com/[Object object].git. Silly javascript smh
the ellipsis syntax turned the string input into a list of characters

Side note / nitpick: I think you should try template strings. You just do stuff like `change repo to ${repoPath}` (those are backticks), which is more readable, imo.

Huh, I'm not sure how I missed that when reading the console myself. Thanks for the help!


Elastalsert sends query from too old date.

I've just started using Elastalert.
At first time it works well.
But after some day later,
Elastalsert sends query from too old date.
Below is my configurations
rules_folder: rules
run_every:
  minutes: 1
buffer_time:
  minutes: 1
es_host: elasticsearch
es_port: 9200
alert_time_limit:
  minutes: 1
writeback_index: elastalert_status

and it is my rule for test.
name: cocofarm-es
type: any
index: my-*
realert:
  minutes: 0
filter:
- query:
    query_string:
        query: "*"
alert:
- command
command: ["smsg.sh", "%(logiss_timestamp)s", "%(logiss_logger_id)s", "%(msg)s"]

with verbose flag, it puts log below
$ date
Mon Aug  7 13:29:31 KST 2017
INFO:elastalert:Starting up
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:11 KST to 2017-08-02 13:12 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:12 KST to 2017-08-02 13:13 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:13 KST to 2017-08-02 13:14 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:14 KST to 2017-08-02 13:15 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:15 KST to 2017-08-02 13:16 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:16 KST to 2017-08-02 13:17 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:17 KST to 2017-08-02 13:18 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:18 KST to 2017-08-02 13:19 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:19 KST to 2017-08-02 13:20 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:20 KST to 2017-08-02 13:21 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:21 KST to 2017-08-02 13:22 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:22 KST to 2017-08-02 13:23 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:23 KST to 2017-08-02 13:24 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:24 KST to 2017-08-02 13:25 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:25 KST to 2017-08-02 13:26 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:26 KST to 2017-08-02 13:27 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:27 KST to 2017-08-02 13:28 KST: 0 / 0 hits
INFO:elastalert:Queried rule cocofarm-es from 2017-08-02 13:28 KST to 2017-08-02 13:29 KST: 0 / 0 hits
...

Why It starts query from 2017-08-02 while current date is 2017-08-07 ?
Any Ideas?

Add to config.yaml
old_query_limit:
  minutes: 1

This sets the maximum time elastalert goes back in time. It tries to do this because it detected that you ran "cocofarm-es" on 08/02 and it goes back up to a week (by default) so that you don't miss any documents, in case elastalert crashes or stops and restarts.

@Qmando
Thanks, It solved my issue.
one more question,
if I want to enable query to old records later,
How I can do that ?
Just change config and reload it ?

You can also force it to run between a start and/or end time. For example,
elastalert --verbose --start 2017-08-03 --end 2017-08-04