Alert name is not specifying in the alert being received when more than one is triggered.

What did you do?
Configured the alert for node memory usage.
**What did you expect to see?
The triggered alert should show the name of all nodes that are triggered.
What did you see instead? Under which circumstances?
Alerts are receiving with the node name when there is one node memory alert is triggered. But the name of node is not showing if more than one node memory alert is triggered.
Environment

Prometheus Operator version:
insert image tag or Git SHA here

Kubernetes version information:
insert output of kubectl version here

Kubernetes cluster kind:
insert how you created your cluster: kops, bootkube, tectonic-installer, etc.

Manifests:

insert manifests relevant to the issue

Prometheus Operator Logs:

insert Prometheus Operator logs relevant to the issue here

This is unrelated to the Prometheus Operator, it's about usage of Prometheus/Alertmanager.
It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.
I hope you understand it doesn't make sense for this repo to be an additional point to solve Prometheus problems, it makes sense to bundle them in one place :slightly_smiling_face: .


Feature/uot 98007: Updates to QAReader/MLapp

For improving QA model, made several changes to the mlapp and QAReader:

Instead of combining inputs into one, then chunking on the max_len, this reads the inputs in as separate contexts and chooses the best from each to compare.
Instead of getting the best answer from each chunk, this calculates a score for the best answer compared to a null_answer (based on this: https://qa.fastforwardlabs.com/no answer/null threshold/bert/distilbert/exact match/f1/robust predictions/2020/06/09/Evaluating_BERT_on_SQuAD.html#Using-the-null-threshold) and only returns answers that score below the null_threshold.
Instead of returning answers in their chunk order, this sorts them by the score so the best answer is returned first.
Instead of returning a list of strings (answers), returns a list of dictionaries ("text", "context", and "score" keys) - so we can identify which context produced the answer

Other changes:

Added comments, typing, reference to source, pulled out model args into config.

Testing/Issues:

Has been tested manually on CPU (not on GPU).
The webapp needs to change to handle the new answer formats and show the source context.

To test manually, spin up the mlapp. In the console:
curl --location --request POST 'http://localhost:5000/questionAnswer' \
--header 'Content-Type: application/json' \
--data '{
    "query": "When is marijuana legalized",
    "search_context": [
        "Virginia'\''s Democratic-controlled Legislature passed a bill legalizing the possession of small amounts of marijuana on Wednesday, making it the 16th state to take the step. Under Virginia'\''s new law, adults ages 21 and over can possess an ounce or less of marijuana beginning on July 1, rather than Jan. 1, 2024. Gov. Ralph Northam, a Democrat, proposed moving up the date, arguing it would be a mistake to continue to penalize people for possessing a drug that would soon be legal. Lt. Gov. Justin Fairfax, also a Democrat, broke a 20-20 vote tie in Virginia'\''s Senate to pass the bill. No Republicans supported the measure. Democratic House of Delegates Speaker Eileen Filler-Corn hailed the plan. Today, with the Governor'\''s amendments, we will have made tremendous progress in ending the targeting of Black and brown Virginians through selective enforcement of marijuana prohibition by this summer she said in a statement. Republicans voiced a number of objections to what they characterized as an unwieldy, nearly 300-page bill. Several criticized measures that would grant licensing preferences to people and groups who'\''ve been affected by the war on drugs and make it easier for workers in the industry to unionize. Senate Minority Leader Tommy Norment also questioned Northam'\''s motives.",
        "We have a governor who wants to contribute to the resurrection of his legacy, Norment said, referring to the 2019 discovery of a racist photo in Northam'\''s 1984 medical school yearbook. The accelerated timeline sets Virginia cannabis consumers in an unusual predicament. While it will be legal to grow up to four marijuana plants beginning July 1, it could be several years before the state begins licensing recreational marijuana retailers. And unlike other states, the law won'\''t allow the commonwealth'\''s existing medical dispensaries to begin selling to all adults immediately. Jenn Michelle Pedini, executive director of Virginia NORML, called legalization an incredible victory but said the group would continue to push to allow retail sales to begin sooner.",
        "In the interest of public and consumer safety, Virginians 21 and older should be able to purchase retail cannabis products at the already operational dispensaries in 2021, not in 2024, Pedini said in a statement. Such a delay will only exacerbate the divide for equity applicants and embolden illicit activity. Northam and other Democrats pitched marijuana legalization as a way to address the historic harms of the war on drugs. One state study found Black Virginians were 3.5 times more likely to be arrested on marijuana charges compared with white people. Those trends persisted even after Virginia reduced penalties for possession to a $25 civil fine. New York and New Jersey also focused on addressing those patterns when governors in those states signed laws to legalize recreational marijuana this year. Northam'\''s proposal sets aside 30% of funds to go to communities affected by the war on drugs, compared with 70% in New Jersey. Another 40% of Virginia'\''s revenue will go toward early childhood education, with the remainder funding public health programs and substance abuse treatment.",
        "Those plans, and much of the bill'\''s regulatory framework, are still tentative; Virginia lawmakers will have to approve them again during their general session next year. Some criminal justice advocates say lawmakers should also revisit language that creates a penalty for driving with an open container of marijuana. In the absence of retail sales, some members of law enforcement said it'\''s not clear what a container of marijuana will be. The bill specifies a category of social equity applicants, such as people who'\''ve been charged with marijuana-related offenses or who graduated from historically Black colleges and universities. Those entrepreneurs will be given preference when the state grants licensing. Mike Thomas, a Black hemp cultivator based in Richmond who served jail time for marijuana possession, said those entrepreneurs deserved special attention. Thomas said he looked forward to offering his own line of organic, craft cannabis. Being that the arrest rate wasn'\''t the same for everyone, I don'\''t think the business opportunities should be the same for everyone"
    ]
}'

Results (Current/Old):
gc-ml-gpu             | Question: When is marijuana legalized
gc-ml-gpu             | 2021-05-14 13:30:06,306 [INFO][PID:11]: ['Wednesday', 'governors in those states signed laws to legalize recreational marijuana this year']

First time:
gc-ml-gpu             | 2021-05-14 13:30:06,332 [INFO][PID:11]: time: 25.6702 seconds

Second time:
gc-ml-gpu             | 2021-05-14 13:32:26,841 [INFO][PID:11]: time: 7.6491 seconds```

Results (New):
gc-ml-gpu             | Question: When is marijuana legalized
gc-ml-gpu             | 2021-05-14 13:35:36,964 [INFO][PID:11]: [{'text': 'it will be legal to grow up to four marijuana plants beginning July 1', 'null_score_diff': -5.801546812057495, 'context': 1}, {'text': 'Wednesday', 'null_score_diff': -5.7262126207351685, 'context': 0}, {'text': '', 'null_score_diff': 0.0, 'context': 2}, {'text': '', 'null_score_diff': 0.0, 'context': 3}]

First time:
gc-ml-gpu             | 2021-05-14 13:35:36,968 [INFO][PID:11]: time: 6.0283 seconds

Second time:
gc-ml-gpu             | 2021-05-14 13:36:14,645 [INFO][PID:11]: time: 6.8298 seconds

can you add some pytests to QA


ARROW-3859: [Arrow][Java] Fixed backward incompatible change.

Throwing exceptions on default breaks a lot of existing client code.
Reverting the change for now, we can check for a deprecation and moving to new
method in the future.
Had to fix the checkstyle comment suppression filter too.

cc @BryanCutler @siddharthteotia please let me know if this looks ok.

@fsaintjacques  Sure changed, but am retaining the fixes for checkstyle comment suppression filter.

@siddharthteotia - can you please review and merge.


Cannot write intercepting middleware

Due to PR #118, it appears we can no longer write middleware that "intercepts" a message from the user (ie: doesn't call await next())
The easiest example is:
public class Interceptor : IMiddleware, IReceiveActivity
{
    public Task ReceiveActivity(IBotContext context, MiddlewareSet.NextDelegate next) => Task.CompletedTask;
}

because the above Middleware implementation doesn't call await next(), the hosting Bot's OnReceive should never be hit; however it will be.
cc @cleemullins @billba @ryanvolum

@brandonh-msft and I have been writing the same middleware in both SDKs.
In the node SDK, this works to intercept messages:
export class TestMiddleware implements Middleware {
    public async receiveActivity(context: BotContext, next: () => Promise<void>): Promise<void> {
    }
}

As expected, not returning next effectively keeps the message from propagating through the rest of the middleware/bot logic, so a bot that uses this middleware is completely unresponsive.

You can write Intercepting Middleware that manipulates the incoming Activity. You just can't currently PREVENT OnReceive from being called.
This is by design. The design may be incorrect, and if so we can change it. I'm open to how folks think this should work.
Current behavior is:

Middleware is run. If Middleware calls next(), then then goto 1 until we're all done.
Once All Receive Middlware is run OnReceive is called.

If I understand correctly you are asserting the following should be true:

If ALL Receive Middleware is run, and ALL of it calls next(), then (and only then) should OnReceive be called.

Is that accurate?
I'm not clear on what the behavior should be, as I can make a case either way.

After a quick consult with @tomlm @stevenic they both believe (as do you) that the following behavior should be what we have:

If ALL Receive Middleware is run, and ALL of it calls next(), then (and only then) should OnReceive be called.

right; and that matches up with what the Node SDK is currently doing as @ryanvolum highlighted.

By intercept messages do you mean prevent them from being sent to the user? @brandonh-msft

@brandonh-msft I'll probably get this taken care of late tonight or tomorrow.

@RyanDawkins I mean you get a message from the user and your Middleware can do something that usurps the rest of the dialog flow of the bot (ie: everything that would've run as defined in OnReceive)

@RyanDawkins This also means that the message doesn't flow through the following middleware of that pipe. This is to say that if you don't return next in receiveActivity, no other receiveActivity middleware will be called, BUT postActivity middleware will be called.

One way to prevent postActivity from returning back is to do IList.clear() on the activities parameter. That is how we are stopping messages from going to the user so we can make a list of messages delay as if the user is typing.

That seems fine, though more generally not returning should stop propagation through that pipe

@RyanDawkins we actually have a 'delay' activity that will let you insert a delay between activities.
@ryanvolum definitely agree that next() should stop propagation. One work around for now would be to look at the responses queue and just no-op your bots OnReceive() handler if there are already responses queued up.

@Stevenic , about the workaround you mentioned in the previous message: isn't there a problem with the QnAMaker middleware by doing this? As this middleware is setting responses if there is a match in the QnA

@nrobert I haven't looked at the QnA Maker middleware implementation but i'm assuming it's currently designed to run on the leading edge of routing which is problematic in general.  QnA Maker can be a bit aggressive about answering queries so more often than not you'd want to run it as a fallback if nothing else answers.  With our latest middleware design you can do that but it looks like C# is currently missing a key addition that I made to the JS SDK to enable this.
You can code QnA Maker to run on the trailing edge of routing which means it will run after everything else has run but you need a way of knowing if the bot has already replied or not. You could look at the responses queue but that's not super reliable as it can get flushed (and it's going away soon) so on the JS side I added a read-only context.responded property you can check to see if any responses have been sent during the turn (it would be a great PR to add that to the C# side.)  Using this flag QnA Maker could conditionally decide to run only when nothing else has sent a message.

This is fixed by PR #146.
Tests around intercepting middleware have also been added into the test suite.
Test showing OnReceive slated to run.
Test showing OnReceive running when the Middlware pipeline is empty.
Test #1 showing an Intercepting Middleware.
Test #2 showing an Intercepting Middlware.


Python Version

Hello
Could you please tell what python version did you use ?
Or if anyone was able to execute this solution, please let me know.
Thanks

What's the issue? If it is related to downloading Python, then you can download it via Microsoft Store and possibly by running the command in Terminal. Remember the fact that some extensions or libraries might not work in the latest Python and vice versa for the later version of Python.


Can the action include the steps to create a badge?

As described in the docs one can generate a badge for the test what is a really cool feature!
Sadly the setup is very complex and error prone, I therefore wonder if the action can be enhanced to include the required shellscripts and action from the current
steps:
- …
- name: Publish Test Results
  uses: EnricoMi/publish-unit-test-result-action@v2
  id: test-results
  if: always()
  with:
    junit_files: "test-results/**/*.xml"

- name: Set badge color
  shell: bash
  run: |
    case ${{ fromJSON( steps.test-results.outputs.json ).conclusion }} in
      success)
        echo "BADGE_COLOR=31c653" >> $GITHUB_ENV
        ;;
      failure)
        echo "BADGE_COLOR=800000" >> $GITHUB_ENV
        ;;
      neutral)
        echo "BADGE_COLOR=696969" >> $GITHUB_ENV
        ;;
    esac

- name: Create badge
  uses: emibcn/badge-action@d6f51ff11b5c3382b3b88689ae2d6db22d9737d1
  with:
    label: Tests
    status: '${{ fromJSON( steps.test-results.outputs.json ).formatted.stats.tests }} tests, ${{ fromJSON( steps.test-results.outputs.json ).formatted.stats.runs }} runs: ${{ fromJSON( steps.test-results.outputs.json ).conclusion }}'
    color: ${{ env.BADGE_COLOR }}
    path: badge.svg

- name: Upload badge to Gist
  # Upload only for master branch
  if: >
    github.event_name == 'workflow_run' && github.event.workflow_run.head_branch == 'master' ||
    github.event_name != 'workflow_run' && github.ref == 'refs/heads/master'
  uses: andymckay/append-gist-action@1fbfbbce708a39bd45846f0955ed5521f2099c6d
  with:
    token: ${{ secrets.GIST_TOKEN }}
    gistURL: https://gist.githubusercontent.com/{user}/{id}
    file: badge.svg

to something like this:
steps:
- …
- name: Publish Test Results and generate badge
  uses: EnricoMi/publish-unit-test-result-action@v2
  id: test-results
  if: always()
  with:
    junit_files: "test-results/**/*.xml"
    badge_branch: 'master'  # this enables generation, but on master branch only
    badge_token: ${{ secrets.GIST_TOKEN }}
    badge_file: 'badge.svg'
    badge_url: https://gist.githubusercontent.com/{user}/{id}
    badge_label: Tests

this will greatly reduce the amount of setup and maintenance and is less error prone.

As in #397, the nice thing about actions is they do very specific things, while they can be combined with other actions doing their specific thing.
The badge creation is provided by an existing action, and the docs show how to combine both. The publish action has been extended so it provides all information needed for this use case, but badge creation is generic and not test-results specific, so combining existing actions should make more sense than having the publish action create the badge. Everyone benefits from improvements to the badge-creation action, but hardly anyone benefits from improvements to a badge creation feature in the publish action.
I agree, the badge-creation example in the docs is pretty long. But it is just an example. GitHub workflows tend to become clumsy when you combine more and more actions. That is the downside of being able to combine actions.

@EnricoMi in this case I can understand that too tight coupling is probably not desired, but do you think the action can probably at laest export some ENV variable asutomatically? e.g. what courently computes:
BADGE_COLOR could automatically be exported as TEST_RESULT_COLOR, and
'${{ fromJSON( steps.test-results.outputs.json ).formatted.stats.tests }} tests, ${{ fromJSON( steps.test-results.outputs.json ).formatted.stats.runs }} runs: ${{ fromJSON( steps.test-results.outputs.json ).conclusion }}' could become TEST_RESULT_SUMMARY.
This can the probably even used with more actions but less setup, e.g the setup_java action exports JAVA_<version>_<architecture> env variables what is really useful!

These two values are highly individual, I doubt there is a one-size-fits-all color or summary.
The badge simply serves as a working example of how to use the JSON output.


How to implement [clrLoading]

Hi
I am using Clarity with angular 4.x. I am unable to implement [clrLoading]. I keep having this error
'ERROR Error: Uncaught (in promise): Error: Template parse errors:
Can't bind to 'clrLoading' since it isn't a known property of 'button'. ("'

@bnlambert: Thanks for filing the issue. Could you please re-submit the issue following our issue template which you can access when you open a new issue? We require the version of Clarity you are using and also a Plnkr which replicates the issue. Without this information, it wont be possible for us to debug the issue. Thanks!
Closing the issue based on this.

I am getting the same error.

Here are the stackblitz templates that can be used to reproduce the issue for different Clarity versions:
https://stackblitz.com/@clr-team


"cereal found more than one compatible output serialization function" when using external split load/save functions with explicit version

Ran into cereal assertion error when trying to use external split load/save functions with explicit version on a subclass which has two constructors, a default constructor and a non-default one.
The non-default constructor seems to have somehow caused some interesting issues. When I did any one of these, the code compiles all okay:

comment out the non-default constructor
Make the non-default constructor explicit like this:explicit Derived(const std::string& name)
...

remove the explicit version
use serialize() instead of load/save.

If I missed something from the doc, please point out. (the doc just said "cereal requires access to a default constructor for types it serializes", in my case, there is a public default constructor.)
Here's the code to produce this issue:
#include <string>
#include <sstream>
#include <memory>
#include <cassert>
#include <cereal/cereal.hpp>
#include <cereal/archives/binary.hpp>
#include <cereal/types/polymorphic.hpp>

struct Base
{
    virtual ~Base() = default;
    int id = 0;
};

struct Derived: public Base
{
    Derived() = default;

    Derived(const std::string& name): name(name){}

    virtual ~Derived() = default;
    std::string name;
};

namespace cereal{

template<class Archive>
void save(Archive& ar, const Base& obj, const std::uint32_t version)
{
    ar(obj.id);
}

template<class Archive>
void load(Archive& ar, Base& obj, const std::uint32_t version)
{
    ar(obj.id);
}

template<class Archive>
void save(Archive& ar, const Derived& obj, const std::uint32_t version)
{
    ar(base_class<Base>(&obj), obj.name);
}

template<class Archive>
void load(Archive& ar, Derived& obj, const std::uint32_t version)
{
    ar(base_class<Base>(&obj), obj.name);
}

}

CEREAL_REGISTER_TYPE(Derived)

int main(int argc, char* argv[])
{
    using namespace cereal;
    std::ostringstream osm;
    BinaryOutputArchive oarc(osm);
    std::shared_ptr<Derived> obj = std::make_shared<Derived>();
    obj->id = 10;
    obj->name = "test name";
    oarc(std::dynamic_pointer_cast<Base>(obj));

    std::istringstream ism(osm.str());
    BinaryInputArchive iarc(ism);
    std::shared_ptr<Base> obj2;
    iarc(obj2);

    auto actual = std::dynamic_pointer_cast<Derived>(obj2);
    assert(obj->name == actual->name);
    return 0;
}

Here's the compile error:
cereal/include/cereal/cereal.hpp: In instantiation of ‘ArchiveType& cereal::OutputArchive<ArchiveType, Flags>::processImpl(const T&) [with T = std::basic_string<char>; typename cereal::traits::detail::EnableIfHelper<(cereal::traits::has_invalid_output_versioning<T, ArchiveType>::value || ((! cereal::traits::is_output_serializable<T, ArchiveType>::value) && ((!(Flags & AllowEmptyClassElision)) || ((Flags & AllowEmptyClassElision) && (! std::is_empty<T>::value)))))>::type <anonymous> = (cereal::traits::detail::type)0; ArchiveType = cereal::BinaryOutputArchive; unsigned int Flags = 1u]’:
cereal/include/cereal/cereal.hpp:347:9:   required from ‘void cereal::OutputArchive<ArchiveType, Flags>::process(T&&) [with T = std::basic_string<char>&; ArchiveType = cereal::BinaryOutputArchive; unsigned int Flags = 1u]’
cereal/include/cereal/cereal.hpp:249:9:   required from ‘ArchiveType& cereal::OutputArchive<ArchiveType, Flags>::operator()(Types&& ...) [with Types = {std::basic_string<char, std::char_traits<char>, std::allocator<char> >&}; ArchiveType = cereal::BinaryOutputArchive; unsigned int Flags = 1u]’
cereal/include/cereal/details/polymorphic_impl.hpp:545:59:   required from ‘static void cereal::detail::OutputBindingCreator<Archive, T>::writeMetadata(Archive&) [with Archive = cereal::BinaryOutputArchive; T = Derived]’
cereal/include/cereal/details/polymorphic_impl.hpp:621:29:   required from ‘cereal::detail::OutputBindingCreator<Archive, T>::OutputBindingCreator() [with Archive = cereal::BinaryOutputArchive; T = Derived]::__lambda10’
cereal/include/cereal/details/polymorphic_impl.hpp:618:13:   required from ‘struct cereal::detail::OutputBindingCreator<Archive, T>::OutputBindingCreator() [with Archive = cereal::BinaryOutputArchive; T = Derived]::__lambda10’
cereal/include/cereal/details/polymorphic_impl.hpp:617:32:   [ skipping 4 instantiation contexts, use -ftemplate-backtrace-limit=0 to disable ]
cereal/include/cereal/details/polymorphic_impl.hpp:706:96:   required from ‘static void cereal::detail::polymorphic_serialization_support<Archive, T>::instantiate() [with Archive = cereal::BinaryOutputArchive; T = Derived]’
cereal/include/cereal/details/polymorphic_impl.hpp:686:5:   required from ‘struct cereal::detail::polymorphic_serialization_support<cereal::BinaryOutputArchive, Derived>’
cereal/include/cereal/archives/binary.hpp:163:1:   required by substitution of ‘template<class T, class BindingTag> typename cereal::detail::polymorphic_serialization_support<cereal::BinaryOutputArchive, T>::type cereal::detail::instantiate_polymorphic_binding(T*, cereal::BinaryOutputArchive*, BindingTag, cereal::detail::adl_tag) [with T = Derived; BindingTag = cereal::detail::{anonymous}::polymorphic_binding_tag]’
cereal/include/cereal/details/polymorphic_impl.hpp:724:86:   required from ‘void cereal::detail::bind_to_archives<T, Tag>::bind(std::false_type) const [with T = Derived; Tag = cereal::detail::{anonymous}::polymorphic_binding_tag; std::false_type = std::integral_constant<bool, false>]’
cereal/include/cereal/details/polymorphic_impl.hpp:738:37:   required from ‘const cereal::detail::bind_to_archives<T, Tag>& cereal::detail::bind_to_archives<T, Tag>::bind() const [with T = Derived; Tag = cereal::detail::{anonymous}::polymorphic_binding_tag]’
simplecerealtest.cpp:53:1:   required from here
cereal/include/cereal/cereal.hpp:472:9: error: static assertion failed: cereal found more than one compatible output serialization function for the provided type and archive combination.

 Types must either have a serialize function, load/save pair, or load_minimal/save_minimal pair (you may not mix these).
 Use specialization (see access.hpp) if you need to disambiguate between serialize vs load/save functions.
 Note that serialization functions can be inherited which may lead to the aforementioned ambiguities.
 In addition, you may not mix versioned with non-versioned serialization functions.

         static_assert(traits::detail::count_output_serializers<T, ArchiveType>::value < 2,

cereal version used is release 1.2.2
Compiler used:
$ c++ --version
c++ (Ubuntu 4.8.4-2ubuntu1~14.04) 4.8.4
Copyright (C) 2013 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Fair warning: this response is more of a triage of the problem than an easy solution. The best way around this currently is to use explicit.
cereal is finding two conflicting functions for serializing std::string, one is the default (in cereal/types/string.hpp) which does not take a version parameter, the other is created through the implicit conversion your constructor provides. You can get an identical error by defining both a versioned and non-versioned variant for a class:
struct Hello
{
  Hello() = default;
  Hello( Snarf const & ) {}
  int x;
};

namespace cereal
{
  template <class Archive>
  void load( Archive &, Hello & )
  { }

  template <class Archive>
  void save( Archive &, Hello const & )
  { }

  template <class Archive>
  void load( Archive &, Hello &, const std::uint32_t )
  { }

  template <class Archive>
  void save( Archive &, Hello const &, const std::uint32_t )
  { }
}

when serialized gives:
workspace/cereal/include/cereal/cereal.hpp:472:9: error: static assertion failed: cereal found more than one compatible output serialization function for the provided type and archive combination.

 Types must either have a serialize function, load/save pair, or load_minimal/save_minimal pair (you may not mix these).
 Use specialization (see access.hpp) if you need to disambiguate between serialize vs load/save functions.
 Note that serialization functions can be inherited which may lead to the aforementioned ambiguities.
 In addition, you may not mix versioned with non-versioned serialization functions.

         static_assert(traits::detail::count_output_serializers<T, ArchiveType>::value < 2,
         ^

A more direct but simplified version of your issue is:
struct Snarf
{
  Snarf() = default;
  int x;
};

struct Hello
{
  Hello() = default;
  Hello( Snarf const & ) {}
  int x;
};

namespace cereal
{
  template <class Archive>
  void load( Archive &, Snarf & )
  { }

  template <class Archive>
  void save( Archive &, Snarf const & )
  { }

  template <class Archive>
  void load( Archive &, Hello &, const std::uint32_t )
  { }

  template <class Archive>
  void save( Archive &, Hello const &, const std::uint32_t )
  { }
}

Serializing Snarf will give the same error. The key to reproducing this is that you have a type with a versioned serialization function that can be implicitly instantiated by a non-versioned type.
When cereal is trying to see how it can serialize the non-versioned type, it has to check for and count all possible valid ways, both versioned and non-versioned, it could be saved and loaded. When it is implicitly convertible to another type that is also versioned, it finds two ways - which creates the error.
We check for these types (see details/traits.hpp line 217) by doing something like:
auto has_load_not_versioned(int) -> decltype( load( std::declval<Archive&>(), std::declval<Type&>() );
auto has_load_versioned(int) -> decltype( load( std::declval<Archive&>(), std::declval<Type&>(), 1u );

Both end up being valid which eventually triggers the static assertion.
There is a way around this, which is what we currently do for minimal serialization, where we have a proxy type that is not implicitly convertible, see details/traits.hpp around line 602. This however creates a lot of annoyances and has some downsides (see #132).
To end this rambling, for now just use explicit and know that resolving this in a clean way is a difficult problem because C++ lacks reflection.


Get user by access token in the middleware

In the "accesstoken" middleware, the user is determined by an access token
which is kept in a store via UserService.UserByAccessToken.
But the store has only one access token per user and provider.
So when user will login from different devices or browsers
access tokens will be different and we will not be able to correctly identify the user.
In the middleware we should query a user by using the Hydra-API, but not via the UserService.
Or another way is to store the tokens for all user sessions
During logout user, we should revoke the access token which is associated with this user session
and not to affect others tokens of this user.
Related #10 .

Clarification: obtain "subject" by token via Hydra API, then user by "subject" == userID (login) from user service.


where does universal get its path from

In your UniversalComponent, your component looks like this:
import universal from 'react-universal-component';

export default universal(({ name }) => import(`./${name}`));

How is the path ./${name}/ resolving?  Is the cwd() not the root of the site?
I am confused as to how this is working.

Despite the fact, that this is closed, for future reference:
Universal Component's root is the shared/components directory. This is signified by the ./ which resolves to the path of the file.


Cybro Repository not found in HACS

I am unable to find the Cybro repository in HACS, is it still up? I am using Cybro HiQ by Robotina in my home and would love to try and integrate it with HA.

I am unable to find the Cybro repository in HACS, is it still up? I am using Cybro HiQ by Robotina in my home and would love to try and integrate it with HA.

Hi,
currently its not yet in the HACS store.
Currently its slowly growing and at the moment there are only LC-10-IQ, power meter and temperature sensors supported.
If you want to try it out, you can manually add the repository on the right top corner (menu -> add user defined repository) and enter https://github.com/killer0071234/ha-cybro as repository with category integration.
But for communicating with the Cybro HIQ plc you need the cybro scgi server running. You can run the scgi server as docker container, see: https://hub.docker.com/r/killer007/cybroscgiserver
BR Daniel

Hi Daniel, thank you for your reply and your continued effort. I will follow the development since right now i could only connect the LC since I don’t have a power meter and the temperature sensors I am using are Netatmo and already have an existing HA integration. It would be amazing if you would do a blinds controller integration, I have interesting use cases for that. Thank you and have a great start to the week!

Hi,
the first release with blind support is online!
but you need to add the new repo also by hand to test it.
https://github.com/killer0071234/ha-hiq
this repository is soon obsolete

Hi,
The new  HIQ integration is now directly available in the HACS store under integrations!
I will close this issue.
BR Daniel


Discover typo in pagination.css

Describe the bug
I found typo in pagination.css
line number is 284
.paginationjs .paginationjs-pages li.paginationjs-next {
  border-right: 1px solid #aaa/9;
}

To Reproduce
Provide a re-produciable demo: http://codepen.io/superRaytin/pen/RRoZBz
.paginationjs .paginationjs-pages li.paginationjs-next {
  border-right: 1px solid #aaa;
}

Expected behavior
A clear and concise description of what you expected to happen.
Versions:

Pagination: [e.g. 2.1.5]
jQuery [e.g. 1.8.3]
Browser [e.g. Chrome]

Additional context
Add any other context about the problem here.

It is not a typo, \9 is a "CSS hack" specific to Internet Explorer 7, 8, & 9.


Grid items rearranging

This issue tracks item rearranging in grid layouts

[x] Find a good way to make items Identifiable
[x] Update inits without breaking changes
[ ] Add examples to supported targets

Related to #65

Now works with #79. Rearranging underlying data will rearrange views based on IDs.


Fb-1816 : CharIndex str func

 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.


variables.Add confusion about aliases

This was discussed on discord (reported by @acmorrow), and there's a fairly simple possible fix - but wanted to capture it in an issue first as there's some room for discussion.   This was mentioned in the context of a pre-4.1 release but exists in the just released 4.1 as well; python versions irrelevant.
If you pass a tuple of strings to Add as the key value, instead of a single string (see the example later in this description to see what it looks like), the tuple is unpacked and the first string used as the key name, the additional ones as aliases.  This isn't actually promised in the docs in words, but a comment in the code says so, and the unit tests definitely test for this alias behavior.
The problem is that for Add, this doesn't work (this is the "issue").  Add and AddVariables share a call to _do_add, which implements actual variable setup, but Add has additional logic that if a tuple alone is passed, it's the result of one of the *Variable functions, and it unpacks this and calls _do_add in a way that doesn't allow the key alias tuple to be recognized.
So three questions for discussion before proposing a PR:

Does Add need to support this behavior of accepting a packed tuple of arguments like AddVariables does instead of passing the positional/keyword arguments that are documented for Add?  The docs don't promise this behavior explictly (although it can be inferred by looking at examples, esp. in the user guide), and it could be disallowed by saying "if you want to pass the result of calling ListVariable (for example). then use AddVariables instead. The overloading of the key parameter so it's both a positional variable and packed set of arguments is, well, awkward. A different approach might be to provide an "alternate constructor" (like providing both Add and Add_as_tuple or some such) - but it's quite possible there is dependence on the existing dual-nature behavior "in the wild" and a change would break things, so that would come with implications.
Should we indeed promote to documented/supported the behavior of passing the key tuple to set up aliases? As noted, the presence of unit tests does suggest it was intended to be considered supported behavior, but one could argue "we never said so, so don't count on it".
At the moment in _do_add, the behavior is that if you pass a key tuple, it behaves as described above, but if you pass a key string, the value is used as the key and the alias - that is, the key is part of the alias list in only one path through the code. The unit tests in fact expect this so it appears semi-intentional, but seems logically odd - I'd expect either the key to always be in the alias list or never. The actual code:

        if SCons.Util.is_List(key) or SCons.Util.is_Tuple(key):
            option.key = key[0]
            option.aliases = key[1:]
        else:
            option.key = key
            option.aliases = [key]

Problem reproducer:
In VariablesTests.py in VariablesTestCase add a third setup that uses a tuple for the key:
        opts.Add(('QUESTION', 'FAQ', 'Q'),
                 help='THE frequently asked question',
                 default="37",
                 validator=check)

and then test for the results:
        # a key which is a tuple should set up aliases
        o = opts.options[2]
        assert o.key == 'QUESTION', o.key
        assert o.help == 'THE frequently asked question', o.help
        assert o.default == "37", o.default
        assert len(o.aliases) == 2
        assert 'FAQ' in o.aliases
        assert 'QUESTION' not in o.aliases

Well, first, I think the Variable Aliases thing is actually really handy. Maybe not needed that often, but I found it crucial when doing some renaming, so that even if a user specified something on the command line via the old name, the setting got re-routed to the new name. I was actually more or less astounded that such a handy feature was right there when I needed it. So, I'd be sad to see it simply dropped.
I'd also be a little disappointed to see this whole thing resolved by saying that I need to use the AddVariables function. I sort of dislike that it would force me to declare all these variables within one function call. It would mean, for the MongoDB build, a function call with close to 100 arguments.
On the other hand, since this variable aliases thing seems to be wholly undocumented, maybe the answer is just to introduce a new keyword argument for Add. I'll muddle through with _do_add until the real Add takes an aliases= argument which takes a list of aliases.

So that's an interesting idea - don't propagate the "use a tuple to define aliases" behavior and introduce a new kwarg.  That would make for a cleaner solution, methinks.  Further thoughts on that?

So if we added an aliases kw arg, then it won't break any existing logic right?
That sounds like the way to go.

I don't find that an objectionable approach... I didn't originally raise the issue, but @acmorrow seems to be okay with it too.


Unable to move the bbox

Hello. Hello. One problem I found is that in the vue2 version of coco-annotator, the bbox can be moved with SelectTool, but the Coco-annotator-ng can't seem to move the bbox

Hi, JinDing123,
Your right, it works in my dev directory,
but doesn't work in my release directory.
I'll try to fix this.

Should be fixed with this commit https://github.com/SixK/coco-annotator-ng/commit/aaf83150f9fc791255a61949263103efb621455e
I had a wrong test on categoryId when checking if object is a BBox.
Thank's for the report


homeassistant-pyozw==0.1.3 fails to install on FreeBSD

Home Assistant release with the issue:
0.91
Last working Home Assistant release (if known):
0.90.2
Operating environment (Hass.io/Docker/Windows/etc.):
FreeBSD
Component/platform:
zwave
Description of problem:
The homeassistant_pyozw fails to install which is a required dependency of home assistant 0.91 for using the zwave component.
   b"/tmp/pip-install-owewm927/homeassistant-pyozw/openzwave-embed/open-zwave-hass/cpp/src/command_classes/SensorMultilevel.cpp:502:55: error: adding 'uint8' (aka 'unsigned char') to a string does not append to the string [-Werror,-Wstring-plus-int]\n"
The problem stems from a compilation bug introduced into the homeassistant open-zwave fork here.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
zwave:

I have created a PR to fix.
https://github.com/home-assistant/open-zwave/pull/4

Same of this? https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=206526

Same of this? https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=206526

No, this is different. The bug is specific to the home assistant fork of open-zwave.

Appears to be effecting macosx too:
https://community.home-assistant.io/t/update-ha-to-0-91-zwave-fail/109462/4

Facing similar issue on Raspbian too, with hass version 0.91.2:
[homeassistant.util.package] Unable to install package homeassistant-pyozw==0.1.3: Failed to restore /test/env/lib/python3.5/site-packages/python_openzwave/ozw_config/__pycache__/
Command "/test/env/bin/python3 -u -c "import setuptools, tokenize;__file__='/tmp/pip-install-rdc7590c/homeassistant-pyozw/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-record-zvm5a0u4/install-record.txt --single-version-externally-managed --compile --install-headers /home/home-auto/env/include/site/python3.5/homeassistant-pyozw" failed with error code 1 in /tmp/pip-install-rdc7590c/homeassistant-pyozw/

This but is blocking a lot of people thinking z wave on different platforms, can we have the PR included for 0.91.1?

@ryanwinter Just randomly noticed your comment on this flying by in Discord's GitHub channel. Made people aware of this, fix should be coming soon. Assigning @pvizeli as he maintains the fork of OZW that this broke in. Also, cc @clementhk, your PR https://github.com/home-assistant/open-zwave/pull/2 seems to have been either improperly applied to the fork or was later broken for *BSD platforms (specific reports of breakage on FreeBSD and macOS which is BSD at heart).
Does anyone know what the consequences would be of just switching back to mainline OZW? Have there been any associated changes in this (home-assistant/home-assistant) repo that would prevent that? I see a couple of cherry picked items onto the fork and a patch or two, so I'm unsure.

The main line is missing a couple of fixes such as the cover, rolling back would break a bunch of things.
My PR just fixes the compilation problem. It just needs to be merged and included in the next version of HA.

Is merged with 0.1.4 - Thanks for PR @ryanwinter

What does this mean for us? Do we have to upgrade to 0.1.4 or whait for a new HA release?

Will be in with 0.92.x

What does this mean for us? Do we have to upgrade to 0.1.4 or whait for a new HA release?

I downgraded to 0.90.2. Will wait for 0.92.x with the fix for this.
Using virtualenv install, used command pip3 install --upgrade homeassistant==0.90.2.

homeassistant 0.92.0b2 listed homeassistant-pyozw 0.1.4 in requirement. But it still fails installation on raspbian. Is that expected?

A temporary work around for Home Assistant, you can manually edit zwave/__init__.py to use homeassistant-pyozw==0.1.4  as shown here..

I'm wondering, what does it mean the issue was pulled from the 0.91.2 milestone? In what version is this fix planned to be released now?

@rschuiling I believe there is new release from ~2 days ago.

@rschuiling This has been fixed in the release of HA 92.0


optimize: Optimize seata-server log pattern.

Ⅰ. Describe what this PR did
optimize: Optimize seata-server log pattern.
Ⅱ. Does this pull request fix one issue?

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

Codecov Report

Merging #2788 into develop will decrease coverage by 0.11%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #2788      +/-   ##
=============================================
- Coverage      50.47%   50.35%   -0.12%
- Complexity      2919     2921       +2
=============================================
  Files            572      573       +1
  Lines          18616    18594      -22
  Branches        2245     2201      -44
=============================================
- Hits            9396     9363      -33
+ Misses          8288     8277      -11
- Partials         932      954      +22

Impacted Files
Coverage Δ
Complexity Δ

...ack/ExtendedWhitespaceThrowableProxyConverter.java
100.00% <100.00%> (ø)
2.00 <2.00> (?)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (ø%)

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterTMRequestConvertor.java
94.73% <0.00%> (-5.27%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterRMRequestConvertor.java
90.47% <0.00%> (-4.77%)
3.00% <0.00%> (ø%)

... and 23 more

@wangliang1986 please explain the display comparison before and after modification.

The comparison chart is in the description, please check it.


Fix current resource path /api-docs according with Swagger 2.0 conventions (/swagger.json)

Fix trails routes for:

Static content (Swagger-UI). What should be the path here? We should avoid any routes conflicts.
cowboy_swagger_handler. The path here should be /swagger.json directly.

NOTE: The idea is let that path totally configurable, as an environment variable of the app, and default value would be /.

…aaaaand closed!


fix: match jest json output by making locations zero-based in json report

Description
This change improves compatibility of the JSON reporter with Jest's --json output by making locations zero-based.
Repro
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
[ ] Ideally, include a test that fails without this PR but passes with it.
[ ] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

[ ] Run the tests with pnpm test:ci.

Documentation

[ ] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

[ ] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

Ignore — was based on a mistaken read of Jest's output.


CASMINST-6035: Fix typo on Vault backup/restore page

Description

Fixes a typo in a link in the table of contents on one page.
Checklist

[X] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[X] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[X] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.4 release/1.3 release/1.2 release/1.0


phpunit 6

drop php 5.5
added phpunit 6 support
travis update
i will do the ::class job after that PR.

For a maintenance release, we cannot drop 5.5, but I can modify the .travis.yml during merge to ensure that it remains tested.
For the next minor release, we can drop 5.5.


Pass in a train dataset and a validate dataset (precompute cv split)?

Is it currently possible to pass in a train dataset and a validate dataset for a skorch NeuralNet?
It looks like I can pass a function that will do the split for me, but I already HAVE two datasets that I want to use, and combining them and then splitting them seems unnecessarily complicated...

Thanks!  train_split seems like it would work well for me.

With these approaches, you have to be careful when pickling your model, since it will include a reference to the data, which you typically don't want.

I'm not sure I understand this, you mean if train_split has a reference to the data, then it will be saved along with the rest of the net?

The most robust solution is probably to really merge the two datasets and then later split them. Depending on your data, you could use cv=GroupKFold and pass a groups argument to fit that indicates whether your data belongs to train or validation.

This seems like it might be on the docket eventually.
Thanks!

I'm not sure I understand this, you mean if train_split has a reference to the data, then it will be saved along with the rest of the net?

I'm not completely certain but that could indeed happen. NeuralNet will has a reference to train_split, and if you define train_split as a function, your data will be part of the enclosing scope -- whether this would result in the data being pickled I'm not completely sure. If this happens, you can define your own train split class that is initialized with your data and implements a custom __getstate__ method where you exclude your data from the object state.
Alternatively, you can use net.save_params, which will only persist the net parameters.

Thanks!

This is addressed in #282.


Improve how ABR works in Plyr

Bug Report
Customer feedbacks lead us to identify 3 issues in the way ABR works in Plyr:

On all browsers, Plyr starts by downloading a 1080p fragment. It should start by a 480p fragment and then adapt up or down depending on the quality of connection. For slow connections, downloading this first fragment of 1080p makes everything very slow,
When ABR is active, the button to switch bitrate is not available anymore. We hid it because it was ineffective in Plyr when ABR is active but this situation creates frustration because people's expectations are based on Youtube. We need to find a way to allow forcing a specific bitrate, even when ABR is available,
On IE11, Plyr starts by downloading the complete video file in 144p without apparent reason.

On all browsers, Plyr starts by downloading a 1080p fragment. It should start by a 480p fragment and then adapt up or down depending on the quality of connection. For slow connections, downloading this first fragment of 1080p makes everything very slow,

This is done in #591

When ABR is active, the button to switch bitrate is not available anymore. We hid it because it was ineffective in Plyr when ABR is active but this situation creates frustration because people's expectations are based on Youtube. We need to find a way to allow forcing a specific bitrate, even when ABR is available,

Activating quality selector is not easy at all. Plyr player and dash does not interact each other (or I didn't found how) and once quality selector enabled a lot of errors are raised if the quality is changed. I have to investigate more on this point

On IE11, Plyr starts by downloading the complete video file in 144p without apparent reason.

¯_(ツ)_/¯ IE11 :heart:


Battery status

Is there one or another way to have the battery status published of the Zwave node?

Hi @pcassimans, that should be possible. How would you prefer to see the option in the UI and topics?

Great! :-)
I don't know what the best option would be. I don't think it's wise to have an update very often as it would consume battery power to get it :-). Maybe a set command could be send to the device on which the device would answer with a status the next time it gets alive. But i don't know if that is possible.
----- Original Message -----
From: "Robin Eggenkamp"<EMAIL_ADDRESS>To: "Edubits/Zway-MQTT"<EMAIL_ADDRESS>Cc: "pcassimans"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>Sent: Wednesday, June 22, 2016 8:39:31 AM
Subject: Re: [Edubits/Zway-MQTT] Battery status (#1)
Hi @pcassimans , that should be possible. How would you prefer to see the option in the UI and topics?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub , or mute the thread .

Any news?


ResNet baseline vs. source only

Hi, thank you for sharing the wonderful work. I have a question regarding the main paper:
In table 3, 5, 7, and 8, what's the difference between "ResNet50/101" (the first line in all tables) and "Source model only"? I'm guessing "ResNet" means using the original ResNet model (replace the last fc, no "fc+bn bottleneck" proposed in the paper) trained on source and evaluate on target without any adaptation. Is that correct?
Also in table 2, what does "Source only" in the first line (with Hoffman's method) mean?
Thank you!

Hi, thank you for sharing the wonderful work. I have a question regarding the main paper:
In table 3, 5, 7, and 8, what's the difference between "ResNet50/101" (the first line in all tables) and "Source model only"? I'm guessing "ResNet" means using the original ResNet model (replace the last fc, no "fc+bn bottleneck" proposed in the paper) trained on source and evaluate on target without any adaptation. Is that correct?
Also in table 2, what does "Source only" in the first line (with Hoffman's method) mean?
Thank you!

Hi, both 'ResNet' and 'Source model only' denote the original model fine-tuned in the source domain and evaluated in the target domain without adaptation. For fair comparisons, we report our 'source model only' since the details of the model (training/ architecture) are not different.
Besides, for digits, 'source only' denote the original model learned in the source domain from scratch and evaluated in the target domain without adaptation.
Hope it addresses your concerns.


Update pysam files for CSP

Updating CSP PySAM files for a custom PySAM release using the develop branch of SSC

@qualand I'm seeing some deleted PySAM modules: TroughPhysicalCspSolver and PhysicalTroughProcessHeat
Can you confirm these changes (and the others in this PR) are correct?

@taylorbrown75 can you confirm these changes?

@qualand I'm seeing some deleted PySAM modules: TroughPhysicalCspSolver and PhysicalTroughProcessHeat
Can you confirm these changes (and the others in this PR) are correct?

Both modules are retired, they were removed intentionally.


UNC path is missing hostname: /

The base image requires auth. Trying again for anapsix/alpine-java...
Retrieving registry credentials for registry.hub.docker.com...
Got more than one input Future failure. Logging failures after the first
java.nio.file.InvalidPathException: UNC path is missing hostname: \/
	at sun.nio.fs.WindowsPathParser.parse(WindowsPathParser.java:113)
	at sun.nio.fs.WindowsPathParser.parse(WindowsPathParser.java:77)
	at sun.nio.fs.WindowsPath.parse(WindowsPath.java:94)
	at sun.nio.fs.WindowsFileSystem.getPath(WindowsFileSystem.java:255)
	at java.nio.file.Paths.get(Paths.java:84)
	at com.google.cloud.tools.jib.image.ReproducibleLayerBuilder$UniqueTarArchiveEntries.add(ReproducibleLayerBuilder.java:69)
	at com.google.cloud.tools.jib.image.ReproducibleLayerBuilder$UniqueTarArchiveEntries.add(ReproducibleLayerBuilder.java:71)
	at com.google.cloud.tools.jib.image.ReproducibleLayerBuilder$UniqueTarArchiveEntries.add(ReproducibleLayerBuilder.java:71)
	at com.google.cloud.tools.jib.image.ReproducibleLayerBuilder$UniqueTarArchiveEntries.add(ReproducibleLayerBuilder.java:71)
	at com.google.cloud.tools.jib.image.ReproducibleLayerBuilder$UniqueTarArchiveEntries.access$100(ReproducibleLayerBuilder.java:44)
	at com.google.cloud.tools.jib.image.ReproducibleLayerBuilder.build(ReproducibleLayerBuilder.java:111)
	at com.google.cloud.tools.jib.builder.steps.BuildAndCacheApplicationLayerStep.call(BuildAndCacheApplicationLayerStep.java:112)
	at com.google.cloud.tools.jib.builder.steps.BuildAndCacheApplicationLayerStep.call(BuildAndCacheApplicationLayerStep.java:37)
	at com.google.common.util.concurrent.TrustedListenableFutureTask$TrustedFutureInterruptibleTask.runInterruptibly(TrustedListenableFutureTask.java:127)
	at com.google.common.util.concurrent.InterruptibleTask.run(InterruptibleTask.java:57)
	at com.google.common.util.concurrent.TrustedListenableFutureTask.run(TrustedListenableFutureTask.java:80)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

Environment
Windows, Gradle 4.10, Java 8
it seems to be a windows-related issue. Not entirely sure what triggers it. The same build worked on very similar machine. And till this day it worked well on both. I recently upgraded to jib 0.10.0, so that could be another possiblity.
One is clearly visible is that UniqueTarArchiveEntries has issues differentiating "/" and "\" paths. When debugging through the stack before the exception is thrown, I see:
recursive calls to  ReproducibleLayerBuilder.add :
tarArchiveEntry.name=\/
tarArchiveEntry.name=\app/
tarArchiveEntry.name=\app\classes/
tarArchiveEntry.name=/app/classes/ch/

initiall one call to ReproducibleLayerBuilder.build:
layerEntry.sourceFile=C:\projects\demo\demo-batch\build\classes\java\main\ch
layerEntry.extractionPath=/app/classes/ch

the tarArchiveEntry.name=\/ then obviously leads to a crash within the Java NIO API.
By hardening UniqueTarArchiveEntries  the issue should get resolved I guess.

Seems obvious that this is a Windows issue. I agree UniqueTarArchiveEntries needs some hardening.

I was investigating it further to find out why it works in some configurations and not others. I ended up seeing that different versions of 'org.apache.commons:commons-compress' can cause the issue. Seems that they already fixed the issue above. This in turn makes the fix here uncessary. But jib should shade all its dependencies along the lines of https://github.com/project-aries/docker-java-shaded.

Apparently, this is a bug in old org.apache.commons:commons-compress, so closing.
About embedding dependencies (by shading, for example) like commons-compress: assessing the situation as of now, I think this doesn't seem really necessary or worth the effort.


Field Component

no background color in old page

There should only be 5 px margin after icon, but there is 5 more pixel after icon container here

why this popup twice

cant put punctuation marks at all, like  comma

autoresize doesnt work too

autoresize doesnt work too

Autoresize is a TODO, not implemented in this PR

conflict dude


issue #113: added auto-formatter as a github action

This is the dotnet default formatting.

ok. this clearly did not work. i found a dotnet-format action in the github marketplace.
https://github.com/marketplace/actions/dotnet-format
We can check that one out next time.


deps: Update ashpd to 0.9

The goal is mostly to move on from wayland-protocols 0.31.
See: https://github.com/bilelmoussaoui/ashpd/releases/tag/0.9.0

Sounds good, thanks!


Make DELETE feature for Review, Note and Create api endpoint class

Refactor api endpoint by create the class for each endpoint.
Create delete feature for CourseReview and Note
Create new test for delete feature
remove mode_grade and add total_grade in StatQuery

Damn it resolve conflict make my code boom boom


MultipleFileUploadEditor - deleting files

Hi,
Can you tell me why some files are able to be deleted and some are not when using the MultipleFileUploadEditor
You can see the first two items have the red cross, but the last two do not.

It's only a css issue: the files that "cannot" be deleted have a filename too long that "hide" the red cross.
I've solved with the following lines of css:
.file-item .filename {
    position: relative;
}

.s-MultipleImageUploadEditor li .delete {
    position: absolute;
    top: 0px;
}

don't know if it's the best approach, but it works!
Bye

Thanks @marcobisio

You're welcome @volkanceylan

Thanks, that sorted out the issue! perhaps a future enhancement would be to truncate the displayed file name as it does look a little messy, but otherwise all good.


divide CUI into std and advanced Command Line

https://github.com/avelino/awesome-go/issues/282
Notes: It was not immediately clear whether I should move the command line email tool to Standard Command Line or leave as a utility (https://github.com/sanbornm/mp). I think the best option is to leave it as a utility because it isn't a tool for building command line apps, it is just a command line app itself.

Dud PR, accidentally picked wrong target


feat: Initial commit for UI #108

The initial commit for UI, as discussed at https://github.com/kubeflow/model-registry/issues/108.
Description
In this commit:

basic Dockerfile
basic Makefile
Scaffold of App and first sample endpoint (http://localhost:4000/api/v1/healthcheck/)
REST API basic infrastructure and error handling

How Has This Been Tested?
> make run
> curl -i localhost:4000/api/v1/healthcheck/
HTTP/1.1 200 OK
Access-Control-Allow-Origin: *
Content-Type: application/json
Date: Wed, 29 May 2024 14:41:13 GMT
Content-Length: 70

{
	"status": "available",
	"system_info": {
		"version": "1.0.0"
	}
}

> make docker-build
>  docker run -p 4000:4000 -e PORT=4000 model-registry-bff
> curl -i localhost:4000/api/v1/healthcheck/
HTTP/1.1 200 OK
Access-Control-Allow-Origin: *
Content-Type: application/json
Date: Wed, 29 May 2024 14:41:13 GMT
Content-Length: 70

{
	"status": "available",
	"system_info": {
		"version": "1.0.0"
	}
}

Merge criteria:

[X] The commits and have meaningful messages; the author will squash them after approval or will ask to merge with squash.
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has manually tested the changes and verified that the changes work

@tarilabs thank you so much for the review! I'll send it on a FUP PR as soon as possible.

@tarilabs thank you so much for the review! I'll send it on a FUP PR as soon as possible.

got it, works for me
/lgtm
/approve


[Docs] Improve metrics/grafana docs to include info on how to utilize the new Ray dashboard features.

Search before asking

[X] I searched the issues and found no similar issues.

KubeRay Component
Others
What happened + What you expected to happen
The current grafana / prometheus docs works well for folks using grafana UI directly. However, in ray 2.2 and above there are metrics features embedded into the Ray Dashboard UI and those features should work with kube-ray's grafana deployment.
Reproduction script
None. Just launch a cluster and check the ray dashboar.
Anything else
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Related Slack discussion: https://ray-distributed.slack.com/archives/CP950VC76/p1689179832076669
Also willing to submit a PR once I have time to figure it out myself :)


<EMAIL_ADDRESS>dependency is deprecated

What are you trying to do?
I'm getting warning "npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated ". This warning is from the dependencies "stylus" of package "@angular-devkit/build-angular".
Will it be fix in next release?
What troubleshooting steps have you tried?
Warning comes in production build or during "npm install"
npm i
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
Reproduction
Steps to reproduce:
npm i
Environment
Angular:
CDK/Material:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

According to source-map-resolve, we should learn how to stop using it. I checked quickly and I see that it is only used once in css in the source-map-support.js file. But I am not sure on what it does, so I can't really propose an alternative and submit it in a PR. I hope somebody will soon.

I also came across this issue, but as a result of an upgrade to webpack 5. Webpack 5 no longer polyfills the url module, so I'm getting an error from this module (css), which is using source-map-resolve, which uses the url module.

@pcart-grandjean Definitely agree there should be a more current approach taken given the points he makes.

I worked on removing the source-map support. This will solve the issue. Trying to reach out to the rework members before going into the fork route (and discussing with my company for the PR route).
ref: https://github.com/reworkcss/css/issues/164

https://github.com/reworkcss/css/blob/b7f3cb62dcb12f569189c38cbdb756553bf0415b/lib/stringify/source-map-support.js seems to be only file that uses source-map-resolve. Is there any other place, it is used in current master branch?

From what I saw, it's not used anywhere else. For our purpose, we don't need source-map files so I directly strip those. I will post my changes as soon as I got the green light from my company. Adding back the source map code shouldn't be too hard then but I will personally let other people do it ;-)

@holblin  Any news on this?!

Yes, we have fork the repository and it's also published into npmjs.
It's currently in rc release and I have some vacation planned so I don't think there will be a lot of updates during the next 2 weeks but I might take a look just after or at least release it in version 4.0.0
https://github.com/adobe/css-tools

Thanks for that, sounds great, happy holidays :-)

a year later, what's the status of this?

a year later, what's the status of this?

Apparently simply use @adobe/css-tools instead.


Using scrollReveal elements on a Bootstrap carousel

Great plugin! I love it and I am trying to combine it with a bootstrap carousel slider.
The bootstrap carousel provides a hook so I can check when a sliding action is occuring
$('#bscarousel').on('slide.bs.carousel', function () {
    var slidno = carouselData.getActiveIndex();
}

I would like to have the scrollReveal actions to start after each slide is loaded. Is there a way of activating the scrollReveal actions after a bootstrap slide transition?
Thanks

I'm trying to use your great PlugIn with Bootstrap Carousel too.
Your link to "Advanced Usage" above is broken, I think: there's no paragraph named Advanced Usage or anything about the init() method.
Could you please provide a short example on how to init scrollReveal with Bootstrap Carousel?

Sure @ksmueller, you’ve resurrected an old issue (thanks for doing a search of past issues) and the response was out of date — let’s take a look at how we can solve this today.

First, to make sure our element isn’t visible before we initialize ScrollReveal, make sure to add visibility: hidden to your element(s):

.myClass { visibility: hidden }

Next, we’ll want to use version 3.0 for access to a powerful new JavaScript API. This will make it easy to setup multiple animations, and trigger them when we want. Grab 3.0 here: https://github.com/jlmakes/scrollReveal.js/tree/v3.0.0-RC

Now let’s instantiate it, and setup our animations:

// sample config

var config = {
  move: 0px;
  scale: { power: '10%', direction: 'up' },
  over: '0.8s',
  viewport: document.getElementById('bscarousel')
}

// create a new instance attached to window so we
// can access it from anywhere within our code

window.sr = new ScrollReveal( config );

I’m not familiar with the events, but for Bootstrap Carousel (based on the original posters code sample), it now might look something like this:

$('#bscarousel').on('slide.bs.carousel', function () {
    sr.reveal('.myClass'}).init();
}

There’s still probably a little more work to get it working the way you want, but hopefully this points you in the right direction. If you post back with your progress, maybe we can piece together a complete working example for posterity.

Hi, thanks for the time you took to improve ScrollReveal.js and create v3!
However, I still have issues with Bootstrap Carousel.
I made a codepen to fiddle with:
http://codepen.io/ksmueller/pen/BjBxvZ
I included bootstrap, jQuery and ScrollReveal-v3.
The BS Carousel is working nicely, the captions have visibility:hidden.
The event is correct, the alert fires, when you un-comment it.
In Firefox the caption gets visible once for each slide (it doesn't move or scale, though, it just fades in), but the captions stay hidden in Chrome
Any help is greatly appreciated :-)

@ksmueller
I’m not sure what to say about Chrome, but the animations only working once in FireFox is what I expected from my suggestion. If you want them to reveal every time, it might be possible… but truthfully, you’re taking a scroll animation library and using it to create CSS animations inside bootstrap carousel.
It would probably be easier to create the CSS animation classes yourself and add/remove them inside the event callbacks, instead of trying to use ScrollReveal for this.


new contact information not sync after contact being add via friendship event

Provide Your Network Information

Where is the location of your server? (i.e. City, or In/Out China)
U.S.
Which cloud platform(AliYun/Qcloud/DigitalOcean/etc) are you using?
own machine

Run npm run doctor or wechaty run doctor(for docker user), paste output here
docker run in container
Expected behavior
bot.on('friendship', async friendship =>
{
    let logMsg
    try
    {
        logMsg = 'received `friend` event from ' + friendship.contact().name()
        console.log(logMsg)
        switch (friendship.type())
        {
            case Friendship.Type.Receive:
                logMsg = 'accepted automatically'
                await delay(10000)
                await friendship.accept()
                break
            case Friendship.Type.Confirm:
                logMsg = 'friend ship confirmed with ' + friendship.contact().name()
                await delay(10000)
                await friendship.contact().sync()
                let fs = await friendship.contact().friend()
                console.log(fs) //////////////// false, but it should be true
                break
        }
    } catch (e)
    {
      logMsg = e.message
    }
    console.log(logMsg)
})

Actual behavior
console.log(fs) / return false, but it should be true
Steps to reproduce the behavior (and fixes, if any)
After friend being add wechaty should refrash contact cache and change the friendship between with bot and new_contact (means new contact cache should fetch again because all information change after add new friend).
this may be same issue as #1517
Full Output Logs
None available for this time
$ docker create -t -i --name=$docker_name \
--mount type=bind,source="${dir}",target=/bot \
-e WECHATY_LOG=silly \
-e TZ="America/Los_Angeles" \
zixia/wechaty:latest mybot.ts
$ docker start -a $docker_name

Show Logs

Paste the full output logs here with WECHATY_LOG=silly set
# PASTE FULL LOG OUTPUT AT HERE:

Good catch!
And it would be terrific if you can fix this problem by sending a PR, would you like to do that?

I would try:)

@kis87988 Did you get any progress about the PR to fix this issue?
Would love to hear back from you!

@zixia
Not really, no luck.
I tried to do sync() in the accept() function, but that doesn't work 😢

Do you think we have to set the contact.payload.friend = true directly for a workaround?


common: bump wrangler v3

If we already use the wrangler v3 controller, the new added webhook sometimes might encounter the following build error:
# github.com/harvester/webhook/pkg/server
vendor/github.com/harvester/webhook/pkg/server/server.go:147:18: cannot use clients.Core.Secret() (value of type "github.com/rancher/wrangler/pkg/generated/controllers/core/v1".SecretController) as "github.com/rancher/wrangler/v3/pkg/generated/controllers/core/v1".SecretController value in struct literal: "github.com/rancher/wrangler/pkg/generated/controllers/core/v1".SecretController does not implement "github.com/rancher/wrangler/v3/pkg/generated/controllers/core/v1".SecretController (wrong type for method AddGenericHandler)
                have AddGenericHandler(context.Context, string, "github.com/rancher/wrangler/pkg/generic".Handler)
                want AddGenericHandler(context.Context, string, "github.com/rancher/wrangler/v3/pkg/generic".Handler)
FATA[0015] exit status 1
make: *** [Makefile:11: default] Error 1

That is because the github.com/rancher/dynamiclistener did not use the corresponding version.
The harvester/webhook v0.1.4 should use the rancher/dynamiclistener v0.3.x
If we already bump wrangler to v3, the dynamiclistener will also bump to v0.6.0.
If that is, using the harvester/webhook v0.1.4 will get the above error.

cc @mingshuoqiu, the network-controller also uses webhook


Uptate the links to the repo.

Updating the links after the github repo has been renamed.
Relates-To: OLPEDGE-1889

Codecov Report

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

@@          Coverage Diff           @@
##           master    #304   +/-   ##
======================================
  Coverage    88.3%   88.3%
======================================
  Files          63      63
  Lines        1918    1918
  Branches      253     253
======================================
  Hits         1694    1694
  Misses        132     132
  Partials       92      92

Continue to review full report at Codecov.

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


fix: resolve html template error

Hi @adamstankiewicz, thanks for your work here. The issue you are experiencing is likely due to you running npm install instead of yarn install. See related https://github.com/react-microfrontends/root-config/issues/13


Gaeilge

In response to YAPC NA 2015 talk.
Three sample personal names from Ireland, each with something you might find interesting to test with.  (They all have accented characters, but that's not particularly interesting.) Pick and choose as you like.
"Seán 'ac Dhonncha": apostrophe at the beginning of a word rather than in the middle
"Máire Mhac an tSaoi": camel-case in name
"Johnny Mháirtín Learaí Mac Donnchadha": name with several parts.

Thank you, Mike!


Remove containers within containers before exiting outside container.

Code looks fine. Add some comments (and link to the right section of the DIND readme) then you're good to merge.


Ignore self signed certificates

I'm having this problem visiting a self signed certificate (https in dev environment)
DEBUG=zombie node zombie.js
  zombie Opened window https://www.example.com/ +0ms
  zombie Resource error +77ms Error: self signed certificate
    at Error (native)
    at TLSSocket.<anonymous> (_tls_wrap.js:1016:38)
    at emitNone (events.js:67:13)
    at TLSSocket.emit (events.js:166:7)
    at TLSSocket._finishInit (_tls_wrap.js:585:8)
  zombie TypeError: self signed certificate
    at /Users/alexserver/dev/play/scripts/node_modules/zombie/lib/pipeline.js:89:15
    at tryCatcher (/Users/alexserver/dev/play/scripts/node_modules/zombie/node_modules/bluebird/js/release/util.js:11:23)
    at Promise._settlePromiseFromHandler (/Users/alexserver/dev/play/scripts/node_modules/zombie/node_modules/bluebird/js/release/promise.js:491:31)
    at Promise._settlePromise (/Users/alexserver/dev/play/scripts/node_modules/zombie/node_modules/bluebird/js/release/promise.js:548:18)
    at Promise._settlePromise0 (/Users/alexserver/dev/play/scripts/node_modules/zombie/node_modules/bluebird/js/release/promise.js:593:10)
    at Promise._settlePromises (/Users/alexserver/dev/play/scripts/node_modules/zombie/node_modules/bluebird/js/release/promise.js:672:18)
    at Async._drainQueue (/Users/alexserver/dev/play/scripts/node_modules/zombie/node_modules/bluebird/js/release/async.js:125:16)
    at Async._drainQueues (/Users/alexserver/dev/play/scripts/node_modules/zombie/node_modules/bluebird/js/release/async.js:135:10)
    at Immediate.Async.drainQueues [as _onImmediate] (/Users/alexserver/dev/play/scripts/node_modules/zombie/node_modules/bluebird/js/release/async.js:16:14)
    at processImmediate [as _immediateCallback] (timers.js:383:17) +12ms

This is the code I'm attemping to running.
const browser = new Browser();
var url = 'https://www.example.com';
browser.visit(url);

This is redirected to localhost via /etc/hosts (not zombie).
I'm having an apache server using a fake certificate https virtual server.
How can I ignore ssl errors like with phantomjs ?

I am having the same problem, please give a solution to ignore https certificate verification.
Thanks!

you can add process.env.NODE_TLS_REJECT_UNAUTHORIZED = '0' before you call .visit() to turn off certificate chain validation.

Try this:
const browser = new Browser({strictSSL: false});


update hidapi submodule to 0.11.2 and bump crate version

https://github.com/libusb/hidapi/releases/tag/hidapi-0.11.2

Thank you!


Tab completion under pipenv?

macOS 10.12.6  Python 3.6.2  pipenv 6.1.6
When I run the CPython REPL from a standard shell tab completion is operative. When I run it via pipenv (pipenv run python3   or  pipenv shell then python3) tab completion is not operative.
Is this a bug, or a configuration failure on my part?

I'm not sure! Very interesting.
Can you test with a standard virtualenv and see if it's the same? Might be a pew side-effect.

Tab completion is operative under a standard virtualenv.

Now create one with pew.

i believe it's $ pew newvenv

The finger seems to point at pew  ...  tab completion is not operative.

Let's open an issue over there and link to it here!

Will do.

✨🍰✨

so aparently this code activates tab complete:
import rlcompleter, readline
readline.parse_and_bind('tab: complete')

when i run this, it enables autocomplete.

maybe this code is being excluded from site.py or something, in pew.

I opened https://github.com/berdario/pew/issues/150

Will use the rlcompleter / readline code as a fix for now. Thanks!

Thanks for noticing this!

Rather than the parse_and_bind call shown above I had to use
readline.parse_and_bind('bind ^I rl_complete') .
Maybe it's a macOS thing.

Interesting, it worked for me on MacOS.

site.py from the Python library has the following code in enablerlcompleter()
    # Reading the initialization (config) file may not be enough to set a
    # completion key, so we set one first and then read the file.
    readline_doc = getattr(readline, '__doc__', '')
    if readline_doc is not None and 'libedit' in readline_doc:
        readline.parse_and_bind('bind ^I rl_complete')
    else:
        readline.parse_and_bind('tab: complete')

Maybe we have different arrangements for accessing a readline library on our respective machines.

@ColMcp can you add this as a comment on https://github.com/kennethreitz/pipenv/issues/497#issuecomment-328190362 ?

Closing for https://github.com/berdario/pew/issues/150

@woozyking how does
$ source $(pipenv --venv)/bin/activate

help in any manner ?

$ pipenv shell
Spawning environment shell (/bin/bash). Use 'exit' to leave.
source /Users/woozyking/.local/share/virtualenvs/test-HaxrPBoG/bin/activate
bash-3.2$ source /Users/woozyking/.local/share/virtualenvs/test-HaxrPBoG/bin/activate
(test-HaxrPBoG) bash-3.2$ # tab completion works partially, not working, for example git commands

$ source $(pipenv --venv)/bin/activate
(test-HaxrPBoG) woozyking-air:test woozyking$ # tab completion works

Not to say it's a definitive solution, but a work around (and may very well be superficial) that works.

@woozyking Thanks ! So this helps with the shell completion, and not the one from the python interpreter, right ?

@ngirard yep

I also had problem using tab-completion in pip-env shell, source $(pipenv --venv)/bin/activate works for me though. My situation: largely using packages, including python, from Nix pkgs on top of Ubuntu 16.04.

Thanks @andyneff, that's very useful! :) I was disappointed to see that PYTHONSTARTUP is not read when using inspect. And neither does the source $(pipenv --venv)/bin/activate workaround.

@andyneff I still see the same problem with Python 3.7. Why did you test for Python 2 in your .pyrc ?

@wichert At the time, I was not having the same issue on my python 3. If you are, then by all means, remove the test for Python 2

Hi, consider re-opening this please.  It was closed after a new bug was filed against pew.  However it doesn't seem they're going to fix it.  The issue is with virtualenv.
Indeed when I create a virtual environment with virtualenv tab completion does not work, however if I create one with venv it does.  Could pipenv use venv instead?

(Note: If I sound frustrated, I am. It’s not really your fault, but because I need to say the same thing so many times.)
We’ve been through this many times in the issue tracker already. venv support is not currently viable because we can’t drop Python 2 support yet, and do not have resources to maintain two backends (or more, venv is not as compatible across Python releases as you might imagine).
If you want to help, try fix this for virtualenv.

venv also is inferior

Python 2 is EOL now, so perhaps this could be revisited.

any updates?  I use the way pipenv run python ./script.py but i got large script directory now, and need to switch between multiple scripts, but autocompletion with tab does not work i have to know the name and location and is hard to keep track, the part that is missing completion is the file completion after the work python


Settings dialog, TTS voices page: display names of TTS engines

@bertfrees commented

In the voices dialog, you capitalize the engine name. Engines also have a "display name". I will make that available in the web API.

Is there a (pipeline) engine issue for this that we can monitor?

See https://github.com/daisy/pipeline-modules/issues/94

Note that https://github.com/daisy/pipeline-modules/issues/94 is done, but I'm not sure if the changes are included in the engine snapshot that you are currently using in the GUI. I think they are.

Yes it's already there but the names need to be shorter.
<tts-engines href="http://localhost:49152/ws/tts-engines">
    <tts-engine name="osx-speech" nicename="macOS" status="available" voices="http://localhost:49152/ws/voices?engine=osx-speech"/>
    <tts-engine message="Property not set: org.daisy.pipeline.tts.google.apikey" name="google" nicename="Google Cloud" status="disabled"/>
    <tts-engine features="speech-rate" name="azure" nicename="Microsoft Azure" status="available" voices="http://localhost:49152/ws/voices?engine=azure"/>
</tts-engines>

Yes, names have been made shorter also but that part has not end up in the snapshot yet.

Ok, well the UI change has been incorporated, it'll use whatever the engine provides. I think we can close this one.


add brian zhang's website

To add your site to the webring, you must be a current student or an alum of the undergraduate Software Engineering program at the University of Waterloo (Waterloo, ON, Canada).
Full Name: \Brian Zhang
Cohort Year: \2028
Full website URL: \https://brianwzhang.com/
LinkedIn or a similar profile: \https://www.linkedin.com/in/brian-w-zhang/
Don't forget to also add your entry to the end of the allSites array in the sites.js file (name, year, personal website)!

Hi @simcard0000, just added the link back to the webring.

Exciting Job Opportunity at Microsoft!
Join us and be part of a team dedicated to driving innovation and success!
How to Apply:
To apply for this exciting opportunity, please click here.
Job Description:

Position: Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan
Company-contributed retirement plan
Flexible working hours
Collaborative and inclusive work environment

Requirements:

Proven experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Familiarity with frameworks such as React, Angular, or Vue.js
Problem-solving skills and ability to work in a team

Note: This position is open to candidates from anywhere in the world.jalvarezit, @SylvainMartel, @tazjel, @kfdslsope, @anthonytwh, @w9w, @vdauchy, @darccau, @SiegeBot, @northrup, @bl4de, @PewZ, @dark1ord007, @thefirstcircle, @theomart, @rmlzy, @modulexcite, @Nsbx, @charlie-goldenowl, @jrdevore

@brian-w-zhang thanks for the update! I'll take a look in the morning. You may have also gotten a notification for a spam post on this PR - do not click any of the links (I deleted the post but like if you got an email as a part of the notif or something 🥲)


fix(documentation): fixed typos in README

Fixes 2 little typos in the README.adoc

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 1.2.0
download it from Artifactory here

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

1.1.1
GitHub release

Your semantic-release bot :package::rocket:


Special char emphasis rendering

Hello,
I've noticed a rendering problem:
some special chars (%, !, ?, ...) are not correctly applying emphasis, if the char is the first char of the emphasis group and if the emphasis group is preceded by a letter
**%**
produce %
100**%**
produce 100**%**
Is it a desired behavior or a known problem?

Is it a desired behavior or a known problem?

That is a desired behavior.
100**%**
   ^^--- these are interpreted as closing emphasis, not opening

Roughly speaking, there are 3 classes of characters to determine whether emphasis pairs are openers or closers:

alphanumeric characters
punctuation characters
whitespace characters

And ** attaches to the ones with most priority. In your case, ** would rather attach to preceding 0 than succeeding %.


Tree features needed for debug console

[ ] somehow need to specify to the tree that the element can not be selected via click and context menu. Currently I was hacking this in via the controller and overriding onClick. Ideas welcome here
[ ] Related to this tree selection disablment I need the native selection still to be respected
[ ] I need getScrollPosition and setScrollPosition. Currently I need it for: users adds a new repl input -> whole repl gets refreshed, I need to figure out if I should reveal the end, I do that only if the end was previously revelied
[ ] I need to navigate through all the visible elements to get their content for Copy All action. I can expose getNode and than go over all looking at the expanded property

Also previosuly I was using monaco-tree-row.has-children for two use cases:

In hover and repl I want to have the pointer hand cursor for only elements with children, otherwise default cursor
In repl some elements with children I reposition their twisty, because input output expressions pairs I actually render as one element, but I want the twistie to actually be in the bottom, not top. Example of this is when user inputs console


[$250] [Group 6][Accessible Pressable] Migrate SignInPageGraphics

HOLD on https://github.com/Expensify/App/issues/16988
Description
Migrate SignInPageGraphics to use PressableWithFeedback
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01192b8c0768c17f36
Upwork Job ID:<PHONE_NUMBER>424855552
Last Price Increase: 2023-06-07

Triggered auto assignment to @joekaufmanexpensify (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero process steps for feature requests for more details.

Can you assign me to this one?

Sorry, we've already hired someone to work on this through a different process than we typically use for external contributors. Please feel free to review our contributing guidelines and look for any issues that still have the Help Wanted label.

Looks like no PR yet here.

@roryabraham I have migration ready but I cannot find where this component is used. Is it possible that this component became obsolete, and can be removed?

@robertKozik Looks like you're right and this component is unused. Let's remove it

To confirm since https://github.com/Expensify/App/pull/21420 is merged and in production, @robertKozik and @mollfpr need to be paid $250 each?

@anmurali Just $250 to @mollfpr. @robertKozik works for Software Mansion and will invoice us separately for work on this project.

https://www.upwork.com/nx/wm/offer/25455403 - @mollfpr

I can't open the offer. In case you need my profile https://www.upwork.com/freelancers/~01de3ccafd86e0bce0

@anmurali bump

https://www.upwork.com/nx/wm/offer/25582335
Can you accept that @mollfpr

Ready for payment @anmurali 🙇🏼

Paid.


Filter items if they have a score

If an item has a score, that's when we should filter them
other we assume that a score hasn't been set in the first place

Coverage remained the same at 49.833% when pulling a50da37eb019fe3e272b085a82924fa8901efd31 on adnasa:correct-score into 870b405834c7b31eeabcbed2e932290c1867f7c9 on vutran:master.

At this point, we can just remove this filter() since it would always return true. The reason why that was added in the first place is to exclude unrelevant results that doesn't match the keyword. Since it's causing some issues, I think we should just remove that filter overall.
Thoughts?

Yes, we can remove the filter for now,
but since we are here, does this mean that we should remove all score implementation in core?

Actually, disregard my last comment as it is incorrect.

ookay :)


ensure handshake done before read

Sometimes we can not  manually call flush,  so we need auto flush before read

I didn't do that for two reasons

I want to avoid doing writes in polll_read
I want to avoid accidental poll_read causing poll_write not using the early data.
like

let (mut reader, mut writer) = split(stream);
join!(
     process(&mut reader),
     process(&mut writer)
);


Bug Report: Green Screen of Death when acessing settings from within console

Environment
Windows build number: 10.0.18362.145
Windows Terminal version (if applicable):

Steps to reproduce
-Open "Windows Terminal (dev build)"
-Open the dropdown menu
-Select "Settings" from dropdown menu

Expected behavior
"Profiles.json" is opened

Actual behavior
Windows crashes to a Green Screen of Death with stop code APC_INDEX_MISMATCH
No further information presented.

Well that's insanely bad. I thought we fixed this with the EnableMouseInPointer thing, but this might be something else.
Actually, thinking on it more, this might be something else entirely. The EnableMouseInPointer thing was for Alt+a causing a GSOD.

For the record, I can't get this to repro locally, so if I could get another verification of this bug report, that'd be appreciated.

For the record, it might be unrelated but Microsoft Word also causes a GSOD on build 1903.

@mrmikeman38 Thanks for the report. Would you mind collecting a memory dump (see here for brief instructions on enabling that if it's not already enabled) and e-mailing it to<EMAIL_ADDRESS>Mention my name ("Dustin Howett") as somebody who they should route it to.

Is it possible to reopen this because this is still a problem. Just installed.
Winver: 18362.175
Windows Terminal: 0.2.1715.0

I can provide information about it, I have done the exact same thing as the owner and I got a bsod.
Informations:
Computer basic informations:
Windows 10 Pro x64
Version 1903 (Build: 18362.175)
BSOD informations:
Bug Check String:
• APC_INDEX_MISMATCH
Bug Check Code:
• 0x00000001
Caused By Driver:
• ntoskrnl.exe
Caused By Address:
• ntoskrnl.exe+1bc8a0
Here is my full Bsod dump uploaded on WeTransfer.com
https://we.tl/t-X23Ccy1nmA