[not reproducible] podman-remote rmi: device or resource busy

@cevich is seeing frequent system-test errors [1] with podman-remote
$  ./bin/podman-remote rmi -f build_test
# Error: error removing containers [10b343023c7d6187aa521fad47934b5d558adc53964ab491017e34d909782751] for image "8659137a98bd8c7f3528c1c2c09a0fc8115bc32ce7659bc7a1d31b7d73dd3029": could not remove container "10b343023c7d6187aa521fad47934b5d558adc53964ab491017e34d909782751": remove /var/lib/containers/storage/overlay-containers/10b343023c7d6187aa521fad47934b5d558adc53964ab491017e34d909782751/userdata/shm: device or resource busy

This is (I think) Ubuntu. I cannot reproduce on my Fedora laptop, even running the varlink server with nice 19 and even with cores disabled to emulate a slow system. My attempted reproducer is a simplified variant of the build system test:
#!/bin/bash

set -xe

tmpdir=$(mktemp -d --tmpdir podman-remote-test.XXXXXX)
cat >$tmpdir/Dockerfile <<EOF
FROM quay.io/libpod/alpine_labels:latest
RUN apk add nginx
RUN echo hi >/myfile
EOF

pr() { /path/to/podman-remote "$@"; }

while :;do
    pr build -t build_test --format=docker $tmpdir
    pr run --rm build_test cat /myfile
    pr rmi -f build_test
done

My only guess is that this might be a race condition with podman-remote run --rm: does anyone know if it is guaranteed to block?
master @ 1ad8fe52412c8cef215e5a5af312e5db680c3916. I am sorry for not being able to reproduce it but Chris seems to run into it constantly.

Thanks Ed.
Thinking about this more, I'm not seeing those particular failures anywhere else.  So there must be something special about PR #3754.  That could simply be new/different VM imges - including images with embedded bugs.
Note: The two failures are on F29 and F30 (not Ubuntu) images:

fedora-30-libpod-5420341990522880
fedora-29-libpod-5420341990522880

I'll try your reproducer (above) using those images vs what we have on master...

@edsantiago good news is, I was able to reproduce the error using your script above and the F30 VM image from my PR.  Trying the VM image from master (expecting no error) now...

...as expected, no error using the VM image from master (fedora-30-libpod-5751722641719296).
Excellent, so now (hopefully) it's a matter of comparing differences. Since the libpod code is common in both VMs, they're running in the same cloud-environment, so there must be a configuration or packaging difference or something like that.

Same on both VMs:

libvarlink-util-18-1.fc30.x86_64
container-selinux-2.113.0-1.dev.git4f7d6bb.fc30.noarch
runc-1.0.0-93.dev.gitb9b6cc6.fc30.x86_64
systemd-241-10.git511646b.fc30.x86_64
slirp4netns-0.4.0-4.git19d199a.fc30.x86_64
containernetworking-plugins-0.8.1-1.fc30.x86_64
containers-common-0.1.37-2.fc30.x86_64

hrmmmmm.

Ran reproducer again, and it worked fine for a while, failing after maybe 10 passes.  So there's def. some race condition here that's somehow more likely with the images in my PR vs from master, even with the exact same libpod code :confounded:
Maybe I should try taking the podman and podman-remote binaries from the breaking-VM over to the working-VM to see if the behavior moves/changes?
Not sure what else could be involved here, @mheon have any ideas?

confirmed: it's not the binaries, behavior remains the same after copying from breaking -> working VM.
Noticed this difference though, the breaking VM is running kernel-5.2.8-200 vs working VM (from master) is using kernel-5.2.7-200.  Going to try upgrading that kernel, rebooting, and see if the problem starts happening...

JJF**($!!343$hg80*&#//**** :angry:
updated both VMs to 5.2.9-200.fc30.x86_64 rebooted, but the pattern remains: Problem reproduces on 1fedora-30-libpod-5420341990522880but notfedora-30-libpod-5751722641719296`.

This could definitely cause a race.
pr build -t build_test --format=docker $tmpdir
pr run --rm build_test cat /myfile
When this command finishes the image is still mounted.  conmon will execute podman container cleanup, but after the run command finishes.
pr rmi -f build_test
If this command executes before podman container cleanup finishes you can get this error.

@mheon Is my assumption correct.  Does podman container cleanup take care of the --rm as well?

Thanks @rhatdan. If this is the case, would it be possible to have run --rm block until conmon finishes cleanup?
And if that's not possible, I'd like to request a clear documentation update stating something like:
 podman-remote run --rm actually means "remove the container some time in
 the future, not necessarily now", and is not actually guaranteed to remove
 the container. If you actually want to deterministically remove the container,
 perhaps because you want to rmi its parent container, please run [some sort
 of TBI command that blocks until container is removed]

Yes I agree it should be cleaned up, but I am not sure if my theory is correct.

podman run --rm should, if attached (not run with -d), block until container removal is complete. I don't think there's a potential for races, either.

Well @edsantiago is seeing what looks like a race when using the varlink connection.

Small correction: I've never seen it; only @cevich has seen it.

@mheon wrong answer.  The correct answer is: Oh, I saw this problem yesterday and just merged a fix :grinning:
@mheon remember this is podman-remote -> podman.  Perhaps podman-remote is exiting before podman/conmon finish their job?
With Ed's reproducer script, this very reliably reproduces for me (using hack/get_ci_vm.sh on the images mentioned).  Would running in debug mode help at all?  How can we prove/disprove Dan's theory?  Or maybe is there a way we can tell the audit subsystem to log process-exits (with timestamps)?

@mheon I managed to catch this in the act by instrumenting the system-test that reproduces it:
--- a/test/system/070-build.bats
+++ b/test/system/070-build.bats
@@ -26,9 +26,11 @@ EOF
     PODMAN_TIMEOUT=240 run_podman build -t build_test --format=docker $tmpdir
     is "$output" ".*STEP 4: COMMIT" "COMMIT seen in log"

+    run_podman ps -a
     run_podman run --rm build_test cat /$rand_filename
     is "$output"   "$rand_content"   "reading generated file in image"

+    run_podman ps -a
     run_podman rmi -f build_test
 }

You can see what the test is doing in the diff.  When it reproduces, it happens on that last rm -f line:
[+0100s] not ok 28 podman build - basic test
[+0100s] # (from function `die' in file test/system/helpers.bash, line 264,
[+0100s] #  from function `run_podman' in file test/system/helpers.bash, line 164,
[+0100s] #  in test file test/system/070-build.bats, line 34)
[+0100s] #   `run_podman rmi -f build_test' failed with status 2
[+0100s] # $ ./bin/podman-remote rm --all --force
[+0100s] # $ ./bin/podman-remote images --all --format {{.Repository}}:{{.Tag}} {{.ID}}
[+0100s] # quay.io/libpod/alpine_labels:latest   4fab981df737
[+0100s] # $ ./bin/podman-remote build -t build_test --format=docker /tmp/podman_bats.hPzLvX/build-test
[+0100s] # STEP 1: FROM quay.io/libpod/alpine_labels:latest
[+0100s] # STEP 2: RUN apk add nginx
[+0100s] # fetch http://dl-cdn.alpinelinux.org/alpine/v3.8/main/x86_64/APKINDEX.tar.gz
[+0100s] # fetch http://dl-cdn.alpinelinux.org/alpine/v3.8/community/x86_64/APKINDEX.tar.gz
[+0100s] # (1/2) Installing pcre (8.42-r0)
[+0100s] # (2/2) Installing nginx (1.14.2-r1)
[+0100s] # Executing nginx-1.14.2-r1.pre-install
[+0100s] # Executing busybox-1.28.4-r1.trigger
[+0100s] # OK: 6 MiB in 15 packages
[+0100s] # 27af317be9360cdb9cf9ec92e3852518e34831223001012fb28aaf496d2462ff
[+0100s] # STEP 3: RUN echo UkyW3wKhN0Uk8z1zdTzgjXxyUypI0WBjXmaKij34vb2vC9cHA2 > /UdFKEYczgNKHkBKXaNwL
[+0100s] # STEP 4: COMMIT build_test
[+0100s] # 102f5447866173d6e131165b85c0955f970cba4d5f2197cd40e1bd506824ce78
[+0100s] # $ ./bin/podman-remote ps -a
[+0100s] # CONTAINER ID  IMAGE  COMMAND  CREATED  STATUS  PORTS  NAMES
[+0100s] # $ ./bin/podman-remote run --rm build_test cat /UdFKEYczgNKHkBKXaNwL
[+0100s] # UkyW3wKhN0Uk8z1zdTzgjXxyUypI0WBjXmaKij34vb2vC9cHA2
[+0100s] # $ ./bin/podman-remote ps -a
[+0100s] # CONTAINER ID  IMAGE                        COMMAND               CREATED                 STATUS                             PORTS  NAMES
[+0100s] # e32470ce70ef  localhost/build_test:latest  cat /UdFKEYczgNKH...  Less than a second ago  Exited (0) Less than a second ago         keen_cohen
[+0100s] # $ ./bin/podman-remote rmi -f build_test
[+0100s] # Error: error removing containers [e32470ce70ef1fabf805c6db9d2bdb763bb0b802e79ac6d2a8b38e21171ac8e4] for image "102f5447866173d6e131165b85c0955f970cba4d5f2197cd40e1bd506824ce78": could not remove container "e32470ce70ef1fabf805c6db9d2bdb763bb0b802e79ac6d2a8b38e21171ac8e4": unlinkat /var/lib/containers/storage/overlay-containers/e32470ce70ef1fabf805c6db9d2bdb763bb0b802e79ac6d2a8b38e21171ac8e4/userdata/shm: device or resource busy
[+0100s] # [ rc=2 (** EXPECTED 0 **) ]
[+0100s] # #/vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv
[+0100s] # #| FAIL: exit code is 2; expected 0
[+0100s] # #\^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

So clearly a race here is possible, but perhaps (somehow) more likely when podman-remote is being used?
is it impossible for the run ... --rm to verify removal before exit?

Have we ever seen this without remote being involved? This should not be possible for local Podman.

@mheon As best as I can recall, I have not seen it.  Even with podman-remote it doesn't always happen.  Is it possible for podman-remote to special-case this, and verify removal happens before exiting (assuming there was no networking-error)?

Looking at the difference between pkg/adapter/containers.go and pkg/adapter/containers_remote.go.
Containers.go has a call to
if ecode, err := ctr.Wait(); err != nil {

And containers_remote.go does not.  I would guess this is the issue.

Oh. That sounds like it.

Oh that's a HUGE relief, thanks @rhatdan !

Closing as #3934 is merged


feat: Event webhook. Closes #2667

Checklist:

[ ] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[ ] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
[x] I've signed the CLA.
[ ] I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
[ ] My builds are green. Try syncing with master if they are not.
[x] My organization is added to USERS.md.

Closes #2667
Tasks

[ ] Extract reapplyUpdate into its own struct so we can re-use it.
[ ] Ensure that problems with a workflow in event_server.go do not result in an error event.

Closing PoC.


Move GOVUK-Request-Id from varnish to nginx.

DO NOT MERGE YET
This gives us tracing in our admin systems.
If we find that we need tracing back between varnish
and the router on our frontend systems we can add
a "GOVUK-Cache-Request-Id" or similar.
If anyone has any changes they'd like to make to the "unique enough" ID, please let me know.
I've confirmed via vagrant that backend-lb-1 and backend-1 will build and boot nginx happily, but had trouble trying to form convincing requests for it.

Updated with config fix, shown working here: https://kibana.integration.publishing.service.gov.uk/kibana#dashboard/temp/AVQPwvUHAWArxCU_s3s8

:+1:


feat(event-handler): API Event handler for AWS Lambda

Description of your changes

New feature for api event-handler in typescript
Feature set follows the lambda powertools python pattern for handling apiGateway events (REST APIs)

Related issues, RFCs
https://github.com/aws-powertools/powertools-lambda-typescript/issues/413

Issue number:
Checklist

[ ] My changes meet the tenets criteria
[ ] I have performed a self-review of my own code
[ ] I have commented my code where necessary, particularly in areas that should be flagged with a TODO, or hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my change is effective and works
[ ] The PR title follows the conventional commit semantics

Breaking change checklist
Is it a breaking change?: NO

[ ] I have documented the migration process
[ ] I have added, implemented necessary warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Hi folks, thank you for opening this PR.
Please allow us some time to go through it and start adding comments.
You can ignore the failed check from Semantic PR, it's failing because it doesn't recognize the event-handler scope, which is expected. On the other hand, other automations related to identifying the source of the PR are failing due to the PR body having been filled incorrectly.
I can edit it if needed, but if possible please take some time to review the PR template and amend at least the "Related issues, RFCs" section using the appropriate format (issue number - i.e. #000 - should be next to the **Issue number:** text).

@dreamorosi  @heitorlessa  - this PR has been open for sometime. Does the Powertools roadmap for 2024 has event-handler feature expansion plan? If not, we can opensource the event-handler as a standalone project. please let us know.

replying today @Muthuveerappanv as soon as I complete internal admin tasks :)

TL;DR: The goal is to launch Event Handler this year with Zod and OpenAPI integration. This year we will continue our focus on feature parity and operational excellence - ETA: v2 and Parser in Q1, Event Handler thereafter.
In the last two quarters we've had to redirect our attention to security, re:Invent, and more recently create a structure for the Powertools team formed by the end of Q3. With some internal setbacks, the operational tax accumulated and ESM headaches pushed the remaining of our focus to get V2 out asap.
This meant we couldn't dig deeper into Event Handler. On the bright side, we did hear from many customers at re:Invent which made the case stronger. They told us not only they're looking forward to this feature but they also want the newest major addition: Data Validation and OpenAPI support.

Timelines

I'm finishing goals, KPIs and key dates for the year we should not miss next week. Then, I'll work with every language to reset their iteration cycle and refresh the roadmap to reflect the headcount and operational work - ETA: early February. To prevent long periods without feedback, among other things, we'll start operational and product reviews every other week - please bear with us!
I highly suspect we might not be able to take the PR as-is, since bringing Zod support for Data Validation and OpenAPI/SwaggerUI support will need a fresh look into the Event Handler experience. By all means, feel free to open source it, we will certainly get inspirations from the great work the team did.
Long answer but necessary given our delays! Let me know if you need any extra details.

@heitorlessa thanks for the explanation, makes a lot of sense.

Validation -  Zod for Parser is something that we use internally along with event-handler through Middleware. If you remember I kick-started the parser-utility with this initial RFC. The Event-Handler has a middleware capability through which we solve parsing, validation, etc. We are open to a different approach (or) customization on top of it if needed
OpenApi - Definitely the roadmap of our Internal DevAx team in 2024. Would be happy to collaborate & share ideas regarding that.

@karthikeyanjp  can share more concrete examples of the event-handler middleware with zod-parser to give you a better understanding.
I suggest that once the Powetools-Typescript team completes the v2 release, we should have a call/deep-dive on Event-Handler to collaborate seamlessly.

As discussed I will close the PR for now, we can continue the discussion under #413 for maximum awareness/visibility.


Get the processor and api ports from env vars if available

I am unable to start the processor and api at the same time because both starts on port 8000. Should both listen on port 8000? if not then i will update that as well in this pr.

To test:
export ROS_API_PORT=8002
pipenv run python3 -m ros.processor.main
and,
export ROS_API_PORT=8003
pipenv run python3 -m ros.api.main

@upadhyeammit Thanks for the changes!
@patilsuraj767 Do you think we can use something like python-dotnev as a dev package to explicitly auto load the vars in the .env file

@saltgen No, I think we should not add python-dotenv.
Reason -

We cannot add python-dotenv package as only dev dependency.
To get python-dotenv to work we will need to edit main.py(which goes in production) and I think we should not touch the production code only to easy developer life from running . .env :sweat_smile:
It creates the risk of overriding env variables which are auto set by openshift/clowder in production.

PR looks good. Can we also append ROS_PROCESSOR_PORT=8004 to scripts/.env file so that we can directly run . scripts/.env to set all development variable at once instead of remembering variable name and running export command manually. Please use port 8004 as 8003 is used by puptoo in docker-compose.

Done! Instead of 8002 and 8004 I have used 8004 and 8005

@saltgen No, I think we should not add python-dotenv. Reason -
1. We cannot add python-dotenv package as only dev dependency.

2. To get python-dotenv to work we will need to edit main.py(which goes in production) and I think we should not touch the production code only to easy developer life from running `. .env` sweat_smile

3. It creates the risk of overriding env variables which are auto set by openshift/clowder in production.

We could list the package as a dev dependency in the Pipfile and then do a `pipenv install --dev`, but yes those will still need to be loaded in config.

Thanks for checking :v:

Rebased!

@upadhyeammit Thanks for the changes! :confetti_ball:


Accessing Payload message in receive_method_request(method_name="XXXX")

Not sure what i should do to access the message payload coming through receive_method_request.
The line print("Payload:" + method_request.data ), seems to hang and hence the direct method fails to send message from Azure Portal. When i remove the statement print("Payload:" + method_request.data ), everything works correct. However i still don;'t know how to get access to the payload  coming through Direct method
    async def method_listener(module_client):
        while True:
            method_request = await module_client.receive_method_request("ams")
            # Wait for unknown method calls
            payload = {"result": True, "data": "Received"}  # set response payload
            status = 200  # set return status code
            print("executed method: " + method_request.name)
            print("Payload:" + method_request.data )
            method_response = MethodResponse.create_from_method_request(
                method_request, status, payload
            )
            await module_client.send_method_response(method_response)  # send response

I am not exactly sure why the line will hang but the MethodRequest object in the azure-iot-sdk-python has no attribute called data. The attribute you may be looking for is payload.


npm install downloads packages in bundledDependencies (npm #6435)

You cannot use NPM 2.x with CleverStack currently because of this issue with NPM, https://github.com/npm/npm/issues/6435

This is no longer an issue as we do not use bundledDependencies[] for loading/referencing modules


Added animation to footer

Added slide in from left animation to footer

#128 resolved by this PR

@arghadipmanna101 Please assign me this PR under GSSoC and label this PR

@arghadipmanna101 Please make me as the assignee for this PR as it is considered for scoring


ngf-select stopped working after upgrade to iOS10 (iphone 5 and 5c)

Hi there,
We have an app that uses the ngf-select directive to upload an image file.
We have noticed that this has stopped working on a couple of iOS 10 devices (iphone 5 and 5c) after upgrade from iOS 9 to 10. We have an iPhone 6 that works correctly after an upgrade.
Cordova 6.3.1
Ionic 2.0.0
IPA built using ionic cloud.
Thanks
Peter

Seconding

Same issue here, also doesn't work withing Chrome browser on IPhone

+1

+1

Can you provide your code? Does it work if you use <input type="file"...>. What exactly not working?

Hi Daniel,
I have the following code:
Change profile image photo_camera

This works fine across most browsers and devices. It was working fine on my iPhone 5 and iPhone 5C devices running iOS 9. After upgrading to iOS 10 when you tap on the 'Change Profile Image' link then nothing happens. Strangely enough works on iPhone 6 running iOS10.
Thanks
Peter

Hi Peter. Did you ever solve this or find out why it is happening? I am still struggling to figure out why its not working on those particular devices for me too.

+1

Hi jj-julia, there is the workaround to use the  tag but I haven't been able to devote much time to looking into this as a possible fix. Depending on your use it may need some additional styling if it does work.

Hi jj-julia, I'm having the same problem on ios 10 with iphone 7. Have you managed to solve this issue?

+1

+1

+1

Follow it here: https://github.com/danialfarid/ng-file-upload/issues/1728#issuecomment-285666176


Support for A1111 extension

Related to #55.
Added two npm commands.

npm run dev-extension: Launch the extension's UI in standalone mode (without A1111 WebUI) for development.
npm run build-extension: Build files for extension

This command generates pages and javascript files. When this extension is released, those files need to be committed.

See here for details of the following files.

style.css
install.py
scripts/openpose_editor_3d.py

Sorry for the delay.
Thank you.

Thanks for your work, I will verify it later

I have a query. Why did you design a vite extension? It seems unnecessary.

@ZhUyU1997
Because it was easier to merge from upstream if the vite config file for extension was separated.
But now, the vite extension file may not be necessary.

@ZhUyU1997 Would you commit the pages and javascript directories? These are required for the extension to work.
After you commit these files, I will update the repo url in the webui extension list to this repo.


[WiP] Moving project structure to ProjectScaffold

This is still very much work in progress, but I thought I'd create the initial PR, so that others can help reviewing this too.

It updates the project to use Paket for dependency management. The restore is invoked automatically from VS build, so this should not cause complications. Paket makes it easier to update dependencies using paket update too. So, all packages.config are replaced with paket.dependencies etc.

There is a build.fsx FAKE script, which drives the whole process. It cleans build outputs, generates AssemblyInfo files, invokes MSBUILD to build the project, runs tests (there are no unit tests yet, so this does nothing) and then it does 3 more things (see next points).

The SourceLink bit is currently commented out, because I couldn't get this to work (cc @ctaggart). AFAIK, this should fix the PDB files, so that the project can be debugged even if you don't have the source locally. I got an exception:

unable to load proj file C:\...\XPlot.GoogleCharts.fsproj with properties: [("Configuration", "Release")], error The value "" of the "Project" attribute in element  is invalid.

Building of NuGet packages should now be done via Paket (paket pack command). This figures out all the information from project, which is quite nice. But at the moment, the setup would produce 1 package for each project (which might actually be better idea - if we included the WPF code in the main package, referencing the package would add reference to the DLL with the WPF code, which I was hoping to avoid). But I'm not sure if having that many packages is a good idea (cc @forki in case he has some recommendations).

Finally, I just moved the existing docs code into "OldDocs" and have not updated this to the new format yet (will do this next!) I'd like to use the same style as in F# Charting, which embeds charts automatically (which we can then also use in FsLab journals...)

I made some more progress on this - In the latest version, it is now possible to automatically embed charts using the F# Formatting evaluator:

Here is one example of the source code
Which produces this work-in-progress homepage as the output!

I'll go through the existing docs and convert them into something basic in the default ProjectScaffold template - your original site is nicer (and you're much better at web design :-)) so I think you can then tweak it to actually look better, if you want.
Once that's done, I think the PR will be in a decent state to get merged (and I can then replace Foogle.Charts with XPlot in FsLab, which is going to be awesome!!)

This is fantastic, we can merge right now if you want and I'll be able to help with the docs conversion since there are lots of demos (and it would be a good opportunity for adding a couple of Deedle + Google Charts samples).

I did a little bit more work on the docs - to get them into a shape that looks like a reasonable start. It is still very incomplete, but it is already better than Foogle Charts :-).

There is reasonably nice home page: http://tpetricek.github.io/XPlot/
I added line & point charts and geo charts tutorials
It builds API Reference from /// comments

All of these need more work (add tutorials for other charts, fill in the comments for all the types), but I'm pretty sure we can ask people to contribute here! I left in the code that lets you embed Deedle frames, so if you want to add examples of this, it should work too.
If you can merge & release it at this point, that would be amazing - I could then look into the next step, which is adding XPlot to the FsLab package. Woohoo :-)
The only two things I did not test in the release process are GitHub releases and pushing to NuGet. This should work nicely if you follow the ProjectScaffold guidelines. This works best if you create release.cmd script as described here - this is already in .gitignore, so it should be safe to put your passwords (for the release process) there.
Ping me, @forki or @pblasucci if something does not work!


Allow filtering of review queues

It would be nice if users can choose to filter (or at least prioritise) particular posts in the review queue, e.g:

Posts only on a specific site
Newest vs oldest posts
Posts with 1 vs 0 feedbacks
Posts with comments
Posts by weight

Do we think this would assist in reviewing, or would it allow people to only review the 'easier' posts and leave the harder ones in the queue for eternity?

Por que no los dos?
I think it's likely to be both. It's certainly likely to help get the review count down and to get a lot of the easier reviews done quicker; also fairly likely that the harder reviews get left in the queue until someone more experienced can come along.
But... I don't think that's a bad thing. The majority of stuff in the queue is relatively easy stuff; this would help get rid of that, which would make a sizable dent in the queue... which is what we need.


Make nullables dublicates of the mappings (And ignore settings of my mapper)

Following the documentation - i made internal mapper to convert properties of the object:
[Mapper(UseDeepCloning = true)]
public static partial class PortfolioMapper
{
    public static partial GetPortfolioResponseDto ToGetDto(this Portfolio source);

    [MapperIgnoreSource(nameof(Agreement.Portfolios))]
    [MapperIgnoreSource(nameof(Agreement.Client))]
    private static partial GetAgreementResponseDto ToGetAgreementResponseDto(Agreement source);

    [MapperIgnoreSource(nameof(Client.Agreements))]
    private static partial GetClientResponseDto ToGetClientResponseDto(Client source);
}

Code Generated includes "?" for nullables for the Portfolio class

But also for the Agreement class where i configured ignore settings for the Client property:

So it uses nullable version of code-generated mapper and doesn't pick up my configuration.
To Reproduce
So I've tried to reproduce the bug, but in small project it works fine - it doesn't create nullable duplicates.
Environment (please complete the following information):

Mapperly Version: 2.6
Target Framework: net7.0
OS: Windows10

Original Classes:
public class Agreement
{
    [Key]
    public int AgreementId { get; set; }

    public Client Client { get; set; }
    public string Code { get; set; }
    public List<Portfolio> Portfolios { get; set; }
    public DateTimeOffset Date { get; set; }
    public DateTimeOffset? TerminationDate { get; set; }

    public bool IsDeleted { get; set; }
}

public sealed class GetAgreementResponseDto
{
    public int AgreementId { get; init; }
    public GetClientResponseDto Client { get; init; }
    public string Code { get; init; }
    public DateTimeOffset Date { get; init; }
    public DateTimeOffset TerminationDate { get; init; }
}
public sealed class GetPortfolioResponseDto
{
    public int PortfolioId { get; init; }
    public GetAgreementResponseDto Agreement { get; init; }
    public string Code { get; init; }
    public DateTimeOffset Date { get; init; }
    public DateTimeOffset TerminationDate { get; init; }
}

public class Portfolio
{
    [Key]
    public int PortfolioId { get; set; }

    public Agreement Agreement { get; set; }
    public string Code { get; set; }
    public DateTimeOffset Date { get; set; }
    public DateTimeOffset? TerminationDate { get; set; }

    public bool IsDeleted { get; set; }
}

Thank you for your bug report. I tried to reproduce this without success.
Could you create a minimal repro project?

Thank you for your bug report. I tried to reproduce this without success.
Could you create a minimal repro project?

Thank you!
I've tried to reproduce already. In console project with my classes and nothing else: mapper doesn't create extra (nullable) methods. With same classes in big production project it repeats. I cann't find any logic.
Are there any mapper configuration for nullables? Or probably project configuration that mapper uses?

The only null related configurations available are ThrowOnMappingNullMismatch and ThrowOnPropertyMappingNullMismatch.
It could be nullable reference types related. Are there any #nullable directives in the project? Is there a <Nullable> defined in any of the msbuild files?

There aren't any #nullable directives in the project. But if i switch csproj <Nullable>enable</Nullable> to <Nullable>disable</Nullable> - mapper handle my methods immediately.

Switching nullables back to enabled solved the problem! Thank you for the hint!
It will be cool to have such property in Mapper (generate nullable methods).

I found a bug related to init only properties and nullable. should be fixed with #257.


The link points to an empty list

https://github.com/biohackrxiv/preview.biohackrxiv.org/blob/51714a302b9713685020cd3b547a6bf19482a174/views/index.slim#L25
Maybe https://biohackrxiv.org/discover can be used instead, but the .json link just below that may not have an equivalent.

This is perhaps because Pjotr's server is down @pjotrp
We may have to have both of them and show a message if the list is not shown correctly

Please point the list to
https://osf.io/preprints/biohackrxiv/discover?q=
for now. We can't maintain the list because of a lack of metadata and
we are working on that.
On Sun, Nov 05, 2023 at 07:42:28PM -0800, Tazro Inutano Ohta wrote:

This is perhaps because Pjotr's server is down @.***
We may have to have both of them and show a message if the list is not
shown correctly
—
Reply to this email directly, [2]view it on GitHub, or [3]unsubscribe.
You are receiving this because you were mentioned. Message ID:
@.***>
References

https://github.com/pjotrp
https://github.com/biohackrxiv/preview.biohackrxiv.org/issues/23#issuecomment-1794037715
https://github.com/notifications/unsubscribe-auth/AAADJ36245KV6Z2WZXLDHL3YDBMCJAVCNFSM6AAAAAA63KY74OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJUGAZTONZRGU

made a PR to fix this https://github.com/biohackrxiv/preview.biohackrxiv.org/pull/24


add validation for auction periods duration

Pull Request Test Coverage Report for Build 40

15 of 18 (83.33%)  changed or added relevant lines in 3 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.7%) to 86.51%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

openprocurement/auctions/geb/models/schemas.py
11
14
78.57%

Files with Coverage Reduction
New Missed Lines
%

openprocurement/auctions/geb/managers/initializators.py
7
93.14%

Totals

Change from base Build 39:
-0.7%

Covered Lines:
1180

Relevant Lines:
1364

💛  - Coveralls

Pull Request Test Coverage Report for Build 40

15 of 18 (83.33%)  changed or added relevant lines in 3 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.7%) to 86.51%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

openprocurement/auctions/geb/models/schemas.py
11
14
78.57%

Files with Coverage Reduction
New Missed Lines
%

openprocurement/auctions/geb/managers/initializators.py
7
93.14%

Totals

Change from base Build 39:
-0.7%

Covered Lines:
1180

Relevant Lines:
1364

💛  - Coveralls


[FEATURE REQUEST] OAuth support

Describe the solution you'd like
I'd love to see support for OAuth2, so I can manage users/authentication via authentik (or any other SSO platform).

+1 Would Love Discord Integration for Login


[Doc]  Under standing Code flow for model conversion and compilation | How to track code flow for llama mode / operators conversion to TIR & down to lower kernels for OpenCL

📚 Documentation
Request

I am very new to TVM Unity, Relax, TIR etc. I tried following multiple tutorials to get understanding of TVM and then got to know TVM Unity is different and uses relax and not relay.

The tuner and auto schedule does not apply to relax ( I guess)?
Does that means the llama mode is hand tuned and or static tuned config is used for every backend (OpenCL)?
How Can we print the kernels or the TIR config / optimiations used final model?
How can we build overall understanding of compilation (Pytorch - Relax - TIR - Model Lib) flow?

I am sorry if this is already explained somewhere. I tried reading the forums, Issues a lot but unable to build a good understanding of the project working and flow.
Can Someone Please help me to get started and list down how the model is written in mlc llm as entry point and how its converted.
@tqchen @Hzfengsy

Is this the entry point ? https://github.com/mlc-ai/mlc-llm/blob/main/mlc_llm/relax_model/llama.py
Is MLC-LLM a frontend for Relax?

created a tracking issue at https://github.com/mlc-ai/mlc-llm/issues/1156

Thanks for the questions
* right now we are not applying tuning, instead we apply an optimization library called dlight

* the kernels can be dumped using debug-dump option

We are in the process of bringing up a LLM.compile flow around a new set of mechanism, after that we will come up to documentation item.
cc @cyx-6

Thanks for your time and replying to my queries.

Can you please point me code where dlight is being applied.  What kind of optimization dlight is doing? Does it gets applied on every backend ?
The TIR model is pre stored : https://github.com/mlc-ai/mlc-llm/blob/main/mlc_llm/dispatch/llama/main.py
I thought that Relax graph is lowered to TIR & operator  prim functions using legalize op and other optimization passes.
Is it not the case right now? Is tuning == Pass Optimizations for which you replied with dlight usage?

Thanks.

Can you please point me code where dlight is being applied. What kind of optimization dlight is doing? Does it gets applied on every backend ?

The interface is at https://github.com/mlc-ai/mlc-llm/blob/main/mlc_llm/core.py#L598, while the dlight codes are at https://github.com/apache/tvm/tree/unity/python/tvm/dlight

The TIR model is pre stored : https://github.com/mlc-ai/mlc-llm/blob/main/mlc_llm/dispatch/llama/main.py
I thought that Relax graph is lowered to TIR & operator prim functions using legalize op and other optimization passes.
Is it not the case right now? Is tuning == Pass Optimizations for which you replied with dlight usage?

Currently we are not using TIR dispatching. The link you provided is a legacy file and we may remove it in the near future.

Thanks for your time and reply @Hzfengsy
I am trying ti understand the flow for the complete project. I wish to contribute to this project in future so I am trying to develop understanding of TVM Unity and mlc llm. I feel that MLC LLM diverges from the TVM Unity Tutorials and Forum posts and does thing in a more "hardcoded"  way (I might be completely wrong).
Do we store the model python code manually here : https://github.com/mlc-ai/mlc-llm/blob/main/mlc_llm/relax_model/llama.py
Is it being used anywhere in any way ( I think it gets used)
I wish to understand atleast the high level overview and major steps that comes into the picture.
I will really appreciate some help..
Thanks


Allow CF_API_KEY on other server types e.g. PAPER, SPIGOT

Create CF_API_KEY resource on other server types to allow for CURSEFORGE_FILES auto download mods.
I'd like to use Curseforge auto downloader to download bukkit mods like: https://www.curseforge.com/minecraft/bukkit-plugins/worldedit
This works without needing to copy them out of mods folder. mc-image-helper knows to put them in plugins/ folder automatically.
Log Snippet
[mc-image-helper] 05:44:47.786 INFO  : Downloaded mod file plugins/worldedit-bukkit-7.3.0.jar

Yes, I am using the CURSEFORGE_FILES  environment.
https://github.com/Tuxprogrammer/homelab/blob/master/kubernetes%2Fapps%2Fminecraft%2Fjava%2Fvanilla%2Fvalues.yaml
That's the specific values for the most part that I'm using.


Feat: next tally settings

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature with a change of default behavior.

What is the current behavior? (You can also link to an open issue here)
In DashboardPanel, when establishing if an AdLib is next, in addition to next pieces' tags and ids, tags from unfinished pieces were checked. That (depending on CSS styling) might lead to displaying unfinished pieces as both on-air and next. This can be considered incorrect in most cases, except some specific use cases.

What is the new behavior (if this is a feature change)?
By default, unfinished piece tags will not be taken into account when establishing if a piece is next.
Two options to are added to the Filter DashboardPanel:

Current part can contain next pieces - this enables taking unfinished piece tags into account when establishing if a piece is next. It's up to the blueprints and/or the custom CSS to make sure the tally indication is correct.
Indicate only one next piece per source layer - if enabled, only the first piece per each sourceLayer will be indicated as next, Piece ids and tags from the next part have priority over unfinished piece tags.

Other information:
Screenshot

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[x] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report

Merging #487 (2979d43) into release34 (ac33337) will increase coverage by 32.11%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           release34      #487       +/-   ##
===============================================
+ Coverage      67.88%   100.00%   +32.11%
===============================================
  Files            252         2      -250
  Lines          15155        32    -15123
  Branches        3372         8     -3364
===============================================
- Hits           10288        32    -10256
+ Misses          4701         0     -4701
+ Partials         166         0      -166

Impacted Files
Coverage Δ

packages/blueprints-integration/src/context.ts
100.00% <ø> (ø)

...teor/server/api/ingest/mosDevice/mosIntegration.ts

meteor/lib/ReactiveStore.ts

...eor/server/migration/deprecatedDatabases/1_13_0.ts

meteor/lib/Settings.ts

meteor/server/migration/0_22_0.ts

meteor/server/api/peripheralDevice.ts

meteor/client/lib/localStorage.ts

meteor/server/api/rundownLayouts.ts

meteor/server/migration/0_19_0.ts

... and 225 more

Continue to review full report at Codecov.

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


Update catalog entry - define engine=zarr as argument for xarray

fixes #167

Sounds good! I fixed the CI in #169.
Let's merge #169 first (needs approval), then merge the master here so we're sure the CI is in good shape.

Sounds good! I fixed the CI in #169.
Let's merge #169 first (needs approval), then merge the master here so we're sure the CI is in good shape.

Closing. New PR #214 fixes this too.


[1.9.1] Cherry-picks for logger modules crash + log rotation size.

High Level Description
This includes two cherry-picks:

https://github.com/dcos/dcos/pull/1565 - A CHECK failure in a logger module when running tasks with a : in the TaskID or ExecutorID.
https://github.com/dcos/dcos/pull/1499 - A logrotation config error, which leads to rotating files when they are 1 MB instead of 256 MB.

Related Issues

CORE-1062 Chronos launching Docker container causes Mesos agent to crash
DCOS-15653 Log rotation happens 256 times more often than intended

Checklist for all PR's

[x] Included a test which will fail if code is reverted but test is not: Test was added to modules repo.
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): https://github.com/dcos/dcos-mesos-modules/compare/7ce5ba397bbaa8fdcc985ef2e6dd8bc81709931a...e46209d55e6e5be2caa2e49ca3c32bdfec5bd427
[ ] Test Results:
[x] Code Coverage (if available): N/A

@kaysoky it seems like this needs a bit of work :-).

although I don't know why the azure test hung/failed (retrying).

Maybe that was as of https://jira.mesosphere.com/browse/DCOS_OSS-1180.


Split distribution packages?

Currently, the excalidraw package is quite large (disk space wise) because it ships dev and prod - umd and esm bundles. We use our own bundler, so we end up pulling 50MB of content into node_modules. It would be great if @excalidraw/excalidraw could be split to multiple dist packages like @excalidraw/dev and @excalidraw/browser. In our case, we only use dist/prod, so we end up pulling 47MB of unused code.

Hi @nishanthkarthik I am not sure how you are getting a mix of umd and esm bundles, in the @next release we only ship esm bundle which has dev and prod bundles and previously in umd also we shipped separate excalidraw.production.min.js and exclidraw.development.js files so you should be able to pull based on the environment.

My bad, I did not look at what's inside dist/browser. It appears browser/ also includes ESM bundles.

I'm not sure what the difference between browser/{dev,prod}, {dev/prod} and {excalidraw} is. Could you please explain the intended use cases for me?
I think my point still holds - just shipping the prod version(s) as a separate npm package would reduce our node_modules footprint.


SideNav: Run 'onClose' and 'onOpen' hooks when opened/closed via swipe

Both hooks were only called in case the sidenav was opened via a button click and closed with a click on the backdrop.
Let me know if there is a better way to do this. This is the only one I could find on the first glance.

Looks good!

Damnit wrong button ;)


dynamic rest: first commit

@ryochiji

added tests for _get_request_fields including the invalid a..b case
added _get_plural_name method and more serializer tests
added _sideload option to viewset such that it can disable the sideloading logic in the renderer
doc commented all new methods and properties

still TODO:

renderer tests
integration tests that hit the DRF router -> DYN viewset -> DYN serializers and DYN renderer (mostly to serve as a model for implementations)

@ryochiji added integration tests and a dummy set of test models and settings that will allow us to make server in order to bring up a demo server. I think it would be nice if we get this up on Heroku, add some fixture data, and allow folks to play around with the requests via the DRF browsable API.
Will also work on a coverage report; at this point I think it is around 90%

Having second thoughts about the need to underscore all of the new methods, given that there are certain methods which I would like to consider "private" in the sense that implementations of dynamic-rest will not need to know about them, and others that should be overridable. Perhaps we should make the public-facing methods and properties look more public.
Thoughts on that?

:+1: for having a demo server.
As for underscores, yeah, I think the underscore-as-private-method convention is so deeply engrained that using it to denote something else is probably not worth the confusion it can cause. What were your concerns around distinguishing Dynamic REST methods and DRF methods?

My concern is mostly internal w.r.t. Dynamic REST; I would like to continue to stay up-to-date with DRF changes and I felt that drawing a clear distinction between the DRF core and the dynamic extensions would help us do that.
However, from an external perspective, consumers of the Dynamic REST interfaces should know what is "public" and what is "private" and I think that should probably take precedence :)
As such, I'll remove the underscores from the "public" methods

At this point, I think include[] and exclude[] are well-supported and tested in DREST at this point, so I am going to merge the PR after making the underscore changes unless you have any other issues.
Here are a few next-steps I can see, in (IMO) priority order:

Port over the prefetching features from V3 and extend them to support Prefetch object chaining which should be useful for filtering down related objects and deferring fields at the queryset level.

Port over the filters{} feature from V3, perhaps modifying the implementation slightly such that it is compatible with the filter_backends interface (https://github.com/tomchristie/django-rest-framework/blob/master/rest_framework/generics.py#L42)

Add support for pagination, perhaps leveraging the new DRF 3.1 pagination interface (https://github.com/tomchristie/django-rest-framework/blob/master/rest_framework/generics.py#L45)

Port over V3RelationField and support lazy-loading in order to deal with circular dependencies within serializers (e.g. users = RelationField('UserSerializer'))

@ryochiji how is your allocation looking for this sprint? Would love to have your help with any of those points or others we can come up with :) -- but if you are tasked with other work, I have another week dedicated to this so I can start to work on them solo. Lets sync up today to discuss.

@aleontiev - Yep, looks good to me -- go ahead and merge. I scheduled some time later for us to sync up as well.


Continue inference process from latest results

Hi! I have run a SimulatedABCSMC inference using a given threshold schedule, for instance
threshold_schedule = [3.0, 2.0, 1.0, 0.5, 0.2]
Now, after checking my posterior distributions, I believe I could get better results by continuing the inference process with smaller thresholds, say
threshold_schedule = [0.1, 0.05]
Is there a way to continue the inference from the latest posterior distributions using only the two new thresholds [0.1, 0.05], without restarting the whole process from the beginning using [3.0, 2.0, 1.0, 0.5, 0.2, 0.1, 0.05] ?
It would save me a lot of time.
Thanks for your help!
Best,
Simon

Hi Simon,
I guess one option would be to use the output of the first stage as the prior for the second stage. You would most likely need to dig into internals of the package for the sampling code.
Evgeny


[request] HMAC Signature

Hash-based message authentication code

:+1:

:+1:

:+1:
Ideally this would implement some existing / draft standard such as draft-cavage-http-signatures-00.

@pcolby  +1

What resources we are targeting here for signature? Shall we follow "draft-cavage-http-signatures-00" as @pcolby advised or we come up with our own standard?

Shall we follow "draft-cavage-http-signatures-00" ... or we come up with our own standard?

By principal, I'd recommend following a standard over inventing our own.

What resources we are targeting here for signature?

It might make sense to make the targeted resources part of the configuration... if that doesn't complicate things too much?
Just my 2c :smile:
PS Tyk, the only API management tool I'm aware of that supports HMAC out of the box, currently implements draft-cavage-http-signatures (not sure which draft).

Thoughts? @lucamaraschi @bigpg

So final plugin implementation  would be similar to Key-Auth except the user will send a digest instead of password as part of Authentication header something like this.
Authentication: hmac username:base64(hmac-sha1(Date))
where Date is the http date header.
Kong will create the signature using the password and match it with request signature to validate.

I like the idea of @shashiranjan84

@pcolby @lucamaraschi @bigpg HMAC Plugin has been released, please give it a try.


I can't find the correct EPG link

Description
I type xumo.tv_us.channels.xml.gz and it got an error. Where is the EPG guide lists like XUMO for correct link ?

https://iptv-org.github.io/epg/guides/us/xumo.tv.epg.xml

Thanks!!! How did you find it? I cant find it anywhere. That way I dont to
bother you lol.
On Mon, Nov 7, 2022 at 5:31 PM Aleksandr Statciuk @.***>
wrote:

https://iptv-org.github.io/epg/guides/us/xumo.tv.epg.xml
—
Reply to this email directly, view it on GitHub
https://github.com/iptv-org/epg/issues/1315#issuecomment-1306316579, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5X34QMLE7W72YPZOHTHCTWHF7KTANCNFSM6AAAAAARZSQ37U
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
In Memory of Spot (FOREVER)

The links to these guides are removed from the description because they are deprecated and will soon stop working.
Links to all supported guides can always be found in README.md.


LGTM.com - false positive Python multiline text

SyntaxError when multiline text is actually OK syntax.
https://lgtm.com/projects/g/scivision/PyLivestream/snapshot/9d210e34128db51b92d1887cd660a8598dba221c/files/pylivestream/ffmpeg.py?sort=name&dir=ASC&mode=heatmap#x5ddb34cb63b681ff:1

We have created a fix for this, which should go live in a few weeks time. I will close this issue once I have verified that the syntax error has disappeared.

Thank you. Is the general timeline for getting fixes going live about a few weeks? because there is another LGTM fix I need that was merged into master about 3 days ago and I haven't seen it go live yet. Just curious.

Yeah, upgrading LGTM.com is a somewhat time-consuming process, as it involves building new snapshots for all of the 135.000+ repos we track there. For this reason it's only done roughly once per fortnight.


Fix confusing log message on breach of overallTimeout duration

Description
The log message printed out on breach of overallTimeout property in Helidon fault-tolerance Retry module is confusing and misleading.
Existing
io.helidon.faulttolerance.RetryTimeoutException: Execution took too long. Already executing: 80344961 ms, must timeout after: 1000 ms.

Updated
io.helidon.faulttolerance.RetryTimeoutException: Execution took too long. Already executing for: 1460 ms, must be lower than overallTimeout duration of: 1000 ms.

Documentation
No doc impact

Hi @spericas, could you please take a look at this?


ensure well known public domains do not get proxied on image imports

In client environments that run with a HTTP_PROXY: that proxy information is lost when the image import is done on the openshift-apiserver side and sent through the konnectivity-sidecar. There are a set of well known domains that are necessary for the MVP set of samples that get deployed in an openshift cluster. These well known public domains should not have to be proxied into the client environment and instead can go direct from the management cluster. Any other defined domains do need to proxy into the cluster network for the ability to access on prem registries.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/retest

/approve

/help

/retitle OCPBUGS-10823 ensure well known public domains do not get proxied on image imports

/retest

/lgtm

/retest

/cherry-pick release-4.12

/cherry-pick release-4.11

/cherry-pick release-4.13

/cherry-pick release-4.10

/cherry-pick release-4.9


:recycle: chore: Convert bun-plugin to jsdoc

Description
Convert bun pluginf from typescript to js with jsdoc
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #549   👈

This comment was auto-generated by Graphite.


Company page - rupee icon is different in same page

Describe the bug
Ruppee icon should be same in all page
https://www.berrycast.com/conversations/05c9827c-b710-5ca5-a342-c16037aaa73f
https://www.berrycast.com/conversations/a196bea9-230b-5e11-937e-6246656befd9

and Check General Settings Default Currency

https://www.berrycast.com/conversations/dd28a043-cee3-5ab2-9e08-48485a627ae4


[BUG] Selecting other asset in Send view don't change from address

Describe the bug
on main branch: When opening Send view and changing the From asset doesn't change the the address.
Expected behavior
Changing address to the respective selected chain
Screenshots

Additional context
on main branch

fixed in #821


Dracula Theme FreeTube

https://github.com/AnyaCodeX/Dracula-Theme-FreeTube

Hey, very cool! ✨
I just sent you an invitation to join the organization.
I also cloned your repository for Dracula's organization, so you can go ahead and delete or archive yours:

Your repository.
New repository.

Thanks for building this! 🎉

🧟 It's alive: https://draculatheme.com/freetube


Location status does not change from locating to tracking

lookup transform service might break the logic

https://github.com/CMU-cabot/cabot/commit/e638cacf9971f90b933b0c38cda66c69b079bfc6


Search at docs.flutter.io is broken due to jquery redirect?

This is a bad regression for our users.
https://code.jquery.com/jquery-2.1.4.min.js Failed to load resource: net::ERR_TOO_MANY_REDIRECTS
typeahead.bundle.min.js:7 Uncaught ReferenceError: jQuery is not defined
    at typeahead.bundle.min.js:7
    at typeahead.bundle.min.js:7
script.js:153 Uncaught ReferenceError: $ is not defined
    at initTypeahead (script.js:153)
    at XMLHttpRequest.<anonymous> (script.js:198)

Looks like our jquery is very old? Could that be contributing to the problem?

I'm not sure if this is dartdoc related or what.  @sethladd might know.

Perhaps intermittent? I was able to do a search for MaterialApp.

But it looks like the latest on the 2.0 branch is https://code.jquery.com/jquery-2.2.4.min.js.

Odd.  May be a google-specific issue.  Works fine in incognito, but not in a normal tab (which has google-specific proxy settings).

Just tested at work. Search is working for me. Does this still happen for you?

Nope.


Enable token authentication to private repos

Following on to https://github.com/Quansight/conda-store/issues/495 discussion around adding a token to install a package from a private repo...
When I add a token with the recommended format (see Read the docs how-to guide for an example), I get a validation error.

This is blocking #495

@kcpevey I've done some research on this and I think that conda-store actually does support this! I am using pkg_resources and the correct format is https://peps.python.org/pep-0508/.
Example
channels:
- conda-forge
dependencies:
- holoviews
- pip:
  - requests @ git+https://github.com/psf/requests.git@main
  - nothing
- ipykernel

I've created a PR to add better docs and error messages around the issue.

I'm reopening this issue since the current error message is not descriptive enough. Still needs improvement cc @nkaretnikov

This issue was conflating two errors. The token syntax in the yaml DOES work... but you have to add the @tag at the end and the tag has to be a release, not @main.
As is discussed in this issue, the following is valid:
channels:
  - conda-forge
dependencies:
  - python = 3.10
  - pip
  - pip:
      - <package name> @ git+https://<username>:<PAT><EMAIL_ADDRESS>  - ipykernel

I have an example now with 2024.1.1 where using @main DOES work. Not sure if this was a bug or is only a bug for some situations?


Two different users with same conversation URL

I am facing an issue where in two different users receive the same chat notification due to which a user can see other users personal chat. What can be the possible cause of the error? Can options.alternateFcmSenderId("alternate_sender_ID") cause this? I followed the init process as described here

Hi @outlander24
These issues are only for Atlas-Android. For other issues, please follow the instructions here to capture logs, and contact<EMAIL_ADDRESS>Thanks
Amar


feat(showcase): Add CloudQuery

Thank you for this great framework. We've been using it for a while 🚀
Implements https://github.com/shuding/nextra/issues/1096#issuecomment-1455088559

Thanks for the quick review and merge @B2o5T 🏅


problem with disabled-from disabled-to

If I define a range such as disabled-from = "17:30" disabled-to = "08:00" is not working properly.
if I just define disabled-from = "17:30" it works fine until 00:00
and if I just define disabled-to = "08:00" it works fine from 00:00 to 07:59 but with both properties together it does not work.

Hi @gautxo,
right now the disabled range expect from to be less than to values. but I will fix this ASAP.
but I tested it when it's used with disabled-from="00:00" disabled-to="08:00" it works as expected and I cannot reproduce this issue in that case.


fix(cilium): k3s has an API server proxy at <IP_ADDRESS>:6444

Proposed Changes

There is no need to use kube-vip LB for this Cilium config, k3s has an API server proxy listening in <IP_ADDRESS>:6444 on all nodes in the cluster. I run with this setup in the https://github.com/onedr0p/cluster-template so just sharing some knowledge to here.
Checklist

[x] Tested locally
[ ] Ran site.yml playbook
[ ] Ran reset.yml playbook
[ ] Did not add any unnecessary changes
[ ] Ran pre-commit install at least once before committing
[x] 🚀

Pinging @sholdee as you initially PRed this.

Not sure why the ipv6 test is failing...

LGTM. CI issue with IPv6 appears unrelated.

If that's the case, shouldn't we also clean up the references to apiserver_endpoint that were int his PR?
https://github.com/techno-tim/k3s-ansible/pull/435/files

If that's the case, shouldn't we also clean up the references to apiserver_endpoint that were in this PR?
#435 (files)

Kube vip is still needed to provide external access to the Kube API.

Essentially, the worker API server proxies are still dependent on registration with the VIP, so we still need to wait for this connectivity to be established before proceeding.

Thank you both!  The ipv6 test has always been kind of flaky, I will revisit this at some point


Document how to develop sao locally and test it

e.g. we should tell people they can do sao ./ output from the root of their folder and add output/ to .gitignore, and we might also want to mention targetPath set to path.join(__dirname, 'output') or something in ava test

hmm how are you testing in AVA? you don't need targetPath if you use sao.mockPrompt https://sao.js.org/#/advanced/testhttps://sao.js.org/#/advanced/test

@egoist why do you use it here? https://github.com/egoist/sao/blob/master/test/index.test.js#L9

oh that's not really necessary, you know that we can use default: ':folderName:' keyword, we actually get the folderName from targetPath, but targetPath in sao.mockPrompt is already set to a random value, and it will not really write any files to disk.


Cache Age not working

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
ios

Platform Version
11.0

SDWebImage Version
4.3.3

Integration Method
carthage

Xcode Version
Xcode 9

Issue Description and Steps
Guys I am using this framework to download and cache the image, using the url which will never change
example url path http://example.com/app/common/images/00108/28.jpg (Not real URL)
The problem is after few mins or hour the Image download again
I tried using
 	// Setting disk cache
        SDImageCache.shared().config.maxCacheSize = 1000 * 1000 * 500 // 100 MB

        // Setting disk cache age
        SDImageCache.shared().config.maxCacheAge = 60 * 60 * 24 * 7 // 7 days

But it keeps requesting for new image from same old path

@lohenyumnam Current implementation, the cache expire removeal process is only happened when your application is entering background. It use a long-term background task to check the expired cache data and remove them.
So if you do not enter background mode, just keep download and store image, the expired cache data will not been cleaned.

Thanks for the reply,
all I want is to download the image once then use the image from the cache for as long as I can.
I don't want to re-download the image again and again.
even if I set the cache age to 7 days or longer, in my app it lasts for like few hours and then it starts downloading the image again. am I missing something here?
// Setting disk cache age
SDImageCache.shared().config.maxCacheAge = 60 * 60 * 24 * 7 // 7 days

@lohenyumnam it depends on the HTTP headers of the response as well. If the server providing the image returns Expires or similar headers, the system may calculate the image needs to be re-downloaded. We can inspect those flags if you provide some test image, otherwise, you need to inspect them yourself. My assumption is this is a setup problem on your server.

@lohenyumnam feel free to reopen or just continue the conversation, I am curious if we can figure it out


Fix case of textEdit link

Incorrect:
https://microsoft.github.io/language-server-protocol/specification#textedit
Correct:
https://microsoft.github.io/language-server-protocol/specification#textEdit
Also added missing parenthesis in some old version of the spec.

Thanks!


How to run estatio in production

After deploying to tomcat the app has a tag at the bottom a red triangle indicating its run in production can some one point out which file I can tweak the env to make estatio run in production

You can enable production mode (rather than prototype) by editing the
'configuration' context-param in web.xml [1]
You will also most likely want to change the JDBC connection to some
external database, in persistor.properties [2]
This configuration can also be externalized outside of the WAR, [3]
HTH
Dan
[1] https://isis.apache.org/guides/rgcfg.html#_using_the_wicket_viewer.
[2] https://isis.apache.org/guides/rgcfg.html#_rgcfg_configuration-files
[3] http://isis.apache.org/guides/ugbtb.html#8.3.-externalized-configuration
On 11 April 2016 at 15:47, Tonderai Khatai<EMAIL_ADDRESS>wrote:

After deploying to tomcat the app has a tag at the bottom a red triangle
indicating its run in production can some one point out which file I can
tweak the env to make estatio run in production
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/estatio/estatio/issues/37


New sign up UI

mocks
image assets will be here
Blocked by: https://github.com/Nonprofit-Exchange-Hub/web-app/pull/272
expanding off of the existing sign up page (client/src/views/SignupCitizen.tsx) please style and add form inputs according the mock above

possible tabbing solution could be similar to existing in client/src/components/Users/Auth/SignUpUserAndNonprofit/SignUpUserAndNonprofit.tsx
does not need to make an api request, functionality will be handled in separate ticket

@jonathanchuangcodes this seems to be blocked by another PR. Ryan is working on that PR. Can you sycn with @jd2rogers2  to get another ticket to work on until 272 is cleared?

@jonathanchuangcodes do you have the open branch for this ticket that you are working on?


Faster control loop

It might be beneficial to provide classes that allow the trajectory following control loop to run independently of the WPILib Command-based framework and its 50Hz frequency constraint. This also means that if for some reason another command took too long to execute, the control loop would not be slowed down.

Added FollowerRunner interface and two implementations: SimpleFollowerRunner and TimedFollowerRunner. SimpleFollowerRunner has been tested to work well IRL. TimedFollowerRunner has not yet been tested. Unit tests still need to be written.

1e49ff6 has added unit tests for follower runners. This issue now closed.


feat(config): import Z-Wave Alliance Files batch 9

Batch 9 for volunteer review

I will checkout this batch, hopefully to find time to go over all files.

@blhoward2 I'm done, I hope everything looks good because english is not my native language.
In 0x0190\adc-s2000-t-ra.json the parameter 4 still needs to be done since I couldnt get my head around the Configuration Bitmask  stuff.


I want to be able to authenticate with username and password on K3OS

At the moment you can't connect to K3OS via ssh and use username and password although the password configuration is set. k3os expects an ssl certificate instead.

Duplicate of #207


User accordions for filters

The filters are kinda bloated, it would be better if we add collapsible sections so the users can focus on the most important stuff and quickly find what they want. It would also make it easier to add more options.

[ ] fix the gap between the "Message" button and the menu popover on Android
[ ] Add an empty state to the search

Will be deployed in the next version


Path type on routes

Hi!
Looking for something equivalent to the flask's path type of segment on routes
Something like:
/my/fixed/path/path:path/the//of/the/url
where if you ask for /my/fixed/path/this/will/be/part/of/path/the/thiswillberest/of/the/url
path will be /this/will/be/part/of/path
rest will be thiswillberest
Any ideas?
Thank

I don't fully understand. Can you point to a flask example or a give a more precise explanation? Ideally code or pseudocode explaining what feature you're describing would be helpful.

You want this https://github.com/channelcat/sanic/pull/632?

I'm closing this since we haven't gotten a response. Please feel free to reopen if the issue is not addressed.


RuntimeError: cannot reshape tensor of 0 elements into shape [0, -1]

H idiap,
Thanks for releasing your code, it really helps me. I'm testing your code on my own dataset, but caught the CUDA Memory Out error. So I follow this issue and change the meshing setting below:
meshing:
  level_set: 0
  resolution: 0.01 #cm # Increase this number to speed up meshing algorithm
  eval_rec: False
  mesh_bound_scale: 1.02

# change to:

meshing:
  level_set: 0
  resolution: 32 #cm # Increase this number to speed up meshing algorithm
  eval_rec: False
  mesh_bound_scale: 1.00

The Memory Out error disappears bug I got another problem:
Traceback (most recent call last):
  File "/home/yanchi/miniconda3/envs/eslam/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
    self.run()
  File "/home/yanchi/miniconda3/envs/eslam/lib/python3.7/multiprocessing/process.py", line 99, in run
    self._target(*self._args, **self._kwargs)
  File "/home/yanchi/qudelin/ESLAM/src/ESLAM.py", line 259, in mapping
    self.mapper.run()
  File "/home/yanchi/qudelin/ESLAM/src/Mapper.py", line 473, in run
    self.mesher.get_mesh(mesh_out_file, all_planes, self.decoders, self.keyframe_dict, self.device)
  File "/home/yanchi/qudelin/ESLAM/src/utils/Mesher.py", line 234, in get_mesh
    for i, pnts in enumerate(torch.split(points, self.points_batch_size, dim=0)):
  File "/home/yanchi/qudelin/ESLAM/src/utils/Mesher.py", line 170, in eval_points
    ret = decoders(pi, all_planes=all_planes)
  File "/home/yanchi/miniconda3/envs/eslam/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/yanchi/qudelin/ESLAM/src/networks/decoders.py", line 165, in forward
    raw = raw.reshape(*p_shape[:-1], -1)
RuntimeError: cannot reshape tensor of 0 elements into shape [0, -1] because the unspecified dimension size -1 can be any value and is ambiguous

It seems that they decoder reshape an empty tensor of shape (0, 4) at the forward step.
I have no idea how to solve this problem, could you provide me something about the setting rule of mesh_bound_scale, resolution, and bound? BTW, would the 3D reconstruction evaluation stay safe if  I do not render a 3D ply file?
I'm looking forward to your reply and thanks in advance.

Hello,
From the first glance, it seems like you have set the resolution too coarse.
You do not need to worry about the mesh_bound_scale. You need to set the bound correctly so that it encompasses your whole room size. Also, please note that you need to provide the model with the first-frame camera pose. It could be an arbitrary camera pose, but the scene bound should be enough to cover the scene w.r.t that arbitrary camera pose. You can then tune the resolution to not have an out of memory error for mesh extraction.
Lastly, the 3D reconstruction evaluation directly uses a 3D ply file. Unfortunately, you need to render that.
I hope this helps.


Potential work around for flaky builds

Windows builds fail intermittently for Windows on Github Actions.  For example:

https://github.com/input-output-hk/cardano-node/pull/2086/checks?check_run_id=1393807980

Retries might help avoid the problem.

bors merge


[PE-D][Tester C] Use of visuals for adding an appointment

Expected:
Thought that it will be nice to include a screenshot to show where the appointment will be added.

Labels: severity.VeryLow type.DocumentationBug
original: SherwynNg/ped#10

Good to fix.
Screenshots were included in UG in the latest updates.


fix swiftlint problems

Task/Issue URL: n/a
Tech Design URL:
CC:
Description:
BitRise updated their SwiftLint which introduced a few violations.
Steps to test this PR:

Update to SwiftLint 0.41
Build and smoke test the app

Internal references:
Software Engineering Expectations
Technical Design Template

LGTM!

Thanks @THISISDINOSAUR !


Fix nits in "Loading WebAssembly modules" article

What's changed, or what was fixed?

Fix nits

[ ] This has been reviewed and approved by (NAME)

[x] I have run gulp test locally and all tests pass.

[x] I have added the appropriate type-something label.

[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

:+1:

:tada: This has been pushed live to https://developers.google.com/web/


Make tsc read tsconfig.json

TypeScript now accepts a lot of config. I found it hard to pick up one that suits most use case, given that one popular framework adopts aggressive options.
So I create one maker that read tsconfig on the fly.  https://gist.github.com/HerringtonDarkholme/34b57329f04d4ec8df6c
I think this is the most sensible default configuration for neomake. It just works out of box. But it uses ugly json eval.
If you think this is acceptable, I will make a pr.

Hmmm. I'd almost rather create a custom executable for tsc instead. I'm making a few guesses here, but tsc uses nodejs, right? So, if you are running tsc, then you should also be able to run node scripts. Therefore we could have a script like this (pseudo code):
#!/usr/bin/env node

var args = [...];
var tsconfig = findFile('tsconfig.json');
var config;
if (tsconfig) {
    config = JSON.parse(readFile(tsconfig));
    for (option in config.compilerOptions) {
        args.push('--' + option, config.compilerOptions[option]);
    }
}

system.execute('tsc', args);

This is nice because:

It doesn't use the potentially unsafe eval as a json parser hack.
It uses tools that tsc users must already have.
It's not vimscript!

Thoughts?

Yes, it is more robust, and more flexible. The only concern is should we ship this launch script with neomake?

I think that would be ok if the script is kept pretty small (which is probably a good idea anyway).

I ran into this today – any resolution here? I'm curious if there's a way to run the make without specifying a file, and still populate the per-file errors/warnings. Specifying a file voids the tsconfig.json (docs), that's my guess as to why it's not getting picked up. Still toying with it.

I'm closing this issue and recommend users to try tsuquyomi.
After using both neomake and tsuquyomi long enough in my production environment, I have to say comparing neomake and tsuquyomi with regards to error reporting is not fair.
tsuquyomi is specialized for TS, it reports very fast, even in a large project. And, more importantly, it respects project configuration. Every thing works out of box.
Neomake is very good at spotting out errors across various file types, but for TypeScript, the not-so-fast-compiled-language, Neomake cannot perform as well as long running language service like tsuquyomi.
Though I still miss Neomake's side bar sign, they are so lovely.

@HerringtonDarkholme, first of all, thank you so much for mentioning tsuquyomi. I just got hired at a place working on enterprise software written in TypeScript and the app is monolithic. You have no idea how much time I have spent trying to configure my Vim settings and plugins to play nicely with numerous TypeScript files and have them compile correctly, include the right exports, understand namespace, etc. I've gone down so many rabbit holes the last two days its unbelievable.
Anyway, as you just mentioned in your last comment about Neomake's side bar error and warning signs, is there no way to make this a reality with tsuquyomi? Believe me, I'm just happy to finally have my tsc compilation working correctly within Vim, but it would be nice to not have the (obtrusive) Vim quick fix window. If there was a way to show the messages on a line by line basis in the navigation bar and simply have an error symbol on the line that is in question, that would be fantastic. While I do not have nearly enough experience with vim plugin libraries (I can barely get the ones that are already written to work properly, ha!), I would love to figure this out if it's a possibility.
Thanks again though, you saved me from wanting to throw my laptop out the window :)

And if you want to use neomake with tsconfig files[], that's the way for it(its bad though, i believe there is a better one):

Put this to vimrc

function! neomake#makers#ft#typescript#tsc()
return {
\ 'args': [
\ '-p', 'PATH TO tsconfig.json directory', '-m', 'commonjs', '-t', 'es2015', '--noEmit'
\ ],
\ 'errorformat':
\ '%E%f %#(%l,%c): error %m,' .
\ '%E%f %#(%l,%c): %m,' .
\ '%Eerror %m,' .
\ '%C%\s%+%m'
\ }
endfunction

And apply this change:

diff --git a/autoload/neomake.vim b/autoload/neomake.vim
index 1928eff..166cdd0 100644
--- a/autoload/neomake.vim
+++ b/autoload/neomake.vim
@@ -75,7 +75,7 @@ function! neomake#MakeJob(maker) abort
let args = a:maker.args
let append_file = a:maker.file_mode && index(args, '%:p') <= 0 && get(a:maker, 'append_file', 1)

if append_file

if (append_file && a:maker.exe != 'tsc')
call add(args, '%:p')
endif
call map(args, 'expand(v:val)')


[bug] Can't style composable parts, such as Dialog.Title

Describe the bug
import { Dialog as TDialog, styled } from 'tamagui'

export const Dialog = TDialog

Dialog.Title = styled(Dialog.Title, {
  fontWeight: 'bold',
  color: '$color12',
  fontSize: '$7',
})

I expect this usage to override Dialog.Title's styles. However, styled doesn't appear to work. This makes it hard to have a reusable dialog component.
To Reproduce
Lmk if you need a repro.

Hm, this works for me in the sandbox, even if i overwrite Dialog.Title:

Closing for now unless you see again.


Endpoints to prepopulate data from Eventbrite

WIP. Showcasing for review.

Probably cancel this given #28

No we merge this, then #28 becomes that one small commit


Port tests for constrained language mode

These tests have been ported from Windows code base and converted to Pester.
Note that some tests are currently pending because PowerShell Core does not yet support Windows platform application allow APIs.  This is being tracked by issue #4778.

@PaulHigin Please execute Feature tests as this PR include Feature tests.
https://github.com/PowerShell/PowerShell/blob/master/docs/testing-guidelines/testing-guidelines.md#requesting-additional-tests-for-a-pr

@JamesWTruher
You have not approved this PR yet.  Thanks.

@adityapatwardhan
All of the new constrained language tests passed.

@PaulHigin A rebase is need to resolve those failures. Can you please rebase and push. Thanks!

@PaulHigin you may also want to squash some commits while rebasing.

Ok. I have performed a rebase.  Hopefully the tests will pass.

@PaulHigin Thanks for adding these tests. 👍

@adityapatwardhan
Thanks!

@PaulHigin Some tests look to me should be able to run on Unix platforms, such as the following:
Describe "Switch -file in constrained language mode"
Describe "Get content syntax in constrained language mode"
Describe "Stream redirection in constrained language mode"
Describe "Invoke-Expression in constrained language mode"
...

But currently, all those tests are made windows only. Is there something prevents them from running on Unix now?
(I didn't go through all conversations in this PR, so sorry that if you already answered a similar question)


Update handler

Is there currently any way to use an update handler function written in a design document?
https://wiki.apache.org/couchdb/Document_Update_Handlers

Not at the moment unfortunately.

Are you still working on this project and planning to implement this?


Updated Team section

Navbar: @SamGuay My "local" spaces between the icon & the menu item are not there on the "non-local" site. Tried to replace the space with   --> change however you like or remove them altogether! 😅
Updated Team section:

[ ] @complexbrains @smoia --> I checked your awesomeee Brainhack Donostia website & saw that you had A LOT of volunteers! ❤ I added the 4 of you & the Brainhack Donostia Organizing Committee as a whole to also recognize smaller, indirect contributions of other Brainhack Donostia volunteers. What do you think?
[ ] @afurgoni20 @eurunuela @danalclop @smoia I added the information you gave me. If you would like me to change or add something, let me know! 😀 --> e.g. if you would like another (non-GitHub) avatar image, if you would like to add your email address and/or Twitter handle!
[ ] Super nice (or that's what I think): the order in which team members appear in the Team section is random => different every time you reload the webpage!

[ ]  @complexbrains @smoia --> I checked your awesomeee Brainhack Donostia website & saw that you had A LOT of volunteers! ❤ I added the 4 of you & the Brainhack Donostia Organizing Committee as a whole to also recognize smaller, indirect contributions of other Brainhack Donostia volunteers. What do you think?

I love this! Thank you!

[ ]  @afurgoni20 @eurunuela @danalclop @smoia I added the information you gave me. If you would like me to change or add something, let me know! 😀 --> e.g. if you would like another (non-GitHub) avatar image, if you would like to add your email address and/or Twitter handle!

Please add my email address (I see I can't push to this branch<EMAIL_ADDRESS>

[ ]  Super nice (or that's what I think): the order in which team members appear in the Team section is random => different every time you reload the webpage!

Cool! But I understand if you (global organizers) want to go first.

If it's not too much to ask, could you please add this picture for @afurgoni20 ?

Please add my email address (I see I can't push to this branch<EMAIL_ADDRESS>
Done!

Cool! But I understand if you (global organizers) want to go first.

Atm, I created two Team subcategories: Brainhack Global Organizing Committee (global organizers) & Volunteers (Brainhack Donostia team & (atm hypothetical 🙈) other volunteers) --> within each subcategory, the order in which team members appear is random. But I'm 💯 open to suggestions!!

@sofievdbos :100: thanks for having taken care of this. I would remove special characters (e.g. ñ or accented characters) from the filenames to avoid potential undesired effects: Eneko, Daniel and I have the chance of having one of those in our names :sweat_smile: . Thanks !

Hi Sofie, thanks a lot for the good work! My Twitter handle is also @danaclop (https://twitter.com/danalclop). And yep, the special characters of my surname might be a bit of a problem 😅 For papers I use 'Daniel Alcala-Lopez'


Can we get PE and PB ratio for a stock?

You can get Symbol P/E and Sectoral Index P/E through /api/equity/{symbol} API call.
Use pdSectorPe and pdSymbolPe under equity metadata.
Refer: https://hi-imcodeman.github.io/stock-nse-india/interfaces/interface.equitymetadata.html


Cancelling orders is broken

I cannot cancel any orders Either on the market trading pg, through the "You have open orders in a resolved Market" or through the portfolio pf, they always fail

not seeing this anymore - closing


feat(slate meta data): separate UI content

Goal
Create easily renderable meta-data without needing to know what is present in the lineup.
Todos

[x] Update Schema
[x] Update json configs
[ ] Connect the dots (unfamiliar with the setup here so I am sure I missed several things)

Review

[x] General @lfishhead

Implementation Decisions
In order for the front end to consistently render titles without fear of exposing internal naming (Coronavirus Slate vs Coronavirus) or getting conjoined returns of title/subtitle this just proposes a couple of new fields to add to the metadata.

displayTitle: This is the title to use for a given slate if required
displaySubTitle: This is the subtitle to use if required
displayTopic: This is the topic that will be used if required/available

I see this as a stopgap in the sense that we all wish for more and better copy associated with the slates and lineups, and even the ability to experiment with the copy itself.  As this repo is configured, any change to this information will require a PR and back end approval.  Copy isn't code.  We should design a better means for marketing or editorial to be able to make changes here that don't involve engineering.  Thanks to @collectedmind  for trying to move things forward in the meanwhile.

Copy isn't code

Agree wholeheartedly.  In the meantime, as we move to production with some of these slates, it will be a challenge to achieve the fluidity we want with lineups and slates without the stopgap.

I am thinking I would also like to ad a displayType but I am still working through the idea there and it is a bit of a rabbit hole

@lfishhead How would you feel about closing this PR?

The challenge of managing slate copy still unsolved. We are tracking this as a project in our backlog: https://getpocket.atlassian.net/wiki/spaces/CP/pages/2509897803/Draft+Data+Products+Meta+Data


Redefine environment variables from other pages

The Policy installation tutorial seems to assume that you start the OCM
installation in the quick start and then progress to the Policy Framework
and then the Policy Controllers in one terminal session. This isn't
always the case, so redefining the required environment variables makes
this clearer.

/cc @dhaiducek


lxd-images: Update simplestream parser for recent breakage

The Ubuntu simplestream format changed earlier today breaking our parser.
This branch makes the parser more tolerant to random fields disappearing
or getting removed (will return nothing rather than blow up).
I'm also keeping support for the old keys in the very likely event that
the simplestream change is reverted.
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

@rcj4747 FYI ^

@stgraber This was an unintended change and is considered a bug.  The version field should not disappear from the stream and the client can assume that it will be present.  The 'release_title' field, along with the 'release_codename', were added for display to users and was not considered for machine consumption (the pesky 'LTS' in there makes it more work to match with this in the 'release_title' field than the 'version' field).
The stream generation bug has been fixed and streams with 'version' re-generated.

Updated my branch to drop release_title, the rest is still relevant as it's basically just improving the way the parser handles failures.


fix(firebase_messaging): Set APNS token if user initializes Firebase app from Flutter.

Description
See title.
Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

@russellwheatley - should these same changes be made to firebase_auth see #7752 not fixed


Bridge networking

Codecov Report

Merging #160 into 1.1.0-alpha will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           1.1.0-alpha     #160   +/-   ##
============================================
  Coverage        16.73%   16.73%
============================================
  Files               15       15
  Lines             1034     1034
============================================
  Hits               173      173
  Misses             850      850
  Partials            11       11

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 b176c05...6d5be6a. Read the comment docs.


commitlint/config-angular no longer allows "chore"

Angular (and therefore @commitlint/config-angular) changed their allowed scopes, removing "chore" in favor of "ci" and "build".
Since I lint my commit messages on CI, all my greenkeeper PRs are failing. Obviously, I can fix this on my side by changing the config of all my projects, but I imagine I'm not the only one blindly following the angular commit guidelines and linting my commits.
(To be clear, I don't have any good ideas for what you might do to deal with this, but I thought it worth bringing to your attention)

what would be a good prefix for bumping a dev dependency in package.json? I use chore: for all kind of things, like edits on .gitignore, LICENSE or .travis.yml. I guess the latter could use "ci", but what about the others?
For reference, this is the commit introducing the change to angular: https://github.com/angular/angular/commit/dff6ee3

Yea, I'm not really sure. I think if I'm reading the angular commit correctly:

dependencies should be build(npm)
.travis.yml should be ci(travis)
.gitignore is probably supposed to be build(git)
LICENSE would maybe be docs(license) or docs(legal)?

These are all guesses and I feel like they replaced chore with two fields that don't cover everything chore used to. While changing greenkeeper to build(npm) seems like it would fix my issue, I'm guessing that'll break anyone who doesn't upgrade their commitlint rules. Though, I guess that's the point of greenkeeper... :)

yeah I wish they would have deprecated it for at least one major version :) Let’s see what  Greenkeeper team thinks, I’m just helping out

I believe that https://github.com/greenkeeperio/greenkeeper/pull/612 allows this issue to be closed as the message is now customizable if so desired.

Seems reasonable to me. I'm not presently using greenkeeper, so I'm no longer in a position to confirm.


A 🚚-load of Tests

More tests across the board, raising coverage dramatically
Tiny bugfixes

Current coverage is 82.39%

Merging #50 into master will increase coverage by +10.01% as of 1f57931

@@            master     #50   diff @@
======================================
  Files           21      21
  Stmts          402     409     +7
  Branches         0       0
  Methods          0       0
======================================
+ Hit            291     337    +46
  Partial          0       0
+ Missed         111      72    -39

Review entire Coverage Diff as of 1f57931

Powered by Codecov. Updated on successful CI builds.


Indentation removed on .toml files

Describe the bug
.toml files that are edited by Decap CMS have their indentation (leading whitespace) removed. For example, I start with this:
[params]
    firstName = "John"
    lastName = "Smith"

And after editing I get this:
[params]
firstName = "Joe"
lastName = "Bloe"

This may be desired in some situations, but for me, when I need to go back in and edit the file manually, it makes it much harder to read.
Applicable Versions:

Decap CMS version: netlify-cms-core 2.55.2, netlify-cms-app 2.15.72
Git provider: Github
OS: Debian 12
Browser version: Chrome Version 116.0.5845.96

CMS configuration
local_backend: true
backend:
  name: git-gateway
  branch: main # Branch to update (optional; defaults to master)
# logo_url:
# display_url:
media_folder: static/img
  # media_library:
  # name: 'disabled' # To disable the media library
public_folder: /img
collections:
  - name: 'config'
    label: 'Site Config'
    editor:
      preview: false
    files:
      - name: 'config'
        label: 'Contact Info'
        file: 'config.toml'
        format: 'toml'
        fields:
        - { label: 'Contact Info',
            name: 'params',
            widget: 'object',
            fields: [
              { label: 'First Name', name: 'firstName', widget: 'string' },
              { label: 'Last Name', name: 'lastName', widget: 'string' },
              { label: 'Email', name: 'primEmail', widget: 'string' },
              { label: 'Phone Number', name: 'primPhone', widget: 'string' },
            ],
          }

Additional context

@schaefsteven thanks for raising this topic. Indentation is optional in Toml, therefore having it vs. not having it is a matter of opinion. This is also potentially a breaking change for some users, who expect this type of formatting.
To conclude: let's keep the current formatting.

This seems like something that could be overridden with a custom formatter: https://decapcms.org/docs/beta-features/#custom-formatters. There's an example of how you could change the yaml formatter in the docs, toml would be very similar.

@mmkal thanks for pointing this out, I was not aware of this feature

This seems like something that could be overridden with a custom formatter: https://decapcms.org/docs/beta-features/#custom-formatters. There's an example of how you could change the yaml formatter in the docs, toml would be very similar.

Thanks for the advice! I will have to investigate this more when I have time, but yes, if I can swap out the formatter, that should be able to achieve the desired results.


Cleaned up details and usecase modularization a bit

Merge #2398 first!

found some issus. don't merge for now


Quartodoc failing on hydamo package

I can reproduce this locally with
pixi run quartodoc-build

  File "D:\repo\ribasim\Ribasim-NL\.pixi\envs\default\Lib\site-packages\quartodoc\builder\blueprint.py", line 173, in get_object_fixed
    raise WorkaroundKeyError(
quartodoc.builder.utils.WorkaroundKeyError: Cannot find an object named: HyDAMO. Does an object with the path hydamo:HyDAMO exist?

I don't understand why it cannot find this, this works ok:
pixi run python
>>> import hydamo
>>> hydamo.HyDAMO
<class 'hydamo.datamodel.HyDAMO'>

Perhaps it doesn't quite like our editable install in pixi.toml:
hydamo = { path = "src/hydamo", editable = true }

EDIT: also try upgrading quartodoc, since the current version doesn't work with griffe v1; https://github.com/machow/quartodoc/releases/tag/v0.7.6

This works now.


Add a function for combine/vcat with source names

Should we add a function like vcat(df1, df2, df3, source = ["X", "Y", "Z"]) which concatenates DataFrames while also labeling their origins? In this example, the output would be equivalent to doing:
df1_alt = copy(df1)
df1_alt[:source] = "X"
df2_alt = copy(df2)
df2_alt[:source] = "Y"
df3_alt = copy(df3)
df3_alt[:source] = "Z"

vcatted = vcat(df1_alt, df2_alt, df3_alt)

The idea is to preserve the semantics of Base.vcat, but allow the introduction of a custom column (whose name comes from a keyword arg) that denotes the origin of each subset of data.

Anyone still interested in this? Could be useful if someone wants to take a stab at it. @cjprybol ?

@oxinabox seems to also want it in join 😄.


Finish story

As a player I want one more map so that I can have more songs to start playing S2VX
Time breakups:

Aaron: 0:00:000 - 0:38:824
Alex: 0:38:824 - 1:15:882
Max: 1:15:882 - 1:53:162

I had time this weekend so I finished my portion of the story max.s2ry.txt

story (2).txt
Finished


feat(VarInt): add VarInt support using bn.js for bitwise operations

What this PR does / why we need it:
Add VarInt support in jellyfish using bn.js for bitwise operations. bn.js was used as default javascript doesn't support bitwise operations beyond 31 bits of Unsigned Integer due to Bitwise limitations of JS and performance issues. See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Bitwise_AND#description

The operands are converted to 32-bit integers and expressed by a series of bits (zeroes and ones). Numbers with more than 32 bits get their most significant bits discarded. For example, the following integer with more than 32 bits will be converted to a 32-bit integer:

Test vectors were directly taken from https://github.com/DeFiCh/ain/blob/a73f92ddef351750650a5eae46718d2f0ad2c7c5/src/test/serialize_tests.cpp#L178-L229, not all test where applicable hence not all were implemented.
Which issue(s) does this PR fixes?:

Fixes part of #1583

ok, let's check if I understood the problem correct @fuxingloh So the Jellyfish had CompactSize implementation as VarInt which is incorrect. now we have renamed all VarInt usage as CompactSize in Jellyfish which still should change to the actual VarInt, But this does not matter for now since < 127 it's the same encoding.
But since cpp side use real VarInt for fields like DCT_ID, we should move Jellyfish side usage to VarInt for 100% accuracy. I guess this will happen in the subsequent issues logged.

Yup, you're spot on with one minor detail. Jellyfish has COMPACTSIZE for implementation for VARINT as it was thought to be the only variable integer size implementation. Since VARINT wasn't actually exposed outside of leveldb operations at the original bitcoin client. Jellyfish just assumed the same which is clearly wrong since we need both variable integer encoding implementations for the different data types.
Yeah, we will move to use VARINT for 100% accuracy, it's actually done with #1588 (together with tests for when ID is >128), just waiting for this to be merged since it stacks it on top of this current PR.