Demo Kit: redirect to new URL prevents back navigation

URL (minimal example if possible):
E.g.: https://openui5nightly.hana.ondemand.com/#/sample/sap.m.sample.ObjectStatus/preview
Steps to reproduce the problem:

Open the above "legacy" link.
User gets redirected to the new URL (without hash)
Try to navigate back to this GitHub page.

What is the expected result?
Users should be able to navigate back, even if they were visiting the page via the "old" URL.
What happens instead?
When trying to navigate back, Demo Kit redirects the user to the new URL again.

Hi boghyon,
I was not able to reproduce this issue, although I remember facing the same issue few days back. Due to the fact that the issue cannot be reproduced anymore, I will close this issue.
Regards Sebastian

Hi @HerzogIgzorn,
Unfortunately, the issue is still reproducible. Could you try again with the environment I tested below?
Fail

Windows

Chrome 78.0.3904.108 (stable) and 80.0.3977.0 (canary)
Firefox 70.0.1 (stable) and 72.0a1 (nightly)
Classic Edge

Android

Firefox 68.3a1 (nightly)

OK

Android Chromium based browsers

Another case:

Visit https://openui5.hana.ondemand.com/#docs/guide/8b325518a2ae48ee91eb2dacda1d1420.html
which redirects the user to https://openui5.hana.ondemand.com/#/topic/8b325518a2ae48ee91eb2dacda1d1420 (In the future, without /#).
Try to nav back.

Hello @boghyon ,
I've managed to reproduce it on Safari.
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards,
Tsanislav

Reopening until internal ticket<PHONE_NUMBER> is processed.

Reopened until<PHONE_NUMBER> has been processed.


Why limiting keys to 100 chars ?

I need to store url as keys. 100 chars is not enough and artificially limiting (most sql indexes are fine with huge keys). Is this some kind of lowest common denominator ?

It's hard to change cause you would have to migrate all existing running instances. And yes the idea is also to have it as the lowest common denominator. Maybe you want to consider hashing the urls. Whats your exact use case?

Nothing in particular, but yes hashing is a workaround.
Maybe ueberDB could do that hash as soon as keys are > 100 in length.

How would hashing work?  Seems to make sense to me but it could confuse Admins..  Re why its a database AFAIK redis feature...
On 19 Feb 2015 08:02, =?ISO-8859-1?Q?J=E9r=E9my_Lal?=<EMAIL_ADDRESS>wrote:
Nothing in particular, but yes hashing is a workaround.
Maybe ueberDB could do that hash as soon as keys are > 100 in length.
—
Reply to this email directly or view it on GitHubhttps://github.com/Pita/ueberDB/issues/73#issuecomment-75011919.


[SPARK-43944][SQL][CONNECT][PYTHON][FOLLOW-UP] Make startswith & endswith support binary type

What changes were proposed in this pull request?
Make startswith, endswith support binary type:
1, in Connect API, startswith & endswith actually already support binary type;
2, in vanilla API, support binary type via call_udf
Why are the changes needed?
for parity
Does this PR introduce any user-facing change?
yes
How was this patch tested?
added ut

cc @beliefer @panbingkun @HyukjinKwon

Do we need to update PlanGenerationTestSuite?
https://github.com/apache/spark/blob/f3db20c17dfdc1cb5daa42c154afa732e5e3800b/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/PlanGenerationTestSuite.scala#L2461-L2467

I don't think so, since the plan is unrelated to the input data types

Do we need to update PlanGenerationTestSuite?
https://github.com/apache/spark/blob/f3db20c17dfdc1cb5daa42c154afa732e5e3800b/connector/connect/client/jvm/src/test/scala/org/apache/spark/sql/PlanGenerationTestSuite.scala#L2461-L2467

https://github.com/apache/spark/pull/41659#issuecomment-1597862195
I don't think so, since the plan is unrelated to the input data types

is there a need to also update the example at https://github.com/apache/spark/blob/master/sql/core/src/test/resources/sql-functions/sql-expression-schema.md to indicate binary type is supported?

is there a need to also update the example at https://github.com/apache/spark/blob/master/sql/core/src/test/resources/sql-functions/sql-expression-schema.md to indicate binary type is supported?

I think it's not needed, since these PRs don't touch SQL syntax at all.

LGTM except one comment @panbingkun left.

I think @panbingkun's comments have already been addressed.

thank you all, merged to master


Adding the new core integration

Change-Id: Ib7bda09337e9a3e5fb7d649311b4308932ca7963

Continue fixing all the tests and improving the sever launcher for the integration tests

Add also tests for the mutiversion stuff

Coverage decreased (-20.8%) to 56.824% when pulling 6c6213fb8c7c31186f2278c10730af67811bd048 on rafinskipg:newcore into 65503795ab23b65da5d1c13e69bb4d0579c17c06 on corbel-platform:newcore.

Coverage decreased (-20.9%) to 56.774% when pulling bcbfd8c5c2f145139409df99871be8cbf3c104ef on rafinskipg:newcore into 65503795ab23b65da5d1c13e69bb4d0579c17c06 on corbel-platform:newcore.

Coverage decreased (-22.9%) to 54.813% when pulling 6a6219bb9b3af51f6ac78ef302653ba4e9cf608c on rafinskipg:newcore into 65503795ab23b65da5d1c13e69bb4d0579c17c06 on corbel-platform:newcore.

Coverage decreased (-21.1%) to 56.557% when pulling a63f744acb8521727b3ff12522e437471c9eae47 on rafinskipg:newcore into 65503795ab23b65da5d1c13e69bb4d0579c17c06 on corbel-platform:newcore.

Coverage decreased (-18.007%) to 59.656% when pulling 0178af0f24ebac6c0bebe2bcbfaaa28bde794736 on rafinskipg:newcore into 65503795ab23b65da5d1c13e69bb4d0579c17c06 on corbel-platform:newcore.


[zh] Update labels-annotations-taints related reference pages

This PR moves /zh/docs/reference/labels-annotations-taints.md to /zh/docs/reference/labels-annotations-taints/_index.md to match the English page, updates it and translates /en/docs/reference/labels-annotations-taints/audit-annotations.md to Chinese.

/approve

/lgtm


ignore directories by default

Re: https://github.com/datproject/dat/issues/741#issuecomment-300489805. We don't need directories in metadata for anything cuz hyperdrive is sweet.
Directories can pollute metadata and can be confusing if mtime changes without a file inside the dat changing (e.g. if it is ignored).
PR adds opts.ignoreDirs to import options, default true.

Codecov Report

Merging #142 into master will decrease coverage by 0.96%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #142      +/-   ##
==========================================
- Coverage   89.39%   88.43%   -0.97%
==========================================
  Files           6        6
  Lines         264      268       +4
==========================================
+ Hits          236      237       +1
- Misses         28       31       +3

Impacted Files
Coverage Δ

lib/import-files.js
80% <100%> (+1.73%)
:arrow_up:

lib/stats.js
81.48% <0%> (-5.56%)
:arrow_down:

Continue to review full report at Codecov.

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


isCrawler() doesnt detect Crawler

Describe the bug
I use this lib on Google Cloud Functions and somehow it doesn't detect any crawlers when I execute isCrawler().
I am not sure if I am doing something wrong but I followed the code in the docs.
At the end I used this:
if (CrawlerDetector.isCrawler(req.get("User-Agent"))) {
        console.log("Crawler detected!")
}

But without specifying the string, it didn't work.

Can you share the User Agent that the module is failing to detected?

I used the Google bot from Chrome Dev Tools.
When I pasted the user agent string directly into the function, it worked. But without specifying the string, it didnt.

This could be a problem with your framework or Node.js have deprecated something in a newer version, with that said please share more details.

Node version?
Frameworks
Print the Http headers array and share it in a comment

I am using Google Cloud Functions with Node version 14.
When it get's executed by the browser I get this request on the server::
>  {
>    'x-forwarded-host': 'localhost:5000',
>    'x-original-url': '/d/sdeqwd',
>    pragma: 'no-cache',
>    'cache-control': 'no-cache',
>    cookie: 'ajs_anonymous_id=...."',
>    connection: 'keep-alive',
>    'upgrade-insecure-requests': '1',
>    'user-agent': 'Mozilla/5.0 (compatible; Googlebot/2.1; +http://www.google.com/bot.html)',
>    accept: 'text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9',
>    'sec-fetch-site': 'none',
>    'sec-fetch-mode': 'navigate',
>    'sec-fetch-user': '?1',
>    'sec-fetch-dest': 'document',
>    'accept-encoding': 'gzip, deflate, br',
>    'accept-language': 'en-US,en;q=0.9,fr;q=0.8,en-GB;q=0.7,nl;q=0.6,de;q=0.5',
>    'x-client-version': 'FirebaseCLI/undefined',
>    host: 'localhost:5001'
>  }

Looks like googlebot isn't in the list of accepted crawlers? I opened a PR heere: https://github.com/JefferyHus/es6-crawler-detect/pull/37

The thing is, when I pass the string directly into the function call. It works !

@gptt916 google bot is already there. The issue could be something else than the regex. As long as passing the string manually works then probably the http headers are not parsed correctly or the headers are not present in runtime.

This is related https://github.com/JefferyHus/es6-crawler-detect/issues/34

@ngmiduc can you test again? I have updated the database with more bots including google bots

This is fixed by https://github.com/JefferyHus/es6-crawler-detect/pull/41


Fail to publish with pnpm

Affected Packages
https://github.com/unional/clibuilder
Problem
Fail to publish, both from changeset action and locally.
> changeset publish

🦋  info npm info clibuilder
🦋  info clibuilder is being published because our local version (8.0.2) has not been published on npm
🦋  info Publishing "clibuilder" at "8.0.2"
🦋  info This operation requires a one-time password from your authenticator.
🦋  Enter one-time password: · 631531
🦋  error
🦋  error packages failed to publish:
🦋<EMAIL_ADDRESS> ELIFECYCLE  Command failed with exit code 1.

I suspect it has something to do with having some local private packages for testing.
But they are all marked with private: true already.
Example of failure in GH Action:
https://github.com/unional/clibuilder/actions/runs/3026236219
> clibuilder-repo@ release /home/runner/work/clibuilder/clibuilder
> changeset publish

🦋  info npm info clibuilder
🦋  info clibuilder is being published because our local version (8.0.2) has not been published on npm
🦋  info Publishing "clibuilder" at "8.0.2"
🦋  error
🦋  error packages failed to publish:
🦋<EMAIL_ADDRESS> ELIFECYCLE  Command failed with exit code 1.
Error: The process '/home/runner/setup-pnpm/node_modules/.bin/pnpm' failed with exit code 1
    at m._setResult (/home/runner/work/_actions/changesets/action/v1/dist/index.js:102:7[25](https://github.com/unional/clibuilder/runs/8281089873?check_suite_focus=true#step:8:26)8)
    at m.CheckComplete (/home/runner/work/_actions/changesets/action/v1/dist/index.js:102:6686)
    at ChildProcess.<anonymous> (/home/runner/work/_actions/changesets/action/v1/dist/index.js:102:5723)
    at ChildProcess.emit (events.js:314:20)
    at maybeClose (internal/child_process.js:1022:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:[28](https://github.com/unional/clibuilder/runs/8281089873?check_suite_focus=true#step:8:29)7:5)
Error: The process '/home/runner/setup-pnpm/node_modules/.bin/pnpm' failed with exit code 1

Proposed solution
No solution yet. It's working fine in other repos, but not on this one.
Using:

pnpm: 7.11.0
@changesets/cli: 2.24.4

Actually, these issue may be the same as #1003


[Workload] SECURITY test: sbom_available

Title: [Workload] SECURITY test: sbom_available
Is your workload test idea related to a problem? Please describe.

A project should have an SBOM file available in order to check its software bill of materials

Describe the solution you'd like

Check for a spdx or cyclondx file in the root directory of the project

Type of test (static or runtime)

static

Documentation tasks:

[ ] Update installation instructions if needed
[ ] Update Test Categories md if needed
[ ] Update USAGE md if needed

[ ] How to run
[ ] Description and details

[ ] What the best practice is
[ ] Why are we testing this

[ ] Remediation steps if test does not pass

QA tasks
Dev Review:

[ ] walk through A/C
[ ] do you get the expected result?
[ ] if yes,

[ ] move to Needs Peer Review column
[ ] create Pull Request and follow check list
[ ] Assign 1 or more people for peer review

[ ] if no, document what additional tasks will be needed

Peer review:

[ ] walk through A/C
[ ] do you get the expected result?
[ ] if yes,

[ ] move to Reviewer Approved column
[ ] Approve pull request

[ ] if no,

[ ] document what did not go as expected, including error messages and screenshots (if possible)
[ ] Add comment to pull request
[ ] request changes to pull request

@CortezFrazierJr

To verify that an SBOM (Software Bill of Materials) is syntactically correct, you can use open-source projects that focus on SBOM validation and compliance. While the provided sources do not directly mention specific tools for SBOM syntax verification, they do highlight various tools and frameworks related to SBOMs. Here are some open-source projects and tools that can be used for SBOM validation, including syntax verification:

SPDX Tools: The Software Package Data Exchange (SPDX) is a standard for communicating the components, licenses, and copyrights associated with software packages. Tools like spdx-tools can be used to validate SPDX documents, which are commonly used for SBOMs. SPDX provides a specification and tools for generating, validating, and analyzing SBOMs in various formats [1].

CycloneDX Tools: CycloneDX is another SBOM format that provides a comprehensive specification for SBOMs. The cyclonedx-maven-plugin and other related tools can be used to generate and validate CycloneDX SBOMs. These tools ensure that the SBOMs adhere to the CycloneDX specification, which includes syntax verification [1].

Syft: Syft is a tool developed by Anchore that can generate SBOMs from container images. While its primary function is to generate SBOMs, it can also be used to validate the syntax of SBOMs generated in formats like SPDX or CycloneDX by ensuring that the generated SBOMs conform to the expected standards [1].

Tern: Tern is a tool for generating SBOMs for containers. It can be used to verify the syntax of SBOMs by ensuring that the generated SBOMs are correctly formatted and adhere to the standards of the SBOM format being used [1].

SBOM Benchmark: While not a tool for direct syntax verification, the SBOM Benchmark mentioned in the sources can be used to evaluate SBOMs for quality, compliance, and errors. This can indirectly help in verifying the syntax correctness of SBOMs by identifying any issues that might affect their compliance [1].

These tools and projects can be used to verify the syntactical correctness of SBOMs by ensuring they adhere to the standards and specifications of the SBOM format being used. It's important to choose the right tool based on the SBOM format you are working with (e.g., SPDX, CycloneDX) to ensure accurate validation.
Citations:
[1] https://github.com/awesomeSBOM/awesome-sbom
[2] https://fossa.com/blog/software-bill-of-materials-formats-use-cases-tools/
[3] https://github.com/CycloneDX/sbom-utility
[4] https://security.googleblog.com/2022/06/sbom-in-action-finding-vulnerabilities.html
[5] https://codesecure.com/learn/sbom-use-cases-and-why-binary-composition-analysis-matters/
[6] https://www.ntia.gov/sites/default/files/publications/howto_guide_for_sbom_generation_v1_0.pdf
[7] https://www.wiz.io/academy/top-open-source-sbom-tools
[8] https://media.defense.gov/2023/Dec/11/2003355557/-1/-1/0/ESF_SECURING_THE_SOFTWARE_SUPPLY_CHAIN RECOMMENDED PRACTICES FOR MANAGING OPEN SOURCE SOFTWARE AND SOFTWARE BILL OF MATERIALS.PDF
[9] https://anchore.com/sbom/the-software-bill-of-materials-sbom-through-an-open-source-lens/
[10] https://medium.com/@interlynkblog/complying-with-nsa-sbom-recommendations-3274c0f67cab


Windows 11 Insider Preview 10.0.23536.1000 not Supported

termsrv.zip
Thanks！

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/b2dfb2ff2f3ad4a7ccad252ab65ad2134213b095


Azure storage explorer

I ran the example, and could find something in the Azure Storage explorer - under blob containers there is the container I created, but I cannot see the messages sent from my device. If I double-click on the files (named by date, e.g. 01>2018>05>31>09>24, I get a prompt asking me how I want to open this file - if I choose the editor, I see a large number of Chinese symbols (扏Ūᐄ癡潲挮摯捥済汵ᙬ癡潲献档浥큡笄琢灹≥∺敲潣摲Ⱒ渢浡≥∺敍獳条≥∬慮敭灳捡≥∺楍牣獯景⹴穁牵⹥敄楶散≳∬楦汥獤㨢筛渢浡≥∺湅畱) - the documentation might mention what exactly one is supposed to see!?

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

ID: 3b0bc375-145f-3e45-e1e9-1869e183a4a3
Version Independent ID: 8dc4d773-0a92-31de-6ce4-4a118d552f6a
Content: Save your IoT hub messages to Azure data storage
Content Source: articles/iot-hub/iot-hub-store-data-in-azure-table-storage.md
Service: iot-hub
Product: unspecified
GitHub Login: @rangv
Microsoft Alias: rangv

@martinfierz Thank you for your feedback! We have assigned this issue to the content author to review and take the right course of action.

@martinfierz The files are stored in binary Avro format: https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-devguide-endpoints#when-using-azure-storage-containers


[WIP] feat: entire application

I decided to go "fuck it" and wrote entire application without any step-by-step progress. Just because it took me long enough and I need the Midtrans mock server on my current company very soon.

pinging @WahidinAji and @YogiPristiawan for review


[Feature Request] Add Last filter

Add Last filter which is analogous to tail command

Thank you! Well, you know the drill: please write an example program which uses the feature, and we'll get to work!

Last can be used to get the last 10 errors as below
script.Stdin().Match("error").Last(10).Stdout()

That's perfectly fine, but not quite what I had in mind. Why would you want this? If you're working on the command line, reading log files, you presumably have tail or less available. But why would a program want the last X lines of something?

One of the usecases that I had was to perform an analysis on recent logs rather than all the logs present in a log file. As you said we can use tail and pipe the result to the go program , but I feel that having this behaviour in the package itself will offer more flexibility.
One more usecase can be that Last paired with Freq can be used to get the least frequent lines.

Analysing 'recent logs' probably needs something a bit more sophisticated than just the last X lines, doesn't it? We wouldn't want to read the same log lines twice, for example.

One more usecase can be that Last paired with Freq can be used to get the least frequent lines.

Of what, and why? I'm not being purposefully obtuse; I'm trying to dig into what the use case is, which isn't the same as 'an example of using it'. A use case is some real task you need to do.

One of the use cases that I encountered recently in my company requires such analysis. The use case was to log the least used control flow path in a critical function, I had thought of using Freq to find it , looking at its implementation , I found that it is sorted in descending order, so with Last the entire implementation can be contained in one go program , otherwise I would have to use tail the output from the go program.
This is what urged me to create the Issue.
One more use case is that finding the Nth line of input such as
p.First(n).Last(1).String()
This is present as tail and head in a shell script that I wanted to port to go using scripts
But as the current implementation did not have Last ( at least not a straightforward one ) , I could not port it successfully.
I cannot go more indepth of its needs as the use cases that I have presented are too specific to my organisation, but I assume that such use cases are present for majority of the organisations.

Oh, that's very interesting! Yes, I hadn't thought of using it with Freq() to get the least frequent lines. Good idea.


Add Cell Count - HumanDevoKidneyAndOrganoids

Make sure Cell Count is both in Ingest and Staging area
Add cell count and export
https://contribute.data.humancellatlas.org/projects/detail?uuid=dd7f2436-0c56-4709-bd17-e526bba4cc15
Acceptance criteria for the task:
Cell count is present in ingest and the staging area

[ ] Ingest
[ ] Staging area

Closing as UCSC is already looking into it


librealsense2-dkms 1.3.22-0ubuntu1: librealsense2-dkms kernel module failed to build - 5.15 iotg kernel

Error Output:
apt-get install librealsense2-dkms
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following packages were automatically installed and are no longer required:
  linux-headers-5.15.0-1026-intel-iotg linux-image-5.15.0-1026-intel-iotg linux-intel-iotg-headers-5.15.0-1026 linux-modules-5.15.0-1026-intel-iotg
  linux-modules-extra-5.15.0-1026-intel-iotg
Use 'sudo apt autoremove' to remove them.
The following NEW packages will be installed:
  librealsense2-dkms
0 upgraded, 1 newly installed, 0 to remove and 14 not upgraded.
Need to get 0 B/4.445 kB of archives.
After this operation, 23,7 MB of additional disk space will be used.
Selecting previously unselected package librealsense2-dkms.
(Reading database ... 266035 files and directories currently installed.)
Preparing to unpack .../librealsense2-dkms_1.3.22-0ubuntu1_all.deb ...
Unpacking librealsense2-dkms (1.3.22-0ubuntu1) ...
Setting up librealsense2-dkms (1.3.22-0ubuntu1) ...
Loading new librealsense2-dkms-1.3.22 DKMS files...
Building for 5.15.0-1038-intel-iotg
Building initial module for 5.15.0-1038-intel-iotg
ERROR: Cannot create report: [Errno 17] File exists: '/var/crash/librealsense2-dkms.0.crash'
Error! Application of patch 92-realsense-hid-5.15.patch failed.
Check /var/lib/dkms/librealsense2-dkms/1.3.22/build/ for more information.
dpkg: error processing package librealsense2-dkms (--configure):
 installed librealsense2-dkms package post-installation script subprocess returned error exit status 6
Errors were encountered while processing:
 librealsense2-dkms
E: Sub-process /usr/bin/dpkg returned an error code (1)

File: librealsense2-dkms.0.crash
cat /var/crash/librealsense2-dkms.0.crash
ProblemType: Package
DKMSKernelVersion: 5.15.0-1038-intel-iotg
Date: Wed Aug 30 19:28:57 2023
Package: librealsense2-dkms 1.3.22-0ubuntu1
PackageVersion: 1.3.22-0ubuntu1
SourcePackage: librealsense2
Title: librealsense2-dkms 1.3.22-0ubuntu1: librealsense2-dkms kernel module failed to build

OS file - os-release:
PRETTY_NAME="Ubuntu 22.04.3 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.3 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"

Kernel Version:
uname -a
Linux my-machine 5.15.0-1038-intel-iotg #43-Ubuntu SMP Wed Aug 16 21:04:08 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

OS: Ubuntu IOT Desktop 22.04 LTS
Camera: Intel Corp. Intel(R) RealSense(TM) Depth Camera 435i

Hi @ionutnechita-intel  As far as I am aware, Ubuntu IOT Desktop has not been previously reported on this support forum to have been used with RealSense cameras.
Do you have the option of building the librealsense SDK from source code instead of installing with packages, please?  If the SDK is built from source code with CMake and the flag -DFORCE_RSUSB_BACKEND=TRUE is included in the CMake build instruction then the SDK will not be dependent on Linux versions or kernel versions and not require a kernel patch.  This makes an RSUSB build of the SDK good for more unusual hardware or OS specifications than the typical consumer PC.

Hi @ionutnechita-intel  Do you require further assistance with this case, please?  Thanks!

Hi @MartyG-RealSense ,
For the installation from the sources, I will try to do all the installation steps myself.
But for Ubuntu IOT, can we have an updated patch for this type of kernel in the next version of librealsense?

I will ask one of my Intel RealSense colleagues who specializes in kernels.
@dmipx  Could you provide advice about the comment above please regarding kernel 5.15 support for Ubuntu IOT?

Hi @ionutnechita-intel
Recently we submitted bugfix for hid and latest kernels doesn't need hid patch.
We have fix for it but new dkms was not released yet.
You can build modules manually, from 'development' branch by running
./scripts/patch-realsense-ubuntu-lts-hwe.sh
https://github.com/IntelRealSense/librealsense/blob/development/scripts/patch-realsense-ubuntu-lts-hwe.sh
If you have internal access to artifactory, you can use this link to get updated dkms for jammy

When will we have the new version, 1.3.24?

Hi @dmipx  Could you assist @ionutnechita-intel with the question in the above comment please?  Thanks!

Hi. I'm checking with team when DKMS will pe promoted.

We promoted DKMS 1.3.24, you can try now.

We promoted DKMS 1.3.24, you can try now.

Ok. Will try to testing with this version.

Hi @dmipx,
I tested today and this version working correctly on Ubuntu 22.04 IOT. Kernel and modules match now.
// ## modinfo videodev
filename:       /lib/modules/5.15.0-1042-intel-iotg/updates/dkms/videodev.ko
version:        realsense-1.3.24
alias:          char-major-81-*
license:        GPL
description:    Video4Linux2 core driver
author:         Alan Cox, Mauro Carvalho Chehab<EMAIL_ADDRESS>Bill Dirks, Justin Schoeman, Gerd Knorr
srcversion:     9A4FB8163941EA975F2CF8A
depends:        mc
retpoline:      Y
name:           videodev
vermagic:       5.15.0-1042-intel-iotg SMP mod_unload modversions
sig_id:         PKCS#7
signer:         glaic3n170 Secure Boot Module Signature key
sig_key:        3D:B9:3E:D6:3E:87:9C:15:88:C4:79:0C:4E:9B:A8:CA:85:09:F6:0D
sig_hashalgo:   sha512
signature:      B3:92:74:60:F9:46:3E:D7:1C:9E:C4:76:9D:B9:6A:27:F7:4C:55:31:
F4:B0:CF:83:40:0E:A5:7B:AE:3C:8C:48:3B:AE:8D:B9:D9:81:42:26:
5E:6B:B8:28:D1:5B:EC:4D:21:08:87:C8:F5:E7:06:09:26:CB:99:7D:
0F:EA:9B:C3:A8:1E:15:40:18:8C:1F:DD:23:B8:C2:81:B1:D2:54:DF:
FB:2E:A6:B0:E3:D6:8B:C0:1C:A6:A8:27:8B:17:A0:50:51:FE:30:A1:
29:7A:FE:08:ED:75:B2:5C:7D:08:F0:15:A7:47:C6:6F:F8:42:C4:B0:
13:CE:BC:03:C3:AB:F4:0E:E1:78:10:F1:B5:4C:18:9B:C0:9E:5A:9F:
9C:34:B0:34:C0:9C:62:F2:70:A5:50:9D:8B:F8:01:2F:F1:97:73:6A:
DD:86:98:1F:99:C9:84:C0:86:4E:78:B0:97:C9:E7:99:7D:6C:EB:9C:
ED:57:68:9F:DB:20:7A:F2:75:C5:58:C6:F9:50:39:4B:01:05:CE:1E:
1E:B4:D4:FA:76:27:72:6C:F6:03:90:33:3C:77:B8:8E:47:13:82:B3:
C3:D6:78:09:30:FB:AA:17:4A:C1:22:D5:ED:76:60:1C:A3:79:7D:24:
84:7A:21:1A:81:12:F2:5E:19:35:9D:F7:44:21:72:8B
Thank you for all support.
Ticket is verified.
Please close this ticket.

Hi @ionutnechita-intel  It's great to hear that you achieved a solution.  As you have done so, i will close this ticket as you suggested.  Thanks again!


color lv 4 syntax feature detection

Hello, it would be possible to target browsers that support the color 4 syntaxes?
Example:
Old syntax hsla(270, 60%, 50%, .15)
New syntax hsl(270 60% 50% / 15%)

I mislinked the can i use link. I was not talking about 8-digit values, but the lv 4 color notation with spaces and percentage opacity, like the example above.
I can't find that feature in browserlist


Change the type of id in ranked_outlier/saved_outlier queries to StringNumber

fix: #83

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

src/graphql/outlier.rs
2.33% <ø> (ø)

:loudspeaker: Thoughts on this report? Let us know!


EVAD

EVAD Formación

@FranJEstrada First stage of verification passed, official domain is evadformacion.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@FranJEstrada Pull request merged. Thank you!


Add Python type annotations to lab1 api, codec, and crypto

This PR adds type annotations to the api, codec, and crypto files of lab1. These annotations make it much easier to understand the code and determine how to use the functions it provides, at least for me.
I tried my best to determine the types by reading through the code, but there may be a couple errors. I created this to be useful to myself on this lab and as a means of reading through the code, but it would be great to have these type annotations incorporated into future labs.

Thanks for the PR.
We thought about adding type annotations initially, but we decided against it as we weren't sure if students would be familiar with them. Perhaps we should revisit that choice if on balance, the annotations help with clarity/reduces type-related bugs (which are very easy to accidentally introduce in Python).
In either case, we probably won't merge this before lab 1 is due.

Thanks for the response. Good to hear you thought about it, and indeed it might be confusing for some students, that's true. I personally find them super helpful especially in a codebase like this where most arguments are class instances, so I personally think they would be a great thing to introduce for future labs and future iterations of the course. I also feel like they're fairly simple/intuitive to learn even for people who haven't seen them. But I definitely understand the hesitation.

Actually, after looking at the code for lab2, I think that there is a very sore need for type annotations in this codebase, or at the very least expected types in docstrings of all the functions (in client.py and public functions in other modules). Trying to understand verify_sign, for example, I have no idea what the datatype of pk should be until I jump through like 5 levels of definitions. I think it would save a lot of time and bring the focus of the labs onto implementation rather than requiring so much time to understand the structure of this codebase.


Changing url does not work with Jest

I'm trying to change the window url in my Jest specs and it throws an error. Not sure if this is an issue with the way that Jest has configured jsdom, or a problem with jsdom. A small example:
import jsdom from 'jsdom';

it('works', () => {
  jsdom.changeURL(window, 'https://example.com?foo=butts')
});

I run it like:
$ jest --config config/jest.json spec/js/my_test.js

And get an error:
    TypeError: Cannot set property '_URL' of undefined

      at Object.<anonymous>.exports.changeURL (node_modules/jsdom/lib/jsdom.js:64:12)
      at Object.<anonymous> (spec/js/my_spec.js:6:45)

Not really much in the way of custom configuration on our part when it comes to Jest. Using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Can you produce a repro without using Jest? It's hard for us to debug otherwise.

jsdom.changeURL has to be called on a document.

@Sebmaster can you give me some more context? The README says to pass in window, and when I try to pass in window.document I get the same error. I'll see if I can do a repro without jest.

Oh, sorry, misread the source. A repo without jest would definitely make debugging easier.

I have the same issue.
https://github.com/tmpvar/jsdom/blob/master/lib/jsdom/living//helpers/document-base-url.js#L42
my baseURL.path = 'blank'.
Is there a way to change that ?

@kopax it looks like you may have a different issue. It tries to parse the url, so if it doesn't have a proper protocol, such as https, it will throw an error.

@mockdeep , not sure, I am trying to trigger a page change using react-router within a unit test.
I have the exact same version as you and I have tried your code into my env and it did the same, helpers like parseURLToResultingURLRecord doesn't get called properly like a real browser..
You can also reproduce it like with this in your test:
window.history.pushState('page2', 'Title', '/page2.php');

@kopax but what you're linking to is the step where it parses the url. For my case it does manage to parse the url just fine, it throws an error later when trying to assign it to doc._URL:
https://github.com/tmpvar/jsdom/blob/7ce11776ce161e8d5921a7a183585327400f786b/lib/jsdom.js#L64

@mockdeep you're right, I landed here because you have an issue with the document mock.

Okay, so I think I've got a representative repro for the issue I'm seeing. This raises the same error:
const jsdom = require('jsdom');
const jsdomConfig = {url: 'https://test.io'};

global.document = jsdom.jsdom(undefined, jsdomConfig);

jsdom.changeURL(global.document, 'https://www.example.com');

Based on the jest source, it looks like they set the markup for the dom to undefined. When this happens we get the _URL error. Not sure what to do with that information, though. Shouldn't jsdom still allow setting the url even if there is no markup?

@mockdeep Your issue is that you're passing document instead of window to jsdom.changeURL. Passing document.defaultView works in your case.

@Sebmaster Ha, yeah, I just realized that. Same here, though it still doesn't work with jest even if I use document.defaultView. I'll keep digging.

@mockdeep Have you figured this out? I'm getting the same error.

No, haven't had a chance to narrow it down. We ended up just wrapping the window url stuff in a separate helper file and mocking that with jest.

same issue here (jest, jsdom) and no idea how to fix it TypeError: Cannot set property '_URL' of undefined

This is caused by having more than one version of idlUtils.implSymbol floating around.
I think this has to do with jest running some variant of
jsdom.evalVMScript(sandbox, vm.Script(fs.readFileSync('utils.js')))

in an attempt to create a sandbox environment, but breaks stuff since Symbol('impl') !== Symbol('impl'). I'm not certain, though

I'm also getting TypeError: Cannot set property '_URL' of undefined when trying to set
jsdom.changeURL(window, 'http://localhost'); for example.
Logging out window does show the correct object

Please for anyone experiencing this, produce some minimal (~10 line) example that uses only jsdom APIs (not Jest APIs) so we can debug it and add it to our test suite.

This is my configuration. Hopefully it helps. Hopefully the issue is on my end :)
// setupFile.js (https://github.com/airbnb/enzyme/blob/master/docs/guides/jsdom.md)
const jsdom = require('jsdom').jsdom;

const documentHTML = '<!doctype html>

';
global.document = jsdom(documentHTML);
global.window = document.defaultView;

Object.keys(document.defaultView).forEach((property) => {
  if (typeof global[property] === 'undefined') {
    global[property] = document.defaultView[property];
  }
});

global.navigator = {
  userAgent: 'node.js'
};

// apit.test.js
import { baseUrl } from '../api';
import jsdom from 'jsdom';

describe('[Utility] api', () => {
  describe('baseUrl', () => {
    console.log(window) // logs out window
    jsdom.changeURL(window, 'https://localhost:9001');

    it('should return a relative URL if it\'s not local', () => {
      expect(baseUrl).toEqual('/v1');
    });
  });
});

Can you produce a version which does not add properties to the global object? That isn't a supported configuration. See https://github.com/tmpvar/jsdom/wiki/Don't-stuff-jsdom-globals-onto-the-Node-global

Also, a version which doesn't depend on Mocha.

Or on syntax which doesn't compile in my Node.js.

thanks for the links!

I am also having a similar issue regarding history (initializing a React app w/ react-router; ran via jest):
(node:1831) TypeError: Cannot read property '_history' of null
    at Window.get history [as history] (/Users/jozsi/Dev/wealthor/node_modules/jsdom/lib/jsdom/browser/Window.js:151:79)
    at /Users/jozsi/Dev/wealthor/node_modules/history/lib/BrowserProtocol.js:93:18
    at updateLocation (/Users/jozsi/Dev/wealthor/node_modules/history/lib/BrowserProtocol.js:82:3)
    at replaceLocation (/Users/jozsi/Dev/wealthor/node_modules/history/lib/BrowserProtocol.js:92:10)
    at /Users/jozsi/Dev/wealthor/node_modules/history/lib/createHistory.js:119:15
    at /Users/jozsi/Dev/wealthor/node_modules/history/lib/createHistory.js:90:9
    at next (/Users/jozsi/Dev/wealthor/node_modules/history/lib/AsyncUtils.js:51:7)
    at loopAsync (/Users/jozsi/Dev/wealthor/node_modules/history/lib/AsyncUtils.js:55:3)
    at confirmTransitionTo (/Users/jozsi/Dev/wealthor/node_modules/history/lib/createHistory.js:80:31)
    at transitionTo (/Users/jozsi/Dev/wealthor/node_modules/history/lib/createHistory.js:100:5)

    get history() {
      return idlUtils.wrapperForImpl(idlUtils.implForWrapper(window._document)._history);
    },

I'll try to allocate some time in setting up a test repo, without jest.

It really looks like there are two different idlUtils.implSymbol as @i8-pi said.
I've added a simple logging inside changeURL before this line:
console.warn('are impl symbols equal', idlUtils.implSymbol === Object.getOwnPropertySymbols(window._document)[0]);

run a test that calls changeURL and got
are impl symbols equal false

in console.
Note that Object.getOwnPropertySymbols(window._document) returns only one symbol Symbol(impl) so I compare symbol contained in window._document with one that is used as a key for getting document instance and they don't match.
So it's probably not a jsdom issue and we should find the resolution in Jest repo.

Getting the similar failure under Jest with HashRouter + jsdom
TypeError: Cannot read property '_history' of null
    at Window.get history [as history] (.../node_modules/jest-environment-jsdom/node_modules/jsdom/lib/jsdom/browser/Window.js:151:79)
    at createHashHistory (.../node_modules/history/createHashHistory.js:71:29)
    at new HashRouter (.../node_modules/react-router-dom/HashRouter.js:48:195)
    at ..../node_modules/react-dom/lib/ReactCompositeComponent.js:292:18
    at measureLifeCyclePerf (.../node_modules/react-dom/lib/ReactCompositeComponent.js:73:12)
    at ReactCompositeComponentWrapper._constructComponentWithoutOwner (.../node_modules/react-dom/lib/ReactCompositeComponent.js:291:16)
    at ReactCompositeComponentWrapper._constructComponent (.../node_modules/react-dom/lib/ReactCompositeComponent.js:282:19)
    at ReactCompositeComponentWrapper.mountComponent (.../node_modules/react-dom/lib/ReactCompositeComponent.js:185:21)
    at Object.mountComponent (.../node_modules/react-dom/lib/ReactReconciler.js:43:35)
    at ReactCompositeComponentWrapper.performInitialMount (.../node_modules/react-dom/lib/ReactCompositeComponent.js:368:34)

Getting the similar failure under Jest with HashRouter + jsdom
TypeError: Cannot read property '_history' of null
    at Window.get history [as history] (.../node_modules/jest-environment-jsdom/node_modules/jsdom/lib/jsdom/browser/Window.js:151:79)
    at createHashHistory (.../node_modules/history/createHashHistory.js:71:29)
    at new HashRouter (.../node_modules/react-router-dom/HashRouter.js:48:195)
    at ..../node_modules/react-dom/lib/ReactCompositeComponent.js:292:18
    at measureLifeCyclePerf (.../node_modules/react-dom/lib/ReactCompositeComponent.js:73:12)
    at ReactCompositeComponentWrapper._constructComponentWithoutOwner (.../node_modules/react-dom/lib/ReactCompositeComponent.js:291:16)
    at ReactCompositeComponentWrapper._constructComponent (.../node_modules/react-dom/lib/ReactCompositeComponent.js:282:19)
    at ReactCompositeComponentWrapper.mountComponent (.../node_modules/react-dom/lib/ReactCompositeComponent.js:185:21)
    at Object.mountComponent (.../node_modules/react-dom/lib/ReactReconciler.js:43:35)
    at ReactCompositeComponentWrapper.performInitialMount (.../node_modules/react-dom/lib/ReactCompositeComponent.js:368:34)

Same issue with next-router...


[DEEPLINK][2.1.0] Request to bring back withDeepLink()

Hi,
During the migration to V2 of the library, we noticed that you mentioned that withDeeplink was removed.
These APIs were removed on v2. There are new and better ways to achieve the same thing.
In our case, we can't achieve the same without it. Our app has different URI schemas by flavors that only the parent module has visibility and injects on our DI. We were building our own graphs and adding the deeplinks to the destinations over there by using the provided schema from DI. Not important, but we had an extension for it, and it is reusing the route + schema.
Unfortunately, we can't achieve the same just by using the annotation, our feature modules don't have the visibility of the schemas so using a constant is not an option, since the schema is provided via DI.
Extension:
public fun <T> DestinationSpec.withDefaultDeeplink(scheme: DeeplinkScheme): DestinationSpec {
    return withDeepLink { uriPattern =<EMAIL_ADDRESS>}
}

Example of a graph:
 destinations =
                listOf(
                    ChatSupportScreenDestination.withDefaultDeeplink(schema),
                    ChatRateScreenDestination,
                ),

Any possibility of bringing this back? 🙏🏽

Hey 👋
So essentially you need to be able to set deep links at runtime. The problem is not where it is set, since there's ways to override things like deep links when importing destinations or graphs. The problem is that you cannot use annotations coz those can only take constants.
Is that it?

I think that will solve our problem

Just realized that by doing your approach we can start generating instead of building our owns.

So, is that good or bad? I'm not sure I understand what you mean 🤔

It's better. We had to have the custom graphs just to include deeplinks, by using that approach it will be not necessary because the deeplinks will be added to the DestinationsNavHost trailing lambda.

Ahh, yes. With v2 there should be no reason for implementing NavGraphSpec of core library manually.

Went with above approach. It is included in beta07 that is currently building, so probably up in about 30min or so.
That version will also contain your PR changes (thanks for that! 🙏 )


vite引入unocss报错

UnoCSS version
0.60.0
Describe the bug
在vite.config.js引入unocss会报错

Reproduction
在0.60.0版本会报错
但是降到0.56.5时能正常运行
System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

The vite plugin is now ESM only. Change your vite.config.ts to vite.config.mts, or set type: module in your package json.

没理解，按照vite的说法https://cn.vitejs.dev/config/，我是不需要配置type:module吧


[MINOR] Adding a testcase for stream-table join in StreamSQL

This PR only adds a testcase for stream-table join in StreamSQL

[X] Any interfaces changed?
No
[X] Any backward compatibility impacted?
No
[X] Document update required?
No
[X] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.
Yes
[X] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/5537/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/6683/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7103/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5879/

LGTM


The elasticsearch PCP role does not properly set the elasticsearch.conf file on RHEL/CentOS 6,7,8

See the variable definitions:
tasks/main.yml:    path: "{{ __elasticsearch_conf_dir }}"
vars/RedHat_7.yml:__elasticsearch_conf_dir: /var/lib/pcp/pmdas/elasticsearch
vars/RedHat_6.yml:__elasticsearch_conf_dir: /var/lib/pcp/pmdas/elasticsearch
vars/CentOS_7.yml:__elasticsearch_conf_dir: /var/lib/pcp/pmdas/elasticsearch
vars/CentOS_6.yml:__elasticsearch_conf_dir: /var/lib/pcp/pmdas/elasticsearch
vars/RedHat_8.yml:__elasticsearch_conf_dir: /var/lib/pcp/pmdas/elasticsearch
vars/RedHat_9.yml:__elasticsearch_conf_dir: /etc/pcp/elasticsearch
vars/CentOS_8.yml:__elasticsearch_conf_dir: /var/lib/pcp/pmdas/elasticsearch
vars/main.yml:__elasticsearch_conf_dir: /etc/pcp/elasticsearch
vars/main.yml:__elasticsearch_conf: "{{ __elasticsearch_conf_dir }}/elasticsearch.conf"

The template module by default does not follow symlinks, but on RHEL/CentOS 6,7,8 /var/lib/pcp/pmdas/elasticsearch/elasticsearch.conf is a symlink to /etc/pcp/elasticsearch/elasticsearch.conf.  But when the template is generated the symlink is removed and only a file is stored in /var/lib/pcp/pmdas/elasticsearch/elasticsearch.conf.

@nhosoi has kindly fixed this in #32


Mistakingly setting a Label as a pointer if it was previously compressed.

An edge case where parsing a compressed packet, then encoding it again, results in a mistake where the compressed Label is still flagged as a pointer, see the comment after the first assert:
{
        let name = "foobar";
        let dup = name;

        let mut original = Packet::new_reply(0);
        original.answers.push(dns::ResourceRecord::new(
            dns::Name::new("@").unwrap(),
            dns::CLASS::IN,
            30,
            dns::rdata::RData::CNAME(dns::Name::new(name).unwrap().into()),
        ));
        original.answers.push(dns::ResourceRecord::new(
            dns::Name::new("@").unwrap(),
            dns::CLASS::IN,
            30,
            dns::rdata::RData::CNAME(dns::Name::new(dup).unwrap().into()),
        ));

        let compressed =  original.build_bytes_vec_compressed().unwrap();
        let decompressed = Packet::parse(&compressed).unwrap();

        let encoded = decompressed.build_bytes_vec().unwrap();

        assert_eq!(encoded, original.build_bytes_vec().unwrap());
        // Error:    mistakenly set the first 2 bits as if it is a pointer ------+
        //                                                                       |
        // encoded:  [ ...header, ...1st answer, 2nd: qname, type, class, ttl, __2__, 6, "foobar", 0]
        // expected: [ ...header, ...1st answer, 2nd: qname, type, class, ttl, __8__, 6, "foobar", 0]

        let parsed = Packet::parse(&encoded).unwrap(); // Err InsufficientData

        assert_eq!(parsed.build_bytes_vec().unwrap(), original.build_bytes_vec().unwrap());
 }

Hey @Nuhvi, thanks for reporting this, I will try to take a look during the week

@Nuhvi can you check if #30 fixes the issue?

@balliegojr Yes that seems to do it. Thanks for the quick fix.

@balliegojr do you plan to publish a new version with the fix this week?

@Nuhvi just did, 0.6.1 is available


New test request for unencoded glyph variants

I think I need a new test for unencoded glyph variants.
These would be the failure modes.

Alternate glyph not in font. Perhaps: Does input glyph have variants in GDEF? e.g. is there an Eng and any Eng.xxxx (imperfect since we don't know what Eng.xxxx is, nor do we know if the default design of Eng happens to be the variant we want. This might more a warn than a fail, since it would require someone to review the font.)
If item1 passes: Is target language tag not used in locl feature?, e.g. "DJR" not used in locl feature
If item1 and item 2 passes, Target language is used in locl feature but target glyph is not changed.

It would be good to also be able test item 3 with other OT features like smcp. Perhaps compare cluster 1 with just smcp on against cluster 2 with locl+smcp on.
I think logic would work.

Let me just check I understand what you're asking for:
The profile specifies an input codepoint (e.g. U+014A Ŋ). This maps to "Eng" in the font, and shaperglot looks to see if there are any other "Eng.XXX" unencoded in the font. If there are not, we get a warning.
If there are, we check that they are triggered by a locl feature for the language we are looking for, and that the glyph gets changed.
Actually now I write it out myself, I think that makes sense. Will work on it...

Thanks. That’s exactly it. I think it should be generic since the local Eng form largely only applies to pan-African Latin. The is one case where I will be checking a Bhook form, so ideally I can add this check to specific profiles as needed. The other element to this is that not all languages have corresponding OT language tags so I would only be adding the check to those that do.   From: Simon CozensSent: Monday, November 21, 2022 11:16 AMTo: simoncozens/shaperglotCc: Neil S Patel; AuthorSubject: Re: [simoncozens/shaperglot] New test request for unencoded glyph variants (Issue #8) Let me just check I understand what you're asking for:The profile specifies an input codepoint (e.g. U+014A Ŋ). This maps to "Eng" in the font, and shaperglot looks to see if there are any other "Eng.XXX" unencoded in the font. If there are not, we get a warning.If there are, we check that they are triggered by a locl feature for the language we are looking for, and that the glyph gets changed.Actually now I write it out myself, I think that makes sense. Will work on it...—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
OK, I've done this. For now, you specify one codepoint at a time, and use it like this:
- check: unencoded_variants
  input:
    text: Ŋ

Shaperglot will pass the language code of the current language definition to Harfbuzz as a BCP47 tag, and Harfbuzz will magically map that to an OpenType tag. If you want to override the language tag passed to Harfbuzz, do it like this:
- check: unencoded_variants
  input:
    text: Ŋ
    language: cuc # Usila Chinantec

Thanks, I'll give it a try.


adu_core_interface.h include directive invalid path.

adu_core_interface.h has an include directive that reference a relative path in azure-iot-sdk-c that does not exist.
#include <azureiot/iothub_client_core_common.h> cannot be resolved by GCC without modifying folder structure of azure-iot-sdk-c project.
Details are located over here:
https://stackoverflow.com/questions/71238001/reference-a-virtual-location-as-include-path-in-gcc

I just saw the install-deps.sh script. I am wondering if it's possible to specify manually an install location. I would like to avoid installing libs on a system-level.

At the end of the day, it would require setting CMAKE_INSTALL_PREFIX (default is /usr/local) on all dependency and in DU agent project--it's not exposed as an option in install-deps.sh today
For example, azure-iot-sdk-c has build.sh that uses $cmake_install_prefix to set -DCMAKE_INSTALL_PREFIX in cmake command to generate the build script.
One could try setting azureiotsdkc_make_options to include cmake_install_prefix.  For DO, catch2, and azure_storage_cpplite, one might be able to just set -DCMAKE_INSTALL_PREFIX:PATH=
Tracking this internally now as a possible improvement to the install-deps.sh/build.sh scripts.

@bennydiamond if you're still looking into how to do this locally you can modify install-deps.sh to add cmake options for each of the targets you want to add to a local folder, however I'm not sure how that effects the build process (eg you specify the libs are installed to ~/aduc_libs that needs to be added to your includes and libs path for the build.sh script to work properly).
If you're just looking at installing the source in a separate directory we have a -k command that keeps all of the downloaded code and a -f command that you can specify a working directory to which these sources are downloaded, built, and then installed to the /usr/include/ and /usr/lib directories for the compiler.
I'm closing the issue for right now. Feel free to open a new issue if this persists or there is new information.


Allow use of Config in cjs lambda

It would be great to have the option to use Config.Secret within my lambdas that are stuck in cjs without the option to switch them over to esm (due to downstream modules not being compatible). The top-level await seems to be the main blocker.
I tried a few workarounds, and ended up needing to just put the env variables into my CI/CD platform and using them that way (by passing them into the function environment directly).
Another option that would be helpful:
I thought I could maybe just go fetch them from SSM in the construct like below:
const DATABASE_URL_VALUE = StringParameter.valueFromLookup(
  stack,
  `/sst/${SST_APP}/${SST_STAGE}/secrets/${DATABASE_URL.name}`
);

and it almost works, except the value is encrypted. So if there was some utility to decrypt this, I could just pass the values directly into the lambda function, but then keep them within the config setup so other esm lambdas could still use Config as intended.

Totally need this as well!
The Config handling looks amazing but I can't use it as I'm also stuck in CJS for now.
@fwang Would you be ok to add the feature if I find some time to implement it?


godep save doesn't make _Workspace and vendor folder.

Expected behavior
Generate vendor folder.
Actual behavior
No vendor folder, and no _workspace folder in Godeps. Just Godeps.json and Readme.
Steps to reproduce behavior
godep save ./...
godep version output
godep v75 (darwin/amd64/go1.7.3)
go version output
go version go1.7.3 darwin/amd64
Contents of Godeps.json file
{
"ImportPath": ".",
"GoVersion": "go1.7",
"GodepVersion": "v75",
"Packages": [
"./..."
],
"Deps": []
}

It happened when Godep doens't changed at all. To reproduce this issue,

Remove vendor dir
Run godep save ./...
Execute ls vendor

$ godep version
godep v75 (darwin/amd64/go1.7.3)
$ go version
go version go1.7.3 darwin/amd64

I am closing this issue because the repo is going to be archived.


Proposal: adding a flag for triad chords on Key.chords

Hey @danigb what do you think about adding a second, boolean, parameter to Key.chords to return the triad chords instead of the seventh chords.
I believe that, as an exception, for the case of a major scale, the 5th grade should still be returned as a seventh chord to keep its dominant quality
So something like:
Key.chords('A minor', true) // [ 'Am', 'Bmb5', 'C', 'Dm', 'Em', 'F', 'G' ]
Key.chords('C minor', true) // [ 'C', 'Dm', 'Em', 'F', 'G7', 'Am', 'Bmb5' ]

What do you think? As always, if you like the feature, I can do a PR to include it

Hi @holden-caulfield,
I see what are you trying to accomplish. My main concern is if we should make it more generic, for example, pass the number of notes (3 for triads, 4 for 7th chords, 5 with 9th chords...). Also I'm reluctant to add more parameters to functions (unless strictly necessary) because it makes them more difficult to use with standard array functions or to compose. For example, when applying map like ["Aminor", "C major"].map(Key.chords), the Key.chords function receives a second parameter with the index, that can be interpreted as false for the first element and true for the rest.
What do you think? Can you elaborate the use case? (what do you need this for?)
Anyway, I'm open to that improvement, but I would like to take the right approach.

Hey @danigb thanks a lot for the input.
Let me elaborate on the use case: I'm working on a small tool to aid my music studies (https://holden-caulfield.github.io/empa-trainer/).
The tool will have exercises on ear training and music theory, similar to other tools out there but more tailored to resemble the kind of exams we take at my school (the EMPA, hence the "EMPA Trainer" name)
I've started with interval detection and now I'm going to work with simple chord progressions on major keys (I-IV-V-I.. that kind of thing). So I would need to pick a random key, and then take the chords for the I, IV and V grade. In the school we usually use triad chords except of the V7 dominant, so that's why I wanted to get triad chords.
I agree with you that my initial approach is not ideal and may affect composability. I'm sending some ideas on a separated comment... this already got too long :P

So here are three options that I believe would work:
Option 1: adding a method on Chord that could get a seventh chord and return the corresponding triad. Something like Chord.toTriad so I could do something like
Key.chords('C major').map(Chord.toTriad) // [ 'C', 'Dm', 'Em', 'F', 'G7', 'Am', 'Bmb5' ]

*Option 2: * same thing but more generic, Chord.toNotes that takes a number and a key and returns a function that takes a chord  and gives the corresponding chord triad/7th/9th/etc chord on that key. Unlike option 1, you need to pass the key, because you may be adding notes instead of just removing.
Key.chords('C major').map(Chord.toNotes(3, 'C major')) // [ 'C', 'Dm', 'Em', 'F', 'G7', 'Am', 'Bmb5' ]

This adds flexibility but I don't like the fact that we are having to repeat the 'C major' parameter
Option 3
Add a notes parameter on Key.chords but make it the first parameter instead of the last, and allow it to be currified. 4 would be a default. So:
Key.chords(3, 'C major') // [ 'C', 'Dm', 'Em', 'F', 'G7', 'Am', 'Bmb5' ]
OR
Key.chords(3)('C major') // [ 'C', 'Dm', 'Em', 'F', 'G7', 'Am', 'Bmb5' ]

And your example could be addressed by
["Aminor", "C major"].map(Key.chords())
OR
["Aminor", "C major"].map(Key.chords(4))

I think this is the one I like the most for composability. But unfortunately is a breaking change for backwards compatibility
What do you think? Any of this sounds like a good plan?

Hi @holden-caulfield thanks a lot for elaborating this!
And I'm not sure about the option. Also, I have the feeling that the natural option for Key.chords would be to return the simple triads (with the dominant seventh chord). But that's a breaking change too...
Let give a little more time to think more about this... meanwhile, maybe you can use a simple function to substitute "Maj7" to "" and "m7" to "m", and map the result using that function.

Just my two cents; I feel like returning a single dominant seventh chord in a list of diatonic triads from is not very intuitive and could cause issues for people.
-Garrett

Thanks for your feedback @grimmdude 👍

@grimmdude I agree. Whatever the solution we turn out creating, I was already expecting to handle the "5th dominant" case as an exception on userland (i.e. my own code), with something like
Key.chords('C major').map((chord, index) => index !== 4 Chord.toTriad(chord) ? chord) // [ 'C', 'Dm', 'Em', 'F', 'G7', 'Am', 'Bmb5' ]

@danigb sure, lets take it easy, no rush for me to get this changes in. In general what I'm doing is I'm isolating this kind of things on a "lib" module built on top of tonaljs on my project and whenever something looks like it could be useful for the broader community I propose to add it the actual library. That makes it easier for me to do the PRs anyways :P

Just noticed that the issue I created (https://github.com/danigb/tonal/issues/61) is a duplicate of this one. @danigb any update on the proposal? Would love to help out if possible!

Hi, guys. Any updates on the issue?

Hi @danigb
Thanks a lot for the follow up. I just tested your suggestion:
const triadOf7th = name => name.slice(0, name.length -1);
Key.chords("C major").map(triadOf7th)

=> ["CMaj", "Dm", "Em", "FMaj", "G", "Am", "Bm7b"]

While this is closer to the expected result, I would intuitively still think that the result should be C, Dm, Em, F, G, Am, Bdim.
Thanks for all your work.

Yes, I've realized yesterday that my proposal has that problem. If we implement the triadOf7th method, it should handle that case.
Anyway, I'd prefer Bb5 over Bdim because Bdim can be confused with a full diminished chord (1P, 3m, 5d, 7d). What do you think?

I don't think my understanding of music theory goes far enough to adequately judge whether Bb5 or Bdim is preferably. If Bb5 avoids confusion then let's go with that 👍

As I understand, the naming of Bdim vs. Bb5 would be contextual.  You wouldn't get agreement on any one single name for these types of things because you name things according to their function.

Great! Thanks both for the responses. Yes @moxious, probably is impossible to differenciate without context. But I'd go with the b5 form if no one sees an impediment.
I'm almost decided to implement the triadOf7th function (instead of changing the chords function). I'm inclined to call it Chord.triadName(chordName)... thoughts?

I'm almost decided to implement the triadOf7th function (instead of changing the chords function). I'm inclined to call it Chord.triadName(chordName)... thoughts?

Makes sense to me!

Hi all,
Almost 1 year after 😱 (sorry to everybody!) I've decided to implement Key.triads function. It's exactly just the same as Key.chords but it returns triads instead of 7 chords (with the exception of the dominant):
 Key.triads("A major") // => ["AM", "Bm", "C#m", "DM", "E7", "F#m", "G#mb5"]

I've opted to use "Xmb5" to denote diminished triads.
It's a proposal (via PR). Not yet merged. Adding another function is IMO the simplest and easy solution, both to use and implement.
Feedback appreciated! 🙏

Ok, next round of commits with the same feature.
First, I've decided (after some research and guided mostly by http://openmusictheory.com/triads.html) to use this:
Key.triads("A major") // => ["A", "Bm", "C#m", "D", "E", "F#m", "G#dim"]

But I created a new function Key.leadsheetSymbols that accepts an array of symbols and create a function like triads or chords:
const chords = Key.leadsheetSymbols(["M", "m", "m", "M", "M7", "m", "dim"])
chords("D dorian") //=> ["Dm", "Em", "FM", "GM7", "Am", "Bdim", "CM"]

In this way, you can use the notation you like most.
My only doubt now is the name of the Key.leadsheetSymbols. I think is not very intuitive...
What do you think?

Man, thank you very much for your efforts in making this requirement
happen. I think it would be clearer if the name was standardTriadNotation.
Em ter, 23 de out de 2018 às 15:45, danigb<EMAIL_ADDRESS>escreveu:

Ok, next round of commits about the same feature:
First, I've decided (after some research, and guided mostly by
http://openmusictheory.com/triads.html) to use this notation:
Key.triads("A major") // => ["A", "Bm", "C#m", "D", "E", "F#m", "G#dim"]
But I created a new function Key.leadsheetSymbols that accepts an array
of symbols (strings) and returns a function like triads or chords:
const chords = Key.leadsheetSymbols(["M", "m", "m", "M", "M7", "m", "dim"])chords("D dorian") //=> ["Dm", "Em", "FM", "GM7", "Am", "Bdim", "CM"]
In this way, you can use the notation you like most.
My only doubt now is the name of the Key.leadsheetSymbols. I think is not
very intuitive...
What do you think?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/danigb/tonal/issues/48#issuecomment-432371141, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ASiaKLivGffBtdY-lB5j8H31X5FiSr17ks5un2PKgaJpZM4QNad3
.

--
Att: Ricardo de Paula Soares
(94) 9114-3270
(94) 8149-1727

Hi @rickdps,
You mean rename triads to standardTriadNotation? Or maybe rename leadsheetSymbols into standardTriadNotation?
If it'ss the second, then you probably didn't understand properly what leadsheetSymbols do. It receives a list of symbols and returns a function. Not related to triads (at least, directly)...
Maybe it's me that I didn't understand you :-D

Hey @danigb,
Sorry for that. I completely misunderstood the usage of leadersheetSymbols. Now I got it and to me it`s name sounds fine.
Once again, thanks for the excellent work.

@danigb Thanks so much for your work, highly appreciated! Would you mind creating a new release for tonal-key so that the feature becomes available via npm? Thanks again!

Thanks @Maarius 🙏 ! Package released: https://www.npmjs.com/package/tonal-key 👍

@danigb great, thanks! Just tested in my app and it looks like it uses the implementation from https://github.com/danigb/tonal/issues/48#issuecomment-432058172. So for me I with package 2.1.0 I get
Key.triads("A major") // => ["AM", "Bm", "C#m", "DM", "E7", "F#m", "G#mb5"]

I thought you wanted to use the notation from https://github.com/danigb/tonal/issues/48#issuecomment-432371141?
Key.triads("A major") // => ["A", "Bm", "C#m", "D", "E", "F#m", "G#dim"]

I'd definitely prefer the latter 🙂

Yes, I've finally decided to use that one... but I added another function to allow other alternatives. It work like this:
const chords = Key.leadsheetSymbols(["", "m", "m", "", "", "m", "dim"])
chords("A major") // => ["A", "Bm", "C#m", "D", "E", "F#m", "G#dim"]

Hope you like it ;-)

Yes, I've just tested and you're right. I probably published without building before. Fixed at v2.1.1: https://www.npmjs.com/package/tonal-key

Perfect, now after updating to 2.1.1 Key.leadsheetSymbols() works as expected and Key.triads() returns the values from https://github.com/danigb/tonal/issues/48#issuecomment-432371141, e.g.
Key.triads("A major") // => ["A", "Bm", "C#m", "D", "E", "F#m", "G#dim"]

Thank you so much for creating this great library and helping to debug my issues.


Multisig in Freewallet

With documentation on counterparty.io showing support for multisig addresses on Counterwallet, is it ever planned to be implemented in the future? Is this function being asked by users at all to be integrated into Freewallet?

Please move to discussions section as an idea.

Currently not much demand for multisig address support at this time


.Net: Adds a memory connector for Azure Cosmos DB for NoSQL

Motivation and Context
Azure Cosmos DB is adding Vector Similarity APIs to the NoSQL project, and would like Semantic Kernel users to be able to leverage them.
Description
This adds a Memory Connector implementation for Azure Cosmos DB's, including support for the new vector search functionality coming soon in Cosmos DB. It is mostly based off the existing connector for Azure Cosmos DB for Mongo DB vCore.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

I believe this is ready to go now @westey-m, @markwallace-microsoft, @stephentoub.

Thanks so much everyone!


:sparkles: Migrate to v4

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] PR title follows the guidelines defined in  https://github.com/ossf/scorecard/blob/main/CONTRIBUTING.md#pr-process

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?
v4 release

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Thanks! I have disabled auto-merge for now though. This needs to be submitted after we have cut the release right?

Yes! after we have the v4 tag. Making any changes I will have to redo this again which is painful.

When are we planning to do the v4 tag?

When are we planning to do the v4 tag?

Mostly mid-Jan. @laurentsimon to confirm.

When are we planning to do the v4 tag?

Mostly mid-Jan. @laurentsimon to confirm.

we don't need to wait for the v4 tag. In fact, if we tag v4, the release will be broken because the tags will say v4 but the code will say v3. This is the problem that happened last time, right?
The HEAD code can be anything and does not affect users of the latest v3.x.y.
So we can merge this now.

if that

When are we planning to do the v4 tag?

Mostly mid-Jan. @laurentsimon to confirm.

we don't need to wait for the v4 tag. In fact, if we tag v4, the release will be broken because the tags will say v4 but the code will say v3. This is the problem that happened last time, right?
The HEAD code can be anything and does not affect users of the latest v3.x.y.
So we can merge this now.

If that is the case then we would have to disable the goreleaser check as it will look for the v3 at this branch.

if that

When are we planning to do the v4 tag?

Mostly mid-Jan. @laurentsimon to confirm.

we don't need to wait for the v4 tag. In fact, if we tag v4, the release will be broken because the tags will say v4 but the code will say v3. This is the problem that happened last time, right?
The HEAD code can be anything and does not affect users of the latest v3.x.y.
So we can merge this now.

If that is the case then we would have to disable the goreleaser check as it will look for the v3 at this branch.

what is this check? Validaion check?

if that

When are we planning to do the v4 tag?

Mostly mid-Jan. @laurentsimon to confirm.

we don't need to wait for the v4 tag. In fact, if we tag v4, the release will be broken because the tags will say v4 but the code will say v3. This is the problem that happened last time, right?
The HEAD code can be anything and does not affect users of the latest v3.x.y.
So we can merge this now.

If that is the case then we would have to disable the goreleaser check as it will look for the v3 at this branch.

what is this check? Validation check?

build-releaser

If that is the case then we would have to disable the goreleaser check as it will look for the v3 at this branch.

what is this check? Validation check?

build-releaser

this file https://github.com/ossf/scorecard/blob/main/.github/workflows/goreleaser.yaml?

https://github.com/ossf/scorecard/blob/192ae4d18fbaaeb7d2510b1116c26c817825e210/Makefile#L142

I am going to remove this as part of this PR and then later add it back.
@azeemshaikh38 / @laurentsimon Thoughts?

I am going to remove this as part of this PR and then later add it back.
@azeemshaikh38 / @laurentsimon Thoughts?

SGTM

I am going to remove this as part of this PR and then later add it back.
@azeemshaikh38 / @laurentsimon Thoughts?

SGTM

Thanks!

@azeemshaikh38 @laurentsimon any idea as to why is this build failing? It is not failing locally for me.

I am going to remove this as part of this PR and then later add it back.
@azeemshaikh38 / @laurentsimon Thoughts?

SGTM

Same here, sgtm. Thanks @naveensrinivasan for the explanation

<EMAIL_ADDRESS>not present in go.sum. Not sure why it's suddenly needed. Is your go.sum different from the committed to the repo?

<EMAIL_ADDRESS>not present in go.sum. Not sure why it's suddenly needed. Is your go.sum different from the committed to the repo?

No, it isn't. Can you try it locally and let me know if you are seeing any difference? Or are you able to build it?

<EMAIL_ADDRESS>not present in go.sum. Not sure why it's suddenly needed. Is your go.sum different from the committed to the repo?

No, it isn't. Can you try it locally and let me know if you are seeing any difference? Or are you able to build it?

make build-scorecard worked. I use $go version: go version go1.17 linux/amd64

@naveensrinivasan any luck on this PR?

@naveensrinivasan any luck on this PR?

Nope!

@naveensrinivasan fyi I'm adding some commits to this PR to get the build to pass.


make reindex-triplestore failing

I was testing some make commands and noticed that the triplestore reindex command is failing with the following:
docker-compose exec -T drupal with-contenv bash -lc 'drush --root /var/www/drupal/web -l ${DRUPAL_DEFAULT_SITE_URL} vbo-exec all_taxonomy_terms emit_term_event --configuration="queue=islandora-indexing-triplestore-index&event=Update"'

  Command "vbo-exec" is not defined.

make: *** [reindex-triplestore] Error 1

I think that's a drush command from the views bulk operations module. It might not be installed/enabled.

Thanks. I just put in a PR to update the docs to reflect this: https://github.com/Islandora/documentation/pull/2239


Modify undo command

Codecov Report

Merging #179 (8f4c327) into master (ee0d6ef) will decrease coverage by 5.47%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #179      +/-   ##
============================================
- Coverage     51.28%   45.81%   -5.48%
- Complexity      467      470       +3
============================================
  Files           106      115       +9
  Lines          2135     2399     +264
  Branches        245      270      +25
============================================
+ Hits           1095     1099       +4
- Misses          977     1234     +257
- Partials         63       66       +3

Impacted Files
Coverage Δ

...va/seedu/address/logic/commands/DeleteCommand.java
25.88% <0.00%> (-0.31%)
:arrow_down:

src/main/java/seedu/address/model/Shortcut.java
52.17% <0.00%> (-27.83%)
:arrow_down:

...a/seedu/address/logic/parser/ArgumentMultimap.java
83.33% <0.00%> (-16.67%)
:arrow_down:

...in/java/seedu/address/commons/util/StringUtil.java
80.00% <0.00%> (-16.00%)
:arrow_down:

...rc/main/java/seedu/address/logic/LogicManager.java
62.96% <0.00%> (-7.88%)
:arrow_down:

...rc/main/java/seedu/address/model/ModelManager.java
52.63% <0.00%> (-7.61%)
:arrow_down:

.../seedu/address/logic/parser/EditCommandParser.java
75.00% <0.00%> (-5.00%)
:arrow_down:

.../seedu/address/logic/parser/AddressBookParser.java
59.25% <0.00%> (-4.75%)
:arrow_down:

...java/seedu/address/logic/commands/UndoCommand.java
4.76% <0.00%> (-0.42%)
:arrow_down:

... and 22 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 ee0d6ef...8f4c327. Read the comment docs.


Feat/678 Perpetual Meta Transactions

Resolves https://sovryn.monday.com/boards/2218344956/pulses/2243577928
For now it is disabled by default.
Navigate to /perpetual and connect your wallet.
If you require balance please contact James Howlett to receive funds.
AC:

When GSN (top right) is enabled, all contract interactions on /perpetual should use the GSN relayer and paymaster. Transactions and Gas should be payed with the Perpetual Token. When disabled transactions should be called as normal.
GSN should automatically disabled, when using hardware wallets.

The buttons on the Perpetuals Account modal will be changed to "BTC Deposit", "BTC Withdraw", "RSK Transfer" respectively. Tooltips may be asked for as well (waiting on decision of vasili)


docs(fix): fix typo

Resolves Jira: DOC-669

Reran the build from within Netlify and passed https://deploy-preview-1589--armory-docs.netlify.app/armory-enterprise/armory-admin/artifacts-github-connect/


Add trade generator

The Python version of the toolbox has a trade generator added; add one for the Ruby version based on Suns of Gold.

This won't be added as Suns of Gold is not a free SWN rulebook


defaultExpanded

Problem Description
I have tried to create a table with raw expanded as default i have tried expandedRows and defaultExpanded but none of them worked (no console error)
Codepen (https://codepen.io/anon/pen/eReewL?editors=0010)
<ReactTable
data={makeData()}
columns={columns}
defaultExpanded={ {0: true, 2: true}  }
expandedRows={ {0: true, 2: true}  }
SubComponent={ ()=>Hello! }
/>

It does appear that defaultExpanded is not working correctly.
But if you're willing to completely take over the expanded rows, if you change expandedRows to expanded it does work. The API changed in 6.0.0 but I do see a reference in the readme still to the old expandedRows api.

I just tried the same and it works with 'expanded' instead of 'expandedRows'.

Indeed, this looks broken.

Hi,It seems that the defaultExpanded  is still not working ,but I have updated my version to 6.5.1

I am facing the same issue. I need a few lines to be expanded by default. when I use defaultExpanded , it's not working at all and when I use 'expanded' property, the rows will be expanded but they won't be collapsed anymore as the user clicks on the row.

What appears to be happening is that the rows start by respecting defaultExpanded, but immediately close on initial render.

This is still happening to me.
using defaultExpanded and pivotBy
expanded does work, but the row cannot be collapsed


adding a new hello WORLD class in java

What does the program do ?
its a hello world program

In what programming language it is written?
Java Language

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Webpack production build

Summary
Updating the script for the webpack production build.
Here is more information about what the production flag does:
https://webpack.js.org/guides/production/#the-automatic-way
Thanks for contributing to Terra.
@cerner/terra

Are you adding a Heroku deployment for terra-ui? I noticed you added a Procfile.

Are you adding a Heroku deployment for terra-ui? I noticed you added a Procfile.

Right now, we are manually pushing to heroku using the cli. Adding the procfile precompiles my assets when pushing to heroku.

https://github.com/cerner/terra-ui.com/issues/13


Prevent reset button from being cropped in world control gui plugin

🦟 Bug fix
Summary
The reset button is cropped in many of the example worlds that have the world control gui plugin specified.

Removing the width property reveals the button, which matches what's done in the default gui.config file
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Would it be better to backport this one #2151?

oh yes, I missed that one. I'll close this and cherry pick instead.


Unable to style the dropdown

For example, if I'm trying to remove the shadow of the dropdown, I see the following way (from dom explorer) to access it to change it:

I tried adding the following to my dropdown.component.css, and even to global style.css, but it didn't change anything, even with !important.
.selected-list .c-btn {
    box-shadow: none !important;
}

// Even tried exact selectors as shown in dom style explorer
.selected-list[_ngcontent-c6]   .c-btn[_ngcontent-c6] {
    box-shadow: none !important;
}

I was not able to make a plunkr demo, but if you are able to create for as a base for people to provide a demo of their issue, that'll be helpful.
Thank you.

Hi, you can overwrite the css by writing your css in style.css(or create and include) file at src level of your angular-cli application. This is the practice that angular suggests to scope css to components and override in case if we want. Hence closing..
.selected-list .c-btn {
box-shadow: none !important;
}
this works..

Thank you for the response.
I started a new project using 'ng new multi-dropdown' (because when I tried to ran this repo locally, it gave an error when running ng serve -- so might want to check into that).
I then installed this component, added the necessary code to html/ts files. And then added the above css to the connected stylesheet. It did not work for me. Can you try?
Attached is the project I made. Download + npm install, then ng serve to run.
multi-dropdown.zip

@ChaudhryAtif   Please take a look at this plunker for styling.. https://plnkr.co/edit/fof4c6PBadEfZFnaWT2G?p=preview


Add default maven goal

Just adding a default goal to avoid error when running ./mvnw

The only problem with this, is the user who wants to build a simple lib (init, maven, java base) without spring boot
The best solution would be:

when adding spring boot, add this default goal if it doesn't already exist

But I'm fine to merge this

Yep, right, we can wait for spring boot migration to module and do that in a postAction then (as replacement won't check if the value is there already)


test: Add wallet connection hook tests

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR introduces a MockStorage class for testing and updates various hooks related to wallet management in the thirdweb library. It enhances test coverage by implementing mock storage functionality and ensuring proper error handling in wallet connection scenarios.
Detailed summary

Added MockStorage class in storage.ts to simulate storage operations.
Updated useSetActiveWallet, useAddConnectedWallet, and useConnect tests to use MockStorage.
Improved error handling and validation for hooks when not used within <ThirdwebProvider>.
Enhanced tests to verify wallet connection and management functionality.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#5689  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


feat: add basic Dockerfile for the api

Description
This PR adds a basic Dockerfile with a build step and a run step.

~200MB saved with NODE_ENV=production :smiley:

This seems nice; However, how are the Prisma migrations handled? Are we assuming that the migrations are run elsewhere?

Optimized ~80MB on the image with moving @nestjs/cli into devDependencies. :bed:
So now we need two npm install one required for the npm build and another for the next stage (so without packages in devDependencies).


Property IndentationControl

The Property IndentationControl seems to have no effect anymore.

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

ID: 2168dc1b-15fd-65db-7869-f0756ca4e1ca
Version Independent ID: 9b9b2bec-8fc0-eed0-04ce-0ae74072c889
Content: IndentationColumn Property - Business Central
Content Source: dev-itpro/developer/properties/devenv-indentationcolumn-property.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: solsen

Hello, Thanks for the feedback!
What version of Business Central are you using? We did some changes in 16 to support the ShowAsTree property. You might have read this, and not sure is is relevant in your case, but the IndentationControl property has the following behavior:

This property is ignored if the ShowAsTree property on the repeater is set to true.
Right-aligned data in columns, such as the integer data type, won't appear as indented.

@jswymer: I'm talking about the current version 16. I was focused on ShowAsTree. You are right IndentationControl is need, if just want to indent something without ShowAsTree.
My suggestion is to change

This property is used by the IndentationControls property and ShowAsTree property. It has no effect if these properties aren't properly set.

to

This property is used by the IndentationControls property. It has no effect if IndentationControls aren't properly set or ShowAsTree is used.

Hello, Thanks!  I agree that the paragraph in question is confusing. Sorry if I misunderstood your suggestion. But the IndentationColumn property comes into play when either the IndentationControls property or ShowAsTree property are set.

The IndentationColumn property is used together with either the IndentationControls property or ShowAsTree property.
The IndentationControls property is ignored when ShowAsTree property is set to true.

This article has been fixed accordingly. The update will be published soon. Thanks for the feedback!


Is User-Agent required?

[Enter feedback here]
Sorry, I am not good at English.
https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/rest-text-to-speech#sample-request-1
It requires "User-Agent".
I use the endpoint of https://japaneast.tts.speech.microsoft.com/cognitiveservices/v1.
It does not require "User-Agent".
API request without "User-Agent" header successes.
I find strange.
If this is right, "User-Agent" is Optional on https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/rest-text-to-speech#request-headers-1.

https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/rest-text-to-speech#sample-request-1
では
"User-Agent"が書いてあるべきです。
なぜなら https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/rest-text-to-speech#request-headers-1
で必須となっているからです。
しかしながら奇妙な現象を発見しました。
私は https://japaneast.tts.speech.microsoft.com/cognitiveservices/v1 エンドポイントを使いました。
"User-Agent"なしのリクエストが成功しました。
もし"User-Agent"がなしでよければ、https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/rest-text-to-speech#request-headers-1 はOptionになっているべきです。

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

ID: e351e013-771a-bfdc-0f76-815316c7717d
Version Independent ID: 8c3a1bbe-196a-13b8-f905-b8ea7bfd0e7a
Content: Text-to-speech API reference (REST) - Speech service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/rest-text-to-speech.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @trevorbye
Microsoft Alias: trbye

@TORIFUKUKaiou we'll review your feedback and get back to you shortly. Thanks.

@trevorbye it seems User-Agent should be "Optional". Can you please confirm if we can update?

We've reached out to subject matter experts and should have an answer on this one soon.

#please-close as it is unclear whether or not it is optional, and it may not be depending on region


No include option

There is no tutorial or any clue that include option is possible in Butterknife.
  <include
        layout="@layout/some_layout"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:id="@+id/my_id"/>

Nither this works:
@BindView(R.id.my_id)
    View myId;

Neither binding of included inner text views. Is that even possible?

<include> tags are inlined by the layout inflater such that the views in the included XML replace the tag. Since Butter Knife just calls findViewById, any view inside the include is available to be bound.

What about calling layout that it is not included? Do I have to create addition class where I inflate it and call inner views from that class?


Cannot call map from Ramda with same arity but different signatures

The way the typings for map in Ramda currently work, if you try and call map in two places with the same arity (arguments passed one-by-one, or both at once), but different type arguments (array, object, functor), Flow will blow up.
For example, try adding
const zs: {[key: string]: string} = _.map((x) => x.toString(), { a: 1, b: 2, c: 3 })

below here
and running the tests
cc/ @alexeygolev

@rjmk will check tonight. Let me know if you have ideas about the possible reason

@alexeygolev I will do some research on Flow / the links on the inference algorithm you left on the PR. At the moment I'm pretty in the dark about all of this!

@rjmk by the way what do you mean by "flow will blow up"?

@rjmk can't reproduce or I'm not understanding the problem correctly. If I change the type of object values to number in your signature flow will complain which means it works the way it should. Your expression typechecks

@alexeygolev Sorry, horrendous of me not to paste the error output. My issue was that in the type checking error message, it would only consider the members of map with array types.
Now, however, I also cannot reproduce. But not could I reproduce when changing the type of object values to number. git diff shows only the insertion of that line. Upon deleting files ignored by the .gitignore the error disappears.
Appears to be anomalous. Sorry for dragging you into that, @a


try to skip docker builds for tests

labeled wip (work in progress) do not merge yet

@alexsong93 I optimized the build in travis, got it down from 24 minutes to 14 minutes.
There was nothing else I was able to skip to get openwhisk deploy, and apigw test to run.


Support Embedded Elixir

Description
I need beautification support for Embedded Elixir, because is template for elixir which use markup same as html
Expected Results
The beautified code should have looked like same as html
Example Code

<%= handler_info @conn %>

Keys for the conn Struct

  <%= for key <- connection_keys @conn do %>
    <%= render "key.html", key: key %>
  <% end %>

Link: http://www.phoenixframework.org/docs/templates

Could you provide some links to documentation and/or example code so we can test? Thank you,

I have added link and also example code now.

@tahmina8765 Could you try submitting some of the Embedded Elixir code samples through http://prettydiff.com/?m=beautify
If that works for you then this becomes a very simple enhancement.

Yes, it works perfectly fine, except it shows an error on elixir syntax <%= %>

@tahmina8765 Could you be more specific or provide an example of code that fails?  I will get a fix published in the next version.

Please check attached image. It works perfectly fine, but shows some red cross in left side.
embeded-elixer.jpg
Thanks for quick response.

By the way, according to contributors doc, I tried to added this language in following repository -
https://github.com/tahmina8765/atom-beautify
I can not understand how to test it, I have run it in travis-ci, and there is 2 jobs failed but not for my code.

Adding a new language for use with an existing beautifier should be a matter of a simple configuration update, such as: https://github.com/Glavin001/atom-beautify/blob/master/docs/add-languages-and-beautifiers.md#how-to-add-a-language
You can see all the other language file settings as examples in: https://github.com/Glavin001/atom-beautify/tree/master/src/languages
But, if you are continuing to notice a defect in testing code samples against a supported beautifier that is either a defect or enhancement that has to be worked separately from that beautifier.

Closing as duplicate of https://github.com/Glavin001/atom-beautify/issues/545


Fix link for texlab

Config.md is auto generated from the lua file, please edit that instead.

sure, closing this in favor of #949


no level is loading

i ran the project in godot 3.3 rc6 , after the title scene and space scene, when i press enter/space bar , level doesn't load

Unable to reproduce in 3.3 stable.


Difficulties in reproducing the results described in ISWC'20 paper

The repository seems to contain several errors partially fixed by the merge request #1. However, despite the new modifications that guarantee the reproducibility of the scripts, the results obtained during the evaluation are lower than the ones reported in the paper, especially for coinfo250 and FakeNewsNet. @rdenaux do you have any ideas and do you know where the problem could be within the pipeline?

Hi @rdenaux,
Thank you for your quick response and help. Regarding the points you discussed:

We successfully collected the datasets necessary for the evaluation and the number of samples correspond to the number described for each evaluation dataset (clef18, coinfo250 and FakeNewsNet). Unfortunately, it is quite difficult to follow the validity of linked sources present in texts (such as URLs pointing to external tweets or articles).
We successfully tested the performances of the models. The claimencoder achieves 83% accuracy on STS-B and the worthinesschecker achieves 95% accuracy on its test set. In addition, we also tried to test the claimneuralindex with some sentence similarity test call.

The results currently achieved for each dataset are:

Clef18 accuracy: 0.3669
Coinform250 accuracy: 0.1774
FakeNewNet accuracy:  0.4726

As far as you know, is it possible that there is some other point in the pipeline that can cause such errors?