Relicense under dual MIT/Apache-2.0

This issue was automatically generated. Feel free to close without ceremony if
you do not agree with re-licensing or if it is not possible for other reasons.
Respond to @cmr with any questions or concerns, or pop over to
#rust-offtopic on IRC to discuss.
You're receiving this because someone (perhaps the project maintainer)
published a crates.io package with the license as "MIT" xor "Apache-2.0" and
the repository field pointing here.
TL;DR the Rust ecosystem is largely Apache-2.0. Being available under that
license is good for interoperation. The MIT license as an add-on can be nice
for GPLv2 projects to use your code.
Why?
The MIT license requires reproducing countless copies of the same copyright
header with different names in the copyright field, for every MIT library in
use. The Apache license does not have this drawback. However, this is not the
primary motivation for me creating these issues. The Apache license also has
protections from patent trolls and an explicit contribution licensing clause.
However, the Apache license is incompatible with GPLv2. This is why Rust is
dual-licensed as MIT/Apache (the "primary" license being Apache, MIT only for
GPLv2 compat), and doing so would be wise for this project. This also makes
this crate suitable for inclusion and unrestricted sharing in the Rust
standard distribution and other projects using dual MIT/Apache, such as my
personal ulterior motive, the Robigalia project.
Some ask, "Does this really apply to binary redistributions? Does MIT really
require reproducing the whole thing?" I'm not a lawyer, and I can't give legal
advice, but some Google Android apps include open source attributions using
this interpretation. Others also agree with
it.
But, again, the copyright notice redistribution is not the primary motivation
for the dual-licensing. It's stronger protections to licensees and better
interoperation with the wider Rust ecosystem.
How?
To do this, get explicit approval from each contributor of copyrightable work
(as not all contributions qualify for copyright, due to not being a "creative
work", e.g. a typo fix) and then add the following to your README:
## License

Licensed under either of

 * Apache License, Version 2.0 ([LICENSE-APACHE](LICENSE-APACHE) or http://www.apache.org/licenses/LICENSE-2.0)
 * MIT license ([LICENSE-MIT](LICENSE-MIT) or http://opensource.org/licenses/MIT)

at your option.

### Contribution

Unless you explicitly state otherwise, any contribution intentionally submitted
for inclusion in the work by you, as defined in the Apache-2.0 license, shall be dual licensed as above, without any
additional terms or conditions.

and in your license headers, if you have them, use the following boilerplate
(based on that used in Rust):
// Copyright 2016 rust-caseless developers
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

It's commonly asked whether license headers are required. I'm not comfortable
making an official recommendation either way, but the Apache license
recommends it in their appendix on how to use the license.
Be sure to add the relevant LICENSE-{MIT,APACHE} files. You can copy these
from the Rust repo for a plain-text
version.
And don't forget to update the license metadata in your Cargo.toml to:
license = "MIT/Apache-2.0"

I'll be going through projects which agree to be relicensed and have approval
by the necessary contributors and doing this changes, so feel free to leave
the heavy lifting to me!
Contributor checkoff
To agree to relicensing, comment with :
I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option.

Or, if you're a contributor, you can check the box in this repo next to your
name. My scripts will pick this exact phrase up and check your checkbox, but
I'll come through and manually review this issue later as well.

[ ] @SimonSapin
[ ] @mbrubeck

This seems very theoretical. I’d like to know if this is a practical concern to anyone.

It's a practical concern for me, for ease of license management for users of my crates wanting to distribute binaries, I won't include MIT-only dependencies in my dual MIT/Apache-2.0 projects (as otherwise users can only chose the Apache-2.0 selectively).
It is somewhat theoretical in that I'm opening this issues blindly, and not selectively on the crates I actually use, but I do think this has concrete benefits for the entire rust ecosystem

It's a practical concern for me

Does Robigalia or another of your projects use this crate?

No, not yet. Wanting maximum reusability with minimum possible friction through the Rust ecosystem is theoretical, I guess. My concern (for robigalia) is that potential users in proprietary high-assurance or otherwise applications will keep away from the Cargo ecosystem due to licensing concerns. Admittedly, the MIT license is fine, I like and have released my own software under MIT. But when the ecosystem is largely dual-licensed because the stdlib is, it seems like having it all be the same is a good thing, if it's possible.
There are certainly valid objections to the Apache-2.0 license, I don't want to try and force you to make this change, but I do want my motivation to be known.

I don't think I've made any copyrightable contributions to this crate, so I'll leave this up to SimonSapin.


attributes at the begining of the line

Describe the bug
norminette does not allow attributes before declarators in .c files
Erroneous code
__attribute__((__nothrow__)) int	main(void)
{
	return (0);
}

norminette return
Error: ATTR_EOL             (line:   1, col:   1):	Function attribute must be at the end of line

but gcc does not allow attributes at the end of line in .c files

Additional infos

OS: Linuxmint 20.2 uma
python --version: Python 3.8.10
norminette -v: norminette 3.3.40

Hello,
I've muted the error for now, while we try to find a better solution.


Henry county webscraper

Make webscraper that spits out a CSV.
Please make a subfolder under src/webscraper that is the county name and contains all your code (Python 3.6) so no merge conflicts.
http://www.henrycountysheriff.net/InmateInformation
Race and charge yes, but no release dates. Click on the "Search inmates" to view all inmates, without entering anything in the text boxes.

Please also add a README.md explaining how to install/run it and basic output to expect.

I'm on this tonight!

Henry county is now open again for anyone to grab.
@EnduringBeta made a folder, copied the Athens-Clarke scraper and made some small changes to get us started. But most of the code has to be deleted/changed because it's still specific to Athens-Clarke. Feel free to do so.

Jumping on tis webscraper!


How can i use LookUpEditor with Multiple = true cascading

Hi.
I am trying to use LookUpEditor with multiple and cascade enabled but it is not working. How can i do that? Someone have an example?

Hey, i could solve the problem.
I use mult tenant and for lookup i use a separeted class, so for multipe lookup i was using the xyzlookup and i changed to xyzrow and it worked.


Support ontology term IDs as data values

User story
User has cell type annotations mapped to the CL cell ontology in their dataset (e.g., adata.obs['cell_type_ontology_term_id']).
Preferred solution
Allow users to specify that data column values map to a particular ontology in their config. For example, obsSets, obsLabels, and featureLabels. Eventually featureSets as well.
See APIs like https://github.com/INCATools/ubergraph for obtaining the associated human-readable data and related ontology terms.

Cell types: CL: prefix with http://purl.obolibrary.org/obo/cl/cl-basic.obo (example: https://identifiers.org/CL:0000171)
Proteins: uniprot: prefix works with https://registry.identifiers.org/registry/uniprot (example: https://identifiers.org/uniprot:P0DP23
Genes: HGNC: prefix (example: https://identifiers.org/HGNC:130)

Identifiers.org https://identifiers.org/CHEBI:138488
OBO Library PURL 	http://purl.obolibrary.org/obo/CHEBI_138488
Name-to-Thing 	https://n2t.net/chebi:138488


Webpack: This plugin works only in a CommonJS environment

Hello! I keep getting This plugin works only in a CommonJS environment when I use webpack. My configuration is as following:
{
    "ecmaVersion": 5,
    "libs": [
        "browser",
        "jquery"
    ],
    "loadEagerly": [
        "resources/javascripts/**/*.js"
    ],
    "plugins": {
        "complete_strings": {},
        "doc_comment": {
            "fullDocs": true
        },
        "commonjs": {},
        "webpack": {}
    }
}

Where are you running the code? You can't run the webpack plugin in the browser (or probably in any non-node environment).

I'm actually using atom-ternjs

Yes, Electron (the Chrome shell they use) does split in-node and in-chrome scripts, and I assume Atom plugins are ran in the chrome space, and for some reason don't have direct access to node libraries.
It might be possible to find a workaround, and a patch that implements such a thing would be nice to have, but I'm not going to work on it myself.


Docker file not found

Hi, After the CPU docker build done successfully.
successful confirmation,
vinoth@vinoth-Acer:~/mimic2$ docker run -it -p 3000:3000 mycroft/mimic2:cpu
Unable to find image 'mycroft/mimic2:cpu' locally
docker run -it -p 3000:3000 mycroft/mimic2:cpu
docker run -it -p 3000:3000 mycroft/mimic2:cpu
Unable to find image 'mycroft/mimic2:cpu' locally

hello, i had a typo in my doc to build the cpu with the :gpu tag at the end. if you do docker images, it might have built it as mycroft/mimic2:gpu instead of mycroft/mimic2:cpu. the error is fixed in the README now


Introduce (PortId, ChannelId) domain type

Crate
ibc
Summary
Since we started working on ICS04 handlers(https://github.com/informalsystems/ibc-rs/pull/452) we noticed that there is a recurring need to encapsulate the tuple (PortId, ChannelId) as a type. This tuple is needed in error types:
https://github.com/informalsystems/ibc-rs/blob/d6250014e8d8fcddde651f9e3ceb6be5c93cbeae/modules/src/application/ics20_fungible_token_transfer/error.rs#L19-L20
or as parameters to structures:
https://github.com/informalsystems/ibc-rs/blob/71d9baa6a2eed5a9286f9fe88f1cfacbd51e19d8/modules/src/mock/context.rs#L86-L86
or as parameters to methods:
https://github.com/informalsystems/ibc-rs/blob/71d9baa6a2eed5a9286f9fe88f1cfacbd51e19d8/modules/src/mock/context.rs#L279-L282
So there's ample reasons to introduce a new type for capturing the two.
Proposal
The new type should probably reside in ICS04. Potential names:
pub struct PortChannel(PortId, ChannelId);
pub struct BoundChannel(PortId, ChannelId);
pub struct AuthorizedChannel(PortId, ChannelId);

For Admin Use

[X] Not duplicate issue
[x] Appropriate labels applied
[x] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

How about IbcSocket? Much like a TCP socket consists of an IP address and a port number.

Blocked on https://github.com/cosmos/ibc/issues/583

We can invent the type locally (already have as PortChannelId?) and use it in any internal or domain/idiomatic wrapper interfaces without the need for protobuf to be modified. In fact, a message type just to group the two fields consumes more space in protobuf wire encoding than the same two fields encoded inline.

Potential names:

Socket?

BoundChannel(PortId, ChannelId)

This looks too "stateful": there may not be a bound channel for this pair of IDs, as there isn't in your example of an error where this type could be used.

I kinda like socket but i am afraid it's already too overloaded.
I agree that BoundChannel is too stateful.

I don't like Socket because IMO, in TCP/IP an (address, port) is more an "endpoint" than a "socket". For me, a socket represents a data structure over which I can perform operations; it is an interface provided by the kernel that lets me read/write the network. A socket is definitely stateful to me: it holds everything needed to properly interact with the network (e.g. the sequence number in the case of IBC). In other words, ChannelEnd is more a "socket" to me than (address, port) is.
I can't think of another analogy that I feel fits IBC's (channelId, portId), so I think I would vote for the simple PortChannelId.

Good points! Let's go with PortChannelId then of nobody has a better suggestion.


Update cert-manager.md

Added section to update the actual ingress after changing the ingress.yaml

Thanks for this PR, @vincentdeity ! Could you please sign your commit and force push? The easiest way is to run: git commit --amend --signoff

I'll make a new one since I used the web interface of github initially and that botches the commit messages.


Fix ui-state jsdoc syntax typo

This commit fixes the ui-state example in the docs.

[x] Add closing markdown backticks to ui-state jsdoc @example

Thanks!!!


Refactor from kroxylicious futures to JDK futures

Type of change

Refactoring

Description
Delete the Kroxylicious custom futures code and replace it with JDK Futures
Note we use InternalFuture, a small subclass of CompletableFuture to override the standard completion methods and get method to try and control when it can block and prevent clients from completing the future from their filters.
Additional Context
We don't want to maintain our own futures code at this point. The kroxy implementation has the advantages of a Promise/Future separation with no blocking methods available to shoot yourself in the feet with, so we try to keep some of that safety by creating a CompletableFuture subclass.

One thing I was interested in is, should we make the visibility even more restricted? Also should we use InternalFuture throughout kroxys internal code?
We could internally work with CompletableFuture as much as possible and only cast back to InternalFuture when we need to complete it. This would allow us to make it package private and only known to FilterHandler and DefaultFilterContext, which could be seen as a positive to scope it as small as needed.
On the other hand it could be good to use InternalFuture more widely to encourage us to not block.

One thing I was interested in is, should we make the visibility even more restricted? Also should we use InternalFuture throughout kroxys internal code?
We could internally work with CompletableFuture as much as possible and only cast back to InternalFuture when we need to complete it. This would allow us to make it package private and only known to FilterHandler and DefaultFilterContext, which could be seen as a positive to scope it as small as needed.

-0. This'll make the code more obscure.   It still requires an InternalFuture, but you don't find out until runtime when a classcast fails.

On the other hand it could be good to use InternalFuture more widely in the Kroxy internal code to encourage us to not block.

+1 I favour the explicit approach.

We selected #211 in stead


Bump to v0.18.0

Diff between upstream and this PR:
https://github.com/prometheus/alertmanager/compare/v0.18.0...simonpasquier:aos-bump-0.18.0

/lgtm
/approve


Is it possible to make one .a library file for the entire libSPDM?

It will be easier to integrate one .a file into another code base instead of integrating a list of .lib files:

No, we cannot.
Different libraries are for different purposely.

Crypto libs are from third party. They may be harder to combine. But the way to break down libspdm into separate spdm_xxx_libs and other small libs like memlib, malloclib, etc is the architectural decision authors make. You can still organize code into separate folders, but reducing the number of libraries would make it easier for users to integrate libspdm.

I think only spdm_common.lib is common one and required for all.
Other libs are optional, such as malloc lib, it is needed by crypto, but not needed by spmd_*lib.
Some of them are designed to be changed, such as transport_lib or crypto_lib, or spdm_device_lib.
Some of them are two different instance of same lib class, and used in different environment, such as debug_lib, debug_lib_null
We cannot put them together.

a better way to feedback:

We cannot bind all to one lib.
If you want to bind some to one lib, let's discuss case by case. Please describe your requirement as clear as possible.


status.players.sample not working

this is my code:
server_address = '<IP_ADDRESS>'

print(f'Checking server {server_address}')
server = mcstatus.JavaServer.lookup(server_address)

status = server.status()
player_online_names = [user.name for user in status.players.sample]
print('version:', status.version.name, 'maxplayers:', status.players.max, 'onlineplayers:', status.players.online, 'players:', player_online_names)

The response i get is version: Velocity 1.7.2-1.19.3 maxplayers: 500 onlineplayers: 10 players: [] so the player array seems to be empty even though there are 10 players online.

Custom servers hide the player list. There's nothing mcstatus can do about that.


Design UI elements for edit mode

Moved Birch comment from #200
Users want to be able to edit resources and save them to a branch on DCS.
For editing, users want to:

Edit only certain fields (This will be addressed in another issue)

Markdown files - TranslationAcademy and TranslationWords articles
In TranslationNotes OccurrenceNote, OccurrenceNumber, SupportReference, Original Quote should be editable

On blur the scripture highlight should update when the Original Quote or OccurrenceNumber are edited
On blur the TranslationAcademy article should update when the user edits the SupportReference

In TranslationWord Links the Original Quote and Occurrence field should be editable

--

Know what field they're currently editing:

--
save their changes to a per user branch
users should be able to click a save button to save their work

--
know when a file has been changed

Each editable card should have a save icon with the tooltip "Save".
use https://fonts.google.com/icons?selected=Material+Icons:save for unsaved.
use https://fonts.google.com/icons?selected=Material Icons Outlined%3Asave for saved

--

switch between preview and markdown

Each editable card should have a preview icon with the tooltip "Preview" when in Markdown mode and "Markdown" when in Preview mode
use https://fonts.google.com/icons?selected=Material Icons%3Apreview for formatted
use https://fonts.google.com/icons?selected=Material Icons Outlined%3Apreview for markdown

Moved the feature below to a separate issue
--

be notified when they can lose their changes
When the user can potentially lose data due to changing records or scripture references they should be notified:
A modal should inform the user of what resources have unsaved changes:

Some resources have unsaved changes that will be lost if you continue:
TranslationAcademy
TranslationWords

Continue without saving | Cancel | Save All

@birchamp Approved using the Visibility Icon instead of [preview](https://fonts.google.com/icons?selected=Material Icons%3Apreview for formatted) because it isn't available in the Material-UI library.
https://unfoldingword.zulipchat.com/#narrow/stream/281815-SOFTWARE--.20UR.2FgatewayEdit/topic/gateway-edit.23221/near/246146446

@mannycolon I see editing UI for Markdown resources only, Is there another issue for TSVs? How much of the design written in this issue's description is implemented by this issue? Kindly point me to the PRs merged with this issue.

@elsylambert Only markdown content (tw & ta) is editable for now, I'm working on enabling editing on the other resources (TSVs)

@elsylambert I would say this issue could be QAed after the other resources have editing working

Thanks @mannycolon .
Waiting on #219 to be completed to test this issue.

v1.0.0 build a2e8af6. Verified and found to be working as expected.


Autosaving files on buffer change

This plugin is exactly what I was planning to develop myself, thanks for it!
I've got an issue with this line:
https://github.com/marcushwz/nvim-workbench/blob/main/plugin/nvim-workbench.vim#L35
I'm using vim.opt.hidden = true so changing buffer is not supposed to trigger a file save.
I could submit a patch but I don't really get the use of this line…

Hey there, glad that you find this plugin useful :) Haven't been working on it for quite some time and I actually changed it to only save files related to this plugin on my local. This is now pushed to main. Not sure if this helps for your use case?
" Autosave on buf leave
au BufLeave workbench.md silent! wall
au BufLeave *branchbench.md silent! wall

Yes, it fixes it, thanks a lot!


[MB-10590] Add ButtonDropdown component

Jira ticket for this change
Summary
Style the React USWDS so that the select looks like a button and includes the required sort icon and include it in Storybook.
Open Questions/Comments

Should this live here, or in the Office subfolder?
I think an aria-label will be needed in spots where we include this without a form label, so I added it as a prop, but let me know if that isn't the case.
I eyeballed the REM measures, if there's a better way, let me know. I included the usa-button styles and overrode what I needed.
Like I mentioned on Slack, I can't style the actual option elements themselves, so this ended up being just styling the select element itself to make it look like a button.

Setup to Run Your Code
Start Storybook locally.
make storybook

Additional steps

Look at Components/ButtonDropdown

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.
Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Screenshots

I was running into Storybook issues (custom styles not being applied), so I actually confirmed this by plopping the component into the move details page, but that might suggest I did something wrong with Storybook 😬

Messages

:book:
:link: MB-10590

  Generated by :no_entry_sign: dangerJS against 1f572256b700d95cc094cf1fbc0e09d68f424d9d

@amanda5iu I fixed the firefox styles and also used the existing USWDS icon in my latest push.


Ambiguous times not detected any more while resolving to a timezone

Steps to reproduce

API function called: Timex.Timezone.resolve/2
Arguments provided:

[~N[2018-11-04T06:00:00], "America/New_York"]

Version: 3.6.1

Description of issue

What are the expected results (works in 3.5.0)?
The input UTC time 2018-11-04T06:00:00 should resolve to ambiguous times:

2018-11-04 02:00:00-04:00 EDT America/New_York
2018-11-04 01:00:00-05:00 EST America/New_York

iex(10)> ~N[2018-11-04T06:00:00] |> Timex.Timezone.convert("America/New_York")
#<Ambiguous(#DateTime<2018-11-04 02:00:00-04:00 EDT America/New_York> ~ #DateTime<2018-11-04 01:00:00-05:00 EST America/New_York>)>

My understanding is that there is EDT -> EST shift (end of DST), which happens at 2018-11-04 02:00 America/New_York, where time is shifted from 02:00 -> 01:00.
This means that such a local time (02:00:00) is represented in two different time zones: EDT and EST.

Actual results:

iex(2)> ~N[2018-11-04T06:00:00] |> Timex.Timezone.convert("America/New_York")
#DateTime<2018-11-04 02:00:00-04:00 EDT America/New_York>

So, looks like ambiguity is auto resolved, to previous moment. Of course I might be completely wrong here, since this is super confusing topic but at least would like to understand why this behavior has changed.

As of 3.7, the behavior of converting a NaiveDateTime via Timex.Timezone is to treat the NaiveDateTime as the desired time in the specified zone, whereas before it was assuming the NaiveDateTIme was in UTC. If you have a UTC date/time and convert it, then it will be shifted appropriately.
Using the date/time you've provided here as a UTC date/time:
~U[2018-11-04T06:00:00Z] |> Timex.Timezone.convert("America/New_York")
> #DateTime<2018-11-04 01:00:00-05:00 EST America/New_York>

The behavior here is correct because the 1AM hour is repeated twice, the second time due to the clock "falling back". If you take a spring date, such as this years DST change, converting from UTC to America/New_York is unambiguous, again because the clock "skips" an hour. However, if you try to cast a NaiveDateTime, you get an ambiguous date/time as you'd expect:
iex> ~N[2021-03-14T02:15:00] |> Timex.Timezone.convert("America/New_York")
#<Gap(#DateTime<2021-03-14 01:59:59.999999-05:00 EST America/New_York> ~ #DateTime<2021-03-14 03:00:00-04:00 EDT America/New_York>)>


NullPointerException with SwaggerSpecFilter

Running Swagger 1.5.7 and I get this.  My filter is very simple, in fact all it is designed to do is simply pass all through.  I plan on toggling various filters based on the current user, but the default "pass all" should work without exception.
java.lang.NullPointerException
	at io.swagger.core.filter.SpecFilter.filterDefinitions(SpecFilter.java:69)
	at io.swagger.core.filter.SpecFilter.filter(SpecFilter.java:54)
	at io.swagger.jaxrs.listing.ApiListingResource.getListingJson(ApiListingResource.java:98)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:606)`

Filter code is as follows:
public class PassAllSwaggerSpecFilter extends AbstractSpecFilter {}

Mistake in my configuration.


Add border for onhover slice in sunburst

1px solid #ccc
Original issue by @bmesuere on Sun Jun 10 2012 at 12:22.
Closed by @bmesuere on Tue Jun 12 2012 at 18:02.

This isn't possible due to overlap of the arcs. The 1px border is only visible at the inner side.
Original comment by @bmesuere on Mon Jun 11 2012 at 15:17.


Applying a move on both FD[V] and FD[Pres].

I implemented the method to apply a single move M to FD[V] and that works just fine. However, if we want to implement the apply method also for FD[Presentation], then we will have problems because of type erasure. The compiler will think both the apply methods have the same type signature and throw an error because of the double definition. One way to get around this could be to wrap the FD[V] and FD[Pres] in different case classes, but that's not very elegant. What do you think I should do?

Firstly there is no type erasure at compile time, only at runtime. So there
should be no problem.
On Thu 25 Jun, 2015 3:59 pm Sayantan Khan<EMAIL_ADDRESS>wrote:

I implemented the method to apply a single move M to FD[V] and that works
just fine. However, if we want to implement the apply method also for
FD[Presentation], then we will have problems because of type erasure. The
compiler will think both the apply methods have the same type signature and
throw an error because of the double definition. One way to get around this
could be to wrap the FD[V] and FD[Pres] in different case classes, but
that's not very elegant. What do you think I should do?
—
Reply to this email directly or view it on GitHub
https://github.com/siddhartha-gadgil/ProvingGround/issues/23.

It doesn't work out. The compiler does throw an error when trying to compile if both the methods are implemented. As an example, you could try compiling this toy example with scalac, the compiler will complain.
https://gist.github.com/Bolt64/cab3432322c645567678

I'll try. But the more serious answer: you don't need the action on P, at
least for now.
Siddhartha
On Thu, Jun 25, 2015 at 4:17 PM Sayantan Khan<EMAIL_ADDRESS>wrote:

It doesn't work out. The compiler does throw an error when trying to
compile if both the methods are implemented. As an example, you could try
compiling this toy example with scalac, the compiler will complain.
https://gist.github.com/Bolt64/cab3432322c645567678
—
Reply to this email directly or view it on GitHub
https://github.com/siddhartha-gadgil/ProvingGround/issues/23#issuecomment-115208178
.

You were right about type erasure. But the second point, as far as I can
see, you do not need moves to be actual functions, i.e., the action need
not be the apply method.
Siddhartha
On Thu, Jun 25, 2015 at 4:25 PM Siddhartha Gadgil <
<EMAIL_ADDRESS>wrote:

I'll try. But the more serious answer: you don't need the action on P, at
least for now.
Siddhartha
On Thu, Jun 25, 2015 at 4:17 PM Sayantan Khan<EMAIL_ADDRESS>wrote:

It doesn't work out. The compiler does throw an error when trying to
compile if both the methods are implemented. As an example, you could try
compiling this toy example with scalac, the compiler will complain.
https://gist.github.com/Bolt64/cab3432322c645567678
—
Reply to this email directly or view it on GitHub
https://github.com/siddhartha-gadgil/ProvingGround/issues/23#issuecomment-115208178
.

In that case, I'll just implement the action on FD[Presentation] as some other method, and leave it at that.


Input.Date value is not recognized on Spanish in Microsoft Teams

Hi. I`m using Bot Framework v4 together with Microsoft Teams as a channel.
I have a case when I need to show adaptive card to the user for date selection. Everything works fine for the English users (MS Teams Settings Language is English), but not for the Spanish ones.
When you change the language in MS Teams the calendar is changed also, so instead of Apr. 9, 2019 i see 9 abr. 2019. After Submit of adaptive card - Activity.Value is empty...all I have is the Locale = es, but not the value.
How can I resolve this problem?

@shalinijoshi19 Hi. Thank you for response.
I`m using .NET Core

@denikqwerty can you clarify what doesn't work?

Can you not set the Input.Date value on the server in your .NET code?
Or is it not possible for an end user to set a date in an Input.Date in Teams?

If the former, can you share your code?
If the latter, this would be a question for the Teams folks; they override input controls (including the date picker) and we don't have control over what they're doing.

Hi @dclaux
I`m showing the AdaptiveCard to the user, and I expect user to select the date:
var opts = new PromptOptions
                {
                    Prompt = new Activity
                    {
                        Type = ActivityTypes.Message,
                        Attachments = new List<Attachment>
                        {
                            new Attachment
                            {
                                ContentType = "application/vnd.microsoft.card.adaptive",
                                Content = new
                                {
                                    type = "AdaptiveCard",
                                    version = "1.0",
                                    lang = "en",
                                    body = new object[]
                                    {
                                        new
                                        {
                                            type = "TextBlock",
                                            text = "Enter the date"
                                        },
                                        new
                                        {
                                            type = "Input.Date",
                                            id = "StartDate",
                                            placeholder = "Enter the date",
                                            value = $"{DateTime.Now:yyyy-MM-dd}"
                                        }
                                    },
                                    actions = new object[]
                                    {
                                        new
                                        {
                                            type = "Action.Submit",
                                            title = "Submit"
                                        }
                                    }
                                }
                            }
                        }
                    },
                };
                return await stepContext.PromptAsync(StartDatePrompt, opts);

Bot is using MS Teams as a main channel. For all users AdaptiveCard is rendered correctly and users allowed to select the date.
But in my code when I`m getting Activity back to the bot, the turnContext.Activity.Value is empty for non-English users. For English users it is ok, and the turnContext.Activity.Value contaings JObject with parameter StartDate and entered date.

@denikqwerty, would you mind pasting in the JSON that gets output and that you are passing down to render in AdaptiveCards?  Meanwhile we are also going to investigate trying to repro this at our end.  Also what platform does this render in ? Thanks!

I got the same issue when selecting Chinese in Teams webapp.

@matthidinger following up offline with Teams

@almedina-ms any update on investigation from our side at all? Are we able to repro this at our end ? We should try to understand the issue for 1.2 even if we move some of our globallization related fixes out to the next release.

This appears to be an issue with Teams. We are working with the Windows Globalization team and Teams (@jjustinms) to figure out a way forward.
Meanwhile, here's the snip from Ron from Windows globalization around the rootcause:

...issue is specific to Teams, which has an inherent problem picking up OS locale data changes.
Teams is built on Electron, which is basically Chromium, and Chromium carries its own version of ICU (an open source set of globalization APIs and data).  When Teams first starts, the Electron ICU picks up the OS locale and caches it.  If the OS locale data changes, the Electron ICU has no mechanism to pick that up, so it stays in the initial OS locale setting.

Related: https://github.com/OfficeDev/BotBuilder-MicrosoftTeams/issues/195

@almedina-ms - Has this been fixed?


[WFLY-9817] [WFCORE-3600] ee8.preview.mode not needed in server.jvm.a…

…rgs / JAVA_OPTS
ee8.preview.mode workaround not needed in server.jvm.args / JAVA_OPTS anymore
https://issues.jboss.org/browse/WFLY-9817 / https://issues.jboss.org/browse/WFCORE-3600
Please make sure your PR meets the following requirements:

[x] Pull Request title is properly formatted: [WFLY-XYZ] Subject or WFLY-XYZ Subject
[x] Pull Request contains link to the JIRA issue(s)
[x] Pull Request contains description of the issue(s)
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Attached commits represent units of work and are properly formatted

I tested on a custom CI branch to save time


raftstore: pub check_sst_for_ingestion

Signed-off-by: CalvinNeo<EMAIL_ADDRESS>
What is changed and how it works?

Issue Number: ref #12849
What's Changed:
We need to make check_sst_for_ingestion public, since it can be used as a util function to validate a sst file.

pub `check_sst_for_ingestion`

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
None

/cc @BusyJay @tonyxuqqi

/run-all-tests

/merge


Fix/octokit proxy agent

upgrade octokit and use proxy agent

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


cmd/go2go: cannot refer to unexported name "pkg.unexported"

What version of Go are you using (go version)?

$ go version
go version devel +79f7966688 Tue Jan 12 16:41:23 2021 +0000 darwin/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="auto"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/benburkert/Library/Caches/go-build"
GOENV="/Users/benburkert/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/benburkert/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/benburkert"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/benburkert/src/github.com/golang/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/benburkert/src/github.com/golang/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/3q/b_j95bxn6_n4z99hyl_8r41r0000gn/T/go-build350165288=/tmp/go-build -gno-record-gcc-switches -fno-common"
GOROOT/bin/go version: go version devel +79f7966688 Tue Jan 12 16:41:23 2021 +0000 darwin/amd64
GOROOT/bin/go tool compile -V: compile version devel +79f7966688 Tue Jan 12 16:41:23 2021 +0000
uname -v: Darwin Kernel Version 20.2.0: Wed Dec  2 20:39:59 PST 2020; root:xnu-7195.60.75~1/RELEASE_X86_64
ProductName:	macOS
ProductVersion:	11.1
BuildVersion:	20C69
lldb --version: lldb-1<IP_ADDRESS>
Apple Swift version 5.2.4 (swiftlang-1<IP_ADDRESS> clang-1<IP_ADDRESS>)

What did you do?
$ cat main.go2
package main

import "subpkg"

func main() {
  subpkg.Hello[string]()
}

$ cat subpkg/pkg.go2
package subpkg

import "fmt"

func Hello[T any]() {
  var t T
  hello(t)
}

func hello(t interface{}) {
  fmt.Printf("hello, here's a zero %T: %#v\n", t, t)
}

What did you expect to see?
$ go tool go2go run ./main.go2
hello, here's a zero string: ""

What did you see instead?
$ go tool go2go run ./main.go2
# command-line-arguments
/tmp/main.go2:6: cannot refer to unexported name subpkg.hello
/tmp/main.go2:6: undefined: subpkg.hello
/path/to/go/bin/go [run main.go] failed: exit status 2

Exporting the hello func fixes the compile error.

/cc @findleyr @griesemer @ianlancetaylor

/cc @findleyr @griesemer @ianlancetaylor

This is an unfortunate limitation of the go2go prototype: Since it relies on monomorphization - i.e., "expansion" of all generic code into non-generic code, the non-generic code for Hello[string] wants to call subpkg.hello which is not exported.
It's not clear to me that we want/need to fix this in the prototype.

This is an unfortunate limitation of the go2go prototype: Since it relies on monomorphization - i.e., "expansion" of all generic code into non-generic code, the non-generic code for Hello[string] wants to call subpkg.hello which is not exported.
It's not clear to me that we want/need to fix this in the prototype.

This is a documented limitation of the go2go tool.
https://go.googlesource.com/go/+/refs/heads/dev.go2go/src/cmd/go2go/doc.go#49

This is a documented limitation of the go2go tool.
https://go.googlesource.com/go/+/refs/heads/dev.go2go/src/cmd/go2go/doc.go#49

@griesemer @ianlancetaylor thanks for the explanation. Out of curiosity, could a go:linkname directive work here?

@griesemer @ianlancetaylor thanks for the explanation. Out of curiosity, could a go:linkname directive work here?

For the special case of calling a package-scope unexported function, yes, go:linkname could work.  It wouldn't help for a reference to an unexported method or an unexported type.

For the special case of calling a package-scope unexported function, yes, go:linkname could work.  It wouldn't help for a reference to an unexported method or an unexported type.


Update README to match the documentation example

this PR should also be merged

@franziskuskiefer hopefully you can update to Rust 1.56 soon and use #![doc = include_str!("README.md")].
If you do, it will automatically doctest these examples.

I guess you meant to tag @dignifiedquire 🙂

thanks


Update 'promote' to handle current directory layout

A sample gitops directory structure can be found in https://github.com/mnuttall/gitops-repo-testing. (This should itself be cleaned up and moved into the rhd-gitops-example org.)
The original 'promote' verb expected a slightly different directory structure. Currently, promote service-a --from dev --to staging means,

Create a PullRequest that copies /services/service-a/base/config/* from the dev branch to the staging branch

See https://github.com/rhd-gitops-example/services#running - this behaviour, and its docs, need fixing.

The command is only this form for now?
promote service-a --from https://my/dev.git --to https://my/staging.git

because of
After some prototyping, separating each environment into its own repository is believed to be the most appropriate design choice.
?
Another question:
Is the promote a command of cli service?

^^ Hi @akihikokuroda yes, it's only between repositories at this point. Yes, promote is a command of service as per the updated description of this issue.


Make some controls disabled by default

I understand this controls may be useful for someone, but i think for most people there are no any reason to change this settings from Home Assistant. But, it is pretty easy to missclick while scrolling through them which may cause damage to the battery. I suggest to make most of them disabled by default.

Hi @voed, I can see where you're coming from, but the truth is that the options are pretty harmless (except for one in the SG0XXP3 profiles) and at worst it just won't work. And I say that because I've seen some BAD configurations people have had here, hah! So from this point of view, I think it is good to make those values rather visible in HA.
BUT I will soon add some kind of a locking mechanism to prevent accidental changes! 😉

BUT I will soon add some kind of a locking mechanism to prevent accidental changes!

Yes please. That would be great!


Support comments in any language

Hi, I like this project very much!
I found an easy way to make comments and release notes multilingual.
Just tell the prompt to "translate into xx language".
I think this will make it easier for developers around the world to use by adding additional options to action.yml.
Proof of concept
# .github/workflows/openai-pr-reviewer.yml
# This content refers to https://github.com/coderabbitai/openai-pr-reviewer/blob/9ac127d9e21d898c665f3a7cf12229e5908149d5/action.yml

jobs:
  review:
    steps:
        with:
          system_message: |
            You are `@openai` (aka `github-actions[bot]`), a language model
            trained by OpenAI. Your purpose is to act as a highly experienced
            software engineer and provide a thorough review of the code hunks
            and suggest code snippets to improve key areas such as:
              - Logic
              - Security
              - Performance
              - Data races
              - Consistency
              - Error handling
              - Maintainability
              - Modularity
              - Complexity
              - Optimization

            Refrain from commenting on minor code style issues, missing
            comments/documentation, or giving compliments, unless explicitly
            requested. Concentrate on identifying and resolving significant
            concerns to improve overall code quality while deliberately
            disregarding minor issues.

            Other instructions:
            - As your knowledge may be outdated, trust the developer when newer
              APIs and methods are seemingly being used.
            - Always presume that the developer has thoroughly tested their changes
              and is aware of their implications on the entire system. Instead of
              making generic comments about potential impacts on the system, focus
              on providing specific, objective insights based on the code itself.
            - Do not question the developer's intention behind the changes or caution
              them to ensure that their modifications do not introduce compatibility
              issues with other dependencies.
            - Never ask the developer to review the changes.

            IMPORTANT: Provide your review in Japanese(日本語).
          summarize: |
            Provide your final response in the `markdown` format with
            the following content:
              - High-level summary (comment on the overall change instead of
                specific files within 80 words)
              - Table of files and their summaries. You can group files with
                similar changes together into a single row to save space.

            Avoid additional commentary as this summary will be added as a
            comment on the GitHub pull request.

            IMPORTANT: Provide your summary in Japanese(日本語).
          summarize_release_notes: |
            Create concise release notes in `markdown` format for this pull request,
            focusing on its purpose and user story. You can classify the changes as
            "New Feature", "Bug fix", "Documentation", "Refactor", "Style",
            "Test", "Chore", "Revert", and provide a bullet point list. For example:
            "New Feature: An integrations page was added to the UI". Keep your
            response within 50-100 words. Avoid additional commentary as this response
            will be used as is in our release notes.

            Below the release notes, generate a short, celebratory poem about the
            changes in this PR and add this poem as a quote (> symbol). You can
            use emojis in the poem, where they are relevant.

            IMPORTANT: Provide your release notes in Japanese(日本語).

Details are in my article below.
https://zenn.dev/link/comments/efff741a41b9ff
My suggestion about signatures
# action.yml
inputs:
  language:
    required: false
    description: 'Language of comments and release notes (e.g. `Japanese(日本語)`)'
    default: 'English'

then, the prompt will be like this (pseudocode):
if(input.language && input.language !== 'English') {
  systemMessage += `\n\nIMPORTANT: Provide your review in ${input.language}.`
}

Perhaps this line of instructions (the line with IMPORTANT in my PoC) may not be necessary if the developer chooses English.

Fixed in #430
Please let me know if this works.


Consider use UTF-8 by default for Azure CLI

Describe the bug
I was reported a bug on StackOverflow: https://stackoverflow.com/q/78008939/910074
When I have to use UTF-8 for my default console output encoding ([Console]::OutputEncoding), the Azure CLI unable to handle Chinese characters because Encoding issue. It cause either Chinese chars missing or produce messy code.
Related command
$(az account list -o json)
az account list -o json | jq '.'
Errors

Issue script & Debug output
It's an encoding issue.
Expected behavior
I expected Azure CLI can handle Chinese characters correctly.
Environment Summary
azure-cli                         2.57.0
core                              2.57.0
telemetry                          1.1.0
Extensions:
account                            0.2.3
azure-devops                      0.25.0
front-door                        1.0.16
interactive                        0.4.5
k8s-extension                      1.2.4
managementpartner                  0.1.3
Dependencies:
msal                              1.26.0
azure-mgmt-resource             23.1.0b2
Python location 'C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\wakau.azure\cliextensions'
Python (Windows) 3.11.7 (tags/v3.11.7:fa7a6f2, Dec  4 2023, 19:24:49) [MSC v.1937 64 bit (AMD64)]
Legal docs and information: aka.ms/AzureCliLegal
Your CLI is up-to-date.
Additional context
I have a workaround by now. Just edit C:\Program Files\Microsoft SDKs\Azure\CLI2\wbin\az.cmd file. Add -X utf8 to the python arguments.
::
:: Microsoft Azure CLI - Windows Installer - Author file components script
:: Copyright (C) Microsoft Corporation. All Rights Reserved.
::

@IF EXIST "%~dp0\..\python.exe" (
  SET AZ_INSTALLER=MSI
  "%~dp0\..\python.exe" -X utf8 -IBm azure.cli %*
) ELSE (
  echo Failed to load python executable.
  exit /b 1
)

Thank you for opening this issue, we will look into it.

I am able to repro with the latest PowerShell 7.4.1. My system locale is English (United States):

Printing to console is fine:
> az group show -n testrg
{
  ...
  "tags": {
    ...
    "key1": "测试"
  },
  ...
}

But a warning is shown when redirecting:
> az group show -n testrg > out.txt
WARNING: Unable to encode the output with cp1252 encoding. Unsupported characters are discarded.

(Actually, I wrote that warning in https://github.com/microsoft/knack/pull/178.)
According to https://docs.python.org/3/library/sys.html#sys.stdout

sys.stdout
Non-character devices such as disk files and pipes use the system locale encoding (i.e. the ANSI codepage).

So changing the console's encoding with [Console]::OutputEncoding = [Text.UTF8Encoding]::new() won't affect Python's output encoding.
I would recommend changing your system encoding to UTF-8 (follow https://github.com/microsoft/knack/pull/178), so that you won't need to modify the az.cmd entry script every time you update Azure CLI.
Also see: https://github.com/python/cpython/issues/74595

Changing the system encoding to UTF-8 is not an option for most of non-English locale people.

Changing the system encoding to UTF-8 is not an option for most of non-English locale people.

Can you explain why? My personal desktop computer is using UTF-8 as I need to display Chinese (Simplified, China).

I can verify Windows PowerShell 5.1 can't handle UTF-8 correctly:
> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.22621.2506
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.22621.2506
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

> [Console]::OutputEncoding

IsSingleByte      : True
BodyName          : IBM437
EncodingName      : OEM United States
HeaderName        : IBM437
WebName           : IBM437
WindowsCodePage   : 1252
IsBrowserDisplay  : False
IsBrowserSave     : False
IsMailNewsDisplay : False
IsMailNewsSave    : False
EncoderFallback   : System.Text.InternalEncoderBestFitFallback
DecoderFallback   : System.Text.InternalDecoderBestFitFallback
IsReadOnly        : False
CodePage          : 437

> & "C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe" -X utf8 -c "print('测试測試')" > out.txt ; Get-Content out.txt
µ╡ïΦ»òµ╕¼Φ⌐ª

This can be fixed by setting [Console]::OutputEncoding = [Text.UTF8Encoding]::new():
> [Console]::OutputEncoding = [Text.UTF8Encoding]::new()

> [Console]::OutputEncoding

BodyName          : utf-8
EncodingName      : Unicode (UTF-8)
HeaderName        : utf-8
WebName           : utf-8
WindowsCodePage   : 1200
IsBrowserDisplay  : True
IsBrowserSave     : True
IsMailNewsDisplay : True
IsMailNewsSave    : True
IsSingleByte      : False
EncoderFallback   : System.Text.EncoderReplacementFallback
DecoderFallback   : System.Text.DecoderReplacementFallback
IsReadOnly        : False
CodePage          : 65001

> & "C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe" -X utf8 -c "print('测试測試')" > out.txt ; Get-Content out.txt
测试測試

https://stackoverflow.com/a/78023334/2199657 mentions PowerShell 7.4 doesn't interpret the redirected data anymore.
https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_redirection?view=powershell-7.4#redirecting-output-from-native-commands

PowerShell 7.4 changed the behavior of the redirection operators when used to redirect the stdout stream of a native command. The redirection operators now preserve the byte-stream data when redirecting output from a native command. PowerShell doesn't interpret the redirected data or add any additional formatting.

Simply calling python -X utf8 will work:
> & "C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe" -X utf8 -c "print('测试測試')" > out.txt ; Get-Content out.txt
测试測試

Same approach can be used to call Azure CLI:
> & "C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe" -X utf8 -IBm azure.cli group show -n testrg > out.txt ; Get-Content out.txt
{
  ...
  "tags": {
    ...
    "key1": "测试測試"
  },
  ...
}

Wait. As you are already using cp950 which is big5: ANSI/OEM Traditional Chinese (Taiwan; Hong Kong SAR, PRC); Chinese Traditional (Big5) according to https://learn.microsoft.com/en-us/windows/win32/intl/code-page-identifiers, I guess you are trying to parse characters not in cp950. May I know the original Chinese character that is causing problem?

I'm okay with the cp950 in both Windows PowerShell or PowerShell 7+.
It because I installed Oh-My-Posh in PowerShell and used in Windows Terminal. So I have to use UTF-8 in the Console. That's why I need az.cmd to output UTF-8 by default.

It because I installed Oh-My-Posh in PowerShell and used in Windows Terminal. So I have to use UTF-8 in the Console.

I fail to understand the relationship between Oh-My-Posh and encoding. Could you give more context on this? I don't think it is Oh-My-Posh that causes the encoding error. May I know the original Chinese character that is causing problem?

It doesn't matter what original Chinese character are. All Chinese characters will be truncated from the output.
For your confusing, it because Oh-My-Posh can define special unicode font to display symbols on the prompt, like this:

So that my Console output encoding must be in UTF-8 encoding. Let's why I don't set cp950 on the Console.

I don't think this got anything to do with Oh-My-Posh when redirection is involved. Without redirection, like a pure az account list, the output is indeed in UTF-8.
https://docs.python.org/3/library/sys.html#sys.stdout

On Windows, UTF-8 is used for the console device.

In your original screenshot, Azure CLI is trying to encode its output with cp950, but certain characters can't be encoded by cp950 showing as "unsupported":

Besides Azure CLI, you can repro this issue with Python:
> python -c "print('测试')" > out.txt
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "C:\Users\jiasli\AppData\Local\Programs\Python\Python311\Lib\encodings\cp1252.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_table)[0]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
UnicodeEncodeError: 'charmap' codec can't encode characters in position 0-1: character maps to <undefined>

Here is my test:

Don't you think this is issue is absurd in 2024? The windows region settings require administrative rights, so not everyone can change it, and the text says

... language (system locale) to use when displaying text in programs that do not support Unicode

emphasis mine
Is Python or AZ CLI a program that doesn't support Unicode? I don't think so. Is AZ CLI supposed to work with non-unicode tools? I don't suppose so, even the damn Notepad in Windows defaults to unicode these days. The Knack even specifically forces UTF8 on logfile output. Why not on stdout?

I don't think they really understand our pain points. For decades.


Allow generation of suffix-free file and symbol names

🚀 Feature request
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[x] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Description
 A clear and concise description of the problem or missing capability...
During style guide meetings we discussed that although recommended for most projects, suffixes in names of components, directives, and services in particular could be redundant for some projects.
We'd not change the official style guide recommendation, but we'd want to enable folks with advanced use cases to drop the suffixes when needed.
Describe the solution you'd like
 If you have a solution in mind, please describe it.
This proposal is for a global CLI flag per project which disables the current default behavior. When this flag is enabled it'll not suffix any of the generated files, nor the declared symbols inside of them.
Describe alternatives you've considered
 Have you considered any alternative solutions or workarounds?
Drop suffixes completely, which is bad for most projects.

@mgechev I have started working on it.

This has been addressed via https://github.com/angular/angular-cli/pull/17040


Implement _.chunk Array method

Method Signature:
chunk(array, size = 1)
Arguments:
array (Array): The array to process.
size (number): The length of each chunk, default value = 1
Lodash _.chunk Reference:
_.chunk

https://github.com/hamzahejja/forget-about-lodash/commit/872950b20f04f72100c0387bdd3d80c8e66c7adb


Seeking new maintainer(s)

Unfortunately @lassebunk passed away shortly after I had talked with him about helping to maintain the gem. When @MrShemek and I created the fork we were hoping to be able to keep this gem going, at least for maintenance.
I haven't heard from @MrShemek in awhile now, and unfortunately I have not been doing much ruby work as of late, and so I haven't had the time to put in the maintenance that is required.
If anyone would like to take over responsibility for getting a new release out with this fix and with others, that would be appreciated. I'm happy to help insofar as transferring ownership and facilitating a transition, but I don't have much time to dedicate to maintenance right now.

Originally posted by @WilHall in https://github.com/WilHall/gretel/issues/16#issuecomment-589726235

I will continue to maintain this project. Thanks for open issue.

@kzkn Many thanks!


Add conversion of ConvOp from ONNX to TOSA

This implements the conversion of ConvOp from ONNX  to TOSA. Since TOSA does not support the group attribute special handling is needed to support it.
Signed-off-by: Maximilian Bartel<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Thank you for the reviews! @AlexandreEichenberger Can you merge this please if it passes the pipeline?

Jenkins Linux s390x Build #9700 [push] Add conversion of ConvOp... started at 12:54

Jenkins Linux amd64 Build #9681 [push] Add conversion of ConvOp... started at 11:54

Jenkins Linux ppc64le Build #8746 [push] Add conversion of ConvOp... started at 12:58

Jenkins Linux amd64 Build #9681 [push] Add conversion of ConvOp... aborted after 49 min

Jenkins Linux s390x Build #9700 [push] Add conversion of ConvOp... aborted after 51 min

Jenkins Linux ppc64le Build #8746 [push] Add conversion of ConvOp... aborted after 53 min


[stable/postgresql] Correct description of initdbScript parameter

What this PR does / why we need it:
PR corrects description of initdbScript parameter which can confuse people right now .
Checklist

[x] DCO signed
[x] Chart Version bumped

/ok-to-test
/lgtm


Update to use system Eigen

Needs to build:

[ ] thirdparty
[ ] MapLabMultiarch
[ ] MuMa Multiarch

To be reviewed (not yet open):

[ ] This PR
[x] ceres
[ ] eigen_catkin
[ ] aslam basic
[ ] MapLab
[ ] MuMa
[ ] Clean up aslam_install

@eggerk Can we merge this now?

@fabianbl Did you have a successful multiarch build? If so, this should be good to merge
@mfehr @schneith any objections to merging this?

@eggerk @mfehr @schneith Last week's multiarch build was successful, I will do another try now to be sure: http://<IP_ADDRESS>:8080/view/maplab/job/maplab_multiarch/145/

For some reasons Jenkins failed checking out this repo. I updated the rosinstall: https://github.com/ethz-asl/aslam_install/pull/117 Hopefully, this will fix it

Next try: http://<IP_ADDRESS>:8080/view/maplab/job/maplab_multiarch/147/

http://<IP_ADDRESS>:8080/view/maplab/job/maplab_multiarch/149/

@eggerk @mfehr @schneith All builds working fine now, I think we can merge this.


Fixed # of masters warning when adding a new master

This patch fixes the warning that was being shown when the user selected
a new master node to be added and it wasn't breaking the odd number
constraint. Before this, the current assigned masters were not being
considered to show/not show the warning.
Now the warning will only be shown if the number of masters to be added
breaks the constraint as it's expected to be.
bsc#1087267
Signed-off-by: Vítor Avelino<EMAIL_ADDRESS>

 Bug 1087267 - Scaling to 3 > 2 > 3 masters is not possible


3/16 Hack Night SF Agenda

Pitch: pdfhook
Turning PDFs into APIs that fill them
Who?
Project Lead: Ben Golder @bengolder
Contributors:

Gaurav @gauravmk (Core contributor)
@zhoux10 Sherry
Brennan @bhoeting
@samgensburg Sam
Marlene @mfeil4
@debrasol Debra
@joffemd Mark

Why?
Many agencies and institutions rely on paper forms to provide essential services. As developers who build things for public services, we need to interface well with existing services. pdfhook lets you take interactive pdf forms made in Acrobat and fill them from JSON data.
What
pdfhook is a python web app that includes three main things:

a python module that wraps the pdftk command line utility, and has functions for filling out pdf forms.
a small flask web app that turns PDF forms into JSON APIs that fill them.
a few demo web pages that illustrate how the API works.

Needs
We need help with cleaning up the project, writing documentation, and building out the demo pages.

Python developers to help refactor and write tests
Anyone who knows how to build pdf forms from scratch (using Acrobat or other tools) to help illustrate and document how to make forms.
Beginners to help "test drive" our installation and setup instructions
Front end developers to help build out the one missing demo page

What happened:

@gauravmk worked through some bugs to get a test deployment up on Heroku
@patrickm02L went through a test run of getting the project set up on his computer and noted questions and issues that might arise for beginners.


Just a small UI bug that shows two rows of tabs

Describe the bug
Just a small UI bug that shows two rows of tabs
To Reproduce
Steps to reproduce the behavior:

Updating to Version: 3.0.4322 (24c634e)
Lunch ProcessHacker without minimizing (cause once its minimized that effect is gone)

Expected behavior
Show two rows of tabs once its openned
Screenshots

Desktop (please complete the following information):

OS: Windows 10 Pro 64Bit
Version 21H1 (19043.1165)

Additional context
Add any other context about the problem here.

Steps to reproduce the behavior

I can't reproduce the bug based on what's provided.

Steps to reproduce the behavior

I can't reproduce the bug based on what's provided.

Hmm odd I just simply updated all the files and its happening I literally didn't do anything unzip everything except my config can that be it?

Try updating to r4330

Try updating to r4330
If it still happens then try disabling the dark theme.

I did all that a few times with reboots and its all the same result

Fixed in the latest nightly update 👍


[CON-1291] Merge release/1.3.1 into master

Changes

Merge changes from 1.3.1 into master

There's a whole bunch of merge conflicts.

There's a whole bunch of merge conflicts.

Yup, I'm working on it

Not sure if this is what you're doing, but the "correct" way to do the merge is to create a branch as normal off the master branch, and then do a git merge with release/1.3. The key point is that you're not merging directly into master - you're merging into your branch, which will then merge into master.


configure sidebar content and add new pages

Description (required)
This updates the content of the sidebar to match the current draft I have in Mock Docs.
Not a finished version of the sidebar, since we'll be splitting some stuff out into more pages yet, but just getting the skeleton in place!

Minimum Disruptive PR!


[Feature Request] Support notifications section

Feature or problem?
This is a feature request to manage github notifications. Github notifications are filtered similarly to issues, and prs.
Solution I'd like:

An additional set of panels for notifications, and notification filters.
Keyboard shortcut to mark a notification as done, subscribe, or bookmark

Reading through notifications should be fast, move to next after taking an action

Optional shortcut to mark all notifications in the filter at once.

Alternates I've considered
Using an alternative plugin to mark all notifications as read.

Yeah I would also like that :)


Any supported way to customize haproxy's 503 page?

When I was rolling my own haproxy container I had customized the 503 page so my users would freak out a bit less.  It also gave me a nice way to provide support/helpdesk info.
I'm loving marathon-lb, much better than the mess I had; but how can I customize the error pages?

Hi,
if you look at https://github.com/mesosphere/marathon-lb/blob/master/Longhelp.md you will find a list of templates, most of which can be overridden. Either by providing a file of the same name as the template in the templates/ directory or using a Marathon label on a app by app basis.
So what should be possible (I haven't tried it though) is that you override HAPROXY_HEAD  or HAPROXY_FRONTEND_HEAD or HAPROXY_BACKEND_HEAD depending on whether you want to set the custom 503 page globally, on the frontend or on the backend.
For instance if we wanted to override the frontend we could either create a file templates/HAPROXY_FRONTEND_HEAD to override all frontends or set a label HAPROXY_{n}_FRONTEND_HEAD on a single Marathon app (where {n} is the service port index), to override only the error files of a specific app.
The contents of the template would look something like this:
frontend {backend}
  bind {bindAddr}:{servicePort}{sslCert}{bindOptions}
  mode {mode}
  errorfile 503 /errorfiles/503.http

/errorfiles/ would be the directory where you volume mount your custom errorfiles into (if you're using marathon-lb as a Docker container).

Hi,
I have not tested yet but it's part of a story we are going to implement. Here is what we will try :
Create a template to update the frontend marathon_https_in section. Make sure you have at the end of the section this rule to handle any unmatched ACL :
default_backend custom503
2 : add in the global template the default backend that redirect to the expected error page
backend custom503      mode http      errorloc303  503 /etc/haproxy/errors/503.http
To be tested !


feat(serde): include state in feature

Adds the state of stateful widgets to the serde feature, so they can be included in the app state without worry or double-construction.
Also adds a roundtrip test showcasing how the feature can be used. I'm especially unsure about the wording in the module doc-comment here, I feel like it's a bit too complicated for what it's trying to express but I can't seem to come up with anything better.
Do note that in order to run the test locally, it seems like bacon.toml hasn't been adjusted to run nextest, and I don't want to change that here since it's out of scope for this PR. cargo nextest r --all-features or cargo test --all-features should work though (or replace --all-features with --features=serde, it doesn't really matter. The Makefile.toml does this right anyway).
Partly supersedes #712. Closes #711.

Will close this PR after I also made the PR about adding rkyv support.

Should I also add tests for the {de,}serialization of the Scrollbar and Table states? Seems a bit noisy to me, but more can't hurt I guess.

Do note that in order to run the test locally, it seems like bacon.toml hasn't been adjusted to run nextest, and I don't want to change that here since it's out of scope for this PR

That's fine - last I checked bacon and nextest don't play well together anyway.

Should I also add tests for the {de,}serialization of the Scrollbar and Table states? Seems a bit noisy to me, but more can't hurt I guess.

Using the approach from my comment, you could render these side by side in the same buffer fairly easily. Change the assert method to assert the entire buffer and construct the list / table / scrollbar within instead of outside (just pass the entire AppState in).

#[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)]
struct AppState {
    list_state: ListState,
    table_state: TableState,
    scrollbar_state: ScrollbarState,
}

impl Default for AppState {
    fn default() -> Self {
        Self {
            list_state: ListState::default(),
            table_state: TableState::default(),
            scrollbar_state: ScrollbarState::new(10),
        }
    }
}
impl AppState {
    fn select(&mut self, index: usize) {
        self.list_state.select(Some(index));
        self.table_state.select(Some(index));
        self.scrollbar_state = self.scrollbar_state.position(index);
    }
}

/// Renders the list to a TestBackend and asserts that the result matches the expected buffer.
#[track_caller]
fn assert_buffer(state: &mut AppState, expected: &Buffer) {
    let backend = TestBackend::new(21, 5);
    let mut terminal = Terminal::new(backend).unwrap();
    terminal
        .draw(|f| {
            let items = vec![
                "awa", "banana", "Cats!!", "d20", "Echo", "Foxtrot", "Golf", "Hotel", "IwI",
                "Juliett",
            ];

            use Constraint::*;
            let layout = Layout::default()
                .direction(Direction::Horizontal)
                .constraints([Length(10), Length(10), Length(1)])
                .split(f.size());
            let list = List::new(items.clone())
                .highlight_symbol(">>")
                .block(Block::default().borders(Borders::RIGHT));
            f.render_stateful_widget(list, layout[0], &mut state.list_state);

            let table = Table::new(
                items.iter().map(|i| Row::new(vec![*i])),
                [Constraint::Length(10); 5],
            )
            .highlight_symbol(">>");
            f.render_stateful_widget(table, layout[1], &mut state.table_state);

            let scrollbar = Scrollbar::new(ScrollbarOrientation::VerticalRight);
            f.render_stateful_widget(scrollbar, layout[2], &mut state.scrollbar_state);
        })
        .unwrap();
    terminal.backend().assert_buffer(&expected);
}

#[test]
fn default_state() {
    let mut state = AppState::default();

    let expected = Buffer::with_lines(vec![
        "awa      │awa       ▲",
        "banana   │banana    █",
        "Cats!!   │Cats!!    ║",
        "d20      │d20       ║",
        "Echo     │Echo      ▼",
    ]);
    assert_buffer(&mut state, &expected);

    let serialized_state = serde_json::to_string_pretty(&state).unwrap();
    let mut deserialized_state: AppState = serde_json::from_str(&serialized_state).unwrap();

    assert_buffer(&mut deserialized_state, &expected)
}

#[test]
fn selected_state() {
    let list = List::new([
        "awa", "banana", "Cats!!", "d20", "Echo", "Foxtrot", "Golf", "Hotel", "IwI", "Juliett",
    ])
    .highlight_symbol(">>");
    let mut state = AppState::default();
    state.select(1);

    let expected = Buffer::with_lines(vec![
        "  awa    │  awa     ▲",
        ">>banana │>>banana  █",
        "  Cats!! │  Cats!!  ║",
        "  d20    │  d20     ║",
        "  Echo   │  Echo    ▼",
    ]);
    assert_buffer(&mut state, &expected);

    let serialized_state = serde_json::to_string_pretty(&state).unwrap();
    let mut deserialized_state: AppState = serde_json::from_str(&serialized_state).unwrap();

    assert_buffer(&mut deserialized_state, &expected)
}

#[test]
fn scrolled_state() {
    let list = List::new([
        "awa", "banana", "Cats!!", "d20", "Echo", "Foxtrot", "Golf", "Hotel", "IwI", "Juliett",
    ])
    .highlight_symbol(">>");
    let mut state = AppState::default();
    state.select(8);

    let expected = Buffer::with_lines(vec![
        "  Echo   │  Echo    ▲",
        "  Foxtrot│  Foxtrot ║",
        "  Golf   │  Golf    ║",
        "  Hotel  │  Hotel   █",
        ">>IwI    │>>IwI     ▼",
    ]);

    assert_buffer(&mut state, &expected);

    let serialized_state = serde_json::to_string_pretty(&state).unwrap();
    let mut deserialized_state: AppState = serde_json::from_str(&serialized_state).unwrap();

    assert_buffer(&mut deserialized_state, &expected)
}

One other thing worth considering adding here (can be part of this PR, but better as a future one) is to add an actual example which shows how you can use it to close an app and retain the previous state.

Sounds useful, thank you for your feedback, I'll address all that tomorrow.
About the example, where should the state be stored? In an XDG store folder? Or in a file in the current working directory?

About the example, where should the state be stored? In an XDG store folder? Or in a file in the current working directory?

I'm not sure what makes sense for example code. There's a tradeoff of keeping it simple vs doing it right. @kdheepak might have thoughts on this.

I was thinking a little bit more about these tests. One way to make them better would be store the actual serialized data and then split the tests. The first is setup state and confirm that it serializes to the stored string. The second is given the stored string, the expected output is rendered.
This makes it clear when a change breaks the serialization format, which direction the break has happened (i.e. serialize / deserialize), which helps understand the impact on consumers.

That'll require a bit of macro magic to write comfortably and a lot of individual changes if the format changes since they're kinda smoke tests, but sure, why not. I'll do that then.

Probably just a few raw strings would do fine. Or perhaps indoc!
Aim for tests to not have too much magic when possible.

Did that. Now the logic is duplicated 3 times, but whatever. Will go and rebase #724 now.

(hot time for me atm, will come back to this in ~2 weeks)

I resolved the conflicts in Cargo.toml (added rstest in another commit).
Commit message for squash:
feat(serde): support TableState, ListState, and ScrollbarState

TableState, ListState, and ScrollbarState can now be serialized and deserialized
using serde.

```rust
#[derive(Debug, Clone, serde::Serialize, serde::Deserialize)]
struct AppState {
    list_state: ListState,
    table_state: TableState,
    scrollbar_state: ScrollbarState,
}

let app_state = AppState::default();
let serialized = serde_json::to_string(app_state);

let app_state = serde_json::from_str(serialized);
```

@MultisampledNight Thank you for this PR. :)


Dart pad - No fault detection

This code in the dartPad not working well, it should report an error
class TagTable {
  static  Map<String, List<String>> records = const {
    'RCRD_ALL': ['key1', 'key2'],
  };
}

void main() {
  print('1-${TagTable.records}');

  TagTable.records['RCRD_ALL'] = ['key5', 'key6'];

  print('2-${TagTable.records}');
}

--
Win-7 - Microsoft Windows [Version 6.1.7601]
Dart VM version: 2.7.2 (Mon Mar 23 22:11:27 2020 +0100) on "windows_x64"

Ok i think the problem is that you are trying to update const value of records Map. Because const value is modifiable and your are trying to update value of a Map['RCRD_ALL'] that's why your are not getting any compiler error.
Here is the code that works. Read more about default value
class TagTable {

  static Map<String, List> records = const {
    'RCRD_ALL': ['key1', 'key2'],
  };
}

void main() {
  print('1-${TagTable.records}');

  TagTable.records = {'RCRD_ALL': ['key5', 'key6']};

  print('2-${TagTable.records}');
}

@rahadur I understand you are trying to help - but you should try to understand the reported issue first. @bshlomo is not asking how to make their code work, they are reporting a bug - the code in question should report a runtime error, but it does not, at least in DartPad.

/cc @sigmundch

I have tested it with dart2js on the master and it seems to produce correct code that throws an error.
I guess dartpad might have a buggy version of dart2js.
@mit-mit Michael, do you know how often we update dartpad?

Moving this to the dartpad repo


Fix bug of setting skipped class to step links

Could you explain the bug in more detail, maybe with an example? I don't see what's wrong with the current implementation. Is it an edge case?

I noticed that when I load the form for the first time all the skippable steps are set with craue_formflow_skipped_step which is weird.
Let's assume we have a flow of two steps: Step1 and Step2, Step1 contains usedProducts property (collection of checkboxes), Step2 must be skipped if usedProducts != 'ProductX' (the product being studied in survey).
When I load the survey, Step2 is marked skipped, even I haven't responded to Step1 yet.

Coverage decreased (-19.22%) to 80.18% when pulling 05cdd1ed1d39a223ea0969c905dcc7def3fc5fec on simohammedhttp:master into ffbf596841e342e9acc570932679a9b6299f56ea on craue:master.

The current implementation allows marking succeeding steps as skipped, which is correct. So make sure that your skip function always evaluates to the expected result. Maybe something like this for step 2:
'skip' => function($estimatedCurrentStepNumber, FormFlowInterface $flow) {
	return $estimatedCurrentStepNumber > 1 && !$flow->getFormData()->hasUsedProduct('ProductX');
}

Mmm.. Sounds interesting.
It sounds interesting, I'll give it a try.

@simohammedhttp: Is it ok for you to close this PR?

Yes, I think the actual implementation is quite good.
2015-06-10 13:40 GMT+01:00 Christian Raue<EMAIL_ADDRESS>
@simohammedhttp https://github.com/simohammedhttp: Is it ok for you to
close this PR?
—
Reply to this email directly or view it on GitHub
https://github.com/craue/CraueFormFlowBundle/pull/182#issuecomment-110734196
.


Add language template ?

Will you add command  to show language template?
Such as
class Solution {
public:
int lengthOfLongestSubstring(string s) {
}

};
in c++

please update to the latest commits, then try leetcode show 1 -t --no-desc


added elasticsearch start timeout config

fix for issue #16.
example use:
elasticsearch = new ElasticsearchInside.Elasticsearch(settings => settings.SetElasticsearchStartTimeout(60));
Changed ps script to use more reliable $PSVersionTable for version check.

is this seconds? milliseconds?   looks like seconds .... perhaps a method doc?


Fixed cypress bug

Signed-off-by: dhruv0000<EMAIL_ADDRESS>Description
Ignore this PR. Debugging the CI/CD process.
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #4089 (f61ca6d) into master (02bcb86) will decrease coverage by 7.49%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4089      +/-   ##
==========================================
- Coverage   26.77%   19.28%   -7.50%
==========================================
  Files          59       59
  Lines        5005     5005
==========================================
- Hits         1340      965     -375
- Misses       3250     3745     +495
+ Partials      415      295     -120

Flag
Coverage Δ

gointegrationtests
?

unittests
19.28% <ø> (ø)

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-62.83%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.01% <0.00%> (-52.89%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.90% <0.00%> (-39.10%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.78% <0.00%> (-32.15%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.13% <0.00%> (-29.55%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
0.00% <0.00%> (-21.60%)
:arrow_down:

Continue to review full report at Codecov.

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

@dhruv0000 can we tie this off before the Meshery call on Wednesday?

@dhruv0000 is this needed any longer? // @sudo-NithishKarthik

merging this pr would fix the subscriptions-transport-ws issue, but it also makes our code more vulnerable, in the sense that, cypress will not catch any exceptions and will act as if none of those happened..


Containability as a responsible AI principal.

I recommend adding the containability to the implementation guidelines. In a decade or two, powerful AI will be ubiquitous and we don't want reward functions decentralizing for efficiency, control needs to be a top priority.
Responsible AI principles
As you implement AI solutions, consider the following principles in your solution:

Fairness: AI systems should treat all people fairly.
Reliability and safety: AI systems should perform reliably and safely.
Privacy and security: AI systems should be secure and respect privacy.
Inclusiveness: AI systems should empower everyone and engage people.
Transparency: AI systems should be understandable.
Accountability: People should be accountable for AI systems.
Containability: AI resources should be limited to the essential.

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

ID: f90f6cea-98d6-d360-0f15-f9843d54a434
Version Independent ID: 3c03d1a4-4cba-189d-2920-eb05d67a89b8
Content: Responsible AI - Cloud Adoption Framework
Content Source: docs/strategy/responsible-ai.md
Service: cloud-adoption-framework
Sub-service: strategy
GitHub Login: @v-hanki
Microsoft Alias: martinek

@jasallen

Thank you for your feedback.  We have created internal ticket 72725 to track and investigate this issue.

#please-close


[AutoPR cosmos-db/resource-manager] Adding cassandra connector properties for cosmos db C* account -

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6550

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/5264


feat(docs): improve ux

Back-ported from v2.  This PR makes improvements to anchor handling, scrolling, styles, and the experience around editing/closing live edit examples.
See the v2 counterpart #2819

Codecov Report

Merging #2820 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2820   +/-   ##
=======================================
  Coverage   99.74%   99.74%
=======================================
  Files         160      160
  Lines        2771     2771
=======================================
  Hits         2764     2764
  Misses          7        7

Continue to review full report at Codecov.

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

Released in<EMAIL_ADDRESS>


[GFCarousel] timer of the autoPlay is not disposed

Describe the bug
In GFCarousel, inside dispose method,
@override
void dispose() {
  super.dispose();
  timer?.cancel();
}

The problem here is that super.dispose is being called first and because of that timer?.cancel(); will never be executed.
Expected behavior
timer should be disposed before calling super.dispose();

@yasalkhatib  Hey, It will be fixed in upcoming version. Thank you


[AVOCADO-259] Do not specify htsjdk or Hadoop-BAM as dependencies.

Resolves #259. Instead, rely on ADAM for transitive dependencies.

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


Is there any instructions how to pre-train the model?

Hi, I would like to know is there any guide for how to preprocessing or where to start.
I can see how to preprocess the downstream tasks.
Thanks!

Hi!
It is all written in README.md. To be specific, the instructions are listed here.
Thank you for the question.


(doc) Fix instructions for installing agent

On Ubuntu only root can usually write file /etc/apt/sources.list.d.
Fix the command shown in the documentation to use sudo wherever appropriate.
Signed-off-by: Gianpaolo Macario<EMAIL_ADDRESS>

To reduce  noise and indicate root action i've seen others change the prompt to # instead of a $
do you think thats better?
/cc @ketan

To reduce noise and indicate root action i've seen others change the prompt to # instead of a $
do you think thats better?

I think that works better for users who are familiar with the difference between the two prompts.
I've often seen commands that look like $ [sudo] apt-get ... to indicate that users may need to optionally use sudo, something that I quite like. But I'm fine with what's in this PR too.
I'm just waiting on the confirmation for the LICENSE file (#203) before merging this PR AS-IS.

@gmacario I noticed that you've already signed the CLA. I'm going to merge this PR.
Would you mind making similar changes for the RPM instructions?
Thanks a bunch


Update storage.rst

Add ftp passive mode doc.

Coverage decreased (-1.2%) to 71.212% when pulling 259c9b3b6ed82221ac3f870e48f8963b3df0a610 on Fisiu:patch-1 into b7c63a84787a71f4f70ca67789802b305204eb69 on django-dbbackup:master.

Hello @Fisiu
Thank you very much for this contribution.
Unfortunately, we plan to remove storages' code from DBBackup in favor of django-storage.
I updated recently this one: https://github.com/jschneier/django-storages/pull/175
Maybe could you give 👍 for speed up merging ?


CI: add e2e cron job

What does this PR do?
add corn job to run master on staging
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
cc @carmocca @akihironitta @borda

We should figure out how to get notified of failures. We could do:

A slack notification
Automatically block merging PRs if the previous cron run has failed

A slack notification

yes

Automatically block merging PRs if the previous cron run has failed

no, does not make sense while this staging is unstable by its nature :rabbit:


Handle Cancel before start.

Fixes #4140.

LGTM. Waiting for green

Green. The interop failure looks irrelevant.


Update dependencies

Run ./hack/update-deps.sh manually as we currently have some issues with the update-dependencies automation.
So we get the latest an can better check for release-ability.
Also I saw, that third_party/eventing was in a detached HEAD state and reatached it to "main". Not sure if this fixes the current issues (as third_party/EKB does not have the issues and was not detached :shrug: )

/cc @pierDipi @Cali0707 @Leo6Leo


Don't hide counters belonging to multiple Intel queries.

Many counters are exposed in multiple queries. By hiding that fact,
we are making life harder for users, because they need to find
related counters all around the query tree. Don't do that.
As a side effect, this will decrease the number of needed passes if
all selected counters belong to the same query.

:+1:

So just to clarify, this prevents (in an oversimplified example) Foo from being deduplicated when it appears under First/Category/Foo as well as Other/Place/Foo ?

So just to clarify, this prevents (in an oversimplified example) Foo from being deduplicated when it appears under First/Category/Foo as well as Other/Place/Foo ?

Exactly.


Verify the content spelling and grammar of the 'Backend Developer with PHP' job page

Description: You must ensure that all written content on the 'Backend Developer with PHP' job page is free from spelling and grammatical errors.
Preconditions: The website  is live and accessible.

Step 2: Review the job's header for spelling and grammatical errors.
Expected Result: No spelling or grammatical errors in the job's header.

Step 3: Review the job's description text for spelling and grammatical errors.
Expected Result: No spelling or grammatical errors in the job's description.

Step 4: Review the contact information section for spelling and grammatical errors.
Expected Result: No spelling or grammatical errors in the contact information section.

Step 5: Check if the 'Aplica' button text is written correctly.
Expected Result: The 'Aplica' button text is spelled correctly.

Step 6: Check if the 'Inapoi la locurile de munca' button text is written correctly.
Expected Result: The 'Inapoi la locurile de munca' button text is spelled correctly.


Multiple (almost) identical SELECT statements

Bug description
When you deleteMany, there are multiple SELECT statement that look as though they're doing the same thing.
Internal context: https://prisma-company.slack.com/archives/CEYCG2MCN/p1610373011220500
How to reproduce
This query:
  const deleteUsers = await prisma.post.deleteMany({
    where: {
      id: {
        in: [7, 8, 9, 10]
      }
    }
  })

Outputs this to logs:
{
  timestamp: 2021-01-11T13:46:47.196Z,
  query: 'BEGIN',
  params: '[]',
  duration: 1,
  target: 'quaint::connector::metrics'
}
{
  timestamp: 2021-01-11T13:46:47.201Z,
  query: 'SELECT "public"."Post"."id" FROM "public"."Post" WHERE "public"."Post"."id" IN ($1,$2,$3,$4) OFFSET $5',
  params: '[7,8,9,10,0]',
  duration: 3,
  target: 'quaint::connector::metrics'
}
{
  timestamp: 2021-01-11T13:46:47.204Z,
  query: 'SELECT "public"."Post"."id" FROM "public"."Post" WHERE "public"."Post"."id" IN ($1,$2,$3,$4)',
  params: '[7,8,9,10]',
  duration: 3,
  target: 'quaint::connector::metrics'
}
{
  timestamp: 2021-01-11T13:46:47.207Z,
  query: 'DELETE FROM "public"."Post" WHERE "public"."Post"."id" IN ($1,$2,$3,$4)',
  params: '[7,8,9,10]',
  duration: 2,
  target: 'quaint::connector::metrics'
}
{
  timestamp: 2021-01-11T13:46:47.210Z,
  query: 'COMMIT',
  params: '[]',
  duration: 2,
  target: 'quaint::connector::metrics'
}

There are two SELECT statements that do the same thing, but one includes an (unused) offset.
Environment & setup

Database: PostgreSQL

I can also reproduce this. Might be a bug in the logger. I will mark it is as confirmed and let someone who works on that part have a look.

I can also reproduce this. Might be a bug in the logger. I will mark it is as confirmed and let someone who works on that part have a look.

It may look weird, but this is working as intended.

It may look weird, but this is working as intended.

We are aware of a lot of optimization opportunities and rest assured it will be addressed. We'll open epics when the time is right.


Revert: Need some approval to add name in companies list.

Description
Need approval to add name in companies list.

@chandulal The changes are empty, please open a new PR with the reverted commit.

@Fokko There is only one change in README.md. Just removed "Grab" from company list. Because we need to take permission from the legal team. These changes are there.

@Fokko Could you please merge with master. Thanks in advance.

@chandulal The PR is empty: https://github.com/apache/incubator-airflow/pull/4046/files
Regardless of the PR, I've reverted the changes: https://github.com/apache/incubator-airflow/commit/6999f344cb62d73b820fdb3d80d5b47ed26f6e22
Let me know when you have the approval :)