make tunnel ingress rule as a standalone resource

Current Terraform and Cloudflare provider version
terraform 1.6.5
cloudflare 4.20.0
Description
make tunnel ingress rule as a standalone resource, so we can reuse tunnel in different project
Use cases
I am using a tunnel to expose services in the internal k8s cluster to the public network. We use the same k8s cluster across different projects, each with its own Terraform repo.
However, the cloudflare_tunnel_config resource is unique, which means I can only set up a single tunnel for each project. This results in a large number of tunnels and pods. If I could set up one tunnel and some public cloudflared when deploying k8s, then add ingress rules as needed each time, things would be more efficient and straightforward, just like clicking 'Add a public hostname' on the web console whenever I need it.
Potential Terraform configuration
resource "cloudflare_tunnel" "cluster_tunnel" {
  // ...
}

resource "cloudflare_tunnel_config" "cluster_tunnel" {
  account_id = "..."
  tunnel_id  = "..."

  config {
    origin_request {
      connect_timeout = "30s"
    }
    ingress_rule {
      service = "http://srv_404.xxxxxxxxx.svc.cluster.local"
    }
  }
}

resource "cloudflare_tunnel_ingress_rule" "my_srv" {
  account_id = "..."
  tunnel_id  = "..."

  hostname = "my_srv.example.com"
  service  = "http://my_srv.xxxxxxxxx.svc.cluster.local"
}

References
No response

the rules here are a part of the config so there isn't a way to manage them individually. I'll pass it onto the service team but closing for now as there isn't anything we can action in the provider for this.


Group consumer "stop-the-world" when one consumer exceed max.poll.interval.ms

In what version(s) of Spring for Apache Kafka are you seeing this issue?
For example:
Spring kafka 2.5.3.RELEASE
Describe the bug
My application have two ConcurrentKafkaListenerContainerFactory separate by topic. Let's say ConcurrentKafkaListenerContainerFactory create consumer A for subscribe topic A and another ConcurrentKafkaListenerContainerFactory create consumer B for subscribe topic B. 2 instances application deploy on K8S.  When consumer B exceed max.poll.interval.ms
What i received:
Member consumer-B-2-05ba4dd6-3a98-4ea8-91eb-003ee4fc7aca sending LeaveGroup request to coordinator <brokers-ip> (id:<PHONE_NUMBER> rack: null) due to consumer poll timeout has expired. This means the time between subsequent calls to poll() was longer than the configured max.poll.interval.ms, which typically implies that the poll loop is spending too much time processing messages. You can address this either by increasing max.poll.interval.ms or by reducing the maximum size of batches returned in poll() with max.poll.records.

I've expected the groupId B would be rebalance and had been assigned partitions again, but nothing happen. Message lag of group B still increased, two instance have stopped consume message . Only group B got stuck, group A worked fine. When I restarted my application and groupId B began consume message. I wonder why this issue happen?
I also check status isRunning of the container. Whenever container return isRunning = false I will restart my application (health check kubernetes). But my application still alive. So I think KafkaContainer bean had been removed by spring application context. Is it possible ? Or I still missing somewhere?
To Reproduce
Very very rarely this issue happen. I still cannot reproduce this issue.
Expected behavior
After LeaveGroup. Kafka consumer will gonna rejoin group and begin consume message.
Sample
I don't have sample code.

My configuration kafka consumer.
public final class KafkaConsumerConfigInit {
    KafkaConsumerConfigInit() {}

    public static <T> ConcurrentKafkaListenerContainerFactory<String, T> getCommonKafkaListenerContainerFactory(TopicConfig topicConfig, Class<T> clazz) {
        ConcurrentKafkaListenerContainerFactory<String, T> factory = new ConcurrentKafkaListenerContainerFactory<>();
        if (topicConfig.getMaxPollRecord() == null || topicConfig.getMaxPollRecord() <= 0) {
            topicConfig.setMaxPollRecord(MAX_POLL_RECORDS_CONFIG);
        }
        if (topicConfig.getMaxPollInterval() == null || topicConfig.getMaxPollInterval() <= 0) {
            topicConfig.setMaxPollInterval(MAX_POLL_INTERVAL_MS_CONFIG);
        }

        if (topicConfig.getShutDownTimeOutInMillie() == null || topicConfig.getShutDownTimeOutInMillie() <= 0) {
            topicConfig.setShutDownTimeOutInMillie(DEFAULT_SHUTDOWN_TIMEOUT);
        }

        if (topicConfig.getNoPollThreshold() == null || topicConfig.getNoPollThreshold() <= 0) {
            topicConfig.setNoPollThreshold(THRESHOLD_NO_RESPONSE);
        }

        String groupId = StringUtils.isEmpty(topicConfig.getGroupId()) ? topicConfig.getTopic() : topicConfig.getGroupId();
        ConsumerFactory<String, T> consumerFactory = getConsumerFactory(topicConfig.getUrl(), groupId,
                topicConfig.getMaxPollRecord(), topicConfig.getMaxPollInterval(), clazz);
        factory.setConsumerFactory(consumerFactory);
        factory.getContainerProperties().setAckMode(ContainerProperties.AckMode.MANUAL);
        factory.getContainerProperties().setShutdownTimeout(topicConfig.getShutDownTimeOutInMillie());
        factory.getContainerProperties().setPollTimeout(topicConfig.getMaxPollInterval());
        factory.getContainerProperties().setNoPollThreshold(topicConfig.getNoPollThreshold());
        factory.getContainerProperties().setMonitorInterval(topicConfig.getMaxPollInterval());
        factory.getContainerProperties().setSyncCommits(topicConfig.isSyncCommit());
        factory.setConcurrency(topicConfig.getConcurrency());

        return factory;
    }
}

Which is
MAX_POLL_RECORDS_CONFIG  = 100,
MAX_POLL_INTERVAL_MS_CONFIG = 120_000,
DEFAULT_SHUTDOWN_TIMEOUT = 300_000,
THRESHOLD_NO_RESPONSE = 3

When I init bean
    @Bean
    public ConcurrentKafkaListenerContainerFactory<String, EventReportMsg> eventReportContainerFactory() {
        return KafkaConsumerConfigInit.getKafkaListenerContainerFactory(eventReportKafka.getTopicConfig(KafkaConst.EVENT_REPORT_TOPIC), EventReportMsg.class); // this is method above.
    }

@KafkaListener
        @KafkaListener(topics = "#{'${event_report}'.split(',')}", containerFactory = "eventReportContainerFactory")
        public void listener(EventReportMsg msg, Acknowledgment acknowledgment) throws DBClusterException {
            log.info("[SENDER] received msg from topic={} with data={}", KafkaConst.EVENT_REPORT_TOPIC, msg);
            this.doConsume(msg);
            acknowledgment.acknowledge();
        }

That doesn't help; you need to provide logs when the problem occurs; also take a thread dump to see if the listener thread is "stuck". someplace.

Member consumer-B-2-05ba4dd6-3a98-4ea8-91eb-003ee4fc7aca sending LeaveGroup request to coordinator <brokers-ip> (id:<PHONE_NUMBER> rack: null) due to consumer poll timeout has expired. This means the time between subsequent calls to poll() was longer than the configured max.poll.interval.ms, which typically implies that the poll loop is spending too much time processing messages. You can address this either by increasing max.poll.interval.ms or by reducing the maximum size of batches returned in poll() with max.poll.records.
this log above is that all I have. No revoked partitions log, JoinGroup log, Attempt HeartBeat fail rebalance log, stuff like that. I wish I could have more log.

Problem in user code. User code call make http request without timeout. Feel free for closing this bug.

Yes, the poll thread actually stuck in user code when make http connection without timeout. Thanks for your support.


handles unreachable network exception

resolves #1019

Hi @sas1ni69
You'll need to add the NetworkError to this file: https://github.com/alexreisner/geocoder/blob/master/lib/geocoder/exceptions.rb
Could you add a unit test?
Thank you for the pull request :)

Hey @TrangPham thanks for the feedback, I'll update the PR :)

@TrangPham updated. Please let me know if you'd like me to add or change anything. Really appreciate you taking the time to look into this.

Sorry I've taken a while to get back to you!
:D It looks good

No worries I know you're busy. Thanks for merging it! 💃


Update copyq to 3.0.2

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Hmm, the commit message not correct -- I used cask-repair -e ... -- it should be something like Add shim script for copyq.

-e does not change the commit message, it only opens the cask for editing before checkign everything else. You can’t change the commit message with cask-repair (and I don’t intend to add the feature).

Why is this change necessary? I don’t see a difference.

Symlinks doesn't work with Qt apps - when the symlink is executed the target app doesn't search the correct directories with libraries and in this specific case fails to load Cocoa Qt plugin.

BTW, I fixed the commit message.

Are you sure the exec is necessary? Also, use /bin/bash, that’s what sh is on macOS, anyway.

I can confirm that this fixes the error we were seeing before (e.g. https://github.com/hluk/CopyQ/issues/719#issuecomment-306408808).
Thanks @hluk

OK, I replaced /bin/sh with /bin/bash.
The exec is not needed but it replaces the bash process and probably saves a bit of memory.


Missing ASL compiler download

It would be nice if you could include a link whereby the MS ASL compiler can be downloaded.

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

ID: 146c2e57-a40b-33e0-c108-37f7555eda8a
Version Independent ID: 81b0b392-c18a-f083-17fa-7e6dca629a52
Content: Microsoft ASL compiler - Windows drivers
Content Source: windows-driver-docs-pr/bringup/microsoft-asl-compiler.md
Product: windows-hardware
GitHub Login: @barrygolden
Microsoft Alias: bagold

The parent to this page states:
Version 5.0 of the Microsoft ACPI source language (ASL) compiler supports the features in the Advanced Configuration and Power Interface Specification, Revision 5.0 (ACPI 5.0 specification). The ASL compiler is distributed with the Windows Driver Kit (WDK) 8.1.

I agree, it would be nice to have a secondary statement on this page that it can be downloaded as part of that kit.  Is it also part of WDK 10?

No joke man. Without a link and trying to troubleshoot my dear and brand new eGPU, I have little patience for reading! Thanks to kyanha for answering. Now I know.

Thanks for reaching out to the Windows Driver Developer Documentation team with your comments about a download link for the ASL complier.
This topic has been updated with information on downloading the Windows Driver Kit (WDK) that includes the ASL compiler.

I have WDK 10.22000.832 fully installed on Windows 11. The path "C:\Program Files (x86)\Windows Kits\10\Tools" does not exist. Please update the documentation if the ACPI tools were discontinued.
You have no idea how frustrating it is when all the internet sources say to download WDK and use asl.exe, only to find that the directory doesn't even exist... on multiple WDK versions.

Hi @johnothwolo sorry for the trouble.  I just searched my recent kit installation and I see asl.exe in C:\Program Files (x86)\Windows Kits\10\Tools\10.0.25330.0\arm\ACPIVerify.  Is that what you're looking for?

Yeah, so this is my Windows Kits directory:

And these are the kits I have installed:

Idk why, but I don't have a Tools directory.

@johnothwolo, I think you have the SDK installed correctly as shown above.
You have not yet installed the WDK (Windows Driver Kit) which will install the asl.exe tool you need.
The SDK and the WDK are two separate kits. You do need to (1) install the SDK as a precursor to the WDK. Then you need to (2) install the WDK.
The WDK can be downloaded here:
https://learn.microsoft.com/en-us/windows-hardware/drivers/download-the-wdk#download-icon-step-3-install-windows-11-version-22h2-wdk

@johnothwolo, when the WDK is installed, you should see something similar to the following for your SDK and WDK install points:

I'm showing Insider Preview versions above, but you should have a similar SDK and WDK version that match (ignore the fonts entry as not relevant here).

Updated the location of asl.exe on the doc page.  This update should reach the live site momentarily.
Thanks again for the feedback!

@johnothwolo, when the WDK is installed, you should see something similar to the following for your SDK and WDK install points:

I'm showing Insider Preview versions above, but you should have a similar SDK and WDK version that match (ignore the fonts entry as not relevant here).

Sorry for not updating you, it was kind of embarrassing when I realized my mistake. I downloaded and installed the right package; I confused the WDK with the SDK.

No worries John, thanks for letting us know.  Glad you got it working.


Enhance node-density heavy to use livenessprobes

Using livenessprobes in addition to readinessprobes increases the load
on the kubelet which is what we want with this test.
Description
Fixes

/rerun all

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_kubeburner
Fail
3
01:28:32

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_kubeburner
Fail
3
01:27:09

/rerun all

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_kubeburner
Fail
3
01:26:25

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_kubeburner
Fail
3
01:29:56

/rerun all

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_kubeburner
Pass
0
00:20:38

@smalleni fixed!, we had a duplicated imagePullPolicy param. And no, I don't have a trigger for perfApp atm (I pushed the image manually yesterday)

@smalleni fixed!, we had a duplicated imagePullPolicy param. And no, I don't have a trigger for perfApp atm (I pushed the image manually yesterday)

AHA, yes my bad, the default is Always anyways, so yes the the other imagePullPolicy was the problem. Thanks for taking care of it, hopefully we stress the kubelet more now :)


New version: Spicetify.Spicetify version 2.35.1

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @afonsojramos,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 870.)

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-03-21 09:07:31.820 [FAIL] Installer failed security check. Url: https://github.com/spicetify/spicetify-cli/releases/download/v2.35.1/spicetify-2.35.1-windows-x64.zip Result: 0x80004005

(Automated response - build 870.)

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-03-21 09:07:31.820 [FAIL] Installer failed security check. Url<EMAIL_ADDRESS>spicetify-2.35.1-windows-x64.zip (download) Result: 0x80004005

(Automated response - build 870.)

@stephengillie hi, most likely it was false positive. Is it possible to run the validation again? (the same goes to v2.35.0)

Yes - ideally my automation would retry every 18 hours, including over the weekend. Working through a bug around that.


vm-builder: Turn off --auto-discover-databases

It causes exporter connection to every database in the Postgres and it was disabled for pods a while ago, but I forgot that VMs have a different build path.
See this for details:
https://github.com/neondatabase/cloud/commit/0a60057f3380e570b540cc823975e07400cd640c
Also remove --exclude-databases it seems to be bogus as well, see:
https://github.com/prometheus-community/postgres_exporter/issues/588

Looks like downscale is failing because the vm-monitor is hitting internal errors. Unfortunately we don't have logs for the VMs themselves, but it's unlikely that it's failing because of this PR — each vm-monitor is pulled from neondatabase/neon's current main, so maybe some recent change or something.
I'll dig into it further.

OK, I'd avoid stacking PR on top of failing tests and it's not urgent, so let me know when it makes sense to re-run / safe to merge

Filed #577 to track the long term solution for moving postgres_exporter configuration into the control plane (among other things).
There's more immediate changes that could be made to make that simpler, if we anticipate needing to do this frequently or with urgency.


Fixed: Added auth user project-member check

refactor helper function so we will have one source of truth while getting project-member.
add the check if authuser is project member or not.

Great Job @naman-dave! Merging this.


make sqlite volume configurable

Signed-off-by: André Bauer<EMAIL_ADDRESS>
make sqlite volume configurable

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.892%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4058

Relevant Lines:
4322

💛  - Coveralls


WIP: Rails 7.0.0 alpha

Just testing this to see what needs to be fixed. Pretty much just required updating devise to fix an issue with it plus some updates sorbet-typed. Otherwise, nothing major. Probably the easiest major Rails upgrade I've ever done.
There is a weird problem with the ActiveStorage signatures caused by delegated methods generating weird signatures (presumably with ..., which Sorbet/Tapioca struggles with?), but I don't think that's worth being too concerned about.

Closing in favor of #2419


ARXIVNG-1296 - Home page

This PR is for a simple reimplementation of the home page:

uses DB to get the the document count by default; falls back to static stats file if DB unavailable. (Legacy defaulted to static stats file and never used DB)
does not display login status (this will be a separate ticket)

Pull Request Test Coverage Report for Build 478

96 of 117 (82.05%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 83.153%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

browse/routes/ui.py
28
29
96.55%

tests/test_browse.py
27
30
90.0%

browse/controllers/home_page/init.py
26
43
60.47%

Totals

Change from base Build 465:
-0.2%

Covered Lines:
3233

Relevant Lines:
3888

💛  - Coveralls


Reducing the embedding size

There's a way to reduce the embedding size? 300d can be very large.

There are some way to reduce the size, e.g. PCA, SVD.
300d is a common setting in open source embeddings, e.g. Word2vec on Google News, GloVe.


Clarity

What does "After the backup finishes successfully using the new location, you can remove the original cache folder." mean?
That we have to run one backup after we change the location? or just that after the service starts it's fine to use?

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

ID: 02ad5c91-acc2-a50b-0b4a-8298c608df41
Version Independent ID: 2e80d5ac-1918-61f8-87d4-8133a0250b1b
Content: Common questions when backing up files and folders with Azure Backup
Content Source: articles/backup/backup-azure-file-folder-backup-faq.md
Service: backup
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thanks for your comment. The feedback has been shared with the content owner for further review. Thanks for your patience.

@saurabhsensharma - Do you know the answer to this question?

Yes, it is a best practice to trigger an ad-hoc backup after changing the cache location. We will update the documentation to make that more clear.

#please-close


.maintain/sentry-node: Add monitoring to docker-compose stack

This pull request adds a basic templated Substrate Grafana dashboard for everyone running a node based on Substrate to use for monitoring. Despite a couple of additions this is based on https://github.com/paritytech/substrate/pull/5310 and thus credit goes to @derfredy.
In addition this pull requests extends the .maintain/sentry-node setup to monitor each node.
To run a fully monitored 3 node network:

Build Substrate cargo build --release

Start 3 Substrate nodes, Grafana, Prometheus and Polkadot UI sudo docker-compose -f .maintain/sentry-node/docker-compose.yml up

Go to localhost:3001 for the Grafana UI.

Go to localhost:9090 for the Prometheus UI.

Outlook
In the future I would like to split this dashboard into single-purpose dashboards (e.g. network, block-production, finality, ...) and add alerting and recording rules. But for now, I think this is a first step.

@ddorgan would you mind taking another look? Any other suggestions by others?

Unfortunately it's hard, just by reading a diff, to figure out what is being tracked and how it looks.
I've been putting some efforts in the past couple of days designing a networking dashboard directly on Grafana, and I think it should eventually converge with what's in this PR.

Unfortunately it's hard, just by reading a diff, to figure out what is being tracked and how it looks.

Yes. I guess one could as well treat it as a binary blob.

I've been putting some efforts in the past couple of days designing a networking dashboard directly on Grafana, and I think it should eventually converge with what's in this PR.

Great! I would be in favor of replacing the network-related graphs of this dashboard with your dedicated network dashboard.

If this is good enough for you @tomaka as a first iteration, would you mind approving the pull request?

Unfortunately it's hard, just by reading a diff, to figure out what is being tracked and how it looks.

I hope the docker-compose file in this pull request turns out to be helpful as one can introspect a dashboard by running docker-compuse up -f ./maintain/[...] and then go to localhost:3001.


Fix boost v1.57 build.  The v1.55 patches no longer are needed

Small fix to the builder for boost v1.57.  The patches have been left, encase we have to revert to v1.55, however the boost.mk package build process no longer needs to have them.  For sure on 1 of them, checked the library code myself, can't remember checking the 2nd one, but removing it too, removed the problem I was having with completing the toolchain build.
Also small 1st test, to push into our repo from one of my own commits & pull request.

...done


ClientError vs ServerError

Describe the feature or improvement you're requesting
Generally, it's a good idea to separate error responsibilities between client and server, just as HTTP status code represents with 400-499 (client error) vs 500-599 (server error).
This library has many error types, but they are all subclass of OpenAIError which makes it hard to handle exceptions at a higher level.
For instance, we'd swallow ServiceUnavailableError on the UI and let the user retry so that devs won't be bothered with the error notification on this. But we'd want to get notified upon InvalidRequestError because it's a bug in our codebase.
If we had ServerError and ClientError, we'd write some code like this:
try:
    response = openai.ChatCompletion.create(**params)
except openai.error.ClientError as e:
    # This is our bug, notify and fix it!
    raise e
except openai.error.ServerError:
    # OpenAI seems to be busy
    # Do not raise error and prompt the user to retry later

I also see tenacity.RetryError for embeddings from time to time, so it would be great if it's handled in the same semantics.
Additional context
The semantics here is:

If we face a client error (403, 404, etc.), we want to raise the exception and report the error (often via tools like Sentry) so that devs can fix it. Do NOT retry with the same set of parameters because it will fail until you fix the bug.
If we face a server error (502, 503, etc.), the issue is usually temporary and retry would fix by itself.

Group those errors into one of the two parent classes, ServerError and ClientError.
class OpenAIError(Exception):
class APIError(OpenAIError):
class TryAgain(OpenAIError):
class Timeout(OpenAIError):
class APIConnectionError(OpenAIError):
class InvalidRequestError(OpenAIError):
class AuthenticationError(OpenAIError):
class PermissionError(OpenAIError):
class RateLimitError(OpenAIError):
class ServiceUnavailableError(OpenAIError):
class InvalidAPIType(OpenAIError):
class SignatureVerificationError(OpenAIError):

I feel this is necessary for handling the exceptions better. It makes life so much easy when the parent aggregator is available to bucket the exceptions
Tested the implementation and here are the results. Please confirm if this aligns with the expectation to be addressed as part of this issue @kenn
def test_openai_api():
    # Set your OpenAI API key
    openai.api_key = api_key  # Replace with your actual API key

    try:
        # Perform an API request that may result in a client error
        openai.Completion.create(
            model="text-davinci-003",
            prompt="Say this is a test",
            max_tokens=-1,
            temperature=0
        )
    except ClientError as e:
        # Handle client errors
        if isinstance(e, InvalidRequestError):
            print("Invalid Request Error:", e)
            # Additional handling logic for invalid request errors
        elif isinstance(e, AuthenticationError):
            print("Authentication Error:", e)
            # Additional handling logic for authentication errors
        else:
            print("Client Error:", e)
            # Default handling logic for other client errors
    except ServerError as e:
        # Handle server errors
        print("Server Error:", e)
        # Additional error handling logic can be added here

test_openai_api()

Results:
Client Error: -1 is less than the minimum of 0 - 'max_tokens'

Great news, we have more granular error classes in the beta of our upcoming v1.0.0! Please try it out and let us know what you think.


Hide /opt

I think it could be better to hide /opt to the finder. It's more compliant with OS design.
You can add chflags hidden /opt to install.
What do you think about that?

It's more compliant with OS design.

No, it is not. /opt is part of the system and has a clear purpose, it’s not created by us.
Hiding things without user knowledge or permission is precisely the type of things we won’t do.

Ok thanks, I don't know why but this folder wasn't present on my system before the caskroom installation and It has been created visible.


load extensions using the C API instead of SQL

This fixes the problem of loading ICU, which needs to replace existing
functions. Replacing existing functions are prohibited when a VM
is running.
I found this problem on Windows x64, debugged by extending icu.c such that it returns a text message.
Another benefit of this change is that the SQL load_extension() function always returns NULL, but
now it is possible to examine the actual return value.

Coverage increased (+0.23%) to 84.01% when pulling 0486deea0daf3e1ba737b158c6f7ee7f2a74b7cb on tajtiattila:master into a141177ca6dad6563ee78bb15495cea8679730fd on mattn:master.

Note that the use of C.sqlite3_load_extension() might not be absolutely necessary: I haven't fully checked, but the SELECT load_extension... might still work without using a prepared statement
(i.e. use conn.Exec directly). Let me know if this should be checked.
I've verified that the calls to C.sqlite3_enable_load_extension() are still necessary. This observation corresponds to the sqlite documentation.

Thank you


I am still awaiting a invite code ? anyone else ?   been 3 weeks or so now.

<EMAIL_ADDRESS>

Looks like you were let in now! I'll go close this ticket

You guys are superb ! :-))


fix github.com issues hyperlink

This fixes #19735, which was automatically closed as not following the issue template.

The gh-pages branch contains the generated website files. You want to edit the source at https://github.com/facebook/react-native-website instead. Thanks for looking into this, by the way.

just signed the CLA


Alerts UI burndown list

Future work

When creating an alert, it's checked immediately. Is that intended? (Discussion here)
Include measure filters when providing the alert input data. To ensure correct data, a refactor is needed: move measure filter resolution from frontend to backend.
Alert on contextual columns, like % change.
Combine the Data and Criteria tabs?
Add a global entrypoint for alerts?
"Are you sure you want to cancel?" dialog (https://github.com/rilldata/rill/pull/4140)
Measure multi-select
Alert detail page: make the Snooze configurable with a simple dropdown

Design polish

Preview modal doesn't format percentages correctly (Slack report)
Sort the Data / Alert preview by the measure in descending order
Data Preview width should always fill available space
Options in the Select element should extend the full width of the element
Our button component doesn't appear to exactly center text vertically
In Alert Detail page's metadata section, we should capitalize the first letter of the "Split by dimension" dimension name

Completed

[x] Switching to the Criteria tab triggers a 400 error from the Aggregation API
[x] Split by dimension/time should include a "None" option (#4187)
[x] "Edit alert" button and dialog (https://github.com/rilldata/rill/pull/4139)
[x] Change alert status backend-to-frontend mapping to match spec (Slack convo) (https://github.com/rilldata/rill/pull/4195)
[x] Add explanation for why an Alert check "Failed" (https://github.com/rilldata/rill/pull/4195)
[x] The below criteria should read "New vaccinations > 200" (#4187)

[x] Clicking on an email's "Open in browser" link leads to this bad URL: http://localhost:3000/rilldata/rill-test-covid/-/alerts/MetricsViewAggregation (#4161)
[x] Clicking a new alert led to a 404 page (Issue not specific to alerts, #4206)
[x] Creating with empty filters fails the backend query. (#4205)
[x] Split by grain is not filled in while editing an alert. (#4205)
[x] Show form validation errors everywhere. They should be triggered by attempting to advance to the next page (button shouldn't be disabled). (#4210)
[x] In the Criteria tab, when the form isn't complete, we need proper copy for the empty Alert Preview section (https://github.com/rilldata/rill/pull/4217)
[x] Fix the time range provided to the query in the Create Alert API (#4240)
[x] Include a time range indication in the filters section (#4240)
[x] Move the "Split by time grain" section to the Delivery tab and call it "Evaluation interval". Discussion here. (#4240)
[x] Exclude dimension filters are displayed as include filters. (#4240)

Thanks Eric, per your comment around proper formatting of percentages in the preview modal, this is what I currently see... also of interest, I'd ideally like to have it mirror the sort order of the expanded dimension table.
A second issue that is evident here is that the bottom of the modal is not visible at my screen size resolution, so I can't see the 'Next' and 'Cancel' calls to action unless I scroll down.


Add uc browser

Added support for recognizing UC Browser.
https://en.wikipedia.org/wiki/UC_Browser
I used their user agent documentation here: http://www.ucweb.com/download/UCBrowser_User_Agent_en.pdf
The UC Browser user agent often includes 'Safari', 'Mobile Safari', so the library currently recognizes UC Browser users as such.
For example:
Mozilla/5.0 (Linux; U; Android 2.3; en-US; MI-ONEPlus) AppleWebKit/534.13 (KHTML, like
Gecko) UCBrowser/<IP_ADDRESS> U3/0.8.0 Mobile Safari/534.13

Thanks for the update. It looks like some user agent strings identified out in the wild as UC Browser don't contain "UCBrowser", for instance UCWEB/8.8 (iPhone; CPU OS_6; en-US)AppleWebKit/534.1 U3/3.0.0 Mobile from http://www.webapps-online.com/online-tools/user-agent-strings/dv/browser212364/uc-browser. Do we care about those?

Good catch! I'm not sure if it's necessary but I made a change to look for UCWEB as well.

Hey @tdumitrescu - just wanted to check in and see if you had a chance to look at this. Thanks!

@kathmath we'll cut a release today!

v2.10.0 has been released with these changes

Thank you @jbwyme @tdumitrescu !!!


Add mosquitto instructions to README

This should fix #2. You must run a mosquitto server.
/to @superkhau

LGTM. Merging.


Field names must be quoted or users must pay attentions to it during use persist function?

Tables having field names that using keyword like 'key' could not be persisted.
Error Message: Message: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'key, value) VALUES ...
To avoid this quote field name in setDefaultTemplate function like below.
    protected function setDefaultTemplate(): void
    {
        $this->setDefaultData(function (Generator $faker) {
            return [
                '`key`' => $faker->word(),
                '`value`' => $faker->randomNumber(),
            ];
        });
    }

The message was throwed at vendor/vierge-noire/cakephp-fixture-factories/src/Factory/BaseFactory.php:334
The persist method must quote as long as it is not virtual field.
But it seems responsibility of CakePHP itself?
I know it's our code's fault, but sometime we must maintain kind of this type project...
I would appreciate if it was fixed, but we can avoid like above so just let me register issue.

@k2rn can't you quote the identifier as described here?
https://book.cakephp.org/4/en/orm/database-basics.html#identifier-quoting


gosec analysis and fixes: 20241202

sweeping scan of our public packages (./pkg/...) with gosec 2.21.4 and related fixes.

/lgtm
/approve


Upload/Download dataset with 0 Spots

If you upload the properties for a dataset and then try to upload the Spots to datasetSpots the server gives an error if there are 0 Spots.
If you try and download all spots for that dataset you get a not found error.
How do I differentiate this error when trying to download from a dataset that exists but has 0 spots from trying to download all spots from a dataset that doesn't exist since it looks like they give the same error?

@jessicagood The datasetSpots controller only deals with spots. If you want to check for the existence of a dataset, you should use the dataset controller, located here:
http://strabospot.org/db/dataset/14725081181717
The datasetSpots controller only looks at spots, and will return a 404 if no spots are found.
If you want me to change this behavior I can, but I'm not sure how else to differentiate the fact that a dataset has no spots and a dataset does not exist.


Filtering related set with escapable characters

As per https://docs.djangoproject.com/en/2.2/topics/db/queries/#escaping-percent-signs-and-underscores-in-like-statements django auto-escapes especial characters like % or _ on LIKE query sets (startswith, endswith, and the likes of those).
I am experiencing in 1.0 a weird behavior, namely, on a custom object I try to get a specific attachment, using the expected attachment name, like so:
from salesforce import models
...
class CustomObject(models.Model):
  ...
  attachment_name_auto = models.CharField(...)
  ...
  def get_attachment(self):
    return self.attachment_set.filter(name__startswith=self.attachment_name_auto)

The problem lies with the naming of the attachment, it follows a pattern like <formatted_date_value>_<contact_name>_<formatted_time_range>, and it causes the queryset to return empty.
I have confirmed that using any part of the name between underscores yields results, only when an underscore is included, the behavior becomes buggy.
Is there a specific way I should escape the string beforehand or is this a bug?

Hah .. I just realized that I had not upgraded the staging environment to 1.0 and was still in 0.9. it does work properly in 1.0, so please disregard this entirely.

It is also a duplicate of #254 where you find details, why it was so and why the current better solution can't be perfect.
Simply: Salesforce uses a simplified escaping and requires it. That doesn't allow to search an exact '%' or '_' in LIKE expressions. It is better to search a string with a wildcard instead of to find nothing and even to be not able to construct any less restrictive string. That is the substance of the fix in v1.0.
The result can be filtered exactly by Python if necessary finally.


[PPP] Xiao Yun

[ ] Add UG extracts
[ ] Add DG extracts

Codecov Report

Merging #353 (13e80c9) into master (86cf21f) will increase coverage by 0.15%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #353      +/-   ##
============================================
+ Coverage     68.37%   68.52%   +0.15%
- Complexity     1133     1136       +3
============================================
  Files           136      136
  Lines          3867     3867
  Branches        521      521
============================================
+ Hits           2644     2650       +6
+ Misses          998      992       -6
  Partials        225      225

Impacted Files
Coverage Δ

.../seedu/address/model/lesson/CalendarEntryList.java
84.61% <0.00%> (+3.55%)
:arrow_up:

Continue to review full report at Codecov.

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


[PURCH-2252] Fix drop shadow and padding on inquiry msg notification

The type of this PR is: Fix

This PR resolves PURCHASE-2252
Description
Adjusts the padding and dropshadow on the inquiry message notification as a QA followup.

PR Checklist (tick all before merging)
After

Before

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

This looks good to me. One question though, should this PR also include the text?
Text is 14px. Unica77 Medium for Title and Regular for text.
Or maybe the text is already that?

Yep, ended up chatting with Jonathan via Zoom to clarify that this was the expected/correct size!


Fast cursor positioning by label

Is your feature request related to a problem? Please describe

At present, the efficiency of line-by-line jump is too low.   If yazi can input labels  to jump to any position of any three views, it will greatly speed up the positioning efficiency
Will you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
When the user presses a command, such as f, it turns on the TAB jump mode, then displays a TAB for all directories and files, and then jumps to them by typing the TAB, like surfingkeys plugin in Google Chrome or flash plugin in nvim
Effect display:
https://github.com/sxyazi/yazi/assets/30348075/4fd7ed20-c6fa-4de4-bac7-eb046e296ed5
refer:
https://github.com/folke/flash.nvim
https://github.com/brookhong/Surfingkeys
Describe alternatives you've considered
No response
Additional context
No response

Hmm I think this is a bit beyond the scope of a file manager.
Do you want to implement it as a Lua plugin once Yazi's plugin system is ready? - We can discuss adding some generic Lua APIs to Yazi that would be necessary for implementing this feature.

Well, that's just a bold suggestion, and if you don't think he's that useful, then let's close it

Well, but let me clarify that I never consider it "useless" - in fact, it's a very useful extension for specific users.
Just saying it doesn't belong to the baseline of a file manager, just like Neovim doesn't natively integrate flash.nvim but allows such plugins to exist and provides the necessary API for them.

no,My idea is to make it work directly on yazi, so that it will work regardless of whether you use nvim or not. Is there documentation for yazi plugin development? Is it possible for a plug-in system to do such a job?
By the way, does yazi's plugin system support other languages for development, since I don't use rust,
Like py,c++,c,go,js. If yazi supports developing plugins for these languages, I should be able to make a plugin for it


Enter the absolute path where the nvm-windows zip file is extracted/copied to:

Issue:

I keep getting this error on nvm 1.1.9 installed with all defaults when running pnpm install
Nvm root is Current Root: C:\Users\olema\AppData\Roaming\nvm
I've tried reinstalling as well, same issue. Also tried different node versions.
How To Reproduce:

Windows 11
Install nvm 1.1.9
nvm install 16.14.2
nvm use 16.14.2
npm install -g pnpm
pnpm install --shamefully-hoist

Then I get prompted with Enter the absolute path where the nvm-windows zip file is extracted/copied to:
so I enter `C:\Users\olema\AppData\Roaming\nvm
which prints
SUCCESS: Specified value was saved.

SUCCESS: Specified value was saved.
\Common was unexpected at this time.

then next time I run pnpm install --shamefully-hoist I'm not asked to enter path, just met with the same:
SUCCESS: Specified value was saved.

SUCCESS: Specified value was saved.
\Common was unexpected at this time.

Expected Behavior:
Not getting prompted by "Enter the absolute path where the nvm-windows zip file is extracted/copied to:"

I'm not sure why this is happening, but it's clear the install.cmd file is running.  I'm not sure when I'll get a chance to look into this.

Hi, I have the same issue.
Is there any fix for this bug?
my script output was:
\Common was unexpected at this time. C:\Work\nvm-noinstall>(echo PATH=C:\Program Files (x86)\Common Files\Oracle\Java\javapath;C:\windows\system32;C:\windows;C:\windows\System32\Wbem;C:\windows\System32\WindowsPowerShell\v1.0\;C:\windows\System32\OpenSSH\;C:\Program Files\SafeNet\Authentication\SAC\x64;C:\Program Files\SafeNet\Authentication\SAC\x32;C:\Program Files (x86)\Kaspersky Lab\Endpoint Agent\;C:\Work\bin;C:\Program Files\Pandoc\;C:\Users\locadm\AppData\Local\Microsoft\WindowsApps) > c:\Work\nvm-noinstall\PATH.txt
Looks like script does not like "Common Files"

I'm having the same issue, was anyone able to fix?

Please change install.cmd to quote the PATH Variable like that:
(echo PATH=%PATH%) > %NVM_HOME%\PATH.txt
--->
(echo PATH="%PATH%") > %NVM_HOME%\PATH.txt

Delete/remove/uninstall the version of node in your "C:\Program Files\nodejs" directory. To the point where there is no more "nodejs" directory at all.
Delete the existence of nvm (C:\Users\xxx\AppData\Roaming\nvm), basically, manually delete the nvm dir.
Download your desired version of nvm (https://github.com/coreybutler/nvm-windows/releases)

Maybe the install.cmd should be renamed to installnvm.cmd its getting put on the path so any install call on windows is executing nvm's install and giving this prompt.  ie user type "install express"  instead of "npm install express"


lisav2: initialize $testSummary for every test run

$testSummary wasn't initialized when a test exception occured. This meant that a previously set $testSummary variable was passed to an aborted test. This fixes https://github.com/LIS/LISAv2/issues/523

Moved $testSummary & $testResult var init outside the try-catch block

Before this PR
ARM Image Under Test  : Canonical : UbuntuServer : 18.10 : latest
Total Test Cases      : 36 (21 Pass, 0 Fail, 15 Abort)
Total Time (dd:hh:mm) : 0:4:11
XML File              : TestConfiguration
ID TestCaseName                                                 TestResult TestDuration(in minutes)
1 BVT-VERIFY-DEPLOYMENT-PROVISION                                    PASS                 7.63
FirstBoot : PASS
FirstBoot : Call Trace Verification : PASS
Reboot : PASS
Reboot : Call Trace Verification : PASS
2 BVT-IS-ROOT-PASSWORD-DELETED                                    ABORTED                 0.99
FirstBoot : PASS
FirstBoot : Call Trace Verification : PASS
Reboot : PASS
Reboot : Call Trace Verification : PASS

After this PR
ARM Image Under Test  : Canonical : UbuntuServer : 18.10 : latest
Total Test Cases      : 2 (1 Pass, 0 Fail, 1 Abort)
Total Time (dd:hh:mm) : 0:0:11
XML File              : TestConfiguration
ID TestCaseName                                                 TestResult TestDuration(in minutes)
1 BVT-VERIFY-DEPLOYMENT-PROVISION                                    PASS                 6.08
    FirstBoot : PASS
    FirstBoot : Call Trace Verification : PASS
    Reboot : PASS
    Reboot : Call Trace Verification : PASS
2 BVT-IS-ROOT-PASSWORD-DELETED                                    ABORTED                 4.22

updated


Wrong result for some inputs

In case a doubled digit equals to 0, the sum is miscalculated and the result is wrong.
For example, the following valid number returns false instead of true:
luhnCheck('4024007174941954');

Or a shorter version:
luhnCheck('406'); // false, but should be true

A possible solution would be to replace the reducer function with this:
(acc, val, i) => (i % 2 !== 0 ? acc + val : acc + ((val *= 2) > 9 ? val - 9 : val))

The fix has been tested against all Paypal's credit card numbers for testing.

In case a doubled digit equals to 0, the sum is miscalculated and the result is wrong.
For example, the following valid number returns false instead of true:
luhnCheck('4024007174941954');

Or a shorter version:
luhnCheck('406'); // false, but should be true

A possible solution would be to replace the reducer function with this:
(acc, val, i) => (i % 2 !== 0 ? acc + val : acc + ((val *= 2) > 9 ? val - 9 : val))

The fix has been tested against all Paypal's credit card numbers for testing.

In this fix also "6011329933655299" is returning true, where it should be false.


Readdir infinity reading fix

readdir function must return FALSE in the end of directory listing.
http://php.net/manual/en/function.readdir.php

Awesome, thanks for taking the time to fix it!


Autogenerated config and cpp/hpp files

The original bebop_autonomy package autogenerate a:

a configure file BebopArdrone3.cfg
header files bebop_driver/include/bebop_driver/
message files from the bebop_msgs/msg/
rst files **states_param_topic.rst and ***settings_param.rst

These are generated with the generate.py script from the bebop_driver/scripts/meta script
These are generated by parsing the xml common.xml (with generate_states) and ardrone3.xml (with generate_states and generate_settings)
The autogenerated C++ files are either for states or settings. Several classes are defined either inheriting from AbstractSetting or AbstratState, both inheriting from AbstractCommand.  These are instantiated in ardrone3_setting_callback_includes.h , ardrone3_state_callback_includes.h and common_state_callback_includes.h and added to the callback_map_ map declared in the Bebop class.  This map is then used in Bebop::CommandReceivedCallback and Bebop::UpdateSettings.
The CommandReceivedCallback is registered at connection with ARCONTROLLER_Device_AddCommandReceivedCallback . These commands are used to publish, on ROS, the states in the subtopics states/{ardrone3,common}/xxxx. Also the object boost::shared_ptr<cb::Ardrone3PilotingStateSpeedChanged>   ardrone3_pilotingstate_speedchanged_ptr; declared in ardrone3_state_callback_includes.h is used to access the odometry published by the nodelet.
From the declared states objects, are also used in the nodelete :

ardrone3_pilotingstate_positionchanged_ptr for getting the GPS info published on the fix topic (type sensor_msgs::NavSatFix)
ardrone3_camerastate_orientation_ptr for getting the timestamp and position of the camera, for publishing with the camera_joint_pub_  publisher on the "joint_states" topic (type sensor_msgs::JointState)
and ardrone3_pilotingstate_attitudechanged_ptr  for setting the orientation component of the odom.pose.position ;  The twist part of the odometry is from the instantaneous velocity -from ardrone3_pilotingstate_speedchanged_ptr) and the position is the integrated velocity

The UpdateSettings is called from the nodelet BebopDriverNodelet::ParamCallback which is the callback assigned to the ROS1 dynamic reconfigure server. Hence, this is used to change the parameters of the bebop once changed from the ROS side.

For now, this is pause ; as not yet necessary (will be required for odometry) and not yet decided on the right way to integrate these procedurally generated codes in the new ROS2 api ;


provide currentHealth and maxHealth values on all categories and actions

Please provide "state" of health on all categories and actions.

Current and max health values should be recorded for all events.


Mixed resource group and account name in command "az ams streaming-endpoint show"

az feedback auto-generates most of the information requested below, as of CLI version 2.9.1

Describe the bug
Mixed resource group and account name in "az ams streaming-endpoint show"
To Reproduce
Run command az ams streaming-endpoint show -g MediaServicesQuickTest -a testmediaaccount8 -n 'default' , but the resource group and account name was mixed in the ourput as below shown.
The Resource 'Microsoft.Media/mediaServices/MediaServicesQuickTest/streamingEndpoints/default' under resource group 'testmediaaccount8' was not found.
Expected behavior
show the right output.
Environment summary
az version                                                                                {
"azure-cli": "2.9.1",
"azure-cli-command-modules-nspkg": "2.0.3",
"azure-cli-core": "2.9.1",
"azure-cli-nspkg": "3.0.4",
"azure-cli-telemetry": "1.0.4",
"extensions": {}
}
Additional context

ams

Hi there, thanks for reporting this. I have opened bug 8272644 on our team and we will be fixing this very shortly. Thank you!


Backport from master to hardknott

This PR backports the commit c462fb9 from master
This commit includes the following changes:
- cb28a8f updatehub: Fix single thread execution
- 4bc7641 updatehub: Update setup mock env execution string

Signed-off-by: Vinicius Aquino<EMAIL_ADDRESS>

It must be a cherry pick with -x.

Ok.


Custom "Example reference"-blocks for examples with manim directive

List of Changes

Add options :ref_classes: and :ref_functions: to the manim directive. Specifying either with a list of space separated class/function names renders a "Example reference" block between source code and video that contains links (!) to the classes/functions in the reference manual.

Motivation
This is a suggestion for an approach tackling #448, and thus strengthening the future connection between our example gallery and the reference manual.
Acknowledgement

[x] I have read the Contributing Guidelines

See the first example at https://manimce--607.org.readthedocs.build/en/607/examples/advanced_projects.html for the new options in action.

Thanks, this is a great addition!

Only, I don't like this yellow box too much.
Can we maybe change it to green? and make it surrounded by a black box?

Thanks, this is a great addition!

Only, I don't like this yellow box too much.
Can we maybe change it to green? and make it surrounded by a black box?

Sure, it can be whatever we want it to! Here is my new suggestion. :-)

Great addition.
Non-blocking: I'd put the See Also block after the video. But whatevs.
On Sat, Oct 24, 2020, 12:44 PM Benjamin Hackl<EMAIL_ADDRESS>wrote:

Thanks, this is a great addition!
[image: ddd]
https://user-images.githubusercontent.com/44469195/97083342-95e40000-160f-11eb-96de-4339730bc04f.png
Only, I don't like this yellow box too much.
Can we maybe change it to green? and make it surrounded by a black box?
Sure, it can be whatever we want it to! Here is my new suggestion. :-)
[image: image]
https://user-images.githubusercontent.com/11851593/97087229-d56b1600-1628-11eb-858e-0ab7e6e65607.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManimCommunity/manim/pull/607#issuecomment-715985167,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAILYAFTEWH2MZ6COFBG2UDSML75XANCNFSM4S5SNUIA
.

Wow, this looks amazing now!
I am unsure if it should be in the middle or beneath the video.
Pro in the middle:

It is completely clear to which example it belongs.

Pro at the bottom:

It is maybe a little more natural to have it there. E.g. Matplotlib does it this way
However, I think it is not clear enough when one example stops and where the new one stars. (I mean the bottom of the screenshot.)
Possible solutions could be for example to insert something that makes the examples more distinguishable: a solid line after one example or all example blocks with a white-grey background to visually advertise where one example block is.

Yes, I've always thought we could use some more whitespace.
On Sat, Oct 24, 2020, 4:28 PM kolibril13<EMAIL_ADDRESS>wrote:

Wow, this looks amazing now!
I am unsure if it should be in the middle or beneath the video.
Pro in the middle:

It is completely clear to which example it belongs.

Pro at the bottom:

It is maybe a little more natural to have it there. E.g. Matplotlib
does it this way
https://matplotlib.org/gallery/lines_bars_and_markers/curve_error_band.html#sphx-glr-gallery-lines-bars-and-markers-curve-error-band-py
However, I think it is not clear enough when one example stops and
where the new one stars. (I mean the bottom of the screenshot.)
Possible solutions could be for example to insert something that makes
the examples more distinguishable: a solid line after one example or all
example blocks with a white-grey background to visually advertise where one
example block is.

[image: sss]
https://user-images.githubusercontent.com/44469195/97092840-81bbf500-1647-11eb-83f0-2afdec4033ec.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManimCommunity/manim/pull/607#issuecomment-716050969,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAILYAGVUWRUQ35VAXU7ZOLSMM2IHANCNFSM4S5SNUIA
.

The reason I put it between video and code is to have the references close to the code; it felt a bit disconnected to put them afterwards.
Suggestion: Let's keep this PR about adding the options for the directive, and I'll look into redesigning the Example blocks in total on a follow-up PR. :-)


Product flaw: filter transaction by categories in UI

Duplicate of #204

Duplicate of https://github.com/AY2425S1-CS2103-F13-1/tp/issues/204


Connecting to pigpiod

Could you please implement a way of using pigpiod daemon instead of using the pipe interface?
When using the managed way you offer in your library, I did not found a way to chose between the two possibilities that PiGpio is offering.

Hi @Hotrunner
As you mention, for the moment, we don't have any support for using pigpio Daemon. We are going to evaluate whether is a good option to implement this alternative in our library, but I think this wouldn't be in the short term.
Could you please give us more information about what are you doing in your app?
How are you using your app, that you need the pigpio daemon?
How implementing this solution could help you to achieve what you need?
It is not possible to achieve what you need using the regular pigpio library (without the daemon)?

Hello @k3z0 ,
the first and most important reason is sending commands over the network.
Another reason is, you don't need sudo.
I think it's not possible to use the default pigpio library without sudo, right?

You are right @Hotrunner, since pingpio access directly to system registers to take control over the IC, it is not possible to use it without sudo.


Error installing hardhat ethers from Yarn Install

I'm getting an error on Yarn Install of 'No candidates found'.
➤ YN0001: │ Error: @nomiclabs/hardhat-ethers@npm:hardhat-deploy-ethers: No candidates found
at le (/Users/benjamingribbin/.yarn/releases/yarn-sources.cjs:441:7865)
at runMicrotasks ()
at processTicksAndRejections (node:internal/process/task_queues:96:5)
➤ YN0000: └ Completed in 4s 864ms
➤ YN0000: Failed with errors in 4s 870ms

Could you please try git pull --rebase, it could you possible that someone has fixed some issue since you have posted here

What OS are you using @bgribbin

SUN DeFi is the safest financial product, profitable every day, every week, high deposit interest in ETH and TROX, USDT
daily interest:
deposit 5000usdt = 630 USDT
deposit 30,000usdt=750 USDT
deposit 60,000usdt= 1920 USDT
deposit 80,000usdt= 2880 USDT
deposit 100,000usdt = 4000USDT
inbox for detailed advice: https://www.funawealth.com?share_address=0x0E2E7bb6D1A83eDAB9C84411C018a050E7BE406C


firefly-iii: init at 6.1.13, nixos/firefly-iii: init

Description of changes
We already have a PR for firefly-iii which has not been merged in #184362. This is an attempt to push in the latest version of a package without using composer2nix and using the latest instructions from the docs.
From the github repo:

"Firefly III" is a (self-hosted) manager for your personal finances. It can help you keep track of your expenses and income, so you can spend less and save more. Firefly III supports the use of budgets, categories and tags. Using a bunch of external tools, you can import data. It also has many neat financial reports available.
Firefly III should give you insight into and control over your finances. Money should be useful, not scary. You should be able to see where it is going, to feel your expenses and to... wow, I'm going overboard with this aren't I?
But you get the idea: this is your money. These are your expenses. Stop them from controlling you. I built this tool because I started to dislike money. Having money, not having money, paying bills with money, you get the idea. But no more. I want to feel "safe", whatever my balance is. And I hope this tool can help you. I know it helps me.

I'd love to get some feedback for this because I've only tested this on my machines with sqlite and postgres. While the module should setup sqlite on its own, the postgres and mysql databases have to be created by the user. There is no web-server setup included in this module either.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I would suggest adding a nginx enable option for ease of use. This is the ngixn setup that works for me:
services.nginx = mkIf cfg.enableNginx {
      enable = true;
      recommendedTlsSettings = mkDefault true;
      recommendedOptimisation = mkDefault true;
      recommendedGzipSettings = mkDefault true;
      virtualHosts.${cfg.virtualHost} = {
        root = "${package}/public";
        locations = {
          "/" = {
            tryFiles = "$uri $uri/ /index.php?$query_string";
            index = "index.php";
            extraConfig = ''
              sendfile off;
            '';
          };
          "~ \.php$" = {
            extraConfig = ''
              include ${config.services.nginx.package}/conf/fastcgi_params ;
              fastcgi_param SCRIPT_FILENAME $request_filename;
              fastcgi_param modHeadersAvailable true; #Avoid sending the security headers twice
              fastcgi_pass unix:${config.services.phpfpm.pools.firefly-iii.socket};
            '';
          };
        };
      };

@PatrickDaG I didn't do this because I believe that these configurations can vary quite a bit between different deployments and because I'm not sure whether nixos wants to privilege nginx over, say, caddy or apache. I know that a lot of projects setup nginx but I've not found it to be universal across web-apps.
And, of course, that also brings up the question of whether we should also setup mysql or postgres if the user sets that in the config.
However, if you believe strongly about this I'll add an nginx option as well.

I think a baseline working setup for nginx is nice to have, even if people deploy differently as it can show which endpoints exist and which extra config you may need. Of course, it would be preferable to have that for all web servers, the same as it would be the best to have automatic setup for any database people may choose.
But for the sake of maintainability I would say it's enough if you setup the software stack you use and if someone wants a complete setup for e.g., Postgres and Apache, they can always post their own pull request.
If you have a very special nginx setup and thus no need for an option as part of the module I can also offer to help maintain that part.  I would use this option, making it easier to maintain, and then you don't have to maintain things you may not need.

@PatrickDaG Let me add the nginx option and push this again in a bit. I might also have a way of building the package from source (though it leaves a few translations out).

Well. Funny coincidence, I was also just trying to get a build from source to work: This seems to work but as you said I think some translations are missing as well as some javascript
Only in result_old/public/v1/js: app.js
Only in result_old/public/v1/js: app.js.LICENSE.txt
Only in result_old/public/v1/js: app_vue.js
Only in result_old/public/v1/js: app_vue.js.LICENSE.txt
Only in result_old/public/v1/js: create_transaction.js
Only in result_old/public/v1/js: create_transaction.js.LICENSE.txt
Only in result_old/public/v1/js: edit_transaction.js
Only in result_old/public/v1/js: edit_transaction.js.LICENSE.txt
Only in result_old/public/v1/js: profile.js
Only in result_old/public/v1/js: profile.js.LICENSE.txt
Only in result_old/public/v1/js/webhooks: create.js
Only in result_old/public/v1/js/webhooks: create.js.LICENSE.txt
Only in result_old/public/v1/js/webhooks: edit.js
Only in result_old/public/v1/js/webhooks: edit.js.LICENSE.txt
Only in result_old/public/v1/js/webhooks: index.js
Only in result_old/public/v1/js/webhooks: index.js.LICENSE.txt
Only in result_old/public/v1/js/webhooks: show.js
Only in result_old/public/v1/js/webhooks: show.js.LICENSE.txt

So far with my short tests it seems to work, but I'm fully confident.

So turns out that's only because those files are built when the develop branch gets built. It shouldn't be a problem until the next version (6.1) comes along.

@PatrickDaG I've made all the requisite changes to this, I think. Please do go over it and see if I've missed anything. If you'd like to be added as a maintainer please tell me, I shall add you and then mark this as ready for review.

@SuperSandro2000 I think I'm satisfied with this at my end. Would appreciate a once over and maybe some more testing.

@drupol Is there anything else I need to do here before this is ready for a merge?

Hey @savyajha,
thank you, for your great work packing firelfy-iii for us! I already tried to set it up using your module and might provide some little feedback on this:
In the current implementation nix fails (even when bundled nginx is not used) without setting the parameter firefly-iii.virtualHost. Especially when using a reverse proxy, a default value of localhost just works fine in my setups.
… while evaluating the option `firefly.services.firefly-iii.virtualHost':
error: The option `firefly.services.firefly-iii.virtualHost' is used but not defined.

Also app.js got delivered without correct mime-type, so the web app is broken when the X-Content-Type-Options: nosniff header is set. I'm still working out what's causing it, maybe someone else will stumble across it.

@PatrickDaG Can you let me know what these options are? While I’ve been dogfooding this module, I might not have covered everything. I was also of the opinion that the defaults (set my the application itself) are sensible.
I will get the changes into a PR this weekend.

I think the mime-type mismatch is actually a 404 not found. In the old version of this package the app.js was not built correctly leading to nginx returning a 404 which has mime-type html resulting in the error you see. I've just tested the newest version, and it seems to be fixed.

I can confirm updating from 6.1.13 -> 6.1.15 fixes the app.js issue.


research unclaimed funds after a term

what happens to un claimed funds at the end of an election term

The funds stay in the contract


fix: orientation on iOS16

I've modified Orientation.m according to what Apple is requiring on iOS16 since [[UIDevice currentDevice] setValue:forKey:]; appears to be buggy.
Notice that the changes are under a condition of the OS being iOS16.

Other than iOS 16, Builds are failing.

Other than iOS 16, Builds are failing.

same issue for me iOS builds are failling.

Fixed failing build by wrapping ios 16 code with #ifdef __IPHONE_16_0
Here's a patch file I made if you're using patch-package
https://gist.github.com/nicolas6422/80cbc4528b4d4798c53ffdd2f56c3625

@nicolas6422 do you know if that's future proof? Like next year's iOS 17?

@gramiro @nicolas6422 any update for release mode because its not working for release mode.

@codal-mpawar ran into a similar issue, for us we build our app with eas so it was about setting the image property of ios to latest in our eas.json file so that our app was built with the latest Xcode 14.1 sdk.

@gramiro If you have a solution can you please fork the library and try to resolve this if possible? it will great help for us.

seems the camera screen is also orientated, is any way to fix?

hii as any one found solution I am also facing issue related to orientation locktolandscape not working
Please @LuczDev if you found any solution please respond asap thanks

@gramiro If you have a solution can you please fork the library and try to resolve this if possible? it will great help for us.

Did you find solution for release mode?


Combat mode is global

This also should not be.

Related #660

Toggling this lags the server a lot


[REQUEST] Dead ESP

ESP that is only turned on while you are dead. Would be very helpful for legit!

This function has been removed. I don't know why

Yeah I want this feature back too.


Added my name to the list and made little changes

I added my name to the list and also made corrections to a wrong syntax.

@dejiii  thnx for contributing, try this year, and make sure you follow me and star this repo for updates about special swags I will be giving.
Contribute in October for Hacktoberfest 2020


Define URL format to refer to project within a repository

However this is closely related to how I define a project ID. Still in some sense it is easier because I just refer to the URL for the repository and then something inside that repository.

Something like https://github.com/CodeSmithyIDE/Project/CodeSmithy/CodeSmithy.csmthprj?project_id=...
If we want to do project_name then names have to be unique. That's not too much to asks but no sure just yet.


Add Form-specific constructors for MPS

This PR adds Form-specific constructors for the MPS struct, which allows us to create Matrix Product States with an specific canonical Form directly on the constructor. I believe this is the proper way to go, and we added a check_canonical_form kwarg that, if true (default), it will check if the MPS fulfills the canonical form.
TO DO:

[ ] Regroup repeated code into an internal function. Maybe it can be named __tn_mps_from_arrays? @mofeing I can't find a good name.

Okay, everything should be ready now @mofeing. I separated the checks of the form in two check_form functions that are dispatched over the Form. I think this is a good idea since we would want to use these functions somewhere outside the constructor. In this case, maybe it's even better that they do not throw an error but just return false? I don't know, for me this is still fine.

Thanks!


Example using rive and skia

Hi there,
I'm trying to get rive and skia working in a GLFW window but am having trouble. I've only got the old flare examples to go on that I've been able to find.  However, a lot seems to have changed since these were written.
Do you have a simple example of at least rendering a rive file to a surface using skia?
Thank you!

Hi! This is an old experimental repo for the Flare C++ runtime! The new Rive one with working Skia renderer (see the Skia/viewer folder) is here: https://github.com/rive-app/rive-cpp

Leaving that comment for posterity, I totally misunderstood.
You have the right repo, wrong examples! You can export new compatible files (v7) from https://rive.app. You can open any file in the community and download the runtime (.riv) file for it.

Thanks for the answers (and for the fantastic product) Luigi - sorry, I should be more clear. I am developing a c++ app:

Create GL window with GLFW (working fine)
Create Skia GL context and surface (working fine)
Draw manually to window using skia (e.g. draw rectangles, lines - working fine)
Load .riv file and draw using skia <-- this is the part I'm having trouble with

For the last step I am trying to use the old flare example as a reference, but rive has changed so much since then. The code is not directly compatible (for example, I cannot find anything called an SkrActor) and there are no example projects in the new rive cpp repo.
Do you have an example of how to render any .riv file using skia once GL and skia are set up and ready?

In case anyone comes across this in the future, there is a good example in the repo but it's a bit tricky to find :)
rive/skia/viewer/src/main.cpp


Server and route metrics are not being collected correctly

{
  "jvm/classes/current_loaded": 6053,
  "jvm/classes/total_loaded": 6063,
  "jvm/classes/total_unloaded": 0,
  "jvm/compilation/time_msec": 49560,
  "jvm/fd_count": 31,
  "jvm/fd_limit": 4096,
  "jvm/gc/Copy/cycles": 15,
  "jvm/gc/Copy/msec": 2002,
  "jvm/gc/MarkSweepCompact/cycles": 2,
  "jvm/gc/MarkSweepCompact/msec": 451,
  "jvm/gc/cycles": 17,
  "jvm/gc/msec": 2453,
  "jvm/heap/committed":<PHONE_NUMBER>,
  "jvm/heap/max":<PHONE_NUMBER>,
  "jvm/heap/used": 145065728,
  "jvm/mem/buffer/direct/count": 5,
  "jvm/mem/buffer/direct/max": 198656,
  "jvm/mem/buffer/direct/used": 198656,
  "jvm/mem/buffer/mapped/count": 0,
  "jvm/mem/buffer/mapped/max": 0,
  "jvm/mem/buffer/mapped/used": 0,
  "jvm/mem/current/Code_Cache/max": 50331648,
  "jvm/mem/current/Code_Cache/used": 6184256,
  "jvm/mem/current/Eden_Space/max": 572653568,
  "jvm/mem/current/Eden_Space/used": 28105728,
  "jvm/mem/current/Perm_Gen/max": 85983232,
  "jvm/mem/current/Perm_Gen/used": 35826344,
  "jvm/mem/current/Survivor_Space/max": 71565312,
  "jvm/mem/current/Survivor_Space/used": 919200,
  "jvm/mem/current/Tenured_Gen/max":<PHONE_NUMBER>,
  "jvm/mem/current/Tenured_Gen/used": 116040808,
  "jvm/mem/current/used": 187087488,
  "jvm/mem/postGC/Eden_Space/max": 572653568,
  "jvm/mem/postGC/Eden_Space/used": 0,
  "jvm/mem/postGC/Perm_Gen/max": 85983232,
  "jvm/mem/postGC/Perm_Gen/used": 32898992,
  "jvm/mem/postGC/Survivor_Space/max": 71565312,
  "jvm/mem/postGC/Survivor_Space/used": 919200,
  "jvm/mem/postGC/Tenured_Gen/max":<PHONE_NUMBER>,
  "jvm/mem/postGC/Tenured_Gen/used": 116040808,
  "jvm/mem/postGC/used": 149859008,
  "jvm/nonheap/committed": 42663936,
  "jvm/nonheap/max": 136314880,
  "jvm/nonheap/used": 41971624,
  "jvm/num_cpus": 1,
  "jvm/postGC/Eden_Space/max": 572653568,
  "jvm/postGC/Eden_Space/used": 0,
  "jvm/postGC/Perm_Gen/max": 85983232,
  "jvm/postGC/Perm_Gen/used": 32898992,
  "jvm/postGC/Survivor_Space/max": 71565312,
  "jvm/postGC/Survivor_Space/used": 919200,
  "jvm/postGC/Tenured_Gen/max":<PHONE_NUMBER>,
  "jvm/postGC/Tenured_Gen/used": 116040808,
  "jvm/postGC/used": 149859008,
  "jvm/start_time":<PHONE_NUMBER>312,
  "jvm/thread/count": 15,
  "jvm/thread/daemon_count": 13,
  "jvm/thread/peak_count": 15,
  "jvm/uptime": 1512674,
  "scheduler/dispatches": 0,
  "scheduler/productivity": 0.8691284656524658,
  "srv/http/app/closechans": 36,
  "srv/http/app/closed": 36,
  "srv/http/app/closes": 0,
  "srv/http/app/connection_duration.avg": 0,
  "srv/http/app/connection_duration.count": 0,
  "srv/http/app/connection_duration.max": 0,
  "srv/http/app/connection_duration.min": 0,
  "srv/http/app/connection_duration.p50": 0,
  "srv/http/app/connection_duration.p90": 0,
  "srv/http/app/connection_duration.p95": 0,
  "srv/http/app/connection_duration.p99": 0,
  "srv/http/app/connection_duration.p9990": 0,
  "srv/http/app/connection_duration.p9999": 0,
  "srv/http/app/connection_duration.stddev": 0,
  "srv/http/app/connection_duration.sum": 0,
  "srv/http/app/connection_received_bytes.avg": 0,
  "srv/http/app/connection_received_bytes.count": 0,
  "srv/http/app/connection_received_bytes.max": 0,
  "srv/http/app/connection_received_bytes.min": 0,
  "srv/http/app/connection_received_bytes.p50": 0,
  "srv/http/app/connection_received_bytes.p90": 0,
  "srv/http/app/connection_received_bytes.p95": 0,
  "srv/http/app/connection_received_bytes.p99": 0,
  "srv/http/app/connection_received_bytes.p9990": 0,
  "srv/http/app/connection_received_bytes.p9999": 0,
  "srv/http/app/connection_received_bytes.stddev": 0,
  "srv/http/app/connection_received_bytes.sum": 0,
  "srv/http/app/connection_requests.avg": 0,
  "srv/http/app/connection_requests.count": 0,
  "srv/http/app/connection_requests.max": 0,
  "srv/http/app/connection_requests.min": 0,
  "srv/http/app/connection_requests.p50": 0,
  "srv/http/app/connection_requests.p90": 0,
  "srv/http/app/connection_requests.p95": 0,
  "srv/http/app/connection_requests.p99": 0,
  "srv/http/app/connection_requests.p9990": 0,
  "srv/http/app/connection_requests.p9999": 0,
  "srv/http/app/connection_requests.stddev": 0,
  "srv/http/app/connection_requests.sum": 0,
  "srv/http/app/connection_sent_bytes.avg": 0,
  "srv/http/app/connection_sent_bytes.count": 0,
  "srv/http/app/connection_sent_bytes.max": 0,
  "srv/http/app/connection_sent_bytes.min": 0,
  "srv/http/app/connection_sent_bytes.p50": 0,
  "srv/http/app/connection_sent_bytes.p90": 0,
  "srv/http/app/connection_sent_bytes.p95": 0,
  "srv/http/app/connection_sent_bytes.p99": 0,
  "srv/http/app/connection_sent_bytes.p9990": 0,
  "srv/http/app/connection_sent_bytes.p9999": 0,
  "srv/http/app/connection_sent_bytes.stddev": 0,
  "srv/http/app/connection_sent_bytes.sum": 0,
  "srv/http/app/connections": 0,
  "srv/http/app/connects": 36,
  "srv/http/app/handletime_us.avg": 0,
  "srv/http/app/handletime_us.count": 0,
  "srv/http/app/handletime_us.max": 0,
  "srv/http/app/handletime_us.min": 0,
  "srv/http/app/handletime_us.p50": 0,
  "srv/http/app/handletime_us.p90": 0,
  "srv/http/app/handletime_us.p95": 0,
  "srv/http/app/handletime_us.p99": 0,
  "srv/http/app/handletime_us.p9990": 0,
  "srv/http/app/handletime_us.p9999": 0,
  "srv/http/app/handletime_us.stddev": 0,
  "srv/http/app/handletime_us.sum": 0,
  "srv/http/app/load": 0,
  "srv/http/app/pending": 0,
  "srv/http/app/read_timeout": 0,
  "srv/http/app/received_bytes": 112387,
  "srv/http/app/request_latency_ms.avg": 0,
  "srv/http/app/request_latency_ms.count": 0,
  "srv/http/app/request_latency_ms.max": 0,
  "srv/http/app/request_latency_ms.min": 0,
  "srv/http/app/request_latency_ms.p50": 0,
  "srv/http/app/request_latency_ms.p90": 0,
  "srv/http/app/request_latency_ms.p95": 0,
  "srv/http/app/request_latency_ms.p99": 0,
  "srv/http/app/request_latency_ms.p9990": 0,
  "srv/http/app/request_latency_ms.p9999": 0,
  "srv/http/app/request_latency_ms.stddev": 0,
  "srv/http/app/request_latency_ms.sum": 0,
  "srv/http/app/requests": 245,
  "srv/http/app/sent_bytes": 3066264,
  "srv/http/app/socket_unwritable_ms": 0,
  "srv/http/app/socket_writable_ms": 0,
  "srv/http/app/success": 245,
  "srv/http/app/write_timeout": 0,
  "srv/http/closechans": 0,
  "srv/http/closed": 0,
  "srv/http/closes": 0,
  "srv/http/connection_duration.avg": 0,
  "srv/http/connection_duration.count": 0,
  "srv/http/connection_duration.max": 0,
  "srv/http/connection_duration.min": 0,
  "srv/http/connection_duration.p50": 0,
  "srv/http/connection_duration.p90": 0,
  "srv/http/connection_duration.p95": 0,
  "srv/http/connection_duration.p99": 0,
  "srv/http/connection_duration.p9990": 0,
  "srv/http/connection_duration.p9999": 0,
  "srv/http/connection_duration.stddev": 0,
  "srv/http/connection_duration.sum": 0,
  "srv/http/connection_received_bytes.avg": 0,
  "srv/http/connection_received_bytes.count": 0,
  "srv/http/connection_received_bytes.max": 0,
  "srv/http/connection_received_bytes.min": 0,
  "srv/http/connection_received_bytes.p50": 0,
  "srv/http/connection_received_bytes.p90": 0,
  "srv/http/connection_received_bytes.p95": 0,
  "srv/http/connection_received_bytes.p99": 0,
  "srv/http/connection_received_bytes.p9990": 0,
  "srv/http/connection_received_bytes.p9999": 0,
  "srv/http/connection_received_bytes.stddev": 0,
  "srv/http/connection_received_bytes.sum": 0,
  "srv/http/connection_requests.avg": 0,
  "srv/http/connection_requests.count": 0,
  "srv/http/connection_requests.max": 0,
  "srv/http/connection_requests.min": 0,
  "srv/http/connection_requests.p50": 0,
  "srv/http/connection_requests.p90": 0,
  "srv/http/connection_requests.p95": 0,
  "srv/http/connection_requests.p99": 0,
  "srv/http/connection_requests.p9990": 0,
  "srv/http/connection_requests.p9999": 0,
  "srv/http/connection_requests.stddev": 0,
  "srv/http/connection_requests.sum": 0,
  "srv/http/connection_sent_bytes.avg": 0,
  "srv/http/connection_sent_bytes.count": 0,
  "srv/http/connection_sent_bytes.max": 0,
  "srv/http/connection_sent_bytes.min": 0,
  "srv/http/connection_sent_bytes.p50": 0,
  "srv/http/connection_sent_bytes.p90": 0,
  "srv/http/connection_sent_bytes.p95": 0,
  "srv/http/connection_sent_bytes.p99": 0,
  "srv/http/connection_sent_bytes.p9990": 0,
  "srv/http/connection_sent_bytes.p9999": 0,
  "srv/http/connection_sent_bytes.stddev": 0,
  "srv/http/connection_sent_bytes.sum": 0,
  "srv/http/connections": 1,
  "srv/http/connects": 1,
  "srv/http/handletime_us.avg": 0,
  "srv/http/handletime_us.count": 0,
  "srv/http/handletime_us.max": 0,
  "srv/http/handletime_us.min": 0,
  "srv/http/handletime_us.p50": 0,
  "srv/http/handletime_us.p90": 0,
  "srv/http/handletime_us.p95": 0,
  "srv/http/handletime_us.p99": 0,
  "srv/http/handletime_us.p9990": 0,
  "srv/http/handletime_us.p9999": 0,
  "srv/http/handletime_us.stddev": 0,
  "srv/http/handletime_us.sum": 0,
  "srv/http/load": 1,
  "srv/http/pending": 1,
  "srv/http/read_timeout": 0,
  "srv/http/received_bytes": 154,
  "srv/http/request_latency_ms.avg": 0,
  "srv/http/request_latency_ms.count": 0,
  "srv/http/request_latency_ms.max": 0,
  "srv/http/request_latency_ms.min": 0,
  "srv/http/request_latency_ms.p50": 0,
  "srv/http/request_latency_ms.p90": 0,
  "srv/http/request_latency_ms.p95": 0,
  "srv/http/request_latency_ms.p99": 0,
  "srv/http/request_latency_ms.p9990": 0,
  "srv/http/request_latency_ms.p9999": 0,
  "srv/http/request_latency_ms.stddev": 0,
  "srv/http/request_latency_ms.sum": 0,
  "srv/http/requests": 0,
  "srv/http/sent_bytes": 0,
  "srv/http/socket_unwritable_ms": 0,
  "srv/http/socket_writable_ms": 0,
  "srv/http/success": 0,
  "srv/http/write_timeout": 0,
  "zipkin/log_span/error/com.twitter.finagle.ChannelWriteException": 1,
  "zipkin/log_span/ok": 0,
  "zipkin/log_span/try_later": 0,
  "zipkin/log_span/unfinished": 1
}

The metrics are being collected correctly, the histogram values appear to have a 10sec refresh interval.


Selecting every Nth frame

Hello,
I've been able to the the ffmpeg library to import a video and extract a sequence of JPG images (1 frame every 300 frames) using the below command.
ffmpeg -i input.mov -vf "select=not(mod(n\,300))" -vsync vfr -q:v 2 img_%03d.jpg
However, I am unable to duplicate this effort when using the ffmpeg.filter function in the ffmpeg-python module. When I run
ffmpeg.input("example.mov).filter("select","not(mod(n\,300))").output('image_%03d.jpg')
Python outputs the following error:
Traceback (most recent call last):   File "<string>", line 32, in <module>   File "C:\Users\CONNOR~1\MINICO~1\envs\general\Lib\site-packages\ffmpeg\_run.py", line 325, in run     raise Error('ffmpeg', out, err) ffmpeg._run.Error: ffmpeg error (see stderr output for detail)
Removing the \' in the not(mod(n,300))` statement allows the code to execute, but it outputs every frame in the video (30 fps * 30 seconds = 900 frames).
Does anyone have any suggestions or can they provide an example where I can use the select filter in ffmpeg-python to extract images at specific intervals?
Thanks in advance!

I am running into the same issue - have you found a solution for this?

Have you checked this out?
I would try using \\, or using python's raw string r"not(mod(n\,300))".
I didn't test this, it's just a test for you to try :)

Thanks,! Yes, I tried that. Getting rid of the backslash altogether actually does seem to work. I am copy+pasting my current def for reference below. I am not sure I understand all the details. The vsync='vfr' is / seems to be important to dial down the frame rate / not replicate frames that have been skipped.
def trim_video(input_file, output_file, start, end, to_skip=100):
    '''
    Trim an input video file to start / end
    and take only a subset of the original frames

    '''
    filter_string = f"not(mod(n,{to_skip}))"
    pipe = (
        ffmpeg
        .input(input_file.as_posix())
        .trim(start=start, end=end)
        .setpts('PTS-STARTPTS')
        .filter('select', filter_string)
        .output(output_file.as_posix(), vsync='vfr')
        .global_args('-loglevel', 'error')
        .global_args('-y')
        .run()
    )

I actually solved the issue by accessing ffmpeg using Python and the subprocess.call function. The \\ was key!

aha, great that it worked!


Large model not evaluating on the devtest set

Hi! We're having trouble sending in our large model for the contest.
When we send in our base model (Model 278) it evaluated correctly on both the dev set and the devtest set. However, when we send in our large model (Model 283) with the exact same handler as the base model, it only evaluated on the dev set, but not the devtest set. I was using batch size 128 for both of these submissions.
I tried reducing the batch size a little (Model 284), the same problem occurs. I also tried increasing the batch size a little (Model 293), and it gave me a takendown status. I'd rather not trial-and-error batch sizes as there is a submission limit per day, so I decided to post an issue instead.
Can we get any advice on what to do here? Thank you in advance for the help!

Hi, we faced a similar issue, where our large model only shows results on the dev set (model 192). Reducing the batch size did not help (model 213). Would be great if we could get some further information on how to resolve this. Thanks!

Hi, can we get more information about running restrictions? For example, GPU memory, maximum running time, and so on. Thanks！

First could you share your model name as well as your model ids ? it's easier for me to look for them in the backend

our large model (Model 283)  it only evaluated on the dev set

this model (finetune-saptarashmi) actually did not run on the devtest, only on dev. I think it's because the name of the model is too long. There seems to be a size limit for job name on AWS and we are reaching it ...
Can you retry with something short ? srph12 or something.

I tried reducing the batch size a little (Model 284)
This one (srph-cbtflores-large) has the same issue than above, but seems to have run fine on dev and test.

I also tried increasing the batch size a little (Model 293),
This one (srph-cbtflores-large) is using a batch size of 192. It seems to have run out of memory on the test set.

@dannigt

Hi, we faced a similar issue, where our large model only shows results on the dev set (model 192). Reducing the batch size did not help (model 213). Would be great if we could get some further information on how to resolve this. Thanks!

Model 192 (mm100bigtrimmedtune) seems to also have the "too long name issue"
Model 213 (ts1626271321-mm100bigtrimmedtunesmallbatch) even longer name, only evaluated on dev.

Hi, can we get more information about running restrictions? For example, GPU memory, maximum running time, and so on. Thanks！

the restrictions are a the one of a ml.p2.xlarge instance, 20 minutes to answer to each batch of 1800 sentences, and apparently a model name of at most 14 characters.

And I'm very sorry to notice this "too long model name" issue only now. Hope you didn't lose to much time.
I'm also looking if it's possible to rerun the missing evaluation on your behalf, without you having to resubmit the model with a shorter name.

@gwenzek Resubmitting the missing jobs will mitigate this issue immediately in the current design, before an updated design is landed.

Hi! We resubmitted our large model (Model 300) with a much shorter name (srph-large) now, but it doesn't seem to be evaluating on anything. The status says "Created" though. Is this delay because of the queue of models being evaluated or is there an error that we're not seeing? Thanks for the quick response btw!

Hi! We resubmitted our large model (Model 300) with a much shorter name (srph-large) now, but it doesn't seem to be evaluating on anything. The status says "Created" though. Is this delay because of the queue of models being evaluated or is there an error that we're not seeing? Thanks for the quick response btw!

We also tried a shorter name (model 299; dlmbtt) but also don't see the results. Could you take a look again? Thanks a lot!

@dannigt the model dlmbtt seems to have a small misconfiguration. It's missing the model_generation.json. Can you check that the file is mentionned in the setup_config.json file ? (you can see it by dynalab init --amend IIRC).
@jcblaisecruz02 the inference seems to have correctly run. I'm investigating on why you don't see the results.

@jcblaisecruz02 the scores should be visible to you now, I'm not sure why they weren't before

@dannigt the model dlmbtt seems to have a small misconfiguration. It's missing the model_generation.json. Can you check that the file is mentionned in the setup_config.json file ? (you can see it by dynalab init --amend IIRC).
@jcblaisecruz02 the inference seems to have correctly run. I'm investigating on why you don't see the results.

Thanks a lot for catching this! We have submitted again with model_generation.json.
Could you kindly take a look again at model 303 (dlmbtt)? We still cannot see results after the submission. Thanks!

This also happens to my model 301 and 302. It seems the system requires more time to create recently. The models have been submitted over 14 hours and it is still in "No data available, the model is still evaluating".

@BaohaoLiao thanks for reporting. This was due to the fact we are starting to see more submissions on the Flores101 full track. Those submissions are taking significantely longer to evaluate and where filling up the evaluation queue.
I've split up the queues, with one queue for small tracks and one queue for the full track. The small track models have all been evaluated now, and you shouldn't see more delay in the future.

@BaohaoLiao thanks for reporting. This was due to the fact we are starting to see more submissions on the Flores101 full track. Those submissions are taking significantely longer to evaluate and where filling up the evaluation queue.
I've split up the queues, with one queue for small tracks and one queue for the full track. The small track models have all been evaluated now, and you shouldn't see more delay in the future.

Thank you for your work.

@gwenzek Sorry for asking again: would you be able to check model 310 (dlmbtts)? It was successfully deployed 4.5 hours ago. I see results for model 309 has been public for a while, so just to check if something went wrong with our model. Thanks a lot in advance!

@dannigt Model 310 did crash: ModuleNotFoundError: No module named 'fairseq'
Can you share with me the setup_config.json of the submission ? The requirements key should be set to true, and you should have a requirements.txt file also.

@dannigt Model 310 did crash: ModuleNotFoundError: No module named 'fairseq'
Can you share with me the setup_config.json of the submission ? The requirements key should be set to true, and you should have a requirements.txt file also.

@gwenzek Thank you so much for checking! Indeed, in that submission I accidentally left the requirements key to false.
Could you further take a look why model 317 and 332 (both named dlmbttss) are not showing results?
Sorry for the frequent questions - we weren't able to run the integration tests because of insufficient docker support on our server.


Re-write the error codes on the Analyse page to be more understandable to users

Error codes on the analyse page are not always clear to users, and don't explain how they might be able to fix the issues e.g:

Use Case:
As the Head of Medicines Optimisation, I want the error codes on the analyse page to be clear on how I can solve the problem so that this doesn't prevent me using this area of the site *
*This particular user tried to use analyse, got an error code, didn't understand it, so didn't use it again
Two things to deal with:
Easy: I can review the error codes we already have (Seb needs to send them across to me)
Hard: We don't have a comprehensive list of all the things that may cause an error to show up on that page. I will make a note of any that come up with demonstrating to users, and will create new github issues as they arise
(this issue can be marked as completed once our current error codes are re-written)
Priority (for rewriting error codes): Low
Ease: Easy

great, plan: seb to generate list error codes, lisa to review and write clearer ones for now, then we rethink

@lmfrench not very user-friendly, but the codes are basically in here - let me know if they are not standing out enough for you! https://github.com/ebmdatalab/openprescribing/blob/master/openprescribing/api/views_spending.py

yes thanks will go through them later in the week!

I think the only error code from that list which might appear to users on the analyse page is this one:
'Error: Codes must all be the same length'
Possible change:
'Error: BNF codes must all be the same length if included in the same search box e.g. if you want to look at Cerelle and Cerazette prescribing, you cannot search for Cerazette_Tab 75mcg (0703021Q0BBAAAA) and Cerelle (0703021Q0BD) under 'on'. You could search for Cerazette (0703021Q0BB) and Cerelle (0703021Q0BD)
Is it clear that by search box I mean under 'on' they must be the same, and under 'versus' they must be too, but that you could have different lengths in each? I couldn't think of a better way to phrase it

Will add this to the FAQs, and then need link added to error code


[Feature][scheduler] Support task group selection from DolphinScheduler

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

The form of sending a task to DolphinScheduler will be added a new selection field -- task groups of DS, and after sending the taskGroupId value will write into DS's database. The task groups is showed relatived to current project code.

3. Fix a bug: when the form window is reopened after sending to DS, some field values are not correctly shown even though the request's callback contains the correct value.

Use case
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

I saw that there is also an intra-group priority configuration for task groups in ds. Would you consider adding it together?

OK, we will add the priority field.


set finder-telescope commands for c-v, c-t, c-x for opening chats in vertical split, horizontal split, and new tab

it would be really nice, do be able to use these keys, again for consistency to all my other telescopes.

awesome, you already implemented it. thx!

:heavy_check_mark: in the latest version


Feature: LTB support.

Hi Maksym,
This looks good, can you squash the commits thought please?
After that I can merge.
Thanks!
Nick


Add "mocha" to the "env" list of .esltinrc

Hi,
My editor is configured to automatically lint all files when an .eslintrc file is present, and files in the test/ dir don't validate because mocha isn't listed as en environment in the .eslintrc
I wish I could've made a PR to propose that change, but I couldn't locate the template for this file.

Hi @louisremi,
I have just added this feature in a new release. It should work now for newly generated projects.
The code you are looking for is located in https://github.com/weblogixx/react-webpack-template. This is the standalone version of the webpack config used by the generator.


support macports, package install bug fix, ensure curl

This pull request has a number of fixes described below. Fixes are in logically independent commits:

Support macports package manager (57d0691 & 	d04b877)
Pass advanced install options via env vars to enable passing the options even when bash executes the script via stdin.
Bug fix - bash syntax error when more than one space separated items are present in a list
install script fails when curl is not installed, so install it via the package manager. I did this only for 'apt' as mac has curl by default and I cannot test for 'yum'.

This looks fantastic, thank you! I'll review it in more detail tonight or tomorrow night.

This works nicely. Verified that CI builds fine without a cache.


Prevent login animation on initial fetch of the table (server side mode)

Issue Type

[x] Question

Specs
What version are you using?
2.21.0
Question
The loading animation when a user interacts with the table is expected but the loading animation during the first load of the page is a little bit annoying for the users. Even if I disable the animation it would still show an empty table for 1-2 seconds then show the populated table.
How can I avoid this? Is there any way to set the initial table data when in remote mode (server side) and prevent fetching it remotely (only the initial data)?

hey @florinfratica there is no loading animation included with VGT. Are you sure this isn't something your App is adding?

Hi, @xaksis.
Indeed the loading animation is something my app is adding. I added it because I don't want to display an empty table for the first few frames (although the animation isn't the best solution for my use case either).
I removed the animation and now the table is displayed empty until the API call is completed. Here is a gif with the behaviour to make it clearer:

As you can see, the table is empty for the first few frames because the API call that fetches table rows hasn't finished when the component is rendered.
So I am wondering if there is any method to provide initial data from client side without having to do the API call (while filtering, sorting etc. would still be server side) in order to avoid displaying an empty table.

Setting the rows and totalRows options solved this issue.
The problem was that I was initializing the rows with an empty array and totalRows with 0.


IndentWithInclude causes error

The new IndentWithInclude property on TemplateContext causes an error if there is no indentation before an include.
Example project: https://github.com/badcel/ScribanBugIndentWithInclude
Error:
Unhandled exception. <input>(2,3) : error : Specified argument was out of the range of valid values. (Parameter 'index')

This should be fixed in the next release 3.0.4

Oops, closed it to early, I still need to fix it, forgot about it in 3.0.4

Ok, now this should be fixed in the next release 3.0.4 😅

Thank you very much :+1:.


duplicity: updated dependencies

Motivation for this change
See #18112
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

fixes #18112
https://github.com/NixOS/nixpkgs/issues/18112

Thanks!


[CASSANDRA-18302] Statement source descriptions

Thanks for sending a pull request! Here are some tips if you're new here:

Ensure you have added or run the appropriate tests for your PR.
Be sure to keep the PR description updated to reflect all changes.
Write your PR title to summarize what this PR proposes.
If possible, provide a concise example to reproduce the issue for a faster review.
Read our contributor guidelines
If you're making a documentation change, see our guide to documentation contribution

Commit messages should follow the following format:
<One sentence description, usually Jira title or CHANGES.txt summary>

<Optional lengthier description (context on patch)>

patch by <Authors>; reviewed by <Reviewers> for CASSANDRA-#####

Co-authored-by: Name1 <email1>
Co-authored-by: Name2 <email2>

The Cassandra Jira

thanks @dcapwell, I've applied some fixes basing on your feedback


[ENG-337] [OATHPIT] Throw ownCloud into the Oath Pit

Ticket
https://openscience.atlassian.net/browse/ENG-337
Purpose
Enable and update the ownCloud provider for aiohttp 3.
Changes

WB .make_request() now supports WebDAV methods

aiohttp.ClientSession only has functions available for native HTTP methods. For WebDAV (a protocol that extends HTTP) ones, WB lets the ClientSession instance call _request() directly and then wraps the return object with aiohttp.client._RequestContextManager.

WB client session now supports customized connector

For providers that use a customized connector such as owncloud, the new session is created (if no session is found in the event loop) with the given connector while an existing session simply ignores (and closes) the new connector. Given that the session is per loop for each instance, the existing session if found must already have a connector with qualified customizations.

Side effects
No
QA Notes
TBD
Deployment Notes
TBD

Coverage increased (+2.6%) to 56.968% when pulling 357febce0e387e65c82eb25f7cc8d4940f8b2e9d on cslzchen:feature/oathpit-owncloud into 12f540acf2a5bb9983b5bc7e213b1513797beb07 on CenterForOpenScience:feature/oathpit.