Docs missing on allowInterop

https://api.dart.dev/stable/2.5.0/dart-js/allowInterop.html
The docs are missing. Maybe they are taken from https://github.com/dart-lang/sdk/blob/36648acfcc5062ad6d5e7bcd18e35e27714487ca/sdk/lib/js/dart2js/js_dart2js.dart#L700-L701
We do have a doc comment at https://github.com/dart-lang/sdk/blob/36648acfcc5062ad6d5e7bcd18e35e27714487ca/sdk/lib/_internal/js_dev_runtime/lib/js/dart2js/js_dart2js.dart#L546-L557

See also https://github.com/dart-lang/sdk/issues/24576

cc @kwalrath @jcollins-g
How do we generate the docs for api.dart.dev - I want to ensure that they show up if I move these doc comments.

I haven't built the docs in a long time, but it looks like this is the code:
https://github.com/dart-lang/sdk/blob/master/tools/bots/dart_sdk.py#L37
@keertip might know the most about how this works.

👍
Was able to run dart third_party/pkg/dartdoc/bin/dartdoc.dart --sdk-docs --output docs
I confirmed that the version of the file it's reading is sdk/lib/js/dart2js/js_dart2js.dart
I'll move the comments to that file.


[Coverage] Merge coverage info after each test

Right now we generate coverage and merge it back into the results object at the end after a test run. This has two disadvantages:

Coverage data is massive and requires a lot of memory
We take a significant amount of time to do this at the end of a test run

by merging coverage info after each test we can avoid both of these issues.

cc @clentfort who expressed interest to help out a bit.

I guess a good starting point for this might be packages/jest-cli/src/reporters/IstanbulTestReporter.js.
But there is a lot of code around for coverage and I'm not even sure if all related code is still alive. I.e in packages/jest-cli/src/Runtime/Runtime.js#getAllCoverageInfo should always return Object {} since this._coverageCollectors is initialized as Object {} and seemingly never reassigned.

It seems like we are already adding coverage information here: https://github.com/facebook/jest/blob/master/packages/jest-cli/src/reporters/IstanbulTestReporter.js#L25
but we also have formatTestResults for JSON printing, which is why we need to retain them. I'd like to take coverage off individual test results and get rid of them after processing them in the IstanbulTestReporter and potentially format them early?

Can I ask why this was closed? My team recently had to turn off coverage requirements in our CI because of process-out-of-memory exceptions. We have a lot of Jest tests :)
Tangentially, has any thought been given to using NYC over istanbul directly? From what I understand NYC wraps istanbul to make it easier to collect coverage from spawned processes. From a cursory glance it looks like they write coverage data from each process to disk instead of trying to hold it in memory or merging on the fly.

Sorry, it looked like we weren't going to focus on this anytime soon but @dmitriiabramov made a strong case to me to improve coverage support :)

I think what @dmitriiabramov is doing should be enough to make this reasonable.


Add Crystal lang syntax highlighting

Is your feature request related to a problem? Please describe.
I'd like to have support for the Crystal programming language in Lapce, as most of my work now is in Crystal.
Describe the solution you'd like
Adding Crystal syntax highlighting

Any updates on this?

Added to #272


How to access data in onMutated

Vue version
3.3.11
Link to minimal reproduction
https://github.com/MCLCG/vue3-new-feature/blob/4b3648f247638f147396e171f7e70a0036083665/project/src/components/OptionLiftCycle.vue#L56
Steps to reproduce
I want to obtain the data of the optional API in onMounted
import {onMounted} from 'vue'
export default defineComponent({
  setup() {
    onMounted(() => {
      console.log('onMounted', xxx.$data); // How to access
    });
  },
  data() {
    return {
      name: 'mike',
    };
  },
  mounted(){
    console.log('mounted',this.$data)
  }
});

What is expected?
Can be implemented without using getCurrentInstance
For example, You can use the 'props' attribute in the onMounted callback function to access data data.
onMounted((props)=>{console.log(props.data)})

What is actually happening?
Unable to retrieve data from props
System Info
No response
Any additional comments?
No response

You can't access data from Options API in setup. That's by design.
We strongly recommend not mixing those APIs.
I'll move this to Discussions so the community can help out with finding you a proper solution


How to log in wandb ?

Hey there, i see that tensorboard is the primary logging mechanism here. Perhaps if i wanted to log all the metrics that tensorboard is logging into wandb, how should I do that ?

You would just need to define a clu.Metric.MetricWriter for wandb. It looks like right now you'd have to then hack that into t5x.trainer.MetricsManager to actually use it. However, we could make it so you can pass in a different set of MetricWriters via gin.

@adarob thanks for replying. This sounds great! , however wandb has a way to absorb the tensorboard logger if it points correctly to it the runner/log file. In the T5X dir structure where would the  tensorboard file be ? also, does T5X pre-training script start logging to tensorboard by default or  do we need to set it ?


Scissors Paper Rock new project

Simple project for beginners

i want to add this for hacktoberfest

Thank you for your suggestion to add a rock/paper/scissors game! We are currently not expanding this to any new lessons and projects. But stay tuned for when we open contributions for additional lessons, and example web apps to build. For now sharing your creations within https://github.com/microsoft/Web-Dev-For-Beginners/discussions would be amazing!


AzureBlobStorage should use default client UserDelegationKey when _sasSigningCredentials is not specified

Fixes
Issue #
Description
desctiption goes here

[ ] I've made sure that this PR is raised against the develop branch.
[ ] I have included unit/integration tests validating this fix.
[ ] I have updated markdown documentation where required.

Sounds reasonable to me, just need to update docs


ODE solving with analytic Jacobians

Description
Solving ODEs with their sensitivities is numerically very intensive. Currently, the sensitivity calculation requires to evaluate the Jacobian wt to states and parameters in each step of the ODE integration. Right now this is done using nested autodiff. Experiments suggest that a >2x speedup is possible when providing these Jacobians in analytic form.
This issue suggests to add functions to stan-math which allow the user to pass in the analytic Jacobians of the ODE RHS wrt to states and parameters
Example
Right now solving ODEs is done with the call
integrate_ode_algo(ode_rhs, y0, t0, times, theta, x_r, x_i, rel_tol, abs_tol, max_steps)
This feature suggests to add an additional signature per algorithm which is
integrate_ode_algo(ode_rhs, Jy, Jtheta, y0, t0, times, theta, x_r, x_i, rel_tol, abs_tol, max_steps)
And Jy & Jtheta are the jacobians of the ode_rhs functor wrt. y and theta, respectively. The signature of the functors could be:
real [] : ode_rhs(real t, real[] y, real[] theta, real[] x_r, int[] x_i)
real [,] : Jy(real t, real[] y, real[] theta, real[] x_r, int[] x_i)
real [,] : Jtheta(real t, real[] y, real[] theta, real[] x_r, int[] x_i)
For efficiency considerations we may consider to pass the ode_rhs result into the Jacobians as well. There can easily be circumstances where this is beneficial and the function value anyway gets computed and will be available. So this is an alternative:
real [] : ode_rhs(real t, real[] y, real[] theta, real[] x_r, int[] x_i)
real [,] : Jy(real t, real[] y, real[] ode_rhs, real[] theta, real[] x_r, int[] x_i)
real [,] : Jtheta(real t, real[] y, real[] ode_rhs, real[] theta, real[] x_r, int[] x_i)
Another consideration is to let the Jacobian functions return eigen matrices instead. While it is more natural to return matrices as matrix types this would create some inconsistency.
How the correctness of the supplied Jacobian is assessed is unclear at this stage - if at all. From the perspective of the stan-math library it can make sense to assume that the supplied Jacobian is correct and no attempt is made to ensure it's correctness.
Expected Output
Same outputs, just a lot faster as nested AD is avoided during ODE integration.
Additional Information
Current Math Version
v2.18.0

What's the status of this issue?


[Bug]: Llama 3.1 405 B FP8 model is not support by vLLM (v0.5.3.post1) on AMD GPU

Your current environment
vLLM version: 0.5.3.post1 (For ROCm)
Model:  meta-llama/Meta-Llama-3.1-405B-Instruct-FP8
AMD MI300x GPU
🐛 Describe the bug

vLLM is throwing value error when loading meta-llama/Meta-Llama-3.1-405B-Instruct-FP8 on AMD MI300x GPU.
Value Erorr: fbgemm_fp8 quantization is currently not supported in ROCm. Refer screenshot for reference.

Hi @jerin-scalers-ai, thanks for reporting the issue. Currently, FP8 is not supported by AMD GPUs. The vLLM and AMD teams are working not it. Please stay tuned.

Thanks for clarification


Create a Simple Automation - Prerequisite links don't work

Tutorials: https://developers.sap.com/tutorials/irpa-simple-automation-helloworld.html
Both the Prerequisite links do not work.
When you click on them you get the message “Sorry, we couldn’t find that tutorial”.
It is not possible to start this tutorial since the Prerequisites are essential to provide the RPA environment.
Please fix the Prerequisite links to enable execution of this tutorial.
Kind regards,
Andrew

By the looks of it iRPA has been subsumed into SAP Build Process Automation, so maybe this tutorial should be removed.

This tutorial was removed.  Thank you.


Fix/arrowfunction with conditional expression body

Summary
Closes #1805
I found an incompatibility issue with Prettier when using arrow functions which body is conditional expression
I found this issue occurs when formatting arrow functions whose body is conditional expression
see below example code when linewidth 17
(someProperty)=>someProperty?'one':'two'

playground
I found prettier enforce space after => this token in this case.
And I tried to implement this without changing to FormatElement but failed. So, I added a HardSpace to FormatElement to implement this.
I am not sure if this approach is acceptable.
Also Documenting can be quite challenging.
Please review my code
Test Plan
add test case

You need to fix the conflicts, then we can merge it


chore: create preview environments for every PR using Uffizzi

This PR adds github workflows which will trigger uffizzi preview environments for PRs to this repo. A PoC has been created at https://github.com/waveywaves/livebook/pull/1. Once this PR is merged, you should be able to see comments like these on PRs to this repo. These comments are created after a preview env has been deployed for the PR.
fixes https://github.com/livebook-dev/livebook/issues/1541

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@waveywaves thanks! I dropped a couple naming suggestions and we can ship it :)


Update separate-of-concern.md

The English version is "No need to learn Kubernetes API deprecations from versions upgrade" for the first point of benefit explanation.
This fixed the explanation.

Description of your changes

I have:

[ ] Read and followed KubeVela's contribution process.
[ ] Update sidebar.js if adding a new page.
[ ] Run yarn start to ensure the changes has taken effect.

Special notes for your reviewer

close this PR, and open https://github.com/kubevela/kubevela.io/pull/1141 for current doc version


Inventory crash with Update 121

When attempting to modify a weapon with the Halloween 2016 update, Goonmod causes the game to crash. Crashlog:
Application has crashed: C++ exception
[string "lib/managers/menu/blackmarketgui.lua"]:814: attempt to index field '?' (a nil value)
SCRIPT STACK

Callstack:
     payday2_win32_release  (???)     ???
     payday2_win32_release  (???)     ???
     payday2_win32_release  (???)     ???
     payday2_win32_release  (???)     zip_get_name
                     ntdll  (???)     RtlAllocateHeap

Same thing. Opening MG42 and "Crash"

Do you have Gage Coin shop enabled? Probably a conflict with the new Continental Coin mod buy feature.
Also it's kinda hilarious how Overkill copies all the functionalities from GoonMod.

Should be fixed in r32. @Joker9944 I work for Overkill.


Some changes

You don't have to merge this yet as I'll be adding more commits soon. I just opened this so you can see some of the changes and additions I've made so far.

Oh no. I did something wrong while rebasing. Now there are duplicate commits.... I'll try to fix it.


Unable to Compile Kernel

Kernel 6.3 fails to compile with LLVM ( LLVM=1 LLVM_IAS=1 make):
make[3]: 'install_headers' is up to date.
  LD [M]  arch/x86/events/amd/amd-uncore.o
ld.lld: error: arch/x86/events/amd/uncore.o: invalid symbol index
make[4]: *** [scripts/Makefile.build:452: arch/x86/events/amd/amd-uncore.o] Error 1
make[3]: *** [scripts/Makefile.build:494: arch/x86/events/amd] Error 2
make[2]: *** [scripts/Makefile.build:494: arch/x86/events] Error 2
make[1]: *** [scripts/Makefile.build:494: arch/x86] Error 2
make: *** [Makefile:2025: .] Error 2

Let's try GCC & binutils (installed in /opt/gnu):
make[3]: 'install_headers' is up to date.
  LD [M]  arch/x86/events/amd/amd-uncore.o
ld: arch/x86/events/amd/uncore.o: bad reloc symbol index (0x6d >= 0x6d) for offset 0x3 in section `.text'
ld: arch/x86/events/amd/uncore.o: bad reloc symbol index (0x6d >= 0x6d) for offset 0x3 in section `.text'
ld: failed to set dynamic section sizes: bad value
make[4]: *** [scripts/Makefile.build:452: arch/x86/events/amd/amd-uncore.o] Error 1
make[3]: *** [scripts/Makefile.build:494: arch/x86/events/amd] Error 2
make[2]: *** [scripts/Makefile.build:494: arch/x86/events] Error 2
make[1]: *** [scripts/Makefile.build:494: arch/x86] Error 2
make: *** [Makefile:2025: .] Error 2

Or LLVM+elftoolschain? (LLVM=1 make)
make[3]: 'install_headers' is up to date.
  LD [M]  arch/x86/events/amd/amd-uncore.o
ld.lld: error: arch/x86/events/amd/uncore.o: invalid symbol index
make[4]: *** [scripts/Makefile.build:452: arch/x86/events/amd/amd-uncore.o] Error 1
make[3]: *** [scripts/Makefile.build:494: arch/x86/events/amd] Error 2
make[2]: *** [scripts/Makefile.build:494: arch/x86/events] Error 2
make[1]: *** [scripts/Makefile.build:494: arch/x86] Error 2
make: *** [Makefile:2025: .] Error 2

Maybe a typo or bug in kernel 6.3? Same errors for kernel 6.3.2
BUT kernel 6.1.8 compiles fine (LLVM=1 LLVM_IAS=1 make).

Let's disable CONFIG_PERF_EVENTS_AMD_UNCORE in the kernel config...
Compile goes a bit further but another error found:
make[3]: 'install_headers' is up to date.
  AR      arch/x86/built-in.a
  LD [M]  arch/x86/kvm/kvm.o
ld.lld: error: arch/x86/kvm/x86.o: invalid symbol index
make[3]: *** [scripts/Makefile.build:452: arch/x86/kvm/kvm.o] Error 1
make[2]: *** [scripts/Makefile.build:494: arch/x86/kvm] Error 2
make[1]: *** [scripts/Makefile.build:494: arch/x86] Error 2
make: *** [Makefile:2025: .] Error 2

Let's disable virtualization, CONFIG_VIRTUALIZATION...
so far no errors...

So far ...
6.1.8 -> No errors. Success
6.3.0 -> fail
6.3.2 -> fail
6.3.3 -> Fail

I haven't played around with LLVM in a while, but what about using Clang/Clang++ with GNU Binutils? I know it's not the ideal, but it would be a workaround while the LLVM and elftoolchain folks doesn't fix this bug.
Funny enough, Linux v6.1.8 compiles fine, per what you've said, so we could also just go with it instead of insisting in other versions.
By the way, what about v5.15.103? Does it also compile fine?

That's true. I have tried other combinations but not yet LLVM+Binutils. That will be my next test

Same issue with LLVM+Binutils, "invalid symbol index"

is it an issue with the linker?

is it an issue with the linker?

If kernel version 6.1.29 compiles fine, I would like to say no... unless something in the kernel source changed starting with 6.2.x and later.

Kernel updates are usually big, so maybe something did. Lets first try a diff linker (eg. mold), and if that doesnt work, follow alpine linux

update: mold requires a SSL library for libcrypto, and that doesnt need to be in a toolchain. probably compile libreSSL and  clang, and then compile mold, then proceed with the rest of stage2

Wow. Never heard of mold... Ha, I guess I might consider replacing LLD with mold.
Let's just say I'm not a fan of rust, so I'm impressed mold is coded in c++

Mold might not be the reason this is happening. There are some patches on chimera linux for the latest kernel, so lets try those

Wow. Never heard of mold... Ha, I guess I might consider replacing LLD with mold.
Let's just say I'm not a fan of rust, so I'm impressed mold is coded in c++

I'd also propose that we just use GNU Binutils' ld(1) instead of LLVM's lld(1), like https://github.com/ClangBuiltLinux/tc-build does.

Yeah, checking for Chimera patches might also be a good idea.

Mold might not be the reason this is happening. There are some patches on chimera linux for the latest kernel, so lets try those

Yeah, checking for Chimera patches might also be a good idea.

Per kisslinux.org:
It can not be used to link the Linux
kernel (due to lack of linker script support)

I only see kernel 6.1.38 for Chimera's linux-lts kernel and 6.1.32 for the linux-rpi kernel ... in the github repo

Well, what linker are they using?
I've suggested to use clang and later link using GNU's ld.bfd, unless 6.1.38 already supports LLVM's ld natively 👀 .
By the way, how the progress on using BSD's elftoolchain is going? I've read 'bout it on Musl-LFS's enhancement issues.

I've already tried LLD and ld.bfd... same problem. In fact, I cannot compile the latest kernel (starting with 6.2.x) with both GCC and clang. My GCC is installed in /opt/gnu but compiles other packages just fine (so it's not broken?).
I have yet to try a 6.2.x+ kernel build on an older MLFS system.
Last I checked, Chimera Linux still used LLD.

Looks like Chimera Linux uses 6.4.x for their kernel-stable... not sure how I missed it. I was only seeing their build for kernel-LTS. There are lots of patches used for the 6.4.x kernel source

Looks like that chimera-linux patch fixed those warnings when compiling:
warning: objtool: gelf_update_symshndx: Invalid argument

Kernel 6.5.4 compiled without errors, using no patches from Chimera Linux.
I am closing this unless someone needs to use kernels 6.2.x and 6.3.x

So far, I have stuck to Kernel 6.5.9 as it compiles fine under my previous CMLFS build (LLVM-15.0.5).
I built another CMLFS system but this time with LLVM-17.0.6 (and updated packages). Now kernel-6.5.9 wont compile.
I checked Chimera-Linux. Looks like at this time of this writing, Chimera is on Kernel 6.8.6.
I download that kernel and patches from Chimera. Kernel appears to compile fine. But when linking up vmlinux.o, build fails:
  LD      vmlinux.o
vmlinux.o: warning: objtool: elf_getdata: Invalid section descriptor
make[2]: *** [scripts/Makefile.vmlinux_o:52: vmlinux.o] Error 1
make[2]: *** Deleting file 'vmlinux.o`
make[1]: *** [/sources/chimera-6.8.6/linux-6.8.6/Makefile:1142: vmlinux_o] Error 2
make: *** [Makefile:240: __sub-make] Error 2

Issue fixed.  Now vmlinux.o links fine... when using Chimera's default x86_64 config


Add newValues parameter in updateOrCreate method

Let's say I have a conversation table with roughly the structure id, conversation, creator, last_msg_id, last_contacted_at
where creator is the user who spoke for the first time in this session, it is only updated in the first session (subsequent updates are not needed), so I can't put it in the original $values parameter of updateOrCreate (this will be updated every time overlay for the most recently spoken user)
adjusted
ImConversation::query()->updateOrCreate(
    [
        'conversation' => 'user1,user2'
    ],
    [
        'last_msg_id' => $message->msg_id,
        'last_contacted_at' => time(),
        // update everytime
    ],
    // This will only set to the model `firstOrNew ` on first insert
    [
        'creator' => $message->from_uid
    ]
);

placing the creator in the first array will affect the behavior and cause the same conversation to be created multiple times.
conversation is identification, creator is variable.


Introduce black formatter with pre-commit hook

I am suggesting using black with pre-commit hooks because

It is an IDE independent formatter.

It seems to be the most popular formatter in the Python community (just an estimation).

If we decide that we like using black, ideally all of our Python projects would use this setup. I can create a small guide on how to add black to PyCharm.
 Then the developer needs to stage the changed files and repeat the commit.

Created a requirements-dev.txt file. This will need to be selected to install dependencies when someone clones this project from scratch. I adapt our confluence instructions accordingly.

https://black.readthedocs.io/en/stable/integrations/editors.html#pycharm-intellij-idea instructions on setting up pycharm for black formatting.


bpf: Fix defines in policy.h

Commit 501332b ("bpf: Remove DROP_ALL define") forgot to remove an #else statement, resulting in a compilation error if policy.h is  included twice.
Marking for backports as I hit this while working on a v1.8 backport.
IMO, we should really enforce the use of #ifdef comments and indentations to avoid this sort of bugs.
Fixes: https://github.com/cilium/cilium/pull/5379

test-me-please

test-1.16-netnext

The previously failing test is passing. Marking as ready to merge.


Is this problem lifecycle?

Description
If we create post with tags then post_id auto injection. it's ok
// PostResource
public function relations(\Lomkit\Rest\Http\Requests\RestRequest $request): array
{
    return [
        HasMany::make('tags', TagResource::class)->requiredOnCreation(),
    ];
}

When we create tag by api/tags/mutate we need post_id
// TagResource
public function createRules(\Lomkit\Rest\Http\Requests\RestRequest $request) {
    return [
        'post_id' => 'required',
    ];
}

But now, we create post with tags relation, post_id is required

Hello @ThanhSonITNIC can you confirm this is still the case with "2.4.2" please ? There was a lifecycle problem before
I'll have a look after your confirmation to confirm this

yes, 2.4.2


Change the visibility of the resolveDefaultValue method to public

The resolveDefaultValue method in Outl1ne\MultiselectField\Multiselect class has a different visibility than the same method in its parent class Laravel\Nova\Fields\Field.
Please change the visibility of the resolveDefaultValue method to public.

Fixed in 4.4.0.


chore(deps): update pydantic_extra_types supported types

Updated supported pydantic_extra_types
Adds support for:

ISBN
LanguageAlpha2 and LanguageName
SemanticVersion

Adds missing unit test for the ULID type.
Checklist

[x] I have added new tests (if necessary).
[x] I have ensured that tests and coverage are passing on CI.
[x] I have updated any relevant documentation (if necessary).
[x] I have used a descriptive pull request title.

Pull Request Test Coverage Report for Build 00d1ad11-1531-4dce-9a07-9bdb6c8642b9
Details

0 of 0   changed or added relevant lines in 0 files are covered.
11 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.2%) to 90.594%

Files with Coverage Reduction
New Missed Lines
%

typing/pydantic_extra_types.py
2
91.67%

_internal/_types/click.py
9
94.84%

Totals

Change from base Build 8ce4b926-f8db-4afe-a719-cdbfb01e0dff:
0.2%

Covered Lines:
992

Relevant Lines:
1095

💛  - Coveralls


Changing intervals under Tasks in Settings doesn't update the UI until changing tabs

Describe the bug
When a user is on the Settings tab of Jellystat, and they change the interval for a Task, the UI will not update showing the new change until after the user clicks on another tab to take them away from the Settings page, and then back again.
Environment Details (please complete the following information):

OS: Ubuntu
Browser Edge
Jellystat Version: 1.1.0
Jellyfin Version: 10.9.2

To Reproduce
Steps to reproduce the behavior:

Go to Settings
Click on a different interval for tasks
Notice the task interval doesn'y update

Expected behavior
Expected Tasks intervals to update like the other dropdowns on the page.
Screenshots
N/A
Task Logs
N/A
Container Logs
N/A
Additional context
I'm using the nightly/unstable docker container version.

Hi, this has been fixed as per 1.1.0


Updating customconfig to string

Description

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

[x] Please add REST spec PR link to the SDK PR
[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

spec: https://github.com/Azure/azure-rest-api-specs/pull/3799

@dsgouda can you merge this? I believe all the comments were addressed


Support email as an inbound channel

We have started with a standard webform but the goal is the support many inbound request channels.
We have a need now to support email, so, email is next!
We already depend on SendGrid for transactional email, so, we will build this feature using SendGrid's inbound parse API:
https://docs.sendgrid.com/for-developers/parsing-email/setting-up-the-inbound-parse-webhook
At a high level, this feature will require the following:

The GovFlow instance administrators set up all the MX config as per SendGrid, and any other SendGrid specific config
We expose a new endpoint on GovFlow for SendGrid to post inbound emails to.
Based on the domain configuration, we establish a pattern of issuing inbound email addresses per jurisdiction, and (i) for parsing association with existing requests from the inbound payload (is an issue new or further correspondence on an existing ticket), and (ii) for parsing department or service information from an inbound payload (ideally via the email name)
New inbound requests go to the inbox
Inbound requests associated with an existing request go to the comment thread of the existing request
Support incoming images by uploading them into the image storage

Looks good!
For 3 (i) I'd consider using the subject line, with a user-friendly ticket number (related to #50). WDYT?

@amirozer

For 3 (i) I'd consider using the subject line, with a user-friendly ticket number (related to https://github.com/govflow/govflow/issues/50). WDYT?


Allow OrchestrationTemplate subclass to customize md5 calculation

Currently, md5 checksum is forcibly calculated on entire orchestration template text content:
OrchestrationTemplate.calc_md5(text)

But this is not always feasible. VMware vCloud, for example, yields randomly ordered elements in XML even when describing the very same orchestration template, hence the MD5 checksum differs. Therefore we need to be able to customize the way the MD5 checksum is calculated.
With this commit we make sure that calc_md5 function from the subclass is used:
ManageIQ::Providers::Vmware::CloudManager::OrchestrationTemplate.calc_md5(text)

Only this way we're able to customize what part of the XML will actually be used to calculate MD5 checksum.
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1553699
@miq-bot assign @Ladas
@miq-bot add_label enhancement,gaprindashvili/yes
/cc @bzwei @agrare @gberginc

@Ladas I assigned you since you're usually involved in refresher stuff. Please let me know if I'm wrong.

@bzwei if I understand correctly we only need to hit the green button here, right?
@miq-bot assign @bzwei

@gmcculloug can you merge it?


Refactoring

@marondeau-mila are you suggesting simply removing all classmethod decorators and having the functions be native instead?

The reason you added the classmethods was for testing. I'm saying that's a hack. The solution is to use mocking to test the methods directly. That is, modify the test to fit the code, not the code to fit the test.


Enhance Database Operation Resilience with Fallback Mechanisms

The current implementation of database operations lacks a robust error-handling strategy, resulting in potential issues and limited resilience to unforeseen errors. The absence of fallback mechanisms and detailed error logging hinders effective debugging and error diagnosis.
// No fallback if the operation breaks down,
public run(sql: string, params: any[]): Promise<RunResult> {
  return new Promise<RunResult>((resolve, reject) => {
    this.db.run(sql, params, function (err) {
      if (err) {
        console.error('Error executing query:', err.message);
        reject(err);
      } else {
        resolve(this);
      }
    });
  });
}

Adjusted in PR (#11) where a transaction mechanism is introduced for critical database operations, ensuring data consistency. PR includes methods for initiating, committing, and rolling back transactions, increasing the reliability of database interactions. You can use the transaction function to encapsulate operations in a single recoverable transaction block.


Installing weserv/images on heroku

Are there any instructions or would it even be possible to just build the nginx module?
I'm trying to build a buildpack for Heroku (which is based on ubuntu 18),
but I have no experience using CMake.
Is there a way to install this as an nginx module like:
./configure \
  ... nginx config options ...
  --with-http_mp4_module \
  --add-module=../weserv_images

Sorry for the delay, this dropped off my to-do list.
The easiest way to install nginx (with the weserv module built-in) is through CMake (this assumes that these dependencies are installed):

Details
# Clone the repository
git clone --recurse-submodules https://github.com/weserv/images.git /var/www/imagesweserv

# Install nginx (with the weserv module built-in) in /usr/local
# Note: this builds the separate API library and CLI tool in
# the project directory (/var/www/imagesweserv/{lib,bin})
cd /var/www/imagesweserv
mkdir build && cd build
cmake3 .. \
    -DCMAKE_BUILD_TYPE=Release \
    -DBUILD_TOOLS=ON \
    -DINSTALL_NGX_MODULE=ON \
    -DNGX_VERSION="1.17.6" \
    -DCUSTOM_NGX_FLAGS="--prefix=/usr/local"
make -j$(nproc)

# Test the CLI tool
cd ../
./bin/imagesweserv-cli public/lichtenstein.jpg x.jpg "w=200"

# Copy nginx config
cp -r /var/www/imagesweserv/ngx_conf/. /usr/local/conf

# Test nginx configuration file
/usr/local/sbin/nginx -t

If you prefer to build and install nginx yourself pass the CMake -DINSTALL_NGX_MODULE=OFF flag:

Details
# Clone the repository
git clone --recurse-submodules https://github.com/weserv/images.git /var/www/imagesweserv

# Install the separate API library and CLI tool in /usr/local
cd /var/www/imagesweserv
mkdir build && cd build
cmake3 .. \
    -DCMAKE_BUILD_TYPE=Release \
    -DCMAKE_INSTALL_PREFIX=/usr/local \
    -DBUILD_TOOLS=ON \
    -DINSTALL_NGX_MODULE=OFF
make -j$(nproc)
make install

# Make sure that the API library can be found during testing the CLI tool
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/local/lib64/

# Test the CLI tool
cd ../
/usr/local/bin/imagesweserv-cli public/lichtenstein.jpg x.jpg "w=200"

# Download nginx mainline version
curl https://nginx.org/download/nginx-1.17.6.tar.gz | tar xvz -C /usr/local/src

# Install nginx (with the weserv module built-in) in /usr/local
cd /usr/local/src/nginx-1.17.6/
./configure --prefix=/usr/local \
    --with-http_ssl_module \
    --with-http_v2_module \
    --with-http_realip_module \
    --add-module=/var/www/imagesweserv \
    --with-cc-opt="-I/usr/local/include" \
    --with-ld-opt="-L/usr/local/lib64 -Wl,-rpath,/usr/local/lib64"
make -j$(nproc)
make install

# Copy nginx config
cp -r /var/www/imagesweserv/ngx_conf/. /usr/local/conf

# Test nginx configuration file
/usr/local/sbin/nginx -t

(lib64 depends on the Linux distribution, you may need to change it to lib or lib/<multiarch-tuple>)
The reason that the API library (libimagesweserv.so) is separated from the nginx module is for easier testing (the unit tests and CLI tool link to this library) and faster deployment (no need to rebuild nginx each time).
I noticed that Heroku supports Docker, wouldn't that be easier? Our Dockerfile is approximately the same as our production environment (the only difference is that the rate limiter is disabled in the Dockerfile).

Hi @kleisauke, thanks for your feedback and your time!
I will try your suggestion to deploy a Docker Image to Heroku first and will get back with any findings that could be relevant in case someone else is interested.

So thanks to @kleisauke's input I managed to deploy this to Heroku! 🎉
Only a few changes are required in the Dockerfile as you can see in the following gist: https://gist.github.com/tkoenig/569ec0bf772500fe06372bd60747d96b
This is just a proof of concept where I replace the port and the hostname in the Nginx config file.
Since the port on Heroku is dynamic I need to replace the port before starting Nginx.
@kleisauke do you think it makes sense to add  a heroku.yml file and specific Dockerfile for heroku which deals with this? If so I'll be happy to prepare a PR with a cleaner Dockerfile.
So thanks again to @kleisauke for your feedback and this awesome project.

@kleisauke : I vote for a PR from @tkoenig with the heroku.yml, what do you think?

Thanks so much for the Docker file. We're currently using it with Heroku.
Has anyone had problems hosting this? I setup Cloudflare + Heroku but the server is still getting hammered and Cloudflare does not seem to be caching much.
Any kind of guidance would be appreciated, would be happy to post back here so it's part of the docs.

Figured it out, this could help others setup. It's just configuring Cloudflare properly (Cache Everything, TTL according to use case - in my case longest possible).
Would it be useful if I submit a new section to the 'hosting weserv/images yourself' documentation?

@cabe56 : If you want, you can suggest any addition to the docs by creating a pull-request at https://github.com/weserv/docs . Configuring specific services is a bit our of our scope, but Cache Everything is indeed used by ourselves.

I'm a bit reluctant to add things I never test or use (like Heroku configuration). We now have a prebuilt Docker image available on GitHub Container Registry, which should suit the needs of most users. In addition, we are doing tests with dynamic nginx modules (see https://github.com/weserv/images/issues/318#issuecomment-962633835 for more info).
Please use a custom nginx configuration if you want to deploy to Heroku. For example:
FROM ghcr.io/weserv/images:5.x

# Update RPM packages
RUN dnf update -y

# Overwrite default nginx config
# For example, change DNS settings:
# https://github.com/weserv/images/issues/206#issuecomment-567015037
# Or default user agent:
# https://github.com/weserv/images/issues/223#issuecomment-616219620
# etc, etc:
# https://github.com/weserv/images/tree/5.x/ngx_conf
COPY imagesweserv.conf /etc/nginx


Adding hindi translation

Hi, adding translations for Hindi which is one of the most widely spoken languages in India, and falls in top 5 languages wrt number of speakers.

Hi @RichardLindhout / @iM-GeeKy, please have a look at this PR when you get some time. Thanks.

Looks good! @RichardLindhout can take a second look and merge it. Thank you @sachinjnd!


create new user

I am trying to create a new user and stuck at this point....
Any help is appreciated.
step1: npm install hfc (passed)
var hfc=require('hfc')
var chain=hfc.newChain('testchain')
chain.setKeyValStore(hfc.newFileKeyValStore('tmp/kvstore'));
chain.setMemberServicesUrl("grpc://xxxxxxxxx_ca-api.blockchain.ibm.com:30303")
var reguser = {roles:["testrole"], enrollmentID: "someuserid01",account:"acct_1",affiliation:"00001",registrar:null};
so from here I tried 2 ways, both ended up with errors:
Method 1:
chain.registerAndEnroll(reguser, function(err){ console.log (err)})
Error: Registar is not set. (sure, bcos I passed null, so how do I set registrar?, I am only trying to add new user to the network)
Method 2:
chain.enroll("testuser","secret123", function (err,usr){console.log(err); console.log(usr);})
Error: Error:{"created":"@1470967403.041000000","description":"End of TCP stream","file:":""......., "grpc_status":14}]
undefined

Enroll WebAppAdmin and set this user as Registrar, you need a registrar in the system to enroll subsequent users.
    name: 'WebAppAdmin',
    secret: 'DJY27pEnl16d'
};
chain.getUser(registrar.name, function (err, userAdmin) {
      if (err) {
          process.exit(1);
      }
      // Enroll the WebAppAdmin user with the certificate authority using
       userAdmin.enroll(registrar.secret, function (err) {
          if (err) {
              process.exit(1);
          }
          chain.setRegistrar(userAdmin);
        })
      })```

Thanks!

@learner4 Thanks for submitting this issue. However, we have transitioned issue tracking to Jira https://jira.hyperledger.org/ Please transfer this issue there and then close this one. Thanks!


Convert interval to string to maintain compatibility with Rails/ActiveRecord 5.2

Hi,
The Interval data type is a limitation to use this gem with Rails/ActiveRecord 5.2.
For the Job and JobStat model, it could be converted to a String with minimal impact on functionality.
I think it is the only limitation for compatibility with 5.2, everything else worked well in my project.
Is it an acceptable compromise? It is a breaking change and it would require version bump.

Thank you @g-langlois !


preference for no sound effects is lost between sessions

Unchecking the preference 'play sound effects' not only results in a sound effect as you return to the main menu (tacky :) ) but it also does not remember this preference for the next app use.

(I'm assuming this is the app downloadable from Citi which installs itself as 'github')

This is a third party command line app that has nothing to do with the Citi provided app which is based on outdated and unsupported Flash technology. Your issue should be reported to Citi, not here.

OK thanks, FWIW, they just recently released a brand new rewrite which appears to avoid using Flash.  It announces itself not as Citibank but as Github (yes, the one downloadable directly from citi), and that brought me here.  Best!

I did not know that, I will checkout the new version. Glad you mentioned it. Thanks.

@therentabrain - I did check out that 'new rewrite'. It lasted for just a few weeks, then stopped working, and now Citi has removed it from their site. At least the online one now works without flash.

Yeah, it's been disabled for a few weeks now.  But yes, it works fine within the website, better than it used to...

But this project no longer works with it?

Nope, login fails...this project is dead.
On September 30, 2020 6:05:28 PM wicadmin<EMAIL_ADDRESS>wrote:

But this project no longer works with it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


feat(dal): remove direct node/edge weight accesses

In order to support the small graph and access to node and edge weights
through the caching layer, we cannot use the petgraph node_weight or
edge_weight methods directly in most cases. Instead we need to access
the weights through a method that will look them up in the caching layer
(or fetch it from the content store). This removes all direct calls to
the petgraph node_weight and edge_weight methods, replacing them
with calls to get_node_weight and get_node_weight_opt. These methods
are sync right now but will become async. Some other accesses remain
(calls to edges_directed, node_weights() and calls to
node_weight_mut, etc.) that also have to be made indirect.

bors r+

Good PR! 🎉


Refine public functions for core.

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

Project coverage is 92.35%. Comparing base (0ad1379) to head (c7eaed7).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #345      +/-   ##
==========================================
- Coverage   92.36%   92.35%   -0.02%
==========================================
  Files          52       52
  Lines        3771     3766       -5
  Branches      349      349
==========================================
- Hits         3483     3478       -5
  Misses        287      287
  Partials        1        1

Flag
Coverage Δ

unittests
92.35% <100.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


fix($compile): Fixes TypeScript compiler issues

TBH, I'm a bit surprised this all works ( the global trick isn't working in my project) but I've fixed all your compiler errors  :D

I figured it out, it's complicated

I've changed the setup ^

Thanks @orta, LGTM 😄


⚠ RPC server - Arbitrum has degraded performance

In f977ca4, RPC server - Arbitrum ($ARBITRUM_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 182 ms

Resolved: RPC server - Arbitrum performance has improved in fef5a64 after 6 minutes.


Add files via upload

My first project as a programmer.

The calculation of work commission according to sales amount that is calculated.


Replaced xlrd with openpyxl

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Cloudfront 403 for Nextjs 13.4 server action POST

If using server actions in nextjs 13.4 this Cloudfront error comes up.
403 ERROR
The request could not be satisfied.
This distribution is not configured to allow the HTTP request method that was used for this request. The distribution supports only cachable requests. We can't connect to the server for this app or website at this time. There might be too much traffic or a configuration error. Try again later, or contact the app or website owner.
If you provide content to customers through CloudFront, you can find steps to troubleshoot and help prevent this error by reviewing the CloudFront documentation.
Generated by cloudfront (CloudFront)
Request ID: opLHZwbJQQ44vAxAjWZ_oHvGKi_OKMKiBdR0a5ek2AZryutDxUdnrQ==

I was able to solve this by changing cloudfront Default(*) behaviour to allow POST methods manually and I'm wondering if I can somehow fix it in the code. Tried with two stages and both had the same issue

Suggested solution from AlcaponeYou#8730:
We can just set the default http method to allow ALL.
for now, pass in cdk.distribution.defaultBehavior.allowedMethods = AllowedMethods.ALLOW_ALL

Fixed in v2.16.3


Decouple the resizer

This commit adds a new Resizer interface with the resizer from nfnt as the default in its own package.
This allows people to use the smartcrop library with their own resize implemention.
My motivation for this is to use this in Hugo. We use github.com/disintegration/imaging to do image processing -- github.com/nfnt/resize is probably equally good, but imaging had a richer API for the Hugo use cases.
I have tested this library and I'm really impressed by the results (and the speed):
http://hugotest.bep.is/resourcemeta/smartcrop/
The relevant Hugo PR is here:
https://github.com/gohugoio/hugo/pull/4376
I know this is breaking some API, but I think people are happy to adjust as long as it is for a greater good. And this is < 1.0 software, and it is super-easy to upgrade.
Again, tanks for this great piece of software.

Coverage increased (+0.2%) to 83.934% when pulling 6921d0c42fe65a18e2bea5c255072d414d074dff on bep:resizer-decouple into 9032446b30f6284f8718490b6bcff8a1d51173cf on muesli:master.

The Travis failures seem to be unrelated.

@bep The Travis issue is indeed unrelated to your changes. I'm in touch with their support, as it affects pretty much all of my projects currently.

Awesome, looks good to me. Thanks for the nice contribution! Do you plan to commit your alternative resizer as well, or do you want to keep it in your own repo / package? (I can see perfectly valid reasons doing it either way)
Looking forward to seeing this supported in Hugo!

Do you plan to commit your alternative resizer as well, or do you want to keep it in your own repo / package?

That wasn't the plan. Not sure how useful that is ...

That wasn't the plan. Not sure how useful that is ...

That said, if you want me to, I could do so, no problem.

@bep No, I think it's fine. It might still be useful to others, so maybe we should at least link to its source. I assume it's in a public repo anyway?

I assume it's in a public repo anyway?

Yes. https://github.com/gohugoio/hugo -- but it's still, for natural reasons, in pull request mode in that repo, too.

Yeah, we should point to that source once it's all merged on both sides. Speaking about it: I'm gonna click a little merge button here now. Thanks for the contribution again!


Load configuration from vue.config.js

Resolves #116 and #145
It appears vue.config.js simply wasn't getting loaded. This fixes it.

Thank you. 1h of testing with the config file, started adding unexpected tokens to see if anything was loaded at all. This fixes the problem!

Are there any problems with this PR. I see it's not merged yet. Will hack in node_modules for now

I hate doing this, but it seems it's needed to get eyes on this PR.
cc/ @yyx990803 Could you please review this? Thanks, and I love the work you've done with vue 👍


Allow for smaller than 40x40 icons

Fenix 5s and Fenix Chronos need 36x36 icons

Also wanted for vivoactive 4s

https://forums.garmin.com/developer/connect-iq/f/discussion/255433/widget-launcher-icon-multiple-resolutions/1219039#1219039

https://developer.garmin.com/connect-iq/core-topics/build-configuration/


AI bros belike "lol I can generateARTS by simply writing prompts" while tryin' hard to mimic the labor done by REAL ARTISTS doing REAL ARTS

Life's good, you AI guys don't need to be so self-abased.

The ultimate goal of AI research is to pass the Turing test, which is to pretend to be a human, so pretending to be a human artist is the normal way to go

The ultimate goal of AI research is to pass the Turing test, which is to pretend to be a human, so pretending to be a human artist is the normal way to go

Nah from what I'm seeing this is more of a try-hard attempt to impress someone made by somewhat incompetent people than, let's say, a legitimate AI research.
Hell, even the quote "the ultimate goal is to pass the Turing Test." Sounds like a try-hard attempt to impress Turing himself deep under the grave🪦

Sir, this is a Wendy's

Sir, this is a Wendy's

and I'm here for the beef

duplicate of #2

duplicate of #2

I'm not discussing about the project's legitimacy, I'm just saying you people should be more confident about yourselves and to not be afraid of accepting your incompetences as they're also a part of who you are.🥰🥰🥰

duplicate of #2

I'm not discussing about the project's legitimacy, I'm just saying you people should be more confident about yourselves and to not be afraid of accepting your incompetences as they're also a part of who you are.🥰🥰🥰

looks same.

duplicate of #2

I'm not discussing about the project's legitimacy, I'm just saying you people should be more confident about yourselves and to not be afraid of accepting your incompetences as they're also a part of who you are.🥰🥰🥰

looks same.
then you should look better, or learn the basics of reading

anyway only Illya can decide to close it or not. Forget my comment :p

I think installing Gentoo might fix this

ESL above me

duplicate of #2

I'm not discussing about the project's legitimacy, I'm just saying you people should be more confident about yourselves and to not be afraid of accepting your incompetences as they're also a part of who you are.🥰🥰🥰

I don’t think drawing large amounts of anthropomorphic animals demonstrates competence in the first place. Regardless of thoughts on ml image generation I’d trust someone able to generate decent images locally with any task over 99% of Twitter artists who have to rely on having quarters thrown at them to sustain themselves while they create “art” consisting of incredibly inappropriate subjects that oftentimes border on illegal.

The ultimate goal of AI research is to pass the Turing test, which is to pretend to be a human, so pretending to be a human artist is the normal way to go

Nah from what I'm seeing this is more of a try-hard attempt to impress someone made by somewhat incompetent people than, let's say, a legitimate AI research.
Hell, even the quote "the ultimate goal is to pass the Turing Test." Sounds like a try-hard attempt to impress Turing himself deep under the grave🪦

Get a life, preferably one that doesn't revolve around drawing furry porn

The ultimate goal of AI research is to pass the Turing test, which is to pretend to be a human, so pretending to be a human artist is the normal way to go

Nah from what I'm seeing this is more of a try-hard attempt to impress someone made by somewhat incompetent people than, let's say, a legitimate AI research.
Hell, even the quote "the ultimate goal is to pass the Turing Test." Sounds like a try-hard attempt to impress Turing himself deep under the grave🪦

Get a life, preferably one that doesn't revolve around drawing furry porn

I evoked the fear of furry lying down deep inside AI bro's instinct without even mentioning the word "furry" once, this PTSD thing of AI bros is going insane, are you guys ok? Cause you guys should definitely go see a therapist.

Wait, where did #2 and #3 go? @JapanYoshi you did something?

Wait, where did #2 and #3 go? @JapanYoshi you did something?

Probably got reported, that's the open-source and open-minded spirit of AI bros right there, folks.

Probably got reported

But the author of #7 remains as ghost. GitHub deleted that user and its entire issues? What is the difference?
Are you familiar with GitHub's Term of Use?

Probably got reported

But the author of #7 remains as ghost. GitHub deleted that user and its entire issues? What is the difference?
Are you familiar with GitHub's Term of Use?

Hmm, that's weird......

I'm afraid that even a 10-years-old user will be erased from Time by a single activity...

Wait, where did #2 and #3 go? @JapanYoshi you did something?

Probably got reported, that's the open-source and open-minded spirit of AI bros right there, folks.

It's completely reasonable. I'm sure you've heard of the paradox of tolerance? An open-minded group can only remain open-minded and kind if it excludes intolerance and hateful individuals. Bullying and harassment is not welcomed here.

Wait, where did #2 and #3 go? @JapanYoshi you did something?

Probably got reported, that's the open-source and open-minded spirit of AI bros right there, folks.

NOTHING MORE; NOTHING LESS.

Good news - discussions are now closed
So when for the issues? 😂

At least illya is watching it

Good news - discussions are now closed
So when for the issues? 😂

lmaoooo as expected! These guys are pretty insecure, aren't they?

@PPAngel345 @Yosshi999 @FieryLionite
Owner of repo has full right to delete the whole issue on github (instead of only closing them, e.g. in gitea; but admins can do that anyway)
So I guess it probably gets deleted.

@ARessegetesStery Oh absolutely, that definitely happened - but I guess the only way forward is to not stop talking about this. I mean, we do keep ratio-ing these guys so that's nice, but staying quiet is the enemy here.
They can control/delete the discussions here, but not on twitter/other social media. I know it has caused heated discussions, at least in Japan.

@ARessegetesStery Oh absolutely, that definitely happened - but I guess the only way forward is to not stop talking about this. I mean, we do keep ratio-ing these guys so that's nice, but staying quiet is the enemy here.

Ah yes, lets keep the heat on and harass, bully another community nonstop so that they'll bend to our will. Oh wait who are the bad guys again??

@ARessegetesStery Oh absolutely, that definitely happened - but I guess the only way forward is to not stop talking about this. I mean, we do keep ratio-ing these guys so that's nice, but staying quiet is the enemy here.

Ah yes, lets keep the heat on and harass, bully another community nonstop so that they'll bend to our will. Oh wait who are the bad guys again??

I don't think anyone believes you're going to change your mind, but speaking out about things you think are wrong or right is a pretty normal thing to do, that's why protests and boycotts exist. People have the right to talk about things they don't like, just as much as you have the right to oppose. I've made it very clear I'm inherently pro-AI, but against the genai usage of datasets without restriction as well as pretending work that was done by a machine was in fact done by yourself. I don't really think it's a hard concept to grasp.

Hi all, as an action of GitHub ToS we must lock this thread.
For future comments please follow the instructions here.


Add integration test for GitHub Actions CI patterns

I was also going to test UV_PYTHON={Python_Root} or whatever but idk this seems like the pattern I'd recommend.


Improve UI of search bar in Landing Page issue#704 resolved

Improved Search bar section by added some CSS properties on it to look it good.
Add search icon embedded on search bar
Make it responsive in all devices.

@panwar8279 Please review it linked with issue #704

Please Create new PRs for this because it contains conflict issue.

Please Create new PRs for this because it contains conflict issue.
@panwar8279 Please check new PR #965 .


Compiled bindings don't seem to work with MultiBindings

If you add the following label to the StackLayout in the "Use compiled bindings" section
      <Label>
        <Label.Text>
          <MultiBinding StringFormat="{}Hue = {0:F2}, Saturation = {1:F2}, Luminosity = {2:F2}">
            <Binding Path="Hue"/>
            <Binding Path="Saturation"/>
            <Binding Path="Luminosity"/>
          </MultiBinding>
        </Label.Text>
      </Label>

and then change e.g. "Hue" to "Hue2" as binding path, the file still compiles.
On the other hand with the verbose writing method of a normal binding
      <Label>
        <Label.Text>
          <Binding StringFormat="{}Hue = {0:F2}" Path="Hue"/>
        </Label.Text>
      </Label>

changing the Path to "Hue2" causes a compiler error as expected.
I don't know if this is a compiler bug or whether this is information missing in the documentation similar to the note saying "Compiled bindings are disabled for any binding expressions that define the Source property. This is because the Source property is always set using the x:Reference markup extension, which can't be resolved at compile time"

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

ID: d8079bc8-3987-9069-717e-c5cbf3512ae1
Version Independent ID: d8079bc8-3987-9069-717e-c5cbf3512ae1
Content: Compiled bindings - .NET MAUI
Content Source: docs/fundamentals/data-binding/compiled-bindings.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @Naryorill
I've chatted to the engineers and as you've identified, compiled bindings don't work with multi-bindings. Therefore I'll update the doc with a note to that effect.
The engineers have an issue to add compiled bindings support to multi-bindings, but it's not something that will happen in the short term.

"*{0} ({1}) {2}" works
" {0} ({1}) {2}" leading space makes a compile error
"{0} ({1}) {2}" different compile errors with nothing between " and {


Is it possible to use capabilities rather than the catchall --privileged?

--cap-add NET_ADMIN for instance?

Good idea, I'm not sure. Could you perhaps try and let me know? I can then update the documentation (or you could submit a PR).
Please note that if some ipv6/ip6tables related kernel modules are not loaded, ipt6tables will automatically load them on first start. This requires mounting /lib/modules (see README), but probably also requires the --privileged.

Closed due to inactivity.

Just a note. I'm using this way:
    cap_add:
      - NET_ADMIN # Perform various network-related operations.
      - NET_RAW # Use RAW and PACKET sockets.
      - SYS_MODULE # Load and unload kernel modules.

It is working for me, but I think NET_RAW is not really needed.
I still need to test it more.
[]'s

@robbertkl Could we reopen this issue? I can submit a PR to update the readme from @navossoc's findings if you like, but don't have a VPS to test at the moment.

Sure, I'm always open to PR's. Especially if it's just a README addition. I've reopened.

It worth mention in README that CAP_ SYS_MODULE is only needed when we bind mount /lib/modules and let ip6tables load necessary kernel modules for us.
Personally I prefer echo ip6_tables >/etc/modules-load.d/ipv6nat.conf to load necessary kmod on boot, and reduce capabilities required by ipv6nat container.

CAP_NET_RAW is required by iptables, but I think it can be avoided without losing any functionality.
If anyone want to patch iptables: (I didn't test it, just memo)

PF_NETLINK accept both SOCK_RAW and SOCK_DGRAM according to netlink(7)
netfilter sockopts should be reachable from tcp/udp sockets, SOCK_RAW can be avoided.

Or replace github.com/coreos/go-iptables/iptables with something that talks to netlink directly without invoking iptables userspace program.

Thanks @zhangyoufu, those are very useful additions. Could you send me a PR with suggested README changes regarding CAP_SYS_MODULE?
Getting rid of requiring CAP_NET_RAW would be great as well, but I’m not sure the proposed changes are worth it right now. If you think this can be done easily, I’m always open to PRs!

@robbertkl
I proposed a PR #54 on README changes.
I have a PoC about iptables without CAP_NET_RAW @ https://github.com/zhangyoufu/alpine, please checkout and try. I may send patches to upstream.


🔍 Implement the profile search

Users can search for developers based on their username of full name.

I want to work on this

I want to work on this

Sure, go ahead, @Te-Stack. ✅


Fix DateRangePicker not passing onDay handlers

Originally the DateRangerPicker did not map the onDay handlers correctly.
This passes the handlers properly to the next component.

Coverage remained the same at 86.384% when pulling 71591774a77566a84df7a6d57e8bc604a425aa3d on omrimor:allow-passing-day-handlers-to-date-range into b662dd245c1663454ddd1617784824a70845d280 on airbnb:master.

Hi @omrimor. This was actually an error on my part from a past refactor. The DateRangePicker does not, by design, take these props (and in fact, even if we were to pass these on as you suggested, the DayPickerRangeController does nothing with them, so the point is moot). The correct solution is actually deleting lines 200 through 204 in the original code. I have that code ready in another commit.
If you would like to have your own logic for the CalendarDay user interactions, I would recommend using the DayPicker as is.

Hi @majapw. Thanks for the replay. I want to use the DateRangePicker in our project but extend the functionality based on our UX design requirement.
The logic there is exactly what we're looking for but we want to be able to use hooks for the mouse enter so we can perform added logic on hover.
I was hoping to avoid writing all the logic that is needed when using the DayPicker component.
also, when using the onPrevMonthClick/onNextMonthClick functions, is there a way to get current month viewed to preform added logic?


Middleware: Inconsistency between UI and Hawkular-API Deployments List

After deploying a new .war, and then later undeploying that same .war - the UI continued to show the new deployed .war, while Hawkular-API no longer showed the .war in it's list.
Seems that Hawkular Serices should be the correct frame of reference, as it is the Middleware manager?
Deployment list validation was done via the MIQ UI and Hawkular API.
Repro:

Add a Hawkular Provider
Deploy one of war/jar/ear, and then wait until the new Deployment is displayed and Enabled in the MIQ UI
Via the Hawkular-API, validate that the new Deployment is returned in the Deployments list
Undeploy the previously deployed Deployment, and wait some time (up to 5 min)
Refresh the Provider, and then note that the Deployment remains in the UI Deployments list.
Again via Hawkular-API, get the Deployments list, and validate that the Deployment is NOT in the list

Yes, I would say this is a duplicate of issue #9769.  PR #10287 is the MIQ-side fix but there is also work on the Hawkular side.

@mattmahoneyrh please see above, it seems this is a duplicate. if indeed so, please close the issue.

Closing due to jshaughn's note on this issue being a dupe.


AWS Security Hub Alerting

User Story
As a PM
I need to understand what security risks we have. As part of this I would like all critical and high AWS Security Hub alerts to be sent to slack - https://moj.enterprise.slack.com/archives/C03CY6451QT.
This should cover all accounts that we are responsible for.
Value / Purpose
This will mean we have a more secure platform
Useful Contacts
No response
Additional Information
Can we use AWS Chatbot integration for this? @richgreen-moj recently created a module for integrating AWS Chatbot with Slack so he may have some thoughts on how to achieve this.
Can we alter our existing PagerDuty integration to filter the critical and high alerts to the high priority channel / SNS topic?
Definition of Done

[ ] Choose appropriate method to send alerts (probably PagerDuty)
[ ] Critical and High alerts are raised in slack in #modernisation-platform-high-priority-alerts channel

this is a duplicate of https://github.com/ministryofjustice/modernisation-platform/issues/8076 - so closing


Add borders and aspect ratio support to resolution_t

With this PR, there's now a way to tell libdragon to have no borders in the output (useful for emulators and modern TV's)
Also with this PR users can add hardware VI borders around the framebuffer to midigate overscan on most CRT TV's
Borders scale the image down instead of cropping it, so that the framebuffer is fully visible.
There's also now an aspect ratio setting which can be used to achieve letterbox widescreen.
With this PR NTSC/PAL/M-PAL systems and monitors should output identical pictures.

I'm working on this with @SpookyIluha, we are testing many edge cases. Putting back to draft status for now

A very similar API change was merged here:
https://github.com/DragonMinded/libdragon/pull/646
This PR also attempted to reverse engineer the actual VI border formula but that part will need to be rewritten as part of the new VI library anyway, so keeping this PR open doesn't help much at this point.


Hook the script paths into the runner drive system

This will allow people to access the scripts using paths like script:/root.toy rather than having raw access to the disk.

Done, hopefully correctly.


Raw query without escaping?

Is there a way to create a query with no escaping?
knex.schema.raw("CREATE UNIQUE INDEX idx ON activities ( (DATE(created_at AT TIME ZONE 'UTC')), user_id, type) WHERE (type = 'daily_visit'))")

results in:

CREATE UNIQUE INDEX idx ON activities ( (DATE(created_at AT TIME ZONE \'UTC\')), user_id, type) WHERE (type = \'daily_visit\')

and I need:

CREATE UNIQUE INDEX idx ON activities ( (DATE(created_at AT TIME ZONE 'UTC')), user_id, type) WHERE (type = 'daily_visit')

The docs show this example:
knex.schema.raw("SET sql_mode='TRADITIONAL'")
  .table('users', function (table) {
    table.dropColumn('name');
    table.string('first_name');
    table.string('last_name');
  })

Outputs:
SET sql_mode='TRADITIONAL';
alter table "users" add column "first_name" varchar(255), add column "last_name" varchar(255);
alter table "users" drop column "name"

but my output shows:
SET sql_mode=\'TRADITIONAL\';

Where are you seeing that output? I'm wondering if it's just an artifact of stringifying the sql value

I see that when I look in console after running this migration:
exports.up = function(knex, Promise) {
  return knex.schema.raw("SET sql_mode='TRADITIONAL'").debug();
};

{ __cid: '__cid1',
  sql: 'SET sql_mode=\'TRADITIONAL\'',
  method: 'raw',
  bindings: [] }

@tgriesser Thanks for looking into this but it was total user error.


Extract AttachmentRedirectUrlUpdater logic into worker

This PR extracts the logic from the AttachmentRedirectUrlUpdater service listener into a new worker. The motivation for this is to ensure that the data required to do the work is fetched at the time the work is done rather than when the job is added to the queue. This is best practice in general for asynchronous jobs and it also means that we can use the worker to help migrate the metadata for existing whitehall assets to asset manager (where we anticipate adding a lot of jobs to the queue).

Rebasing on master in preparation for merging.


Handle non-presence of artifacts without exception

Eliminating messages like this one

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


please post package in downloads

Would you please post a zip package of the WikiNotes source in the
downloads area, for convenience?  THX.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 8 Mar 2008 at 2:27

I have put a zipped version of the source code for WikiNotes for Android in the
downloads area. However I would encourage everyone to use subversion to get the
source instead of the zip file if at all possible. I will try and remember to
update
the zip file on important changes, but the subversion repository will always
represent the latest sample code.

Original comment by<EMAIL_ADDRESS>on 12 Mar 2008 at 11:47

Changed state: Fixed


Is there an already built app for this project.

When trying to build this tool, it didn't run, so is there an already built tool to use right away

Not yet.


measurement table additional columns

For CohortConstructor it would be really helpful to have these 3 additional columns in the measurement table: value_as_number, value_as_concept_id, and unit_concept_id

To make sure all tables contain all the columns:

omopgenerics::omopColumns("person", required = TRUE)
omopgenerics::omopColumns("person", required = FALSE)
omopgenerics:::fieldsTables

always create all the columns


gateway should allow additional configs

Hi @shefaliv, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (shvohra). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Update const.py unit change of electric meter

Added
UNIT_VOLTAGE = "V"
UNIT_AMPERE = "A"
UNIT_WATT = "W"
UNIT_ENERGY = "kWh"
and changed the electric meter units to match the correct one (used to be °C)

I wanted to add energy class and last_reset so the entity can be added to the energy dashboard but can't get it working on my installation...

Thanks for the PR, I don't have a power meter installed in mine so I missed these, will take a look at getting it to work with HA energy dashboard.


RUNTIME mismatch

jobsub_lite uses the RemoteWallClockTime classad for a job to get its RUNTIME attribute for running jobs in jobsub_q output.  A user rightly noted that this doesn't match the output of condor_q.  This seems to be because condor_q, when run, is querying its own internal database, and may NOT be updating the classad:
https://github.com/htcondor/htcondor/blob/ae1a4a42d6e4f8994575a67ff5217e131e713f6c/src/condor_q.V6/queue.cpp#L4226
We should ask the condor folks about this and how to get an accurate RUNTIME.

The condor folks say we can do the following:
So the reported run time in condor_q is (serverTime-shadowBirthdate)
RemoteWallClockTime only increments at the end of the job–and is cumulative of all the time the job ran.  Condor_q shows current run time, condor_q -cumulative-time can show all time accumulated.  “ ShadowBday”

So look for the ShadowBday classad, and perhaps try to get the server time.  If not, we can use the client time as a reasonable approximation, since those shouldn't be off by more than a few seconds under normal circumstances.

Looking good so far:
[testuser@submit_host jobsub_lite]$ bin/jobsub_q -G fermilab --schedd-for-testing=schedd@domain
JOBSUBJOBID                             OWNER           SUBMITTED     RUNTIME   ST PRIO   SIZE  COMMAND
373.0@schedd@domain          testuser           10/15 11:06   0+00:00:00 I    0    0.0 sleep 300
375.0@schedd@domain          testuser           10/15 11:29   0+00:00:00 I    0    0.0 sleep 300
376.0@schedd@domain          testuser           10/15 11:30   0+00:01:52 R    0    0.0 sleep 300
3 total; 0 completed, 0 removed, 2 idle, 1 running, 0 held, 0 suspended
[testuser@submit_host jobsub_lite]$ bin/jobsub_q -G fermilab --schedd-for-testing=schedd@domain
JOBSUBJOBID                             OWNER           SUBMITTED     RUNTIME   ST PRIO   SIZE  COMMAND
373.0@schedd@domain          testuser           10/15 11:06   0+00:00:00 I    0    0.0 sleep 300
375.0@schedd@domain          testuser           10/15 11:29   0+00:00:00 I    0    0.0 sleep 300
376.0@schedd@domain          testuser           10/15 11:30   0+00:02:53 R    0    0.0 sleep 300
3 total; 0 completed, 0 removed, 2 idle, 1 running, 0 held, 0 suspended

I used the expression:
IfThenElse(JobStatus == 2, ServerTime - ShadowBday, RemoteWallClockTime)

for the runtime.  The idea is that if a job is running, then ServerTime - ShadowBday is correct.  If a job is idle, then RemoteWallClockTime will be 0, and if the job is held or completed, then RemoteWallClockTime will have the proper accumulation of time across all runs.

The old behavior of this column was to use RemoteWallClockTime, which is supposed to include the cumulative run time across all executions.  We were missing that here; so if a job went held, and it was then released, the RUNTIME would only have the current execution run time, which is incorrect.
The remedy was adding the RemoteWallClockTime back in to the running-job RUNTIME.  If a job was running for the first time, RemoteWallClockTime is 0.  Otherwise, it's got the accumulated time, which we need to add back in.
So the final expression is:
IfThenElse(JobStatus == 2, (ServerTime - ShadowBday) + RemoteWallClockTime, RemoteWallClockTime)

Testing this now.


On Internet Explorer, RouterMixin's "Scroll Position Management" is setting the url path to "undefined"

When RouterMixin's enableScroll is not set to false, something is happening to cause the url path to get set to undefined.  For example, if I go to www.example.com/?query1=true&query2=false, and then I scroll down any amount, the url gets replaced with www.example.com/undefined.  Something is not playing well with IE's history management I think.  It works fine on Chrome, Firefox, Safari.  It's just IE (more specifically IE11, via VMWare on Mac, Windows 7 and 8).
The temporary fix is just to disable it by passing {enableScroll : false} when initializing the component with the RouterMixin

@phaddius  Hmm.  Are you using 0.6.1?  We fixed this issue in that release, with PR https://github.com/yahoo/flux-router-component/pull/88

I'm on 0.6.0, I'll try out the latest, thanks!

Cool.


[BUG][Connector-V2] Iceberg source lost data with parallelism option

Purpose of this pull request
Fix https://github.com/apache/seatunnel/issues/5661

Does this PR introduce any user-facing change?

No
How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

@ic4y

Thanks @kangdw0x80 !


Fix domain cache refresh bug

Coverage increased (+0.03%) to 66.929% when pulling ee233b69321b2f2f55b963d7dc2df3770865f61a on bugfix into a781595a30ff4feea7bfc38305e091a6a29d1f07 on master.


What's the difference btw many latency in the result?

There are many latencies.
I know latency definition in the context of mlperf.

What is the difference ?

Min
Max
Mean
50,90,95,99, 99.9 percentile?
*) What is  "QPS w/ and w/o loadgen overhead"?

Thank you !

Perhaps this will help: https://en.wikipedia.org/wiki/Percentile

Do you have any idea about  "QPS w/ and w/o loadgen overhead"?

There is max latency and min latency in summary.
Since latency is "The latency for a query is the difference between the time time when the LoadGen receives a reply and the time when the query was scheduled."
What is the meaning of those term?
Btw every query latency, most high latency is max and low is min latency ?
Can I understand like this ?

Do you have any idea about "QPS w/ and w/o loadgen overhead"?
w/  = samples / duration
w/o = 1 / (average latency)

Where:

Duration includes time spent in the loadgen.
Average latency does not include time spent in the loadgen.

Since latency is "The latency for a query is the difference between the time time when the LoadGen receives a reply and the time when the query was scheduled."
What is the meaning of those term?

scheduled = just before loadgen calls SystemUnderTest::IssueQuery
receives a reply = just after SUT calls QuerySamplesComplete

Btw every query latency, most high latency is max and low is min latency ?
Can I understand like this ?

I think that's correct.


Extend Env for command line arguments and environment variables

Environment variables are in raw KEY=VALUE form. Also, it's probably
better to use getenv(3) to access environment variables.

#108

Great!


Is it possible to only list images without GIF?

Thank you for providing this useful multiple images selection library.
I understand that we can perform filtering for image only
imagePicker.settings.fetch.assets.supportedMediaTypes = [.image]
However, is there a way to perform a more specific filtering, by only listing out PNG and JPG only, and rejecting GIF file? This is because at the moment, our app doesn't support GIF
Thanks.

I am facing too


Islesboro siting data review

Go over data sources that were looked at. Determine which ones were valuable or not. What gaps do we have in knowledge. Determine some high level exclusion criteria, and goals for siting.

Wait for information about ICS LPA location to include in analysis.


Update version number in version.json

Version.json had out of date version number

A single fix has been reviewed in Dev branch.


Fix bug with big time periods

If I try to run analytics for more than 15 days, my time period will be reduced.

After investigating the issue, I discovered that the behavior is due to a limitation in the Google Calendar API, which currently allows a maximum of 250 events to be retrieved at once. To resolve this, we need to update the collector class to enable the gathering of large amounts of events by breaking them into smaller chunks.


More detailed setup intructions

Are there by chance any more detailed setup instructions? I have a server admin background, but not a Laravel background....
Regards,

I suggest checking out the Laravel documentation.  This project uses version 5.0 which is a slightly older version of Laravel, but the docs for this version are still at laravel.com.
Laracasts is also a good resource for helping you get started with the framework.


[8.x] Return false on trashed() if the value in deleted_at is a future time…

The current behavior of the trashed() method in the SoftDeletes trait is to return true (item is trashed) if the value in the deleted_at column is anything other than null. But that column is a timestamp, so we could let users take advantage of that fact by only returning true if the value is a timestamp right now or in the past. This would allow the user to schedule an item to be soft deleted in the future.
A sample use case would be a contact management system that pulls contact information from external directories. When a contact disappears from a directory, it's likely that they've left the organization and that change should be reflected in my own database. But maybe they've actually just changed their name so they're still there but my scraper isn't finding them. If I can set any newly "not found" contacts to be deleted at some time in the future, I can give my ops team the time to investigate these newly missing contacts and reconcile any non-departure changes before we drop them from our database.
This PR is limited to just adding the check for a past timestamp rather than any non-null value to the trashed() method. My original vision was to build out a "scheduled soft deletion" feature, but I think for now that might be a better fit in userland. I'll probably build a package implementing my vision of it.
The reason I'm sending in this PR rather than just handling it in my package is that if this one method is changed, then my and any other userland code that might try to implement scheduled soft deletions is simple. I don't have to totally reproduce the trait or override the trashed() method via insteadof.
The original behavior of the method and the trait overall is preserved for anyone who uses the SoftDeletes trait as documented.

I would add an willBeTrashed() for free to check if the deleted_at is set in the future. Should return false in any other case (null or set in the past).
I don't think this is a BC for anyone that has set the deleted_at timestamp in the future, but there may exist some cases. I honestly don't know why someone would do that.

The change would be much more involved than just this method. For example, the scope methods in SoftDeletingScope.php would need to change, etc.
We would probably also consider this a fairly large breaking change since 3rd party packages could also be relying on "non-null" checks to determine if a record is soft deleted.

Should be pushed to 9.x?


Minimap crashes when updating element with attacher

See related issue in JIRA and diagram-js-minimap issue.

Related Support case: SUPPORT-3811


Decide how to handle empty JARs on a classpath

It seems like we treated empty JARs as non-existent with runtime normalization in 5.5, while they are reported as different in 5.6. At least in the build scan comparison, an empty JAR showed up as "not there" in Gradle < 5.6 while it shows up as changed in 5.6.
We should make a conscious decision what behaviour we want to have.

cc: @gradle/build-cache

The PR https://github.com/gradle/gradle/pull/9344 causes that a zip marker is added for each included zip file. Therefore, we are not ignoring empty zips any more.

We didn't do anything here in two years. Let's close this now and re-open if it comes up again in the future.


AstropyDepreciationWarning

Hello everybody! I am new to SORA, I am trying to predict occultations of Io based on a paper that suggested occultations in 2019 and 2021. If I try to predict them, using this code:
` io = Body(name='Io',
ephem=['/Users/gianmarcobroilo/Desktop/Tudat/SORA/input/jup365.bsp', '/Users/gianmarcobroilo/Desktop/Tudat/SORA/input/de441_part-1.bsp'])
print(io)
pred = prediction(body=io, time_beg='2019-09-09',time_end='2019-09-12',mag_lim=16)
print(pred)`
I get this issue only when it actually finds occultation possibilities:
Searching occultations in part 1/1 Generating Ephemeris between 2019-09-09 00:00:00.000 and 2019-09-11 23:59:00.000 ... Downloading stars ...     65 Gaia-EDR3 stars downloaded Identifying occultations ... WARNING: AstropyDeprecationWarning: ``id_type``s 'majorbody' and 'id' are deprecated and replaced with ``None``, which has the same functionality. [astroquery.jplhorizons.core] Traceback (most recent call last):
Thank you for the help!

Hi @GianmarcoBroilo . This is not an error, so the code continues running without any problem.
This warning is raised due to a recent update on astroquery regarding parameters when calling jplhorizons. But the current parameters work just fine. This will be a simple modification in SORA to fix it, but it will take some time until a release is made.
You can just ignore this warning.
I know the printing of many warning may bother, so I will try to provide you a script to safely ignore it.


iOS7, Navigation Bar Black-Style, Status-Bar not shown

In the iOS7 isn't shown the status bar for a black style navigation bar. The text color of the status bar is also black. There are any possibility to change it into white color?
In iOS6 isn't it a problem.

What's the fix for this? I'm still seeing this issue


Why Testflight showing Ask Permission Request will be sent to Parent/Guardian while purchasing subscription

I am developing an application using in_app_purchase from flutter.
it works fine on android but on ios after subscribing to the in-app product(auto-renewing subscription) it asks permission. My client's iPhone does not have family sharing turned on and we are on testflight with internal testing.
check the video to see behavior video link
I think because of ask permission it is not allowing the purchase completed.
the code works perfectly fine in android
my purchase stream code
    purchaseUpdates.listen((purchases) async {
      for (PurchaseDetails purchase in purchases) {
        if (purchase.status == PurchaseStatus.error) {
          print(purchase.error.code);
          print(purchase.error.message);
          print(purchase.error.details);
          reUse.show_toast("payment problem if");
          reUse.show_toast(purchase.status.toString());
          in_app_purchase.completePurchase(purchase);
        } else if (purchase.status == PurchaseStatus.purchased) {
          await reUse.add_purchase_details_to_user_account(
            user_id,
            purchase.purchaseID,
            purchase.transactionDate,
            purchase.billingClientPurchase.purchaseToken,
            purchase.billingClientPurchase.signature,
            selected_plan,
            purchase.billingClientPurchase.orderId,
          );
          reUse.show_toast("success");
          in_app_purchase.completePurchase(purchase);
          setState(() {
            subscribed = true;
          });
        } else {
          reUse.show_toast("payment problem else");
          reUse.show_toast(purchase.status.toString());
        }
      }
    });

@junaidlodhi22
This looks very similar to this issue. Please check this issue comment and see if it helps to resolve your issue.

thanks, @darshankawar. the message of ask permission gets removed. Getting a message that you are successfully subscribed.
But the above purchase code is still not running
 purchaseUpdates.listen((purchases) async {
      for (PurchaseDetails purchase in purchases) {
        if (purchase.status == PurchaseStatus.error) {
          print(purchase.error.code);
          in_app_purchase.completePurchase(purchase);
        } else if (purchase.status == PurchaseStatus.purchased) {
          await reUse.add_purchase_details_to_user_account(
            user_id,
            purchase.purchaseID,
            purchase.transactionDate,
          );
          in_app_purchase.completePurchase(purchase);
        } else {
          reUse.show_toast("payment problem else");
        }
      }
    });

@junaidlodhi22 Good to know your permission is now resolved.
Regarding your other issue, can you provide flutter doctor -v and which use case isn't working exactly related to purchase subscription ?
Thanks.

I got it resolved. @darshankawar thank you again for helping...
actually, I was using the client's mobile to test so I could not see the error...
it was because of purchasewrapper class parameter billingClientPurchase
it is null on ios and that's why my code was breaking. I went into the code and saw that.


Set display orientation in web config

This way no one has to care about this option BEFORE flashing. If the setup is wrong, no new flashing is needed.
I build in this function mainly because i have an enclosure which i can place in two positions.
In one position it is flat and i put it on my computer monitor to have a straight view on the display. When i turn the device 180 degree, the display is angeled. This way i can put it on a table and view "from above". Therfore i have to set the orientation of the display.

Thanks for the contribution.   I have made a couple of comments to locations in the PR.   We will not be changing the original MIT license and copyright.  Please remove the additions to the copyright.   You will be listed as a contributor in the Readme.md as well as here in the GitHub history.


Consolidate plugin files

My main goal of this was to:

reduce complexity of plugin files (no need for inheritance)
reduce the size of the baseline file

Everything here should be functionally the same, it is just a refactor.

Mind rebasing this?


Web socket connections started by web workers are not proxied

What is your Scenario?
Hello TestCafe team.
We have a situation where WebSocket connections over web workers are not being proxied.
We need to pass authentication cookies to a web socket connection but the connection is started by a web worker, in that specific case, we can see that the websocket connection URL is not proxied (see attached screenshot), and the auth cookies are not being attached to the request.
What is the Current behavior?
Secure web socket connections initiated by a web worker are not proxied.
Authentication cookies added during the test execution, are missing from the web socket connection.
What is the Expected behavior?
Websockets connection initiated by a web worker should be proxied.
Authentication cookies should be added to web socket connections initiated by a web worker.
What is your public website URL? (or attach your complete example)
https://github.com/kvindascr/test-cafe-cookie-issue
Follow the readme instructions to setup and run the example.
What is your TestCafe test code?
import { debug } from '../testcafe/log';
import { ReactSelector } from 'testcafe-react-selectors';
import {ClientFunction, Selector} from 'testcafe';
const STABLE_TEST_ROOM = 'stable-baa';
const url1 = 'google.com';
const url2 = 'news.ycombinator.com';
const url3 = 'linear.app';
const setCookiesBtn = ReactSelector('AppContainer');
const callWSBtn = ReactSelector('AppContainer').withProps({ clientId: this.clientId }).findReact('BrowserAppContainer');
export const getPageHTML = ClientFunction(() => {
return document.documentElement.outerHTML;
});
fixture('TestFixture').beforeEach(async (t) => {
debug('starting fixture');
});
test('add browser and test core actions', async (t) => {
await t.navigateTo('https://local.myapp.app/');
// await t.debug();
await t.wait(5000);
await t.click('#setCookiesBtn');
await t.click('#callSocketBtn');
await t.wait(2000);
// await t.debug();
await t.expect(Selector('#result').textContent).contains('Cookies found');
});
Your complete configuration file
No response
Your complete test report
$ node ./run-test-cafe.js
 Running tests in:
 - Chrome 103.0.5060.53 / macOS 10.15.7

 TestFixture
starting fixture
 ✖ add browser and test core actions

   1) AssertionError: expected 'ERROR Cookies NOT found' to include 'Cookies found'

      Browser: Chrome 103.0.5060.53 / macOS 10.15.7

         24 |    await t.wait(5000);
         25 |    await t.click('#setCookiesBtn');
         26 |    await t.click('#callSocketBtn');
         27 |    await t.wait(2000);
         28 |    // await t.debug();
       > 29 |    await t.expect(Selector('#result').textContent).contains('Cookies found');
         30 |});
         31 |

         at <anonymous> (/Users/kvindas/Development/js-projects/switchboard/test-cafe-cookie-issue/testing/e2e/tests/CookiesTest.test.ts:29:53)
         at fulfilled (/Users/kvindas/Development/js-projects/switchboard/test-cafe-cookie-issue/testing/e2e/tests/CookiesTest.test.ts:5:58)

 1/1 failed (12s)
error Command failed with exit code 1.

Screenshots
The URL marked with red should have been proxied, but as can be seen, it is a non proxied url.

Steps to Reproduce

Follow the setup steps from the readme.md.
Exec docker-compose up at repo root
Exec npm run start at frontend dir
Exec npm run start at backend dir

To manually reproduce it.

Open local.myapp.app
Press the Set cookies button
Press the Call socket via worker button.
You should see Cookies Found message, if running outside test cafe.

To run it via test cafe

Exec npm run e2e at testing dir
You should see an error message as a result.

TestCafe version
"^1.15.1"
Node.js version
v16.13.2
Command-line arguments
node ./run-test-cafe.js
Browser name(s) and version(s)
Chromium 98, Chrome 103
Platform(s) and version(s)
Mac 12.3
Other
No response

Hello @kvindascr,
Thank you for your example project. I reproduced the issue.
For the Team:
I've attached a simplified project to see the problem with testcafe-hammerhead playground:

npm install
node ./index.js
Run testcafe-hammerhead proxy
Navigate to the http://localhost:4000 url. You will see two buttons. Open the DevTools
Click the "Send WS message" button. You will see that the web socket connection url looks like ws://localhost:1401/some-random-symbols/http://localhost:4000
Click the "Send WS message over new Web Worker" button. You will see that the web socket connection url is not proxied: ws://localhost:4000

Hello,
We've released a new TestCafe version(v2.3.0-rc.2) that includes the experimental proxyless mode. This mode uses native browser automation. In Proxyless mode, a few issues are already fixed. This issue should also be fixed in Proxyless mode since it should not proxy any URLs, including web sockets. However, I was not able to run your example on my side due to issues that occur on the step with certificates. Would you please check if your sample is working correctly in v2.3.0-rc.2 with the experimental-proxyless mode enabled?
This option is available in all interfaces:
// Command-line
testcafe chrome tests --experimental-proxyless

// Programmatic
const testcafe = await createTestCafe({ experimentalProxyless: true });

// Configuration file
{
   "experimentalProxyless": "true"
}

Note that at present it is an experimental mode. Also, the Proxyless mode is implemented only in Google Chrome. It will not work correctly if you run tests in a non-Chrome browser or in a combination of other browsers.


Documentation on how it was put together and how to set up a local instance.

I would to be able to use some of the yats alongside some other django apps, but it is very difficult to get an idea about how this was put together, and how to setup a local (non vagrant) setup, such as a container.
Is there documentation available that can be checked in about setting up local development. configuring the various requirements etc?

I just deployed an instance, pretty much just patched bootstrap.sh and web.ini with the paths I wanted. The only glitch was having to remove sudo from the command unzipping the xapian tarballs (the buid step failed because the folder was owned by root).

OK. So did it work for you or do you need further help?

Worked for me, I just wanted to comment here in order to help someone trying to do the same. I will repeat the process soon, if you think it is a good idea I can document the steps and send a PR.

Yes Please, I need help how to run it on local.

Hello, i need help running it locally. I am using Anaconda, and have a environment in it.

Sorry, I am not familiar with Anaconda.
You need all python modules in the modules folder and all from requirements.txt
YATS is a Python 2.x Django 1.11 Project. For more info on Django see: https://www.djangoproject.com
So you need the web folder in sites:
cd into that and do: python manage.py check
read bootstrap.sh and see, what is done for installation.
And you need a xapian installation. You can use other search engines - see http://haystacksearch.org for more info. You can even disable haystack in settings.py under INSTALLED_APPS.
if it all does not work, git vagrant a try ;-)
https://www.vagrantup.com

Thank you for the response @Henner .
I see that this is a Python 2x app.
Do you have any plans to upgrade it to make it Python 3x compatible.

yes, as this is an active project, we have to until the end of this year ;-)

can anyone help me with the steps to run this project locally on ubuntu

the easiest way is:

install vagrant
install virtualBox
checkout this project
cd into the vagrant folder
on a shell type: vagrant up

I need to check the demo site for my reference, but the site is down from couple of weeks.

what site is down? Our demo here?
https://yats.mediafactory.de/
I can open it and we are getting lots of test tickets by others...

ok thanks sir.


Implement Next.js app directory

Included in https://github.com/Frojd/Wagtail-Pipit/releases/tag/v9.0.0-beta


output complete information on summary and json output formats. Refac…

…tor the reporting of errors to use dictionaries

@rgardler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Add credential database object

Summary
Create new database object that can be used when accessing object storage.
Specifications

Create
CREATE CREDENTIAL my_creds
PROVIDER aws
OPTIONS (access_key_id = '<access-key>', secret_access_key = '<secret>')
[COMMENT 'a comment'];

Drop
DROP CREDENTIAL [IF EXISTS] my_creds;

Usage (table creates)
CREATE EXTERNAL TABLE my_table
FROM s3
OPTIONS (credentials = my_creds, ...);

Usage (copy to)
COPY (SELECT * FROM glare_catalog.tables)
TO 's3://sean-test-glaredb/test2.json'
OPTIONS (CREDENTIALS my_creds);

Usage (table func)
SELECT * FROM read_parquet('s3://...', my_creds);

(Note we don't have a read_parquet function yet.)
Like tunnel, credentials will be a top-level item and will live in its own unique namespace.
When resolving credentials during a table create, we’ll ensure that the data source type matches the credential type (e.g. s3 tables should only allow aws credentials).
Rationale

Definite
Impact

Tasks

Do we want credentials to be referred or once a datasource is created, they become static?
Basically, does a table refer to credentials and if the credentials are updated (altered), should the table be able to pick up the new credentials or when creating the table with credentials the values are saved in each table from the creds object?
To actually have any benefits of saving the creds reference, we need to have an ALTER CREDENTIALS ... statement, else it's pretty useless.

I'm currently going with just copying creds from the object into tables/databases. We can change this later if we want.

Copying the creds is fine for now, but I'd like for us to move towards referencing them in the near term.

To actually have any benefits of saving the creds reference, we need to have an ALTER CREDENTIALS ... statement, else it's pretty useless.

As an alternative to this, we could have the ALTER TABLE/DATABASE command allow altering which credentials to use. So to update, you'd create a new credentials object, then run ALTER TABLE to point to that new creds object.

As an alternative to this, we could have the ALTER TABLE/DATABASE command allow altering which credentials to use. So to update, you'd create a new credentials object, then run ALTER TABLE to point to that new creds object.

We can still do this by just copying the creds.
I feel we should refer credentials and have an ALTER CREDENTIALS ... to implement this behaviour.


Distance and Direction addin UI should not have decimals in outputs for measurements

@kgonzago and @csmoore can distance and direction interactive outputs in the UI not be crazy lengths for each measurement? Crazy being more than four decimal places

@topowright please rewrite the question above as an issue if it is not behaving as expected. Add a screenshot showing the current behavior and how you want the results to be for expected results.

@topowright A simple one liner fix, we can definitely limit the display of numeric values to four (4) significant digits.

Changes to address this issue is in this PR

Verified #598.

@ACueva @kgonzago @topowright This implementation was not correct. The display was not supposed to be to two decimal places. The issue was specifically for the feedback in the UI not on the map. The display on the map was to remain as whole numbers.