Inconsistent output for weekly groupings with offset and timezone parameter during time change

Steps to reproduce:

Connect to Influx using -precision rfc339 option, like influx --database <database> --username <username> --password <password> -precision rfc3339
Write a query that include a change hour from winter time to summer time grouping by week and starting from monday (4 days offset), for example with italian timezone:

SELECT sum("value") FROM "datapoint_values" WHERE "plant"='0001' AND "datapoint"='PWER' AND time >= '2020-03-20T00:00:00Z' AND time <= '2020-04-30T00:00:00Z' GROUP BY time(1w,4d) fill(0) tz('Europe/Rome')

Expected behavior:
time                      sum
----                      ---
2020-03-16T00:00:00+01:00 882795
2020-03-23T00:00:00+01:00 818340
2020-03-30T00:00:00+02:00 713050
2020-04-06T00:00:00+02:00 477533
2020-04-13T00:00:00+02:00 418878
2020-04-20T00:00:00+02:00 424883
2020-04-27T00:00:00+02:00 187801

Actual behavior:
time                      sum
----                      ---
2020-03-16T00:00:00+01:00 882795
2020-03-23T00:00:00+01:00 818340
2020-03-30T00:00:00+02:00 0
2020-03-30T01:00:00+02:00 713050
2020-04-06T00:00:00+02:00 477533
2020-04-13T00:00:00+02:00 418878
2020-04-20T00:00:00+02:00 424883
2020-04-27T00:00:00+02:00 187801

An additional line with zero result is printed (2020-03-30T00:00:00+02:00 0), also one line time is not midnight located (2020-03-30T01:00:00+02:00 713050)
Additional info:

The same behavior occurs with an offset of days whose module is> = 4 and <= 6

With offset day time null or < 4 (unitl sunday) the output looks like consistent:

SELECT sum("value") FROM "datapoint_values" WHERE "plant"='0001' AND "datapoint"='PWER' AND time >= '2020-03-16T00:00:00Z' AND time <= '2020-04-30T00:00:00Z' GROUP BY time(1w,3d) fill(0) tz('Europe/Rome')
name: datapoint_values
time                      sum
----                      ---
2020-03-15T00:00:00+01:00 846944
2020-03-22T00:00:00+01:00 1100599
2020-03-29T00:00:00+01:00 712901
2020-04-05T00:00:00+02:00 496406
2020-04-12T00:00:00+02:00 417564
2020-04-19T00:00:00+02:00 424568
2020-04-26T00:00:00+02:00 247997

SELECT sum("value") FROM "datapoint_values" WHERE "plant"='0001' AND "datapoint"='PWER' AND time >= '2020-03-16T00:00:00Z' AND time <= '2020-04-30T00:00:00Z' GROUP BY time(1w) fill(0) tz('Europe/Rome')
name: datapoint_values
time                      sum
----                      ---
2020-03-12T00:00:00+01:00 243645
2020-03-19T00:00:00+01:00 1472722
2020-03-26T00:00:00+01:00 657964
2020-04-02T00:00:00+02:00 596316
2020-04-09T00:00:00+02:00 425270
2020-04-16T00:00:00+02:00 418544
2020-04-23T00:00:00+02:00 427514
2020-04-30T00:00:00+02:00 5004

Environment info:

Linux 4.14.193-113.317.amzn1.x86_64 x86_64
InfluxDB v1.8.2 (git: 1.8 6a1299e8c6c81ef8fd573ed2b700217121baf04e)
InfluxDB shell version: 1.8.2

same behavior with updated version InfluxDB v1.8.3 (git: 1.8 563e6c3d1a7a2790763c6289501095dbec19244e)

Hi, any news about this issue?

Hi, this bug still remains with a change of the output. Now there isn't an additional line, but the result of the week where the time change is always 0
SELECT sum("value") FROM "datapoint_values" WHERE "plant"='0001' AND "datapoint"='PWER' AND time >= '2020-03-20T00:00:00Z' AND time <= '2020-04-30T00:00:00Z' GROUP BY time(1w,4d) fill(0) tz('Europe/Rome')
name: datapoint_values
time                      sum
----                      ---
2020-03-16T00:00:00+01:00 882795
2020-03-23T00:00:00+01:00 1531390
2020-03-30T00:00:00+02:00 0
2020-04-06T00:00:00+02:00 477533
2020-04-13T00:00:00+02:00 418878
2020-04-20T00:00:00+02:00 424883
2020-04-27T00:00:00+02:00 187801


Specify the import and pre-compute pipelines

This includes (but not limited to):

Geolocation
Expanding Reporting Context (e.g. containers for a given id)
Identifying matching identifiers (e.g. handle, doi and URL point to same resource)
Identifying connected events (e.g. workflow steps)

Import and pre-compute pipelines have been specified here:
https://docs.google.com/document/d/1aa02icZyr6a-qtD6pxR-lw7BMCjL4kgU84p2yLHlYAY/edit#
These have now already moved into implementation, so I'm moving this one to Review and we can close it off as implementation gets more complete


Octopus Agile: electricity current rate is wrong for the first 15-90 seconds of every 30 minutes interval

Describe the bug
I'm using Octopus Agile and trying to build automation where one of the conditions checks the electricity current rate to switch on the load if it is below a certain value. However, the electricity current rate doesn't get updated for the current time period for anywhere between 15 and 90 seconds after the start of the interval. For example, here is the delay of 1 minute and 10 seconds:

Reproduction steps
Just open the graph of Electricity Current Rate sensor and look at the update timestamps.
Expected behaviour
The current rate is correct as of the current time, without delays.
Tariff Code
E-1R-AGILE-FLEX-22-11-25-H
Integration Version
v7.5.2
Home Assistant Version
2023.7.3
Fresh Install?
After upgrading
Home Assistant Logs
Still obtaining, will update.

Forgot to say: the current workaround is to use "next rate" instead of "current rate".

Hi there. All sensors are set to update every 1 minute. If any sensors take longer than this, then this is under the mercy of Home Assistant. This is by design to get a balance between speed of updates and not degrading people's home assistant instances. If you are wishing to make this more frequent then HAs recommendation is to use automations via https://www.home-assistant.io/integrations/homeassistant/#service-homeassistantupdate_entity


refactor(api)!: renamed getCurrent functions to avoid ambiguity

closes #10193

I dislike this so much as a Rust dev 😭

That's a really good idea

Should we also change getAll to getAllWindows etc?
Also probably need to update examples

done

Also here https://github.com/tauri-apps/tauri/blob/59f89b2126d89d2ffef48211980e9c40575a0aee/examples/api/src/views/Window.svelte#L16


Get-DbaAgentJob, fix slowness

Verified issue does not already exist?
Yes
What error did you receive?
No errors, just lots of time spent waiting for output in instances with a lot of jobs
Steps to Reproduce
Get-DbaAgentJob -SqlInstance xxx

Are you running the latest release?
Yes
Other details or mentions
No response
What PowerShell host was used when producing this error
PowerShell Core (pwsh.exe), Windows PowerShell (powershell.exe), Windows PowerShell ISE (powershell_ise.exe)
PowerShell Host Version
it's slow on any version
SQL Server Edition and Build number
it's slow on any version

thanks to @wsuhoey for reporting, we identified two main fixes in search of "speeeeeed"
#newmethod

$d = Connect-DbaInstance name_of_instance

Measure-Command {
    $res = $d.Query("select name, job_id from msdb..sysjobs_view")
    foreach($j in $res) {
        $d.JobServer.GetJobByID($j.job_id) | format-list *
    }
}

#FASTER!!!!

$e = Connect-DbaInstance name_of_instance
$FieldsJob = 'LastRunOutcome', 'CurrentRunStatus', 'CurrentRunStep', 'CurrentRunRetryAttempt', 'NextRunScheduleID', 'NextRunDate', 'LastRunDate', 'JobType', 'HasStep', 'HasServer','CurrentRunRetryAttempt', 'HasSchedule', 'Category', 'CategoryID', 'CategoryType', 'OperatorToEmail', 'OperatorToNetSend', 'OperatorToPage'
$initFieldsJob = New-Object System.Collections.Specialized.StringCollection
[void]$initFieldsJob.AddRange($FieldsJob)

$e.SetDefaultInitFields([Microsoft.SqlServer.Management.Smo.Agent.Job], $initFieldsJob)
Measure-Command { $c.JobServer.Jobs | format-list * }

I'm going to add a merge of those two (the first avoids the complete enumeration, the second avoids SMO being the "slow-loader" guy) in the next few days.


Unit tests for api.go

Added unit tests for v1alpha/api.go
Note: These tests depend on the v1_suite_test.go test suite file from this PR: #24.

Working fine.
Running Suite: V1alpha Suite
============================
Random Seed:<PHONE_NUMBER>
Will run 18 of 18 specs

••••••••••••••••••
Ran 18 of 18 Specs in 0.000 seconds
SUCCESS! -- 18 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS

Ginkgo ran 1 suite in 1.119981223s
Test Suite Passed


Move dataset/rados.h to util/rados_internal.h

This is a multi-line test comment

With GitHub Markdown :sparkles:
Created by [create-or-update-comment][1]
[1]: https://github.com/peter-evans/create-or-update-comment


[Program]: Write a Kotlin program to convert time from 24-hour to 12-hour format

Description
Write a Kotlin program to convert time from 24-hour to 12-hour format
Input  : 12:05:45
Output : 12:05:45PM

How to contribute

Save the solution in program/program/convert-time-from-24-hour-to-12-hour-format/program/ConvertTimeFrom24-hourTo12-hourFormat.kt file

!assign


Should reset mock DAO in tear down

It doesn't make sense in the docs to setup mock and then reset immediately. It was very confusing for me when I was setting up test

Good catch. Thanks!

:+1:


streamingccl: add minimum flush interval cluster setting

PR: https://github.com/cockroachdb/cockroach/pull/60160
From release notes:

Added the bulkio.stream_ingestion.minimum_flush_interval cluster setting, which allows the user to control how often the stream ingestion job will flush. Note that the job may still flush more often if the in-memory buffers are filled. [#60160][#60160] {% comment %}doc{% endcomment %}

Closing after discussing with @pbardea as this setting is already doc'd in the generated table here: https://www.cockroachlabs.com/docs/dev/cluster-settings.html#settings and controls a feature that isn't public yet.


[Bug] Admin View has a random issue whereby route is not loaded. It would appear the query have some issues. This happens randomly

503 is service unavailable, which indicates that the admin service is hitting capacity problems. Likely related to the download csv feature.

There are a couple of Network Error entries in the logs. Probably related to the recent stability problems that we have had when we made the Download CSV link available.

I dun quite understand why the csv download affect only the route view and
not all aspect of admin view. When i check bookings and route passes,
everything looks ok
On Fri, 27 Oct 2017 at 7:05 AM, LoneRifle<EMAIL_ADDRESS>wrote:

503 is service unavailable, which indicates that the admin service is
hitting capacity problems. Likely related to the download csv feature.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/datagovsg/beeline-admin/issues/332#issuecomment-339825617,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVtU30t31WCTVmk40Be2feb_FRBoMNZ9ks5swRA1gaJpZM4QH0vK
.

Should we take away download csv for route pass first? Cause route page is
pretty improtant for ops.
We will put it back when it is more stable
On Fri, 27 Oct 2017 at 7:28 AM, Swee Chin Chua<EMAIL_ADDRESS>wrote:

I dun quite understand why the csv download affect only the route view and
not all aspect of admin view. When i check bookings and route passes,
everything looks ok
On Fri, 27 Oct 2017 at 7:05 AM, LoneRifle<EMAIL_ADDRESS>wrote:

503 is service unavailable, which indicates that the admin service is
hitting capacity problems. Likely related to the download csv feature.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/datagovsg/beeline-admin/issues/332#issuecomment-339825617,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVtU30t31WCTVmk40Be2feb_FRBoMNZ9ks5swRA1gaJpZM4QH0vK
.

We could upgrade the dyno as an alternative, though bear in mind that's $25/mth compared to the $7 we are currently paying

That is one way. I am just wondering why would the most record heavy
booking csv is ok compare to route pass. Cause rightfully, grab cant be
spamming the download of csv too
On Fri, 27 Oct 2017 at 7:31 AM, LoneRifle<EMAIL_ADDRESS>wrote:

We could upgrade the dyno as an alternative, though bear in mind that's
$25/mth compared to the $7 we are currently paying
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/datagovsg/beeline-admin/issues/332#issuecomment-339829955,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVtU39V5hvnEYqvQqlJ26LORaJeJqnrXks5swRZigaJpZM4QH0vK
.

It isn't okay - the csv dump is causing the subsequent stability issues. My worry is that the server dyno isn't doing proper cleanup after something like that happens, and this has knock-on effects.
I've upgraded the dyno to the standard 1X for now for stability sake and giving its increasing importance. Closing for now.


SampleApp: Missing camera permissions

Since Android 6 any app needs runtime permissions.
(see: https://stackoverflow.com/questions/26305107/how-to-fix-fail-to-connect-to-camera-service-exception-in-android-emulator)
In the SampleApp it was missing and the app failed to connect to the camera service.
I added in MainActivity.OnCreate:
            var permission = Android.Support.V4.Content.ContextCompat.CheckSelfPermission(this, Manifest.Permission.Camera);
            if (permission != Android.Content.PM.Permission.Granted)
            {
                Android.Support.V4.App.ActivityCompat.RequestPermissions(this, new string[] { Android.Manifest.Permission.Camera }, 50);
            }

But that is just a quick hack to get the SampleApp up and running.

HI @TedStryker thanks for saving my day ;) this resolved my issue.

@TedStryker
Updated to version 4.2.1. New method
bool allowed = await GoogleVisionBarCodeScanner.Methods.AskForRequiredPermission();

You can use this method to request for permission in your xamarin form project.
You will need Plugin.Permission and Plugin.CurrentActivity to do this, please refer to the sample / readme.md


allow custom_keyword! to support different struct name

I'd prefer to have uppercase Struct names, consistent with the actual keywords in syn::token::*.
I propose a syntax, like custom_keyword!(var as Var), that could be non-breakingly be introduced.
If this is something you'd consider, I'd be willing to implement it.

I think you can try using use var as Var; as a temporary workaround for this issue.


3.0 Does not work with an empyt catalogue

What I'm trying to achieve
Starting up the dashboard with a clean Saleor instance
Steps to reproduce the problem

New Saleor backend without populating db
Create superuser and login to dashboard

What I expected to happen
See an empty dashboard
Screenshots

Essentially there is a lot of code which looks like:
searchAttributes.result.data?.search.edges and countAllProducts.data?.products.totalCount--> however this needs to be updated to searchAttributes.result.data?.search?.edges and countAllProducts.data?.products?.totalCount

Which version of Saleor? AFAIR it should be resolved in 3.0

Sorry this was a while back - i think it would have been 3.0.0-a.7 looking at the timelines based on the issue open date


powerranks problem with chat formatting

powerranks problem with chat formatting
line #2 present this in config.yml
chat: {enabled: true, format: '%clans_colored_clan_tag% &l&4**&r%powerranks_prefix%&l&4**[player]&r» %powerranks_chatcolor%[msg]'}
the rank color rgb what I use in the tablist is number #3
tablist_modification: {enabled: true, format: '&a[%clans_clan_tag%]&r[prefix]&r[player]'}
the problem is because the chat formatting no support the color, actualize plz
Screenshots

can you give us debug info? such as Server version and PowerRanks version. or just screenshots of these commands:

/ver
/ver PowerRanks

this looks like it is working as intended, what's the issue?

can you give us debug info? such as Server version and PowerRanks version. or just screenshots of these commands:

/ver
/ver PowerRanks

powerranks version 1.9.9R3

To top on this, the gradient will work if you don't have powerranks chat formatting, ( i demonstrate here by showing a non-pr formatted chat ( % test cuz we all know % breaks formatting ), and one without


Separate query params and body content params

Rough support for body params
Updated swagger spec

For now, this will do until we get the rest of the stuff working.
@designmatty added a some fontello, and put most of the new styles in the component sass file

I'll make an issue to support nested json specs in the body params. We'll probably do that after most of the other interactive stuff is finished

cool looks nice.

Closes https://github.com/azavea/raster-foundry/issues/1495


[RFC] Liger FlexChunkLoss: Alignment and Distillation loss

🚀 The feature, motivation and pitch
We want to support various alignment and distillation loss functions.
Refer this PR on ORPO: #362
Progress
Alignment

[x] ORPO https://github.com/linkedin/Liger-Kernel/pull/362
[x] CPO https://github.com/linkedin/Liger-Kernel/pull/382
[x] DPO https://github.com/linkedin/Liger-Kernel/pull/378
[x] SimPO https://github.com/linkedin/Liger-Kernel/pull/386
[x] IRPO
[ ] KTO
[ ] f-PO

Distillation

[ ] KL divergence
[ ] cosine_similarity
[ ] earth_mover_distance
[ ] JSD
[ ] KVD

Design
Approach Overview:
The core idea is to extend the methods used in chunked Fused Linear Cross Entropy (FLCE) to various alignment algorithms. Here's how the process is structured:

Modular Optimization Process:

Every alignment algorithm’s optimization can be broken into three key steps:

Linear layer computation
Loss computation
Gradient calculation

Fused Linear and Loss Computation:

Similar to FLCE, we aim to fuse the linear layer with the loss computation for efficiency.

Chunking & Forward Optimization:

Since this is the final step in the model’s forward pass, we can also compute gradients directly during the forward pass instead of waiting for a separate backward pass.
We also chunk the input within the forward pass of the model, allowing significant reduction in peak gpu memory required.

Torch Compile for Kernel Optimization:

Instead of manually handling kernel-level optimizations, we let torch.compile automatically optimize kernel execution. This reduces the need for low-level optimizations while still achieving performance gains.

By combining these strategies, we efficiently optimize alignment algorithms while also simplifying development.
Key Findings
By leveraging torch.compile alongside optimization techniques like chunking, online softmax, etc, we observed close to custom triton kernel performance and reduced development time. This is why we want to introduce torch.compile as a key component of Liger.
References:

#227
https://gist.github.com/Chillee/22cd93e11b887db1f596ab754d60a899#file-lce_benchmark-py

Interface
Have a base class FlexChunkLoss that handles chunking, accumulation and compiling strategies.
A custom loss class wraps the  FlexChunkLoss and implements the loss fn that operates on a given chunk.
class Mycustomloss(FlexChunkLoss):
  def loss_fn(...):
    ..do something here

Alternatives
No response
Additional context
No response

I can take fused linear kl div. BTW, really nice illustration on the chunk linear op fusion from the paper. Very clear to new contributors 😄

@shivam15s @ByronHsu I think we should also consider including some of the loss functions commonly used for training embedding models, especially the popular ones supported in Sentence transformers.
It's quite common for embedding models to require large batch sizes to be trained well. Coupled with the fact that their batch/input structure is kind of similar to RLHF where we have positive and negative pairs, I believe that this can prove to be useful. I'd recommend supporting CoSENTLoss, MatryokshaLoss and TripleLoss for starters. Perhaps this can be its own roadmap separate to this one although the idea of chunking and fusing remains the same.

@pramodith that is a good idea! do you know if the models in embedding also has large vocab and suffer from memory bottleneck?

@ByronHsu most embedding models have a final Linear layer of shape (hidden_dim, hidden_dim), so vocab size doesn't really come into the picture for them so you're right to point it out, but it is common to have an effective batch size of 65k

Then i think chunk loss is still helpful given the large batch size

Then i think chunk loss is still helpful given the large batch size

Yes, I think so too. I can give this a try after we wrap up all the important RLHF and distillation losses. I'll also get Tom Aarsen's perspective since he's the lead of Sentence Transformers.

#take Simpo and Irpo since they are just extensions of CPO.

I will #take KTO  as the next

A little update on kto I am working now on the tests

Update on KTO loss I am done with the loss but I have problem with assertions.I am working on it.

Here's a refined version of your paragraph:

I was following this thread and working on a chunked, fused linear KL-divergence implementation for distillation use cases. Since distillation losses differ from preference losses, introducing a LigerFusedLinearDistillationBase parent class could be helpful.
In general, the distillation pipeline involves three key inputs: teacher_logits, student_logits, and ground_truth_label. The first two inputs are used to calculate the soft loss (KL divergence), while the latter two are used to compute the hard loss (cross-entropy). The final distillation loss is typically a weighted sum of these two components.
To leverage chunked, linear-fused optimizations, we could design the solution to accept inputs as teacher_tensor (BT, hidden_dim_teacher), student_tensor (BT, hidden_dim_student), and true_label (BT,). Using these inputs, we can apply the chunked, linear-fused approach to efficiently compute both the KL-divergence loss and the cross-entropy loss.
cc @ByronHsu, @shivam15s, @pramodith: What are your thoughts on this? Do you think it makes sense to include the cross-entropy loss as part of the DistillationBase class? Thanks for your feedback!

@hongpeng-guo yes! I like your approach it's cleaner to create a new Base class for distillation losses, we're kind of doing the same for the Alignment losses to by computing the nll (cross-entropy loss of the accepted responses inside the Base class.) https://github.com/linkedin/Liger-Kernel/blob/7e3683e23f8a9a5663913fd0ea7b0b03ea1a667b/src/liger_kernel/chunked_loss/fused_linear_preference.py#L37.

+1 on @hongpeng-guo proposal. @shivam15s can help polish the base class

Sounds good @hongpeng-guo, a separate base class for distillation is absolutely needed!

please review and comment my PR on KTO here https://github.com/linkedin/Liger-Kernel/pull/410

there is an update about #410

Is CPO-SimPO planned? This can be implemented in SimPO.
Reference: https://github.com/fe1ixxu/CPO_SIMPO
Quote
CPO and SimPO share similar objectives but have different goals. CPO adds a BC-regularizer to prevent the model from deviating too much from the preferred data distribution.
$L_{CPO}(\pi_\theta;U) = -E_{(x,y_w,y_l) \sim \mathcal{D}} \Big[ \log \sigma \Big( \beta \log \pi_{\theta}(y_w | x)  - \beta \log \pi_{\theta}(y_l | x) \Big) + \log \pi_\theta(y_w| x)\Big]$
SimPO incorporates length normalization and target reward margin to improve model performance and prevent the generation of long but low-quality sequences:
$L_{SimPO}(\pi_\theta;U) = -E_{(x,y_w,y_l) \sim \mathcal{D}} \Big[ \log \sigma \Big( \frac{\beta}{|y_w|} \log \pi_{\theta}(y_w | x)  - \frac{\beta}{|y_l|} \log \pi_{\theta}(y_l | x)  - \gamma  \Big) \Big]$
These two objectives can be jointly used, which we call CPO-SimPO:
$L_{CPO-SimPO}(\pi_\theta;U) = -E_{(x,y_w,y_l) \sim \mathcal{D}} \Big[ \log \sigma \Big( \frac{\beta}{|y_w|} \log \pi_{\theta}(y_w | x)  - \frac{\beta}{|y_l|} \log \pi_{\theta}(y_l | x)  - \gamma  \Big)+ \alpha \log \pi_\theta(y_w| x)\Big]$

@ccdv-ai I think this can be done via the existing set of hyperparams of setting compute_nll_loss=True and alpha for the BC regularizer and right now all our alignment loss functions do assume length normalization  https://github.com/linkedin/Liger-Kernel/blob/bd65c47999cebc2ac3dce39447ecec051b8b6159/src/liger_kernel/chunked_loss/fused_linear_preference.py#L51

there is an update about KTO on #410


reading COMTRADE files and getting a convert string to float error

I have files from new SEL-735 PQ meters that when downloaded from the relay itself as a COMTRADE file, will cause a string to float conversion error.

C:/Users/spark/Desktop/Comtrade Files/New folder/ZIGZAG_735 - ER 10015 15_57_54_736.cfg
Traceback (most recent call last):
File "c:\Users\nv7\Dropbox\files\PyCharm\COMTRADE-Reader\735plot.py", line 34, in
rec.load(sel_file + '.cfg', sel_file + '.dat')
File "c:\Users\nv7\Dropbox\files\PyCharm\COMTRADE-Reader\comtrade.py", line 754, in load
self._load_cfg_dat(cfg_file, dat_file)
File "c:\Users\nv7\Dropbox\files\PyCharm\COMTRADE-Reader\comtrade.py", line 767, in _load_cfg_dat
self._cfg.load(cfg_filepath)
File "c:\Users\nv7\Dropbox\files\PyCharm\COMTRADE-Reader\comtrade.py", line 324, in load
self._read_io(cfg)
File "c:\Users\nv7\Dropbox\files\PyCharm\COMTRADE-Reader\comtrade.py", line 454, in _read_io
self._time_multiplier = float(line)
ValueError: could not convert string to float: '1 \x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a\x1a'

I looked into the cfg file and noticed a additional space for null data

1,IA,A, ,A,   0.06466082,0,0,-32768,32767,   80.0000,1,P
2,IB,B, ,A,   0.06466082,0,0,-32768,32767,   80.0000,1,P
3,IC,C, ,A,   0.06466082,0,0,-32768,32767,   80.0000,1,P
4,IN,N, ,A,   0.06466082,0,0,-32768,32767,   80.0000,1,P
5,VA,A, ,V,   0.03667481,0,0,-32768,32767,    1.0000,1,P
6,VB,B, ,V,   0.03667481,0,0,-32768,32767,    1.0000,1,P
7,VC,C, ,V,   0.03667481,0,0,-32768,32767,    1.0000,1,P

which I removed but then I get a further errors of a similar type. The script I am using has been functional in the past when I have received an SEL event report file with a cev format and then converted it with SEL SynchroWAVE but this is the first time dealing directly with the relay COMTRADE file itself.  I tested to see if a file converted from SEL's SynchroWAVE software, which can convert to 1999 and 2013 formats, caused any issues as both binary and ascii formats but they all worked.
Attached is one COMTRADE file that is not sensitive in nature and is causing the error.
files.zip

Hi @spark706 , thanks for reaching out and attaching a sample. The text files (both .cfg and .hdr) doesn't end on EOF, but on a sequence of 0x1A (substitute character), which are not handled by python's method such as strip().

I updated the module so it will remove these characters of last lines of cfg files and both hdr/inf files. This solves the issue for the attached sample.


Listen on a unix socket rather than a port?

I generally prefer to have my web apps listen on a unix socket such as /var/run/my_app.sock instead of a TCP port so I don't have to try to keep track of which ports are used/available. All WSGI servers I have worked with offer this option as well. Would it be possible to add this to robyn, assuming it can't already do this?

@ibrewster , thank you for creating this issue. It is not possible to do it in Robyn right now, but I would love to add this in the future :D


Application title stuck on "Settings" after Use active terminal title as application title is disabled

Windows Terminal version
1.11.3471.0
Windows build number
10.0.19044.1586
Other Software
No response
Steps to reproduce
Enable "Use active terminal title as application title"
Disable "Use active terminal title as application title"

Expected Behavior
Title should revert to "Windows Terminal"
Actual Behavior
Title remains "Settings"

Weird. Thanks for the report!


issue with containg

the line
assertThat(Arrays.asList(1,1), org.hamcrest.Matchers.contains(1));
fail.

This is the intended behaviour. contains only matches if all parameters have a one-to-one correspondence to the list elements. So this would work:
assertThat(Arrays.asList(1,1), org.hamcrest.Matchers.contains(1, 1));

To check if the list has at least one element use
assertThat(Arrays.asList(1,1), org.hamcrest.Matchers.hasItem(1));

#140 ;)


refactor: Update to use EKS module which follows EKS Blueprints v5 direction

What this PR does / why we need it:

Update to use EKS module which follows EKS Blueprints v5 direction

Which issue(s) this PR fixes:
Reference https://github.com/aws-ia/terraform-aws-eks-blueprints/issues/1421
Quality checks

[ ] My content adheres to the style guidelines
[ ] I ran make test or make e2e-test and it was successful

I tried to run make test but got this:
make test
bash hack/run-tests.sh 'terraform' '*'
Error: Please specify a module
make: *** [test] Error 1

And when I try make e2e-test:
make e2e-test
make: *** No rule to make target `e2e-test'.  Stop.

I don't see an e2e-test target in the Makefile, and I'm not sure what a "module" is to target with make test. Any pointers on how to test would be appreciated
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Going to close this since the rearchitecture work will move away from this design


e2e/cypress: fix "Remove Credential" issue

#720 should be considered a workaround. For some reason, cypress thinks the "Remove Credential" button wasn't visible; because chef-dropdown had display:none.
I am not sure why this is happening, and what brought this change about -- digging through git didn't yield anything (for me! please try yourself).

What's odd is that when I click on the "..." button in the cypress window, all I see is another "..." button:

What could be related is this recently released Cypress version: https://docs.cypress.io/guides/references/changelog.html#3-3-2 (but I haven't verified this).

are these failures still happening? i haven't noticed them but i haven't been looking closely either


Issues with Refactor... command while moving the part of code

If I try to move the code by selecting the code then click on refactor from the context menu and move to a file
and select the file
it transfer's the whole files code to that file instead of moving the selected code

The issue lacks

mentioning of the language you are using
the extension you are using
a copy-paste friendly complete, but minimal, sample
precise steps

well All I am saying is the selected code is not being taken into account while moving the code using Refactor... or while generating the tests from the copilot

Do I understand correct that this is an issue with inline chat, e.g you do

right click > select "Copilot > Generate Tests"?

Yes
But it happens even if I try to move certain functions from file to another file using refactor menu from the right click context menu

Yes

Ok, please share the sample code snippet and where your cursor is when invoking inline chat

But it happens even if I try to move certain functions from file to another file using refactor menu from the right click context menu

That sound like another unrelated issue

both are related in my opinion as both the cases it is not considering the selection

please share the sample code snippet and where your cursor is when invoking inline chat

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Meilisearch - Added metadata as filterable attributes and polling on enqueued tasks

Hello there! back again with another improvement for Meilisearch component, I added metadata of the input documents to the metadata of meilisearch hits so no metadata information is lost, I also added all of the metadata content as filterable attributes to enable meilisearch filtering through the component. Moreover,some of meilisearch requests return an "enqueuedTask" , it creates an enqueuedTask depending on the request (adddocuments/update/delete/create-index etc...), so it was necessary to add proper polling on these created tasks to ensure correct upserting of large datasets and to avoid taking any action before these tasks are fully completed.
Example: loop on add documents enqueuedTask to ensure that the upsert function does not end before adding documents already happened on meilisearch cloud
This is the shape of the updated component, it is fully tested and working on production environment, feel free to ask about any of the proposed changes. Thanks

you are welcome @HenryHengZJ ! thanks for your review 👍


Correctly skip project classification for utils/

project is a list here, not a string.

@metaflow PTAL


Fix StreamParameters query string

@jhorstmann @lukasniemeier-zalando
The query string is broken right now, because the & is missing between the individual parameters.

:+1:


Error trying to execute multiple package.json scripts

The following lines were taken from package.json's script section.
"start": "concurrently --kill-others -p name -n \"caching,worker,server,webapp\" -c \"blue,white,green,yellow\" \"yarn:start-cache\" \"yarn:start-core\" \"yarn:start-server\" \"yarn:start-webapp\"",
"start-cache": "node --optimize_for_size --max_old_space_size=28672 ./packages/caching/index.js",
"start-core": "node ./packages/core/index.js",
"start-server": "node ./packages/server/server/server.js",
"start-webapp": "yarn --cwd packages/webapp react-scripts start",

When yarn start is executed on terminal, it fails with following error.

yarn run v1.15.2
$ concurrently --kill-others -p name -n "caching,worker,server,webapp" -c "blue,white,green,yellow" "yarn:start-cache" "yarn:start-core" "yarn:start-server" "yarn:start-webapp"
$ node ./packages/server/server/server.js
$ node --optimize_for_size --max_old_space_size=28672 ./packages/caching/index.js
$ yarn --cwd packages/webapp react-scripts start
$ node ./packages/core/index.js
error An unexpected error occurred: "Cannot create property '-npm-sap-com' on string '{"-npm-sap-com":"","-npm-sap-com-":""}'".

-npm-sap-com part is very foreign to me since I cannot find it in my repository.
Just to show that my code works properly and concurrently works when not referring any script in package.json, I am attaching the working concurrently script from my package.json.
"start": "concurrently --kill-others -p name -n \"caching,worker,server,webapp\" -c \"blue,white,green,yellow\" \"node --optimize_for_size --max_old_space_size=28672 ./packages/caching/index.js\" \"node ./packages/core/index.js\" \"node ./packages/server/server/server.js\" \"yarn --cwd packages/webapp react-scripts start\"",
It's output in terminal.

yarn run start                                                                                yarn run v1.15.2                                                                                $ concurrently --kill-others -p name -n "caching,worker,server,webapp" -c "blue,white,green,yellow" "node --optimize_for_size --max_old_space_size=28672 ./packages/caching/index.js" "node ./packages/core/index.js" "node ./packages/server/server/server.js" "yarn --cwd packages/webapp react-scripts start"                                                                                $ C:\Users\MohanKiranMathew\workspace\meraki\node_modules.bin\react-scripts start

Tag me if you require any more inputs from my side. Thanks.

It's been 2+ years, I'll close this issue.
-npm-sap-com seems the npm registry from the SAP company, which also has nothing to do with concurrently.
Possibly something in the dependencies.


trying to install mapnik in windows

i'm so confused about this process and documentation not that great, i installed mapnik 2.2.0 binary and added the path to environment path and i can use mapnik through the cmd
then i try to install mapnik via npm as a depedancy for a project i get this error:
C:\Users\ibrahim>npm install -g mapnik

<EMAIL_ADDRESS>preinstall C:\Users\ibrahim\AppData\Roaming\npm\node_modules.st
aging\mapnik-4095ae5e
npm install node-pre-gyp

C:\Users\ibrahim\AppData\Roaming\npm
`--<EMAIL_ADDRESS>npm ERR! Windows_NT 6.2.9200
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\ibrahim\AppData
\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "node-pre-gyp"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.8
npm ERR! path C:\Users\ibrahim\AppData\Roaming\npm\node_modules.staging\mapnik-
4095ae5e
npm ERR! code EBUSY
npm ERR! errno -4082
npm ERR! syscall rmdir
npm ERR! EBUSY: resource busy or locked, rmdir 'C:\Users\ibrahim\AppData\Roaming
\npm\node_modules.staging\mapnik-4095ae5e'
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\ibrahim\AppData\Roaming\npm\node_modules.staging\mapnik-4
095ae5e\npm-debug.log
npm ERR! Windows_NT 6.2.9200
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\ibrahim\AppData
\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "-g" "mapnik"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.8
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>preinstall: npm install node-pre-gyp
npm ERR! Exit status<PHONE_NUMBER>
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>preinstall script 'npm install node-pre-gyp
'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the mapnik package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm install node-pre-gyp
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs mapnik
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls mapnik
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\ibrahim\npm-debug.log
C:\Users\ibrahim>

I got the same error with node 6.9.5

Node Mapnik currently does not support Mapnik 2.x - You will need to use Mapnik 3.0.x series.

Re-opening because I realized there is more to this -- node-mapnik installed using npm install will attempt to bring its own prebuilt binary of mapnik.
I have no idea the current versions that node-pre-gyp supports but this is an issue with node-pre-gyp not with mapnik from what I can see. It appears that node-pre-gyp is not being installed.

Also please see https://github.com/mapnik/node-mapnik/issues/728#issuecomment-281399620


fix: Always set a committer on tags to resolve compatibility issue with GitLab.

Closes #235

@Shadow53 will you give this a try please?

Merging so that it will be in a release and therefore easier to test!


Improve explanation about out-of-resource eviction signals for CPU resource.

Either I'm misunderstanding something or the page is a bit confusing in regards to how evictions happen, the QOS classes and CPU.
Having a CPU request != cpu limit or cpu limit not defined makes the pod class automatically burstable or best effort.
https://kubernetes.io/docs/tasks/configure-pod-container/quality-service-pod/
However this doc seems to imply the QOS class is used to determine eviction which would imply a pod can be evicted for not defining cpu or going over cpu requests.
https://kubernetes.io/docs/tasks/administer-cluster/out-of-resource/#evicting-end-user-pods
I don't believe a pod can be evicted for CPU because the system can always continue to run and gain back CPU it needs, whereas memory disk are harder to reclaim.  Thus CPU does not provide an eviction signal
https://kubernetes.io/docs/tasks/administer-cluster/out-of-resource/#eviction-signals

/retitle Improve explanation about out-of-resource eviction signals for CPU resource.
https://kubernetes.io/docs/tasks/administer-cluster/out-of-resource/#node-conditions lists the conditions that can trigger eviction actions; these are specifically MemoryPressure and DiskPressure. As you have noted, CPUPressure is not a thing in Kubernetes.
https://kubernetes.io/docs/tasks/administer-cluster/out-of-resource/#eviction-signals confirms this.
@bbgobie it is a complicated topic to explain; can you suggest a change that you feel would explain it better?

In my opinion, CPU is an elastic resource, and there is no need to trigger pod eviction when exceeding it; QoS scheduling only depends on cpu.request and cpu.limit Distribute.Unable to trigger eviction based on increased CPU usage.

/retitle Improve explanation about out-of-resource eviction signals for CPU resource.
https://kubernetes.io/docs/tasks/administer-cluster/out-of-resource/#node-conditions lists the conditions that can trigger eviction actions; these are specifically MemoryPressure and DiskPressure. As you have noted, CPUPressure is not a thing in Kubernetes.
https://kubernetes.io/docs/tasks/administer-cluster/out-of-resource/#eviction-signals confirms this.
@bbgobie it is a complicated topic to explain; can you suggest a change that you feel would explain it better?

I think your post is actually pretty clear if anyone saw it, so what if the anchor for eviction signals was added to the evicting end user pods section.  Also perhaps there should be a page on compressible resources, I think an explanation would make it pretty clear for many users.
"If the kubelet is unable to reclaim sufficient resource on the node, kubelet begins evicting Pods."
"If the kubelet is unable to reclaim sufficient resources on the node, kublet begins evicting Pods based on eviction signals"

/kind cleanup
/triage accepted
/language en

This feels relevant to discussion in #19081

/reopen
Might be worth revisiting. If not picked up I would let it get closed by the bot though.


[BUG] Default notificationProvider has duplicate keys issue

Describe the bug
When two notifications of the same kind appear in close progression, the notifications can have same keys, causing both to disappear at once, even though the latest snackbar only just appeared.
Steps To Reproduce
When you edit a form entry (using useForm and the default notificationProvider), then quickly edit the same form again, then you will get two notification snackbars with the same key, causing unexpected behavior.

Expected behavior
What should happen instead is that both snackbars should have unique keys, so that they do not disappear together when they are closed.
Packages
"@refinedev/cli": "^2.16.37",
"@refinedev/core": "^4.54.0",
"@refinedev/devtools": "^1.2.7",
"@refinedev/kbar": "^1.3.12",
"@refinedev/mui": "^5.20.0",
"@refinedev/react-hook-form": "^4.9.0",
"@refinedev/react-router-v6": "^4.6.0",
"@refinedev/simple-rest": "^5.0.8",

Additional Context
No response

Hey @PercivalFigaro can you provide a minimal reproducible example? Are you building notification yourself? It's hard to judge without example.

Hey @PercivalFigaro, we're using predictable keys for notifications based on the type and resource etc. I think triggering a mutation repeatedly with a very close interval is not very common 😅 Also having them unique keys will be an issue if users are using preventDuplicate prop of <SnackbarProvider /> 🤔
I think it will be better to pass the key explicitly to the notification props in this case. Assuming you're also experimenting this based on your other issue report 😅 Let me know if we're on the same track here 🙏

@aliemir Okay, sounds good. I'm fine with leaving this as is. Thanks!


Reach

It doesn't work！


Front page makes the project look unsupported

Just thought Id give you a heads up that this is going on:

It looks like people put in a lot of effort here so it's shame that the front page doesn't reflect how updated the project actually is

Thanks!
The date is because github shows the commit author date and not the push date; we can't do anything about that.
For the releases, i'd either have to start remembering to make a release for every new package update (ugh) or delete the existing ones.
I've made a release for the latest version, at least.


Add "Layer1" as focus option to application form

Also makes the "Main Focus" mandatory.
Build preview:

/apply/grant
/apply/lts

FYI: I just did a test submission on the live website and everything seems to work.

@dergigi Let me know if you need further testing on this.

Thanks, I think we're all good @Bayernatoor - no further testing needed.


Consolidate get(Opaque|Opened)TypeArchetypeProperties into archetypeProperties()

When I introduced archetypeProperties, I missed the other two functions that do similar things. Consolidate them all under a new ArchetypeType::archetypeProperties().

@swift-ci please smoke test

@swift-ci please smoke test Linux


log missing during core start/stop

What version of V2Ray are you using?
master build
V2Ray 5.15.1 (V2Fly, a community-driven edition of V2Ray.) Custom (go1.22.2 linux/amd64)
What's your scenario of using V2Ray?
Not related
What problems have you encountered?
logs are missing during v2ray-core starting / stopping.
What's your expectation?
Completed log output should be sent and processed by log handler during starting / stopping.
Please attach your configuration here
{
  "log": {
    "loglevel": "debug"
  }
}

Server configuration:
// Not related

Client configuration:
// Not related

Logs output got during stop
No debug logs output when I hit Ctrl-C
2024/04/17 13:51:45 [Debug] app/dns: domain fonts.gstatic.com will use DNS in order: [UDP:<IP_ADDRESS>:53 DOH//<IP_ADDRESS>] [TypeA]
2024/04/17 13:51:45 [Debug] app/dns: UDP:<IP_ADDRESS>:53 querying DNS for: fonts.gstatic.com.
2024/04/17 13:51:45 [Debug] transport/internet/udp: dispatch request to: udp:<IP_ADDRESS>:53
2024/04/17 13:51:45 [Debug] transport/internet/udp: UDP original destination: udp:<IP_ADDRESS>:53
2024/04/17 13:51:45 [Debug] [3389796255] proxy/dokodemo: processing connection from: <IP_ADDRESS>:40438
2024/04/17 13:51:45 [Info] [3389796255] proxy/dokodemo: received request for <IP_ADDRESS>:40438
2024/04/17 13:51:45 [Info] [3389796255] app/dispatcher: taking detour [dns-out] for [udp:<IP_ADDRESS>:53]
2024/04/17 13:51:45 [Info] [3389796255] proxy/dns: handling DNS traffic to udp:<IP_ADDRESS>:53
2024/04/17 13:51:45 <IP_ADDRESS>:40438 accepted udp:<IP_ADDRESS>:53 [dns-out]
2024/04/17 13:51:45 [Info] app/dns: UDP:<IP_ADDRESS>:53 got answer: fonts.gstatic.com. TypeA -> [<IP_ADDRESS>] 2.203529ms
2024/04/17 13:51:45 [Debug] app/dns: UDP:<IP_ADDRESS>:53 updating IP records for domain:fonts.gstatic.com.
2024/04/17 13:51:45 [Debug] transport/internet/udp: UDP original destination: udp:<IP_ADDRESS>:53
2024/04/17 13:51:45 [Debug] [1341554531] proxy/dokodemo: processing connection from: <IP_ADDRESS>:53992
2024/04/17 13:51:45 [Info] [1341554531] proxy/dokodemo: received request for <IP_ADDRESS>:53992
2024/04/17 13:51:45 [Info] [1341554531] app/dispatcher: taking detour [dns-out] for [udp:<IP_ADDRESS>:53]
2024/04/17 13:51:45 [Info] [1341554531] proxy/dns: handling DNS traffic to udp:<IP_ADDRESS>:53
2024/04/17 13:51:45 <IP_ADDRESS>:53992 accepted udp:<IP_ADDRESS>:53 [dns-out]
2024/04/17 13:51:46 [Debug] app/dnscircuit/ospf: received <IP_ADDRESS>-><IP_ADDRESS> OSPFv2 | Link State Acknowledgment | PacketLength: 44
AreaId: 0 | RouterId: <IP_ADDRESS>
LSAcknowledgement Content:
  {LSAge:1 LSType:5 LinkStateID:2899947008 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:49626 Length:36 LSOptions:2}
^C2024/04/17 13:51:46 [Debug] app/log: Logger closing // <--- This is where I hit Ctrl-C
root@debianpov:~#

Logs output expected during stop
Expection: Debug logs during stopping should still be printed when I hit Ctrl-C.
This is done by commenting log app close
2024/04/17 13:55:13 [Debug] transport/internet/udp: UDP original destination: udp:<IP_ADDRESS>:443
2024/04/17 13:55:13 [Debug] [467673545] proxy/dokodemo: processing connection from: <IP_ADDRESS>:60025
2024/04/17 13:55:13 [Info] [467673545] proxy/dokodemo: received request for <IP_ADDRESS>:60025
2024/04/17 13:55:13 [Warning] [467673545] app/dispatcher: default route for udp:<IP_ADDRESS>:443
2024/04/17 13:55:13 [Info] [467673545] proxy/freedom: opening connection to udp:<IP_ADDRESS>:443
2024/04/17 13:55:13 <IP_ADDRESS>:60025 accepted udp:<IP_ADDRESS>:443 [direct]
2024/04/17 13:55:13 [Debug] transport/internet/udp: UDP original destination: udp:<IP_ADDRESS>:53
2024/04/17 13:55:13 [Debug] [3741104488] proxy/dokodemo: processing connection from: <IP_ADDRESS>:42201
2024/04/17 13:55:13 [Info] [3741104488] proxy/dokodemo: received request for <IP_ADDRESS>:42201
2024/04/17 13:55:13 [Info] [3741104488] app/dispatcher: taking detour [dns-out] for [udp:<IP_ADDRESS>:53]
2024/04/17 13:55:13 [Info] [3741104488] proxy/dns: handling DNS traffic to udp:<IP_ADDRESS>:53
2024/04/17 13:55:13 <IP_ADDRESS>:42201 accepted udp:<IP_ADDRESS>:53 [dns-out]
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: received <IP_ADDRESS>-><IP_ADDRESS> OSPFv2 | Link State Acknowledgment | PacketLength: 84
AreaId: 0 | RouterId: <IP_ADDRESS>
LSAcknowledgement Content:
  {LSAge:1 LSType:5 LinkStateID:1746484736 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:6138 Length:36 LSOptions:2}
  {LSAge:1 LSType:5 LinkStateID:1746484480 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:8944 Length:36 LSOptions:2}
  {LSAge:1 LSType:5 LinkStateID:2890097152 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:6247 Length:36 LSOptions:2}
^C2024/04/17 13:55:13 [Debug] app/log: Logger closing // <--- This is where I hit Ctrl-C
2024/04/17 13:55:13 [Info] transport/internet/udp: failed to read UDP msg > read udp <IP_ADDRESS>:12345: use of closed network connection
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: flushing 4 self-originated external LSAs before shutting down
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: area 0 is pre-maturing LSA({LSType:5 LinkStateId:3030619648 AdvRouter:3232257801})
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: area 0 is pre-maturing LSA({LSType:5 LinkStateId:1746484736 AdvRouter:3232257801})
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: area 0 is pre-maturing LSA({LSType:5 LinkStateId:1746484480 AdvRouter:3232257801})
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: area 0 is pre-maturing LSA({LSType:5 LinkStateId:2890097152 AdvRouter:3232257801})
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: area 0 flushing 1 self-originated LSAs before shutting down
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: area 0 is pre-maturing LSA({LSType:1 LinkStateId:3232257801 AdvRouter:3232257801})
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: sent via interface ens192 <IP_ADDRESS>-><IP_ADDRESS> OSPFv2 | Link State Update | PacketLength: 172
AreaId: 0 | RouterId: <IP_ADDRESS>
LSUpdate Content:
  NumOfLSAs: 4
  LSAheader: {LSAge:3600 LSType:5 LinkStateID:3030619648 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:60661 Length:36 LSOptions:2}
LSAContent: {NetworkMask:4294967040  ExternalBit:128 Metric:10000 ForwardingAddress:0 ExternalRouteTag: 0}
  LSAheader: {LSAge:3600 LSType:5 LinkStateID:1746484736 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:6138 Length:36 LSOptions:2}
LSAContent: {NetworkMask:4294967040  ExternalBit:128 Metric:10000 ForwardingAddress:0 ExternalRouteTag: 0}
  LSAheader: {LSAge:3600 LSType:5 LinkStateID:1746484480 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:8944 Length:36 LSOptions:2}
LSAContent: {NetworkMask:4294967040  ExternalBit:128 Metric:10000 ForwardingAddress:0 ExternalRouteTag: 0}
  LSAheader: {LSAge:3600 LSType:5 LinkStateID:2890097152 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:6247 Length:36 LSOptions:2}
LSAContent: {NetworkMask:4294967040  ExternalBit:128 Metric:10000 ForwardingAddress:0 ExternalRouteTag: 0}
2024/04/17 13:55:13 [Debug] app/dnscircuit/ospf: sent via interface ens192 <IP_ADDRESS>-><IP_ADDRESS> OSPFv2 | Link State Update | PacketLength: 64
AreaId: 0 | RouterId: <IP_ADDRESS>
LSUpdate Content:
  NumOfLSAs: 1
  LSAheader: {LSAge:3600 LSType:1 LinkStateID:3232257801 AdvRouter:3232257801 LSSeqNumber:2147483650 LSChecksum:50785 Length:36 LSOptions:2}
LSAContent: {Flags:2 Links:1 Routers:[{Type:2 LinkId:3232257793 LinkData:<PHONE_NUMBER> Metric:10 TOSNum:0 TOSs:[]}]}
2024/04/17 13:55:14 [Debug] app/dnscircuit/ospf: received <IP_ADDRESS>-><IP_ADDRESS> OSPFv2 | Link State Acknowledgment | PacketLength: 124
AreaId: 0 | RouterId: <IP_ADDRESS>
LSAcknowledgement Content:
  {LSAge:3600 LSType:5 LinkStateID:3030619648 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:60661 Length:36 LSOptions:2}
  {LSAge:3600 LSType:5 LinkStateID:1746484736 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:6138 Length:36 LSOptions:2}
  {LSAge:3600 LSType:5 LinkStateID:1746484480 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:8944 Length:36 LSOptions:2}
  {LSAge:3600 LSType:5 LinkStateID:2890097152 AdvRouter:3232257801 LSSeqNumber:2147483649 LSChecksum:6247 Length:36 LSOptions:2}
  {LSAge:3600 LSType:1 LinkStateID:3232257801 AdvRouter:3232257801 LSSeqNumber:2147483650 LSChecksum:50785 Length:36 LSOptions:2}
2024/04/17 13:55:14 [Debug] app/dnscircuit/ospf: exiting ens192 runSendLoop
2024/04/17 13:55:14 [Debug] app/dnscircuit/ospf: exiting ens192 runReadDispatchLoop
2024/04/17 13:55:15 [Debug] app/dnscircuit/ospf: exiting ens192 runReadLoop
root@debianpov:~#

From what I've seen, this is caused by improper sequence of starting / stopping features, as log is designed to be a plugable feature too.
I am considering add an interface like IsLogHandlerApp in log handler registry, to help core tell which app "looks like" a log handler, then it can explicitly manipulate the sequence during start and stop.

/remove stale


Strong Server but My Stream freezes why

Hi
My Stream freezes why
I have 2 high server but my stream Freezing after seconds
Server one on contabo
CPU : 1-2 x Intel Xeon E5 2630v4 1-2 x (10 x 2.20 GHz)
RAM : 256 GB REG ECC
Port / bandwidth : 1 Gbit/s port UNLIMITED Traffic
Operating system : Ubuntu 16.04
Using this Tutorial For install YouPHPtube And NGINX RTMP
https://github.com/YouPHPTube/YouPHPTube/wiki
Sent stream Via FFMPEG With Loop
Code Loop
#!/bin/bash while :; do ffmpeg -re -i http://adtv.ercdn.net/adsport2/adsport2_720p.m3u8 -s hd720 -r 60 -g 60 -b:v 3M -c:v libx264 -x264-params keyint=60 -bufsize 500k -c:a aac -strict -2 -ar 44100 -b:a 128k -f flv rtmp://myipserver/live?p=4adc701664135e50b2354496086d53dd/5d70337586747 </dev/null > /dev/null 2>&1 done &
Server Two From a local distributor
RAM 16 GB
Processeur CPU 8 Cores / 8 Threads [Intel Atom™ 2,5 GHz]
Débit Internet 1 Gb/s (Garanti 700 Mb/s)
Operating system : Ubuntu 16.04
Using This Tutorial For install YouPHPtube And NGINX RTMP
https://github.com/YouPHPTube/YouPHPTube/wiki
Sent stream Via FFMPEG With Loop
Code Loop
#!/bin/bash while :; do ffmpeg -re -i http://adtv.ercdn.net/adsport2/adsport2_720p.m3u8 -s hd720 -r 60 -g 60 -b:v 3M -c:v libx264 -x264-params keyint=60 -bufsize 500k -c:a aac -strict -2 -ar 44100 -b:a 128k -f flv rtmp://myipserver/live?p=4adc701664135e50b2354496086d53dd/5d70337586747 </dev/null > /dev/null 2>&1 done &
This My Nginx Config http://lamadya.com/nginx.conf
But My Stream freezes why Please Help Me

freezes like how? have you seen your logs? without being able to see what's in logs and knowing if your source runs without problems i can't help you


Update LlamaSteeds.java

yo can i make it chickens instead of llamas thanls?

Hello! I won't be merging this, as this is where the actual code for Llama Steeds that needs to work for llamas is stored; however, feel free to do whatever you would like in your own version! The license permits modification and redistribution as long as you provide proper attribution. Here are a few resources to get you started:

Setting up a mod development environment
Fabric modding terminology
Fabric mixins (the fancy things that lets the mod override what prevents llamas from being saddled)


Support deployment of GenAIExamples/CodeGen

At least including:

helm chart
manifests

Completed by the following PRs
#42
#40
#36
#27
#22
#14


Prom atlas/crumbl plugin

Looks pretty dang good, and I like the use of messageService.sendPagedEmbed :)


Level 7 wrong rest times

Summary
I am somehow getting the wrong values from the restTimes supplier in level 7. i used the same concept in level 5 where i only called get once. For this level, i called get under run in Simulator7 to check if the server decides to rest (if the rest time from the supplier is more than zero) but so far i am only getting 0.0.
Description
Describe the problem and code snippets if necessary
Screenshots (if any):
Insert Images here if necessary

Hello I had one similar issue before. I got the wrong timimgs because I wrapped the restTime supplier in a Lazy. You can check my prev post here  to see if you are getting the same issue.

Hi I think using the supplier directly would give non 0 values since Lazy calls for the supplier's get() once and stores it as an attribute. Therefore, when calling for Lazy's get(), it would repeatedly return the first value, which is 0.0


The status of the rule is Stopped

1、create stream
curl -X POST
http://<IP_ADDRESS>:9081/streams
-H 'Content-Type: application/json'
-d '{"sql": "create stream demo() WITH (FORMAT="JSON", TYPE="edgex")"}'
2、create rule
curl -X POST
http://<IP_ADDRESS>:9081/rules
-H 'Content-Type: application/json'
-d '{
"id": "rule1",
"sql": "SELECT uint8 FROM demo WHERE uint8 > 20",
"actions": [
{
"rest": {
"url": "http://localhost:48082/api/v1/device/7d80374d-cb55-4cab-ab68-358d2b9d1538/command/a02b8dbb-61cf-4e5d-9fae-2838105a24ff",
"method": "put",
"dataTemplate": "{"Bool":"true", "EnableRandomization_Bool": "true"}",
"sendSingle": true
}
},
{
"log":{}
}
]
}'
3、show rule status
Connecting to <IP_ADDRESS>:20498...
[
{
"id": "rule1",
"status": "Stopped: cannot get the plugin file path: invalid name edgex: not exist."
}
]
question：
When creating a stream,TYPE="edgex"，What is the meaning of “TYPE” ，and “ cannot get the plugin file path: invalid name edgex: not exist”，the “edgex” is the file “edgex.yaml” in etc/source?? this file is exists.

Could you please refer to this link - https://github.com/emqx/kuiper/issues/569? It can be solved by compile a native build with edgex support.

I was build kuiper with "make build_with_edgex",so Kuiper should support EdgeX. Kuiper's plugins/source folder is empty.

The edgex will not compiled into the kuiperd binary file, it will not be a separated so file in plugins/source folder. From your log, it shows that the edgex has not been compiled into kuiperd binary file.

Shouldn't Kuiper be compiled into edgex enabled binaries?
How can I check whether edgex is compiled into kuiperd binary?
_build/log/ stream.log file shows "Listen error: listen tcp :20498: bind: address already in use" file="server.go:72"
Thank you very much.

Shouldn't Kuiper be compiled into edgex enabled binaries?
How can I check whether edgex is compiled into kuiperd binary?

The separated binary version of Kuiper does not include EdgeX source because EdgeX requires additional libraries (zeromq library), and Kuiper cannot be started without installing the lib. Kuiper is not only for EdgeX, so it will confuse user that do not use EdgeX.

build kuiper with "make build_with_edgex" ,return "Build successfully"
_build/log/ stream.log file shows "Listen error: listen tcp :20498: bind: address already in use" file="server.go:72"
Thank you very much.

It seems there already have an existed kuiperd process, you need to kill it.

Thank you very much for your patient answers . I understand what you mean. But I think it’s not a compilation problem .  Because I ran the source code of edgex-go, installed zeromq, and compiled and ran successfully. The readme file under kuiper is written to be compiled with "make build_with_edgex", which means that this version of kuiper contains the source of Edgex.
When I get data from the EdgeX Message bus(MQTT server) when creating a stream, the rule state is running.


Custom Row Container?

I'm new to typescript and just threw this together really quickly, but what do you think about a customRowContainer prop to allow, for example, alternating color schemes?
Also, I haven't looked too closely but some way to expose row props to the row component would, in addition, allow for onRowClick actions.
I'll clean this up with your suggestions if you think something approximate to what I have is a valid feature request.

@colemars sweet thanks for the PR! I definitely like the idea of adding in more row styling. I'll take a closer look when I get a bit more time. yea I'm thinking of adding a rowStyle prop that can optionally take an index. I'll make an issue out of it.
but I think it'll look like this:
<Table
  data={data}
  columns={columns}
  rowStyle={object | (index: number) => object}
/>

And yea I like the idea of adding a customRow, it might be an advanced feature.


Scene change sequentially

I want to change multiple scenes sequentially like animation when the tour button is clicked.
I used "viewer.loadScene("Scene1"); viewer.loadScene("Scene2"); viewer.loadScene("Scene3");".
But there is an error in change. I just want to change scene's panorama image quickly. What should i do?

Could you elaborate on the error you see?
JavaScript is generally asynchronous, as is the Pannellum API, so the loadScene function returns immediately, not when the scene has finished changing. You need to use an event listener to see when the scene has actually changed before changing to the next scene, e.g.
viewer.on('load', function() {
  window.setTimeout(function() {
    viewer.off('load');
    viewer.on('load', function() {
      window.setTimeout(function() {
        viewer.off('load');
        viewer.loadScene('Scene3');
      }, 3000);
    });
    viewer.loadScene('Scene2');
  }, 3000);
});
viewer.loadScene('Scene1');

will load Scene2 three seconds after Scene1 finishes loading and then load Scene3 three seconds after Scene2 finishes loading.

There is not console error. But scene image changing does not work properly. But when I drag mouse, only hot spot buttons move.The code you provided works well. But I want to use setTimeout function in for loop.
ex) var sceneList = ["Scene1","Scene2","Scene3"];
for (var i .... sceneList length)
{
viewer.loadScene(sceneList[i]);
}
How do I fix the code?

Hi Matthew
Quick question.
What's the purpose of the line
viewer.off('load');
in the code above?
Only asking because of problems with on(load) firing early, and maybe there is a potential solution for me here.
Best wishes
Derek

In this example, the goal was to replace the original event listener with a new one. Thus, viewer.off('load') is called to remove the old load event listener before adding the new one, so each event listener is only fired once.


Should be able to use negative timestamps for m command

Started in branch
feat-43

Implemented in https://github.com/z5267282/mve/pull/111


Cannot convert value to type System.String.

Update-MSCommerceProductPolicy -PolicyId AllowSelfServicePurchase -ProductId $product.ProductID -Enabled $false
Update-MSCommerceProductPolicy : Cannot process argument transformation on parameter 'ProductId'. Cannot convert value
to type System.String.
At line:1 char:78

... licyId AllowSelfServicePurchase -ProductId $product.ProductID -Enable ...

                                           ~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidData: (:) [Update-MSCommerceProductPolicy], ParameterBindingArgumentTransformatio
nException
FullyQualifiedErrorId : ParameterArgumentTransformationError,Update-MSCommerceProductPolicy

[Enter feedback here]

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

ID: 5082f7ed-ad40-04c1-5c6c-59e67b253257
Version Independent ID: 3d8b109d-285c-952f-2fa4-9fa244c2a3f9
Content: Use AllowSelfServicePurchase for the MSCommerce PowerShell module
Content Source: microsoft-365/commerce/subscriptions/allowselfservicepurchase-powershell.md
Service: o365-administration
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@ParaGlideRocks Thank you for your feedback. This looks like the variable $ product did not get captured correctly.
before you run the command, please capture the value for the variable, Below is an example
$product = Get-MSCommerceProductPolicies -PolicyId AllowSelfServicePurchase | where {$_.ProductName -match 'Power Automate'}
Once the variable is captured, you can run the cmdlet
Update-MSCommerceProductPolicy -PolicyId AllowSelfServicePurchase -ProductId $product.ProductID -Enabled $false
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @ParaGlideRocks Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Exception with useGeneratedKeys = false

Exception with  useGeneratedKeys set to false.  same code set to true will be worked!
Suppressed: java.lang.ClassCastException: class java.lang.Long cannot be cast to class java.lang.Integer (java.lang.Long and java.lang.Integer are in module java.base of loader 'bootstrap')
...
        <dependency>
            <groupId>org.mariadb</groupId>
            <artifactId>r2dbc-mariadb</artifactId>
            <version>1.1.3</version>
        </dependency>
...

This is my test code in mapper:

//  create table TEST1(ID bigint NOT NULL AUTO_INCREMENT , NAME VARCHAR(10) NULL, PRIMARY KEY(ID));
  @Insert("INSERT INTO TEST1(NAME)" +
      " VALUES(#{name})")
  @Options(useGeneratedKeys = true, keyProperty = "id", keyColumn = "ID")
  Mono<Long> addTest1(Tmod mod);  // **This is Worked.**

  @Insert("INSERT INTO TEST1(NAME)" +
      " VALUES(#{name})")
  @Options(useGeneratedKeys =false)
  Mono<Long> addTest1( @Param("name")String name); // **This throws  Exception**

//  create table TEST2(ID bigint NOT NULL  , NAME VARCHAR(10) NULL, PRIMARY KEY(ID));
  @Insert("INSERT INTO TEST2(ID,NAME)" +
      " VALUES(#{id},#{name})")
  Mono<Long> addTest2(@Param("id") long id, @Param("name")String name);// **This throws  Exception too**

I change result type to Mono
  @Insert("INSERT INTO TEST2(ID,NAME)" +
      " VALUES(#{id},#{name})")
  Mono<Void> addTest2(@Param("id") long id, @Param("name")String name);

throws exception:
	Suppressed: org.apache.ibatis.binding.BindingException: Mapper method 'pro.chenggang.example.reactive.mybatis.r2dbc.spring.mapper.Test2.addTest2' has an unsupported return type: class reactor.core.publisher.Mono 		at pro.chenggang.project.reactive.mybatis.support.r2dbc.binding.MapperMethod.rowCountResult(MapperMethod.java:151) 		at pro.chenggang.project.reactive.mybatis.support.r2dbc.binding.MapperMethod.execute(MapperMethod.java:100) 		at pro.chenggang.project.reactive.mybatis.support.r2dbc.binding.MapperProxy$PlainMethodInvoker.invoke(MapperProxy.java:160) 		at pro.chenggang.project.reactive.mybatis.support.r2dbc.binding.MapperProxy.invoke(MapperProxy.java:83) 		at net.bytebuddy.renamed.java.lang.Object$ByteBuddy$OQMpyh45.addTest2(Unknown Source)

I located this error is :
result.getRowsUpdated() .
Release Notes for MariaDB Connector/R2DBC 1.1.3
Notable Changes

To support the R2DBC 1.0.0 specification, the following implementation has been added or changed:

Result.getRowsUpdated() now returns Mono. Prior to this release, Result.getRowsUpdated() returned Mono.

HI Copper,
Thanks for reporting this.
Since I've been busy for a long time, I haven't adapted the 1.0.0 R2DBC specification yet.I would finish adapting the 1.0.0 specification of R2DBC driver in this month.As long as I have time to do this.I'll also check the recently updates of mybatis3 and mybatis-spring,and try to catch up official repository as soon as possible.

Finished compatible with r2dbc-spi both (0.9.1.RELEASE) version and 1.0.0.RELEASE version. See branch: 3.x


Client needs to close SQL cursor before running more queries

Example testcase:
create or replace table qtemp.tt as (
select * from qsys2.syslimits_basic
) with data on replace delete rows;

select * from qtemp.tt;

create or replace table qtemp.tt as (
select * from qsys2.syslimits_basic
) with data on replace delete rows;
select * from qtemp.tt;

This works if you scroll through to force the client to fetch all rows of the select result set (which implicitly closes the cursor). Otherwise, the second create or replace fails with [SQL0951] Object TT in QTEMP not altered. (due to the cursor still being open from the select)

FYI @ThePrez, this query is always returning this result below, even on main.
-- this returns success false (as below)
create or replace table qtemp.tt as (
select * from qsys2.syslimits_basic
) with data on replace delete rows;

-- but this still works
select * from qtemp.tt;

{
  "id": "query3",
  "has_results": false,
  "error": "null",
  "success": false
}


Feature: hotkey map and commentable

Description
feature

Allows writing a comment to a selected notification

Inspired by junegunn's gstash function

Display a preview screen with all hotkeys (accessible with ?)

Listing of all hotkeys and an information to set the preferred editor
--bind "change:first"

Will be triggered whenever the query string is changed and goes to the first hit

U/D performs incremental scrolling changes, W/S advances half a page with each keystroke

formatting

Slight changes in the formatting

the header text is dimmed
Reduction of the margin and limitation of the border to the upper edge

GIF
Hotkey map

Writing a comment with vim

This is awesome! you're a fzf wizard 🧙♂
I might prefer different different bindings for scrolling the preview. Maybe in the future we can add a config file?

Thnak you.

Two ways forward to solve the keybinding issue.
1. Easy solution

Remove all preview scrolling keyboard bindings from gh notify and write in the Hotkey Map/Help and README section that the user can change the FZF_DEFAULT_OPTS environment variable in the .zshrc/.bashrc file and refer to the man page or README of fzf for instructions.

The following example is my fzf setup.
# <--------- fzf command-line fuzzy finder ---------- >
#  ~/.zshrc
export FZF_DEFAULT_OPTS="
--layout=reverse
--info=inline
--multi
--header 'CTRL+V - "$(grep -Eom1 '^[^(0-9)]+' <($EDITOR -h))" | CTRL+Y - Copy | ? - Toggle'
--preview '([[ -f {} ]] && (bat --style=numbers --color=always {} || cat {})) || ([[ -d {} ]] && (tree -C {} | less)) || echo {} 2> /dev/null | head -200'
--border top
--color 'bg+:233,fg+:255,fg:regular:250,hl+:40,hl:dim:40'
--color 'prompt:80,info:40,header:italic:23,border:dim'
--color 'pointer:9,spinner:92,marker:46'
--margin 2%,1%,2%,1%
--pointer='▶' --marker='✓'
--bind change:first
--bind 'ctrl-a:select-all'
--bind '?:toggle-preview'
--bind 'ctrl-v:execute("$EDITOR" {+})'
--bind 'ctrl-w:preview-up,ctrl-s:preview-down'
--bind 'ctrl-w:preview-half-page-up,ctrl-s:preview-half-page-down'
--bind 'ctrl-y:execute-silent(echo {+} | pbcopy)'"

# man page fzf
   AVAILABLE KEYS: (SYNONYMS)
       ctrl-[a-z]
       ctrl-space
       ctrl-\
       ctrl-]
       ctrl-^      (ctrl-6)
       ctrl-/      (ctrl-_)
       ctrl-alt-[a-z]
       alt-[*]     (Any case-sensitive single character is allowed)
       f[1-12]
       enter       (return ctrl-m)
       space
       bspace      (bs)
       alt-up
       alt-down
       alt-left
       alt-right
       alt-enter
       alt-space
       alt-bspace  (alt-bs)
       tab
       btab        (shift-tab)
       esc
       del
       up
       down
       left
       right
       home
       end
       insert
       pgup        (page-up)
       pgdn        (page-down)
       shift-up
       shift-down
       shift-left
       shift-right
       alt-shift-up
       alt-shift-down
       alt-shift-left
       alt-shift-right
       left-click
       right-click
       double-click
       or any single character

   AVAILABLE ACTIONS:
       A key or an event can be bound to one or more of the following actions.

         ACTION:                   DEFAULT BINDINGS (NOTES):
           abort                      ctrl-c  ctrl-g  ctrl-q  esc
		   [...]
           preview(...)               (see below for the details)
           preview-down               shift-down
           preview-up                 shift-up
           preview-page-down
           preview-page-up
           preview-half-page-down
           preview-half-page-up
           preview-bottom
           preview-top
           previous-history           (ctrl-p on --history)
           print-query                (print query and exit)

2. Config file

...

I would lean towards solution No.1, I try to have as few external configuration files as possible.

@LangLangBart ah nice, yeah just changing fzf settings will work for me


Add updateURL mutation to schema

Add updateURL endpoint to graphql api schema and implement resolver.

Fix bug in url update statement that causes update to fail persistance.

Update wired dependencies.

I will add more edge cases to verify that all behaviors are working as intended and so that codecov checks pass.

@byliuyang I've implemented some of the required changes and will be working on completing the rest. Thank you for the feedback!

Sorry for the long delay. I have finished fixing all the merge conflicts and just need to add the last code review changes. :smile: @byliuyang

Hi @oatovar , thank you so much for working on this feature. Please separate db, use case, and GraphQL API changes into separate PRs!

Each PR should be roughly ~200 lines.

Hi everyone. I have opened #847 as the first of the smaller PRs related to this. I broke down this PR to reduce the mental strain of having to read 800 lines of additions and modifications.

Hi everyone. I have opened #847 as the first of the smaller PRs related to this. I broke down this PR to reduce the mental strain of having to read 800 lines of additions and modifications.

Thank you for splitting it! If there will be no more discussion in this PR you can close it


SQL Server IndexedProperties example

Provides correct example of SQL Server IndexedProperties
The existing example was syntactically wrong and could never have worked. This makes me question whether anyone has ever used this feature before.

@willtsai  @greenie-msft


Modified YAML Test Scripts - Nov 24th

Modified Test Scripts

TC-BI-1.1
TC-BOOL-1.1
TC-CC-1.1
TC-CC-4.2
TC-CC-6.2
TC-DIAGTH-1.1
TC-EMR-1.1
TC-FLW-1.1
TC-ILL-1.1
TC-LVL-1.1
TC-MC-1.1
TC-OCC-1.1
TC-OO-1.1
TC-PCC-1.1
TC-PRS-1.1
TC-RH-1.1
TC-TM-1.1
TC-TSTAT-1.1
TC-TSUIC-1.1
TC-WNCV-1.1

Testing
Executed all YAML test scripts, attached logs for reference
Execution_log.txt

I will disucss with the team and update the test step

Fast tracking, given this is adding tests, but still pends review.

Mereged this YAML scripts in PR #12397


Add set schema to middleware

Fix #828, breaking change: computed columns now only work if they are on the config schema.
Fix #835, tests now not depend on the search_path of the postgrest_test_authenticator.

By the way, does this affect the pleasant side effect where setting db-schema = "" in the config file falls back to the search_path?

from how the code looks, i think it affects it. we should keep that sideffect. when the value of schema is emptty string, there should be no "set schema" statement

one more thing, since these functions live in the api schema, what happens if you call them directly? do they get displayed in the openapi output?

The computed columns get displayed in the openapi output, taking always_true as the example:
{
  "/rpc/always_true": {
      "post": {
          "tags": [
              "(rpc) always_true"
          ],
          "produces": [
              "application/json",
              "application/vnd.pgrst.object+json"
          ],
          "parameters": [
              {
                  "required": true,
                  "schema": {
                      "type": "object"
                  },
                  "in": "body",
                  "name": "args"
              },
              {
                  "$ref": "#/parameters/preferParams"
              }
          ],
          "responses": {
              "200": {
                  "description": "OK"
              }
          }
      }
  }
}

In psql I'm able to call it as a function with both:
select test.always_true(row(1));
select test.always_true(i) from test.items i;

However through PostgREST I'm not able to call them with a GET/POST rpc/always_true since it doesn't take a named argument.
So it seems it's not useful to list the computed columns as rpc in the openapi output, I added a comment on #956 with a proposal for a different format.

The db-schema = "" doesn't work as before since that would also set the search_path TO '' ignoring the authenticator search_path.
If the idea of the db-schema was to add an extra layer of security allowing it to be "" seems wrong to me(btw just tested that and it leaves the openapi output empty). I think that is a bug and it should be restricted to a non-empty value.
If we want to allow the user to use a search_path instead of a fixed schema I propose that we change the db-schema to db-search-path this would take values in the usual format like public, test, for this to work I'll have to remove the prepending of the config schema, that isn't required now anyway so it's doable without breaking previous functionality.

it is intact doable, and yes that is the "right/flexible" way of doing it but here is the downside.
i guarantee that instead thinking about it and creating a clear separation for the public api, ppl will start just dumping in that config all the schemas and accessing the tables directly. sooner or later that is going to lead to security issues and guess who is going to get the blame.
The PostgREST way of building apis is new and i'd rather people (including us) learn to walk before they start running :) and the reality is there is not much db expertise around for people to truly understand the implications of search_path.
I would argue that in the majority of the cases, it's not only enough to have one schema as a public api but in fact that is the better method. At least for now it'd rather have this "hack" remain undocumented and explained to users that really need the "feature".
i would say in this pr, jsut change the function to output an empty string/query when the param is empty to preserve the previous "hack".

Thing is we're not offering good support for users that need more than one schema with that hack, when doing the db-schema = "" and relying on the search_path the openapi output is broken(it doesn't list any tables or rpc's) and it's likely that there'll be more bugs(by the way I just ran the test suite with the config  changed to the empty db-schema with the postgrest_test_authenticator search_path changed to public,test and there are 61 failures).
Not reasonable to fix the bug's bugs for supporting that use case like that, for me it'd be better to give explicit support with db-search-path.
How about supporting both db-schema and db-search-path? We could make them exclude each other for the config to be valid, and the documentation would have the db-schema as the preferred way(the only one included in the tutorials) and db-search-path only mentioned as an advanced config option(with a warning), both would be validated to be nonempty.

it is intact doable, and yes that is the "right/flexible" way of doing it

I think (assuming we're careful as I describe below) we should support the most elegant and flexible solution which is the search path.

i guarantee that instead thinking about it and creating a clear separation for the public api, ppl will start just dumping in that config all the schemas and accessing the tables directly

Yeah...convenience does tempt people to do regrettable things. I think it's a matter of how we present it though, of guiding people to the safest way by default and only revealing the extra functionality as necessary.

How about supporting both db-schema and db-search-path?

Actually what about supporting just db-schema like we do now, but treating comma separated values as a search path if someone knows about that feature? That way it's backward compatible and the docs can stay the same, but it provides the most flexibility for power users?
In another less prominent docs section about versioning we could describe how setting e.g. db-schema="v2,v1" can help reduce view duplication.

i have no strong opinion about this,  if you guys think we should make db-schema to actually be the search-path value then i'm fine with it.
Maybe the only thing needed here is something like on boot, if the value has any commas then say something like
"It seems like you are exposing more then one schema as an api, this is discouraged and we hope you know what you are doing" ... and that would be warning enough.
@begriffs you write the message :)

Maybe a message like, "CAUTION: Multiple schemas detected in db-schema parameter. This might leak unintended information."

Insofar as it adds a set schema call this PR is good to merge. After it's merged I'll open one to try and remove the schema prefixing from names and support the full search_path.

@begriffs I was already working on it, on this branch https://github.com/steve-chavez/postgrest/commits/set-schema-2.

You rock! By all means carry on then. 😄

My bad, I thought this PR couldn't be merged because it was incomplete, but reviewing it again it seems is good to merge as it is, and the search_path is and additional feature that could be done on another PR.
@begriffs What about if I merge this branch and you continue where I left on this commit https://github.com/steve-chavez/postgrest/commit/e61579f170e0a1b7f06b9064734edd6839fbf711?

Since this PR has a breaking change(also the other one regarding memory usage) I think we could release a new version now(v<IP_ADDRESS>) and include the current PR changes in v<IP_ADDRESS>, that way we would be complying to semver.

Sounds good, let's revisit this after the next minor release.
This PR, and additional work for the more generalized search_path behavior can go into the next major version.


Forums and Tags

Feature Request
Add an option for a tag, or let the user choose a category

In Progress.

Have started working on this

Actually, this issue has not been resolved yet. A user is unable to add tags when they ask a question!

Currently being worked on in issue-4 branch


Add support for distros that use Busybox like Alpine

Checks for busybox by looking at the realpath for timeout
If busybox is detected, uses timeout with a -t option and
uses nc instead of /bin/tcp
References issue #5

I'm certainly no hero when it comes to bash scripting, so some of my choices may have more elegant solutions.  Lmk if anything needs help.

+1

Is there something preventing this being accepted? Alpine-Linux is a very common option in Dockerland and I'd love to see this in the main repo.

@ksnyde : Please see issue #5. If someone builds a pull request that doesn't remove the /dev/tcp BASH idiom, I would consider merging it.  This specific pull request breaks the core use case of this tool, which is a BASH compatible script that requires no external dependancies.

Before I can merge this, I need to add Travis testing scripts that will test the script in all possible working environments.  I have zero time to do this, so I'm eager to work with volunteers.

Are there existing tests? Don't see any in the repo. Extending existing tests will be much easier for folks to help out with.

How this can be run in alpine if it is using bash and alpine comes with ash?

@lordofthejars this pr works for me, I want to run a database built on top of cassandra. After I install bash in alpine, the original wait for it does not work, showing timeout: can't execute '15': No such file or directory, however switch to the fork in the PR works.

Works with java:8-alpine plus bash as per this docker file.  Also passes automated tests.

This script only works if there is bash installed in alpine, which is not in the latest alpine version, so it assumes that you need to install bash first?

Yes - it requires bash to be installed.  I see that there's a version that doesn't require bash, but does require netcat over at github.com/raphaelahrens/wait-for-it. This should work with the official Alpine docker image.

@douglas-gibbons  it doesn't work with alpine, since alpine uses busybox which has a slightly differen timeout command.  See https://github.com/raphaelahrens/wait-for-it/issues/1

fyi, https://github.com/mirror/busybox/commit/c9720a761e88e83265b4d75808533cdfbc66075b


Stop button lost

Peraphs it's since Hass upgrade, i have lost Stop and restart button of add-on and the add-on is stuck
When i try to call service restart, start or stop add-on, nothing happens
Home Assistant Supervisor: Stop add-on
There is no error in log
Home Assistant Core 2022.6.2
Home Assistant Supervisor 2022.05.3
Home Assistant OS 8.1

I had this bug two time it disappeared only with  host reboot.

Happy to hear that you have fixed your problem, but this would not have been an issue with the addon. The addon has no control over those interface elements, but sometimes those kinds of things can happen after an update and can be fixed by clearing your browser cache or restarting Home Assistant.

This bug come back offen and force me to reboot host device. I have no error, no log nothing but integration don"t want to sart ,... That why don't have stop and restart buttun,..
I believe i can try to run integration in cmmand line but i don"t found it.


Type checking for Dimension ID

It is not clear from the UI that the Dimension ID has to be an integer.  Add type checking to the UI and do not permit submission where the Dimension ID is not an integer.  Bonus if you make the default the highest unused Dimension ID in the database.
Object { _message: "Dimension validation failed", name: "ValidationError", message: "Dimension validation failed: dimensionID: Cast to Number failed for value \"TT\" (type string) at path \"dimensionID\"", … }

Done


Test: Adds support for bitmap based postings in Index

This PR updates the index file to write and read postings that are written in bitmap formats by using github.com/dgraph-io/sroar library. It also adds a FormatV3 as index file format for the new indexes written in bitmap postings format.
Note: The aim of this PR is to test the performance of the implementation.

/prombench main

/prombench cancel

Discussed in the bug-scrub meeting.
Since this is draft and no progress in 2 years, we will close.  Feel free to reopen if you come back to it.


PyLint claiming zmq.REP and zmq.REQ cannot be found

Discovered on Windows10 using Anaconda Python 3.6.1 with the pyzmq library installed using pip install zmq.
Pylint output:
44,50,error,E1101:Module 'zmq' has no 'REQ' member
60,50,error,E1101:Module 'zmq' has no 'REP' member

Given that this is an inaccurate report from pylint, perhaps ask pylint? I'm not sure if pylint can handle runtime-defined names. Maybe there's something we can do here to help them find what they are after, but the attribute is defined and in zmq.__all__, so I'm not sure what more there is to do.

If a module includes a name in its __all__, then that provides a way for the module to signal to tools like pylint that the name will exist at runtime.
Pylint could continue to warn the project defining __all__ that it's including a variable that doesn't exist, and that project can include a comment to disable pylint E0603 on that line if it wants. But the calling project is doing nothing wrong, and the information is available in the imported module's __all__ to confirm that it is an intended variable name of the project (which, if it doesn't exist, is that project's fault, not the importer's).

@chrisjbillington well, however, it is quite hard to decide on what runtime variables exist since zmq in different versions have different enums.

@adamcavendish right, and I see that actually zmq.REP is not in __all__ as it is defined in source. it too must be getting added to __all__ at runtime, so inspecting source __all__ would still not reveal it,

same problem with zmq.PUB, zmq.LINGER, zmq.SUB, zmq.SUBSCRIBE see also https://github.com/PyCQA/pylint/issues/478

Closing as there doesn't seem to be anything to do. Static analysis isn't going to find dynamically defined names.

Here's a quick workaround cheatsheet for others hitting this. In .pylintrc, add:
[TYPECHECK]
# zmq.{LINGER,REQ,ROUTER,NOBLOCK} are dynamically generated and so pylint
# doesn't see them, causing false positives.
generated-members=LINGER,REQ,ROUTER,NOBLOCK

(I didn't need REP but you can add other attributes following the same pattern).


[HOLD for payment 2023-08-10] [HOLD for payment 2023-08-08] Pressing Enter key on Split bill page doesn't navigate to next step

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Click on 'FAB' menu
Click on 'Split bill' button
Enter amount & press 'Enter' key

Expected Result:
User should be navigated to the next step
Actual Result:
User is not navigated to the next step
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.48-0
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/05a0531a-66e2-407a-b832-fdc084a64671
https://github.com/Expensify/App/assets/43996225/0ddafca2-2b55-4658-b8f4-f18b2c8e37cf
Expensify/Expensify Issue URL:
Issue reported by: @natnael-guchima
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690829632233769
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

@marcaaron did some debugging, i think the root cause of this issue is from onyx, the update of the amount is delayed and initially it's 0 so it will navigate us back from MoneyRequestParticipantsPage here:
https://github.com/Expensify/App/blob/1fa4b723d8ef649afa56780b52ef453a39bbc9fc/src/pages/iou/steps/MoneyRequstParticipantsPage/MoneyRequestParticipantsPage.js#L77-L79
and there is onyx version upgrade 5 days ago deployed to staging NoQA] Perf/onyx upgrade use cache directly #23749

@hannojg This seems to persist after react-native-onyx change https://github.com/Expensify/App/pull/23749 https://github.com/Expensify/react-native-onyx/pull/280 , cache logic(cahcedState) implementation might be the cause for this. With the earlier version 1.0.52 issue is not reproducible.

 If so, not sure if I understood you properly or what the "coincidence" you are referring to is. Thanks!

@getusha Is that sarcasm? If so, not sure if I understood you properly or what the "coincidence" you are referring to is. Thanks!

@marcaaron  It's just that @Pujan92 and i commented almost the exact same time.

I am currently running into an issue where I can't open the split bill page due to this crash. is it just me or everyone?

Try creating a new fresh account in a new browser instance. I think the crash is reproducible

Ya that looks like this issue: https://github.com/Expensify/App/issues/23951

Okay, however with these changes, there is still the issue that when pressing the Enter-Key,

You will open a second instance of the same page (split bill)
On the second press you get to the next page

There was a separate issue for that, not sure if that's also due to the onyx changes will double check!

https://github.com/Expensify/App/issues/23962

Fixed also with this PR:

https://github.com/Expensify/App/pull/23996

Looks like this was fixed by https://github.com/Expensify/App/pull/24003! Not a deploy blocker anymore

Hello @marcaaron, no reporting bonus is issued here.

A gentle bump for reporting payment @isabelastisser

@Natnael-Guchima I sent you an offer in Upwork now. Please accept it and I will process the payment. Thanks!

I have accepted the offer. Thanks @isabelastisser


Unable to specify HYBRID_SHARD for FSDP which requires process group or device_mesh to be passed

System Info
- `Accelerate` version: 0.29.1
- Platform: Linux-5.19.0-46-generic-x86_64-with-glibc2.35
- `accelerate` bash location: /home/yuchao/miniconda3/envs/TorchTTS/bin/accelerate
- Python version: 3.10.13
- Numpy version: 1.23.5
- PyTorch version (GPU?): 2.2.2+cu118 (True)
- PyTorch XPU available: False
- PyTorch NPU available: False
- PyTorch MLU available: False
- System RAM: 125.48 GB
- GPU type: NVIDIA GeForce RTX 4090
- `Accelerate` default config:
fsdp_plugin:
  sharding_strategy: 4

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[X] My own task or dataset (give details below)

Reproduction
Specify fsdp strategy to ShardingStrategy.HYBRID_SHARD or _HYBRID_SHARD_ZERO2
File "/home/yuchao/miniconda3/envs/TorchTTS/lib/python3.10/site-packages/accelerate/accelerator.py", line 1434, in prepare_model
    return self.prepare_model(obj, device_placement=device_placement)
  File "/home/yuchao/miniconda3/envs/TorchTTS/lib/python3.10/site-packages/accelerate/accelerator.py", line 1434, in prepare_model
    model = FSDP(model, **kwargs)
  File "/home/yuchao/miniconda3/envs/TorchTTS/lib/python3.10/site-packages/torch/distributed/fsdp/fully_sharded_data_parallel.py", line 448, in __init__
    model = FSDP(model, **kwargs)
  File "/home/yuchao/miniconda3/envs/TorchTTS/lib/python3.10/site-packages/torch/distributed/fsdp/fully_sharded_data_parallel.py", line 448, in __init__
    _init_process_group_state(
  File "/home/yuchao/miniconda3/envs/TorchTTS/lib/python3.10/site-packages/torch/distributed/fsdp/_init_utils.py", line 118, in _init_process_group_state
    _init_process_group_state(
  File "/home/yuchao/miniconda3/envs/TorchTTS/lib/python3.10/site-packages/torch/distributed/fsdp/_init_utils.py", line 118, in _init_process_group_state
    raise ValueError(
ValueError: ('Manual wrapping with ShardingStrategy.HYBRID_SHARD', 'requires explicit specification of process group or device_mesh.')
    raise ValueError(
ValueError: ('Manual wrapping with ShardingStrategy.HYBRID_SHARD', 'requires explicit specification of process group or device_mesh.')

Expected behavior
Should provide a way to provide the process group or device_mesh as they're parameters for FSDP https://pytorch.org/docs/2.2/fsdp.html#torch.distributed.fsdp.FullyShardedDataParallel

auto_wrap_policy may be another way. But I find no way to setup process_group and device_mesh which is in the interface of FSDP
torch.distributed.fsdp.FullyShardedDataParallel(module, process_group=None, sharding_strategy=None, cpu_offload=None, auto_wrap_policy=None, backward_prefetch=BackwardPrefetch.BACKWARD_PRE, mixed_precision=None, ignored_modules=None, param_init_fn=None, device_id=None, sync_module_states=False, forward_prefetch=False, limit_all_gathers=True, use_orig_params=False, ignored_states=None, device_mesh=None)

True, we need to enable this in the FullyShardedDataParallelPlugin, and make it work nicely when things are set via accelerate launch.

Sounds like one implication here is that transformer Trainer's --fsdp hybrid_shard / --fsdp hybrid_shard_zero2  can't possibly run successfully, since process_group  is not amoung trainer arg

any updates on this?

any updates?


Clair conf change

1, update the interval of clair updater to 12 hours
2, update the interval for scan all to 2 hours

Coverage increased (+0.02%) to 54.703% when pulling c1bbcb5babff081828bae346f1863e8c0a14e4d4 on reasonerjt:clair-conf-change into 8a1b10ad7e81e8ff738e9857a7e72c2db22038f4 on vmware:master.

Coverage increased (+0.02%) to 54.703% when pulling c1bbcb5babff081828bae346f1863e8c0a14e4d4 on reasonerjt:clair-conf-change into 8a1b10ad7e81e8ff738e9857a7e72c2db22038f4 on vmware:master.


Use VersionType::from_string for macos

Codecov Report

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

@@           Coverage Diff           @@
##           master     #202   +/-   ##
=======================================
  Coverage   84.22%   84.22%
=======================================
  Files          12       12
  Lines         488      488
=======================================
  Hits          411      411
  Misses         77       77

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 c6bbae2...b082ed5. Read the comment docs.


Add format to support MDX and plain markdown

Add format to support MDX and plain markdown

Default to detecting whether input is mdx or md based on whether the given
file has an extension in mdExtensions and to handle both mdExtensions and
mdxExtensions in integrations
Support format: 'md' to treat input as markdown and to handle mdExtensions
in integrations
Support format: 'mdx' to treat input as MDX and to handle mdxExtensions
in integrations
Switch which elements will turn into configurable components, previously that
was everything that was written in the source markdown, now this is everything
not explicitly written as JSX
Add tests for integration with rehype-raw
Fix docs on evaluate options

Closes GH-17.

@ChristianMurphy Do you see the default being detect as a good idea? I think so

/cc @remcohaszing @ggoodman WDYT?

Do you see the default being detect as a good idea?

I think so. :+1:
There is an edge case, that being if people use non-standard md and mdx extensions and still want the type to be inferred there isn't a way to override at the moment.
But until that use case pops up, YAGNI

There is an edge case, that being if people use non-standard md and mdx extensions and still want the type to be inferred there isn't a way to override at the moment.

The way I’ve set it up now is that there’s two lists of extensions: one for markdown, one for mdx. Folks can overwrite those to set their own funky preferences.
It’s indeed a weird case, but with {mdExtensions: ['.mdx', ...], mdxExtensions: [...]} one could load .mdx as markdown, or with {mdExtensions: [...], mdxExtensions: ['.md', ...]} one could set .md as MDX. Again, I think it’s really weird, but it’s possible.

I mostly looked at the docs, because although I’ve looked at the internals, I’m not very familiar with them.
Anyway, looks good to me :+1: