create_user: Use silent mentions in personal notification.

Fixes #27243

Fixes:  Use silent mentions in personal notification that invite was accepted.
Added new query because my change requests for user_profile.full_name
sql: SELECT "zerver_userprofile"."id", "zerver_userprofile"."full_name" FROM "zerver_userprofile" WHERE ("zerver_userprofile"."is_active" AND "zerver_userprofile"."realm_id" = 2 AND UPPER("zerver_userprofile"."full_name"::text) = UPPER('Fred Flintstone'))
So I have increased query count from 92 to 92 in QueryCountTest

Screenshots and screen captures:
earlier

now

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@zulipbot review

So I have increased query count from 92 to 92 in QueryCountTest

This seems like a typo?

Please squash your commits. As https://zulip.readthedocs.io/en/latest/contributing/commit-discipline.htm notes:

It should pass tests (so test updates needed by a change should be in the same commit as the original change, not a separate “fix the tests that were broken by the last commit” commit).

Ready for review!

Works for me in manual testing! @timabbott perhaps this is quick for you to review?

locally backend tests.test_invite is fine but here they are failing on line 958
Screenshots and screen captures:

local
github

Merged, and replacing the commit message with something less repetitive and that explains why the change was made.
Thanks for the contribution @sayyedarib!


ld doesn't find libevent

Tried to compile a crystal binary, but it couldn't find the libevent library that I installed with nix.
I found the solution by adding an alias the library with the following symlink:
sudo ls -s /lib/x86_64-linux-gnu/libevent-2.1.so.6 /lib/x86_64-linux-gnu/libevent.so
For more details: https://github.com/alexherbo2/snippets/issues/1
So the problem is solved and this could be closed. But I already encountered similar library problem with nix, so I'm wondering if I am missing something when I'm installing package  or is it nix that miss creating the right symlink ?

you are supposed to run the compiler inside nix-shell with appropriate dependencies available: https://nixos.wiki/wiki/FAQ/I_installed_a_library_but_my_compiler_is_not_finding_it._Why%3F

having a /lib would mean impurity, which is someting that nix/nixpkgs/nixos tries to stay clear of, @symphorien suggestion is correct. You should pass the build store paths.

Thanks you both for your quick answer

you are supposed to run the compiler inside nix-shell with appropriate dependencies available: https://nixos.wiki/wiki/FAQ/I_installed_a_library_but_my_compiler_is_not_finding_it._Why%3F

I think the link you sen. I put it in the category of documentation that is only clear for people who don't really need the documentation ;)
I'm not a compiler and c/c++ expert, I just want to  sudo make install to run smoothly.
In ubuntu the workflow is usually:
sudo apt install dep1 dep2 ...
sudo make install

What is the equivalent with nix ?

For just building?
nix-shell -p dep1 dep2 ...
make

for installing, I would look at https://www.youtube.com/watch?v=LiEqN8r-BRw where I package a small c application locally

Thanks @jonringer !
I though for building it would simply be:
nix-env -i libfoo
I know it's not recommanded because it's slow, but it work fine for me !


genericTagQueryList does not correctly serialize

        String new_geohash = "2vghde";
        List<String> geohashList = new ArrayList<String>();
        geohashList.add(new_geohash);
        genericTagQuery.setValue(geohashList);
        Filters filters = NIP01.createFilters(null, null, null, null, null, null, null, null, new GenericTagQueryList(genericTagQuery));

If you send this, then the sent serialized REQ string looks like this:
INFO: Sending serialized message: ["REQ","de3daa94-ff45-496f-b1a4-13d6ba50abe6",{"genericTagQueryList":[{"nip":1,"#g":["2vghde"]}]}]

When it should be more like:
INFO: Sending serialized message: ["REQ","de3daa94-ff45-496f-b1a4-13d6ba50abe6","#g":["2vghde"]}]}]

I'll take a look
Sent with Tuta; enjoy secure & ad-free emails:
https://tuta.com
17 Apr 2024, 04:34 by @.***:

@tcheeric https://github.com/tcheeric>  I added the > @JsonUnwrapped>  annotation to genericTagQueryList, but that hasn't changed the behavior, I don't know why.
—
Reply to this email directly, > view it on GitHub https://github.com/tcheeric/nostr-java/issues/144#issuecomment-2060282333> , or > unsubscribe https://github.com/notifications/unsubscribe-auth/ABQMG7DJG3X4LBS7V4SISI3Y5XUTVAVCNFSM6AAAAABGKGURKKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRQGI4DEMZTGM> .
You are receiving this because you were mentioned.> Message ID: > <tcheeric/nostr-java/issues/144/2060282333> @> github> .> com>

This is actually currently not supported. I will look into it.

I think you are referring to the serialiser, right? Please confirm. (deserialising is not supported)

I meant serialize.

This made me realise a mistake in the implementation, due to a misinterpretation of the spec.
NIP-01 defines Filters as follows:
{
  "ids": <a list of event ids>,
  "authors": <a list of lowercase pubkeys, the pubkey of an event must be one of these>,
  "kinds": <a list of a kind numbers>,
  "#<single-letter (a-zA-Z)>": <a list of tag values, for #e — a list of event ids, for #p — a list of pubkeys, etc.>,
  "since": <an integer unix timestamp in seconds, events must be newer than this to pass>,
  "until": <an integer unix timestamp in seconds, events must be older than this to pass>,
  "limit": <maximum number of events relays SHOULD return in the initial query>
}

Filters do not accept GenericTagQueryLists, as incorrectly assumed, but instead, GenericTagQuerys
I have made the corrections in the Filters class, and adjusted the corresponding encoder and serializer.
More test cases are needed to ensure the current implementation is bullet-proof, but it looks promising.
I have pushed the changes in the branch feature/Issue_144 and will add more test cases tomorrow.


feat(Image): handle children

There is interesting example in SUI docs.

It sends Label to Image as child while we don't handle this in Image component. While I'll work on Label docs I'll check all examples, but now my initial proposal is add label shorthand for Image.

Agreed, seems like shorthand here.  Something to the effect of:
<Image label={{ corner: 'left', icon: 'heart' }} src="/images/wireframe/image.png" />

In general, we're trying to adhere to "no children for components that don't have children" such as an img or an input.

we're trying

Nice try, comrade 😄  I am afraid that the markup of Image options may be too difficult for the shorthands like in Input, after I'll do Label docs it will be understood.

Oh boy, yes... wish I could have avoided that one!
I think we can do it with shorthand, though not completely opposed to having to use children as in the Input. If image children are provided, it would just dictate the wrapped markup (a ui image div wrapping the img tag).

In #576 I don't find new use cases, so I'll label shorthand after #576 will be merged.


Interface explicit lifetime parameters - like MergedObject

I would add this myself but I didn't see the changed code/PR referenced in the original issue.
Basically, the problem is that Interfaces currently don't seem to support explicit lifetime parameters instead giving a implicit elided lifetime not allowed here.
This is very similar to the issue that used to exist for MergedObjects (https://github.com/async-graphql/async-graphql/issues/333) and I suspect the solution will be very similar.
If no-one has time to fix this if @sunli829 would you happen to have the PR that fixed #333 I can have a look at it myself?

Ah it actually does seem to work I just had to specify the lifetime in the type 🤦
#[derive(Interface)]
#[graphql(
    field(name = "text", type = "String"),
    field(name = "position", method = "position", type = "Position<'_>")
)]
pub enum Annotation<'a> {
    Definition(Definition<'a>),
    Term(Term<'a>),
}


Clarify when keys are created

Fixes #15792
Signed off by: Chris Swan<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "fixes-15792" git@github.com:cpswan/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

LGTM

the commit is still signed incorrectly

@jfrazelle needs email rather than handle???

I mean i just dont think it was done via the cli it looks off... and the regex isnt working :(

Please, do not move this forward until the commit is properly signed, with the email.

Sorry @jfrazelle and @calvera this is my first PR to Docker, and despite trying my hardest to follow the excellent guidance from @moxiegirl I've obviously still managed to mess it up
If I follow Gordon the Turtle's guidance then I see:
git commit --amend -s --no-edit
[fixes-15792 10d3487] Clarify when keys are created
 Author: Chris Swan<EMAIL_ADDRESS> 1 file changed, 3 insertions(+), 2 deletions(-)

Then I did git push -f and everything seemed to look OK on Github. So what do I need to do now to make this work?
PS since this is a couple of lines in docs I don't see how this could have anything to do with Jenkins build Windows-PRs 12700 has failed, but I'm willing to suspend disbelief on that.

I think I got it - having only set my user.email locally for the docker-fork it wasn't applied when I cloned to another repo per the @GordonTheTurtle guidance (and I'd messed up my original commit by missing the -s flag).

@cpswan Looks like you are almost there. Your signature went through but you need to rebase -- pull the latest material from docker/docker's master branch. This explains the rebase:
http://docs.docker.com/project/create-pr/#rebase-your-branch

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "fixes-15792" git@github.com:cpswan/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@RichardScothern agreed that the the 'makes use of' needs to be fixed, but it's not an issue that's been introduced by this PR. If team Docker think it's something that should be done whilst we're here then I'm happy to do another (not just signing) update.

I've fixed the nearby typo that @RichardScothern spotted

@cpswan Thank you. LGTM


NavLink suggestion

It's great that className and isActive were added in NavLink component properties:
https://github.com/yahoo/fluxible-router/blob/master/lib/NavLink.js
I suggest to add one more parameter which is tagName. Example : I would like that the whole  row of Customers Table was link to the customer's page so I should be able to write something like
<NavLink className='app-table-row' tagName='tr' routeName='customer' navParams={customer.id} {...this.props} />

I like the idea of allowing non anchor tags, but how would followLink behavior work in that case?
Perhaps we should provide a generic Nav component that can wrap arbitrary tags. @mridgway @lingyan thoughts?

As much as I like the declarative syntax, imho this is a case where the onClick handler (eventually using the History API or makePath with the navigate action) fits better. NavLink should be only for links, e.g. a replacement for anchor tags.
(however using the navigate action will scroll to the top of the page)

This isn't something that we're considering adding. Closing.


Enable trimming

From Default.rd.xml remove:
<Assembly Name="Application" Dynamic="Required All" />
Have to rewrite using DI source generator:
https://github.com/CommunityToolkit/Labs-Windows/discussions/463
Other MSBuild options you can set in your .csproj/.props to save size:
<DisableStackTraceMetadata>true</DisableStackTraceMetadata>
<DisableExceptionMessages>true</DisableExceptionMessages>

First one should save ~8% of binary size
Second one can save a few hundred KBs
(this is all per CPU arch)

Early test shows upto 50% reduction (50mb -> 25mb) with the following Default.md file:
    <!--<Assembly Name="*Application*" Dynamic="Required All" />-->

    <!-- .NET Native fix -->
    <Assembly Dynamic="Required All" Name="NLog"/>
    <Assembly Dynamic="Required All" Name="Microsoft.Extensions.DependencyInjection"/>
    <Assembly Dynamic="Required All" Name="Microsoft.Extensions.Options"/>
    <Assembly Dynamic="Required All" Name="Microsoft.Extensions.Logging"/>
    <Assembly Dynamic="Required All" Name="Microsoft.Extensions.Http"/>
    ```

Problem is serializers returning null due to reflection, will need to rewrite using source generator:
https://learn.microsoft.com/en-us/dotnet/standard/serialization/system-text-json/source-generation

I was told to use System.Text.Json 8.0 Preview 1 for better source generators.

Sample:
```cs
[JsonSourceGenerationOptions(PropertyNameCaseInsensitive = true)]
[JsonSerializable(typeof(AirQuality), GenerationMode = JsonSourceGenerationMode.Metadata)]
internal partial class MyJsonContext : JsonSerializerContext
{
}
..
await JsonSerializer.DeserializeAsync(stream, MyJsonContext.Default.AirQuality);


GH-36601: [MATLAB] Add a MATLAB "type traits" class hierarchy

Rationale for this change
To make it easier to write generic code for arrow.array.Array and associated objects in the MATLAB interface, it would be helpful to have some kind of "type traits"-like objects (e.g. arrow.type.traits.StringTraits, arrow.type.traits.UInt8Traits, etc.).
These "type traits" objects would help centralize various type-specific information in one place (e.g. MATLAB double <-> arrow.type.Float64Type <-> arrow.array.Float64Array), simplifying generic client code.
This would help reduce the number of "switch-like" statements across the MATLAB code base that branch based on type.
What changes are included in this PR?

Added new arrow.type.traits.TypeTraits classes (e.g. arrow.type.traits.UInt8Traits and arrow.type.traits.StringTraits).
Added new arrow.type.traits.traits "gateway" function for creating "type traits" objects from MATLAB class strings (e.g. "double" or "datetime") and arrow.type.ID enumeration values (e.g. arrow.type.ID.Timestamp or arrow.type.ID.UInt8).

Are these changes tested?
Yes.

Added MATLAB tests for the new arrow.type.traits.TypeTraits classes.
Added MATLAB tests  (ttraits.m) for the new arrow.type.traits.traits "gateway" function.

Are there any user-facing changes?
Yes.
There are now MATLAB "type traits" classes that can be used to simplify writing generic client code that switches based on type.
Note: It may make sense to eventually mark these "type traits" APIs as "internal" so that they aren't encouraged to be used by client code outside of the MATLAB interface. The "type traits" classes expose some implementation details such as the Proxy classes that are used under the hood to represent a given Array object in C++.
Example
% Construct a "type traits" object from an arrow.type.ID enumeration value
>> logicalTraits = arrow.type.traits.traits(arrow.type.ID.Boolean)

logicalTraits =

  BooleanTraits with properties:

       ArrayConstructor: @arrow.array.BooleanArray
         ArrayClassName: "arrow.array.BooleanArray"
    ArrayProxyClassName: "arrow.array.proxy.BooleanArray"
        TypeConstructor: @arrow.type.BooleanType
          TypeClassName: "arrow.type.BooleanType"
     TypeProxyClassName: "arrow.type.proxy.BooleanType"
      MatlabConstructor: @logical
        MatlabClassName: "logical"

% Construct a "type traits" object from a MATLAB class string
>> stringTraits = arrow.type.traits.traits("string")

stringTraits =

  StringTraits with properties:

       ArrayConstructor: @arrow.array.StringArray
         ArrayClassName: "arrow.array.StringArray"
    ArrayProxyClassName: "arrow.array.proxy.StringArray"
        TypeConstructor: @arrow.type.StringType
          TypeClassName: "arrow.type.StringType"
     TypeProxyClassName: "arrow.type.proxy.StringType"
      MatlabConstructor: @string
        MatlabClassName: "string"

Future Directions

Re-implement switch statement in RecordBatch code to use "type traits" classes instead.
Look for more opportunties to leverage "type traits" to simplify code in the MATLAB interface.

Notes

Thanks @sgilmore10 for your help with this pull request!

Closes: #36601

We renamed arrow.type.traits.Traits to arrow.type.traits.TypeTraits because the fact that this PR contained both arrow.type.traits.traits and arrow.type.traits.Traits was leading to a CI failure on Windows and macOS, which have case-insensitive filesystem behavior.
Thanks @sgilmore10 for your help debugging this!

+1


[IOTDB-6190] Increase the threshold for Ratis to shut itself down if it detects that a process is stuck

After Ratis was updated to version 3.0, the way this parameter is set has changed, and the previous PR is no longer valid and needs to be modified

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (20bdc07) 48.97% compared to head (77aeeec) 48.91%.
Report is 10 commits behind head on master.

Files
Patch %
Lines

...confignode/manager/consensus/ConsensusManager.java
0.00%
3 Missing :warning:

...org/apache/iotdb/consensus/config/RatisConfig.java
75.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11865      +/-   ##
============================================
- Coverage     48.97%   48.91%   -0.06%
- Complexity    25594    25614      +20
============================================
  Files          2908     2908
  Lines        181274   181648     +374
  Branches      21691    21750      +59
============================================
+ Hits          88776    88860      +84
- Misses        92498    92788     +290

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


MoM 1.7.0 - Broken image (sprites) on Windows

With the latest version of MoM on steam, I see that the sprites are not valid anymore.
I'm talking about the token and non playable characters pictures.
Can someone (who has not done the update), share with me the two following files ?
In directory %USERPROFILE%\AppData\Roaming\Valkyrie\MoM\import\img\
SpriteAtlasTexture-ATLAS_FEATURES_CORE (Group 0)-2048x1024-fmt12.dds
and Atlas_Main.dds
For the technical explanation, I think FFG changed the tile placement on the picture.

related to #868

Can't link them on GitHub doesn't like the file format.
I'll share a folder on google drive to you @BenRQ

Fix is now available, new hotfix version incoming


ng-serve won't check for status

added a boolean to the sendCommand() method to choose whether or not to check for status.
Resolves #55

This will be fixed in #51


Allow to pass the verbosity level to the --verbose long option.

The help text for the verbosity option mentions that it is possible to pass a value to set the verbosity level:
  -v|vv|vvv, --verbose  Increase the verbosity of messages: 1 for normal output, 2 for more verbose output and 3 for debug

However, when I try this I get an error message:
$ php application.php demo:greet Fabien --verbose=3

  [Symfony\Component\Console\Exception\RuntimeException]
  The "--verbose" option does not accept a value.

The -vvv option works fine, but the POSIX long option --verbose doesn't accept a value. This is clearly the intention since these values are being read in Application::configureIO():
            if ($input->hasParameterOption('-vvv', true) || $input->hasParameterOption('--verbose=3', true) || $input->getParameterOption('--verbose', false, true) === 3) {

The solution is to simply change the input option from VALUE_NONE to VALUE_OPTIONAL.

Thanks for contributing this pull request. However, the repositories of individual Symfony Components are read-only, so pull requests are ignored. Please, submit this pull request to the main Symfony repository at https://github.com/symfony/symfony/pulls
Thank you!

I did not realize this, thanks for letting me know! I have made a new PR against the main repo: https://github.com/symfony/symfony/pull/22875


Documentation - Build sphinx documentation on Read the Docs for RDC

Use rocm-docs-core template for documentation
Includes User guide (https://docs.amd.com/bundle/ROCm-Validation-Suite-Guide-v5.3/page/Introduction_to_ROCm_Validation_Suite_Guide.html) and API reference (built by doxygen)
Rationale:

https://github.com/RadeonOpenCompute/rocm-docs-core#purpose

want to add the dependabot?

Merged in Gerrit


git tag "latest" is 3 years old

I know this is not really that important but it might trip someon up in the future.
Anyways, while looking through the repo I noticed that the git tag "latest" points to a commit from 2018, which is quite obviously not the latests version.

Thanks for reporting! I removed that tag to avoid confusion.


Load and execute script with special character

System information:

Operating system (distribution) and version
Windows 10 Pro
DBeaver version
<IP_ADDRESS>906240635

Describe your question:
Hi, I'm investigating an error on my application and discover that is because an special character (NULL) in SQL script (pic. 1). I wondering if is possible to reproduce those characters on DBeaver editor.
Picture 1

Is this issue still actual for you?
Null can be used in DBeaver SQL Editor.

there is no update on ticket for a long time. if issue is still actual for you - feel free to reopen the ticket


[BUG] Lack of Real-Time Transactions Display in Iroha 2 Events Pipeline

OS and Environment
MacOS, Docker Hub
GIT commit hash
b1855454
Minimum working example / Steps to reproduce
$  docker compose up -d
$ ./iroha_client_cli events pipeline
$ ./iroha_client_cli domain register --id=dsfdf

Actual result
The events pipeline does not display the passing transactions in real-time.

Expected result
The events pipeline should output all the events related to Iroha 2 in real-time
Who can help to reproduce?
@astrokov7
Notes
No response

We should investigate if this is true of all peers, or just one. It's possible that the event isn't ready by the time it's supposed to be emitted.
We should also supplement (and leave in) some logs to see the process through. Pipeline events don't happent frequently and the extra diagnostics are worth the performance trade-off IMO.


replace OlapMeta with KVStore

1.replace OlapMeta with KVStore

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @Narcasserun you could run
cd build-support/
sh clang-format.sh

for format BE code to pass starrocks_clang-format test.

Hi @Narcasserun you could run
cd build-support/
sh clang-format.sh

for format BE code to pass starrocks_clang-format test.

I tried, but it suggested： which: no clang-format in，Can you help me

if you OS is mac you can setup a brew from https://brew.sh/ and then run
brew info clang-format

and if you OS is ubuntu you just run
sudo apt-get install clang-format

to setup clang-format


Added 4+5 to preview/stable process

/azp run

@MikeHillberg reassigning to you as this is more up your alley than mine now


Usage of schema_format = :sql

Greetings!
I'm still all hopped up on those sweet sweet Rails World vibes, and am presently writing a SQLite powered Rails app.
I'm having to resort to a sql-based schema format (sometimes you gotta be hardcore like that). It seems like Solid Queue doesn't like this setting, I end up with a blank queue_structure.sql file.
I have a sample repo here https://github.com/jherdman/baconator5k you can see the action played out in 3 commits.

Hey @jherdman, this looks like a bug in Rails 🤔  It seems very similar to https://github.com/rails/rails/issues/52829, but that one should have been fixed. However, I understand your queue_schema.rb was emptied when you ran db:migrate, no? Which seems exactly the same thing there...

Yup! That definitely seems like the one. I'll close this in favour of the aforementioned. Thanks, as always, for being so responsive!


How to insert date/time into script?

Looks like the only way is to generate a lua script from my batch script

I think this is a common scenario, because if you are looking for sth like Expect, you must be scheduling configuration backups of network devices.

Is this satisfy your needs ?
sendln("echo " ..  os.date("%Y/%m/%d %H:%M:%S") )

Great! I've totally forgot this is a lua script.
I also didn't notice the arg[], which partly meets my needs

Can I close this issue ? > @IzualYang

Yes, of course


english_words package doesn't work in DartPad

What happened?

english_words is listed in Directly importable packages, but when I import it, I get Error: Couldn't resolve the package 'english_words' in 'package:english_words/english_words.dart'.
I confirmed I was able to import and use a different package as expected (bloc).
Steps to reproduce problem

Example code from https://pub.dev/packages/english_words:
(Printing the top 50 most used nouns in the English language)
import 'package:english_words/english_words.dart';

main() {
  nouns.take(50).forEach(print);
}

Additional info
Browser
Browser:
Chrome, MacOS
Version:
Version 106.0.5249.119 (Official Build) (arm64)
Are you using any extensions/plugins that affect website behavior
(particularly those that affect iframes, such as ad blockers)?
No
Are there any warnings or errors in your browser's JavaScript console?
If so, paste them below:
Error compiling to JavaScript:
Info: Compiling with sound null safety
Error: Couldn't resolve the package 'english_words' in 'package:english_words/english_words.dart'.
lib/main.dart:1:8:
Error: Not found: 'package:english_words/english_words.dart'
import 'package:english_words/english_words.dart';
       ^
lib/main.dart:4:3:
Error: Undefined name 'nouns'.
  nouns.take(50).forEach(print);
  ^^^^^
Error: Compilation failed.

That'd be due to the fact that it is currently listed as a Flutter package, not a Dart package. Fix in flight.

Confirmed it's working for me now. Thanks for the quick fix!


Receiver and Route fixes

Change how we handle nameless routes and how we save yaml

We just filtered out nameless routes which caused problems with default routes and manually created routes. This now gives a default name of route-${i} which if saved will add the name to the route.

Verified that editing and deleting still works when order has changed and new routes are present.

Made the yamlSaveOverride consistent with receivers since it wasn't working when going to edit as yaml from the create page.

Resolving receiver issues

Edit as yaml on a created resource wasn't calling the yamlSaveOverride
Erroneously set required_tls default for webhooks
camel cased a field that wasn't supposed to be

rancher/dsahboard#1239
rancher/dashboard#1238

Hold off on this one. Turns out we can't have names at all https://github.com/rancher/dashboard/issues/1239#issuecomment-721406280
I was able to see the logs for myself.

Since we couldn't actually save the names of receivers without breaking the alertmanager service I made it so that it's index based. We have a default name of route-${1} which we generate and don't allow the user to set. All of the names get removed before actually saving the config.
This is what it looks like


Tags with accented characters (such as é and á) issue

Tags with accented characters (such as é and à) issue are published on Mastodon without these characters (i.e. #éléphant is published #lphant)

Indeed, that's because a lot of characters are not valid for tags. I'll have a look precisely which characters are allowed and I'll adapt the regex :)

Hello.
Same problem here with the french word "icône" spelled "icne" on Mastodon.
Source : https://gilles.wittezaele.fr/liens/?xY3YtQ
Result : https://mamot.fr/@ParigotManchot/99665721843326402
The shaarli2twitter addon has no problem with this word.

This commit should fix the issue: https://github.com/kalvn/shaarli2mastodon/commit/f293ee362f6e9a93d7ed2bffc7c0daad861a051b


Tidy up and C#ify methods/properties

Just reformatted and removed some unused things as well as rename methods and properties to more C# standard naming.
No functionality has been changed.
Not a problem if you'd rather not merge into your repo, just thought I'd offer as I needed to do it for myself.

You are right about the naming of methods. What happened is that while I translated the code from JavaScript I kept their original casing. Since they are private methods there is no harm in changing their name
The original library decided to remove the code about pages, so we should too. I just forgot about it.
I am not going to rename private fields, because there is really no standard. The naming convention about fields mentions that the standard does not apply to private fields, while the general naming convention says that you should not use underscore. Now, many projects do use underscores, so it is not wrong to do so, but there is really no reason to change either.

I will integrate the pull request in the next few days.

And thank you for your pull request @jamie-lord !


Could not switch Tab Sheets on a window

I have following items of an application written in Delphi, printed with print_control_indentifiers and I want to select the tab with title="Units":
TfrmExportSetup - 'Export Setup'    (L0, T0, R400, B600)
['Export SetupTfrmExportSetup', 'TfrmExportSetup', 'Export Setup']
child_window(title="Export Setup", class_name="TfrmExportSetup")
   |
   | TPageControl - ''    (L6, T29, R394, B555)
   | ['TPageControl', 'pnlExportActive1TPageControl']
   | child_window(class_name="TPageControl")
   |    |
   |    | TTabSheet - 'Maintenance'    (L10, T53, R390, B551)
   |    | ['MaintenanceTTabSheet', 'Maintenance', 'TTabSheet', 'TTabSheet0', 'TTabSheet1']
   |    | child_window(title="Maintenance", class_name="TTabSheet")
   |	|
   |    | TTabSheet - 'Units'    (L10, T53, R390, B551)
   |    | ['UnitsTTabSheet', 'TTabSheet3', 'Units']
   |    | child_window(title="Units", class_name="TTabSheet")

I tried different methods and none works. Some of the failures:
1.
Code:
self.export.child_window(class_name="TPageControl").menu_select("Units")
Error:
pywinauto.controls.menuwrapper.MenuInaccessible

2.
Code:
page_control = self.export.child_window(class_name="TPageControl")
page_control.child_window(title="Units", class_name="TTabSheet").click_input()
Error:
pywinauto.findwindows.ElementNotFoundError: {'title': 'Units', 'class_name': 'TTabSheet', 'top_level_only': False, 'parent': <win32_element_info.HwndElementInfo - '', TPageControl, 3742488>, 'backend': 'win32'}

3.
Code:
self.export.child_window(class_name="TPageControl").menu_bar_click_input("Units", self.app)
Error:
AttributeError: Neither GUI element (wrapper) nor wrapper method 'menu_bar_click_input' were found (typo?)

4.
Code:
page_control = self.export.child_window(class_name="TPageControl")
tab_list = page_control.descendants(class_name="TTabSheet")
tab_menu = tab_list[2]
print(tab_menu)
tab_menu.click_input()
Output:
It prints: hwndwrapper.HwndWrapper - 'Units', TTabSheet
But clicks on center on the window, not on the Units tab

I've added screenshots with the TPageControl and Units elements highlighted by Inspect.exe
I've struggled a lot with this and couldn't find the issue...
Thanks in the advance for the help!

Issue solved!
If anyone else has this problem I managed to find the solution by using TabControlWrapper.
The solution that works:
page_control = self.export.child_window(class_name="TPageControl")
tab_control = TabControlWrapper(page_control.wrapper_object())
tab_control.select(2)

Hi @fbarbu15 thanks for the report! More detailed info can be obtained if you switch Inspect.exe from MSAA mode to UI Automation mode.
It looks like you used backend="uia" first and then re-used TabControlWrapper from backend="win32". This is perfect. We just need to simplify this by providing something like that:
tab_control = page_control.interpret_as("TabControl", backend="win32")
page_control.interpret_options() # should print possible control types and backend combinations

Would it make sense?
Also this particular case can be investigated in the light of library improvements. Maybe it's possible to use backend="uia" only considering few possible structures of tab control hierarchy. Marking as enhancement.

Hi @vasily-v-ryabov Thanks for looking into this. I'm testing an old (more than 10 years) application written in Delphi. I'm using the default backend="win32"  . I tried with backend="uia" but some things didn't work so I've sticked with win32.

I have the exactly same issue, but when I tried your solution the result is:

Hi @belnscm you're using "win32" backend, but @fbarbu15 uses Application(backend="uia"). Probably it makes difference.

Hi, do you know what do I need to change to get TabControlWrapper working for me? My goal is to paste "text" word in a specific tab of MobaXterm app.


KAFKA-10199: Add interface for state updater

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks @cadonna , merged to trunk.


[mongodb-replicaset] WIP - Add name label to pods

Requirements
gcr.io/google_containers/mongodb-install:0.5 must be built and pushed for these changes to work.
Description
This pull request adds a new entrypoint to the init image to add a label name to every pod in the ReplicaSet through the Kubernetes API. It is equivalent to manually run:
$ for i in 0 1 2; do kubectl label pods "messy-hydra-mongodb-replicaset-$i" name=messy-hydra-mongodb-replicaset-$i; done

It allows exposing every pod through a service with the IP of the pod as the only endpoint on the service by making its selector unique; thus, the service will target just one pod. It allows to connect external applications to the MongoDB replica set with a URI, like this:
mongodb://${IP_KUBE_NODE_0}:30371,${IP_KUBE_NODE_1}:31818,${IP_KUBE_NODE_2}:31365/?replicaSet=rs0

It partially solves #1569, but it won't expose the pods with a service for you.
Problem
Expose the first pod with the selector as they are without the changes on this pull request:
$ kubectl expose pods messy-hydra-mongodb-replicaset-0 --type=NodePort
service "messy-hydra-mongodb-replicaset-0" exposed

Describe the pods
$ kubectl get pods -o wide
NAME                            READY  STATUS  ... IP          ...
messy-hydra-mongodb-replicaset-0   1/1    Running ... <IP_ADDRESS> ...
messy-hydra-mongodb-replicaset-1   1/1    Running ... <IP_ADDRESS> ...
messy-hydra-mongodb-replicaset-2   1/1    Running ... <IP_ADDRESS> ...

The service is exposing all the pods because it uses the labels on them to create the selector, and there is no difference between the labels on the pods that are part of the StatefulSet
$ kubectl describe svc messy-hydra-mongodb-replicaset-0
Name:                   messy-hydra-mongodb-replicaset-0
Namespace:              default
Labels:                 app=mongodb-replicaset
                        release=messy-hydra
...
Selector:               app=mongodb-replicaset,release=messy-hydra
Type:                   NodePort
IP:                     <IP_ADDRESS>
Port:                   <unset> 27017/TCP
NodePort:               <unset> 30483/TCP
Endpoints:              <IP_ADDRESS>:27017,<IP_ADDRESS>:27017,<IP_ADDRESS>:27017
...

Add a label to the second pod
$ kubectl label pods messy-hydra-mongodb-replicaset-1 name=messy-hydra-mongodb-replicaset-1
pod "messy-hydra-mongodb-replicaset-1" labeled

Check the labels
$ kubectl get pods --show-labels
NAME                            READY     STATUS  ... LABELS
messy-hydra-mongodb-replicaset-0   1/1       Running ... app=mongodb-replicaset,release=messy-hydra
messy-hydra-mongodb-replicaset-1   1/1       Running ... app=mongodb-replicaset,name=messy-hydra-mongodb-replicaset-1,release=messy-hydra
messy-hydra-mongodb-replicaset-2   1/1       Running ... app=mongodb-replicaset,release=messy-hydra

Expose the second pod through a service
$ kubectl expose pods messy-hydra-mongodb-replicaset-1 --type=NodePort
service "messy-hydra-mongodb-replicaset-1" exposed

Describe the service
$ kubectl describe svc messy-hydra-mongodb-replicaset-1
Name:                   messy-hydra-mongodb-replicaset-1
Namespace:              default
Labels:                 app=mongodb-replicaset
                        name=messy-hydra-mongodb-replicaset-1
                        release=messy-hydra
...
Selector:               app=mongodb-replicaset,name=messy-hydra-mongodb-replicaset-1,release=messy-hydra
Type:                   NodePort
IP:                     <IP_ADDRESS>
Port:                   <unset> 27017/TCP
NodePort:               <unset> 31818/TCP
Endpoints:              <IP_ADDRESS>:27017
...

The service is only exposing one pod since its endpoint only matches the IP of the second pod
$ kubectl get pod messy-hydra-mongodb-replicaset-1 -o wide
NAME                            READY ... IP          ...
messy-hydra-mongodb-replicaset-1   1/1   ... <IP_ADDRESS> ...

I signed it!

@unguiculus I added support to create a service per pod. I tested it with services of type NodePort which is what I am using. Do you think is worth to provide support to services with LoadBalancer type? If that's the case, I'll test the load balancers tomorrow and improve the readme and the notes.

2017-08-17T19:31:30.273-0500 I NETWORK  [thread1] getaddrinfo("insights-mongodb-replicaset-1.insights-mongodb-replicaset.default.svc.cluster.local") failed: Name or service not known
2017-08-17T19:31:30.307-0500 I NETWORK  [thread1] getaddrinfo("insights-mongodb-replicaset-2.insights-mongodb-replicaset.default.svc.cluster.local") failed: Name or service not known
2017-08-17T19:31:35.836-0500 I NETWORK  [thread1] getaddrinfo("insights-mongodb-replicaset-0.insights-mongodb-replicaset.default.svc.cluster.local") failed: Name or service not known
2017-08-17T19:31:35.837-0500 W NETWORK  [thread1] No primary detected for set rs0
2017-08-17T19:31:35.837-0500 I NETWORK  [thread1] All nodes for set rs0 are down. This has happened for 3 checks in a row.
2017-08-17T19:31:35.837-0500 E QUERY    [thread1] Error: connect failed to replica set rs0/a9d2a121a83aa11e7b2be065c31768d6-1086025196.us-west-2.elb.amazonaws.com:27017,a9d308e7083aa11e7b2be065c31768d6-995244097.us-west-2.elb.amazonaws.com:27017,a9d35f5b083aa11e7b2be065c31768d6-1028377341.us-west-2.elb.amazonaws.com:27017

Status
I was testing the LoadBalancer services when I ran into an error:
2017-08-17T19:31:30.273-0500 I NETWORK  [thread1] getaddrinfo("insights-mongodb-replicaset-1.insights-mongodb-replicaset.default.svc.cluster.local") failed: Name or service not known
2017-08-17T19:31:30.307-0500 I NETWORK  [thread1] getaddrinfo("insights-mongodb-replicaset-2.insights-mongodb-replicaset.default.svc.cluster.local") failed: Name or service not known
2017-08-17T19:31:35.836-0500 I NETWORK  [thread1] getaddrinfo("insights-mongodb-replicaset-0.insights-mongodb-replicaset.default.svc.cluster.local") failed: Name or service not known
2017-08-17T19:31:35.837-0500 W NETWORK  [thread1] No primary detected for set rs0
2017-08-17T19:31:35.837-0500 I NETWORK  [thread1] All nodes for set rs0 are down. This has happened for 3 checks in a row.
2017-08-17T19:31:35.837-0500 E QUERY    [thread1] Error: connect failed to replica set rs0/a9d2a121a83aa11e7b2be065c31768d6-1086025196.us-west-2.elb.amazonaws.com:27017,a9d308e7083aa11e7b2be065c31768d6-995244097.us-west-2.elb.amazonaws.com:27017,a9d35f5b083aa11e7b2be065c31768d6-1028377341.us-west-2.elb.amazonaws.com:27017

The hosts used to create the replica set must be resolvable to the MongoDB client.
Proposal
Peer finding
I'd like to write a script similar to peer-finder to get the IPs or domain names that the MongoDB client can reach, instead of looking up SRV records.
Security
When the operator needs to use LoadBalancer services, the bootstrap initialization container must guarantee the creation of firewall rules for peer communication between the pods outside the Kubernetes network.
Creation order and replica set creation
The current headless service has an annotation to tolerate unready endpoints to skip waiting until the pods are ready. The bootstrap script already polls the status of the pods by trying to get a MongoDB connection from the pods. Is this enough? Do we need to guarantee the order of creation of resources, first the services and then the StatefulSet?
Use Cases

The sysadmin needs to connect an application to the MongoDB cluster within the same networking stack, but the sysadmin can not deploy the application in the same Kubernetes cluster.

The sysadmin has to deploy the application in a cluster with no access to the StatefulSet API, so he decided to create a new Kubernetes cluster to install the MongoDB-ReplicaSet chart.

The sysadmin is migrating to the cloud, but she has not containerized the application yet. However, she needs to migrate the database as soon as possible since it is causing serious performance issues. Besides, the team is willing to tolerate the delay to connect the application from the company's data center to their MongoDB cluster on the new cloud provider.

The sysadmin will deploy MongoDB in a multi-region cloud environment to guarantee high availability.

I'd appreciate any feedback to continue working on this.

Unready endpoints must be tolerated. Otherwise the whole bootstrapping with the peer finder would not work. But that shouldn't really bother you. The additional services you create don't have that annotation.
To be honest, I'm not sure if we should really add this functionality. It's a very special use case that you need access from outside the cluster. It makes the whole chart a lot more complicated.

@mariomarin Please update the PR.

Marking a stale. Please update within one week.

Sorry about the delay. I pushed some changes to get the list of peers from the LoadBalancer and NodePort services. However, I am trying to fix an issue when trying to connect to the LoadBalancer services to initialize the replica set, which I think is related to the IPTables' MASQUERADE rule.

I think there's some more work to be done here - it's still using the hacky curl to talk to the api which is not ideal. Also looks like this will race with https://github.com/kubernetes/charts/pull/1784 (and probably go in after it). So, perhaps we should have this PR use an image tagged with 0.6.

Bump! Does this need another review from @foxish or @unguiculus? 😄
This is especially handy if working locally with a minikube instance that has all your infrastructure but you're developing outside of minikube. Unless you know of a better way to handle that...

We'll likely have https://github.com/kubernetes/kubernetes/pull/55329 go into 1.9, in which case we can avoid a lot of the complexity associated with getting different labels associated with each statefulset pod.
Secondly, I think this PR adds a lot of complexity that doesn't apply to everyone. @ekryski, I'd prefer if we could figure out the separate items here (e.g, coredns support, name label, etc), discuss the solutions proposed in an issue, and break this down into separate PRs for each of those features.

The sysadmin needs to connect an application to the MongoDB cluster within the same networking stack, but the sysadmin can not deploy the application in the same Kubernetes cluster.

The sysadmin has to deploy the application in a cluster with no access to the StatefulSet API, so he decided to create a new Kubernetes cluster to install the MongoDB-ReplicaSet chart.

@mariomarin I should have taken a look at the proposal earlier. Those two don't sound like use-cases we even want to support with this chart. The chart is meant to be a fully featured mongodb installation for a quick turn-up. The use-cases you mention, or the one @ekryski mentioned with minikube are too specific IMO. @unguiculus wdyt?

@foxish I agree with you.

An update from me. I managed to figure out a work around without needing to expose this publicly (that felt kind of dirty).
In order to able to connect from outside minikube we just use kubectl port-forward be able to hit the master in the replicaset. This works great for normal development, however you're not actually able to test failover. Not ideal but when we need to test how things work with actual failover we are able to deploy our apps that talk to Mongo right into the cluster. 🤷♂

Closing as this has been worked around in other ways. Please re-open if needed.


Update relationships.md

Update the variable name being passed to context.Entry(), to help the reader discern the difference between the Course type and the variable course

I don't think there is any confusion between type and variable name.

I wonder if there was confusion, because the variable name was course2, however the usage of that variable name wasn't carried through to the last line of the code.

@krptodr the EF docs do assume that the reader is familiar enough with C# to understand the difference between types and variables plus the type is Course and the variable is course, as is standard. It seems better to follow the way code is actually written - I definitely wouldn't call a variable course2 in my code.

@krptodr Thanks for this suggestion, but we're going to leave the doc as is based on the discussion with the team above.


add action.yml to support local testing

Signed-off-by: Paul Czarkowski<EMAIL_ADDRESS>

@paulczar I think this fixes things, but take a look and see what you think.

@tybritten Can you take a look at this too and try it out? I think this is a nice addition.


Upgrade to react 16

Current peerDependencies is:
"peerDependencies": {     "react": "^0.14.0 || ^15.0.0",     "react-dom": "^0.14.0 || ^15.0.0"   }
It would be nice to update to version 16.

Fixed in<EMAIL_ADDRESS>


Make dispatch functions and action creator functions have different names

when there are several functions called setUser in one file, you know you're gonna have a bad time

we no longer use redux, so this can be closed


Configure producer using environment variables

Is there a way to configure the producer using environment variables.  The README doesn't specify that this can't be done, but since the producer setting is expected to be a list (https://github.com/zendesk/racecar/blob/master/lib/racecar/config.rb#L35), it's not clear to me how to go about it.
Instead I've had to configure this one setting in config/racecar.rb:
Racecar.configure do |config|
  config.producer = ['message.max.bytes=10485760']
end

racecar using king_konf under the hood, so you can use something like this to configure the producer via env vars
export RACECAR_PRODUCER='message.max.bytes=10485760,retries=111111'
``

Thanks for the response @deepredsky


tensorRT whisper is not working on multilingual!! (I'm korean)

anyone is there Who working on TensorRT whisper on multinugal?
i do try multiple on smal/medium/large-v3 model.
but all model is printing on english output.
please help me~!!!@
server
root@9a18589c6385:/home/WhisperLive# python3 run_server.py -p 9090 -b tensorrt -trt /root/TensorRT-LLM-examples/whisper/whisper_medium -m
# client
audio_file_path =  "C:/archive1/develop/test_audio/06_01_104204_220818_0003_NV_cut30.wav"
host="<IP_ADDRESS>"
port = 9099
lang = "ko"
model ="medium" #"large-v3"
from whisper_live.client import TranscriptionClient
client = TranscriptionClient(
host,
port,
lang=lang,
translate=False,
model=model,
use_vad=False,
)
client(audio_file_path)

i think
we must to this
wget --directory-prefix=assets https://raw.githubusercontent.com/openai/whisper/main/whisper/assets/multilingual.tiktoken

tokenizer.py
vocab_path catch teh that multiliugual.tiktoken!!!!

@ssifood thanks for pointing this out we are looking into this. we will update here with the fix.


Tweaks to env vars in Spaces

I think this is clearer, and it will also prevent Python noobs like me from having to google how to read an env var!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Recover from the panic when pushing a message with namespace consts.ParitySharesNamespaceID

Summary of Bug
Currently, if we manually add a message with namespace ID: consts.ParitySharesNamespaceID to a block and propagate it, the application panics during ProcessProposal as this latter is higher than the maximum allowed namespace.
This panic should be recovered from, as a malicious block proposer could propose an invalid block containing such a message, and halt the chain.
Panic message:
panic: pushed data has to be lexicographically ordered by namespace IDs: last namespace: ffffffffffffffff, pushed: fffffffffffffffe [recovered]
	panic: pushed data has to be lexicographically ordered by namespace IDs: last namespace: ffffffffffffffff, pushed: fffffffffffffffe

Version
f5a07435f5969bb558b72f4189611db9ff1eefad
Steps to Reproduce
Uncomment the following test:
https://github.com/celestiaorg/celestia-app/pull/592/files#r938092524
For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

I think #595 covers this and is a tad bit more general? wdyt?

also, what if the parity namespace is placed exactly at the last share of the original square, won't those shares still be in order and therefore we won't see a panic?

That is actually true. In fact, I was checking process proposal and I guess even for a block which has a message whose namespace is of the parity shares, the node will not panic, because, it will be caught before.
Currently, process proposal starts by sanitizing the messages. And then, it proceeds to calculate the data availability root. This latter is the one that panics when receiving a message that has a wrong namespace. But, in our case, it will never be reached.

yeah, that's a great point, after #592, this specific panic isn't actually reachable.


Get followers

New feature to get followers of an account

Sorry, one more fix in comments, will issue new PR


Flaky test (node_route_controller_test.go) observed in windows unit test CI run

Describe the bug
-- FAIL: TestControllerWithDuplicatePodCIDR (5.00s)
130 Error:     node_route_controller_test.go:153: Test didn't finish in time
131 Error:     node_route_controller_test.go:156: missing call(s) to *testing.MockClient.InstallNodeFlows(is equal to node1, is equal to 00:00:00:00:00:01, is anything, is equal to <IP_ADDRESS>, is equal to 1, is equal to 0) D:/a/antrea/antrea/pkg/agent/controller/noderoute/node_route_controller_test.go:129
132 Error:     node_route_controller_test.go:156: missing call(s) to *testing.MockInterface.AddRoutes(is equal to <IP_ADDRESS>/24, is equal to <IP_ADDRESS>, is equal to <IP_ADDRESS>) D:/a/antrea/antrea/pkg/agent/controller/noderoute/node_route_controller_test.go:130
133 Error:     node_route_controller_test.go:156: aborting test due to missing call(s)
134 FAIL
135 FAIL	github.com/vmware-tanzu/antrea/pkg/agent/controller/noderoute	5.107s
To Reproduce
run unit tests with windows platform multiple times
Expected
Test should pass consistently.
Actual behavior
Observed a failure in one of the CI runs. Re-running the test resolved the problem.
Versions:
Please provide the following information:
Antrea TOT master branch

This test is failing very frequently now. Any update on a possible fix?

Thanks for your report. I will take a look.

The test on windows is slow, it migh be the root cause. I will double confirm it.

@ruicao93 I have seen the test fail on Linux as well. For example, see https://github.com/vmware-tanzu/antrea/runs/1593012375?check_suite_focus=true

@ruicao93 I have seen the test fail on Linux as well. For example, see https://github.com/vmware-tanzu/antrea/runs/1593012375?check_suite_focus=true

Agree @antoninbas this is not specific to windows, and not really because of the timeout. I can reproduce the failure on linux even extending the timeout to 10 seconds. It should be some race condition causing event missed.

Agree @antoninbas this is not specific to windows, and not really because of the timeout. I can reproduce the failure on linux even extending the timeout to 10 seconds. It should be some race condition causing event missed.


Clean up more unneeded IExternals

Just some more house keeping, shrinking the internal interfaces used to split out Power Fx.

Pull Request Test Coverage Report for Build 307629

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 58.103%

Files with Coverage Reduction
New Missed Lines
%

/src/libraries/Microsoft.PowerFx.Core/Public/DisplayNames/DisabledDisplayNameProvider.cs
1
94.44%

/src/libraries/Microsoft.PowerFx.Interpreter/Environment/OptionSet.cs
6
77.78%

Totals

Change from base Build 307224:
0.01%

Covered Lines:
19813

Relevant Lines:
34100

💛  - Coveralls


Include 'name' property in MigratorConfig

PR #3416 added the ability to specify a filename for the up and down commands, but that parameter was left out of the MigratorConfig interface, causing compilation errors for those using that property and typescript.

Thanks!


Distribute on a CDN

I use a WordPress blog and would like to add this to my site. I imagined it would be as simple as adding a script tag from a CDN and using the element. After all, that's the point of a web component.
Distributing the dist on a CDN would be a good idea for such use cases where you don't have a module bundler or want to avoid an npm installation for just one package.

We use unpkg for this on the demo site. Can you use something like that?
https://github.com/github/clipboard-copy-element/blob/4969cf8c11a71759d06da2afa1318d67e44513f1/examples/index.html#L5

Oh, sure! That works for me. Perhaps be nice to document it in the README? What do you think?

I hesitate because we don't control that service, and we're going to have to update all the README's that reference it if it ever goes away. I'd much rather distribute the library on github and npm and leave how to include the library in their application up to the implementors.

I agree with @koddsson here. I would also add that there are many CDNs which are available to use, such as JSDelivr, Unpkg, CDNJS... promoting one over any others might be considered an endorsement of that CDN by our team or by GitHub writ large, it could also be seen as an endorsement of hot linking code to CDNs in general, which is a practice I personally wouldn't like to endorse.
I think I'll close this issue because the original question has been answered, and I don't think there is anything more actionable here. Thanks everyone!


Correct the code example for parallel tuning in the documentation page

ray.init(n_cpus=16)
should be
ray.init(num_cpus=16) in the documentation page
https://microsoft.github.io/FLAML/docs/Use-Cases/Task-Oriented-AutoML#parallel-tuning

Please assign me this issue.

Hi @qingyun-wu thanks for assigning me this issue. I looked through other places as well and didn't find any error which resembled this one.


Resolve urls inside separate css-files

Do you want to request a feature or report a bug?
Request a feature or request docs addition
Images in url() do not resolve if separate css-files setup is used.
https://github.com/zeit/styled-jsx#styles-in-regular-css-files
Here it is documented how to use separate files for styles. However if we do that we can no longer insert images resolved by Webpack.
I tried to use css-loader but then the styles are rendered in such a way that it won't work at all.
Are there any plans or ideas how to handle this properly? Css inside components doesn't seem right but images have to be handled with Webpack. Normally there are some means to parse them out of url(). Without Webpack I can't use image loaders like those that encode images in base64. Without separate files I can't use IDE css helping tools. Something is off.
I ended up rolling my own Webpack loader to handle this but it seems like too much complexity.

Do you know how other projects do this with loaders and such? Eg. if you have a sass project and build it with webpack (no styled-jsx). How do you solve this issue?

I think in majority of frontend projects that use Webpack css-loader is always used. This loader is the one responsible for resolving url()s in css-files. But css-loader does something else too while doing this. Its output is somehow incompatible with styled-jsx.
Most often other projects use a combination of style-loader and css-loader  called at the very end of styles processing. I.e. first different kinds of loaders like sass and so on are called. css-loader is always the last but one being called. And style-loader is normally the last one.
But I do not know how to integrate it with styled-jsx

Yes the styled-jsx loader turns CSS into JavaScript, so the url resolving should be done earlier. Have you tried to run your CSS through https://www.npmjs.com/package/resolve-url-loader before passing it to the styled-jsx's loader?

I tried now. If invoked before styledJsxLoader it seems to do nothing for some reason. Urls remain unchanged. If onvoked after styledJsxLoader then error occurrs during build:
Error: resolve-url-loader: CSS error <path to my file>: Unknown word.
I am not sure how this loader works. I used it without any options. Maybe it should configured somehow.

I was having the same issue, ended up creating my own custom loader:
function isAbsolutePath(url) {
  return /^[a-zA-Z][a-zA-Z\d+\-.]*:/.test(url)
}

module.exports = function (content) {
  if (this.cacheable) this.cacheable()
  this.addDependency(this.resourcePath)

  let index = 0
  let imports = ''
  const output = content.replace(/(?:url(?:\s+)?)\((?:'|")?(.*[^'"])(?:'|")?\)/gi, (...args) => {
    const url = args[1]
    if (isAbsolutePath(url)) {
      return args[0]
    }
    const name = `_temp_url_${index}`
    const nameAsVar = '${' + name + '}'
    imports = `${imports} import ${name} from '${url}';`
    index++
    return `url("${nameAsVar}")`
  })

  this.callback(null, `${imports} ${output}`)
}

Put this loader in between styled-jxx/webpack and options.defaultLoaders.babel (if you are using next.js) something like this:
config.module.rules.push({
  test: /\.scss$/,
  use: [
    options.defaultLoaders.babel,
    {
      loader: path.join(__dirname, './lib/styled-jsx-url-loader'),
    },
    {
      loader: require('styled-jsx/webpack').loader,
      options: {
        type: 'scoped',
      },
    },
  ],
})

It's not sure sophisticated, basically just one big regexp replace so there could be some false positives, but it does the job for me.

I made also a loader for that https://github.com/danielhusar/styled-jsx-url-loader


Report a runtime error when found an undefined foreign class

In the previous code, such class made ASSERT() fail in debug mode, and triggered a segfault in production, in the better case. In the worse case, it was a security hole (UB) and all sorts of bad.
It was one of two - dereferencing an uninitialized pointer or out-of-bounds array access. Neither way a good thing, and security men are expert in seriously crafting code so it will hit the out-of-bounds case and to some sensitive data. We're all hate it.

I would change the error message to "Could not find foreign allocator for class $ in module '$'.", and maybe also promote missing finalizer from ASSERT to error while at it.

I would change the error message to "Could not find foreign allocator for class $ in module '$'.", and maybe also promote missing finalizer from ASSERT to error while at it.

I would change the error message to "Could not find foreign allocator for class $ in module '$'."

I think this falls into the bucket of exposing internal details to the end user. We test for allocator only because a foreign class must have an allocator, and so returning allocate == NULL && finalize != NULL is an error in the C code. This is the reason we assert for that, too. But if we don't care about speed, we could in the same manner test for allocate == NULL && finalize == NULL.

and maybe also promote missing finalizer from ASSERT to error while at it.

I don't understand your intention.

I would change the error message to "Could not find foreign allocator for class $ in module '$'."

I think this falls into the bucket of exposing internal details to the end user. We test for allocator only because a foreign class must have an allocator, and so returning allocate == NULL && finalize != NULL is an error in the C code. This is the reason we assert for that, too. But if we don't care about speed, we could in the same manner test for allocate == NULL && finalize == NULL.

and maybe also promote missing finalizer from ASSERT to error while at it.

I don't understand your intention.

This is mostly a message that will happen during development. Regular users
should never see it, so exposing internals or in other words what is
missing is makes the implementor more aware of the issue.
The intension is to avoid heisenbug, where a rogue module would remove
finalisers in build mode. Don't trust users, promote assert to handled
errors.

This is mostly a message that will happen during development. Regular users
should never see it, so exposing internals or in other words what is
missing is makes the implementor more aware of the issue.
The intension is to avoid heisenbug, where a rogue module would remove
finalisers in build mode. Don't trust users, promote assert to handled
errors.

This is mostly a message that will happen during development. Regular users should never see it, so exposing internals or in other words what is missing is makes the implementor more aware of the issue.

For this case, if assertions are on, and you provide only a finalizer, a similar error is indeed what you see. But if you provide neither, then you wanted to say "no such class". This is the same thing that happens when you return NULL from bindForeignMethodFn, and this is what the example uses.

The intension is to avoid heisenbug, where a rogue module would remove finalisers in build mode. Don't trust users, promote assert to handled errors.

But it is valid to have a foreign class without finalizer, what can I do?

This is mostly a message that will happen during development. Regular users should never see it, so exposing internals or in other words what is missing is makes the implementor more aware of the issue.

For this case, if assertions are on, and you provide only a finalizer, a similar error is indeed what you see. But if you provide neither, then you wanted to say "no such class". This is the same thing that happens when you return NULL from bindForeignMethodFn, and this is what the example uses.

The intension is to avoid heisenbug, where a rogue module would remove finalisers in build mode. Don't trust users, promote assert to handled errors.

But it is valid to have a foreign class without finalizer, what can I do?

If it is valid, then the ASSERT is not necessary, and some parts of wrenFinalizeForeign could be simplified.
I can see some usage to not have finalizers with POD, so I think it is a safe optimization to not assume that finalizers are required.

If it is valid, then the ASSERT is not necessary, and some parts of wrenFinalizeForeign could be simplified.
I can see some usage to not have finalizers with POD, so I think it is a safe optimization to not assume that finalizers are required.

If it is valid, then the ASSERT is not necessary, and some parts of wrenFinalizeForeign could be simplified.
I can see some usage to not have finalizers with POD, so I think it is a safe optimization to not assume that finalizers are required.

I can see many reasons to not have finalizers.
I'm aware about the simplification of wrenFinalizeForeign(), but decided to postpone that to PR #911, because it is more related there.

If it is valid, then the ASSERT is not necessary, and some parts of wrenFinalizeForeign could be simplified.
I can see some usage to not have finalizers with POD, so I think it is a safe optimization to not assume that finalizers are required.

I can see many reasons to not have finalizers.
I'm aware about the simplification of wrenFinalizeForeign(), but decided to postpone that to PR #911, because it is more related there.


New feature request - add "expense ratio"

It would be great if "expense ratio" from the Yahoo Finance Summary tab would be added to the info that can be obtained from .info

Since this feature has been long due and nothing is changing just download the clone the repo
Edit base.py file and add the following after # get info and sustainability         data = utils.get_json(ticker_url, proxy, self.session) is defined:
expenseRatio = (data
.get('fundProfile', {})
.get('feesExpensesInvestment', {})
.get('annualReportExpenseRatio', None))
    try:
        self._info['annualReportExpenseRatio'] = expenseRatio
    except Exception:
        pass

This should be implemented. it's very easy as well.

@moedi85 @carlos-ventura  @Courvoisier13 Ran has granted me permissions to merge PRs etc, but help is appreciated in creating and/or reviewing PRs - anyone can do this.


B21576 int add order types for sc too

Agility ticket
B21576
Summary
In B21575, 2 new order types were added for the customer:  Student Travel, Early Return of Dependents.  B21576 adds these new order types for SC and TOO.  When the SC or TOO edits orders, they will be able to select the new order types and "has dependents" will update to "yes" on the backend.  In addition, the new order types will display in the "Orders" section of the "Move Details Page" for the SC or TOO.  And, finally, when SC creates a move, the "Orders" page will behave exactly like the customer version of this page.
How to test
In the .envrc.local file, add the following to turn on the feature flag:
export FEATURE_FLAG_ENABLE_ALASKA=true
restart server and client
=====
Create a move with a HHG shipment and submit it.

Login as customer
On the Orders page,

Set order type to: permanent change of station
Set has dependents to: no.

Note the move code.
Keep the customer page open.  The orders section will be viewed to confirm changes.
Open a browser window as SC
Open a browser window as TOO

=====
Test:  SC: edit orders, Student Travel

Login as SC
Open the move that was just created
Order type: Student Travel

Edit the orders
Select order type Student Travel
Save changes
In the "Orders" section, verify "Orders Type" is "Student Travel"
On the customer page,

Go to the "Orders" section: move main page > review your request > orders
Verify

Orders type is "Student Travel"
Dependents is "Yes"

Reset Orders

Edit orders

Set "Orders type" to "permanent change of station"
Set "dependents" to "no"

Click button "return home"

On TOO page

Open the move using search and the move code
Go to the "Move Details" tab
Confirm "orders type" is "Student Travel" in the Orders section.
Close the move

=====
Test:  SC: edit orders, Early Return of Dependents

Login as SC
Open the move that was just created
Order type: Early Return of Dependents

Edit the orders
Select order type Early Return of Dependents
Save changes
In the "Orders" section, verify "Orders Type" is "Early Return of Dependents"
On the customer page,

Go to the "Orders" section: move main page > review your request > orders
Verify

Orders type is "Early Return of Dependents"
Dependents is "Yes"

Reset Orders

Edit orders

Set "Orders type" to "permanent change of station"
Set "dependents" to "no"

Click button "return home"

On TOO page

Open the move using search and the move code
Go to the "Move Details" tab
Confirm "orders type" is "Early Return of Dependents" in the Orders section.
Close the move

=====
Test:  SC: edit orders, Select neither Early Return of Dependents nor Student Travel

Login as SC
Open the move that was just created
Order type: Local move

Edit the orders
Select order type Local move
Save changes
In the "Orders" section, verify "Orders Type" is "Local move"
On the customer page,

Go to the "Orders" section: move main page > review your request > orders
Verify

Orders type is "Local move"
Dependents is "No"

Reset Orders

Edit orders

Set "Orders type" to "permanent change of station"
Set "dependents" to "no"

Click button "return home"

On TOO page

Open the move using search and the move code
Go to the "Move Details" tab
Confirm "orders type" is "Local move" in the Orders section.
Close the move

=====
Test:  SC: create move

login as SC
search for customer using name "Spaceman".
select a customer and click "create move".
at the move orders page

set initial values all fields:

order type: Permanent Change of Station
remaining fields: any value is fine

verification

verify: order type,  changing existing type to another existing type

change order type to local move
verify no fields are changed

verify: order type, changing existing type to new type (student travel)

change order type to student travel
verify "are dependents included" defaults to "yes"
verify "are dependents included" is disabled

verify: order type, changing new type (student travel) to existing type

change order type to permanent change of station
verify "are dependents included" defaults to no selection
verify "are dependents included" is enabled

verify: order type, changing existing type to new type (early return of dependents)

change order type to student travel
verify "are dependents included" defaults to "yes"
verify "are dependents included" is disabled

verify: order type, changing new type (early return of dependents) to existing type

change order type to permanent change of station
verify "are dependents included" defaults to no selection
verify "are dependents included" is enabled

verify: order type, changing new type (early return of dependents) to to new type (student travel)

change order type to early return of dependents
verify "are dependents included" defaults to "yes"
verify "are dependents included" is disabled
change order type to student travel
verify "are dependents included" defaults to "yes"
verify "are dependents included" is disabled

=====
Test: turn off feature flag
In the .envrc.local file, add the following to turn on the feature flag:
export FEATURE_FLAG_ENABLE_ALASKA=false
restart server and client

SC, Create a move: On the orders page, verify neither Student Travel or Early Return of Dependents shows up in the drop down list
SC, Edit orders: On the orders page, verify neither Student Travel or Early Return of Dependents shows up in the drop down list

ANDI Screenshots
SC: move, edit orders

SC: move, orders section

TOO: move, orders section

SC: create move

Also noting the concern here that edit orders page doesn't have an option to choose "has dependent" for office users. This means if SC choosesStudent Travel and then saves -> then goes back and chooses a different orders type -> the has_dependent will remain true.


Allow slf: Py/PyRef/PyRefMut in pymethods

Looks like
#[pymethods]
impl Reader {
    fn get_iter(slf: PyRef<Self>, keys: Py<PyBytes>) -> PyResult<Iter> {
        Ok(Iter {
            reader: slf.into(),
            keys: keys,
            idx: 0,
        })
    }
}

and can be called like usual methods from python.
Resolves https://github.com/PyO3/pyo3/issues/356#issuecomment-465742221
Currently only slf: PyRef is tested.
Another concern is I observed an undesirable segfault when using arg: PyRef<PyBytes> in pymethod(though I'm not sure it's related to this PR

In general for the self type I think it's better if we had a trait (or maybe reuse extract) that is implemented for all three types instead of trying to guess the right method based on the string-representation of the type

I think it's better if we had a trait

Experimentally I introduced a new trait named FromPyPointer, which registers the pointer to GIL pool and returns the reference of it.
If you come up with a better design, please let me know.

I'll have to check if we already have the traits that can do what FromPyPointer does or with we can replace some other code with the new FromPyPointer, but otherwise it's looking good

@konstin
I want this PR move forward.
Though FromPyPointer is not implemented for Py and PyObject, we should fix this after #446.
It remove lots of Py::from_owned_ptr*s.

Merging master should make travis ci pass again (or at least only fail for minimum nightly)

Now CI passed, thanks


Include Modules in building cv.js

Hello,
I'm completely new to emscripten so my question may be stupid, but I needed to rebuild the cv.js file to include new haar files.
But when compiling the new file it seems that it didn't include some of the modules since I receive a
cv.VideoCapture is not a constructor
when calling new cv.VideoCapture(this.myVideo);
Did I made something wrong, or is the make.py file provided missing something?
Thank you.

As said here it takes some time to load all the modules, so you should either wait a constant time before executing your function or bind your function to the 'onRuntimeInitialized' event:
const cv = require("./opencv.js");
cv['onRuntimeInitialized']=()=>{
          let mat = new cv.Mat();
          console.log(mat.size());
          mat.delete();
};


delete_disk_reference fails

Versions (please add them if you have more releases for this issue)
CPI      Version:
BOSH     Version:
CF       Version:
Diego    Version:
Stemcell
Version:
Distro:

[x] Ubuntu
[ ] CentOS

Environment

Azure Environment:

[ x] Azure
[ ] AzureChinaCloud
[ ] AzureUSGovernment
[ ] AzureGermanCloud
[ ] AzureStack
[ ] Others:

Location: (e.g. East US)
Subscription ID:
Resource Group Name:
[ ] Use Managed Disks

Issue Details
bosh cck --resolution delete_disk_reference --resolution delete_vm_reference on azure to delete stale disk references and are getting the error
Logs (Please redact all sensitive information in your logs)
Task 210 | 18:52:49 | Scanning 11 persistent disks: Looking for inactive disks (00:00:06)
                    L Error: CPI error 'Bosh::Clouds::CloudError' with message 'This function should only be called for unmanaged disks' in 'has_disk' CPI method
Task 210 | 18:52:55 | Error: CPI error 'Bosh::Clouds::CloudError' with message 'This function should only be called for unmanaged disks' in 'has_disk' CPI method

Repro steps (if necessary, delete otherwise)

Delete a vm with persistent disk
run bosh cck --resolution delete_disk_reference --resolution delete_vm_reference

Could you please run "bosh task 210 --debug > 210.log" to collect the task debug log and send it to the email<EMAIL_ADDRESS>Are you using azure managed disks? You can check the property use_managed_disks in bosh.yml.
What's the CPI version?

issue happens at this line in cloud.rb and line in disk_manager.
After a managed disk is removed, has_disk? will also check if it has a unmanaged disk with the same diskid.

@tinygrasshopper
@gossion has fixed the issue, and the fix is included in the version v34.
Could you please try it and let us know if you have any issues?

Closing as stale, feel free to mention us and have it re-opened.


PreventHome works only if mount is directly under /home

What happened?
node-exporter is unable to do a statfs if a filesystem is mounted under a subdirectory of /home/
error reported
"msg="Error on statfs() system call" err="permission **denied"**

How to reproduce it (as minimally and precisely as possible):
mount a filesystem under /home//foobar
launch node-exporter
get the aformentioned error
Environment
linux, ubuntu 18.04

Role version:
latest published on galaxy

Anything else we need to know?:
The bug is located at this place https://github.com/cloudalchemy/ansible-node-exporter/blob/master/templates/node_exporter.service.j2#L37
the test should be replaced with {% for m in ansible_mounts if m.mount.startswith('/home') %} at least it worked for me.

This role has been deprecated in favor of a the prometheus-community/ansible collection.


add support for a homebrew installation/update

Is your feature request related to a problem? Please describe.
I would like to have a homebrew tap to manage the installation and upgrade of the cmctl
Describe the solution you'd like
Ideally I would like to have a command like:
brew install cmctl
Describe alternatives you've considered

I could download a release the problem is that I would not get the upgrade automatically.
I could do a go install command but it seems complex to find the right command to install it using the go toolchain.

/kind feature

Hi! Thanks for opening an issue for this, I also wish we had an Homebrew formula (official or tap) for cmctl.
/area ctl

I also ran in to this issue. As I happen to already use the asdf version manager, I was able to install cmctl as follows:
asdf plugin add cmctl https://github.com/asdf-community/asdf-cmctl.git
echo "cmctl 1.9.0-beta1" >> ~/.tool-versions
asdf install
cmctl check api

This seems to have resolved this satisfactorily for me.

FYI opened https://github.com/Homebrew/homebrew-core/pull/114104 for inclusion in Homebrew core.

/reopen
/remove-lifecycle rotten

@remyleone @eblume @maelvls Homebrew accepted a merge of cmctl into homebrew/core, so as of today you can install cmctl with just brew install cmctl (you might need to brew update first of course). 🎉
I'm going to add it to the documentation for cmctl too before this issue is complete.

And that's the docs merged too. You can see the homebrew instalation instructions on https://cert-manager.io/docs/reference/cmctl/#homebrew.
@irbekrm can we close this issue?

Thanks for implementing and documenting this @JamesLaverack !


designspaceLib: allow to construct descriptors from keyword args

... like data classes.
For now, I resisted the temptation to rewrite the whole thing use attrs or dataclasses, as I am time-constrained. I just wanted to be able to initialize axis/source/instance descriptors using their default constructor, which is more idiomatic than creating an incomplete object and then mutating its attributes afterwards.
I kept the same defaults as before (basically all None or False). The defaults are mostly meaningless, which is another pain point when using dsLib -- you don't know what piece of data is actually required by what consumer.
We shall figure that out one day.

FWIW, when programmatically creating DesignSpace objects, I find I frequently use helper functions like
def addAxis(doc, name, tag, minimum, default, maximum):
    axis = doc.newAxisDescriptor()
    axis.name = name
    axis.tag = tag
    axis.minimum = minimum
    axis.default = default
    axis.maximum = maximum
    doc.addAxis(axis)

I usually don't need to descriptor objects themselves, but do miss a convenient way to add the to the ds document. Your proposed change makes things a little easier, but requires importing the right classes in addition to the DesignSpaceDocument class.
Perhaps we can consider adding methods of the following form:
	def addAxisDescriptor(self, name, tag, minimum, default, maximum):
	    axis = self.newAxisDescriptor()
	    axis.name = name
	    axis.tag = tag
	    axis.minimum = minimum
	    axis.default = default
	    axis.maximum = maximum
	    self.addAxis(axis)

yeah, I like that. I'm on it

Awesome, thanks!


If "console" is false, then windows' mouse pointer doesn't be disappeared

In the config.json file, if I change "console" to "false", windows' mouse pointer (not the game's one) won't be disappeared.
It might be disappeared when the first run after changing the parameter, but even if it's disappeared, while playing game, windows' mouse pointer would be suddenly shown.

Known and fixed, will be in next release.
Can be closed @yamashi

Known and fixed, will be in next release.
Can be closed @yamashi


Problem with CUAD

Hi Lars,
I've just updated abapGit to latest build and I tried pulled the FALV but as in SAPLink I'm having problems with CUAD. I have to manually change variable l_nok to be empty while debugging to be able to install CUAD.

Additionally after installing I still see difference between local and remote version. Which should not be I think.

Cheers
Łukasz

what version(SAP_BASIS) are you using?

750 SP2

thanks,
no problems on 740 SP10
note 2159455 introduces the CHECK_ADM form, I've earlier done an evil hack to get it working, see:
https://github.com/larshp/abapGit/blob/d093c2b6045db26a7d48905a7b929b62ccbf32a9/src/zabapgit_objects.prog.abap#L1446
After applying the latest version of note 2159455 I have reproduced the issue on my test system

hmm, it works if a dummy text is inserted in the ADM variable, however I do not know the consequences of this
perhaps just skip RS_CUA_INTERNAL_WRITE, and insert the data in the relevant places instead

I'd leave it adding this dummy text, I've checked on the code and if it's empty then this function always return l_nok = 'X', but if it's not then it does some additional checks. So for me it's sounds better to set dummy text instead of doing all manipulations directly

yeah, but filling setting ADM-ACTCODE ='TEST'. might be bad, and have some unforeseen consequences

Did some more testing, creating CUA saving to abapGit and reinstalling works., as ADM data is written, see https://github.com/larshp/PROG/blob/master/zfoobar.prog.xml#L72
Looking at the code for SAPLINK and the description in SAP note 2159455, I would regard this CUA as corrupt, as SAPLINK deletes the ADM fields.
You can:

Edit the XML manually and add some values in the ADM fields, however I do not know what the correct values should be(caution, I do not recommend this approach). Perhaps you can debug the standard code to find the correct logic.

Recreate the CUAs manually, I tried deleting the objects in the function group and recreating, but it seems that this does not initialize the ADM data. So perhaps the entire function group has to be deleted and recreated(most objects can be copied)

Tried deleting the function group and recreated a CUA manually, ADM values are written to the XML:

No risk no fun, I've debug a bit and for me the numbers which are stored in ADM are the latest ID used in STATUSes and TITLEs, so I've manually entered proper entries to XML and now everything works like a charm. I know it's ok, because before when I was trying to edit the GUI status I was receiving an error that this status is inconsistent so I can only view it. Now I can edit it without problems.

Thanks a lot.


Create a new publishing task in gradle

This will be used to generate the artifacts to be published on maven.
Closes #195

@yigit I updated this PR to include source code in the release.  PTAL. Thanks!

Thanks both!


Update terms and conditions

@TimKardol should we update priority?

Yes, should be added to the footer as two links. I think both policies are up to date or shouldn't change. Maybe check them for the years. I'll put this as high priority.

Updated texts for the privacy policy and cookie consent
Add banner for cookies that actually functions

I wanted to create a bug report for this. This is actually legally required since you're storing PII so would be nice to add it...

@Lucky-Loek Yup, we are aware of it...

Closed in #587


Fix random crashes when using yield()

e8611966de4dfc9c28a7a4de1798f3f10ff87f80 intended to replace indexed operator by pointer arithmetic, but in the way altered the meaning of the code in a subtle way.
I was getting random crashes at random places where I was using yield() heavily.
This PR makes everything work again, while keeping the intent of the referred commit.
~UPDATE: Getting crashes now on Android. Looking deeper...~
UPDATE: Now it's fixed. I had got lost in the expression and my first attempt was wrong, more or less the same that happened to the original author. @hpvb, If only there were GPS for these, don't you think? :)

Yes, I see my mistake now. My apologies!
LGTM


[Ruby 2.5] Hash#transform_keys

https://bugs.ruby-lang.org/issues/13583
Specs are based on the specs for Hash#transform_values.
There are similar specs in Rails repository. Not sure we should just copy-past them

Again, I'm really sorry I didn't check your PR for this; not fun for you and more work for me lol!
Very good PR. I merged it.
I took the liberty from removing the "returns nil if we break from the block" for a couple of reasons:

it doesn't return nil, it returns whatever value was given to break (of nil when none is given as in your example)
this is not really relevant to transform_keys/values. For example, if you implement transform_keys/values in Ruby, it is simply not possible to not return the break value (short of raising an error within an ensure or something silly like that)
we'd have to test that for all methods using blocks, and we'd also have to test with throw, raise and return. And next too.
In short, I feel that Rubyspec takes for granted that any implementation will have a consistent handling of blocks with regards to flow control.


XZ (LZMA) Backdoor

Hey everyone, I haven't had a chance to look into it, but does anyone know if the XZ version we have here is vulnerable to the recently discovered backdoor? If so, I'll update the version and pull the old one :)

the BCR only has 5.4.5 https://github.com/bazelbuild/bazel-central-registry/tree/main/modules/xz, affected versions are only 5.6.0 and 5.6.1

Legend, thank you!


Favicon

The fontawesome feather would be a good fit

More details on how to implement here: https://rstudio4edu.github.io/rstudio4edu-book/rmd-fancy.html


Can't log in or register. Site appears to be bugged out

It's been like this for a few hours now.
I got logged out and cannot access anyone's calcs or log back in or register.

Thanks for the report! Something went wrong with mLab service, I've restarted the server and now all seems back to normal.


chore(components): optimize component templates with each directive

Overview
This PR is for consolidating repeating occurrences of HTML fragments in component demos, by more utilizing Handlebars features.
Added
Handlebars {{#each}} usage in component demo templates.
Testing / Reviewing
Testing should make sure no component demo is broken.

Thanks for going through and updating all of this @asudoh!! I went through all the components and the only thing that looked off was the Data Table V2

@alisonjoseph Thanks a lot for reviewing and spotting those! Fixed.

Thanks @alisonjoseph!


Added Programming languages with alphabet "B"

I have added all the programming language with alphabet "B" in dropdown

I have raised a PR kindly merge it
Also by mistake I also committed programming languages with alphabet "I" in the same PR so please merge it as well


Update NearFutureConstruction for 1.1, primary moved to SpaceDock

Not sure why the build failed. JSONLint says the syntax is valid...

Now it complains about B9PartSwitch, which - as far as I can tell - points towards an external Netkan file. Is that error expected in such a case? Will it still work?
Or does the error mean that there is no 1.1 compatible release at the end of that external Netkan file? Which would be kind of odd, since Nertea bundled a 1.1 compatible version with the direct SpaceDock download, and he'd have to have gotten it from somewhere.

There's no 1.1 release on github.

Tests fine locally. As a heads up this won't be available via CKAN until a version of B9PartSwitch is available.

I see in the B9PartSwitch repository that @blowfishpro has what appears to be a ready to go 1.1 version of the .dll but no release yet.

I was waiting for something that actually requires it to be released.  I'll package up a release today.

New version released, should be picked up by NetKAN on the next run


SREP-1863: dedicated-admin systemgroup added to [cluster]rolebindings

This enables the customer to create any SA in the dedicated-admin namespace and then get dedicated-admin privileges with that SA.
Feature is documented here: https://docs.openshift.com/dedicated/admin_guide/service_accounts.html#dedicated-admin-granting-service-accounts-access

/hold
Couple of things to work out still..

/hold cancel

RoleBindings for existing clusters are not updated as a part of this.  Thought about creating new rolebindings but nobody w/ a live cluster has asked for this feature so they don't care.  I'll propose we simply cleanup manually by forcing recreation of the rolebindings:  oc get rolebinding -A | grep dedicated-admin | grep -v "^openshift" | awk '{print "oc delete rolebinding " $2 " -n " $1}' | sh

Offhand this looks good, but in order to provide better feedback, I'm trying to run it locally and see if it does what we expect.

@jewzaam One suggestion based on testing in minikube:
I had to make this change in order for the ClusterRoleBinding to be created:
[dakini@nibbana dedicated-admin-operator ((66c8dd3...) *$%)]git diff
diff --git a/manifests/02-dedicated-admins-cluster.ClusterRoleBinding.yaml b/manifests/02-dedicated-admins-cluster.ClusterRoleBinding.yaml
index a9aeb5f..75d09df 100644
--- a/manifests/02-dedicated-admins-cluster.ClusterRoleBinding.yaml
+++ b/manifests/02-dedicated-admins-cluster.ClusterRoleBinding.yaml
@@ -1,4 +1,4 @@
-apiVersion: authorization.openshift.io/v1
+apiVersion: rbac.authorization.k8s.io/v1
 kind: ClusterRoleBinding
 metadata:
   name: dedicated-admins-cluster

During testing, dedicated-admin-operator created the following rolebindings automatically when a new namespace was created, so the operator is working:
[dakini@nibbana ~]$ kubectl get rolebindings -n test20190806
NAME                         AGE
dedicated-admins-project-0   14s
dedicated-admins-project-1   14s

And the dedicated-admin namespace exists. So that looks good too.
[dakini@nibbana ~]$ kubectl get serviceaccount -n dedicated-admin
NAME      SECRETS   AGE
default   1         9m20s

I wasn't able to test if SAs within the dedicated-admin namespace are automatically granted dedicated-admin permissions. I might need a full openshift cluster for that test.


[ENG-13419] Produce logs in json format

Task linked: ENG-13419 Failure encountered during metadata extractor processing


Removing the foreground widget from a tab panel causes it to be hidden

When removing the foreground widget from a tab panel, either by adding it to another region or sending it a close-request, the TabPanel will hide the foreground widget.
The culprit is the currentChanged handler in TabPanel, which hide()s the previous foreground widget without checking to see if the previous widget is still a child of the panel.
I have some free time this weekend, so I can submit PRs for this and #362 if you'd like.

Thanks for reporting this!

Although I wonder if previousTitle should be null for this case. i.e. the true bug might be in the tab bar. Perhaps the previousTitle should be null if it's already been removed.


Support the Secret Service API

This is the closest thing to a standardized secret management API on Linux distributions.
Pros:

People can use kbs2 as an integrated secret manager in any tool that knows how to use the Secret Service API

Cons:

I have to think about D-Bus

xref: https://www.freedesktop.org/wiki/Specifications/secret-storage-spec/


ci: Update fake-seam-connect

@razor-x I think this PR can be closed in favor of https://github.com/seamapi/javascript-http/pull/150 which updates fake-seam-connect and unskips the tests?


Resizing log window does not reflow log messages

Description

Resizing the log window does not reflow log messages. As a result, they go under the scroll bar
Steps to reproduce

Open the log tab
Shrink the tab until a message would go under the scroll bar

Expected results
Messages reflow on resize so they do not become hidden under the scroll bar
Actual results
Messages do not reflow and become hidden under the scroll bar. Also, the scroll bar is transparent so it looks wrong
Versions

Deephaven: 0.2.1

@mattrunyon I'm unable to reproduce this on 59deee2c, is this still reproducible for you?

Can't reproduce in 0.3.1. Reproduced in 0.2.1, so it must have been fixed by something in between


fix: runtime namespaces should always override buildtime values

The previous behavior precluded operator-level defaulting at runtime
because setting the generate-with-watched-namespaces property was
marking the controller as having explicitly set namespaces. We now only
set that status if and only if namespaces were set using annotations.
Fixes #653

/cc @Jamstah @shawkins

Is it possible to have a variant of this that acts over all controllers?

Is it possible to have a variant of this that acts over all controllers?

Not sure I understand the question

Will this have to be specified for each controller?

Sorry I mean could a variant of generate-with-watched-namespaces work over all controllers?

Sorry I mean could a variant of generate-with-watched-namespaces work over all controllers?

Ah, that's a different issue, which we could indeed support. Can you open a ticket for it, please? Not sure if I'll get to it before my vacations, though.

This doesn't seem to have worked for me.
I built the keycloak operator with:
mvn clean package -Doperator -Dquarkus.container-image.build=true -Dquarkus.operator-sdk.controllers.keycloakrealmimportcontroller.generate-with-watched-namespaces=JOSDK_WATCH_CURRENT -Dquarkus.operator-sdk.controllers.keycloakcontroller.generate-with-watched-namespaces=JOSDK_WATCH_CURRENT

And run with the env var:
QUARKUS_OPERATOR_SDK_NAMESPACES=""

And I got:
2023-07-13 18:09:55,212 INFO [io.qua.ope.run.OperatorProducer] (main) Quarkus Java Operator SDK extension 6.2.2-SNAPSHOT (commit: a01a9fb on branch: fix-operator-level-namespaces) built on Thu Jul 13 17:20:47 GMT 2023
2023-07-13 18:09:55,368 INFO [io.jav.ope.Operator] (main) Registered reconciler: 'keycloakrealmimportcontroller' for resource: 'class org.keycloak.operator.crds.v2alpha1.realmimport.KeycloakRealmImport' for namespace(s): [openshift-operators]
2023-07-13 18:09:55,401 INFO [io.jav.ope.Operator] (main) Registered reconciler: 'keycloakcontroller' for resource: 'class org.keycloak.operator.crds.v2alpha1.deployment.Keycloak' for namespace(s): [openshift-operators]
2023-

Does the runtime property have to be something other than the empty string perhaps?

Does the runtime property have to be something other than the empty string perhaps?

Probably, yes, especially if it's set as an env variable because I think an empty value for an env var unsets it.

Although I did a test setting:
QUARKUS_OPERATOR_SDK_NAMESPACES="jammy"

and it worked:
2023-07-13 18:50:54,259 INFO [io.jav.ope.Operator] (main) Registered reconciler: 'keycloakrealmimportcontroller' for resource: 'class org.keycloak.operator.crds.v2alpha1.realmimport.KeycloakRealmImport' for namespace(s): [jammy]
2023-07-13 18:50:54,282 INFO [io.jav.ope.Operator] (main) Registered reconciler: 'keycloakcontroller' for resource: 'class org.keycloak.operator.crds.v2alpha1.deployment.Keycloak' for namespace(s): [jammy]

However, that doesn't work with OLM. The value OLM passes to tell the operator to watch all namespaces is the empty string, so while this PR helps, its not quite a complete solution to what we need.

However, that doesn't work with OLM. The value OLM passes to tell the operator to watch all namespaces is the empty string, so while this PR helps, its not quite a complete solution to what we need.

Can you elaborate on this part, please? When you mean OLM, which part of OLM exactly?

https://github.com/quarkiverse/quarkus-operator-sdk/issues/656

And, thanks for this part :)


Batch API error responses are (sometimes) not according to protocol

If all objects in upload / download operations have errors, we need the entire request to return an error status code, most likely 422 (the API docs are somewhat unclear on that).
See https://github.com/datopian/giftless/blob/master/giftless/view.py#L65

from https://github.com/git-lfs/git-lfs/blob/master/docs/api/batch.md#response-errors:
Response Errors

LFS servers can respond with these other HTTP status codes:

    401 - The authentication credentials are needed, but were not sent. Git LFS will attempt to get the authentication for the request and retry immediately.
    403 - The user has read, but not write access. Only applicable when the operation in the request is "upload."
    404 - The Repository does not exist for the user.
    422 - Validation error with one or more of the objects in the request. This means that none of the requested objects to upload are valid.


Autostart on login

I can start on login through osx but it will open the full window, can we get it to just start the app with the statusbar indicator and nothing else? # @

There is no such features now.


feat: Torch conversion for ONNXIdentityOp

Added:

New torch conversion for Identity operator
Lit tests
Docstrings

We agreed in the helprack call that this operation disappears in ONNX-MLIR and this PR should be closed with "won't do".