Running the bot on debian (Mono)

Hello, I have been trying to make the bot work on my odroid c2 without success...
I only have ssh access to the odroid (no gui), so I copied a configuration from Windows.
This is the output I have (I enabled the debug flag inside ASF.json):
2016-08-18 15:42:23|INFO|ASF|Init() ASF V<IP_ADDRESS>
2016-08-18 15:42:23|INFO|ASF|IsRuntimeSupported() Your Mono version is OK. Required: 4.4 | Found: 4.7.0
2016-08-18 15:42:25|INFO|ASF|InitEnhancedLoggers() Logging module initialized!
2016-08-18 15:42:25|INFO|ASF|CheckForUpdate() ASF will automatically check for new versions every 24 hours
2016-08-18 15:42:25|INFO|ASF|CheckForUpdate() Checking new version...
2016-08-18 15:42:25|WARN|ASF|UrlGetToContentRetry() Request failed even after 5 tries
2016-08-18 15:42:25|WARN|ASF|CheckForUpdate() Could not check latest version!
2016-08-18 15:42:25|INFO|account_name|Start() Starting...
2016-08-18 15:42:26|INFO|account_name|OnDisconnected() Disconnected from Steam!
2016-08-18 15:42:26|INFO|account_name|OnDisconnected() Reconnecting...
2016-08-18 15:42:35|INFO|account_name|OnDisconnected() Disconnected from Steam!
2016-08-18 15:42:35|INFO|account_name|OnDisconnected() Reconnecting...

And it just keeps on going on and on.
I tried following the Mono wiki page but there's no issue like this mentioned...
Do I have to run/install something else to make
(I replaced my account name with account_name)
Thank you for your awesome work!

Solved it, sorry for the spam.
I run ASF.exe with the --trace argument on mono and I found out I had no certificates...
Even if I had a newer version of Mono I had to import the certificates with the command: mozroots --import --sync
Anyway, it'd be nice to have that kind of error (the certificate error) shown, at least when the debug flag is up.

Anyway, it'd be nice to have that kind of error (the certificate error) shown, at least when the debug flag is up.

Ask Mono team for that.


case sensitive for searching

hi
is it possible to disable the case sensitive for searching ? or add an option for that ?
i've made a patch if you want. thanks

I'd suggest to file a PR and link it here, if you already did that feature.


Window Rule not working perfect

[winshadows]
clip_shadow_inside = false
enabled_views = type is "toplevel" & floating is true & maximized is false
glow_color = \#3584E4FF
glow_emissivity = 1.000000
glow_enabled = false
glow_intensity = 0.500000
glow_radius = 0
glow_radius_limit = 150
glow_spread = 8.000000
glow_threshold = 0.030000
horizontal_offset = 0
include_undecorated_views = true
shadow_color = \#00000099
shadow_emissiveness = 0.000000
shadow_radius = 100
vertical_offset = 0

maximized windows still have shadows

It happens to when a window is unmaximized and then get maximized, there is no issue if window created as maximized by default

Upstream issue, see linked issue and PR.


[5.9 05-01-2023] Support resolving code file references in ConvertService

Explanation: Support resolving tutorial code file references in ConvertService requests.
Scope: Tutorial content in ConvertService requests.
Issue: rdar://107965493
Risk: Low risk. Isolated and limited scope
Testing: New tests verify that file references are resolved and that the Tutorial Render JSON include the code file's content and their line differences.
Original PR: #570
Reviewers: @ethan-kusters

@swift-ci please test


Error Trying to install postgres

Expected behavior
Create postgres container
Actual behavior
(HTTP code 500) server error - {"message":"Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)"}
Information about the Issue
Steps to reproduce the behavior

Open kitematic
New
Postgres
create

Please make sure that you're on a stable connection and use a solid DNS such as Google DNS.


Failed prop type: undefined is not an object (evaluating 'Object.keys(flattened)')

Description
The following warning is thrown when displaying animations that don't have a style object.
Failed prop type: undefined is not an object (evaluating 'Object.keys(flattened)')

Steps to Reproduce

Create a LottieView without a style-property, i.e.

export class Animation extends React.Component {
    animation

    componentDidMount() {
        this.animation.play()
    }

    render() {
        return <LottieView ref={animation => (this.animation = animation)} source={require('./animation.json')} />
    }
}

Observe the animation through your app

Expected behavior: Animation plays without warnings.
Actual behavior: The above warning is thrown.
Versions
Introduced in 2.5.1 and still exists in 2.5.6. Both iOS and Android.

Fixed on master:
https://github.com/react-community/lottie-react-native/commit/ec60a087ccd819fc7901d8d8355407f5b1f6c543
The next release (2.5.10) will include this fix

@emilioicai when can we expect 2.5.10?


Undefined symbols: Linker command failed with exit code 1 (use -v to see invocation)

Description
I'm getting this error "Undefined symbols: Linker command failed with exit code 1 (use -v to see invocation)" after updating the firebase from 10.29.0 to 11.1.0

Firebase SDK Version
11.1.0
Xcode Version
15.0.1
Installation Method
CocoaPods
Firebase Product(s)
Crashlytics
Targeted Platforms
iOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Please show the full error message and fill out the full issue template.

You can see this video
https://github.com/user-attachments/assets/bfeb0b6a-ec20-4091-8d93-e83c920dc0be

Select the below icon and click on the build logs for the failing build. The undefined symbols should appear when expanding the failing build command.

https://github.com/user-attachments/assets/9e1e2911-aa45-4d15-97b2-dec1578c7d9c

Firebase requires at least Xcode 15.2. See the release notes.


UI improvements

Added the tabbed UI (but brought back the standard one that we'll keep for now)
Improved user list with grouping
Stream list improved - icons for private/public and description text
Added account info screen

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!


Reset-trust-status in client

Client command to reset trust status, returns that reset status.
Plain text:
coda.exe client reset-trust-status -ip-address <IP_ADDRESS>
0.0000, Unbanned

JSON:
coda.exe client reset-trust-status -ip-address <IP_ADDRESS> -json
{"trust":0.0,"banned":"Unbanned"}

Partially implements #2658.

If you want to learn more about how the graphql interface works, it shouldn't be too hard to graft on support for a GraphQL mutation.
To do so:

open up graphql.ml
in module Mutations, make a let reset_trust a field that
Takes an input of the ip addr string (wrapped in a ResetTrustInput)
And in the resolver invokes Coda_commands.reset_trust
(look at delete_wallet for an example)

But I'm willing to approve without that, if you are too busy

@bkase Can we add the GraphQL stuff in a separate PR?


[1.1] kill k8s latest boskos resource

We used to have dedicated boskos resources for the "Latest" GKE version. At the time this was 1.11, and other tests ran on 1.10.
Today, we cannot even create a 1.10 GKE cluster, since they bumped the minimum version, so we can just consolidate these to save on the number clusters we have.
This will fail until https://github.com/istio/test-infra/pull/1953/files is merged

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
a523fa74905434ce728a8a6c1ee44b765a708fae
link
/test istio-unit-tests

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

/retest

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
a523fa74905434ce728a8a6c1ee44b765a708fae
link
/test istio-pilot-multicluster-e2e

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


Does not start playing automatically

The player does not automatically start playing the video, and waiting for the "Play" button to be pressed.  For example, if you make it the default player in LazyMedia Deluxe...and in other programs too... It decreases usability :((
My device is Ugoos-AM6, ATV, Android 9

Is this caused by the Auto frame rate matching? Does it work (start the playback) when you disable it? If yes, than it is the same issue as in #179. In this case, please provide me a bug report.


Update font-awesome from v4.5.0 to v4.7.0

I updated the font awesome library to version v4.7.0 which includes a lot of new useful icons.

Hello @Tafkas,
thank your for submitting this pull request! :+1:


Privacy policy for the embedded version

Hello,
Thanks for your great work. I plan to embed openspeed in a mobile application. I would like to know about the privacy policy. Which data is collected ?
Thanks

Use https://github.com/openspeedtest/Speed-Test on your own server; it does not collect any data.

Thanks for your answer.

Use https://github.com/openspeedtest/Speed-Test on your own server; it does not collect any data.

The icons at the bottom take the user to the openspeedtest website. I can see from the url that the data is actually probably going there just for display purposes, but that also gives you the opportunity to collect it. Would it be possible to do the display locally. Failing that, is there a way to disable creating those links?

@xpufx link to Openspeedtest? Guage Icon?


upgrade to babel 7

Rebased on dev and against dev, as requested in #52

Pull Request Test Coverage Report for Build 150

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.177%

Totals

Change from base Build 147:
0.0%

Covered Lines:
278

Relevant Lines:
282

💛  - Coveralls


net: ieee802154: Add transmission with multiple CCA for nRF SoCs

The Kconfig IEEE802154_MULTIPLE_CCA option is added. The net_pkt_cb_ieee802154 is extended allowing to express the number of additional CCA attempts to perform.
The ieee802154_nrf5.c is updated allowing to pass extra cca attempts to nRF 802.15.4 Radio Driver.
This PR is to enable usage of new feature of nRF 802.15.4 Radio Driver from Nordic Semiconductor.
The nRF 802.15.4 Radio Driver will be delivered soon to https://github.com/zephyrproject-rtos/hal_nordic

@jciupis @ahasztag @LuDuda @edmont Please review and let me know if this satisfies your needs

https://github.com/zephyrproject-rtos/zephyr/pull/60390 has been merged so this can be closed, I guess?

@fgrandel

It's the proper PR and Kconfig description that says that this is a non-standard feature. And of course non-standard features can be supported just not in a namespace starting with IEEE802154 IMHO. :-D

What about naming all such extensions on Kconfig level as IEEE802154_EXTENSION_* and particularly this one IEEE802154_EXTENSION_MULTIPLE_CCA ? They can be in the zephyr/drivers/ieee802154/Kconfig or in some additional zephyr/drivers/ieee802154/Kconfig.extensions
Is there a common way to add such extensions before they get into standard?

What about naming all such extensions on Kconfig level as IEEE802154_EXTENSION_*

@ankuns Spot on. :-) You know by now that we had the exact same idea regarding this.


Dropzone accessibility

does not support keyboard

https://github.com/Panenco/panenco-ui/issues/400


update k8s to 1.25.2

Changes

update k8s library to 1.25.2

Fixes #2598

Release Note

Update k8s library to 1.25.2

/cc @psschwei

/hold

/retest

@pierDipi @lionelvillard seems all the eventing downstream tests get broken with the update to k8s 1.25.2, I think a big part of it is updating the version of cloud-events

retesting after https://github.com/knative/eventing/pull/6552
/retest

It's an antlr problem. k8s 1.25 picks up a newer not-backward compatible antlr. I'll submit a PR in the ce SDK repo.

btw k8s 1.25 requires go 1.19. It does not seem to use any new go features so the build does not break but I think it's probably better to align with what k8s is doing.

/retest
For eventing.

/retest
For eventing.

seems like a different issue this time

@lionelvillard
do you think we can get this fixed for the release next week?

@lionelvillard @psschwei does this PR need to be merged in first then we fix the downstream issues? or how does this work?

@lionelvillard @psschwei does this PR need to be merged in first then we fix the downstream issues? or how does this work?

Yeah, we should make the update here first and then the fixes downstream, assuming everyone's ready. We should be good from the serving side, @lionelvillard can confirm for eventing.

the only caveat is easycla is broken for knative/eventing

EasyCLA should be fixed now, so...

/hold cancel


Fix - #418 Feature section responsiveness added

Fixes #418
Before Fix :

After fix :

Hi @anuragverma108
When you have a moment, could you please review it and let me know if any further changes are needed? If everything looks good, I would appreciate it if you could merge it.

Closed due to no response recieved.


[Enhancement] Use BrokerService instead of PulsarClient for topic lookup

Is your feature request related to a problem? Please describe.
Currently KoP use Pulsar broker's builtin PulsarClient for topic lookup, see https://github.com/streamnative/kop/blob/9630e5c0de041119ec2027502f655a3ed814f12b/kafka-impl/src/main/java/io/streamnative/pulsar/handlers/kop/KafkaTopicManager.java#L211-L212
There's a limit for this way. For example, all connections share the same PulsarClient instance. However, there's a pending requests limit on PulsarClient, i.e. if there're many connections, the load on a PulsarClient will be high and TooManyRequestsException could be thrown. There's another case that if a namespace has many topics, when a Kafka client tries to list all topics, the TooManyRequestsException could also be thrown because all topics need to be looked up in METADATA request handler.
Describe the solution you'd like
Use BrokerService instead of PulsarClient for topic lookup.

Can we use TopicLooupBase.lookupTopicAsync directly to do topic lookup? We could specifiy advertisedListenerName to get the kafka advertised address directly, could also resolve the problem of #633

Yeah, I'm working on this way recently.

@wangjialing218 It looks like not easy to use TopicLookupBase.lookupTopicAsync in KoP. The broker side may send a LookupResponse whose redirect field is true to client, then Pulsar client will resend a LookupRequest to the redirected broker. However, there may be no way for Kafka client to do this like Pulsar client.
I'll take a further look to this problem. If you have any idea, feel free to share it.

After reading the source code of NamespaceService#findBrokerServiceUrl,  only if no one owns the bundle could the redirect field be set.
            // First check if we or someone else already owns the bundle
            ownershipCache.getOwnerAsync(bundle).thenAccept(nsData -> {
                if (!nsData.isPresent()) {
                    // No one owns this bundle

                    if (options.isReadOnly()) {
                        // Do not attempt to acquire ownership
                        future.complete(Optional.empty());
                    } else {
                        // Now, no one owns the namespace yet. Hence, we will try to dynamically assign it
                        pulsar.getExecutor().execute(() -> {
                            searchForCandidateBroker(bundle, future, options); // NOTE: redirect is set here
                        });
                    }

I think we can handle the case like how we handle the future.complete(Optional.empty()) case.


Feature/authors info display

Major updates:

The individual results page for a record pings crossref API to retrieve the authors of the dataset if a publication DOI is present in the record metadata.
The author names link to their ORCID profiles if the ORCID number is returned by crossref.

Additional updates:

Calculation details box is made expandable through mouse click rather than by hovering. The metadata can be copied using a 'Copy calculation details' button at the bottom of the calculation metadata window, which is cleaner than mouse highlight to select text and then copy.
The calculation details display is cleaned up to display a neater JSON parsed text.
Cleaned up table display to remove a strange gap.

Thanks Sathya, those are very useful changes!
A quick question about the crossref lookup: does the loading of the record page wait until the lookup is complete? If so, what happens if the lookup hangs? Or does it update that content once it received the info from crossref?
I would suggest three small changes:

the naming of the api/works route - is there a  reason for this route name that I'm missing?
A few tests for the crossref lookup - add in a few random DOIs with author lists you look up and test to see that they're correct.
Handling of 'bad' DOIs: Since users can enter ~anything into our DOI field, we should handle the case where the crossref lookup comes back empty/with an error of some kind. Having a bad DOI in the tests would also be useful.

Cheers

@jkshenton Besides making changes as you suggested, I have added test cases to fetch 1 good doi and 1 bad doi.
For good doi, I have pre-defined authors' names as list of strings to compare that the names are in the returned response (author names part of the total API response). For bad doi, the test checks that he appropriate error message and the error status code are returned. Please see commit de3872a
Do you have any improvement suggestions?

Hi @Sathya-S3 , that looks good to me now - thanks!


[0.8.x] GUVNOR-2373: Modal for artifacts table in Project editor-Dependencies is too small for the content

See https://issues.jboss.org/browse/GUVNOR-2373
This PR is for 0.8.x (cherry picked from commit 2e14355c84dca1d939369c25ade5140bad686f28)

Part of an ensemble:-
https://github.com/uberfire/uberfire-extensions/pull/203
https://github.com/droolsjbpm/guvnor/pull/243
https://github.com/droolsjbpm/kie-wb-common/pull/248

0.8.x: 2dcb1665bddbe9b60456598b53a93a5fb218b7f4


Update to current prerequisites

.NET 4.5.1 and early versions of VS 2017 will not work

Thank you @ErikEJ!


A post comparing agent frameworks

https://datarevenue.com/en-blog/ml-platforms-dataiku-vs-alteryx-vs-sagemaker is the sort of article that indicates a maturing market/trend or possibly a paradigm. I think If we do a post comparing the RL and cog sci / neuroscience frameworks we know about, that would be the sort of thing that could help (in a small way) our agenda to progress the agent paradigm

Yeah, feels like we've already done a fair bit of the work here, and it'd be satisfying to finalize (a v1) and post it somewhere.


Toggle / hide titlebar visibilty

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
The electron titlebar not only takes up valuable screen real-estate, but the grey colour contrasting against the dark theme is pretty off putting. The lost space is exacerbated when using a tiling window manager.
Adding the option to toggle / hide the titlebar in preferences would be great, and would not be inaccessible once the titlebar is hidden, since the preferences icon is still visible in the GUI in the bottom left.
.bru file to reproduce the bug
No response
Screenshots/Live demo link

This might be a duplicate of this issue: https://github.com/usebruno/bruno/issues/1315

I did search, but apparently not effectively. Closing as duplicate


Removed unused variable

Type of change
Sample update

cc @ealsur

BTW, any specific reason DeleteContainerStreamAsync() is used instead of DeleteContainerAsync()?

No other reason but the fact that I'm not really interested in the serialization of the response

No other reason but the fact that I'm not really interested in the serialization of the response

Thank you. Just learned something from you 🙂
Sorry to pollute the PR thread, but where can I find more documentation on the API differences?

/azp run

@SeanFeldman thank you for your contribution.


Question Marks in Graph

I have changed the font a few times and still having issues. If I remember exactly, suppose to be dots. Haven't used vtop in awhile. I am on a fresh install of Arch Linux. Using i3wm and Terminator.
screenshot

I think this is a problem with your system itself. According to what I read on the internet, one needs to glymph of that particular set of characters. BTW, I am having the same issue.

Same issue. Braille working in other apps.

Launching vtop with a UTF-8 locale may help. See #120.

Same with MacOSX and edex-ui.

Same issue

Resolved by adding the following line in ~/.bashrc or ~/.zshrc
export LC_ALL=en_US.UTF-8
export LANG=en_us.UTF-8


Does supplying cookie options need to be so complex?

You're obviously smarter than me so I may be missing something. Have I done this in the way that was intended?
let expiresTimestamp = (new Date).setFullYear(2025);

class MyCookieOpts implements CookieOptionsArgs {
    path = '/';
    domain = 'localhost';
    expires = new Date(expiresTimestamp);
    secure = false;
}

this.cookieService.put('key', 'value', new CookieOptions(new MyCookieOpts));

If so, is this really necessary? The learning curve for TypeScript/Angular2 is already high enough...

Hey @stewhouston
Actually, you can do the following:
let expires:Date = new Date();
expires.setFullYear(2025);

let options: CookieOptionsArgs = <CookieOptionsArgs> {
        expires: expiresTimestamp
    };

this.cookieService.put('key', 'value', options);

But if you want to override defaults options, check the last part of the readme file.

Thanks mate, that's much cleaner. As Angular2 gets more popular the repo might pick up steam. There will probably be a few more jQuery-for-everything converts like me coming by. This snippet would be very helpful in the readme.


🌱 Descriptive Upgrade Constraint Policy constants

Addresses #1151
This PR renames the constant values used in the API for UpgradeConstraintPolicy and updates their references in the documentation.

Conflicts because PR https://github.com/operator-framework/operator-controller/pull/1237 has merged

Approved and then looked at CI - looks like you need to:

rebase and resolve merge conflicts
regenerate manifests with make manifests

I've rebased and resolved conflicts. The CI failures are because we are changing the API right ?

@skattoju It looks like there is still a sanity failure. I do expect the go-apidiff failure though.

Looks like the sanity failure is due to the newly added types documentation

looks like sanity is passing after doing make verify and committing changes to docs/refs/api/operator-controller-api-reference.md 👍


cond-expr: Selecting a new mapping with cond expr enabled show old

Selecting a new mapping when the cond expr is enabled for the old mapping still shows old cexpression.

I don't see this issue, there might be some condition to reproduce

Procedure:

Establish 2 different mappings
Select mapping-A, select the conditional expression icon
Enter an arbitrary expression
Select mapping-B, select the conditional expression icon
Select mapping-A - the conditional expression retains the expression for mapping-B

Workaround:
Close the expression box and re-open it.


Linux bot uses outdated Chrome version

In #8205 we noticed that the Linux bot is running a really old version of Chrome, which prevents us from using ES6 syntax properly. We should therefore update this manually on the Linux bot.
/cc @yurydelendik @brendandahl

Fixed by #8236.


dot.net with permalinks

https://dotnet.microsoft.com/en-us/download/dotnet/7.0 expands the latest minor version by default, but when you want to link a specific version, the page should support permalinks to make it easier for sharing precise links in docs and conversations.

Thanks for your feedback @kasperk81. We're planning to surface this functionality at some point.
/cc @richlander


Build folders created by Vivado

Currently Vivado creates an in_context directory, and synth and impl and created as subdirectories.
I would like to remove the in_context directory, and just have the synth and impl directories, so that the directory structure is consistent regardless of what synth/impl tool is used.

When I run designs, it creates an in_context and ooc directory, both of which contain synth and impl as subdirectories, would I have to differentiate between the in and out of context synth and impl directories by naming them something like ooc_synth/impl and in_context_synth/impl?

@reillymck @KeenanRileyFaulkner @dallinjdahl Probably worth a larger discussion here.
I think I would prefer to just have "synth" and "impl".  Making a new directory for every variation of synthesis tool seems potentially problematic.  For example, suppose one flow wanted to run synthesis with DSPs, but another flow wanted synthesis without DSPs.  Would we then create more "synth_*" sub-directories to distinguish between these synthesis options?
I understand the reasoning for needing separate directories since we moved to a build/design/flow structure, but if that's going to cause us more problems that it's worth.
It seems safer to me to go back to build/flow/design to make sure variations in how different flows may invoke the tools do not cause conflicts that are missed.
Thoughts?

The way this is currently set up is that we treat normal vs ooc as separate tools, which happen to both be implemented by vivado.  This choice is validated due to the significant difference in the outputs and build scripts between the two.
The reason we switched to build/design/flow is so that we can reuse results from previous runs if they are relevant.  That relevance check is done by ninja, and we've taken pains to ensure that the dependency graph is complete.  If it isn't that's a bug.
As an example of this completeness, the flow_args are templated into viv_synth.tcl.mustache to generate viv_synth.tcl.  The synthesis step explicitly depends on viv_synth.tcl, and the templating step depends on the mustache file.  If the flow_args change or the mustache file is updated, everything that depends on that is considered out of date and will be overwritten, so there should be no conflicts.  If you want to keep the results for a given run, those files (under the current paradigm) should be copied elsewhere for archival purposes.  This mirrors the way most command line utilities work on linux.
Another note is that we don't support running multiple flows against a given design in a single run, which means that the flow_args can't change within a run, so a run is guaranteed to be internally consistent.
In conclusion, normal vs ooc is a much larger difference than with/without DSPs, so they are treated as separate tools.  Anything specified as flow_args is done within the respective version (ooc or normal), and build targets are reused when possible, and invalidated when necessary.

For additional insight into the level of completeness, the following link illustrates the relevant aspect:
https://cr.yp.to/redo/honest-script.html
ninja automatically records dependencies on the rules and builds, and we specify all the pertinent files in the dependency list.  That includes build scripts like the tcl file.  Currently the line is drawn at utilities.  Utilities are expected to be present and functional, and we do not depend on them explicitly.
The fuzzy area is in ninja_utils.  We currently treat them as utilities, and they are intended to be written as such, so that they could run as a standalone tool if necessary.  However, we could treat them as build scripts with respect to dependencies.

The fuzzy area is in ninja_utils. We currently treat them as utilities, and they are intended to be written as such, so that they could run as a standalone tool if necessary. However, we could treat them as build scripts with respect to dependencies.

With regards to this, I have an issue open, #336 that will add the ninja_utils python scripts as implicit dependencies wherever they are used by ninja.

I agree with Dallin. And at least in my use case, I generally am running vivado with the same arguments between runs, although the tools afterwards differ. The mustache file would cause the vivado run to be redone if those arguments changed, but as is I think it's useful to have the file structure work on the assumption that vivado runs will be the same vs each flow will require a new run.

If the flow_args change or the mustache file is updated, everything that depends on that is considered out of date and will be overwritten, so there should be no conflicts.

OK, that's good to know.  Given that then I agree keeping it as build/design/flow is ideal.
I would just stick with synth and impl for the usual flow.  I agree that ooc synthesis is different enough to call it synth_ooc, but also, I'm not sure the motivation of calling it something else?  All of the synthesis tools thus far have just used the synth folder, versus having tool-specific naming (eg synth_vivado, synth_icecube, etc).  But if a flow is going to run both ooc and regular synthesis, then I suppose having separate names would be important.
Is there such as thing as out-of-context implementation?

There is such a thing as ooc implementation, it just needs to be ran in the same vivado session as implementation. In my experience, opening an checkpoint from a ooc design post-synthesis and then running implementation does not work. I'm not sure if I tried this with Vivado 2023.1 and I definitely have not tested it with 2023.2


Error parsing some coinbase transactions

Getting errors when running Transaction.fromHex() on some some coinbase transactions, depending on the value of the input scripts. The majority of the coinbase transactions i've tested work, however there are some outliers.
The following testnet transactions are failing failing:

2e5dd635c33562dee6ef8be47d3cd7e23fa9e318e6a26aaf6f88ea6d39141fb6
73e2aa791918b8a9f5d4dd37c3ada0d1362095749e788f41fe0b466043f74fe8
cf10f6e38861cee65d03e0b21c24f2297ed612d621e470ece9703653676b37c1
8f780508d52563d400f7cf03e740b0bd553dc6110c4f5c1417703078b18c4154
946626c46be810e5378148f5a011d706e06ab9450d02219b5acbb2e132e38cfc
83f9cc5a2d13072e5446c1f7bf903491de1b54669ecdabb2126e009ce575f4ea

RuntimeError: unreachable
    at wasm://wasm/0031f49e:wasm-function[1366]:0xb6f35
    at wasm://wasm/0031f49e:wasm-function[960]:0xb1030
    at wasm://wasm/0031f49e:wasm-function[126]:0x53616
    at wasm://wasm/0031f49e:wasm-function[142]:0x59a21
    at wasm://wasm/0031f49e:wasm-function[105]:0x49a6e
    at wasm://wasm/0031f49e:wasm-function[451]:0x90a41
    at wasm://wasm/0031f49e:wasm-function[678]:0xa30d8
    at Function.fromHex (/node_modules/bsv-wasm/bsv_wasm.js:3524:24)

Additionally, the following testnet transactions are failing with a different error

422f2631ee4f87f6b04c7006ed3096c81f1c22a4d6134a0148b02bcdfbf8ef06
917b9fcf0e507c6a11a25b8ef9c81ff95c9cc865234344aaa3c541745faf7583

Error when deserialising Script: OP_IF statement requires an OP_ELSE code

this is fixed in<EMAIL_ADDRESS>or 6 by @deanmlittle


Fix handsoap debug logging in the broker.

Requires https://github.com/ManageIQ/handsoap/pull/1 as well as a new tag on the handsoap_1_2_5 branch.
Prior to the handsoap change, if the client sets the log_header_callback,
but not the log_body_callback, debug would still be set to true. In the
post-response case, this would cause the XML to be pretty formatted for
no reason at all, which can be incredibly expensive. This new code only
does the message processing if the callback has been set.
To leverage these new savings, we must ensure in the broker that we are
not setting the log_body_callback when not necessary.
Broker RSS memory, post handsoap response, with a large environment on
the initial preload:
Before 564.1MB / After 444.8MB / Savings 119.3MB (~21.1%)
@matthewd @jrafanie @dmetzger57 @chessbyte Please review.

Awesome find 😍


The  lakeFS CLA link leads no where....

On the community page. Click it, and you'll see :-)

Fixed after integrating with cla-assistant


'NoneType' object has no attribute 'renew_token'

Hello
Could you explain how to make a refresh request please ? I tried with Basic auth and Bearer Auth but always receive this error.
Thanks for help.
Regards

Hi.
Did you install and setup durin inside your django project ? Your settings.py should have the following:
INSTALLED_APPS = (
  ...
  'rest_framework',
  'durin',
  ...
)
REST_FRAMEWORK = {
    'DEFAULT_AUTHENTICATION_CLASSES': ['durin.auth.TokenAuthentication'],
    ...
}


✨  Allow scoping permission claims by name and/or namespace

Summary
This change allows objects to be claimed by name and/or namespace on the PermissionClaim.ResourceSelector field.
Related issue(s)
Part of #1937

/retest
For both tests:
Error: failed to publish images: error building "ko://github.com/kcp-dev/kcp/cmd/syncer": GET https://cgr.dev/token?scope=repository%3Achainguard%2Fstatic%3Apull&service=ghcr.io: unexpected status code 504 Gateway Timeout
Error: failed to publish images: error building "ko://github.com/kcp-dev/kcp/test/e2e/fixtures/kcp-test-image": GET https://cgr.dev/token?scope=repository%3Achainguard%2Fstatic%3Apull&service=ghcr.io: unexpected status code 503 Service Unavailable

Prow infra issues
 error: error creating buildah builder: reading signatures: downloading signatures for sha256:a68a70657f3ca82bb42b2830574edc9c02ba911a54aa7cb058a170ad845f1b42 in registry.build01.ci.openshift.org/ci/managed-clonerefs: received unexpected HTTP status: 504 Gateway Time-out

/retest


Prod hotfix - Revert #544

Prod hotfix - Revert #544

Close as issue is with other pull request.  Thank you.


i have cloned the react native chime sdk demo and deployed the chime-sdk-js to server. I want to find out the active speaker. what changes i will have to make and in which files to achieve this. Kindly help me out

Hi @rnqayush,
There will be changes needed on both native iOS/Android and also react native. I will take iOS as example and you will find similar files and APIs on Android.

Update ios/RNDemo/MeetingObservers.h and .m to implement ActiveSpeakerObserver and send an event in the activeSpeakerDidDetectWithAttendeeInfo:(NSArray<AttendeeInfo *> * _Nonnull)attendeeInfo method
Register the activeSpeaker observer here
Add a listener in src/containers/Meeting.js to listen to the event sent in step 1 and capture the active speakers and and your own handling logic

Let us know if you have more questions on this

thank you @zhinang-amazon ,
It would be grateful if you could tell it for android too. i have applied the code for active speaker in NativeMobileSDKBridge.kt but getting error shown below.

and this is code i applied.

Hi @rnqayush , could you check if you have imported it by import com.amazonaws.services.chime.sdk.meetings.audiovideo.audio.activespeakerdetector.ActiveSpeakerObserver same for all other classes.
import com.amazonaws.services.chime.sdk.meetings.audiovideo.AttendeeInfo
import com.amazonaws.services.chime.sdk.meetings.audiovideo.audio.activespeakerpolicy.DefaultActiveSpeakerPolicy
?

thanks @zhinang-amazon , it was import issue and got resolved after it

@zhinang-amazon , I have implemented active speaker in my mobile app and working fine for mobile but when I am joining same meeting with web it isn't detecting the active speaker of web.

Hi @rnqayush, Could you provide more details on how you tested the active speaker between mobile and web? When you say web, are you referring to the JS SDK? Is active speaker detected between mobile and mobile? How did you confirm the web client is in the same meeting? Are you receiving any ActiveSpeaker callbacks at all? Are you receiving any other callbacks from web? Do you have logs from mobile side and web you can share?

@dylonChime thanks for your reply. By saying web I was referring to Js sdk.
I have checked the attendees joining and active speaker with the Log.i ()

Actually my microphone was not working last time I have tested it again and now its working correctly as expected. I apologize for this mistake.


Fix RWC runner so that it will read default library file from correct location

@mhegazy @vladima for code-review

Address in #4488


Update make_pretty_igv.py

Fixed the PAM 'nubbs' not being displayed on the igv output. 'full_start' and 'full_stop' need to be set as the start and end of the whole gRNA.

Tested and merged - thanks!


Enhance homepage announcement styling and add form components with re…

…act-hook-form integration

LGTM


UserWarnings on loading tl_calamancy_trf-0.1.0

Given #37, the user may likely manually install spacy-transformers, which will install the most recent version.
The most recent version introduces some deprecation warnings, and a user warning due to changes in the dependency transformer https://github.com/explosion/spacy-transformers/releases/tag/v1.3.1.
Upon loading tl_camancy_trf-0.1.0, we get the traceback:
(my_path_name)\site-packages\spacy\util.py:918: UserWarning: [W094] Model 'tl_calamancy_trf' (0.1.0) specifies an under-constrained spaCy version requirement: >=3.5.0. This can lead to compatibility problems with older versions, or as new spaCy versions are released, because the model may say it's compatible when it's not. Consider changing the "spacy_version" in your meta.json to a version range, with a lower and upper pin. For example: >=3.7.5,<3.8.0
  warnings.warn(warn_msg)
(my_path_name)\site-packages\transformers\utils\generic.py:441: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.
  _torch_pytree._register_pytree_node(
(my_path_name)\site-packages\transformers\utils\generic.py:309: UserWarning: torch.utils._pytree._register_pytree_node is deprecated. Please use torch.utils._pytree.register_pytree_node instead.
  _torch_pytree._register_pytree_node(
(my_path_name)\site-packages\spacy_transformers\layers\hf_shim.py:137: UserWarning: Error loading saved torch state_dict with strict=True, likely due to differences between 'transformers' versions. Attempting to load with strict=False as a fallback...

If you see errors or degraded performance, download a newer compatible model or retrain your custom model with the current 'transformers' and 'spacy-transformers' versions. For more details and available updates, run: python -m spacy validate
  warnings.warn(warn_msg)

Thanks for the report! Let me look into this and the other issues on the weekend. Also feel free to make a PR if you're interested! :)


fix last references to docker image on openzwave/.

@robertsLando you missed a couple of references in #496

I also just noticed that the dockerbuild.yml is actually building dev tag using latest tag :cry: could you fix that? Just replace latest with dev here


[Documentation] Add HPA best practices to review guidelines.

After reviewing the nginx-ingress and spark charts which both have HPA's
defined I am making the following recommendations based on the
nginx-ingress chart specifically (after #7560) be added to the
review guidelines.
Signed-off-by: Paul Czarkowski<EMAIL_ADDRESS>

/lgtm

So I am working on https://github.com/helm/helm/pull/5110, where an hpa template is generated with the helm create command. In the values file I call it hpa rather than autoscaling however because I know that there is also something called a vertical pod autoscaler, which may wind up in values files somewhere eventually... thoughts @scottrigby @paulczar ?


PEX issues #335

This is a 2 PR change fixing https://github.com/tendermint/tendermint/issues/335
1 PR: https://github.com/tendermint/tendermint/pull/370
In this PR:

3 new API functions for addrbook
starting/stopping the book inside the reactor, because the former is the main component of the latter and the reactor should be responsible for starting and stopping of its components

LGTM. While I agree that the reactor should stop/start the addrbook, I wonder if maybe thats not true because the addrbook is passed in on NewPEXReactor. I think it's fine for now but maybe we'll revisit as our peer/seeds/trustedpeers policies evolve

While I agree that the reactor should stop/start the addrbook, I wonder if maybe that's not true because the addrbook is passed in on NewPEXReactor.

I also thought about it and my thoughts were as follows:
PEXReactor does not work properly without an addrbook being started (mostly because changes won't be saved). Coming from the Ruby world, passing an instance of an addrbook is definitely a good thing (aka dependency injection). But if we're giving control over the starting/stopping addrbook, we could run into troubles (if someone forgot to start/stop the addrbook)

I rebased and removed the 2 functions from PexReactor since who ever started the pex should have access to the address book directly and can call these funcs if it needs to


Remove ts namespace import from checker

I'm getting a little tired of auto-imports in checker.ts trying to use the ts namespace import rather than named imports. The import was only used in two places; hasType was a naming conflict (the new name for the internal function IMO is clearer as to what it means), and typeHasCallOrConstructSignatures which is a weird helper defined outside the checker.
I figured we'd do this as part of the "direct import" PR, but I think getting this targeted change in is good to do now.

@typescript-bot perf test this

Thanks for the review!

I'd prefer not to add another method to the checker when there was already a way to do it, but I don't think it's a huge deal.


autoSend is not a function

Hi, I just installed this module. It is linked correctly but when I try to send a SMS I get this error:
"_reactNativeGetSmsAndroid2.default.autoSend is not a function". Why I get this error?
This is my code:
`import SmsAndroid  from 'react-native-get-sms-android';
const instructions = Platform.select({
ios: 'Press Cmd+R to reload,\n' +
'Cmd+D or shake for dev menu',
android: 'Double tap R on your keyboard to reload,\n' +
'Shake or press menu button for dev menu',
});
export default class App extends Component<{}> {
sendsms = () => {
SmsAndroid.autoSend("0966666622", "hello", (fail) => {
console.log("Failed with this error: " + fail)
}, (success) => {
console.log("SMS sent successfully");
});
}
render() {
return (

<Button title="Press" onPress={() => this.sendsms()}/>

);
}
}
`

Hi there. This looks like you're using an older version of the plugin. Make sure you've got the latest version installed.

I tried to update and these are my versions:
"react-native": "0.49.2",
"react-native-get-sms-android": "^1.3.2"
but it doesn't work. I tried to send,delete and list but I always get this error. Why?

@andreatoso I've updated the package on npm. It's now version 1.4.
@davidstoneham Good catch

@briankabiro It works like a charm, thank you for updating! You can close.


Issue due to Axis objects not having a timestamp when loading from a DataSaverLoader

Issue due to Axis objects not having a timestamp when loading from a DataSaverLoader:

2024-06-19 18:31:49,718 - pymodaq.browsing - ERROR - "Attribute ('timestamp') does not exist in node '/RawData/Scan000/Detector000/Data1D/CH00/Axis00'"
Traceback (most recent call last):
File "C:\Users\sebqu\miniforge3\envs\PyMoDAQ_dev\lib\site-packages\tables\attributeset.py", line 546, in getitem
return self.getattr(name)
File "C:\Users\sebqu\miniforge3\envs\PyMoDAQ_dev\lib\site-packages\tables\attributeset.py", line 283, in getattr
raise AttributeError(f"Attribute {name!r} does not exist "
AttributeError: Attribute 'timestamp' does not exist in node: '/RawData/Scan000/Detector000/Data1D/CH00/Axis00'

Proposed fix: adding 'if' block checking the data type before setting it's timestamp:
(I can pull request along with  #332 if the fixes are accepted)
in pymodaq\utils\h5modules\data_saving.py:
class DataSaverLoader(DataManagement):
    ....
    def load_data(self, where, with_bkg=False, load_all=False) -> DataWithAxes:
        ....
        if 'axis' not in self.data_type.name:    # PROPOSED FIX
            data.timestamp = data_node.attrs['timestamp']
        return data

yes that will do, I was just about looking into this when I saw your issue!!

before doing anything, I'll have to look at why no test were covering this. Are you familiar with pytest?

please do it and I will review if tests are needed. Please do a PR including both https://github.com/PyMoDAQ/PyMoDAQ/issues/332 and https://github.com/PyMoDAQ/PyMoDAQ/issues/333

before doing anything, I'll have to look at why no test were covering this. Are you familiar with pytest?

Not yet 😕 sorry.
I did the PR #338, thank you for the review !

I did a PR #341 to add the tests


Add keep operator as deanonymizer

Implements the keep operator for deanonymisation too.
It keeps all the methods from the Keep class and extends only the ones that return the type.
Concrete classes MUST subclass Operator type, since operator discovery is done with Operator.__subclasses__()
This PR fixes issue #1254
Checklist

[x] I have reviewed the contribution guidelines
[ ] I have signed the CLA (if required)
[ ] My code includes unit tests
[ ] All unit tests and lint checks pass locally
[ ] My PR contains documentation updates / additions if required

Hello, I'm just creating this PR in case you (de core dev team) find it interesting. If so I'll pay a visit to the testing and other features

@microsoft-github-policy-service agree

/azp run

/azp run


Lower sidekiq retries to 4 (from default 25)

don't forget :dead => false :)

On staging, retries don't seem triggered at all.


Component overwrites global hass state

Checklist:

[x] I updated to the latest version available
[ ] I cleared the cache of my browser

Release with the issue:
Last working release (if known):
Browser and Operating System:

Description of problem:
The component overwrites global hass object state cause wrong values to be displayed in all dialogs.
I think this lone is the (or one of) culprit.
https://github.com/MindFreeze/ha-sankey-chart/blob/5e64f26943f4984752051b92689323c3a0fbd616/src/ha-sankey-chart.ts#L77

Javascript errors shown in the web inspector (if applicable):

Additional information:

Good catch. Fixed in v1.3.2


[Windows] Small improvements to PowerShell installation scripts

Reset-Node.ps1: "rm -f" will fail because "-f" is ambiguous
Prepare-ServiceInterface.ps1: change taskkill command to print an INFO log
instead of an ERROR log if the process does not exist, to avoid confusing
users

Codecov Report

Merging #2088 (1cd23d1) into main (b118f2d) will decrease coverage by 5.96%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2088      +/-   ##
==========================================
- Coverage   60.97%   55.01%   -5.97%
==========================================
  Files         270      270
  Lines       20366    20366
==========================================
- Hits        12418    11204    -1214
- Misses       6649     7967    +1318
+ Partials     1299     1195     -104

Flag
Coverage Δ

kind-e2e-tests
40.34% <ø> (-11.31%)
:arrow_down:

unit-tests
41.37% <ø> (ø)

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

Impacted Files
Coverage Δ

pkg/apis/controlplane/helper.go
25.00% <0.00%> (-75.00%)
:arrow_down:

pkg/apis/controlplane/v1beta2/helper.go
25.00% <0.00%> (-75.00%)
:arrow_down:

pkg/controller/networkpolicy/mutate.go
0.00% <0.00%> (-71.77%)
:arrow_down:

pkg/controller/networkpolicy/store/group.go
5.00% <0.00%> (-70.00%)
:arrow_down:

pkg/controller/networkpolicy/validate.go
0.00% <0.00%> (-69.47%)
:arrow_down:

pkg/k8s/name.go
33.33% <0.00%> (-66.67%)
:arrow_down:

pkg/agent/controller/networkpolicy/packetin.go
6.20% <0.00%> (-65.52%)
:arrow_down:

...formers/externalversions/security/v1alpha1/tier.go
0.00% <0.00%> (-64.29%)
:arrow_down:

...ers/externalversions/core/v1alpha2/clustergroup.go
0.00% <0.00%> (-64.29%)
:arrow_down:

...s/externalversions/core/v1alpha2/externalentity.go
0.00% <0.00%> (-64.29%)
:arrow_down:

... and 60 more

/test-all


Pixel: Add tab activity stats to a daily pixel

Task/Issue URL: https://app.asana.com/0/1207418217763355/1208414748460831/f
Description
This PR adds the following parameters to the m_tab_manager_clicked_daily pixel:

tab_count: The number of open tabs
tab_active_7d: The number of tabs access within the last 7 days
tab_inactive_1w: Tabs not accessed for more than a week, but less than 2 weeks
tab_inactive_2w: Tabs not accessed for more than 2 weeks, but less than 3 weeks
tab_inactive_3w: Tabs not accessed for at least 3 weeks

All stats are reported within buckets due to privacy reasons:

A new column was added to the TabEntity — the lastAccessTime, which is updated every time a tab is activated.
Steps to test this PR
DB migration

[x] Run the app with existing data with a previous DB version
[x] Verify the migration to version 56 succeeds (no crashes)

Pixel firing

[x] Run the app
[x] Open the tab switcher
[x] Verify the m_tab_manager_clicked_daily pixel is fired with all of the parameters mentioned above

Last access time updates

[x] Run the app and access a couple of different tabs
[x] Open the App Inspector in AS
[x] Select the DDG process
[x] Select the app.db database
[x] Select the tabs table
[x] Verify the lastAccessTime was updated for the access tabs using correct time

@karlenDimla I’ve addressed your comments in the code.


Add AnyDictionary Abstraction and Take ArrayRef in DictionaryArray::with_values

Which issue does this PR close?

Closes #.
Rationale for this change

https://github.com/apache/arrow-datafusion/issues/5471#issuecomment-1675875694
What changes are included in this PR?

Are there any user-facing changes?

Integration test failure seems unrelated, will see if sorts itself out tomorrow, if not I'll raise an upstream report


troubleshooting: Fix OpenSSL version section

https://github.com/ome/omero-install/pull/197 updated the command, but the surrounding text wasn't updated.

Thanks @manics


handle missing prop data

Motivation
#356
Fills missing property data based on data type:
if float, missing data is NaN
if str, missing data is ''
if int and contains some missing data, promote to float and missing data is NaN

@bendichter Would it be possible to add specific tests for this behavior?

@bendichter Thanks, just waiting for tests to finish.
I know this kind of thing was resolved somewhere in the #348 changes, but pinning it down is tricky. I'm sure that'll come up again in the piecewise refactors but this is good as a quick standalone fix for our current purposes.

@bendichter Oh my, tests are actually failing now: https://github.com/catalystneuro/nwb-conversion-tools/runs/4719902648?check_suite_focus=true
Looks like a simple indexing issue though, should be easy to fix.


could not import github.com/go-sql-driver/mysql (cannot find package "github.com/go-sql-driver/mysql" in any of

Error:
could not import github.com/go-sql-driver/mysql (cannot find package "github.com/go-sql-driver/mysql" in any of
C:\Program Files\Go\src\github.com\go-sql-driver\mysql (from $GOROOT)
C:\Program Files\Go\src\github.com\go-sql-driver\mysql (from $GOPATH))
Code:
`package main
import (
"database/sql"
"fmt"
"html/template"
"net/http"
_ "github.com/go-sql-driver/mysql"

)
func main(){
...
}
`
How to solve this? I saw the given problem, but the solution didn't help. I reinstalled Go, I put a different $GOPATH but nothing helps

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Push Notifications don't work properly O365

Note: I've attempted several times to work with Office 365 support unsuccessfully to resolve this.
If I use passwordless logon with outlook.com (microsoft account) when an auth is required, there is a push notification on the device paired (in my testing either an iPhone XSMax (with and without a paired Apple Watch 2 with the authenticator app) and iPad Pro) with that account.  You click on it and must unlock said device and you are then prompted to select the proper code.
When attempting to do the same with a corporate account (O365) there is no push.  Rather no real notification.  However if you unlock said paired device and open the Authenticator app, you will get a list of the codes to choose from, but it takes a few extra seconds or in some cases requires a pulldown refresh.
I've tried unregistering devices, turning off MFA on the account and turning it back on as well as disabling the tenant and reenabling it and setting up MFA and converting it to passwordless.
I'd love to find the right team to work with to figure out if there is some error or if this is a bug of some sort.  I can provide much more detail if I find the right avenue.

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

ID: ee6ac70d-5853-6a3a-0964-23b46910b4c4
Version Independent ID: a5beda87-9845-8807-6221-7fdc64079587
Content: Password-less sign-in with the Microsoft Authenticator app (preview) - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-authentication-phone-sign-in.md
Service: active-directory
Sub-service: authentication
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@dwatsonit Thank you for your query. We will investigate and update further on this.

Do you get the notifications when you select "check for notifications"?

Affirmative

Hi @dwatsonit,
If you send me an email at<EMAIL_ADDRESS>I can open a support case. I am uncertain if this is a bug and since I don't have a testing environment for this can't confirm.

Closing this out as moving to email/support. I will update the thread when we have a resolution.

@dwatsonit I am seeing the exact same behaviour in my tenant. Did you find a solution to this yet?

@rviljoen  I forwarded all the information to MSFT outside of this thread but have not heard anything more regarding a solution as of yet.

Seeing the same behaviour here, going to raise this with support as well.

Just in case someone else finds this, I resolved this issue by going into the Microsoft Authenticator app, choosing to "Disable phone sign-in" for my AAD Work Account, and then enabling it again right after. This updated the icon in the https://myprofile.microsoft.com portal from a normal MS Authenticator padlock icon, to a phone sign-in icon like you see in the app itself. After about half a day, the push notifications then started working for the passwordless sign-in flow.

@MarileeTurscak-MSFT was there ever a solution to this? Passwordless sign-in is working for me but I don't get any push notifications in the app - push notifications for approve/deny are working fine

@MarileeTurscak-MSFT was there ever a solution to this? Passwordless sign-in is working for our team but we don't get any push notifications in the app - push notifications for approve/deny are working fine

Not that I was ever given. O365 generic support (meaning I do not pay for premier/classic or unified support) said this was by design.  It’s a pretty crummy UX the way it is IMHO but that was the best I could get.

We got the fix for you!! Seems “Azure Multi-Factor Auth Connector” was disabled on our accounts. This will allow you to get the push notifications for the number matching challenge without the need to manually poll by refreshing the pulling the app.

Open PowerShell and use “Connect-MsolService”
Login with Global Administrator Account
Run “Get-MsolServicePrincipal -AppPrincipalId 1f5530b3-261a-47a9-b357-ded261e17918”
If it is False, run “Set-MsolServicePrincipal -AppPrincipalId 1f5530b3-261a-47a9-b357-ded261e17918 -AccountEnabled:$true”
In the Authenticator App on your phone, click the arrow to run “Update phone sign-in” again, the exclamation mark should go away

At this point you can close the Authenticator App and try to login to your Office 365 account. It should start prompting you without manually opening the App!
Hope that helps!
reference
https://www.jasonsamuel.com/2019/03/04/how-to-setup-password-less-phone-sign-in-authentication-with-microsoft-authenticator-azure-ad-and-citrix-workspace/

@aollivierre thanks, this worked!


Coding error in SnackbarEvent constructor

If I'm not completely mistaken, there is a coding error in JFXSnackbar.java:
Line 389 should be: this(message, null, actionText, timeout, persistent, actionHandler);
instead of this(message, null,null, 3000, false, null);

Hello,
Definitely, thanks for pointing out the issue 👍
Regards,


feat(data-classes): Add KafkaEvent and KafkaEventRecord

Issue number: #1484
Summary
This change adds a data class for Kafka event sources either from self-managed Apache Kafka or Amazon MSK.
Changes
A new data class KafkaEvent is introduced which includes a records property/generator. The returned KafkaEventRecord class has a variety of useful properties for decoding the record key, value and headers.
User experience
from aws_lambda_powertools.utilities.data_classes import event_source, KafkaEvent

@event_source(data_class=KafkaEvent)
def handler(event: KafkaEvent, context: LambdaContext):
	for record in event.records:
		print(record.decoded_key, record.json_value)

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
Not a breaking change
RFC issue number: ?
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

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

View rendered docs/utilities/data_classes.md

Hi @lyoung-confluent! Indeed this is an awesome PR! Thanks a lot for taking the time to help improve the project! 🥇
I'm going to start doing a review on this PR today and let you know if we need to make some small changes.

Codecov Report

Merging #1485 (bd9c8a6) into develop (2cf38cc) will decrease coverage by 0.40%.
The diff coverage is 92.85%.

@@             Coverage Diff             @@
##           develop    #1485      +/-   ##
===========================================
- Coverage    99.89%   99.48%   -0.41%
===========================================
  Files          124      126       +2
  Lines         5588     5672      +84
  Branches       634      661      +27
===========================================
+ Hits          5582     5643      +61
- Misses           2       14      +12
- Partials         4       15      +11

Impacted Files
Coverage Δ

...a_powertools/utilities/data_classes/kafka_event.py
92.75% <92.75%> (ø)

...mbda_powertools/utilities/data_classes/__init__.py
100.00% <100.00%> (ø)

...bda_powertools/utilities/data_classes/alb_event.py
90.47% <0.00%> (-9.53%)
:arrow_down:

aws_lambda_powertools/__init__.py
100.00% <0.00%> (ø)

aws_lambda_powertools/utilities/batch/__init__.py
100.00% <0.00%> (ø)

aws_lambda_powertools/event_handler/api_gateway.py
100.00% <0.00%> (ø)

...ws_lambda_powertools/utilities/batch/exceptions.py
100.00% <0.00%> (ø)

...lambda_powertools/utilities/data_classes/common.py
100.00% <0.00%> (ø)

.../utilities/data_classes/api_gateway_proxy_event.py
100.00% <0.00%> (ø)

aws_lambda_powertools/utilities/batch/sqs.py

... and 2 more

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

Looks good to me, thanks for the quick feedback/review @leandrodamascena!


Update branch in deploy-docs.sh.

The source branch of druid-io/druid-io.github.io was changed from "master" to "src" recently.

👍


Adding support for JFACT?

From<EMAIL_ADDRESS>on February 08, 2013 08:24:35
I know that supporting FacT++ has historically been a pain.  We've found JFACT to be easier to get working consistently with the OWL-API.  This performs better than HermiT does with our ontologies, so would be great if you could integrate it into the list of available reasoners.
Original issue: http://code.google.com/p/owltools/issues/detail?id=50

From<EMAIL_ADDRESS>on February 08, 2013 10:26:47
Okay. That was easy. JFact 1.0.0 is available as maven artefact.
I have added the options for JFact to owltools and OORT CLI (not in the GUI).
Status: Started
Owner<EMAIL_ADDRESS>Labels: -Type-Defect Type-Enhancement


My title edited

My comment

pouet

super

?

test

test

test

@kadocbot could you receive me?

@kadocbot ?

@RoukTest foo

@kadocbot test

@kadocbot pouet

@kadocbot pouet

@kadocbot merge


panic: etcdclient: old cluster version

Here's the error occured when starting zrpc server:
2023-11-24 13:46:07.438 CST	 error 	etcdclient: old cluster version	caller=zrpc/server.go:88
panic: etcdclient: old cluster version

What kind of behavior you prefer?

same problem,
It looks like to upgrade the etcd sever version.

same problem, can someone give the solve method.

upgrade your etcd server version (>= 3.4.0)

upgrade your etcd server version (>= 3.4.0)

My etcd version is 3.5.13, but this error is still reported

upgrade your etcd server version (>= 3.4.0)

My version is the latest, but this problem is still reported

why don't support < 3.4.0 ?


use of console.logs causes issues

https://github.com/kratiahuja/broccoli-tslinter/blob/master/index.js#L95 for example, prevents output sensitive tools like test output processors from working correctly.
Let me recommend instead https://github.com/visionmedia/debug

@stefanpenner Thank you for catching that! I'll fix this sometime this week.


Validation of getFees directCommad arguments is wrong

Describe the bug
On validateCommandAPIParameters the check is returning 'true' instead of 'false' if 'ddo' or 'serviceId' are missing
else if (command === PROTOCOL_COMMANDS.GET_FEES) {
      if (!requestBody.ddo || !requestBody.serviceId) {
        return {
          valid: true
        }
      }

closed on https://github.com/oceanprotocol/ocean-node/pull/194


Update npm diff project dependency

= 3.5.0

Already at 3.5.0 in package.json:
https://github.com/microsoft/vscode-python/blob/640a62b072a4d74b42e24f5fc927e2f4cd030a9e/package-lock.json#L6629-L6633


Allow for or extend checking connectedness of the provider

Is your feature request related to a problem? Please describe.
I want to properly check the connectedness status of the provider, I am unsure if the connected flag of the WebrtcProvider is properly representative of this status, as it is true if the room is created, but not if any networking has been done.
Describe the solution you'd like
One of the following come to mind:

The connection being properly represented as a flag so I can check it with e.g. promise.until from lib0
Connection events to resolve new promises
An async function in the provider connectedAsync that resolves if connected at call time or when the connection is established

Describe alternatives you've considered
Checking the connected flag, but as stated earlier, I am not sure if this is fully representative of what I would expect.
Additional context
I am not certain if a connection status can generally be determined in this configuration and if I am asking for a feature that cannot be implemented.

I think the core issue is the I am having problems when I want to dynamically create and destroy providers but run into issues that a room is already in use because I cannot properly determine when all the room and connection management is completed.


Getting: Invalid volume signature! for jetson-tx2/2.13.0+rev1

<EMAIL_ADDRESS>

Ubuntu 16.04 x86-64
$ sudo resin local configure resin.img
 Invalid volume signature!

If you need help, don't hesitate in contacting us at:

GitHub: https://github.com/resin-io/resin-cli/issues/new
Forums: https://forums.resin.io

tried it with Raspberry Pi 3 img and it didn't reproduce
 Front conversations

@telphan Any idea what's up here? Has anything changed in the partition layout for the TX2 recently that might have started causing this?

@pimterry the boot partition on the tx2 is 12.
I think the CLI should adapt to that, probably still uses partition 1 for this board?

the boot partition on the tx2 is 12.

@floion is that true?  https://api.resin.io/config for the tx2 has:
"configuration": {
	"config": {
		"partition": {
			"primary": 1
		},
		"path": "/config.json"
	}
}

and parted <tx2 2.13.0+rev1.prod> gives me:
Number  Start   End     Size    File system  Name         Flags
 1      4194kB  88.1MB  83.9MB  fat16        resin-boot   boot, esp
 2      88.1MB  3238MB  3150MB  ext4         resin-rootA
 3      3238MB  3242MB  4194kB  ext4         resin-rootB
 4      3242MB  3246MB  4194kB               resin-state
 5      3246MB  3262MB  15.7MB               resin-data

I've had a look at this - it actually does seem to work with os configure (I think on partition 1), and just fails with local configure. I think the issue is that we're using an old version of reconfix (and thereby resin-image-fs) in there.
I'm going to update that and do some testing, but it looks like that should fix it, unless I'm missing something?


Panel doesn't handle multiple renders during animations well

https://github.com/limefrogyank/BlazorFabric/issues/27
Turns out this bug in Panel was causing the weird errors in the new version of ResponsiveLayout

Seems to actually be a bug in Panel. When Panel gets re-rendered during a close animation, that triggers the flicker. Yes, there might be one extra unnecessary render call from ResponsiveLayout, but that shouldn't affect the Panel if it were working properly. It's working better now, but still sometimes shows the flicker.

I can't get it to flicker anymore.

Aha!  It only does it in Debug mode with Server-side blazor.  In Release mode, it's fine.

I'm going to close this for now


http://mirror.envygeeks.io/ is down

FROM jekyll/jekyll:3.2.1
that uses alpine
it fails fetching from:
http://mirror.envygeeks.io/alpine/v3.5/main/x86_64/APKINDEX.tar.gz

forget this, my bad, it uses
https://hub.docker.com/r/envygeeks/alpine/


Rangeslider problem when min and users min are 0

In the case users min and the min are 0 the tracker/button for min dissapears.

Can you provide a fiddle please?


Issue: Python Version Compatibility with Jetson Orin AGX 64G and Text Generation WebUI

Environment Details
Hardware: Jetson Orin AGX 64G
PyTorch Version: Python 3.8, aarch64 compatible
Container Used: text-generation-webui from your containers.
Issue Description
After installing the prerequisites, I executed the following command to run the Docker container for text-generation-webui:
user@ubuntu:~/Desktop/jetson-containers$ ./run.sh $(./autotag dustynv/text-generation-webui) /bin/bash -c "cd /opt/text-generation-webui && python3 server.py --model-dir=/data/models/text-generation-webui --model=Intel_neural-chat-7b-v3-1 --loader=transformers --api --bf16 --verbose --listen --extension openai --auto-devices"
However, I encountered this error:
2023-11-24 13:05:12 ERROR:Failed to load the extension "openai". Traceback (most recent call last):   File "/opt/text-generation-webui/modules/extensions.py", line 36, in load_extensions     exec(f"import extensions.{name}.script")   File "<string>", line 1, in <module>   File "/opt/text-generation-webui/extensions/openai/script.py", line 27, in <module>     from .typing import (   File "/opt/text-generation-webui/extensions/openai/typing.py", line 8, in <module>     class GenerationOptions(BaseModel):   File "/opt/text-generation-webui/extensions/openai/typing.py", line 9, in GenerationOptions     preset: str | None = Field(default=None, description="The name of a file under text-generation-webui/presets (without the .yaml extension). The sampling parameters that get overwritten by this option are the keys in the default_preset() function in modules/presets.py.") TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'
Possible Cause
Upon investigation, it seems the issue stems from the text-generation-webui Docker container using Python 3.10+ syntax, while only Python 3.8 is available in the Docker. This discrepancy leads to the above error. An attempt to rewrite the typing part in Python 3.8 resolved this error, but embedding still did not function correctly.
Request for Assistance
Could you please verify and address this compatibility issue? Any help in resolving this would be greatly appreciated.
Thank you in advance for your assistance.
Best Regards,

Up please

Up please

jetpack 6 is coming with python 3.10(ubuntu 22.04). Release today


[Update] Spore got update (10.2024)

Describe the bug
A clear and concise description of what the bug is.
Can't launch Spore with mods via Mod Manager
Screenshots
If applicable, add screenshots to help explain your problem.

Platform (please complete the following information):

OS: [e.g. Windows 8.1, Kubuntu Linux 20.04]

Windows 10 x64 22H2
Additional context
Add any other context about the problem here.
Bug occured because of game update. Therefore need to update Mod Manager, please.

@VanillaCold see above


⚠ ClamAV has degraded performance

In 42d6629, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 78 ms

Resolved: ClamAV performance has improved in fa16fba.


the project has an open source plan?

This product is what I have been looking for. Thank you very much for your efforts. I am interested in this project， if the project has an open source plan?

Unfortunately, this seems to be proprietary software just using Github as ticketing system… :disappointed: See: #210

Unfortunately, this seems to be proprietary software just using Github as ticketing system… :disappointed: See: #210


Calculator does not render well on Firefox

Fix html so the CF table renders properly on Firefox

Fixed in https://github.com/TEC-Toolkit/Semantic_Machine_Learning_Impact_Calculator/commit/c692a2b359390bfb9c53cb7e8c7a701ddff8e193


Change to use enum instead of enum class for MessageLevel.

This allows us to create C interface with MessageLevel later.

Looks ok, even though it seems weird to qualify the enumerant name with the enum type name.  ?!  A corner I'm unsure about.
You get an LGTM if both bots pass.


fixed the errors in deploying.rst and i18n.rst

Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[ ] Run make lint from the Wagtail root.

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

thoughts ?

Nice finds @0saurabh0  - merged in, thanks for the fixes!

thanks @lb-  it's been my pleaseure


added measurements to availability and dependency, renamed dependency…

added measurements to availability and dependency
updated availability schema
renamed dependencyTypeName to type
deprecated handledAt

:shipit:

Tests failing as Breeze do not support some of those changes yet. Working on it


Upload project sources and submit archives to GCS

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
Upload project sourced for build and ipa/aab for submits to gcs
How
Test Plan
Run build, build with auto submit, submit with path option, submit from existing build

/changelog-entry chore Upload project sources for EAS Build and archives for EAS Submit to GCS.


proposal: Add package.json like support on go.mod

As, many of you already know, in javascript, when using "npm init" command, it asks several questions for project level metadata. Also, developers can add custom commands in "script" section. Some other features like "repository, config" etc.
Can Go have a similar feature when using "go mod init" command? What do you guys think?
Or, in summary, can we have more rich features for go modules?

If I understand this correctly, it seems to me that a more Go-like approach would be options for go mod init.
In any case, can you give some specific examples of values that would be set?  I don't understand your example of "repository, config."  Thanks.

It appears to be described here, in particular repository and config. ISTM these are things that just work with Go as part of the go tool, or are part of Go syntax.

We aren't going to add questions to go mod init.  In general we've decided that we won't store arbitrary configuration data in the go.mod file.  Closing this issue.

We aren't going to add questions to go mod init.  In general we've decided that we won't store arbitrary configuration data in the go.mod file.  Closing this issue.


Adds flag to clean up cilium state before startup

test-me-please

test-missed-k8s

test-me-please

test-me-please

test-missed-k8s

test-me-please

test-missed-k8s

test-me-please

test-me-please

test-me-please

test-me-please

submitted another PR using a different branch. This new PR is merged.
Closing this PR as its not required.


Setting Default role with Yii2-user

How do I modify Yii2-user such a way that I can set  default user role in Yii2-rbac . ??

At the moment you can override beforeSave method of User model and set the role there

Sorry, but can you write, what i must override in that method?

I know this old, but i spent some time solving this one, and this is what i used.  You can use the authManager component to define defaultRoles.
http://stackoverflow.com/questions/28936747/rbac-yii2-doesnt-work-with-default-roles
'authManager' => [
    'class' => 'yii\rbac\DbManager',
    'defaultRoles' => ['admin', 'author'], // your define roles
],

Добрый день!
Использую Ваши компоненты yii2-user & yii2-rbac.
Не пойму как назначить роль по умолчанию пользователю который авторизуется через соц. сети?

AuthManager is used by RBAC, so when setting that component up under in config, you can specify the default role that AuthManager object apply to the logged in user.  You wont see any entries in the RBAC tables as it looks to be an just an object property.

This is how I solved it
public function afterSave($insert, $changedAttributes)
{
        parent::afterSave($insert, $changedAttributes);

        if ($insert) {
            $this->getDb()
                ->createCommand()
                ->insert('{{%auth_assignment}}', [
                    'item_name' => \app\helpers\RoleHelper::SUBSCRIBER,
                    'user_id' => $this->id,
                    'created_at' => time(),
                ])
                ->execute();
        }
}

I don't know if it's the proper way since I cannot do it in beforeSave.


First pass on Info

Checks

I have...

[x] read and understood the Contributing Guidelines
[x] Updated any nessecary files such as the README.md and/or CHANGELOG.md.

Type of Pull request

[ ] Minor Change
This Pull request doesn't break existing configuration.
[ ] Major Change
This Pull request will break existing configuration.
[ ] Bug fix
This Pull request will fix a (critical) bug.
[x] Documentation
This Pull request only changes documentation (README.md, CHANGELOG.md, etc.)
[ ] Other: __________

Description
First pass on updating the README with valuable information.
Things left to do:

[ ] Add Workflow structure

Closes #

@Andre601 Can we provide an explanation on how this repository is linked with the Readme-Icons repository? The default replacers for readme icons will be added in #2 .

Not really sure it's worth to mention this...
At most maybe a "See also" section that just links to the other projects

@Andre601 Should we merge this? Or is there anything left to be added?
Asking because there's still in progress label here.

@Andre601 Should we merge this? Or is there anything left to be added?
Asking because there's still in progress label here.

I think there is, because Andre added Add workflow structure in the description.

I forgot to update the description. It should now be ready, but I would request you to quickly check the workflow setup for if it would be the right syntax, or if it would require some changes.

env:
	GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

@Andre601 We might need token to commit in repo. Refer this file

I forgot to update the description. It should now be ready, but I would request you to quickly check the workflow setup for if it would be the right syntax, or if it would require some changes.

@abhijoshi2k Check the syntax yourself, and comment if OK.

env:
	GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

@Andre601 We might need token to commit in repo. Refer this file

done


elchecking/example: add support for MokListTrusted variable

Newer Shims measure if the kernel should trust the keys stored in MokList.
Currently requires a tpm2-tools version with this patch applied: https://github.com/tpm2-software/tpm2-tools/pull/3144

Codecov Report

Merging #1126 (3b3f92d) into master (1951b34) will decrease coverage by 0.16%.
The diff coverage is 100.00%.

Additional details and impacted files

Flag
Coverage Δ

packit-e2e
72.26% <100.00%> (-0.17%)
:arrow_down:

testsuite
54.26% <0.00%> (-0.01%)
:arrow_down:

unittests
14.65% <0.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

keylime/elchecking/example.py
94.00% <100.00%> (+0.06%)
:arrow_up:

keylime/revocation_notifier.py
52.20% <0.00%> (-9.44%)
:arrow_down:


no matching versions for query "v8"

go get github.com/hdt3213/delayqueue@v8
go: github.com/hdt3213/delayqueue@v8: no matching versions for query "v8"

It is beacause https://github.com/golang/go/issues/57122
I would fix this issue tonight.


ReTerraForged Compat (found working solution)

Add compatibility with the 1.20.1 version of ReTerraForged
Additional Context
https://github.com/racoonman2/ReTerraForged

UPDATE: works with fork https://github.com/GamingCrew69/ReTerraForged/tree/1.20.1
The original repo didn't build, but after copying the offending mixin file from the main fork of RTF it has worked perfectly so far.

@ColossalRenders
Don't mind me but would you perchance have a copy of this version of RTF that functions with Strata? I am no modder myself so I have a feeling if I tried to build the file it would probably explode lol.
Appreciated!


Fix bugs, when multiple DCs have instances of the same id, Syner over…

Fix bugs, when multiple DCs have instances of the same id, Syner overwrites the registered instance
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run go build go test go fmt go vet to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

LGTM


Disable Helm kube-controler and scheduler on GCE

What did you do?
On Google the kubernetes is managed by your. And i can't have access to this component.
Do you know if it exists a solution to have those metrics; or to disable it on kube-prometheus charts?

just delete the alerts if you don't want them :slightly_smiling_face:


[android] Lockscreen playback notification has no controls

Samsung S5 Android 6.0.1

Ukryta zawartość = hidden content

what do you suggest we do here?
Should we just make it unavailable when the screen is locked, thus protecting the user's privacy?

CC: @marcelinkaaa

Well, I'd at least add the pause button - The music is still playing (So protecting the user's privacy by not displaying what is playing is weird). We need to unlock the screen to change that (and that's a hassle if someone has an elaborate swipe pattern).

@grzesiekrzaca Check your settings. Some content like messages is hidden on lock screen for privacy. It could be similar with players. On my phones I am getting the standard player we have (although I agree the player itself could have more options - I will upload what I mean to your ticket about that).

Yeah, I guess it's the android version thing and they changed something in permissions again. I can see other players on my lock screen just fine. Which android do you have?

HTC M8 and LG Nexus - both running Android 6

Hmm, weird. All other players (default Samsung one and MX Player)  have normal controls in the notification on the lockscreen but FW notifications have their contents hidden.

I just downloaded bunch of other players and all of them are open on lockscreen including FW.
Will ask others what they have. Maybe its phone related only or maybe something with permissions on our end?

Thank you for noticing, suggesting and fixing this issue @grzesiekrzaca


Add GoCardless

Awesome job putting this list together!
I've added GoCardless (https://gocardless.com/) as we have a variety of articles and incident reviews that might be useful.

Fwiw, I think the build failures are happening on main too?

Thank you @lawrencejones  🙌


maps-app FATAL EXCEPTION for Android version 6.0.1 due to SecurityException

Device Nexus 5, Android 6.0.1
On start, the application immediately throws the following exception because as of Android 6, apps needs to explicitly ask for permissions even though the permissions are already in the manifest.  See this doc.
FATAL EXCEPTION: main
Process: com.esri.android.mapsapp, PID: 28375
java.lang.SecurityException: "network" location provider requires ACCESS_COARSE_LOCATION or ACCESS_FINE_LOCATION permission.
at android.os.Parcel.readException(Parcel.java:1620)
at android.os.Parcel.readException(Parcel.java:1573)
at android.location.ILocationManager$Stub$Proxy.getLastLocation(ILocationManager.java:717)
at android.location.LocationManager.getLastKnownLocation(LocationManager.java:1200)
at com.esri.android.map.LocationDisplayManager$a.a(SourceFile:894)
at com.esri.android.map.LocationDisplayManager$a.a(SourceFile:810)
at com.esri.android.map.LocationDisplayManager.start(SourceFile:131)
at com.esri.android.mapsapp.MapFragment.startLocationTracking(MapFragment.java:1107)
at com.esri.android.mapsapp.MapFragment$4.onStatusChanged(MapFragment.java:549)
at com.esri.android.map.MapView$1.b(SourceFile:431)
at com.esri.android.map.MapView$1.handleMessage(SourceFile:354)
at android.os.Handler.dispatchMessage(Handler.java:98)
at android.os.Looper.loop(Looper.java:148)
at android.app.ActivityThread.main(ActivityThread.java:5417)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

This is due to an invalid site redirect URI.  To configure a redirect URI

Sign into ArcGIS for developers
Go to the Authentication tab for your maps-app application on ArcGIS for Developers
Add a new redirect URI to the list at the bottom of the page.  These redirect URI's represent a valid place that a user can be redirected to after a successful sign on.

You can also configure a redirect URI on ArcGIS Online or your own on-premise portal.  You cannot use a redirect URI that begins with file://.

@zinfin We should better handle the exception when attempting to log in w/o a redirect URI in place.

Verified fixed with #126


Graph based droppability

Restructures a lot of old code having to do with parenthesis wrapping and droppabilty. Things that were changed:

Removes the old precedence, socketLevel, and classes fields, replacing them with parseContext, nodeContext, and shape.

parseContext must always be a string that can be passed to the parser indicating what AST node to start the parse at when reparsing this node.
nodeContext must be an instance of NodeContext, and has information indicating how to un-paren-wrap this node.
shape is one of the old socketLevel constants and represents whether to put a nubby in on the block.

Removes the old XML-based serialization and replaces it with a new JSON-based serialization which is easier to modify.
Changes the socket reparse behavior to add back the feature where saying a * [b + c] in that socket will add parentheses around (b + c). (TODO add test for this)
Changes reparse behavior to reparse less things less often to improve performance.

Perhaps parseContext can be named as parseContextType to reflect that it's a string type not a NodeContext object?

I'm adding a few more high-level questions that I wanted to ask in the meeting:

Is the use of Dijkstra (or DFS) essential for estimating the droppability in real-time?? For performance, maybe we could flatten the tree with all possible children (as array) for each rule, if the paths are not so complex. Also it seems not using the path cost to make use of the Dijkstra (UCS) algorithm.
How can we go about generating the grammar trees for Python, JS, etc.?

Hmm, you're right. What do you think good names would be for parseContext and nodeContext? parseType and innerType, maybe or parseType and unwrappedInfo? parseContextdescribes the outermost parse context used for the block and is used during reparsing, whilenodeContext` describes how to completely unwrap the block of all its parentheses.
I do not know if Dijkstra is essential; the graphs usually do not have a lot of edges so I think performance is not a huge problem (the library I found that finds shortest paths just happens to use it).
It uses the path cost when it figures out how to wrap in parentheses; it seeks to wrap using the fewest number of parenthesis rules possible. For instance, when a + b is dropped into a statement context where it needs to become a + b;, it could theoretically also wrap (a + b);, but the extra () are unnecessary, so we want the least cost path. Not sure whether to pre-flatten -- @davidbau any thoughts?
Right now CoffeeScript and JS still use custom droppability rules, implementing the drop(node...context) function themselves. If we wanted to transition to the new graph-based droppability, we would probably have to write the graph by hand (and possibly add more specific type labeling in the parsers). Depending on how complex we discover the languages to be, this might be easy or difficult.

How about "outmostClass" (for "parseContext") and "classComponents" (for "nodeContext")? I tried to avoid the words "context" and "type" that are already used in other places with (I think) different implications.

Sorry, I overlooked that you are using the shortestPath() with costs involved. That probably justifies the use of tree and searching.

@dabbler0 If you have "example-c.coffee," do you mind checking it in to whichever the up-to-date branch? Thanks!


Add description in the meta categories file

You can now add a meta description to the categories, to be used in the templates, for example in the homepage to further add details to each group of links.

Looks good, merging. Thank you!

Thank you! :)


Consolidate Page Options for Rendering?

Describe the problem
Page options for rendering use three separate variables, i.e. export const prerender, export const ssr, and export const csr. The issue is that it is possible to specify multiple of these options at the same time, which doesn't have one clear interpretation.
Describe the proposed solution
If there was just a single variable, e.g. export const render that can be one of "prerender", "ssr" or "csr", it would not be possible to specify two conflicting options at the same time.
Alternatives considered
No response
Importance
nice to have
Additional Information
No response

Whoops, I think I misunderstood the usage of these options from the docs.


#19 Add cookie extractor

Coverage decreased (-0.4%) to 99.642% when pulling 6905ee95325c2b39a872ce38ed686205b86df294 on feature/19-cookie-extractor into 5bb3319455bd2eb2d897074d2ffe6b55a3b1a883 on master.

Coverage decreased (-0.4%) to 99.642% when pulling 6905ee95325c2b39a872ce38ed686205b86df294 on feature/19-cookie-extractor into 5bb3319455bd2eb2d897074d2ffe6b55a3b1a883 on master.

Coverage is OK. I don't know why coveralls blocking here.


tab panel fixes

fixes the following issues

VoiceOver reads the tab header as presentation
VoiceOver reads the tab name twice
Focus indicator not visible for focused tab header
Should be able to navigate between tabs using arrow keys

Focus indicator for tab under the group header - added more pixels to the group header bottom border so that it is not overlapping the focus indicator

Focus indicator is now visible for active tab

How screen reader reads the tab - previous it is being read as: presentation Home Home tab...


Added a note on usage of textarea element

Maybe everyone else just knew about how to properly use textarea elements, but I spent half an hour trying the wrong way and got bit frustrated with them.
Setting the text area contents through child elements fails when the textarea elements are removed.  Regardless of which element is removed, the content of the last text area vanishes and the remaining text areas get the previous contents in the original order. That is, the textarea contents jump from one element to another.

Is there ever a good case to put child nodes in a textarea, or should that argument be entirely removed?  I've never seen it done nor done it myself, and removing the List of Html children would prevent this error in the future.  If someone really needed to attach children they could just call node "textarea" anyway as an advanced case.

I would recommend changing the textarea to omit the child nodes.
The spec says that "The textarea element represents a multiline plain text edit control. The contents of the control represent the control's default value." but in context of elm, I think the default value is better supplied in other ways.

This documentation is helpful (to me, found tonight), but it also potentially leads the user down the path of creating a worse experience when setting value reactively (the cursor will seem to not move as it gets updated with every key press - see #52).
Could align with the sentiment on https://github.com/elm-lang/html/issues/55#issuecomment-233513659 and suggest using defaultValue.


Add support for probing camera devices on socket file descriptors

Close #23

This also needs to drop the permission from the flatpak manifest of the demo.
Note the build will fail currently but that's caused by a new zbus beta.

Would you mind rebasing your PR? The CI should pass correctly now. Thank you

Thanks a lot for this MR! I am looking forward to correctly get those ids.


DartPad not caching artifacts?

Each time I click "Run" on the counter sample, I see >10MB sent over the network per the image below:

This includes multi MB artifacts that should be cached I think: dart_sdk.js , flutter_web.js, canvaskit.wasm - they are sent every time.

The 1.8 MB compileDDC xhr is a red flag as well.  That should be far smaller.  Peeking at the result, it looks like we're compiling and sending firebase libraries (at least) every time Run is clicked.  Those should be fetched only once - e.g., in flutter_web.js.

I suspect this is a result of using a throw away sandbox iframe for each new request. There is no shared state.


PII Should Be Censorable With a Click Of a Button

Should be a way to censor all PII with a click of a button so that logs bans and such and been screen shotted for ban appeals etc. Should be pretty easy with some JS

Yeah I wanna do something along the lines of a user setting that uses local storage to keep stuff like dark mode setting and such, not really much to add there now tho


Added a config plugin that makes this module compatible with the Expo managed workflow

I've added an expo config plugin, making this module compatible with the expo managed workflow. This does not mean it can be ran within expo go, since it still needs the native code, however you can create an expo dev-client to mimic expo go.
To use it, all a user has to do is add the following in their app.json:
"expo": {
    "plugins": [
      "react-native-android-auto"
    ]
  }
}

After this, they can use the module through javascript, just like in react native bare and the expo ejected workflow.

Can you enlighten me on the benefit of using the Expo Config Plugin in this project? This project already requires a lot of native code setup.

Hi, I don't know if you are familiar with the Expo managed workflow, but the idea is that a developer can create an app without ever touching any native code. However one of the shortcomings is that you cannot change configurations such as the AndroidManifest or the Info.plist. As you said your project requires a lot of native code setup, and that's exactly what this config plugin tackles.
An Expo config plugin takes care of the native setup part automatically, meaning an Expo user wouldn't have to eject out of his managed workflow. If you'd like to read more about it, have a look at the expo documentation. If you have any more questions, let me know! :)

I did go through the documentation of Expo plugins. One thing that I don't understand is the purpose of the plugin. Reading the documentation it looks like this plugin is going to add the CarService to the user AndroidManifest.xml which shouldn't be necessary as this service is already included in the library's manifest file and the Android build system will merge the library's manifest file with the user's manifest file. The only real benefit of this plugin will be the addition of meta-data and automotive_app_desc.xml file which I'm thinking of moving to the library itself.
Please let me know if I misunderstood it.

You are indeed correct, since the CarService is already defined in the library manifest it doesn't need to be in the plugin. I've removed it in case you'd still want to merge the plugin into the library. And indeed the only benefit is the addition of the meta-data and the automotive_app_desc.xml. Were you to move those into the library itself, it would solve the problems that this plugin tackles.

I will look into that when I get some time. I'm keeping this PR open just in case those attributes can't be moved into the library itself.


specify the rootElement of d3 tip in DOM

I would like to suggest a configuration of the tip to change the default root DOM element from document.body to every child element you want.

@vdh this looks like a nice improvement.

This will have to be updated to resolve the conflicts.

Updated in PR #170 .  I've been using the stetro/d3-tip PR for a few months now.  It would be nice to get back on the main branch.

@isaacplmann Thanks for the contribution! :)

@isaacplmann @stetro I made a 0.8.0-alpha.1 pre-release, since I realised I was also using an obscure fork. Hopefully the d3 v4 & micro libraries upgrade doesn't cause too much of a headache for you. If you can't upgrade to v4, let me know. I should probably be able to back port it into 0.6.x for v3.


showNavbar with time interval option?

Currently the function "showNavbar" doesn't have an argument of time delay and the function itself by default delays 0.1 sec. I am wondering if in the future this function can take the time delay argument so that we have more flexibility to change how fast the navigation bar will show up?

Hey @TunaYangSkey
I've added an optional parameter in showNavbar(_:duration:) and hideNavbar(_:duration:)