MQTT LWT to AMQP LWT

The LWT feature is not a really "MQTT related" but is having it on top of AMQP.
In the mqtt-lwt project I see :

the Netty MQTT coded dependency is used just for the MQTT QoS enum, that could be removed (using an "ad hoc" enum or just 0,1 or 2) -> no dependencies on MQTT related projects anymore
the only two AmqpWillMessage and AmqpPublishMessage classes involved use standard AMQP but add two annotations "x-opt-mqtt-qos" and "x-opt-retain-message". The first one could be not related to MQTT so we could remove the "-mqtt-" and in part it's decoded using the durable AMQP field (https://github.com/EnMasseProject/enmasse/blob/master/documentation/mqtt-over-amqp/docs/operations_connection.md#will-information-to-the-last-will-and-testament-service). The retain part is more MQTT specific (for the will message) even if you could not specify it having retain=false by default.

My instinct (but more thinking is needed) is to have an amqp-lwt service, so outside of the whole "mqtt" umbrella.
Wdyt ? @lulf @grs @rgodfrey

I agree, it can be independent of mqtt. Perhaps just x-opt-qos with defined values (even if they are same absolute values as mqtt, it would be defined as an amqp extension). Likewise for retain.

I'm updating this for having a more generic AMQP connectivity/presence service (so not just LWT). It means not just detecting disconnection and sending the "will" message but even detecting connection and publishing events for that.

The MQTT LWT → AMQP LWT service, could become a more generic “AMQP Connectivity/Presence” service
Speaking about pure AMQP clients, if a client wants to provide such feedback on his connectivity status, it has to attach a link to such a service (today it happens from the MQTT gateway to the $lwt address of the MQTT LWT service). Having this attached link, the service can publish a message on a system address (or maybe configurable ?) like “$connection” with info about the connected client (something got from the “link name” established by the client, i.e. client-id for MQTT use case).
On this link, as it already happens today, an AMQP “will” message can be sent to the service from the client; such message will be stored by the service and delivered on brute client disconnection to the “will topic”.
When a client detaches this link (brute or not), the service can send a message on a system address (or maybe configurable ?) like “$disconnection” with info about the disconnected client. At same time, as already happens today, the stored "will" messages is just deleted or sent to the "will topic".

In this way, if a service is interested in clients connection/disconnection it can acts as a receiver for $connection/$disconnection addresses.
The “will testament” part continues to work as today : when a brute disconnection happens, the service publishes the related “will” message to the “will topic”. At same time, the AMQP Connectivity service is in charge to store the received “will” messages (and delete them when needed) in a corresponding LWT store (today we have only a simple “in memory” implementation for that).

closing


css breaks command palette

The introduction of the new css code from #20 breaks the appereance of all other prompts e.g. the command palette (Ctrl+P) from the core plugins.

I have also encountered this issue.

Thank you for reporting this bad issue. It should be fixed now with version 1.0.52. Please reopen this issue, if it still occurs.


Topic sk main 1

Merge to main after resolving conflicts.

Updating image versions with autoscaling functions issues addressed
Remove requirement from network validation script for private load balancer to open to <IP_ADDRESS>/0 CIDR.


Need blurb text

Hi,
do you have a nice and brief blurb text that I could add to
http://cbor.io/impls.html
Right now, the D part is a bit naked...

Hey, sorry for late answer. Here is some text:
A compact D implementation. Dub package.


Save Latest Firmware of A9G

Hi,
I have an A9G module which has below firmware version.
AT+GMR

V02.02.20190915R

Before starting to flash my firmware, i was curious if i could somehow save the complete firmware in case i want to revert back to original factory firmware so that atleast i could use the module using AT commands on UART?
I figured out we could flash Factory/Dev firmware using "fpugrade" tool but the lastest available firmware is of V02.02.20180825/20181011 version and there is not a single info on the current firmware(20190915) that is flashed on the A9G.
I dont know why Ai-thinker is not updating the info properly.
The HW specs are really interesting but finding the toolchains seems to be really daunting due to missing links:-(.
Thanks!

OK, it's possible. but not straightforward. basically you need a "save flash to SD Card" utility, then, you need to "ramrun" it from coolwatcher without flashing it, I have a some code to do it (I needed to restore an A9G accidentally-completely-wiped during experimenting with random calls). It's late here at the moment, but I'll dig it out and post it here.
I'm not sure if there's an easier option or utility that does this already, I'm sure others will mention if so.

Hi @ihewitt ,
do you mean inserting a SD card into the A9G module ?
Could you share the documentation/code about "ramrun"?
Thanks for your hint!

Ah I was getting muddled with one of my other devices... hmm, I'm not sure if there is an equivalent of "run without burning" in coolwatcher.
anyway.... this code: https://gist.github.com/ihewitt/7ef825261cc642398cf795f394af7539
When burnt will dump the contents of the flash onto the SD card, although when burnt you will lose any "APP" already installed, i.e. you wouldn't get a copy of the default AT app, but this will take a copy of the platform and bootloader (i.e. base SDK). Perhaps anyone else can comment/suggest if it's possible to run without burning?

haha spoke too soon, ok I think I've got the binary protocol to query the memory. I'll see if I can write a command line "dump" via HST app.

How did you find the HW protocol?
I'm trying to figure out how coolwatcher is communicating on the COM port using serial sniffer.
let me know if you have any update, I'm curious.

On Tuesday, 6 October 2020 21:40:00 BST ViHu wrote:

How did you find the HW protocol?
I'm trying to figure out how coolwatcher is communicating on the COM port
using serial sniffer.
let me know if you have any update, I'm curious.

In coolwatcher you can open up the "coolhost" window... in there you log and
view the serial I/O there's a "save com data" on the "other" tab which you can
then use to open up and view the traffic.
I've got a small basic serial app that can read the flash now.... although I'm
not 100% sure I've got the serial configured quite right since it dies after
some I/O has gone through. Will play with it a bit more after work and paste
to gist.

Here's a gist of what I'm trying to get working.. proof of concept:
https://gist.github.com/ihewitt/5969b7d427fc7248306cb894ec20cace
almost works. needs a fresh boot of the A9 will start to dump flash contents (of the app section - https://ai-thinker-open.github.io/GPRS_C_SDK_DOC/zh/more/flash_map.html)
but I'm not a serial expert and I can't figure out the right way to get the protocol working. but it starts to work (on a fresh booted A9G) but gets checksum errors and breaks.... :(

Done. Proof of concept. Needs editing to extract different parts of flash (and its slow) but can be used to extract all firmware sections without flashing anything.
https://github.com/ihewitt/ivrtrack/blob/main/util/dump.c

Found something interesting ;-)


Update fix-weightClass and fix-vf-meta

Both scripts has issues with setting usWeightClass values correctly.
fix-vf-meta:
I remove the hairline style. Fixes #221
fix-weightclass:
I've used the fontbakery.parse module. This script will now set the usWeightClass correctly for otfs, ttfs and vfs.

Thanks Mark! I just tested it and everything work as expected :)


fix: applications status are updated in a single transaction

The code coverage is decreasing. Ca you please check ? It may due to dead code appearing or missing tests


Setup page on fresh install

Make a setup page when there are no users nor settings in the database. So when a user (or in the future - a Dockerfile) does:
# migrate DB
./manage.py migrate

# start the app
./manage.py runserver 0:80

# or other more production deployment like apache2 with mod wsgi

..they are greeted with setup page. Probably this needs to be done as the last issue to contain the most settings.

Very trivial page setup pushed in 95e885c797bab22f3826bfe6b72fcef69ea0048c


Add tooltip for table column headers and align header and cell

This adds the toolTip option for TableColumn and also aligns the padding left of headers and cells so that it all aligns.

@kisantia just fyi, it's pretty useful to include a screenshot of the impact of the change for style updates like this.  Otherwise, one has to try to imagine the problem and proposed solution 😄.


Rewrites OkHttp interceptor as BraveTracingInterceptor

This is an OkHttp interceptor which
traces client requests. It should be applied as both an application
interceptor and as a network interceptor.
The implementation models the application request as a local span. Each
network request will be a child span. For example, if there's a redirect,
there will be one span for the application request and two child spans
for the associated network requests.
Trace identifiers of each network attempt are propagated to the server
via headers prefixed with X-B3. These spans are also reported out of
band, with cs (client sent) and cr (client receive) annotations,
binary annotations (tags) like http.url, and sa including the
server's ip and port.
Configuration
Since this interceptor creates nested spans, you should use nesting-aware
span state like InheritableServerClientAndLocalSpanState. If using
asynchronous calls, you must also wrap the dispatcher's executor
service. Regardless, the interceptor must be registered as both an
application and network interceptor.
Here's how to add tracing to OkHttp:
brave = new Brave.Builder(new InheritableServerClientAndLocalSpanState(localEndpoint))..

// The request dispatcher uses an executor service.. wrap it!
tracePropagatingExecutor = new BraveExecutorService(
    new Dispatcher().executorService(),
    brave.serverSpanThreadBinder()
);

client = new OkHttpClient.Builder()
  .addInterceptor(tracingInterceptor)
  .addNetworkInterceptor(tracingInterceptor)
  .dispatcher(new Dispatcher(tracePropagatingExecutor));
  .build();

Naming the application span
The span associated with the application request is by default named
according to the SpanNameProvider supplied to the interceptor.
If you prefer to use the OkHttp request tag instead, set
BraveTracingInterceptor.Builder.tagIsApplicationSpanName(true)
In the below example, the span name will be "get-gummy-bears":
tracingInterceptor = BraveTracingInterceptor.builder(brave)
  ...
  .tagIsApplicationSpanName(true).build();

request = new Request.Builder()
    .url("https://myhost.com/gummybears")
    .tag("get-gummy-bears").build();

Naming the server
When calling an service that isn't traced with Zipkin, such as a cloud
service. You'll want to assign a server name so that it shows up in
Zipkin's service dependency graph. You can set this via
BraveTracingInterceptor.Builder.serverName()
tracingInterceptor = BraveTracingInterceptor.builder(brave)
  ...
  .serverName("github").build();

request = new Request.Builder()
    .url("https://api.github.com/repos/square/okhttp/issues")
    ...

I created this for use in an okhttp-specific elasticsearch impl for zipkin, but @tburch might be interested, too.
https://github.com/openzipkin/zipkin/issues/1312

I'm actually going to rewrite the interceptor and deprecate the old one

actually found a way to do what I wanted..

cc @jakewharton @swankjesse next change after this will take advantage of network interceptors by logging a "ws" (wire send) annotation for each network attempt.

Here's a sample trace I got from an elasticsearch client I'm working on. You can see the outbound request (which is okhttp) continuing from a server one.

thinking about an adjustment, which is to make the "application" request a local span. This would make each network attempt a child client span. I think this would be more helpful to explain latency, especially if a tag is set.
Ex. if using feign, you could set the okhttp tag corresponding to the api name called. For example, github#list-users would be a span, and it would have a child named "get" for each network attempt. This would pretty-much obviate any need to make custom feign instrumentation (ex #158)
This needs the special nested-aware span state class, as often the caller of okhttp is already a local span. Lucky thing is that we have that. (InheritableServerClientAndLocalSpanState)
cc @schlosna @spencergibb @rafabene @ewhauser @marcingrzejszczak @shakuzen

ok refactored the thing to do nested spans. very hard to test this, since there's no way in brave to control nanotime or trace id generation. I think I learned more than I wanted to about mockito.
Anyway, will update examples after trying against elasticsearch again

Here's how it looks now (using okhttp tags to name application spans)

@swankjesse in the image I just pasted, you can see application two okhttp requests resulting from a server. "search-..." are okhttp tag names I assigned to the requests. Each one has a child span "post" corresponding to the actual network request.
I suspect that in the future, we could add things to explain the offset. Ex the first was probably longer due to connection setup and/or dns. For example, we could trace the Dns type or any other hooks that might arise.

@schlosna I'm going to rewrite this so that it is easier to add custom headers as tags

Here's a fun one.. I traced the okhttp crawler example, showing a mix of requests taken from local cache and network responses

going out as 3.13.0 and will be used almost immediately in zipkin when tracing its traffic to elasticsearch

Thanks @adriancole, I didn't get a chance to test it today, but sounds like this should cover our need.


Secure review - v1.01 release

Secure code review of the newly created KomodoPlatform dragon_logs flutter package hosted at  https://pub.dev/packages/dragon_logs. This removes a lot of logging-related code from the main project and handles it here much more efficiently using a novel approach for web storage.
This is the first version to be reviewed so it needs a full review. I've tried to re-use versions of dependencies previously reviewed in other projects.

Thanks for the review @DeckerSU.

I will add a note that some browsers have experimental support. The API has perhaps also matured in the 10 months since the packages release (which affects the package without an update needed).
I see that the .remove() call is the only experimental call we are using for supported browsers. I will replace it with the [removeEntry](https://developer.mozilla.org/en-US/docs/Web/API/FileSystemDirectoryHandle/removeEntry#browser_compatibility) API in the next release which has stable support.

For share_plus, I've had to reference the pub-hosted version because packages can't be published with git-hosted dependencies. Our internal security is the main priority so, alternatively, I can create a private fork with the git reference.
I'm also working on setting up a private Dart package hosting server for the org (repo).


Validate Memory Usage for given resource pool for Vsphere provider

Issue #, if available:
Description of changes:
Add a preflight validation that checks for requested memory is available in the resource pool available before proceeding with the create or upgrade operation.
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest eks-anywhere-presubmit

/retest-required

/approve
/lgtm


Error when initialising

Hello - if I initialise with non empty highlightStyles and selectStyles, I get the following error:
TypeError: Cannot read property 'selector' of undefined

If I initialise with  empty highlightStyles and selectStyles it initialises OK.
Here is some test code that shows options that work and options that do not work:
import cytoscape from "cytoscape";
import viewUtilities from "cytoscape-view-utilities";
viewUtilities(cytoscape); // register extension
const cy = cytoscape()
const optionsWorking = {
    highlightStyles: [],
    selectStyles: {},
};
const optionsNotWorking = {
    highlightStyles: [
        {node: {'border-color': '#0b9bcd', 'border-width': 3},
            edge: {
                'line-color': '#0b9bcd',
                'source-arrow-color': '#0b9bcd',
                'target-arrow-color': '#0b9bcd',
                'width': 3
            }
        },
        {node: {'border-color': '#04f06a', 'border-width': 3},
            edge: {
                'line-color': '#04f06a',
                'source-arrow-color': '#04f06a',
                'target-arrow-color': '#04f06a',
                'width': 3
            }
        },
    ],
    selectStyles: {
        node: {'border-color': 'black', 'border-width': 3, 'background-color': 'lightgrey'},
        edge: {'line-color': 'black', 'source-arrow-color': 'black', 'target-arrow-color': 'black', 'width': 3}
    },
};

let api = cy.viewUtilities(optionsWorking);

export default cy;

Can anyone suggest please what might be the issue?
thanks!

@bootrino Since view-utilities is a UI extension, you shouldn't run it in headless cytoscape instances. In other words, you should provide a container to your cytoscape(). If you still want to run cytoscape headlessly, then you need to set styleEnabled option to true to be able to use this extension.

@hasanbalci hmm OK thanks for the prompt reply.
Actually I mount the container a bit later instead of during initialisation like this:
        cy.mount(container.current)

Thanks for the pointer though - I'll see if I can find a way to initialise cytoscape.js-view-utilities after my mount command.


[outreachy] Registry: click on tag for additional filtering

Outreachy task for @basiratkareem
Based on your user experience report, please provide the following suggested improvement via a PR:

Implement a feature where clicking on a tag (like native, sandbox, etc.) automatically filters the records with the same tag, allowing users to quickly apply filters without needing to scroll back up to the search bar.

Hello @svrnm , as an outreachy contributor I want to work on this issue

Hello @svrnm , as an outreachy contributor I want to work on this issue

hey @crocmons, please review #5290, and follow the instructions for outreachy contributors there.

Thank you @svrnm I'll work on it and will update my progress.

Hello @svrnm I have made the changes and created a PR


Implement Cloud Source Repositories API crawler

Cloud Source Repositories allows to query list of repos per project. We need to implement a scanner capable of querying list of projects.

Hey @mshudrak, I would like to work on this issue. Please assign this to me.

Thanks, done.


Completed status values

Added the following values:

GettingStatus
Quarantined
FilteredAsSpam

Docs Build status updates of commit 2fcdc45:
:clock10: Full build: the average full build time is 12 min(s) 48 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 2fcdc45:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/mail-flow/Get-MessageTrace.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Allow isolatedModules: true with noEmitOnError: true

Search Terms

isolatedModules
noEmitOnError

Suggestion
Allow isolatedModules: true with noEmitOnError: true
Use Cases
What do you want to use this for?
My understanding of the purpose of isolatedModules: true is that it does extra validation to ensure that separate compilation is safe. We want to use it for that purpose.
Allowing emit in the presence of errors is contrary to that goal. We're doing the validation to stop dangerous code from getting through.
What shortcomings exist with current approaches?
We use TypeScript in our compilation pipeline.
The only workaround I can think of is some hack to compile to a different location and copy files over once checks pass. This doesn't seem great for watch mode, and could be error-prone or racy.
Examples
tsconfig.json:
{
    "compilerOptions": {
        "noEmitOnError": true,
        "isolatedModules": true,
    }
}

Actual behavior: tsconfig.json(5,9): error TS5053: Option 'noEmitOnError' cannot be specified with option 'isolatedModules'.
Expected behavior: no error message, just works
Checklist
My suggestion meets these guidelines:

[x ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

I think this restriction was added during the period of time in which we were confused about isolatedModules meant, and can be trivially removed

Thanks Ryan!
There's a PR to allow the options to be used together here: https://github.com/bloomberg/TypeScript/pull/34 against the Bloomberg repo. We can reopen against the TS repo, but will get 👀 on it first.


migrating VMs from vmware using vcsim and fake vddk.

introducing vddk-test-image  that is built by cdi https://github.com/kubevirt/containerized-data-importer/tree/main/tools/vddk-test
for our testing, it replaces the real vddk side-car container.

LGTM. Do we need also #13 for it to work? Also, later on - we may want to remove this change and do it straight into the tests in forklift repo (e2e for vmware).

No this is a replacement for #13, I've updated this PR to consume the image from  http://quay.io/kubevirt/vddk-test ,  so we don't need to worry about building/maintaining this container separately.

also  bumped kubevirt version  to v0.58.0 due to https://github.com/kubevirt/kubevirt/pull/8451


Home Update: Banners

Issue #1442 outlines what should happen when there are no banners

@katarinabatina Do you think this is a must in the first public version?

@alloy not necessarily but if we're moving forward with changing search then the current design will still need to be adjusted somewhat

@katarinabatina I’m not entirely sure what the part is that would still need to be adjusted. Could you elaborate?

@alloy adjustments based on discussion we had this morning continuing to use existing data.

@katarinabatina Thanks! Just want to confirm that we’re using the name attribute on both iPhone and iPad and ignoring the mobile_title attribute?
{
    "name": "ArtRio 2016 #3",
    "mobile_title": "ArtRio",
}

(I don’t know what purpose the mobile_title is supposed to serve, maybe it’s purely a martsy thing?)

@alloy I am not sure the difference between them, I imagine the mobile one displays on martsy/iphone and the name displays on desktop. Perhaps we should pull the mobile_title to be safe.

@katarinabatina Ok, what I’ll do is use mobile_title on iPhone and name on iPad.

@alloy yeah that sounds like a good idea. It's interesting because the "name" isn't actually used to display the text on the banner in desktop (the text is an image). My guess is that "name" is used to identify which banner is being shown (hence the "Art Rio #3")

Ahhhhh, I naively assumed that that was just a staging data thing. Ok, so then it doesn’t really make sense to ever use name, right? I’ll just use mobile_title which probably just holds ‘the textual version of the same title’?

@katarinabatina What’s the black background rectangle with 30% opacity about? Is it an overlay over the image that’s meant to darken it?

Yeah, I bet it's to make sure that any pure whites on the image don't make the text unreadbale

Yup, what Orta suggested!
Sent with thumbs,
Katarina Batina
Product Designer
Artsy.net

On Oct 10, 2016, at 8:07 AM, Orta<EMAIL_ADDRESS>wrote:
Yeah, I bet it's to make sure that any pure whites on the image don't make the text unreadbale
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Gotcha! 👍

@alloy looking at this today, looks pretty good I think just

@katarinabatina continuing here https://github.com/artsy/eigen/issues/1912


Flesh out unit tests

Work for this issue will begin begin on branch work-liquid-labs/question-and-answer/104.


Add Figure releases to announcements

WIP I'll copy the rest across tomorrow

Ah, now find Figure 3.0.0 onwards are already here! I wasn't going to bother listing 2.0.1 since it's just a PyPI fix, if that's okay @jburel ? So this is good for final review now, I'll list it for Monday if you don't have time to look at it today.

@hflynn ignore 2.0.1.

https://snoopycrimecop.github.io/www.openmicroscopy.org/announcements/ all update now, ready for you to review @will-moore - thanks

Do we support Youtube videos in-line?
It can be nice to see them on the page, if the formatting is not too hard to handle?
Or maybe add a link instead?
E.g. releases 1.0.0, 1.1.0, 1.2.0.
I realise we've not been adding links to PRs in the release announcements. I'm not sure why, but maybe this was simply because they were available on the figure.openmicroscopy.org posts and we mostly pointed users there?
However, if those posts won't be available in future, it would be nice to have the links to PRs here, since this gives a lot more info to those who are interested - and can be useful for our own reference too.

I followed the pattern of release announcements for other bits of the project, where we do not link directly to PRs. There are some PRs listed in the CHANGELOG, I'd keep them there instead.
I'm not sure if we can embed Youtube videos in the markdown pages, I can certainly add the links though.

Of course the videos work embedded, the set up here is the same as where you originally posted them! Added now anyway.

https://snoopycrimecop.github.io/www.openmicroscopy.org/2014/10/07/figure-1-0-0.html
https://snoopycrimecop.github.io/www.openmicroscopy.org/2015/04/20/figure-1-1-0.html
https://snoopycrimecop.github.io/www.openmicroscopy.org/2015/11/24/figure-1-2-0.html
updated

This is ready for final sign off now

Is this good to merge now @will-moore ?

Looks great, thanks.


Clion bazel plugin shows "Bazel test" for Helloworld.cc class from examples project

Description of the bug:
The Clion Bazel plugin shows the "Bazel test" for the Helloworld.cc class from the sample CPP of the example project.
What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
Import Clion project and do a sync. Then go to https://github.com/bazelbuild/intellij/blob/master/examples/cpp/simple_project/src/hello_world.cc class and click on Green icon beside  int main(int argc, char** argv) { main method. it shows Bazel test src which is confusing.
 for reference https://screenshot.googleplex.com/95UVRLzpsj9zPHp
Which Intellij IDE are you using? Please provide the specific version.
CLION 2023.2.
What programming languages and tools are you using? Please provide specific versions.
CPP
What Bazel plugin version are you using?
v2023.11.07
Have you found anything relevant by searching the web?
No response
Any other information, logs, or outputs that you want to share?
No response

I reviewed the notes for manual testing and yes in C++ the binary and test runs should be invoked from the target in the BUILD file. But when we tried running the test from the .cc file worked, I'm not sure if fixing it for the binary is something we want to keep in mind or close this bug as not planned.

There is a cc_test target for the .cc file here that I did not notice before, so I'm not sure if this is WAI now? @blorente who added this target, what do you think?

This is intentional as per https://github.com/bazelbuild/intellij/pull/4786
The heuristic introduced there goes like: If there is a "main" function and a cc_test target, please treat it as a test.
It's definitely a heuristic more than a hard rule, we (internally, at Apple) just found that it suited us well for tests that were essentially "just a main function".
I assumed, perhaps incorrectly, that a main function would be either a test or a binary, not both.
I'm happy to:

Revert https://github.com/bazelbuild/intellij/pull/4786.
Separate the example so that the cc_test is it's own separate main function.

I think what needs to be updated is if there is also a cc_binary target for the same "main" function then which rule should be used. I think we can separate the targets in the example project for now.


Display HTTP error response to the user

Fixes https://github.com/gradle-nexus/publish-plugin/issues/91
This PR logs the (error) response body in the Exception thrown by the plugin whenever a HTTP response fails. This allows the user to fix the problem indicated by the Nexus server.

Thanks @hiddewie . Spotless has some formal objections, but in general it LGTM.

Fantastic - I was about to make my own PR as we're facing the same issue:
java.lang.RuntimeException: Failed to release staging repository, server at https://oss.sonatype.org/service/local/ responded with status code 500, body: io.github.gradlenexus.publishplugin.shadow.okhttp3.ResponseBody$1@7a0c0fab

Ah yes I messed up the indenting, hopefully fixed.

Running ./gradlew spotlessKotlinCheck sometimes might be faster ;-).

Thanks again. It will be part of the 1.1.0 release.

Thanks for the help and the quick feedback!


SOCIAL_AUTH_COMPLETE_URL_NAME = 'complete' but should be 'socialauth_complete'

the correct setting is 'socialauth_complete'
but readthedocs.org has the incorrect value http://django-social-auth.readthedocs.org/en/latest/configuration.html
and for some reason the master branch in github also has the incorrect value
https://github.com/omab/django-social-auth/blob/master/doc/configuration.rst
but the pypi site has the correct value:
http://pypi.python.org/pypi/django-social-auth/
if you have the incorrect value, you will see this error message:
Reverse for 'complete' with arguments '(u'facebook',)' and keyword argumeants '{}' not found.
i originally typed this info at:
https://convore.com/django-social-auth/social_auth_complete_url_name-complete-vs-socialauth_complete/

Fixed on https://github.com/omab/django-social-auth/commit/a854c8bad4f465d48bb3e3499b58ff339a61f058


Fix ExtensionInterface answer - Update bundles.yml

The good answer is public function load(array $config, ContainerBuilder $container) doc

Correct :) Thank you @oktapodia :)


Switch to *.podspec.json format for better automation.

As part of #532 we need to have the Azure SDK specs in *.podspec.json format so that we can run some automation to enforce versioning logic etc. This is the initial step (cutting over to *.podspec.json.

@bsiegel regenerated.


Service worker doesn't always update

https://github.com/excalidraw/excalidraw/pull/1588 didn't completely fix the SW caching issue. It still often happens the latest isn't fetched (e.g. https://github.com/excalidraw/excalidraw/issues/2061). No idea what are the conditions for the update to not work.
We need to figure out a 100% reliable solution.
/cc @tomayac

@tomayac you're posting a SKIP_WAITING message to the service worker, but aren't subscribing and acting on it from the looks of it:
https://github.com/excalidraw/excalidraw/blob/4718c31da5b6a40d4f3a7ede75836355fadd6fa1/src/index.tsx#L181

The service worker update logic is handled by create-react-app, see this SO post that covers the mechanics. For me, it reloads reliably whenever a new service worker is available.

Well, it's missing from our script. FWIW, CRA completely removed their script as of now, anyway.
Fixed by https://github.com/excalidraw/excalidraw/pull/2084

Interesting. It definitely was there when I added the logic. I don’t follow CRA closely, but I think they’re redoing their service worker logic anyway.

No, you're right. They're dynamically adding that part on build, which is super weird.
So we're back to square one a need to figure out a different solution :(.

The related issue seems to be https://github.com/facebook/create-react-app/issues/9141.

Mhm.. who knows when they'll release it. @lipis can we install CRA from a commit?

@dwelle I would wait for an official release, instead of doing some ninja moves ;)


narrower type inference of for..in loop variables

Flow seems to always infer string type for the variable in for..in loops, but it could make narrower inferences for enum types.  The following code fails to typecheck:
// @flow

type Axis = 'x' | 'y'

const axisProps: {[axis: Axis]: {}} = {}

for (let axis in axisProps) {
  const props = axisProps[axis]
}

src/temp.js:8
  8:   const props = axisProps[axis]
                               ^^^^ string. This type is incompatible with
  5: const axisProps: {[axis: Axis]: {}} = {}
                              ^^^^ string enum

Found 1 error

heh, funny that this came up again. someone +1'd a duplicate of this yesterday: https://github.com/facebook/flow/issues/1736. i'll re-title that one, no wonder it's impossible to find.

@mroch yeah I figured someone else would have come across this.  Thanks!


snakefmt all snakefiles

See Documentation
Installed it on biowulf at /data/CCBR_Pipeliner/bin/snakefmt

Testing in

Testing in /gpfs/gsfs8/users/CBLCCBR/kopardevn_tmp/issue57_testing_6


Failed to convert back into a xml file

I need to convert xml to json and then make some changes to json and convert it back to xml.  This module seems very promising.
When I tried it on some xml (https://raw.githubusercontent.com/GovReady/govready/master/prototypes/ssg-rhel6-oval.xml)
The converted xml is not valid:  I loaded it using Firefox browser, but it complained XML Parsing Error: not well-formed.
Here is a simple code snippet that I adopted from your example.
var convert = require('xml-js');
var fs=require("fs");

xml=fs.readFileSync("ssg-rhel6-oval.xml", "utf8");

var result1 = convert.xml2json(xml, {compact: true, spaces: 4});

rxml = convert.json2xml(result1, {compact: true});

console.log(rxml);

Any ideas?  Appreciate your help!

Hi.
You may try other converter.
https://javadev.github.io/xml-to-json/

@jinq102030 did you solve this?

I didn't need it any more.  You can close this ticket.

'<' and '&' need to be represented as character /numeric entities in attribute values (See XML 1.0 Standard section 2.4, https://www.w3.org/TR/xml/#syntax). This does not happen and causes the trouble with the sample file of jinq102030.


verbose for qeuue:work ?

Is there a verbose version of queue:work or/and queue:listen?
I'm pushing jobs to the queue, when I run the
./artisan queue:work

no outputs show.

We're open to PRs.


[Union] attribute on an partial record instead of interface?

Example usage
[Union]
public partial record Result
{
    partial record Ok(string Message);

    partial record Error(Exception Exception)
    {
        // Can override specific methods on the generated types
        public override string ToString() => Exception.ToString();

        // ...or add new ones without resorting to extension methods
        public bool CanIgnore => Exception is OperationCanceledException;
    }
}

// Instantiate the Error case directly:
var result = new Result.Error(new Exception("hi :)"));

// Use the factory methods that return `Result`:
var otherResult = Result.NewError(new Exception("another way"));

result.Match(
    (Result.Ok _) => 1,
    (Result.Error e) => e.CanIgnore ? 1 : 0);

// Can use named arguments to make `match` more explicit:
result.Match(
    error: e => e.CanIgnore ? 1 : 0,
    ok: m => m.Message.Length);

Benefits

With the generated code below, it is impossible to create another type that derives from Result as it has a private constructor.
It is possible to add your own method to the Result class and the case classes directly rather than having to write them as extension methods.
Untested hypothesis, but I imagine the JIT would generate better code with an abstract Match method, as opposed to the extension method that you have currently.

Source generator produces
abstract partial record Result
{
    // No other classes can derive from this class
    private Result() {}

    public static Result NewOk(string message) => new Ok(message);
    public static Result NewError(Exception exception) => new Error(exception);

    public abstract T Match<T>(Func<Ok, T> ok, Func<Error, T> error);

    public sealed partial record Ok : Result
    {
        public override T Match<T>(Func<Ok, T> ok, Func<Error, T> error) =>
            ok(this);
    }

    public sealed partial record Error : Result
    {
        public override T Match<T>(Func<Ok, T> ok, Func<Error, T> error) =>
            error(this);
    }
}

I really like this idea! Seems more powerful and easier to generate sources for.
Acceptance note:
This should probably be implemented as an alternate code path where the generator gets triggered when an abstract record marked with the Union attribute changes. Don't think it would be a good idea to try to fit this functionality into the current generator.

With this implementation we should also be able to do:
using static Result;

// Instantiate the Error case directly:
var result = new Error(new Exception("hi :)"));

// Use the factory methods that return `Result`:
var otherResult = NewError(new Exception("another way"));

Closing as the core functionality is implemented by #21 and #25 and released in https://github.com/domn1995/dunet/releases/tag/v0.5.0
Will track factory method generation in #12.
Thanks so much for this awesome suggestion!


namespace: don't change namespaces which are not belonged to the CT

In a config file we specify which namespaces have to be created and which configuration have to be applied for them. Currently if a configuration is specified without a proper namespace, it is applied to the host namespace. It's unexpected behavior, isn't it?
With these patches we will get an error if a config file contains configuration without a proper namespace.

i have refactor this pr base on @LK4D4 's suggest. have a look please.

@xiaods @avagin I'm not a fan of such development model. For now I suggest you to decide which PR we should leave opened, but next time I will appreciate if you wil make PRs to other's PR or just reviewing it. Thanks!

Agree with @LK4D4

@LK4D4 got it. thanks for your comments.

Can we continue this discuassion? @mrunalp, you probably should like this functionality, because libcontainer recently destroyed your host mount namespace;).

@avagin Ha ha, yes you make a compelling case :) Reviewing.

LGTM

LGTM. (I will send a follow on PR to add this to userns code path.)


Windows 10 error

win_ble 1.0.2 (same issue with version 1.0.1)
flutter 3.16.4
(No issues) OS Name Microsoft Windows 11 Pro - Version 10.0.22621 Build 22621
(Issue)        OS Name Microsoft Windows 10 Pro - Version 22H2 Build 19045.3803

We have two different behaviors if the application runs on Windows 10 or Windows 11.
The application runs smoothly on Windows 11, but in Windows 10 it gets an "Unspecified error. 3 Required characteristic not found"   during the write and subscribe command.
Below the logs (the error appears translated into Italian because Windows is set to Italian).
We have also tried on other computers with Windows 10 and 11 and the behaviour is always the same.
Thanks in advance.
Windows 11 log (with no issue):
flutter: DEBUG - 2023-12-15 10:20:42.540147   Instance of 'BleClientConnectionStateUpdateEvent'
flutter: Received Message : {_type: response, _id: 0, result: BluetoothLE#BluetoothLE60:e3:2b:e9:2d:3d-0c:8b:95:77:10:ca}
flutter: Received Message : {_type: response, _id: 1, result: [{00001801-0000-1000-8000-00805f9b34fb}, {00001800-0000-1000-8000-00805f9b34fb}, {0000faf0-0000-1000-8000-00805f9b34fb}]}
flutter: DEBUG - 2023-12-15 10:20:44.351165   Instance of 'BleClientConnectionStateUpdateEvent'
flutter: DEBUG - 2023-12-15 10:20:44.352165   Get hello
flutter: DEBUG - 2023-12-15 10:20:44.352165   Send SET: Instance of 'HelloRequest'
flutter: Received Message : {_type: response, _id: 4, result: null}
flutter: Received Message : {_type: response, _id: 2, result: 1}
flutter: Received Message : {_type: valueChangedNotification, subscriptionId: 1, value: [1, 1, 161, 2, 0, 0, 0, 0, 0, 0, 9, 0, 1, 0, 229, 58, 8, 0, 1, 4, 0]}
flutter: {address: 0c:8b:95:77:10:ca, serviceId: 0000faf0-0000-1000-8000-00805f9b34fb, characteristicId: 0000faf2-0000-1000-8000-00805f9b34fb, value: [1, 1, 161, 2, 0, 0, 0, 0, 0, 0, 9, 0, 1, 0, 229, 58, 8, 0, 1, 4, 0]}
flutter: DEBUG - 2023-12-15 10:20:44.605119   HelloResponse decode
flutter: DEBUG - 2023-12-15 10:20:44.606125   Get setting info
flutter: DEBUG - 2023-12-15 10:20:44.607123   Send SET: Instance of 'GetSettingsInfoRequest'
flutter: Received Message : {_type: valueChangedNotification, subscriptionId: 1, value: [1, 1, 17, 51, 87, 0, 0, 0, 0, 36, 146, 0, 0, 0, 255, 255, 4, 8, 1, 255, 100, 0, 0, 9, 9, 9, 9, 1, 255, 100, 0, 0, 9, 9, 9, 9, 1, 255, 100, 0, 0, 9, 9, 9, 9, 2, 9, 100, 0, 0, 7, 9, 9, 9, 0, 0, 26, 186, 122, 101, 24, 0, 24, 0, 0, 0, 1, 0, 0, 26, 186, 122, 101, 7, 0, 7, 0, 0, 0, 1, 0, 0, 26, 186, 122, 101, 7, 1, 7, 1, 0, 0, 1, 0, 0, 26, 186, 122, 101, 7, 2, 7, 2, 0, 0, 1, 0, 0, 26, 186, 122, 101, 7, 3, 7, 3, 0, 0, 1, 0, 0, 26, 186, 122, 101, 25, 0, 25, 0, 0, 0, 1, 0, 0, 26, 186, 122, 101, 28, 0, 28, 0, 0, 0, 1, 0, 0, 26, 186, 122, 101, 6, 0, 6, 0, 0, 0, 1]}
flutter: {address: 0c:8b:95:77:10:ca, serviceId: 0000faf0-0000-1000-8000-00805f9b34fb, characteristicId: 0000faf2-0000-1000-8000-00805f9b34fb, value: [1, 1, 17, 51, 87, 0, 0, 0, 0, 36, 146, 0, 0, 0, 255, 255, 4, 8, 1, 255, 100, 0, 0, 9, 9, 9, 9, 1, 255, 100, 0, 0, 9, 9, 9, 9, 1, 255, 100, 0, 0, 9, 9, 9, 9, 2, 9, 100, 0, 0, 7, 9, 9, 9, 0, 0, 26, 186, 122, 101, 24, 0, 24, 0, 0, 0, 1, 0, 0, 26, 186, 122, 101, 7, 0, 7, 0, 0, 0, 1, 0, 0, 26, 186, 122, 101, 7, 1, 7, 1, 0, 0, 1, 0, 0, 26, 186, 122, 101, 7, 2, 7, 2, 0, 0, 1, 0, 0, 26, 186, 122, 101, 7, 3, 7, 3, 0, 0, 1, 0, 0, 26, 186, 122, 101, 25, 0, 25, 0, 0, 0, 1, 0, 0, 26, 186, 122, 101, 28, 0, 28, 0, 0, 0, 1, 0, 0, 26, 186, 122, 101, 6, 0, 6, 0, 0, 0, 1]}
flutter: Received Message : {_type: response, _id: 3, result: 2}
flutter: Received Message : {_type: response, _id: 5, result: null}
flutter: Received Message : {_type: valueChangedNotification, subscriptionId: 1, value: [1, 1, 161, 2, 1, 0, 0, 0, 3, 0, 36, 0, 1, 0, 5, 1, 0, 0, 2, 0, 171, 1, 0, 0, 2, 0, 227, 0, 0, 0, 1, 0, 108, 0, 0, 0, 2, 0, 188, 0, 0, 0, 1, 0, 64, 0, 0, 0]}
flutter: {address: 0c:8b:95:77:10:ca, serviceId: 0000faf0-0000-1000-8000-00805f9b34fb, characteristicId: 0000faf2-0000-1000-8000-00805f9b34fb, value: [1, 1, 161, 2, 1, 0, 0, 0, 3, 0, 36, 0, 1, 0, 5, 1, 0, 0, 2, 0, 171, 1, 0, 0, 2, 0, 227, 0, 0, 0, 1, 0, 108, 0, 0, 0, 2, 0, 188, 0, 0, 0, 1, 0, 64, 0, 0, 0]}
flutter: DEBUG - 2023-12-15 10:20:44.645242   GetSettingsInfoResponse decode
flutter: DEBUG - 2023-12-15 10:20:44.677927   Send SET: Instance of 'MonitorEnableRequest'
flutter: Received Message : {_type: response, _id: 6, result: null}
flutter: Received Message : {_type: valueChangedNotification, subscriptionId: 1, value: [1, 1, 161, 2, 2, 0, 0, 0, 9, 0, 0, 0]}
flutter: {address: 0c:8b:95:77:10:ca, serviceId: 0000faf0-0000-1000-8000-00805f9b34fb, characteristicId: 0000faf2-0000-1000-8000-00805f9b34fb, value: [1, 1, 161, 2, 2, 0, 0, 0, 9, 0, 0, 0]}
flutter: DEBUG - 2023-12-15 10:20:44.844795   MonitorEnableResponse decode
flutter: DEBUG - 2023-12-15 10:20:48.869576   AppLifecycleState.inactive
flutter: DEBUG - 2023-12-15 10:20:52.536680   Send SET: Instance of 'KeepAliveRequest'
flutter: Received Message : {_type: response, _id: 7, result: null}
flutter: Received Message : {_type: valueChangedNotification, subscriptionId: 1, value: [1, 1, 161, 2, 3, 0, 0, 0, 2, 0, 0, 0]}
flutter: {address: 0c:8b:95:77:10:ca, serviceId: 0000faf0-0000-1000-8000-00805f9b34fb, characteristicId: 0000faf2-0000-1000-8000-00805f9b34fb, value: [1, 1, 161, 2, 3, 0, 0, 0, 2, 0, 0, 0]}

.....

Windows 10 log (with issue):
flutter: DEBUG - 2023-12-15 09:43:34.337885   Instance of 'BleClientConnectionStateUpdateEvent'
flutter: Received Message : {type: response, id: 0, result: BluetoothLE#BluetoothLE00:0a:cd:2f:ad:2f-7c:87:ce:0d:ef:da}
flutter: Received Message : {type: response, id: 1, result: [{00001801-0000-1000-8000-00805f9b34fb}, {00001800-0000-1000-8000-00805f9b34fb}, {0000faf0-0000-1000-8000-00805f9b34fb}]}
flutter: DEBUG - 2023-12-15 09:43:34.928884   Instance of 'BleClientConnectionStateUpdateEvent'
flutter: DEBUG - 2023-12-15 09:43:34.929877   Get hello
flutter: DEBUG - 2023-12-15 09:43:34.929877   Send SET: Instance of 'HelloRequest'
flutter: Received Message : {type: response, id: 2, error: Errore non specificato.
3

Requested characteristic not found}
flutter: Received Message : {type: response, id: 3, error: Errore non specificato.
3

Requested characteristic not found}
flutter: ERROR - 2023-12-15 09:43:35.162883   Errore non specificato.
3

Requested characteristic not found
flutter: ERROR - 2023-12-15 09:43:35.162883   runZonedGuarded 2: #0      WinConnector.invokeMethod (package:win_ble/src/utils/win_connector.dart:52:32)
<asynchronous suspension>
#1      WinBle.subscribeToCharacteristic.<anonymous closure> (package:win_ble/src/win_ble.dart:300:13)
<asynchronous suspension>
#2      WinBle.subscribeToCharacteristic (package:win_ble/src/win_ble.dart:296:5)
<asynchronous suspension>

flutter: ERROR - 2023-12-15 09:43:35.162883   Errore non specificato.
3

Requested characteristic not found
flutter: ERROR - 2023-12-15 09:43:35.162883   runZonedGuarded 2: #0      WinConnector.invokeMethod (package:win_ble/src/utils/win_connector.dart:52:32)
<asynchronous suspension>
#1      WinBle.subscribeToCharacteristic.<anonymous closure> (package:win_ble/src/win_ble.dart:300:13)
<asynchronous suspension>
#2      WinBle.subscribeToCharacteristic (package:win_ble/src/win_ble.dart:296:5)
<asynchronous suspension>

flutter: Received Message : {type: response, id: 4, result: null}

.....

@rohitsangwan01 is there any news on this issue? Do you need any more information? Thank you.

@ffamar can you cross check with another windows 10 issue to verify if its windows 10 issue or something else

@ffamar can you cross check with another windows 10 issue to verify if its windows 10 issue or something else

@rohitsangwan01 I got the same result using 2 notebooks and 1 desktop of different brands.

I'm having the same issue. I can produce an MSIX installer which runs fine on Windows 11. On Windows 10, I can connect to the peripheral device but the subsequent interaction yields an "Unspecified error  Requested characteristic not found"

I'm having the same issue with win_ble 1.1.1 on windows 11 on a dell latitude 5540. "errore non specificato" (in italian, means  "unspecified error") when i try to subscribe to the characteristics. On another pc with windows 11 the same code works.

@Giovanni-Fantin - recommend you switch the universal_ble

i don't have Windows 10 to test yet, that's why am not able to replicate this issue, though am also one of the maintainer in universal_ble plugin, both plugins uses different Apis, on win_ble we are using win32 and on universal_ble we are using winrt apis, but will also try to fix this one if i get access to Windows10


Document and test Vehicle.close

@tcr3dr Can you explain the Vehicle.close semantics - when/why should this be called, and what exactly is the state of the vehicle after doing so? Is this called automatically when a script exits/falls through? etc.

This will be fixed by #384 (adds API reference for method). This is already documented in the migration guide.

Just changing this issue to note there needs to be a few more tests for vehicle.close() to ensure its behavior matches that in the docs.

Or update the docs to make sure they match what it does -  I more or less just documented what I thought it should do and was relying on you to catch me being wrong.


Add update gofmt script and fix gofmt

Two fixes:

Add update-gofmt script

Fix gofmt

@ICGog @deepak-vij @shivramsrivastava

/assign @shivramsrivastava

@m1093782566 Thanks for this PR.
Do we need a separate PR for the changes in PR #27, can't it be merged in this same PR itself?

@shivramsrivastava
This PR is related to gofmt and #27 is about golint. I would like to keep them seperated.

/lgtm

/retest

@shivramsrivastava
The CI job pull-poseidon-verify should not pass for now, right?
Is it a blocking issue that prevents this PR getting merged?

@m1093782566 its a bug in the verify script.
The script it trying to use a missing command  'bc'.
I have rasied a PR #30 , will do a reset after that is merged.

@shivramsrivastava
PR changed, would you please take a look again and LGTM it if you are happy with it :)

/retest

@shashidharatd
Comments are addressed now. Please take a look again when you have a chance. Thanks!

/lgtm

/retest

/retest

@shashidharatd @shivramsrivastava
/retest seems dose not work for pull-poseidon-bazel job.

In addition, close this PR and reopen would not trigger pull-poseidon-bazel job re-run once.

/test pull-poseidon-bazel

[trigger plugin was re-enabled]
/test pull-poseidon-bazel

ah this needed #33 as well
/test pull-poseidon-baze

/test pull-poseidon-bazel


How get json in OnFileComplete event

Hi !
I would like to know how I can get an address in the metadata of the file.
Currently I'm trying like this:
var dataJson = metadataDict ["datajson"]. GetString (Encoding.UTF8);
but the return is a string "[object object]"
Thank you !

Sounds like an encoding error from the client.  "[object object]" is what you get if you do obj.toString() in JavaScript instead of JSON encoding (using JSON.stringify).

Did you manage to solve this issue?

No answer for 6 days. Closing due to inactivity.


Add unauthorized method for Laravel Http Client

While working with the Http Client and making requests to services that require some form of authentication,  I found that I have been doing:
$response = Http::withToken('apikey')->post('http://example.com', [
      'data' => $data,
]);

if ($response->status() == 401 ||  $response->status() == 403) {
    # authentication error
}

and it became repetitive. With this pull request, everything written above can now be achieved with:
$response = Http::withToken('apikey')->post('http://example.com', [
      'data' => $data,
]);

if ($response->unauthorized()) {
    # authentication error
}

Hopefully someone else finds it useful 😀

Just check the status code you want directly.

If you still want to use this as a method the Response class is macroable.

Alright Thanks for the tip @dennisprudlo


Remove EE and OSS differentiation. signature_v4 access via riak-cs.conf

Some parts commented rather than removed at this stage in case they later prove necessary.

Some comments from old guy.
At first, I'm very happy to look at the new activity in this repo :smile:
This comment does not to intent two points below should be addressed
before merge, but just to notice some old issues/concerns.

rebar.config.script was just for OSS/EE deference. It can be removed completely
if there is only one OSS flavor. rebar.config will just suffice.
v4 authentication is NOT COMPLETED yet.  The current status is not as secure as S3 is [1], and some API does not support v4 [2] [3].
I agree to make it accessible from conf, but it may be better to include some "unfinished" caution in the @doc.

[1] https://github.com/basho/riak_cs/issues/1059
[2] https://github.com/basho/riak_cs/issues/1058
[3] https://github.com/basho/riak_cs/issues/1060

I tried removing rebar.config.script but CS refused to make packages without it. Guessing it's a packaging issue. I will see if I can correct for that in future updates.
Thank you for the additional v4 comments. I have changed the comment in our internal copy of riak-cs.conf to "experimental support" and that will probably be reflected here sooner or later.

Thanks for the update.
Just for the future change, a comment about rebar.config.script (my previous comment was insufficient, sorry).
It creates pkg.vars.config file for packaging. Because the file was different for OSS and EE versions, it was created dynamically. Now OSS version is the only one, the file can be static.
What should be done are:

remove the line from .gitignore https://github.com/basho/riak_cs/blob/develop/.gitignore#L14, and
add the file to git

Thanks again!

Oops, I missed another thing the script does.
git mv dialyzer.ignore-warnings.ee dialyzer.ignore-warnings is needed.

Looks nice!

Thank you for your help. Looking to tag this as 2.1.2rc2


Net Core version 5 Template?

Hello,
can you release a net core 5 version of the project or guide us how to do the upgrades ourselves. i tried using updated assemblies and framework target but it broke the project for me.
thanks

hi @aloksharma1 ,
Sure will plan .Net 5 version in few days. or you can share detail of error or even screenshot of it. I can help you on it.

hi, thanks for replying... most issues are coming from health checks upgrade (it cant find db config implementations) and some issues with mediator.. you can see them just by changing target framework and upgrading the dll versioons from nuget or dotnet cli.

Regarding Health check, yes even I found there is some issue on it when I check few months back. sure will try to upgrade to .net 5 next month when I get free time.. you can watch this repo for any changes on it.

Hi @aloksharma1 ,
I have upgraded .net 5 with healthcheck feature. You can take latest code
Regards,
Amit Naik


Visa

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

@ramiksa11 please fill out the issue template if there is an issue with the site. Closing issue as there is not enough information to help. Please comment back if you have more information. Happy coding!


ocaml-ng.ocamlPackages_4_11.ocaml: 4.11.0+α3 → 4.11.0+β1

Motivation for this change
Announcement<EMAIL_ADDRESS>Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build ocaml-ng.ocamlPackages_4_11.utop


Breaking change in purego DLL stuff

Latest version does not compile on Darwin anymore.
purego.DLopen and purego.DLsym introduced an additional error return variable.

Which version of purego are you using?

I just tried out oto on my mac, using
go get github.com/hajimehoshi/oto
i have no clue, which purego version was installed as a dependency,
but i expect it to compile out of the box.

You can see the version of oto at your go.mod

My guess is that you are using v0.3.0-alpha versions. purego's 0.2.0 and 0.3.0 are incompatible.


Question: Can be used to take photo

Hi, is it possible to use this lib to take photo?

@aresares Hello, It can be used to capture photos and videos and also select recent photos.


Update to ACK runtime v0.27.1, code-generator v0.27.1

Issue #, if available:
Description of changes:

Update to ACK runtime v0.27.1, code-generator v0.27.1
requirements.txt is updated to refer commit 5a09bbdb961ea14a65b15b63769134125023ac61 from test-infra

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/retest


Plumb authorization through tasks

Closes #11668, #11669.

Looks good, only saw one stylistic issue, nothing substantive.


Ionic3: Does not find the provider after assembly with the flag "--prod"

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x]3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
After assembling and running the application in production mode on the device, an error is thrown about not finding a provider. In the development mode, there is no such problem. All pages of the application I switched to lazy loading mode. All components and pipes have their own module, which is connected as necessary to the page modules. Application providers are imported only to the main module.

Expected behavior:
Correct application startup
Steps to reproduce:

ionic cordova:run android --prod

Related code:

app.module.ts

@NgModule({
  declarations: [
    MyApp
  ],
  imports: [
    BrowserModule,
    BrowserAnimationsModule,
    HttpModule,
    IonicModule.forRoot(MyApp, {}),
    NavigationMenuComponentModule
  ],
  bootstrap: [IonicApp],
  entryComponents: [
    MyApp
  ],
  providers: [
    services,
    NATIVE_PLUGIN_PROVIDERS,
    { provide: ErrorHandler, useClass: IonicErrorHandler }
  ]
})
export class AppModule { }

components.module.ts

@NgModule({
    declarations: [
        components_list
    ],
    entryComponents: [
        /** entryComponents **/
    ],
    exports: [
        components_list
    ],
    imports: [
        CommonModule,
        FormsModule,
        ReactiveFormsModule,
        PipesModule,
        BadgeModule,
        ItemModule,
        IconModule,
        ButtonModule,
        CardModule,
        ListModule,
        BackdropModule,
        GridModule,
        LabelModule,
        AvatarModule,
        CheckboxModule,
        InputModule,
        ThumbnailModule
    ]
})
export class ComponentsModule { }

pipes.module.ts

@NgModule({
    declarations: [
        pipes_list
    ],
    exports: [
        pipes_list
    ]
})
export class PipesModule { }

Ionic info:
Cordova CLI: 6.5.0
Ionic Framework Version: 3.0.0-beta.3
Ionic CLI Version: 3.0.0-beta.5
ios-deploy version: Not installed
ios-sim version: Not installed
OS: macOS Sierra
Node Version: v7.7.2
Xcode version: Xcode 8.2.1

Hello, thanks for using Ionic! Would you be able to share a repo to your app? Id love to take a look at it myself. Also, could you look at this branch of the conf app. That is our ionic 3 branch of the conf app with lazy loading set up and may be a good example for you.

Yes, before moving the project to version 3, I was guided by this repo and this document. As I mentioned above in the development mode it works without errors, the error is only displayed at the start in the device. It seems to me that the error is thrown after the "ngc" compilation. Unfortunately, the repository is private. I can only provide sections of code. Any ideas?

Is it possible to disable compression after ngc in the production assembly as a way to get a readable stack of errors?

Got ya. So one question, are you using this provider from a component or a page?
Is it possible to disable compression after ngc in the production assembly as a way to get a readable stack of errors?

Not that we are aware of at the moment

@konrin I have a similar application structure and get the same error only on production builds. You can pass a custom uglify config to app-scripts to find out what t really is:
module.exports = {
  mangle: false,
  compress: false,
  comments: false
};

For me it's one of my custom services. No joy trying to fix it so far though.

I found the issue with the provider in your case! This was due to the fact that aot is incorrect processed connection providers.

Broken connection

let providers = [
    /** providers list **/
]

if(ENV_NAME.indexOf('dev') > -1) {
    providers.push(Firebase);
} else {
    providers.push({ provide: Firebase, useClass: FirebaseMock })
}

In development mode it works but not after aot

Hello @konrin , are you still having this issue?

No, I decided it, thank you.

No problem!


Open vSwitch bridged interfaces are not deleted after container stop

Ubuntu 14.04, docker 1.6.1, ovs-vsctl (Open vSwitch) 2.0.2
Steps to reproduce:

start container that is bridged to an existing OVS bridge:

# pipework ovs-int $(docker run -d -p 8080:8080 satnam6502/go-webserver-helloworld
/gopath/src/github.com/satnam6502/go-webserver-helloworld/hello) dhcp 0a:d0:cc:e0:02:01

identify container

# docker ps
CONTAINER ID        IMAGE                                       COMMAND                CREATED              STATUS              PORTS                    NAMES
084bd6d0b240        satnam6502/go-webserver-helloworld:latest   "/gopath/src/github.   About a minute ago   Up About a minute   <IP_ADDRESS>:8080->8080/tcp   insane_mccarthy

stop container:

# docker stop 084bd6d0b240

interface that have been created for container is still up and bridged to OVS:

# ovs-vsctl show
    Bridge ovs-int
        Port "veth1pl6217"
            Interface "veth1pl6217"
        Port ovs-int
            Interface ovs-int
                type: internal

Expected result:
Virtual interface should be unbridged & deleted after container stop as stated at documentation

Hi! Thanks for reporting this. Unfortunately, I don't have an OVS setup to reproduce this here :(
Could you also attach the output of docker inspect for this container?

docker inspect a0ab9c643513
[{
    "AppArmorProfile": "",
    "Args": [],
    "Config": {
        "AttachStderr": false,
        "AttachStdin": false,
        "AttachStdout": false,
        "Cmd": [
            "/gopath/src/github.com/satnam6502/go-webserver-helloworld/hello"
        ],
        "CpuShares": 0,
        "Cpuset": "",
        "Domainname": "",
        "Entrypoint": null,
        "Env": [
            "HOME=/",
            "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/goroot/bin:/gopath/bin",
            "DEBIAN_FRONTEND=noninteractive",
            "GOROOT=/goroot",
            "GOPATH=/gopath"
        ],
        "ExposedPorts": {
            "8080/tcp": {}
        },
        "Hostname": "a0ab9c643513",
        "Image": "satnam6502/go-webserver-helloworld",
        "Labels": {},
        "MacAddress": "",
        "Memory": 0,
        "MemorySwap": 0,
        "NetworkDisabled": false,
        "OnBuild": null,
        "OpenStdin": false,
        "PortSpecs": null,
        "StdinOnce": false,
        "Tty": false,
        "User": "",
        "Volumes": null,
        "WorkingDir": ""
    },
    "Created": "2015-06-16T09:16:06.253802302Z",
    "Driver": "aufs",
    "ExecDriver": "native-0.2",
    "ExecIDs": null,
    "HostConfig": {
        "Binds": null,
        "CapAdd": null,
        "CapDrop": null,
        "CgroupParent": "",
        "ContainerIDFile": "",
        "CpuShares": 0,
        "CpusetCpus": "",
        "Devices": [],
        "Dns": null,
        "DnsSearch": null,
        "ExtraHosts": null,
        "IpcMode": "",
        "Links": null,
        "LogConfig": {
            "Config": null,
            "Type": "json-file"
        },
        "LxcConf": [],
        "Memory": 0,
        "MemorySwap": 0,
        "NetworkMode": "bridge",
        "PidMode": "",
        "PortBindings": {
            "8080/tcp": [
                {
                    "HostIp": "",
                    "HostPort": "8080"
                }
            ]
        },
        "Privileged": false,
        "PublishAllPorts": false,
        "ReadonlyRootfs": false,
        "RestartPolicy": {
            "MaximumRetryCount": 0,
            "Name": "no"
        },
        "SecurityOpt": null,
        "Ulimits": null,
        "VolumesFrom": null
    },
    "HostnamePath": "/media/vconts/docker/containers/a0ab9c643513553b4149fb48e40d6a4d9da4b8b52224c6bbf5f823bec5cd77d3/hostname",
    "HostsPath": "/media/vconts/docker/containers/a0ab9c643513553b4149fb48e40d6a4d9da4b8b52224c6bbf5f823bec5cd77d3/hosts",
    "Id": "a0ab9c643513553b4149fb48e40d6a4d9da4b8b52224c6bbf5f823bec5cd77d3",
    "Image": "f01bb1047dd12a0c4ec25b98923289c246c07fa9ed0478014e06e869c8542cdd",
    "LogPath": "/media/vconts/docker/containers/a0ab9c643513553b4149fb48e40d6a4d9da4b8b52224c6bbf5f823bec5cd77d3/a0ab9c643513553b4149fb48e40d6a4d9da4b8b52224c6bbf5f823bec5cd77d3-json.log",
    "MountLabel": "",
    "Name": "/drunk_franklin",
    "NetworkSettings": {
        "Bridge": "",
        "Gateway": "",
        "GlobalIPv6Address": "",
        "GlobalIPv6PrefixLen": 0,
        "IPAddress": "",
        "IPPrefixLen": 0,
        "IPv6Gateway": "",
        "LinkLocalIPv6Address": "",
        "LinkLocalIPv6PrefixLen": 0,
        "MacAddress": "",
        "PortMapping": null,
        "Ports": null
    },
    "Path": "/gopath/src/github.com/satnam6502/go-webserver-helloworld/hello",
    "ProcessLabel": "",
    "ResolvConfPath": "/media/vconts/docker/containers/a0ab9c643513553b4149fb48e40d6a4d9da4b8b52224c6bbf5f823bec5cd77d3/resolv.conf",
    "RestartCount": 0,
    "State": {
        "Dead": false,
        "Error": "",
        "ExitCode": 2,
        "FinishedAt": "2015-06-16T09:16:33.017358979Z",
        "OOMKilled": false,
        "Paused": false,
        "Pid": 0,
        "Restarting": false,
        "Running": false,
        "StartedAt": "2015-06-16T09:16:06.516538418Z"
    },
    "Volumes": {},
    "VolumesRW": {}
}
]

I finally got to reproduce this!
It looks like it's a normal behavior of OVS; when a port is deleted, it remains in the database and it still shows up. It doesn't seem to be related to pipework. If someone has an idea about how to change this, let me know!
But meanwhile I'll close this issue.
Thank you!


Some changes in "Connect application to the network"

This section, https://github.com/IBM/Create-BlockchainNetwork-IBPV20/blob/master/README.md#5-deploy-blockchain-network-smart-contract-on-the-network, has been modified in a recent release. A few users have complained that they can't see this step.
Please, if you have the time, update this section with the new flow on how to connect via the SDK, https://cloud.ibm.com/docs/blockchain?topic=blockchain-ibp-console-smart-contracts#ibp-console-smart-contracts-connect-to-SDK-panel

Please revise this tutorial. After the latest release, 2.5, some of the instructions here are outdated. Please refer to this documentation, https://cloud.ibm.com/docs/blockchain?topic=blockchain-ibp-console-build-network

@n0rbs I've started working on the updates for this pattern and should have it merged soon.

can we load the provided application as a cloud function in IBM cloud ??


OCM-5632 | feat: support password generate/hash in terraform provider

Signed-off-by: marcolan018<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR is to add the feature for secure cluster credential to terraform provider.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story (OCM-xxxx):
Fixes #
https://issues.redhat.com/browse/OCM-5632
Change type

[x] New feature
[ ] Bug fix
[ ] Build
[ ] CI
[ ] Documentation
[ ] Performance
[ ] Refactor
[ ] Style
[ ] Unit tests
[ ] Subsystem tests

Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.

/ok-to-test

/retest

/ok-to-test

Please complete the following:

Generate documentation by running make generate
Add subsystem tests and fix existing tests following the changes

@nirarg
in the latest commit, I added the subsystem test for the changes, can you take a look at it? Thanks

/approve
/retest
@bardielle can you have another review here?

/lgtm
/approve


XML Support

I read this issue: https://github.com/thephpleague/fractal/issues/178
And it is mentioned that it should be possible to output xml somehow with a custom serializer.
However I don't get yet how this should work exactly.
Can anyone help with some more details?
I use the following endpoint in element-api.php
// Sitemap
'sitemap' => function() {
	return [
		'criteria' => [
			'section' => 'pages',
		],
		'transformer' => new SitemapTransformer(),
		'serializer' => new XmlSerializer()
	];
},

But then i'm unsure how to write the XmlSerializer. As I see almost all methods from SerializerAbstract should return an array so how can I make it return xml?
/edit1:
As I see now the DefaultController uses the Scope class which does this:
public function toArray() {
 ...
 return $data + $meta;
}

And besides the DefaultController also uses a JSON Formatter:
public function actionIndex(string $pattern): Response {
 ...
 $formatter = new JsonResponseFormatter([ ... ]);
}

So basically that makes it look impossible to actually have an xml response. Did this change over time of why was it possible before?

In the meantime I wrote myself a module to return xml myself. The controller uses this code to return xml:
public function actionIndex() {
  $response = Craft::$app->getResponse();
  $response->content = '<myxml>...</myxml>';
  $response->format = Response::FORMAT_XML;
  return $response;
}

Why did you close this? It's still a very valid issue if true? I was just about to start down this journey.

Here is the docs on the Serializers: https://fractal.thephpleague.com/serializers/
If you have an XML serializer you built, it needs to implement the SerializerAbstract class for Fractal to pick it up.

This is actually a valid point, the serializer returns arrays of data which then gets converted into JSON by the scope, with the toJson method, so the serializer only ever returns array data.
Rather than the scope returning JSON, you can use toArray and do something like this
$data = $fractal->createData()->toArray();

$sxml = new SimpleXMLElement('<root />');
array_walk_recursive($data, [$sxml, 'addChild]);

return $xml = $sxml->asXML();

this is obviously not ideal. when theSerializer could build the SimpleXML object on the fly, and return the XML, so maybe we need to change the burden of where the conversion (not serialization) to JSON / XML happens.

Im going to close this for now as it seems like we found a good solution and no follow up. If this is still an issue please reopen.


Remove duplicate dbt script entry

Description
Resolves: https://github.com/dbt-labs/dbt-core/issues/5314
There is a duplicate dbt script entry in setup.py
When installing the wheel, the console_script is overwriting the hand-written code in scripts/dbt. pip should not be overwriting files during installation and will begin to log warnings at some point in future. See: https://github.com/pypa/pip/issues/4625#issuecomment-320506401
You can confirm this by installing dbt-core into a venv. The contents of the dbt script is an autogenerated console_script defined in entry_points and not the handwritten script.
/tmp
myvenv ❯ cat myvenv/bin/dbt
#!/private/tmp/myvenv/bin/python3.9
# -*- coding: utf-8 -*-
import re
import sys
from dbt.main import main
if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw|\.exe)?$', '', sys.argv[0])
    sys.exit(main())

This also results in errors when using pypa/installer which is stricter than pip.
/tmp
❯ python3 -m venv dbt_venv

/tmp
❯ source dbt_venv/bin/activate.fish

/tmp
dbt_venv ❯ python -m pip install installer
Collecting installer
  Using cached installer-0.5.1-py3-none-any.whl (452 kB)
Installing collected packages: installer
Successfully installed installer-0.5.1
WARNING: You are using pip version 22.0.4; however, version 22.1.1 is available.
You should consider upgrading via the '/private/tmp/dbt_venv/bin/python -m pip install --upgrade pip' command.

/tmp
dbt_venv ❯ python -m installer --destdir /tmp/example ~/Downloads/dbt_core-1.1.0-py3-none-any.whl
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File<EMAIL_ADDRESS>line 87, in _run_code
    exec(code, run_globals)
  File "/private/tmp/dbt_venv/lib/python3.9/site-packages/installer/__main__.py", line 85, in <module>
    _main(sys.argv[1:], "python -m installer")
  File "/private/tmp/dbt_venv/lib/python3.9/site-packages/installer/__main__.py", line 81, in _main
    installer.install(source, destination, {})
  File "/private/tmp/dbt_venv/lib/python3.9/site-packages/installer/_core.py", line 109, in install
    record = destination.write_file(
  File "/private/tmp/dbt_venv/lib/python3.9/site-packages/installer/destinations.py", line 203, in write_file
    return self.write_to_fs(
  File "/private/tmp/dbt_venv/lib/python3.9/site-packages/installer/destinations.py", line 167, in write_to_fs
    raise FileExistsError(message)
FileExistsError: File already exists: /tmp/example/private/tmp/dbt_venv/bin/dbt

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

@groodt  Thanks for the help getting this into dbt-core!!


Update to AutoMapper 4.2.0

Hi,
are you planning an update to AutoMapper 4.2.0? And if so, when?
Thx!
Andreas

Is there any particular reason why you can't just use AutoMapper 4.2 with this package?

Hi,
in AutoMapper 4.1.1 there are constructores for MappingEngine:
public MappingEngine(IConfigurationProvider configurationProvider)
      : this(configurationProvider, MappingEngine.DictionaryFactory.CreateDictionary<TypePair, IObjectMapper>(), MappingEngine.DictionaryFactory.CreateDictionary<ExpressionRequest, LambdaExpression>(), configurationProvider.ServiceCtor)
    {
    }

public MappingEngine(IConfigurationProvider configurationProvider, IDictionary<TypePair, IObjectMapper> objectMapperCache, IDictionary<ExpressionRequest, LambdaExpression> expressionCache, Func<Type, object> serviceCtor)
    {
     // Some Code
    }

In AutoMapper 4.2.0 there is only one constructor
public MappingEngine(IConfigurationProvider configurationProvider, IMapper mapper)
    {
      // Some code
    }

Now, i´ll get an Exception when using your RegisterMapper method

The type AutoMapper.MappingEngine does not have a constructor that takes the parameters (IConfigurationProvider)

You are injection one constructor parameter in the RegisterMapper Method:
...
....
UnityContainerExtensions.RegisterType<IMappingEngine, MappingEngine>(container, (LifetimeManager) new ContainerControlledLifetimeManager(), new InjectionMember[1]
      {
        (InjectionMember) new InjectionConstructor(new object[1]
        {
          (object) typeof (IConfigurationProvider)
        })
      });
....
....

Thx!
Andreas

Okay, I'll try to fix it. Are you using Unity 3 or Unity 4?

Hi,
great! Thx!
i´m using Unity 4.0.1

So you're using 4.0.12 package?

Hi,
yes! I´m using 4.0.12!

5.0.16 is up for you to try.

Thx for the quick help!
I will test it on Monday!

Hi!
Seems to work perfectly :-)!
Thx!


zref-clever User Manual desperately needs examples of usage

The User Manual needs considerable expansion written from the viewpoint of, especially, the new user, through inclusion of EXAMPLES of usage!
Its current focus seems to be more from the viewpoint of the author(s), as a reference manual that enumerates exhaustively all the options.
I'm a fairly experienced user of LaTeX, including of hyperref and cleveref, but still I find the zref-cleveref User Manual forbidding

Hi @murrayE , I thank you for your feedback, but that seems unfair. Indeed the manual tries to cover the options, this is intended. But the only shortcoming you mentioned, which is the supposed lack of examples, is simply not true: there's a whole section of "How tos", covering some use cases of configuration I thought would be common. Besides, the basics are the same as the standard referencing system: set a label (with \zlabel in this case), and make a reference (with \zcref). And that's about it, and that is covered in the short "User interface" section.

Gustavo,
My purpose was not to criticize per se but to offer comment towards making the package more useable!
You are quite correct about the How-tos section beginning on page 22, which is useful.
But it would still help greatly to have some examples interspersed with all the detailed reference material on the preceding 21 pages.
The User Interface section gives no actual examples of what kind of options might be useful, or how.
In fact, to entice new users of the package to get started, the most effective approach could be to put some typical examples of usage first and then follow that by the reference material.
Murray

On 14 Feb2022, at 4:19 PM, Gustavo Barros @.***> wrote:
Hi @murrayE https://github.com/murrayE , I thank you for your feedback, but that seems unfair. Indeed the manual tries to cover the options, this is intended. But the only shortcoming you mentioned, which is the supposed lack of examples, is simply not true: there's a whole section of "How tos", covering some use cases of configuration I thought would be common. Besides, the basics are the same as the standard referencing system: set a label (with \zlabel in this case), and make a reference (with \zcref). And that's about it, and that is covered in the short "User interface" section.
—
Reply to this email directly, view it on GitHub https://github.com/gusbrs/zref-clever/issues/8#issuecomment-1039571581, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABILR2NXZ7PONSV4NNJMRMTU3FWUNANCNFSM5OMRVRIQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.

——
Murray Eisenberg                                @.***
Mathematics & Statistics Dept.
Lederle Graduate Research Tower      phone<PHONE_NUMBER> (H)
University of Massachusetts
710 North Pleasant Street
Amherst, MA 01003-9305

A "typical" example is just something like:
\documentclass{article}
\usepackage{zref-clever}
\usepackage{hyperref}
\begin{document}
\section{Section 1}
\zlabel{sec:section-1}
A reference \zcref{sec:section-1}
\end{document}

But I don't think this would add much to the documentation. It is just the usual stuff. And it "not being there" certainly is no significant barrier to interested users, in my judgement. Indeed, I would argue that even if the How-tos are a little more elaborate, they also cover the need of supplying examples of what the most basic (and probably the most common) usage is.
And most certainly, it is not granted to say the User manual "desperately needs EXAMPLES of usage"...

I see no reason not to include such a very basic example in the documentation: you never know what experience a potential user may  have.
P.S. My interest in zref-clever is that, for a book-length project, I’ve been using hyperref + cleveref + crossreftools along with babel-french, which originally caused havoc with my use of colons in labels (e.g., thm:main, lem:first). This required at least two hacks offered on tex.stackexchange.com http://tex.stackexchange.com/ to get around babel-french's making he colon as an active character.
I was trying to see if zref-clever would eliminate the need for those hacks — on the principle that it is safer in general to use packages “as is” rather than hacked — and possibly allow an alternative to crossreftools for my purposes.
But with so few examples, I’m finding it hard to tell.

On 14 Feb2022, at 4:45 PM, Gustavo Barros @.***> wrote:
A "typical" example is just something like:
\documentclass{article}
\usepackage{zref-clever}
\usepackage{hyperref}
\begin{document}
\section{Section 1}
\zlabel{sec:section-1}
A reference \zcref{sec:section-1}
\end{document}
But I don't think this would add much to the documentation. It is just the usual stuff. And it "not being there" certainly is no significant barrier to interested users, in my judgement. Indeed, I would argue that even if the How-tos are a little more elaborate, they also cover the need of supplying examples of what the most basic (and probably the most common) usage is.
And most certainly, it is not granted to say the User manual "desperately needs EXAMPLES of usage"...
—
Reply to this email directly, view it on GitHub https://github.com/gusbrs/zref-clever/issues/8#issuecomment-1039599033, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABILR2IT3DJA5EGI663MKETU3FZV3ANCNFSM5OMRVRIQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.

——
Murray Eisenberg                                @.***
Mathematics & Statistics Dept.
Lederle Graduate Research Tower      phone<PHONE_NUMBER> (H)
University of Massachusetts
710 North Pleasant Street
Amherst, MA 01003-9305

P.S. My interest in zref-clever is that, for a book-length project, I’ve been using hyperref + cleveref + crossreftools along with babel-french, which originally caused havoc with my use of colons in labels (e.g., thm:main, lem:first). This required at least two hacks offered on tex.stackexchange.com http://tex.stackexchange.com/ to get around babel-french's making he colon as an active character.
I was trying to see if zref-clever would eliminate the need for those hacks — on the principle that it is safer in general to use packages “as is” rather than hacked — and possibly allow an alternative to crossreftools for my purposes.
But with so few examples, I’m finding it hard to tell.

I'm glad you're interested in the package, and even happier if it happens to solve some of the problems you face. I do hope you find it useful. However, there are trade-offs, and you should ponder them. Do take a look at section 11 "Limitations". Also, depending on the nature of the project you are into, do read section 2 "Warning" and consider it carefully.

I see no reason not to include such a very basic example in the documentation: you never know what experience a potential user may  have.

I'll keep your suggestion in mind but, for the time being, I think the "How-tos" section already covers sufficiently basic use cases. So I'm closing this one.

I see no reason not to include such a very basic example in the documentation: you never know what experience a potential user may have.

+1 on this. Examples do always help. The more covering of different experience levels of users, the better for any kind of documentation (for both directions, from beginners to very very experienced users)

+1 on this. Examples do always help. The more covering of different experience levels of users, the better for any kind of documentation (for both directions, from beginners to very very experienced users)

The second how-to currently in the manual is the following:
\documentclass{article}
\usepackage{zref-clever}
\newtheorem{lemma}{Lemma}[section]
\begin{document}
\section{Section 1}
\begin{lemma}\zlabel{lemma-1}
A lemma.
\end{lemma}
\zcref{lemma-1}
\end{document}

It is simple enough, and it does illustrate basic usage. There is really no need to add a how-to on "making a cross-reference to a section". I do hope I'll be able to extend the How-to section one day, but more in the direction of more interesting examples, not in adding even more basic ones than the basic ones already there. The manual presumes basic knowledge on how to use LaTeX cross-references and will continue to do so, it does not intend to replace a "LaTeX Introduction" for which there are plenty and better sources.

Ok, vox populi. I have added to my todo list to add a "zref-clever for the impatient" section to the manual, and will do so when I have some time.
However, @dbitouze , regarding comparisons with cleveref, what I've tried to emphasize in the manual is that zref-clever can reach parity to the standard referencing system in terms of features. Or, at least, near parity, as much as the support for zref allows. And that's what the example you mentioned tried to capture. That's deliberate. I do think cleveref is a great package, I have been its user for several years, and zref-clever itself owes much to it, so I would not like to make the relation between the packages any sort of "contest". Even if, admittedly, this may make the path to information a little longer.
That said, if you have a concrete example (or examples) to suggest for the "How-tos" section within this spirit (that is, something which is interesting on its own), I'd be glad to consider it for inclusion. But, if you do so, please open a separate issue for it, and not revive an issue that was closed for due reason to add further (different) requests.

Thanks for considering!
About "contest", I understand. I rephrase: "A useful addition would be examples which highlight the differences between of zref-clever and cleverref."
About concrete example (or examples) to suggest, the point is that I hoped since a long time to have the time to study zref-clever in details, but I never succeed. And that's why it would be helpful to have a quick overview: it would help to read what follows :smile:. If I succeed and find useful examples, I'll open a new issue (sorry to use the current one but opening a new one for my addition would probably be considered as a duplicate).

About concrete example (or examples) to suggest, the point is that I hoped since a long time to have the time to study zref-clever in details, but I never succeed. And that's why it would be helpful to have a quick overview: it would help to read what follows :smile:.

It sounds like you'd be interested in https://github.com/gusbrs/zref-clever/issues/9. I'd say this is as far as I'm willing to go with a comparison.


Launch App no longer works on Quest 2

it seems that "Launch app" no longer works with Quest 2/Android 10. In order to launch BMBF, adb "am start" just doesn't open it, however, using "adb shell monkey -p com.weloveoculus.BMBF -v 500" does launch it

Fixed


Divider and Position Issue

Thanks for the component @isaacplmann
I am facing two issues here,

Even when I set the 'divider' to true (tried in both template and menu options approach), the divider is not getting visible.
The position is not proper, it always goes with some offset to top left position.

Note that, I am using the styles defined for .angular2-context-menu in the documentation.
Can you please help me resolve this? Thanks in advance.

Can you paste in the relevant code?  Show me your css, template file and typescript that relate to the context menu.
Thanks.

Thanks for your response @isaacplmann .
The issues are fixed. The issue is with the relative positioning override and divider works fine when defining it in the template.
I have referred your demo project and resolved these issues. Thanks.


24374 Fix tf.einsum so it computes the trace correctly

This is in relation to and fixes issue #24374.  tf.einsum is not.  Einsum is not properly calculating Trace.  This fix functions by properly identifying a trace call and utilizing math_ops.trace.  Previously It would receive an error
Subscript not supported: an axis appears more than once: i
or something similar.  Before that error, people with older versions were receiving the wrongly calculated value.

The tests passed on my end @drpngx

So this is working on my end, I'll continue to tweak it to see if there are any better approaches while I await feedback.

Thoughts?

OK, let me run the tests again to see what's wrong.

Did you test this test?
FAIL: test_simple (__main__.EinsumTest)
test_simple (__main__.EinsumTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/b/f/w/bazel-out/k8-opt/bin/tensorflow/python/special_math_ops_test.runfiles/absl_py/absl/third_party/unittest3_backport/case.py", line 37, in testPartExecutor
    yield
  File "/b/f/w/bazel-out/k8-opt/bin/tensorflow/python/special_math_ops_test.runfiles/absl_py/absl/third_party/unittest3_backport/case.py", line 162, in run
    testMethod()
  File "/b/f/w/bazel-out/k8-opt/bin/tensorflow/python/special_math_ops_test.runfiles/org_tensorflow/tensorflow/python/ops/special_math_ops_test.py", line 292, in test_simple
    self.run_test(case)
  File "/b/f/w/bazel-out/k8-opt/bin/tensorflow/python/special_math_ops_test.runfiles/org_tensorflow/tensorflow/python/ops/special_math_ops_test.py", line 359, in run_test
    self.assertLess(err, 1e-8)
AssertionError: 36.89950854256984 not less than 1e-08

I believe I did?  Let me go back and check

So not quite sure if I missed it or what but I was able to find and fix the error/bug.  The failing test was due to the added trace case('ijji').  The output is correct from the function, the issue is that the unit test does not correctly calculate the trace.  For now I removed the test and have left only one simple test case for trace('ii').  Let me know if that case is enough, otherwise, I can look into adding it back in and adding in code to ensure the unit test calculates the right value.  @drpngx

Infra :: RabbitMQ :: Connection on connection should assign uri to connection:
AssertionError: expected undefined to equal 'amqp://guest:guest@<IP_ADDRESS>:5672//?heartbeat=30'
at Context. (test/unit/infra/messaging/RabbitMQConnect.spec.js:18:32)
at process.topLevelDomainCallback (domain.js:121:23)

Infra :: RabbitMQ :: Publishing Messages with a connection and no exchange defined should reject publish call with missing exchange:
AssertionError: expected undefined to equal 'Publish failed - no exchange missing.ex on connection default is defined'
at Context.it (test/unit/infra/messaging/RabbitMQPublishing.spec.js:24:34)

Infra :: RabbitMQ :: Publishing Messages with a connection and exchange defined should not error on publish calls:
Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

Infra :: RabbitMQ :: Publishing Messages with a connection and exchange defined "after all" hook:
Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

Infra :: RabbitMQ :: Subscription "before all" hook:
Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

utils :: emails when email is valid creates and sends the email:
Error: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

Socket.io :: Integration API :: POST /api/chats when sent data is ok should emit chat event (via socket) when a chat is added:
Error: Timeout of 2000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

Yes, it would be nice to have that case work as well. Let me know when it's ready. Thanks!

@drpngx I fixed the test and added it back in.  Thanks for the patience.  Its tested on my side and should be ready

Thank you for the quick turnaround! Kicking off the tests...

Could you fix the lint errors and reupload?
FAIL: Found 6 non-whitelited pylint errors:
tensorflow/python/ops/special_math_ops_test.py:355: [C0326(bad-whitespace), ] Exactly one space required around assignment
tensorflow/python/ops/special_math_ops_test.py:356: [C0326(bad-whitespace), ] Exactly one space required around comparison
tensorflow/python/ops/special_math_ops_test.py:357: [W0311(bad-indentation), ] Bad indentation. Found 8 spaces, expected 6
tensorflow/python/ops/special_math_ops_test.py:357: [C0326(bad-whitespace), ] Exactly one space required after assignment
tensorflow/python/ops/special_math_ops_test.py:358: [W0311(bad-indentation), ] Bad indentation. Found 8 spaces, expected 6
tensorflow/python/ops/special_math_ops_test.py:360: [W0311(bad-indentation), ] Bad indentation. Found 8 spaces, expected 6

Yeah, sorry will fix those shortly

Linting issues fixed @drpngx

Testing...

Tests run and things are linted on my side, not sure what is off with the internal builds

Yeah the errors are unrelated. @ymodak it's ready.

Yay!  Thanks for the review!

Woohoo!


Don't use outdated "namespace"

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @AngularClass

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
bug

What is the current behavior?
We are using the outdated namespace in src/custom-typings.d.ts.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).

What is the expected behavior?
According to https://palantir.github.io/tslint/rules/no-namespace/ this shouldn't be used anymore because it's outdated.

What is the motivation / use case for changing the behavior?
To comply with the tslint rules.

Please tell us about your environment:

Angular version: 2.0.0-beta.X
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

thanks, done via https://github.com/AngularClass/angular2-webpack-starter/commit/9df5eff93ca3ee027324e42d8d01fc8a7b84fe7c


The pre-training loss can not converge.

Dear Author,
I tried to pre-train the model on book-corpus data only, whose training config is exactly the same as yours.
But I found that the training loss is around 0.76 after 3 epochs.
It seems that the model could not converge.
And the reconstruction images are bad.
Could you tell me what's the problem?
Thanks,
Chawdoe

Hi Chawdoe,
could you perhaps paste as much information as possible on the config you're using and your environment setup (torch version, python version, libraries, etc.)? Otherwise it's very hard for me to diagnose what might be wrong. Also have you tried to take a small subset of the data (e.g. 1000 examples) and see if the model is able to overfit it?


Math mode text colour is applied after escaped dollar sign

In Neovim v0.9.5 and typst.vim, typing an escaped dollar sign \$ is interpreted as if text after the dollar sign belonged to math mode.
For example, let's start with the following piece of text, typed into an empty buffer:
90 per month

Some text

This all looks good for me:

If I wrap 90 with a $ on both sides, it still looks fine:

However, if I change the first line to \$90 per month, everything after that part will be coloured blue:

As a matter of fact, if I have an unescaped opening $, intentionally leaving the closing $, it looks the same. (Except for the $ highlighted in red. That's an error about the missing closing $ reported by LSP.)

Thank you for the fix!


Prevent duplicate mentee applications

Purpose

The purpose of this PR is to fix #227
Goals

Prevent mentees from applying twice for the same program
Approach

Added the relevant validations
Screenshots

N/A
Checklist

[x] This PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.
[x] I have read and understood the development best practices guidelines ( http://bit.ly/sef-best-practices )
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

Related PRs

N/A
Learning

N/A

Thanks a lot for your review, obviously it was not working properly, but with your corrections, it should be better.
Sry for the failed attempt, I am fairly new to github.

Thanks for you time and your patience with me :)


[SPARK-18024][SQL] Introduce an internal commit protocol API

What changes were proposed in this pull request?
This patch introduces an internal commit protocol API that is used by the batch data source to do write commits. It currently has only one implementation that uses Hadoop MapReduce's OutputCommitter API. In the future, this commit API can be used to unify streaming and batch commits.
How was this patch tested?
Should be covered by existing write tests.

This is the same as https://github.com/apache/spark/pull/15696
but rebased with https://github.com/apache/spark/pull/15633

Test build #67855 has started for PR 15707 at commit 0647959.

This lgtm, modulo the comments in #15696

Test build #3384 has started for PR 15707 at commit 0177ded.

Test build #67855 has finished for PR 15707 at commit 0647959.

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

Merged build finished. Test PASSed.

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

This LGTM, just a minor comment

Test build #3386 has started for PR 15707 at commit 65ba5c1.

Test build #67865 has started for PR 15707 at commit 65ba5c1.

Test build #67865 has finished for PR 15707 at commit 65ba5c1.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Looks like the test failed due to a flaky test, but other than that everything else was fine. I'm going to merge this optimistically.

Test build #3384 has finished for PR 15707 at commit 0177ded.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class HadoopCommitProtocolWrapper(path: String, isAppend: Boolean)

Test build #3386 has finished for PR 15707 at commit 65ba5c1.

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