feat: extended card title

This PR implements an extension to the displayed card title partly as described in issue #851.
I added this because sometimes one doesn't know how many cards are left to review and thus is discouraged to continue if the time one has is short, but if the user exactly knows how many cards are left, then one could guess how long it will take and just pull through with it.
Another good reason for this addition is that the card title will display more useful information in the random card mode.
Before:

It just displayed the current deck name and the number of cards left in it.
After (with random card random deck mode enabled):

In random mode it displays now the deck chosen, with the number of cards left and in the subtitle the deck out of which the card was drawn.
After (with sequential mode enabled):

In sequential mode the title will display the deck chosen, with the number of cards left and in the subtitle the number of subdecks, the current deck name and the number of cards left in it.

Hi, everything is now resolved.


hard and softlink tar handling, modified tarfile in current directory

We should review this after a bit more testing.

Note:  This PR comes from ticket INC000001147543 in ServiceNow, where a mu2e user previously was unable to submit a job with a tarball, and still won't be able to after 1.1.1 is deployed.  This PR should fix that issue.
I took the user's tarball, and reproduced the error in 1.1.1, and have confirmed that with this PR's code checked out, the error disappears and jobs can be submitted.  We'll be reaching out to the user to test on Monday, as well as running our own tests, as @marcmengel mentioned above.


Don't modify original Flatpak manifest

Draft because I didn't even compile-tested it, and I'm not sure this is the correct way to implement this fix.
Reviews appreciated.
Fixes https://github.com/bilelmoussaoui/flatpak-github-actions/issues/61

The tests seems to be working fine, can you try it with obs now and see? just point the action to your flatpak-github-actions fork

You can verify here that this works: https://github.com/GeorgesStavracas/obs-studio/runs/4300417030

Thanks !


fix: When running npt in keep-alive mode then the keep-alive loop should never exit due to an exception from npt.run()

Resolves #1182
- What I did
fix: When running npt in keep-alive mode then the keep-alive loop should never exit due to an exception from npt.run()
- How I did it
Added additional catch clauses to the keep-alive loop, and ensure that they do not lead to the loop terminating if keep-alive is set.
- How to verify it
@cconstab please can you test against this branch?

@cconstab have you had a chance to test this?

I will sorry!

@cconstab ping

Discussed with @XavierChanth - we will merge this PR and come back to the ticket if the problem occurs during subsequent real-world testing


Column name with % and ) causes error in profiler

Affected module
Ingestion
Describe the bug
I have a table with a column called Ozone gas concentration (%03).  This throws an error in the Profiler logs:

{'Ozone gas concentration (%03)': {'Static Metrics': {"unsupported format character ')' (0x29) at index 46"}}}

Version:

OpenMetadata version: 0.12.2

@ayush-shah lets handle this in 0.13.1

@harshach I believed we said we won't fix this has it requires some very intricate logic for an edge case


Release prebuilt robotjs packages

Do so in https://github.com/jitsi/robotjs which we were using before. We might need to publish the package on npm I think, we can do that as @jitsi/robotjs. The package has been "orphaned" for years, AFAICT, and we have no intention of becoming the maintainers, but providing prebuilt packages, which can be done with a GH action easily would simplify things.

To get this started I have provided all upstream an my (context-aware) commits via https://github.com/jitsi/robotjs/pull/2

Followup: Draft PR to provide prebuilds via GH actions: https://github.com/jitsi/robotjs/pull/3

PR for prebuild robotjs packages is now ready: https://github.com/jitsi/robotjs/pull/5

Prebuilds of robotjs are now ready to be tested: https://github.com/jitsi/robotjs/releases/tag/v0.6.5

🚀

Published on npm.


Cant find modules

System information

OS: Windows 10
TypeScript Hero version: 2.3.1
VSCode version: 1.21.0

Description
In angular (maybe somewhere else also IDK) in app.module.ts file when i want to import modules (for example: RouterModule) it cant find modules classes etc.
It also does not works in component.ts files (in classes).
here is an image:

Same issue here
System information
OS: Windows 8.1
TypeScript Hero version: 2.3.1
VSCode version: 1.21.0
Seems to be after the last update of VSCode to version 1.21.0

Same issue:
OS: macOS Sierra
TypeScript Hero: 2.3.1
VSCode: 1.21.0


Making EvalBat round up to nearest whole number

Hi Mate,
Love this little thing, I use it heaps. Running into an issue and wondering is there a way to make it round up to the nearest whole number when doing math instead of showing all the decimals? Just need that for something I'm working on. If rounding isn't possible then just leaving off any number to the right of the decimal point and only showing the whole number. Thank you!

If you have the most recent version, you can use the Ceil(xxx) function which is an additional math function in the script which will round a floating point expression up to the next higher Int value.  There is a similar function Floor() which rounds down (and the built in Int() conversion works to round down as well).
You can also use the VBScript function Round() to round to nearest.

Look at the EvalBat_vbs.vbs source code to see additional functions: comparisons, Pi(), trigonomic expressions, etc.

I can't seem to get floor or ceil to work with variables trying Call EvalBat.bat Ceil (%var1%/%var2) and it just shows the answer as normal with decimals

You have a typo in the example you gave: "%var2" should be "%var2%"
It is missing a % after var2
On Tue, Jan 31, 2023 at 3:33 PM Galikin @.***> wrote:

I can't seem to get floor or ceil to work with variables trying Call
EvalBat.bat Ceil (%var1%/%var2) and it just shows the answer as normal with
decimals
—
Reply to this email directly, view it on GitHub
https://github.com/adisak/EvalBat/issues/1#issuecomment-1411097685, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAA33ZE7Z6GT4HCLFTJELDWVGAKXANCNFSM6AAAAAAUL6YVSY
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

You have a typo in the example you gave: "%var2" should be "%var2%"
It is missing a % after var2
Try this expression:
Call EvalBat.bat Ceil ( %var1% / %var2% )

Tested that it works as designed even with batch variables
set aa=300
set bb=65
evalbat_echo Ceil(%aa% / %bb%)
Input: Ceil(300 / 65)
Result: 5

If you need to debug expressions in the future, try using EvalBat_Echo.bat instead of EvalBat.bat to see the what the input and output are being fed to EvalBat.bat by your call


Format include?

Operating System (+Version): macOS 10.13.2
VS Code Version: 1.20.1
beautify Version: 1.3.0

NOTE: Please provide code snippets instead of screen shots. Your issue needs to be replicated, and we need the failing code (text) to do this.

How can I format an "include" so that it looks like a normal declaration and not in one line?

First of all, thanks for providing a screen shot and making me type out all your code myself. 👎

Secondly, I don't think you can. Beautify is specifically for CSS and doesn't have any options for formatting this kind of stuff. You could work on supporting it by contributing to js-beautify.

Apologies, I mis-read it I thought you wanted it in a screenshot (I thought it was an odd request).  Thanks for your help at any rate.


fix: NaN rewards on DHT

LGTM @JChiaramonte7 can you please remove package-lock.json from your commit?


onLanguage:javascript  can't active my extension.

I just do a simple change based on the hello world example.
Changes:  Add contents onLanguage:javascript and onLanguage:typescript for the activationEvents attr in the package.json file.
Expect: When I open a js file, I can see the Congratulations info from the console.
But when I open a js file, I can't see the Congratulations console info, and when I open a ts file, I can see the console log.

My changes are as follows.
       "activationEvents": [ 		"onLanguage:javascript", 		"onLanguage:typescript" 	],
Is this the right result? Or I need to do some change.

Your have a file association that makes *.js be treated as JavaScript react. Add  onLanguage:javascriptreactto fix this


Invalid IPubsubMessage.data type

https://github.com/googleapis/nodejs-pubsub/blob/76db007f270a646e8570768fa827ea2a97b62cbc/protos/protos.d.ts#L441
IPubsubMessage.data type seems to be Buffer and it is typed as UInt8Array|string.

@HNicolas Thanks for the report!
@alexander-fenster This looks like an issue with proto generation maybe?

@feywind Yes, seems like I will need to add Buffer here: https://github.com/googleapis/gax-nodejs/blob/master/tools/compileProtos.ts#L141 - we do some serious hacking on the generated protos.d.ts. I will fix it (but probably not earlier than I come back from vacation), or feel free to make a fix there and release!  (it will, unfortunately, trigger regeneration of all libraries)

I'm moving this over to a cleanup item because it's going to cause a lot of churn, and maybe it's not the best time for it. I'm leaving it on the backlog here for now though.

I'm transferring this issue over to gax-nodejs since that's where the problem appears to be.


Fix formfield cursor behaviour

Fixes #718
Fixes https://github.com/subconsciousnetwork/subconscious/issues/708
Fixes https://github.com/subconsciousnetwork/subconscious/issues/674
Changes

ValidatedTextField has an internal @State var that holds the current field value

Edits are replayed to the store to keep the two in sync

New ValidatedFormField component which wraps ValidatedTextField wires up everything to talk to a backing FormField<>
Replaced all fields by FormFields
Nickname can be entered with capitals but will always persist and present in lowercase
Bio will automatically truncate >280 chars
Clarified profile memo lifecycle

It would also be worth investigating why this happens for this component, but not other input fields that are driven by ObservableStore.
One noted difference is the closure in this model. I wonder if it might be confusing SwiftUI's change detection under the hood.

It would also be worth investigating why this happens for this component, but not other input fields that are driven by ObservableStore.
One noted difference is the closure in this model. I wonder if it might be confusing SwiftUI's change detection under the hood.

I think there was a little bit of a red herring here, thought I'm still not 100% sure. Based on my testing here is my understanding:
I think that the fundamental cause of the cursor issues is a race condition between updating the binding via the store and internal input events of the TextField. It seemed like FormFields were experience more cursor issues but I saw (and have seen) the same issues crop up in other fields. I recall the first report I heard of this problem was the Gateway URL field, which did not use the FormField abstraction until this PR.
After decoupling the input binding from the store the cursor issues resolved, but I was seeing stale validation results instead. To fix that I introduced the ValidatedFormField API that takes the entire FormField struct as a property (as opposed to the current value and a computed hasError property). I suspected that the change detection would be more reliable checking struct equality vs. a computed field and it appears to behave correctly now, though I'll admit I am hand waving.
(Sidenote: sheets made no difference to anything)


mapStateToProps method only receive react-router params if use withRouter function

I'm trying use react-redux-router and face with a problem, the "ownProps" parameter of mapStateToProps function don't receive react-router "params" attribute. To fix this we need to pass our "connect" function to react-router withRouter function.
Don't work:
const VisibleTodoList = connect(
  mapStateToProps,
  mapDispatchToProps
)(TodoList)

Work:
import { withRouter } from 'react-router';
const VisibleTodoList = withRouter(connect(
  mapStateToProps,
  mapDispatchToProps
)(TodoList))

I think we need update the documentation with this information:
https://github.com/reactjs/react-router-redux#how-do-i-access-router-state-in-a-container-component
Thanks.

This library doesn't sync router params because it only interacts with history. You should use a separate route component if you need access to the params. It can shove them into the store in its constructor or cWM.


Error: You must supply options.input to rollup

Hi,
I am using vite-plugin-web-extension v1. Firstly i removed this line"default_popup": "popup.html from manifest.json then i run build command and Vite threw error.
Error: You must supply options.input to rollup
if i add popup.html again to manifest.json. There is no problem.
i dont want to use popup. How to remove it ?
how do i fix this problem ?
Error log

My manifest
{
  "manifest_version": 3,
  "name": "Resmo Chrome Extension",
  "$schema": "https://json.schemastore.org/chrome-manifest.json",
  "description": "",
  "version": "1.0",
  "background": {
    "service_worker": "only_background.js"
  },
  "icons": {
    "16": "public/images/icon-16.png",
    "32": "public/images/icon-32.png",
    "48": "public/images/icon-48.png",
    "128": "public/images/icon-128.png"
  },
  "action": {
    "default_icon": {
      "16": "public/images/icon-16.png",
      "32": "public/images/icon-32.png"
    },
    "default_title": "Click to view a popup"
  },
  "permissions": [
    "alarms",
    "identity",
    "identity.email",
    "scripting",
    "storage",
    "tabs",
    "webRequest",
    "declarativeNetRequest",
    "declarativeWebRequest",
    "*://*/*"
  ],
  "host_permissions": [
    "http://*/*",
    "https://*/*"
  ],
  "content_scripts": [
    {
      "matches": [
        "http://*/*",
        "https://*/*"
      ],
      "js": [
        "contentScript.js"
      ]
    }
  ],
  "content_security_policy": {
    "extension_pages": "script-src 'self'; object-src 'self'"
  }
}

See #10 and #40. V2 will support extensions without a UI, but right now you need at least 1 html file.
Here's a work around for now: https://github.com/aklinker1/vite-plugin-web-extension/issues/10#issuecomment-1089367768

Okey, thank you. I'm looking forward to the new version.


Remove beta markings from DashCanvas

Exciting!

closed with https://github.com/xh/hoist-react/commit/e6791e72eba60d637e42a6a2d40e1dab0b4605e2


Add member hub content

What:
Add member hub content
Why:
Relates to: https://app.asana.com/0/1168997577035609/1194726062822489/f
How:

Create a prop in the footer to hide it if necessary
Add content for the member hub page

Media:
https://www.loom.com/share/6d2d0baa34d7471eaf5fa0e725aa35cf

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


fix: cosmwasm-std version up

Summary of changes

Report of required housekeeping

[ ] Github issue OR spec proposal link
[x] Wrote tests
[x] Added a relevant changelog entry to CHANGELOG.md

(FOR ADMIN) Before merging

[ ] Added appropriate labels to PR
[ ] Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
[ ] Confirm added tests are consistent with the intended behavior of changes
[ ] Ensure all tests pass

Codecov Report

Merging #4 (a7b30f1) into main (65193ac) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a7b30f1 differs from pull request most recent head dfbf77e. Consider uploading reports for the commit dfbf77e to get more accurate results

@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage   87.10%   87.10%
=======================================
  Files          18       18
  Lines        3172     3172
=======================================
  Hits         2763     2763
  Misses        409      409

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


chore: remove obsolete patch to nuxt content

Blocked by https://github.com/nuxt/content/issues/1856

Done now


Update package-lock.json

https://github.com/louislam/uptime-kuma/pull/71 was merged incorrectly into master branch...

Thank you, but I also add a new dependency, so I just build it again already in the new commit.

@louislam: OK, I see. But please delete your package-lock.json and run npm install - for me (I'm using NPM 6.14.13), the formatting was changed in this file
But maybe the best solution will be to delete package-lock.json and add it to .gitignore


Typed Serialization

Codecov Report

Merging #211 into baker-3.0.0 will decrease coverage by 1.57%.
The diff coverage is 75.63%.

@@              Coverage Diff               @@
##           baker-3.0.0    #211      +/-   ##
==============================================
- Coverage        78.87%   77.3%   -1.58%
==============================================
  Files              122     139      +17
  Lines             3262    3534     +272
  Branches           145     148       +3
==============================================
+ Hits              2573    2732     +159
- Misses             689     802     +113

Impacted Files
Coverage Δ

...r/runtime/actor/recipe_manager/RecipeManager.scala
95.65% <ø> (ø)
:arrow_up:

...ypes/src/main/scala/com/ing/baker/types/Type.scala
20.68% <ø> (ø)
:arrow_up:

...aker/runtime/actor/ClusterBakerActorProvider.scala
0% <ø> (ø)
:arrow_up:

...tor/process_instance/ProcessInstanceProtocol.scala
33.33% <ø> (ø)
:arrow_up:

...cala/com/ing/baker/runtime/core/RuntimeEvent.scala
64.28% <ø> (ø)
:arrow_up:

...ing/baker/baas/server/protocol/StateResponse.scala
100% <ø> (ø)

...baker/runtime/actor/serialization/Encryption.scala
100% <ø> (ø)
:arrow_up:

...ker/runtime/actor/process_index/ProcessIndex.scala
81.42% <ø> (ø)
:arrow_up:

.../baker/runtime/actor/LocalBakerActorProvider.scala
100% <ø> (ø)
:arrow_up:

...ialization/protomappings/RuntimeEventMapping.scala
100% <100%> (ø)

... and 65 more

Continue to review full report at Codecov.

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


How to listen changes of stored property?

Hello! Good Job, I like your repo. But I have a question :)
I have cell with UITextField and configure, it with default text. TextField from cell after editing change stored property in View Controller. When I scroll tableView down and up again textfield text setting to default textFieldText value. How to listen changes of textFieldText?
// stored property
var textFieldText = "TextField"

override func viewDidLoad() {
    super.viewDidLoad()

   // .. configurate manager
    manager.memoryStorage.addItem(textFieldText)
}

// cell
 func update(with model: String) {
     textField.text =  model
 }

Hey!
The simplest would option is probably to use property observers, something like this:
var textFieldText = " foo" {
    didSet { manager.memoryStorage.setItems([newValue]) }
}


Lua Wrapper

I wrapped it with Swig. But, I can not wrap TF_NewWhile because of the const pointer. I do not know why it says that it is read-only, because I tested it with another C program and it compiled. If anyone can explain why it says it is read-only I would like to learn the reason it refuses to compile it.

@darkSKULL-labs ,
In order to expedite the trouble-shooting process, could you please provide the following information
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version:
Python version:
Installed using virtualenv? pip? conda?:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
and the exact sequence of commands / steps that you executed before running into the problem
Thanks!

I don't think you can help I don't think it is a tensorflow problem.

@darkSKULL-labs ,
Without the reproducible code, it would be difficult for us to debug the issue. In order to expedite the trouble-shooting process, could you please provide a minimal code snippet and the TensorFlow version you are using.
Also, if this is not a TensorFlow bug or feature request, it is better asked on StackOverflow. There is a larger community that reads questions there. Thanks!

I am pretty sure it is a problem with GCC, the swig wrapper is so simple you could make it in five seconds if you want to test it.

This is the swig wrapper
%module lua_tensorflow
%{
#include <tensorflow/c/c_api.h>
%}

%ignore TF_NewWhile;
%include <tensorflow/c/c_api.h>

The problem is GCC will not assign the result from TF_NewWhile because ti believes it is read only but it doesn't make any sense to me why it thinks that.

@darkSKULL-labs If this is not a TF problem, then please post it in Stackoverflow where there is a huge community to help each other.
This is not Build/Installation or Bug/Performance issue. Please post this kind of support questions at Stackoverflow. There is a big community to support and learn from your questions. GitHub is mainly for addressing bugs in installation and performance. Thanks

Ok thanks

Yes, I found out what the problem was and can fix it.

Yes, I just wanted to show why Google uses Swig for tensorflow. Thanks.

Butler, you are supposed to learn now and close the thread.


Links

Changes made:
Description
All links go to a new tab, we were just missing one lil t at the end of target.
Testing:
Before making this pull request, I:

[ ] Cleaned the code the way Vue likes it - run 'npm run lint --fix'
[x] Made sure all tests run
[ ] Ran WAVE plugin 508 compliance tool

I can confirm this has been checked on:

[x] Chrome
[x] Safari
[x] Edge
[x] Firefox
[ ] Samsung Internet
[ ] Internet Explorer 11 (not supported, but still needs at least a working user redirect page)

Bleh, unfortunate branching Marty...do better.  Guess this small change will have to wait for the sidebar bug to be approved.


Hipifying a Cuda file that has a call to a reduction function

Hi!
I'm using a CUDA code that calls a function from the NVIDIA CUB library.
This is the code:
    // Determine temporary device storage requirements
    void     *d_temp_storage = NULL;
    size_t   temp_storage_bytes = 0;

    cub::DeviceReduce::Sum(d_temp_storage, temp_storage_bytes, in_d, sum_d, N);

    // Allocate temporary storage
    CHECK(cudaMalloc(&d_temp_storage, temp_storage_bytes));

    // Run sum-reduction
    cub::DeviceReduce::Sum(d_temp_storage, temp_storage_bytes, in_d, sum_d, N);

'in_d' is the input array with the values to be reduced, 'sum_d' is an array with a single position to include the result of the complete reduction of the array, and 'N' is the number of elements in the array that are to be reduced.
Is there any equivalent HIP library function for Sum reduction that I can use on an AMD Fiji card (AMD R9 Nano)?

Hi,
As of today no. Currently, HIP only supports just programming APIs.

Thanks for the fast reply!
So how can I implement something like shown here?
https://devblogs.nvidia.com/parallelforall/faster-parallel-reductions-kepler/
If I convert the kernel (using hipify):
inline device
int warpReduceSum(int val) {
for (int offset = warpSize/2; offset > 0; offset /= 2)
val += __shfl_down(val, offset);
return val;
}
It converts to:
inline device
int warpReduceSum(int val) {
for (int offset = hipWarpSize/2; offset > 0; offset /= 2)
val += __shfl_down(val, offset);
return val;
}
I get errors with hipcc, saying that it does not recognize 'hipWarpSize'.
What is the correct translation between this piece of CUDA code and HIP code?

Hi,
We are working on hipWarpSize. For now, you can define it as a macro of 64 for HIP and 32 for CUDA. shuffle instructions are implemented in the compiler.

Is there already some HIP Sum reduction kernel for floating points?

Hi,
Not yet. Did you try changing hipWarpSize to 64?
__inline__ __device__
int warpReduceSum(int val) {
for (int offset = hipWarpSize/2; offset > 0; offset /= 2)
val += __shfl_down(val, offset);
return val;
}```

By "implemented in the compiler" do you mean '__shfl_down' it should work with hipcc?
Because at least for the version I'm using hipcc complains about it.
I believe it only works with nvcc (the nvidia compiler).

Hi,
Here are the APIs in header.
https://github.com/GPUOpen-ProfessionalCompute-Tools/HIP/blob/master/include/hcc_detail/hip_runtime.h#L405

When using:
val += __shfl_down(val, offset, hipWarpSize);
I get when compiling with hipcc:
error: :0:0: in function ZN12_GLOBAL__N_154_Z18deviceReduceKernel16grid_launch_parmPfS0_i_functor19__cxxamp_trampolineEiiiiiiiiiiiijPKfPfi void (i32, i32, i32, i32, i32, i32, i32, i32, i32, i32, i32, i32, i32, float addrspace(1), float addrspace(1), i32): unsupported call to function __hsail_activelanepermute_b32
Failed to get ehdr: invalid `Elf' handle
clang-3.5: error: linker command failed with exit code 1 (use -v to see invocation)
Died at /opt/rocm/bin/hipcc line 230.

This is the complete code:
#define hipWarpSize 64  // AMD: 64, NVIDIA: 32
inline device
TYPE warpReduceSum(TYPE val) {
for (int offset = hipWarpSize/2; offset > 0; offset /= 2)
val += __shfl_down(val, offset, hipWarpSize);
return val;
}
inline device
TYPE blockReduceSum(int val) {
shared TYPE shared[64]; // Shared mem for 32 partial sums
int lane = hipThreadIdx_x % hipWarpSize;
int wid = hipThreadIdx_x / hipWarpSize;
val = warpReduceSum(val);     // Each warp performs partial reduction
if (lane==0) shared[wid]=val; // Write reduced value to shared memory
__syncthreads();              // Wait for all partial reductions
//read from shared memory only if that warp existed
val = (hipThreadIdx_x < hipBlockDim_x / hipWarpSize) ? shared[lane] : 0;
if (wid==0) val = warpReduceSum(val); //Final reduce within first warp
return val;
}
global void deviceReduceKernel(hipLaunchParm lp, TYPE *in, TYPE *out, int N) {
TYPE sum = 0;
//reduce multiple elements per thread
for (int i = hipBlockIdx_x * hipBlockDim_x + hipThreadIdx_x;
i < N;
i += hipBlockDim_x * hipGridDim_x) {
sum += in[i];
}
sum = blockReduceSum(sum);
if (hipThreadIdx_x==0)
out[hipBlockIdx_x]=sum;
}

The __shfl_down has only been implemented recently in the hcc compiler with the GCN ISA backend (if you see a link /opt/rocm/hcc --> /opt/rocm/hcc-lc).  You could get the compiler update either by building it from source (you'll also have to rebuild hip with the new compiler) or you could wait for a few days more to get a new binary update from apt-get.

As the conversation is inactive, closing issue.


test: make unit tests FIPS provider version aware

This PR is to address the CI failures when running 3.0.x against the validated 3.0.0 FIPS provider.
Adding additional test cases is separate.
Fixes #19171

[ ] documentation is added or updated
[x] tests are added or updated

An alternative would be to version check all of the currently loaded providers against the testable version rather than just the FIPS provider.

Looks ok to me Pauli

@paulidale
As @t8m suggested changing the code in EVP_PKEY_eq() to this seems to work in most cases..
    if (a->keymgmt != NULL || b->keymgmt != NULL) {
        int selection = SELECT_PARAMETERS;

        if (evp_keymgmt_util_has((EVP_PKEY *)a, OSSL_KEYMGMT_SELECT_PUBLIC_KEY))
            selection |= OSSL_KEYMGMT_SELECT_PUBLIC_KEY;
        else
            selection |= OSSL_KEYMGMT_SELECT_KEYPAIR;
        return evp_pkey_cmp_any(a, b, selection);
    }

30-test_evp_extra.t still fails..

With that last change the test log is now:
04-test_encoder_decoder.t        (Wstat: 256 Tests: 3 Failed: 1)
  Failed test:  3
  Non-zero exit status: 1
25-test_verify.t                 (Wstat: 512 Tests: 182 Failed: 2)
  Failed tests:  134-135
  Non-zero exit status: 2

@paulidale
As @t8m suggested changing the code in EVP_PKEY_eq() to this seems to work
    if (a->keymgmt != NULL || b->keymgmt != NULL) {
        int selection = SELECT_PARAMETERS;

        if (evp_keymgmt_util_has((EVP_PKEY *)a, OSSL_KEYMGMT_SELECT_PUBLIC_KEY)
             && evp_keymgmt_util_has((EVP_PKEY *)b, OSSL_KEYMGMT_SELECT_PUBLIC_KEY))
            selection |= OSSL_KEYMGMT_SELECT_PUBLIC_KEY;
        else
            selection |= OSSL_KEYMGMT_SELECT_KEYPAIR;
        return evp_pkey_cmp_any(a, b, selection);
    }

30-test_evp_extra.t passes if you also check b

The way I'm reading this is that with this change, all the stuff done in test/ in this PR can be dropped.  Did I read that wrong?

@paulidale
As @t8m suggested changing the code in EVP_PKEY_eq() to this seems to work
    if (a->keymgmt != NULL || b->keymgmt != NULL) {
        int selection = SELECT_PARAMETERS;

        if (evp_keymgmt_util_has((EVP_PKEY *)a, OSSL_KEYMGMT_SELECT_PUBLIC_KEY)
             && evp_keymgmt_util_has((EVP_PKEY *)b, OSSL_KEYMGMT_SELECT_PUBLIC_KEY))
            selection |= OSSL_KEYMGMT_SELECT_PUBLIC_KEY;
        else
            selection |= OSSL_KEYMGMT_SELECT_KEYPAIR;
        return evp_pkey_cmp_any(a, b, selection);
    }

30-test_evp_extra.t passes if you also check b

The way I'm reading this is that with this change, all the stuff done in test/ in this PR can be dropped. Did I read that wrong?

Yes you read it wrong.. There are a group of tests that access EVP_PKEY_eq() internally that currently fail.. such as the ssl related tests and cms. This change fixes that issue.. There are numerous other issues because we added tests to check new behaviour, or we  broke tests because we changed behaviour (such as not allowing Explicit EC curves).

As @slontis wrote, that fixes the pkey eq problems.  There are plenty of other bug fixes.

Yeah, I pushed the authorship change to the wrong repo.  FFS.
Fixed now I hope.

I'll be diverting from the two remaining fixes (EC explicit curve related I suspect) to working on adding the additional required tests tomorrow.

Do you mean the CI workflow?

Yes, we need additional CI workflows to validate future fixes.

@paulidale The commit I've added should fix the failures related to explicit curves handling.

Thanks @t8m !

I can confirm that these changes allow the entire test suite to pass against the 3.0.0 FIPS provider.
@t8m, I made a small change to the detection of the 3.0.0 FIPS provider -- the check routine doesn't return a Boolean, can also return a failure code.
I've also set the comments to a standard format: FIPS(3.0.0): <reason> #issue which should help us eventually remove them.

Yes, not having them makes debugging the new one much easier.

This is what happens when you prebuild :)

Maybe leave out the test of the 3.0.0 provider since it is currently broken by expired certs?

This is testing 3.0 fips against master, which is good for new commits..
What about one for LTS releases..i.e 3.0.X against 3.0 provider?

branch: 3.0 covers this.

Still OK with the fixups.

@paulidale if you are going to force push, instead of doing fixup commits can you at least comment on what you changed please.

I squashed the fixup commit.

Any idea why the buildbot errors are happening?

No idea unfortunately.

I wonder if buildbot  is still not clearing old build folders..

I wonder if buildbot is still not clearing old build folders..

That really looks like it as this PR is not merged yet. @levitte? We absolutely must build into clean directory to avoid any interference in between PR builds.

It's not quite past the 24 hour delay period, so not merged.
I'm kind of expecting problems merging this to 3.0 as well.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master and 3.0.  3.0 required dropping the KDF_CTX_dup part of evptest but was otherwise clean enough.


Surface inner exception message to ErrorDetails message

Currently, if there is an exception, the cmdlet wraps it with a generic exception and the error message the user gets interactively is a generic one that something bad happened.
They then will look at the innerexception message to determine the right course of action.
This change puts the innerexception.Message as the ErrorDetail message
Addresses https://github.com/PowerShell/PowerShell/issues/2942

restarted macOS due to tests failing to launch.

LGTM.


Debugger doesn't work with gdb 12

Describe the bug
Ghidra debugger doesn't work with gdb 12, the dynamic listing and the memory view won't show the correct result.
To Reproduce
Steps to reproduce the behavior:

Use the Ghidra debugger with the gdb release on Ubuntu 22.04 or archlinux
Check the dynamic listing and memory view

Expected behavior
The dynamic listing should display the instructions being executed
Screenshots

Attachments
secrecy.tar.gz
Environment (please complete the following information):

OS: [e.g. Archlinux]
Java Version: [11.0.15]
Ghidra Version: [10.1.4]
Ghidra Origin: [official ghidra-sre.org distro]
Gdb  Version: [12.1]

@ucasqsl Happy to try to troubleshoot this - any chance you could run the following two commands, either in gdb proper or in the Interpreter window?
info proc mappings maintenance info sections ALLOBJ nosections

Wow, hmmmm - those are the answers I would have expected, which should mean you’d have good values in the Modules and Regions views.  Bad/missing values there are typically the cause of the failure in the dynamic listing.
The other possibility is that the trace itself has somehow failed.  I can’t see what’s in Threads, Registers, or most of the trace-driven windows in your screenshot, but the fact that you have entries in the Time window suggests a trace was started.
Unfortunately, am on the road at the moment without my computer, but have asked @nsadeveloper789 to take a look at this too.

I'm not sure I've seen that layout for info proc mappings before. I'm guessing GDB changed it. sigh. On the good news front, it looks like they added a "Perms" column!!! The "Debug Console" may provide some hint at the errors, but I suspect to solve this, we'll have to try GDB 12 ourselves. In the meantime, and I realize this may be unpleasant, the best workaround is probably to compile and install GDB 11. I recommend setting the install prefix to somewhere in your home directory, so you're not modifying your system out-of-band from the package manager.
To be sure this is the only issue, could you please expand the "Modules" node in the Objects tree and confirm the accuracy of that list? Or is it empty?

To be sure this is the only issue, could you please expand the "Modules" node in the Objects tree and confirm the accuracy of that list? Or is it empty?

Only libc and ld, but the program itself doesn't appear.

Hmm. The address ranges are also 000s. I'll note that, too for GDB 12 compatibility. Thanks!


💚 Move email tests to "tests with deps" job

Change Summary
We need to run tests on CI with and without email dependency group installed. This allows combining both runs into cumulative coverage report.
Related issue number
None
Checklist

[X] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

At the moment of writing this comment main has 94.63% coverage and this PR has 94.68%.

please review


Create a sample with Azure DevOps

Now available at https://github.com/microsoft/delta-kusto/blob/main/documentation/tutorials/az-dev-ops/README.md.


Gulp prepublishNonBundle takes a really long time on a Mac

Sometimes this task can take 40 minutes
I believe it's all the check-datascience-dependencies part. Webpack --profile can be super slow.
Perhaps there's another way we can compute dependencies for webpack.

Closing as this has been resolved.


encoding/protojson: handling google.protobuf.Empty

Version: v1.34.1
Hi, I've encountered interop problems with protojson encoding between different languages
The following code produces error in golang:
anyResult := &anypb.Any{}
data := []byte(`{"@type":"type.googleapis.com/google.protobuf.Empty"}`)
err = protojson.Unmarshal(data, anyResult)
fmt.Println(err)

Output:
proto: (line 1:53): missing "value" field

Yet, this snippet in Java:
@Test
void test() throws InvalidProtocolBufferException {
    final var typeRegistry = TypeRegistry.newBuilder()
        .add(Empty.getDescriptor())
        .build();

    final var protoMessageAny = Any.pack(Empty.getDefaultInstance());

    final var jsonString = JsonFormat.printer()
        .preservingProtoFieldNames()
        .includingDefaultValueFields()
        .omittingInsignificantWhitespace()
        .usingTypeRegistry(typeRegistry)
        .print(protoMessageAny);

    System.out.println(jsonString);
}

produces output:
{"@type":"type.googleapis.com/google.protobuf.Empty"}

python, same thing:
from google.protobuf.any_pb2 import Any
from google.protobuf.empty_pb2 import Empty
from google.protobuf.json_format import MessageToJson

a = Any()
a.Pack(Empty())
print(MessageToJson(a))

Output:
{
  "@type": "type.googleapis.com/google.protobuf.Empty"
}

It doesn't seem golang implementation should expect value field

https://github.com/golang/protobuf/issues/759#issuecomment-593793968
Found this issue, says it was fixed, but the same behaviour reproduces on<EMAIL_ADDRESS>

Verified, v1.20.0 still reports missing "value" field as does the current v1.34.1.

This is the same as https://github.com/golang/protobuf/issues/759. Someone responded with https://github.com/protocolbuffers/protobuf/issues/5390#issuecomment-476424326.  I'm uncertain if that is affirmative as there has been no further action/decision and the issue was simply closed as inactive.  May want to reopen that issue.

I think this is the other side of #759: I don't know if we should marshal an Empty with a value of {} or no value, but we definitely should parse what other languages are producing.

The documentation for google.protobuf.Any.value says:

[The value field] must be a valid serialized protocol buffer of the above specified type.

Elsewhere, the documentation for google.protobuf.Any says:

If the embedded message type is well-known and has a custom JSON representation, that representation will be embedded adding a field value which holds the custom JSON in addition to the @type field.

Then, the documentation for google.protobuf.Empty says:

The JSON representation for Empty is empty JSON object {}.

Thus, it seems that the current behavior is correct. That said, the protobuf ecosystem is full of inconsistencies where the implementations do not follow what is documented. Thus, I agree with what @neild says. Pragmatically we should just do whatever the other languages do.

I’m on the same page as dsnet. Technically, we’re following the spec as written… but that doesn’t much matter if other implementations are accepting this and we’re not.

Changing behaviour of golang protojson.Unmarshal can be a viable compromise, otherwise compatibility would break.

I don't see an urgent reason to change Marshal's behavior. However, Unmarshal must be able to parse the output of other languages, even if that output is technically wrong. (I have no opinion on whether it is technically wrong or not.)

I definitely wasn’t advocating for changing the marshalling output. Only for accepting.

We still end up with the problem that the output generated by Go cannot be parsed by the other languages. But I agree that extending the unmarshalling makes it more permissive and thus might not be fine (unless someone depends on an error being reported).

output generated by Go cannot be parsed by the other languages

Perhaps I missed it, but the original post only mentioned protojson.Unmarshal. Is there a report of whats generated by protojson.Marshal not being accepted in another language implementation?

I only tested it with C++, but C++ doesn't accept:
{"@type":"type.googleapis.com/google.protobuf.Empty","value":{}}
It fails with
invalid JSON in   google.protobuf.Any  @ <any>: message google.protobuf.Empty, near 1:62  (offset 61):   no   such  field:   'value'

Ah, thanks for looking into that.
Given the amount of inconsistency in the ecosytem, I suspect every implementation will need to accept both the case where it is preset with {} and also missing. That said, I think it's up to the protobuf team to declare which form is "more correct" and we should probably output that.

https://github.com/golang/protobuf/issues/759 is the report for V1's marshaling issue.  With the response in https://github.com/protocolbuffers/protobuf/issues/5390#issuecomment-476424326, I think we decided to keep the behavior for V2.
I was mistaken in my comment above that this is similar to that marshaling issue.  I don't mind having the unmarshaling logic accept what the other languages output.

I also encountered this issue when using the GCP go SDK (I wonder if this is the most common scenario?).  Google LROs use anypb.Any for the result field, and there are multiple APIs service which do not send the value field when the result is emptypb.Empty, triggering this problem.  One mitigating factor with the GCP go SDK is that it only applies to protojson, and the GCP go SDK seems to default to true proto encoding now for most services.
However for protojson, the GCP Go SDK does pass both AllowPartial and DiscardUnknown.  I would think that AllowPartial=true should mean that a missing value should not be an error.  I think what has happened is that we create a new UnmarshalOptions in protojson and do not pass down AllowPartial.  I proposed a CL to pass down AllowPartial and not return an error if value is not set.  For me, this seems to fix the problem with GCP Go SDK, and (I would argue) is a bug fix rather than a behavioral change - so perhaps easier to justify.
I don't know if we should also pass down DiscardUnknown.  I would think yes, from the perspective of obeying the user's intent (and then maybe we should pass down all the options?).  But also because I think DiscardUnknown is important - we presumably do not want to fail when the service adds new fields.  Because this doesn't happen all the time, I'm guessing that this is only relevant for a new field in anypb.Any, so maybe this isn't very likely, but it still seems like good future-proofing.
The CL is https://go-review.git.corp.google.com/c/protobuf/+/617516

My primary concern here would be that if the AllowPartial is permitted in an anypb.Any, wouldn’t this then permit missing values for something more serious like a structpb.Value?
The problem here is wholly only with wrapping the emptypb.Empty type, and we should probably be allowing this missing value field for that type regardless of what AllowPartial is.

we don't honor the user's AllowPartial instruction for this case.

We don’t honor it for good reason, it’s a well-known type, and thus has a canonical representation in JSON:

If the Any contains a value that has a special JSON mapping, it will be converted as follows: {"@type": xxx, "value": yyy}. Otherwise, the value will be converted into a JSON object, and the "@type" field will be inserted to indicate the actual data type.

The problem is that arguably not emitting the {} for the Empty WKT, in the Any is a violation of the standards, and shouldn’t be produced by any of the official protobuf libraries. But it seems others have read the same standard and come to a different decision on this specific situation. That’s why we raised the issue in the upstream mainline protobuf project. Unfortunately, it seems it hasn’t received any attention. 😢
But since other of the Big4 generators are marshalling and unmarshalling it, even when AllowPartial is disabled, and this is the behavior that would need to be replicated. It’s the whole reason we separated off the json.UnmarshalOptions and didn’t copy over the AllowPartial in this case.

I don't think AllowPartial is the right solution here. AllowPartial disables required field checks. The problem here is not a missing required field, it's a disagreement on what a valid representation of an Any containing an Empty is. If we start making AllowPartial affect the parsing of Any, we're asking people to set an option that universally affects required fields to work around a bug. That's confusing, and will have bad knock-on effects for many years to come.
We should change the Go JSON parser to accept {"@type":"type.googleapis.com/google.protobuf.Empty"} as a valid Any containing an Empty.
Every parser that doesn't accept {"@type":"type.googleapis.com/google.protobuf.Empty","value":{}} as a valid Any containing an Empty should be changed to do so.
The protobuf project maintainers should decide which form is the correct one, and we should eventually change all encoders to produce it. Although that's going to be tricky to do without breaking anything, so we'd probably want to wait some period of time after aligning parsers to handle either form.
We should not change AllowPartial to affect the parsing of Any.

Fair enough :-)

We should change the Go JSON parser to accept {"@type":"type.googleapis.com/google.protobuf.Empty"} as a valid Any containing an Empty.

I'll send a version that does this.

https://go.dev/cl/618395 changes the Go JSON unmarshaler to accept the output produced by C++/Java, but C++ (and I think Java) still don't accept the output produced by Go. Fully fixing this issue requires addressing that.
Just changing the Go marshaler isn't going to work, since we'd start producing output that can't be parsed by earlier versions of protojson. Addressing the issue will require coordinating with the C++/Java maintainers (at least). Perhaps that should be a new issue, but I'm going to reopen this one for now so we don't lose track of it.

This edge case should also be added the conformance test suite to ensure that future implementations agree with each other.

@justinsb thank you so much for the fix.
Can you make a release with this version?

@chressie
Can you make a release with this fix?

@orloffv we just cut a new release https://github.com/protocolbuffers/protobuf-go/releases/tag/v1.35.2, which includes the change.


Clarify opt-out consent vs opt-in

Opt-in > opt-out. Let's write a section on this

From https://medium.com/mydata/consent-lost-gdpr-and-found-eprivacy-e85cf881ffb

An underlying issue is “opt-out” consent compared to “opt-in”. The old directive left scope for the interpretation that opt-out consent was sufficient — that consent was assumed unless otherwise indicated. Since the GDPR requires a statement or clear affirmative action, consent has to be opt-in — no response means no consent.


Logstash often stops to send data to influxdb after influxdb shutdown/restart or network disconnect

Hi,
I have the cluster of 10 RHEL 6.4 servers.
9 servers with logstash-1.5.0.rc2-1
1 server with logstash-1.5.0.beta1-1
All servers have logstash-output-influxdb.
I check the logstash behavior in case of influxdb crash/kill/shutdown for several minutes and restart, or network disconnect for server seconds or minutes
The results as follow:
Kill the influxdb for several minutes: --Message start to appear in logstash.log on each server
"{:timestamp=>"2015-03-25T14:21:13.456000+0200", :message=>"Failed to flush outgoing items", :outgoing_count=>1, :exception=>Errno EBADF: Bad file descriptor - Bad file descr
iptor>, :backtrace=>["org/jruby/RubyIO.java:2099:in close'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/ftw-0.0.42/lib/ftw/connection.rb:176:inconnect'", "org/jruby/RubyArray.java:1613:
in each'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/ftw-0.0.42/lib/ftw/connection.rb:142:inconnect'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/ftw-0.0.42/lib/ftw/agent.rb:421:in c onnect'", "org/jruby/RubyProc.java:271:incall'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/ftw-0.0.42/lib/ftw/pool.rb:49:in fetch'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/ftw-0.0 .42/lib/ftw/agent.rb:418:inconnect'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/ftw-0.0.42/lib/ftw/agent.rb:283:in execute'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/ftw-0.0.42/lib /ftw/agent.rb:181:inpost!'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/logstash-output-influxdb-0.1.3/lib/logstash/outputs/influxdb.rb:237:in post'", "/opt/logstash/vendor/bundle/jruby /1.9/gems/logstash-output-influxdb-0.1.3/lib/logstash/outputs/influxdb.rb:231:inflush'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/stud-0.0.19/lib/stud/buffer.rb:219:in buffer_flush'", "org/jruby/RubyHash.java:1341:ineach'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/stud-0.0.19/lib/stud/buffer.rb:216:in buffer_flush'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/stu d-0.0.19/lib/stud/buffer.rb:193:inbuffer_flush'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/stud-0.0.19/lib/stud/buffer.rb:159:in buffer_receive'", "/opt/logstash/vendor/bundle/jruby/1 .9/gems/logstash-output-influxdb-0.1.3/lib/logstash/outputs/influxdb.rb:196:inreceive'", "/opt/logstash/lib/logstash/outputs/base.rb:88:in handle'", "(eval):4529:ininitialize'", "org/j
ruby/RubyProc.java:271:in call'", "/opt/logstash/lib/logstash/pipeline.rb:279:inoutput'", "/opt/logstash/lib/logstash/pipeline.rb:235:in outputworker'", "/opt/logstash/lib/logstash/pipe line.rb:163:instart_outputs'"], :level=>:warn}"
--Data doesn't sent to the influxdb.
Starting the influxdb:
--The message stops to appear on all servers.
--Several servers started to send data again to the influxdb as well.
--Other never recover the sending, only kill -9 and new start solved the issue.
--Even '/etc/init.d/logstash stop' doesn't work as in the bug #2796.
I tried this test several times - each time another logstashes recovered and other stucks.
I checked the network status on servers with stucked logstash - the port 8086 (influxdb rest) was in close_wait status.
In the second test I shutdown the NIC for few minutes and started it back.
Same logstash's behavior found except one - no errors appeared in the log file at all.

There's another issue that I fixed here:
https://github.com/logstash-plugins/logstash-output-influxdb/pull/15
It retries over and over to POST data to InfluxDB when it's down (good!), but duplicates the batch data on every attempt (bad!).

I was able to isolate that issue.
In the below snippet, you can see that if you have an exception it sleep 1 and retry. So in my case influx hosted what sending me a weird http response and I had http parse exception from ftw. All I did was to put a begin/rescue block around post() or any other logic if you want to retry a couple of times. But otherwise, it will freeze logstash completely requiring a kill -9.
@buffer_state[:outgoing_items].each do |group, events|
          begin
            if group.nil?
              flush(events,final)
            else
              flush(events, group, final)
            end

            @buffer_state[:outgoing_items].delete(group)
            events_size = events.size
            @buffer_state[:outgoing_count] -= events_size
            items_flushed += events_size

          rescue => e

            @buffer_config[:logger].warn("Failed to flush outgoing items",
              :outgoing_count => @buffer_state[:outgoing_count],
              :exception => e,
              :backtrace => e.backtrace
            ) if @buffer_config[:logger]

            if @buffer_config[:has_on_flush_error]
              on_flush_error e
            end

            sleep 1
            retry
          end
          @buffer_state[:last_flush] = Time.now
        end

      ensure
        @buffer_state[:flush_mutex].unlock
      end

This doesn't solves the issue with influxdb plugin.
The plugin already uses begin/rescue block:
begin
@logger.debug("Post body: #{body}")
response = @agent.post!(@url, :body => body)
rescue EOFError
@logger.warn("EOF while writing request or reading response header from InfluxDB",
:host => @host, :port => @port)
return # abort this flush
end

I suggest that the issues caused by FTW::Agent, seems that FTW cannot recreate the connection.
So, I disabled the FTW and just implemented Net::HTTP::Post instead.
Now plugin reconnects.

This should be fixed in v 5.0.0 Thanks to #55. I'm going to close this issue given that. If you continue to encounter an issues please open a new issue. Thanks :)


Improve frontend

Freeze time validation for withdraw button
Improve UI
Add timer clock

I would like to work on frontend thing and it will be great if you can show me type of design you want @SANTHOSH17-DOT

Hey @JagveerGagaan ,
I would like to have a simple and compact front end since it'll be a browser extension.

Have a timer like this for the time left.
I am open to any suggestions. Attach screenshots once you are done.

ok I will let you know about further steps : )

Hey @JagveerGagaan
Any update on this?

@SANTHOSH17-DOT Is it under IWOC if it is then please assign this to me sir

thanks sir @SANTHOSH17-DOT

can you give some like how should frontend look like basically i  am asking about basic hint you can give me @SANTHOSH17-DOT

like what should be the your materials to be added @SANTHOSH17-DOT

I do not get you. Frontend for the application is already done. You have to work on UI enhancement. A timer needs to be added.

ok sir

like timer have to be added in the forntpage?

@SANTHOSH17-DOT

like timer have to be added in the forntpage?

No, it should start after the user deposits their crypto

ok

sir can you explain it in simple words like i am not able to understand what you want to say @SANTHOSH17-DOT

Sorry @Bharadwajshivam28 It needs some prerequisite knowledge. You should be comfortable with react and interacting with smart contracts

ok sir


In conflict resolution, branch name get over the rest if the UI.

When you have the part of your UI where branch name are come over other panels when those panel are over text with merge issue.
I think the attached file illustrate it better than me.

Yes, I came here to report the same thing. :+1:

#188 covers this, duplicate bug.

Yup, this is a duplicate of #188; please subscribe there for updates.


Values of boolean parameters are always resets to true

When I try to set "DE_ONLY" to false (see image below) it is always automatically changed to true when focus is lost.

Where is this dialog? FMU parameters?

The is a parameter for an FMU. Scalar variable example:
<ScalarVariable name="dummy" valueReference="3" causality="parameter" variability="fixed" initial="exact"><Boolean start="false" /></ScalarVariable>


Problems assigning variables

Hi,
I have a pipeline with the following structure:
node {

        stage ("Build & Launch Tests") {

            def tests = [:]
            def nrOfThreads = 2

                for (int i = 1; i <= nrOfThreads; i++) {                    tests[i] = {

                        lock(label: 'afthost', variable: 'LOCKED_SERVER', quantity:1) {

                            lock(label: 'slave_node', variable: 'LOCKED_SLAVE', quantity:1) {

                                stage ("Build Tests "  + testGroup) {
                                    cleanWs()

                                    echo env.LOCKED_SLAVE
                                    echo env.LOCKED_SERVER

                                    if (env.LOCKED_SERVER == null) {
                                        error("Server not locked properly")
                                    }

                                    if (env.LOCKED_SLAVE == null) {
                                        error("Slave not locked properly")
                                    }

                                }

                            }
                        }
                    }
                }
            tests.failFast = true
            parallel tests
        }

}

At one time it caused the issue:
Trying to acquire lock on [Label: afthost, Quantity: 1]
Found 0 available resource(s). Waiting for correct amount: 1.
[Label: afthost, Quantity: 1] is locked, waiting...
Lock acquired on [Label: afthost, Quantity: 1]
[Pipeline] {
[Pipeline] lock
Trying to acquire lock on [Label: slave_node, Quantity: 1]
Found 0 available resource(s). Waiting for correct amount: 1.
[Label: slave_node, Quantity: 1] is locked, waiting...
Lock acquired on [Label: slave_node, Quantity: 1]
[Pipeline] {
[Pipeline] node
Running on Jenkins in /apps/jenkins/latest/jobs/TAAS_PARALLEL_CAMPAIGN_POC/workspace@2
[Pipeline] {
[Pipeline] stage
[Pipeline] { (Build Tests PX_POOL1)
[Pipeline] cleanWs
[WS-CLEANUP] Deleting project workspace...
[WS-CLEANUP] Deferred wipeout is used...
[WS-CLEANUP] done
[Pipeline] echo
null
[Pipeline] echo
null
[Pipeline] dir

In the above log I print both locked resources, but they are null.
It has happened once, but I could not find any references to a similar problem. I have tried reproducing this in a smaller project but so far to no avail.
In summary I have:

two group of locked resources
since job is in parallel they are locked in parallel
I use scripted pipeline

I just wanted to signal this, in case anyone knows about it.

This sounds a lot like the bug fixed in 2.7, very strange...

Ah thanks. That explains a few things:

The issues arrived in 2.6
I Tried reproducing in 2.7 (hence my inability to reproduce) :)

Thanks I'll try and upgrade my plugin.

For reference: The bug fixed in 2.7 has existed since at least 2.2 (maybe even since this plugin started supporting pipeline)

Sorry for my late response. It worked so good I forgot to close this issue. Thanks. It works like a charm.


⚠ RPC server - Conflux has degraded performance

In 7f47d05, RPC server - Conflux ($CONFLUX_RPC_URL) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: RPC server - Conflux performance has improved in 14fac95 after 4 minutes.


Dev plan for Voyage-6

Started in #220 (Dev plan for voyage 4)
Main features

State of the front-end part. React Context API or Redux.
A few missing pages / forms on the front-end.
Testing components with Jest and Enzyme.
Real content and content serving. We search for ready-to-use APIs for jobs, events, projects to contribute as well as user stories.
Back-end and DB:
There are 2 options. Gatsby is working well with Contentful CMS (headless back-end, quite similar to Wordpress) and graphQL server or serverless backend (similar to Node and its Express framework). Last option has own solutions for DB.
We can use one of options for back-end or both together.

MVP

We can collect and serve user data.
User data forms on the front-end, user authentication, back-end for storing data.
Jobs page is ready and serves a real content. (I know at least 1 API which we can use).

Hey guys 👋
@mar-bi @miljan-fsd
For reference take a look at this for the Context API in React and Gatsby: https://blog.scottspence.me/react-context-api-with-gatsby
There's some links in there to a couple of YouTube videos which cover it beautifully, let me know if there anything you need 💪

More details on MVP - https://github.com/chingu-voyage6/grad.then/issues/132#issuecomment-398388436


Added in-built media player to play video without navigating to another window

@ramdevcm I don't quite get it what you are willing to add? Before making any off-topic pull request. Please do create issue.

can I know why this is labelled as invalid?

can I know why this is labelled as invalid?

Deployment got failed.

can I know why this is labelled as invalid?

because it fails ci deployment on vercel

This was the issue

should I need to create an issue?

should I need to create an issue?

Yes! you should. It will be better, and check what error you got while deploying

@ramdevcm checkout logs right here.
13:51:13.738  	Cloning github.com/space-voyager-21/space-voyager (Branch: master, Commit: 7915063)
13:51:14.576  	Cloning completed: 837.411ms
13:51:14.601  	Analyzing source code...
13:51:15.908  	Installing build runtime...
13:51:18.322  	Build runtime installed: 2.414s
13:51:20.823  	Looking up build cache...
13:51:23.356  	Build cache downloaded [32.94 MB]: 2312.579ms
13:51:24.625  	Detected package.json
13:51:24.625  	Installing dependencies...
13:51:24.936  	yarn install v1.22.11
13:51:25.038  	[1/4] Resolving packages...
13:51:25.819  	success Already up-to-date.
13:51:25.826  	Done in 0.90s.
13:51:26.363  	><EMAIL_ADDRESS>build /vercel/path0
13:51:26.363  	> react-scripts build
13:51:27.977  	Creating an optimized production build...
13:51:30.782  	Failed to compile.
13:51:30.783  	./src/components/apod/Apod.js
13:51:30.783  	Cannot find module: 'react-player/lazy'. Make sure this package is installed.
13:51:30.784  	You can install this package by running: yarn add react-player/lazy.
13:51:30.799  	npm ERR! code ELIFECYCLE
13:51:30.799  	npm ERR! errno 1
13:51:30.802  	npm ERR<EMAIL_ADDRESS>build: react-scripts build
13:51:30.803  	npm ERR! Exit status 1
13:51:30.803  	npm ERR!
13:51:30.803  	npm ERR! Failed at the<EMAIL_ADDRESS>build script.
13:51:30.803  	npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
13:51:30.816  	npm ERR!  Make sure this package is installed.
13:51:30.784 You can install this package by running: yarn add react-player/lazy.
😑

ig he didnt push package.json

Are you sure? you mean any problem in dependencies?

13:51:30.783 Cannot find module: 'react-player/lazy'. Make sure this package is installed. 13:51:30.784 You can install this package by running: yarn add react-player/lazy.
😑

Ohh yea! here it is.

yes added a new package

then commit and push package.json   without   closing this pull request

can you add it??
or should I PR again?

u dont need to close this pr , just commit and push @ramdevcm

Code Pushed


add door exception handling and tests

Next PR in the series of #168. Exception handling for Doors

 Logging


Populate "About" area of repository with useful information

The "About" area for the repository currently displays "No description, website, or topics provided." I think it would be a good idea to put some useful information in there about the project.

I like the description that was added to the "About", but it's missing a period at the end. Please add a period to the end of that sentence.
I would also like to see a link to or the URL of the GitHub Pages documentation URL added to the "About" area.

@esabol Sure, thanks. I will add period.
Not sure, what do you propose concerning url? I've added the url to pgsphere.org. It is appeared under the About section. It is proposed to to be the primary project place. Now it redirects to the GitHub Pages. Later, a simple site will be implemented with some information and documentation about the project.

I didn't know pgsphere.org worked! I was just thinking a link to https://postgrespro.github.io/pgsphere/, but whichever.


provide ortholog links for priority fungal pathogens (and ideally make IDs searchable to reach pombe orthologs)

WHO priority pathogens Oct 2022
source: https://iris.who.int/bitstream/handle/10665/363682/9789240060241-eng.pdf?sequence=1

Priority Level
Pathogen
In PANTHER

Critical
Cryptococcus neoformans
YES

Critical
Candida auris

Critical
Aspergillus fumigatus

Critical
Candida albicans

High
Nakaseomyces glabrata (Candida glabrata)

High
Histoplasma spp.

High
Eumycetoma causative agents

High
Mucorales (e.g., agents causing mucormycosis)

High
Fusarium spp.

High
Candida tropicalis

High
Candida parapsilosis

Medium
Scedosporium spp.

Medium
Lomentospora prolificans

Medium
Coccidioides spp.

Medium
Pichia kudriavzeveii (Candida krusei)

Medium
Cryptococcus gattii

Medium
Talaromyces marneffei

Medium
Pneumocystis jirovecii

Medium
Paracoccidioides spp.

I put all priorities but we may restrict to Critical/high

Find out which are in PANTHER and request any exclusions
Use PANTHER to provide ortholog data store in Chado
Link to UniPort entries in ortholog page section (open website ticket when ready)
Make UniProt? IDs searchable in pombe simple search and gene mapper

using PANTHER directly won't work here
https://pantherdb.org/panther/speciesTree.jsp
because only includes species with a lot of data for transfer  in the core trees
We might be able to use PANTHER family IDs from InterPRO though?


WIP: Add new classifiers

We should wait for those classifiers to be added to PyPi.

I think I see what you mean with the 4.1 etc. I think we should do something like just 4. 4.1, 4.0, 4.2 etc will all be compatible

Okay.and it will be easier to add those in pypi than requesting a minor version each time... I will update this !

For now those classifiers do not exist on PyPi. We can still reopen this later.


hypercorn 0.9.0 not working on Windows 10

This is a known issue of hypercorn and will be fixed (already been fixed, not released currently) by its author in the future.
Flask works fine as always. Just use app.py or twistd.

Now version 0.9.2 works just fine


Use message queue gem

Part of ticket: https://trello.com/c/z8b6VmCU/317-update-panopticon-when-links-are-sent-to-publishing-api

@rboulton @alext
I've squashed and updated this PR. It should be good to merge, but I hold off doing that until we've added the panopticon PR. That might surface some issues relevant to this repo. But if you can review this PR in the meantime I'd be happy.

One tiny (non-merge-blocking) nit, but otherwise this is fine.

Updated PR to use the new environment var version of the gem.

I've squashed & rebased the commits here. I think this should work, but haven't done a manual test.
I'm closing this and making a new ticket on Trello:
https://trello.com/c/rCUEgyjz/413-make-content-register-use-the-message-queue-gem
We'd like to focus on getting panopticon running with this gem first.


Unknown react install found

@AllenFang I'm using a local fork of the great REACT-BOOTSTRAP-TABLE. :slightly_smiling_face:  From what I see in webpack.umd.config.js, you have react and react-dom specified in externals still installing it locally in my another project it shows me extraneous dependency of react within my project.
Using the command npm ls react shows output

This is showing duplicate react error in the browser console.
I'm not sure what's causing the error here, but recently today I pulled upstream from your repo. Could this be related to react-bootstrap-table in any case?
npm version: 5.3.0
react-version: 15.6.1

Well @AllenFang this seems to be an issue on my end and the library works as required :+1: . I learned that the externals config in webpack.umd.config.js prevents bundling react and moreover you've perfectly specified react as a peerDependency in package.json.
Cheers to that :beers:. Closing this out.

:beers:


Not able to use example provided in pseudo types.

Not able to use example provided in pseudo types section. - https://github.com/acorn-io/aml#types-pseudo
% aml eval Acornfileschema
2023/10/03 15:31:16 value kind object does not support || operation: Acornfileschema:12:28 (backtrace 15:9->15:9)

cat Acornfileschema
localData:
{
let types: {
    // Types by convention start with an uppercase letter following PascalCase
    StringItem: {
        item: string
    }
    NumberItem: {
        item: number
    }

    Item: types.StringItem || types.NumberItem
}

items: [types.Item]

// Functions are defined using the function keyword
myAppend: function {
    // Args are defined using the args keyword
    args: {
        // Args are defined using the name of the arg and the type of the arg. The type of the arg
        // follows the schema syntax described later in this document
        head: string
        tail: string
    }

    someVariable: "some value"

    // The return of the function should be defined using the return key
    return: args.head + args.tail + someVariable
}

// The arguments will be applied by the order they are assigned
callByPosition: myAppend("head", "tail")

// The arguments can also be applied by name
callByName: myAppend(tail: "tail", head: "head")
}
                containers: {
                        mywebnew: {
                                image: "nginx"
                                ports: publish: "80/http"
                                files: {
                                        "/usr/share/nginx/html/testn1.html":   localData.callByPosition
                                        "/usr/share/nginx/html/testn2.html":   localData.callByName
                                }
                        }
                }

Also smal typo in this text:
The following pattern can be used to define reusable types. Custom types are not a first class object in the language but instead objects will schema fields can be reused.
This needs to be corrected:
 but instead objects **with** schema fields can be reused.

You can't add schema to the Acornfile.  You need to add it to a file and run it as an argument of aml eval --schema-file.

Able to test psedo types example:
aml eval --schema-file schemanew.acorn filenew.acorn
2023/10/09 17:56:56 schema violation key items:
	option 1: [schema violation key item: expected kind string but got bool with value (true) [schema path types.StringItem]],
	option 2: [schema violation key item: expected kind number but got bool with value (true) [schema path types.NumberItem]]

% cat schemanew.acorn
// By convention put types in a let field named types. This ensures the objects
// are not included the evaluated output.
let types: {
    // Types by convention start with an uppercase letter following PascalCase
    StringItem: {
        item: string
    }
    NumberItem: {
        item: number
    }

    Item: types.StringItem || types.NumberItem
}

items: [types.Item]
% cat filenew.acorn
items: [{ item: "hello" },
{item: 9},
{item: true}
]

 aml eval --schema-file schemanew.acorn filenew.acorn
{
    "items": [
        {
            "item": "hello"
        },
        {
            "item": 9
        }
    ]
}

 % cat filenew.acorn
items: [{ item: "hello" },
{item: 9},
]

% cat schemanew.acorn
// By convention put types in a let field named types. This ensures the objects
// are not included the evaluated output.
let types: {
    // Types by convention start with an uppercase letter following PascalCase
    StringItem: {
        item: string
    }
    NumberItem: {
        item: number
    }

    Item: types.StringItem || types.NumberItem
}

items: [types.Item]


Z-targeting: Unable to play/unequip piano

The piano cannot be unequipped or played in Z-Targeting.
https://user-images.githubusercontent.com/104460803/175122131-8ae8146e-f3d5-4f09-87ca-38a0c34f04c9.mp4

The piano can be equipped and its working fine on my side.. Can you try reproducing it?
The issue is that it can not be unequipped rn. So, to unequip it, Press "space" and then press R.

Hey @Loryhoof I think this is okay, we just need some UI to indicate to the user how to exit.
The space key R is good so that you can use the R key to play, but it's not clear to users how to leave.
@avaer any thoughts on this?


Add a title tag to the audio

Description
To show the track name on mobile devices can we add the "title" property to the  tag?
https://stackoverflow.com/questions/39883200/use-mp3-metadata-or-html-to-display-artwork-and-track-name-on-iphone-lock-screen

Done. you can use audioTitle option


GithubMediaStore Pagination not getting the correct offset

Description
The pagination on the Media Manager screen isnt getting the correct offset, when you click for page 2 on the media manager nothing loads, after pulling it apart a bit, I found that the MediaListOptions offset is 0 on the first (expected) and then 100 on the second page, but the limit is only 50, therefore nothing is loaded.
Steps to reproduce
Have more than 50 media items, click the 2 in the bottom right corner, and nothing loads.

Environment
Tinacms Package Versions
"tinacms": "^0.37.0"
I am happy to add a PR, if someone could point me in the right direct? As not sure what is calling MediaStore.list

50 is the default, do you pass any options to your Media List?

No I dont pass any options, I was using the default set by GithubMediaStore

Hi again,
So I have created a small repo with over 50 images in it, to show the issue:
https://github.com/louMoxy/tinacms-media-issue
https://user-images.githubusercontent.com/20976565/109432598-d6131f00-7a03-11eb-8304-93cd11916f6b.mp4
Any help or advice would be apprciated, thank you


Need a way to quickly evaluate optical tracker position estimate quality

As I mentioned in #13 , there needs to be a good way to evaluate the quality of the optical tracker position estimation to know whether or not the position should be used to correct the state-space model. We shouldn't implement this until #13 is done because we will use the model to help with quality estimation. Just writing a few thoughts for now...

Maybe we can compare the new calculated position to the predicted position at the appropriate snapshot and ignore the position update if it's more than X cm away (X ~5).

But this can quickly snowball out of control if IMU drift causes us to cross the threshold.

If we get a couple frames in a row that are too far away, but the velocity and acceleration are both low (i.e., the controller isn't moving), then take the position update anyway.

As part of the DS4 changes that I'm pulling up to master each of the trackers computes a "estimated optical pose" of the tracker. Part of the info in this struct is a "projected pixel area". When filtering the position and orientation we take the average pixel area (averaged from all trackers that can see the controller) and turn it into a "tracker quality" value. The min and max pixel areas needed to compute the "tracker quality" for the position and orientation filter are editable in the controller configs. I currently use the tracker quality value to blend between optical position tracking and IMU tracking for the DS4 and for blending the optical orientation (used to fight drift) on the DS4.
This likely isn't the final word on optical quality model, but I figure it's a starting point to experiment with and it helps get the quality rating framework in place.


How to get the background color for a default profile photo for users without one?

There is a profile_accent_color_id field under the user class. But from the description it seems that this is a custom color premium users instead of the default color? If so, which api should one use to get the default color? If there is no such api, can you share the algorithm for generating the color? I see that it is consistent across the official clients.

Use user.accent_color_id: "Identifier of the accent color for ... backgrounds of profile photo ..".
The note "For Telegram Premium users only" in the documentation is redundant and will be removed.

Thank you I was confused about that word.


Refactor/get payments

Describe the changes you have made in this PR
This is a follow up of the Transactions page https://github.com/getAlby/lightning-browser-extension/pull/2035. Same as on the transactions page, in DefaultView and AllowanceView payments are fetched and transformed to transactions. In the transactions feature this transformation is abstracted into the convertPaymentsToTransactions util function. This PR applies the convertPaymentsToTransactions abstraction for the two mentioned screens.
This by itself is actually a rather small refactor. However as we are dealing with transactions most of the time and the component consuming the data is also using the property transactions, I renamed a lot of properties from "payments" to "transactions" for consistency reasons.
Link this PR to an issue [optional]
Fixes #2057
Type of change

refactor: Follow up of #2027 https://github.com/getAlby/lightning-browser-extension/pull/2035

How has this been tested?
I tested manually all screens that are affected.
Checklist

[x] My code follows the style guidelines of this project and performed a self-review of my own code
[x] New and existing tests pass locally with my changes
[x] I checked if I need to make corresponding changes to the documentation (and made those changes if needed)

@rolznz discussing the transactions page you suggested refactoring those two components, I suggested doing this in a follow up. Maybe you could also review this PR. Thanks.

Thanks @lujakob !


PGleaderchk component movement from a2ha to automate

Description
Movement of these components
automate-backend-pgleaderchk  (Go version needs to be updated)
Acceptance Criteria :
Movement of pgleaderchk to automate

PR link: https://github.com/chef/automate/pull/5666


Suggestion: split the variants with and without the remainder into two separate functions

In https://github.com/tc39/proposal-reversible-string-split#could-this-be-an-extra-option-for-the-split-method, it suggests an options bag. I'm not convinced one is necessary, not if you split the two variants into two different methods. You'd get the conciseness of the second with the  clarity of the third.
Something like this:

"a|b|c".splitWithRemainder("|", 2) - returns "a" and "b|c"
"a|b|c".splitWithoutRemainder("|", 2) - returns "a" and "b"

Names are intentionally ugly - that's not the point here.

It’s not that it’s necessary, it’s that it provides an opportunity to give a better name to the existing “count” parameter, and also provides forwards compat for any changes to split in the future.


Can't use /origin with custom origins in .mcfunction files

I have a .mcfunction file that checks if a player has a custom origin every tick with the following command:
execute store result score @a playerOrigin as @a at @s run origin has origin @s origins:origin evolution:fire_elemental
The problem is that this function file loads before the origin itself, throwing the following error:
java.util.concurrent.CompletionException: java.lang.IllegalArgumentException: Whilst parsing command on line 1: Unknown origin: evolution:fire_elemental
When reloading the data pack, the function loads and works fine using the /function command but does not execute every tick (as far as I can tell).
My log and .mcfunction files are attached below.
origins_bug_report.zip

This has been fixed at some point and is no longer an issue. (Sorry, forgot to close this whenever the fix for this was released.)


Problem in getting Max Length And Min length

We are unable to get maxLength and minLength properties in XSD while converting into JSON

Is there any news relating to this issue?
I am also experiencing the same problem.
If further details are required, please let me know.


Add root level api

This PR adds root level api route that responds server version number.

Looks good!


Switch fetching device info to go over the websocket for RPC devices

Avoid the REST call since we have to create another connection to the device when we do the rest call which will stay open for a bit. By doing everything of the websocket we put less pressure on the device, and setup is a bit faster

I don't think this will work
2024-08-27 10:13:47.295 ERROR (MainThread) [homeassistant.components.shelly] Unexpected exception
Traceback (most recent call last):
  File "/Users/bdraco/home-assistant/homeassistant/components/shelly/config_flow.py", line 216, in async_step_credentials
    device_info = await validate_input(
                  ^^^^^^^^^^^^^^^^^^^^^
  File "/Users/bdraco/home-assistant/homeassistant/components/shelly/config_flow.py", line 109, in validate_input
    await rpc_device.initialize()
  File "/Users/bdraco/home-assistant/venv/lib/python3.12/site-packages/aioshelly/rpc_device/device.py", line 189, in initialize
    await self._connect_websocket()
  File "/Users/bdraco/home-assistant/venv/lib/python3.12/site-packages/aioshelly/rpc_device/device.py", line 202, in _connect_websocket
    self.shelly["auth_domain"],
    ^^^^^^^^^^^
  File "/Users/bdraco/home-assistant/venv/lib/python3.12/site-packages/aioshelly/rpc_device/device.py", line 433, in shelly
    raise NotInitialized
aioshelly.exceptions.NotInitialized

Did some more testing. All seems good.


[WIP] repo.name includes the user/repo so split to get just repo

Note, I didn't yet look to see why this changed.  We might want to do this differently but this is what got it working locally for me.

What changed is that I made the default name be fully qualified where before it defaulted to short name

To be replaced with a similar change in #21


[SPARK-33483][INFRA][TESTS][3.0] Fix rat exclusion patterns and add a LICENSE

What changes were proposed in this pull request?
This PR fixes the RAT exclusion rule which was originated from SPARK-1144 (Apache Spark 1.0)
Why are the changes needed?
This prevents the situation like https://github.com/apache/spark/pull/30415.
Currently, it missed catalog directory due to .log rule.
$ dev/check-license
Could not find Apache license headers in the following files:
 !????? /Users/dongjoon/APACHE/spark-merge/sql/catalyst/src/main/java/org/apache/spark/sql/connector/catalog/MetadataColumn.java
 !????? /Users/dongjoon/APACHE/spark-merge/sql/catalyst/src/main/java/org/apache/spark/sql/connector/catalog/SupportsMetadataColumns.java

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CI with the new rule.

Test build #131332 has started for PR 30424 at commit 8ac37df.

Thank you, @HyukjinKwon .

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35933/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35933/

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/35933/
Test FAILed.

Merged to branch-3.0. All related tests passed.

Test build #131332 has finished for PR 30424 at commit 8ac37df.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131332/
Test PASSed.

Thank you for merging, @HyukjinKwon !


Ability to set filepath in v3

Problem
Currently  spotdl [trackUrl] downloads to a predefined filename  -
It would be good to manually specify the filename to be downloaded to, as well as the target folder.
Solution
restore the ability from V2 to set --output_filepath

a5a4e72
There's -o <filepath>  or --output...

Maybe I do not understand how to use the -o <filepath> flag
I want to specify the target folder and filename for the downloaded file.
But -o <filepath> only specifies the target folder. It still uses a generated filefname.
if not os.path.isdir(arguments.path):
            sys.exit("The output directory doesn't exist.")

You can change the FileName around here in Download.py
https://github.com/spotDL/spotify-downloader/blob/3c66180e73461083e4d9d3bb0b938038e081594e/spotdl/download/downloader.py#L140-L171


feat: Calendar deep link to course component

LEARNER-9888: Navigate to a particular course component upon tapping the link from the native calendar application synced with the calendar for that specific course.
Demo:
https://github.com/openedx/openedx-app-ios/assets/11990137/447de2bb-3605-4601-be89-d8be6ee44bcb

@shafqat-muneer can you open full flow to this component?
for example, open from push:
https://github.com/openedx/openedx-app-ios/assets/76485998/f237be3d-b671-4cfa-83c1-d8c9b461160d

@shafqat-muneer can you open full flow to this component?
for example, open from push: https://github.com/openedx/openedx-app-ios/assets/76485998/f237be3d-b671-4cfa-83c1-d8c9b461160d

@eyatsenkoperpetio This flow will execute from native calendar link which is demoed in this video. Do you have any scenario in which it can execute through push notification?

@shafqat-muneer can you open full flow to this component?
for example, open from push: https://github.com/openedx/openedx-app-ios/assets/76485998/f237be3d-b671-4cfa-83c1-d8c9b461160d

@eyatsenkoperpetio This flow will execute from native calendar link which is demoed in this video. Do you have any scenario in which it can execute through push notification?

@shafqat-muneer, I believe @eyatsenkoperpetio didn't mean to open that from a push notification, we probably need to open the full user path
MainView -> Course -> Course Sequential (Optional) -> xBlock
instead of
MainView -> xBlock
as @eyatsenkoperpetio demonstrated on video for deep links from pushes.
I think this is the right way too. If we give users the full path, they can just hit the back button to return to the course and keep learning with the next or previous section.

@volodymyr-chekyrta It appears I misunderstood. Thank you for providing clarification.
@eyatsenkoperpetio I'll adjust accordingly based on that.

@eyatsenkoperpetio I have completed the changes and have also included an updated demo video in the description of the PR. Please review it.


Contributor role has been removed

Necessary Information
Email address to add<EMAIL_ADDRESS>Committers, that could take over: @almadigabor @FaGru3n
GitHub username: dvasunin
I was a contributor already but my role was revoked just now. I've got email:
You’ve been removed from the "automotive-tractusx-contributors" team on the "Eclipse Tractus-X" organization.
Cheers & Octocats,
GitHub Support

Invite was sent again


.kv file encoding issue

My setup is Win7 x64 Ultimate with latest version of Kivy. I use
Notepad++ for python development.
Now Notepad++ by default is set to use UTF-8 encoding. When I save .kv
files in this encoding
parser fails. But when I encode it in ANSI there is no problem.
I know it is not a very big problem. But it annoys a little.
Is it by design?
Parser says:
'Invalid data after declaration'
kivy.lang.ParserError: File "", line 0:
to reproduce:
go to \Kivy-1.0.9-w32\kivy\examples\guide\designwithkv
replace controller.kv file with exactly same file, BUT with utf-8 encoding (with BOM)
now run main.py
voila :)
error!

Sorry to re-open an old issue, but I have a problem that is very similar to this one.
I'm developing a multi-platform app for Linux and Windows 7. All my files were first written on Linux and encoded as utf-8, but when I open this same project on Windows the kv file is read as if encoded with cp1252. The same thing does not seems to happen to my .py files maybe because I'm using python3.
As a consequence the Unicode characters written on the kv file won't render correctlly on the Kivy app. The string 'Título' will show as TÃ-tulo.
My settings are: Kivy=1.9.1, Python=3.4.4, Windows 7 x64 Home Premium.
To reproduce the problem:
Write a kv file encoded with Unicode:
# test.kv
<myButton@Button>:
    text: 'Título'

On python interpreter or .py script:
import kivy
from kivy.lang import Builder
from kivy.uix.button import Button

Builder.load_file('test.kv')
class myButton(Button):
    pass

print( myButton().text == 'Título' ) # False
print( myButton().text.encode('cp1252').decode() == 'Título' ) # True

The multi-platform workaround I found was this:
# test.kv
<myButton@Button>:
    text: str(b'T\xc3\xadtulo'.decode())

@VinGarcia This is a different problem. Please open a separate issue.

Ok, thank you.


I get error when loading content script: "Cannot find anchor"

I am using the default template see:
import './style.css';

export default defineContentScript({
  matches: ['<all_urls>'],
  // 2. Set cssInjectionMode
  cssInjectionMode: 'ui',

  async main(ctx) {
    // 3. Define your UI
    const ui = await createShadowRootUi(ctx, {
      name: 'example-ui',
      anchor: '#anchor',
      position: 'inline',
      onMount(container) {
        // Define how your UI will be mounted inside the container
        const app = document.createElement('p');
        app.textContent = 'Hello world!';
        container.append(app);
      },
    });

    // 4. Mount the UI
    ui.mount();
  },
});

But i get this error when I run:

I am expecting a shadow root to be created but its not.

Also I just checked the manifest, content_script is not populated, see:
{
  "manifest_version": 3,
  "name": "@scoutahead/extension-new",
  "description": "manifest.json description",
  "version": "0.0.0",
  "icons": {
    "16": "icon/16.png",
    "32": "icon/32.png",
    "48": "icon/48.png",
    "96": "icon/96.png",
    "128": "icon/128.png"
  },
  "commands": {
    "wxt:reload-extension": {
      "description": "Reload the extension during development",
      "suggested_key": {
        "default": "Alt+R"
      }
    }
  },
  "background": {
    "service_worker": "background.js"
  },
  "action": {
    "default_title": "Default Popup Title",
    "default_popup": "popup.html"
  },
  "host_permissions": [
    "<all_urls>",
    "http://localhost/*"
  ],
  "content_security_policy": {
    "extension_pages": "script-src 'self' 'wasm-unsafe-eval' http://localhost:3000; object-src 'self';"
  },
  "permissions": [
    "tabs",
    "scripting"
  ]
}

Duplicate of https://github.com/wxt-dev/wxt/issues/434#issuecomment-1935399083.
anchor: "#anchor" is a query selector used as an example. You need to fill it out with your own query selector, or remove the option if you just want to add it to the body.
As for the manifest not containing the content script, in dev mode they're registered dynamically at runtime so they can be reloaded on change. See https://github.com/wxt-dev/wxt/issues/434#issuecomment-1934830092 for more details.
Since this is the second time someone has had this problem, I'll update the example to use #your-anchor-query-selector instead of just #anchor.

I've updated the docs: see https://github.com/wxt-dev/wxt/commit/a0d16430219c9c90f8733f12229ebf665c64f005 and https://github.com/wxt-dev/wxt/commit/a0d16430219c9c90f8733f12229ebf665c64f005


MWPW-143262: offer preview removed as auto-block

As discussed in #milo-dev slack channel, offer preview block should not be listed as an auto-block. This PR removes it from utils.js.
Resolves: MWPW-143262
Test URLs:

Before: https://main--milo--adobecom.hlx.page/?martech=off
After: https://MWPW-143262--milo--adobecom.hlx.page/?martech=off

Nit: the provided test URL is a 404 and should rather point to https://mwpw-143262--milo--axelcureno.hlx.page/?martech=off instead.


BULK VALIDATION missing descriptions 7

Docs Build status updates of commit 5a320fc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Skype/UCMA/tasks-of-the-host-net-application.md
:white_check_mark:Succeeded

Skype/UCMA/telephony-call-controls.md
:white_check_mark:Succeeded

Skype/UCMA/terminating-a-conversation.md
:white_check_mark:Succeeded

Skype/UCMA/threading-model.md
:white_check_mark:Succeeded

Skype/UCMA/tonecontroller.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix build validation errors
@lindalu-MSFT
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you’d like to see. Thank you!

Docs Build status updates of commit c8e3c14:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Skype/UCMA/tasks-of-the-host-net-application.md
:white_check_mark:Succeeded

Skype/UCMA/telephony-call-controls.md
:white_check_mark:Succeeded

Skype/UCMA/terminating-a-conversation.md
:white_check_mark:Succeeded

Skype/UCMA/threading-model.md
:white_check_mark:Succeeded

Skype/UCMA/tonecontroller.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit da0761e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Skype/UCMA/tasks-of-the-host-net-application.md
:white_check_mark:Succeeded

Skype/UCMA/telephony-call-controls.md
:white_check_mark:Succeeded

Skype/UCMA/terminating-a-conversation.md
:white_check_mark:Succeeded

Skype/UCMA/threading-model.md
:white_check_mark:Succeeded

Skype/UCMA/tonecontroller.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Ensure SQLModel and SQLAlchemy Alembic migration parody with automated tests.

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
NA

Description
Add tests that implement the same model in SQLModel and SQLAlchemy, then runs an Alembic migration against an array of databases.  This is one way to find disparities in the underlying models generated by SQLModel.
Using Alembic to test the generated models deltas is a good real-world test. Moving from existing SQLAlchemy models to SQLModel's and running migrations is going to be a very common use case.
Wanted Solution
Create a set of isolated docker containers that run sets of Alembic migrations and see if there are any deltas.
Example workflow:

Spin up Postgres 13
Auto-generate an alembic migration based on the SQLAlchemy schema
Apply the Alembic migration to the DB
Auto-generate another alembic migration based on the SQLModel schema
Ensure that Alembic does not think there should be any migrations.

This could be done on a set of different DBs that SQLAlchemy supports.
Wanted Code
NA

Alternatives
Directly comparing the generated SQLAlchemy and SQLModel model's could be a solution, but it lacks the real-world use case and added complexity of using Alembic.
Operating System
Linux
Operating System Details
Docker
SQLModel Version
0.0.4
Python Version
3.9
Additional Context
I would be happy to set up these types of tests as this is my exact real-world use case.

parity != parody


Ability to expand/collapse the neighbors list

In Hierarchy Browser, if an edge list is present the connected neighbors show automatically and can't be closed. This panel should have a button to expand/collapse as desires.

Implemented in PR #33.

Implemented in PR #33.


NullPointerException when trying to change Account record type

When you try to change the record type of an account you get a NPE:

**lurch: add

Tracking W-018647

Included in beta release 1.35 (Beta 2)

Included in production release rel/1.35


docs: MenuItem Documentation

Closes #31

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
v1.0.0

Your semantic-release bot :package::rocket:


Feedback System

I have a feature request. After resolving and closing a support ticket, it would be advantageous to incorporate a feature prompting users to rate their experience. This feature entails the bot sending a post-ticket message to users, soliciting their rating on a scale from 1 to 5. To ensure user interaction and convenience, Discord's Select Menu feature can be utilized, allowing users to easily select their rating.
Subsequently, the bot should process and record the feedback, forwarding it to a specified chat room dedicated to feedback review and analysis.
I think this interactive rating system fosters user engagement and provides actionable insights for continual improvement of support services.
Thanks a lot!

Hello, this feature already exists in the bot and can be configured in the config.yml file. To enable it, you need to at least have either the delete DM embed or DM transcript enabled and the ratingSystem. The user will receive an embed with 5 options from 1 to 5 stars in a select menu then will be prompted with a discord modal to submit feedback after selecting the rating.
Once that is all done, the rating and feedback are sent to a channel of your choice based on your log channels configuration.