[RDBMS] Improvements to Management Plane for MySQL and PostgreSQL

Description

Note: Recreating this PR to update the history/rebase with hotfix changes for rdbms.

This PR improves existing commands for the 'Flexible Servers' APIs within the RDBMS component.
Breaking changes:
Parameter name changes

-- time to --restore-in-time for restore command
Why: PM requested change to make it clearer for customer.  Documentation on our side will also be updated.
--force to --yes
Why: Consistency across different delete prompt commands for customer experience.
--server-name to --name for CRUD operations
Why: PM requested change for consistency
--name to --rule-name in firewall rule
Why: PM requested for clearer & self-explanatory params for customer experience.

Changes include:

Update MySQL Connection string in create command
Replace server name check with CheckNameAvailability API for MySQL & PostgreSQL
Update help text and parameters for commands to be more descriptive and helpful to user (commands/arguments: vnet related, replica, firewall, restore).  These were to improve customer experience.
Update validators to use result from list-skus commands/API.
Enabled more tests for code coverage and command testing (validator & replica tests)
Added new recording files with default locations for MySQL & PostgreSQL

Testing Guide

To run tests (these cover existing commands and include new tests):
azdev test rdbms

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR title and description has followed the guideline in Submitting Pull Requests.

[X] I adhere to the Command Guidelines.

Please merge this with a squash commit :)

RDBMS

/azp run

Reverted last two accidental commits.

could you refine description part according to https://github.com/Azure/azure-cli/tree/dev/doc/authoring_command_modules#submitting-pull-requests?

could you refine description part according to https://github.com/Azure/azure-cli/tree/dev/doc/authoring_command_modules#submitting-pull-requests?

Made the update thanks!


[CELEBORN-1363] AbstractRemoteShuffleInputGateFactory supports celeborn.client.shuffle.compression.codec to configure compression codec

What changes were proposed in this pull request?
AbstractRemoteShuffleInputGateFactory supports celeborn.client.shuffle.compression.codec to configure compression codec.
Why are the changes needed?
AbstractRemoteShuffleInputGateFactory only supports LZ4 compression codec via hard code at present. AbstractRemoteShuffleInputGateFactory should support celeborn.client.shuffle.compression.codec to configure compression codec like ZSTD etc.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
No.

Ping @FMX, @RexXiong, @TanYuxin-tyx.


Order of events doesn't match the order of records in a partition

I have a single-partition topic. I want to use the EventBridge sink connector to send events to EventBridge and SQS. However, the messages in SQS seem in random order.
Steps to reproduce the behaviour:

Create an EventBridge Rule and set SQS or SQS FIFO as a destination.
Produce some records to the Kafka topic.
Start the connector.
Receive a message from SQS.
Records are not in order.

Expected behaviour
Events are processed in the same order as in a Kafka topic. They are ordered by offset.
Additional context
Is it by EventBridge design, or can the connector be tweaked to support ordering?

Thx @consal for reaching out! I'm assuming you're using SQS FIFO queues? Can you elaborate on why you need strict ordering for your particular use case? I'm just curious, not saying you don't need it ;) (see comment below)
Your understanding is correct: today EventBridge Event Buses do not maintain order. If you have strict ordering requirements we generally propose two options in EventBridge:

Use EventBridge Pipes if it suits your requirements
Implement a custom re-sequencer between [event bus] - [re-sequencer] - [your target] e.g., using Lambda with a state store
Handle out-of-order events in the consumer*

* To be honest with you, IMHO this is the only way to deal with this issue in general. For example, there are various situations which can lead to events for the same record key in Kafka partitions to arrive out of order e.g., not using transactional consumer with session handling, application concurrency bugs in the producer, producer reads from external source and duplicates events leading to out of order, etc. Following general code design and Domain Driven Design practices, the consumer should be prepared for these situations (incl. duplicated events) and handle those, which by the way will also increase overall throughput in your system. However, I do understand that sometimes it is not possible to defer that logic to the consumer.

In a Kafka topic, I have events in the exact order as they happened in the source system. I'm migrating an app to use an SQS queue instead of Kafka, and I need events in the same order as in the Kafka topic.
Now that I understand that the EventBridge connector doesn't support that, I'll look into alternative solutions. Unfortunately, EventBridge pipes don't support Kafka as a source in my region.

It's Zurich. Thank you for the explanation 😊 Please let me know if you have any updates on the feature's availability.

Passed to product team, thank you! Closing this for now.

Thank you. Is there any timeline for introducing it in Zurich?

Can't comment on this publicly here. Feel free to reach out to the AWS support team which will get you in touch with the service team.


Make loading snapshots parallel

What does this PR change? What problem does it solve?
Loads snaphots files in parallel, like already done by restic check.
This speeds up many commands if there is a large number of snapshots in the repository.
Note that this should be also included in #3121 (either here or there depending on the order of PR merge)
Was the change discussed in an issue or in the forum before?
The issue came up quite often, but so far only the possibility to change the repo format (snapshot blobs) has been discussed.
Checklist

[x] I have read the Contribution Guidelines
[x] I have enabled maintainer edits for this PR
I have not added tests for all changes in this PR, but existing tests still pass
I have not added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Hmm, this reminds me of https://github.com/MichaelEischer/restic/commit/3da6b9de9d51290a868c034418f662cf8d6a2250 from my https://github.com/MichaelEischer/restic/commits/streamtrees branch. From what I see the main difference is the function interface func StreamSnapshots(ctx context.Context, wg *errgroup.Group, repo restic.Repository, excludeIDs restic.IDSet) <-chan SnapshotItem. The main difference is that I've used a channel instead of a callback (the parameters themselves are equivalent). Using a channel avoids using a mutex but requires a bit more ceremony, see https://github.com/MichaelEischer/restic/commit/7c68668ef76bde63347a2094a8d31bbbd2eecfaf . @fd0 do you have a preference?
I'd also prefer to not add additional code to the restic package, but I'm not sure whether a new "parallel" package would be a solution.

@MichaelEischer Thanks for the hint - I didn't fully remember what was included in your change.
Besides the decision about how to handle this (I'm open to both possibilities) we should add this parallel loading of snapshots not only to the processing of all tress. All snapshots are also read is for instance also used when searching for the latest snapshot which is used in several places. This is why I proposed to change it within internal/restic.

@MichaelEischer I think I prefer the callback variant. This allows easier error handling IMHO. :)

@aawsome Most snapshot users use the Find*Snapshot functions, see https://github.com/MichaelEischer/restic/commit/44dec3328f17de18764d12d9a77b7fb1b5f9a925 . LoadAllSnapshots is only used by prune (see https://github.com/MichaelEischer/restic/commit/1bd0370e6e34b9ab8662eb9126fc67486fffe8f7), that change is relatively independent of my StreamTrees PR (well, the merge conflict there should be rather easy to resolve).
We could of course move the function to find snapshots to a different package. That way we would limit the restic package to fundamental data structures and put other helper somewhere else.

@MichaelEischer Thanks for asking. I was in fact waiting for directions and just realized that I didn't understand the hint in your comment last week. Thanks for clarifying your proposal! I now changed debug and prune as you suggested (the parallelized tree walk needs to be adjusted in your tree PR).
I think this is now ready for merging and IMO should be merged. Sorry again for "stealing" your work, I didn't realize that you were already working on this topic...


Page is loading differently each time execution

Complete page loading different ui some time time it load full sometime not …please help to resolve this

please submit a link to your repository and reproduction scenario of your issue

gg

hi if u have admin right to modify content ...please delete the information
on this ticket
6929


Room.findAll() get error when rooms more than 400

Enviroment
Starting Wechaty v0.13.72 with Node.js v9.6.1 ...

<EMAIL_ADDRESS>doctor /wechaty
npm run check-node-version && ts-node bin/doctor

<EMAIL_ADDRESS>check-node-version /wechaty
check-node-version --node ">= 8.5"

Wechaty Doctor

Wechaty version: #git[b5c0c14]
Linux x64 version 3.16.0-4-amd64 memory 353/998 MB
Docker: true
Node version: v9.6.1
Tcp IPC TEST: PASS

PROBLEM
When the bot has too many rooms(400-500), when I try Room.findAll() will get the following error. While if the room is under 200, all works well.
17:52:35 WARN Room parse() on a empty rawObj?
17:52:35 ERR Room contactGetter(@@c5529e069a00362de84bcd47411e5937b42aa5cced47485f6dd88e4a86fc3aec) exception: no this.obj set after contactGetter
17:52:35 WARN Room parse() on a empty rawObj?
17:52:35 ERR Room contactGetter(@@f06db6946a1b6700c3ce5fc316236bcffc07e8085d2c30b8eef0130b246656e3) exception: no this.obj set after contactGetter
^C17:52:36 WARN PuppetWebBridge proxyWechaty() exception: Protocol error (Runtime.evaluate): Target closed.
17:52:36 WARN PuppetWebBridge proxyWechaty() exception: Protocol error (Runtime.evaluate): Target closed.
17:52:36 WARN PuppetWebBridge proxyWechaty() exception: Protocol error (Runtime.evaluate): Target closed.
17:52:36 WARN PuppetWebBridge proxyWechaty() noWechaty exception: Error: Protocol error (Runtime.callFunctionOn): Session closed. Most likely the page has been closed.
17:52:36 WARN PuppetWebBridge proxyWechaty() noWechaty exception: Error: Protocol error (Runtime.callFunctionOn): Session closed. Most likely the page has been closed.
17:52:36 WARN PuppetWebBridge proxyWechaty() noWechaty exception: Error: Protocol error (Runtime.callFunctionOn): Session closed. Most likely the page has been closed.
17:52:36 WARN PuppetWebBridge proxyWechaty() noWechaty exception: Error: Protocol error (Runtime.callFunctionOn): Session closed. Most likely the page has been closed.

REPRODUCE
snip code as follows
import { Wechaty, Room } from 'wechaty'
const bot = Wechaty.instance({ profile: 'sara' })

bot
.on('scan', (url, code) => {
  if (!/201|200/.test(String(code))) {
    let loginUrl = url.replace(/\/qrcode\//, '/l/')
    console.log(`${url} \n ${code}`)
  }
})
.on('login', async function (user) {
   console.log(`${user}login`)

  setTimeout(async function() {
    console.log('加载群组信息如下')
    const list = await Room.findAll()
    console.log(list)
  }, 10 * 1000)
})
.on('logout', function (user) {
   console.log(`${user.name()} logout`)
})
.on('error', err => {
   console.error(`@@@@@@@@@@@@ exit for ${JSON.stringify(err)} @@@@@@@@@@@@`)
})
.init()
.catch(e => {
  console.error('Bot', 'init() fail: %s', e)
})

This docker  version can works well
zixia/wechaty@sha256:4406ddb154bb1b90329ba6a94abe812ce8e3f5a0c983f7ed2985b03304517642

After v0.16, the puppet-puppeteer had been rewriten.
So I'll close issue, and please file a new issue at wechaty-puppet-puppeteer if you stil have this problem, thank you very much.


'nitropy nk3 update' prints a lot of spsdk debug messages to the console

A lot of these debug messages are printed to the console, which means the progress bar is not visible and it quickly fills up the scrollback buffer of the terminal:
DEBUG:spsdk.mboot.interfaces.usb:OUT[60]: 02, 00, 38, 00, A4, 40, 8B, BD, C3, 25, 57, CD, 18, 95, 86, 09, FC, B0, B3, 8F, 4B, 0E, 80, 26, 35, 4D, 48, 01, D6, 50, B1, 41, 6E, C7, 86, CF, F3, 31, A5, 77, A7, 94, 52, C3, FD, 69, 41, DF, 93, 0A, 6A, 39, D1, 7A, D0,
10, A5, 6D, 55, 48 (22642ms since start, usb.py:125)

spsdk seems to default to no level (NOTSET) which means it logs everything
https://github.com/nxp-mcuxpresso/spsdk/blob/2facbebd4e1f023099557803d163820e07937d1b/spsdk/__init__.py#L111
I found a way to set this to INFO, create ~/.spsdk/logging.yaml which contains:
version: 1
incremental: True
loggers:
  spsdk:
    level: INFO

After this only INFO level spsdk messages are printed:
$ nitropy nk3 update firmware-nk3-v1.4.0.zip
Logging config loaded from /home/jernej/.spsdk/logging.yaml
Command line tool to interact with Nitrokey devices 0.4.36
Current firmware version:  v1.4.0
Updated firmware version:  v1.4.0
The version of the firmware image is the same as on the device.  Do you want to continue anyway? [y/N]: y

Please do not remove the Nitrokey 3 or insert any other Nitrokey 3 devices during the update. Doing so may damage the Nitrokey 3.
Do you want to perform the firmware update now? [y/N]: y

Please press the touch button to reboot the device into bootloader mode ...

INFO:spsdk.mboot.mcuboot:Connect: MCU HID GENERIC DEVICE (0x20A0, 0x42DD) path=b'/dev/hidraw5'
INFO:spsdk.mboot.mcuboot:CMD: ReceiveSBfile(data_length=451152)
INFO:spsdk.mboot.mcuboot:CMD: GetProperty('MaxPacketSize', index=0)
INFO:spsdk.mboot.mcuboot:CMD: Max Packet Size = 56
Perform firmware update: 100%|█████████████████████████████████████████████████████████████████▊| 450k/451k [00:05<00:00, 76.9kB/s]INFO:spsdk.mboot.mcuboot:CMD: Successfully Send 451152 out of 451152 Bytes
INFO:spsdk.mboot.mcuboot:CMD: Reset MCU
INFO:spsdk.mboot.mcuboot:Closing: MCU HID GENERIC DEVICE (0x20A0, 0x42DD) path=b'/dev/hidraw5'
Perform firmware update: 100%|██████████████████████████████████████████████████████████████████| 451k/451k [00:05<00:00, 75.6kB/s]
Finalize upgrade: 100%|███████████████████████████████████████████████████████████████████████████| 100/100 [00:01<00:00, 83.54%/s]

It might make sense to set this to an even higher severity like WARNING, ERROR or CRITICAL. If debug messages are wanted, they could be saved to a log file in a tmpfs (like nitropy does to /tmp/nitropy.log.*) by adding a new handler to logging.yaml.
Of course this should be configured by nitropy itself, if it can set imported spsdk logger config itself so that this config file is not needed. I'm not sure how or if a change in spsdk is needed.

https://github.com/nxp-mcuxpresso/spsdk/issues/54

This is probably because I am using spsdk 1.10.0 instead of the pyproject.toml specified 1.7, the logger was added in spsdk 1.9.0. So it's not a problem when using the "correct" dependency versions, but will need to be adressed when updating the spsdk dependency.

This will be fixed in https://github.com/Nitrokey/pynitrokey/pull/364.


Revert FileSystemError changes due to Windows breakage

Related revert in TSC is in https://github.com/apple/swift-tools-support-core/pull/169.

apple/swift-tools-support-core#169
@swift-ci please smoke test

@MaxDesiatov apple/tools-support-core#169 is not a revert, its a fix - I don't think that this combination of changes is going to work

Sorry, I didn't notice. This PR is irrelevant then.


Add the limit returned by service to edit-scope's queryInfo ?

currently we can get total from using edit-scope prop of FeathersVuexFind component, but we can't get the limit sent back by the server as easily. I know it's not that important , because usually limit is somewhat hard coded on both sides, but I find it a bit useful for testing some pagination components quickly.
It would be nice to have it.

After reading the vuex code and doing some logging I found that this is already available, but not in queryInfo, inside pageInfo.pageParams.$limit.


DUPLO-26210 TF:Ingress with TLS: change not detected when removed tls block

Overview
TLS block when removed from Ingress resource and plan diff not appearing
Learning :- We cant keep computed true for optional block if nested attribute is of required scope.
Summary of changes
Removed computed true from TLS block
This PR does the following:

Removed computed true from tls block
...

Testing performed

[ ] Using unit tests
[ ] Manually, on my local system
[ ✓] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-26210 TF:Ingress with TLS: change not detected when removed tls block


Why do some fonts not provide correct bounding box

Note: I've edited this question as I see now that it's a diff between svg.js and opentype.
If I draw some text using SVG.js which I believe will render a font using CSS on SVG (rather than vector paths) and also print the font with OpenType I get two different lengths of font using same font size. This only happens on some typefaces such as the one I am attaching (temp link).
https://www.dropbox.com/s/p6nzm0zewhfk7wb/franklin_gothic_demi_cond.woff2?dl=0
The top line is OpenType and the bottom is SVG.js

What data in the font could be doing this and what might be the best way to correct this ( I can dit the font files if needed)
Thanks for any ideas.

My bad. nvrmind.


Can you made Preview Pane that is not integrated to Files

What's the Problem?
No response
Solution/Idea
I sometimes use dual pane for multitasking but at that time opening preview pane consume more space as it is intergrated to Files.
Can you make Preview Pane that will work as a separate window, it will be helpful.
Alternatives
I sometimes use dual pane for multitasking but at that time opening preview pane consume more space as it is intergrated to Files.
Can you make Preview Pane that will work as a separate window, it will be helpful.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
<IP_ADDRESS>
Windows Version
Window 11 21H2 22000.469
Comments
No response

Perhaps you could install QuickLook which works with Files and shows a preview as a separate window by pressing Space.


Update codemeta.json

coverage: 86.644% (-0.3%) from 86.977% when pulling 5a3ab91fe7a1a4a07d8c24baf6f261097847b0b6 on apdavison-patch-1 into 8fdca601b57eab746b694e05efa0663c9435cde4 on master.


404 /pulse

Pulse Version
1.0@beta-2
Laravel Version
10.34.2
PHP Version
8,2.12
Database Driver & Version
Mariadb
Description
I just installed Pulse, I think I followed all the instructions

composer require laravel/pulse
set up db and set connection in .env
php artisan migrate

/pulse exists in my Routes when I run Route:list
I have cleared the route cache.
I get a 404 when I visit /pulse on my server.
I have tried changing the path in the config file to no avail.
Is there a simple solution, have I missed a step?
Steps To Reproduce
All I did was

composer require laravel/pulse
set up db and set connection in .env
php artisan migrate

This might well be the case, but I have a catchall wildcard route at the end of all my routes, so it matches anything that doesnt match my routes.   I'm unsure how to get around this.
I have had other third party packages that register Routes that seem to work ok, is there a reason pulse is different?


A9G giving fixed location (of hongkong)

Hello There,
A9g pudding board stuck at giving FIXED LOCATION
I just want to know this issue is related to Hardware or Software?
or is this because of damaged antenna ?

check NMEA or demo output if  fixed? fix mode should >=2

yes, @Neutree actually that's what I wanna ask.
I'm always getting GPS data with GPS FIX MODE:1 AND BDS FIX ** MODE: 1** I already read other Issues.:)
that's why I'm asking is there any problem of CODE or Hardware or antenna?
here is the data of one A9G pudding Hardware, this remains always the same(HONGKONG city location).
GPS fix mode:1, BDS fix mode:1, fix quality:0, satellites tracked:0, GPS sates total:0, is fixed:no fix, coordinate:WGS84, Latitude:22.606128, Longitude:113.839978, unit:degree,altitude:0.000000
and this one is another hardware OUTPUT this also remains the same.
GPS fix mode:1, BDS fix mode:1, fix quality:0, satellites tracked:0, gps sates total:0, is fixed:no fix, coordinate:WGS84, Latitude:23.194863, Longitude:72.637818, unit:degree,altitude:0.000000
so can you please tell me what is the issue?

the log shows us the module didn't fix, means the latitude and longitude is invalid,
please place you antenna in open area and try again
And you can show your board and antenna here with picture, it's more helpful for us to resolve it

yes fix quality:0 means GPS is not locked so it's giving invalid lat/lon. My device is working perfectly even with passive GPS antenna. fu-techiot I think there is a problem with antenna.

**thank you ** @Neutree @updesh31 For your quick response,
yes, I'm able to get proper GPS Location in OPEN area, I think there are a power issue and connection issues.

I had the same problem, I solved it with an external power supply and put the antenna outside the window.
<EMAIL_ADDRESS>

i have the same issue. does it compulsory to place the antenna in window or under open sky?
If we have the antenna inside the room, can we not get the updated the GPS coordinated?
How this antenna function is different than the working of a cellphone GPS.

If you have to stick a GPS antenna outside of your window to get a signal then yes, that is compulsory in your case.
Phones use other sources of information to get your location, like cell towers and nearby WiFi access points. Use an app like GPS Status to see how many GPS satellites your phone can really see and lock on to.

If you have to stick a GPS antenna outside of your window to get a signal then yes, that is compulsory in your case.
Phones use other sources of information to get your location, like cell towers and nearby WiFi access points. Use an app like GPS Status to see how many GPS satellites your phone can really see and lock on to.

Is there something that I can program in the module to know the information about the nearest cell towers?


[BACKLOG-40847] Unit test fixes for JDK17

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 15m 17s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
plugins/put-a-file-with-ftp/impl

:ok_hand: All tests passed!
Tests run: 5, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Enable swapfile on dchi

Demo server (try.gitea.io aka dchi) needs a swapfile so Gitea doesn't crash on start...
If someone knows how to pull a git-repo instead of copy-pasting them here, do tell. Until then I've just copy-pasted this one which I hope works :trollface:  https://github.com/kamaln7/ansible-swapfile

Maybe relaunch the machine with a swap partition directly? Why does it fail now? It worked in the past totally fine.

@tboerger Relaunching the machine means I need to backup /etc/defaults/secrets and add somehow (And I have no clue how to do all of that :trollface: )
It fails because of OOM-killer, I haven't looked into where it fails, but my guess is it's loading a bunch of repos into memory 😒 Another possibility is SQLite being the culpit...
2017/05/14 21:40:18 [I] Log Mode: File(Info)
2017/05/14 21:40:22 [I] XORM Log Mode: File(Info)
2017/05/14 21:40:22 [I] Cache Service Enabled
2017/05/14 21:40:22 [I] Session Service Enabled
2017/05/14 21:40:22 [I] Mail Service Enabled
2017/05/14 21:40:22 [I] Notify Mail Service Enabled
2017/05/14 21:40:22 [I] Git Version: 2.11.2
2017/05/14 21:40:22 [...gitea/models/repo.go:155 NewRepoContext()] [E] Failed to set git user.email(fork/exec /usr/bin/git: cannot allocate memory):


The error occurred while executing the backward method using the LinkAsTorchModel class

I am trying to call a chainer model using the LinkAsTorchModel class.
However, I am getting the following error while executing the backward method.
Can you give me any advice?

I want to call a chainer model from a pytorch trainer.
I refer to the following site to implement the code.
https://chainer.github.io/migration-guide/
But that is difficult for me.
I need more sample codes.

Can you show the code of how do you create the model and how do you call backward?

Thank you for your comment.
I copied my code below.
chainer device
            import chainer
            self.chainer_device = chainer.get_device(self.device)

device
        if self.device >= 0:
            self.device = 'cuda'
        else:
            self.device = 'cpu'
        self.device = torch.device(self.device)

MLP class
class MLP(chainer.Chain):

    def __init__(self, n_mid_units=100, n_out=10):
        super(MLP, self).__init__()
        with self.init_scope():
            self.l1=L.Linear(None, n_mid_units)
            self.l2=L.Linear(None, n_mid_units)
            self.l3=L.Linear(None, n_out)

    def forward(self, x):
        h1 = F.relu(self.l1(x))
        h2 = F.relu(self.l2(h1))
        return self.l3(h2)

The sub class of MLP and classifier
from myfw.mychainer.model.MLP import MLP

class MLPAndClassifier(MLP):

    def forward(self, x):
        y = super(MLPAndClassifier, self).forward(x)
        return F.log_softmax(y)

create model
            from myfw.mychainer.model.MLPAndClassifier import MLPAndClassifier
            self.model = MLPAndClassifier()

            self.input_size = (28 * 28,)

wrap  model
            import chainer_pytorch_migration as cpm
            self.model.to_device(self.chainer_device)
            self.model(self.chainer_device.xp.zeros((1,) + self.input_size).astype('f'))
            self.model = cpm.LinkAsTorchModel(self.model)

set  optimizer
create trainer
        self.trainer = create_supervised_trainer(self.model, self.optimizer, F.nll_loss, device=self.device)

set event handler
run
        self.trainer.run(self.train_loader, max_epochs=self.max_epochs)

is F.nll_loss a chainer or a torch function?

Also, when wrapping a chainer model with LinkAsTorchModel you should use a torch manual training loop instead of the chainer trainer. The optimizer needs to be a torch optimizer too.
https://towardsdatascience.com/the-pytorch-training-loop-3c645c56665a

import torch function at the top of the file.
# import
import torch
import torch.nn.functional as F
from torch.utils.data import DataLoader
from torch import optim

I have already set the torch optimizier.
        self.optimizer = optim.SGD(self.model.parameters(), lr=self.lr, momentum=self.momentum)

Can you please share a gist with the complete example so I can download it and run it straight away?
Thank you

I'll upload my code tonight.

I uploaded my codes to the github.
https://github.com/take0212/pytorch-chainer-combination
Could you please check them?

This case is the contrary
for LinkAsTorchModel the migration guide says
The graph (forward/backward) must be constructed and traversed in Chainer
This means that the loss function must be a chainer function too.
You need to pass a custom step function to your ignite trainer to deal with the loss calculation here as in chainer models the loss is the return value of the Link forward
https://pytorch.org/ignite/concepts.html#engine

Thank you for your kind support.
I will confirm it within a few days.

I set the chainer loss function.
        from chainer.functions.loss.softmax_cross_entropy import softmax_cross_entropy
        self.trainer = create_supervised_trainer(self.model, self.optimizer, softmax_cross_entropy, device=self.device)

I can't understand the following comment.

You need to pass a custom step function to your ignite trainer to deal with the loss calculation here as in chainer models the loss is the return value of the Link forward

So the following error happens.

I implemented the custom process function to convert Tensor to cupy and passed it to ignite.engine.Engine.
My sample code can run without error.
But in fact, it can't be learned at all.
I updated my code to my github.
Could you check it again?
I would like to use Chainer model in PyTorch code(torchvision.datasets and ignite.engine).
Is it possible?
Should I convert torch.utils.data.DataLoader to chainer.iterators.SerialIterator instead of passing the custom process function to ignite.engine.Engine in order to convert a Tensor to a cupy?
I also imported pytorch-pfn-extras.
I use chainer pytorch_pfn_extras.training.extensions and I can evaluate the model in case of PyTorch model and PyTorch code(torchvision.datasets and ignite.engine).
I also use pytorch_pfn_extras.nn.LazyLinear.
Thank you.

Sorry, the code is difficult to follow.
Can you create a single .py file without all the boilerplate code that reproduces the error?

I am currently referencing the sample code below.
https://gist.github.com/emcastillo/8be64990940ce23147cc08584bea985b

The issue seems solved so let us close it.
Thanks

I published it.
https://github.com/take0212/pytorch-chainer-combination


Cannot install Split-diff

I am on Mojave and when I try to install split-diff 1.6.1 package it's giving me the following error. How can I resolve it?
Installing<EMAIL_ADDRESS>failed.Hide output…
npm ERR! code UNABLE_TO_GET_ISSUER_CERT_LOCALLY
npm ERR! errno UNABLE_TO_GET_ISSUER_CERT_LOCALLY
npm ERR! request to https://registry.npmjs.org/diff failed, reason: unable to get local issuer certificate
npm ERR!      /Users/zzzz/.atom/.apm/_logs/2020-01-13T15_03_54_386Z-debug.log

This error comes up when I try to install any of the packages


TypeError: Cannot read properties of null (reading 'useContext')

[x] Check if updating to the latest Preact version resolves the issue

Describe the bug
Hello,
I'm not able to run Preact components using SWR, in a test environnement.
In dev/prod environnement Preact handles perfectly SWR, but not in test environnement.
When I launch the test, it returns the error :
TypeError: Cannot read properties of null (reading 'useContext')
To Reproduce
Barebone example : https://github.com/dolie/preact-swr-vitest-bug
Steps to reproduce the behavior:

git clone git@github.com:dolie/preact-swr-vitest-bug.git
pnpm i
pnpm vitest

Expected behavior
Preact handles SWR in a test environnement, as it is running in dev/prod environnement.
Additional Context
SWR
Vitest
MSW
Testing-library

Vitest unfortunately has problems aliasing, this is something we've seen a few times.
Can't go looking for reference issues right now (on phone, will take a look later) but I'm sure you can pretty easily find them on the Vitest repo.
I think this is something Vitest needs to address, not us here.

Thank you for you fast answer !
Ok, so I added preact/compat using pnpm aliases, but it throws a new error about useSyncExternalStore ?

That error says you're loading multiple copies of Preact at once; hooks, in both Preact and React, require that the library is loaded as singleton (only once).
Probably the dual-package hazard: https://github.com/vitest-dev/vitest/issues/4365. Again, something that unfortunately needs to be addressed in Vitest (as far as I am aware).

Vitest team found a workaround😉
I updated the repo with it.
Thank you for your help @rschristian @JoviDeCroock 🙏


🐛 fix(bal-label): Labels don't break

Current Behavior
The labels don't break anymore, hence the input width stretches with label length no matter what is set as a width on the input
Expected Behavior
label and input should respect the set grid width, which means the label has to break in the next line in some situations
Steps to Reproduce
regular input and label use
Version
13.2.1
Browsers
No response
Device
No response
Code Reproduction URL
No response
Additional Information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

/cib


edge case with mathtools v3

Hi,
I have encountered an edge case when converting from mathtools to mml (see attached idml).
Mathtool sometimes (I do not understand when|why) emit an empty @MathToolsML on a charrange containing text content and this content get lost due to the <p:delete name="remove-MathTools-Markup"... line in idml2hub.xpl.
Solution is to modyfy above line from
<p:delete name="remove-MathTools-Markup"          match="*[@MTMathZone][not(descendant::mml:math or descendant::Rectangle)]                       |CharacterStyleRange[descendant::mml:math]/Content[not(normalize-space())]"/>
to
<p:delete name="remove-MathTools-Markup"          match="*[@MTMathZone][not(descendant::mml:math or descendant::Rectangle)][not(MathToolsML='')]                       |CharacterStyleRange[descendant::mml:math]/Content[not(normalize-space())]"/>
I think this should be pretty safe and (hopefully) should not introduce any regression.
test.idml.zip

I haven’t looked at the code and at the input yet, but if you say that sometimes @MathToolsMl is empty, shouldn’t it read [not(@MathToolsML='')] instead of [not(MathToolsML='')]?

I haven’t looked at the code and at the input yet, but if you say that sometimes @MathToolsMl is empty, shouldn’t it read [not(@MathToolsML='')] instead of [not(MathToolsML='')]?

No, because @MathToolsMl has been  already transformed to <MathToolsML> in a previous step.
I have tested the above code and it works.

I applied the change directly here on Github. Let’s see what our integration tests say…

The tests haven’t complained. I don’t think we have very broad coverage of MathTools content anyway. If it’s working for you, you might close this issue.

Yes, it is ok for me.


Getting the size of plaintexts stored in memory under CKKS

Hello, I'm trying to include a pre-processing step to store needed plaintexts (under CKKS) ahead of time and would like to determine the number of bytes my plaintexts need in total. Based on the information in plaintext.h, if I have the parameters set as
polynomial modulus degree = 8192
coefficient moduli bit widths = {50, 36, 36, 36, 50}
could I simply do 8192 poly mod deg * 5 primes * 8 bytes = 327680 bytes per plaintext or am I thinking about this incorrectly? I also see that I need to take the size of the parms_id. Is there a way to calculate the size of parms_id?

What you have calculated is the maximum size of Plaintext::data_. Other data members are also serialized here. These data members except data_ have fixed sizes; for example, sizeof(parms_id_type) tells you its size. Further, there is a SEALHeader created and save for each serialized object too, which also have a fixed size.
The best way to learn the size of an serialized object is follow the examples. The 327680 bytes that you have calculated is an upper bound of data_, which is usually smaller when compression is enabled.

Thank you so much, that was incredibly helpful.


Return default value when cli config get cannot find config value

When I run cli config get --key prompt for example, and I haven't set any config value with this key, the command is just returning nothing.

In my opinion, this is not really a great user experience. It can be really confusing for users. When I encounter this, I have to look it up in the docs to find the default value for this config variable.
I suggest that when a config value is not found, we just return the default value. (I guess this is a breaking change?)
In a lot of commands, we retrieve config values as follows:
cli.getSettingWithDefaultValue<boolean>(settingsNames.autoOpenLinksInBrowser, false)

I don't think it's a good approach to define the default value in the command itself. A better approach would be that we centralize all default values and provide a new function where the user should only specify the config variable.
E.g.:
cli.getSetting<boolean>(settingsNames.autoOpenLinksInBrowser)

Awesome suggestion!

The idea behind providing the default value was not to return the value that's default when no value has been configured in the CLI, but rather a value that makes sense for the particular scenario. We can of course revisit this stance, but when reviewing it, let's consider it before we apply any changes.

The idea behind providing the default value was not to return the value that's default when no value has been configured in the CLI, but rather a value that makes sense for the particular scenario.

Personally, I don't really understand why. Is there any command that doesn't use the CLI default value? Why would you want to use your own value?
Worst case we can make the default value parameter optional so you don't have to define the default value in every command over and over again.

Re-reading the original issue, I think we've got two things that I think we should take on separately:

Retrieving and showing the config value using the cli config get command
Refactoring cli.getSettingWithDefaultValue to cli.getSetting

Refactoring our code isn't a breaking change, because it's our code, and not something that's exposed to users.
As for updating the cli config get command, returning the default value if none has been configured makes sense and doesn't really break anything in my opinion. I suggest we update the issue to non-breaking.
Agreed with both refactorings.


HDDS-4117. Normalize Keypath for listKeys.

What changes were proposed in this pull request?
Normalize Keypath for listKeys.
When ozone.om.enable.filesystem.paths, OM normalizes path, and stores the Keyname in the OM DB KeyTable.
When listKeys uses given keyName(not normalized key path) as prefix and Starkey the list-keys will return an empty results.
Similar to HDDS-4102, we should normalize Starkey and keyPrefix.
Note:
This PR is based on top of #1328
To get CI, posted a PR with both fixes included.
The changes for this commit start from ab0cee5f44094a569d2a83e41889c515d08d380c
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4117
How was this patch tested?
Added a test.

Codecov Report

Merging #1331 into master will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1331      +/-   ##
============================================
- Coverage     74.41%   74.35%   -0.07%
- Complexity    10301    10324      +23
============================================
  Files           984      985       +1
  Lines         50394    50549     +155
  Branches       4914     4937      +23
============================================
+ Hits          37503    37587      +84
- Misses        10531    10600      +69
- Partials       2360     2362       +2

Impacted Files
Coverage Δ
Complexity Δ

...src/main/java/org/apache/hadoop/ozone/OmUtils.java
80.66% <100.00%> (+1.08%)
49.00 <3.00> (+3.00)

...ava/org/apache/hadoop/ozone/om/KeyManagerImpl.java
65.34% <100.00%> (+0.43%)
140.00 <4.00> (+4.00)

...pache/hadoop/ozone/om/request/OMClientRequest.java
81.17% <100.00%> (-1.25%)
29.00 <1.00> (-2.00)

...hdds/scm/container/common/helpers/ExcludeList.java
78.26% <0.00%> (-21.74%)
17.00% <0.00%> (-5.00%)

...doop/ozone/om/exceptions/OMNotLeaderException.java
48.48% <0.00%> (-18.19%)
3.00% <0.00%> (-2.00%)

.../apache/hadoop/hdds/scm/node/StaleNodeHandler.java
88.88% <0.00%> (-11.12%)
4.00% <0.00%> (ø%)

...e/hadoop/ozone/recon/tasks/OMDBUpdatesHandler.java
60.93% <0.00%> (-9.66%)
9.00% <0.00%> (-8.00%)

...ache/hadoop/ozone/om/codec/S3SecretValueCodec.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (-1.00%)

...va/org/apache/hadoop/hdds/utils/db/RDBMetrics.java
92.85% <0.00%> (-7.15%)
14.00% <0.00%> (-1.00%)

...ozone/container/ozoneimpl/ContainerController.java
76.31% <0.00%> (-5.27%)
13.00% <0.00%> (-1.00%)

... and 44 more

Continue to review full report at Codecov.

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

Closing this. Opened #1451 to track this issue.


Closes #26 rename/rerun programs from admiral.test/admiralvaccine

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[ ] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[ ] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[ ] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[ ] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[ ] Address any updates needed for vignettes and/or templates
[ ] Update NEWS.md if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[ ] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[ ] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[ ] Link the issue in the Development Section on the right hand side.
[ ] Address all merge conflicts and resolve appropriately
[ ] Pat yourself on the back for a job well done! Much love to your accomplishment!

Summary of the updates:

replace dev folder by data-raw
remove prefix admiral_ in both programs and datasets
rename update_domain.R programs to domain.R
add programs for ex/cm/sv/ts/vs/dm (direct copy from CDISC pilot study)
add set.seed in pc.R/qs_ophtha.R to avoid changes for future rerun

@pharmaverse/admiraldata
I removed all raw_domain.rda data, and updated to read them directly from the URL. Do you see any issues with this approach? should we save a copy of the raw data under data-raw folder?

Thanks for working on this @kaz462 - there are a few things I would change in the individual scripts to make them tidier and more homogeneous, but as discussed yesterday for nowit's more important to get the package structure nailed down.

I removed all raw_domain.rda data, and updated to read them directly from the URL. Do you see any issues with this approach? should we save a copy of the raw data under data-raw folder?

I think that's OK because it is unlikely that the CDISC pilot data is removed or changed.


flatten repo

to me, it seems like the nested double package.json structure adds unnecessary extra steps for installation and various other processes

Kinda realized I was swimming upstream here, create-reactapp/react-scripts is designed to run in its own client folder with its own package.json


Remove cargo unleash CI job

The CI job is currently breaking, because we released some crates. We also don't need the job
currently because releases are not done. So, the job currently just annoys people and eats CPU time.
This brings us to the conclusion that this job is currently not required and can be readded at a
later point when we are sure that we need it.

I’m hazy on what we loose with this but given it’s broken we should remove it.

We will loose the check that the current workspace could be "released" to crates.io. However, even if that breaks, fixing it should be doable in 1-2 days. But, we will have a much easier life for now.


Imp: use gost bytes pool

What this PR does:
use gost bytes pool to reuse byte slices.

This branch has conflicts that must be resolved.

binary type is not used regularly, this pr will not improve performance for most situation.

@wongoo can not agree with u. if binary is used, the performance is better. the array length is 6KiB which is not a little slice.

@AlexStocks I wonder is there some place can also use the bytes pool, not just for binary type.

@wongoo none. All those bytes are return to user without this binary func.

LGTM


Fix the docs deployment

Fixes #726

Oops. Already slipped in a possible fix before seeing this one.
Thanks. If my fix doesn't work, I'll fall back to this one. I'm not sure I understand it though, as is seems not to make a change to the final repo string. The "protocol" `https:\" is still there.

Codecov Report

Merging #727 (b3d2c59) into dev (51c486b) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #727      +/-   ##
==========================================
- Coverage   84.87%   84.70%   -0.17%
==========================================
  Files          36       36
  Lines        3352     3315      -37
==========================================
- Hits         2845     2808      -37
  Misses        507      507

Impacted Files
Coverage Δ

src/measures/measures.jl
69.07% <0.00%> (-1.81%)
:arrow_down:

src/measures/loss_functions_interface.jl
72.72% <0.00%> (-0.49%)
:arrow_down:

src/show.jl
36.64% <0.00%> (-0.40%)
:arrow_down:

src/interface/data_utils.jl
96.49% <0.00%> (-0.29%)
:arrow_down:

src/utilities.jl
83.89% <0.00%> (-0.22%)
:arrow_down:

src/resampling.jl
88.11% <0.00%> (-0.16%)
:arrow_down:

src/composition/learning_networks/machines.jl
90.24% <0.00%> (-0.12%)
:arrow_down:

src/measures/finite.jl
98.10% <0.00%> (-0.10%)
:arrow_down:

src/data/datasets_synthetic.jl
95.41% <0.00%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

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

See line 22. The "https://" will not be passed when we call deploydocs, which is the important part.

The repo kwarg to deploydocs is a different concept from the repo kwarg to makedocs.

In makedocs, you do still want to pass the "https://", which is why I'd recommend reverting the change you pushed to dev.

In the repo kwarg to deploydocs, you should not include the .git suffix.

PkgTemplates.jl will generate a docs/make.jl file that is compatible with the Documenter.jl requirements.
Here is a repository that I generated recently using PkgTemplates: https://github.com/bcbi/TableCellCounts.jl/blob/51ac308adbbc57f3bd4b21d3105f613079cf3458/docs/make.jl

In the repo kwarg to deploydocs, you should not include the .git suffix.

I guess both ways work then. I found .git option in the Documenter documentation

@OkonSamuel @DilumAluthge I'm happy for you two to take over here, thanks!

In the repo kwarg to deploydocs, you should not include the .git suffix.

I guess both ways work then. I found .git option in the Documenter documentation https://github.com/JuliaDocs/Documenter.jl/blob/5e5d2db1d0600cddec14b4ea3139bc5acc48e401/src/Documenter.jl#L381-L383

Ah, good find! Yeah, it looks like either with or without will work. I think that PkgTemplates generates a template without, which is probably why that's why I tend to choose.


Updating your new builds fails

Why does the below happen when I tried to update?
WARNING in ./~/<EMAIL_ADDRESS>329:2-21 "export 'ɵDomAnimationEngine' was not found in '@angular/animations/browser'
WARNING in ./~/<EMAIL_ADDRESS>382:43-63 "export 'ɵNoopAnimationEngine' was not found in '@angular/animations/browser'
ERROR in<EMAIL_ADDRESS>(8,119): Module '"/node_modules/@angular/animations/browser"' has no exported member 'ɵDomAnimationEngine'.

never mind upraded cli and worked


Store the expected number of test rows in the Trained* object

This allows us to double-check that the correct objects are being used in classify_*.
This requires deprecation of the function that accepts an intersection, because there's no way to get the number of rows from the test dataset; this would need to be replaced by a function that accepts the number of rows explicitly.

Closed by c0447ce8b3392c7b43f43b1c2d06fcffb4a2da2e.


k8s: fix incorrect EndpointSlice API version

This PR fixes a bug that k8s watcher incorrectly chooses the discovery/v1beta1 EndpointSlice in environments where discovery/v1 EndpointSlice is available. The agent doesn't wait for the discovery/v1 EndpointSlice to be received on its restart and ends up removing live services because of this bug.
The issue happens in the following scenario:

The agent initializes and runs the informers for Service and EndpointSlice.
The reflectors list the resources from kube-apiserver and replace DeltaFIFO.
The informers pop items from the DeltaFIFO and call the onAdd handler one by one.
If the Service handler is called first, ServiceCache.UpdateService doesn't emit the UpdateService event, because the service is not ready. (No backend)
The EndpointSlice handler is called next, ServiceCache.updateEndpoints emits UpdateService event.
The agent doesn't wait until the event triggered at step 5 is processed, and calls SyncWithK8sFinished.
The agent removes live services because it hasn't heard about them. (It is supposed to hear about those services in the event process triggered at step 5)

This issue doesn't persist in v1.14 and later because K8sAPIGroupEndpointSliceOrEndpoint is used in those versions. The affected versions are v1.13 and v1.12.
Fixes: #27215
Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

/test

/test-backport-1.13

Good catch!

/test-1.25-4.19

I think we can merge this one, but we also need this change in 1.12, right?


The separator between seconds and milliseconds is culture independent in date and longdate layouts

NLog version: 5.2.8
Platform: .NET 4.5.2
Current NLog config
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
	  internalLogLevel="Debug"
      internalLogFile="c:\temp\nlog-internal.txt"
      internalLogToConsole="false"
      throwConfigExceptions="true">
	<targets>
		<target
			name ="AsyncTrimCSV"
			xsi:type="AsyncWrapper"
			timeToSleepBetweenBatches="0"
			batchSize="200"
			queueLimit="10000"
			overflowAction="Block">
			<target
					  xsi:type="File"
					  encoding="utf-8"
					  fileName="${basedir}/Logs/ResistorAdjustment/${shortdate}.csv"
					  keepFileOpen="False"
					  concurrentWrites="True"
					  maxArchiveDays="730">
				<layout xsi:type="CsvLayout">
					<column name="Date" layout="${date}" />
					<column name="Time" layout="${time}" />
					<column name="Filename" layout="${event-properties:item=filename}" />
					<column name="Resistor Serial Number" layout="${event-properties:item=ResistorSerialNumber}" />
					<column name="Resistor Value After Trim" layout="${event-properties:item=ResistorValueAfterTrim}" />
					<column name="Resistor Start Value" layout="${event-properties:item=ResistorStartValue}" />
					<column name="Target Resistor Value" layout="${event-properties:item=TargetResistorValue}" />
					<column name="Laser Precision" layout="${event-properties:item=LaserPrecision}" />
					<column name="Power" layout="${event-properties:item=Power}" />
					<column name="Speed" layout="${event-properties:item=Speed}" />
					<column name="Freq" layout="${event-properties:item=Freq}" />
					<column name="Pulse Width" layout="${event-properties:item=PulseWidth}" />
					<column name="Beam Wobble" layout="${event-properties:item=BeamWobble}" />
					<column name="Path To Image" layout="${event-properties:item=pathToImage}" />
				</layout>
			</target>
		</target>
		<target name="logfile" xsi:type="File" fileName="${basedir}/Logs/task.log"
				layout="${longdate} ${logger} ${event-properties:task_name} ${message}${exception:format=ToString}"/>
	</targets>
	<rules>
		<logger name="asynctrim" minlevel="Info" writeTo="AsyncTrimCSV" />
	</rules>
</nlog>

What is the current result?
With "ru-RU" culture the date looks like this:
2024.02.26 16:06:38.212
What is the expected result?
2024.02.26 16:06:38,212
Did you checked the Internal log?
Yes
Please post full exception details (message, stacktrace, inner exceptions)
There are no exception
Are there any workarounds? yes/no
Yes, for example "date:format=yyyy-MM-dd HH:mm:ss,fff", but this is works only for one culture and not for the others.
Also, you can separate the date and time, while the time works correctly with the culture
Is there a version in which it did work?
Didn't look
Can you help us by writing an unit test?
I don't know how, but I'll try

By the way time layout works well with different cultures.

Both ${date} and ${time} uses public CultureInfo Culture { get; set; } = CultureInfo.InvariantCulture; by default in NLog v5, unless one have specified LogEventInfor-FormatProvider (which probably should be fixed in NLog v6, so a layoutrenderer can override the LogEventInfor-FormatProvider).
It would be very helpful if you can write a small unit-test that reproduces the issue, or provide a simple code-example that shows how you are logging and reproduces the issue.

NLog.config
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
	<targets>
		<target name="logconsole" xsi:type="Console"
				layout="${date} ${time} ${message}"/>
	</targets>
	<rules>
		<logger name="console" minlevel="Info" writeTo="logconsole" />
	</rules>
</nlog>

Code Example
void Test()
        {
            Thread.CurrentThread.CurrentCulture = CultureInfo.GetCultureInfo("ru-RU");
            NLog.LogManager.Configuration.DefaultCultureInfo = Thread.CurrentThread.CurrentCulture;
            Console.WriteLine(Thread.CurrentThread.CurrentCulture.Name);
            Logger logger = NLog.LogManager.GetLogger("console");
            logger.Info("Not correct");

            Thread.CurrentThread.CurrentCulture = CultureInfo.GetCultureInfo("en-EN");
            NLog.LogManager.Configuration.DefaultCultureInfo = Thread.CurrentThread.CurrentCulture;
            Console.WriteLine(Thread.CurrentThread.CurrentCulture.Name);
            logger.Info("Correct");
        }

Result
ru-RU
2024.03.01 14:14:34.587 14:14:34,5873 Not correct
en-EN
2024/03/01 14:14:34.603 14:14:34.6035 Correct

The only problem is separator between seconds and milliseconds. It's strange to see different separators in the same culture in date and time layout.

Can see that $[time} + ${processtime} was updated to have extra culture-support with #1556
The ${date} uses the default Format=yyyy/MM/dd HH:mm:ss.fff, and it seems that DateTime.ToString only applies some culture adjustements (but not the milliseconds-seperator).
If you know how to make DateTime.ToString to also perform culture adjustment for the milliseconds-seperator, then pull-request are wellcome.
I guess a work-around could be: ${date:format=yyyy/MM/dd} ${time}

Closing due to inactivity.


Fix leak of inline cache list when invalidating

Change reviewed in PR #1019.   Created new PR to target release/1.2 instead.

LGTM. Thanks.

@dotnet-bot test this please


Seems some widgets don't need a limit selector

Description:
Some over-time widgets don't have a table corresponding to the time-series data plot. Hence the limit selector is not needed.
Monitoring/error summary over time
Monitoring/API latency over time
Monitoring/API traffic over time
Suggested Labels:
Type/Improvement, Priority/Normal
Affected Product Version:
3.2.0-alpha

The limit selector is not used limit the number of entires in the table but rather to limit the number of data entries retrieved from the backend. Removing this may result in a large data load being retrieved at once which could cause memory outage. Hence closing this issue.


Fix #49

It's should be #69 ? 😄

Codecov Report

Merging #86 into master will increase coverage by 1.72%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #86      +/-   ##
==========================================
+ Coverage   21.39%   23.12%   +1.72%
==========================================
  Files          40       40
  Lines        3487     3555      +68
==========================================
+ Hits          746      822      +76
+ Misses       2704     2694      -10
- Partials       37       39       +2

Impacted Files
Coverage Δ

pkg/types/route.go
0% <ø> (ø)
:arrow_up:

pkg/config/config.go
12% <ø> (ø)
:arrow_up:

pkg/router/types.go
9.52% <ø> (ø)
:arrow_up:

pkg/config/parser.go
3.24% <0%> (-0.09%)
:arrow_down:

pkg/router/routematcher.go
92.45% <0%> (+0.56%)
:arrow_up:

pkg/router/routeruleimpl.go
46.29% <0%> (+16.66%)
:arrow_up:

pkg/router/virtualhostimpl.go
59.09% <0%> (+19.74%)
:arrow_up:

Continue to review full report at Codecov.

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


Hymba multipack support

Description
Support for Hymba multipacking
Motivation and Context

How has this been tested?

Screenshots (if appropriate)
Types of changes

Social Handles (Optional)

a small e2e test for this with and without packing and validating the train_loss would be good to have.

rebased in https://github.com/axolotl-ai-cloud/axolotl/pull/2178


ClusterIssuer is missing field

error: error validating "10-cert-manager/issuer.yml": error validating data: ValidationError(ClusterIssuer.spec): unknown field "ignoreError" in org.freeipa.certmanager.v1beta1.ClusterIssuer.spec; if you choose to ignore these errors, turn validation off with --validate=false

The Cluster Issuer is missing the field ignoreError however the error which is suppressed by this field still appears when trying to sign with the issuer.

@StefanAbl can you test with the latest commit?


APP-1561: Add getInvoicePdf proto

WIP branch for public API changes in support of APP-1561

lgtm! out of curiosity, does the response automatically download the PDF? or does the frontend have to process the GetInvoicePdfResponse to then download it?

I haven't figured out how to get the PDF to donwload automatically, yet. There might be a header setting that makes this happen, so I'll continue playing with it.
For now, though, the streaming response makes it pretty easy for the front-end to construct a Blob that the client can then download through a data URL

Collect streaming file response:
https://github.com/viamrobotics/app/blob/a476253396d4c949549ad19c992f94653747ffb4/frontend/src/lib/api/billing.ts#L219-L238
Download:
https://github.com/viamrobotics/app/blob/a476253396d4c949549ad19c992f94653747ffb4/frontend/src/pages/orgs/billing.vue#L99-L107


CarbonCharts and Jest don't work together

https://cengage.slack.com/archives/C8Y4FR63U/p1719005312208249

All Charts show the proper custom styles, looks good here!

I reviewed the charts at https://next--upbeat-sinoussi-f675aa.netlify.app/api/chart/ and everything is looking to good to me too.


Add hydrological forcing comparison recipe

Description
Hi everyone, this recipe compares between ERA5, ERA-interim, and MSWEP datasets for a certain catchment.

Closes #issue_number
Link to documentation: https://esmvaltool--2013.org.readthedocs.build/en/2013/recipes/recipe_hydro_forcing.html

Todo:

[x] https://github.com/ESMValGroup/ESMValCore/pull/986 merged

Before you get started

[ ] ☝ Create an issue to discuss what you are going to do

Checklist
It is the responsibility of the author to make sure the PR is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 PR has a descriptive title for the changelog
[x] 🛠 Code follows the style guide
[x] 🛠 Documentation is available
[x] 🛠 YAML files pass pre-commit or yamllint checks
[x] 🛠 Circle/CI tests pass
[x] 🛠 Codacy code quality checks pass
[x] 🛠 Documentation builds successfully on readthedocs

New or updated recipe/diagnostic:

[x] 🛠 Provenance information has been added
[x] 🛠 New dependencies are added to the project requirements
[x] 🧪 Documentation for the recipe/diagnostic clearly describes what it calculates from a scientific point of view
[ ] 🧪 Recipe runs successfully on @esmvalbot or some other machine without modification (MSWEP data not available)
[x] 🧪 Figure(s)/data look as expected
[x] 🧪 Code is well documented and scientifically sound

To help with the number pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

Hi @Peter9192 , I think this PR is ready. Could you do a technical review?

I can test it if you want, but I will need a suitable shapefile

I can test it if you want, but I will need a suitable shapefile

@jvegasbsc That would be great! I'm using the shapefiles from here: https://github.com/eWaterCycle/recipes_auxiliary_datasets/tree/master/Lorentz_Basin_Shapefiles
You will also need the MSWEP data: https://docs.esmvaltool.org/projects/ESMValCore/en/latest/develop/fixing_data.html?highlight=mswep#mswep
(or just run with ERA5 / ERA-Interim)

Thanks. I've just run it for ERA-Interim, as we only have monthly data for ERA5 in jasmin

Thanks @jvegasbsc , I have updated the code. I think it is better now 👍

Hi @jeromaerts , could you do a scientific review of this recipe? Especially, could you have a look at the documentation, descriptions, captions, etc., whether they make sense scientifically?

@stefsmeets, Looks great, also from a scientific point of view.
Some small notes on the documentation:

"3. Plot the monthly climate statistics over a certain period"
Change: "3. Plot the monthly climatology statistics over a certain period"

If possible extent the plotting of climatology to also include a daily climatology plot.

"All hydrological recipes require a shapefile as an input to produce forcing data."
Change, produce to select.

"All recipes are tested with the shapefiles that are used for the eWaterCycle project."
"All recipes are tested with shapefiles from HydroSHEDS that are used for the eWaterCycle project."

Looking at the figures, the dates on the horizontal axis are very close to each other, perhaps slightly reduce the label font-size.

Hi @jeromaerts , thanks for the comments! I have updated the code and added the new plot. Let me know what you think. If you are happy with the changes, could you approve the PR?

Hi @jvegasbsc , have you had a change to take another look at the changes I made? If you agree, could you approve this PR so we can get this merged? If you have any other comments, please let me know!

Looks better now! I agree with @nielsdrost comment about tha nimng of the entry_point but that's a very minor concern.
And sorry for the delay, I went straight to holidays after the release

No worries, probably well-deserved! 😅 Will fix it next week 👍

Thanks @stefsmeets !


fix(innerRef): allow ref as object

React 16.3 comes with a createRef API where you can add ref as an object
What:
Allow innerRef to be passed as an object.
Why:
Because React 16.3 comes with a createRef API, the ref to elements is not only functions anymore, it may also be objects.
How:
I have updated prop type for the innerRef prop as well as the TypeScript interface.
Checklist:

[ ] Documentation N/A
[ ] Tests N/A
[x] Ready to be merged
[x] Added myself to contributors table

I have updated where I can see that we set types for innerRef, but it would be great if someone took a quick look as well.
I'm happy to update if there are adjustments needed.

Ah, looks like there's an issue with the snapshots.  Thanks!

Just glad I was able to help! Snapshots are now updated.

Codecov Report

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

@@          Coverage Diff          @@
##           master   #409   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          10     10
  Lines         177    177
  Branches       50     50
=====================================
  Hits          177    177

Impacted Files
Coverage Δ

src/create-glamorous.js
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Implement Like/Deslike mode style

Hey, My name is Sachin Jethanandani and I'm trying to prepare for gsoc next year. I want to start my contributions. Thanks.

Hi @sachin-0102 !! 👋🏻
That's so nice to know you are preparing for gsoc! Good luck with that! I hope we could help you in anyway.
I'll create an issue for you and link it here asap. Do you have any preference for React or pure HTML/CSS ?

I'm more comfortable with CSS...

@tiagostutz  I understand the Like/Deslike mode but what is another mode Choice Between ?

The Choice Between is actually in a more mature state and the interaction is to choose between two options. The Like/Deslike mode is an interaction where the user just say whether he likes or not the presented single option.

@tiagostutz I think I can take this one - ill reach out if I have questions. It looks like basically a state update onClick right?

To be fare, It is a bit more thant that. But not a big deal.
Currently the project has two types of interaction: Choice mode (choose between two options) and Like/Deslike mode (one option presented and the user just like or deslike, pretty much like tinder). To put the Like/Deslike mode on the trail is the goal here. You don't need to take care of the events for now, you can just focus on the HTML/CSS part of this component, so it will use Tailwind and will have an aesthetic like the ChoiceMode, that is a bit more advanced.
What do you think?

sure! just another question: are you using tailwind across the board on the site too?

Yep, the ideia is to use Tailwind all over.

@tiagostutz yup I will get started on this this week, I will reach out if I have questions =]

@tiagostutz I was running into issues with my set up. could you give me a hand? after forking and cloning it to my local machine, and running npm install im getting start time errors due to there not being a package.json in the project although there is -

Hi @accordingtoherr !
Let's check this. Could you try to run npm run dev instead of  npm start? If it works, please fix the README. Also, are you running the command inside the app folder? Last but not least, which node version are you using.

I’ll try tomorrow! And I have the most up to date version of node - I can download an older version if needed for this project w nvm if needed
Sent from my iPhone

On Sep 23, 2020, at 10:02 PM, Tiago de Oliveira Stutz<EMAIL_ADDRESS>wrote:

Hi @accordingtoherr !
Let's check this. Could you try to run npm run dev instead of npm start? If it works, please fix the README. Also, are you running the command inside the app folder? Last but not least, which node version are you using.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi there @accordingtoherr ! Any updates on that?
If you don't have time to work on that now, that's OK.! I can assign this issue to someone and we can work on another issue for you when you are ready to go.

@tiagostutz no i am still not able to set up the environment i was getting errors. already checked node versions and made sure all the dependencies  are up to date

Ah, I see... let's work on that. Please check those steps:

enter de app folder (cd app)
remove node_modules
run yarn install
run yarn run dev

After you run that, what are the errors you got? Could you please paste here the screenshots?

$ REACT_APP_BACKEND_URL=http://localhost:8008 yarn start
'REACT_APP_BACKEND_URL' is not recognized as an internal or external command,
operable program or batch file.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
confirming i removed the packages, added back in and installed a couple that did not install initially. - not sure what is going on

@tiagostutz i am still unable to run this project locally - you can assign it to someone else since I tried a lot of things/fixes and nothing worked so I do not want a possible machine config to hold up the ticket

Hi @tiagostutz! I would like to work on this issue. Can you please assign it to me?

Hi @tiagostutz! I would like to work on this issue. Can you please assign it to me?

Absolutely @shoaibkamalkhan ! I've just assigned the issue to you.
Take a quick look at the contribution guidelines and feel free to ask if you have any question.
Thank you so much.! =)

Absolutely @shoaibkamalkhan ! I've just assigned the issue to you.
Take a quick look at the contribution guidelines and feel free to ask if you have any question.
Thank you so much.! =)

Thanks a lot @tiagostutz! I will take a look.😀

Thanks a lot @tiagostutz! I will take a look.😀

Hi @tiagostutz! I have forked and cloned the repo and when I try to run the app I get the following errors:
I have first used npm install and then npm run dev:
The default interactive shell is now zsh.
To update your account to use zsh, please run chsh -s /bin/zsh.
For more details, please visit https://support.apple.com/kb/HT208050.
ShoaibKhansMBP:stop-analyzing-embed shoaibkhan$ cd app
ShoaibKhansMBP:app shoaibkhan$ npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@testing-library/react-hooks
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/react-test-renderer
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@testing-library/react-hooks
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/shoaibkhan/.npm/eresolve-report.txt for a full report.
npm ERR!      /Users/shoaibkhan/.npm/_logs/2021-01-26T14_56_13_324Z-debug.log
ShoaibKhansMBP:app shoaibkhan$ npm run dev

<EMAIL_ADDRESS>dev
REACT_APP_BACKEND_URL=http://localhost:8008 yarn start

sh: yarn: command not found
npm ERR! code 127
npm ERR! path /Users/shoaibkhan/Documents/Treehouse/Open Source Projects/stop-analyzing-embed/app
npm ERR! command failed
npm ERR! command sh -c REACT_APP_BACKEND_URL=http://localhost:8008 yarn start
npm ERR!      /Users/shoaibkhan/.npm/_logs/2021-01-26T14_56_33_589Z-debug.log
ShoaibKhansMBP:app shoaibkhan$

Hi @tiagostutz! I have forked and cloned the repo and when I try to run the app I get the following errors:
I have first used npm install and then npm run dev:
The default interactive shell is now zsh.
To update your account to use zsh, please run chsh -s /bin/zsh.
For more details, please visit https://support.apple.com/kb/HT208050.
ShoaibKhansMBP:stop-analyzing-embed shoaibkhan$ cd app
ShoaibKhansMBP:app shoaibkhan$ npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@testing-library/react-hooks
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/react-test-renderer
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@testing-library/react-hooks
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/shoaibkhan/.npm/eresolve-report.txt for a full report.
npm ERR!      /Users/shoaibkhan/.npm/_logs/2021-01-26T14_56_13_324Z-debug.log
ShoaibKhansMBP:app shoaibkhan$ npm run dev

<EMAIL_ADDRESS>dev
REACT_APP_BACKEND_URL=http://localhost:8008 yarn start

sh: yarn: command not found
npm ERR! code 127
npm ERR! path /Users/shoaibkhan/Documents/Treehouse/Open Source Projects/stop-analyzing-embed/app
npm ERR! command failed
npm ERR! command sh -c REACT_APP_BACKEND_URL=http://localhost:8008 yarn start
npm ERR!      /Users/shoaibkhan/.npm/_logs/2021-01-26T14_56_33_589Z-debug.log
ShoaibKhansMBP:app shoaibkhan$

Don't worry, let's troubleshoot this.
Which node version are you using? Could you try v14?

Don't worry, let's troubleshoot this.
Which node version are you using? Could you try v14?

I have v15.6.0.

I have v15.6.0.

I'll try with v15.6.0. I use nvm to manage my node versions...

I'll try with v15.6.0. I use nvm to manage my node versions...

I have tested here with v15.6.0 and had the same problem. So, if you use the v14 I think it will work, as it did here.

I have tested here with v15.6.0 and had the same problem. So, if you use the v14 I think it will work, as it did here.

Ok, I will try this and let you know how it went. Thanks a lot for helping me!😀🙏🏻👍🏻🎶

Ok, I will try this and let you know how it went. Thanks a lot for helping me!😀🙏🏻👍🏻🎶

Hi @tiagostutz!
I was able to install nvm and then install node version 14.5.4 and I was able to progress and then install yarn, but now I am getting this error when running npm run dev. Do you know what this error means?
ShoaibKhansMBP:app shoaibkhan$ npm run dev

<EMAIL_ADDRESS>dev /Users/shoaibkhan/Documents/Treehouse/Open Source Projects/stop-analyzing-embed/app
REACT_APP_BACKEND_URL=http://localhost:8008 yarn start

yarn run v1.22.10
warning ../../../../../package.json: No license field
$ npm run dev:css && react-scripts start
npm WARN lifecycle The node binary used for scripts is /var/folders/08/2sy0rbdx76l_qm4_7__vn0ym0000gn/T/yarn--1611758954838-0.4330973958038482/node but npm is using /Users/shoaibkhan/.nvm/versions/node/v14.15.4/bin/node itself. Use the --scripts-prepend-node-path option to include the path for the node binary npm was executed with.

<EMAIL_ADDRESS>dev:css /Users/shoaibkhan/Documents/Treehouse/Open Source Projects/stop-analyzing-embed/app
postcss src/styles/tailwind.css -o src/styles/main.css

sh: postcss: command not found
npm ERR! code ELIFECYCLE
npm ERR! syscall spawn
npm ERR! file sh
npm ERR! errno ENOENT
npm ERR<EMAIL_ADDRESS>dev:css: postcss src/styles/tailwind.css -o src/styles/main.css
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev:css script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?
npm ERR!      /Users/shoaibkhan/.npm/_logs/2021-01-27T14_49_15_327Z-debug.log
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>dev: REACT_APP_BACKEND_URL=http://localhost:8008 yarn start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?
npm ERR!      /Users/shoaibkhan/.npm/_logs/2021-01-27T14_49_15_403Z-debug.log
ShoaibKhansMBP:app shoaibkhan$

Hi @tiagostutz!
I was able to install nvm and then install node version 14.5.4 and I was able to progress and then install yarn, but now I am getting this error when running npm run dev. Do you know what this error means?
ShoaibKhansMBP:app shoaibkhan$ npm run dev

<EMAIL_ADDRESS>dev /Users/shoaibkhan/Documents/Treehouse/Open Source Projects/stop-analyzing-embed/app
REACT_APP_BACKEND_URL=http://localhost:8008 yarn start

yarn run v1.22.10
warning ../../../../../package.json: No license field
$ npm run dev:css && react-scripts start
npm WARN lifecycle The node binary used for scripts is /var/folders/08/2sy0rbdx76l_qm4_7__vn0ym0000gn/T/yarn--1611758954838-0.4330973958038482/node but npm is using /Users/shoaibkhan/.nvm/versions/node/v14.15.4/bin/node itself. Use the --scripts-prepend-node-path option to include the path for the node binary npm was executed with.

<EMAIL_ADDRESS>dev:css /Users/shoaibkhan/Documents/Treehouse/Open Source Projects/stop-analyzing-embed/app
postcss src/styles/tailwind.css -o src/styles/main.css

sh: postcss: command not found
npm ERR! code ELIFECYCLE
npm ERR! syscall spawn
npm ERR! file sh
npm ERR! errno ENOENT
npm ERR<EMAIL_ADDRESS>dev:css: postcss src/styles/tailwind.css -o src/styles/main.css
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev:css script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?
npm ERR!      /Users/shoaibkhan/.npm/_logs/2021-01-27T14_49_15_327Z-debug.log
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>dev: REACT_APP_BACKEND_URL=http://localhost:8008 yarn start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?
npm ERR!      /Users/shoaibkhan/.npm/_logs/2021-01-27T14_49_15_403Z-debug.log
ShoaibKhansMBP:app shoaibkhan$

Hi @tiagostutz! I was wondering if there was any status on this issue? I also posted my issue on stackoverflow; you can visit the link here:
https://stackoverflow.com/questions/65908908/i-need-some-help-in-getting-my-first-open-source-project-running/65909903?noredirect=1#comment116542998_65909903
The user eol was very helpful and stated the following:

The error message tells us that yarn is not installed. Looking at the dev script inside the package.json we can see the actual command that is run:
"dev": "REACT_APP_BACKEND_URL=http://localhost:8008 yarn start",
The problem is that it's not listed as a (dev-) dependency in the package.json (you might want to create a PR for that), hence you don't get it installed when doing npm i .
To fix this and install yarn, you should run the following: npm i -D yarn or install it globally using npm i -g yarn.

Do you think we can do a Zoom call or any other screen sharing app and get this issue sorted as I really want to get started on this issue and submit my first PR.

Hi @tiagostutz!
I was able to install nvm and then install node version 14.15.4 and I was able to progress, but now I am getting this error when running npm run dev. Do you know what this error means?
The default interactive shell is now zsh.
ShoaibKhansMBP:app shoaibkhan$ npm run dev

<EMAIL_ADDRESS>dev /Users/shoaibkhan/Documents/Treehouse/Open Source Projects/stop-analyzing-embed/app
REACT_APP_BACKEND_URL=http://localhost:8008 yarn start

sh: yarn: command not found
npm ERR! code ELIFECYCLE
npm ERR! syscall spawn
npm ERR! file sh
npm ERR! errno ENOENT
npm ERR<EMAIL_ADDRESS>dev: REACT_APP_BACKEND_URL=http://localhost:8008 yarn start
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?
npm ERR!  /Users/shoaibkhan/.npm/_logs/2021-01-27T23_41_41_615Z-debug.log
ShoaibKhansMBP:app shoaibkhan$

Hi, I have 2 doubts

Should I implement frontend or connection with state,
Getting Error like below while running "npm run dev",

<EMAIL_ADDRESS>dev
REACT_APP_BACKEND_URL=http://localhost:8008 yarn start

'REACT_APP_BACKEND_URL' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>dev: REACT_APP_BACKEND_URL=http://localhost:8008 yarn start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:

Hi @MDPuneethReddy !

Frontend only
What is your environment (OS, node version...)?

Hi @tiagostutz !
I want to get started I am a beginner so what and how should I begin?

Awesome.! Do you have any experience with React, HTML/CSS? You could begin bringing the project up and running locally. Tell me if you have problems with that.

@tiagostutz I have much experience in HTML/CSS and I have a very little in react (but I am determined to learn it on the way) so can you please guide me how to proceed?

Awesome!
You can start following the REAMDE instructions on how to run the project locally.

i want to contribute can u share issues related html,css,js

Hi, @tiagostutz I am Amandeep and I am preparing for GSOC next year. I would like to contribute in any open source project in HTML, CSS or Javascript. Could you share some beginner issues that I could solve?

Hi, @tiagostutz  I am Sarfraz. I would like to contribute to any open source project in HTML, CSS, and JS and also familiar with the basics of react js as well.. are there any issues for me?

Hi there @omnaladkar  and @AmandeepSingh285  ! 👋🏻
Welcome to stop analyzing.!
Let's get something for you to tackle. Have you read the README instructions? Were you able to bring the project up and running? Need some help with that?
That's the first step, to put the project running and then let's tackle some issues. Ok?

Hi! @tiagostutz. I would love to work on it.
To my understanding, I have to style the like mode choice card using Tailwind CSS and add a close button to it as well. Is that it?

Hi @accordingtoherr !!
Part of... the interaction of the like mode is different from the choose mode. In the choose mode, you have 2 options and select the one that you prefer. In the "like mode" the user is presented just one option and he/she will like or deslike this option. So, there's some interaction change and it is a good idea to break these two types of interactions in different components.

hi, @tiagostutz  can I start working on styling the like mode??

Absolutely! Go ahead and let me know if you have any issues... =)

Hi @tiagostutz  I have a couple of questions

Do I have to use tailwind css or normal css would work fine?
Since main.css file is incredibly large, can I create another css file to style the like mode card, if not please specify what file I can use.
Is there any finished card that I can look upto for reference.

Hi @tiagostutz I have a couple of questions

Do I have to use tailwind css or normal css would work fine?

Since main.css file is incredibly large, can I create another css file to style the like mode card, if not please specify which file I can use.

Is there any finished card that I can look upto for reference.

Hey @tiagostutz
Can you assign an issue for me to work on ?
It would be great to start with HTML, CSS, Bootstrap, JS

Hi. I'm looking for couple of issues for solving for my midterm university project. I wonder if you can help me to do some
easy one's. I'm a beginner html,css developer so the topic I want is html and css.
@tiagostutz I would appreciate if you can help me out

Hi, i am prem and I am newbie in issue solving or i can say i want to start my journey from here. So, can you provide me some issue so that i can solve them? I am looking for webdesigning issue as I am starting from easy one.

@tiagostutz is this issue still open?
I would love to contribute.

Hey Folks! Is anyone working on this ? I would like to contribute. Can take the issue or if anyone is assigned or working on this issue I can help them

Hola Folks!! I am an absolute newbie to opensource. Is this issue still open ? If yes, then please tell me what to do. I would really love to do my first contribution...

Hey is it still available ?

hi I'm new to open source completely and would like to contribute and learn

Hello , I'm Bindu vani new to open source and I'm trying to prepare for gsoc 2024. I want to start my contributions and learn more!!!

Im intrested in working on this issue. could you please assign this issue to me?


SQLITE_ENABLE_JSON1 flag does not appear to compile sqlite3 with this option

SQLiteCpp with the SQLITE_ENABLE_JSON1 #289  doesn't appear to be working. I'm using the following query to test, which returns zero.
SELECT sqlite_compileoption_used('ENABLE_JSON1')

I changed the line in ./build.sh to
cmake -DCMAKE_BUILD_TYPE=Debug -DSQLITE_ENABLE_JSON1=ON -DSQLITECPP_USE_ASAN=ON -DSQLITECPP_USE_GCOV=OFF -DSQLITECPP_BUILD_EXAMPLES=ON -DSQLITECPP_BUILD_TESTS=ON ..

And created a quick test in Database_test.cpp - which fails. for ease, you can pull it from my fork here
#ifdef SQLITE_ENABLE_JSON1
TEST(Database, json1)
{
    remove("test.db3");
    {
        // Create a new database
        SQLite::Database db("test.db3", SQLite::OPEN_READWRITE | SQLite::OPEN_CREATE);
        EXPECT_FALSE(db.tableExists("test"));
        EXPECT_EQ(1, db.exec("SELECT sqlite_compileoption_used('ENABLE_JSON1')"));
    } // Close DB test.db3
    remove("test.db3");
}
#endif // SQLITE_ENABLE_JSON1

Any thoughts? Perhaps I'm missing something obvious here.

Hello, can you provide more details on your setup? What OS? Aka is your project really recompiling the sqlite library or using an existing one from your distribution?
I am not in front of a computer so I cannot check the feature (coming from PR #288)

Sure, MacOS 12.16 here

I think that's the reason, by default on Unixes CMake will detect the existing sqlite3 dynamic library installed in your system and use it
You want to use the cmake flag to force using the internal library built from source code

I know documentation is far from perfect, from the readme on troubleshooting:
or turn on the option(SQLITECPP_INTERNAL_SQLITE "Add the internal SQLite3 source to the project." ON) in CMakeFiles.txt
Let me know if you have any trouble with that

option(SQLITECPP_INTERNAL_SQLITE "Add the internal SQLite3 source to the project." ON))
looks to be on by default this is also mentioned again here
perhaps something here is overriding this:
cmake -DCMAKE_BUILD_TYPE=Debug -DSQLITE_ENABLE_JSON1=ON -DSQLITECPP_USE_ASAN=ON -DSQLITECPP_USE_GCOV=OFF -DSQLITECPP_BUILD_EXAMPLES=ON -DSQLITECPP_BUILD_TESTS=ON ..

I hope this helps
Cheers!


Feature/anmodninger via frontend

References

https://github.com/Energinet-DataHub/team-phoenix/issues/73

Codecov Report

Merging #2163 (fa0c07b) into main (1cce0d9) will increase coverage by 11.16%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main    #2163       +/-   ##
===========================================
+ Coverage   19.95%   31.11%   +11.16%
===========================================
  Files          81       98       +17
  Lines        2070     2439      +369
  Branches      278        0      -278
===========================================
+ Hits          413      759      +346
- Misses       1647     1664       +17
- Partials       10       16        +6

Files
Coverage Δ

...s/dh/api-dh/source/DataHub.WebApi/GraphQL/Query.cs
0.81% <0.00%> (+0.04%)
:arrow_up:

... and 25 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Request] Fluid conduit pipes transparancy

Got a question here about the fluid conduits
im using sphax, and i made the ender fluid conduit looking the same as the normal conduit hoping to see its liquid flowing throug it but i cannot see fluid :( is this code wise disabled or is it still a texture problem ?
If it is code wise could there be a option to make it viewable ? :) i love eyecandy :D

Ender fluid conduits don't have contents. Same as item c. They teleport directly from source to target.
Sent from my iPhone

On 31. Jul 2017, at 11:32, Jason<EMAIL_ADDRESS>wrote:
Warning: This message has had one or more attachments removed
Warning: (msg-2266-18.html).
Warning: Please read the "j-e-b-Attachment-Warning.txt" attachment(s) for more information.
Got a question here about the fluid conduits
im using sphax, and i made the ender fluid conduit looking the same as the normal conduit hoping to see its liquid flowing throug it but i cannot see fluid :( is this code wise disabled or is it still a texture problem ?
If it is code wise could there be a option to make it viewable ? :) i love eyecandy :D
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/SleepyTrousers/EnderIO/issues/4300
This is a message from the MailScanner E-Mail Virus Protection Service
The original e-mail message contained potentially dangerous content,
which has been removed for your safety.
The content is dangerous as it is often used to spread viruses or to gain
personal or confidential information from you, such as passwords or credit
card numbers.
Due to limitations placed on us by the Regulation of Investigatory Powers
Act 2000, we were unable to keep a copy of the original attachment.
The content filters found this:
MailScanner: Found a script in HTML message
--
Postmaster
MailScanner thanks transtec Computers for their support

Aaah that explains :) Thnx m8


Link to included template with  is broken

Hello,
It seems that last updates have broken the feature of showing a link to a included template using < cfinclude > tag.
Furthermore, I've noticed that the CHANGELOG.md file has not been updated for a while.
Whereas last version is v0.6.19, the changelog shows v.0.6.12 as the last one.

Damn, really ? I fixed this recently, perhaps something with the new file type has broken it, will have a look tomorrow

<cfinclude template="file.cfm"> does not show a link to "file.cfm".
(I am not using cfscript).

In a .cfm template?

Yes. A simple .cfm template including another one.
Just checked that last working version for me was 0.6.14.

I wonder if a .CFS file is required to activate parts of the plugin now, I'll have to investigate further, because it seems to work ok for me

Ok, I understand. I will have a try with a .CFS file and see if it works on 0.6.19.

Otherwise are there some more specifics you could help me identify? Are there any errors in debug output ? Specific template paths that don't work and those that do ? Do you use a workspace or individual directories?

I had someone let me know it was broken recently, but I think I resolved it for them

All template paths don't work for me. They stopped working since v0.6.15.

try uninstalling / reinstalling or possibly look for conflicting extensions ? do you know anyone else using the extension that might be able to help determine if its something environmental ? I'm on a Mac, are you on windows ? that could play a factor ? Sorry I don't have time to diagnose this at the moment, perhaps pull the project and see if you can work out why its not working ?

I've just tried uninstalling/reinstalling, without any luck.
I am on Windows. I have colleagues who also are experimenting this issue, although we use almost the same set of extensions.
Thanks for your effort. I will try to debug or detect conflicts with other extensions.

@eescovar I haven't given up, just a bit busy at the moment and its difficult when you cant reproduce something, have you tried vscode.dev to see if it works there ? it could have something to do with relative paths and the correct path separator or maybe case ?, I might check with other members of our team at work who are on windows. Also, I found this https://github.com/microsoft/vscode/issues/108256

Thanks, I really appreciate that.
Tried now on vscode.dev (great suggestion, btw). I've installed only CFML Editor. Same problem.

@eescovar I've managed to see this first hand, and it appears this is broken on windows, I'm going to try source a windows machine or VM so I can try debug the cause and fix it. Might be a week or two away though, any contributions would be welcome.

Ok! I just want to emphasize that it works on v0.6.14 (v0.6.15 has broken it).

@eescovar sorry a colleague seems to think its working on windows in v0.6.19 but that it can take a few seconds to appear, please reach out on cfml slack if you can?

@eescovar there could be some kind of conflict occurring with the old cflint extension, please ensure you're using the associated cfmleditor one.

Pushed some updates that might help

Sorry for the late response.
Just tried uninstalling all CFML Editor versions, then restared vscode, and then reinstalled v0.6.20.
Failed. :-(
Rolled back to v0.6.14, links work as expected.
I guess it is something related to the Windows OS that got broken since v0.6.15.

@eescovar a colleague had a similar issue but was running the old cflint extension, uninstalled it and installed cfmleditor-lint and it fixed the issue, could you check this isn't a similar problem for you ?

I didn't have the old cflint extension installed.
Just in case, I have installed cfmleditor-lint to check if this makes links work again. No luck.
For now, me and my colleagues will stay with v0.6.14, because we appreciate the link feature.

@eescovar v0.6.21 should fix this issue, finally got access to a windows laptop where I was able to replicate the issue, please let me know.

Issue fixed, @ghedwards! Thank you very much!


Adding additional Inovelli Switches: LZW31, LZW30, LZW30-SN, LZW36

Hey, just a heads up. We'll need to hold off on merging these for now. The multi target automatic updates seem to be buggy still. Unfortunately I can't investigate why until I am back from paternity leave in January.

No worries, we actually only one of the devices has a multi target config. If I remove that one hopefully that will allow it to go through. Let me know if that works or if there is more I need to do.


fix: various improvements to play better with shared caches

This PR introduces a collection of changes to integrate better with CDNs:

When a file has its contents updated or is deleted we add to gin's context a key FileChanged with the value set to the file's ID. The purpose of this key is so we can pick it up in any CDN middleware that may be interested in it for purging purposes.
When a presignedURL is requested we calculate the correct max-age based on the presignedURL's creation time and its expiration time. This way if a cache caches the file it will be cached for as long as the URL is valid only.
When getting files (including via presigned URLs) we set a response header Surrogate-Key that has as value the file ID being downloaded. The purpose of this key is to be able to tell a CDN to purge all cached objects related to the same file (i.e. images manipulated with various options and presigned URLs)
We added a middleware to integrate with Fastly's CDN and automatically purge files as needed.
We treat conditional requests to s3 correctly
Support Range header for GetFile

Closing in favor of individual PRs


Total amount of REP it says I have claimable does not match up with the amounts to claim in the list

see .7 REP vs .3985 REP
i claimed and my balance went up by .7629
so something must be missing from modal

not seeing this anymore actually