NFSAAS-2236 update python SDK tests for R3.5

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Codecov Report

:exclamation: No coverage uploaded for pull request base (restapi_auto_netapp/resource-manager@aa65abe). Click here to learn what that means.
The diff coverage is n/a.

@@                           Coverage Diff                           @@
##             restapi_auto_netapp/resource-manager    #4684   +/-   ##
=======================================================================
  Coverage                                        ?    49.5%
=======================================================================
  Files                                           ?     6345
  Lines                                           ?   177639
  Branches                                        ?        0
=======================================================================
  Hits                                            ?    87935
  Misses                                          ?    89704
  Partials                                        ?        0

Continue to review full report at Codecov.

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

Thanks @leonardbf !!!


gadgets: Standardize common information

Fixes #1567
This PR introduces some types and helpers to easy the gadget development process. By using the new types (specially struct gadget_process) and the gadget_fill_current_process and gadget_socket_fill_process helpers, gadgets can provide all information related to processes without having to duplicate the same logic in all of them.
Specifically, this PR:

Adds support for column alias: This is needed to avoid printing long names like proc.user.uid
Introduces new types and helpers for handling common information
Port almost all gadgets to use them (see missing ones on the TODO below)

Backwars Compatibility
The json output of this PR is changed, consumer of the json output will have to update their unmarshaling logic. The columns isn't changed (at least, not a lot)
Discussion
This new solution us a little bit less efficient than the previous one, for instance, many gadgets need to get the mount ns namespace, pid or uid fields to understand if a given event has to be ignored, then the call to gadget_fill_current_process will get all that information again. I think the advantages are more and we can optimize it back on #706.
Peformance Numbers
I did some performance experiments to better understand the impact of this PR, these are the numbers. In order to run the experiments, I modified Inspektor Gadget  to collect statistics on the programs and run both versions of the gadgets while generating events.

Gadget
Main
This PR
%

trace_open
590
629.44
6.65%

trace_exec
1527,4
1533,7
0.412%

top_file
384,1
388,6
1.15%

The performance impact seems to be low from the eBPF POV. Users interested in super efficient gadgets could create their own without using these helpers and providing only the specific information they're interested in.
Reviewing
There was a lot of copying and pasting when updating the gadgets, it's very likely that something went wrong, so please check them carefully.
TODO

[x] Fix #3459
[x] Understand how to handle the mount namespace id. Should it go under process?
[x] Add helpers for socket enricher (enrich process information fron an skb)
[x] Port missing gadgets
[x] Document annotations
[x] Release note
[ ] Update missing gadgets (help wanted)

[ ] fnsnotify: Depends on #3460, will be done later on
[ ] trace_oomkill (@eiffel-fl ?)
[ ] trace_tcpdrop (@blanquicet ?)

Release Notes
(under "Breaking changes")
The format of the json output for the image-based gadgets was changed on #3506. Now all the process related information such as command name (comm), process identifier (pid), thread identifier (tid), user identifier (uid), group identifier (gid), parent process (pcomm) and mount namespace inode id (mntns_id) is consolidated on the "proc" leaf. If you are consuming the json output from the image-based gadgets, please update the unmarshaling logic. This is usually done by updating the types used to represent the events generated by the gadget.
Please notice that this change only affects the image-based gadgets (the ones under gadgets/). Built-in ones (under pkg/gadgets/) aren't affected by this).
Before
The process related fields where printed on the event root, and according to the gadget some fields could be present or not.
{
  "comm": "cat",
  "mntns_id":<PHONE_NUMBER>,
  "pid": 51092,
  "tid": 51092,
  "gid": 0,
  "group": "root",
  "uid": 0,
  "user": "root"
}

Now
{

  "proc": {
    "comm": "cat",
    "mntns_id":<PHONE_NUMBER>,
    "parent": {
      "comm": "sh",
      "pid": 50991
    },
    "pid": 51092,
    "tid": 51092,
    "creds": {
      "gid": 0,
      "group": "root",
      "uid": 0,
      "user": "root"
    }
  },
}

I removed the fsnotify from this PR as it depends on #3460. We can handle it later on.

Failing test is related to #3361.


Smabuk/theming

What this does

Reverts App.cs and NavMenu.razor.css back to their templates
Uses lower case for the theme names rather than mixed which can get confusing and looks more natural in the html
Uses a media query to handle system preferences
The html data-theme attribute switches between "dark", "light" and not being set

Down sides
For ease of creating this I've just duplicated the values for "light" and "dark" but these could easily be derived from a master set of values
Not related
I had to put the base line back into the index.html to get this to run on my machine

Looks good to me, thank you. I reckon we'll take a look at in our next stream and merge it in.
Yeah, the base URL is a pain. We have a plan to fix it with a devops script.
I'd be keen to cull the reused variable so they share a base, but we can work that out on stream.


[Documentation suggestion] What's under the hood for each platform

I think it'd be great if there's an explanation of how this library does its magic for each platform. Something like, "Android above version X uses native support (no extra download). Otherwise, it uses the embedded native library."
So that the users of this library can make informed decisions about whether it's worth it to use this or default to the built-in ones (e.g., for WebP).
What do you think?

Thanks for the suggestion. The plugin comes with libavif binaries on all platforms it supports and uses them regardless of the platform's native support. I think it would be beneficial to the user if we add a section to the readme about this.
We can also document the binary size impact of the plugin on each platform. I haven't measured it on all platforms but for example on Android, the plugin adds between 1-2 MBs to the APK. (release build with ABI splitting)


US134385 - Use the SubscriberRegistryController in filter

Add the new SubscriberRegistryController directly into d2l-filter to be able to register subscribers interested in getting updates on which filters are active.  The only consumer of this for now will be a new version of the applied filters lab component, but eventually that will be fixed up and moved into core once we have an official tag component.  PR for that is here:

Will need to use Sauce to figure out why Safari is unhappy when I get back.


ImportError: cannot import name DEFAULT_CIPHERS (Fixed)

An error occurred when I run the program under python3.11 (and all the libs are the latest) :
Fixing method : install a version < 2. pip install 'urllib3<2'

Traceback (most recent call last):
File "D:\Programs\KMP-Kemono-Downloader-<IP_ADDRESS>-pre\KMP-Kemono-Downloader-<IP_ADDRESS>-pre\KMPDownloader.py", line 12, in
import cfscrape
File "C:\Program Files\Python311\Lib\site-packages\cfscrape_init_.py", line 19, in
from urllib3.util.ssl_ import create_urllib3_context, DEFAULT_CIPHERS
ImportError: cannot import name 'DEFAULT_CIPHERS' from 'urllib3.util.ssl_' (C:\Program Files\Python311\Lib\site-packages\urllib3\util\ssl_.py)

However I found the problem, DEFAULT_CIPHERS is removed from urllib3 2.0.0 while the program still imports it.
Thanks to https://github.com/Yelp/elastalert/issues/2306

Thanks for bringing up this issue and sorry for the late response. I will be returning to working on this project again soon and will include a script to fix this issue.


language detection

language detection of the dataset

use langdetect
https://pypi.org/project/langdetect/


Assign persistent IDs to StreamBlock items

This PR adds an ID field to the data model for items in a StreamField / StreamBlock. The database JSON representation of a StreamField now looks like:
[
    {"type": "heading", "value": "Hello world", "id": "3482a437-4f50-49c4-bfbf-6db8cc8e081d"},
    {"type": "paragraph", "value": "
Lorem ipsum...
", "id": "49359c63-2100-4da7-9baa-433bb589b58b"}
]

IDs are assigned a random UUID on save (if not already present), are preserved in the edit form, and are exposed in the API.
The immediate use-case for this feature is for a client site that's using page save/publish signals along with the Wagtail API, to push StreamField-based page content to an external content store. Keeping track of IDs means that we can update existing records in that content store where appropriate, rather than replacing the whole lot on every publish. It could also potentially be used to make the revision comparison view smarter, since we can now definitively tell when blocks have been added, deleted or reordered, rather than having to guess based on comparing content.

Hi @kaedroho and @gasman , please how can I access the new 'id' attribute in a template? It seems not to be visible.
Thank you

@pyMan You can access it as the id attribute of the StreamChild object (i.e. the object you get when you iterate over the StreamField value):
{% for block in page.body %}

{% include_block block %}

{% endfor %}

If the block defines a template of its own (to be rendered with {% include_block %}), you won't be able to access the ID there, because that template only has access to the block value, not the StreamChild. You'd have to pass it explicitly instead:
{% for block in page.body %}

{% include_block block with block_id=block.id %}

{% endfor %}

Thank you @gasman, I was missing to pass the id explicitly since I'm using the include_block template tag.


Tests for 3.1.1

Let's see how the tests handle this. The odds are that there are some schema changes causing #98.

Thank you :)

Looks like Travis does not support YAML 1.0 anymore (just a guess tho) since builds are not starting. I'll provide a fix soon.

Fixed it already :)

Never mind then. Looks like they really dropped the YAML 1.0 support in July. The config was not changed but 'missing config' just popped up.


robots.txt fetch fails

This is a regression. Was working as of 2.8
https://github.com/hydroshare/hydroshare/issues/5125

also

interestingly, robots.txt is functional on https://dev-1.hydroshare.org/robots.txt

settings look okay to me
https://www.hydroshare.org/admin/robots/rule/

also https://www.hydroshare.org/sitemap.xml timed out for me


Update the header

This:

adds the environment indicator (staging, integration, development) to the header. From user research we've seen that it might prevent confusion, especially because we currently show the tag in all interfaces
changes the production colour from blue to red (against my strong objections!!)
use GOV.UK Publishing as the product name (product_name is no longer configurable)

https://trello.com/c/05wMNlxg

Aw I like the blue too.
Could we maintain the customisability of the name and whether the environment label is there in this heading, but default to "Publishing"? Reason I ask is that we've been trying to use this header for the component guide itself.


Static documentation with Github Pages

It would be great, if the documentation was hosted in Github Pages.
Using a framework like: docusaurus or docsify.
The reading and user experience is much better than Github, in addition to having support for multiple languages, navigation bar and advanced search.
The project documentation is very extensive, leaving some people lost.
The readme.md of the project could be simplified and leave the documentation responsibility for Github Pages.
Leaving readme.md cleaner and exploring further the purposes and goals of the tool.

I definitely feel like the documentation can be improved. Especially for first time users.
I don't think I'd go the GitHub pages route or anything move complex than the current markdown pages being used right now though. I like everything in one place and without any more tech than is being currently used.
If Feri got ridiculously popular I envision a dedicated site with all sorts of videos, interactive goodies, and such. Feri may never get that popular though. ;)


Integration with Paket Powershell

Could that be bundled in with this project so when installing you automatically get the Powershell components as well? If you need to go "out of the box" with Paket VS currently e.g. custom commands or whatnot, I currently have to resort to going back to the command line or opening another tool e.g. Atom (with which I love the paket integration, but hate the slow startup time + performance with an equal passion).
If we had the Powershell add-in then that would complete the integration within VS for me.

/cc @ctaggart


[5.4] merge config recursive

I have use this way to merge config many time.  It's has been very useful for me to merge recursively and allow override with many package .
So here my change  apply to the merge config function in service provider.

I don't plan to add this to the core but you could add it to your own package.


Incorrect SQL with row_number() and arithmetic operation

Hi, I'm using Ibis with Impala. I'm getting the row numbers in a table like so:
w = ibis.window(order_by=table['age'])
expr = table.mutate(new_col=ibis.row_number().over(w))
print(ibis.impala.compile(expr))
print db_con.execute(expr)

As expected, this produces:
SELECT *, row_number() OVER (ORDER BY `age`) - 1 AS `new_col`
FROM ibis_testing.`insight_oracle_test_table3`

    age    new_col
0   8.0    0
1   10.0   1
2   40.0   2
3   45.0   3
4   70.0   4
5   80.0   5
6   NaN    6

However, if I try to perform any arithmetic operation on the row number (e.g. divide by 2), the wrong SQL gets produced because it's not surrounded in parentheses. E.g.:
w = ibis.window(order_by=table['age'])
expr = table.mutate(new_col=ibis.row_number().over(w) / 2)
print(ibis.impala.compile(expr))
print db_con.execute(expr)

produces:
SELECT *, row_number() OVER (ORDER BY `age`) - 1 / 2 AS `new_col`
FROM ibis_testing.`insight_oracle_test_table3`

    age    new_col
0   8.0    0.5
1   10.0   1.5
2   40.0   2.5
3   45.0   3.5
4   70.0   4.5
5   80.0   5.5
6   NaN    6.5

So here, instead of dividing by 2, it subtracted by 1/2. (I need to do this operation in order to be able to calculate quantiles.)

@paavanmathur Thanks for the report!


Tuya IoT Cloud Terms Updated

          Jason,

Pardon the intrusion i received this notice from Tuya.
I haven't had to pay for the Tuya site in three years do i have to pay something for the service now?
Noticed my "Groups" were also depreciated?
Highest Regards,
Bailey

Tuya IoT Platform Service Notification
Dear developer,
Your subscription IoT Core, Cloud Develop Base Resource Trial remaining usage is 0.00%. Once the quota limit is exceeded, the service will be suspended.
If you want to raise the limit,
you can click the link below to upgrade your plan.
https://www.tuya.com/vas/commodity/IOT_CORE_V2
You are the account owner of the Tuya IoT Platform and have the right to be notified of any change in your subscription plan. If you do not use the service mentioned above in your business, please ignore this email.
If you are unclear about how to proceed, please contact your account manager or file a service ticket.
This is an automated email, please do not reply to this email. If you have any questions, please email us at @.***

Originally posted by @sjpbailey in https://github.com/jasonacox/tinytuya/issues/414#issuecomment-1792374811

Thanks @sjpbailey , I'm moving this to a new issue.  It looks like there was a recent change.  See https://developer.tuya.com/en/docs/iot/membership-service?id=K9m8k45jwvg9j.

The benefits of IoT Core were updated on October 31, 2023. See the table below for the latest pricing details. Your existing subscriptions will not be affected. If you choose to upgrade or renew your subscription, you will be eligible for the benefits listed in the table below.

Tuya has always charged for the use of their IoT cloud. Most users only need or sign up for the trial account to get the local keys to manage devices via TinyTuya (local access). However, as you can see on the Tuya site, the trial account is limited in number of requests and features and prohibited for commercial use:

The Trial Edition only applies to individual developers or works for debugging purposes. Commercial use is prohibited.

This change could impact this community (some of us with a lot of devices). However, It doesn't seem to limit the duration of the trial account just the amount of use per month, but I suppose we will see.

Thank You Jason for Your Valuable Time!
Hello Jason!
So i have been using the cloud instead of  local for Cloud control.
Your code relies on stored json files to reference, because of the stored Json file referenced in your Module needed to find local devices. UDI would not let me write to the customers router, individual json files for local control. I have to grab it online and use it then. No future reference is allowed!
(any ideas?)
If a customer has to do python or anything then UDI said no to file storage and reference with devices not being, automatically discoverable without customer intervention.
How it works:
I wrote code to cloud API interface to Tuya and users were using the API trial to get their devices exposed to the Tuya Cloud.
Then, Post, Put, Pull worked fine with NO, "oh someone might get to my network".

Then all API's out there are dangerous?!?
Now it is on a the UDI store that customers can install their devices to have them on Universal Devices frontend for program routines.
If this is "commercial use prohibited" then i am out of business as most people are cheap and think their device purchase covers API costs.
I need to look at the monthly cost, if too high then i need to get ready to throw all my Tuya products in the recycle bin and buy Z-wave devices that i HATE.
Thank You again for Your Teams efforts and one of the coolest projects i have been following!
https://polyglot.universal-devices.com/

Oh it has been code, code, coding and three year for $45!
The band Yes said it best, "10% of nothing."

Ha! I understand.  You could look at https://github.com/blakadder/tuya-uncover to extract the keys if that is all you are needing.

Tuya seem to have released a "Smart Home Basic" Service API at some point recently.
This is added by default as a free replacement for the IOT Core service.
Service: https://iot.tuya.com/cloud/products/detail/?abilityId=1667049392684671055
Docs: https://developer.tuya.com/en/docs/cloud/smart-home-basic-service?id=Kconis1yii4be
Perhaps this can be TinyTuya can be refactored to use this API instead?

Hello Jason,
Yes also could not get a straight answer on commercial use from Tuya.
However they said it was complex and Home Assistant has a green light and is free to use Basic API within up to 10 devices Tuya with TinyTuya!
Highest Regards,
Bailey
Tuyas Response,
"Hi there,
Let me explain it in more detail:

As we all know, the Home Assistant plugin is free to use. This means that the open API in Home Assistant is not intended for commercial use. Customers are using the trial version of the IOT core service.

It is difficult to explain how the third-party plugin works since it is not an official plugin.

You have two options to choose from:
3.1. You can allow your customers to use the trial version of the IOT core service.
3.2. Alternatively, you can purchase the IOT core service, but your web application may not be free to use in your business model.

Please let me know if there's anything else I can assist you"

On Jan 3, 2024, at 6:22 PM, phrak @.***> wrote:
Tuya seem to have released a "Smart Home Basic" Service API at some point recently.
This is added by default as a free replacement for the IOT Core service.
Service: https://iot.tuya.com/cloud/products/detail/?abilityId=1667049392684671055
Docs: https://developer.tuya.com/en/docs/cloud/smart-home-basic-service?id=Kconis1yii4be
Perhaps this can be TinyTuya can be refactored to use this API instead?
—
Reply to this email directly, view it on GitHub https://github.com/jasonacox/tinytuya/issues/419#issuecomment-1876220669, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMRVB3LL6LGGWMVJDIQO7CDYMYG6VAVCNFSM6AAAAAA65VRIA2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZWGIZDANRWHE.
You are receiving this because you were mentioned.

https://github.com/jasonacox/tinytuya/issues/441


Authorizations need to comma separated.

Authorizations need to be comma separated.
@tfitzmac as  discussed in Teams

@MSFTMith : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MSFTMith - thanks for making this fix

#sign-off


main doesn't pass it's own test

(MLFlowClient) pkg> status
Project MLFlowClient v0.4.6
Status `~/.julia/dev/MLFlowClient/Project.toml`
  [48062228] FilePathsBase v0.9.21
  [cd3eb016] HTTP v1.10.5
  [682c06a0] JSON v0.21.4
  [605ecd9f] ShowCases v0.1.0
  [5c2747f8] URIs v1.5.1
  [ade2ca70] Dates
  [cf7118a7] UUIDs

(MLFlowClient) pkg> test
     Testing MLFlowClient
      Status `/tmp/jl_1EBWsE/Project.toml`
  [48062228] FilePathsBase v0.9.21
  [cd3eb016] HTTP v1.10.5
  [682c06a0] JSON v0.21.4
  [64a0f543] MLFlowClient v0.4.6 `~/.julia/dev/MLFlowClient`
  [605ecd9f] ShowCases v0.1.0
  [5c2747f8] URIs v1.5.1
  [ade2ca70] Dates
  [8dfed614] Test
  [cf7118a7] UUIDs
      Status `/tmp/jl_1EBWsE/Manifest.toml`
  [d1d4a3ce] BitFlags v0.1.8
  [944b1d66] CodecZlib v0.7.4
  [34da2185] Compat v4.14.0
  [f0e56b4a] ConcurrentUtilities v2.4.1
  [460bff9d] ExceptionUnwrapping v0.1.10
  [48062228] FilePathsBase v0.9.21
  [cd3eb016] HTTP v1.10.5
  [692b3bcd] JLLWrappers v1.5.0
  [682c06a0] JSON v0.21.4
  [e6f89c97] LoggingExtras v1.0.3
  [64a0f543] MLFlowClient v0.4.6 `~/.julia/dev/MLFlowClient`
  [739be429] MbedTLS v1.1.9
  [4d8831e6] OpenSSL v1.4.2
  [69de0a69] Parsers v2.8.1
  [aea7be01] PrecompileTools v1.2.1
  [21216c6a] Preferences v1.4.3
  [605ecd9f] ShowCases v0.1.0
  [777ac1f9] SimpleBufferStream v1.1.0
  [3bb67fe8] TranscodingStreams v0.10.7
  [5c2747f8] URIs v1.5.1
  [458c3c95] OpenSSL_jll v3.0.13+1
  [56f22d72] Artifacts
  [2a0f44e3] Base64
  [ade2ca70] Dates
  [b77e0a4c] InteractiveUtils
  [8f399da3] Libdl
  [56ddb016] Logging
  [d6f4376e] Markdown
  [a63ad114] Mmap
  [ca575930] NetworkOptions v1.2.0
  [de0858da] Printf
  [9a3f8284] Random
  [ea8e919c] SHA v0.7.0
  [9e88b42a] Serialization
  [6462fe0b] Sockets
  [fa267f1f] TOML v1.0.3
  [8dfed614] Test
  [cf7118a7] UUIDs
  [4ec0a83e] Unicode
  [c8ffd9c3] MbedTLS_jll v2.28.2+0
  [14a3606d] MozillaCACerts_jll v2022.10.11
  [83775a58] Zlib_jll v1.2.13+0
     Testing Running tests...
Test Summary:    | Pass  Total  Time
createexperiment |    2      2  0.6s
Test Summary:                      | Pass  Total  Time
getexperiment                      |    5      5  0.1s
  getexperiment_by_experiment_id   |    2      2  0.0s
  getexperiment_by_experiment_name |    2      2  0.1s
  getexperiment_not_found          |    1      1  0.0s
Test Summary:         | Pass  Total  Time
getorcreateexperiment |    4      4  0.1s
deleteexperiment: Test Failed at /home/anabrid/.julia/dev/MLFlowClient/test/test_experiments.jl:53
  Expression: length(experiments) == 1
   Evaluated: 2 == 1

Stacktrace:
 [1] macro expansion
   @ ~/.julia/juliaup/julia-1.9.4+0.x64.linux.gnu/share/julia/stdlib/v1.9/Test/src/Test.jl:478 [inlined]
 [2] macro expansion
   @ ~/.julia/dev/MLFlowClient/test/test_experiments.jl:53 [inlined]
 [3] macro expansion
   @ ~/.julia/juliaup/julia-1.9.4+0.x64.linux.gnu/share/julia/stdlib/v1.9/Test/src/Test.jl:1498 [inlined]
 [4] top-level scope
   @ ~/.julia/dev/MLFlowClient/test/test_experiments.jl:48
Test Summary:    | Fail  Total  Time
deleteexperiment |    1      1  0.7s
ERROR: LoadError: Some tests did not pass: 0 passed, 1 failed, 0 errored, 0 broken.
in expression starting at /home/anabrid/.julia/dev/MLFlowClient/test/test_experiments.jl:47
in expression starting at /home/anabrid/.julia/dev/MLFlowClient/test/runtests.jl:3
ERROR: Package MLFlowClient errored during testing

shell> git status
On branch main
Your branch is up to date with 'origin/main'.

nothing to commit, working tree clean

shell> git show HEAD
commit a31b41a91dc32107fa7dcd6ad5a581c116a4ff9c (HEAD -> main, origin/main, origin/gitlab_client, origin/HEAD)
Author: Jose Esparza<EMAIL_ADDRESS>Date:   Tue Mar 5 11:23:19 2024 -0500

    Bump version

diff --git a/Project.toml b/Project.toml
index 6c4ebe8..202535a 100644
--- a/Project.toml
+++ b/Project.toml
@@ -1,7 +1,7 @@
 name = "MLFlowClient"
 uuid = "64a0f543-368b-4a9a-827a-e71edb2a0b83"
 authors = ["@deyandyankov, @pebeto, and contributors"]
-version = "0.4.5"
+version = "0.4.6"

 [deps]
 Dates = "ade2ca70-3891-5945-98fb-dc099432e06a"

shell> git log
commit a31b41a91dc32107fa7dcd6ad5a581c116a4ff9c (HEAD -> main, origin/main, origin/gitlab_client, origin/HEAD)
Author: Jose Esparza<EMAIL_ADDRESS>Date:   Tue Mar 5 11:23:19 2024 -0500

    Bump version

commit 776695b6bb4d92d732b162ae92de72b8ebbf4aba
Merge: de98ac2 60a2535
Author: Jose Esparza<EMAIL_ADDRESS>Date:   Mon Mar 4 18:14:18 2024 -0500

    Merge pull request #37 from pebeto/improving_code_base

    Improving code base

commit 60a25356276ab61aed12eba7af45b944adc85d44
Author: Jose Esparza<EMAIL_ADDRESS>Date:   Mon Mar 4 18:11:11 2024 -0500

    Ignoring `deprecated.jl` in coverage

commit dfed176c092e0194d6e220434cfd06a7d346b41c
Author: Jose Esparza<EMAIL_ADDRESS>Date:   Sun Jan 7 23:29:12 2024 -0500

    Increasing coverage

commit 3190751aef412958151d509d8866c8877636ba38
Author: Jose Esparza<EMAIL_ADDRESS>Date:   Sun Jan 7 23:13:06 2024 -0500

    Fixing Documenter failing on pipeline

commit 8886aa835e9fb176e1c774e36e69593c983c440d
Author: Jose Esparza<EMAIL_ADDRESS>Date:   Sun Jan 7 22:22:54 2024 -0500

    Increasing maintainability by nice function separation in files

commit de98ac2fddbd49e00a3724d05184d1462f8a291c
Author: deyandyankov<EMAIL_ADDRESS>Date:   Sun Jan 7 21:58:13 2024 +0000

    @JuliaRegistrator register

commit 26674e04bf1313942d78f682924e6683ac6106c7
Author: deyandyankov<EMAIL_ADDRESS>Date:   Sun Jan 7 21:54:10 2024 +0000

    bump version

commit e27ac1026627327d58fd4b63aeeef62c738b0977
Author: Jose Esparza<EMAIL_ADDRESS>Date:   Sun Jan 7 16:53:07 2024 -0500

    Migrating from /api to /ajax-api endpoint (#36)

commit 5a4237fbd28a611c40d37b6cc705a3c0b2de2c50
Author: deyandyankov<EMAIL_ADDRESS>Date:   Thu Aug 10 09:54:40 2023 +0100

I tried to bisect the issue but output of test changed non deterministically as i walked back the tree and it now works for a31b41a91dc32107fa7dcd6ad5a581c116a4ff9c which is cursed. I have no idea why this would happen.

(@v1.9) pkg> activate .
  Activating project at `~/.julia/dev/MLFlowClient`

(MLFlowClient) pkg> test
     Testing MLFlowClient
      Status `/tmp/jl_q5UEKV/Project.toml`
  [48062228] FilePathsBase v0.9.21
  [cd3eb016] HTTP v1.10.5
  [682c06a0] JSON v0.21.4
  [64a0f543] MLFlowClient v0.4.6 `~/.julia/dev/MLFlowClient`
  [605ecd9f] ShowCases v0.1.0
  [5c2747f8] URIs v1.5.1
  [ade2ca70] Dates
  [8dfed614] Test
  [cf7118a7] UUIDs
      Status `/tmp/jl_q5UEKV/Manifest.toml`
  [d1d4a3ce] BitFlags v0.1.8
  [944b1d66] CodecZlib v0.7.4
  [34da2185] Compat v4.14.0
  [f0e56b4a] ConcurrentUtilities v2.4.1
  [460bff9d] ExceptionUnwrapping v0.1.10
  [48062228] FilePathsBase v0.9.21
  [cd3eb016] HTTP v1.10.5
  [692b3bcd] JLLWrappers v1.5.0
  [682c06a0] JSON v0.21.4
  [e6f89c97] LoggingExtras v1.0.3
  [64a0f543] MLFlowClient v0.4.6 `~/.julia/dev/MLFlowClient`
  [739be429] MbedTLS v1.1.9
  [4d8831e6] OpenSSL v1.4.2
  [69de0a69] Parsers v2.8.1
  [aea7be01] PrecompileTools v1.2.1
  [21216c6a] Preferences v1.4.3
  [605ecd9f] ShowCases v0.1.0
  [777ac1f9] SimpleBufferStream v1.1.0
  [3bb67fe8] TranscodingStreams v0.10.7
  [5c2747f8] URIs v1.5.1
  [458c3c95] OpenSSL_jll v3.0.13+1
  [56f22d72] Artifacts
  [2a0f44e3] Base64
  [ade2ca70] Dates
  [b77e0a4c] InteractiveUtils
  [8f399da3] Libdl
  [56ddb016] Logging
  [d6f4376e] Markdown
  [a63ad114] Mmap
  [ca575930] NetworkOptions v1.2.0
  [de0858da] Printf
  [9a3f8284] Random
  [ea8e919c] SHA v0.7.0
  [9e88b42a] Serialization
  [6462fe0b] Sockets
  [fa267f1f] TOML v1.0.3
  [8dfed614] Test
  [cf7118a7] UUIDs
  [4ec0a83e] Unicode
  [c8ffd9c3] MbedTLS_jll v2.28.2+0
  [14a3606d] MozillaCACerts_jll v2022.10.11
  [83775a58] Zlib_jll v1.2.13+0
Precompiling project...
  1 dependency successfully precompiled in 1 seconds. 24 already precompiled.
     Testing Running tests...
Test Summary:    | Pass  Total  Time
createexperiment |    2      2  0.9s
Test Summary:                      | Pass  Total  Time
getexperiment                      |    5      5  0.1s
  getexperiment_by_experiment_id   |    2      2  0.0s
  getexperiment_by_experiment_name |    2      2  0.1s
  getexperiment_not_found          |    1      1  0.0s
Test Summary:         | Pass  Total  Time
getorcreateexperiment |    4      4  0.1s
Test Summary:    | Pass  Total  Time
deleteexperiment |    1      1  0.4s
Test Summary:     | Pass  Total  Time
restoreexperiment |    2      2  0.0s
Test Summary:                            | Pass  Total  Time
searchexperiments                        |    6      6  0.1s
  searchexperiments_get_all              |    1      1  0.0s
  searchexperiments_by_filter            |    2      2  0.0s
  searchexperiments_by_filter_attributes |    2      2  0.0s
  searchexperiments_filter_exception     |    1      1  0.0s
Test Summary:                  | Pass  Total  Time
createrun                      |    4      4  0.6s
  createrun_by_experiment_id   |    2      2  0.5s
  createrun_by_experiment_type |    2      2  0.0s
Test Summary: | Pass  Total  Time
getrun        |    2      2  0.1s
Test Summary:                              | Pass  Total  Time
updaterun                                  |   25     25  0.5s
  updaterun_by_run_id                      |    5      5  0.4s
  updaterun_by_run_info                    |    5      5  0.0s
  updaterun_byrun                          |    5      5  0.0s
  updaterun_by_run_info_and_defined_status |    5      5  0.0s
  updaterun_by_run_and_defined_status      |    5      5  0.0s
Test Summary:           | Pass  Total  Time
deleterun               |    2      2  0.2s
  deleterun_by_run_info |    1      1  0.2s
  deleterun_by_run      |    1      1  0.0s
Test Summary:                     | Pass  Total  Time
searchruns                        |   10     10  1.5s
  searchruns_by_experiment_id     |    1      1  0.3s
  searchruns_by_experiment        |    1      1  0.1s
  searchruns_by_experiments_array |    1      1  0.2s
  searchruns_by_filter            |    1      1  0.0s
  searchruns_by_filter_params     |    1      1  0.1s
  searchruns_filter_exception     |    1      1  0.0s
  runs_get_methods                |    4      4  0.1s
Test Summary:                          | Pass  Total  Time
logparam                               |    8      8  0.1s
  logparam_by_run_id_and_key_value     |    2      2  0.0s
  logparam_by_run_info_and_key_value   |    2      2  0.0s
  logparam_by_run_and_key_value        |    2      2  0.0s
  logparam_by_union_and_dict_key_value |    2      2  0.0s
Test Summary:                           | Pass  Total  Time
logmetric                               |   12     12  0.2s
  logmetric_by_run_id_and_key_value     |    3      3  0.1s
  logmetric_by_run_info_and_key_value   |    3      3  0.0s
  logmetric_by_run_and_key_value        |    3      3  0.0s
  logmetric_by_union_and_key_arrayvalue |    3      3  0.0s
Test Summary:                            | Pass  Broken  Total  Time
logartifact                              |    5       1      6  0.3s
  logartifact_by_run_and_filenameanddata |    1              1  0.0s
  logartifact_by_run_id_and_file         |    1              1  0.1s
  logartifact_by_run_and_file            |    1              1  0.0s
  logartifact_by_run_info_and_file       |    1              1  0.0s
  logartifact_using_IOBuffer             |    1              1  0.0s
  logartifact_error                      |            1      1  0.0s
Test Summary:       | Pass  Total  Time
logbatch            |    4      4  0.6s
  logbatch_by_types |    2      2  0.4s
  logbatch_by_dicts |    2      2  0.2s
Test Summary:               | Pass  Total  Time
listartifacts               |    3      3  0.2s
  listartifacts_by_run_id   |    1      1  0.2s
  listartifacts_by_run      |    1      1  0.0s
  listartifacts_by_run_info |    1      1  0.0s
     Testing MLFlowClient tests passed

Hi.
The error is happening because the previous run doesn't ended correctly. To improve this here, we need to define a post-test routine to delete the experiments even when a test fails.
This can be flagged as an improvement in the next version for the test suite, but it's not a bug.

Right. The tests were written with the assumption that the mlflow test instance will be wiped out after test runs with a separate process and not the tests themselves. This way if the tests fail, one could debug what happened using test output and existing objects in mlflow before wiping out mlflow. So the behaviour was actually intentional.
The local development workflow is:
have an mlflow instance running
run the tests
wipe out the instance
run the tests
[...]

The same thing happens when the tests are run in github. A new mlflow instance is spun up and then its automatically wiped out by the job rather than cleaned up by the tests.


New API: Similar to EASTL::hash_map::find_as

I would like to request the addition of a new API that is similar to EASTL::hash_map::find_as. This new API would provide a convenient way to search for a specific value in an unordered_dense::map using a custom comparison function.
references:
https://github.com/electronicarts/EASTL/blob/05f4b4aef33f2f3ded08f19fa97f5a27ff35ff9f/include/EASTL/hash_map.h#L84
https://github.com/electronicarts/EASTL/blob/05f4b4aef33f2f3ded08f19fa97f5a27ff35ff9f/include/EASTL/internal/hashtable.h#L1075

I don't think that is needed, this can be done with is_transparent feature. See here: https://github.com/martinus/unordered_dense#324-heterogeneous-overloads-using-is_transparent


[Bug]: The interface for recovering bad disk is timing out.

Contact Details
<EMAIL_ADDRESS>Is there an existing issue for this?

[X] I have searched all the existing issues

Priority
low (Default)
Environment
- CubeFS version:3.4.0

Current Behavior
After canceling the decommission operation for a bad disk, the interface to recover the bad disk times out.
Expected Behavior
The interface for recovering bad disk returns immediately after being called.
Steps To Reproduce
1. Decommission a bad disk.
2. Cancel the deommission progress for bad disk.
3. Recover the bad disk.

CubeFS Log
No response
Anything else? (Additional Context)
No response

closed by #3563


Uninstalling on Windows does not remove right click context menu entry

Uninstalling Hyper, on Windows, does remove the context menu (right click) entry.

Open \HKEY_CLASSES_ROOT\Directory\Background\shell\ (with regedit) and remove the Hyper folder.

It seems that it's fixed in version 1.2.1.
I downloaded the latest Windows version today, and uninstall it from control panel, the context goes away.

Can confirm as not fixed in 1.2.1 (Windows 10, latest updates, Hyper 1.2.1).

This is still not fixed in 1.3.3.

Still not fixed... Seems small but it leaves a bad taste in my mouth.

For me worked: HKEY_LOCAL_MACHINE\SOFTWARE\Classes\LibraryFolder\background\shell\git_shell and \git_gui

They uninstaller do nothing, and leave lots of garbage in registry, thats why i love portable soft,
Try open regedit and just search for 5KPlayer after uninstalling, and you will be shocked

may be a duplicate of #3090

Tested on latest canary, seems to be fixed, no results on searching 'Open Hyper here' in registry after uninstall.


Screenshot cover doesn't save scanline effect

When taking a screnshoot for cover (F7) , the saved image is saved without the scanline effect.
Not a big defect.

Yeah I'm pretty sure that would mean allowing any color to be used for the cover image. If the colors are indexed I don't think you could store more than 16 colors at a time.

That make sense.


check_ceph_osd_db: Decode bytes output to string

Avoids TypeError: cannot use a string pattern on a bytes-like object when executing re.findall().
Popen.communicate() returns bytes if the streams weren't opened in text mode, which is not the case here, as far as I can tell. See https://docs.python.org/3/library/subprocess.html#subprocess.Popen.communicate

Noted this issue and it fixed the problem for me.


Remove unused variable from LockTime example

Following up on a comment from #3026 and removing an unused variable from the LockTime example

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 remained the same at 83.153%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19634

Relevant Lines:
23612

💛  - Coveralls

Rebased


Azure App Service gRPC support

Could you please add a link to info about the experimental support for gRPC support in Azure App Services so we get a chance to test it? And it says only in some regions, could you add a link where we could check what reagions are supported?
BR
Marcus

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

ID: 76d3f0ff-f65f-97d2-940f-4509b0d425bd
Version Independent ID: ea62de0b-d7e5-e543-b892-a587d6e9dc84
Content: gRPC on .NET supported platforms
Content Source: aspnetcore/grpc/supported-platforms.md
Product: aspnet-core
Technology: aspnetcore-grpc
GitHub Login: @JamesNK
Microsoft Alias: jamesnk

Thank you for contacting us. Due to a lack of activity on this discussion issue we're closing it in an effort to keep our backlog manageable. If you believe there is a concern which hasn't been addressed, please file a new issue.


use setuptools_scm

@dtrifiro, I propose migrating to py-template straight away. Let me know if you need any help, I did that last week with scmrepo.

Codecov Report
Base: 90.67% // Head: 91.38% // Increases project coverage by +0.70% :tada:

Coverage data is based on head (d06f8bb) compared to base (086659f).
Patch coverage: 60.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #278      +/-   ##
==========================================
+ Coverage   90.67%   91.38%   +0.70%
==========================================
  Files          21       21
  Lines         751      720      -31
==========================================
- Hits          681      658      -23
+ Misses         70       62       -8

Impacted Files
Coverage Δ

dvclive/version.py
60.00% <60.00%> (-12.23%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Related: https://github.com/iterative/dvclive/issues/272

@dtrifiro, I propose migrating to py-template straight away. Let me know if you need any help, I did that last week with scmrepo.
Yeah, I think it's worth it.

Superseded by #281


Error: Building package ogre:x64-linux failed with: BUILD_FAILED

Building the ogre package failed on my windows machine using the windows subsystem for linux with ubuntu. The errors are:
CMake Error at ports/ogre/portfile.cmake:86 (file):
file must be called with at least two arguments.
Call Stack (most recent call first):
scripts/ports.cmake:71 (include)
Commenting out line 86 in ports/ogre/portfile.cmake makes the installation run through. Line 86: file(REMOVE ${REL_CFGS} ${DBG_CFGS}) seems to cause the issue.

@YgabrielsY, does this still be a problem for you, could you get latest souce and try again?
I can't repro this issue with WSL Ubuntu 18.04:
Computing installation plan...
The following packages will be built and installed:
ogre[core]:x64-linux
Starting package 1/1: ogre:x64-linux
Building package ogre[core]:x64-linux...
ogre currently requires the following library from the system package manager:
Xaw
It can be installed on Ubuntu systems via apt-get install libxaw7-dev
-- Using cached /home/ubuntu18/vcpkg/downloads/OGRECave-ogre-8083067c1835147de5d82015347d95c710e36bc0.tar.gz
-- Using source at /home/ubuntu18/vcpkg/buildtrees/ogre/src/c710e36bc0-fd78f2aff5
-- Configuring x64-linux-dbg
-- Configuring x64-linux-rel
-- Building x64-linux-dbg
-- Building x64-linux-rel
-- Installing: /home/ubuntu18/vcpkg/packages/ogre_x64-linux/share/ogre/copyright
-- Performing post-build validation
-- Performing post-build validation done
Building package ogre[core]:x64-linux... done
Installing package ogre[core]:x64-linux...
Installing package ogre[core]:x64-linux... done
Elapsed time for package ogre:x64-linux: 38.36 min
Total elapsed time: 38.36 min
The package ogre:x64-linux provides CMake targets:
find_package(OGRE CONFIG REQUIRED)
# Note: 17 target(s) were omitted.
target_link_libraries(main PRIVATE OgreHLMS OgreMain Codec_EXR OgreBites)

Thanks for coming back at me, no the problem doesn't persist anymore!


adding litmus tutorial

Adding a tutorial on how to use Litmus + Keptn to evaluate the resilience of microservices

Preview: https://6059e18d8cce000007b0bc0f--keptn-tutorials.netlify.app/

Thanks for finding the typos @johannes-b

thanks - will merge as soon as the release branch of the integration is ready


Allow pushing of non-master branches

Hello, thanks for this fantastic project @rcarmo! I just successfully deployed my first piku instance. :tada:
I'm about to issue a PR for something else but I wanted to write this down here while I can as it surprised me. I tried to push a non-master branch to my piku instance and nothing happened whereas on other PaaS this is not a problem and the app deploys as expected.

Yeah, well, that’s due to the very minimalist design. You can work around that by pushing that branch explicitly as master (I forget the right incantations, but I’ve done it).

On 22 May 2019, at 10:24, Chris McCormick<EMAIL_ADDRESS>wrote:
Hello, thanks for this fantastic project @rcarmo https://github.com/rcarmo! I just successfully deployed my first piku instance. 🎉
I'm about to issue a PR for something else but I wanted to write this down here while I can as it surprised me. I tried to push a non-master branch to my piku instance and nothing happened whereas on other PaaS this is not a problem and the app deploys as expected.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/rcarmo/piku/issues/22?email_source=notifications&email_token=AAC73267YUYYSBVMSRNHZNLPWUGMVA5CNFSM4HOSK5UKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GVFCTIQ, or mute the thread https://github.com/notifications/unsubscribe-auth/AAC7325FPLNMREZSR6CBACLPWUGMVANCNFSM4HOSK5UA.

Here you go Stackoverflow has this neatly presented answer.
Right now I don't intend to implement this myself, but I will leave this opened in case there's a PR to address it :wink:

Have you tired git push piku yourbranch:master?

Thank you! BTW I consider the minimalist design a major feature.

Just noticed that I had a TODO for it...


feat(parser): support "jsonb" as typed string

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This PR add a keyword JSONB and a variant AstDataType::Jsonb in parser, so that we can use jsonb as typed string.
select jsonb '{}';

This syntax is used in regress tests.
Checklist

[ ] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[x] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

added a new way to create jsonb value:
jsonb '{"key": 1}'

This can be added as an example in the table.

I believe the correct way to support this is not to add a new keyword, but to overcome this limitation in parser:
https://github.com/risingwavelabs/risingwave/blob/bb100a743b593e1c95527a4fe9728c8f43df0be7/src/sqlparser/src/parser.rs#L537-L548
Alternatively, we can also workaround regress tests by copying them out / update them in-place. This is what I did for int2 / int4 / int8.
The workaround feels bad as well. But keywords are a user-facing change.


Let's use a fake calendar, it will look better for the judges.

Fixed in b1700a1d5b5508870c9b179b3e5186a907697e4e, with a working calendar!.


can this project easily modified into react frontend?

React is my main frontend skill stack, and dont want to turn to vue now.

Hey, as stated in the README.md you can easily remove the frontend. But you will then have to create the frontend stuff yourself.
I did find a similar template where fastAPI and React were used together.

Hey, as stated in the README.md you can easily remove the frontend. But you will then have to create the frontend stuff yourself.
I did find a similar template where fastAPI and React were used together.

Thanks, I like your thing. And I have stared your project.

Thanks this is a dope project

Yep, it should be quite doable. In fact, I've been wanting to refactor it to use React. I have to update a lot of things in this project but first I want to finish with SQLModel to include it here. 🤓


feature request: How can I write a function to have actions.cd add path to zoxide before cd-ing into it?

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
Not asking for this to be implemented inside oil.nvim,
but I am wondering how I can write a function to have the ~ run a zoxide add $PATH command before :cding into it.
How can I start?  Is this a callback function to write?
Provide background
No response
What is the significance of this feature?
strongly desired
Additional details
No response

I am trying to get it to work with something like this:
                ['`'] = {
                    callback = function()
                        local oil = require('oil')
                        local dir = oil.get_cursor_entry()
                        print(vim.inspect(dir['name']))
                        -- vim.cmd('zoxide add ' .. dir)
                        require('oil.actions').cd()
                    end,
                },

But the require('oil.actions').cd()
Is throwing /Users/me/.config/nvim/lua/plugins/oil/init.lua:39: attempt to call field 'cd' (a table value) even with the vim.cmd commented out.

Try something like this:
['`'] = {
  callback = function()
    local dir = require("oil").get_current_dir()
    if dir then
      vim.system({'zoxide', 'add', dir})
      vim.cmd.cd({args = {dir}})
    else
      vim.notify("Cannot :cd; not in a directory", vim.log.levels.WARN)
    end
  end,
},

That worked beautifully.  Thank you @stevearc


precleanCompilation deletes .o files created by Rcpp::sourceCpp, causing compilation to fail for C++ linked via nimbleExternalCall

I noticed nimble's new precleanCompilation option #1393 in version 1.1.0, introduced to fix #1368, caused some of my project code to break on OSX (M2, Ventura 13.1) and Linux (Rocky Linux 8.6).  I can keep my project code running by setting nimble::nimbleOptions(precleanCompilation=FALSE), but wanted to document the issue in case others run into the same.  I also wanted to ask if you thought it might be possible to re-implement the precleanCompilation option to be more precise in which files it deletes?  Or, can you recommend another fix?  I recognize this is probably a low-priority need.  An MWE to reproduce and resolve the issue is below.
I use Rcpp::sourceCpp (Rcpp version 1.0.11) to manage C++ include paths while compiling cpp files with dependencies on the RcppEigen and BH packages, for example, so I can call Boost's differential equation solver from nimble models.  The precleanCompilation option deletes object files Rcpp::sourceCpp creates, which prevents nimble models with nimbleExternalCall functions from compiling.  The MWE does only depends on Rcpp and nimbleExternalCall, though.
Thanks!
# MWE modified from help("nimbleExternalCall")

library(nimble)

sink('add1.h')
cat('
 extern "C" {
 void my_internal_function(double *p, double*ans, int n);
 }
')
sink()
sink('add1.cpp')
cat('
 #include <cstdio>
 #include "add1.h"
 void my_internal_function(double *p, double *ans, int n) {
   printf("In my_internal_function\\n");
     /* cat reduces the double slash to single slash */
   for(int i = 0; i < n; i++)
     ans[i] = p[i] + 1.0;
 }
')
sink()

# modification: use Rcpp::sourceCpp to compile instead of direct system call
# system('g++ add1.cpp -c -o add1.o')

# compile files
Rcpp::sourceCpp(file = 'add1.cpp')

# find compiled file
ofile = normalizePath(
  path = dir(
    path = getOption("rcpp.cache.dir", tempdir()),
    pattern = 'add1.o',
    full.names = TRUE,
    recursive = TRUE
  ),
  winslash = '/'
)

Radd1 <- nimbleExternalCall(
  function(x = double(1), ans = double(1), n = integer()){},
  Cfun =  'my_internal_function',
  headerFile = file.path(getwd(), 'add1.h'),
  returnType = void(),
  oFile = ofile
)

# check: Rcpp output file exists
file.exists(ofile)

# Error: will not compile when  nimbleOptions('precleanCompilation') == TRUE
CRadd1 <- compileNimble(Radd1)

# precleanCompilation unintentionally deletes the required object file
file.exists(ofile)

# rebuild object file
Rcpp::sourceCpp(file = 'add1.cpp')

# Success: will compile when precleanCompilation is disabled
nimbleOptions(precleanCompilation=FALSE)
CRadd1 <- compileNimble(Radd1)

Note from Perry: Given that the --preclean fixed a bug we only ever saw during testing, we could decide that when nimbleExternalCall has been used, it will always turn off the --preclean option. Or we could at least add a note to its roxygen.

Both sound like fine options since at the moment it mostly feels like two corner cases bumping into each other, thanks!

For now I am just adding a note to the roxygen for nimbleExternalCall. On quick glance I didn't see an easy way to "partially" preclean.


Facebook Login iOS issue when Facebook app is also installed

🐛 Bug Report
When using Facebook login when the Facebook app is installed (requires a real device), the facebook app returns to the webview and does not actually log the user in. This leads to the user being stuck in a loop of trying to log in.
I see that there is another similar open issue which has had some attention but was not resolved. I hope someone else may be able to re-create this with the information I am posting here in this new issue
To Reproduce
https://user-images.githubusercontent.com/3537850/145878665-8a49582e-de14-49be-9ec1-2f3567d0788a.mp4

Clone the example code on a Mac
Replace the App ID, Client token and CFBundleURLSchemes in ios/RNFBSDKExample/Info.plist with your Facebook App ID
Install Facebook on your iOS device
Build and Run the example App on your iOS device via Xcode
Try to log in with a know user which would otherwise work on Android, or work on iOS when the Facebook app is not installed

For this test I created a new Facebook App in the Facebook developer console, and followed the relevant instructions for adding the Facebook login product to the app.
Expected Behavior
I would expect the Facebook app to return to the Example app after successfully authenticating
Code Example
I am using the Example App exactly as it is, aside from changes in Info.plist for the following keys
<key>FacebookAppID</key>
<string>MY_APP_ID</string>
<key>FacebookClientToken</key>
<string>MY_CLIENT_TOKEN</string>
<key>CFBundleURLTypes</key>
<array>
  <dict>
    <key>CFBundleURLSchemes</key>
    <array>
      <string>fbMY_APP_ID</string>
    </array>
  </dict>
</array>

Environment
System:
OS: macOS 12.0.1
CPU: (8) arm64 Apple M1
Memory: 1.74 GB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 17.2.0 - /opt/homebrew/bin/node
Yarn: Not Found
npm: 8.1.4 - /opt/homebrew/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.11.2 - /Users/davidwestgate/.rvm/rubies/ruby-3.0.2/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
Android SDK: Not Found
IDEs:
Android Studio: 4.2 AI-202.76<IP_ADDRESS>51085
Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
Languages:
Java: 11.0.13 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.66.3 => 0.66.3
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found

@djwesty I believe would be best if you could post your bug report together with the other issue and make sure you're using latest code available.As from last comment from Mike, all SDKs are updated now.

Still an issue with latest build.

I was having the same issue, you should update your AppDelegate.m to handle linking right. Facebook should be on top of RCTLinking Manager like the code below:

I was having the same issue, you should update your AppDelegate.m to handle linking right. Facebook should be on top of RCTLinking Manager like the code below:

This worked. Thanks

Hello everyone :)
I have the same issue but I am using SceneDelegate.
Is there any alternative of that works for SceneDelegate because right now the "openURL" method doesn't get called from my AppDelegate.
Thanks!

Ok I think I figure out a solution..
I added this method on my SceneDelegate:
- (void)scene:(UIScene *)scene openURLContexts:(NSSet<UIOpenURLContext *> *)URLContexts
{
  if ([URLContexts.allObjects count] > 0) {
    NSURL *url = URLContexts.allObjects.firstObject.URL;

    if ([url.absoluteString<EMAIL_ADDRESS>!= NSNotFound) {
      [[FBSDKApplicationDelegate sharedInstance]
      application: [UIApplication sharedApplication]
      openURL: url
      sourceApplication: nil
      annotation: nil];
    }
  }
}


Fixes to built-in token to_smart`to_classic` methods

What

Properly sign the args
Properly spend classic amount (signs were flipped)

Why
Bug fixes, getting this fixed while I continue working on tests.
Known limitations
N/A

Good fixes, thank you!


[BUG] reintroduce n_features in compute_dual_scaling: sometimes a test fails

Found while working on #131 : in #133 to lighten the API n_features was removed from compute_dual_scaling, and instead inferred in the function  as screened.shape[0].
This is not clean as in the inner solver of Celer, we do not screen, and pass, for screen,  dummy_screened = np.zeros(1), making n_features to be inferred as 1.
2 things are weird:

the failure is not deterministic, while there should not be randomness, neither in the data nor in the solver (possible source of randomness is np.argpartition ?)
ws_size == n_features evaluates to True in the outer loop (n_features is correctly inferred and n_features is passed for ws_size)  and to False in the inner (n_features is 1 so ws_size != n_features), which is in the end the desired behavior.

When the WS size reaches n_features (possible on small datasets), in the inner loop we start using screening since ws_size == n_features evaluates to True (so it feels like we are in the outer loop) There is no boundscheck performed on dummy_screened, hence random values are used, which may sometimes be true, causing the feature to be skipped.
The dual norm can thus be wrong (too low), d_obj is too high, a negative gap is sometimes reached.
Adding a variable in_inner (to replace ws_size == n_features) should fix it.

Codecov Report

Merging #136 into master will decrease coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage   83.16%   83.07%   -0.09%
==========================================
  Files          11       11
  Lines         974      975       +1
  Branches      145      145
==========================================
  Hits          810      810
- Misses        115      116       +1
  Partials       49       49

Impacted Files
Coverage Δ

celer/tests/test_lasso.py
100.00% <100.00%> (ø)

celer/dropin_sklearn.py
89.05% <0.00%> (-0.73%)
:arrow_down:

Continue to review full report at Codecov.

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


MS-Windows: Shift-Arrow keys do not work in command-line version

It is no longer possible to use Shift-Arrow keys to highlight text in the command-line version of Vim on Windows.
Reproduction steps:

Start command-line version of vim on Windows
iasd to insert some text in the buffer (so there's something to select)
Press Shift-Left (while still in insert mode)

I tested against binaries from https://github.com/vim/vim-win32-installer and was able to confirm that v8.0.1367 was the last build that worked correctly; v8.0.1376 exhibits the issue. Looking at the patch notes, I'm guessing this is a side effect of patch 8.0.1371, but have not looked further into this.
Vim 8.1 also exhibits this bug, even at latest patch 0369, although this inserts different garbage characters than 8.0.

I'm guessing this is a side effect of patch 8.0.1371, but have not looked further into this.

8.0.1371 was written by @mattn. So mentioning him.

Can you please check, if this is fixed by https://github.com/vim/vim/commit/2bc152ab53c4b01072edf6ec2ff61e504cb03cbe ?

Yes, that seems to have done the trick - 0450 still had the bug, 0451 does not. Thank you!


how to develop on personal smtp mail server?

the local deployment is using mailhog and it works fine. But I would like to use mailgun for testing purposes.
What  should  I  do in order to config correctly?

It's not something that's documented, but you should see SMTP settings in your nhost/config.yaml file:

We're working on a new configuration setup that will make this much easier in the future: https://github.com/nhost/cli/pull/597


[LB] - When empty uri is sent when creating LB , we get "503 Service Unavailable" when trying to access the LB ipaddress.

Server - build from mater on Apr 7
When creating LB , have some text and delete it from the "Check URL" field for LB config .
uri gets sent as empty string in this case:
healthyThreshold: 2
interval: 2000
responseTimeout: 2000
type: "loadBalancerHealthCheck"
unhealthyThreshold: 3
uri: ""
Lb rule gets created successfully.
But we get "503 Service Unavailable" when trying to access the LB ipaddress.
Following is the haproxy configuration in LB Agent:
backend 6cc3efd3-7f98-4bcd-b625-7884cb3d0691_backend
mode http
balance roundrobin
timeout check 2000
option httpchk
server 4948f728-b657-4601-805f-44246fa59628 <IP_ADDRESS>:80 check inter 2000 rise 2 fall 3
server 88e9dfa5-0264-4c23-9965-e56d1fffd960 <IP_ADDRESS>:80 check inter 2000 rise 2 fall 3

That is a valid httpchk, so it's using the defaults and making OPTIONS / HTTP/1.0 requests, getting no response and marking all the targets as down...  With no targets up you get back 503 responses from HAProxy.
Not setting httpchk if the uri (soon requestLine) is empty/null makes sense.  The UI will either send null or the full "<method> <uri> <version>" form.

Tested with Server - V0.16.0
Created LB with LB config using the following health check policy having empty string as "uri"
healthCheck: {interval: 2000, responseTimeout: 2000, healthyThreshold: 2, unhealthyThreshold: 3, uri: ""}
healthyThreshold: 2
interval: 2000
responseTimeout: 2000
unhealthyThreshold: 3
uri: ""
id: "1lbc4"
kind: "loadBalancerConfig"
lbCookieStickinessPolicy: null
When this LB is programmed in the LB agent , there is no "option httpchk" programmed in haproxy.cfg.
Lb rule works as expected.
frontend 7fd3168e-a063-4145-8c1d-caa180fde2e1_frontend
bind <IP_ADDRESS>:9102
mode http
default_backend 7fd3168e-a063-4145-8c1d-caa180fde2e1_backend
backend 7fd3168e-a063-4145-8c1d-caa180fde2e1_backend
mode http
balance roundrobin
timeout check 2000
    server fd8ba599-c316-4732-aa35-5789f1aebec7 <IP_ADDRESS>:80 check inter 2000 rise 2 fall 3
    server 7b950fab-e022-4dda-98cb-9c89f8fb6549 <IP_ADDRESS>:80 check inter 2000 rise 2 fall 3


Commit search shows a 500er error

Gitea version (or commit ref): Gitea version 1.11.1 built with GNU Make 4.3, go1.13.8 : bindata, sqlite, pam
Git version: 2.26.0
Operating system: ArchLinux
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No
[ ] Not relevant

Log gist: https://gist.github.com/Bodenhaltung/4c2fae24a5879894f8cdba3cc48c3936

Description
The commit search shows a 500er error consistently on any searchterm

What Gitea version are you using?

What Gitea version are you using?

 I thought you've fixed the issue. Reopening. :flushed:

I'd suggest you first upgrade to 1.11.3, since 1.11.1 has some interesting issues that will affect you sooner or later.
As for your problem, I think it's worth attempting to rebuild the code indexes. Those are usually in the data folder (e.g. /var/lib/gitea/indexers, depending on your system configuration). You can delete the repos.bleve directory and Gitea will rebuild it for you. You must shutdown the instance before deleting the directory.

@guillep2k commits search is not code search. Commits search will search commit via git command but not indexes.

This is probably related to:
<EMAIL_ADDRESS>https://bugs.exim.org/show_bug.cgi?id=1749
I'm not familiar with arch linux or anything but this change might be related to fixing it:
https://git.archlinux.org/svntogit/packages.git/commit/trunk?h=packages/pcre2&id=920838a6a4ca3ff466b8751d8dc4c24c5383767f
Do you have that version? Or can try updating (I don't know if git needs to then be rebuild or however arch linux works).

Thanks for your help, i have core/pcre2 10.34-3 and now also: Gitea version 1.11.3 built with GNU Make 4.3, go1.14.1 : sqlite, pam
But still the same error.
I understand that the problem is not gitea related more arch vs. pcre2 vs. git.


homebrew module regression

Commit f4ca9ebfc913453f98ba6912a8c42684fd742cc1 introduced a regression in the homebrew module where it won't load.
Originally the bail-out code was:
if [[ "$OSTYPE" != (darwin|linux)* ]]; then
  return 1
fi

The regression is:
if ! is-darwin || ! is-linux; then
  return 1
fi

Fix:
if ! is-darwin && ! is-linux; then
  return 1
fi

Ah! Thanks for pointing it out, will create a PR or do you want to do the needful?

I'm still seeing:
command not found: is-darwin
command not found: is-linux

When loading the homebrew module.
I believe modules/homebrew/init.zsh is missing a pmodload 'helper', so depending on which modules you're loading and the order you load them in, it's possible that the helper functions are not available when the homebrew module is loaded.


Add UK National Insurance Number Recognizer

Change Description

Add recognizer to detect UK National Insurance Number + tests
Update docs on creating pre-defined recognizers to use config file

Issue reference
NA
Checklist

[x] I have reviewed the contribution guidelines
[x] I have signed the CLA (if required)
[x] My code includes unit tests
[x] All unit tests and lint checks pass locally
[x] My PR contains documentation updates / additions if required

/azp run

/azp run


[enhancement]: better nodes graph validation errors

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
If a node graph validation fails, you get a rather unhelpful 422 (Unprocessable Entity) error. Would be very helpful (probably essential) to get better errors.
A useful shape for this might be:
const errors = {
  nodes: [
    {
      node_id: '123',
      field: 'width',
      error: 'Invalid width',
      data: 0.5,
    },
    {
      node_id: '456',
      field: 'scheduler',
      error: 'Invalid scheduler name',
      data: [1, 2, 3],
    },
  ],
  edges: [
    {
      source_node_id: '123',
      source_field: 'height',
      target_node_id: '456',
      target_field: 'prompt',
      error: 'Expected str input, got int',
    },
  ],
};

Alternatives
No response
Aditional Content
No response

we do actually get this, I just didn't know hwere to look


small typo prints blank page, may be worth adding an error

Took me one hour to find that a small typo in one of my embl templates was causing the page to render nothing, without any error message. Just thought it may be worth signalling it in case it is possible to detect this case and print a compile error when it occurs:
| This line will not be rendered at all when there is the typo below
a. click="destroyModel"
| This line will not be rendered at all when there is the typo above

=> typo is in the a., should be only a

Moved to #276


Create Length of the longest subarray with positive product.java

Added 7 Feb 2023 GFG POTD JAVA code.

Fixes Issue

Closes #

CheckList (Check all the boxes)

[ ] My code follows the code style of this project.
[ ] My code will not cause a merge conflict.
[ ] The title of my pull request is a short description of the requested changes.
[ ] I have named & renamed the file & folders correctly before making this pull request.
[ ] I have formatted the code accordingly.
[ ] My code is different than the existing code in this repository (if any).
[ ] I removed all comments and commented lines from the main code. (class Sotulion{...})

Screenshots

Note to reviewers

@kunalnandre
Fixes Issue
Closes #

CheckList (Check all the boxes)
 My code follows the code style of this project.
 My code will not cause a merge conflict.
 The title of my pull request is a short description of the requested changes.
 I have named & renamed the file & folders correctly before making this pull request.
 I have formatted the code accordingly.
 My code is different than the existing code in this repository (if any).
 I removed all comments and commented lines from the main code. (class Sotulion{...})

When you create a pull request you have to provide details of which issue is linked with this pull request by
Fixes Issue
Closes #
and you need to check mark all the required checklist.


x/tools/gopls: 'references' on embedded field returns refs only to field, not type

The form struct {T} is unique in the Go grammar in that the identifier T is both a reference (to a type) and a declaration (of a field). When gopls responds to a references query on the identifier, it chooses arbitrarily to the interpret it as a field, and does not report references to the type. I think it should report the union of both.

If the embedded type is popular (e.g. "bytes.Buffer"), you could lose the field references in the sea of type references. Maybe we can keep the field references before type references in the results? Or is there some other way for the LSP client to distinguish them?

That's a good point. There's a related situation in renaming where an embedded field couples a field and a type: renaming a reference to the the type must of course rename the field, but renaming a reference to the field probably shouldn't rename the type as well. So, the renaming tool gives an error hinting that, if the bigger renaming was intended, it should be initiated on a reference to the type. But I don't see any way that 'references' can return a hint along with the smaller result. Ordering field refs before type refs is all we can hope for; or leaving things as they are.

any way that 'references' can return a hint along with the smaller result

Could gopls issue a notification or whatever that shows up for the user?

Good question. We should experiment.

Ordering field refs before type refs is all we can hope for; or leaving things as they are.

I'm not sure if all clients preserve the ordering of references, though I think most do.
I do think that this is inconsistent behavior, but probably not a high priority.


Issue#730 solved

Issue that this pull request solves
Closes: #730
Proposed changes
-Added Patch API in Question and Answers to update isApproved field.
Brief description of what is fixed or changed

isApproved field of Question and Answers Schema can be updated to 'true' by sending PATCH request.

Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] My changes does not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface
Other information
Any other information that is important to this pull request

@prafullapal Please write the test for both the apis as well.

Approving Answer through PATCH request

Disapproving Answer through PATCH request

Approving Question with PATCH request

Disapproving Question with PATCH request


continuous-integration/appveyor/pr fails recently

Description

I saw recent prs continuous-integration/appveyor/pr  ci  test all fails, and stuck in
bash.exe -elc "export PATH=/c/tools/mingw64/bin:/c/gopath/src/github.com/containerd/containerd/bin:$PATH ; mingw32-make.exe integration"
+ integration
time="2019-04-18T07:23:57Z" level=info msg="running tests against containerd" revision=dcf0be28642d610d6ff22d8aa2d7b179d74c031c runtime= version=v1.2.0-494-gdcf0be28
*** Test killed: ran too long (10m0s).
exit status 1

test can not start

See https://github.com/containerd/containerd/pull/3222


Hdds 3188 test

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
What is the link to the Apache JIRA
(Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HDDS-XXXX. Fix a typo in YYY.)
Please replace this section with the link to the Apache JIRA)
How was this patch tested?
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

Codecov Report

:exclamation: No coverage uploaded for pull request base (HDDS-2823@0f64aab). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             HDDS-2823      #10   +/-   ##
============================================
  Coverage             ?   73.65%
  Complexity           ?    10440
============================================
  Files                ?     1000
  Lines                ?    51710
  Branches             ?     5005
============================================
  Hits                 ?    38089
  Misses               ?    11215
  Partials             ?     2406

Continue to review full report at Codecov.

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


Update README.md

Hello,
TL;DR This allows the MCP clients to execute CLI commands securely.
Added to: Other Tools and Integrations
Thanks

Added here and also added to https://glama.ai/mcp/servers


[ISSUE #3760]if block ends with a return statement, so drop this else and outdent its block

Fixes #3
if block ends with a return statement, so dropped this else and outdented its block

@qqeasonchen I have resolved the conflict.


support sarif output for scan

Fix #197
Tested successfully in combination with #257
PS: I think there is a bug in the output format SARIF for the verify command, it does not log all the errors

Closing following inactivity, feel free to reopen or ask for help if appropriate, thanks!


Automated cherry pick of #7380

Cherry pick of #7380 on cloud.
/cc  @hmhealey

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.


Each component should have a constructor so that the logger is added by the constructor

All components that use a logger should have a constructor so that can add the component name to the logger -  log.With(logger, "component", "ops")
This
func (c *disputeChecker) Exec(ctx context.Context, logger log.Logger){
	logger.
}

should change into
func NewDisputeChecker(logger log.Logger) *DisputeChecker{
	return &disputeChecker {
		logger:  log.With(logger, "component", "dispute checker")
	}
}

func (c *DisputeChecker) Exec(ctx context.Context){
	c.logger
}
``

actualy we can do this as part of the actual PRs that add the logging


Add commit and rollback nodes

Purpose
$subject
Part of https://github.com/wso2-enterprise/eggplant-product-management/issues/15

Closing this as this is not required at the moment

Part of implementing transaction handling flow nodes.


Add task list

Please describe the changes this PR makes and why it should be merged, as well as any and all proof of successful tests:
This PR adds a sample task list command, allowing users to view and edit the state of tasks related to the skills they've started.
Semantic versioning classification:

[x] This PR changes the framework's interface (methods or parameters added)

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[ ] This PR only includes non-code changes, like changes to README, etc.

Actually I'm mistaken, they're all used, please rename the docs for clarity about the purpose of time if you can

I'd rename "time" to "interval", "no deadline" to "ongoing", and maybe change the time limit to the integer number of days because I think that's what we'll be tracking in the database to make processing easier

All that should be done now, lmk if I missed something.

Looking good, just two things, "change the time limit to the integer number of days because I think that's what we'll be tracking in the database to make processing easier" aka change each of "x1 MONTH(S)" -> 30, "x1 WEEK(S)" -> 7 etc. And add documentation for frequency parameter

Right, that should be it.

wait, has the branches merged or am i trippin'


TPC-H test suite added

This PR refactors the single existing TPC-H query to perform operations using standard SQL rather than directly with DataFrames. This was done to expedite the process of increasing TPC-H coverage, since all of the queries are already available in SQL. It also creates a new folder to hold all queries from https://github.com/apache/spark/tree/master/sql/core/src/test/resources/tpch

Thanks!

Thanks!


ClientCertificateCredential implements IDisposable

closes #12080

It won't be a leak, the finalizer would clean the native resource up. And for customers that want to do it earlier, we can recommend loading cert themselves and disposing the instance.

It won't be a leak, the finalizer would clean the native resource up.

On .NET core finalizers are not guaranteed to execute. And while users can create the X509Certificate2 themselves to control the lifetime when using ClientCertificateCredential they don't have this ability when using it indirectly such as with the EnivornmentCredential or DefaultAzureCredential. So if we made this change we would want to make those types disposable as well.

On .NET core finalizers are not guaranteed to execute

Do you mean the shutdown scenario?

using it indirectly such as with the EnivornmentCredential or DefaultAzureCredential.

Don't forget the ChainedTokenCredential as well :)

I'm going to close this for now unless there is more customer interest. It seems that the finalizer is mostly sufficient and I hadn't considered the potential fan out of types that would also need to handle conditional dispose logic.


Decide on visualization app

Grafana, datadog, nagios, etc

Decided grafana, but needs to elaborate more before closing.

See https://github.com/DevelOpsITU/MiniTwit/issues/99#issuecomment-1068085003
The reasoning for picking Grafana is basically the same as with Prometheus.

As we had chosen Prometheus as our monitoring tool, we wanted to support the information from Prometheus and visualize it in a better way.
When looking into visualization tools we had several options. Some of these tools include but are not limited to Grafana, Datadog, and Nagios. All of the tools looked to be viable options. However, some were better than others.
Grafana was quickly our go-to option due to several reasons.
Firstly some of our team members had prior experience with working with Grafana along with the data collected from Prometheus. This gave us some knowledge of how to set it up and some of its capabilities.
Additionally, it is a free visualization tool, which in comparison to e.g. Datadog is a big plus.
Besides Grafana being compatible with Prometheus, it also works well with the logging tool Loki, which we use for logging in our system.


Add support for all widgets in systelab-components-test

In order to successfully migrate existing Protractor test cases to WDIO, this repository should provide full compatibility with the widgets included on http://github.com/systelab/systelab-components-test repository.

PR #1 should address this issue


Invalid HMAC signature

Trying to use the PHP API client and I'm getting an error when running any command.
[Recombee\RecommApi\Exceptions\ResponseException]
  {"statusCode":401, "error":"Invalid HMAC signature"}

I've set it up exactly how it says in the docs.
$client = new Client(
            config('services.recombee.database'),
            config('services.recombee.token')
        );

$client->send(new AddItemProperty("name", "string"));

I've searched your API docs and it says the APIs take care of the HMAC signature already. Can you help with this issue please?

I'm glad you solved it :+1:

@OndraFiedler My bad, I was using the original token when I started the account, not knowing there was a different token for each new database created.


Iterators (WIP)

Overview
This pull request adds a stream-based approach to the query engine via iterators. This initial example only includes a FloatIterator and an implementation of MinIterator for floats. It shows how an aggregate iterator would be built and shows how iterators can be composed.
Performance
The following is a benchmark to calculate a windowed min() across a variety of intervals. This uses 10 input iterators (which would be shards within InfluxDB).
$ goo test -v -bench=Iterator ./tsdb
PASS
BenchmarkFloatMinIterator_1m-4 	50000000	        34.3 ns/op	       0 B/op	       0 allocs/op
BenchmarkFloatMinIterator_10m-4	50000000	        34.1 ns/op	       0 B/op	       0 allocs/op
BenchmarkFloatMinIterator_1h-4 	50000000	        33.9 ns/op	       0 B/op	       0 allocs/op
BenchmarkFloatMinIterator_1d-4 	50000000	        33.8 ns/op	       0 B/op	       0 allocs/op
ok  	github.com/influxdb/influxdb/tsdb	22.762s

This is showing that it can run a min() across 1 minute, 10 minute, 1 hour, & 1 day intervals with about the same performance per point. These benchmarks ramp up to 50M points and operate at ~30M pts/sec.

This looks pretty nice.

Thanks, Philip. I also wanted to note on this PR that the windowed iterators work because they expect input iterators to send their windows in order (although points within those windows do not need to be in order).

How would it look if the windowing was performed outside of the FloatMinIterator? For my use case the data is already windowed and so the FloatMinIterator would just need to exhaust the iterator looking for the min.
Maybe create WindowedFloatIterator that does all the windowing separate from the MinIterator?

@nathanielc If you're only looking for a single value then you can use the MinIterator with a 0 Interval. I'm not sure how I would be able to separate out windowing from the MinIterator since the iterator needs to know what the next window is in order to compute a min value for it.

Could you do something like this?
// Get iterator for underlying data
fitr := FloatIterator(...)
// Nest iterator in windowed iterator.
witr := WindowedIterator(fitr)
witr.Interval = time.Minute
// pass windowed iterator to MinIterator
minItr := NewFloatMinIterator(witr)

The implementation for WindowedItrator just returns nil once the window has been exhausted. And MinIterator reads all values from its iterator and returns the min.
I am probably missing something....

Right now nil returns signify an EOF on the iterator. It would have to use some kind of marker if it wanted to distinguish between end-of-window and end-of-stream. I'm not sure that would be cleaner than how it is now. Can you show a pseudo-impl?

I was missing the fact that the MinIterator etc are reused for multiple windows, but it makes sense why.
Might be possible to just delimit windows with nil and multiple nils in a row indicate EOF. Just a wild guess/suggestion. I need to think about this a bit more to see how it would fit with what I need.
Overall though I like the concept and it looks great.

I added tsdb.Join() in 8c5c8b1234ab380f98bc5463eeb52be245186d92. This combines iterators so they all emit values at the same timestamps for every unique input timestamp.
This is useful for joining series together like:
SELECT sum(x.value) + sum(y.value) FROM x, y GROUP BY time(10m)

The join allows us to ensure that every x.value point and y.value point  has a matching point on the other side with the same timestamp (even if that other point's value is null).
This is also one example of a join. Another type of join might be one that merges all values from one side with a range on the other side, such as in @corylanou's example of SELECT-clause subqueries:
SELECT value + (SELECT avg(value) FROM y GROUP BY time(10m)) FROM x

Also, I added benchmarks for the join but it's harder to give representative numbers with a synthetic benchmark here. If there's no jitter between timestamps (as is in the case of group by intervals) then there's no allocations. However, if a point doesn't exist for an iterator for a given timestamp then, obviously, an allocation would have to occur to create that point.
So, best case scenario, the join can combine 10 separate iterators with no jitter in about 150ns on my 2012 MBP. If every iterator has a different point then that's 9 allocations per point and it jumps to about 2µs.

I'm closing this one in order to do incremental PRs against the query-engine branch.


Issues with ABR with bandwidth variations

Have you read the FAQ and checked for duplicate open issues?
yes
What version of Shaka Player are you using?
2.5.10
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
Own
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
Chrome Version 115.0.5790.98 (Official Build) (arm64) and MACOS Ventura 13.4.1 (c)
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
No
What are the manifest and license server URIs?

https://storage.googleapis.com/shaka-demo-assets/bbb-dark-truths/dash.mpd
What configuration are you using?  What is the output of player.getConfiguration()?

streaming: {
rebufferingGoal: 1,
bufferBehind: 6,
bufferingGoal: 10,
durationBackoff: 1,
ignoreTextStreamFailures: true,
jumpLargeGaps: true,
smallGapLimit: 0.5,
startAtSegmentBoundary: false,
retryParameters: {
timeout: 0,
maxAttempts: 5,
baseDelay: 1000,
backoffFactor: 3,
fuzzFactor: 0.5
}
}
What did you do?

Play asset having Multiple resolution, select high resolution from the resolution list.
Throttled network using Charles Proxy/ Chrome network throttling or when network bandwidth fluctuates.
Video and audio segments of selected resolution is getting downloaded and buffering is seen.

What did you expect to happen?
Video and audio segments of lower profile should be downloaded and playback should be smooth.
What actually happened?

If high resolution is selected from the resolution list, when bandwidth fluctuates Video and audio segments of lower profile is not getting downloaded. High profile is getting downloaded and a lot of buffering is seen.

https://github.com/shaka-project/shaka-player/assets/10974291/82a45e55-ea8b-478d-a2a1-9484424b1e1e

For Adaptive Bitrate Streaming to Work you must select Auto in quality panel, When you select any other value, it is enforced and ABR is turned off and any change in network speed wont affect the resolution variant. Refer ABR Configuration
player.configure({abr: {enabled: true}});

@WINOFFRG  It is not same for other's player like AVPlayer (iOS), ExoPlayer and so on.
If you manually select a specific quality level (e.g., 720p or 1080p) from the quality panel, ABR is not turned off or disabled. The media player will continue to adapt the video quality within the selected resolution variant based on the available network speed. If the network conditions change, the player will still adjust the bitrate within the chosen resolution to maintain a stable playback experience.

Hey! Kindly note that Demo UI is a player implementation. You can use it the way you want, Also Shaka Player allows you to select any variant without disabling the ABR. You can directly select a variant and you will just get a warning Changing tracks while abr manager is enabled will likely result in the selected track being overriden. Consider disabling abr before calling selectVariantTrack(). Which you can ignore. And based on the network it will auto switch to some other variant just as you asked.

@WINOFFRG In the Shaka Player version 2.5.10 I tried setting below config when I select quality level as 720
shakaPlayer.configure({
abr: {
enabled: true,
restrictions: {
maxHeight: 720
}
}
});
I see only resolution variant 720 is picked in both case stable network and change in network speed.
Is there any issue with Shaka version 2.5.10 ?
From which version for Shaka Player abr works fine when I choose quality level as high (eg 720) and there is change in network speed?

@WINOFFRG In the Shaka Player version 2.5.10 I tried setting below config when I select quality level as 720
shakaPlayer.configure({ abr: { enabled: true, restrictions: { maxHeight: 720 } } });
I see only resolution variant 720 is picked in both case stable network and change in network speed.
Is there any issue with Shaka version 2.5.10 ? From which version for Shaka Player abr works fine when I choose quality level as high (eg 720) and there is change in network speed?
In the latest Shaka version 4.3.8 I see useNetworkInformation: true in the configuration with this abr is working fine when I choose quality level as high (eg 720) and there is change in network speed.
I see related PR #2663

@WINOFFRG any comment

I'm not exxactly able to understand the issue here, Can you share some recording or explain in detail?

I see only resolution variant 720 is picked in both case stable network and change in network speed.

For me ABR seems to work fine, When you restrict height or width means any variant above that wont be selected or shown in getVariantTracks You may disable ABR when calling selectVariantTrack to prevent ABR from auto switching to any other track based on bandwidth if you dont do so depending upon network conditions it will switch automatically.

@WINOFFRG I have enabled abr and set restrictions maxHeight: 720. what i expect is if network bandwidth is good it should pick track with bandwidth 720 , if network bandwidth is low it should pick track with less than 720.
In shaka-player version 2.5.10 it's not working as expected, it's only picking track with bandwidth 720 .
In latest shaka player it's working as expected. I wanted to know is there any limitations in v2.5.10?

Sorry I haven't worked with v2.5.10 have no idea about it. However it is an outdated version I guess no support for that. Other members might be able to help.

See maintained-branches.md for the up-to-date list of maintained branches of Shaka Player.

Thanks!


Customize --dev options

I would like to be able to customize the options added by --dev (I personally dislike 6 and 9 in some code bases).
I think it would also be nice to have -keep-locs and -short-paths there too.

Indeed, I haven't really thought where this customization should go. It's probably something that should be configured per-project via a configuration file

Why not just in an arbitrary jbuilder, and it changes the default for all the folders that are under it ? You can still compose projects and it's fairly intuitive.

True. I guess it could be something like this:
(dev_mode
 ((flags (...))
  (ocamlc_flags (...))
  (ocamlopt_flags (...))))

In any case -short-paths looks fine by default. I've added it. I didn't know about -keep-locs, but I would imagine that this is more a global setting. Any reason you would want to enable it on a per project basis rather than globally? I imagine that you can do this with OCAMLPARAM

I use keep-locs because it improves the error messages for wrong signature inclusions a lot. It's pretty essential when you do weird things with functors. The trade-off is that it bloats cmt* files, so I don't need to use it for release.

Ok, I tried it on jbuilder itself and the size difference of the _build directory was quite small, so I added -keep-locs to the dev mode by default.
That makes me think, in parsexp we have a Positions module to encode a sequence of positions in a compact way. Maybe we could use something like this in the compiler to always keep locations.

@Drup Please re-open if the env stanza doesn't address any of your concerns.


Simulation Mode - does not move to the next round

Hi, running the bot is simulation mode to check in Quote strategy  it says BOT STARTED, waiting for next round and then doesnt go to the next round nor record in the history?

I initially had this problem until I setup a free endpoint at quicknode.com. When you setup, just ignore the paid add-ons.
Then copy your endpoint 'https://...' and paste for BSC_NODE_PROVIDER_URL in .env file.

@spacechicken I think its the same problem of  #7, check your BSC_NODE_PROVIDER_URL.

Yeah signing up for Quick node fixed the issue straight away by replacing the created node with the BSC_NODE_PROVIDER_URL.
I would be greatly interested if you add the martingale strategy to it as well - thanks for your help. Ran each strategy at it is not profitable at the moment but everything works well so its a good base to and better strategies to.
Thanks again!


refactor(protocol-kit): add predict Safe address method as a util function

What it solves
Resolves #355
How this PR fixes it
Added predictSafeAddress method as a util function

Implemented in #415


Newlines showing as literal '\n'

According to the Slack API for multi-line messages (which was taken from Get-Help Send-SlackMessage -Parameter Text) you send a multi-line message by adding \n. However when adding this to text the actual \n is shown in the next.
The code I used was from the README:
> Send-SlackMessage -Uri $Uri `
                   -Channel '#test' `
                   -Parse full `
                   -Text 'Hello @pauby,\njoin me in #general!'

> New-SlackMessageAttachment -Color $([System.Drawing.Color]::red) `
                            -Title 'The System Is Down' `
                            -TitleLink https://www.youtube.com/watch?v=TmpRs7xN06Q `
                            -Text 'Please\nDo The Needful' `
                            -Pretext 'Everything is broken' `
                            -AuthorName 'SCOM Bot' `
                            -AuthorIcon 'http://ramblingcookiemonster.github.io/images/tools/wrench.png' `
                            -Fallback 'Your client is bad' |
     New-SlackMessage -Channel '#test' `
                      -IconEmoji :bomb: |
     Send-SlackMessage -Uri $Uri
``

Results in:

![image](https://user-images.githubusercontent.com/12760779/49687366-b4af7b00-faf9-11e8-996b-b0f0619bb025.png)

I've tried this from two different computers and posting to two different slack groups and different slack channels all with the same results.

Is this an issue, expected behaviour or something else?

Hiyo!
Looks like PowerShell / String fun.
this should work:
Send-SlackMessage -Token $t -Text "this`nshould`nwork" -Channel "@pscookiemonster"

ok      : True
channel : D03PKG3GY
ts      :<PHONE_NUMBER>.000700
message : @{type=message; subtype=bot_message; text=this
          should
          work; ts=1544239940.000700; username=Slack API Tester; bot_id=B3C7A8V1V}
```powershell

This links might come in handy:

* [about_Quoting_Rules](https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_quoting_rules)
* [about_Special_Characters](https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_special_characters)

Cheers!

That does work. Kicking myself here. I tried that but of course but I must have only tried it with single quotes. The frustration was building and I assumed it wasn't the module.
Anyway - thank you. Apologies for the interruption!


Errors running example in terraform 0.12-rc1

Following example, I've tried to use this module with terraform v0.12-rc1.
provider "aws" {
  region = "us-east-2"
  version = "~> 2.0"
}

module "vpc" {
  source = "terraform-aws-modules/vpc/aws"
  version = "1.64.0"

  name = "vpc-mgm"
  cidr = "<IP_ADDRESS>/16"
  enable_dns_hostnames = true
  create_database_subnet_group = false

  azs             = ["us-east-2c"]
  private_subnets = ["<IP_ADDRESS>/24"]
  public_subnets  = ["<IP_ADDRESS>/24"]

  enable_nat_gateway = true
  single_nat_gateway = true
  one_nat_gateway_per_az = false

  vpc_tags = {
    Name = "vpc-mgm"
  }
}

And got the following errors running terraform plan:
Error: Missing resource instance key

  on .terraform/modules/vpc/terraform-aws-modules-terraform-aws-vpc-c0c3b52/main.tf line 31, in resource "aws_vpc_ipv4_cidr_block_association" "this":
  31:   vpc_id = "${aws_vpc.this.id}"

Because aws_vpc.this has "count" set, its attributes must be accessed on
specific instances.

For example, to correlate with indices of a referring resource, use:
    aws_vpc.this[count.index]

Error: Unsupported argument

  on .terraform/modules/vpc/terraform-aws-modules-terraform-aws-vpc-c0c3b52/main.tf line 295, in resource "aws_default_network_acl" "this":
 295:   ingress = "${var.default_network_acl_ingress}"

An argument named "ingress" is not expected here. Did you mean to define a
block of type "ingress"?

Error: Unsupported argument

  on .terraform/modules/vpc/terraform-aws-modules-terraform-aws-vpc-c0c3b52/main.tf line 296, in resource "aws_default_network_acl" "this":
 296:   egress  = "${var.default_network_acl_egress}"

An argument named "egress" is not expected here. Did you mean to define a
block of type "egress"?

I have the same issue. any way to resolve it?

On TF < 0.12, if your count attr was exactly 1, then accessing such resource as aws_vpc.this would be ok. To keep consistency, 0.12 enforces the use of index referring when count is used even if the value is 1. That need to be addressed here #228

v2.0.0 has been released.
Please give it a try and report if you find any issues.

Upgrading to ~> 2.0 fixed this issue for me.

Can anyone elaborate please how exactly it is resolved?


Replaced hardcoded log path with given parameter path

Specifying a path with registering the file logger only created the folder, but never used it in creating the write stream.
Fixed it so that the parameter gets used in creating the write stream.

 See here: https://docs.github.com/articles/about-gpg/

Signed the commit


system_bd.tcl Vivado version upgrade gotcha

In fpga/parallella/headless_e16_z70[12]0/system_bd.tcl:
When upgrading Vivado, you must change the line with:
set scripts_vivado_version 2015.4

to match the Vivado version. Even though this file is re-generated in the
build, it is required by the build to create an initial block design.
Ugh!!!

I am using Vivado version 2017.1
I have followed the instructions in https://github.com/parallella/oh/issues/103#issuecomment-298262178 but I still have the following error.

[ERROR]  : Can't read BIT file - ./system.runs/impl_1/system_wrapper.bit
cp: cannot stat 'system_wrapper.bit.bin': No such file or directory

I have attached the error log file below.
error.txt

When I open the system.xpr in GUI, I have this warning in synthesis:

[Project 1-486] Could not resolve non-primitive black box cell 'system' instantiated as 'system_i' ["/home/promach/Documents/Grive/Personal/Digital/Parallella/oh/src/accelerator/fpga/system.srcs/sources_1/bd/system/hdl/system_wrapper.v":16]


1.1.0 update

Hi,
I tried to update the mod so it no longer crash in 1.1.0.

Could you maybe fork it instead?

I'll look at it when I can -- feel free to fork it in the meantime. I glanced through the code changes and they look fine, so I'll approve after I can test it out (plus I'll credit you as a contributor on Nexus and Steam).
Sorry for the long turnaround, work's crazy.


Timeout for Github Actions

We should probably add a timeout for Github Actions. Just to make sure we're don't waste resources on a long running job.

I just saw it, too. I included timeouts in the main branch actions.