Update aws-java-sdk-core to 1.12.672

About this PR
📦 Updates com.amazonaws:aws-java-sdk-core from 1.12.459 to 1.12.672
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #720.


Change pre-batching from 1M to 1000

Right now, we're sending basically all returned results along to the next hop (1M ids). This makes a few KPs quite angry. We should test out changing the pre-batch size to 1000.
Should be a very simple change found here

After a little digging, this is most likely a bigger change than expected. It looks like right now what we call the "mini-batches" can be limited to 1,000 curies, but we do a second round of batching that groups multiple waiting requests into one larger request. This second batching happens in the trapi_throttle and currently doesn't expose a way to limit the number of curies in each final request, only the number of requests that it will batch together.

BTE is now complaining that we're sending too large of requests. Molepro sends a 413 error.

Update: We can now effectively set the batch size limit per query. However, setting it lower inevitably causes the query to timeout unless the overall query timeout is also bumped up. We need to wait for all batches to be returned before finishing, which takes a lot longer now that we're sending more queries, and those queries are actually being done (instead of erroring out quickly because the query was too big). This would also require the returning of partial results (https://github.com/ranking-agent/strider/issues/185).


Refactor code in the web/ folder to use async/await

Split off from PR #9944 as requested by @brendandahl.
Much more manageable diff with https://github.com/mozilla/pdf.js/pull/9995/files?w=1

/botio-linux preview

/botio-windows preview

Looks really good. Thanks!


Feature request: Concurrent exports

Based on the preview, when exporting an entire folder to PDF it seems to export one file at a time, for large folders this is quite slow, it would be great if this could be done concurrently.
Thank you for the great plugin.

Currently, preview rendering is not implemented in parallel. Exporting is asynchronous and parallel. The main time-consuming point is in the preview rendering stage. This will be optimized in the next step.


Add HACS support

It would be lovely if I could add & update this integration using HACS, and not have to manually copy files

I get the following error when trying to install via HACS

It should work now. when trying to add a dev brach to test some things it installed just fine for me


[util/vendor.py] Enable patching in combination with only_subdir

Git can only apply patches in the root directory of a repository. This PR makes sure patches can also be applied when only a sub directory of the upstream repo is actually being used.

Hm, that doesn't look right. And looking at sw/host/vendor/mpsse, the current behavior seems to work as expected. What path do you have in your patch files? The paths we have in the mpsse patches look as expected to me.

What I am trying to do is to integrate some IPs from OT in a different repo. I.e., vendor out instead of in (I think this should not make a difference though). My only_subdir is "hw/ip/prim". The path in an example patch file is a/rtl/prim_ram_2p_async_adv.sv. This does not work, the patch is skipped. The only thing that is working is to do it relative to the root directory of the repo: correct the path to a/hw/ip/prim/rtl/prim_ram_2p_async_adv.sv + apply the change in this PR.
Maybe the problem is that the actual file to be patched is inside another directory rtl in the subdirectory. For mpsse, there are no other hierarchy levels. The files to be patched are directly inside mpsse.

Something fishy is going on here. Can you please upload the problematic files in a zip here, or push them to a repository somewhere for me to have a look?

It is indeed fishy, I cleaned up the example files and corrected the path in the patch file back to a/hw/ip/prim/rtl/prim_ram_2p_async_adv.sv and it suddenly works without the change in this PR.

I am thus closing this PR. Thanks for your input @imphil .

The path should actually be relative to whatever is being copied to the target directory, i.e. the contents of hw/ip/prim. That would mean a path in the patch like a/rtl/prim_ram_2p_async_adv.sv. So there still could be a bug somewhere, if you get a chance, please upload a tree somewhere for me to have a look.

Here you go: https://github.com/vogelpi/vendor_example


WFCORE-648 Ear exclusions cascaded to sub-deployments

This is the integration test for WFCORE-648.
BE CAREFUL, this requires WFCORE 2.0.0-Alpha6+ (with commit for WFCORE-648 : https://issues.jboss.org/browse/WFCORE-648)

Can one of the admins verify this patch?

this is ok to test
even if it fails :)

Linux Build 6827 is now running using a merge of 1136ddcaf80a8fe5b68c4d85050c2880a3d8ce74

Linux with security manager Build 1724 is now running using a merge of 1136ddcaf80a8fe5b68c4d85050c2880a3d8ce74

Windows Build 1901 is now running using a merge of 1136ddcaf80a8fe5b68c4d85050c2880a3d8ce74

Linux with security manager Build 1724 outcome was SUCCESS using a merge of 1136ddcaf80a8fe5b68c4d85050c2880a3d8ce74
Summary: Tests passed: 855, ignored: 322 Build time: 0:06:29

Windows Build 1901 outcome was FAILURE using a merge of 1136ddcaf80a8fe5b68c4d85050c2880a3d8ce74
Summary: Tests failed: 1 (1 new), passed: 2875, ignored: 392 Build time: 0:46:31
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.deployment.classloading.ear.EarJbossStructureCascadeExclusionsTestCase: <no details avaliable>

Linux Build 6827 outcome was FAILURE using a merge of 1136ddcaf80a8fe5b68c4d85050c2880a3d8ce74
Summary: Tests failed: 4 (4 new), passed: 2869, ignored: 392, muted: 3 Build time: 0:47:02
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.deployment.classloading.ear.EarJbossStructureCascadeExclusionsTestCase: <no details avaliable>

org.jboss.as.test.integration.ejb.remote.distinctname.EarDeploymentDistinctNameTestCase.testRemoteSLSBInvocation: <no details avaliable>

org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>

org.jboss.as.test.integration.ws.wsse.signencrypt.SignEncryptMultipleClientsTestCase.encryptedAndSignedRequestFromJohn: <no details avaliable>

Note to self:
http://brontes.lab.eng.brq.redhat.com/viewLog.html?buildId=59483&tab=buildResultsDiv&buildTypeId=WF_WildFlyCoreIntegrationExperiments
is a run of a branch w/ WFCORE master + the core PR included, plus WFLY master + this PR included.


Cli Fails "panic makechan: size out of range"

Hello, I have copied the following example and tried running it locally but I get an error message:
Example:
https://github.com/openfga/sample-stores/blob/main/stores/banking/store.fga.yaml

My env:
Linux alexander-p14s 6.8.7-arch1-1 #1 SMP PREEMPT_DYNAMIC Wed, 17 Apr 2024 15:20:28 +0000 x86_64 GNU/Linux

fga version:
fga version v`0.4.0` (commit: `7f8f86973799d9be6ae443147977e36595461bc9`, date: `2024-05-07T13:11:24Z`)

docker compose:
version: '3.8'

networks:
  openfga:

services:
  postgres:
    image: postgres:14
    container_name: postgres
    networks:
      - openfga
    ports:
      - "5432:5432"
    environment:
      - POSTGRES_USER=postgres
      - POSTGRES_PASSWORD=password
    healthcheck:
      test: [ "CMD-SHELL", "pg_isready -U postgres" ]
      interval: 5s
      timeout: 5s
      retries: 5

  migrate:
    depends_on:
      postgres:
        condition: service_healthy
    image: openfga/openfga:latest
    container_name: migrate
    command: migrate
    environment:
      - OPENFGA_DATASTORE_ENGINE=postgres
      - OPENFGA_DATASTORE_URI=postgres://postgres:password@postgres:5432/postgres?sslmode=disable
    networks:
      - openfga

  openfga:
    depends_on:
      migrate:
        condition: service_completed_successfully
    image: openfga/openfga:latest
    container_name: openfga
    healthcheck:
      test: [ "CMD", "/usr/local/bin/grpc_health_probe", "-addr=openfga:8081" ]
      interval: 5s
      timeout: 30s
      retries: 3
    environment:
      - OPENFGA_DATASTORE_ENGINE=postgres
      - OPENFGA_DATASTORE_URI=postgres://postgres:password@postgres:5432/postgres?sslmode=disable
      - OPENFGA_LOG_FORMAT=json
    command: run
    networks:
      - openfga
    ports:
      # Needed for the http server
      - "8080:8080"
      # Needed for the grpc server (if used)
      - "8081:8081"
      # Needed for the playground (Do not enable in prod!)
      - "3000:3000"

  init_store:
    depends_on:
      openfga:
        condition: service_healthy
    image: nginx
    command:
      - sh
      - -c
      - "curl -X POST http://openfga:8080/stores -d '{\"name\": \"bankerise-corporate\"}'"
    networks:
      - openfga

Error:

panic: makechan: size out of range
goroutine 1 [running]:
github.com/openfga/openfga/pkg/storage/storagewrappers.NewBoundedConcurrencyTupleReader(...)
/home/runner/go/pkg/mod/github.com/openfga/openfga@v1.5.4-rc1/pkg/storage/storagewrappers/boundedconcurrency.go:45
github.com/openfga/openfga/pkg/server.(*Server).Check(0x99b0278, {0x8cac2e8, 0x93ea1a0}, 0x9e1f9b0)
/home/runner/go/pkg/mod/github.com/openfga/openfga@v1.5.4-rc1/pkg/server/server.go:792 +0xa5e
github.com/openfga/cli/internal/storetest.RunSingleLocalCheckTest(...)
/home/runner/work/cli/cli/internal/storetest/localtest.go:18
github.com/openfga/cli/internal/storetest.RunLocalCheckTest(0x99b0278, {{0x9c078a4, 0x9}, {0x9c97ce0, 0x1d}, 0x9f86028, 0x99b7980}, {0x9adaf08, 0xd, 0xd}, ...)
/home/runner/work/cli/cli/internal/storetest/localtest.go:53 +0x2b9
github.com/openfga/cli/internal/storetest.RunLocalTest(0x99b0278, {{0x995ea20, 0x18}, {0x0, 0x0}, {0x0, 0x0, 0x0}, {0x0, 0x0}, ...}, ...)
/home/runner/work/cli/cli/internal/storetest/localtest.go:242 +0x1b2
github.com/openfga/cli/internal/storetest.RunTest(0x9a18218, 0x99b0278, {{0x995ea20, 0x18}, {0x0, 0x0}, {0x0, 0x0, 0x0}, {0x0, ...}, ...}, ...)
/home/runner/work/cli/cli/internal/storetest/tests.go:29 +0x1f4
github.com/openfga/cli/internal/storetest.RunTests(0x9a18218, 0x9c16640, {0x8b4247f, 0x7})
/home/runner/work/cli/cli/internal/storetest/tests.go:47 +0x22e
github.com/openfga/cli/cmd/model.init.func3(0x93b3260, {0x98fc300, 0x0, 0x6})
/home/runner/work/cli/cli/cmd/model/test.go:60 +0x254
github.com/spf13/cobra.(*Command).execute(0x93b3260, {0x98fc2d0, 0x6, 0x6})
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.8.0/command.go:983 +0xaa0
github.com/spf13/cobra.(*Command).ExecuteC(0x93b2960)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.8.0/command.go:1115 +0x458
github.com/spf13/cobra.(*Command).Execute(...)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.8.0/command.go:1039
github.com/openfga/cli/cmd.Execute()
/home/runner/work/cli/cli/cmd/root.go:48 +0x23
main.main()
/home/runner/work/cli/cli/cmd/fga/main.go:21 +0x17

Hi @sashokbg
What command do you run when getting the error?
I tried:

Running docker compose up
fga store list to get the store ID
export FGA_STORE_ID=
running fga store import --file store.fga.yaml, that uploads the model/tuples
run
fga query check customer:anne can_make_bank_transfer account:123 --context '{"transaction_amount":"10", "new_transaction_limit_approved":"0"}'

I got:
{
  "allowed":true,
  "resolution":""
}

Hello @aaguiarz
Here is what I did:
➜  openfga git:(develop) ✗ fga store list
{
"continuation_token":"",
"stores": [
{
"created_at":"2024-05-27T12:52:09.88043Z",
"id":"01HYX1QE0RQMK3KP25XYBJ7NHC",
"name":"store",
"updated_at":"2024-05-27T12:52:09.88043Z"
}
]
}
➜  openfga git:(develop) ✗ fga model test --store-id 01HYX1QE0RQMK3KP25XYBJ7NHC --tests store.fga.yaml

That works for me :(
In any case, that command will use the model from the store.fga.yaml and run the tests in memory, it won't run them on the server.
If you run fga store import --file store.fga.yaml it will import the model/tuples (you can verify that by running fga model get fga tuple read.
To run the tests on the server, you'd need to remove the model/tuples from the store.fga.yaml and the rests will be run on the server.
This behavior is not obvious, we'll clarify in docs and in the CLI output.

Even fga store import --file store.fga.yaml does not work for me - I get just "null" result and nothing happens :(

I'm getting null too, which seems to be a bug when you are providing the store (if you don't specify it, it creates one), but seems to be working.
Try doing fga model get after running it.

Added a new issue for the null bug here https://github.com/openfga/cli/issues/334

Hello, I may have downloaded the x86 architecture instead of adm64 .. I just reinstalled my cli and restarted all my db and docker etc. and everything works now !


Update communication.

I have change MPI to required because I don't think that we want to test that DTK works in serial (doesn't matter for now but it will once we add global search) but I can change it back.

Codecov Report

Merging #261 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #261   +/-   ##
=======================================
  Coverage   96.95%   96.95%
=======================================
  Files          49       49
  Lines        2496     2496
=======================================
  Hits         2420     2420
  Misses         76       76

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 0a2ff66...9ccd88a. Read the comment docs.

@aprokop while I agree someone could in theory use DTK in serial. I don't see why someone who want to do that -> I don't see what we would test that -> if it is not tested, it doesn't work.

@Rombur I don't have any specific counterarguments, partially because I don't know our user base all that well. I just don't think supporting Serial is hard, and if we add a test in #259 it makes it even easier. I assume that some people/codes may want to run DTK even on a single workstation with OpenMP.

@aprokop I'll reverse the comment for MPI since we don't use it right now anyway and I'll add a comment about one more possible configuration in #259


fix(terraform destroy): fixed the issue doing terraform destroy when …

Description
Closes #1
Issue
If AWS is out of capacity and terraform fails to apply, we wouldn't be able to do terraform destroy without commenting the modules outputs.
$ terraform apply

...
module.dedicated-host.aws_cloudformation_stack.dedicated_hosts: Still creating... [30m0s elapsed]

Error: error waiting for CloudFormation Stack creation: timeout while waiting for state to become 'CREATE_COMPLETE, CREATE_FAILED, DELETE_COMPLETE, DELETE_FAILED, ROLLBACK_COMPLETE, ROLLBACK_FAILED' (last state: 'CREATE_IN_PROGRESS', timeout: 30m0s)

$ terraform destroy

Error: Invalid index

  on main.tf line 44, in output "dedicated-host":
  44:   value = module.dedicated-host.dedicated_hosts["HostID"]
    |----------------
    | module.dedicated-host.dedicated_hosts is empty map of string

The given key does not identify an element in this collection value.

Fix
CloudFormation outputs will be an empty string in case of error.
Added output with the host_id to be used instead of the CloudFormation map output

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


typo in comparison and possible script-flow glitch

looking into _background.sh lines 104...
  updateDynDomain=0
  if [ ${recheckPublicIP} -eq 1 ]; then
    echo "*** RECHECK PUBLIC IP ***"

    # execute only after setup when config exists
    if [ ${configExists} -eq 1 ]; then

      publicIPChanged=$(/home/admin/config.scripts/internet.sh update-publicip | grep -c 'ip_changed=1')

    # check if changed
    elif [ ${publicIPChanged} -get 0 ]; then

      # refresh data
      source /mnt/hdd/raspiblitz.conf

      # only restart LND if auto-unlock is activated
      if [ "${autoUnlock}" = "on" ]; then
        echo "restart LND with to pickup up new publiIP"
        sudo systemctl stop lnd
        sudo systemctl start lnd
      else
        echo "new publicIP but no LND restart because no auto-unlock"
      fi

      # trigger update if dnyamic domain (if set)
      updateDynDomain=1

    else
        echo "public IP has not changed"
    fi

  fi

I guess the line elif [ ${publicIPChanged} -get 0 ]; then should be elif [ ${publicIPChanged} -gt 0 ]; then
further looking into the program flow it might not do as intended....
if the config file exsits then the check for changed global IP is executed, but I guess that the elif [ ${publicIPChanged} -get 0 ]; then  part which will evaluate the above return value will not be executed.
I might be wrong as I'm not so familiar with bash programming.

Jupp the -get is a bad typo and the if just makes sense ... must have been a late night edit of mine. Good catch. Thanks.


Can't delete or rename any file !

I don't know what is the problem but I can't rename or delete any file and it's giving error :
Error renaming file
http://imgur.com/Fn1IXwG
I tried to open Brackets with sudo access but It didn't work and still it's giving error.
Please somebody help me :(

As the message says you cannot rename or delete files that are outside the current project.
Unfortunately this is a limitation of Brackets...
Alternatively you could open the project which contains those files or simply do that on the file explorer on your system.


Proper use of addon tag in DataTable.vue

May I ask some help how can use the addon tag in DataTable?
Following example is working for me:
                 "_actions": {
                       "toolbar": {
                          "create": False,
                          "delete_all": False
                       },
                       "buttons": {
                          "show": False,
                           "delete": False,
                      },
                      "addon": {
	                      "button": {
	                          "label": "testlabel",
	                          "href": "/ddd/",
	                       },
                      }
                   },

How can I get to append the selected line _id's to the url? Is there any way for it?

Thank You. It is working! Save my day :)


10.1.1

:rocket: Prepare release 10.1.1

Would be good to add #2455 after getting merged otherwise LGTM 🚀

@KingDarBoja #2455 is non-essential for the release.


Release

[x] Standardize
[x] Initial PAPI Testing
[ ] Benchmarking on PAPI
[ ]  Dockstore Upload
[ ]  FireCloud Method Upload
[ ]  Firecloud Workspace Creation
[ ]  Firecloud Workspace Documentation

moving issue to dsde-docs


Clean up README

Consistent use of ATX-style headers
Fix some incorrect syntax
Follow Markdown syntax more strictly in some cases (due to some renderers being more sensitive [e.g., Chef Supermarket])

This also includes an opinionated change to the headers throughout, with the addition of a title at the top. Totally cool to strip out that change if it irks anyone.

Thanks for your contribution @jeffbyrnes!
Kind regards,
David


after crouton update -u kali won't start X (connection refused)

name: kali
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/kali...
crouton: version 1-20161129162558~master:5f2f11f9
release: kali-rolling
architecture: amd64
xmethod: xiwi
targets: xiwi,xfce
host: version 9000.29.0 (Official Build) beta-channel cyan
kernel: Linux localhost 3.18.0-13497-gbec0bdb #1 SMP PREEMPT Wed Dec 14 22:05:02 PST 2016 x86_64 GNU/Linux
freon: yes

Please describe your issue:
sudo startxfce4
Entering /mnt/stateful_partition/crouton/chroots/kali...
/usr/bin/startxfce4: Starting X server
_XSERVTransmkdir: Owner of /tmp/.X11-unix should be set to root
X.Org X Server 1.19.0
Release Date: 2016-11-15
X Protocol Version 11, Revision 0
Build Operating System: Linux 3.16.0-4-amd64 x86_64 Debian
Current Operating System: Linux localhost 3.18.0-13497-gbec0bdb #1 SMP PREEMPT Wed Dec 14 22:05:02 PST 2016 x86_64
Kernel command line: cros_secure console= loglevel=7 init=/sbin/init cros_secure oops=panic panic=-1 root=/dev/dm-0 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="1 vroot none ro 1,0 3584000 verity payload=PARTUUID=1d4d5bda-7556-364a-b7cf-105796f2ff37/PARTNROFF=1 hashtree=PARTUUID=1d4d5bda-7556-364a-b7cf-105796f2ff37/PARTNROFF=1 hashstart=3584000 alg=sha1 root_hexdigest=4e2c477dd543151f7a10719395701f4fec7df592 salt=d6d9f861b6b203f45699385a91b4bb1911a91ddc36bb4e35fdac797f572bbc9f" noinitrd vt.global_cursor_default=0 kern_guid=1d4d5bda-7556-364a-b7cf-105796f2ff37 add_efi_memmap boot=local noresume noswap i915.modeset=1 tpm_tis.force=1 tpm_tis.interrupts=0 nmi_watchdog=panic,lapic
Build Date: 16 December 2016  07:30:27PM
xorg-server 2:1.19.0-3 (https://www.debian.org/support)
Current version of pixman: 0.34.0
Before reporting problems, check http://wiki.x.org
to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
(++) from command line, (!!) notice, (II) informational,
(WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(++) Log file: "/tmp/Xorg.crouton.3.log", Time: Tue Jan  3 22:18:22 2017
(++) Using config file: "/etc/X11/xorg-dummy.conf"
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
/usr/lib/xorg/Xorg: symbol lookup error: /usr/lib/xorg/modules/drivers/dummy_drv.so: undefined symbol: ChangeWindowProperty
sudo startxiwi -b xterm/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: Connection refused
/usr/bin/xinit: server error
Broke yesterday after:
Inside Kali: croutonversion -u -d -c
Exit;
Outside chroot: sudo sh ~/Downloads/crouton -u -n kali
(Finished without errors, now won't start X, but enter-chroot works)
Redownloading crouton-script and re-renning update command above won't fix it.

@The1andONLYdave,
It reported an Xorg log here:

(++) Log file: "/tmp/Xorg.crouton.3.log", Time: Tue Jan 3 22:18:22 2017

Could you post the output of that log file or give us a gist or pastebin link, etc. to it please?
-DennisL

＃2953 would help

I created a new log with same error: http://pastebin.com/maM0eAMw

@The1andONLYdave,
Thanx for the Xorg log.
I'm afraid I'm out of my depth when it comes to analyzing and interpreting the data, I'm hoping someone more knowledgeable about it might be able to help.
Of note though I think are these GLX entries:
[ 13108.059] (II) LoadModule: "glx"
[ 13108.062] (II) Loading /usr/lib/xorg/modules/extensions/libglx.so
[ 13108.082] (II) Module glx: vendor="X.Org Foundation"
[ 13108.082]    compiled for 1.19.0, module version = 1.0.0
[ 13108.082]    ABI class: X.Org Server Extension, version 10.0

and further down:
[ 13108.101] (II) AIGLX: Screen 0 is not DRI2 capable
[ 13108.101] (EE) AIGLX: **reverting to software rendering**

And there are some about the DUMMY driver too:
[ 13108.083] (II) DUMMY: Driver for Dummy chipsets: dummy
[ 13108.083] (WW) Falling back to old probe method for dummy
[ 13108.083] (WW) VGA arbiter: cannot open kernel arbiter, no multi-card support

I'm afraid I don't know if these warnings relate to 'xinit giving up' or not though.
Maybe the issue is the one mentioned by @bigeagle above concerning the /usr/lib/xorg/Xorg: symbol lookup error ???
Hope this helps,
-DennisL

@bigeagle
I don't suppose you can give us a short tutorial about how to do that?  I know how to git clone and such, and make as well.  Did you just do it inside your chroot and make install?

@middleagedman No, I did this on my Linux PC.
``
git clone https://github.com/nxtr/crouton
git checkout origin/xiwi-xf86-video-dummy-0.3.8
make
you'll get a runnable crouton script.

@bigeagle
Thanks!  I do have the misfortune of having a x86 linux box and a rockchip asus flip, but I'll see if I can narrow a build environment for just the video-dummy

facepalm nm.. make just builds an installer script.  Good to go

In case anyone has similar problems, I had to do the following to overcome this error:
$ enter the chroot
$ sudo apt-get install git
$ git clone https://github.com/nxtr/crouton
$ cd crouton
$ git checkout origin/xiwi-xf86-video-dummy-0.3.8
$ sudo apt-get install make
$ make
$ cp crouton ../Downloads/
However, this still didn't solve the problem, so I ran the crouton update (-u) using the new crouton script. Then it worked. :-)

I have the same problem but with sid

I think this fix has been merged.


[FEATURE] Add option to disable cursorline default

Is there a specific reason to have the autocmd to set cursorline on the focused window? I imagine it has something to do with making it more obvious split is currently focused, but either way, would it be possible to have a setting to disable it?

Also some food for thought, you can do something like:
set winhighlight=Normal:FocusedWindow,NormalNC:UnfocusedWindow

to actually highlight the focused split with a different color. You'd need to define the FocusedWindow and UnfocusedWindow, preferably with :h hi-default so that users can override them, as well as gating this behind a variable to completely disable it.
See a proposed implementation of this in #5.

Either way, awesome plugin, it'd just be nice to be able to configure it a bit more.

This resolves my problem :) Nice!


Add TypeScript types for DefinePlugin

DefinePlugin doesn't have any types in webpack. There are some in @types/webpack but they are incomplete: https://github.com/DefinitelyTyped/DefinitelyTyped/blob/796b838a15fad73287bad7a88707a9ca04e60640/types/webpack/v3/index.d.ts#L726-L728
webpack.DefinePlugin.runtimeValue is missing both types and docs

You should send a PR to DefinitelyTyped

Why does webpack have some types but not others?


[FLOC-1581] List containers state

https://clusterhq.atlassian.net/browse/FLOC-1581

Thanks.  Looks good apart from those minor points.  Apart from that ... end-to-end tests?  Are we doing those for the container API?  Later?
Please address inline comments, maybe file a follow-up issue for end-to-end-testing, and merge.

First acceptance we have is FLOC-1567, and that will need to use this code path too so should do.


Partial Refresh of Feed

The feed should refresh incrementally - reloading only modified items in existing timeline. Currently, the whole feed is refetched as a temporary implementation. (upon like, unlike, comment)

Completed and verified tests.


read FixedString from BinaryInputStream

ClickHouseRowBinaryInputStream can't read FixedString(n) and throw exception in this case, therefore the functionality for reading FixedString(n) has been added.
this issue

@ayudovin , please add a test, checking behavior for zero-padded string (e.g. when FixedString(10) and inserted "hello", should return "hello\0\0\0\0\0"

@alex-krash, I have added changes by your suggestions.

insert with fixedString is already supported?

insert with fixedString is already supported?

general write string should work for you

@ayudovin , thanks!


Cannot load JSON neither YAML in Android

Hi.
I've been trying (a lot) to use jpmml to load PMML-based models to Android apps - the goal, of course, is to have on-device inference - but I've been facing too many problems with too many approaches.
The one I'm reporting here is for both JSON and YAML models.
Let's take, as example, the decision tree pmml model over the iris dataset that you made available on the archived jpmml-android repository (but you can use any other). I've been using version 1.5.15 of both jpmml-model and jpmml-evaluator.

The first step was to translate the pmml model to json/yaml (successfully) as follows:
java -cp pmml-model-example/target/pmml-model-example-executable-1.6-SNAPSHOT.jar org.jpmml.model.example.TranslationExample --input result/model.pmml --output result/model.json
or
java -cp pmml-model-example/target/pmml-model-example-executable-1.6-SNAPSHOT.jar org.jpmml.model.example.TranslationExample --input result/model.pmml --output result/model.yaml

Then, placed both files on the assets folder of the Android app (successfully);

For model.json file: after loading it from the assets manager, I've used the JacksonUtil.readPMML(inputStream) method from the pmml-model/pmml-model-jackson lib. I had no luck and got a huge stack trace;

For model.yaml file: I had a little more work but faced a similar problem. I had to import the com.fasterxml.jackson.dataformat.yaml package to set the YAMLFactory, which I set on the objectMapper. Again, a huge stack trace, pretty similar to the one of the json file.

Both log files are attached to this issue but what you will see is a lot ot "Unable to resolve java.lang.Class<org.dmg.pmml.XXXX> annotation class YYYY" and a "java.lang.NoClassDefFoundError: Failed resolution of: Ljavax/xml/bind/annotation/adapters/XmlAdapter" which leads to "Failed resolution of: Lorg/dmg/pmml/adapters/NumberAdapter".
Android_json_log.txt
Android_yaml_log.txt
Any idea why this is happening and how can we, indeed, use json/yaml pmml-based files in Android since, as per the README, it is one of the supported platforms?
A clear tutorial would be nice though.
Cheers,
Bruno

The pmml-model-jackson module is intended for Java SE/EE platforms, as they assume the availability of JAXB runtime.
If you want to use this module on Android platform, then you'd probably need to include some 3rd party helper libraries to compensate for the missing JAXB runtime.
IIRC, you can use Jackson in several modes - following JAXB annotations, following JSON annotations, or a mix of both (JSON overriding JAXB, etc). I can't remember which one is the default mode.
First make sure which annotations are recognied by your Jackson runtime.

you will see is a lot ot "Unable to resolve java.lang.Class<org.dmg.pmml.XXXX> annotation class YYYY" and a "java.lang.NoClassDefFoundError: Failed resolution of: Ljavax/xml/bind/annotation/adapters/XmlAdapter" which leads to "Failed resolution of: Lorg/dmg/pmml/adapters/NumberAdapter".

There are @XmlJavaTypeAdapter annotations in the generated JPMML-Model classes, which reference appropriate XML-to-Java type adapters.
Try including a 3rd party library into your Android project that would provide this interface. For example, jakarta.xml.bind:jakarta.xml.bind-api(:2.3.3). What then?

how can we, indeed, use json/yaml pmml-based files in Android since, as per the README, it is one of the supported platforms?

It is supported in a sense that all JPMML-Model library core classes/interfaces only use Android-compatible Java APIs.
See here:
https://github.com/jpmml/jpmml-model/blob/1.5.15/pom.xml#L254-L278

A clear tutorial would be nice though.

It's been on my TODO list for several years now.

Other things you may try if the JSON/YAML pathway is blocked:

Use plain old Java serialization.
Use Kryo serialization, as implemented by the org.jpmml:pmml-model-kryo module.

Using binary serialization you can "transfer" both org.dmg.pmml.PMML objects, or fully-initialized org.jpmml.evaluator.Evaluator objects. With JSON/YAML you're limited to the former.

Villu,
I don't know if you want to discuss it here (or in a different issue) but I also add problems with the jpmml-transpiler v1.1.10.
java -jar target/jpmml-transpiler-executable-1.1-SNAPSHOT.jar --pmml-input result/model.pmml --jar-output result/model.pmml.jar --class-name com.model.DecisionTreeB
Not to load the jar at runtime and mess with Android's DVM, I've declared the jar as a dependency in my project and, when trying to instantiate the corresponding class, I get this stack trace (using jpmml-evaluator v1.5.15):
E/AndroidRuntime: FATAL EXCEPTION: main   java.lang.NoClassDefFoundError: Failed resolution of: Lorg/jpmml/evaluator/ResourceUtil;     at com.model.DecisionTreeB.<clinit>(DecisionTreeB.java:38)     at com...model.PmmlEvaluator.load_evaluator(PmmlEvaluator.kt:14)     at com...MainActivity.onCreate(MainActivity.kt:44)     at android.app.Activity.performCreate(Activity.java:7136)
Any on this? Do we need the pmml-evaluator-extension?
Because if we do, we see again a "Failed resolution of: Lorg/dmg/pmml/adapters/NumberAdapter".
About plain old Java serialization, well, I had problems there too (even though I was able to workaround them with the archieved jpmml-android). I guess I will open a new issue for that.
Thanks.

E/AndroidRuntime: FATAL EXCEPTION: main
java.lang.NoClassDefFoundError: Failed resolution of: Lorg/jpmml/evaluator/ResourceUtil;
Do we need the pmml-evaluator-extension?

Yes, your Android application would need to include both org.jpmml:pmml-evaluator and org.jpmml:pmml-evaluator-extension modules. The ResourceUtil utility class resides in the latter module.

Because if we do, we see again a "Failed resolution of: Lorg/dmg/pmml/adapters/NumberAdapter".

The org.jpmml:pmml-evaluator module should automatically include the org.jpmml:pmml-model module, which contains all JAXB type adapter classes. Perhaps it cannot be loaded because of the missing @XmlJavaTypeAdapter annotation class again? If so, then you'd need to include JAXB API classes explicitly.

Ok, it seems that everything is leading towards the same issue...
Loading the JAXB lib is likely to fix the issue (but will come with a cost in the app's size).
For me, you can close this issue!
Thanks.

Loading the JAXB lib is likely to fix the issue (but will come with a cost in the app's size).

The jakarta.xml.bind:jakarta.xml.bind-api(:2.3.3) library JAR file weighs around 100 kB (0.1 MB). I don't see how that could be a deal-breaker.

The  jakarta.xml.bind:jakarta.xml.bind-api(:2.3.3) isn't enough. The Xerces2 J is also required (another 1.3 MB). That, together with the 600kb of the org.jpmml:pmml-evaluator lib and a few more kb of the jar model...
Nonetheless, with the following libs one is able to run transpiled PMML objects in Android successfully (it is also likely to solve JSON/YAML problems - didn't try it though):
jakarta.xml.bind:jakarta.xml.bind-api:2.3.3
xerces:xercesImpl:2.12.1
org.jpmml:pmml-evaluator:1.5.15
org.jpmml:pmml-evaluator-extension:1.5.15

The jakarta.xml.bind:jakarta.xml.bind-api(:2.3.3) isn't enough. The Xerces2 J is also required (another 1.3 MB).

I'm not aware of any source code-level functionality that would require Xerces2J.
Can you elaborate, what's the exception if Xerces2J is not available? Is there some specific class missing again (some SAX class/interface declaration)? If so, then there's probably a smaller replacement library available (something with an -api suffix).

java.lang.Error: javax.xml.datatype.DatatypeConfigurationException: Provider org.apache.xerces.jaxp.datatype.DatatypeFactoryImpl not found
at javax.xml.bind.DatatypeConverterImpl.<clinit>(DatatypeConverterImpl.java:877)
at javax.xml.bind.DatatypeConverter.initConverter(DatatypeConverter.java:125)
Caused by: javax.xml.datatype.DatatypeConfigurationException: Provider org.apache.xerces.jaxp.datatype.DatatypeFactoryImpl not found
at javax.xml.datatype.DatatypeFactory.newInstance(DatatypeFactory.java:106)
... 52 more
It seems to be Android-related (https://developer.android.com/reference/kotlin/javax/xml/datatype/DatatypeFactory).

java.lang.Error: javax.xml.datatype.DatatypeConfigurationException: Provider org.apache.xerces.jaxp.datatype.DatatypeFactoryImpl not found

See the Javadoc description of DatatypeFactory.
The Xerces2J is the default implementation, which is used if any other implementation is not configured/made available. It should be possible to implement a dummy DatatypeFactory subclass (eg. all methods throw UnsupportedOperationException), and instruct the Android application to use it instead.
The idea is that a transpiled org.dmg.pmml.PMML object should not need any Java XML APIs to function (you'd only need some interfaces/annotations as exemplified with the jakarta.xml.bind:jakarta.xml.bind-api module, but not the actual implementation classes).
Also, it whould be safe to catch and swallow javax.xml.datatype.DatatypeConfigurationException. The JPMML part of the codebase should work fine even after that.


Position from reactFlowInstance.project incorrect

Describe the Bug
I have a customNode Board, and I want to drag some customNodes into it.
I refer to the "drag and drop" part of examples and the code is like below:
const onDrop = useCallback(
    (event: any) => {
      event.preventDefault();

      const nodeInfo = event.dataTransfer.getData("application/reactflow");
      const { nodeType, label, nodeConfig } = JSON.parse(nodeInfo);

      // check if the dropped element is valid
      if (typeof nodeType === "undefined" || !nodeType) {
        return;
      }

      if (reactFlowWrapper.current && reactFlowInstance) {
        const reactFlowBounds =
          reactFlowWrapper.current.getBoundingClientRect();

        const position = reactFlowInstance.project({
          x: event.clientX - reactFlowBounds.left,
          y: event.clientY - reactFlowBounds.top,
        });

        const boardRects: any[] = [];
        const boards = nodes.filter((node) => node.type === "BOARD");

        boards.forEach((node) => {
          const { x, y, width, height } = getRectOfNodes([node]);
          boardRects.push({
            left: x,
            right: x + width,
            top: y,
            bottom: y + height,
          });
        });

        for (let index = 0; index < boardRects.length; index++) {
          const rect = boardRects[index];
          const board = boards[index];
          if (
            position.x > rect.left &&
            position.x < rect.right &&
            position.y > rect.top &&
            position.y < rect.bottom
          ) {
            const newNode = addNode(
              nodeType,
              label,
              position,
              nodeConfig,
              board.id
            );
            setNodes((nodes) => nodes.concat(newNode));
          }
        }
      }
    },
    [reactFlowInstance, reactFlowWrapper.current, nodes]
  );

The initial Board data is like below:
{
	"id": "67bd04dc-0da8-46aa-b38f-05dbf70140c6",
	"type": "BOARD",
	"position": {
		"x": 0,
		"y": 0
	},
	"data": {
		"name": "Board",
		"width": 800,
		"height": 800
	},
	"zIndex": -1,
	"selected": false,
	"width": 800,
	"height": 800,
	"positionAbsolute": {
		"x": 0,
		"y": 0
	},
	"dragging": false
}

Here is the screen record for this problem.
Video
Your Example Website or App
https://codesandbox.io/embed/silly-frost-q4dngv?fontsize=14&hidenavigation=1&theme=dark
Steps to Reproduce the Bug or Issue

Drag a node from sidebar to the graph
The new node's position is not the position at the mouse cursor

Expected behavior
The position is right
Screenshots or Videos
(video)[https://bonbon.oss-cn-shanghai.aliyuncs.com/iShot_2023-07-11_15.23.54.mp4]
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

The projected position isn't wrong from what I can tell, it's the fact that you add the node as a child node and don't consider the parent's position when projecting.
This should work I believe:
const parentNode = getNode(parentNodeId);

const position = reactFlowInstance.project({
  x: event.clientX - reactFlowBounds.left - parentNode.position.x,
  y: event.clientY - reactFlowBounds.top - parentNode.position.y
});

Thanks for your reply! Your suggestion is very useful.
I will have a try right now and give you a feedback.

I change my code to below:
const boardPostion = board.position;
const newNode = addNode(
  nodeType,
  title,
  {
    x: position.x - boardPostion.x,
    y: position.y - boardPostion.y,
  },
  nodeConfig,
  board.id,
  nodeAttributes
);

and it's really works!
Thanks!
It's important to caculate the parentNode's position when using project


Could not find the '@angular-devkit/build-angular:dev-server' builder's node package.

🐞 Bug report
Command

[x] build
[x] serve
[x] run

The globally required package has been installed as well as
node 16.13.2
angular 13.2.0
npm 8.4.0
Description
 Created a clean angular project in docker. 'npm i' was successful. when trying to 'npm start/ ng serve' gave an error.
🔬 Minimal Reproduction

🔥Angular package.json

 "scripts": {
    "start": "npm i && ng serve",
    "startt": "export NODE_OPTIONS=--max_old_space_size=4096 && npm i --no-audit  --legacy-peer-deps && ng serve -c production",
    "vm": "npm i  --no-audit -s --legacy-peer-deps --only=production && npm update && export NODE_OPTIONS=--max_old_space_size=4096 && ng serve -c production --host <IP_ADDRESS> --port 4200 --disable-host-check",
    "test": "npm i -D --no-audit  --legacy-peer-deps && ng serve -c production"
  },
  "private": true,
  "dependencies": {
    "@angular/animations": "^13.1.3",
    "@angular/cdk": "^13.1.3",
    "@angular/common": "^13.1.3",
    "@angular/compiler": "^13.1.3",
    "@angular/core": "^13.1.3",
    "@angular/forms": "^13.1.3",
    "@angular/material": "^13.1.3",
    "@angular/platform-browser": "^13.1.3",
    "@angular/platform-browser-dynamic": "^13.1.3",
    "@angular/router": "^13.1.3",
    "@auth0/angular-jwt": "^5.0.2",
    "angular-highcharts": "^13.0.1",
    "angular2-text-mask": "^9.0.0",
    "highcharts": "^9.3.2",
    "moment": "^2.29.1",
    "rxjs": "~7.5.2",
    "tslib": "^2.3.1",
    "zone.js": "~0.11.4"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "^13.1.4",
    "@angular/cli": "^13.1.4",
    "@angular/compiler-cli": "^13.1.3",
    "@types/jasmine": "~3.10.3",
    "@types/node": "^17.0.8",
    "jasmine-core": "~4.0.0",
    "karma": "~6.3.10",
    "karma-chrome-launcher": "~3.1.0",
    "karma-coverage": "~2.1.0",
    "karma-jasmine": "~4.0.1",
    "karma-jasmine-html-reporter": "~1.7.0",
    "typescript": "^4.5.5"
  }

🌍 Your Environment

FROM node:16

RUN npm i -g<EMAIL_ADDRESS>RUN npm i -s -g @angular/cli
RUN npm i -s -g @angular/compiler-cli @angular/language-service @angular-devkit/build-angular

# Set the locale
ENV LANG en_US.UTF-8
ENV LC_ALL en_US.UTF-8
ENV LANGUAGE en_US.UTF-8
ENV NODE_ENV production
WORKDIR /app
COPY package*.json .
COPY . ./
CMD cd client && ng serve

Anything else relevant?
I went into the docker container and tried to create a new project, and I noticed that with the command 'npm i' in the folder node_modules/@angular-devkit/
no files at all

Hello, we reviewed this issue and determined that it doesn't fall into the bug report or feature request category. This issue tracker is not suitable for support requests, please repost your issue on StackOverflow using tag angular-cli.
If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.

The problem however is because setting the  environment variable NODE_ENV to "production" causes npm install to ignore dev dependencies.


Support helm

Wonder if we should provide support (optionally?) for Helm https://github.com/kubernetes/helm

#70 should cover this. Further tickets will be raised for charts support and addons


Readme - Updates

With the release of RPP package we need to revisit the README to fix & update current flow.

#230 - track this issue here


Audit Fix: fee overflow

From ottersec:
We noticed a potential overflow issue ( overflow while casting uint64 to int64 )that passed ValidateBasic  and caused a protocol panic here when users withdraw with a fee greater than Max.int64.
Solution: In ValidateTickFee we ensure that fee < MaxTickExp. This makes it impossible to overflow or wrap the tick uint

https://github.com/neutron-org/neutron-tests/actions/runs/9258877729


Use supplemental-ui to configure the header

It could be nice to have a dedicated header per module (so we could contextualize the links) but until then I've replaced the default header with links on the Asciidoctor repository (GitHub), Issue tracker (GitHub) and Chat (Gitter).

@mojavelinux Anything wrong ?
I'm not sure if you've already planned to create a custom UI (or if you want to usethe default UI) but in the meantime could we merge this small improvement ?
I'm using this documentation site for the Asciidoctor.js User Manual and it would be better if the header is relevant 😄

Nope, no problem. I just got sidetracked. Over the next few weeks, Sarah and I will be working to complete the docs migration to docs.asciidoctor.org. We do plan to create a custom UI, but we can use the default UI with these customizations in the interim.

Nope, no problem. I had just gotten sidetracked. I'll proceed with merging. Thanks for submitting these modifications.
Over the next few weeks, Sarah and I will be working to complete the docs migration to docs.asciidoctor.org. While we do plan to create a custom UI, we can use the default UI with these customizations in the interim.

Nope, no problem. I had just gotten sidetracked. I'll proceed with merging. Thanks for submitting these modifications.
Over the next few weeks, Sarah and I will be working to complete the docs migration to docs.asciidoctor.org. While we do plan to create a custom UI, we can use the default UI with these customizations in the interim.

Nope, no problem. I had just gotten sidetracked. I'll proceed with merging. Thanks for submitting these modifications.
Over the next few weeks, Sarah and I will be working to complete the docs migration to docs.asciidoctor.org. While we do plan to create a custom UI, we can use the default UI with these customizations in the interim.

Nope, no problem. I had just gotten sidetracked. I'll proceed with merging. Thanks for submitting these modifications.
Over the next few weeks, Sarah and I will be working to complete the docs migration to docs.asciidoctor.org. While we do plan to create a custom UI, we can use the default UI with these customizations in the interim.

Nope, no problem. I had just gotten sidetracked. I'll proceed with merging. Thanks for submitting these modifications.
Over the next few weeks, Sarah and I will be working to complete the docs migration to docs.asciidoctor.org. While we do plan to create a custom UI, we can use the default UI with these customizations in the interim.

Nope, no problem. I had just gotten sidetracked. I'll proceed with merging. Thanks for submitting these modifications.
Over the next few weeks, Sarah and I will be working to complete the docs migration to docs.asciidoctor.org. While we do plan to create a custom UI, we can use the default UI with these customizations in the interim.

Nope, no problem. I had just gotten sidetracked. I'll proceed with merging. Thanks for submitting these modifications.
Over the next few weeks, Sarah and I will be working to complete the docs migration to docs.asciidoctor.org. While we do plan to create a custom UI, we can use the default UI with these customizations in the interim.

Thanks Dan :smile:

I think that due to GitHub recent failures, the build was not triggered on Netlify.
I manually trigger a deployment and we are now live! :tada:


decreased tooth number

decreased tooth number
DEF: reduction in the number of the bony structures of the upper and lower jaws used in mastication
EXACT SYN: reduced tooth number
EXACT SYN: reduced number of teeth
EXACT SYN: missing teeth
NARROW SYN: hypodontia
NARROW SYN: oligodontia
decreased incisor number
DEF: reduction in the number of the long teeth, normally consisting of two pairs, top and bottom, that are the most anterior and prominent in the jaw
SYN: reduced incisor number
SYN: reduced number of incisors
absent teeth (add new term, absent teeth MP:0000124 will be obsoleted because of incorrect definition)
DEF; absence of all of the bony structures of the upper and lower jaws used in mastication
NEW SYN: complete tooth agenesis
NEW SYN: complete dental agenesis
NEW SYN: toothless
NEW SYN: anodontia
NEW SYN: tooth aplasia
NEW SYN: edentulous

Added:
decreased tooth number MP:0030608
decreased incisor number MP:0030609
absent teeth MP:00306010


Fix Farsi Hyperlinks

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

This pull request aims to fix some HTML-Proofer errors that are caused by a small number of outdated hyperlinks found inside the Farsi translations directory. Oddly enough, they randomly started showing up as I was working on #2791. I noticed that those hyperlinks exist in the main branch as well (and they don't work as expected on the main website) which prompted me to create this pull request.

#a-checklist-before-you-contribute

Why??

Bnb

BNB/#+999@Henrytech2+98$/💯 2754bnB

GoMYwALlEt99HeNBIa100999R2Y..  BnB code of my day

Am ready to Marge with anyone from Bnb

Go Bnb

Please, you have to explain me. I don´t understand. so sorry
so 9. 4. 2022 v 17:07 odesílatel Henrytech22 @.***>
napsal:

Go Bnb
—
Reply to this email directly, view it on GitHub
https://github.com/github/opensource.guide/pull/2792#issuecomment-1094062541,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQM4U7CP3QVTQ6OLVBG4URDVEGMKJANCNFSM5SUGF5RA
.
 Missed this PR until I'd done this myself manually 🤦🏻 . Thanks anyway @ChristolisOfficial!


How do I compile this?

I'm sorry this might be a very novice questions for most people, but how do I set up those folders to compile? The Readme doesn't specify anything, if I just open the terminal and type catkin_make --pkg astra_camera it obviously won't work.
Do I just place eveything inside ~/catkin_ws/src/ros_astra_camera and compile using the  catkin_make --pkg astra_camera (or would I put it inside ~/catkin_ws/src/astra_camera? it seems the name of the project is ros_astra_camera but the first command to compile the package is astra_camera). Do I also need to install any drivers? How?
Thank you

Do I just place eveything inside ~/catkin_ws/src/ros_astra_camera and compile using the catkin_make --pkg astra_camera (or would I put it inside ~/catkin_ws/src/astra_camera?

Yes.
navigate to your catkin_ws folder and

open the terminal and type catkin_make --pkg astra_camera

If you look into package.xml , the package name is astra_camera. The name of the folder you put it in doesn't matter.

Were you able to successfully run this package with astra_launch? After running the catkin_make command, when I run roscd astra_camera, it says "roscd: No such package/stack 'astra_camera'".  What could I be doing wrong? Thanks

Just worked. Thanks @parthc-rob .


MessageDigestPasswordEncoder is no longer consired secure by Spring team

As you can read on https://docs.spring.io/spring-security/site/docs/5.6.0/api/org/springframework/security/crypto/password/MessageDigestPasswordEncoder.html, the use of MessageDigestPasswordEncoder is deprecated and no longer considered secure.
Please consider updating the article.

Hi @pythonmeister ,
Thanks for the heads up. We'll take care of this.
I assume you're referring to the https://www.baeldung.com/java-password-hashing article.

Yes, that is the article I'm referring to.

We've updated the article.

Thanks for taking care :-)


Add recent list to FindReplaceDialog

Previous dialog had a separate button to show this list. Instead, use an editable DropDown control.
There is probably no need to persist this list between sessions.
Related to #36.

Fixed in 442b1ca5.


fix support for osx

This PR adds the minimum amount of changes needed to make OSX applications work again. It's not the most elegant code, but let's start small.
Summary of changes:

Recover utils.get_executable from the original appinst
Fix class signatures
Conform to the menuinst.install(...) API expected by conda (same as windows)
Comply with the modern plistlib API
Slugify script name (last component of/Applications/${NAME}.app/Contents/MacOS/${NAME})
Bring some placeholders used on Windows too

I'd just like to say that it would be great to have these changes added to menuinst. If there's any way in which I could help make that happen, please do let me know.

This looks good to me. Can we enable tests for this so that we don't accidentally break this in the future?

Thanks @isuruf! I don't know if we want to pursue this series of updates though (this PR, and the one for Linux).
There's a full rewrite on #91 that implements the same functionality with a new cross-platform schema. It could be considered a new major version, though.
Enabling non-Windows also requires changes in conda itself.
I think we should close this PR, and #89, and work our way towards cross-platform compatibility through #91.

I think we should close this PR, and #89, and work our way towards cross-platform compatibility through #91.

I agree!


Use r-lib/actions@v2 for installing and caching the dependencies

Use r-lib/actions/setup-r-dependencies@v2 for installing and caching the dependencies
Remove RSPM as it's now automatically used internally in r-lib/actions/setup-r@v2 (if we want to stop this, we can set use-public-rspm to false)
Remove considerations on the case of R < 4.0 because it's been a while since R 4.0 was released.
Set a version on actions/upload-artifact
Use actions/checkout to checkout extendr/rextendr

Thanks!


[SPARK-6562][SQL] DataFrame.replace

TODO:
Python support

Test build #29445 has started for   PR 5282 at commit 06f2c63.

Test build #29445 has finished for   PR 5282 at commit 06f2c63.

This patch passes all tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

case class AtLeastNNonNulls(n: Int, children: Seq[Expression]) extends Predicate

This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29445/
Test PASSed.

Test build #29690 has started for   PR 5282 at commit 84e935f.

Test build #29690 has finished for   PR 5282 at commit 84e935f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29690/
Test PASSed.

Test build #30141 has started for   PR 5282 at commit 4b72434.

Test build #30141 has finished for   PR 5282 at commit 4b72434.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30141/
Test PASSed.

I'm going to merge this now. The resolve thing needs a broader (df-wide) discussion. That should happen but outside of this change.


ARC: tests/kernel/critical/kernel.common times out on nsim_em and nsim_sem

Test case just hangs on nsim_em and nsim_sem simulator:
***** Booting Zephyr OS v1.14.0-rc1-1250-g4e097b4d541f *****
Running test suite kernel_critical
===================================================================
starting test - test_critical

It looks like the test eventually completes, but it takes a long time, longer than the allowed timeout in sanitycheck for the test.
On qemu_x86, the test takes about 16 seconds.
On nsim_em, the test takes about 105 seconds on my machine.

This may be fixed by #14558, where i used 10 Mhz  host timer as timer resource.
If the timer is accurate,  the critical test will run 1500-1600 ms.

this was already fixed by #14558


What is the best way to define a model with nested properties?

What is the best way to define a model with nested properties?
I know that I can reference other models in my props but I was wondering if I can simply define nested properties inside a single class?

Same question. Anyone knows how to define model with nested properties...?


Wistia support

Is it possible to download videos, which are hosted by Wistia? Do you plan to support it in the future?

Yeah, I am working on the this. The APIs are changed, so it will require to
rewrite some parts.
On Thu, Nov 2, 2023, 1:02 PM Akos Kovacs @.***> wrote:

Is it possible to download videos, which are hosted by Wistia? Do you plan
to support it in the future?
—
Reply to this email directly, view it on GitHub
https://github.com/sumeetweb/Thinki-Downloader/issues/50, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF734XXZU5YKK4NGGVAEMSLYCNECDAVCNFSM6AAAAAA62NVQAWVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE3TGNRXGA2DOMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


dashboard/config: switch to ORC unwinder

Jann pointed out that the frame pointer unwinder fails to unwind double
fault stacks. Switch to using the ORC unwinder instead.
https://www.kernel.org/doc/html/latest/x86/orc-unwinder.html

(@ramosian-glider KMSAN config will need to be regenerated once this is merged.)

Codecov Report

Merging #1583 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1583   +/-   ##
=======================================
  Coverage   56.91%   56.91%
=======================================
  Files         144      144
  Lines       26856    26856
=======================================
  Hits        15285    15285
  Misses      10791    10791
  Partials      780      780

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 93e5e33...02a2951. Read the comment docs.

@thejh
The last time I tried ORC, it did not work, caused random KASAN reports all over the place or something. It was a year ago or so, but ORC was already the default back then.
Has anybody tried to run syzkaller with ORC today?
ORC also does not work with KMSAN:
https://github.com/google/kmsan/issues/48
So even if it works with KASAN, we need to update the KMSAN config script to switch back to FRAME_POINTER.
We also need to figure out what to do with bisection. If we switch to ORC now, syzbot will start doing bisections with ORC back in history. At some point ORC is not present and I assume kconfig will switch back to FRAME_POINTER (or to GUESS? we don't want GUESS). But most likely there is a period when ORC was present but was all broken (even when it was default, but I assume there was a period when it was present but not default precisely because it wasn't working yet). We need some cut off point for ORC and switch back to FRAME_POINTER in pkg/vcs/linux.go.

Ah, I don't know about that - I've never actually run syzkaller myself.

Has anybody tried to run syzkaller with ORC today?

The USB fuzzing instance has been running with ORC just fine.

So even if it works with KASAN, we need to update the KMSAN config script to switch back to FRAME_POINTER.

Done.

Has anybody tried to run syzkaller with ORC today?

The USB fuzzing instance has been running with ORC just fine.

OK, let's assume it works now.

So even if it works with KASAN, we need to update the KMSAN config script to switch back to FRAME_POINTER.

Done.

We need something for bisection too. If we merge this, it may break silently and corrupt dozens of bisections until we notice.

I've added switching to FRAME_POINTER for bisection. PTAL.


Items doesn't seem to have the required tags for reforge

Minecraft Version
1.19.x
Mod Loader Version
1.14.19
Mod Version
1.2.7
Describe the Issue
Issue : Can't reforge stuff because the required items don't have the right tags
I have been trying to reforge some of my items, but I couldn't put the amethyst shard inside the right slot on the reforge tab.
After a bit of research, I found on your page that the Amethyst Shard and the required ingots should have the right tags in order to be used in the reforge.
Weirdly enough, Flint seems to have the right tag, but the amethyst and my ingots doesn't. Is there a way to fix it in some config files by myself ?
Thank you in advance and have a beautiful week. I'll upload some screenshots to make the subject clearer.

latest.log
latest.log
I can also give you the server logs if needed.
Do you use any other mods except the required ones?
Another Quality Modpack 2 (1.5.2)

Also, I realized it's probably irrelevant to ask you instead of the modpack creator because it surely is not your fault if the tags are breaking.
You can close the issue if you think I shouldn't post this here, sorry for the disturbance.

I found out what the right item was for the amethyst slot.
Closing the issue, because I think it comes from my modpack.

Glad you figured it out :)


Update MS docs: Spring on Azure document

As title.
Please refer to parent issue(#95) to get more information.

Closing this issue, because contents already contained in this PR: MicrosoftDocs/azure-dev-docs-pr#2487


Unity crash - Example 34

Unity version - 5.4.0b24 personal - Windows
Unity crashes on opening GUI example - 034_Controls_InteractingWithUnityUI

This is an issue with unity beta b24, it seems to work ok on beta b20.
I'm assuming it's something Unity need to address, as any piece of code in the toolkit should not crash the Unity application.


feature: Finer control over filtered LSP messages

Did you check the docs?

[X] I have read all the noice.nvim docs

Is your feature request related to a problem? Please describe.
I find mini LSP notifications useful, but I would like to filter out some clients or some particular messages while still using the UI that this plugin provides.
Describe the solution you'd like
Extra options in lsp.progress or more options in routes.
Describe alternatives you've considered
I tried using the snippet given in the wiki, but this results in a bunch of unformatted notifications (I want to keep the formatting from the mini view):

The other issue is that such snippet wouldn't allow me to continue using the mini view to display these filtered notifications...
Additional context
That being said, I love this plugin! <3

You can use something like the below:
    table.insert(opts.routes, {
      filter = {
        event = "notify",
        find = "No information available",
      },
      opts = { skip = true },
    })

Please open a discussion instead of an issue next time for questions.

You can skip specific lsp progress messages like this:
    table.insert(opts.routes, {
      filter = {
        event = "lsp",
        kind = "progress",
        find = "workspace", -- skip all lsp progress containing the word workspace
      },
      opts = { skip = true },
    })

You can skip specific lsp progress messages like this:

    table.insert(opts.routes, {

      filter = {

        event = "lsp",

        kind = "progress",

        find = "workspace", -- skip all lsp progress containing the word workspace

      },

      opts = { skip = true },

    })

Thanks for the quick response! However, I'm the most interested in filtering by client, which I haven't been able to do because it seems that the highlights prevent the string match (?)

Ohh, is this because the client name isn't part of the message's content but part of the title? @folke

To exclude a client you can use:
    table.insert(opts.routes, {
      filter = {
        event = "lsp",
        kind = "progress",
        cond = function(message)
          local client = vim.tbl_get(message.opts, "progress", "client")
          return client == "lua_ls" -- skip lua-ls progress
        end,
      },
      opts = { skip = true },
    })

That worked perfectly, thanks a lot @folke!
Is this filter setting documented? I couldn't find it...

Also thanks again for all your amazing plugins. I truly admire your work!

Noice is a bit too configurable tbh. Even I need to dive in the code all the time to check how I can achieve something :)
There's definitely a lot of documentation missing as well.
And thank you for the kind words!

This was super helpful. I bet you did it because of null_ls. That thing is too noisy when it comes to these progress messages.

This was super helpful. I bet you did it because of null_ls. That thing is too noisy when it comes to these progress messages.

Yep. I also find the "code action" messages a bit too noisy since I'm using the gitsigns builtin and there's a code action for blaming a line, which is applicable to pretty much any line lol.


Autosuggest and table cells (insider build v1.5.4)

As of the v1.5.4 insider build, autosuggest no longer works in the context of editing a cell with the live-preview table editor. As seen below, the dialog only shows up for a split second.

This is a bug in Obsidian v1.5.4. It will be fixed in the upcoming 1.5.5


feat(components, app, labware-library): migrate to react-router-dom v6.24.1

closes AUTH-564
Overview
This PR updates react-router-dom to v6.24.1 in the stack to match PD's react-router-dom version
Big changes between v5 and v6:

useHistory is deprecated and replaced with useNavigate. Instead of history.push('path') and history.goBack() it is navigate('path') and navigate(-1)
Route no longer has children, instead the component goes into the element prop.
Switch is deprecated and is now Routes
Route must be wrapped in Routes (should mainly have to worry about this in unit testing since the app components are wrapped in Routes already from the top level)
<Redirect/> is deprecated and is now <Navigate/> (if it is wrapped in Routes then you may use Route and Navigate goes into the element prop)

Test Plan
Smoke test labware-library, protocol designer but most importantly, smoke test the app and ODD.
Changelog

update react-router-dom in Labware-library and app
deprecate it in components
fix all affected components and tests

Review requests
see test plan
Risk assessment
high since we have a 8.0.0 release and this touches a bunch of code. we need to smoke test well before merging

the migration guide would be useful to review this pr
https://reactrouter.com/en/main/upgrading/v5

For ODD, we would need to check the followings
unboxing flow [without software update]
welcome -> network setup -> software update -> e-stop screen -> name robot -> finish setup -> robot dashboard with modal
unboxing flow [with software update]
welcome -> network setup -> software update -> reboot the robot -> e-stop screen -> name robot -> finish setup -> robot dashboard with modal
Navigation item after unboxing flow

protocols
quick transfer
instruments
settings

Robot settings Dashboard

network setup flow
rename flow

making my way through this PR but chiming in to say that i can devote time early next week to test this on a bot!


Implement a prototype for the generalized-alpha solver

We would like to implement a prototype version of the generalized-alpha time integration scheme into OpenTurbine as part of the proof-of-concept (POC) study to model the dynamics of a rigid pendulum. To achieve this, we want to implement the algorithm presented in the paper by Arnold and Bruls. A basic description of the generalized-alpha method and the algorithm for constrained, nonlinear structural dynamics problems is provided in the Overleaf document for the POC (Section 5.1).

@faisal-bhuiyan -- Overall, the generalized alpha scheme looks fine.  Can I suggest, though, that we you implement the Newton iterations in terms of incremental accelerations rather than incremental displacements?  I find this a more natural choice for structural dynamics problems and the Jacobians needed; this is also what we are using in the tight coupling algorithm being implemented now in OpenFAST.
In general, supporting algebraic constraints as proposed proposed looks fine.  But I would encourage the use of relative coordinates and algebraic elimination of constraints in the multi-body formulation of OpenTurbine as much as possible. The solution will be much more computationally efficient with fewer algebraic constraints.

@jjonkman Thanks for the comments and suggestion, and we will keep them under consideration.  At least for this proof-of-concept problem, we are interested in testing the displacement-based DAE-3 formulation, which is appealing for its simplicity.


Once JSON language mode sees an error, it never lets go

Issue Type: Bug
I create foo.json with  these contents:
{
    "title": "Tutorial 2",
    "ranks": [
        {
            "type": "Input",
            "id": "InputA"
        },
        {
            "type": "Input",
            "id": "InputB"
        }
    ]
}

I want to add a new "rank", so I type a comma after the curly brace that follows InputB. It's immediately highlighted as an error, the filename turns red. At that point, this file shows up in the IDE as having an error. If I then immediately delete the comma I just added, it's not fixed. Nor is it fixed if I add a new rank.
In fact, it seems that there's nothing I can do to fix it short of restarting the editor. From this point on, errors in JSON are just spurious noise; there's no real relationship to the state of the file. Saving the file doesn't help, formatting the file doesn't help, closing the file doesn't help. Renaming it DOES help, but if you then rename it back, the (wrong) errors reappear. The only things I've found are quitting and restarting all of VSCode, or (just discovered) "Developer: restart Extension Host".
Expected result: when the JSON is correct and parseable, it should not show invalid errors in the file, and the filename in the explorer should not indicate errors.

Maybe the error comes from an extension? Does it reproduce with extensions disabled?

I apologize. I had an extension ("TextMate Languages") that was breaking it. I had forgotten I installed it.


Fix Linear Interpolation

Long story short my original linear interpolation was bugged. I had tested things but then made a few mistakes trying to arrange how the extension worked and caused an issue. So here is the fix with a cleanup as well as tests for snapping and linear interpolation.

Very nice!  Love the tests!


Redesign of code

Main purpose:
Shrink the code by eliminating redundancy and by auto-generating menu controls and reusing code and just making the entire thing cleaner and better designed in general
Tasks/goals (many will be broken off into subtasks)

[x] Turn all menu options under Events into generated code.  This will be done in two main parts

[x] Turn menu items into a base set of controls that can be generated from variables, which will be loaded from/saved into a .JSON file
[x] Take the code that is currently used by the menus and turn it into a string that can be run against variables for the same effect as it has now, but without all the redundant copies

Code redesign is finished and uploaded.  Menu system works as it should, and the save and load features do properly save and load the menus

All tasks completed, Closing Task


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=403741538
https://github.com/Azure/azure-cli-extensions/commit/5a2c3eb7fd7010fbd34131df82fa8b6cfdce1d2a

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

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


Allow no token owner in archive db (rampup)

Same as #12841, for rampup.

!ci-build-me

!ci-build-me


Broken Event

Latest commit breaks the 12.41 event. Everything else seems fine.

I’m aware of the issue I will fix tomorrow

Fixed.


Introduce an ante round before preflop

I experimented a bit with changes to GameState to support an ante in addition to the small blind and big blind. The approach in this pull request is to introduce a separate "Ante" round before "Starting" and "Preflop", which is admittedly quite intrusive and impacts a lot of code. Anyway, I am sharing it here in case it is of interest.
A separate commit adjusts the selection of the player first to act after each round (see #102).

Looks good though it needs a format:
 cargo clippy --fix --allow-staged --allow-dirty --all
 cargo fmt --all

Added commit to fix clippy lints and fmt, thanks for the feedback.


doc(#103): Add documentation for RuleAllTestsHaveProductionClass

Add documentation for RuleAllTestsHaveProductionClass .
Closes: #103

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here


testSslNoClientAuthenticationFailsWithMultipleKeyStoreUnspecified() test is flaky

This test makes an assumption about the order in which a key store returns the keys that it contains. Should rewrite or remove this test since it looks like this can be system-dependent.

Fixed by #51


Some features show information from the wrong/previous repository visited

Description

repo-header-info show invalid repository's stargazer-count.

How to replicate the issue + URL

Clear cache of refined-github
Go to https://github.com/refined-github/refined-github
Go to it's organization with Left top's breadcrumb

Go to other repository
Check your repo-header-info stargazer-count

Extension version
23.10.5
Browser(s) used
Chrome

The reason is CachedFunction's cacheKey is initialized with refined-github/refined-github, and visiting other repository with out script reloading (soft-rendering), CachedFunction still has cacheKey to refined-github/refined-github
https://github.com/refined-github/refined-github/blob/main/source/features/repo-header-info.tsx#L20-L28
There can be same cache issues which initializes CachedFunction like this case.

That cacheKey is dynamic, it depends on location.href
Maybe the issue is that it's fired at the wrong time due to:

https://github.com/refined-github/refined-github/issues/6437

That cacheKey is dynamic, it depends on location.href
Maybe the issue is that it's fired at the wrong time due to:

https://github.com/refined-github/refined-github/issues/6437

Yes. I've checked cacheKey is correctlly updated on feature init callback.
But, the cacheKey is passed to CachedFunction's constuctor as value, cannot apply location.href dynamically, when soft-rendering cases.

I have a repro for another feature, default-branch-button:

Clear cache
Open this in a new tab https://github.com/sindresorhus/p-retry/releases/tag/v6.1.0
Click "separate package"
Open a file or folder

https://github.com/refined-github/refined-github/assets/1402241/e28c4aa8-4540-4c66-af4d-4776a6bb14c4

Oof, it doesn't simply mess up the current view, but it also messes up the cache. I thought this was just a few issue, but for some reason it takes the previous value and stores it on the current URL.
My guess is that getRepo is called at different times, or that webext-storage-cache is memoizing the function call incorrectly.

Is this really just a cache / memoization issue?

Did you clear the cache? I see 507

I disable and re-enabled the extension, that didn't help. Now I also tried clearing cookies and other data from the URL header in Firefox (after which I had to login again). The count is still at 2.1k.

I disable and re-enabled the extension, that didn't help

There's a "Clear cache" button in the extension's options page, not the browser's

I didn't know about that. Now it just fails to show the header info entirely, even after navigating around a bit.

Not sure, but as you said, It appeares frequently and commonly on other features also.

The issue is:

https://github.com/refined-github/refined-github/issues/6637
Memoization
specifically api.v4


Fix GnuTLS error -110: The TLS connection was non-properly terminated

Fixes GnuTLS error -110: The TLS connection was non-properly terminated issue

See #116 for fix

Great fix!
We can simplify this since .end() returns the socket.
this.dataSocket.end().destroy();

If you can make this change and rename the commit message to follow the Angular commit guidelines per the Contribution guide, eg: fix(connector): gracefully end data socket

Thanks @trs. I'll have to start using those commit conventions more. Thanks for the feedback

For sure, the reason I use it is that it allows for automatic versioning, tagging, and releasing with semantic-release. Releases are easy and automated with CI.


Can you delete subboards?

Can't seem to figure out how 🤔

Yes, the delete button in the ribbon menu allows you to delete a (sub)board. If you have suggestions on where it could be placed to be more useful I'd be happy to hear!

oh... I feel silly now. I kinda assumed that would delete my currently selected nodes or something.
This is probably good placement, although one idea I have had is to have the delete button inside the subboard switcher dropdown.

I like this idea. I'll give that a whirl.


Can't define a namedExports

Hey there! I'm trying to set a namedExports in a Lerna project as follow:
commonjs({
  include: '**/*',
  namedExports: {
    'packages/button/node_modules/classnames/index.js': ['classnames']
  }
}),

I have tried other keys such as: classnames, node_modules/classnames/index.js, ./node_modules/classnames/index.js without success... in all cases I'm getting the following error:
λ ~/Workspace/react-spectre/ master* yarn build -- --verbose
yarn build v0.24.6
$ cross-env NODE_ENV=production rollup -c --verbose

/a/b/c/react-spectre/packages/button/src/index.js → packages/button/dist/button.umd.js...
[!] Error: 'default' is not exported by packages/button/node_modules/classnames/index.js
https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module
packages/button/src/Button.js (3:7)
1: import React from 'react'
2: import PropTypes from 'prop-types'
3: import classnames from 'classnames'
          ^

Same issue with 8.3.0

Hello,
I got same problem I think

import React from 'react';

...
/** non of this options work, 1st should be more correct */
import createHashHistory from 'history/lib/createHashHistory';
import { createHashHistory } from 'history/createBrowserHistory';
...

Dependency: https://github.com/ReactTraining/history/releases/tag/v1.17.0
I can't share project code, because or non-disclosure
const rollup = require('rollup-stream'); //  ^1.24.1 + "rollup": "^0.56.0",
const jsonify = require('rollup-plugin-json'); // ^2.3.0
const cjs = require('rollup-plugin-commonjs'); // ^8.3.0
const npm = require('rollup-plugin-node-resolve'); // ^3.0.3
const babelify = require('rollup-plugin-babel'); // ^3.0.3

                    input: entry,
                    format: 'iife',
                    external: ['react-dom'],
                    sourcemap: true,
                    plugins: [
                        jsonify(),
                        babelify({
                            babelrc: false,
                            exclude: 'node_modules/**',
                            presets: [
                                [
                                    'es2015',
                                    {
                                        modules: false
                                    }
                                ],
                                'stage-0',
                                'react'
                            ],
                            plugins: ['external-helpers']
                        }),
                        npm({
                            jsnext: true
                        }),
                        cjs({
                            exclude: [
                                '**/*.css'
                            ],
                            include: [
                                'node_modules/create-react-class/**',
                                'node_modules/fbjs/**',
                                'node_modules/object-assign/**',
                                'node_modules/react/**',
                                'node_modules/react-dom/**',
                                'node_modules/prop-types/**'
                            ]
                        }),
                    ]

                            include: [
/** by adding this >>> */ '**/*',  /** solved issue */
                                'node_modules/create-react-class/**',
                                'node_modules/fbjs/**',
                                'node_modules/object-assign/**',
                                'node_modules/react/**',
                                'node_modules/react-dom/**',
                                'node_modules/prop-types/**'
                            ]

Hey folks (this is a canned reply, but we mean it!). Thanks to everyone who participated in this issue. We're getting ready to move this plugin to a new home at https://github.com/rollup/plugins, and we have to do some spring cleaning of the issues to make that happen. We're going to close this one, but it doesn't mean that it's not still valid. We've got some time yet before the move while we resolve pending Pull Requests, so if this issue is still relevant, please @ me and I'll make sure it gets transferred to the new repo. :beer:


Ec slope hint

Integration test?

I just try but there is still unimplemented hints to do an integration test

Codecov Report

Merging #257 (9831f40) into main (061c955) will decrease coverage by 16.61%.
Report is 45 commits behind head on main.
The diff coverage is 48.35%.

@@             Coverage Diff             @@
##             main     #257       +/-   ##
===========================================
- Coverage   75.29%   58.68%   -16.61%
===========================================
  Files          20       41       +21
  Lines        2040     5204     +3164
===========================================
+ Hits         1536     3054     +1518
- Misses        453     1896     +1443
- Partials       51      254      +203

Files Changed
Coverage Δ

pkg/builtins/builtin_runner.go
0.00% <0.00%> (ø)

pkg/hints/hint_utils/hint_reference.go
99.37% <ø> (ø)

pkg/hints/hint_utils/secp_utils.go
0.00% <0.00%> (ø)

pkg/parser/parser.go
57.89% <0.00%> (ø)

pkg/types/exec_scope_utils.go
0.00% <0.00%> (ø)

pkg/utils/math_utils.go
0.00% <0.00%> (ø)

pkg/builtins/poseidon.go
30.28% <4.00%> (-58.36%)
:arrow_down:

pkg/builtins/signature.go
5.44% <5.44%> (ø)

pkg/vm/program.go
17.94% <14.28%> (-3.27%)
:arrow_down:

pkg/builtins/pedersen.go
19.72% <19.72%> (ø)

... and 28 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

A general comment for the whole PR, let's be consistent with the naming conventions, like using camel case and shorter names when applicable (for example, vm instead of virtual_machine or virtualMachine).

other than the namings, PR looks good 👍


[Bug] [Blazor Webassembly] System.Diagnostics.Process is not supported on this platform

When I try to start a workflow, an exception occurs as mentioned in the subject.
The error is in line 41 of TemporalConnection.cs:
options.Identity ??= System.Diagnostics.Process.GetCurrentProcess().Id
Please use the System.Environment.ProcessId method instead, it works under Webassembly and also more efficient:
https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1837

Please use the System.Environment.ProcessId method instead

That is not supported across all .NET versions this library supports. We could use compiler directives to use this newer approach with newer .NET versions, but you're going to hit other WASM issues with this SDK.

it works under Webassembly and also more efficient:

We use a Rust-based core for this library and therefore it won't be Blazor/WASM compatible anyways.

(closing as question, but feel free to respond or join us on Slack or our forums)


Add a tab for PackageReference on the package details page

In addition to the existing "Package Manager", ".NET CLI" and "Paket CLI" tabs, I'd like to see one for PackageReference:

That would make it really easy to copy a PackageReference XML tag for a specific version and paste it directly into your project file.
Questions that needs answers:

What should the name of the tab be?
Where would the new tab be placed in relation to the others? What's the ordering?

@khellang I like the idea but I am not sure this is the recommended flow for users (especially beginners). Some of the drawbacks of directly editing the project files:

For Legacy PackageReference projects like WPF, WinForms etc., one will need to unload the project, then edit the project file to add this there. Its not as simple as .NET Core based projects
As you suggested, PrivateAssets and other tags that tooling will automatically bring to the PackageReference node based on some metadata on packages. Something not so simple to know with this proposed flow.
Where exactly in the project file to put this PackageReference node for somebody adding it for the first time? Though not so hard for advanced users but for beginners this option may cause more confusion.

In addition, I am not sure if we will confuse users working with packages.config with this option. Obviously .NET CLI too doesn't work with packages.config but using the tool will error out while putting this inside the project file for packages.config may result in weird cases.

I am not sure this is the recommended flow for users (especially beginners)

I agree. I'd much rather have a nice UI to do this. Unfortunately, the NuGet UI in Visual Studio is unbearably slow and stuttery. Editing the project file is so much faster.
Sure, it might be a bit confusing the first time for beginners, but does that mean we can't provide convenience for more experienced users? I seriously think you're underestimating the average .NET developer.
If this isn't a recommended flow, why do you provide documentation for it? Adding a PackageReference is literally the first point in the docs, which this tab would link to.

For Legacy PackageReference projects like WPF, WinForms etc., one will need to unload the project, then edit the project file to add this there. Its not as simple as .NET Core based projects

Yes, I'm painfully aware of this, as I do it on a weekly basis. Still, it doesn't mean that this won't be helpful. It'll still save me valuable time compared to typing it out.

As you suggested, PrivateAssets and other tags that tooling will automatically bring to the PackageReference node based on some metadata on packages.

I still think a best effort is better than nothing here.

Where exactly in the project file to put this PackageReference node for somebody adding it for the first time?

This is already detailed in the docs. And chances are that there's already an ItemGroup with package references in there already.

In addition, I am not sure if we will confuse users working with packages.config with this option. Obviously .NET CLI too doesn't work with packages.config but using the tool will error out while putting this inside the project file for packages.config may result in weird cases.

Shouldn't the Visual Studio/MSBuild tooling also error out instead of "resulting in weird cases"? I'd argue this should be the case regardless of this tab in the gallery.

I do not disagree on the usefulness of this option for advanced users who already know about PackageReference and this option will definitely save some time. My only concern is for beginners who I feel get the most out of these options. And this option may be a distraction for them.
I definitely want this option to be documented - today its missing on the doc topic: Ways to Install a package

Devaju/market research over at https://twitter.com/Nick_Craver/status/987658040819273729?s=09

In the world of cross-platform, multi targeting, dotnet core and the CPS. I rarely use Visual Studio in any form. I always use sublime, vim or other things when  updating packages references because ReactiveUI uses the directory.props convention. Even if tooling was built, I still wouldn't use it. This has value.

I'm often on a package's NuGet.org page when I want to add it to a project, with the .csproj text editor open in VS Code or Visual Studio. Anything other than allowing copy and paste is going to be making me jump through less-fun hoops. :D

Hey @anangaur, what other angles should we consider here? The PR has been open over a year now.

Merge it!!

I kept it at abeyance because we were thinking around Central Package Version Management that will not have a version with PackageReference. But lets cross the bridge when we come to it.

Landed in d157ea7

@khellang , we will close this after it's deployed to nuget.org :)

Oh, sorry 😅

This is now live! Thanks for the contribution @khellang!


Platform annotations on Process struct

Signed-off-by: John Howard<EMAIL_ADDRESS>Extracting pieces from the proof of concept PR for Windows OCI support at https://github.com/opencontainers/runtime-spec/pull/504. This PR tidies up the Process structure by putting annotations for platform support on Capabilities, Rlimits and NoNewPrivileges, none of which are used on Windows. In addition, it removes what appears to be an erroneous blank line that appeared to serve no purpose.
Note I'm assuming (probably incorrectly :innocent:) , that Rlimits and NoNewPrivileges are also relevant on Solaris too, but I'm not sure. If that is incorrect, I will update the PR to remove the solaris annotation.

On Wed, Sep 14, 2016 at 01:51:28PM -0700, John Howard wrote:

// Capabilities are Linux capabilities that are kept for the container.

// Capabilities are Linux capabilities that are kept for the container. (this field is platform dependent)
Capabilities []string json:"capabilities,omitempty" platform:"linux"

Consistency. It's all over this file whenever a platform annotation is present.

+1 for consistency.  How do folks feel about consistently removing
this from the comment when a platform field exists?  And there are
fields like mounts[]options with different rules depending on the
platform, but I expect that folks will recognize that without having a
Go comment here to remind them.  Are people really reading the Go
comments and skipping the Markdown spec?

@wking - Yes, it all does seem more than a tad superfluous IMO. But that said, it's still a little orthogonal to this PR itself :innocent:

On Wed, Sep 14, 2016 at 02:44:53PM -0700, John Howard wrote:

@wking - Yes, it all does seem more than a tad superfluous IMO. But
that said, it's still a little orthogonal to this PR itself
:innocent:

This PR does two things (as of 103926ff):
a. Add ‘solaris’ to some ‘platform’ tags.
b. Add “(this field is platform dependent)” to some Go comments.
I think we want to drop (a) 1, but am fine waiting on Solaris-dev
feedback there.  And I think we want to go the other way with (b).
If this PR is just about (a), I'm happy to file my other-(b) as a
separate PR.

@wking - Let's try and keep these two things independent, and handle the go comments as a separate issue/PR. I think that will be cleaner in the long run.

On Wed, Sep 14, 2016 at 04:11:27PM -0700, John Howard wrote:

@wking - Let's try and keep these two things independent, and handle
the go comments as a separate issue/PR. I think that will be cleaner
in the long run.

Sounds good.  I spun off the comment removal into #568.

LGTM

(side note: we ought to define these tags somewhere)

On Thu, Sep 15, 2016 at 07:28:19AM -0700, Vincent Batts wrote:

LGTM

Really?  Solaris does use these fields?
And I agree we should define the tags.  We already have the
enum here 1, just need to link it with the JSON tags.

@wking chill.
Solaris supports more of the Linux API than I realize. I'll leave it up to folks like @jlb13 to correct me.

@vbatts @wking Removed the annotation for solaris

On Thu, Sep 15, 2016 at 10:25:59AM -0700, John Howard wrote:

@vbatts @wking Removed the annotation for solaris

ce3ac33d7 looks good to me :).

LGTM

What's the process for full approval and merging in this repo? There's two LGTMs above, but I'm not sure if more are still required - it appears it does from above, but I'm not sure why. Forgive my ignorance 😇

@jhowardmsft We have a .pullapprove.yml which set reset_on_push: true and reviewers: required: 2, so after you force pushed, we still need two LGTMs before merging.

Ah, I see. Thanks for confirming :smile:

LGTM


label3 PR merged then reverted

@sanjay-kv
https://github.com/HimanshuNarware/CareerZunction_Intern/issues/398
this is the issue where PR got merged bt then reverted

rasie pr


[config] Make IP filter disableable in an effort to reduce cost.

Part of https://artsyproduct.atlassian.net/browse/PLATFORM-1835
This flamegraph generated by clinic.js seems to suggest that in a prod build of the server code a large amount of time is spent in the express-ipfilter middleware. I don’t really feel like I can trust this tool yet, as I'm struggling trying to get things to run without too much noise (e.g. from things like webpack), but figured it was worth a try nonetheless.
After deploying we should remove the IP_DENYLIST from production and restart the service.

👍 sounds good. We prefer Cloudflare for this sort of operation these days anyway, so maybe it's fine to remove express-ipfilter altogether (after porting the currently blocked IPs to the similar firewall rule)?

cc @zephraph

Let’s first see if it works, then next week we can decide to move it elsewhere?

@alloy my only concern (if we just deploy this and unset the config) is we'd be subject to abuse from those known-bad sources.

Merging as discussed elsewhere. I've gone ahead and copied the current list into a Cloudflare challenge rule so you should feel free to unset this whenever you want to monitor.


Separate build script for git4mps from the review plugin

This change is to separate out the git4mps plugin from the main code reviewer plugin.
I've also change the output from zip file to the unzipped form and let gradle build script take care of the zipping step.
One thing I noticed is that .ds_store keep getting zipped even though gradle zip task should by default ignore them (or maybe I am wrong), I need some hints to do this part correctly. Currently MPS complains when trying to install the plugins because of the .ds_store files.
This is still work in progress, but I'd like to get some early feedback is possible.

Everything works now.
Gradle now zips the plugins correctly and I've pulled out the build solution into its own plugin so people don't need to depend on mbeddr when using git4mps.
MPS no longer complains when installing these plugins.


Pagination: Sorting

I've flown around the documentation and noticed that there isn't enough of a note about setting up your api to serve sorting in pagination. Mind you, I'm using Laravel 5.4 and paginate().
Am I missing something?

If you use something like fractal with the serializers, the default serializer works mostly works, I think just some of the default names are not quite right. The array serializer is what I typically use. Though if not using fractal you would need to map it out yourself.
<?php

namespace App\Serializers;

use League\Fractal\Pagination\PaginatorInterface;
use League\Fractal\Serializer\ArraySerializer as BaseArraySerializer;
use Illuminate\Contracts\Pagination\LengthAwarePaginator;

class LaravelArraySerializer extends BaseArraySerializer
{
    public function paginator(PaginatorInterface $paginator)
    {
        $currentPage = (int) $paginator->getCurrentPage();
        $lastPage = (int) $paginator->getLastPage();

        $parentPaginator = $paginator->getPaginator();
        if ($parentPaginator instanceof LengthAwarePaginator) {
            $pagination = [
                'total' => (int) $paginator->getTotal(),
                'count' => (int) $paginator->getCount(),
                'per_page' => (int) $paginator->getPerPage(),
                'current_page' => $currentPage,
                'last_page' => $lastPage,
                'next_page_url' => $parentPaginator->nextPageUrl(),
                'prev_page_url' => $parentPaginator->previousPageUrl(),
                'from' => (int) $parentPaginator->firstItem(),
                'to' => (int) $parentPaginator->lastItem()
            ];
        } else {
            $pagination = [
                'total' => (int) $paginator->getTotal(),
                'count' => (int) $paginator->getCount(),
                'per_page' => (int) $paginator->getPerPage(),
                'current_page' => $currentPage,
                'total_pages' => $lastPage,
            ];
        }

        return ['pagination' => $pagination];
    }
}

https://ratiw.github.io/vuetable-2/#/Data-Format-JSON
Is where you can see the structure that vue-table expect and should be able to help build the responses from the api correctly.

Hey Jordan, thanks for the reply (and especially edit); I read the first part and wen't "huh, did he gloss over sorting?". ;)
Your spot on correct with that, too, because in my search (and failure to find anything), I ended up doing nearly exactly what you did... I'm still a little unsure about the choice of putting a pipe-bound value in the querystring, but this is what we have.

Have you check out this repo? Specifically this section


Not working well with MouseREgion

Describe the bug
onEnter and onExit is call multiple time
To Reproduce
Steps to reproduce the behavior:

Go slowlt to child of MouseRegion
See error

Expected behavior
only call when needed
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser edge
Version latest

@Trung15010802
Are you passing trying to use the controller within the callback function?
Yes?


Upgrade to batik 1.9

Hi,
if possible update to batik 1.9 since 1.8 brings in some vulnerabilties.
mvn org.owasp:dependency-check-maven:aggregate -Dowasp

Best regards
Jörg

Hey, that's an interesting plugin!

Yep. Discovered it today. @work I'll suggest to put it in the CI pipeline.
grtnx -j

Just thought about putting it into my parent pom 👍