FetchXML flaw for ordering and then doing distinct, not the other way around.

When calling the API to do distinct you use ?fetchXml=<fetch distinct="true"> but before that, I'd like to order by createdon desc and then do the distinct to remove duplicates. Is this possible in some way?

I'm assuming the problem is how they do the distinct algorithm. They probably do (if next is same -> remove) rather than (if list contains dups -> remove). So, if you sort first, the distinct algorithm fails?

@wrharper -
Thanks for your feedback.
This forum is for feedback about the documentation, and it works best when the feedback is associated with a specific article. The way that these get linked is by opening the issue using the feedback button at the bottom of the article.
Which article is your feedback related to? I expect it must be this one Use FetchXML to construct a query
I recommend using FetchXml builder to test your FetchXml query and observe the result. When you send the request via Web API, you should get the same results.

I wish Microsoft owned that; 3rd party stuff isn't something most people are allowed to use.
What I am trying to do doesn't appear to be possible at all. What do you do in that case?

@wrharper -
I'm working with the dev team on some improvements to the fetchXml documentation. But these are going to be pretty general in nature. I don't think they will address your specific question.
I can't offer technical support here, so I can't help troubleshoot this specific issue.
The purpose of these issues is to identify things we can do to improve the docs.
What do you think we can add to the docs that will help address this challenge and save you time?
Also, some context about your use case might be helpful. I know you are using Web API and you want to do aggregation & grouping. Is this to generate a report?
If so, have you looked at Power BI?
BTW - An alternative to using FetchXml builder is to use the legacy Advanced Find and download the fetchxml it generates.

At this point, it looks like it's just not possible with fetchxml. What I would really like is to know if there is an active github about fetchxml features and what it "cannot" do and see if such a thing can be requested.

Thanks Wayne.
Positively documenting capabilities is possible. We have much room for improvement here just to do this.
Trying to maintain documentation about all the things that can't be done is not realistic since everyone has different expectations about what they should be able to do. We are going to focus on documenting known capabilities better.
I'll share this feedback with the dev team and update this if I get a response.

ok, my current query is quite complex already. it looks like to go further I will just have to do more than 1 query. I was hoping to get everything I needed in 1 go instead of sending multiple calls to get the info I need while being distinct and sorted in a certain way. I'm pretty positive this isn't possible now because of how "distinct=true" works.
I'm just trying to optimize overall speed of output to users.

@wrharper-AASP
I recommend that you contact tech support for help with this question. A dev reviewed this thread and it isn't clear what specific change to the documentation we could make that would help you.
Closing this.


feat: update API responses to better match old JSON file

Description

Add "ensure_ascii": false param to account for special characters
Add more formatting to directly match the old JSON format.
Fix formatting error with subgroups object
Change how names are displayed.
Only add approvers array if there are approvers.

Issue Ticket Number and Link

N/A
Checklist

Remove any options that are not applicable

[X ] I have reviewed the how-to guide and the contributing file.

[X] I have run required code checks and resolved any blockers.

[X] I have created the necessary tests to show my changes are effective and work as intended.

[X] I have thoroughly commented my code, especially in more complex areas.

[X] I have updated any necessary documentation.
Screenshots / Recordings
N/A

Looks good! As a test, did you try comparing the output of affiliations_list_json_format to the old JSON file?

I did compare the two files- they are not identical but nearly, the differences are below:

guidelines_name and guidelines_url values were not migrated, so those are expected to be missing.
some of the affiliations are soft-deleted so they do not appear in this response (this can be changed if soft-deleted values should be returned).
affiliation_fullname values have GCEP, VCEP, or both appended to the end depending on the subgroups in the affiliation.
approvers have not been added into the affiliation microservice yet so those are expected to be missing. (those values are only in the JSON files and were not a part of the CSV file, so I will be migrating those values for V2).
the API response is not ordered, the JSON file was in ascending order and the order never changed. From my understanding JSON objects are not ordered, so this is also to be expected.

Are we adding approvers before we hook up the affiliations service to the GCI/VCI? It seems like we would need to. Or are we planning on having both the affiliations service and the JSON files used at the same time?

Yes! I just completed a script to add approvers from the file to the microservice. This should be run before launching V2 so that all relevant information is in the microservice and the JSON files can be deprecated. See this PR . I also added a script to compare the JSON file with a file of the API response to catch any differences, which is also in that PR.


Chores: Updated packages and removed unnecessary imports

Looks good in my test pass.  Thanks for the updates!

@all-contributors add @MdeBruin93 for code


[IMPROVMENT] AMS does not support to create table under the Sentry managed dir

Precondition:
Installed a hive with sentry permission controlled.
Also, the user didn't have root permission for hive warehouse dir.
Phenomenon:
When user try to create a table with new database, AMS will try to create dir on HDFS under the hive warehouse dir. Due to the given role had no permission for hive warehouse dir, it will throw AccessControlException: Permission denied.
Genrally, there are 2 thoughts to solve the issue:

Give the user root permission for hive warehouse dir. However, this cannot been achieved due to the Hive warehouse itself been managed by Sentry, where you unable to specify role manaually using setfacl command.
Another way is manully create and give permission by using Hive and Sentry. However, the Hive created database include ".db" suffix but Iceberg does not, thus it will still not found and try to create database then throw exception again.

Possible solution:
Let the iceberg create a database with ".db" suffix.
Dicussion:
Should we support this feature which allow user to create table under the Sentry managed location? Or leave this to the enterprise-class feature?

Hi, @hhippodnsla
Thanks for trialing arctic and providing feedback.
As I see,  you want create arctic table under hive warehouse dir, so you want hms manage arctic or iceberg table?
If yes, a beter path would be:

Using arctic adapting hive table：a new arctic table type adapting hive table，detail information would be found in #38  and will be released in arctic v0.3.1
Waiting arctic support iceberg hive catalog: this is a high priority improvement for arctic and will be released soon

Okay


Avoid saving sparse ChunkedArrays in pyarrow tables

Fixes https://github.com/huggingface/datasets/issues/5541

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


String: Add Reverse Words in a String in Java

https://leetcode.com/problems/reverse-words-in-a-string/

@akgmage assign this issue it to me please.

Please assign this to me i  will do it


%LOAD magic + doc

The %LOAD magic command doesn't behave like documented.
For example, this works fine (with testdb.db existing in the same folder as the notebook):
%LOAD testdb.db

Giving it an absolute/relative path works as well
%LOAD /absolute/path/to/testdb.db

On the other hand, this doesn't work:
%LOAD "testdb.db"

Result: "Error: The path doesn't exist." Same issue with single quotes, with absolute or relative paths.

huh, thank you for capturing this!
Feel free to open a PR to change it :)

This is fixed by https://github.com/jupyter-xeus/xeus-sqlite/pull/116


Errors on build main branch

Wanted to import and run
    let domain = Domain::new("google.com").unwrap();
    let permutations = domain.all().unwrap();

Getting errors at bottom.
❯ cargo --version
cargo 1.72.0 (103a7ff2e 2023-08-15)

great-squat on  main [!] is 📦 v0.1.0 via 🦀 v1.72.0 on ☁  (us-east-1)
❯ rustc --version
rustc 1.72.0 (5680fa18f 2023-08-23)

errors:
❯ cargo build
   Compiling twistrs v0.5.2 (https://github.com/haveibeensquatted/twistrs.git?branch=main#5ebc1d17)
error[E0277]: the trait bound `String: Serialize` is not satisfied
    --> /home/tscanlan/.cargo/git/checkouts/twistrs-ab64395ec7e7a9bf/5ebc1d1/twistrs/src/enrich.rs:90:24
     |
90   | #[derive(Debug, Clone, Serialize, Default)]
     |                        ^^^^^^^^^ the trait `Serialize` is not implemented for `String`
91   | pub struct DomainMetadata {
92   |     /// The domain that is being enriched.
     |     -------------------------------------- required by a bound introduced by this call
     |
     = help: the trait `Serialize` is implemented for `str`
note: required by a bound in `enrich::_::_serde::ser::SerializeStruct::serialize_field`
    --> /home/tscanlan/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.188/src/ser/mod.rs:1865:12
     |
1859 |     fn serialize_field<T: ?Sized>(
     |        --------------- required by a bound in this associated function
...
1865 |         T: Serialize;
     |            ^^^^^^^^^ required by this bound in `SerializeStruct::serialize_field`

error[E0277]: the trait bound `Vec<std::net::IpAddr>: Serialize` is not satisfied
    --> /home/tscanlan/.cargo/git/checkouts/twistrs-ab64395ec7e7a9bf/5ebc1d1/twistrs/src/enrich.rs:90:24
     |
90   | #[derive(Debug, Clone, Serialize, Default)]
     |                        ^^^^^^^^^ the trait `Serialize` is not implemented for `Vec<std::net::IpAddr>`
...
95   |     /// Any IPv4 and IPv6 ips that were discovered during
     |     ----------------------------------------------------- required by a bound introduced by this call
     |
     = help: the following other types implement trait `Serialize`:
               bool
               char
               isize
               i8
               i16
               i32
               i64
               i128
             and 93 others
     = note: required for `std::option::Option<Vec<std::net::IpAddr>>` to implement `Serialize`
note: required by a bound in `enrich::_::_serde::ser::SerializeStruct::serialize_field`
    --> /home/tscanlan/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.188/src/ser/mod.rs:1865:12
     |
1859 |     fn serialize_field<T: ?Sized>(
     |        --------------- required by a bound in this associated function
...
1865 |         T: Serialize;
     |            ^^^^^^^^^ required by this bound in `SerializeStruct::serialize_field`

error[E0277]: the trait bound `Vec<(std::net::IpAddr, String)>: Serialize` is not satisfied
    --> /home/tscanlan/.cargo/git/checkouts/twistrs-ab64395ec7e7a9bf/5ebc1d1/twistrs/src/enrich.rs:90:24
     |
90   | #[derive(Debug, Clone, Serialize, Default)]
     |                        ^^^^^^^^^ the trait `Serialize` is not implemented for `Vec<(std::net::IpAddr, String)>`
...
106  |     /// IP addresses resolved through GeoIP lookup to City, Country, Continent.
     |     --------------------------------------------------------------------------- required by a bound introduced by this call
     |
     = help: the following other types implement trait `Serialize`:
               bool
               char
               isize
               i8
               i16
               i32
               i64
               i128
             and 93 others
     = note: required for `std::option::Option<Vec<(std::net::IpAddr, String)>>` to implement `Serialize`
note: required by a bound in `enrich::_::_serde::ser::SerializeStruct::serialize_field`
    --> /home/tscanlan/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.188/src/ser/mod.rs:1865:12
     |
1859 |     fn serialize_field<T: ?Sized>(
     |        --------------- required by a bound in this associated function
...
1865 |         T: Serialize;
     |            ^^^^^^^^^ required by this bound in `SerializeStruct::serialize_field`

error[E0277]: the trait bound `String: Serialize` is not satisfied
    --> /home/tscanlan/.cargo/git/checkouts/twistrs-ab64395ec7e7a9bf/5ebc1d1/twistrs/src/enrich.rs:115:24
     |
115  | #[derive(Debug, Clone, Serialize)]
     |                        ^^^^^^^^^ the trait `Serialize` is not implemented for `String`
...
120  |     /// Message received back from the SMTP server
     |     ---------------------------------------------- required by a bound introduced by this call
     |
     = help: the trait `Serialize` is implemented for `str`
note: required by a bound in `enrich::_::_serde::ser::SerializeStruct::serialize_field`
    --> /home/tscanlan/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.188/src/ser/mod.rs:1865:12
     |
1859 |     fn serialize_field<T: ?Sized>(
     |        --------------- required by a bound in this associated function
...
1865 |         T: Serialize;
     |            ^^^^^^^^^ required by this bound in `SerializeStruct::serialize_field`

error[E0277]: the trait bound `String: Serialize` is not satisfied
    --> /home/tscanlan/.cargo/git/checkouts/twistrs-ab64395ec7e7a9bf/5ebc1d1/twistrs/src/permutate.rs:36:26
     |
36   | #[derive(Default, Debug, Serialize)]
     |                          ^^^^^^^^^ the trait `Serialize` is not implemented for `String`
...
41   |     /// The top-level domain of the FQDN (e.g. `.com`).
     |     --------------------------------------------------- required by a bound introduced by this call
     |
     = help: the trait `Serialize` is implemented for `str`
note: required by a bound in `enrich::_::_serde::ser::SerializeStruct::serialize_field`
    --> /home/tscanlan/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.188/src/ser/mod.rs:1865:12
     |
1859 |     fn serialize_field<T: ?Sized>(
     |        --------------- required by a bound in this associated function
...
1865 |         T: Serialize;
     |            ^^^^^^^^^ required by this bound in `SerializeStruct::serialize_field`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `twistrs` (lib) due to 5 previous errors

@tompscanlan just ran this on main without issues on the same version. What environment are you running this on by any chance?

It's windows 11 wsl.
sent from a phone, Tom,<PHONE_NUMBER>
On Thu, Aug 31, 2023, 5:44 AM Juxhin @.***> wrote:

@tompscanlan https://github.com/tompscanlan just ran this on main
without issues on the same version. What environment are you running this
on by any chance?
—
Reply to this email directly, view it on GitHub
https://github.com/haveibeensquatted/twistrs/issues/57#issuecomment-1700713468,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2AEUNEU23X3JCRZL7YSYLXYBMIPANCNFSM6AAAAAA4E4RYR4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm also running this on WSL2/Windows 11. If you're using it as a dependency, I'd recommend pulling the latest version from crates.io.

@tompscanlan not sure if you got round to solving this. I'm going to close the issue -- feel free to re-open it should it issue persist.


Configure checkstyle plugin to check for copyright notice in files

The Maven License Plugin might be useful for it, or the the Maven Checkstyle Plugin.

Checkstyle seems to be enabled now. @meltsufin please reopen if I'm wrong.


Formatting adds a space at end of line

Expected behavior
For steps with table param, running gauge --format specs should not add an extra space at the end of step text.
Actual behavior
For steps with table param, running gauge --format specs adds an extra space at the end of step text.

Not reproducible.


Filter dapps by category

See writeup here for requirements: https://www.notion.so/valora-inc/651e35300967405a969d31808247ad07?v=dd3840385dc04284a6e1878b7ae66ccb&p=d9cd9761e9bf4bcaa005f44d9a675cce


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-observability-instances
module
patch
v2.9.1 -> v2.9.2

github.com/terraform-ibm-modules/terraform-ibm-resource-group
module
patch
v1.0.5 -> v1.0.6

Release Notes

terraform-ibm-modules/terraform-ibm-observability-instances (github.com/terraform-ibm-modules/terraform-ibm-observability-instances)
v2.9.2
Compare Source
Bug Fixes

added missing moved block for atracker log analysis targets (#360) (45c30ce)

terraform-ibm-modules/terraform-ibm-resource-group (github.com/terraform-ibm-modules/terraform-ibm-resource-group)
v1.0.6
Compare Source
Bug Fixes

update usage to show hashicorp syntax instead of github (#435) (9469adf)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Bump oauth2 version

Cleaner version of #5
De-duplicates dependency info by using gemspec in Gemfile

Thank you for the PR @rahearn, we'll take a look.  Looks like ruby_dep needs Ruby version >= 2.2.3, which is causing the Travis checks to fail.

@rahearn while I was updating the client, I went ahead and added the requirement of oauth at least 1.0.  And I also switched to https://rubygems.org/, thanks for that suggestion!  The travis checks on master are still failing for jruby, I hope to fix that (the client works fine in jruby).
I like the idea of using +gemspec, looks like it cleans things up nicely.  Thanks again for the your efforts!

@JaredCorduan glad to see that the gem has been updated!


Leave room with runtime error: Invalid memory address or nil pointer dereference

Here is the problem I've encountered. Checked out the leaveRoom function. No Idea why this happened.

Try to remove your $GOPATH/github.com/kataras/go-websocket and upgrade it with go get -u github.com/kataras/go-websocket.

I think I have the latest version and got the same panic


Question about litecoin branch 0.10.x

Apparently (if I'm informed correctly) the rpc interface for 0.10.x has changed, does mpos know how to handle this? And if so, starting from what version?
Thanks!
Chris

I have not read about any breaking changes. Would have to check. But I know Bitcoin is working fine on the 0.10.x version so I would assume Litecoin is similar.

The only thing changed is how to talk to litecoind:
litecoind no longer sends RPC commands. You must use the separate litecoin-cli command line utility.

And since I hit send too quickly, this does not affect how MPOS talks with the RPC. It still uses RPC-XML via TCP socket.

Ok, cool! That's the thing I noticed, that the rpc stuff was gone from the main executable. Now it all comes back to me :)
Chris

10 seconds Google foo could have replaced a ticket ;-) But good to know! Had no idea they did that even when recently testing 0.10.x since I never run command line RPC commands. It's mostly MPOS stuff I have to test ;-)

Stratum-mining not support the latest versions of BTC and LTC ?
bitcoin-cli getwork
error: {"code":-32601,"message":"Method not found"}

MPOS doesn't care about  the getwork implementation and nor does Stratum. That's more of a mining client issue. You won't be able to use a getwork only client anymore unless adding a proxy.
I have a Litecoin testnet running on <IP_ADDRESS> which works fine.

I am using the node.js stratum by the way.


Add stress buster tests in skip list from extended execution for metrics

Error from jenkins run: Reading XML source '/home/fspcibld/output.xml'; failed: ParseError: mismatched tag: line 5152, column 2 14:38:07 14:38:21 Caused by: hudson.remoting.ProxyException: com.ctc.wstx.exc.WstxParsingException: Unexpected close tag </arguments>; expected </assign>. 14:38:21 at [row,col {unknown-source}]: [5152,11] From the robot community: "This could happend, for example, if your libraries use threads, but there are also many other ways to corrupt output files."

Thanks @lgon for the input.  Hope this fixed my run

https://gerrit.openbmc-project.xyz/#/c/7745/


YJDH-157 | OIDC backchannel logout

Description :sparkles:
Add backchannel logout to the shared oidc component. Implementation loosely based on tunnistamo and: City-of-Helsinki/tunnistamo#154
Issues :bug:
YJDH-157
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #139 (32d0672) into develop (0fa4fd9) will increase coverage by 22.89%.
The diff coverage is 96.82%.

@@             Coverage Diff              @@
##           develop     #139       +/-   ##
============================================
+ Coverage    72.20%   95.10%   +22.89%
============================================
  Files          255       63      -192
  Lines         5217     1858     -3359
  Branches       347        0      -347
============================================
- Hits          3767     1767     -2000
+ Misses        1386       91     -1295
+ Partials        64        0       -64

Impacted Files
Coverage Δ

backend/shared/shared/audit_log/audit_logging.py
100.00% <ø> (ø)

backend/shared/shared/audit_log/tests/conftest.py
100.00% <ø> (ø)

...hared/shared/audit_log/tests/test_audit_logging.py
100.00% <ø> (ø)

backend/shared/shared/audit_log/tests/utils.py
60.00% <ø> (ø)

backend/shared/shared/audit_log/viewsets.py
90.38% <ø> (ø)

.../shared/shared/azure_adfs/tests/test_adfs_login.py
100.00% <ø> (ø)

backend/shared/shared/azure_adfs/urls.py
100.00% <ø> (ø)

backend/shared/shared/common/tests/conftest.py
100.00% <ø> (ø)

backend/shared/shared/oidc/admin.py
100.00% <ø> (ø)

backend/shared/shared/oidc/auth.py
88.15% <ø> (ø)

... and 211 more

Continue to review full report at Codecov.

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


Support groups as scopes

Right now moauthd only supports three hardcoded scopes: "public", "shared", and "private". Would be nice to extend this to all UNIX groups so that a user login maps that user to all of the user's groups as scopes.

[master 49a3340] Add support for Unix groups as scopes (Issue #12)


Address pytoml build warnings

_WouldIgnoreField: `dependencies` defined outside of `pyproject.toml` would be ignored.
!!

        ********************************************************************************
        ##########################################################################
        # configuration would be ignored/result in error due to `pyproject.toml` #
        ##########################################################################

        The following seems to be defined outside of `pyproject.toml`:

        `dependencies = ['pydantic>=2']`

        According to the spec (see the link below), however, setuptools CANNOT
        consider this value unless `dependencies` is listed as `dynamic`.

        https://packaging.python.org/en/latest/specifications/declaring-project-metadata/

        For the time being, `setuptools` will still consider the given value (as a
        **transitional** measure), but please note that future releases of setuptools will
        follow strictly the standard.

        To prevent this warning, you can list `dependencies` under `dynamic` or alternatively
        remove the `[project]` table from your file and rely entirely on other means of
        configuration.

        By 2023-Oct-30, you need to update your project and remove deprecated calls
        or your builds will no longer be supported.
        ********************************************************************************

!!

Addessed in #10


Add code input sections for Mission Start and Mission End to configuration

Basically porting onMissionStart.sqf and creating similar functionality for mission end.

[ ] (Frontend) Add "onMissionStart" and "onMissionEnd" code input sections to 3den configuration.
[ ] (Backend) add functionality for "onMissionStart" to "onMissionEnd" to be called.

"onMissionEnd" is necessary as any gm can just use an event handler for it, so discussion can be had as to whether it should be implemented.

Going to go ahead and say endMission code isn't needed, as any GM can just use the mission ended event handler.


[Question] How to get embedded array object of document result?

Hi,
Does this library support search operation like db.getCollection('media_collection').find({tags: {$elemMatch: {value: "javascript"}} })?
For example i have embedded array object of documents like:
"tags" : [
  {
    "key" : "code",
     "value" : "code"
  },
  {
    "key" : "javascript",
    "value" : "javascript"
  },
  {
    "key" : "lo",
    "value" : "lo"
  }
],

What operation should i use to get the result? maybe something like $elementMatch

@mydevlocal yes. You can do someting like this
CollectionModel::where('tags.value', 'like', '% javascript%')->get();
Or you can use whereRaw method

Oh great, it works. Thank you.


PR : Removed horizontal overflow : issue no. #184

Pull Request Checklist
Solved issue #184

[x] I have added screenshots and videos to show the working of my code.
[x] I have ensured that the screen size is set to 100% while making the video.
[x] I have synced the latest fork with my local repository and resolved any conflicts.

Kindly share screenshot before how it is looking and after what you have done

There are check errors in fork pr.


adding index.html

Feel free to add whatever html you see fit

Sorry sent you two pull request one from my master branch and you from the feature branch disregard the first one from the master branch


FK connector lines will not draw past CSS 'area' property boundries

From<EMAIL_ADDRESS>on July 01, 2009 21:32:41
What steps will reproduce the problem? 1. Load a database structure that is larger than the 'area' property
boundary in the 'styles/style.css' config file
2. Create a new foreign key constraint, or view existing FK constraint
connector lines
3. Scroll to the bottom of the defined CSS 'area' boundary and either view
that the existing connector lines going to tables located below the
boundary area are 'cut off', or drag a table with an existing FK connector
line below the boundary and note that the lines are cut off. What is the expected output? What do you see instead? Expected that the FK connector lines would continue to be drawn past the
'boundary', as the tables are correctly rendered past this point.
I see that the lines are 'cut off', although when dragging a table around
under the 'boundary', the lines continue to update and be drawn above the
'boundary'. What version of the product are you using? On what operating system? Latest stable download as of 06/30/2009. Fedora 10 Linux
<IP_ADDRESS>-170.2.56.fc10.x86_64 #1 SMP Mon Mar 23 23:08:10 EDT 2009 x86_64
x86_64 x86_64 GNU/Linux Please provide any additional information below. Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:<IP_ADDRESS>) Gecko/2009061118
Fedora/3.0.11-1.fc10 Firefox/3.0.11
Original issue: http://code.google.com/p/wwwsqldesigner/issues/detail?id=24

From ondrej.zara on July 01, 2009 13:38:51
This is intended behavior. Vector canvas must have fixed dimensions and these are set
to correspond with the "drawing canvas" area.
Feel free to enlarge this element by adjusting the relevant CSS properties. This will
adjust the vector space as well.
Status: WontFix


Readme client example throws an AssertionError

Long story short
Readme client example throws an AssertionError:
assert isinstance(url, URL), url
AssertionError: http://python.org

Client example runs.
Actual behaviour

assert isinstance(url, URL), url
AssertionError: http://python.org

Steps to reproduce
Run the code under Python 3.6

Your environment
Up-to-date MacBook with Python 3.6

fixed with yarl 0.9.6


WIP: cudaMapper sample application

Add a cudaMappper sample application that shows a simple use of the APIs
closes #340

@mimaric a couple of things:

I couldn't figure out how to create an overlapper object with the public API. For instance the matcher has create_matcher() . Does it make sense to have something similar for the overlapper?
the use of index_descriptor for grouping reads also isn't part of the public API. Should it be? It seems useful for the full cudamapper pipeline without having to reimplement a  the grouping/batching logic.

Still a few things that need to be resolved for this. What should the "default" fasta file be? I made some assumptions (in terms of read size as well as the all-to-all assumption) are those alright?

I couldn't figure out how to create an overlapper object with the public API. For instance the matcher has create_matcher() . Does it make sense to have something similar for the overlapper?

Yes, this is a bug/missing feature. There should be a make_overlapper() function. I created #509

the use of index_descriptor for grouping reads also isn't part of the public API. Should it be? It seems useful for the full cudamapper pipeline without having to reimplement a  the grouping/batching logic.

This is an interesting question. Do you think it could be used exactly in the current form? When I was developing it I had our IndexCache strategy in mind, and as that should definitely be a private cudamapper thing I didn't consider putting anything else in the public interface.
Would you like to create an issue for it?

Still a few things that need to be resolved for this. What should the "default" fasta file be? I made some assumptions (in terms of read size as well as the all-to-all assumption) are those alright?

I'd ask @edawson and @vellamike about this

Regarding IndexDesciptor, I think it would make sense to expose it in public interface. Would you like to do it now or merge as it is and then follow up with a separate PR?

Regarding IndexDesciptor, I think it would make sense to expose it in public interface. Would you like to do it now or merge as it is and then follow up with a separate PR?

I think merging it soon and then following up with another PR would be good.

Rebased + squashed commits

Regarding IndexDesciptor, I think it would make sense to expose it in public interface. Would you like to do it now or merge as it is and then follow up with a separate PR?

I think merging it as is and then following up with another PR would be good.

I agree. Created issue #517

@nvvishanthi FYI after addressing a comment we just write "fixed", "done" or similar and let the comment originator mark it as resolved

@nvvishanthi FYI after addressing a comment we just write "fixed", "done" or similar and let the comment originator mark it as resolved

Ah ok thanks for letting me know. Sorry about the self-resolve


feat(s3): add regionOverride property

some s3 compatible storage solutions (minio, for example) take the region into account when validating incoming requests but allow you to set non-aws region identifiers. this change allows you to explicitly set the region identifier as an override via regionOverride. it also allows you to set the region for configurations using the endpoint setting.

@robzienert PTAL
@imosquera FYI

@robzienert can do!

@robzienert documented. i also removed setting the region when it was set to preserve existing functionality. in the end, it wasn't what i was trying to actually fix anyway.


Gurkan collection game

Gurkan collection game
The gurkbot will randomly post a gurkan reaction on one of the latest messages in the server. The first user to click it, consumes it (which makes the bot remove both reactions). At any point, you can use the !gurkancount command to view a leaderboard of who has collected the most candies.
Occasionally the bot will also post a skull reaction, and if the user who clicks this has any gurkans collected, it will steal up to 3 of them and post a spooky message in the channel saying that the player has been cursed and has lost x gurkans. if the user does not have any gurkan, the skull disappears and no message is posted.
Required Features

[ ] The bot should have a small chance of posting a gurkan reaction on all new messages. Under 5%
[ ]  The bot should have an even smaller chance of posting a skull reaction if a new message is posted. maybe 2%.
[ ]  Whenever one of the past 10 messages gets a new reaction, there should be a slightly larger chance the bot posts a reaction as well. perhaps 10% and 5% respectively.
[ ]  When the reactions are clicked, trigger corresponding events.

[ ]  If the gurkan is clicked, add one gurkan to the users count, and remove both reactions.
[ ]  If the skull is clicked, remove up to 3 gurkans from the users count, remove both reactions, and post a message explaining that the user has been cursed and has lost however many gurkans you just removed.

[ ]  The gurkancount leader board will be connected to the gurkan games leaderboard

I would like to work :)

This should be able to use a lot of the code from the candy game on Sir Lancebot (SeasonalBot).
Instead of reacting with a skull, it would be fun if it reacted with another fruit or some more related thing. A skull is a bit unrelated imo.
I also think we should limit this to a certain channel. We don't want the bot to spam cucumber reactions everywhere. Even with a 1% chance, it gets really annoying over time.

This should be able to use a lot of the code from the candy game on Sir Lancebot (SeasonalBot).
Instead of reacting with a skull, it would be fun if it reacted with another fruit or some more related thing. A skull is a bit unrelated imo.
I also think we should limit this to a certain channel. We don't want the bot to spam cucumber reactions everywhere. Even with a 1% chance, it gets really annoying over time.

How about a knife 🔪?

That’s terrifying.

Lol, what do you think we should use then?

I'd probably like to go with a watermelon or mango. That could be fun.


Use a select for the language selection

Showing all languages here are not good-looking. Change it to a select.

Working on this issue.

Fixed by: https://github.com/casbin/casdoor/pull/175


How to download Kudu VM for 1.2.0 version?

I tried to download kudu VM using script:
curl -s https://raw.githubusercontent.com/cloudera/kudu-examples/master/demo-vm-setup/bootstrap.sh | bash

VM set up is done successfully. But kudu -version is 1.0.0
We are not able to use IN_LIST predicate with this version.
Is there any other location to download VM with latest Kudu version?

The VM has since been updated to use CDH 5.11 (i.e. Kudu 1.3.0).


undefined vnode.state

Description:
vnode.state is not copied from vnode.tag when Mithril 1.0.0 "rewrite" is running in node.
Steps to Reproduce:
Using this gist: https://gist.github.com/magnusleo/ac28737dd659db6465d9db47ab54495b
npm test

My test was run with node v5.9.1 on Mac OS X 10.11.5, installed using http://brew.sh/
Expected:
The test runs flawlessly.
Actual:
component.js:15
                return m('li', {onclick: vnode.state.clickCurry(i)}, item); // Fails in node
                                                    ^

TypeError: Cannot read property 'clickCurry' of undefined

I just looked into it.
Simple solution was to also copy all of tag to state. Just did that.
But I think I should try again to use mithrils rendering directly (like @isiahmeadows recommended) in order keep up with changes there.

Thank you very much.
Without deeper knowledge, I agree that using as much of Mithril's rendering as possible is a good idea for the same reason you mentioned.


Error when selecting a specific facility

As user, I'm refining a search by selecting "European Syncrothron Radiation Facility" in the facility dropdown, I see "Something went wrong" message in the results panel.
See screenshot below.
I would expect to see "No results available" message if no results are available or the facility results.
The browser console logs the following error:
Error: 404
    e SWRProvider.js:10
    Babel 9
    Va SWRProvider.js:4
    W index.mjs:757
    l index.mjs:50
    l index.mjs:31
    Le index.mjs:24
    Le index.mjs:20
    W index.mjs:700
    De index.mjs:1023
    Tt Immutable
    jt index.mjs:612
    Tt Immutable
    vr useApi.js:23
    ka DocumentList.js:13
    React 13

I don't see a problem with this. The box says "Something went wrong" as the server replied with 404.
I can't seem to find anything wrong with the "provider filter" implementation either. You may inspect it yourself here.
I think the issue lays with the endpoint.

@nitrosx do you confirm @noobadmin analysis that the problem is with the endpoint i.e. federated search? Thanks for taking a look.

@noobadmin
When the user select the facility, It seems that the portal sends the query directly to the facility by-passing the federated search.
Is that correct?
For the ESRF issue, the URL used to update the portal is the following:
https://icatplus.esrf.fr/api/documents?filter={"query":"diffraction","limit":50}
unfortunately, the ESRF API is case sensitive and, to work, you should use the following:
https://icatplus.esrf.fr/api/Documents?filter={"query":"diffraction","limit":50}
If you can fix the portal, to take care of this, it should work, although I'm skeptical about the approach to skip the federated search and go directly to the facility backend from the portal (I can be convinced otherwise)
@andygotz I think we need to do a brainstorm first to define the scope of this functionality.
The way I had envisioned was that the facility selector was acting on the results set returned to the data portal and not asking the data portal to contact the specific facility. Hopefully my explanation makes sense.

Max thanks for the feedback. A quick fix is already a start. Yes a brainstorm sounds like a good idea. I thought the portal was filtering on the results it already had retrieved. But the approach of Jiri sounds sound i.e. ask the facility endpoint.
Jiri can you try the quick fix? Thanks!

Hi @nitrosx
Apart of the fact of by-passing the federated search API that I have no strong opinion yet, if we want this to work smoothly I would suggest to harmonize the endpoints. If I am not mistaken, the error 404 sounds good to me because we are not implementing such endpoint.
Currently, the portal sends the requests to the federated search API with lower case, example:
/documents
Later, and for some reason I do not know, the federated search API forward the request to the local implementations with capital letter:
/Documents
In the documentation there is a mix of upper and lower case that makes it confusing:
https://github.com/panosc-eu/search-api/blob/master/doc/api-calls.md#call-5
In general, the recommendation is that URLs should be case-sensitive (https://www.w3.org/TR/WD-html40-970708/htmlweb.html) so I would harmonize this and I would not expect everybody to be case insensitive, it is probably unnecessary and prone to errors.
Just my opinion!

@antolinos I agree with you we should harmonize the URLs.
Do you think we should go all lower case or first letter uppercase?

I think that the federated search api should forward the request to the local implementations without altering it. So, it is the responsability of the portal to use the right case.
So, from the federated search It means that if /documents in then /documents out.
Lower or upper case?
I use to develop endpoints in lower case but as far as same it is used everywhere in the software chain I will be happy.

I prefer lowercase too. Uppercase begs the question then of camel case - should we capitalise words or not? Underscores would be preferred to camel case then. My 2 cents.

If you can fix the portal, to take care of this, it should work, although I'm skeptical about the approach to skip the federated search and go directly to the facility backend from the portal (I can be convinced otherwise)

@nitrosx Is there another way? I've actually naively tried to implement it by filtering on provider key first but without success.

A nice feature might be to send as parameter a list of providers (optional). The providers will be taken into account by the federated search. I think it is fairly easy to implement.
Example:
/api/documents&filter=...&provider=ESS,ESRF,ILL

This would fix the issue and will allow in a future to filter not only by single facilities as today but by type of facility, i.e: photons, neutrons, xfel etc...
The user interface might propose to select by neutrons then:
/api/documents&filter=...&provider=ESS,ILL

or photons:
/api/documents&filter=...&provider=ESRF,SOLEIL,DLS,HZB,PSI

This might add value to the search and Max will be happier :D
Just an idea...

A nice feature might be to send as parameter a list of providers (optional). The providers will be taken into account by the federated search. I think it is fairly easy to implement.
Example:
/api/documents&filter=...&provider=ESS,ESRF,ILL

This would fix the issue and will allow in a future to filter not only by single facilities as today but by type of facility, i.e: photons, neutrons, xfel etc...
The user interface might propose to select by neutrons then:
/api/documents&filter=...&provider=ESS,ILL

or photons:
/api/documents&filter=...&provider=ESRF,SOLEIL,DLS,HZB,PSI

This might add value to the search and Max will be happier :D
Just an idea...

I have this implementation ready to go already but as I mentioned above the federated endpoint didn't support this last time I've checked. @nitrosx

@andygotz I've fixed the casing, should work now but there are still issues with some endpoints...

PSI and ESRF fail to respond to queries with (nested) filters. I undertand they aren't implemented at those sites...

@noobadmin Could you please provide a couple of examples? Thanks

@nitrosx
https://data.panosc.eu/search/?q=lung&facility=ESRF&technique=absorption+microtomography
https://data.panosc.eu/search/?q=lung&facility=PSI&sample_chemical_formula=Cu

@noobadmin thanks a lot for fixing the case, filtering on ESRF now works nicely!
@nitrosx there are still a number of facilities with errors and/or returning inconsistent results e.g. searching on 'covid-19' still has MAX-IV, ESS, XFEL returning results which have no relation with covid-19?

@andygotz long and complicated story short, to the eyes of the scoring service, searching for covid-19 is the almost the same as searching for covid nineteen. That's why you see results which apparently are not relevant to covid-19. They are relevant to the word nineteen which should be in the scoring info.
This is one of the biases that are introduced by the technique that I used for the scoring. I've been thinking about how to correct the bias, but I have not found a robust solution yet.
Interesting enough, if you search just for covid, you will exclude all the unwanted results.
There might be enough material for a paper there

@nitrosx I see. Do you mean the number '19' is searched for or the text 'nineteen' - I presume the number because I cannot see any text nineteen in the entries. A simple solution is to not allow searching for numbers but only for literal text i.e. covid-19 is treated as a single text. If this is not possible (why?) then remove the numbers from the text. Not being able to search for numbers e.g. "1", seems better to me rather than have results which are meaningless. Users could always enter numbers in the parameters if they need to.

In the PaNOSC scoring, all the number are translated to their word representation and to allow to be searched for.
It is possible to have some sequence of words treated as single word and go unchanged through the ingestion that the scoring perform, but it needs to be set-up correctly.
It is possible to remove numbers from the information ingested by the scoring, but that would required a change at all the facilities that have adopted the PaNOSC scoring.

I still don't understand where 'nineteen' is in the text of the results? If the search text changes numbers to words what about the text that is being search? Would it match the nineteen in<PHONE_NUMBER>9? If we drop treating numbers separately could we still keep the option of searching for text with numbers in it e.g. ID19 should still be searchable for.
I understand this is a change for all sites deploying the scoring. What is the work for each site? Is it limited to updating the docker image (are all sites using this)? Or does it involve more work?

If you can fix the portal, to take care of this, it should work, although I'm skeptical about the approach to skip the federated search and go directly to the facility backend from the portal (I can be convinced otherwise)
@nitrosx Any updates on this? I'd be happy to implent the provider filter through federated endpoint if you show me the required query.


Fix content links of picture schema in contracts

The contract was not compliant with the documentation and the actual output of the endpoint. When I generated an api client from the specification, I had no way to get the picture links. This PR fixes that issue.

Docs Build status updates of commit b0c3a7f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav/api-reference/v1.0/contracts/BCOAS1.0.yaml
:white_check_mark:Succeeded

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


Refactor: AddressInfo and EtherscanButton

[x] Remove type in EtherscanButton. It can be inferred using value length. If it's > 42 is assumed to be a TX else an address. If the value is > (66 a TX) can be assumed to be a data value, but for this case, EtherscaButton should not be used.

[x] Rename AddressInfo to EthHashInfo and make the required changes to work with the new EthersacanButton interface.

[x] @fernandomg requested to stop events propagation for EtherscanButton and copyButton. That's to avoid problems with <AddressInput /> component.

Travis automatic deployment:
https://pr50--safereactcomponents.review.gnosisdev.com

Travis automatic deployment:
https://pr50--safereactcomponents.review.gnosisdev.com

Travis automatic deployment:
https://pr50--safereactcomponents.review.gnosisdev.com

sorry, it was too early. Check it now @fernandomg, please.

Travis automatic deployment:
https://pr50--safereactcomponents.review.gnosisdev.com

Travis automatic deployment:
https://pr50--safereactcomponents.review.gnosisdev.com

Travis automatic deployment:
https://pr50--safereactcomponents.review.gnosisdev.com

Travis automatic deployment:
https://pr50--safereactcomponents.review.gnosisdev.com

@mikheevm I'm merging this PR so we can move forward. If you think any other change needs to be done, I'll work on it in a new PR.


Git pull

Users are often confused by fetch and pull. The expect the local directory to change when they press fetch. We should support this behaviour by introducing pull feature.
Git pull = fetch + merge
We can ignore the case with conflicts for now. So it will just be fetch + checkout to remote branch.

Are you having success with the fetch prototype?

this may help: https://stackoverflow.com/questions/39651287/doing-a-git-pull-with-libgit2 it talks about checkout having the wrong default strategy

This new command (lg2) should enable pulling using a terminal command: https://github.com/thebaselab/codeapp/pull/887

Method for reference: https://github.com/libgit2/objective-git/blob/c5ff513bee3c0d02c48e11c38f049577db0f3554/ObjectiveGit/GTRepository%2BMerging.m#L76


Non-native JS traits do not allow default parameters

Consider:
trait Foo extends js.Any {
  def foo(x: Int = 1): Unit
}

This fails with:
Members of non-native JS traits must either be abstract, or their right-hand-side must be `js.undefined`.
    def foo(x: Int = 1): Unit
                     ^

Adding @js.native to the trait makes compilation succeed.

The error message could certainly be improved. But I don't see how to actually make it work.

I'm not sure I understand. Isn't a default parameter in a facade merely a definition of a calling convention? So basically, the following two are equivalent IIUC:
trait Foo extends js.Any {
  def foo(x: Int = 1): Unit
}

trait Foo extends js.Any {
  def foo(): Unit
  def foo(x: Int): Unit
}

The problem is when you implement that trait in a non-native JS class: you don't have to provide a default value, but the compiler also doesn't know where to get it from.

Ah, I see. Fair enough. In this case, only the error message needs improvement indeed.

Hah, this causes an IR checking error:
trait A extends js.Any {
  def foo(x: js.UndefOr[Int] = js.undefined): Int
}

object Impl extends A {
  def foo(x: js.UndefOr[Int]): Int = x.getOrElse(5)
}


Checks if animator is running before resuming it on rescale

Fixes #208 and https://github.com/CartoDB/cartodb.js/issues/631
cc @javisantana

it needs tests

ok, ref for the record: https://github.com/CartoDB/torque/issues/144

Test added, waiting for #210 merge


radio options

This PR will add three properties to the Form component.

radio takes a list of fields to be rendered as radio button groups.
radioGroupWrapperComponent takes a tag name or a component which will wrap radio button groups.
radioWrapperComponent takes a tag name or a component which will wrap individual radio buttons.
radioComponent takes a tag name or a component which will render each radio button.

They are all optional therefore no breaking change is introduced.
This should address #54
TODO

[x] Rename radioWrapperComponent to radioGroupComponent.
[x] Add new radioWrapperComponent to wrap around individual radio options.
[x] Ensure default tags have all the proper accessibility features.

@danielweinmann I think we should default the radio group wrapper to a fieldset and the group label to a tag legend to follow the mdn example (the second one).
Thoughts?

@danielweinmann I think we should default the radio group wrapper to a fieldset and the group label to a tag legend to follow the mdn example (the second one).
Thoughts?

👌🏼

@diogob, I think radioGroupWrapperComponent could be called simply radioGroupComponent. What do you say?

@danielweinmann you can take a look now, introducing unit tests will require a little more research and I'll do it in a future PR.


Problem with Vue and invocations (probably programmer error)

Bug or feature request?
If there is an issue, it's a bug.
Description:
On Firefox, I am receiving an error when trying to use invocations as described in the "Testing" section of the invocation documentation (https://xstate.js.org/docs/guides/communication.html#testing).
I DO NOT receive the service makes an actual call with axios to a a backend service -- so that tells me there is something wrong with how I am using promises, potentially.   But I am just following the example I found.
The machine/service/promise work on codepen, but I can easily reproduce it with a brand new vue project.
(Bug) Expected result:
No error in a browser console.
(Bug) Actual result:
I am receiving the following error in my console:
TypeError: "state is undefined"
    execute interpreter.js:172
    update interpreter.js:182
    start interpreter.js:283
    spawn interpreter.js:496
    exec interpreter.js:437
    execute interpreter.js:173
    execute interpreter.js:172
    update interpreter.js:182
    start interpreter.js:283
    created HelloWorld.vue:62
    VueJS 27
    <anonymous> main.js:6
    js app.js:3073

...
(Bug) Potential fix:
I believe it has something to do with how the promise is working, because axios promises work, but the example promise from the Testing section causes the error to show up (but it does NOT show up in codepen for some reason).
Link to reproduction or proof-of-concept:
To easily recreate the error with a brand new vue project, do this (assumes vue-cli 3):

vue create xstate-test	[I chose default (babel, eslint)]
cd xstate-test
npm install xstate@latest --save
Then overwrite the HelloWorld.vue component with this:

<template>
  <div class="home">
    Hello.
  </div>
</template>

<script>
import { Machine, interpret } from 'xstate';

// An example service that can be invoked by the state machine
const services = {
  getService: (ctx, event) => fetchApp(ctx.appId)
}

const fetchApp = async appId => {
  return {
    appId: appId,
    other: 'Other Data'
  }
}

const machine = new Machine(
  {
    id: 'master',
    initial: 'init',
    strict: true,
    states: {
      init: {
        invoke: {
          id: 'serviceExample',
          src: 'getService',
          onDone: {
            target: 'test_state'
          }
        },
      },
      test_state: {}
    }
  },
  {
    services
  }
)

export default {
  name: 'home',

  data: () => ({
    service: null,
    current: null,
    context: {
      config: null
    }
  }),

  created: function() {
    this.service = interpret(machine.withContext( this.context ))

    // Start service on component creation
    this.service.onTransition(nextState => {
      this.current = nextState;
    }).start()
  },

}
</script>

npm run serve

If you open a browser console, and open http://localhost:8080/ you will see the error.

Next time can you please put this on CodeSandbox? Makes debugging much quicker.
I think the core issue is that the promise returned from fetchApp(...) evaluates instanceof Promise to false in Firefox for some reason. Still trying to figure this one out, but if you have any ideas, I'm all ears.
In the interpreter, it checks if it's a promise (src instanceof Promise) to see if it should spawn a promise service or a machine otherwise. It gets state is undefined because it assumes src is a machine (with an initial state) and not a promise, which is wrong.

Yeah, this seems to be a Vue error, as this works perfectly in Firefox (latest) without any framework: https://codesandbox.io/s/pw9l3zxzn0
Please investigate this within Vue.

It also works with axios when you do a HTTP.get(...), so there's something different between how they are returning promises versus the syntax in the example.  Also, it works perfectly within Vue on CodePen.

Must be a CodeSandbox issue then.

Add to index.html:
<script>
  window.originalPromise = Promise;
</script>
<!-- built files will be auto injected -->

Add at the bottom of HelloWorld.vue
window.promiseInHelloWorld = Promise

console.log('HelloWorld.vue: original Promise?', Promise === window.originalPromise)

Add to node_modules/xstate/interpreter.js just before the if (source instanceof Promise) block:
console.log('interpreter.js: original Promise?', Promise === window.originalPromise)
console.log('same Promise?', Promise === window.promiseInHelloWorld)
console.log('source instanceof original Promise?', source instanceof window.originalPromise)
console.log('source instanceof Promise?', source instanceof Promise)
console.log('source instanceof Promise in HelloWorld.vue?', source instanceof window.promiseInHelloWorld)

Yields the following in Firefox:
HelloWorld.vue: original Promise? false

interpreter.js: original Promise? false
same Promise? false
source instanceof original Promise? false
source instanceof Promise? false
source instanceof Promise in HelloWorld.vue? true

This behavior appears to indicate the Vue dev server environment is giving each compiled file a unique Promise polyfill...? Wild.
Looks like a Vue thing. Perhaps it can be configured to be a global polyfill somewhere?

In the early days of Promises, it was typical to check for a Promise via duck-typing (basically, support any "then-able"), given how many implementations were floating around.
This alternate check:
if (typeof source === 'object' && typeof source.then === 'function') {

appears to work fine.
A duck-typed check for a then-able similar to the above is about the only thing XState could do to aid in any situation where a function is returning an instance of a different Promise polyfill than the global Promise.

For comparison, Chrome yields:
HelloWorld.vue: original Promise? false

interpreter.js: original Promise? true
same Promise? false
source instanceof original Promise? true
source instanceof Promise? true
source instanceof Promise in HelloWorld.vue? true

HelloWorld.vue still appears to have a different Promise, but at least the async function is return the original global Promise. (And XState gets the original global Promise as well.)

I'll accept a PR for this.

In the early days of Promises, it was typical to check for a Promise via duck-typing (basically, support any "then-able"), given how many implementations were floating around.
This alternative check:
if (typeof source === 'object' && typeof source.then === 'function') {

Perhaps instead of an alternative check, it's an additional check?  I do think checking to see if it has a then(...) function is the way to go.  Maybe something like:
if (source instanceof Promise || (typeof source === 'object' && typeof source.then === 'function'))

I strongly feel that source instanceof Promise should just work. Anything else is an external issue not related to XState, but I'm willing to accept a PR for duck-typing that it is an object with a .then function.

Worth noting that my example above is not an ideal thenable check, as:
> typeof null
"object"

Per the Promise/A+ specification (https://promisesaplus.com), which defines the rules for interoperability for conformant Promise implementations, a:

1.2 “thenable” is an object or function that defines a then method.

So, a better check would be (encapsulated as a helper):
const isThenable = (x) => x && (typeof x == 'function' || typeof x === 'object') && typeof x.then === 'function'

Given the existence of the Promise/A+ specification, the popularity of some of those implementations (Bluebird comes to mind), and their use in libraries in the wild, it's probably worth going the thenable route for convenience. Other conformant Promise implementations can be "adopted" via Promise.resolve(someThenable), but first we have to detect it is a thenable you want to adopt. (Even that has some wrinkles.) Since we just care to .then() here, anyway, adopting is unnecessary.
(Much like state machines and statecharts, Promises have surprising depth as a subject. I understand where you're coming from, though, now that evergreen environments typically include native Promises, and strongly prefer those where possible.)

This should be fixed in https://github.com/davidkpiano/xstate/pull/415. We'll reopen if the issue appears again.


tools/flow: reference to parent struct does not create dependency on contained tasks

What version of CUE are you using (cue version)?

cue version v0.5.0-alpha.1

       -compiler gc
     CGO_ENABLED 1
          GOARCH amd64
            GOOS darwin
         GOAMD64 v1

Does this issue reproduce with the latest release?
if this alpha is considered latest release, then yes
What did you do?

package servetest

import (
	"tool/cli"
)

#Things: [mapkey=string]: #Thing & {Name: mapkey}
#Thing: {Name: string, Purpose: string}

Things: #Things & {
	"MyThing1": {
		Purpose: "To serve butter ..."
	}, "MyThing2": {
		Purpose: "Ketchup bot."
	}, "MyThing3": {
		Purpose: "Aggressively collect crumbs."
	}
}

command: "servetest": {
	doServe: {
		for thingName, thing in Things {
			let serveText = thing.Name + "'s purpose is to " + thing.Purpose
			"doPrint-\(thingName)": cli.Print & {
				text: serveText
			}
		}
	}

	afterServing: {
		// note: careful about the success, otherwise the whole command (will repeat)
		$after: doServe // runs many times, doesn't wait
		//		$after:    doServe.$success  // runs once, but doesn't work
		printDone: cli.Print & {text: "done serving "}
	}
}

run: cue cmd servetest
and run it multiple times, since it will sometimes wait
What did you expect to see?
I expect "afterServing" to wait until doServe has completed, and print only once.
What did you see instead?
There are 2 scenarios here:
1:
$after: doServe will repeat the doServe N=3 (in collection) times
2:
$after: doServe.$success, runs once, but does not wait until completion.

note that there is a condensed version of:
https://github.com/kghenderson/cue-companion/blob/main/examples/direnv_example/tools/serve/serve_tool.cue

Both versions of $after: <...> do not create an inter task dependency. You have to refer to an incomplete field within the task(s) you wish to wait on. For your example, you would want something like
$after: [ for thingName, _ in Things doServe["doPrint-\(thingName)"].$done ]

the subfield of the task to wait on doesn't matter so much, just that it's incomplete. It should be referenceable, so you would need to add $done: _ in the print tasks, and you never need to fill it. For tasks which have incomplete fields they fill, you can wait on those

thank you @verdverm. i'm going to leave this issue open though because i think this behaviour as it is, is very unintuitive. i'd have a hard time trying to explain this to others.

@kghenderson thanks for raising this. My analysis on the problem is slightly different to @verdverm's.
First to point out one thing: the use of $after must be within a task. All it does is create a dependency between tasks where there would otherwise not be a reference to create such a dependency.
In your original code, the $after field appears as a sibling of a task.
Consider a slight rewrite of your example therefore:
package servetest

import (
	"tool/cli"
)

#Things: [mapkey=string]: #Thing & {Name: mapkey}
#Thing: {Name: string, Purpose: string}

Things: #Things & {
	"MyThing1": {
		Purpose: "To serve butter ..."
	}
	"MyThing2": {
		Purpose: "Ketchup bot."
	}
	"MyThing3": {
		Purpose: "Aggressively collect crumbs."
	}
}

command: "servetest": {
	doServe: {
		for thingName, thing in Things {
			let serveText = thing.Name + "'s purpose is to " + thing.Purpose
			(thingName): cli.Print & {
				text: serveText
			}
		}
	}
	printDone: cli.Print & {
		$after: doServe
		text:   "done serving "
	}
}

The $after field in printDone now references the doServe struct, which contains the three tasks MyThingN.
Looking at a mermaid dependency graph of the initial dependencies:
graph TD
  t0("command.servetest.doServe.MyThing1 [Ready]")
  t1("command.servetest.doServe.MyThing2 [Ready]")
  t2("command.servetest.doServe.MyThing3 [Ready]")
  t3("command.servetest.printDone [Ready]")

This highlights the problem: there is no dependency between any of the MyThingN tasks and printDone. Consider a further rewrite:
package servetest

import (
	"tool/cli"
)

#Things: [mapkey=string]: #Thing & {Name: mapkey}
#Thing: {Name: string, Purpose: string}

Things: #Things & {
	"MyThing1": {
		Purpose: "To serve butter ..."
	}
	"MyThing2": {
		Purpose: "Ketchup bot."
	}
	"MyThing3": {
		Purpose: "Aggressively collect crumbs."
	}
}

command: "servetest": {
	doServe: {
		for thingName, thing in Things {
			let serveText = thing.Name + "'s purpose is to " + thing.Purpose
			(thingName): cli.Print & {
				text: serveText
			}
		}
	}
	printDone: cli.Print & {
		$after: [ for _, v in doServe {v}]
		text: "done serving "
	}
}

The change here being in the $after field. Now the dependency graph looks like this:
graph TD
  t0("command.servetest.doServe.MyThing1 [Ready]")
  t1("command.servetest.doServe.MyThing2 [Ready]")
  t2("command.servetest.doServe.MyThing3 [Ready]")
  t3("command.servetest.printDone [Waiting]")
  t3-->t0
  t3-->t1
  t3-->t2

(note there is no order between the MyThingN tasks, I'm assuming that was intended from your side).
So the bug here is that the reference to doServe does not capture the three MyThingN tasks when it should. That is a bug.
@verdverm note that you don't need to reference a field in a task, a reference to the task itself is sufficient.
So @kghenderson your intuition here is correct. The problem is a bug is getting in the way! I will repurpose this issue and summarise the bug in the next comment.

Precise summary of the bug described in https://github.com/cue-lang/cue/issues/2078#issuecomment-1315085636.
What did you do?
exec cue cmd servetest
cmp stdout stdout.golden

-- x_tool.cue --
package servetest

import (
	"tool/cli"
)

command: "servetest": {
	doServe: {
		print: cli.Print & {
			text: "this is a test"
		}
	}
	printDone: cli.Print & {
		$after: doServe
		text:   "done serving "
	}
}
-- stdout.golden --
this is a test
done serving

What did you expect to see?
Passing test, 100% of the time.
What did you see instead?
Failing test, ~50% of the time:
$ ts repro.txtar
> exec cue cmd servetest
[stdout]
done serving
this is a test
> cmp stdout stdout.golden
--- stdout
+++ stdout.golden
@@ -1,2 +1,2 @@
-done serving
 this is a test
+done serving

FAIL: /tmp/testscript1603975044/repro.txtar/script.txtar:2: stdout and stdout.golden differ

FYI I just landed https://review.gerrithub.io/c/cue-lang/cue/+/546541 to make debugging dependency issues easier.

@verdverm  made a comment in slack last week that sparked a light-bulb for me, when he said:

tasks are values ... which have the $id field within them
basically, the engine walks the value tree, looking for the $id field

once this dawned on me, i understood the error of my ways (thank you tony)
i had simply (incorrectly) assumed that 'parent tasks' were groupings of 'sub-tasks', which obviously isn't the case.
note that i do think that we should revisit this behaviour, but there's a lot more to unpack with that,
as it ties in to multiple other requests and proposals.
re: referring to a task and not done, i was getting incorrect behaviour for this, but i will retest in light of that potential bug-fix :-)

i'll also add that the cue documentation seems to imply (in my flawed interpretation) that tasks are the keys within a command map, i.e. one level down. but i was using the cuetorials commands as reference which showed that wasn't strictly the case, and in my own experiments, the nested "tasks" worked as well. so then i just figured any descendant key of the command was a 'task'. again, this obviously isn't correct, but just wanted to explain the rationale. so this might need to be addressed in the description of 'tasks' on the cue-lang site.

i had simply (incorrectly) assumed that 'parent tasks' were groupings of 'sub-tasks', which obviously isn't the case.

Just to clarify though, the declaration of a dependency from one task to some other tasks should work when the target is a reference to a value that contains tasks, i.e. a parent struct (because for cue cmd structs are walked to find struct values which define an $id field, as you point out). That does not work today, but it should. So your intuition is correct.

Adding a note to self: need to clarify whether this is a feature request or not. From memory, the reason I thought this was a bug is that there is/was a place in the CUE code base that suggested this should work. Just noting in passing because we are trying to be brief in our big issue garden!

I think this is a bug. It's counterintuitive that reference to parent task creates no dependency. For now the workaround [ for _, v in tasks {v}] can be used.

The way cue/flow determines dependencies, any field within a task referencing another task will (should) create a dependency. It's basically a big walk over the CUE value / lattice at the ADT level to find any references.
I believe the $after was more to let people know by having something in the schema they see in the docs.


It seemed to me that this would be better, because this is the only…

…solution when my application started working with the Discord SDK without obvious errors.

I think that issue might lie in the way you included this library into your project.
Are you using Gradle or did you copy the files into your project directory?

Are you using Gradle or did you copy the files into your project directory?
I copied the source code of your product into my project.

Ok, so the problem is probably that the folder you stored the native libraries in (native/) is not part of the class path.
I use the getResourceAsStream-Method which tired to find the resource anywhere in the class path. I prefer that, because it makes it possible to load resources from JAR files.
Please try to place the native/ inside your class path. This could for example be in the bin/ directory.
If you need help with that, please tell me which IDE you are using.

Oh. My mistake. Excuse me. I guess I just didn't understand some things. In this case, your code will be the best solution. My bad.


Clean up confusing variable in dataplane.py

aws_nodes_to_provision was confusing previously

Merge blocked on: https://github.com/skyplane-project/skyplane/pull/770


Disable realtime aggregate metrics if multi value dimension present

Consumption will stop in the case of aggregateMetrics=true in combination with multi-value dimension. More details in https://github.com/apache/incubator-pinot/issues/3867
This will be fixed in a separate PR. Until then, disabling the realtime aggregation in this scenario

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1f9d096). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #3868   +/-   ##
=========================================
  Coverage          ?   67.17%
  Complexity        ?        4
=========================================
  Files             ?     1029
  Lines             ?    50796
  Branches          ?     7097
=========================================
  Hits              ?    34122
  Misses            ?    14341
  Partials          ?     2333

Impacted Files
Coverage Δ
Complexity Δ

...manager/realtime/LLRealtimeSegmentDataManager.java
70.86% <0%> (ø)
0 <0> (?)

.../core/indexsegment/mutable/MutableSegmentImpl.java
67.66% <0%> (ø)
0 <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 1f9d096...65416e3. Read the comment docs.

Issue: https://github.com/apache/incubator-pinot/issues/3867


More Workflow example?

More Workflow example?

more sparsectrl sample will be most appreciated!

SparseRGB/Sparse Scribble model really make me confused, would you kindly provide some samples for these?

https://civitai.com/models/258137/sparse-controlnet-animate-diff-video-to-video-extras


fix(line-numbers): line height will be zero when  is hidden

When the <pre> element is hidden by default, lineNumberSizer.getBoundingClientRect().height  will be 0.
When the user sets hidden to false,line numbers will be displayed as follows.This is not the expected display.

Yeah, I don't think we should fix this internally. If the pre is display: none initially, my suggestion would be to rehighlight it when you reveal it. I'd be really hesitant to make this change cuz I'd definitely be concerned about breaking consumers.
@RunDevelopment I'd be ok with closing (we still appreciate the contribution!), but I'd like to hear your opinion given you've played with this.

@mAAdhaTTah I agree. Rehighlighting will trivially solve this issue (though it's a bit of a waste performance-wise).
I think we should rather focus on #2413. After this PR, we can let the browser do the positioning of line numbers (and line highlights), so we don't have to worry about this (and a few other issues) at all.


Ubuntu 16.04 - Can't install mdns

Hello.
I tried to install mdns on 2 different servers and I got the same message on both :
><EMAIL_ADDRESS>install /home/power/Desktop/tmp/core/node_modules/mdns
> node-gyp rebuild

make: Entering directory '/home/power/Desktop/tmp/core/node_modules/mdns/build'
  CXX(target) Release/obj.target/dns_sd_bindings/src/dns_sd.o
In file included from ../src/dns_sd.cpp:1:0:
../src/mdns.hpp:32:20: fatal error: dns_sd.h: No such file or directory
compilation terminated.
dns_sd_bindings.target.mk:153: recipe for target 'Release/obj.target/dns_sd_bindings/src/dns_sd.o' failed
make: *** [Release/obj.target/dns_sd_bindings/src/dns_sd.o] Error 1
make: Leaving directory '/home/power/Desktop/tmp/core/node_modules/mdns/build'
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:126:13)
gyp ERR! stack     at ChildProcess.emit (events.js:214:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)
gyp ERR! System Linux 4.13.0-43-generic
gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/power/Desktop/tmp/core/node_modules/mdns
gyp ERR! node -v v8.11.1
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/power/.npm/_logs/2018-06-02T00_49_12_258Z-debug.log

Please take a look at the README and read the install section.


Help description for options with nil default values, fix #81

Awesome, thanks for this fix!

Awesome, thanks for this fix!


Clear renderCalendarInfo timeout before assigning a new timeout (#1323)

This is a do-over of @mstanaland's work on https://github.com/airbnb/react-dates/pull/1736 to fix a memory leak issue where the timeout is not cleared before being set again. I am doing this so that we can be current with master, since the original PR went unmerged for six months due to lack of a unit test. This branch does contain a test displaying that setState is called by the timeout only once.
Original PR description by @mstanaland:

Clears the setTimeout assigned to this.setCalendarInfoWidthTimeout before a new setTimeout is assigned/reassigned to prevent a possible memory leak.
Fixes issue #1323

Looks like the failures in Travis CI are not related to the code changed by this PR. @ljharb is this normal for this project?

In the future, please do not create a second, duplicate PR - post a link to your branch and I can update the original PR. Now both of these need to stay open and kept in sync for the duration.

If that's the only failure, it's something I can try to look into in the next week or so.

Okay sure, would you like me to close this PR?

@smit1625 no, please don't close it :-) i'll need to keep it in sync with #1736 until both are landed.


Role administration with MariaDB?

System information:

Operating system (distribution) and version: Windows 10
DBeaver version: 7.2.2

Connection specification:

Database name and version: MariaDB 10.5

Describe your question:
Is it possible to administrate roles? Create and edit roles and assign roles to users?

Hi @Dimas-sc ,
unfortunately it isn't implemented yet.
I've labeled your ticket as feature request. Thanks for suggestion


Tags are sorted in 'random' order in OpenAPI.

Currently the endpoints documentation contains a map of tag name to PathItem:
final class OpenApi private (
    val info: Info,
    val paths: Map[String, PathItem],    // <----
    val components: Components
)

Since this uses a map underneath the tags appear random in the resulting openAPI (see for example https://otm5.opentripmodel.org/, which is btw a nice example of what endpoints4s can achieve!). This order also changes if a new tag is added since Maps are unordered in their keys.
I think it would be quite nice if we can sort the keys, for example alphabetically, so it looks a little more tight. I would also be willing to implement this feature myself.
My suggestion would be to add an optional 'sorting' to the OpenApi class that is used when present.

I wonder if this is more the responsibility of the OpenAPI UI generator instead? I think the current implementation keeps the paths in the order they are passed to the call to openApi(...)(endpoint1, endpoint2, ..., endpointN) (but we have no test for that, so I’m not sure).

As far as I can see the openApi(...)(endpoint1, endpoint2, ..., endpointN) will group the endpoints by tag using a simply groupBy.  The result is then later processed sequentially by simply calling .iterator on the scala.Map and processed in that order. In any case the UI we use simply uses the order it is present in the JSON.
Would you be open for a PR? I can make something locally and confirm I can ensure an user defined order (defaulting to just a String Ord).

This is the piece of code (in endpoints4s.openapi.EndpointsWithCustomErrors#openApi) where the ordering changes:
endpoints
        .groupBy(_.path)
        .map { case (k, es) =>
        ...
        }

Right, the groupBy breaks the ordering!
Do you think it would be enough to just keep the order in which the endpoints are passed to the openApi call? It would be simpler to implement it with https://github.com/scala/scala-library-next/issues/51, but we can do it manually in the meantime.

Yes, sorry if that was not clear, but if keeping the original insertion order works for you that would be a simple solution (we could make it more sophisticated in the future if needed). So yes, you will have to use a LinkedHashMap and to implement the groupBy operation by yourself (so that it returns a LinkedHashMap instead of a default Map).

NP :) I still think we might misunderstand each other. To be clear, we need a PR in endpoints4s to make this possible. The Map implementation in the openApi method has to change to use LinkedHashMap instead. I think everything else will work from there on out. Since the JSON creation (also in endpoints4s) will automatically use the original order since that map is used under the hood.
Shall I make a PR? I think it is just a one-liner

I thought we could change the implementation of openApi without changing the signatures of other methods, but I might be wrong. In any case yes, a PR is welcome :)

Yes, I think so too, I will try :smile:


TypeError: prisma_client__WEBPACK_IMPORTED_MODULE_1_.Prisma.sql is not a function

Bug or Error if run in ubuntu
any solve this problem ?

👋
Could you provide more details, such as when it occurs, your setup, how to reproduce it, and a minimal reproduction?


[Spark] Fix InCommitTimestampSuite to make master compilable

Which Delta project/connector is this regarding?

[X] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[ ] Other (fill in here)

Description

The merge of these two PRs:

https://github.com/delta-io/delta/pull/2902
https://github.com/delta-io/delta/pull/2883
broke master. This renames a variable in InCommitTimestampSuite to unbreak it.

How was this patch tested?

Test-only fix.
Does this PR introduce any user-facing changes?

No

Force merging


Fix/radius ux

Added padding for radius. Changed name:radius to name:distance.
Distance filter only appears when location selected.
Closes #129

@JacOng17 can you fix the small css lint issues on codeclimate before merging

@Br3nda Done! :smile:  Commit ref: 0d3b763


🐛Bug Fix: Centrally align and justify the profle title

Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

This pull request is meant to give alignment to the pod title.
@ricardoprins @GustavoElizarraras please review

I have tried out the changes locally and LGTM :+1:.

@khattakdev


"az acr build"  doesn't work with "Azure-Samples / acr-build-helloworld-node"

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az acr build
Errors:
The command failed with an unexpected error. Here is the traceback:
'NoneType' object is not callable
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/acr/build.py", line 109, in acr_build
    variant=platform_variant
TypeError: 'NoneType' object is not callable

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

git clone https://github.com/Azure-Samples/acr-build-helloworld-node.git
cd acr-build-helloworld-node
az acr build --registry {xxx} --image helloacrtasks:v1 .

Expected Behavior
Environment Summary
Linux-5.4.0-1055-azure-x86_64-with-debian-10.2 (Cloud Shell)
Python 3.6.10
Installer: DEB

azure-cli 2.26.0 *

Extensions:
ai-examples 0.2.5
ssh 0.1.5

Additional Context

I suspect it may be due to the acr being windows rather than linux?

route to service team

Thanks for reporting it and sorry for the inconvenience.
2.26.1 has been released to address the issue.
https://docs.microsoft.com/en-us/cli/azure/release-notes-azure-cli?tabs=azure-cli
On cloud shell, you can upgrade to 2.26.1 by running
curl -L https://aka.ms/InstallAzureCli | bash

after upgrading it is working now.
Thank you a lot for very rush reaction (new release in couple of hours only - it is simply fantastic)


Support Setting UI Options Programmatically

Currently not exposed when using a base activity.

Pick one:

supportSetUiOptions(..)
setSupportUiOptions(..)
setUiOptions(..)
getSupportWindow().setUiOptions(..)

as executed in the context of the Sherlock activities?

So which one of the four did you implement? I cant find any method related to setUiOptions..

getSherlock().setUiOptions(...)

My bad, i did try getSherlock() before posting here but doesnt match any method, turns out its due to the version that i'm using (RC1)..its working great with the 4.0 release, thanks for the answer anyway..


result and segments are empty

For this code
const filePath = `${RNFS.DocumentDirectoryPath}/whisper/ggml-tiny.en.bin`
const sampleFilePath = `${RNFS.DocumentDirectoryPath}/youtube_audio_3LopI4YeC4I.wav`

const whisperContext = await initWhisper({ filePath })

const options = { language: 'en' }
const { stop, promise } = whisperContext.transcribe(sampleFilePath, options)

const { result, segments } = await promise
// result: (The inference text result from audio file)

console.log('---result---', result);
console.log('---segments---',segments);

I got this print
---result---
---segments--- Array []
This audio file is a English content, and the pronunciation is pretty clear
I do not know why got this.
Thanks!
@jhen0409

Is anyone here? I've struggled with this a whole day : (

You can check the WAV file is 16khz mono first, only this format is currently supported in whisper.cpp.
Otherwise, you can also try to find related issues in whisper.cpp repo.

Thanks, my process is to download the audio file in webm format, then use ffmpeg-kit-react-native to convert it into the same metadata as jfk.wav, it still doesn't work...


Link parse issue

I wrote this in my markdown file:
[https://www.example.com](https://www.example.com/?page=5&load=4)

And then, I got this by parsedown:

https://www.example.com

But what I wanted is:

https://www.example.com

Parses fine in http://parsedown.org/demo

@kminchev: It does not.
@772807886: Duplicate of #358.
Note: You can disable auto-linking by setting $Parsedown->setUrlsLinked(false).


Change G12 banner when all drafts complete

In our user research we discovered that users expected the banner to change when they had completed all their drafts.
Change the banner to a success message if the user has no unsubmitted G12 recovery drafts (see #1325).
Content on this banner is not final.
There is G12 recovery logic to show a banner on the /frameworks/<string:framework_slug>/services route, however I've not updated this as we've now added a new route (/all-services, see #1301) for the G12 recovery suppliers to see their services and therefore the banner will never be shown on the /services route.

https://trello.com/c/p3BVSpj6/740-2-change-banner-when-all-drafts-complete

You have marked all your services as complete. Your services will be automatically submitted on 25 February.
[some suppliers may not complete all draft services. should we also consider?]
You have marked X services as complete. Your services will be automatically submitted on 25 February.


devon4j templates -  ETO should derive from RevisionedEto rather than AbstractEto

On devon4j, in case some project is using Envers via RevisionedPersistenceEntity:
We would have an entity implementing RevisionedPersistenceEntity (or RevisionedEntity). Using that entity as input. the generated ETO should derive from RevisionedEto rather than AbstractEto.
Related PRs
devon4j#16.
Related issues
#751

This should be an opt in right? Otherwise, you would do that for everyone, which is mostly not needed.

Yep, I also agree. Pseudocode:
if (pojo instanceof RevisionedPersistenceEntity) {
  generateWithBaseClassIfNothingElseToExtend(RevisionedEto.class);
} else {
  generateWithBaseClassIfNothingElseToExtend(AbstractEto.class);
}

Please note that this applies to both spring-data as well as DAO in devon4j.


Task/jh/addon manager packaging

addon manager packaging

Sorry I was late to this review. I didn't see an email notification until it had already merged. One thing I did for the AddOnManager that proved very useful was to build and save the artifacts in TeamCity on every build. Makes it easy for anyone to download the Add-on for drag-and-drop install and testing.


Lemur and SSLMate

Has anyone tried to integrate Lemur with SSLMate as a CA? Which plugin should be used?

Hi, a plugin for SSLMate does not currently exist. One would need to be created before it could be used. Sorry for the inconvenience!


Fix a typo in the word tailwind in index.css

The layer name had a typo in the word tailwind.
I don't think it was intentional as the code works fine even when the typo is fixed. So adding a PR for that. Great work on the repo btw 🥳

Thanks for the PR


Fix toggling debug info crashing the editor

Fixes https://github.com/superlistapp/super_editor/issues/2254

Changes Stack in SuperEditorImeDebugVisuals to SliverHybridStack because the render object now lives in a sliver
Implements forgetChild inside ContentLayersElement. This is necessary because the content of ContentLayersElement is being reparented using GlobalKey so it needs to be excluded from visitChildren when unmounting.

@knopp are you still working on this, or do you think you responded to all my comments? Whenever you do, please re-request a review so that I'm notified.

I added missing comments.

@knopp I don't see any code comments in the change log

@knopp I don't see any code comments in the change log

That's because I forgot to actually push the changes :) Sorry about it. Should be there now.

@knopp can you give this PR another pass for comments?

@matthew-carroll, let me know if this is any better.

@angelosilvestre can you cherry pick this?


[multi-chart] bump charts to latest common library version

Special notes for your reviewer:

bumped common library to 1.6.1
bumped all charts by minor version
zigbee2mqtt will expectedly fail due to the "USB device not found" issue
Disclaimer added to the charts about upstream projects
updated OWNERS file to include bjw-s and myself
left out jackett and ombi, these 2 were covered in other PRs

For real closes https://github.com/k8s-at-home/charts/issues/183
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [radarr])

I may hav to pull zibgee2mqtt into it's own PR, but let's see how far it gets before it fails


[Android] Can't load ads from Ad Manager

On Android, but probably ios as well, wasn't able to debug ios yet (ads don't load at all on ios so far).
Loading Ads with admob unit id works but loading with AdManager doesn't.
In https://github.com/NativeScript/firebase/blob/3de8339d6056bbe5ecf584dbb79a3241ddd990f2/packages/firebase-admob/src-native/android/admob/src/main/java/org/nativescript/firebase/admob/FirebaseAdmob.kt#494
fun load(adLoader: AdLoader, request: String, isAdManager: Boolean)
The load method shows isAdManager boolean parameter, but the load method in BannerAd for example only takes a RequestOptions:
load(options?: RequestOptions)
Where RequestOptions are:
export interface RequestOptions {     contentUrl?: undefined | string;     keywords?: string[];     networkExtras?: undefined | {         [key: string]: string;     };     requestAgent?: undefined | string;     requestNonPersonalizedAdsOnly?: undefined | false | true; }
Maybe it's the requestAgent field, but I can't find any documentation

Issue seemed to be on ad manager side.


Hailxor.java

Related Issue
Solved issue number: #151
Describe the changes you've made
Hail XOR algorithm in java language
Language used for coding
Java

Nice work :)


Low-allocation version of JSON Pointer and other improvements

Implements a low-allocation version of JSON Pointer.
API is mostly the same, but it does break.  Primarily, JsonPointer is now a struct.
Previously, JsonPointer held a collection of PointerSegments.  These were all classes and each segment held a string.
Now, JsonPointer is a struct that holds a string representation of the pointer and a collection of Ranges that identify each segment.  This architectural change improved run time and memory consumption a little.
Further investigation revealed that much of the memory allocation came from actually parsing the string, where lists were used to manage collections.  Because these lists are only used within the method scope, we can make another optimization to use Memory<Range> instead (via the MemoryPool<T> class).  This change cut run time by 40% and memory usage by 51%!
Parsing Pointers
This benchmark parses the pointers that are in the spec n times.

Method
n
Mean
Error
StdDev
Gen0
Allocated

master branch
1
5.830 us
0.2559 us
0.7300 us
4.6387
9.48 KB

local
1
3.188 us
0.0492 us
0.0460 us
2.2583
4.62 KB

master branch
10
54.627 us
1.0469 us
2.9355 us
46.3867
94.77 KB

local
10
31.482 us
0.4489 us
0.4199 us
22.5830
46.17 KB

master branch
100
537.704 us
10.6843 us
22.5369 us
463.8672
947.66 KB

local
100
329.112 us
2.6580 us
2.4863 us
226.0742
461.72 KB

JSON Schema Test Suite
Additional improvements were made to schema processing, including refactoring to perform more analysis at registration time and more targeted memory management.
This benchmark runs the test suite n times.

Method
n
Mean
Error
StdDev
Gen0
Gen1
Allocated

master branch
1
412.7 ms
14.16 ms
41.30 ms
27000.0000
1000.0000
82.66 MB

current
1
301.6 ms
5.93 ms
10.07 ms
23000.0000
7000.0000
78.41 MB

master branch
10
1,074.7 ms
22.24 ms
63.82 ms
218000.0000
11000.0000
476.56 MB

current
10
945.9 ms
18.64 ms
32.15 ms
216000.0000
5000.0000
472.94 MB

Something feels off about this, and I can't quite identify it.  I'm not going to merge this until I'm completely comfortable with the changes.


mysql 0.4.1 throws exception

With the database_cleaner 1.5.1 and mysql 0.4.1 I get the following exception:
 Specified 'mysql2' for database adapter, but the gem is not loaded. Add `gem 'mysql2'` to your Gemfile (and ensure its version is at the minimum required by ActiveRecord). (Gem::LoadError)

I think this issue may not caused by database_cleaner, it's caused by the mysql2 version you're using.
See: https://github.com/rails/rails/issues/21544

You are right. Thank you.

:smile:


Fix translator code to check retry error reason

The error response from translator api
{
  "error": {
    "code": 403,
    "message": "User Rate Limit Exceeded",
    "errors": [
      {
        "message": "User Rate Limit Exceeded",
        "domain": "usageLimits",
        "reason": "userRateLimitExceeded"
      }
    ]
  }
}

As can be seen its not contained inside a "data" key

@quartzmo I checked to see if there where any tests and as you said found none, its my intention to add them once I clear some time on my schedule

@adrian-gomez Thank you very much! I will wait to hear from you until next week.

@quartzmo tests added, let me know what you think

Not sure why the tests are failling,  running rake ci locally works for me

@adrian-gomez Thank you very much for this contribution, much appreciated.

thank you for maintaining this, any idea when a new release including this fix might be out?

I'm pretty sure it will happen early next week (if not earlier). But don't be afraid to ask for status on this, or anything else we can do to help you out.

@quartzmo any updates on the release? We are waiting for the exponential backoff built on the gem to work, or we will have to implement it ourselves in our app

We'll get this out by end of day, today.

New gem release google-cloud-translate/0.23.1 is now available, containing the changes made in this PR.

Thanks @adrian-gomez!

Thanks @quartzmo and @blowmage :D


Playwright: how to automate scrolling thru the table

I have a table with several rows (300+) and with the scroll bar can you please clarify how to scroll the page using playwright?  the table  load the records as user scrolls the page, and when the scrolling is done all the way to bottom of the table, it loads all the rows in the table. Is there a way in playwright testing library to scroll down till the end of the table ?
Before scrolling, the table shows only certain number of rows, and all other records are invisible, so when we search for something from invisible rows from the table, it is not able to find, eg: 1234 is the order no that i want to search , however it is at the bottom of table, so when i run the following code, it says it is unable to find.
Can you please advise whether i should scroll each time and search for the order no if so can you please tell me how i can scroll thru the table using playwright /playwright dom?
and how to make sure that i scroll thru all the very to bottom of the table?
note: it is infinite scroll where we the table loads certain no of rows as user scroll thru the page . and user can not scroll any further after all rows are loaded.
The table screenshot is attached below.

        let awardRound1 = await screen.queryByTestId('paged-table-Vacation').within().getByRole('row', {name: RegexParser("1234")}).within().queryByTestId(/td-IFVacationCrewMemberVacationAccruals-awardedRound1/).textContent();
        console.log("awardRound1" + awardRound1)
        let originalAccruedDays = await screen.queryByTestId('paged-table-Vacation').within().getByRole('row', {name: RegexParser("1234")}).within().queryByTestId(/td-IFVacationCrewMemberVacationAccruals-originalAccruedDays/).textContent();
        console.log("originalAccruedDays" + originalAccruedDays)

Hi @InduKrish, I'll refer you to my comment on the related/duplicate issue that was posted.

[...] I'm sorry, but this really has nothing to do with playwright-testing-library. I'd recommend checking out the Playwright Community resources and perhaps ask in Discord or on Stack Overflow.

You might try selecting the last row in the table and invoking Locator.scrollIntoViewIfNeeded().