Connection timeout when using AWS Lambda

I have a Spring Cloud Function in AWS Lambda that returns a connection timeout when trying to parse a website using JSoup. This connection timeout happens only with cold starts.
This function is triggered in prod by an EventBridge rule.
The website is parsed/scraped correctly when the function is warmed up, but after 1.5h to 2 hours, the function goes cold and on the next attempt in prod to parse the website, it results in a connection timeout.
When warmed up, the function usually completes within 8 to 10 seconds successfully. The lambda function has a timeout of 1 minute.
Currently, I'm using AWS Lambda without any VPC, as the function doesn't access any resources in my account.
I have verified the internet connection before using JSoup as you can see in my code.
Also, I can test the function using the "test" option of the console and everything works normally.
The URL is hidden in the logs for clarification.
What could be causing this issue?
2024-08-19T18:05:56.418Z
Internet is available
2024-08-19T18:05:56.419Z
Trying to connect to https://pra.uf**/ (attempt 1)
2024-08-19T18:06:12.712Z
Failed to connect to https://pra.uf**/ on attempt 1: Connection timed out
2024-08-19T18:06:13.712Z
Trying to connect to https://pra.uf**/ (attempt 2)
2024-08-19T18:06:29.094Z
Failed to connect to https://pra.uf**/ on attempt 2: Connection timed out
2024-08-19T18:06:30.094Z
Trying to connect to https://pra.uf**/ (attempt 3)
2024-08-19T18:06:45.478Z
Failed to connect to https://pra.uf**/ on attempt 3: Connection timed out
2024-08-19T18:06:46.478Z
Trying to connect to https://pra.uf**/ (attempt 4)
2024-08-19T18:06:55.461Z 7bcf94b5-6b47-42d2-815d-46cf3dd4ff7a Task timed out after 60.08 seconds

public boolean isInternetAvailable() {
  try {
    final URL url = new URL("http://www.google.com");
    final URLConnection conn = url.openConnection();
    conn.connect();
    conn.getInputStream().close();
    return true;
  } catch (IOException e) {
    return false;
  }
}

@Component
public class ScraperRU implements IScraperRU {

    private static final int TIMEOUT_CONNECTION = 35000; // 35 seconds
    private static final int RETRY_DELAY = 1000; // 1 second
    private static final int MAX_RETRIES = 4;
    private final Utils utils;
    private final ScraperHelper scraperHelper;
    private final String ruUrl;

    public ScraperRU(Utils utils, ScraperHelper scraperHelper, @Value("${RU_URL}") String ruUrl) {
        this.scraperHelper = scraperHelper;
        this.utils = utils;
        this.ruUrl = ruUrl;
    }

    public Document connectScraper(String webURL) throws InterruptedException {
        if (!utils.isInternetAvailable()) {
            throw new RuntimeException("No internet connection available");
        }

        int attempt = 0;

        System.out.println("Internet is available");

        while (attempt < MAX_RETRIES) {
            try {
                attempt++;
                System.out.println("Trying to connect to " + webURL + " (attempt " + attempt + ")");

                Connection.Response response = Jsoup.connect(webURL).timeout(TIMEOUT_CONNECTION)
                        .userAgent("Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.124 Safari/537.36")
                        .execute();

                System.out.println("HTTP Status Code: " + response.statusCode());
                System.out.println("HTTP Status Message: " + response.statusMessage());
                System.out.println("Response Headers: " + response.headers());

                if (response.statusCode() == 200) {
                    return response.parse();
                } else {
                    System.out.println("Unexpected HTTP status code: " + response.statusCode());
                    throw new RuntimeException("Failed to retrieve content from the website due to unexpected HTTP " + "status code: " + response.statusCode());
                }
            } catch (IOException e) {
                System.out.println("Failed to connect to " + webURL + " on attempt " + attempt + ": " + e.getMessage());

                if (attempt >= MAX_RETRIES) {
                    throw new RuntimeException("Failed to retrieve content from the website after " + MAX_RETRIES + " attempts");
                }

                Thread.sleep(RETRY_DELAY);
            }
        }

        throw new RuntimeException("Failed to retrieve content from the website after " + MAX_RETRIES + " attempts");
    }

Hi there,
I don't see this as a jsoup issue. You could switch your isInternetAvailable() method to use jsoup.connect() to validate that. jsoup just uses a plain Java HttpURLConnection to make requests and the connection timeout is applied there. That's the same mechanism that implements the url.openConnection.
One guess is that your backend URL is also going cold between runs and takes longer than the connection timeout to spin up. Or some firewall along the way doesn't like your request.
Good luck rooting it out!


Jobresult

Closes #17 with Adding Job<Result<_,_>

Hi @TheAngryByrd, I am currently held up. Will merge the changes over this weekend.

@TheAngryByrd I am getting this weird build error on my machine, and because of it, I am not able to publish the package. In CI also, it is failing!
~/FsToolkit.ErrorHandling/src/FsToolkit.ErrorHandling.JobResult/JobResultCE.fs(157,15): error FS0074: The type referenced through 'System.OperationCanceledException' is defined in an assembly that is not referenced. You must add a reference to assembly 'System.Threading.Tasks'. [~/FsToolkit.ErrorHandling/src/FsToolkit.ErrorHandling.JobResult/FsToolkit.ErrorHandling.JobResult.fsproj]

I tried adding this package, but it didn't succeed.
Any idea?

@demystifyfp https://github.com/demystifyfp/FsToolkit.ErrorHandling/pull/47 should fix it


I cant export from blockbench if I have animations

I have that issue where I cant export if I have an animation and it gives me this error message whenever I do it:

Nvm I just had to update java and it worked


edb.errors.InternalServerError: 'NoneType' object has no attribute 'is_single'

create type a;
create type b;
with  x := (insert b) insert a;

edb.errors.InternalServerError: 'NoneType' object has no attribute 'is_single'
(nightly 4.0-dev.7669+aff1ecb)

Whoops, wish I hadn't missed this issue!
Fixed by https://github.com/edgedb/edgedb/pull/6287


Update front-end to support the comparison of two different readGroupSets (when applicable)

Update the front-end so the user is able to select two different readGroupSets for the IGV to render a comparison graph between the two.
This would involve the changes in main.js

The preliminary coding stage is done, need to think about what to do when there's only one readGroupSet available.

Changes committed in https://github.com/CanDIG/ga4gh-server/commit/3ead63a1fcb3c68566b6f40476b2d30741d53a11
need more cleanup before it can be merged into authz

There's a bug in igv.js, where when users select two readGroupSets that are the same, igv will fail to load the data, further investigations are needed. To avoid the issue, we can either dynamically change the dropdown options (remove the ones that are selected by the user), or only pass one of the selected elements if both are the same.

The fix for this issue was committed, more info available from #24

After a local fresh install, I believe they are working as expected.


Unit testing needed

Unit testing needed for helpers.KeyFromPassword and cli* from encryptor.go

More testing added. More needed.


Tweaks for run_benchmarks.rb

Will need to deconflict the OUT_CSV_PATH line with https://github.com/Shopify/yjit-bench/pull/194 but should be trivial.
See individual commit messages for the changes.

I wonder if we can rename harness directories to something like:
harness/bips
harness/continuous
harness/perf
harness/stats
harness/warmup
harness/default # renamed from harness

so that we can just pass --harness=warmup instead of --harness=harness-warmup. We're getting more harnesses lately. It would tidy up the repository root too.

The downside of that is ruby's -I is not enough then to specify the harness.
IMO it's not worth the complexity and I don't see any issue with a few harness-* at the root, it's actually to show there is not just one harness.
We could have special handling for the --harness option to automatically append -harness if not already there though, I can add that.

This is now ready for review.
It fixes https://github.com/Shopify/yjit-bench/pull/201/files#r1139215845
And makes this work:
$ ruby run_benchmarks.rb --rss --harness warmup -e "ruby::ruby" fib

And also the RSS are saved in the JSON file.

As a note, RSS is always measured since #201. So the --rss option is just whether to include the RSS measurements in the various outputs. I'll update the option description to clarify that.


[F5] [F5.1] [F5.1.2] Update specific user's attempt's notes

Update specific user's attempt's notes
Related Feature: Attempts details
Description
Update attempt's notes. Can only be done by user himself.

Closing - completed in PR #110


[Event Request] Codeunit 402 "Purchase Header Apply"

I would like to ask you to create an Integration event in codeunit 401 "Purchase Header Apply", in trigger "OnRun".
trigger OnRun()
begin
PurchHeader.Copy(Rec);
with PurchHeader do begin
PayToVendorNo := "Pay-to Vendor No.";
VendLedgEntry.SetCurrentKey("Vendor No.", Open);
// >>>
OnRunOnBeforeFilterVendLedgEntry(CustLedgEntry, PurchHeader);
// <<<
VendLedgEntry.SetRange("Vendor No.", PayToVendorNo);
VendLedgEntry.SetRange(Open, true);
OnRunOnAfterFilterVendLedgEntry(VendLedgEntry);
if "Applies-to ID" = '' then
...
end;
Event signature:
[IntegrationEvent(false, false)]
local procedure OnRunOnBeforeFilterVendLedgEntry(var VendorLedgerEntry: Record "Vendor Ledger Entry"; PurchHeader: Record "Purchase Header")
begin
end;
In our extension, we need it because we would like to filter Vendor Ledger Entry on extra data from Purchase Header.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 41782.


feat: catch POOL_TOO_LOW_PRIORITY author rpc error

Signed-off-by: Gregory Hill<EMAIL_ADDRESS>

Could you try adding some integration tests to check that outdated nonce & too low priority checks work? I thought we had one for outdated nonce, but if so, it's clearly not working correctly. I feel like this error handling is super fragile (this is the third pr in a couple of weeks related to error handling), so it warrants automated regression testing


Event ID Failures

Any information would be helpful in solving these errors.
Event ID 7211 - The Secondary Account Primary Refresh Token prerequisite check failed.
Followed by
Event ID 7206 - Windows Hello for Business failed to locate a certificate registration authority.

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

ID: 49d45db5-5172-45cb-2c4d-d73a9e15d699
Version Independent ID: fa08e81d-3307-514e-129a-843e812e97a9
Content: Hybrid Windows Hello for Business key trust Provisioning (Windows Hello for Business)
Content Source: windows/security/identity-protection/hello-for-business/hello-hybrid-key-whfb-provision.md
Product: w10
GitHub Login: @MikeStephens-MS
Microsoft Alias: mstephen

Hi JonBrekhus.
Sorry that you are having issues.
Please understand that this site is a product documentation site and not a support nor a feature request site.
As such, the feedback section here is intended for feedback on the documentation and not intended for feedback on product features nor troubleshooting.
For product feature requests you can submit via User Voice.
For troubleshooting you can visit support.microsoft.com, answers.microsoft.com or open a support ticket.
I hope this gets you pointed in the right direction.
Recommend closure.

@AndreaBarr - This issue can be closed. Thank you.

Stupid!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

How the do i delete my post then?

This sucks!

Hi Jon.
No need to delete.
This issue was closed and will slide into a different bucket.
Best of luck with your troubleshooting!


Release 0.8

hi admin, i would like to read bookinfo sample code, please approve my pull request!

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kyessenov
If they are not already assigned, you can assign the PR to them by writing /assign @kyessenov in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@superdullard: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Hi @superdullard. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

What does this PR do?

We are not planning any release out of the release-0.8 branch.
If anything is needed - please use 1.0 branch.


relative url should be prefixed with a /

Todos do not load on react native or in mobile view due to complete URL resolving to http://localhost:8000api/v1/todos/user with process.env.WEB_ADDR not set and using default URL

Thank you!


Escalate failed master builds via Google Chat

Currently there is no escalation mechanism in case CI build fails in the master branch.
In order to notify the team members of such events, a Google Chat bot should be implemented.
Supported functionality

Every hour the bot should review the status of the latest builds in master branches of Spine libraries:

base
time
core-java
web
gcloud-java
bootstrap
money
jdbc-storage

In case the latest master branch build is failed, the bot should create a new thread in the "Spine Developers" room in the Google Chat. The message should list

The message stating the build has failed and the name of the library.
The link to the build.
The time since when the build has been broken.
The title of the latest commit.

In case the build is not fixed in an hour, the bot sends the corresponding message to the same thread again.

If the build is fixed, the bot notifies the group via the same thread that the issue is now resolved.

Hard reset
As an optional step, the bot may be configured to perform the hard reset to the latest successful commit to the master. Such a drastic measure may be taken if the build does not get a fix in 12 hours since the initial issue discovery.
However, such an operation is a risky action, as the commit history may be lost. So the hard reset option needs to be discussed additionally, perhaps, with an inclusion of a repository backup procedure (i.e. an upload of the whole repository along with its history to the Google Storage).
Implementation
The chat bot application should be written in Spine. Once done, it will be hosted in the Google Cloud.
The codebase for the chat bot should reside in the distinct SpineEventEngine/chat-bot repository.

@armiol, I haven't heard from ChatBot recently. Are we that accurate with builds, or is ChatBot dead?

@alexander-yevsyukov
That's because we don't have any broken builds on Travis. We are pretty accurate with that.
We'll have to discuss how to act in case one of GH Actions runs fails. For instance, our Windows-based builds aren't very stable, and what we really need for them is to restart first. And if it fails again, then report.


Review Ticket

Assignment
Points
Grade
Evidence

Pull Request (Integration)
2
1.75

Relevancy Checklist (Peer)
2
1.8

Lesson (Group)
1
0.92

Homework, Popcorn Hacks
1 x 8
7.325

Individual Contribution
1
0.93
    Mutator, Constructor, Constructor popcorn hacks, Mutator popcorn hacks

Personal Notebooks / Blogs
1
0.93
 Unit 5 Home. Team Teaches Overview Blog I wrote an overview notebook with a summary notes page for each lesson taught. I think it will be useful to reference in the future! It will be of great help for me and for when we make our N@TM projects!

Total
15
13.655
I think there is still room to improve. First off, I think we could have easily gotten a better score in the integration if we had ensured everything worked fine. We had issues on the naming conventions and didn't have a navbar, which Mortensen had mentioned us to add. I think the solution would have been talking and consulting with Mr. Mortensen more to clarify any doubts. I think other aspects like my individual contributions are quite solid. However, I do agree that we should have put less hacks in our lesson. I think we should have consolidated the hacks since a lot of them were just stepping stones. We could've combined a lot of hacks.

Skill
Points
Grade
Evidence

Work Habits (Analytics)
1
0.91

Team Planning (Issue)
1
0.92
https://github.com/KinetekEnergy/studentCSA/issues/8

Presentation Memories
1
0.9
   For the team memories, I chose to add some funny memories we made. I think people usually only add stuff like their lessons or plans. Personally, I think you should include personal things too. Memories should be fun! They should be times where you were enjoying yourself with your team! For these reasons, I chose to add a couple screenshots of our Slack DMs since I found them funny. I think they were a good memory for me! However, I still added some evidence of the specific lesson that I taught.

Grading and Feedback
1
0.92
Grading

Beyond Perfunctory
1
0.95
  I tried to make my presentation interesting by adding humor. I think that often times, most of the team teaches have very dry language. They are so formal, and they don't feel fun. I tried to switch things up by adding a bit of slang and making it less formal while still keeping the important information students need to learn. Our group also used the banana as a pointer, and we made it a reward for the person who answers the most questions. I think that students payed more attention due to this since it was interesting and unique. I tried to also keep everything Minions themed so the presentation would be more engaging to students. I think our presentation was one of the most unique and had a great theme. No other group did what we did, and no other group made a poster (we made a minion poster for the room).

Total
5
4.6

Assignment
Weightage
Grade
Comments

College Board Coverage
20
TBD
TBD

Java Examples
30
TBD
TBD

Popcorn Hack Usage
10
TBD
TBD

Homework
10
TBD
TBD

Grading Plan
10
TBD
TBD

Original an Creative
10
TBD
TBD

Total
90

Note that I submitted by blogs to teams, but they haven't graded mine yet. I am still waiting on that. Once they have gotten back, I will update it ASAP.

I like personality and effort you put into answering some of these questions.

included many images and made many connections to the original presentation such as the minions. His notes are well organized and have references to what he learned during the team's teaching.

Good and thorough notes, even on topics you are familiar with, which shows dedication to each lessons.

Memories were fun to talk about
Funny memories
Very organized log page for easy future access

I  like how you review ticket and notes for each topic. It's concise, personalized, and easy to read.

I like all the images you use as evidence for your self grading. It really looks like you put in reflection which is nice and genuine.

Overall, it is solid work and then the blog and the issue are very organized as well. The issue included images and good summaries.

I like how organized your overview blog is and how you have easy access to the previous lessons. I like how you incorporated lots of code cells.

I liked your presentation as it was pretty engaging, one suggestion is maybe go quicker.

I like how you organized or unit summaries with different explanations and code snippets explaining the specific unit.

I liked how detailed your notes were and how you linked the lessons for quick reference.

Good organization with quick links for each lesson. The notes are very cleanly written with descriptions, code snippets, and highlights of key terms.

Notes is very good, and examples for each one help understand the material better. The whole blog looks really organized.

The presentation was really personalized and fun to listen to, and it was also informative.

Your content is organized and includes good ways to help you remember the content, such as the code cells and different symbols

Blog - in association with Units and Teaching
Memory - something to recall your individual presentation
Relevancy - something impactful you learned
Beyond Perfunctory - something that shows you want to learn
Presentation memory and minions poster.  Discussed constructors and DM's in lesson planning.   Had links to home work and discussed notes to breakdown key elements.   DIscussed code snips in his vocab.   Brain rot was memborable.

you kept your presentation very entertaining and consience

great organization, nice highlights, good summary of each topic from CB

I like how you included quick links for you to be able to remember specific concepts and how you included fun memories from your group.

The poster was well done showing the passion  and the work is concise and very indepth.

I like how you talked about all your memories with your group and other team teaches

I think the memories were done very well and showed understanding of CSA

Very nice presentation Aashray.  Very thorough presentation.


Add Nats Pub/Sub to ingestor once GraphQL abstraction is complete

Based on https://github.com/guacsec/guac/issues/216, the ingestor is currently not publishing via NATS. Once the abstraction is complete add publish to ingestor and subscribe to assembler.
	if js != nil {
		assemblerInputsJSON, err := json.Marshal(assemblerInputs)
		if err != nil {
			return nil, err
		}
		_, err = js.Publish(emitter.SubjectNameDocParsed, assemblerInputsJSON)
		if err != nil {
			return nil, err
		}
		logger.Infof("doc parsed: %+v", docTree.Document.SourceInformation)
	}

Closing as not needed. GraphQL abstraction will take care of this.


Add support for "content root" per language

For bigger multilingual documentation sites etc., having all the translated version in the same folder can be a little hard to administer, even if you split into sections/sub folders.
You can, currently, somehow get around this by setting a translationKey, but you will still have to put the language code in the file name.
I suggest that we introduce a "content root" setting per language:
content
├── en
│   └── docs
│       ├── some-doc-1.md
│       ├── some-doc-2.md
│       ├── some-doc-3.md
│       └── some-doc-4.md
└── jp
    └── docs
        ├── some-doc-1.md
        ├── some-doc-2.md
        ├── some-doc-3.md
        └── some-doc-4.md

The simple rule being: All content without a language identifier in the filename will fallback to the "root 's language".

@RickCogley any thoughts/objections?

No, that looks clean to me. I assume Hugo would be able to determine a file's translation status by comparing the file names, and matching ones that are the same from lang to lang?

Hugo would be able to determine a file's translation status by comparing the file names, and matching ones that are the same from lang to lang?

Yes.

It begs the question though, about bundled assets. I think currently you can put any language-shared images in the bundle, and access them from both files, say, index.md and index.en.md in my case, where index.md is Japanese. I'd have to copy the shared asset to the second bundle folder, no?

I'm not sure. But note that you don't have to use this and you can mix and match. This is mostly targeted at "big and content heavy" sites.

@RickCogley I have pondered a little about how to implement this, but now I think I know:
So, given this:
content
├── en
│   └── docs
│       ├── some-doc-1.md
│       ├── some-doc-2.md
│       ├── some-doc-3.md
│       └── some-doc-4.md
└── jp
    └── docs
        ├── some-doc-1.md
        ├── some-doc-2.md
        ├── some-doc-3.md
        └── some-doc-4.md

Will transformed become:
content
├── en
│   └── docs
│       ├── some-doc-1.md
│       ├── some-doc-2.md
│       ├── some-doc-3.md
│       └── some-doc-4.md
└── jp
    └── docs
        ├── some-doc-1.md
        ├── some-doc-2.md
        ├── some-doc-3.md
        └── some-doc-4.md

Will when looket at as "one tree" look like this:
content
└── docs
    ├── some-doc-1.jp.md
    ├── some-doc-1.md
    ├── some-doc-2.jp.md
    ├── some-doc-2.md
    ├── some-doc-3.jp.md
    ├── some-doc-3.md
    ├── some-doc-4.jp.md
    └── some-doc-4.md

Which would effectively solve all the shared bundle issues you mention.

@bep works the magic as usual 👍

@bep works the magic as usual 👍

@RickCogley I just tested the conceptual technical part and it looks beautiful. Note that I would not have thought of the simple solution without your input.

Would this improvment still allow still allow a folder structure in the form of:
en/
Index.MD
Page.MD
JP/
Index.MD
Page.MD
Scoping this out as a possible framework, our current setup uses the organization above. With 3,000+ source files * 6+ languages, a flat folder structure feels like a regression. Does Hugo support browser accept-languge switching our-of-the-box?

Would this improvment still allow still allow a folder structure in the form of

Yes, indeed. For general questions, see https://discourse.gohugo.io/

Yes, per my edit what is preventing this proposed folder structure now?

Yes, per my edit what is preventing this proposed folder structure now?

Read my edits.


Changed separated background-* declarations to background shortcut

Hi Twolfson,
the title of this comment says it all: I changed the sprite mixin in all style templates to output the background shortcut instead of background-image, background-position to save a couple lines of code. I chose to fork your repo, because I use it extensively in a bunch of projects via grunt-spritesmith, but as I said before I wanted to have the shortcut instead of separate declarations.
Maybe you want to merge my changes with your original json2css repo.
Best regards!
webastronaut

Deciding to close this issue since the shortcut requires a delcaration of no-repeat which some people might want and others might not (e.g. background sprites vs normal sprites).


'awaiting for primary endpoint to be ready' takes forever

it looks like patroni cannot be initilized. It might be some networking or other issue.
thus, charm is stuck in awaiting for primary endpoint to be ready status
another issue on top of this is that when juju emits on update status, then postgres charm sets the charm status to active with message Primary, which simply overrides patroni waiting message

CC: @marceloneppel
Just for the history, fixed internally as https://warthogs.atlassian.net/browse/DPE-789 on October 14.
Resolving as no longer topical.

@taurus-forever
it is not clear from Jira if the issue is fixed or not.
can you clarify on it?

You are right, the issue is not yet fixed.
My mistake. Reopened.

This has also happened to me on a clean installation but I'm afraid the issue is intermittent. The details of my env are the following:
MicroK8s v1.26.1 revision 4595
juju 2.9.38-ubuntu-amd64

Dear @arturo-seijas , thank you for bumping this up!
The issue had low priority due to random reproducibility. Searching for steps-to-reproduce reliably.
The priority revised to investigate it further the next pulse 23#6.
Is it constantly reproducible for you? Any specific actions to reproduce it?
I have fired new multipass machine and postgresql (revision 47) started properly:
> multipass launch -c8 -m8Gb -d30Gb charm-dev
...
> multipass shell charm-dev

> juju deploy postgresql-k8s --channel edge --trust
Located charm "postgresql-k8s" in charm-hub, revision 47
Deploying "postgresql-k8s" from charm-hub charm "postgresql-k8s", revision 47 in channel edge on jammy
...

> juju status
Model       Controller  Cloud/Region        Version  SLA          Timestamp
default38x  microk8s    microk8s/localhost  2.9.38   unsupported  11:56:36+01:00

App             Version  Status  Scale  Charm           Channel  Rev  Address         Exposed  Message
postgresql-k8s           active      1  postgresql-k8s  edge      47  <IP_ADDRESS>  no       Primary

Unit               Workload  Agent  Address     Ports  Message
postgresql-k8s/0*  active    idle   <IP_ADDRESS>         Primary

> snap list | grep 'microk8s\|juju'
juju        2.9.38         21790  2.9/stable     canonical**  classic
microk8s    v1.26.1        4595   1.26/stable    canonical**  classic

I will continue searching the way to reproduce the issue reliably and will share my findings.

Hi @taurus-forever,
No, haven't been able to reproduce it since, but I'll keep an eye on it too.
Thanks

I implemented a fix through https://github.com/canonical/postgresql-k8s-operator/pull/116 for the original situation reported by Maksim (he reported me on MM that something might got wrong with microk8s network, after that some pods died, but PG hasn't recovered).
@arturo-seijas, can you try revision 58 (the latest on edge) and see if you face the issue again? If so, can we get more details about how are you deploying?

I implemented a fix through #116 for the original situation reported by Maksim (he reported me on MM that something might got wrong with microk8s network, after that some pods died, but PG hasn't recovered).
@arturo-seijas, can you try revision 58 (the latest on edge) and see if you face the issue again? If so, can we get more details about how are you deploying? I tried revision 47 in an environment similar to yours (same Juju and microk8s version), but couldn't reproduce the error.

I haven't been able to reproduce the issue with revision 58. As far as I'm concerned, I think we can close the ticket. Thank you very much

@marceloneppel I am +1 for closing considering #116

Thanks @arturo-seijas and @beliaev-maksim!


docs: Fix simple typo, mousedowm -> mousedown

There is a small typo in awesomplete.js.
Should read mousedown rather than mousedowm.

Thanks!


:loudspeaker:  Ubuntu 18.04 hits end-of-life this month

Tweet:
https://www.omgubuntu.co.uk/2023/05/ubuntu-18-04-end-of-support

:ok_hand:

Closing this issue: there is nothing because #2931 has been rejected.


Create  READ ME FIRST: Admin Quick Start Guide.md

Overview
TL;DR

Questions
<If relevant, write a list of questions that you would like to discuss related to the changes that you have made.>
Next Steps
<If incomplete, create a task list of items that are still being worked on within the Pull Request.>
Review
<If complete, or ready for :eyes:, use @mentions for quick questions, specific feedback, and progress updates.>

readme giud


[OSCI] Update deprecated Babel plugins

Description
These changes update two deprecated Babel plugins in the OUI devDependencies list. The deprecated @babel/plugin-proposal-class-properties is replaced with @babel/plugin-transform-class-properties. Likewise, the deprecated @babel/plugin-proposal-object-rest-spread is replaced with @babel/plugin-transform-object-rest-spread.
After the changes, yarn build, yarn build-docs, and yarn test run successfully.
Issues Resolved
Fixes #1130
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[x] All tests pass

[x] yarn lint
[x] yarn test-unit

[x] Update CHANGELOG.md
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@JohnathonBowers I think you also need to update the comments and usage in the .babelrc file here:
https://github.com/opensearch-project/oui/blob/24592a68111ab47d511fc250ed18ef03ccac6f25/.babelrc.js#L34-L37

Thanks for pointing this out, @joshuarrrr ! I've made the changes in .babelrc.js and pushed a new commit.


Add link tracking to GovSpeak component

What
Option to track all links rendered within a GovSpeak component
Why
This will allow us to add custom tracking to content pages where tracking is required for reporting purposes.
https://trello.com/c/BHqdvxKg/688-set-up-analytics-for-briefing-video-page

Just read Alex's review and I agree with his concerns. As he said, could tracking not be achieved using the existing track-click.js script?

Just read Alex's review and I agree with his concerns. As he said, could tracking not be achieved using the existing track-click.js script?

Just read Alex's review and I agree with his concerns. As he said, could tracking not be achieved using the existing track-click.js script?

I tried that initially but the problem is that it starts tracking any clicks within the div (not just links) and it didn't doesn't allow support for using the labels as actions etc.

Just read Alex's review and I agree with his concerns. As he said, could tracking not be achieved using the existing track-click.js script?

I tried that initially but the problem is that it starts tracking any clicks within the div (not just links) and it didn't doesn't allow support for using the labels as actions etc.

Closing this to keep the changes within the app as this is not something we want to encourage GOVUK wide. New PR https://github.com/alphagov/government-frontend/pull/1966

Closing this to keep the changes within the app as this is not something we want to encourage GOVUK wide. New PR https://github.com/alphagov/government-frontend/pull/1966


Feature/2702 watch ext

All done I believe. Thank you for your review!

It looks like you changed it to minutes, not seconds.

you are right :) - fixed

Merged, thanks for pull request.


chore - remove archived BouncyCastle.Crypto and instead use latest nuget package (2.3.1) / Remove target net40 and add targets net461, net6, and net8

Ok


Missing emotes

Didn't see anything in changelogs about removal of specific emotes, but the "Flipping" emotes seem to have disappeared. Flipping Sexy was one of them, can't remember the name of the other.

I notice that too! Also California Gurls disappeared again ;c

This is so strange. I don't even feel like I touched the list of emotes last update. There's some supernatural stuff going on lol. Let me check on these when I get home


Broken communication

I have implemented the libosdp on my gateController (cp mode). Communication is being established using SCBk, the problem is after specific time (30 minutes) The cp stop sending poll request to the reader, and so I can not get any event-callback. After other 30 minutes the connection work again and the cp back to send poll request and functions fine.
I don't understand why cp stop communicate aftet 30 minutes expired and work again after othet 30 minutes?
The debug log doesn't show any error.

I ran into what seems like the same issue. In my case, I tracked it down to an overflow in osdp_millis_now/usec_now on some platforms (I am running on Raspberry Pi, where the fields of struct timeval and the constant 1000L are 32-bit). I was able to work around it by providing a definition of osdp_millis_now in my program and statically linking with libosdpstatic.a and libutils.a.
The problematic code is this line:
https://github.com/goToMain/c-utils/blob/9f2ea57abc8ed432b990a408d09468e6d4e62e2a/src/utils.c#L78
	usec = tv.tv_sec * 1000L * 1000L + tv.tv_usec;

@katsl, You are right, usec_now() can can overflow if struct timeval::tv_sec and struct timeval::tv_usec are only 32 bits wide. THB, I didn't know they could even be defined that way :). Thanks for letting us know.
Anyways, I have a fix for that situation by changing that line to:
usec = tv.tv_sec * 1000LL * 1000LL + tv.tv_usec;

@mazen2000, please check the most recent master and let us know if it fixes your issue.


error!

I download docker image, trained and ran it.
Only got an error!.
Could you figure out what went wrong?
Read Wiki Articles: 100%|███████████████████████████████████████████████████████████████████████████████████████████| 21427/21427 [02:16<00:00, 157.48it/s]
Completed!
Traceback (most recent call last):
File "machine.py", line 18, in
claf_machine = registry.get(f"machine:{machine_name}")(config)

File "/workspace/claf/claf/machine/open_qa.py", line 34, in init
self.load()

File "/workspace/claf/claf/machine/open_qa.py", line 55, in load
self.rc_experiment = self.make_module(reasoning_config.reading_comprehension)

File "/workspace/claf/claf/machine/base.py", line 64, in make_module
experiment = Experiment(Mode.PREDICT, experiment_config)

File "/workspace/claf/claf/learn/experiment.py", line 48, in init
self.load_setting()

File "/workspace/claf/claf/learn/experiment.py", line 89, in load_setting
cuda_devices, checkpoint_path, prev_cuda_device_id=prev_cuda_device_id

File "/workspace/claf/claf/learn/experiment.py", line 101, in _read_checkpoint
f"cuda:{prev_cuda_device_id}": f"cuda:{cuda_devices[0]}"

TypeError: 'NoneType' object is not subscriptable

To run claf_machine on a computer where cuda is available, please specify cuda_devices in the machine config file where an experiment type module configuration is defined(seems like you are using open_qa so this would be the reading_comprehension module), like:

Thanks, it finally works. But it gives other kind of errors. It seemed trade-off between memory overflow and max_sequence_size. The bigger the size, the more probable it gives memory overflow error. The smaller the size, the more probable, it gives size overflow error. What should I do?

I'm afraid there is no silver bullet for the tradeoff issue... 😥 The choice of max_sequence_size depends on the GPU (memory of the GPU) that you use. I think finding the optimum value by trial and error is the best way.

OK, it's no problem to set max_sequence_size smaller one. But how do I avoid doc. with more than max_sequence_size when choosing docs?

I think that when the document is long, splitting the document into paragraphs with smaller than max_sequence_size will produce better results.


Feature Request: Printing

When printing a Net Doc page, remove the left and right sidebars.

Not quite closed yet, need to do documents, passwords and processes.


Jumping scroll in IE 10/11 WIndows 7

Hello,
When you scroll in IE 10/11 (only on WIndows 7) it is scrolled with jumps (I mean scroll bar rail). Please see attached video: https://dl.dropboxusercontent.com/u/73896408/jumping scroll.avi
Maybe you could help with this problem or specify what the problem may be related with.
With best regards

Related with #160, and there is no really good solution for the time being. Some workarounds are suggested in the thread, so please have a look.
Sorry that it can't help so much :disappointed:


Support css animation classes mimic reactjs/react-transition-group v4

Addresses #35
This is the same React library’s latest version. They have changed to an approach that will work but does result in a breaking change.
The breaking change is ember-css-transition users are going to have to update their css. The difference is enter is added, then removed and enterActive is added.

closed in favor of https://github.com/peec/ember-css-transitions/pull/38


Fix dart usegpu.

@hcho3. Is there any way I can look into the stalled Windows executor?  Like whether it's truly failed allocation or some plugins gone wrong?

@trivialfis The EC2 plugin in Jenkins is to blame here. The plugin lost connection to the agent but somehow the agent still counted toward the instance cap.

Please let me know how do I help.

@trivialfis For now, you can log into the EC2 console (I gave you credential) and watch if there is a P2 instance running that's not showing up in https://xgboost-ci.net/computer/. If so, you should terminate it so that a new instance can be launched.
In the long run, we need something like https://github.com/apache/incubator-mxnet-ci/tree/master/services/jenkins-autoscaling, a Lambda function that queries the job queue in the Jenkins server and provisions EC2 workers. There are lots of moving parts here, but I'll try to get it done by end of this year.

@trivialfis https://github.com/pycontribs/jenkinsapi seems useful too

@hcho3 Huge thanks!  That will be great.  Seems to be quite a project by itself.  I need to learn some basic about operating Jenkins.

@trivialfis I decided to take small steps instead of making a big script in one go, since small scripts are much easier to test and debug.


Problem with .jshintignore

JSHINT not useing .jshintignore for excludeing directories.
And be very good if

if (!opt) opt = '.jshintrc';

Just for the record for anyone else that comes across this it is working for me with the following setup:
/
  gulpfile.js
  .jshintignore
  webroot/
    source/
      js/
        modified.third.party.lib.js

and .jshintignore having the following in it:
webroot/source/js/modified.third.party.lib.js

and my gulp task being:
gulp.task('lint', function () {
  gulp.src(jsSourceGlob)
    .pipe(jshint('.jshintrc'))
    .pipe(jshint.reporter('jshint-stylish'));
});

@joshstrange That's the ticket +1


Research android unit testing & writing unit tests

We should research on the topic to be able to write some unit tests for our project. A good place to start is the link below:
http://futurice.com/blog/android-unit-testing-in-android-studio-and-ci-environments

@bulenthacioglu

We should use Espresso instead of Robolectric. Espresso is official UI testing framework of Android. I have added an example UI test on current Android project.Also I have written how to run it from command line on READ.ME of Android project.


dts: Add led nodes to nxp boards, refactor mcux gpio driver to use dts

Based upon the work done in #5381, adds led and button gpio nodes to nxp boards. Refactors the mcux gpio driver to use device tree for base addresses, irqs, and device names.
Tested samples/basic/blinky on every board touched.
@pfalcon: Last time I mucked around with gpio device names I broke things for you (zephyr.js or micropython IIRC). I believe I preserved the names with this change, but please double check.

Codecov Report

Merging #7129 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7129      +/-   ##
==========================================
- Coverage   58.62%   58.61%   -0.01%
==========================================
  Files         464      464
  Lines       47440    47440
  Branches     8790     8790
==========================================
- Hits        27810    27809       -1
- Misses      15797    15798       +1
  Partials     3833     3833

Impacted Files
Coverage Δ

lib/posix/pthread.c
69.34% <0%> (-0.51%)
:arrow_down:

Continue to review full report at Codecov.

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


chore(papyrus_config): use get_packaget_dir instead of env var

Stack:

#2403
#2402
#2401
#2400
#2399 ⬅
#2398
#2395
#2394

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is


Fix YAML deserialization for Beaker::Platform

The version of Psych shipped with Ruby 1.9.3-p194 did not support
subclasses of String that added additional ivars to the String class.
The Beaker::Platform class does just that - subclassing String and
additional ivars - which meant that round tripping a Beaker::Platform
object would create an object with improperly initialized instance
variables. Psych commit e2fcf9af9e95535401f816bc893839b9ad743a9e
resolved that issue but we still use platforms that have the old version
of psych.
To resolve this issue, this commit implements a custom #init_with method
that explicitly sets all instance variables on the Beaker object that
were defined inside of the YAML map instance, and then reconstructs the
string value based on those fields.
/cc @er0ck

This should have some fairly exhaustive verification of the behavior across versions of Ruby, but I'm Pretty Confident (TM) that this should work correctly on all versions of Ruby that use Psych.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1842/
Test FAILed.

@er0ck could I get the magic jenkins incantation uttered to kick this build?

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1843/
Test PASSed.

opened a ticket for this: BKR-634
@adrienthebo quick work on this one!
if you have a second can you associate this PR/commits with the ticket above?
if not, i can probably just push to your branch...

FYI, the magic incantation is
'rete st
th1s
please'
or some parts of it, all one one line, speled correctly :-D

@er0ck I just updated the commit message to reference BKR-634, do you need anything else for this?

looks good to me!
i guess the PR tests kickoff isn't smart enough to know when it's just a commit msg change...

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1850/
Test PASSed.


Identitcal test passed in Mocha but fail in Jest

Demo repo:  https://github.com/ivawzh/firebase-jest
At the point https://github.com/ivawzh/firebase-jest/commit/a61cc0b70ab804a6f03923a313338d2f41cb82a6, Jest test fails and does not say which test unit/line the error comes from.
I got really desperate and implemented Mocha with the identical tests.
Mocha told me the beforeEach block in doSomething test unit is causing timeout. Hence I increased the timeout in test unit and got the problem solved https://github.com/ivawzh/firebase-jest/commit/366a50264834b2e1efdd4ddc17e197d19ffe97d8#diff-537bafeaa73dd43646d91171494af162R8.
And then I came back to Jest and tried the same trick of increase timeout. However, the problem still persists with Jest 😞
Now I have lost all my confidnece against Jest... Would you please tell me how I can fix this simple test in Jest?

I had a similar error, and found that setting the timeout to a really high value (around 60 seconds), and running the test multiple times results in it passing some of the time. Don't know if it's a cache issue, or some jsdom quirk, but it is pretty frustrating

@ivawzh, jest uses browser environment ad default. I had the same issue, until I added this in my package.json:
  "jest": {
    "testEnvironment": "node"
  }

and everything just worked =)

@fdaciuk The thing is, firebase should be able to run in both browser and node environments (it uses the browserify "browser" flag), so it could still need to be tested in the default browser environment. In my case,  firebase starts up fine, and I get a response from firebase.io; but trying to login/register or change any data fails with a network connection interrupted error.
I think the underlying cause is this: https://github.com/tmpvar/jsdom/issues/1195 Jsdom doesn't support websockets, which might be incorrectly detected by the firebase browser code; will investigate further

@TheAncientGoat got it, you right. In my case, I'm just using firebase-admin, so this configuration solved my problem xD

Jest uses jsdom to emulate browser env, so if websockets are needed for Firebase to work and they're not covered in the library, this may be problematic (it isn't for Mocha, because it can run in a browser).
You can try to mock the websocket somehow, but it certainly won't work for "real-world" testing you're trying to achieve. Please bump the jsdom issue mentioned by @TheAncientGoat.
Closing for now.


[1.19.2]: Cannot plant seeds on Terrestria Farmland

To reproduce:

Place Terrestria's Andisol Farmland
Attempt to plant the seed
The seed does not get planted

Other mods like Croptopia work fine, and regular farmland also works fine.
Versions:
Terrestria: 5.0.8
Farmer's Delight: 1.3.9
Fabric API: 0.76.0

Update: Cabbage seeds seem to work fine, maybe only tomato seeds are affected?

Hello @Martmists-GH,
Thanks you for report the issue, this is already done and it's a duplicate of (#30, #90 and #157). It will be fixed in next release normaly.

From what I can gather in the related issues it's caused by MoreTags, however I do not have the mod installed, nor does any mod include it as dependency.


Update from 6.2.16 to 6.2.17 results in poor performance

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[X] elastic.v6 (for Elasticsearch 6.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v2 (for Elasticsearch 1.x)
Please describe the expected behaviour
No change in request latencies when updating to 6.2.17.
Please describe the actual behaviour
Request latencies more than doubled.
Any steps to reproduce the behaviour?
I just updated to 6.2.17 and noticed that all requests (measured with a timer around bulkRequest.Do()) took more than twice as long. Looking at the ES metrics I could find no abnormal behaviour regarding latencies or rates, and looking at tracing of my client code did not reveal anything either. Switched back to 6.2.16 and everything looked fine again.
I'm curious if anyone else has experienced this.

Interesting. I cannot see why that is. Here's the list of changes between the two versions: https://github.com/olivere/elastic/compare/v6.2.16...v6.2.17
Although there were changes in the BulkService, they shouldn't have any impact on the performance. https://github.com/olivere/elastic/commit/07d32edbf0aca2e283726a15f8c2a663d3ae202e is the only change to the code. Other changes were purely cosemtic, such as changing the links to the documentation from version 6.2 to 6.7.

Yes, I looked through the commit log and couldn't find anything obvious either, so hopefully it's just something on my side, but I thought I would create an issue if anyone else have observed any similar problems. Below I've attached a screenshot from my environment during the time that 6.2.17 was deployed, and just switching back to .16 with no other changes restored the performance.

Are you using easyjson for that matter? I had to recreate the generated code for the serialiser die to the change of the Version type. Maybe we need to dig into that.

Are you using easyjson for that matter? I had to recreate the generated code for the serialiser due to the change of the Version type. Maybe we need to dig into that.

Interesting, yes, I'm setting UseEasyJSON on BulkIndexRequest, particularly because I experienced somewhat poor performance earlier (indexing perhaps unusually large bulks).

Hmm... yes, serialization in standard library is not the fastest in the world, but it works okay for the common use cases. Maybe then easyjson is the culprit here.

Perhaps, but this also affected search performance.

I disabled easyjson and tried to upgrade again and this time there was no performance difference compared with 6.2.16 👍 Why it affected search when I only set it on indexing requests I have yet to figure out, but that could likely be something strange with my code.

Thank you for your quick response and help, this can be closed if you have no reason to keep it open 👍

Sure. Happy to help.


Ignore feature on models for creating routes

As a developer, I should be able to hide a model in auto-created routes.
import { Model } from "axe-api"

class User extends Model {
  get ignore() {
    return true
  }
}

export default User

Obviously, we can use the handlers method to manage this. But the ignore() method would be easier.

Client should be able to use those models in related queries.

Howdy @ozziest If you don't mind me to step-up on this issue? If you haven't assign this ToDo issue?

Hi @vaish567, welcome to the board. 🥳 The issue is yours.
We haven't created good documentation yet but you can use dev-api repository for your development environment. Also, if you have any questions please let me know. If you can create something about it but you may not be sure, please open a Draft PR, so I can review it to help you.
Thank you very much for your help. 🥇

Hello @ozziest Should I use model in Model.js ?

As a developer, I should be able to hide a model in auto-created routes.
import { Model } from "axe-api"

class User extends Model {
  get ignore() {
    return true
  }
}

Hello @vaish567, yes. But the default value should be false in Model.js. If the developer wants they can make it true n their models.


Add luhn exercise

/cc @canweriotnow do you have cycles to review this?

I'd say drop the unnecessary %1 %2 for sure and choose none/some of the other style preferences and this looks good to me. @canweriotnow?


[WFMP-157] Upgrade wildfly-channel 1.0.0.Alpha14

JIRA: https://issues.redhat.com/browse/WFMP-157
Signed-off-by: Jeff Mesnil<EMAIL_ADDRESS>

This is related to https://github.com/wildfly-extras/wildfly-channel/issues/56


Add WorkQueue debugging for ACDC - wmagent branch

1.1.8_wmagent version of #8260

These debugging lines have been merged into 1.1.8 branch (went in in tag 1.1.8.patch2), so I think we can close this PR.


Update to allow for better handling of multiple schema uris per tag uri

Fixes a small bug which prevents validation of a tag which has more than one schema associated with it, but which does not have a loaded converter extension.

Hi @WilliamJamieson,
do you have an example at hand where one tag is associated with more than one schema?
I would like to look more into how and where to apply that concept.

@CagtayFabry there is a contrived example here in the tests.
The asdf-format/asdf-transform-schemas#40 PR was originally intended to also demonstrate this feature; however, we found some bugs in asdf (one of which is fixed by this PR) which prevented us from implementing examples in the schema files. These bugs were entirely within the testing infrastructure, and not with the actual implementation.
The multiple schema uris per tag is intended to help us tame the large number of new schemas in the asdf-transform-schemas package that results from changing the transform-1.x.0 schema and cut down on the duplication of otherwise identical schemas. In asdf-format/asdf-transform-schemas#40, we are introducing schemas for cos, tan, arcsine, arccos, and arctan based models. All of these models use the same parameter sets as the sin transform just swapping the trigonometric function involved (note that the inverse-trigonometric functions implement the parameters so that they are exactly the inverse of their trigonometric counterpart). This means that essentially the sine1d, cosine1d, tangent1d, arcsine1d, arccosine1d, and arctangent1d schemas are identical save for their tags. The multiple schemas per uri would let one instead define one trigonometric1d schema as follows:
%YAML 1.1
---
$schema: "http://stsci.edu/schemas/yaml-schema/draft-01"
id: "http://stsci.edu/schemas/asdf/transform/trigonometric1d-1.0.0"
title: >
  One dimensional trigonometric model.

description: >
  One dimensional trigonometric.

type: object
properties:
  amplitude:
    anyOf:
      - $ref: "../unit/quantity-1.1.0"
      - type: number
    description: Oscillation amplitude.
  frequency:
    anyOf:
      - $ref: "../unit/quantity-1.1.0"
      - type: number
    description: Oscillation frequency.
  phase:
    anyOf:
      - $ref: "../unit/quantity-1.1.0"
      - type: number
    description: Oscillation phase.

required: ['amplitude', 'frequency', 'phase']
...

Then one can define the tags for each of the trigonometric transforms in the manifest using:
- tag_uri: tag:stsci.edu:asdf/transform/sine1d-1.0.0
  schema_uri:
    - http://stsci.edu/schemas/asdf/transform/transform-1.2.0
    - http://stsci.edu/schemas/asdf/transform/trigonometric1d-1.0.0
  title: One dimensional sine model.
  description: |-
    One dimensional sine.
- tag_uri: tag:stsci.edu:asdf/transform/cosine1d-1.0.0
  schema_uri:
    - http://stsci.edu/schemas/asdf/transform/transform-1.2.0
    - http://stsci.edu/schemas/asdf/transform/trigonometric1d-1.0.0
  title: One dimensional cosine model.
  description: |-
    One dimensional cosine.
- tag_uri: tag:stsci.edu:asdf/transform/tangent1d-1.0.0
  schema_uri:
    - http://stsci.edu/schemas/asdf/transform/transform-1.2.0
    - http://stsci.edu/schemas/asdf/transform/trigonometric1d-1.0.0
  title: One dimensional tangent model.
  description: |-
    One dimensional tangent.
- tag_uri: tag:stsci.edu:asdf/transform/arcsine1d-1.0.0
  schema_uri:
    - http://stsci.edu/schemas/asdf/transform/transform-1.2.0
    - http://stsci.edu/schemas/asdf/transform/trigonometric1d-1.0.0
  title: One dimensional arcsine model.
  description: |-
    One dimensional arcsine.
- tag_uri: tag:stsci.edu:asdf/transform/arccosine1d-1.0.0
  schema_uri:
    - http://stsci.edu/schemas/asdf/transform/transform-1.2.0
    - http://stsci.edu/schemas/asdf/transform/trigonometric1d-1.0.0
  title: One dimensional arccosine model.
  description: |-
    One dimensional arccosine.
- tag_uri: tag:stsci.edu:asdf/transform/arctangent1d-1.0.0
   schema_uri:
    - http://stsci.edu/schemas/asdf/transform/transform-1.2.0
    - http://stsci.edu/schemas/asdf/transform/trigonometric1d-1.0.0
  title: One dimensional arctangent model.
  description: |-
    One dimensional arctangent.

Note that by doing this we do not need to add schemas for any of the sine1d, cosine1d, etc.
Moreover, note that trigonometric1d-1.0.0 does not have an allOf between transform-1.2.0 and its additional properties, this is instead handled by the manifest because it implies an allOf between all of the schema uris listed in the entry. This means that if a new transform schema were to be needed, we could simply update the manifest instead of essentially duplicating trigonometric1d with only the transform version updated. We plan to extend this usage to the rest of the transform schemas.

Thank you for that thorough write-up @WilliamJamieson, makes it much clearer!
The implied allOf is probably not what I have in most use cases but certainly good to know this pattern for the future.

The implied allOf is probably not what I have in most use cases but certainly good to know this pattern for the future.

Please feel free to suggest additional methods of combining the schemas into the tag. If there is a good use-case for other methods of combining them please let me know. It should be relatively simple to add a control property to each manifest entry to signal other modes of combining them (getting it to work on the backend is a different matter).

The implied allOf is probably not what I have in most use cases but certainly good to know this pattern for the future.

Please feel free to suggest additional methods of combining the schemas into the tag. If there is a good use-case for other methods of combining them please let me know. It should be relatively simple to add a control property to each manifest entry to signal other modes of combining them (getting it to work on the backend is a different matter).

I'll take some time and look into it and let you know, thank you 👍


Write good docstrings for each class / method

Classes should provide an overview of what the particular buildpack does, methods should explain the reasoning for why it does what it does.

oops! closing in lieu of #211


feature/CLS2-680-include-global-account-manager-in-investment-project-index

Description of change
Add the one_list_group_global_account_manager field to the investment project search index. This field comes from the investor company or its global headquarters company the project is assigned to.
The opensearch index for investment projects is reindex when a company one list account owner is updated.
Test instructions
You should be able to see the additional field one_list_group_global_account_manager when querying the Investment project on open search
E.g. http://localhost:8000/v3/search/investment_project
Sample field structure:
------>8------
"one_list_group_global_account_manager": {
    "name": "Barry Oling",
    "last_name": "Oling",
    "id": "a80ff5fd-8904-4940-bf96-fe8047e34be5",
    "first_name": "Barry"
},
------8<------

There are two cases for investment projects belong to:

Investor company

Create/select an investment project for a company.
Assign it an One list account owner through the admin.
This should now be listed in the one_list_group_global_account_manager field when queried through open search.

Global headquarters company

Create/select a subsidiary company.
Create/select an investment project for the subsidiary company.
Assign another company as its Global headquarters.
For the Global headquarters company assign an One list account owner through the admin.
The investment project of the subsidiary company should have the One list account owner assigned to the Global headquarters as the one_list_group_global_account_manager value when queried through open search.

Checklist

[x] Has this branch been rebased on top of the current main branch?

Explanation
The branch should not be stale or have conflicts at the time reviews are requested.

[x] Is the CircleCI build passing?

General points

Other things to check

Make sure fixtures/test_data.yaml is maintained when updating models
Consider the admin site when making changes to models
Use select-/prefetch-related field lists in views and search apps, and update them when fields are added
Make sure the README is updated e.g. when adding new environment variables

See docs/CONTRIBUTING.md for more guidelines.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (383f0c8) 97.01% compared to head (f0e7444) 97.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5126      +/-   ##
==========================================
- Coverage   97.01%   97.00%   -0.01%
==========================================
  Files         893      894       +1
  Lines       21214    21257      +43
  Branches     1892     1897       +5
==========================================
+ Hits        20580    20621      +41
- Misses        497      498       +1
- Partials      137      138       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Local gulp not found in ~/dev

konard@konard-MS-7982:~/Desktop/dev$ npm run packages;

><EMAIL_ADDRESS>packages
> (npx gulp packages:get) && (npx gulp packages:ci)

Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated
npm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 does not receive security updates since 2019. Upgrade to chokidar 3 with 15x fewer dependencies
[20:12:02] Failed to load external module ts-node/register
[20:12:02] Error: Cannot find module 'ts-node/register' from '/home/konard/Desktop/dev'
[20:12:02] Failed to load external module typescript-node/register
[20:12:02] Error: Cannot find module 'typescript-node/register' from '/home/konard/Desktop/dev'
[20:12:02] Failed to load external module typescript-register
[20:12:02] Error: Cannot find module 'typescript-register' from '/home/konard/Desktop/dev'
[20:12:02] Failed to load external module typescript-require
[20:12:02] Error: Cannot find module 'typescript-require' from '/home/konard/Desktop/dev'
[20:12:02] Failed to load external module sucrase/register/ts
[20:12:02] Error: Cannot find module 'sucrase/register/ts' from '/home/konard/Desktop/dev'
[20:12:02] Failed to load external module @babel/register
[20:12:02] Error: Cannot find module '@babel/register' from '/home/konard/Desktop/dev'
[20:12:02] Local gulp not found in ~/Desktop/dev
[20:12:02] Try running: npm install gulp

konard@konard-MS-7982:~/Desktop/dev$ npm run packages;

><EMAIL_ADDRESS>packages
> (npx gulp packages:get) && (npx gulp packages:ci)

[20:13:25] Failed to load external module ts-node/register
[20:13:25] Error: Cannot find module 'ts-node/register' from '/home/konard/Desktop/dev'
[20:13:25] Failed to load external module typescript-node/register
[20:13:25] Error: Cannot find module 'typescript-node/register' from '/home/konard/Desktop/dev'
[20:13:25] Failed to load external module typescript-register
[20:13:25] Error: Cannot find module 'typescript-register' from '/home/konard/Desktop/dev'
[20:13:25] Failed to load external module typescript-require
[20:13:25] Error: Cannot find module 'typescript-require' from '/home/konard/Desktop/dev'
[20:13:25] Failed to load external module sucrase/register/ts
[20:13:25] Error: Cannot find module 'sucrase/register/ts' from '/home/konard/Desktop/dev'
[20:13:25] Failed to load external module @babel/register
[20:13:25] Error: Cannot find module '@babel/register' from '/home/konard/Desktop/dev'
[20:13:25] Local gulp not found in ~/Desktop/dev
[20:13:25] Try running: npm install gulp

Workaround
npm install gulp


Safari Technology Preview supports WebGPU

It shipped in Safari Technology Preview 190, in March 2024.

https://developer.apple.com/documentation/safari-technology-preview-release-notes/stp-release-190#WebGPU


No CFG involved in the training process?

Hi OpenSora Team,
I would like to extend my gratitude for your open-source contributions to the AIGC field. While researching your codebase, I noticed that OpenSora defaults to using cfg=5.5 during the inference stage, which suggests that it might be optimal to drop the input text prompt (or zero it out) with a certain probability during training. However, I have not been able to locate the code responsible for dropping the text in the current training codebase. If I have overlooked something, please kindly correct me. Thank you for your assistance.

Great, it was an oversight. We will add it in the next version.


Getting weird formatting for my dates

Did you use * { box-sizing: border-box } ? It will remove the padding/margin on datestring which puts it in the correct position.

I think the problem is likely due to box-sizing: border-box
Maybe related to this issue : #3

hello. I encountered this issue as well. I am glad to find this thread.
I am using material-ui-react and I am using the CssBaseline component. (https://material-ui.com/style/css-baseline/)
afaik this component applies box-sizing: border-box by default, as such, I had to edit the minified css file to fix the positioning of the dates. I notice that the VerticalTimelineElement is also slightly off the vertical center. This seems like an issue. I would love a prop to deal with this.
Thanks for the awesome package. Honestly, well done.


editor.renderLineHighlight: "line" - doesn't work anymore

VSCode Version: 1.12.1
OS Version: Windows 10

Option editor.renderLineHighlight: "line" is not working anymore. The current line is no longer highlighted with a border, it is only highlighted with a background - it's like having the "all" option enabled.

@MariusVatasoiu what theme are you using?

Now I use Dark+ Material, but the problem also appears on other themes downloaded from the marketplace. Somehow for VS Code default themes, for example Dark (Visual Studio), this option works correctly.

@MariusVatasoiu I couldn't find the Dark+ Material theme on the marketplace, what is its exact name?
I've tried with One Dark Pro and it works for me:

@MariusVatasoiu if you can provide me the link on marketplace of your theme then I can have a test. It seems the theme doesn't handle the background/border explicitly.

@rebornix This is the link:
https://marketplace.visualstudio.com/items?itemName=vangware.dark-plus-material
In "Dark+ Material" is like this:

And in "Dark (Visual Studio)" is like this:

Before the update, the "line" option worked as in the second image (with border).
@alexandrudima Your example works like mine, but I want the current line to be highlighted with a border, not with a background.

@MariusVatasoiu the reason that you saw this behavior is Dark+ Material added this explicitly https://github.com/vangware/dark-plus-material/commit/2151a043f6d0e36e243a636a90988b8003d7cf10#diff-f2dfccb8f7b770ba7f97b22c2d98b6bbR21 to make the current line has a background. Please file issues there as that's the purpose of that line of code.


[Meru AB] Dust app tweaks for suggestions

changed prompt to determine if instructions are finished or not
moved names to gpt3.5
asked to provide the instructions suggestions as questions
a few tweaks / fixes in instructions

Risk

Deploy Plan

Uncontroversial (only app spec change, gated), merging


Correct location of read scale-out setting

@rajeshsetlem

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

This PR has been included in an internal PR that also updates the article. This PR will automatically be closed when the internal PR is merged.


Add comment about user existence

I added a comment about things in the thing. Mostly a test. @DonnyLi

Ayyy lmao :shipit:


Missing Floating UI Size Middleware

@awcodes - let me just start by saying that this library is excellent! It works just as expected and is very much appreciated.
I'm working on an implementation and am trying to match the size of the dropdown to the reference width using the size middleware (https://floating-ui.com/docs/size). It appears, however, that this library doesn't support the size middleware at all yet. Are there plans on your end for implementing it? If you have no plans to do so I may be able to take a shot at it in the next few weeks.

I didn't see the point of it, but I'll see what I can do. Thanks.

Thanks @awcodes! My case was Matching Reference Width, which would be difficult to do outside of the plugin.

Resolved by #4

This has been released in v3.5.0


2 Bar and 2 Line chart are not aligning with each other.

Expected behavior
Code block
const config = {
  type: 'bar',
  data: data,
  options: {
    scales: {
      y: {
        beginAtZero: true
      }
    }
  },
};

const labels = Utils.months({count: 7});
const data = {
  labels: labels,
  datasets: [
  {
    label: '1',
    type:'line',
    data: [65, 9, 80, 81, 56, 55, 40],
    backgroundColor:'rgba(255, 99, 132)',
    borderColor: 'rgb(255, 99, 132)',
    borderWidth: 1,
      yAxisID: 'y',
      stack:'a'
  },
  {
    type:'line',
    label: '2',
    data: [24, 15, 88, 34, 23, 44, 66],
    backgroundColor:'rgba(255, 199, 132)',
    borderColor: 'rgb(255, 199, 132)',
    borderWidth: 1,
      stack:'a'
  },
  {
    label: '3',
    data: [34, 13, 65, 34, 76, 24, 23],
    backgroundColor:'rgb(0, 199, 132)',
    borderColor: 'rgb(0, 199, 132)',
    borderWidth: 1,
    pointStyle:'line',
      yAxisID: 'y'
  },
  {
    label: '4',
    data: [67, 56, 58, 65, 97, 65, 33],
    backgroundColor:'rgb(100, 22, 255)',
    borderColor: 'rgb(100, 22, 255)',
    borderWidth: 1
  }
  ]
};

The line and bar to be in same line.

Current behavior
Line dots fall between the two bars.

Reproducible sample
https://codepen.io/topratiksharma/pen/wvNOxBK
Optional extra steps/info to reproduce
https://codepen.io/topratiksharma/pen/wvNOxBK
Possible solution
No response
Context
No response
chart.js version
v4.4.1
Browser name and version
edge
Link to your project
No response

This is working as expected because there are multiple bars. The position of the dots is solely based on the X axis and the bars don't factor into that calculation

Can we align it by some way ?

As @etimberg outlined, this is default behavior. You can use a custom plugin to do that.
See in SO: https://stackoverflow.com/questions/76060933/aligning-linechart-with-grouped-barchart-using-chart-js/76061773#76061773


Added a Benchmark for Rotary and Improved Rotary Performance

changed rotary block tiling config resulting in 15% speed up
added a benchmarking script to test rotary performance

Apologies. Meant to pull this into a forked repo.


Impliment Skip Fusion

I need to actually compile (?:a|b)(?:c|d) down to skip 2

This should be done at the AST level I think.


pip3 install get's stuck

Hey,
This project looks really exciting and kudos on the great work! I am trying to install the pymagnitude package, using the command pip3 install pymagnitude, but it get's stuck after downloading and doesn't proceed to install. I supplied the verbose flag with the command to see what it's getting stuck at and turn's out, it's getting stuck at:
Running setup.py (path:/tmp/pip-build-60o8i7g_/pymagnitude/setup.py) egg_info for package pymagnitude
Running command python setup.py egg_info

I tried installing it in a fresh virtual environment just to ensure that it was not the fault of my environment, but it did not work there too. Is it actually getting stuck or am I just being impatient and the package take 5+ minutes to install?
For your reference, I am running Python 3.6.5 on an Ubuntu 16.04
Any help would be appreciated. Thank you!

Just released version 0.1.46. You can now disable downloading the wheel and force it to compile instead by running this command in your shell instead:
SKIP_MAGNITUDE_WHEEL=1 pip3 install pymagnitude==0.1.46 -vvvv

I think the downloading and installing of the wheel is just taking extra long on your computer / network, but if you really need to skip it, try running this command.
Note: It is actually slower than installing a wheel since it will have to run GCC to compile some large C code instead of using a pre-built binary. But it seems like the bottleneck on your computer is downloading the wheel.

This did the job for me. Thank you so much for your prompt response!


Question: did you want to implement sqlite driver for golang with pure golang ?No gcc ?

Question: did you want to implement sqlite driver for golang with pure golang ?No gcc ?
Is it true?

This project is some infrastructure which may be used in the backend of the service.


MINI-IMDB

Description
IT IS A MINI-IMDB PROJECT THAT PROVIDES INFORMATION ABOUT ANY MOVIE OR TV SERIES WHEN SEARCHED.
This project is coded in python language and uses basics of python data structure - lists, OOPS and some basics of database.
Do remember to read the documentation at the top of the source code for smooth learning.
Fixes #(issue_no)
No issues related to this repository is fixed. Tried to add a new project that copy the data from IMDB database.
Type of change
Choosing one or more options from the following as per the nature of your Pull request.

NOTE: These boxes can be checked using [X]

[ ] Bug fix (non-breaking change which fixes an issue)

[ @] New feature (non-breaking change which adds functionality)

[ ] Documentation Update

Checklist:
Please tick all the boxes that are fulfilled by your Pull Request.

[ ] I have named my files and folder, according to this project's guidelines.
[x] My code follows the style guidelines of this project.
[ @] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[ @] I have commented on my code, particularly in hard-to-understand areas.
[ @] I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
[ ] I have included a requirements.txt file (if external libraries are required.)
[x] My changes do not produce any warnings.
[x] I have added a working sample/screenshot of the script.

Sorry for the @ signs. I am a beginner so I wasn't aware of how to tick the boxes.

Sorry for the @ signs. I am a beginner so I wasn't aware of how to tick the boxes.

This kind of script already exists in our collection


AddressAttribute: validate each field separately

Give more significant error message
Choose which field is required

This looks like a great start, but since there's no way this will merge cleanly with develop we're going to be doing a lot of work that we essentially have to re-do over again in develop. I'm going to close this; if you'd like to re-open this against develop we'd look to merge it there.


Delete exercise does not work in case plagiarism cases exist

Describe the bug
When you delete an exercise (e.g. in an exam) for which plagiarism cases (potentially with posts) exist, the deletion does not work due to database constraints.
To Reproduce

Create a text exercise with 2 larger identical submissions
Run plagiarism checks and accept the found plagiarism
Make sure a plagiarism case was created
Delete the exercise --> Internal Server Error

Expected behavior
Deleting the exercise in such a case should be possible.
All associated plagiarism cases (and posts) should be delete beforehand
Screenshots
No response
What browsers are you seeing the problem on?
Chrome
Additional context
No response
Relevant log output
No response

@krusche I'm not able to reproduce this anymore. Here is the video with the flow I follow: https://www.loom.com/share/ab028925577d4bb394ede1326edd74d6.

Do I follow the correct flow?
If yes: Do you agree that the issue is solved by now?
If no: What should I perform to reproduce that?

The flow seems correct.
Entities related to plagiarism get deleted by cascading:

the server deletes PlagiarismResult first. This by cascading removes PlagiarismComparison:

https://github.com/ls1intum/Artemis/blob/9504ec832d1e81cf6e4afb0b61369ba575cedce1/src/main/java/de/tum/in/www1/artemis/service/ExerciseDeletionService.java#L139
https://github.com/ls1intum/Artemis/blob/9504ec832d1e81cf6e4afb0b61369ba575cedce1/src/main/java/de/tum/in/www1/artemis/domain/plagiarism/PlagiarismResult.java#L35

then the server removes the Exercise which by cascading removes all PlagiarismCase:

https://github.com/ls1intum/Artemis/blob/9504ec832d1e81cf6e4afb0b61369ba575cedce1/src/main/java/de/tum/in/www1/artemis/service/ExerciseDeletionService.java#L139
https://github.com/ls1intum/Artemis/blob/9504ec832d1e81cf6e4afb0b61369ba575cedce1/src/main/java/de/tum/in/www1/artemis/domain/Exercise.java#L138

Interestingly, neither of those lines was edited after this issue was created. But, manual testing and code analysis show that the anticipated flow works.


Update the way of decaying the learning rate

Avoid the problem of training speed become slower iteration by iteration mentioned in this issue

In your code learning rate is going to be changed just once, but don't you think that a better approach is to allow multiple corrections of learning rate. Please check my commit c018d838d3b3ee2a4770f60fdfb44e754e8290bd

No it will decay every cfg.TRAIN.STEPSIZE steps with a base of 0.1:
You can get more details here

Yes, you are right. I have been testing the wrong code :(

This commit is right. The training becomes very slower on my machine.


Refactor /health route

Summary
Change /health route behaviors to fit more the common use.
Related to this discussion.
Motivations
health route in meilisearch was used to inform about service maintainances, which is not the idomatic use of it.
The health route should only inform about the availability of meilisearch.
Explanation
the health route:

NEEDS to ensure that service is completely up if it returns 201
DOES NOT block service process (write transaction)
NEEDS to use the less network bandwidth as possible when it's triggered
NEEDS to use the less service resources as possible when it's triggered
DOES NOT NEED any authentication
MAY be named /health

Implementation
A trivial approach would be make /health route always return 201
a more complex approach would be to enumerate all checks we need to do to consider meilisearch as available/healthy.
set_healthy and set_unhealthy should be removed
source code to refactor here
test code to adapt here
Related

[ ] Validated by the team
[ ] Test added
[ ] Documentation TBD
[ ] SDK/Integrations TBD

Hello! Why does this route returns a 201? 201 is used when a resource is created.

is used when a resource is created

I agree, I think we should use 200

sorry for the typo, I wanted 204

Hello @LegendreM Can I please work on this ticket as an opensource contribution ?

Hey @AlyHKafoury, yes don't hesitate!

hi @AlyHKafoury , are you still working on this? if not i may pick this up.

What is the final decision about this issue:

A trivial approach would be make /health route always return 204

a more complex approach would be to enumerate all checks we need to do to consider meilisearch as available/healthy.

I guess the trivial approach is chosen, can you confirm?
Summary
If I understand well:

update health route is removed
get health route will always return a 204 if we success to reach the MeiliSearch.

Do I forget something?

Yep @curquiza  the trivial approach is chosen because we don't have any sanity check to do during run time. If meilisearch as an hard failure the server doesn't run and health route doesn't respond.

Thanks for the confirmation! Good to me then 🚀

What are possible error responses from health route in case it is not a '204' ( responses with a Meilisearch error body )? If some exists of course.
Can we receive the maintenance 'errorCode'?


postal dont send email

After the Postel installation process
I tried sending an email
i have this problem (SOFT FAIL)
https://prnt.sc/17hfexc

That message usually means that your server provider has blocked outgoing emails. Are you able to talk to them to confirm whether you can get it unblocked?


Remove BUILDFIXED and MAKEFIXED.

Remove BUILDFIXED support.
Split out MAKEFIXED into a separate 'makefixed' util that is easy to use if we want to regenerate/verify inffixed.h

This builds on top of PR/374 and includes that commit, so please review only the second commit here.

Fixed indentation error in makefixed.c


An implementation of IDiagnosticContext from Serilog.Extensions.Hosting

Since SerilogTracing will frequently fit the role otherwise played by UseSerilogRequestLogging() from Serilog.AspNetCore, it would be nice if the IDiagnosticContext interface, commonly used with that setup, could be implemented atop System.Diagnostics.Activity when using SerilogTracing.
It seems like AddProperty() and SetException() from that interface would map naturally onto the model here; the trick I think would be to reliably identify the root web-request-handling activity and enrich that, rather than simply add details to Activity.Current.

For now, we'll leave this alone. Using SerilogTracing, tagging the current activity is probably sufficient:
Activity.Current?.SetTag("A", 42);

There could be some value in an activity-based implementation of IDiagnosticContext within Serilog.Extensions.Hosting, but that's not in this project - a change would likely only be made there if/when there's some activity handling support in Serilog itself.


Known hacks

This issue will contain a list of known hacks to be removed.

Retrieving the address for env_map data pointer for shinyness of objects is a hack on PSX.


Add support for installing / updating extensions

Auditors: @darkdh @bridiver

@darkdh @bridiver this is ready for another review.  You'll notice I didn't end up needing to uninstall to upgrade, so this adds some code for uninstalling that is not currently used, but I think it's useful to keep anyway.

++


docs: add a comment for the cw721 example's callable points

This PR adds a comment for the cw721 example's callable points.
This comment is for telling these callable points do not cover all of cw721 query and messages.

Could you add anything to the DESCRIPTION?

fixed


Add pr merge doc updater

Add an AI sublayer_documentation updater github action.
Runs when a PR is merged to Sublayer main.

Been thinking about these github-related sublayer actions and how they're different from the ones in the sublayer_actions repo and the ones used elsewhere. I do think we should standardize on doing these things one particular way where we use the octokit api rather than shelling out directly to the filesystem.
I think what we probably want to do is package that repo up as a gem and then have the ability to require different sub-sections of the actions for what you're trying to do...so something like:
gem "sublayer"
gem "sublayer_actions/github"

or
require "sublayer_actions/github"

I find myself copy/pasting these files all over the place and finding a bug in one means having to figure out all the places we've done it that way in the past...

and actually thinking...maybe a pattern similar to what devise offers...where you can rely on the controllers/views/etc if you want to use the defaults, but if you want to change them, it'll write the files out for you in your project for you to customize...


Associating with an ancestor creates a rolling change set

$ git log --oneline
b93138d Remove changelog
6610881 Remove contributing
b97c669 Add changelog
11fd99b Add contributing
fcc8655 Initial commit
$ git changes
b93138d Remove changelog
6610881 Remove contributing
$ git changes associate HEAD^
fix-broken-stuff has been associated with HEAD^
$ git changes
b93138d Remove changelog
$ git changes associate
HEAD^
$ touch file.txt
$ git add -A
$ git commit -m "Add file"
[fix-broken-stuff 808ff4b] Add file
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 file.txt
$ git changes
808ff4b Add file

The expected behavior would result in:
$ git changes
808ff4b Add file
b93138d Remove changelog

Having a rolling change window could be useful. There just needs to be a way for toggling keeping an ancestor reference as-is or resolving it.

Creating a rolling change set really doesn't make sense. So it won't be supported.


Updating vertical scrollbar.

Describe your chaotic idea
Hey @vansh-codes ,
I  want to update the vertical scrollbar.

Can you please assign me this issue ?
Add Screenshots or Sketches

Confirm

[X] I understand that my idea will add to the chaos and I'm okay with it
[X] I agree to follow this project's Code of Conduct
[X] I want to work on this feature
[X] I'm GSSOC'24 Extd registered Contributor
[ ] I'm Hacktoberfest registered Contributor

Go ahead!


Check if req.body has been parsed already

When used with express/any other framework that does body-parsing, the web hook doesn't work any longer. Added a block to check if the body has already been parsed and then skip the whole buffer/stream stuff and calculate the signature from JSON.

Looks like your editor may have done a heavy-handed reformat of the file, would you mind updating to just the bits that have changed because it's not clear to me on first glance what's been materially changed. Also, would you mind thinking up a test case for this, even if it has to involve importing Express as a devDependency, as I don't use Express myself I don't run into this problem so it would be good to have test coverage for it. Cheers!

yeah, beautify-on-safe, sorry for that. I'll re-think the solution as it's not really the nicest way to do that but quick & dirty.

actually, tests 21 (handler accepts a signed blob with alt event) and 17 (handler accepts a signed blob) are failing. Once they're fixed I can create a new branch for the body-parser implementation with tests.


Unexpected behaviour when mouting a volume containing a symbolic link

Hi
How to reproduce:

Host command: echo 'host' > /tmp/a
Spin up a empty/minimal docker image: docker run -it -v /home/test:/test some:image /bin/bash
Container command: echo 'container' > /tmp/a
Container command: ln -s /tmp/a /test/a
Exit container command: Ctrl+p, Ctrl+q
Host command: more /home/test/a

The result:

The echo of the file /home/test/a returns 'host'

The expected result:

I would expected that the file /home/test/a returns 'container'.

Thoughts:

Seems counter-intuitive that a mounted symbolic link in a container is resolved to the host file system instead of begin resolved within the the file system of the the container only.
This can produce unforeseen behaviors, as one might perform changes on a symbolic linked file in hope of changes in the container images, but in reality the changes are only performed on the host.

Is this an expected behavior?
Running: Centos7, Docker 1.6.0/8aae715, Physical box
Containers: 6
Images: 110
Storage Driver: devicemapper
 Pool Name: docker-253:19-1191-pool
 Pool Blocksize: 65.54 kB
 Backing Filesystem: extfs
 Data file: /dev/loop0
 Metadata file: /dev/loop1
 Data Space Used: 2.721 GB
 Data Space Total: 107.4 GB
 Data Space Available: 2.27 GB
 Metadata Space Used: 5.64 MB
 Metadata Space Total: 2.147 GB
 Metadata Space Available: 2.142 GB
 Udev Sync Supported: true
 Data loop file: /var/lib/docker/devicemapper/devicemapper/data
 Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
 Library Version: 1.02.93-RHEL7 (2015-01-28)
Execution Driver: native-0.2
Kernel Version: 3.10.0-229.4.2.el7.x86_64
Operating System: CentOS Linux 7 (Core)
CPUs: 2
Total Memory: 7.698 GiB
Name:
ID:

Client version: 1.6.0
Client API version: 1.18
Go version (client): go1.4.2
Git commit (client): 8aae715/1.6.0
OS/Arch (client): linux/amd64
Server version: 1.6.0
Server API version: 1.18
Go version (server): go1.4.2
Git commit (server): 8aae715/1.6.0
OS/Arch (server): linux/amd64

Linux alpha 3.10.0-229.4.2.el7.x86_64 #1 SMP Wed May 13 10:06:09 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Is this an expected behavior?

Yes it is.
Symbolic links are resolved at the time of their lookup, and in the context of their path at the time of lookup.

Still seems a bit counter-intuitive that symbolic links can cross the host/container boundary. Can imagine a lot of nasty stuff and guess I might not be the only not knowing about this behavior.

Can imagine a lot of nasty stuff from this behavior

Such as?
I can't think of any. However it is very easy to abuse this behavior if it worked as you describe. For example, from the host I could do ln -s /etc/shadow /home/test/shadow, and get access to the /etc/shadow file within the container, where normally I would not have access to it.

Lets say you setup SSH in a container, and you for some reason perform ln -s /etc/ssh/sshd_config /root/config/sshd_config in the container and mount volume with  -v /home/service:/root/config . Then you will actually modify the host sshd_config instead of the container sshd_config. This specific use case requires --privileged=true. This use case may be bad.
Seems like symbolic links and docker containers must be used with care.

and mount volume with  -v /home/service:/root/config

In that case, the "dangerous" part is in mounting a host-directory in the container, not the symlink. By mounting a host-directory in a container, you are explicitly breaking the boundary between the container's filesystem and the host. You should always be careful doing that.
If you're worried about making changes to a bind-mounted directory, you can also mount with the :ro option to make it read-only.

I will take care setting up the production containers. Lets close this. Thanks for feedback


Various improvements to the DateFilter widget

Includes various fixes to the DateFilter:

On picker mode cast start/end/value to datetime.date
Override the value Parameter by a CalendarDate or DateRange Parameters (let me know if there's a better way, that looks hacky!)
Bidirectionally link the widget to the value Parameter, this allows URL query parameter changes to be synced back to the widget.

This is work in progress and requires other changes both in Param and Panel. I've also noticed accumulating calls in get_data on picker mode so there's something wrong there.

Codecov Report

Merging #276 (f92a7cb) into master (99a62e3) will decrease coverage by 0.19%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #276      +/-   ##
==========================================
- Coverage   65.42%   65.22%   -0.20%
==========================================
  Files          57       57
  Lines        5611     5628      +17
==========================================
  Hits         3671     3671
- Misses       1940     1957      +17

Impacted Files
Coverage Δ

lumen/filters/base.py
63.07% <0.00%> (-2.70%)
:arrow_down:

lumen/ui/variables.py
36.36% <0.00%> (-2.20%)
:arrow_down:

lumen/ui/launcher.py
47.25% <0.00%> (-2.18%)
:arrow_down:

Continue to review full report at Codecov.

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

This is ready for review @philippjfr
The accumulating calls to get_data I saw when picking a date were due to an issue in Param. The last commit just allows to link visible and disabled to the date widget, they were not linked which looks like an oversight.
A few other PRs were required to get this properly working:

https://github.com/holoviz/panel/pull/3513
https://github.com/holoviz/panel/pull/3515 + https://github.com/holoviz/panel/pull/3529
https://github.com/holoviz/param/pull/627
https://github.com/holoviz/param/pull/625
https://github.com/holoviz/param/pull/629

@philippjfr I've reworked this PR quite a bit, https://github.com/holoviz/lumen/pull/276/commits/c89cb89b0a5b90e5651c255f17dea3d117bd42f2 is where you can see all the changes I've made.
Based on our discussion and write-up in https://github.com/holoviz/lumen/issues/293, I've implemented multi for the date filters, and I've added datetime filters. There are a couple of decisions I've made which we'll need your approval:

For multi=True and mode='picker', as there's no DateRangePicker available in Panel yet I've decided to fallback to a DatetimeRangePicker and emit a warning. This is the option 2 I mentioned in https://github.com/holoviz/lumen/issues/293
For multi=False and mode='slider',  as there's no DatetimePicker I've decided to emit a warning and fall back to a DatetimePicker

It'd probably be worth:

[ ] Adding some tests!

Adding tests separately. Thanks @maximlt!


Better error message for non-local imports with --m=comonjs

importing a file in the same directory, with non-relative path in node causes an error for unknown file. the error message can be a bit helpful if the file is around.
import * as ns from "file1";

generates error:

Error TS2307: Cannot find module 'file1'.

can be:

Error TS2307: Cannot find module 'file1'. did you mean ./file1

comonjs is misspell?