Pattern: Content Group Card - Accessibility - The heading and the body in each card is given as link.

Detailed description
Device - JAWS FF, VO iPhone Safari
Issue :- The heading and the content in each card is given as link. This makes it redundant.
Suggestion:-

If both the links in each card is expected to perform the same action, there is no need to provide different links. Either the heading or the content under the heading can be given as link.
It is not recommendable to given the entire textual content as a link. Because it makes the screen reader over verbose. The link text or the label should be concise.

Additional information

The solution for this should be the same as issue (#1482)

@oliviaflory Can you verify if this is a problem from an accessibility standpoint that has to be fixed.

@RobertaJHahn I tried to do some research on the accessibility site about repetitive links, but am having difficulty finding a recommendation.
cc @wonilsuhibm

This is a bad UX for users using screen readers at the minimum. This should be read once as a whole.

Assigned a sev 4. Can be fixed in the future release.

Also consider the mobile experience - are there two spots for taping on mobile with blank spots in-between that could incorrectly be attempted to be tapped with no results?
Why can't the whole card be the clickable / tapable region? Then the screen reader only hears the heading (link label) once.

Yes, we want the whole card to be clickable but only one link label so we have to investigate how to fix it.

@Praveen-sr
Could you please test this again?
We added an aria-label on the link,
Thanks!

@RaphaelAmadeu Ram re tested this on experimental and staging but Screen reader is not identifying the link in each card, all are announced only as text.

@Praveen-sr
Here's the link,
https://ibmdotcom-react.s3.us-south.cloud-object-storage.appdomain.cloud/deploy-previews/1877/index.html
Thanks!

@Praveen-sr
Do we have any updates on this?
Thanks!

@RaphaelAmadeu Yes, ram verified on the link you provided, but Screen reader is not identifying the link in each card, all are announced only as text.

@Praveen-sr
Another update was made,
Could you please test this once more?
Thanks!

@RaphaelAmadeu Ram verified this, issue still exists. Can we have a discussion on call and resolve it?
Thanks.

@RobertaJHahn
After investigating this with @kennylam we found out that this is an issue inherited from Carbon,
They are documented working on the Card used component Tile
It will be fixed, as soon as they update the component,
The same can be said for the issue #1482
Thank you!


default tabs

how can i set default tabs when i load or reload pages ?
now when i set tab conent 2 is active ,but when load pages is always show the tab content 1

Just add "active" class to required tab on page load. Currently active tab won't save by default if you close the page. You should do it manually, by saving it for example in local storage or somewhere else


Introduce Efficient Node Localization in Syntax Tree for Unit Tests

Description
Currently, there is no mechanism to efficiently refer to specific node in syntax tree for the need of unit test. For example to create Diagnostic.highlights and attach them to a specific node in a syntax tree within diagnostic formatters unit tests. The absence of such a feature makes it cumbersome to identify and highlight syntax nodes for diagnostic or verification purposes.
Proposed Solution by @ahoppen:
Introduce a NodeLocator struct to facilitate the process. Below is a proposed implementation idea:
/// Refers to the first node in a syntax tree after `marker` that satisfies `match`.
///
/// For example, if you have
/// ```swift
/// var x: Int
/// 1⃣func foo() {}
/// ```
///
/// Then `NodeLocator(marker: "1⃣")` refers to the `CodeBlockItemSyntax` that
/// contains the function declaration and
/// `NodeLocator(marker: "1⃣", matches: { $0.is(FunctionDeclSyntax.self) })`
/// refers to the function within.
struct NodeLocator {
  let marker: String
  let match: (Syntax) -> Bool = { _ in true }

  func findNode(in tree: Syntax) -> Syntax? {
    // TODO: Implement
  }
}

[!note]
Before starting implementation, let's discuss this approach here.

This feature could be a significant step towards enabling more precise and easier-to-write unit tests, especially those involving diagnostic highlights.

For the current needs, I've implemented unit tests to examine how the range of source code to highlight is calculated. The relevant tests can be found here:
https://github.com/apple/swift-syntax/blob/b6fcad4496cce446faa8e4f79b7558a33710a842/Tests/SwiftDiagnosticsTest/SyntaxHighlightRangeCalculatorTests.swift#L76-L90
https://github.com/apple/swift-syntax/blob/b6fcad4496cce446faa8e4f79b7558a33710a842/Tests/SwiftDiagnosticsTest/SyntaxHighlightRangeCalculatorTests.swift#L276-L296
And this approach is far from optimal.

Tracked in Apple’s issue tracker as rdar://116256883


Multiple processes metrics

Summary
This PR covers functionality requested in #2761 :

AllowMultipleProcessesMetrics configuration flag was added; it allows to return metrics from multiple processes via /metrics endpoint. By default its value is false and /metrics endpoint behaves as currently: when multiple processes match default process filters, no data is returned. When set to true, /metrics endpoint can return metrics of multiple processes, and:

all metrics returned by /metrics endpoint will contain additional labels: process_name and process_id. Those labels identify process which is the source of given metric
process_name label returns name of app pool for IIS processes (that is, when system process name is w3wp) and "regular" system process name for other processes

Release Notes Entry
Added option to track metrics of multiple processes via /metrics endpoint

@dotnet-policy-service agree

Thank you for your contribution. I put some initial feedback in the PR, but please hold off on additional changes until we figure out some of these design points:

Should we support multiple metrics on connect mode
Configuration options: allow the same level of configurability but across N processes or keep all options the same.
How do we extract a meaningful process name? Ideally we don't need specific understanding of w3wp or dotnet.

Thank you for your feedback. I'll wait for your decisions.

(...)

How do we extract a meaningful process name? Ideally we don't need specific understanding of w3wp or dotnet.

I suppose some extensibility for "process label factory" could be introduced, similarly to egress extensibility model.


add difficulty curve configuration system and associated Hitbloq Bot commands

add difficulty curve configuration system and associated Hitbloq Bot commands
(this refers to how difficulty ratings are used when determining CR rewards)

done ages ago.


Makefile

Hi, I would like to embed this into my system, instead of putting obj files to it in command line or GUI. Could you give instructions how to compile and run this code, perhaps using makefile?
-- Never mind, I didn't expect compiling it would be this simple that you don't even need a Makefile! Good work!

If anyone is interested: g++ -o simplify Main.cpp (inside src.cmd)


Should New-MgUserAuthenticationPhoneMethod be listed?

I've been looking for a method to populate the authentication method for users to use SSPR without their intervention. Whilst I am finding the Phone field will sync from on-premises to Azure it doesn't register the authentication method automatically. The user still needs to add the mobile number and verify.
I've since done testing with New-MgUserAuthenticationPhoneMethod with Microsoft Graph and have noted that this sets the authentication method and does not require end-user validation to register or review the number set. This looks to be a better approach without any intervention required. Is Microsoft Graph the way to go for doing this now rather than the suggested PowerShell commands?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: aae83fda-27b9-edca-1b59-1140de4cc3f5
Version Independent ID: c93eb84b-f04b-de03-f3f5-57d4672b0559
Content: Pre-populate contact information for self-service password reset - Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/authentication/howto-sspr-authenticationdata.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@MichaelRyan87
Thanks for your feedback! We will investigate and update as appropriate.

Hi @MichaelRyan87 , there's a note at the beginning of the article stating that the Microsoft Graph REST API (beta) for authentication methods is the current engineering focus to provide modern interaction.
My understanding is that phoneAuthenticationMethod is supported in the beta version but not in v 1.0 yet, and the beta APIs aren't recommended for production. That said, when it gets out of beta I agree that it should be listed.
Adding @Justinha for visibility and in case I missed something though https://docs.microsoft.com/en-us/graph/api/resources/authenticationmethods-overview?view=graph-rest-beta


dnanexus client support

plugged into the tester.py script support for dnanexus client

Thanks @AlexanderVi, looks good.


dev: improve latency of apibara indexing

Describe the enhancement request

Note: needs further scoping by code owners, @tcoratger or @greged93
Context: I understand that Apibara latency is ~3s due to an env variable for pending block indexing
Describe Preferred Solution
We should aim to make this as low as possible to improve UX.

this has already been changed here
https://github.com/kkrt-labs/kakarot-rpc/commit/e71a0b0328081649d3f47f391875dea4db9810c2


Changed readSnapshot  to retrieve s3 bucket region prior to retrieval

Created constructor for readSnapshot to retrieve s3 bucket region to stop PermanentRedirect: The bucket you are attempting to access must be addressed using the specified endpoint. error when accessing config bucket from another region.
Because the AWS Config bucket can be configured as a bucket in an external account it's possible that the home region for CloudTrail and the AWS Config bucket region may not match. Therefore I've added a constructor which finds the config bucket's S3 region in order to allow the getObject call to address the correct endpoint.
I appreciate this PR is a little larger than my previous ones, but I welcome any feedback you have on alternative approaches!
I confirm these files are made available under CC0 1.0 Universal (https://creativecommons.org/publicdomain/zero/1.0/legalcode)

Hi @Sam-Martin, any thoughts on the question above?

Hi @aaronkhoo, sorry, didn't get a chance to look at this yesterday!
My original intention was to try and split out the readSnapshot function to be a bit more readable while still keeping it as the same object (as the bit I was splitting out wasn't terribly re-usable).
However, on further reflection it's a pretty overwrought solution and actually the same thing can be achieved with the latest commit I've just added.
JavaScript isn't my strongest language so I'm more than happy to defer to your recommendation on the best way to implement this.

The new code looks fine. Merging.


Interface item examines don't appear

As in title. When the "Show interface item examines" box is checked, examining an item in the inventory won't show the examine text in a infobox. It will only appear in the chat window

I'll look into it, I think I saw something about the ops for examines in the inventory changing recently. I don't know when I'll have time to push a fix though. Thanks for raising the issue.

I did a quick check, looks like inventory examines now work the same way as other interfaces, so taking out the specific logic for that and bringing it back to the other type seems to fix the issue. If you are able to build hub plugins, you may test the PR I linked to this issue.
I don't have time to 100% test this yet but I'll try to PR this to the hub before the end of the week.


Remove support?

What about nodes remove support? I understand that with a current architecture it's not really possible because it will invalidate NodeId, but what if I really-really have to remove a node?
UPD: I just noticed that you rewrote the whole thing...

What does "remove" mean that detach doesn't cover? Dropping items could be done with Tree<Option<T>> or similar, but reclaiming storage in the Vec invalidates the entire tree.

Do'h. I missed it.
But what about values() iter? It will return detached nodes anyway (it doesn't matter for my case, but still).

but reclaiming storage in the Vec invalidates the entire tree

You can store a list of free "nodes" so a new node will take a free space and will not expand the vector.

Yeah Values and Nodes iterators can't really do anything about the structure of the tree.
Maintaining a free list really sounds like it would complicate the structure to the point where it would make sense to use a different tree representation.

I see. Then I'm closing it for now. I thought maybe this is a planned feature.


Remove "standalone services" docs

Now they are separate projects

@LavyshAlexander can you please move these pages to the according readme files in metadata/mempool repos?


Function createGlobalStyle don't work in esm and cjs bundles by wrong SERVER global variables in build config

Environment
Any clients side environment
System:

OS: Windows 10 10.0.22000
CPU: (16) x64 AMD Ryzen 7 5800X 8-Core Processor
Memory: 12.25 GB / 31.91 GB

Binaries:

Node: 17.3.1 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.17 - ~\AppData\Roaming\npm\yarn.CMD
npm: 8.3.0 - C:\Program Files\nodejs\npm.CMD

npmPackages:

styled-components: =5.3.3 => 5.3.3

Reproduction
Import node_modules\styled-components\dist\styled-components.cjs.js or node_modules\styled-components\dist\styled-components.cjs.js to client side code by any bundler.
Steps to reproduce
import {createGlobalStyle} from "styled-components";
import styles from "./../../../../node_modules/swiper/swiper.min.css";

export const CssGlobalSwiper = createGlobalStyle`
    ${styles}
`;

Expected Behavior
Insert global styles by bundles:
node_modules\styled-components\dist\styled-components.esm.js - work
node_modules\styled-components\dist\styled-components.cjs.js - work
node_modules\styled-components\dist\styled-components.js - work
Actual Behavior
Insert global styles by bundles:
node_modules\styled-components\dist\styled-components.esm.js - don't work
node_modules\styled-components\dist\styled-components.cjs.js - don't work
node_modules\styled-components\dist\styled-components.js - work
Where problem
In this line
https://github.com/styled-components/styled-components/blob/main/packages/styled-components/src/constructors/createGlobalStyle.ts#L61
used variable __SERVER__. In rollup config this variable set to false for js bundle, and true for cjs and esm bundles. But its totaly wrong, because modern bundler can use cjs and esm for import script. CJS and ESM its not a version for nodejs, its versions for modern runtimes, also included a browsers.
How fix it
Set variable __SERVER__ in runtime, not a build time.
Small improvement
In 2022 yeahr better way will dont make bundle, just transpile ts files to target es2015 and esnext modules.
Rellated issue
https://github.com/styled-components/styled-components/issues/2670

I can also make a pull request that will fix this error. If you wish.

@LabEG, are you using any workaround for this in the meanwhile?
I've been using:
import { createGlobalStyle } from 'styled-components/dist/styled-components';

It is fine, but I'm curious if there is something neater.

At first, I did the same.
But then I made script in postinstall that replace entry point for module in package.json of styled components.

Can confirm that createGlobalStyle isn't working in client for<EMAIL_ADDRESS>

I'm running into this same issue. Compiled with Vite, everything works fine in serve mode, but under a production build I get an error in the browser console:
Uncaught TypeError: ot is not a function
    at Qe (styled-components.browser.esm.js:1:15599)
    at vi (styled-components.browser.esm.js:1:22570)
    at GlobalStyles.js:8:22

This is with Vite 4.2.1, Node 16.18.0, tried with styled-components v5.3.0, 5.3.3, and 5.3.10.

On my end it turned out to be a misconfiguration, causing the server export of styled components to be loaded in the client.

@Xiphe can you be a little more specific about what you misconfigured?

It was highly specific to my app, so I doubt it will be helpful. But nonetheless:
I had issues where esbuild (for some reason I don't remember) was not able to use the cjs build of  styled-components. So I was forcing it to use the esm build (by setting up an typescript alias, [might also happen with any other alias]). At that time I was not aware that styled components uses different runtimes on server and client so I accidentally forced all import styled from 'styled-components' to use the esm server bundle.
Currently I'm monkey-patching the package.json of styled-components to expose it's esm build for the server as "main" - this way client builds still go with the "browser" field.
Ideally this would be solved by fine-tuning the way the bundler imports this specific package instead of monkey patching, but in remix we don't have such granular settings.

Adding my use case here, I am working on a project that streams HTML rendering for Node.js CLI application, and expect styled-components to work with JSDOM as close as how it works in a browser.
And this __SERVER__ hard-coded by compiler is kind of a blocker here. Also, there's a similar issue with IS_BROWSER. Though not hard-coded, but requires import() expression to delay the evaluation for ESM.
https://github.com/vilicvane/backpage/blob/ec0ec49f25b91d3b6e92f3d82a935342a0993145/src/examples/styled-components.tsx#L6-L15
I don't see notable benefit doing so instead of lazy evaluation.


hessian protocol not support？

How to integrate Hession protocols in my projects？

ps please update this issue while you progress!

but actually this issue should be in brave, not zipkin

@lukylent can you please open an issue in Brave instead? https://github.com/openzipkin/brave/issues


Error eagerly evaluating toplevel property

The following fails
shared void run(){
    print("done.");
}

interface Foo { }
Foo noFoo = nothing;

with
$ ceylonDev compile-js && ceylonDev run-js com.example.simple
Note: Created module com.example.simple/1.0.0

/Users/jvasileff/Dropbox/Repos/ceylon/ceylon-dist/dist/repo/ceylon/language/1.1.1/ceylon.language-1.1.1.js:265
if (!cond)throw wrapexc(AssertionError(msg),loc,file);
                ^
ceylon.language::AssertionError "nothing may not be evaluated: 'false' at nothing.ceylon (11:11-11:17)"
    at AssertionError (/Users/jvasileff/Dropbox/Repos/ceylon/ceylon-dist/dist/repo/ceylon/language/1.1.1/ceylon.language-1.1.1.js:2227:6)
    at asrt$ (/Users/jvasileff/Dropbox/Repos/ceylon/ceylon-dist/dist/repo/ceylon/language/1.1.1/ceylon.language-1.1.1.js:265:25)
    at Object.getNothing (/Users/jvasileff/Dropbox/Repos/ceylon/ceylon-dist/dist/repo/ceylon/language/1.1.1/ceylon.language-1.1.1.js:3210:791)
    at $valinit$$2 (/Users/jvasileff/Desktop/js/simple/modules/com/example/simple/1.0.0/com.example.simple-1.0.0.js:27:40)
    at /Users/jvasileff/Desktop/js/simple/modules/com/example/simple/1.0.0/com.example.simple-1.0.0.js:27:66
    at /Users/jvasileff/Desktop/js/simple/modules/com/example/simple/1.0.0/com.example.simple-1.0.0.js:34:37
    at /Users/jvasileff/Desktop/js/simple/modules/com/example/simple/1.0.0/com.example.simple-1.0.0.js:1:83
    at Object.<anonymous> (/Users/jvasileff/Desktop/js/simple/modules/com/example/simple/1.0.0/com.example.simple-1.0.0.js:33:2)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)

Based on my reading of the spec, toplevel properties should be lazily initialized:

8.2.10. Initialization of toplevel references

A toplevel reference has no associated frame. Instead, the lifecycle of its persistent value is associated with the loading and unloading of a module by the module runtime. The first time a toplevel reference is accessed following the loading of its containing module, its initializer expression is evaluated, and the resulting value is associated with the reference. This association survives until the toplevel reference is reassigned, or until the module is unloaded by the module runtime.

Initialization of a toplevel reference may result in recursive initialization of other toplevel references. Therefore, it is possible that a cycle could occur where evaluation of a toplevel reference occurs while evaluating its initializer expression. When this occurs, an InitializationError is thrown.

Hrm. I would not be averse to weakening this language if @chochos thinks it would be a pain or otherwise bad to implement that behavior in js.

I've been trying to track down a bug for a while, and I lost track if this issue was a contributing factor, but there may be related problems. For instance, is it always ok to eagerly evaluate? Or is it possible that some methods or other properties may not yet be defined?
Also - nothing is convenient as a placeholder. So if this behavior changes, I think ideally it should fail on Java as well, to avoid surprises.
Finally, IIRC, these module-load-time problems are very hard to diagnose if the error occurs during the loading of a dependent module. (At least in some cases, the error doesn't provide useful information, just something like:
ceylon.language::AssertionError "Assertion failed: 'exists m = modules.find(moduleNameAndVersion[0], moduleNameAndVersion[1])' at tool.ceylon (90:15-90:89)"

But again, I don't recall all of the circumstances for that particular error.

Well Foo noFoo => nothing is the lazy version, right? I hadn't realized Foo noFoo=nothing should also be lazy.

My understanding is Foo noFoo => nothing should evaluate every time, whereas Foo noFoo=nothing initializes/evaluates once, either at module load time (currently) or the first time it is needed (as in JVM's impl.)

Well Foo noFoo => nothing is the lazy version, right? I hadn't realized Foo noFoo=nothing should also be lazy.

Well the second is "lazy", but it's also memoized.

yes it's fixed now (it was already memoized, but was being initialized eagerly)

OK, great :)


README.md changes

Here we'll keep track of the commits related to the README.md.

Paulo made README.md Prolog backend agnostic: 0d0709cf6de26fd824ab516d8bb4a70294d9d879.

The README.md will be renamed to NOTES.md to go align with the Logtalk project (see the intervals library for example).


Allow node title to be shown for public node logs [OSF-7192]

Purpose
Accessing public node logs as anonymous user via the API prompted a permissions check when getting the title, which would fail!
Changes

Display the node title if the node is public
Two unrelated changes stumbled upon:

fix rendering in node log docstring
remove expect_errors from a test expecting a 200 success

Side effects
None anticipated
Ticket
https://openscience.atlassian.net/browse/OSF-7192

Is now fixed, no longer needed!


Publish message is sent to all in queue group

When publishing a message, every consumer in a queue group receives the message. Only one should receive and create response.

It seems that queue is not defined for messages.


Point tesseract-ocr and libtiff-ios submodules to most recent stable releases and update libs

I've been trying to update Tesseract and have faced an issue.
Neither of the tags from the upstream tesseract can be compiled. There is a broken commit ec9581d8f2b3895e7307609043a3ec1bc9c603c5 right before the tag 3.04.00, which doesn't allow to build Tesseract. And that commit has been reverted at 66a76a94772343582a70783a712f6248a81879ab. So all the latest tags are between those commits and cannot be compiled.
So there are 2 ways available to fix this.

Wait for the next stable upstream Tesseract release marked with the next tag.
Update to any other commit before the broken one or after the reverted one. But there are no more tags. So it seems that those history points could be unstable.
So I suggest the first way and won't update the tesseract so far.

I agree, I think 1 is the best option. Also I'm sorry I haven't reviewed your PRs yet; things are really busy at work but I'll try to take a look at them this weekend.

There is new version of Tesseract 3.04.01 have released. I'm trying to compile it but stuck with following output from make
cd /Users/podkovyrin/Projects/Tesseract-OCR-iOS/TesseractOCR/libtiff-ios ; \
	until `/Applications/Xcode.app/Contents/Developer/usr/bin/make -s`; do sleep 5; done
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:04 --:--:--     0
curl: (78) RETR response: 550
/bin/sh: ../configure: No such file or directory
make[1]: *** [/Users/podkovyrin/Projects/Tesseract-OCR-iOS/TesseractOCR/libtiff-ios/libpng-1.6.20/arm-apple-darwin7/Makefile] Error 127
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:--  0:00:02 --:--:--     0

The wrong path to libpng
Ther is no files at this path anymore. You should manually fix the path to new files stored at libpng repo.

@ws233 it worked, thank you

@podkovyrin, would you mind uploading a PR with your changes?

Sure, as soon as corresponding PR will be merged https://github.com/ashtons/libtiff-ios/pull/13

ashtons merged commit 71ad425 into ashtons:master on Feb 25


Update Kafka DL script to 2.13-3.8.0

Description

Fixes Kafka download script to use an available Kafka version.
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

/merge


cpp_assert_signature does not work correctly with lists

cpp_assert_signature does not work correctly with lists.
Let's say you have a function:
function(my_fxn arg0 arg1 arg2)
    cpp_assert_signature("${ARGV}" bool int str)
endfunction()

and you call it like:
my_fxn(TRUE 1 "foo;bar")

In debug mode, you will get an error: "Function takes 3 argument(s), but 4 was/were provided". The problem is cpp_assert_signature sees that it should get three arguments, but it then sees that ARGV is a 4 element list. Without manually passing ARGV0, ARGV1, ...to maintain the partitioning, I'm not sure how to avoid this at the moment.

I think I found a possible solution using macros. It turns out, macros can loop over the argument lists of the functions in which they are called in if you use a very specific syntax, while still allowing to loop over their own arguments. The documentation on this behavior is here:
https://cmake.org/cmake/help/latest/command/macro.html#argument-caveats
Example:
macro(test_macro)
    # Hacky workaround to prevent the macro's ARGC from being inputted,
    # since only the literal ${ARGC} is replaced with the macro's count
    set(_tm_c "C")

    # CMake is weird and includes the stop, so we have to subtract one
    math(EXPR _tm_arg_count "${ARG${_tm_c}} - 1")

    foreach(arg RANGE ${_tm_arg_count})
        # Only exact ${ARGV0} etc is replaced with macro arguments
        # so this expression is left alone and evaluated according to the
        # calling function arguments
        message("Argument: ${ARGV${arg}}")
    endforeach()

    set(macro_args "${ARGV}")
    foreach(arg IN LISTS macro_args)
        message("Macro args: ${arg}")
    endforeach()

endmacro()

function(test_func)
    test_macro(1 2 "Hello")

endfunction()

set(l element1 element2)
test_func("${l}" "func1" "func2" 3 4)

Prints:
Argument: element1element2
Argument: func1
Argument: func2
Argument: 3
Argument: 4
Macro args: 1
Macro args: 2
Macro args: Hello


fix cat for UnivariateFiniteArray

#337

Codecov Report

Merging #339 into dev will decrease coverage by 0.55%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #339      +/-   ##
==========================================
- Coverage   82.56%   82.00%   -0.56%
==========================================
  Files          31       31
  Lines        2237     2212      -25
==========================================
- Hits         1847     1814      -33
- Misses        390      398       +8

Impacted Files
Coverage Δ

src/univariate_finite/arrays.jl
100.00% <100.00%> (ø)

src/univariate_finite/methods.jl
82.56% <0.00%> (-5.62%)
:arrow_down:

src/measures/finite.jl
89.34% <0.00%> (-2.96%)
:arrow_down:

src/data/datasets.jl
86.11% <0.00%> (-2.78%)
:arrow_down:

src/composition/networks.jl
65.56% <0.00%> (-1.32%)
:arrow_down:

src/hyperparam/one_dimensional_range_methods.jl
91.86% <0.00%> (-0.37%)
:arrow_down:

src/univariate_finite/types.jl
90.72% <0.00%> (-0.19%)
:arrow_down:

src/composition/composites.jl
90.67% <0.00%> (-0.15%)
:arrow_down:

src/resampling.jl
88.88% <0.00%> (+0.93%)
:arrow_up:

src/composition/arrows.jl
27.27% <0.00%> (+2.27%)
:arrow_up:

Continue to review full report at Codecov.

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


Task log timings are inverted

In JkLog, #startTask and #endTask act on opposite ends of the START_TIMES LinkedList:
#startTask pushes the current time into the list, meaning that it is the first element. #endTask however polls last, which removes the final element of the list, meaning the very first time startTask was called. This results in sub tasks claiming to take longer than the entire project starting up does:
Initializing class com.chocohead.loom.LoomCommands...
│ Instance decorated with plugin class com.chocohead.loom.JkPluginLoom[runOffline = false]
│ Fetching manifests ...
│ │ <work>
│ └ Done in 1027 milliseconds.
│ Instance decorated with plugin class dev.jeka.core.tool.builtins.scaffold.JkPluginScaffold[]
│ Instance decorated with plugin class dev.jeka.core.tool.builtins.eclipse.JkPluginEclipse[jreContainer = null, useVarPath = true]
│ Run instance initialized with options []
└ Done in 562 milliseconds.

Either they'll both want to be switched to push and pop to add subtasks to the start of the list, or addLast/offerLast and removeLast so the two operations are consistent.
I'd also say it would be nice for JkLog#endTask(String) to expose the time taken in some way so it can be logged as part of the message rather than it having to be calculated by the caller. Potentially using String#format if message contains %d, or changing the method to take a DoubleFunction<String>. That being said, I haven't actually needed to use it myself yet so feel free to ignore the suggestion ;)

Great. I like the %d idea . I will introduce it in this fix.


chore: add node version file

Codecov Report

Merging #143 (0767951) into main (d39ace7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #143   +/-   ##
=======================================
  Coverage   89.85%   89.85%
=======================================
  Files          76       76
  Lines        2475     2475
  Branches      308      308
=======================================
  Hits         2224     2224
  Misses        181      181
  Partials       70       70

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Portuguese version

Do you think this is good to merge?

I untracked .gitsubmodules because I am creating the Portuguese content to divinum-officium and my .gitsubmodule is going to https://github.com/ofrades/divinum-officium.git instead of https://github.com/DivinumOfficium/divinum-officium.git.
I think that was the only change I did to your repo.

Thank for the PR. I'll review it in the beginning of the week.

Hi, I quickly glanced through your PR. There's a lot of good work, but so far I have found couple of problems that need to be fixed before it can be merged.

I don't get why have you untracked .gitsubmodules. It's essential and it should point to https://github.com/DivinumOfficium/divinum-officium.git.

There's plenty of propers that are not translated or partially translated. Couple of examples:

/pt/2020-03-21 Lectio mismatch between PT and LA
/pt/2020-04-04 Graduale is in Polish
/pt/2020-04-05 doesn't work
/pt/2020-04-06 all sections except Prefatio and Oration super populum are in Eng
/pt/2020-04-08 wrong order of sections; Introit on third position
/pt/2020-04-09 all sections are in Latin
/pt/2020-04-10 only one section is being displayed
/pt/2020-04-11 most sections are in Latin
/pt/2020-04-12 all sections are in English
/pt/2020-04-13 sections in Latin and English, encoding issues, e.g. Confit�mini
/pt/2020-04-29 Polish comment and some of the sections
/pt/2020-05-15 Polish comment and some of the sections

There's 127 tests failing in tests/test_propers.py and 2 tests failing in tests/test_apiv3.py. Probably related to the version of linked DivinumOfficum

Ordo, Supplement and Info are not translated at all

On a different note, I have added an alternative way to handle complex Masses, such as Palm Sunday or Ash Wednesday. You can create a pre-generated json file to have more control on how the proper is displayed (smilar to Ordo). You might consider using it instead of regular way. Palm Sunday example:
https://www.missalemeum.com/pl/2020-04-05
https://github.com/mmolenda/Missal1962/blob/master/missal1962/static/data/pl/tempora:Quad6-0r:1:rv.json

I am doing the divinum officium portuguese version from scratch. There is no portuguese translation online, so what I  doing is to create the content step by step, like every week or so. Only in September I will have completed all the propers. And that is why my gitsumodules are pointing to my forked version of divinum officium. Makes sense?

I am the author of Polish translation of Divinum Officium Mass part so I understand your approach, though I recommend pushing any divoff work to the main branch as soon as it's done to make it available for the users right off. And this way have one source of truth from missalemeum perspective.

I will do that and do a PR after is done.
Also I am not sure about the pre-55 holy week, I tried some things but I am not sure of the approach.

You can handle Holy Week using approach I described in the comment https://github.com/mmolenda/Missal1962/pull/68#issuecomment-602172257


Linux x64 OpenVR - throws NullPointerException

Environment

LWJGL version: 3.2.0
LWJGL build #: 12
Java version: 1.8.0_151
Platform: Linux 4.4.0-134-generic x86_64
Module: openvr

Description
Issue:
Calls to either:

VRSystem_GetStringTrackedDeviceProperty
VRSystem_GetRecommendedRenderTargetSize

both throw java.lang.NullPointerException.
I suspect that all calls to OpenVR may also fail, as I've closely scrutinized my parameters and can't find fault.
FYI: I've gotten the C++ hello world working by running the following command:
~/.steam/steam/ubuntu12_32/steam-runtime/run.sh ./cpphellovr
So I know that OpenVR, my headset, and my runtime are working.
The issue seems to be somewhere in the LWJGL-OpenVR chain.
Steps to reproduce:

Download gist (Kotlin)
Build JAR (I use IntelliJ's "Build Artifacts")
~/.steam/steam/ubuntu12_32/steam-runtime/run.sh java -jar out/artifacts/kotlinviz_jar/kotlinviz.jar

Thank you very much for the library!

Hey @DavidYKay,
I cannot test your code without a HMD. Could you please post the NPE stacktrace? Also, try printing the value of OpenVR.VRSystem.GetStringTrackedDeviceProperty & OpenVR.VRSystem.GetRecommendedRenderTargetSize, they should not be 0. If they are, it means that LWJGL failed to retrieve function pointers from your OpenVR runtime. Which version do you have installed? Another thing you could try is running the program with -Dorg.lwjgl.util.Debug=true -Dorg.lwjgl.util.DebugLoader=true and seeing if there's any useful output.
One thing that's definitely wrong with your code is this line: return String(stringBuffer.array()). LWJGL buffers are never backed by a Java array on the heap, they are always direct. Try returning MemoryUtil.memUTF8(stringBuffer, unRequiredBufferLen - 1) instead.

First, thank you SO much for the prompt reply. I am hugely in your debt. :)
Second, it looks like it was user error. I followed your instructions and found that VRSystem wasn't being initialized properly. Because I was missing a call to it. I am so sorry for sending you on a goose chase!
Here's a working Hello World for any future OpenVR devs: https://gist.github.com/DavidYKay/639760f4f3be8ca77658a076fdf55916
Thanks again!


Make sure imageRenderer evaluates to an array for further processing

It might happen that $this->runtime->evaluate($this->path . '/' . $fusionPath, $this) evaluates to an empty string and thus can not be merged with array_merge.
We have to make sure to get an array out of this evaluation.

https://github.com/networkteam/Networkteam.Neos.ContentApi/blob/4bc0f802173ebe7d9fe48365bba6a401f59dd1df/Classes/Fusion/PropertiesImplementation.php#L95C22-L95C22

Ah this only happens, if Networkteam.Neos.Util:ImageUriAndDimensions does not evaluate correctly and throws an error. Thus, this might not be an issue normaly.


Update struct.md

Readonly static members are possible. I may have not exhausted the list of members excluded from being applicable to both readonly and static (those that can modify the structure instance), so further edits may be necessary.
Summary
Fixes unclear and misleading information regarding the usage of static readonly members.

Hi 👋 @chessboards
That statement on disallowing readonly should stay. It's meant to cover this example:
struct S
{
    // CS 8657: Can't apply "readonly" to a static method:
    public static readonly void M()
    {

    }
}

How did you interpret that statement? Would a different wording make that more clear?

I think the wording could be changed to make the distinction clearer between static members that should not interact with the structure instance or state (methods, events, props with setters), and those that have no state changing logic (therefore can be made readonly). The wording of members led me to include static fields and get-only properties as also being excluded, which they are not. https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/members

thanks again @chessboards
I've set this to auto merge as soon as the builds finish.  You should see it live tomorrow, on our regular publishing cycle.
Thanks again!


fix: Only attempt to parse the AppConfig response when it is not empty

Description
Based on the API documentation, the call to AppConfig GetLatestConfigurationAsync will return an empty response when the configuration has not changed.  This PR fixes a thrown Json parsing exception due to that empty response.
Motivation and Context
The periodic refresh was frequently throwing a JsonReaderException because the empty response could not be parsed as valid json.
Testing
Tested existing functionality by running unit and integration tests.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I have read the README document
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

License

[x] I confirm that this pull request can be released under the Apache 2 license

I remember there was a bug, when you were using lambda layer for AppConfig and the config was bigger, then the ContentLength we 0 even the response was there.
Let's check, but maybe it's not a case anymore because it was in the AppConfig Lambda Layer v1.x and now we have v2.x.

I remember there was a bug, when you were using lambda layer for AppConfig and the config was bigger, then the ContentLength we 0 even the response was there.
Let's check, but maybe it's not a case anymore because it was in the AppConfig Lambda Layer v1.x and now we have v2.x.

I believe the code to get the data when using a Lambda extension is in a different method. So the change in this PR wouldn't be affected by that, right?

I remember there was a bug, when you were using lambda layer for AppConfig and the config was bigger, then the ContentLength we 0 even the response was there.
Let's check, but maybe it's not a case anymore because it was in the AppConfig Lambda Layer v1.x and now we have v2.x.

I believe the code to get the data when using a Lambda extension is in a different method. So the change in this PR wouldn't be affected by that, right?

No, it's the same code. Let's just check if it works with AWS Lambda Extension.

I remember there was a bug, when you were using lambda layer for AppConfig and the config was bigger, then the ContentLength we 0 even the response was there.
Let's check, but maybe it's not a case anymore because it was in the AppConfig Lambda Layer v1.x and now we have v2.x.

I believe the code to get the data when using a Lambda extension is in a different method. So the change in this PR wouldn't be affected by that, right?

No, it's the same code. Let's just check if it works with AWS Lambda Extension.

@mgorski-mg The code I changed is inside GetDataFromServiceAsync().  The code for LambdaExtension is in GetDataFromLambdaExtensionAsync().  The code paths to reach those methods are from GetDataAsync() which picks one or the other (see below).  So I don't think LambdaExtensions will be affected.  But we could test it anyway.  Is that testing something you could help with?
https://github.com/aws/aws-dotnet-extensions-configuration/blob/b02f43cfc51b868eb5b96af23c55425cf05ba13e/src/Amazon.Extensions.Configuration.SystemsManager/AppConfig/AppConfigProcessor.cs#L78-L89

@tylerohlsen right! 🤦♂ then there was no topic, sorry 😊

Thanks for the PR. I'll work on getting the fix released and post back when the fix is out.

Version 4.0.1 is out with this PR. Thanks again!


[Question] Talks archive

I was wondering whether there is an option to archive/reset talks from a previous year before opening CFP for the new year so that they are not visible anymore? Is there a more elegant/meaningful way other than wiping out talks data?
Or to formulate it differently: what's the recommended workflow for reusing CFP between conferences?

@nikolaposa I think the usual process is to simply delete everything from the talks table but keep the user accounts in place.
If I were to implement this in code I would add a column to the talks table that indicates a talk is archived or not and then allow the archiving to occur via the bin/opencfp CLI tool.

If I were to implement this in code I would add a column to the talks table that indicates a talk is archived or not and then allow the archiving to occur via the bin/opencfp CLI tool.
@chartjes That could be a useful feature.

Feel free to close this issue, I got the input I needed. Thanks!

I am not a conference organizer, but I think this could be a good feature in the context of evaluating submissions, for example

do we want to hear this speaker again talking about a very similar topic
statistics

Europeans may want to worry about the GDPR though.

Europeans may want to worry about the GDPR though.

And to my understanding, "flagging as archived" would not be compliant. I think it'd require a full deletion of all requested user data. That wouldn't be that bad honestly, especially now that we've shored up better interfaces for AccountManagement.
I recently had to touch GPDR with an Event Store implementation I maintain. Fun stuff. Reasonably easy to comply with (on the surface, at least).


Client Type "docker-v2" doesn't exist

I have put the .jar file into the /opt/keycloak/providers/ directory. However, I can't find any "docker-v2" option there. How am I able to find that "docker-v2" option?

Thank you

Thank you @alexanderwolz for replying.
I have solved the problem. It turns out we need KC_FEATURES=docker,token-exchange as an environment variable. Your example really helps me a lot. However, I suggest that you put this environment variable in the setup section to make someone like me avoid this issue.
By the way, I am using Keycloak 21.0.1 in this case.
Cheers

Hi @aulia-adil
Thanks for the feedback, I have updated the Readme accordingly.
I'm happy that my work helps you.
BTW: I had issues regarding custom themes with 21.0, so I suggest you update to 21.1+ if you try to introduce your own design.


Get information_schema for pulsar connector to work

Motivation
Currently the information_schema that presto provides by default is not working for the Pulsar Presto connector

rerun java8 tests


No stamina to spellpower for demonology warlocks

Describe the bug
Using the "Calculate stat weights" as a demonology warlock gives a weight of 1 stamina = 0.00 +- 0.00 DPS EP, however due to the talent "Demonic Knowledge" 1 stamina ~ 0.14 spellpower.
The lack of value for stamina in the weights calculation seems to imply that the conversion of stamina to caster spellpower is completely skipped in the simulation, or that there is a simple bug in the weights output.
To Reproduce
Go to warlock simulation and choose any demonology spec and simulate weights
Expected behavior
1 stamina ~ 0.14 spellpower.
Sim Links and Screenshots
https://wowsims.github.io/wotlk/warlock/
Desktop
Browser chrome

Found the issue, we were applying extra SP from pet stm but it wasn't dynamically updating and so the 'bonus stats' that the EP system uses wasn't added to stm before the SP bonus was added.
I am working on a fix now.

https://github.com/wowsims/wotlk/pull/1983


lxd/apparmor: Workaround socket handling

It appears that on some kernel at least, the connect operation is
evaluated against the non-dereference path when dealing with a symlink.
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

jenkins: test this please


stream the whole topology in one json message

we should treat the streaming as snapshot, in which we send one message that contains all streamed API's
it should look like this
“metadata” : {
"id" : 15,
"timestamp" : “1970-01-01T00:00:01Z”,
“type” : “full”
},
"links" :      {
“links_list” : []
},
"devices" : {},
"ports" : {
“ports_list” : []
},
"cables" : {},
"alarms" : {}

maybe also should contain ufm name as part of json.
the receiver will get such data from multiple ufm servers and will need way to distinguish

maybe also should contain ufm name as part of json.
the receiver will get such data from multiple ufm servers and will need way to distinguish

FluentD accept additional parameter called tag in which we could put source of the data instead of adding it in the json itself

Done


GPT4 augment prompt

I found the test prompt in cole_designerintention_v1_samples had gpt4 augmented prompt.
How to augment the original prompt? Could you give the gpt4 prompt?

Hi, the augmented prompt is provided by the original COLE. I checked the paper but could not find the exact prompt fed to GPT4. Please ask it to the original COLE authors, and it would really appreciate it if you also share it with me;

Feel free to open if you want to discuss more!


Upgrade eslint & commander (out of current ranges)

The only scary changelog here is commander, which has jumped from v2 to v4 without a v3 and without much in the way of migration information.
However, since we only use it in lib/cli.js, and npm start -- --help and npm start -- --config=config.json both work after this change, it sure looks like we're fine to update this.
eslint is behaving fine – doesn't look like any of it affects us.

Coverage remained the same at 97.126% when pulling 1836f1960704cd845bded868bdcc20c7734b8dec on Throne3d:upgrade/deps-2019-12-14 into 33d12d7b3841bd8935a29a428af15d3a3158ce82 on reactiflux:master.


adding timeout to alert.post and http_post

Adding timeout to alert.post and http_post nodes.  Currently
http.DefaultClient is used, which blocks indefinitely when the endpoint
accepts the connection but does not respond to the HTTP request.
Switching to using a non-default http.Client with a timeout defined per
node.
Addresses #1461
Required for all non-trivial PRs

[x] Rebased/mergable
[x] Tests pass
[x] CHANGELOG.md updated

@sputnik13 This looks good, I see one issue. The Update function does not propagate a new Timeout value to existing handlers since the time out is set only when the handler is created. One possible change is to use context.WithTimeout on the request.WithContext method to apply a timeout per request so that client need not be destroyed and recreated with changes to the handler config.

@desa Could you also review this PR?

On it.

@nathanielc got it, I'll take a look and push up an update

@nathanielc @desa sorry for the delay, seems like I had this just sitting on my computer, is this what you had in mind?

it's passing tests for me locally, does the ci gate run a different set of tests than test.sh?

it should be the same suite of tests. Looks like the following tests are failing in CI--- FAIL: TestBatch_AlertStateChangesOnly (0.03s)--- FAIL: TestBatch_AlertStateChangesOnlyExpired (0.02s)--- FAIL: TestStream_HttpPost (0.01s)--- FAIL: TestStream_HttpPostEndpoint (0.01s)--- FAIL: TestStream_Alert (0.02s)--- FAIL: TestStream_Alert_NoRecoveries (0.05s)--- FAIL: TestStream_Alert_WithReset_0 (0.05s)--- FAIL: TestStream_Alert_WithReset_1 (0.02s)--- FAIL: TestStream_AlertDuration (0.03s)--- FAIL: TestStream_AlertHTTPPost (0.02s)--- FAIL: TestStream_AlertHTTPPostEndpoint (0.05s)--- FAIL: TestStream_AlertSigma (0.02s)--- FAIL: TestStream_AlertComplexWhere (0.02s)--- FAIL: TestStream_AlertStateChangesOnly (0.04s)--- FAIL: TestStream_AlertStateChangesOnlyExpired (0.01s)--- FAIL: TestStream_AlertFlapping (0.04s)FAILFAIL	github.com/influxdata/kapacitor/integrations	4.798s--- FAIL: TestServer_ListServiceTests (0.02s)--- FAIL: TestServer_AlertHandlers (1.01s)    --- FAIL: TestServer_AlertHandlers/post-8 (0.09s)FAILFAIL	github.com/influxdata/kapacitor/server	20.777sMichael Desa
        On Sep 6 2017, at 4:57 pm, Min Pae<EMAIL_ADDRESS>wrote:

          it's passing tests for me locally, does the ci gate run a different set of tests than test.sh?

—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.

looking in to it

@desa looks like tests are passing now, thanks for hint on test environment

LGTM.

@nathanielc @desa I've been digging in to the reason for the race condition, and it seems due to how the DefaultTransport used by the DefaultClient is using a separate goroutine to perform the actual write.
When a .Do is called on the client, it ultimately ends up calling .RoundTrip on the default transport (https://github.com/golang/go/blob/master/src/net/http/transport.go#L341), which ends up writing the write request to a channel (https://github.com/golang/go/blob/master/src/net/http/transport.go#L1955), which is then handled by the writeLoop (https://github.com/golang/go/blob/master/src/net/http/transport.go#L1761) in a separate goroutine.
When a request or context is cancelled, there seems to be no feedback/feedforward to this writer.  Looking through the http request code, it seems like there's an expectation that ownership of the object that provides the Body is handed off to the client (https://github.com/golang/go/blob/master/src/net/http/request.go#L779) since it will attempt to close the object.  Coupled with the fact that the http lib seems to do a shallow copy of the request at every opportunity, this means that ownership of the buffer has to be relinquished when it is passed to hc.Do as part of the Request.
I'm wondering whether this means in order to utilize a pool of buffers, a new buffer pool implementation is needed such that closing the buffer returns it to the pool, rather than requiring the caller to Pool.Get also Put it back to the pool.

Question...  is the motivation for using a buffer pool to eventually put a limit on maximum memory usage or is it to avoid GC?

this latest commit has a quick stab at a buffer pool that provides a closable buffer and uses .Close() on the buffer to return it to the pool

@sputnik13 Thanks!


Album View - Show Year & Genre

When an album is open to the track listing, there is nowhere that shows the year and genre of the album, as is shown on the equivalent screen in Apple Music.
It'd be great to get this added in - I'm not fussed about genre but I refer to year pretty often when listening to an album.
PS: Excellent work, love the web app and use it all the working day.

@hcuk94 Thank you for the feature request! The release date and genres are not shown on the Album view page.


List comprehensions do not work

Consider the following program.
{-# OPTIONS_GHC -fplugin Plugin.SMLPlugin #-}
module Example where
count = [i | i <- []]

Compiling the program yields the following error.
*** Exception: No match in record selector syn_expr

This was probably broken by the upgrade to GHC 9.1.
I'll try to fix it now...

This was probably broken by the upgrade to GHC 9.1.
I'll try to fix it now...

Well, this is harder than expected.
There was a simple error due to upgrading GHC.
Remaining problems:

Do-Notation works without segmentation fault, but fails at core-lint.
List conprehensions complain about a missing instance MonadND []

Both problems seem to arise from the same issue: Somewhere a list type is not lifted to ListND.

Well, this is harder than expected.
There was a simple error due to upgrading GHC.
Remaining problems:

Do-Notation works without segmentation fault, but fails at core-lint.
List conprehensions complain about a missing instance MonadND []

Both problems seem to arise from the same issue: Somewhere a list type is not lifted to ListND.


docs: update import instructions

See https://github.com/Codex-/cosmiconfig-typescript-loader/issues/36#issuecomment-1228371492

Codecov Report

Merging #39 (a9df5c3) into main (242d85f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #39   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           24        24
  Branches         2         2
=========================================
  Hits            24        24

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


Minimal changes to fix editor on MacOS with Apple Silicon

Fixes bloom shader on Metal
Swap out CSCore for UPM version as the previous one had a dll that didn't support Apple Silicon. The UPM version is pure C#

Closed - CSCore isn't working. I'll switch to just disabling audio viz on MacOS and reopen this PR


Update to v6.5.3:

version bumped up and sha256 updated
explicit pinning in host section
removed nbclassic from test requirements

nbclassic seems to be unused. It is also not mentioned in the upstream in the test requirements, although conda-forge still lists it. I have removed it.


Really finish refactoring of UBKG-API

Statement of Problem
The UBKG API still contains code that should be in hs-ontology-api.  Everything below the comment HUBMAP/SENNET ENDPOINTS should be in hs-ontology-api.
This file contains code that is only used by the hs-ontology-api:
Breadcrumbsubkg-api/src/ubkg_api/common_routes
/common_neo4j_logic.py
This file in hs-ontology-api imports a function from ubkg-api. The function is only used in hs-ontology-api, so it should be in a file in the hs-ontology-api.
(https://hubmapconsortium.slack.com/archives/GNJ4QBLEL/p1693924359451179)
https://github.com/x-atlas-consortia/ubkg-api/blob/2-refactor-application-endpoint[…]ll-suggestions/src/ubkg_api/common_routes/common_neo4j_logic.py
Import statement:
from ubkg_api.common_routes.common_neo4j_logic import query_cypher_dataset_info

The change was made as part of this issue


Viewing Terms or Privacy links from a Public Link, and using the browser back button causes Runtime Errors

Happens in both iOS (mobile Safari) and Android (Chrome).
To reproduce in iOS:

View a Public Link created in LCW (example)
LCW opens browser
Scroll to the bottom of the verified credential in the Public Link
Select either Terms of Use or Privacy Policy
Browser displays Terms / Privacy as expected
Use the back button in the browser, browser returns to the same public link (example)
Browser  reports two errors and does not display the verified credential
Refreshing the page using the refresh button will reload and display the verified credential as expected.

Error Messages from Chrome

Error Messages from Mobile Safari

This is happening in desktop browser too.

Pr: 96

Confirmed bug is fixed.


Blocking I/O with rayon threadpool could lead to deadlock.

The closure passed to the threadpool is:
fn serve(engine: impl KvsEngine, tcp: TcpStream) -> Result<()> {
    let mut r = tcp.try_clone()?;
    let mut reader = Deserializer::from_reader(BufReader::new(&tcp));
    let mut writer = BufWriter::new(&tcp);
    let req = Request::deserialize(&mut reader)?;
    macro_rules! send_resp {
        ($resp:expr) => {{
            let resp = $resp;
            serde_json::to_writer(&mut writer, &resp)?;
            writer.flush()?;
        }};
    }
    match req {
        Request::Get(key) => send_resp!(match engine.get(key) {
            Ok(value) => GetResponse::Ok(value),
            Err(e) => GetResponse::Err(format!("{}", e)),
        }),
        Request::Set(key, value) => send_resp!(match engine.set(key, value) {
            Ok(_) => SetResponse::Ok(()),
            Err(e) => SetResponse::Err(format!("{}", e)),
        }),
        Request::Remove(key) => send_resp!(match engine.remove(key) {
            Ok(_) => RemoveResponse::Ok(()),
            Err(e) => RemoveResponse::Err(format!("{}", e)),
        }),
    };
    let mut buf = Vec::new();
    r.read_to_end(&mut buf)?;
    Ok(())
}

This code needs process a request from a TCP connection with a client, and then sends response back.
When client received response, it will shutdown WRITE and exit.
Importantly, the KvsEngine is a wrapper about Arc<Mutex<SharedKvStore>>, so in serve() function it will try to require lock to process request.
In my benchmark about 1000 concurrent TCP connections with different threadpool, use rayon threadpool sometimes will lead to deadlock at Server Side.
I have tried install() and spawn() method in rayon threadpool, but it doesn't work.
Does rayon threadpool support this usage? if support, how to use? Thx!

It's not inherently a problem to make blocking calls in the rayon threadpool, but this could introduce deadlocks if the thing that's meant to unblock that is also running in the pool, like #592.


fix fix test_leading.out mistake delete

test_leading.out was mistake deleted, add it back

run buildall

run cloud_p1


Replace 'None' with 'NULL' in table sample

There was an inconsistency between the table samples shown in the metastore app to the ones accessible from the left sidebar of the hive/impala apps. Nulls in the data were coming through as "None" rather than NULL, so this patch fixes that (copied code block from describe_table.mako).
(p.s. do I need to create a JIRA too?)

This is simple enough so we can skip the jira and it looks good!
Thanks! https://github.com/cloudera/hue/commit/9e0eb37981ac1848be96d52fa47f6c898ec0bd13


feat: add visible for ExternalSecretData

Problem Statement
DataFrom is toooooo complicated, while template is very simple to use.
However, when using template, you don’t want the key to be applied to appear in the secert (although it can be exposed, it is not elegant).
Adding a switch to the Data field to determine whether to render to secert can better control the content of the generated secert.
Related Issue
#3869
https://github.com/external-secrets/external-secrets/discussions/3854
Proposed Changes

Add a rendering visible switch to Data

If the community thinks that this switch makes sense, we will continue to discuss whether this solution should be implemented.
Checklist

[ ] I have read the contribution guidelines
[ ] All commits are signed with git commit --signoff
[ ] My changes have reasonable test coverage
[ ] All tests pass with make test
[ ] I ensured my PR is ready for review with make reviewable

@gusfcarvalho

Hey! Thank you for the contribution 🥳
Although I think this is already implemented 😄  - i added a comment in the issue https://github.com/external-secrets/external-secrets/issues/3869#issuecomment-2330856033

Hey! Thank you for the contribution 🥳
Although I think this is already implemented 😄 - i added a comment in the issue #3869 (comment)

aha. But I think my conversion tool is still very useful, and it would be even better if it could appear in the external secert ecosystem.

I agree :smile:  maybe changing this PR to add it to our docs session under community ?

I agree 😄 maybe changing this PR to add it to our docs session under community ?

Sounds good!

@gusfcarvalho hi. can you help me to solve "License Compliance — 6 issues found" ? I not understand what's this.

@gusfcarvalho hi. can you help me to solve "License Compliance — 6 issues found" ? I not understand what's this.

inconvenience on our side - nothing to worry


Fix most recenty nightly breakage due to removed await! support

This bumps also rustc-ap-* libs to v546
r? @topecongiro
cc https://github.com/rust-lang/rust/issues/63195
cc https://github.com/racer-rust/racer/pull/1063

Thank you!


Cannot read property 'getBondedDevices' of null

Mobile Device Environment
Provide a list of operating systems on which this issue is relevant.

Device: Iphone 12 mini
OS: ios 16.6.1

Application Environment
Provide information about your development environment:

React Native version: v0.72.5
RN Bluetooth Classic version: v1.60.0-rc.29

Describe the bug
Android worked fine, when I switched Expo to build ios, I am getting "Cannot read property 'getBondedDevices' of null" when attempting to use RNBluetoothClassic.getBondedDevices(). Upon logging RNBluetoothClassic I get this result:
{"_eventEmitter": {"_nativeModule": {}}, "_nativeModule": null}
As opposed to the same call on Android which produces this:
{"_eventEmitter": {"_nativeModule": {}}, "_nativeModule": {"accept": [Function promiseMethodWrapper], "addListener": [Function nonPromiseMethodWrapper], "availableFromDevice": [Function promiseMethodWrapper], "cancelAccept": [Function promiseMethodWrapper], "cancelDiscovery": [Function promiseMethodWrapper], "clearFromDevice": [Function promiseMethodWrapper], "connectToDevice": [Function promiseMethodWrapper], "disconnectFromDevice": [Function promiseMethodWrapper], "getBondedDevices": [Function promiseMethodWrapper], "getConnectedDevice": [Function promiseMethodWrapper], "getConnectedDevices": [Function promiseMethodWrapper], "getConstants": [Function anonymous], "isBluetoothAvailable": [Function promiseMethodWrapper], "isBluetoothEnabled": [Function promiseMethodWrapper], "isDeviceConnected": [Function promiseMethodWrapper], "openBluetoothSettings": [Function nonPromiseMethodWrapper], "pairDevice": [Function promiseMethodWrapper], "readFromDevice": [Function promiseMethodWrapper], "readMultipleFromDevice": [Function promiseMethodWrapper], "readOneFromDevice": [Function promiseMethodWrapper], "removeAllListeners": [Function nonPromiseMethodWrapper], "removeListener": [Function nonPromiseMethodWrapper], "requestBluetoothEnabled": [Function promiseMethodWrapper], "setBluetoothAdapterName": [Function promiseMethodWrapper], "startDiscovery": [Function promiseMethodWrapper], "unpairDevice": [Function promiseMethodWrapper], "writeToDevice": [Function promiseMethodWrapper]}}
To Reproduce
Steps to reproduce the behavior:

Call RNBluetooth.getBondedDevices() on an ios device

Expected behavior
Expect to return a list of bonded devices

From my understanding there are a bunch of issues related to Bluetooth and Expo.   There was a secondary/addon library documented in order to get it working https://kenjdavidson.com/react-native-bluetooth-classic/expo/ you'll need to look into that.   I don't do much Mobile development anymore, and I've never used Expo, so this is out of my realm.
Also, have you followed all the required functionality for IOS, ie. MFi protocol strings?  This may be compiling on Expo but failing here instead of at the protocol setting portion.

Yes I have the MFi protocol string in the expo plugins "with-rn-bluetooth-classic" section, thanks for the quick response though I will do some more research!

Can you email me the protocols?
What's the device?
The error implies that the native module isn't loading. Have you tired ejecting and doing locally?

Following up for those that may come across this issue.  It looks like the IOS side of things is not auto-linking the native module.   @woflmao is going to attempt to:

Reinstall the POD and hope that it picks up
Attempt to manually link using the appropriate React Native docs

Chances are that React Native changed how Auto linking of native modules works, I recently read something in 0.70 version about native modules being changed.  If this is the case, I'll attempt to break off:

0.64.x-rcY to maintain lower version
0.70.x-rcY to start looking at fixing how React native manages native modules.

PRs will be appreciated if so.

Update: The steps shared by @kenjdavidson above did work to solve this issue. Thanks again for all the help.

@woflmao I am having the same issue. I am using Expo managed workflow with dev client. Do you have an idea about what I need to do to fix the issue?


[Icon Request] Support for Remark

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: filename
Icon Name: remark
Sample original Icon:
Filenames: .remarkrc.js

More info: Remark is a tool that works with Markdown. It includes a parser, linter, and CLI, and itegrates with other tools. The CLI uses a file named .remarkrc.js for its configuration.

The official icon as you can see at RemarkJS GitHub is the entire remark work. So not  very excited to make a text-only icon.

My suggestion is to use the red r and the black m (but not in black for dark themes).

@JimiC We should take into account these ones too: UnifiedJS

Do we have a request for UnifiedJS?

Not what I mean, just scroll down and check the other 2 syntax along with remark: Rehype and Retext 🤔

I see. Same principal. rm for remark (red-ish), rh for rehype (yellow-ish), rt for retext (green-ish).


ctool

In Process

#176 (paa)

### Product
- [ ] https://github.com/voedger/voedger/issues/279
- [ ] https://github.com/voedger/voedger/issues/276
- [x] https://github.com/voedger/voedger/issues/370
- [ ] https://github.com/voedger/voedger/issues/371

### Technical
- [x] https://github.com/voedger/voedger/issues/727

Migrated to https://github.com/orgs/voedger/projects/11/views/15


nativescript-accordion inside any contanier height problem

nativescript-accordion inside any contanier height problem ??

Not sure what you are asking here but you should be able to set the height on the view in the template

iwant this height be auto not static


[Set Audio Device] Unknown system error upon executing command

Extension – Set Audio Device
Author: @benvp

Description
All commands are failing for me with the following error:
There was an error fetching the audio devices. Command failed with Unknown system error -86: [~homepath]...

Steps To Reproduce

Install extension
Attempt to run either "Set Output Device" or "Set Input Device" commands

The current behavior
Error message (pictured above)
The expected behavior
Audio inputs/outputs should be set accordingly.
Raycast version
Version 1.34.1
Of note, I am running macOS 12.3.1 on an M1 computer that currently does not have Rosetta 2 installed. I had seen the "system error -86" crop up on another extension where a dependency appeared to require that Rosetta be present, so I suspect that could be the case here.
Assuming so, I'm not sure if there's a way to get this working without Rosetta being present, but that would be preferred if possible.

Did a little bit more digging on this, and the full error reads as follows:
FAILURE There was an error fetching the audio devices. Command failed with Unknown system error -86: ~Library/Application Support/com.raycast.macos/extensions/audio-device/audio-devices list --output --json

It appears this is occurring upstream on the macos-audio-devices package, as I'm also getting an error trying to install it directly: Bad CPU type in executable
I opened up an issue on that package to see it's feasible to compile for M1 chips: karaggeorge/macos-audio-devices#23

Bumping to keep open

@pernielsentikaer, would it be possible to reopen this one? The issue is still occurring for me so I was hoping to keep it tracked. Thanks!

Issue is still occurring.

Still valid

Still relevant

Bump!

@benvp do you have an idea here if we can do anything?

+1

Never had this issue before. Do you have that checkbox checked? This determines which implementation for getting the audio devices is being used.

Thanks for replying! It wasn't checked. When I checked it, I didn't get the error anymore, but "No Results" instead @benvp

So it uses the audio-devices implementation. That's using a binary which lives in the repo. So probably this one is still an intel-compiled version and requires Rosetta. Would it be possible for you to check, if installing rosetta fixes it?

@benvp Yes!!! It works now 😄 Thank you so much
For anyone coming from google, install Rosetta 2 by going to Terminal > softwareupdate --install-rosetta

You're welcome. So we should think about a way of adding an Apple Silicon specific binary.

I noted in the original issue that this was likely an issue related to Rosetta 2 not being present, but agreed that it would be great to include Apple Silicon support.
The issue I opened upstream is linked above, and has since had a pull request opened to build a universal binary in karaggeorge/macos-audio-devices#27. I took a look at those changes and am hoping they'll be merged upstream, but if not we could potentially use a forked version with the same change.

Oh, sorry, I completely skipped over that paragraph @djpowers

Still an issue, awaiting an upstream fix!


chore(main): release 3.1.0

:robot: I have created a release beep boop
3.1.0 (2023-11-01)
Features

add automerge (f079be3)
update ci (85c008e)

Fixes

add nl (24629af)
automerge (c0b641a)
ci (3379ecc)
delete example.js (43afe01)
disabled customStuffs.js (89d2d9b)
Remove logs for automato body script (8d68abf)
remove unused ci code (299de25)
use gh token for auto pr (6390c75)
Use gh token to automerge (cf3c33a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Yarden-zamir/Create.run/releases/tag/v3.1.0 :sunflower:


Multi-core and zcat

unzip VCF file using zcat blocks VCF parsing for every block. This results in stretch of single core use alternating with full cores. It may be interesting to tweak this behaviour.

related to https://github.com/pjotrp/bioruby-vcf/issues/17

Actually, the bio-vcf VCF reader is running in its own thread (the main one) and since the output writer has its own thread there is no improvement possible. When unzipping a file, it is zcat that is blocking. Use 'unpigz -c' instead of zcat and performance should be even better.


UI menu

Hi! I can't seem to find a mention of the keybind (or it in itself, tried all the keys) for the config menu, its not mentioned in controls, and I can't find an option for it in settings. I see that you have a picture of it, but not any description of how to get into the menu.
Thanks for your time! I absolutely love the ability to have the clock in game :D

I probably should've written this out in the mod description but you can only access the config menu using mod menu


test the game page by clicking through all the pictures

to see if the info and links are accurate, to see if the images are not duplicated

done


Add dataprepper as a part of the source image param

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
Add dataprepper as a part of the source image param
Issues Resolved
#2157
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2245 (63c2101) into main (9b021e5) will increase coverage by 5.43%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #2245      +/-   ##
=============================================
+ Coverage     94.56%   100.00%   +5.43%
=============================================
  Files           205         6     -199
  Lines          4254       105    -4149
  Branches         29        19      -10
=============================================
- Hits           4023       105    -3918
+ Misses          225         0     -225
+ Partials          6         0       -6

Impacted Files
Coverage Δ

src/sign_workflow/signer.py

...sts/jenkins/jobs/DetectTestDockerAgent_Jenkinsfile

...ests/jenkins/jobs/AssembleManifest_tar_Jenkinsfile

src/test_workflow/dependency_installer.py

src/ci_workflow/ci_check_manifest_component.py

src/build_workflow/build_args.py

src/test_workflow/bwc_test/bwc_test_runner.py

...ests/jenkins/jobs/AssembleManifest_rpm_Jenkinsfile

...nkins/jobs/RpmDashboardsDistValidation_Jenkinsfile

tests/jenkins/jobs/BuildAssembleUpload_Jenkinsfile

... and 189 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 9b021e5...63c2101. Read the comment docs.


Create shipment for Transfer Order failed if there are multiple order items with the same product

Current behavior
Create shipment for Transfer Order failed if there are multiple order items with the same product.
Expected behavior
Create shipment for Transfer Order should work fine even if there are multiple order items with the same product.
Steps to reproduce the issue

Can you handle fixing this bug by yourself?

[ ] YES
[ ] NO

Environment details

Browser:
OS:
Code Version:

Additional information

Issue verified and working fine in release v2.5.0. Hence, closing it.


Player Info Bug Temp Fix

Fixes #120

Codecov Report

Merging #121 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #121      +/-   ##
==========================================
+ Coverage   93.44%   93.46%   +0.01%
==========================================
  Files          22       22
  Lines         809      811       +2
==========================================
+ Hits          756      758       +2
  Misses         53       53

Impacted Files
Coverage Δ

espn_api/base_league.py
92.42% <100.00%> (+0.11%)
:arrow_up:

espn_api/football/league.py
97.91% <100.00%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


Adding pages to the WelcomeCoordinatorLayout at a later time messes up the whole view

I'm trying to add a page upon an button click to the layout but it messes up the whole view after being added to the layout.
I'll try to upload a screen record soon.

https://www.youtube.com/watch?v=Fd52H7ILOfs

Having the same problem, i'm trying to add pages dynamically depending on user interactions and i do not know how many pages there will be in total and the order of them when initializing the pages in onCreate, with the current implementation of this library it seems to be impossible, is there any chance that there is a small workaround for this?

maybe you can workaround with setting up all pages and on button click move to the place to go in with
coordinatorLayout.setCurrentPage(x, false)
where x is the page number to go on from now.
When you dont know the amount of pages setup at least 100 if there cannot be more than 100 and dynamically change the pages content as needed


Example of usage with Blazor WASM?

Could you provide me with an example of how to use the player with Blazor WASM or point me in the right direction? Not sure how I should connect any javascript interop for it....
I´m trying to play with it here in BlazorFiddle (and my .net 5 code locally of course) but the player doesn´t play anything.

I would also be interested in a working Blazor WASM example - any update please?

More info here @garethkelly-dev if you like https://docs.microsoft.com/en-us/answers/questions/377441/can-you-use-the-azure-media-player-in-a-blazor-was.html
But I haven´t worked on getting it into Blazor yet... when I go to .net 6 I´ll give it another go

Hello @sturlath. I got the azure media player to load a SmoothStreaming URL in a Blazor WASM (hosted) project.
Here is a link to the GitHub repo. Cheers.
https://github.com/mtrinder/blazor-azure-media-player

@mtrinder the repo is probably private because I can't access it.
But the wonderful @stsrki at @megabit aka Blazorise  has a full fledge video player for Blazor that will hopefully be in a preview release this weekend.
Please try that out with me when it hits!

@sturlath Thank you for spreading the word.
The new Blazorise v1.0 preview 1 is now up and you can see the video component in action at https://preview.blazorise.com/docs/extensions/video

@mtrinder the repo is probably private because I can't access it.
But the wonderful @stsrki at @Megabit aka Blazorise  has a full fledge video player for Blazor that will hopefully be in a preview release this weekend.
Please try that out with me when it hits!

Thanks @sturlath - it was a bad link to my repo. I've fixed it now.


Status v2 before merging

Hi friends,
I was thinking is there something we need from the old list before merging structured-v2 into master? I think we should do this ASAP and make clear contributing guidelines, such as don't link to youtube accounts, but playlists or specific tutorials. Projects should be hosted on Github. etc. Then we can post this in the FB group to get people to contribute.
What do you guys think? @tomaspietravallo @hongweitang

Also, I noticed that for playlists for video tutorials. It is awesome when the playlist is on a specific subject within Spark rather than a collection of all the tutorials. Should we take that as a requirement for playlists, that they have to be about a specific subject?

I think the only thing left from the old list is are gumroad & paid patches. About the playlists, it'd be nice to have all the playlists divided up, that's true

@tomaspietravallo I've already added som on the "Other Resources" section. I also got Josh to make a playlist for his LUT's, so I've updated that. Also the contribution file with info on sharing projects and videos

Yes love the new direction of the structure and how the content has to be prepared.
The contribution guidelines are gonna be crucial for how that will continue. Clear explanations of the contributed links make it more searchable and findable through the internet too.

Merged.


typo: Protodol -> Protocol

@nschonni : Thanks for your contribution! The author, @spelluru, has been notified to review your proposed change.

Thanks again, @nschonni
#sign-off


Passing variables to triggered build

Feature request:
When triggering a pipeline from release build, we would want to pass some variables.
Use case :
I am trying to use this extension to run tests against every deployment.
So from the release build, I would want to pass the name of the environment to the pipeline as a variable.

You can add value to trigger a specific stage.


CollectionsWrite with published explicitly set to false gets treated as published and unpublished in CollectionsQuery

It appears that the underlying search-index library does not handle indexing of boolean values.


[DISCUSSION][UX] Inbox search clarification

Description of issue
There are multiple related UX problems associated with the search feature which is initiated by clicking the magnifying class icon in the current Android client "inbox" shown here:

The first issue is that when the icon is clicked we see the same list of active chats along with a preview of the most recent message in those chats, which suggests that the search will match the typed string against the messages themselves:

when in fact it only searches for users/groups to chat with. The preview of the messages is misleading.
A different but related issue is that one would expect an inbox search to search the contents of the inbox -- after all the magnifying glass is pinned to the inbox itself -- so I would expect the search to match against strings found in my inbox, meaning messages in DMs and the names of users/groups/channels that I am chatting with. I would not expect the inbox search to search across ALL users/groups/channels on the server, including those that I do not have an open chat with.
Solution
These two issues have confused many of our users and we think the UX would be more intuitive if inbox search button led to a search of the inbox content. This is pretty standard functionality but for an example look at how WhatsApp inbox search (which uses the same magnifying class icon) works. As one would expect, it searches across users and groups and messages within the inbox and not beyond that.
Associated change needed
If this issue is fixed as per the above, we would then eliminate the only way that new chats/groups are created using the current Android client. In my opinion, this should be resolved via a FAB button, which was implemented but then rejected here: https://github.com/RocketChat/Rocket.Chat.Android/pull/891. Note that the React Native RC client uses a FAB button as does Hip Chat, FB messenger, WhatsApp, and many more. In fact, if you look up the documentation of the FAB button, the example given for when to use a FAB is in fact exactly this use case -- to create new chat's in a chat app inbox. See https://material.io/design/components/buttons-floating-action-button.html and this image:

@bizzbyster Thanks for pointing that to us, makes a lot of sense to me. 👍
Searching messages is not as easy as it sounds for us, because we don't have all the content on the phone and we don't have a "global search" yet.
What do you think about changing the layout of the cells to only show the name of the chat when search is activated? This may not be the perfect solution, but should work in the UX perspective and resolves the problem very easily for now.
Related to the "+" button: agreed, if we change the behaviour a little bit on what happens once you tap it. I think the ideal case would be: when I press the + I would be able to start a DM (with existing or not user) and create a new group/channel if that's what I want. What do yo think?
Again, thanks for the feedback! 💯

@rafaelks Thanks for commenting on this. I'd like to think on it a bit more but my first reaction is this:
If we add the "+" FAB, do we still need the search magnifying glass? Why not just at that back once we have the ability to do the "global search" (what I call inbox search) properly?
Incidentally, we may be able to help with this search feature if you point me to where it is being discussed and/or designed.

@rafaelks No doubt this additional piece belongs in a new issue but the new chat/group that results from clicking the "+" FAB button, can also include an option to "invite" users that are not already on the server. This could integrate this work https://github.com/WideChat/Rocket.Chat.Android/commit/cb786166fc6ad45199bcbc0cf25fdc65c7cebce9 which is documented here https://forums.rocket.chat/t/invite-new-users-feature/668. So by clicking the '+' you 1) start a new chat with a user already on the server or join an existing public group, 2) start a new private group or public channel, or 3) invite new users to the server. Thoughts?

On further thought and discussion @rafaelks I agree with what you propose:

changing the layout of the cells to only show the name of the chat when search is activated
when I press the + I would be able to start a DM (with existing or not user) and create a new group/channel if that's what I want -- where the "+" is a floating action button.

Next time I'll think a bit before responding. :)

Adding @thiagosanchz and @filipedelimabrito to the discussion here.

Also, when I press the + FAB and go to a new chat dialog which allows me to search for existing users on the server it would be great if we could also give the option to invite users via the share intent. The work for this can be pulled from here https://github.com/WideChat/Rocket.Chat.Android/pull/91 (with feature discussion here https://forums.rocket.chat/t/invite-new-users-feature/668) with the difference being that the button to trigger it would be on this create a new DM/group/channel dialog. This would really distinguish the inbox search and the + / FAB / new chat actions while maintaining the way that they overlap in that they can both find users/groups to chat with.


WIP: Added debug container to csp pod and change heatbeat to 10 secs.

WIP: This adds another container to the CSP pod that can be used to query the ubbagent.

no longer needed

no longer needed


Epic: Refactoring

Development has gotten a bit hackish as the user stories were not developed enough to have a solid foundation for development. Therefore we need to refactor the app before public deployment.
Separating out Invoice
The biggest change is that logic related to the invoice should be in a separate Invoice (#61) table. We can therefore remove the following from Event:
    currency = models.TextField(null=True, blank=True)  # Show as an input
    payment_deadline_days = models.IntegerField(default=30)
    payment_details = models.TextField(blank=True, default="")

currency needs to be added to RegistrationOption (a single event can have registration in multiple currencies).
In Registration, invoice_no should be replaced with a foreign key. Similarly, paid and paid_date should be removed. extra_invoice_text should be a column (with a different name) in RegistrationOption.
Changing registration options
The table RegistrationOption should only be for registering for an event. A single person can only register once for an event. The column add_on can be removed. Registration can therefore have a foreign key (instead of ManyToMany) to RegistrationOption.
Registration status
Instead of a column active, we should have a choices field (models.TextField(choices=) called status. This can be:

Application submitted
Selected
Wait listed
Declined
Registered (payment pending)
Registered
Withdrawn

Discounts
This functionality is nice to have, so let's comment it out for now
Membership
The column honorary should be changed to a choices field with possibilities:

Normal
Board
Business
Honorary

This isn't shown to users when they are registering - board and honorary are created manually, and there is a separate signup for business members.

Should we have invoice_no in an old form (year + no) as a PK in the Invoice table? ('In Registration, invoice_no should be replaced with a foreign key').
Should paid and paid_date be moved to Invoice instead of completely removed?
Related question (probably, the most important now): do we have any changes to the man payment procedure: do we still have to change the paid status (in the Invoice table?) on a payment success event?
I don't understand the idea of registration options changes. I'll ask about it later.
Furthermore, I'll need explanations on a status flow (when we should switch to which statuses).
Have seen PR #52: It'd probably be better to generate PDFs on the fly, not to store them as blobs.
What is the template field for there?

Should paid and paid_date be moved to Invoice instead of completely removed?

Yes, this seems reasonable. These two fields are both needed, but maybe instead of paid we have status with more than on/off choices, for example.

Have seen PR https://github.com/Depart-de-Sentier/dds_registration/pull/52: It'd probably be better to generate PDFs on the fly, not to store them as blobs.

@jsvgoncalves is a big fan of this idea as well. Let's do that.

What is the template field for there?

Because we can have more than one kind of invoice :)

Should we have invoice_no in an old form (year + no) as a PK in the Invoice table? ('In Registration, invoice_no should be replaced with a foreign key').

Sure, why not? It must be unique, and doesn't need to be sequential.

Related question (probably, the most important now): do we have any changes to the man payment procedure: do we still have to change the paid status (in the Invoice table?) on a payment success event?

Yes, we will need to integrate with both Wise and Stripe to automate this - but that is not a requirement for v1 (we need to ship v1 ASAP).

Furthermore, I'll need explanations on a status flow (when we should switch to which statuses).

Almost all of that is manual, except for payment being made, and even that will be manual for now.

Sure, why not? It must be unique, and doesn't need to be sequential.

Probably, we could have the normal auto-incremented integer PK and construct invoice no by adding the year.


Autorotate image

Autorotate image on upload based on EXIF metadata

This pull request needs to be rebased and reopened because we did some history rewriting, if you need help with this head into #dev in our slack channel
#4814


bugfix: fix loader path in startup scripts

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
New PR which is related to #5010
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #5018 (2a36b79) into develop (83ac9f3) will decrease coverage by 0.21%.
The diff coverage is n/a.

:exclamation: Current head 2a36b79 differs from pull request most recent head 5557dfc. Consider uploading reports for the commit 5557dfc to get more accurate results

@@              Coverage Diff              @@
##             develop    #5018      +/-   ##
=============================================
- Coverage      49.19%   48.97%   -0.22%
+ Complexity      4101     4075      -26
=============================================
  Files            733      733
  Lines          25958    25958
  Branches        3202     3202
=============================================
- Hits           12771    12714      -57
- Misses         11833    11888      +55
- Partials        1354     1356       +2

Impacted Files
Coverage Δ

...va/io/seata/server/console/vo/GlobalSessionVO.java
22.05% <0.00%> (-33.83%)
:arrow_down:

...er/src/main/java/io/seata/server/ServerRunner.java
50.00% <0.00%> (-15.39%)
:arrow_down:

...java/io/seata/server/storage/SessionConverter.java
80.00% <0.00%> (-9.10%)
:arrow_down:

...rage/redis/store/RedisTransactionStoreManager.java
63.53% <0.00%> (-4.56%)
:arrow_down:

.../java/io/seata/spring/util/TCCBeanParserUtils.java
30.64% <0.00%> (-1.62%)
:arrow_down:

...torage/file/store/FileTransactionStoreManager.java
56.27% <0.00%> (-0.65%)
:arrow_down:

...erver/storage/file/session/FileSessionManager.java
49.04% <0.00%> (-0.64%)
:arrow_down:


kong: FIX Kill wait-for-postgres if host or port not set

Fixes #27
The script runs forever and simply prints 'waiting for db' if KONG_PG_HOST or
KONG_PG_PORT are not set. Which isn't very helpful.
This commit hard fails if either of those environment variables are not set,
making it easier and quicker to diagnose problems.
As a note, the reason why they may not be set is when a user provides there own
database but doesn't set pg_port or pg_host.
Additional changes:

Message is slightly more informative, so if connecting takes awhile,
the use can see host and port being attempted.
Set the default postgres port

@yasn77 Can you rebase this on top of next and then it should be good to merge.

@yasn77 Can you rebase this on top of next and then it should be good to merge.

Done :+1:


Support scaling

Add support for scaling to be able to draw maps like the ones in Door in the woods.

Giving this a look-see locally. Thank you!

I like the overall effect, and it works well. So I'll merge it, and add an issue to note that there's a little more to do: the mouse position should be scaled as well.


rootElement sometimes variable sometimes function

The rootElement function call uses functor to set the rootElement variable to a function.
The rest of the code then tries to set rootElement.append or similar instead of rootElement().append. This inconsistency makes the library impossible to change the tip location from a body to div.

There's been some unresolved issues with rootElement ( See #175 ). I haven't been able to round up help to fix it, and I'm unfortunately not active enough to be able to fix it myself.


dummy change to test PR build with Maven 3.5.2

Signed-off-by: Aurélien Pupier<EMAIL_ADDRESS>

ok to test

it has not worked as i expected.
Maven version has been changed in the job config for jenkins but for Travis it is still using Maven 3.3.9 https://travis-ci.org/fabric8io/fabric8-maven-plugin/jobs/397348506#L1454
I don't know how to modify it, I found no reference in codebase to 3.3.9 and one for 3.5.2 (which seems not used in the build then?)

@apupier : Right now we don't specify maven version in these integration tests. I think it can be configured here using option useMaven3Version: https://github.com/fabric8io/fabric8-maven-plugin/blob/e7ea8bff53bb7f8035d27b0e587df0810d71b144/rt/src/test/java/io/fabric8/maven/rt/BaseBoosterIT.java#L204-L209

I'm closing this issue. it shows that the problem was not in fabric8-maven-plugin itself.
Based on that I was able to find that the issue is related to an incompatibility within the embedded Maven version in Eclipse and fabric8-maven-plugin. I think the issue is more on Eclipse side.
Thanks for your help!


updating to k8s 1.28 and doing any conversions for removed fields

Relates: operator-framework/operator-sdk#6554

Updating the dependencies to support k8s 1.28, doing so requires some code changes for fields that were removed in controller-rumtime v0.16.x.

@acornett21 I did this in #253


Update EveMarketer.Js

Fixed undefined error, also used uppercase in case strings for ability to use uppercase locations in spreadsheet. Took off console.log

Also could use parameter update.


'applies to' is listing the wrong information

at the top of the page you can find
Windows 10
Windows Server 2016 and above
at least wrong for option 16, which can be used starting with version 1709
Server 2016 (Gui version ) is version 1607 - and therefore option 16 cannot be used here
would be good to have availability information for the single options based on version information
Windows 10 and Server 2016 is missleading for those dealing with Server with GUI or LTSB/C versions
thanks

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

ID: 6d4ecadb-c54e-5da1-03ae-1301a35ef64a
Version Independent ID: 69d28732-f557-9ae1-cec4-594a2ec097a2
Content: Understand WDAC policy rules and file rules (Windows 10) - Windows security
Content Source: windows/security/threat-protection/windows-defender-application-control/select-types-of-rules-to-create.md
Product: w10
Technology: windows
GitHub Login: @jsuther1974
Microsoft Alias: dansimp

@AndreasDiet - Thank you for submitting feedback.
From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/windows-itpro-docs/commit/17cbe48c1727bef184e25b64ae06c4e7c8c20b41.
Thank you for your contribution to make the docs better! Much appreciated!


Testing for default connection

Is there a way to test if a default connection is available in .dwrc before calling dwave_micro_client.Connection()?  If not, this is a feature request.
As of v0.2.2 calling dwave_micro_client.Connection() without a .dwrc for me results in,
...
  File "/usr/local/lib/python2.7/site-packages/dwave_micro_client.py", line 168, in __init__
    url, token, proxies, self.default_solver = load_configuration(url)
  File "/usr/local/lib/python2.7/site-packages/dwave_micro_client.py", line 1440, in load_configuration
    raise IOError("Could not load configuration from {}".format(file_path))
IOError: Could not load configuration from /Users/carleton/.dwrc
unable to parse stdin as a json document

Absolutely valid feature request. We're already doing something along those lines with the auth refactor that's in progress. :)

As of v0.3.0 (Mar 8), configuration via .dwrc is deprecated in favor of dwave.conf.
Introduction to the new config system is available in the docs. Detailed docs on how to use it under dwave.cloud.client.Client.from_config.
Now there are several ways/layers on which you can define the default solver: via config file - under solver section and under defaults section, via env var, and during call to from_config via kwargs.
Lastly, feature-based solver selection (i.e. not name/id-based) is also available in v0.4.3+. It's not yet visible on RTD, but it's documented in docstring.
You can, for example, get the first QPU-based solver with at least 1000 qubits (but no upper bound) like this:
first_qpu_solver = client.solvers(qpu=True, num_qubits=[1000, None])[0]


Implement reset method in the PTB example

Fix #6196.

Jenkins, test this please

Jenkins CI test (for commit e9a5c1146f4946d530c0e76265fd7c99f02bef7b, target branch master) failed with status FAILURE.

LGTM


fix: covers case where AGOL sends appended 1=1 along with other query fields

We noticed sometimes that AGOL would send 1=1 at the end of a query string with many other query parameters.  When this would happen the field from 1=1 would get converted to properties => 1 and break the query.

@efbenson, I'm also not sure why 1=1 would be appended. This seems like a reasonable patch to just ignore that field.


Music and genre UI options

In this branch I:

Created UI options to create a music_album, list all albums, and list all genres.
Created a modules folder to hold the modules.
Created a decorator module that takes any item class and adds an author, a genre, and a label with the user input.
Changed the publish_date validation to be more concise and precise.
Solved the cyclomatic complexity by changing how the options are dispatched (hash table instead of switch case)

CLOSES #14

Status: Aproved!
Hello @tomasesquivelgc , Your code looks great! there's nothing else to say other than... It's time to merge!!
✔Linters pass
✔Tests are correct
✔Implemented requested changes
✔All classes working as intended