Tax being added on zero rated Item

Describe the bug
Tax is being added to an item shown as zero rated on website
To Reproduce
Steps to reproduce the behavior:

Go to 'hadhab.com'
Click on 'Add to cart'
Click the Cart Icon and click checkout
FIll in details and click place order
The Thawani Payment Page will add tax to the product even if its set as zero rate on website

Expected behavior
Tax not to be calculated on zero Rate items.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Microsoft Edge
Version: 116.0.1938.62 (Official build) (64-bit)

Also how can I partially refund customer?

Hello @Hadhab  , as for the issue, I'm looking into it.

Also how can I partially refund customer?

This is can't be done with thawani, you may need to refund the customer fully or manually if partial refund is required.

@Hadhab I tried to replicate the same setting and it seems that you have missconfiguration on your end..
please advice your developer to check if he/she did change something in the plagin or the zero rate tax has a value.
best regards

Only changes mentioned in this issue was applied
https://github.com/PhazeRoOman/thawani-for-woocommerce/issues/80

protected function get_tax_precent($order)
    {
        $get_tax = 0;
        //need the first array
        foreach ($order->get_items('tax') as $item) {
            $get_tax = $item->get_data();
            break;
        }

        return $get_tax['rate_percent'];
    }
    /**
     *
     * create a products payload for the request payload
     *
     * @param int $order_id order id
     *
     * @return array products
     */
    protected function prepare_products($order_id)
    {

        $order_items = wc_get_order($order_id);
        $items = $order_items->get_data()['line_items'];
        // get the tax set of the order
        $tax  = $this->get_tax_precent($order_items);

        $products = [];

        foreach ($items as $item) {
            $product_price  = (float) $item->get_data()['total'];

            if ((float)$tax > 0)
                $unit_price = $this->format_price($product_price + ($product_price * ($tax / 100)));
            else
                $unit_price = $this->format_price($product_price);

            $product_name = $item->get_data()['name'];
            if (strlen($product_name) > 40)
                $product_name = mb_substr($product_name, 0, 30, 'UTF-8') . '...';
            $products[] = [
                'name' => $product_name,
                'unit_amount' => (int)ceil(($unit_price / (int) $item->get_data()['quantity'])),
                'quantity' => $item->get_data()['quantity'],
            ];
        }
        //do the shipping
        $shipping_total = $order_items->get_data()['shipping_total'];
        if ((int) $this->format_price($shipping_total) > 0) {
            $products[] = [
                'name' => 'Shipping',
                'unit_amount' => $this->format_price($shipping_total + ($shipping_total * ($tax / 100))) //changes made to this line,
                'quantity' => 1,
            ];
        }
`

I have put the code above indicating where the changes were made

Hello, I have a problem with Checkout
When the payment process is rejected and shown
(Error processing order completion. Please try again)


Build a new signalOrStart API to replace the signalWithStart API

Is your feature request related to a problem? Please describe.

The signalWithStart is confusing to understand the word with which means a lot
Sometimes people don't really need an additional signal when starting it
Even they need it, it can be included into the start parameter anyway

More importantly, users don't have to deal with the fact that signal method can be invoked before workflow method...Although, this could still happen due to reset. See post: https://medium.com/@qlong/gotchas-about-signalwithstart-in-cadence-temporal-c3783fe1cc2e
Describe the solution you'd like
This should be really easy to build based on the existing signalWithStart API.
Describe alternatives you've considered
NA
Additional context
Some discussion in Slack: https://indeed-pte.slack.com/archives/C02E8TQA01G/p1695229310008899

Thanks for reporting this issue. It seems like you are experiencing confusion with the SignalWithStart method and its terminology, as well as the general concept of signal methods being invoked before the workflow method.
To understand the situation better, I have some questions:

Could you provide an example of how you are using SignalWithStart in your code?
What specific behavior are you observing that you consider undesirable or confusing?

Once I have this context, I can help you find a solution or suggest a workaround.

Thanks for reporting this!
I understand the confusion around the signalWithStart method and its naming.  Temporal is actively working on improving the API for this feature, but in the meantime, here's how you can work around the current API:

Use the startWithSignal method: As shown in the Signal-With-Start documentation, the startWithSignal method provides a cleaner way to start a workflow and send a signal simultaneously.

Consider using startWorkflowAsync and signalAsync separately: You can also start a workflow using startWorkflowAsync and then immediately send a signal using signalAsync for the same workflow.

Let me know if you have any other questions.

Thank you for reporting this issue!
Temporal's signalWithStart API, while powerful, can lead to confusion regarding the order of Signal and Workflow method execution. You've pointed out the potential for invoking the Signal method before the Workflow method.
The signalWithStart API is designed for situations where you want to ensure a Workflow is running, but it's not always the most intuitive approach.
To address this, Temporal is considering an alternative API:

startWithSignal: This API would explicitly start a new Workflow instance and then immediately send the Signal. This would eliminate the ambiguity about the order of execution.

Here are some workarounds for your current situation:

Workflow ID Check: You can check the Workflow ID before sending the Signal. If the Workflow is already running, you can send the Signal using the regular Signal method. Otherwise, you can start a new Workflow and then send the Signal.

Signal-With-Start Logic: You could add a check within the Workflow itself to determine whether it was started with a Signal. If so, it could perform the necessary setup.

We understand the value of clearer APIs and will continue to explore options for improving the workflow execution process.
Thanks for your feedback!


Does not work with h265

Hi @Nidheeshv0312 , We currently have no plans to support h265.

Fair enough.  Do we know how this behaves when the stream is h265, for reference?


Description Request results in HTTP 500 response if resource does not exist on requested connector

If a description request (/admin/api/request/description) is sent to the connector with a requested resource URI that does not exist in the requested connector, then the connector returns a HTTP Status Code 500 with the message.
In the example setup with two connectors the following request to the data-consumer connector:
https://localhost:8080/admin/api/request/description?recipient=https%3A%2F%2Flocalhost%3A8081%2Fapi%2Fids%2Fdata&requestedArtifact=https%3A%2F%2Fw3id.org%2Fidsa%2Fautogen%2FpublicKey%2F78eb73a3-3a2a-4626-a0ff-631ab50a00f9
would give the following response:
Metadata could not be saved: Could not resolve type id 'ids:RejectionMessage' as a subtype of [simple type, class de.fraunhofer.iais.eis.DescriptionResponseMessage]: known type ids = [ids:DescriptionResponseMessage]  at [Source: (String)"{"@context":{"ids":"...
Expected:
Status Code 404 with the rejection message from the requested connector as payload.

Hello,
we will try to reproduce the problem. Which version of the connector are you using?

We were able reproduce the problem. It will be added to the todo list.

As of commit b4082fb7e2155c9c115ae42cc445391508aa1c18 the behaviour of this endpoint has changed. This issue will be closed. If you still consider the current behaviour a problem, feel free to reopen this or another issue.

As of commit b4082fb7e2155c9c115ae42cc445391508aa1c18 the behaviour of this endpoint has changed. This issue will be closed. If you still consider the current behaviour a problem, feel free to reopen this or another issue.


net: TestDialCancel is flaky on linux/arm64-buildlet

http://build.golang.org/log/4bd4487d4aaf77131c30e73075ee875f1087e445
http://build.golang.org/log/b6a2ac3ecf7464fccdf847418f1a7d19c5dd62ee
http://build.golang.org/log/7272fc9849e8f20ccc5139e16d771e3ab0971bba
http://build.golang.org/log/f0bba6d903ad208abb6ccfbd7637179600968749
http://build.golang.org/log/c3fb4e2e81dda126312d9ca67b715f72ec7fec2e

Odd:
--- FAIL: TestDialCancel (0.01s)
	dial_test.go:870: dial error after 0 ticks (5 before cancel sent): dial tcp <IP_ADDRESS>:1234: getsockopt: network is unreachable
FAIL
FAIL	net	1.745s

Why would it return ENETUNREACH, but only sometimes? Amusingly, getsockopt's man page (http://linux.die.net/man/2/getsockopt) doesn't even mention this error.
/cc @ianlancetaylor @davecheney @minux

If you look at the code in netFD.connect in fd_unix.go, you'll see that (most likely) getsockopt is not returning ENETUNREACH.  Instead, getsockopt(SO_ERROR) is succeeding in retrieving the error associated with the socket, and that error is ENETUNREACH.  The error is really coming from connect, and it means that there is no route to the IP address.

I'm just going to disable this test for now. I think that machine (on Linaro) has different routes than we've normally assumed for tests.
For the record,
root@r2-a25-go1:/home/brad.fitzpatrick# ifconfig
eth0      Link encap:Ethernet  HWaddr 00:16:3e:0c:9c:8a
          inet addr:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
          inet6 addr: fe80::216:3eff:fe0c:9c8a/64 Scope:Link
          UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
          RX packets:5151248 errors:0 dropped:0 overruns:0 frame:0
          TX packets:996113 errors:0 dropped:0 overruns:0 carrier:0
          collisions:0 txqueuelen:1000
          RX bytes:5114537899 (5.1 GB)  TX bytes:3534764800 (3.5 GB)

lo        Link encap:Local Loopback
          inet addr:<IP_ADDRESS>  Mask:<IP_ADDRESS>
          inet6 addr: ::1/128 Scope:Host
          UP LOOPBACK RUNNING  MTU:65536  Metric:1
          RX packets:2065172 errors:0 dropped:0 overruns:0 frame:0
          TX packets:2065172 errors:0 dropped:0 overruns:0 carrier:0
          collisions:0 txqueuelen:0
          RX bytes:6848213327 (6.8 GB)  TX bytes:6848213327 (6.8 GB)

lxcbr0    Link encap:Ethernet  HWaddr a6:d1:77:00:04:d2
          inet addr:<IP_ADDRESS>  Bcast:<IP_ADDRESS>  Mask:<IP_ADDRESS>
          inet6 addr: fe80::a4d1:77ff:fe00:4d2/64 Scope:Link
          UP BROADCAST RUNNING MULTICAST  MTU:1500  Metric:1
          RX packets:0 errors:0 dropped:0 overruns:0 frame:0
          TX packets:7 errors:0 dropped:0 overruns:0 carrier:0
          collisions:0 txqueuelen:0
          RX bytes:0 (0.0 B)  TX bytes:570 (570.0 B)

root@r2-a25-go1:/home/brad.fitzpatrick# ip route show
default via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/24 dev lxcbr0  proto kernel  scope link  src <IP_ADDRESS>
<IP_ADDRESS>/16 dev eth0  proto kernel  scope link  src <IP_ADDRESS>

I've started seeing these as well on freebsd-arm-paulzhol:
https://build.golang.org/log/d89169422b2e1c3f4765d7d9093faa56510e21ec
https://build.golang.org/log/6b83f0f1b1ba97c0efb74fae4bb05280c26b8a22
https://build.golang.org/log/664eb5529a95b2b051d874ebc6d4f0412266984e
I'm not sure why it started appearing now. There have been some changes in the environment: switched to a buildlet based builder, upgrade to FreeBSD 11.1 etc. But they don't seem to be related.
For my setup I can track the cause to the router/firewall replying with a TCP RST segment when dialing to the <IP_ADDRESS>/15 subnet:
19:13:12.724990 IP 192.168.X.Y.29436 > <IP_ADDRESS>.1234: Flags [S], seq<PHONE_NUMBER>, win 65535, options [mss 1460,nop,wscale 6,sackOK,TS val 157886 ecr 0], length 0
19:13:12.725786 IP <IP_ADDRESS>.1234 > 192.168.X.Y.29436: Flags [R.], seq 0, ack<PHONE_NUMBER>, win 0, length 0

It is OpenBSD pf firewall's recommended behavior according to https://www.openbsd.org/faq/pf/example1.html:
block in quick on egress from <martians> to any
block return out quick on egress from any to <martians>

Packets coming in on the egress interface should be dropped if they appear to be from the list of unroutable addresses we defined. Such packets were likely sent due to misconfiguration, or possibly as part of a spoofing attack. Similarly, our clients should not attempt to connect to such addresses. We'll specify the "return" action to prevent annoying timeouts for users. Note that this can cause problems if you're doing double NAT.

Where <martians> is a table containing <IP_ADDRESS>/15 as well as other non-routable address ranges.
The block return rule's behavior according to the pf.conf manual is

This causes a TCP RST to be returned for TCP packets and an ICMP UNREACHABLE for other types of packets.

Another one on linux-arm: https://build.golang.org/log/17e81bd9a7f4cae294a6327b6d2bab1a3c987609

linux-arm again:
https://build.golang.org/log/7460cac43be8414f1f3766051dcc583aa7cbfb8f

linux-arm: https://build.golang.org/log/806bf66f3ac7ce31e8a6f521e619d48485c65b97

linux-arm again: https://build.golang.org/log/29ecd5fb9d6853a2370163d0d81c265a86343e70

linux-arm continues: https://build.golang.org/log/019db9d64edc807077516ba20399da59bd7a949e

Same test failure happening on the MIPS builders.
2019-10-29T12:23:21-ac346a5/linux-mips64le-rtrk
2019-10-29T18:32:59-ca70ada/linux-mips64-rtrk
2019-10-29T19:58:24-cc47b0d/linux-mips64-rtrk
2019-10-30T03:48:03-9e094ea/linux-mips64le-rtrk
2019-10-30T08:17:29-f4e32ae/linux-arm
2019-10-31T16:02:25-d5caea7/linux-mips64-rtrk
2019-10-31T17:09:48-a9b37ae/linux-mips-rtrk
2019-10-31T17:21:56-48c0cef/linux-mips-rtrk
2019-11-05T05:22:07-3c0fbee/linux-mips64-rtrk
2019-11-05T15:40:02-8550a58/linux-mips64-rtrk
2019-11-05T18:37:06-c3cef0b/linux-mips64-rtrk
2019-11-05T20:21:34-552987f/linux-mipsle-rtrk
2019-11-05T20:47:22-fb37821/linux-mips64-rtrk
2019-11-05T20:47:22-fb37821/linux-mipsle-rtrk
2019-11-05T21:26:19-649f341/linux-mips-rtrk
2019-11-05T21:26:19-649f341/linux-mips64le-rtrk
2019-11-06T09:08:53-6108998/linux-mips-rtrk
2019-11-06T09:08:53-6108998/linux-mips64le-rtrk
2019-11-06T09:09:21-0ea7440/linux-mips-rtrk
2019-11-06T09:09:21-0ea7440/linux-mips64-rtrk
2019-11-06T09:09:59-0c5d545/linux-mips64le-rtrk
2019-11-06T13:55:04-6dc250f/linux-mips64-rtrk
2019-11-06T13:55:04-6dc250f/linux-mips64le-rtrk
2019-11-06T13:55:04-6dc250f/linux-mipsle-rtrk
2019-11-06T14:33:39-f891b7c/linux-mips-rtrk
2019-11-06T14:33:39-f891b7c/linux-mips64le-rtrk
2019-11-06T14:33:39-f891b7c/linux-mipsle-rtrk
2019-11-06T14:34:46-b824048/linux-mips-rtrk
2019-11-06T14:34:46-b824048/linux-mipsle-rtrk
2019-11-06T14:56:38-1bd974e/linux-mips-rtrk
2019-11-06T14:56:38-1bd974e/linux-mips64-rtrk
2019-11-06T14:56:38-1bd974e/linux-mips64le-rtrk
2019-11-06T14:56:38-1bd974e/linux-mipsle-rtrk
2019-11-06T15:08:19-cf3be9b/linux-mipsle-rtrk
2019-11-06T16:17:30-a5936a4/linux-mips-rtrk
2019-11-06T16:17:30-a5936a4/linux-mipsle-rtrk
2019-11-06T17:03:51-a2b1dc8/linux-mips64le-rtrk

The test assumes that trying to connect to <IP_ADDRESS> or 2001:2::254 will hang.  I guess that is usually true, and I guess that it is not always true.  I will send a CL to tweak the test.


Modify standardization & use of crash data to permit crashes by month rather than week

Certain cities (Philadelphia and Brisbane at least) provide crash data with no day of month property, for privacy reasons.
For these cities to work we need to look at upgrading at least two processes:

the standardize_crashes script needs to understand when the raw crash data has no day of month property and record the available date accordingly
the modeling process needs to know the city uses per-month crash data (by reading the config?) and generate predictions of the same nature.


User profile errors out

This user profile errors out.

Issues around creating empty track record charts

fixed on main


Bump manageiq-smartstate gem to 0.1.4

manageiq-smartstate gem has been updated to version 0.1.4 and pushed out to Rubygems.
Change manageiq to use the new version.
This is in support of https://bugzilla.redhat.com/show_bug.cgi?id=1463780
@roliveri @Fryguy @agrare please review and merge
Links

https://bugzilla.redhat.com/show_bug.cgi?id=1463780

@jerryk55  this bumps smartstate and vmwarw_web_service https://github.com/ManageIQ/manageiq/pull/15749

@agrare then do you want to merge your commit?  We need that change for a BZ.

Merged via https://github.com/ManageIQ/manageiq/pull/15749


Develop library for computing the differences in KRM resources when they change and produce concrete differences

Develop library for computing the differences in KRM resources when they change and produce concrete differences. This will provide concrete outputs for what is added/modified/deleted so that  users of this library can take appropriate actions.

done, part of cond SDK


Adopt PowerShellBuild

https://github.com/psake/PowerShellBuild
PSSA, PlatyPS, Pester baked in... what's not to love??

This looks sweet.


Impossible to use GPU in docker desktop

Hello,
After successfully using zerocostdl4mic to train a model using stardist3D, I now would like to train locally using Dl4miceverywhere.
I use Ubuntu 22.04, installed Cuda (the last version, 12.4) and Docker. The Stardist3D notebook work perfectly when I don't use the GPU, but when I use it I get the error [could not select device driver “” with capabilities: [[gpu]] ...
My Cuda installation is normally working. Is there anything I missed ?
Thank you in advance !

Hi @mvelean1 ,
The message that you are getting might be because Nvidia is not properly registered with Docker. If your CUDA installation is normally working, the possible suggest you to restart the Docker daemon:
sudo systemctl restart docker

In case this is still not working, it might be that the nvidia-cuda-toolkit is not correctly installed in your computer. Could you check what is the output of the following command?
nvcc -V

Please let me know if this restarting Docker daemon worked or if you might need to install nvidia-cuda-toolkit.
Thank you very much for using DL4MicEverywhere! 😄

Thank you for your answer and your help !
It didn't work after restarting Docker. Here's the output of nvcc -V :
nvcc: NVIDIA (R) Cuda compiler driver Copyright (c) 2005-2024 NVIDIA Corporation Built on Tue_Feb_27_16:19:38_PST_2024 Cuda compilation tools, release 12.4, V12.4.99 Build cuda_12.4.r12.4/compiler.33961263_0

Okey so the cuda toolkit is installed, can you check to run a simple Docker image from Nvidia to test if that works correctly?
docker run --gpus all --rm nvidia/cuda:12.3.2-devel-ubuntu22.04 nvidia-smi

It should give you the same output as if you run nvidia-smi in your terminal.
Does this work?

Okey, then the issue seems to be not with the nvidia-cuda-toolkitbut with the nvidia-container-toolkit that is needed to make the connection between nvidia and the docker containers. For that you will need to install nvidia-container-toolkit and restart Docker daemon again just in case. Try with the following commands:
sudo apt-get update
sudo apt-get install -y nvidia-container-toolkit
sudo systemctl restart docker

In many blogs they comment that this fixed their issues, so I hope that this will fix the problem. If not we can find another way to fix it 😄

I already tried that, sounded like it would solve my problem, but it didn't :( I get the same error unfortunately !

I think it's a problem of permission, when I run docker in rootless mode, I can run a docker image from Nvidia. However, I didn't succeed to run DL4MicEverywhere in rootless mode

Unfortunately, no :(
'CUDA Version 12.3.2
Container image Copyright (c) 2016-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
This container image and its contents are governed by the NVIDIA Deep Learning Container License.
By pulling and using the container, you accept the terms and conditions of this license:
https://developer.nvidia.com/ngc/nvidia-deep-learning-container-license
A copy of this license is made available in this container at /NGC-DL-CONTAINER-LICENSE for your convenience.
Failed to initialize NVML: Unknown Error
'

Hi @mvelean1,
That is wonderful a new!!
I see, sorry that I missed that part of registering the nvidia toolkit into the docker configuration.
I'm super glad that you found the solution because I was not finding other things.
Yeah I think that is clear, also for other people that might have the same error in the future.
Thanks a lot for all the feedback and putting all the steps and output that you have gone through!! 😄


account overview and typos

Added a brief overview steps to get an account. Corrected typos in account pages and elsewhere.

I don’t see the changes described included

Thanks and also, phooey. I'll get it straightened out.
On Tue, Jun 4, 2024 at 4:49 PM Josh Eisinger @.***>
wrote:

I don’t see the changes described included
—
Reply to this email directly, view it on GitHub
https://github.com/NOAA-RDHPCS/noaa-rdhpcs.github.io/pull/77#issuecomment-2148393041,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4T5JW765GQWXBP3IT4OZM3ZFYR6RAVCNFSM6AAAAABIZJF43OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBYGM4TGMBUGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Update jquery/bootstrap to the latest versions

This PR tracks work upgrading both jquery and bootstrap to the latest versions. There isn't much jquery which makes that piece okay, but the update to Bootstrap 4 is somewhat involved and still WIP.

This has been deployed and now matches the desired format well enough, will follow up with additional tweaks as needed.


Maybe move the minmap to left side? Thank U!

It's achievable but a bit tricky to implement. I'll visit this later.

It's achievable but a bit tricky to implement. I'll visit this later.

Thanks! I think if it looks like a PPT thumbnail, that would be great. So canvas can be used as PPT。

É exatamente o meu pedido, estes dois tópicos podem ser mesclados!
https://github.com/ifree/Obsidian-canvas-minimap/issues/12


Adding barcode generator

This branch contains QmlBarcodeReader example and QmlBarcodeGenerator example.

Add signals to all properties in SBarcodeGenerator
QQmlExpression: Expression qrc:/qml/GeneratorPage.qml:187:11 depends on non-NOTIFYable properties:
    SBarcodeGenerator::fileName
QQmlExpression: Expression qrc:/qml/GeneratorPage.qml:141:11 depends on non-NOTIFYable properties:
    SBarcodeGenerator::eccLevel
QQmlExpression: Expression qrc:/qml/GeneratorPage.qml:136:11 depends on non-NOTIFYable properties:
    SBarcodeGenerator::margin
QQmlExpression: Expression qrc:/qml/GeneratorPage.qml:131:11 depends on non-NOTIFYable properties:
    SBarcodeGenerator::height
QQmlExpression: Expression qrc:/qml/GeneratorPage.qml:126:11 depends on non-NOTIFYable properties:
    SBarcodeGenerator::width

Looks like sometimes you pass a string to an integer. Ensure that you convert text to int in such cases
qrc:/qml/GeneratorPage.qml:143: Error: Cannot assign QString to int
qrc:/qml/GeneratorPage.qml:127: Error: Cannot assign QString to int

The back button is not necessary. Please remove it.

Increase bottom margin for buttons

After saving a file display a confirmation popup.


StringBuilder.AppendFormat doesn't show up as a link on the String.Format page

On the "String.Format Q & A" section of the String.Format page, the StringBuilder.AppendFormat method doesn't show up as a link while Console.WriteLine does.
I tried both on Win10 x64 with Microsoft Edge 41.16299.248.0, Firefox 58.0.2 and Chrome 64.0.3282.186, same problem.
There's also a typo at "The use of mutiple" at the bottom of the screenshot below:

Thank you for reporting this @GenesisFR and @svick for the fix. It's fixed now.
This issue can be closed. (I don't have write permissions to do it)


Process Explorer Environment Variables tab should scrub env values to avoid XSS

This is discovered by a security group and MSRC ticket was opened against Kudu.   Given one exploiting and being exploited needs to be an App Owner, it is less of the a security than to clean the UI up.
To repro, add an AppSetting (or ConnectionString or anything that will be displayed on Process Explorer's Properties' Environment Variables tab) whose the value contains XSS such as .     Browse to Kudu and open Process Explorer's Properties' Environment Variables tab.    The alert script will be executed and browser dialog popup.
The simple fix would be to remove "<", ">" and "&" from value before displaying.

this is fixed


cleanup!: use the google-cloud-cpp-common project

BREAKING CHANGE: The common libraries have been moved
to the google-cloud-cpp-common repository.
While this may not be a technically breaking change (the API and ABI
remain unchanged, the include paths are the same), it will require
application developers to change their build scripts, so I think labeling
as such is useful.

This change is

PTAL.

On Mon, Oct 14, 2019 at 1:43 PM Todd Derr<EMAIL_ADDRESS>wrote:

@mr-salty approved this pull request.
[image: :lgtm:]
https://camo.githubusercontent.com/41b3ec3419116e3b3f84507ad965646ce4ce1f0c/68747470733a2f2f72657669657761626c652e696f2f6c67746d2e706e67
Reviewed 154 of 156 files at r1, 11 of 11 files at r2.

I have no idea why it says "154 of 156", I have the little green eyeball on
every file in the list...  and i'm not going to count them.

Reviewable
https://reviewable.io/reviews/googleapis/google-cloud-cpp/3196#-:-LrAMUEl-Mz1izunFSGN:bnfp4nl
status: all files reviewed, 3 unresolved discussions (waiting on @coryan
https://github.com/coryan)

ci/kokoro/docker/Dockerfile.fedora-install, line 95 at r2
https://reviewable.io/reviews/googleapis/google-cloud-cpp/3196#-LrAJdkSAZUXieXwmIj0:-LrAJdkT3yS_UdLE-sD5:b-ejjf1m
(raw file
https://github.com/googleapis/google-cloud-cpp/blob/8e9105fc92456c36cf577360b00dff4fa8340d79/ci/kokoro/docker/Dockerfile.fedora-install#L95):
RUN tar -xf v0.13.0.tar.gz
WORKDIR /var/tmp/build/google-cloud-cpp-common-0.13.0
Compile without the tests because we are testing spanner, not the base
this should say something other than "spanner".
Applies to a bunch of Dockerfiles (I only saw it go by 5 times or so
before it registered...)
ci/test-install/WORKSPACE, line 51 at r2
https://reviewable.io/reviews/googleapis/google-cloud-cpp/3196#-LrAKDkO4ehr9XAmAms_:-LrAKDkP2meYVPqXUUh8:b4cm4um
(raw file
https://github.com/googleapis/google-cloud-cpp/blob/8e9105fc92456c36cf577360b00dff4fa8340d79/ci/test-install/WORKSPACE#L51):
)
https://github.com/bazelbuild/bazel/issues/1550
nit: get rid of extra indent since you deleted the top line
google/cloud/README.md, line 11 at r2
https://reviewable.io/reviews/googleapis/google-cloud-cpp/3196#-LrALMQW0vnQRGCXDzLD:-LrALMQW0vnQRGCXDzLE:b-ruwcsk
(raw file
https://github.com/googleapis/google-cloud-cpp/blob/8e9105fc92456c36cf577360b00dff4fa8340d79/google/cloud/README.md#L11):
the
s/, the/; they/
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googleapis/google-cloud-cpp/pull/3196?email_source=notifications&email_token=AB2T57FB3XOLRFI6ZTQY62LQOSVVBA5CNFSM4I77DQ22YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCH34EOA#pullrequestreview-301449784,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2T57BKXI4OLZ7UUJMFJ2LQOSVVBANCNFSM4I77DQ2Q
.


HibernateTypesContributor throws java.lang.NoClassDefFoundError: javax/money/MonetaryAmount

Spring boot 2.7.2, I am receiving this errors when trying to upgrade:
Cannot resolve reference to bean 'entityManagerFactory' while setting constructor argument; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'entityManagerFactory' defined in class path resource [org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaConfiguration.class]: Invocation of init method failed; nested exception is java.lang.NoClassDefFoundError: javax/money/MonetaryAmount

Fixed.


Added multifile support

Closes #132

I think this is a big enough feature that we need  immediate tests on it to ensure it actually works. If we had other work waiting for this to be approved i would do it now, but we might as well get it in before merging.

The split file tests cannot be run asynchronously because the generated code may match in content but differ in order. This results in the verified files causing an error.


buffer has wrong return type?

See here: https://github.com/ReactiveCocoa/ReactiveObjC/blob/master/ReactiveObjC/RACSignal+Operations.h#L140
Shouldn't this return type be a RACSignal<RACTuple>?

See https://github.com/ReactiveCocoa/ReactiveObjC/pull/66


Delete Dockerfile as it downloads Synthea JAR that uses Log4j < 2.16.0

Description of what I changed

E2E test

TESTED:
N/A since it is just a Dockerfile
Checklist: I completed these to help reviewers :)

[ ] My IDE is configured to follow the code style of this project.
No? Unsure? -> configure your IDE, format the code and add the changes with git add . && git commit --amend

[ ] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides. Note, when in conflict, OpenMRS style guide overrules.

[ ] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)
No? -> write tests and add them to this commit git add . && git commit --amend

[ ] I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

[ ] All new and existing tests passed.
No? -> figure out why and add the fix to your commit. It is your responsibility to make sure your code works.

[ ] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command git pull --rebase upstream master

The build fails because we have deleted the test docker files (to remove log4j issues); this will be fixed once #240 is done.


download as JSON

As a user I want to download JSON parse result of a single file I uploaded so I can see details of the recorgnized results

Save as JSON done in the develop branch


Adds Gnollhack as a variant and adds Notdnethack to the first ascension type

Adds Gnollhack as variant and updates Notdnethacks list of ascension types and makes them first ascension listed.
The file schema/addgnoll.sql
will add the records for Gnollhack tracking.
NOTE because you added the log files for "tnnt 2022" but didn't push them, your max(logfiles_i) may be different than mine, in which case you would need to adjust that field prior to running it.

there were mistakes but I fixed them myself, see e3eed585


[BUG]

Rancher Server Setup

Rancher version: v2.7.s1

Information about the Cluster

Kubernetes version: 1.22.15 or 1.23.*
Cluster Type (Local/Downstream):

If downstream, what type of cluster? Custom

User Information

What is the role of the user logged in? Admin

Describe the bug
Setup a kubernetes fresh cluster one of the cloud servers in india. Trying to add that cluster to rancher using the registeration command. It doesnt complete the process. Finding Management Cluster is struck in pending
apiVersion: management.cattle.io/v3
kind: Cluster
When I took the yaml and ran in the kubectl in terminal on the kuberenetes environment finding this error
error: resource mapping not found for name: "c-2d2zd" namespace: "" from "rancher-mgmt.yaml": no matches for kind "Cluster" in version "management.cattle.io/v3"
ensure CRDs are installed first
To Reproduce
Copy the yaml of c-2d2zd.yaml
apiVersion: management.cattle.io/v3
kind: Cluster
and run in the machine with kubectl apply -f c-2d2zd.yaml
Result
Cluster not getting added to rancher console
Expected Result
cluster addition
Screenshots

@muthufmass can you please edit the title of this issue so that it's easier to know what it's about?


Visual bug: Share your Close Contact Codes button has incorrect colored drop shadow

@helloantoine to advise correct hex value.  The drop shadow is currently showing as black.

@helloantoine ?

fix, avail in next release


[NIFI-2559] UI - Controller Service update missing icons

Reviewing

+1 Verified font changes, icons, alignment. Merging


The frontend should display any rules which could not be tested due to missing files/data

Related to #45.  The tool should probably flag up validations that have not run because the applicable data was not provided.
For example if the user accidentally misses out the OC3 csv entirely they might like to know that rules 131-133 (etc) could not be tested, rather than being left with the impression there are no errors with their data.  Similarly for rules that depend on previous year data it would be useful to remind/highlight that those validations haven't been run when the relevant previous year data has not been loaded.
On the other hand I can see benefits to being able to intentionally upload only a single file or subset of files/data for validation so would not like to see any changes which would prevent that behaviour.  Maybe any such warnings (this or #45) should be in expandable panes like the existing selection list of validation rules?
def _validate(dfs):
        if 'Episodes' not in dfs:
          return {} # This is where we discover the relevant file/data isn't available.

I was just thinking about this very issue - agree this functionality definitely needs to make its way into the tool in some form.
It might do to just infer the relevant tables from the columns listed in affected_fields, but as pointed out in https://github.com/SocialFinanceDigitalLabs/quality-lac-data-beta-validator/issues/347, some fields are duplicated between tables so that might cause trouble
Otherwise we could update the ErrorDefinition class to also include the required tables. Bit of a tedious copy/pasting job, and we'd have to make a note to add that to all future rules, but certainly doable.
It'll also take a bit of thought to figure out how best to display this in the frontend. It would be cool to be able to group select/deselect the rules in the 'Validation Rules' & 'Filter' dropdowns based on the affected tables and/or fields


Submitting Cifer chain's CIF Token to CosmosStation

Currently, $CIF tokens are live on the CiferAI testnet and can be sent and received via Keplr wallet using our website, block.cifer.ai. We are actively developing the platform and plan to utilize $CIF in multiple ways as real-use cases:
Gas Fee and Transaction Fee: $CIF will serve as the primary currency for paying gas fees and transaction fees on the CiferAI blockchain. This will incentivize users to hold and utilize $CIF within the ecosystem.
AI Model Training Incentives: Users who provide data or computational resources for training AI models on the CiferAI platform will be rewarded with $CIF tokens. This incentivizes participation and ensures a robust and decentralized training process.
Governance and Voting: In the future, $CIF holders will have voting rights on key decisions and proposals related to the CiferAI protocol and development roadmap. This grants token holders control and fosters community engagement.
We are continuously working on integrating additional use cases for $CIF within the CiferAI ecosystem. We will keep the communities updated on our progress.

Our customer support team notified me that they have sent out an email regarding this PR.
The current PR is not able to be merged because of the following reasons.

The token needs to be on Mainnet
The token needs to have a real use case on the Cosmos Ecosystem dApps.

Once the tokens are live on mainnet and have a real use case on Dapps on Cosmos ecosystem, please re-submit the PR with details. Thank you and we hope you have a great day


Basic OpenGL rendering

[x] Create a window using GLFW
[x] Enable VSync on the window
[x] Application's window shall be a singleton. There is no need for multiple windows
[ ] Use GLAD for GL 3.0 context loading
[ ] Shaders?

Ditching OpenGL for DirectX


Remove unneeded abstractions in model code

I don't see any reason to have functions in the class that just call one line functions. We might as well call them directly. We actually do this already for some parts of the model, so this is also a more consistent approach.
My overall ambition is to make models as lightweight as possible and encourage users to make models with minimal overhead. Possibly, going as far as to not provide any prebuilt models in the package itself (but provide ample examples in documentation).

I like getting lighter code. I would just wait for @dylanhmorris' take on this before merging.

Noting that I updated the original description for anyone who had previously read it.

Once this is merged I think the _rv suffix for those class attribute RandomVariables can be removed, but that's a separate PR.

Happy to discuss further. I know people are generally opposed to variable names that state the variable type, but I think it is an easy remedy to avoid having code like latent_hosp_admissions = latent_hosp_admissions.sample.

I have mangled the git history here somehow. Retrying on https://github.com/CDCgov/multisignal-epi-inference/pull/207.


$cordovaPush:notificationReceived event not broadcasted

As per the docs, I was trying to listen for notifications using:
$rootScope.$on('$cordovaPush:notificationReceived', func(event, notification)), which just wouldn't work.
Replacing that to listen for the pushNotificationReceived instead made it work for me. So I'm calling:
$rootScope.$on('pushNotificationReceived', func(event, notification)) and the listener successfully works.
Not sure if the docs are wrong, or if I'm doing something wrong on my side so I figured I post here.

Leon
http://miyonline.net/dollar
23.03.2015, 22:49, "Leon Sasson" notifications@github.com:As per the , I was trying to listen for notifications using:As per the , I was trying to listen for notifications using:
$rootScope.$on('$cordovaPush:notificationReceived', func(event, notification)), which just wouldn't work.
Replacing that to listen for the pushNotificationReceived instead made it work for me. So I'm calling:
$rootScope.$on('pushNotificationReceived', func(event, notification)) and the listener successfully works.
Not sure if the docs are wrong, or if I'm doing something wrong on my side so I figured I post here.
—Reply to this email directly or .

I too am listening for $cordovaLocalNotification:clicked but it never get broadcast

@leonsas Which version of ngCordova do you use?
Event name has changed on 0.1.11-alpha version.
CHANGELOG

I've just realised I'm talking about LocalNotifications not
PushNotifications. I'm on the 0.1.0
On Thu, Mar 26, 2015 at 1:07 PM Arnaud Spanneut<EMAIL_ADDRESS>wrote:

@leonsas https://github.com/leonsas Which version of ngCordova do you
use?
Event name has changed on 0.1.11-alpha version.
CHANGELOG
https://github.com/driftyco/ng-cordova/blob/master/CHANGELOG.md#push-notifications
—
Reply to this email directly or view it on GitHub
https://github.com/driftyco/ng-cordova/issues/693#issuecomment-86506170.

I'm on the 0.1.8-alpha, so it makes sense that I still need to use pushNotificationReceived.

Hey Guys I've tried both ways and nothing works.. this is still an issue? I've followed this article but I cannot get the device resgistration id.

Bruno, it was a while ago but I did get it working (LocalNotifications). I
don't have the working code anymore I'm afraid.
Kevin
On Thu, Sep 3, 2015 at 5:55 PM bruno-serfe<EMAIL_ADDRESS>wrote:

Hey Guys I've tried both ways and nothing works.. this is still an issue?
I've followed this article but I cannot get the device resgistration id.
—
Reply to this email directly or view it on GitHub
https://github.com/driftyco/ng-cordova/issues/693#issuecomment-137510516
.

Hi Kevin, thanks anyway, I've found the solution using this new plugin from phonegap https://github.com/phonegap/phonegap-plugin-push I hope this also works for others.
Best regards,

@bruno-serfe I added support for the new plugin in #984, also covered in issue #963.

Closed with the resolution of #963.


Plugin update to make it work with IntelliJ 2020.3 until the latest 2022.1

Some output from ./gradlew runPluginVerifier:
Plugin com.github.mateusz512.crxcleaner:0.0.5 against IC-212.5712.43: Compatible
Plugin can be loaded/unloaded without IDE restart
Plugin com.github.mateusz512.crxcleaner:0.0.5 against IC-213.7172.25: Compatible
Plugin can be loaded/unloaded without IDE restart
Plugin com.github.mateusz512.crxcleaner:0.0.5 against IC-221.5080.210: Compatible
Plugin can be loaded/unloaded without IDE restart
Plugin com.github.mateusz512.crxcleaner:0.0.5 against IC-203.8084.24: Compatible
Plugin can be loaded/unloaded without IDE restart
Plugin com.github.mateusz512.crxcleaner:0.0.5 against IC-211.7628.21: Compatible
Plugin can be loaded/unloaded without IDE restart

Could you please update the IntelliJ Marketplace with a new release if you merge this? Thx!

LGTM. Thanks for the contribution!

Thank you! Could you cut a new release 🙏 ?

Pipeline is having some issues. I'll try fixing it tomorrow.

@Mateusz512 maybe this helps to get the pipeline working again? https://tomgregory.com/build-gradle-projects-with-github-actions/#6_Use_gradle-build-action

@wimsymons nope, that was not it, but thanks anyway :D
v0.0.5 is pending release on plugin marketplace. Should be available within a day or two.


Update DataProtection

Hi @BrennanConroy, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@Tratcher To take a look at the cookie sharing specifically

:+1:

@Rick-Anderson @danroth27

https://github.com/aspnet/Docs/issues/1105

:shipit:


[C000560] Sen. Thomas Coburn

Sen. Thomas Coburn:
http://www.coburn.senate.gov
Likely contact form URL to start with:
http://www.coburn.senate.gov/public/index.cfm/contactsenatorcoburn?p=ContactForm

YAML file lost. I am working on it.


Update depreciation.py

writing if statement in one line

@martinwicke made the changes

Nagging Assignee @martinwicke: It has been 17 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

It has been 43 days that this pull-request has stalled. Please create a new pull-request with the requested changes.


Fix bug in OpentelemetryOban.insert_all/4

What does this PR do?
Fixes a bug in OpentelemetryOban.insert_all/4 that causes the request to fail due to OpenTelemetryOban not existing in Oban

I'm ignorant of how this works, what do you mean by OpenTelemetryOban not existing in Oban and how does it relate to this PR?

Sorry @tsloughter, this is an example error:
16:20:13.912 [error] ** (RuntimeError) no config registered for OpentelemetryOban instance, is the supervisor running?
(oban 2.13.1) lib/oban/registry.ex:37: Oban.Registry.config/1
(oban 2.13.1) lib/oban.ex:420: Oban.insert_all/5

The other functions in this module use the Oban instance.

👍 looks good
Though, from README:

Oban also supports inserting jobs using Oban.insert/4, Oban.insert_all/2
and Oban.insert_all/4. These are currently not supported by OpentelemetryOban
and are just proxied through to Oban.

So even though it fixes that API, it won't actually add any tracing.

Ah yep that's very true. I think we use OpentelemetryOban.insert_all/4 for parity with our other usages of it.


PdfAsync using "Microsoft Print to PDF" instead of "Save as PDF"

Description
We've had some issues with weird printing bugs, that we couldn't reproduce in any chromium instance.
Then we've tried the headless configuration, and we found out that the PDF method actually prints it with "Microsoft Print to PDF" option, which produces the exact document with the bugs that we've seen, and not the Chromium bulit-in "Save as PDF" which works well.
Versions

Which version of PuppeteerSharp are you using? 1.18
Which .NET runtime and version are you targeting? 4.7.2

Removing the Microsoft printer from the Windows installation fixed the problem.


Stop testing build

This will stop build.sh changes always failing.
I think @mgsdk's work on the NetKAN build script would apply to the CKAN-Meta. So I might take a look into porting it.

This looks great to me. Does jenkins keep KSP install state between runs, hence the need to "forget" them?

@pjf possibly it did previously, I was getting annoyed during testing locally - can remove it if required.
Every build now gets a fresh docker container which is terminated and removed post build.

@techman83 : Having thing which make sense for local testing is awesome. Thank you! :)

Merging with great thanks and the knowledge that if this screws things up, jenkins will tell us. :)


Add link to speech from dev conference to Readme

Video contains speech "Scaling React.js Applications"  delivered
by Mas Stoiber at oredev conference 2016. It provides insights
about scalable design of React applications which used in
react-boilerplate.

Coverage remained the same at 98.217% when pulling 056ef0230e4c5041d0e373a42324b55c69ecf34f on jeggett:oredev-video-link into b0e3f8c6650f7ac4163af8ca931a9e91d177ade6 on mxstbr:master.


Command Line Interface

I'm opening this as a PR so people have the chance to try it out and comment on it before it's merged into master. It replaces the current Rails-specific methods of working jobs (rake que:work and autostarting when the server runs) with a command-line interface. This will hopefully fix a lot of the persistent problems people have been having with workers running during migrations, asset compilations, in the console, etc.
From the changelog:

A command-line program has been added that can be used to work jobs in a more flexible manner than the previous rake task. Run que -h for more information.
The rake que:work rake task that was specific to Rails has been removed in favor of the CLI, and the various QUE_* environment variables no longer have any effect.
The worker pool will no longer start automatically in the same process when running the rails server - this behavior was too prone to breakage. If you'd like to recreate the old behavior, you can manually set Que.mode = :async in your app whenever conditions are appropriate (classes have loaded, a database connection has been established, and the process will not be forking).

Currently, the que command must be invoked with the path of the application to load before jobs can be worked, and actually, any number of filenames can be provided and they'll be required in order. For people using Rails, you'll probably want to invoke the command like que config/environment. I considered trying to check whether the command was being called in the context of a Rails app by checking for the presence of the config/environment.rb, but this felt hacky to me. I understand that most people using Que are probably using it with Rails, though, so I suppose I could be talked into it.
Also be aware that the docs haven't been updated to reflect these changes, so take them with a grain of salt.
Feedback and experimentation (and even PRs to merge into this PR) welcome! And thanks to GoCardless for bringing up the idea a while ago, I took some inspiration from Hutch.

Note that, per one of @godfat's suggestions, the script no longer adds '.' to the load path, so for the Rails example, it'd have to be invoked like que ./config/environment.rb.

It replaces the current Rails-specific methods of working jobs (rake que:work and autostarting when the server runs) with a command-line interface

How does that tie in with avoiding a separate worker dyno on Heroku? Would we still do a CLI call during application startup for this case? Or can the CLI be used after application deployment to start workers in the same dyno? Sorry if I’m missing something.

In a nutshell, if you're using rake que:work before, you use que ./config/environment.rb instead. If you want to avoid an extra process, then you have to put Que.mode = :async somewhere in your code. If you're using a forked server, you could put it in your after fork callback.

Yeah, Que won't start by itself anymore at all, since doing so was too prone to issues. You can spin it up manually by setting the mode to :async, but it'll be up to you to figure out how to do so safely for your app. For people using forking webservers, the after_fork callbacks will probably be the best bet, but I imagine it'll be affected by when classes are loaded, when the DB connection is established, etc.


[BUG] 404 error when playing a song

Bug Description
404 error when playing a song
Reproduction Steps
Steps to reproduce the behavior:

Try to play a song
404 error happens

Expected Result
song should play
Attachments
Playing YOASOBI "Racing into the Night"  Official Music Video //song that i tried to play
//error code
node:internal/process/promises:245
          triggerUncaughtException(err, true /* fromPromise */);
          ^

Error [ERR_UNHANDLED_ERROR]: Unhandled error. ('Status code: 404')
    at new NodeError (node:internal/errors:329:5)
    at Player.emit (node:events:358:17)
    at PassThrough.<anonymous> (/var/bot/node_modules/discord-music-player/src/Player.js:705:24)
    at PassThrough.emit (node:events:381:22)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:94:5) {
  code: 'ERR_UNHANDLED_ERROR',
  context: 'Status code: 404'
}

Versioning:

OS: Arch Linux 5.11.11 as well as Windows 10 - 20H2
NPM Version 7.10.0
Module Version 7.2.0

The exact code the bot is running https://ghostbin.com/paste/2wnx2 (it's copy paste from docs anyway)
I also tried to reinstall it multiple times. Rebooting machine and so on.

That issue was fixed in v8 - #177


Request ignored due to cluster ID mismatch?

I'm attempting to get etcd working correctly on 633.1.0 with an initial cluster of 5 machines.
As suggested previously I'm using etcd2 running inside a docker container.  In order to override the inbuilt etcd in 633.1.0 I created a service file in /etc/systemd/system
I set my initial cluster size to 5 when creating the URL. So extra machines should fall into a proxy mode.  Does that work?
/etc/systemd/system/etcd.service:
[Unit]
Description=etcd container
After=docker.service
Before=fleet.service

[Service]
Restart=always
RestartSec=10s
LimitNOFILE=40000
EnvironmentFile=/etc/environment
ExecStartPre=-/usr/bin/docker kill etcd
ExecStartPre=-/usr/bin/docker rm etcd
#ExecStartPre=/usr/bin/docker pull quay.io/coreos/etcd:v2.0.8
ExecStart=/usr/bin/docker run \
  -v /var/lib/etcd:/var/lib/etcd \
  -v /usr/share/ca-certificates/:/etc/ssl/certs \
  -p 4001:4001 -p 2380:2380 -p 2379:2379 --name etcd quay.io/coreos/etcd:v2.0.8 \
  -name %H \
  -data-dir /var/lib/etcd \
  -initial-advertise-peer-urls http://${COREOS_PRIVATE_IPV4}:2380 \
  -listen-peer-urls http://<IP_ADDRESS>:2380 \
  -discovery=https://***my unique discovery url***
ExecStop=/usr/bin/docker stop etcd

[Install]
WantedBy=multi-user.target

In order to allow restarting etcd, I'm setting the data directory and mounting it locally so it's not lost and it will allow me to hopefully switch to etcd2 when the native version comes out and I won't be needing to put it into docker.
The problem is however, it doesn't work.  The discovery URL is being populated, but I have these errors on my nodes repeated over and over again.
May 08 01:37:34 service2-3 docker[957]: 2015/05/08 01:37:34 rafthttp: request ignored due to cluster ID mismatch got a7c6bb9a1ba51b7d want 29ac8734ae277c83
May 08 01:37:45 service2-3 docker[957]: 2015/05/08 01:37:45 rafthttp: request ignored due to cluster ID mismatch got a7c6bb9a1ba51b7d want 29ac8734ae277c83

Can someone please please write a recipe, or tell me what is the recommended way to start a cluster from CoreOS stable using etcd2 without switching to Alpha channel.  Or tell me what's wrong with the above.
Ideally, write some documentation on how to do it.  Since to switch to Alpha means to migrate my entire cluster.  I'm not going to do that.  I just want etcd2.   I can't use 0.4x, I don't think anyone can possibly run it for more than a few days without it crashing your hosts.  So how do I do it?
And please, more examples of running bare metal.  I'm using some scripts to populate COREOS_PRIVATE_IPV4 and doing the double-init thing.  That works.

I thing you somehow misconfigured your cluster. Can you please provide the full log of your etcd members?

Unfortunately not.  I tore it down and went with a static config. It's the only one I could get to actually work.

@hookenz It works good with me. And I cannot get this reproduced. Reopen if you can reproduce it and get some meaningful log. Thanks!


angular material not working

when I install npm install --save @angular/material
It gives me following response :
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
But this are just warnings , but when I try to actually import it in app.module.ts npm start breaks :
npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "build"
npm ERR! node v6.10.0
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: tsc -p src/
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'tsc -p src/'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular-quickstart package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     tsc -p src/
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular-quickstart
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular-quickstart
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     E:\My Projects\angular-quickstart-master\npm-debug.log
npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "start"
npm ERR! node v6.10.0
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>prestart: npm run build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prestart script 'npm run build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular-quickstart package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular-quickstart
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular-quickstart
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     E:\My Projects\angular-quickstart-master\npm-debug.log
I am not sure what is the exact issue. Can any one please help.

Refer this if you want to use angular 4 with newest angular material https://stackoverflow.com/a/44557421/4465743

Hey,
The quick start is using an old version of typescript, to fix this problem you need to rock your application with a version of typescript  which is above 2.2, update it and you will be just fine.

Also you will probably be in need of @angular/animations, so you will need to install that too and then configure your systemjs.config.js file again with the new library and your typically done with the issue.

This repository is now deprecated. The Angular Quickstart project was a nice starting point for creating Angular applications. Now we recommend using the Angular CLI to create new Angular projects.
Starting from 1 November 2017, all the Angular documentation, at angular.io, is based on the Angular CLI.
Let's get started


Accept mining fee for TransactionBuilder

Transaction builder is computing mining_fee for the transaction. User/clinet is not able to set the mining fee.
Enables TransactionBuilder to accept mining fee and use it as the mining fee instead of computing the fee and make sure it compute the fee unless the mining fee is not provided.

Context behind this PR is, I want to set minig fee explisitly for various reason. Current version computes the fee based on the rate.

Thanks!


Update audio_stream_effects.c

This may slightly improve performance and be more welcoming for new users despite being an more advanced feature. void * usually throws an error in most compilers and it would be better to just avoid it.
Also added <stdbool.h> because booleans are, sometimes, not defined by <stddef.h>.

@raysan5 Fixed! (sorry for drafting, I'm getting used to GitHub lol)

@lipx1508 thanks for the review!


Add bounds enforcement pipeline operators

Currently, some hard-bounds enforcement is tightly integrated within perturbation pipeline operators.  However, this can be decoupled into a separate bounds operator.
There can be a bounds operator that clips values and another one that flags an individual for bounds constraints violations.

Discussion with @markcoletti: we already have a consistent pattern for doing this with the existing hard_bounds argument approach.  Also, logic for dealing with constraint violations can be very operator specific: for example, some strategies do re-sampling rather than clamping in a particular way.  So it does not make sense to separate bounding into a separate operator in all cases.


id is reserved word in relay modern

Relay modern does not like id fields that are not string global ids:
https://github.com/facebook/relay/issues/1682
Would it be possible to bring back something like rowId?

Found classicIds flag 😸


website link to github

The website should link here (in footer, header, about page?) so people know to file issues if things are broken/missing.

Since there is no about page, I thought header would be better for this. If css takes care of aligning, fontsize etc... could it be this simple as I've done here?
https://github.com/Fraasi/Rainbow-Code/blob/master/_jekyll/includes/2-base/navigation.html
Just added this line:
Github

I don't know about jekyll, but since it is directing to another website, this should be ok.
Thoughts?

This would add a link to GitHub that looks the same way as the other links in the header. The only thing I would add is a target="_blank" to open it in a new tab.

Agreed. Added target and made a pull request.

Closing as fixed by #460


Update back-end-server.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/1720
Updated SFB server 2019 supported SQL AlwaysOn editions/versions

@SerdarSoysal Hi Serdar, changes look correct. Could you please check and merge? Thanks

@SerdarSoysal Hi Serdar! Just following up on this PR. Could you please check and merge? Thanks.

@microsoftheidi please review.

@MicrosoftHeidi Hi Heidi, any updates on this issue?

This is pending review, and I have no timeline for that.

This should not be merged, and was handled in a different update request. These changes would now clash, and the article as it is today should be correct.


Allow option to provide classes for select input

Currently, we are able to pass a class attribute to the ui-select-match element directive and it will get included on the button.  There is not, however, functionality to allow for the same on an input.
The use case for this is using input-sm and btn-sm on the bootstrap ui-select.

Any news on this enhancement?

This won't be an easy fix since and maybe we should resolve https://github.com/angular-ui/ui-select/issues/186 first

Any news on this one?

+1

+1

if you're using a pre-processor you can use the mixin:
@include button-size($btn-padding-y-sm, $btn-padding-x-sm, $font-size-sm, $line-height-sm, $btn-border-radius-sm);
on .ui-select-toggle etc
(this may be specific to BS4, there's probably a BS3 version)

+1

any news on this? I want to use btn-sm ! :(

+1

+1

+1

+1

+1

+1

A bit late to the party, but it seems that wrapping the <ui-select ... /> with an element which has a form-group-sm class seems to do the job.

+1

+1


Connect the exernal_user_ids to the User Retirement Process

Now that we have an optional sink to transfer exernal_user_ids to Clickhouse, we would need to keep this table up to date.
When a user is retired in the LMS, the view destroys the PII information for the user.
https://github.com/openedx/edx-platform/blob/a313b3fd20b4e12d3d1b0b35a0c5eaef1d0cc771/openedx/core/djangoapps/user_api/accounts/views.py#L1175
However the update should also go through to clickhouse even if the delete operation might be costly.
This issue came to my attention thanks to the Spanish universities when discussing aspects and GDPR.

fyi @Ian2012

I think that is very reasonable. Discuss and decide before implementing anything.
In my view, the fact that the system does not store PII information is a great starting point. All info from the xAPI statements is already safe to store as it is already anonymized.
For in campus usage however having only anonymized data is not enough, mostly teacher want to be able to understand trends in the course so they can still have an effect on individual students that are at risk of failing a course. To achieve this we can turn on the optional event_sink.external_id table from PR#43. This sink dumps data from the ExternalId model into clickhouse. According to the code it copies:
            "external_user_id",
            "external_id_type",
            "username",
            "user_id",
            "dump_id",
            "time_last_dumped",

Now the thing is that even if for campus usage the teachers get access to de-anonymized data. This should not go on forever. Specially if the user deletes its account, the PII info should not longer be accesible. I understand this as simply deleting the row in the sink table in clickhouse (the LMS retirement process already does more than this) but as clickhouse mostly dumps new data in, then as it works now the old records would not be deleted when the row in the LMS is deleted.
The only desired outcome I propose in this issue is that the corresponding row in the sink table is deleted or obfuscated. Any other xAPI statement or DBT processed facts table should remain untouched.

We will very likely want to combine the data from the external id table in ClickHouse into downstream tables for performance reasons, meaning more tables will need to be touched (deleted from / updated / rebuilt) to clean up the PII

I had not considered this. I imagined that we already had the data anonymized so we would do all calculations with the actor_id and if and when someone wanted to de-anonymize then that would be done joining the profile table right in the graph query.
Right now I'm thinking about two possible paths worth exploring. The first would be to double down and make an obfuscation transformation so that we can alter the user_profile table as well. In this case I'm no longer sure if removing the actor_id to user_id mapping is necessary given. Probably that would also have limitations as there might be cases where the obfuscation does not propagate well into the downstream tables.
The other path I can imagine at the moment would be to separate the table user_profile table into the PII fields and the non_PII fields. And then we could propose guidelines that recommend that PII data is not combined in downstream tables while maintaining the capability to further process things like language, gender or country.

-If not carefully considered, operations on those downstream tables may cause data to change, confusing users
-If any downstream tables need to be rebuilt (for example if a new column is added in an update) and they inner join back to the external id table which no longer has rows for retired users, the data contained in them will change, confusing users

These two are related to confusing users which I think can be mitigated if we are doing obfuscation instead of deleting the rows.

If a lot of users retired in a short period of time it might cause performance issues

Do you think this risk warrants doing some rate limiting in the lms? for instance in the task that is dispatched when the user_profile changes.

We can mitigate some of the impact on database performance by creating a command to re-generate the entire MVs or to batch update the records in downstream tables and do it in batches for multiple users if needed

I'm talking with Altinity about this now to get their advice. Rebuilding MV tables could result in lengthy downtimes once the database has significant data in it. One possibility is to use a DICTIONARY type to cache PII data in memory like we do for course data. That should be performant enough to join on, and act as a buffer to updates on the base PII tables, at the cost of using more ClickHouse memory.
I'll let you know what Altinity suggests.

I'm thinking of tackling this with the following steps, likely one PR per step.
Comments/corrections/suggestions @bmtcril  @felipemontoya @Ian2012  ?
Clickhouse/Alembic migrations in tutor-contrib-aspects:

[ ] Partition the ASPECTS_EVENT_SINK_USER_PROFILE_TABLE by userid % 100 (configurable)
[ ] Partition the external_ids table (https://github.com/openedx/tutor-contrib-aspects/pull/521/) by userid % 100
[ ] Create an in-memory dictionary to join user profile PII with external_id, which refreshes every 15m (configurable).
Join to this dictionary for all external_id or PII dataset/chart queries (worth an ADR).

Event bus (new plugin repo):

[ ] Create tutor plugin for event-bus-redis IDA
[ ] Configure event bus to run with openedx-event-sink-clickhouse

Retire user in openedx-event-sink-clickhouse

[ ] On receipt of the USER_RETIRE_LMS_MISC signal from the event bus, queue request to retire user data in Clickhouse.
Records will be deleted from the external_id and user profile tables; dictionary will refresh with these changes automatically.
Management command provided to manually retire a given user.

Monitor event sink/user retirements:

[ ] Add Event Sink tab to the Operator Dashboard, showing data from the event sink's celery queue.
Chart queue size and error count over time.

This all sounds great, my expectation is that if the ASPECTS_ENABLE_PII flag is off the table and dictionary will be empty, making my main concern about memory usage mostly moot. I think we will want a separate flag for whether retirement runs, defaulting to the same as ASPECTS_ENABLE_PII, just in case someone turns the flag on and then back off for some reason and still needs to retire data. Or perhaps it's just always on, I just want to minimize the chances of there being orphaned PII in the database.

@bmtcril I'm leaning towards leaving the "retire user" event sink on all the time, since User Retirement is in the platform by default. If ASPECTS_ENABLE_PII is false, then there won't be any data to delete, but that should run quickly. And if someone does toggle ASPECTS_ENABLE_PII on and off, then user retirement will still happen.

@bmtcril I think this issue can be resolved?

I believe so, @felipemontoya does the completed work meet your needs?

I believe this is set now, closing.


Y70-70 boot loop

Hi,
Thanks for your nice work.
I am using Y70 which is the same as Y50 except the screen size. I have no issue when using Y50 Clover EFI for Catalina. The problem is the system will restart several times (may be 10 )like a boot loop, then have a chance to work properly.
System will shows:
Panic Medic Boot:
A third party kernel extension was preventing the machine from successfully booting......
Here is the error report that system generated when login.
panic(cpu 2 caller 0xffffff800d3efa76): Kernel trap at 0xffffff7fae82c930, type 14=page fault, registers:
CR0: 0x0000000080010033, CR2: 0xffffff7fae82c930, CR3: 0x00000000226f0000, CR4: 0x00000000001626e0
RAX: 0x0000000000000000, RBX: 0x0000000000000000, RCX: 0x000000000000000b, RDX: 0x0000000000000000
RSP: 0xffffffb0aa96bca8, RBP: 0xffffffb0aa96bcf0, RSI: 0x0000000000000078, RDI: 0xffffff7fae840600
R8:  0x0000000100000201, R9:  0xffffff8011936035, R10: 0x0000000000000003, R11: 0x0000000000000000
R12: 0xffffff800e209850, R13: 0x0000000000000007, R14: 0x0000000000000001, R15: 0xffffff937a13e5e0
RFL: 0x0000000000010246, RIP: 0xffffff7fae82c930, CS:  0x0000000000000008, SS:  0x0000000000000010
Fault CR2: 0xffffff7fae82c930, Error code: 0x0000000000000010, Fault CPU: 0x2, PL: 0, VF: 1
Backtrace (CPU 2), Frame : Return Address
0xffffffb0aa96b6c0 : 0xffffff800d2bc66d
0xffffffb0aa96b710 : 0xffffff800d3ff073
0xffffffb0aa96b750 : 0xffffff800d3ef6aa
0xffffffb0aa96b7a0 : 0xffffff800d261a2f
0xffffffb0aa96b7c0 : 0xffffff800d2bbf0d
0xffffffb0aa96b8e0 : 0xffffff800d2bc1f8
0xffffffb0aa96b950 : 0xffffff800dabee1a
0xffffffb0aa96b9c0 : 0xffffff800d3efa76
0xffffffb0aa96bb40 : 0xffffff800d3ef75d
0xffffffb0aa96bb90 : 0xffffff800d261a2f
0xffffffb0aa96bbb0 : 0xffffff7fae82c930
0xffffffb0aa96bcf0 : 0xffffff800d986bdc
0xffffffb0aa96bd40 : 0xffffff800d98413c
0xffffffb0aa96bda0 : 0xffffff800d9989dd
0xffffffb0aa96bdf0 : 0xffffff800da0eebe
0xffffffb0aa96be40 : 0xffffff800d9ee39b
0xffffffb0aa96bef0 : 0xffffff800d9eddcf
0xffffffb0aa96bf50 : 0xffffff800d9f0b46
0xffffffb0aa96bfa0 : 0xffffff800d26113e
Process name corresponding to current thread: kernel_task
Boot args: brcmfx-driver=2 brcmfx-country=#a -igfxnohdmi
Mac OS version:
20B50
Kernel version:
Darwin Kernel Version 20.1.0: Sat Oct 31 00:07:11 PDT 2020; root:xnu-7195.50.7~2/RELEASE_X86_64
Kernel UUID: 84C6DC45-6B02-335F-9439-5D2A9BC385A4
KernelCache slide: 0x000000000d000000
KernelCache base:  0xffffff800d200000
Kernel slide:      0x000000000d010000
Kernel text base:  0xffffff800d210000
__HIB  text base: 0xffffff800d100000
System model name: MacBookPro11,2 (Mac-3CBD00234E554E41)
System shutdown begun: NO
Panic diags file available: YES (0x0)
Hibernation exit count: 0
System uptime in nanoseconds:<PHONE_NUMBER>
Last Sleep:           absolute           base_tsc          base_nano
Uptime  : 0x00000001c5e4e202
Sleep   : 0x0000000000000000 0x0000000000000000 0x0000000000000000
Wake    : 0x0000000000000000 0x00000006713d02ba 0x0000000000000000

This EFI is designed for Lenovo Y50-70. The hardware of Y50-70 and Y70-70 are not the same of course. You may need some changes. I can help you with it.
Whats your iGPU? I believe it's not Intel HD 4600, must be something newer I guess.

There are two GPUs. one is GTX 860M. The other one is HD4600

As I have the Y50-70 UHD, so I don't know if it helps for you.
I don't have the Broadcom card (Wifi/Bluetooth) of this Repo added to my machine, so I needed to delete the corresponding kexts (don't forget to remove them from config.plist).
Otherwise I ended up boot looping.

@861 did you remove or edit anything from the EFI?
Try to add -v in boot-args to operate in verbose mode.
By doing this we should see the errors and try to fix the bootloop for you.

As I have the Y50-70 UHD, so I don't know if it helps for you.
I don't have the Broadcom card (Wifi/Bluetooth) of this Repo added to my machine, so I needed to delete the corresponding kexts (don't forget to remove them from config.plist).
Otherwise I ended up boot looping.

Hi Thanks for your reply.
Did you delete BrcmFirmwareData.kext?

@861 did you remove or edit anything from the EFI?
Try to add -v in boot-args to operate in verbose mode.
By doing this we should see the errors and try to fix the bootloop for you.

Thank you.
I have attached  the photos for each restart.

I also attached my catclina EFI which is working perfectly.
https://mega.nz/file/udNXzYKJ#XDF-YBij_YHVrvg3owzx-M954cMpGw_9777NYl7dFaY

As I have the Y50-70 UHD, so I don't know if it helps for you.
I don't have the Broadcom card (Wifi/Bluetooth) of this Repo added to my machine, so I needed to delete the corresponding kexts (don't forget to remove them from config.plist).
Otherwise I ended up boot looping.

Hi Thanks for your reply.
Did you delete BrcmFirmwareData.kext?

If you don't have a compatible wifi/bluetooth card you need to remove the following from your config.plist and delete the files as well:

BrcmBluetoothInjector.kext
BrcmFirmwareData.kext
BrcmPatchRAM3.kext

If you don't know how to remove them from your config.plist you can download  OpenCore Configurator (OCC), load your config.plist and delete these kexts from your config by going to Kernel -> Add.
Hope this will help you!


Assign vagrant box to instance

Issue Type

Feature request

Molecule and Ansible details
ansible <IP_ADDRESS>
molecule, version 1.22.0

Molecule installation method: pip
Ansible installation method: pip

Desired Behaviour
I would like to select specific box to a vagrant instance (like docker where you can select your OS!).
Something like:
vagrant:
  providers:
    - name: virtualbox
      type: virtualbox
      options:
        memory: 256
        cpus: 1
  platforms:
    - name: wheezy64
      box: debian/wheezy64
    - name: jessie64
      box: debian/jessie64
    - name: stretch64
      box: debian/testing64
  instances:
    - name: vagrant-debian-wheezy
      platform_name: wheezy64
    - name: vagrant-debian-jessie
      platform_name: jessie64
    - name: vagrant-debian-stretch
      platform_name: stretch64

@HanXHX Maybe this issue will help.  Hopefully @tknerr has some insight as to what he did.

@HanXHX @retr0h I haven't solved that yet. We are still on molecule 1.8.4, and from what I experimented in the issue mentioned above it's not possible with molecule v1 v via molecule.yml.
I believe with molecule v2 this would be fixed, but haven't had a chance to try it yet.
For molecule v1 my approach would be to simply write the Vagrantfile by hand instead of having molecule generate it for you. It seems you can point to a custom Vagrantfile template via vagrantfile_template. That way you have full control over the Vagrantfile, and it should still play nicely with molecule.
(note: I have not verified that this works yet, but this would be my attempt)

@HanXHX Closing.  Really no good way to deal with this under v1.


⚠ Bofrost.de has degraded performance

In 99ff0ef, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1304 ms

Resolved: Bofrost.de performance has improved in 960ef2b.


CI run failed in VCS(git) package

git clean can not clean src directory.
Failed in: https://github.com/2014CAIS01/AUR-packages/runs/2056864144

If upstream update the code,  $pkgver will be changed in check PKGBUILD and fail.
https://github.com/2014CAIS01/AUR-packages/runs/2065355972

The first one is fixed.
The second failure is expected, but there can be a feature request for auto-updating the version when the result of pkgver is changed.


react native mmkv is not supported in expo go

I'm using react native expo and stuck at this error. Guide me how to fix it?

react-native-mmkv is not supported in Expo Go! Use EAS (expo prebuild) or eject to a bare workflow instead.

here's my expo link
https://snack.expo.dev/@elamandeep/zustand

This isn't a bug. MMKV requires that you use an expo dev client due to its native code requirements.

exactly what @ChronSyn said. Also, this is literally what it says in the error message.

Can config plugins in Expo also be made to work with Expo Go, or is that not possible?

Can config plugins in Expo also be made to work with Expo Go, or is that not possible?

Not possible, as config plugins typically add native code, which requires a new native build

I think I've read something that expo supports native code? Or am I wrong?

I think I've read something that expo supports native code? Or am I wrong?

Yes, Expo can not support native code.
Yes. Expo dev client builds can support custom native code.
Expo Go can not support custom native code.

Thanks for clarifying Beste Grüße,Nils BaumgartnerAm 14.01.2024 um 23:55 schrieb Scott Pritchard @.***>:
I think I've read something that expo supports native code? Or am I wrong?
Yes, Expo can not support native code.
Yes. Expo dev client builds can support custom native code.
Expo Go can not support custom native code.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

All you need to do is:

npx expo prebuild     // This command will create android folder in root
npx expo run-android

All you need to do is:

npx expo prebuild     // This command will create android folder in root
npx expo run-android

That is not the same as running it in Expo Go.

Yes @ChronSyn you're right but it is an alternate way.


Remove Homebrew release tasks

The Pick Homebrew formula is now maintained within the
Homebrew/homebrew-core project and updating it can no longer be done
automatically.

The Arch Linux release tasks do not work very well so I'll follow up with a PR removing those as well.
I think maintaining system packages outside of this repo creates a clear separation of concerns.

LGTM.

Thanks @mike-burns. I'll merge this.


Fixing resolving of Config for projector resolution (fixes #2349)

Now resolves Config.

I do not understand why the config should have to be find here. The config is load in core/base.js. Then is is available in the DS and you should always be able to get it via $scope.config(...).

See #2349

Now the controller accesses the config via $scope.config but skips if $scope.config is not a function. This is the case if loadGlobalData hasn't finished yet, because until then $scope.config is undefined.

Ok, please rebase.

@normanjaeckel Ping

Ok.

Ok.


Switch from tox build wrapper to charmcraft.yaml overrides

As defined by https://github.com/canonical/data-platform-workflows/releases/tag/v22.0.0 to support charmcraftcache-hub

thank you!
looks good except extra files part!
also, you may experience issues with data-platform-workflows because the versions in pyproject.toml and poetry.lock do not match the versions in your yaml files:
https://github.com/canonical/wazuh-dashboard-operator/blob/7bf611fb16e3d22fa0155bc04c5ef9763a702885/pyproject.toml#L103
(data-platform-workflows only supports using the same version for all workflow files & python packages—you may experience issues with mixing versions)
although opensearch-dashboards-operator 2/edge seems to have this same issue (https://github.com/canonical/opensearch-dashboards-operator/blob/9345b74d662c56c61d16e203ff4f28cd1c2d12d2/pyproject.toml#L103) cc @Mehdi-Bendriss @phvalguima, so maybe they got lucky and there aren't compatibility issues

I think I can integrate the changes from upstream whenever they are available and Wazuh upgrades the underlying OpenSearch version, WDYT?


[Test]Get card to update while chess pieces move

I put this up bcos, I wanted to experiment moving a chess piece and sending that action back up to the card. It has failed. I guess bcos when I use chess.js  the import shows some illegal __esModule export.
Note: previously I use cm-chessboard for my library but I wanted to provide move validation and update the board fen string so I needed another library -- chess.js. It seems these libraries don't have the right build that our loader accepts.

AI performed bad and the reason is probably it doesn't understand FEN string very well. Perhaps, we might want to consider the position inside of a containsMany. Note-to-self. So, have 2 way edits. Edit FEN string or Edit contains many and they should be binded. Perhaps, FEN string is a computed perhaps.

Oh gosh.  I am using the wrong chess library

I know this seems a little verbose. The pgn is modelling the state of the board whereas the game moves are modelling the state of the game. In chess its common to have this fork in analysis mode, where there is a game state but you analyse different positions hence the complexity. Ideally, a forked position should be able to be saved in a field linking to another chess game.

This prob can go in.


chore(main): release 1.5.0

:robot: I have created a release beep boop
1.5.0 (2024-03-26)
Features

main: Added changelog to documentation (b06a727)

Bug Fixes

main: Cleaning up images (8b9fab9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/CoreySpohn/lod_unit/releases/tag/v1.5.0 :sunflower:


python38Packages.astropy-healpix: 0.4 -> 0.5

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.8-astropy-healpix/versions.
meta.description for python38Packages.astropy-healpix is: '"BSD-licensed HEALPix for Astropy"'.
meta.homepage for python38Packages.astropy-healpix is: '"https://github.com/astropy/astropy-healpix"

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.5 with grep in /nix/store/ymv3cjfld51mdljdxa5xh35888bkfg3k-python3.8-astropy-healpix-0.5

directory tree listing: https://gist.github.com/0372a571314366228f77dfa8105c6ab9

du listing: https://gist.github.com/bea839fb48401a4ddf11551f87e58f0b

Rebuild report (if merged into master) (click to expand)

21 total rebuild path(s)
6 package rebuild(s)
5 x86_64-linux rebuild(s)
5 i686-linux rebuild(s)
5 x86_64-darwin rebuild(s)
6 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
python37Packages.aplpy
python37Packages.astropy-healpix
python37Packages.reproject
python38Packages.aplpy
python38Packages.astropy-healpix
python38Packages.reproject

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ymv3cjfld51mdljdxa5xh35888bkfg3k-python3.8-astropy-healpix-0.5 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A python38Packages.astropy-healpix https://github.com/r-ryantm/nixpkgs/archive/bd2c0ac8ac1e55da33ac9d4aef719bc02dce31e1.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ymv3cjfld51mdljdxa5xh35888bkfg3k-python3.8-astropy-healpix-0.5
ls -la /nix/store/ymv3cjfld51mdljdxa5xh35888bkfg3k-python3.8-astropy-healpix-0.5/bin

cc @smaret for testing.

nix-review pr succeeds on Debian:
5 package were build:
python37Packages.aplpy python37Packages.astropy-healpix python37Packages.reproject python38Packages.astropy-healpix python38Packages.reproject

On macos, it gives the following error:
1 package failed to build:
python37Packages.aplpy

4 package were built:
python37Packages.astropy-healpix python37Packages.reproject python38Packages.astropy-healpix python38Packages.reproject

The failure isn't related to this update but to another dependency of aplpy, python37-docker, that fails to build:
builder for '/nix/store/320aynm8i2bnks3ijcyhv69x6k9bcvbf-python3.7-docker-4.1.0.drv' failed with exit code 1; last 10 log lines:
          self.addCleanup(shutil.rmtree, base)
          for d in ['foo', 'bar']:
              os.makedirs(os.path.join(base, d))
          sock = socket.socket(socket.AF_UNIX)
          self.addCleanup(sock.close)
  >       sock.bind(os.path.join(base, 'test.sock'))
  E       OSError: AF_UNIX path too long

I would rather wait for this failure to be fixed before merging this PR.

darwin builds can be addressed outside this PR, for the version bump of this package, it's good


mix ecto.migrate gives an error because of callback in ecto

I get the following error when running mix ecto.migrate. I think the migration works as it should, but the command gives an error exitcode. This is a problem for me when I build do builds.
mix ecto.migrate

19:16:03.206 [info]  == Running MyApp.Repo.Migrations.CreatePosts.change/0 forward

19:16:03.208 [info]  == Migrated in 0.0s
** (UndefinedFunctionError) undefined function: Mongo.Ecto.stop/2
    (mongodb_ecto) Mongo.Ecto.stop(#PID<0.151.0>, 5000)
    (ecto) lib/mix/ecto.ex:75: Mix.Ecto.ensure_stopped/2
    (mix) lib/mix/cli.ex:55: Mix.CLI.run_task/2
    (elixir) lib/code.ex:363: Code.require_file/2

@blacksails could you try with current master?

will do :+1:

Works like a charm. Good job :)

I am having the same problem:
$ mix ecto.migrate

15:10:41.178 [info]  Already up
** (UndefinedFunctionError) undefined function Mongo.Ecto.stop/3
    (mongodb_ecto) Mongo.Ecto.stop(MyApp.Repo, #PID<0.176.0>, 5000)
    (ecto) lib/mix/ecto.ex:94: Mix.Ecto.ensure_stopped/2
    (elixir) lib/enum.ex:604: Enum."-each/2-lists^foreach/1-0-"/2
    (elixir) lib/enum.ex:604: Enum.each/2
    (mix) lib/mix/cli.ex:58: Mix.CLI.run_task/2
    (elixir) lib/code.ex:363: Code.require_file/2

In my mix.exs of my Phoenix Framework project, I'm pulling what based on the version listed in this repo's source should be the master:
{:mongodb_ecto, ">= 0.1.2"}

I've cleaned and pulled dependencies again, but the same thing occurs. Is there some other way to get the master into a Phoenix project?

Note on my above post...I have now changed my dependency to this:
      {:mongodb_ecto, github: "michalmuskala/mongodb_ecto"},

And re-pulled dependencies, and I'm still getting exactly the same error.

I suppose I should have listed all of my dependencies....here they are:
  defp deps do
    [
      {:phoenix, "~> 1.1"},
      {:phoenix_ecto, "~> 2.0"},
      {:phoenix_html, "~> 2.3"},
      {:phoenix_live_reload, "~> 1.0", only: :dev},
      {:mongodb_ecto, github: "michalmuskala/mongodb_ecto"},
#      {:mongodb_ecto, "~> 0.1.2"},
      {:cowboy, "~> 1.0"},
      {:exrm, "~> 1.0.0-rc7"},
      {:gettext, "~> 0.9"}
    ]
  end

I am still getting the same error.


Graph algorithms for HetNets

Hi all,
since you are working with heterogeneous information networks, I think it is interesting to point out a couple of works that might help you in this regard.

Some work on metapaths that are abstract description of paths surrounding a specific node.
They are useful in order to compare nodes in the graph.

Meng, C., Cheng, R., Maniu, S., Senellart, P., & Zhang, W. (2015). Discovering Meta-Paths in Large Heterogeneous Information Networks (pp. 754–764). Presented at the the 24th International Conference, New York, New York, USA: ACM Press. http://doi.org/10.1145/2736277.2741123
Sun, Y., 0001, J. H., Yan, X., Yu, P. S., & Wu, T. (2011). PathSim - Meta Path-Based Top-K Similarity Search in Heterogeneous Information Networks. Pvldb.

Graph embeddings: project the graph into a multi-dimensional space and perform similarity operations in that space (e.g., k-means):

Chen, Y., & Wang, C. (2017). HINE: Heterogeneous Information Network Embedding. In Database Systems for Advanced Applications (Vol. 10177, pp. 180–195). Cham: Springer International Publishing. http://doi.org/10.1007/978-3-319-55753-3_12
metapath2vec: https://www.youtube.com/watch?v=t9yujaCpX_E

Some work on node classification:

http://www.cs.cmu.edu/~deswaran/papers/vldb17-zoobp.pdf
These are just a few, but if you tell me what you need there might be more customized ideas out there!
Regards,
Davide

Adding another reference to a related work named MetaExp mentioned to me by @jexp. From their manuscript titled MetaExp: Interactive Explanation and Exploration of Large Knowledge Graphs:

We present MetaExp, a system that assists the user during the exploration of large knowledge graphs, given two sets of initial nodes. At its core, MetaExp presents a small set of meta-paths to the user, which are sequences of relationships among node types. Such meta-paths do not overwhelm the user with complex structures, yet they preserve semantically-rich relationships in a graph. MetaExp engages the user in an interactive procedure, which involves simple meta-paths evaluations to infer a user-specific similarity measure. This similarity measure incorporates the domain knowledge and the preferences of the user, overcoming the fundamental limitations of previous methods based on local node neighborhoods or statically determined similarity scores. Our system provides a userfriendly interface for searching initial nodes and guides the user towards progressive refinements of the meta-paths. The system is demonstrated on three datasets, Freebase, a movie database, and a biological network.


Nuxt SPA build issue

When using a SPA built with Nuxt, the site does not load correctly - it just shows a loading/error page, and a couple of errors appear in the console.
This happens using both embeddedServer.active true and false, as well as running the tauri app in dev mode, and ensuring the build.devPath points to a local server (I.e. npm.im/serve) running the exact same dist folder (although @nklayman said that was not the case for him).
The logs that appear are as follows:
embeddedServer.active: true
SCRIPT5022: SCRIPT5022: Expected identifier, string or number
embeddedServer.active: false
HTML1507: Unexpected character: U+0000 NULL. Null characters should not be used.
To Reproduce

Create a new project with NuxtJS, and make sure to select spa mode.
Build the project with yarn build, and either init tauri, or copy the dist folder to a tauri project.
Configure tauri.conf.json to use the dist folder as distDir.
Build the tauri project, and run it.

Expected behavior
The SPA site should load correctly inside the app, as happens with a usual VueJS app.
Screenshots
Loading/Error screen:

Platform and Versions:
Operating System - Windows_NT(10.0.19541) - win32/x64
Node.js environment
Node.js - 13.5.0
tauri.js - 0.3.1
Rust environment
rustc - 1.40.0
cargo - 1.40.0
tauri-cli - 0.1.2
Global packages
NPM - 6.13.4
yarn - 1.21.1
App directory structure
/src-tauri
App
tauri - 0.3.1
mode - no-server
build-type - bundle
CSP - default-src res: blob: data: filesystem: ws: http: https: 'unsafe-eval' 'unsafe-inline'
Windows - Edge
distDir - dist
devPath - http://localhost:5000

we released v0.3.2 of tauri at crates yesterday - however that treated a different problem of the event API being improperly addressed in rust - so that probably won't have an impact here. I know @lucasfernog was going to look into a basePath issue. Maybe he has some more insight.

I'm checking it tonight, I don't know how Nuxt dist works.

Adding this to the nuxt.config.js file, it "works":
build: {
    /*
    ** You can extend webpack config here
    */
    extend (config, ctx) {
      if (!ctx.isDev && ctx.isClient) {
        const tauriConfig = require('./src-tauri/tauri.conf.json')
        if (!tauriConfig.tauri.embeddedServer.active) {
          const TauriRequirePlugin = require('@tauri-apps/tauri-webpack/plugins/tauri-require').plugin
          config.plugins.push(new TauriRequirePlugin())
          console.log(config)
          config.output.publicPath = '_nuxt/'
        }
      }
    }
  }

now, it starts on a 404 page, but I can click on "Back to home page" and then it goes to the right page. Can you give me a hint on how can that happen? Maybe I didn't load the routes before Nuxt kicks in?

Here's my work so far: #306

I'm closing this issue as stale, we've moved a lot recently. Please feel free to open a new issue if the problem persists.

Did you have luck getting nuxt going with tauri @thetre97  ?
I was thinking of trying the nuxt generate, for the static mode maybe

@gregbenner I haven't had reason to try it recently - but as said above, a lot has changed since.
Their discord is super helpful if you have any issues...


gui: bump iced 0.9

Have you checked doing a reproducible build with the new version?

./contrib/reproducible/guix/guix-build.sh did not failed


Mnemonic Backup Word BIP39 support

Zcoin has since 0.13 had HD key support.
With @riordant recent updates, Sigma updates are also now backed up using the same seed.
To build in BIP39 support direct into the QT wallet when making a new wallet.
https://github.com/bitcoin/bips/blob/master/bip-0039.mediawiki
Take a note on this though:
https://bitcoin.stackexchange.com/questions/87407/can-a-bip39-12-word-mnemonic-be-used-to-restore-a-wallet-through-bitcoin-cli
There are also various derivation paths.
BIP32 BIP44 BIP49 BIP84 BIP141
so some research needs to be done on which has the most wide support.
A good summary is this.
https://iancoleman.io/bip39/?

Solved by #641