Appstudio update rhtap-demo-component-fqvx

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: rhtap-demo-component-fqvx-on-pull-request-
creating pipelinerun rhtap-demo-component-fqvx-on-pull-request- in namespace rhtap-demo-dev-iwrj-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


MINOR: doc change for deprecate removal

Committer Checklist (excluded from commit message)

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

@mjsax addressed comments.

retest this please

@vvcephei @mjsax comments addressed.


Avoid undefined behavior with std::minmax and rvalues.

The behavior of std::minmax with rvalue arguments is undefined after the
end of the expression that contains the call to minmax. This code to
special-case the comparisons for ARM vs. Thumb and macOS vs. Darwin was a
little overly clever anyway. Rewrite it to use straightforward comparisons.
This fixes a test failure in Serialization/target-incompatible.swift when
building with a recent version of clang, because these checks were completely
optimized away due to the undefined behavior. rdar://problem/28700005

@swift-ci please smoke test

@jrose-apple Can you please review?


Add jpeg support

Add support for encoding jpeg images. This changes the API for *InitialsAvatar.DrawToBytes by adding an optional encoding parameter to specify  what kind of image is to be generated. Setting encoding to jpeg will encode the image to jpeg format, the default is png.

@holys  the continuous integration failure has nothing to do with this PR. All the tests pass( from the travs log).

:+1:

@gernest what does undersize and oversize do?
https://github.com/holys/initials-avatar/pull/5/files#diff-16b4d18092846142ce07a1b970d69adbL19

@holys nothing for the moment. It was my intention to check if the limits like max image size are set  correct or not. Which is an anticipation because I can't find anything in the API.

@holys I am paranoid and infected with testing fever haha! You can just remove them, and maybe I will add them when the need arise :smile:

@gernest  :)

:smile:


Allow SQS region by config

Allow SQS region by config, so we can move the queues if needed.

Maybe squash into a single commit (as it's such as small change), but otherwise looks good to me :+1:


Remove class in child that forces left-0 on a centered menu

Subject: We should not force the dropdown window of navigation-menu.tsx to position: absolute and  left: 0 when the menu itself is center.
Type: Bug
Current behaviour:

Desired behaviour:

@shadcn This might be better to set up this change as a component variant for a centered navigation-menu and leave the default classes as is. The left-0 makes things nice and aligned left for navigations that are left aligned (like https://ui.shadcn.com/) but breaks if the navigation menu is centered like the screenshots in the original post.

@leabs Looking into it. Thanks for the heads up.

This has been fixed. I'm closing this but feel free to re-open if not. Thanks for your help fixing this issue. Appreciate it.

Making the changes suggested to the NavigationMenuViewport this PR works for me


Moneyed dependency

After merging pull request #3 new dependency in file utils.py has been created:
import moneyed

A reason why modules were messed up is not clear. There was two independent modules moneyed for storing money type and djmoney_rates for money convertation.
If someone wants to use these modules together - why not, but there are no needs to inject one into another.
By pull request #3 was implemented one particular usage case of this module. This is not right and breaks module structure.
CC: @tzenderman

Your comment makes sense, but I didn't think anyone that is using this would not be using the py-moneyed package. @synasius we could remove the util function convert_money https://github.com/evonove/django-money-rates/pull/3/files#diff-cec83698a31b7e3204fb01ff85b29c16R58 and add it to the Readme as an option for the developer to implement in his own project.


Pongo in dbless mode (--no-postgres) using kdbl is failing to recognize second custom plugin

Hello,
Please note that I am running kong/kong-gateway:<IP_ADDRESS>-amazonlinux-2023 image.
I've built an image with a custom plugin named jwk_collector with its source located in:
/usr/local/share/lua/5.1/kong/plugins/jwk_collector
And I am running Kong image <IP_ADDRESS>/<IP_ADDRESS> with it and developing one more plugin. Using DB-less mode (--no-postgres) with kdbl Kong reports that the plugin is not enabled:
in 'services':
  - in entry 1 of 'services':
    in 'routes':
      - in entry 1 of 'routes':
        in 'plugins':
          - in entry 1 of 'plugins':
            in 'name': plugin 'jwk_collector' not enabled; add it to the 'plugins' configuration property
stack traceback:
	[C]: in function 'error'
	/usr/local/share/lua/5.1/kong/init.lua:793: in function 'init'
	init_by_lua(nginx-kong.conf:60):3: in main chunk

  Run with --v (verbose) or --vv (debug) for more details

I am sourcing kong.conf and kong.yaml with the necessary configuration (e.g. jwk_collector -> enabled: true) before running kdbl
Kong.conf:
..
plugins = bundled,jwk_collector

and of course, the location has kong:kong perms:
/usr/local/share/lua/5.1/kong/plugins/jwk_collector

The Dockerfile for the image contains:
ENV KONG_PLUGINS=bundled,jwk_collector

so you have 2 related custom plugins? that rely on each other?
Have you considered to put the two in the same repo? since they rely on each other anyway? In a default plugin structure, they are namespaced and can live next to eachother in the same repo.
Then you probably do not have to add anything, nor create a custom image. Pongo will insert and enable both plugins.

Well, I removed their dependency, so they run separately from one another. Now, I managed to get them in a single repo as your suggestion - Thank you!
I tried this before, but it didn't work. I did it now and it works!

thx for reporting back.


[Hooks] An official global state-management solution (a la redux)

Related to #68
I'm unsure if this is the appropriate place for this. If not, feel free to close.
The useState hook feels very similar to the way atoms work in reagent.cljs, except for one important part: in reagent, multiple components can share one atom so that all of them re-render when it changes, whereas useState only creates state local to the component that called it and only that component (and its children) re-render when that state changes. To say it another way, React hooks offer little to no affordance for doing scalable global state management in the fashion of redux or mobx, whereas reagent atoms do. However, I think it would be relatively simple to implement atoms for React with a light abstraction on top of useState and useMutationEffect.
I've already implemented a minimal version of Atom for react, and so far I really like the result. I'm curious to know what others think.
I'm just calling it react-atom for now and have published it (with TypeScript typings) to npm. Here are the docs. And here is a JS CodeSandbox and TS CodeSandbox to try it out live. If you're wondering, the API for it is mostly just plagiarized from clojure/script atoms.
I think there would be a lot of benefit to react.js providing something like this as an official way for doing global state management. Doing so would really help unify the community and library ecosystem, and it would allow some performance optimizations to be managed internally by react. It could be made more plug-able too, so as to enable the various niceties made possible by redux middleware.
The new Hooks proposal seems like the perfect opportunity to introduce something like this. What do you think?

Here is a basic example of what using an Atom looks like (remember, it's just using useState and useMutationEffect internally):
import React from 'react';
import ReactDOM from 'react-dom';
import {Atom, deref, swap} from '@dbeining/react-atom';

/////////////////////// APP STATE /////////////////////////
const stateAtom = Atom.of({
  count: 0,
  text: '',
  data: {
    // ...just imagine
  },
});

/////////////////////// EFFECTS /////////////////////
/**
 * `swap` applies a function to an Atom's current state to set it's next state
 * Then it rerenders all the function components that `deref`ed that atom
 * so they read its new state
 */
const increment = () =>
  swap(stateAtom, (state) => ({...state, count: state.count + 1}));

const decrement = () =>
  swap(stateAtom, (state) => ({...state, count: state.count - 1}));

const updateText = (evt) =>
  swap(stateAtom, (state) => ({...state, text: evt.target.value}));

const loadSomething = () =>
  fetch('https://jsonplaceholder.typicode.com/todos/1')
    .then((res) => res.json())
    .then((data) => swap(stateAtom, (state) => ({...state, data})))
    .catch(console.error);

///////////////// COMPONENT  /////////////////////
export const App = () => {
  /**
   * `deref` reads the current value of the atom
   * also, on the first render, it subscribes the component to the Atom
   * so that it re-renders when that Atom changes.
   * It will automatically unsubscribe from updates when the component un-mounts.
   */
  const {count, data, text} = deref(stateAtom);

  return (

Count: {count}

Text: {text}

      <button onClick={increment}>Moar</button>
      <button onClick={decrement}>Less</button>
      <button onClick={loadSomething}>Load Data</button>
      <input type="text" onChange={updateText} value={text} />

{JSON.stringify(data, null, "  ")}

  );
};

ReactDOM.render(<App />, document.getElementById('root'));

Doesn't this proposal provide a solution? https://github.com/reactjs/rfcs/pull/89

@mfrachet , yeah that's what @gaearon pointed out to me. #89 wasn't posted until the day after I finished prototyping react-atom so I missed it when I posted here.  Context.write does seem to be getting at a similar problem, but I'm curious what differences there are in the implications for each approach.
react-atom simply uses useState hooks internally to trigger component re-renders on state change, so it works with concurrent rendering and Suspense to the same degree that useState does, and as far as I can tell,react-atom has no problem sharing/syncing state across multiple React roots -however, I probably don't understand well enough the issues @acdlite described about concurrent rendering, Suspense, and dealing with multiple roots.
@acdlite , if you happen to see this, I'd be super grateful to know what you think about react-atom and its viability as an alternative to enhancing the Context API.

Hi, thanks for your suggestion. RFCs should be submitted as pull requests, not issues. I will close this issue but feel free to resubmit in the PR format.


Google Play returns "Varies by Device"

Compare of versions crashes when Google Play store has old versions for old devices...

Do you have an example app where I can reproduce the issue?
Thanks.

com.trixiesoft.clapp on the play store
On Fri, Feb 26, 2016, 6:58 AM Javier Santos<EMAIL_ADDRESS>wrote:

Do you have an example app where I can reproduce the issue?
Thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/javiersantos/AppUpdater/issues/13#issuecomment-189264064
.

Fixed in the latest version of the library (v1.2.1). Unfortunately, apps showing "Varies by Device" on Google Play are not supported by the library. Thanks for letting me know!
dependencies {
    compile 'com.github.javiersantos:AppUpdater:1.2.1'
}


style(standard): DRY out PropTypes

If we're going to go this route I'd rather destructure on the import, eliminates even more code.
import { Component, PropTypes } from 'react'.
I understand that your method was done since the Provide plugin is being used to expose React globally, but I'd consider dropping that plugin to eliminate this oddity. Personally I didn't really consider the inclusion of the ProvidePlugin as thoughtfully as I should have, and I'd really like to steer the project away from too much webpack magic. If you'd like to wait for me to drop it before making this change that might be best, unless you have other thoughts.

You're right, I did it in light of the ProvidePlugin because normally I do:
import React, { PropTypes } from 'react'

I'm fine with resubmitting something in that vein. It's a useful demonstration too, since some people just getting into ES6 don't know you can do that.

Also, I haven't checked, but does this global injection carry over to tests? If not, that's a good reason to drop it.

It does carry over into tests, so no problems there. I'll mull it over a bit tonight and figure out whether or not to drop the ProvidePlugin, and we can go from there.

+1 for import React, { PropTypes } from 'react'. ProvidePlugin is a lot of magic and in some IDEs throws errors because of missing imports.

I will get to this tomorrow :)

+1  for that from me too. Also in IDEs that support auto completion (like Webstorm, specificly when you add typescript definitions for react) it breaks the autocompletion for React and PropTypes.

By "tomorrow" I mean hopefully by the end of this week. ;)

Reverted to pre-ProvidePlugin w/ a few additional tweaks: https://github.com/davezuko/react-redux-starter-kit/pull/410

Updated PR

Thank you, appreciate it.


1745 Get simplified login

Do we want to merge it on master or to feature branch?
I did not notice and the previous one was merged to master - but we have feature branch and this one is connected to simplified login

we are going to be lost having some code in master and some on feature branch and NOW yes it's going to be complicated to merge :(


fix: use starknet::ContractAddress in #[system]

fixes #995
use starknet::ContractAddress instead of ContractAddress
use world_address instead of world_dispatcher

Is it possible to just use IWorldDispatcher instead of ContractAddress? if we can't use the dispatcher we should probably change the variable name to world_address instead of world_dispatcher

Hm as a matter of fact it's the IWorldDispatcher which is stored, it just happens that the serialised data is identical. Probably better to change it.

updated to world_address

back to world_dispatcher: IWorldDispatcher


[SPARK-44863][UI][3.5] Add a button to download thread dump as a txt in Spark UI

What changes were proposed in this pull request?
This PR added a button to download thread dump as a txt w.r.t. jstack formatting
Why are the changes needed?
The formatting of raw jstack can be relatively easy to read and analyze using various thread tools.
Does this PR introduce any user-facing change?

How was this patch tested?
Raw Dump File
driver.txt
Reporting by external tools
https://fastthread.io/my-thread-report.jsp?p=c2hhcmVkLzIwMjMvMDgvMjAvZHJpdmVyLnR4dC0tMTMtNTMtMjI=&
Was this patch authored or co-authored using generative AI tooling?
no

@dongjoon-hyun yes, thanks for the reminder.
@yaooqinn let's keep this feature only in master, sorry for asking for merging to 3.5

It's okay @zhengruifeng. And thank you, @dongjoon-hyun, for the clarification.

Thank you for the decision, @yaooqinn and @zhengruifeng .


clipboard box is always empty

I'm using brackets 0.44 and the clipbox 0.2.1 extension on ubuntu 14.04 within a virtual machine.

Same on my machine (Arch Linux). Actually it worked for a short time but then stopped and I have no clue why. I will try to debug it within the next days.

Same problem with Linux Mint 17.2 (Ubuntu 14.04 based)


[Bug Report] el-table setting type=selection,el-checkbox no response on safari 9.0 System。Hope to get back to you as soon as possible

Element UI version
2.14.1
OS/Browsers version
Safari9.0
Vue version
2.5.13
Reproduction Link
https://codepen.io/YiXueQing/pen/qBapNxd
Steps to reproduce
Click the multiple box in the list to check no response; the multiple box in div cannot be checked
What is Expected?
Normal selection and cancellation of multiple chexkbox
What is actually happening?
No response from multiple chexkbox

Safari9.0 radio component not availablec

When can reply and provide solutions

When can reply and provide solutions


chore: Add documentation for common POST */react errors

Description

Adds documentation for Reactor errors and common scenarios.

Testing required outside of automated testing?

[X] Not Applicable

Screenshots (if appropriate):

[X] Not Applicable

Rollback / Rollforward Procedure

[X] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Send welcome email to new users

Would SES suffice for sending mail via SMTP? There's an official SDK for Golang that I think would work quite well for this, especially because you can just import the stuff you need for specific services. There are also some other unofficial libraries that might work in place.

Using SMTP would be fine though I don't think there's a good reason to restrict it to Amazon SES. If the interface to send email is SMTP, any provider could be used.
I have a HTML template that can be used for the welcome email - I'll push that to a branch.

I've created a basic HTML template here, on a branch:
https://github.com/mattbostock/leavediary/blob/welcome_email/templates/welcome_email.tmpl

Ah, I see: those who run this provide their own SMTP details. In that case, yes - much better to make connections to the details the user provides.


fix python crate with the changes to logical plan builder

Which issue does this PR close?
fix python crate with the changes to logical plan builder
Closes #.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@jorgecarleitao please help review this if you have time, thanks


Add iconz collection

Add iconz collection metadata for a single item to test before migrating more

Can I get some feedback on what's wrong here? Perhaps the inscription icon? Can you guide me on how to set that up properly?


[FlakyTest] CI failure caused by oom

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

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

Describe the subtask
CI failure caused by oom
KyuubiOperationPerConnectionSuite:
- KYUUBI #647 - async query causes engine crash
- client sync query cost time longer than engine.request.timeout
- sync query causes engine crash
- test asynchronous open kyuubi session
- test asynchronous open kyuubi session failure
- open session with KyuubiConnection
./build/mvn: line 99:  1710 Killed                  ${MVN_BIN} $MAVEN_CLI_OPTS "$@"
Error: Process completed with exit code 137.

https://github.com/apache/incubator-kyuubi/runs/6317041935?check_suite_focus=true
https://github.com/apache/incubator-kyuubi/runs/6318021157?check_suite_focus=true

Parent issue
#1733
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

cc @yaooqinn

I've had the same problem many times


Special characters don't look right in organization field

Certain special characters don't seem to be showing up properly. For example, "Federal University of Uberlândia" shows up as "Federal University of Uberl├óndia".
As you can see in the linked example, this problem isn't universal - the "â" shows up correctly in the title but not the organization field. This suggests it is an AMS/ISS issue. Sure enough, this organization doesn't show up correctly in Membersuite.
@jessicaaasheaashe Can you ask Membersuite about this?

The unicode characters are incorrect in MemberSuite, so they'll bleed over into the Hub, Stars. the Bulletin, etc.  Need to fix in MemberSuite, and the changes will ripple out to the other systems.
I'll guess the resource listed was entered through the Hub interface, which does seem to handle unicode characters correctly -- that's why the â is right in the resource title.  (And it's wrong in the organization name because that's what comes from MemberSuite.)

All,
I think this was a migration error.  I think it would be easier to just correct these in Member Suite manually instead of asking them to do some sort of global fix.  I just fixed the Federal University of Uberlândia one.  If you could identify other records where this is happening, let's go ahead and fix manually.
Sincerely,
Jessica

Got it! I've started fixing manually in MS and will continue to do this. I'll close this ticket here.


Fix for a function_clause crash in couch_native_process

Overview
In case when CouchDB has a native query server enabled and we call an Erlang map function for the first time (therefore there is no view index) couch_mrview_updater:start_update calls run(State, [<<"map_doc">> , Doc]) from the couch_native_process and after this view index is built and data is read from it. Unfortunately when the same Erlang map function is used for changes feed with _view filter, couch_native_process crashes with:
{"error":"error","reason":"{{function_clause,\n     [{couch_native_process,ddoc,\n          [{evstate, ....

Changes feed with _view filter allows to use existing map function as the filter and it doesn’t query the view index files. Furthermore seq_indexed option doesn't work correctly in the mrview (more details in this thread) hence fast_view filter (which allows to use a dedicated seq and keyseq indexes) was disabled in the CouchDB 2. As a consequence changes feed with _view will always call run(#evstate{ddocs=DDocs}=State, [<<"ddoc">>]) and this call will crash with the aforementioned error.
This crash is caused by a missing implementation of ddoc function for <<"views">> FunPath case, because for requests like /_changes?style=all_docs&filter=_view&view=test/repl the FunPath will be: [<<"views">>,<<"repl">>,<<"map">>]. For JS map functions this does work because in this case couch_query_server delegates map to an external couch_js OS process.
This fix effectively provides that missing ddoc clause and it is based on the FilterFun one but matches return values of the erlang:put() function (which is called from the native Emit function) and it also matches ok and false return values for the cases when docs were not emitted and Erlang view just returned either ok or a false atom.
Testing recommendations
Here is a simple shell script which makes curl requests in order to populate a test db with dummy docs and design docs containing Erlang and JS map functions:
#!/bin/sh

# creating the db
curl -u admin:admin1 -X PUT http://<IP_ADDRESS>:5983/db1

#insert ddoc with erlang view
curl -u admin:admin1 -H "Content-Type: application/json" -X PUT http://<IP_ADDRESS>:5983/db1/_design/test -d '{"_id":"_design/test","language":"erlang","options":{"include_deleted":true,"include_design":true},"views":{"repl":{"map":"fun({Doc}) ->\n    Repl = lists:keyfind(<<\"repl\">>, 1, Doc),\n    if\n         Repl =:= {<<\"repl\">>, <<\"unsafe\">>} ->\n              ok;\n         true ->\n              DocId = case lists:keyfind(<<\"_id\">>, 1, Doc) of\n                   {<<\"_id\">>, Id} -> Id;\n                   false -> undefined\n              end,\n              Emit(DocId, null)\n    end\nend."}}}'

#insert second ddoc with js view
curl -u admin:admin1 -H "Content-Type: application/json" -X PUT http://<IP_ADDRESS>:5983/db1/_design/testjs -d '{"_id":"_design/testjs", "language":"javascript","options":{"include_deleted":true,"include_design":true},"views":{"repl":{"map":"function (doc) {\n\tif (doc.repl !== \"unsafe\") emit(doc._id, null);\n}"}}}'

#populate with 50 repl safe docs
for IDX in {1..50}; do curl -u admin:admin1 -H "Content-Type: application/json" -X POST http://<IP_ADDRESS>:5983/db1 -d '{"index":"'$IDX'", "repl": "safe"}'; done

#populate with 10 repl unsafe docs
for IDX in {51..60}; do curl -u admin:admin1 -H "Content-Type: application/json" -X POST http://<IP_ADDRESS>:5983/db1 -d '{"index":"'$IDX'", "repl": "unsafe"}'; done

#populate with 10 random docs
for IDX in {61..70}; do curl -u admin:admin1 -H "Content-Type: application/json" -X POST http://<IP_ADDRESS>:5983/db1 -d '{"index":"'$IDX'", "data": "test"}'; done

In order to test just run changes feed for both JS map:
curl -u admin:admin1 'http://<IP_ADDRESS>:5983/db1/_changes?style=all_docs&filter=_view&view=testjs/repl'

and Erlang one filter=_view&view=test/repl and compare the output including the "last_seq" which ought to be the same.

paging @eiri for PR review please

@wohali @AlexanderKaraberov Just to let you know, I'll get to it later this week, suffering from a flu right now.

This is rather embarrassing, but I still can't find time to look into it.
@iilyak, @nickva - if one of you have free cycles, can you revew this, please?

This PR needs tests. I do believe that following would be sufficient. I am going to test it tonight:
diff --git a/src/couch/test/couch_changes_tests.erl b/src/couch/test/couch_changes_tests.erl
index 673f2faad..f529eb312 100644
--- a/src/couch/test/couch_changes_tests.erl
+++ b/src/couch/test/couch_changes_tests.erl
@@ -153,7 +153,9 @@ filter_by_view() ->
             fun setup/0, fun teardown/1,
             [
                 fun should_filter_by_view/1,
-                fun should_filter_by_fast_view/1
+                fun should_filter_by_fast_view/1,
+                fun should_filter_by_erlang_view/1
+
             ]
         }
     }.
@@ -665,6 +667,39 @@ should_filter_by_user_ctx({DbName, _}) ->
             ?assertEqual(UpSeq, LastSeq)
         end).

+should_filter_by_erlang_view({DbName, _}) ->
+    ?_test(
+        begin
+            DDocId = <<"_design/app">>,
+            DDoc = couch_doc:from_json_obj({[
+                {<<"_id">>, DDocId},
+                {<<"language">>, <<"javascript">>},
+                {<<"views">>, {[
+                    {<<"valid">>, {[
+                        {<<"map">>, <<"fun({Doc}) ->"
+                            " case lists:keyfind(<<\"doc3\">>, 1, Doc) ->"
+                                " Emit(<<\"doc3\">>, null); "
+                                " _ -> ok"
+                            " end "
+                        "end.">>}
+                    ]}}
+                ]}}
+            ]}),
+            ChArgs = #changes_args{filter = "_view"},
+            Req = {json_req, {[{
+                <<"query">>, {[
+                    {<<"view">>, <<"app/valid">>}
+                ]}
+            }]}},
+            ok = update_ddoc(DbName, DDoc),
+            {Rows, LastSeq, UpSeq} = run_changes_query(DbName, ChArgs, Req),
+            ?assertEqual(1, length(Rows)),
+            [#row{seq = Seq, id = Id}] = Rows,
+            ?assertEqual(<<"doc3">>, Id),
+            ?assertEqual(6, Seq),
+            ?assertEqual(UpSeq, LastSeq)
+        end).

@AlexanderKaraberov: I did test the change with the test above. It fails without your change and passes with it. Could you either add the test to your PR or write another one if you think of better ways to test it?
Thank you for your contribution.

@iilyak
Yes sure. Sorry, I forgot to write proper tests before actually submitting my PR. I will run your test suite, verify whether it covers all the use cases and will include it in the PR. Hopefully today or tomorrow.

Hi @iilyak , I've checked the test you had added and it's completely fine. Thanks!
should_filter_by_erlang_view is exactly what's needed and I've included this test in the PR.

@iilyak
Excuse me for inconvenience, I've screwed up my fork. Will re-open this PR as a new one to avoid all the clutter if you don't mind.


Warnings in Chrome - SVG's SMIL animation are deprecated

Chrome is logging the following message when using slick "SVG's SMIL animations (, , etc.) are deprecated and will be removed. Please use CSS animations or Web animations instead."
Do you plan on moving to CSS animations?

I do use CSS animation?


IPv6 on shields.io

:question: Question
I'm trying to create a badge that queries my server to display the progress of a project of mine.
The problem is, my server is only accessible via IPv6. If I use the following, I get unresponsive
https://img.shields.io/badge/dynamic/yaml.svg?color=green&label=Progress&query=progress&url=http%3A%2F%2Fmerspieler.duckdns.org%3A8080%2Fprogress.php

If I replace the url with the IPv6 address(with [ and ]), I get Inaccessible.
Do you support IPv6 on shields.io?

Since there's no update on this, I close this issue now.


chore: use builtins iter to define os_order_iter

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#351  👈
#344 : 1 other dependent PR (#348 )
#319
#296 : 1 other dependent PR (#297 )
#246 : 2 other dependent PRs (#298 , #312 )
#235
#234
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.49%. Comparing base (4733c8e) to head (63717fb).

Additional details and impacted files
@@                              Coverage Diff                              @@
##           arni/declare/compilation/compilation_file     #351      +/-   ##
=============================================================================
+ Coverage                                      82.32%   82.49%   +0.16%
=============================================================================
  Files                                             33       33
  Lines                                           1607     1622      +15
  Branches                                        1607     1622      +15
=============================================================================
+ Hits                                            1323     1338      +15
  Misses                                           219      219
  Partials                                          65       65

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


MSbuild Task/Visual Studio Integration

Would be nice to be able to have this as part of my MSBuild chain, so when I build the .NET assembly I can have it automatically build the "native" wrapper

@SamuelEngland First need to fix test environment

@realvictorprm no rush 😄

c.c. @mhutch who has been thinking about IDE integration for e4k

@jonathanpeppers added some docs explaining how to setup post-build event for Visual Studio here: https://mono.github.io/Embeddinator-4000/getting-started-java-android.html

@tritao VSMac custom "After Build" steps are deprecated. I would suggest instead that we have a NuGet with MSBuild targets that can be simply added to the rpoject.

There is a XM sample that shows how to fire custom build steps:
https://github.com/xamarin/mac-samples/blob/master/UseMSBuildToCopyFilesToBundleExample/UseMSBuildToCopyFilesToBundleExample/CustomBuildActions.targets
But yes, we should have target people can pull in.

See also #644.


[Bug]: CRUD >> JSONForm | Insert Modal from an empty page does not show Table's actual fields in Modal

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Insert modal from an empty Table page while table still shows correct column names:

Insert modal from a Table Page where rows are filled with data:

Steps To Reproduce

Generate a CRUD page from a Datasource, eg: MySQL, Postgres etc
Observe the table & JSON form shows the selected row's data
Deploy the app
Navigate to a page in Table where there are no records present
Try to insert a record from this page [Click Insert button]
Observe the modal - fileds shown are not the table columns & is confusing
Navigate back to a page where table has data & try to insert record from this page
Observe the modal - fields from table are correctly shown so user can insert a new record
Expected: Insert Modal to always show the right column fields for proper insertion of records

Public Sample App
No response
Version
Prod/Release v1.7.1

@ashit-rath is there a way to populate the JSONForm fields from the table column headers?


Throw error if route config path has leading slash symbol

Fix for issue #1889
Currently it is not obvious for developer that something has gone wrong if route config paths have leading slashes. With this fix, error is being thrown on validation of route configs.
Added new test to cover this case.

I thought about proposed implementation too, but decided to go with warning, because:

it does not add complexity to path handling logic
although it is nice to allow both ways, our focus is to provide universal solution, so setting some rules might not hurt
paths without slash just seem cleaner :-)


Drop HHVM in .travis.yml

refs: https://github.com/Seldaek/monolog/pull/997#issuecomment-379158443

Thanks


does it support powershell

I used termion for terminal backend and unfortunately it does not support windows. Probably crossterm can be used instead.


No resource identifier found for attribute...  error

Hi,
Android 7.1.1. api25 I am unable to set any of the custom attributes, as I get "no resource found.." as per the example below:
No resource identifier found for attribute 'swipe_unselectedItemTitle' in package '...'
My root attribute has:
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
             xmlns:app="http://schemas.android.com/apk/res-auto"

and all other custom attributes work fine.
I also tried
xmlns:app="http://schemas.android.com/apk/lib/com.roughike.swipeselector"

Just check the gradle of Swipe selector with your gradle.


Added solutions and objective for "Headline with the h2 Element"

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

@KevinatorTrainer5 Please remove the Problem and Objective sections as the user can always view the original challenge for this information.
Thank you

Hi @KevinatorTrainer5 ,
As there has been no response from your end for a while, we are closing this PR.
Feel free to request this PR to be reopened or create a new PR, if you still wish to make the changes you originally added plus any additional change requests asked of you.  Also, remember to update your fork, so you have the latest version of master in case it has changed significantly.
Happy Coding!


Add a getting started example/tutorial

Add a getting started example/tutorial on how to add a new/custom Rust program to Motor OS image.

Done: https://github.com/moturus/motor-os/blob/main/docs/recipes/hello-moturus.md


Cordova plugin with custom onActivityResult requestCode

I am attempting to use the InAppBilling plugin (https://github.com/j3k0/cordova-plugin-purchase) for android, and it is using a custom requestCode in the onActivityResult callback. It is a random number (10001) in this case, but all that seems to be returned to the onActivityResult callback is 0. Am I missing something in the implementation? Is this plugin doing something non-standard that tabris does not support?
Any help on this is appreciated.

In general that should work. There were some comments on the result not being passed here: https://github.com/eclipsesource/tabris-js/issues/924. Maybe it is related.

It does seem to be a related issue, is there a roadmap to a resolution?

@dested Yes. I will look into it.

Thank you. This issue is critical to our application as we have a number of plugins we've experienced callback issues with.
https://github.com/j3k0/cordova-plugin-purchase
https://github.com/Telerik-Verified-Plugins/Facebook
https://github.com/dariosalvi78/cordova-plugin-health

@dested thanks for the pointers


Fix "original in English" href

The original href='/" will refer to http://ncase.me instead of http://ncase.me/crowds/ as excepted.

Whooooops. Thanks for catching that!


flot: fix unhighlight typo

The method is "unhighlight", but the type definition declared a version
of it as "unhightlight".

@ksheedlo thanks mate!


Quiz Model: moved created_at & quiz_code attributes to QuizBase and used QuizInDBBase

see files changed tab for changes in models/quiz/quiz.py, ignore test files

when creating a quiz, do we allow custom quiz_code?


Please expose common organization secret so same can be used across all repositories

PYPI_USERNAME and PYPI_PASSWORD secrets have been added to the organization secrets.


Missing f-string at the error messages in torchvision.utils._parse_colors

🐛 Describe the bug
At torchvision.utils._parse_colors, the following lines don't use f-string so that the error message prints the literal {colors}, not the value of colors variable.
    607     raise ValueError("`colors` must be a tuple or a string, or a list thereof, but got {colors}.")
    608 elif isinstance(colors, tuple) and len(colors) != 3:
    609     raise ValueError("If passed as tuple, colors should be an RGB triplet, but got {colors}.")

Versions
Collecting environment information...
PyTorch version: 2.4.1
Is debug build: False
CUDA used to build PyTorch: None
ROCM used to build PyTorch: N/A
OS: Debian GNU/Linux 12 (bookworm) (aarch64)
GCC version: (Debian 12.2.0-14) 12.2.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.36
Python version: 3.11.2 (main, May  2 2024, 11:59:08) [GCC 12.2.0] (64-bit runtime)
Python platform: Linux-6.6.31+rpt-rpi-v8-aarch64-with-glibc2.36
Is CUDA available: False
CUDA runtime version: No CUDA
CUDA_MODULE_LOADING set to: N/A
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:                         aarch64
CPU op-mode(s):                       32-bit, 64-bit
Byte Order:                           Little Endian
CPU(s):                               4
On-line CPU(s) list:                  0-3
Vendor ID:                            ARM
Model name:                           Cortex-A72
Model:                                3
Thread(s) per core:                   1
Core(s) per cluster:                  4
Socket(s):                            -
Cluster(s):                           1
Stepping:                             r0p3
CPU(s) scaling MHz:                   100%
CPU max MHz:                          1800.0000
CPU min MHz:                          600.0000
BogoMIPS:                             108.00
Flags:                                fp asimd evtstrm crc32 cpuid
L1d cache:                            128 KiB (4 instances)
L1i cache:                            192 KiB (4 instances)
L2 cache:                             1 MiB (1 instance)
Vulnerability Gather data sampling:   Not affected
Vulnerability Itlb multihit:          Not affected
Vulnerability L1tf:                   Not affected
Vulnerability Mds:                    Not affected
Vulnerability Meltdown:               Not affected
Vulnerability Mmio stale data:        Not affected
Vulnerability Reg file data sampling: Not affected
Vulnerability Retbleed:               Not affected
Vulnerability Spec rstack overflow:   Not affected
Vulnerability Spec store bypass:      Vulnerable
Vulnerability Spectre v1:             Mitigation; __user pointer sanitization
Vulnerability Spectre v2:             Vulnerable
Vulnerability Srbds:                  Not affected
Vulnerability Tsx async abort:        Not affected
Versions of relevant libraries:
[pip3] numpy==2.1.1
[pip3] torch==2.4.1
[pip3] torchaudio==2.4.1
[pip3] torchvision==0.19.1
[conda] Could not collect

Thanks for reporting, just created PR https://github.com/pytorch/vision/pull/8639


Filter doesn't work for columns other than CHARACTER VARYING with postgresql database

Extension Version
v1.14.3
OS and Version
macOS Sequoia v15.0
VS Code Version
1.93.1
Database name and Version
PostgreSQL 17
Steps to reproduce
1 - Install extension
2 - create .devdbrc file in project root directory
3 - addconfig for postgresql: [
{
"name": ,
"type": "postgres",
"host": ,
"port": "",
"username":,
"password": ,
"database": <database_name>
}
]
4 - in DevDB UI try to filter any column other than one with type CHARACTER VARYING
Expected behavior
I expect to get rows filtered
Screenshots

Additional context
No response

Thanks for reporting this @arielaa86
Fixed in the latest version v1.14.6
Please updrade to >= v1.14.6

You are welcome!
Sadly, the issue persists. Columns UUID and INT doesn't filter properly.


OSD-19453: drain-reboot-worker-node script

Renamed directory and amended script description.
What type of PR is this?
Feature
What this PR does / Why we need it?
We have changed the directory name to represent the script name, and changed the script description.
Which Jira/Github issue(s) does this PR fix?
_Resolves #_https://issues.redhat.com/browse/OSD-19453
Special notes for your reviewer
Pre-checks (if applicable)

[x] Validated the changes in a cluster
[x] Included documentation changes with PR

/lgtm
/approve


master broken: missing node argument after automatic layer view creation

🐛 Bug
All the examples on Master are currently broken - the error I get on add_image.py initially is
  File "/Users/nicholassofroniew/Github/napari/napari/_qt/qt_viewer.py", line 149, in _add_layer
    vispy_layer = VispyBaseLayer(layer)
  File "/Users/nicholassofroniew/Github/napari/napari/util/layer_view.py", line 52, in __new__
    return super().__new__(cls)
TypeError: Can't instantiate abstract class VispyBaseLayer with abstract methods _on_data_change

if i make this no longer an abstract method, the error i get is
  File "/Users/nicholassofroniew/Github/napari/napari/_qt/qt_viewer.py", line 149, in _add_layer
    vispy_layer = VispyBaseLayer(layer)
TypeError: __init__() missing 1 required positional argument: 'node'

which is because the VispyBaseLayer takes two arguments the layer and the node which doesn't seem to work out well with the approach of #488. We could drop the node argument and append it inside the layer specific Vispy views, but I defer to @kne42 on how to fix this.
Unfortunately this was not caught by any of our existing tests, which I find really confusing (they all pass locally for me even though I get this error). Maybe @kne42 can also figure out some tests to add that will catch this too!

@sofroniewn Oops, sorry about that.
I was able to fix the error, which was due to the files not being run and therefore not registered. It's at kne42:layerview-import-fix (commit).
However, I discovered that due to some weird things that PySide2 is doing with metaclasses, you cannot actually take arguments in the class definition line 😞. It does work with PyQt5 though.
So to fix that we can do one of the following:

revert the entirety of #488
revert part of #488 with regards to the Qt files and keep it for the VisPy ones
refactor the Qt files such that they wrap the Qt objects instead of inheriting from them

I'm going to be on PTO for the rest of the day so if you want a quick fix you should go with 1)

ok let's revert #488, merge, and then revisit early next week

maybe #2) reverting just for vispy is best if not too hard


Copy value from JSON Viewer

It'd be awesome if you could copy the value directly from the JSON viewer object.
Flow would be something like

Right click object in JSON Viewer
Click Copy
Viola

Good feature! will work on it

@visch ,
I am developing this feature and it is almost complete. Would you please mind if I ask you to test it and provide any early feedback?
Please note: currently, it copies node name, value, and path. Will try to extend it to copy the while array/object in near future.
How to test

Download binary from build room https://ci.appveyor.com/project/kapilratnani/json-viewer as appropriate e.g. win32 or x64 or ARM64
Open Notepad++ plugin folder

Open plugin folder, go to Plugins menu on Notepad++
Click on Open Plugins Folder

Navigate to NPPJSONViewer and rename the existing NPPJSONViewer.dll for backup.
Copy the downloaded binary (in step # 1) to NPPJSONViewer folder.
Run some test
Once done, restore the backup. To restore,

Delete the new file (NPPJSONViewer.dll)
Rename the old file to NPPJSONViewer.dll
Now backup is restored.

@visch , I am developing this feature and it is almost complete. Would you please mind if I ask you to test it and provide any early feedback?
Please note: currently, it copies node name, value, and path. Will try to extend it to copy the while array/object in near future.
How to test

Download binary from build room https://ci.appveyor.com/project/kapilratnani/json-viewer as appropriate e.g. win32 or x64 or ARM64

Open Notepad++ plugin folder

Open plugin folder, go to Plugins menu on Notepad++
Click on Open Plugins Folder

Navigate to NPPJSONViewer and rename the existing NPPJSONViewer.dll for backup.

Copy the downloaded binary (in step # 1) to NPPJSONViewer folder.

Run some test

Once done, restore the backup. To restore,

Delete the new file (NPPJSONViewer.dll)
Rename the old file to NPPJSONViewer.dll
Now backup is restored.

Slight issue with the Json Viewer not working until I hit the refresh button. Video here https://autoidm.slack.com/files/U02DQB1GWKT/F0436FV9CKG/screenshare_-_2022-09-21_4_10_00_pm.webm
Other than that (and the array issue) it looks to be working!

@visch Thanks for sparing some time and testing it.

Slight issue with the Json Viewer not working until I hit the refresh button. Video here (uploaded as the link before didn't work)

What Noptepad++ version do you use? I'm unable to reproduce it with Notepad++ v8.4.5   (64-bit). I will check 32 bit as well.


Cross compile for version 2.11.x

It would be nice if we could get binaries for older (still in use) scala version 2.11

There really haven't been any bugs found or changes to this library in quite a while (I think since we were publishing the 2.11 version).
Is there some problem you want addressed? If so, please report a bug.
I would be willing to add a 2.11 build back if we had some motivation, but I think you should be fine to use the last version published for 2.11 (which I use at work myself).


chore: Remove dynamic home view feature flags

This PR resolves ONYX-1398
Description
Clean Up Old Home View: PR I:
This PR removes the feature flags related to the new dynamic home view.
PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Clean Up Old Home View code - ole

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (Clean Up Old Home View code - ole - olerichter00)

  Generated by :no_entry_sign: dangerJS against 34a0d3c88fca8bd1f7512743c8c3b590c5028a71


Fix example

Closes https://github.com/MicrosoftDocs/office-docs-powershell/issues/10144

Learn Build status updates of commit 6e38e07:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsAutoAttendant.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@tiburd Please merge this PR. Thanks


Add detail error logs when 'Unknown Device' error happens if devicema…

Dear maintainers,
This PR is to add the detail error logs for device mapper storage when the error "unknown device' happens as below when tries to restart a container:
"Handler for GET /containers/d355ff91fe74b635b8ed325b880c5e1654e5e57b9b23e6455c3e8fb2b30a1f4f/json returned error: Unknown device d355ff91fe74b635b8ed325b880c5e1654e5e57b9b23e6455c3e8fb2b30a1f4f"
In order to find the reason why read the data failed from devicemapped device, these error logs are added.
Please review it, thanks a lot.

@mYmNeo , thanks Shengbo, I have amended the PR again, please help to review it again, thanks.

Dear maintainers,
The reason for experimental job failed is related to the test environment:
05:17:37 FATAL: Unable to delete script file /tmp/hudson3111005131340468747.sh
05:17:37 java.lang.InterruptedException
05:17:37 	at java.lang.Object.wait(Native Method)
05:17:37 	at hudson.remoting.Request.call(Request.java:147)
05:17:37 	at hudson.remoting.Channel.call(Channel.java:780)
05:17:37 	at hudson.FilePath.act(FilePath.java:979)
05:17:37 	at hudson.FilePath.act(FilePath.java:968)
05:17:37 	at hudson.FilePath.delete(FilePath.java:1422)
05:17:37 	at hudson.tasks.CommandInterpreter.perform(CommandInterpreter.java:104)
05:17:37 	at hudson.tasks.CommandInterpreter.perform(CommandInterpreter.java:64)
05:17:37 	at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
05:17:37 	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:782)
05:17:37 	at hudson.model.Build$BuildExecution.build(Build.java:205)
05:17:37 	at hudson.model.Build$BuildExecution.doRun(Build.java:162)
05:17:37 	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:534)
05:17:37 	at hudson.model.Run.execute(Run.java:1738)
05:17:37 	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
05:17:37 	at hudson.model.ResourceController.execute(ResourceController.java:98)
05:17:37 	at hudson.model.Executor.run(Executor.java:410)
Please help to check this PR, thanks a lot.

LGTM

Dear maintainers,
After i amended this PR according to @cpuguy83 's suggestion, actually, all the cases are passed. The fail occurred in  the same place:
03:09:56 FATAL: Unable to delete script file /tmp/hudson5863554771796508595.sh
03:09:56 java.lang.InterruptedException
03:09:56 	at java.lang.Object.wait(Native Method)
03:09:56 	at hudson.remoting.Request.call(Request.java:147)
03:09:56 	at hudson.remoting.Channel.call(Channel.java:780)
03:09:56 	at hudson.FilePath.act(FilePath.java:979)
03:09:56 	at hudson.FilePath.act(FilePath.java:968)
03:09:56 	at hudson.FilePath.delete(FilePath.java:1422)
03:09:56 	at hudson.tasks.CommandInterpreter.perform(CommandInterpreter.java:104)
03:09:56 	at hudson.tasks.CommandInterpreter.perform(CommandInterpreter.java:64)
03:09:56 	at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
03:09:56 	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:782)
03:09:56 	at hudson.model.Build$BuildExecution.build(Build.java:205)
03:09:56 	at hudson.model.Build$BuildExecution.doRun(Build.java:162)
03:09:56 	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:534)
03:09:56 	at hudson.model.Run.execute(Run.java:1738)
03:09:56 	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
03:09:56 	at hudson.model.ResourceController.execute(ResourceController.java:98)
03:09:56 	at hudson.model.Executor.run(Executor.java:410)

@hopkings2008 CI is just having problems.

@cpuguy83 and other maintainers, i have amended this PR according to @cpuguy83 's suggestion again, and please help to check it, thanks.

LGTM


m365 entra user get sends incorrect query to Graph API. Closes #5788

Closes #5788

Ready to merge 🚀

Resolved renaming aad to entra

merged manually. Thank you for your awesome work 👏


Best way to update custom Tabbar?

I'm trying to update my tabbar messages tab badge with a number.
I'm currently using mobx, but it seems that the tabbar component doesn't update directly with mobx.
So I tried to pass it as a prop to tabbar, but it updates the entire tabbar scene, causing a flick.
So I removed this prop. When I switch tabs, it updates the number.
Any tips on this? Is there a way to update only the custom tabbar?
Thanks in advance

Nevermind. Got it working by using component directly.


Onboarding improvements

Is your feature request related to a problem? Please describe.
I assume that 70%+ of people don't enable the notifications but they are crucial part of the product.
Describe the solution you'd like
Explainer screen that doesn't let people use the app without enabling notifications, explaining how important it is

I quickly created an onboarding screen in Figma to match the current designs. Does this work, or do we need different illustrations?

Update

[ ] This task should include speech profile in onboarding, connection and ws shared by same

[ ] Should find a better way to ask for permissions

[ ] Bluetooth doesn't appear when you opened (very bad experience)
[ ] Ideally connect your device is less skippable
[ ] Notifications permission is better asked.

[ ] Probably check bluetooth permissions, and ask the user to turn the device on or explain the lights indicators.

Aha moment should take 30 seconds for them to start seeing transcript in the screen.
Relates to https://github.com/BasedHardware/Omi/issues/606
When they open the app, they should instantly see new memory creation.

A sample memory.

This sample memory should explain that once the conversation finishes, they will get a memory, etc.

Onboarding Flow:
Notification permission -> Bluetooth Permission -> Connect Device (Also explain light indicators through UI) -> Setup Speech Profile (skippable?) -> If speech profile skipped, speak few words (show transcript) -> Show memory (aha moment) and then home

[ ] We should also include here a set name page here, during onboarding, to confirm the user name, so should use firebaseAuth first name, and ask them "How should we call you"  it should call @router.patch('/v1/users/name', tags=['users'])
[ ] People should be able to go to settings and change the name (specially existing users) cause some of them might have AnonymousUser value from apple hidden name.

[ ]  We should also include here a set name page here, during onboarding, to confirm the user name, so should use firebaseAuth first name, and ask them "How should we call you"  it should call @router.patch('/v1/users/name', tags=['users'])
[ ]  People should be able to go to settings and change the name (specially existing users) cause some of them might have AnonymousUser value from apple hidden name.

An endpoint to get the user details as well? To show their details like the name, email address etc in the personal details section in the app

@mdmohsin7 name should be changed directly with firebase auth, check auth.dart, but once changed we need to call the backend, to modify the text contained in facts, to match the new user name
"Kaitlin likes chocolate"
modified to
"Kaitlin2 likes chocolate"


TypeError: ignorePattern is not a function on<EMAIL_ADDRESS>

I tried to npm-start running my react native but encounter this error:
jest-haste-map: Watchman crawl failed. Retrying once with node crawler.
Usually this happens when watchman isn't running. Create an empty .watchmanconfig file in your project's root folder or initialize a git or hg repository in your project.
Error: Watchman error: ignorePattern is not a function. Make sure watchman is running for this project. See https://facebook.github.io/watchman/docs/troubleshooting.html.
ERROR  ignorePattern is not a function
TypeError: ignorePattern is not a function
at HasteMap._ignore (../node_modules/jest-haste-map/build/index.js:703:5)
at stdout.trim.split.filter.x (../node_modules/jest-haste-map/build/crawlers/node.js: 100 :58)
at Array.filter (native)
at Socket.child.stdout.on (../node_modules/jest-haste-map/build/crawlers/node.js: 100 :45)
at emitOne (events.js:101:20)
at Socket.emit (events.js:191:7)
at Pipe._handle.close [as _onclose] (net.js:511:12)
Version:

jest: 20.0.0
node: 7.10.0
watchman: 4.7.0

I run npm-start normally on<EMAIL_ADDRESS>

Can you please provide a repro?

I'm not sure about the repro, I tried npm-start then it requires jest, then I install latest version @20.0.0 where this error is occurred. I retried with @19.0.2 to make it work. Here is my package.json
`/**

Sample React Native App
https://github.com/facebook/react-native
@flow
*/

import React, { Component } from 'react';
import {
AppRegistry,
StyleSheet,
Text,
View
} from 'react-native';
export default class auth extends Component {
render() {
return (

Welcome to React Native!

To get started, edit index.android.js

Double tap R on your keyboard to reload,{'\n'}
Shake or press menu button for dev menu

);
}
}
const styles = StyleSheet.create({
container: {
flex: 1,
justifyContent: 'center',
alignItems: 'center',
backgroundColor: '#F5FCFF',
},
welcome: {
fontSize: 20,
textAlign: 'center',
margin: 10,
},
instructions: {
textAlign: 'center',
color: '#333333',
marginBottom: 5,
},
});
AppRegistry.registerComponent('auth', () => auth);
`

I mean, it would be nice to have a repository where I could checkout and triage the issue :)

This shouldn't be an issue unless you are using npm2, maybe.

found this problem too.
npm version: 5.0.0
node version: V8.0.0
react-native : 0.42.3
jest-haste-map: 20.0.3
os sys: windows
when npm start in cmd, it occurs that TypeError: ignorePattern is not a function
so  I have to downgrade jest-haste-map to 18.0.0 to keep it working.

Also having this problem. Source code: https://github.com/lucaszanella/jscam/tree/3f77d06aa6616d9803b8400888774297408c5a7b/src/jscam
If I include the _rn-cli.config.js on the project, that is, take _ out, the error occurs


Add insert multi omit tests

will replace #1146

Codecov Report

Merging #1439 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1439      +/-   ##
==========================================
+ Coverage   56.93%   56.96%   +0.02%
==========================================
  Files          48       48
  Lines        8137     8137
==========================================
+ Hits         4633     4635       +2
+ Misses       2888     2887       -1
+ Partials      616      615       -1

Impacted Files
Coverage Δ

session_insert.go
56.16% <0%> (+0.36%)
:arrow_up:

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 3f2ca6e...918d06a. Read the comment docs.


Crash on android 6.0.1 (100%) - Pusher. java.lang.NoClassDefFoundError - ma.d

Crash logs
Fatal Exception: java.lang.NoClassDefFoundError: ma.d
       at com.pusher.client.Pusher.<init>(Pusher.java)
       at com.pusher.client.Pusher.<init>(Pusher.java)
       at com.pusher.channels_flutter.PusherChannelsFlutterPlugin.init(PusherChannelsFlutterPlugin.java)
       at com.pusher.channels_flutter.PusherChannelsFlutterPlugin.onMethodCall(PusherChannelsFlutterPlugin.java)
       at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage()
       at io.flutter.embedding.engine.dart.DartMessenger.invokeHandler(DartMessenger.java)
       at io.flutter.embedding.engine.dart.DartMessenger.lambda$dispatchMessageToQueue$0(DartMessenger.java)
       at android.os.Handler.handleCallback(Handler.java:739)
       at android.os.Handler.dispatchMessage(Handler.java:95)
       at android.os.Looper.loop(Looper.java:148)
       at android.app.ActivityThread.main(ActivityThread.java:7406)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)

Flutter logs

flutter_error_exception
PlatformException(error, null, null, java.lang.NullPointerException at la.c.x() at la.c.onMethodCall() at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage() at io.flutter.embedding.engine.dart.DartMessenger.invokeHandler() at io.flutter.embedding.engine.dart.DartMessenger.lambda$dispatchMessageToQueue$0() at io.flutter.embedding.engine.dart.DartMessenger.a() at io.flutter.embedding.engine.dart.a.run() at android.os.Handler.handleCallback(Handler.java:739) at android.os.Handler.dispatchMessage(Handler.java:95) at android.os.Looper.loop(Looper.java:148) at android.app.ActivityThread.main(ActivityThread.java:7406) at java.lang.reflect.Method.invoke(Native Method) at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120) )

Thanks pusher team

I believe there may be an error in our support versions documentation, Android 6 may not be supported by default and additional steps are required to function on Android 6 devices.
Are you able to follow the desugaring guidance at https://developer.android.com/studio/write/java8-support#library-desugaring and let me know if this resolves the issue? Note that the instructions are for native Android applications, but you should still be able to update the dependencies as required.
Do you see the same crashes on Android 7?

We've not had a response here so I'll close this, if you need further support please get in touch at https://support.pusher.com/hc/en-us/requests/new


Consul agent fails to start when confab's node.external_ip is null

Running in a deployment with dynamic networks, the rendered confab.json looks like this:
{
   "consul" : {  },
   "node" : {
      "name" : "kube-node",
      "external_ip" : null,
      "index" : 0
   }
}

When the external_ip is null, the bind_addr option in the generated consul configuration is set to the empty string. This forces consul to attempt to calculate the advertise address by looking at the network adapters with addresses in the private IP space.
If multiple adapters exist with private IPs, consul fails with the following:
==> Starting Consul agent...
==> Error starting agent: Failed to get advertise address: Multiple private IPs found. Please configure one.

The multiple adapter scenario was encountered while doing a rolling deployment of a bosh job that contained bridge interfaces for overlay networks and containers.
When the spec.address from bosh is null, it seems like the bind or advertise address should be calculated by connecting to the consul server and using the local address of the connection. This would avoid the problem while still using spec.address when bosh has that information.

@sykesm
There's some discussion in https://github.com/cloudfoundry-incubator/consul-release/issues/32 about a possibly upcoming spec.ip method provided by BOSH.  Would that solve your problem?
/cc @cppforlife

@Amit-PivotalLabs It depends on how it's done and how it works. If there's only one dynamic network, it will probably be fine. If there are multiple dynamic networks and you chose the one associated with the default gateway, it might.
At the end of the day, something needs to choose the network(s) and the addresses. This is why we suggested you dial the consul server to find the local address.

I'd prefer to know the exact requirements in addition to the proposed solution.  Given what I think your requirements are, having a BOSH-provided helper would be preferable.  It sounds like the problem you're describing might impact things other than Consul, and might make more sense at the BOSH level.  Have you discussed this with the BOSH team at all?  Are they aware of your exact requirements?

With bosh "v1", my understanding is that you have access to all of the networks via spec.networks so once the IP is available, you'll still have the problem of choosing whether you listen on all networks or deciding which one.
For bosh "v2", links can be associated with a network so, if links are used, you should be covered.

Closing in favour of #32

@sykesm I m planning to add network: blah on the job similar to links for the operator to specify which ip to expose to the job templates and continue defaulting to default: gateway picked network.


Improve when certain actions run

Our actions seem to have a lot of duplication in when they run. For example, it'd be great if we could some how hash or cache the last time our integration test ran, and only run them when code changes. For a lot of these PRs, we have them running even when no code has been updated (build tooling has changed, though)

@aulneau Would you mind creating a new issue in the DevOps repo for this so we can prioritize this on our board?


Create IfC vs IaC documentation

Based on feedback from new prospects (IAG, MongoDB, Oracle, etc) and inspired by the detailed documentation at https://www.twingate.com/docs/how-twingate-works, update docs with detailed Nitric documentation included a IfC vs IaC page to educate new folks to the concept and Nitrc.

Here are some articles that could be helpful to look back at for ideas:
https://nitric.io/blog/iac-vs-ifc
https://poetryincode.dev/product-engineering-with-ifc
https://thenewstack.io/infrastructure-as-code-is-dead-long-live-infrastructure-from-code/
https://serverlessfirst.com/emails/infrastructure-from-code-vs-infrastructure-as-code/
https://asher-sterkin.medium.com/cloud-application-infrastructure-from-code-ifc-398140babf13
https://yehudacohen.substack.com/p/exploring-the-emerging-cloud-development

iac vs. ifc is a search query we've gotten some traffic for. We could likely get more traffic on this query over time, so it might be worth constructing this page with that particular phrase in mind.


A few fixes to the style etc of the ggplot2 cheatsheet

The meat of this PR is the removal of the old style ..var.. notation in favour of after_stat(var).
Further, it removes lineend and linejoin from the list of aesthetics since they are really not relevant there.
Additional considerations I did not add in this PR:

All mapping should be moved to geom_sf()/coord_sf() (relevant for geom_map(), coord_quickmap(), and coord_map())
We have never fully embraced the vars() notation for facets in interactive use - both me and Hadley continue to use formula notation. Nothing wrong with using vars() in the cheatsheet but we don't really advocate it elsewhere
There is no mention of scale_linewidth() but it would take up some additional space in an already packed column and wouldn't bring much new to the table

Addressed vars(), did not address mapping and linewidth scale in this PR, will leave these for next round of updates (tracking in #310)


UAVCAN - update ROMFS_FW_PATH to /etc/uavcan/fw. to match documentation

The UAVCAN firmware update PATH changed and does not correspond to the documented one here. Is there a reason for that?
Should we change the path here or the documentation?

There's a sequence where the factory firmware will be copied to the SD card. But if the SD card contains an updated version that version will be used. I would just check this change and make sure it hasn't broken that functionality

Looks good. https://github.com/PX4/Firmware/blob/74990cadc8fe61bf541c2ec6fa29db3b11dd1feb/Tools/uavcan_copy.sh#L2
This should probably be baked into the build system.


Hash Time Lock Contracts

This PR is a first step towards more interesting verifiers and cross-chain atomic swaps. Already there are enough verifiers to support swaps between two Tuxedo chains with the HTLC verifier :rocket:
Before we can realize the goal of serving on the arbitrating side of the Farcaser protocol we would need Either a verifier that supports bitcoin script, or dedicated farcaster verifiers, both of which are feasible and put off until a future PR.
Some references I followed:

https://en.bitcoin.it/wiki/Hash_Time_Locked_Contracts
https://lightning.network/lightning-network-paper.pdf Section 4
https://arxiv.org/pdf/1801.09515.pdf Section 4.1
https://www.youtube.com/watch?v=tcZGf8Qb_XA
https://www.youtube.com/watch?v=8FeAXjkmDcQ

Changes to the Verifer trait
First, we've made a small change to the verify method where we also pass in the block number. This may seem like the beginning of a slippery slope about what data will be made available to verifiers. I think some good boundaries are block header data and stuff that is available to bitcoin scripts.
Second, we've added an associated type Redeemer. Previously we just accepted a slice of bytes to represent the redeemer. And the first thing that every redeemer did was decode it to some expected type. By having an associated type, the executive can do the decoding and handle errors. This makes life easier for Verifier writers. See the changes to ThresholdMultisig as an example of why the new interface is cleaner.
New verifier implementations

Simple hash lock. Can be spent simply by supplying a preimage.
Simple time lock. Can be spent by anyone after a given block height.
P2PKH. Private ownership just like a simple sig check but doesn't leak the pubkey. Inspired by bitcoin.
HTLC. Has a spend path that requires a signature of the recipient and a preimage. Also has a refund path which requires a time delay and a signature by the refunder.

This will be merged or rebased after #170 goes in. Then this will just be about the hashlock timelock and HTLC.


x/tools/cmd/gopls: errors reported in package with no errors

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

$ go version
go version go1.12.5 linux/amd64
$ go list -m golang.org/x/tools
golang.org/x/tools v0.0.0-20190524210228-3d17549cdc6b

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN="/home/myitcv/gostuff/src/github.com/myitcv/govim/cmd/govim/.bin"
GOCACHE="/home/myitcv/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/myitcv/gostuff"
GOPROXY="https://proxy.golang.org/"
GORACE=""
GOROOT="/home/myitcv/gos"
GOTMPDIR=""
GOTOOLDIR="/home/myitcv/gos/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/myitcv/gostuff/src/github.com/myitcv/govim/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build230798786=/tmp/go-build -gno-record-gcc-switches"

What did you do?

Opened a file in Vim (using govim) that is part of a package that compiles and on which tests pass; received the following error message:
failed to run analyses for file:///home/myitcv/gostuff/src/github.com/myitcv/govim/cmd/govim/main.go: analysis skipped due to errors in package: [/home/myitcv/gos/src/net/lookup_unix.go:324:23: undeclared name: cgoLookupPTR /home/myitcv/gos/src/net/lookup_unix.go:124:24: undeclared name: cgoLooku
pCNAME /home/myitcv/gos/src/net/lookup_unix.go:108:23: undeclared name: cgoLookupPort /home/myitcv/gos/src/net/lookup_unix.go:96:24: undeclared name: cgoLookupIP /home/myitcv/gos/src/net/lookup_unix.go:81:24: undeclared name: cgoLookupHost]

This appears to have been introduced as part of https://go-review.googlesource.com/c/tools/+/178681
What did you expect to see?
No error message.
What did you see instead?
The above error message.

cc @stamblerre @ianthehat

Is this error message logged or shown as a diagnostic? That CL changed it so that it would not show that diagnostic message anymore.

Sorry, I forgot to include that critical detail: it's logged.

It comes through as an error message though; so my expectation was the user should know about it. i.e. there has been an error, something has gone wrong.

I encounter the same error. It occurs when I freshly start vim. If I leave the file/buffer open for a while it starts working after a minute or so. I haven't found the right timing yet. I just opened the buffer switched to this tab to describe the problem, went back to verify the error and the error was gone. I tried that a few times and it seemed to solve the error. Could it be that the server needs a while to start?
I found this recent change which you labeled as "fixing a race condition" and it is concerned with package scanning. If I install gopls before that commit, the error is gone.
https://github.com/golang/tools/commit/d532c0723df8fac5ff45eea34e5435634ba6ada5
My setup is neovim with languageClient-neovim and gopls. If I can help in any way hit me up!

I think the other point to note, at least in my case, is that there should not be any type checking errors. The package in question compiles without error.

Exact same issue here with latest version from master. Package compiles, but I'm getting the same error message from gopls.

[Error - 上午9:17:45] failed to run analyses for file:///Users/elvizlai/xxxx/main.go: analysis skipped due to errors in package: [/usr/local/Cellar/go/1.12.5/libexec/src/net/lookup_unix.go:324:23: undeclared name: cgoLookupPTR /usr/local/Cellar/go/1.12.5/libexec/src/net/lookup_unix.go:124:24: undeclared name: cgoLookupCNAME /usr/local/Cellar/go/1.12.5/libexec/src/net/lookup_unix.go:108:23: undeclared name: cgoLookupPort /usr/local/Cellar/go/1.12.5/libexec/src/net/lookup_unix.go:96:24: undeclared name: cgoLookupIP /usr/local/Cellar/go/1.12.5/libexec/src/net/lookup_unix.go:81:24: undeclared name: cgoLookupHost]
[Trace - 9:17:54 AM] Sending request 'textDocument/hover - (4)'.

@elvizlai
See https://github.com/golang/go/issues/29202#issuecomment-496007059

I believe that these errors are related to build tags / CGO, which is why the package compiles but still has errors. They didn't appear before because of a race condition. However, these errors are being logged, rather than published as diagnostics, so I'm wondering why they are being shown to the user? I would not expect these to be user facing errors.

@stamblerre

I believe that these errors are related to build tags / CGO

I think so too, occur only on darwin.

However, these errors are being logged, rather than published as diagnostics, so I'm wondering why they are being shown to the user?

At least the autozimu/LanguageClient-neovim will display internal error if server side has error. This implements different to show diagnostic. Therefore user shown(maybe)
Note that I don't know vscode behavior.

I believe that these errors are related to build tags / CGO, which is why the package compiles but still has errors

Is this being tracked somewhere? Because there should be no errors, at least in the instance I reported here.

However, these errors are being logged, rather than published as diagnostics, so I'm wondering why they are being shown to the user?

If an error is reported by gopls, what is the expectation on the client?
An error of this sort suggests to me (although I'm hoping the spec is more prescriptive) "there was a problem with the language server, you should know about this"

@myitcv

Is this being tracked somewhere? Because there should be no errors, at least in the instance I reported here.

See

https://github.com/golang/go/issues/29202#issuecomment-495926430
https://github.com/golang/go/issues/31705

There appears to be a lot of Mac discussion there; I'm on Linux. Is this the same issue?

So there is a window/logMessage call and a window/showMessage call. In this case, we are just logging a message, so I would not expect it to be shown to the user. We actually log errors a lot in gopls, in cases where we would want to see this information in logs, but it's not necessary for the user.

Yes, it should be the same error. Any code that depends on packages with build tags will probably cause this error.

@myitcv
Ah, sorry for confusing 🙏

@stamblerre
FYI, autozimu/LanguageClient-neovim is incorrect(see below) implemented window/logMessage.

https://github.com/autozimu/LanguageClient-neovim/blob/6ee4c8994ee84609f8370aa4766b66c7cbad1424/src/language_server_protocol.rs#L1797-L1809

self.vim()?.echomsg(&msg) will be shown any error msg to vim users with echo area. therefore vim users confusing.
This API might be interpreted differently depending on the editors behavior.
In vim behavior, vim can see echo area log.
If use echo command, not stock any msg to echo log. If use echomsg, vim will stock any msg to echo log. (maybe @myitcv already know)
So, That behavior might be correct on vim. It is both logMessage and showMessage on vim.

Sorry noisy.

That most recent commit also fixed an issue I was about to report; I have an export_test.go file I use to export things to other packages for testing (similarly to what's done in the Go tests themselves).
Before that commit, files which used functions exported from export_test.go would show as errors, but now work correctly. This scenario used to work a few weeks ago but broke fairly recently (and is now back to working). Things exported in _test.go files might be something to test in addition to build tag differences.

Hmm, after doing more work the issue came back. Maybe it wasn't fixed and that commit wasn't related after all; I'll try to put together a repro and file a new issue.

@stamblerre

So there is a window/logMessage call and a window/showMessage call

Perfect, thank you, that's exactly the distinction I was looking/hoping for.
@zchee - I think the LogMessage client method is, as @stamblerre kindly pointed out, just that, for logging a message, with a given level of severity. In govim I am simply going to have that message be added to the govim log. From Vim, the user can ask "where is the govim log?" but I'm not actively going to surface any govim/gopls log messages.
However the ShowMessage method will require some thought (for now gopls does not call this method). In this first instance I think it would be acceptable to echom the message, with a highlight colour appropriate for the severity, for example for an error:
echohl ErrorMsg | echom "message here" | echohl None


Rename class HueSensorData?

As title, could rename to class HueData or HueRemoteData?

Even better @robmarkcole, we can totally remove that data object, as Paulus pointed on https://github.com/robmarkcole/Hue-sensors-HASS/pull/226.
I'll move the changes from my POC of the remotes on main hue when I have some free time


Display the coverage percentage

I want to display the coverage percentage in the status bar at the bottom of the editor

Hi @kentaro-m! Are you planning to work on that in the next weeks?
Thank you very much for you nice package by the way ;)


Please properly credit serde_yaml's authors

Running a quick diff against serde_yaml, this project seems to be a copy of dtolnay's crate with the changes being limited to:

find-and-replace serde_yaml to serde_yml
slight changes to rustfmt settings
loads of AI-generated "documentation" that is verbose at best and straight-up incorrect at worst
a couple new test cases

The README has a tiny note at the bottom that thanks dtolnay for "inspiring" it. Clearly, this repository was more than inspired by serde_yaml. Please give proper, prominent (i.e. at the top of the README) attribution to those who actually wrote the code.
The same goes for your website, where you take falsely take credit for the library.

@caelunshun thanks for your feedback, more than happy to make any changes both to the library and the website and give the full and complete credits.

@caelunshun
I have added the below:
Credits

This library is a fork of the excellent [Serde YAML][17] library by [David Tolnay][16].
David deserves the lion's share of the credit behind the work he has done on his library.
The intent of this repository is purely to keep an active version of his library and continue my learning into Rust.
For those migrating from the orginal version of [Serde YAML][17], please be aware that this Library is not intending to replace at all the original one.

Let me know if you have any other suggestions you would like me to add

Thanks!


Set up continuous integration tests

I've been running the test suites manually before deploys, but should push this to Travis or similar to do it automatically.

Getting the Elasticsearch tests working was tricky, but finally passing now!


Symbols are duplicated

monaco's language service needs to be disabled when the json extension is in charge, otherwise symbols are provided twice. we see duplicates in Theia's outline view.

the old language services are now removed in theia.


Switch to static MethodHandle

Avoids the reflection overhead on each call.

I can throw in a dedicated unit test if you prefer.

@karussell done :wink:

This may also benefit GraalVM compilation. Static method handles are supported.


mexNormalizeHistogramChar.cpp

Is it possible to obtain mexNormalizeHistogramChar.cpp file?
This repository contains only compiled version (mexNormalizeHistogramChar.mexa64) which does not work for Windows and Mac... :/

Hi. The file comes from DSP-SIFT and only DSP-SIFT needs it. I do not actually compile it. Please ask DSP-SIFT author for the file.


for some reason there is no icon in the menu

Geode Issue

[X] I confirm that this bug is NOT related to a mod but directly to Geode Loader itself.

Platform
Windows
SDK commit
No response
Geode Version
latest
Mods Installed
none i tried reinstalling gd
Expected Behavior
to show the icon and work
Actual Behavior
does not show the icon
Steps to Reproduce
idk
Additional Information
No response

Same problem here, It ocured after upgrading to Windows 11

если вы недавно обновили GD

thanks

Yeah! me too... i just recently updated it... that might be the problem (contact the discord cuz i dont have a discord)

it's because of the 2.206 update

same

just was thinking what if as a temp fix (and a sucky compatibility fix for old as heck mods) the installer could just change the version installed like just reinstall old vers?


Idea: suggest the right address

I am working with many addresses too, some problem I face almost everyday is wrong address (district-amphoe-province-zipcode dont match)
When the address is wrong, some courier does not accept the request so I can not get the tracking address
It would be great if this repo can suggest that too
(I've done some of work about this, some case can be predict while some case are not)
thanks!
a 425degree fan :)

That's a great suggestion! Should we do a function to see if a given address match or doesn't match the pattern? Or you want something more complicated e.g. suggesting correct address. I have to do some prototyping to see if it works tho.

@chaya-tan so we just updated the parser which should work quite well now! I might work on the correction soon. Let's chat on how to get data that need correction on (district-amphoe-province-zipcode that doesn't match) so that I can build the library around those examples?

@titipata I'd love to!


[Flake]: Timeout while building docker image.

Two builds failed.  They timed out while building the docker image:
https://source.cloud.google.com/results/invocations/df4ae4d5-a5a4-44f4-a3dd-b0b830c51a54
https://source.cloud.google.com/results/invocations/cf482fc7-d267-47f5-b639-89f9dcce27f7
The log is empty because we do not print the log of this phase unless it fails, and the timeout is managed outside our script.  We should change the script to timeout the docker image creation within 60 minutes (that is a rough guess, it should be more than 30 minutes, but less than the 120 minutes for the full build).

The Ubuntu servers seem to be really slow today. Is it viable to look into using an alternate download mirror?


Separate toLMDB into create and add to LMDB

Resize of lmdb also possible? See https://github.com/lmdbjava/lmdbjava/issues/104 Otherwise maybe not needed.

done


Add version sensor

Description:
The topic about displaying the current Home Assistant version as sensor is coming up again and again. There are multiples solutions available to do but this sensor may be the most efficient one.
Successor of #8224.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#3163
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: version

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Good point.

Instead of setting the sensor state in the tests, mock out the version, set up the sensor, then just get the state and check that it's equal to what had been mocked.


Add template for metric-federation Thanos Ruler

This MR adds a new template that deploys another instance of Thanos Rule. This Rule instance will be used to federate certain metrics from the MST instance to the Telemeter instance.

Looks good, but how we make sure we only install that in one namespace? (not in both MST and Telem)?

That's why we created a new template. We will deploy this new template in only the telemeter namespaces (like we do for conprof and jaeger).


Add support for touch events

It's not really mouse related, but didn't know where else to open this issue.
Add support for touchstart, touchmove, touchend global events.

I think we want to keep it all about mouse events. I recommend using ports for touch events for now.


Unable to escape characters for string literals[BUG]

Describe the bug
Unable to escape characters for string literals
Or for query issues:

Run the query SELECT 'i\'m'
See unexpected result i\'m

Expected behavior
The result should be i'm
Screenshots

This issue affects TDVT tests:

exprtests/standard\setup.logical.txt

Adding more info to this bug:

PPL and SQL both affected
character escaping rules are not applied to all string literals

Some examples:
SELECT 'Pat O''Hanrahan & <Matthew Eldridge]'''

returns
Pat O''Hanrahan & <Matthew Eldridge]'' instead of Pat O'Hanrahan & <Matthew Eldridge]'
SELECT * FROM staples WHERE `Customer Name` = "Mary O'Rourke";

returns valid result, but
SELECT * FROM staples WHERE `Customer Name` = 'Mary O\'Rourke';

yields empty result set.
In case of PPL
source=staples | WHERE `Customer Name` = "Mary O'Rourke";

returns valid data, but
source=staples | WHERE `Customer Name` = 'Mary O\'Rourke';

doesn't.
staples table is a sample data from TDVT suite.

Adding the following test to sql/src/test/java/org/opensearch/sql/sql/parser/AstExpressionBuilderTest.java will also reproduce the problem.
 @Test
  public void canBuildEscapedStringLiteral() {
    assertEquals(
            stringLiteral("O'Rourke"),
            buildExprAst("'O\\'Rourke'")
    );
  }

Tested and I think this is already fixed in https://github.com/opensearch-project/sql/pull/696:
POST _plugins/_sql
{
  "query": """
    SELECT 'O\'Rourke'
  """
}
{
  "schema": [
    {
      "name": """'O\'Rourke'""",
      "type": "keyword"
    }
  ],
  "datarows": [
    [
      """O\'Rourke"""
    ]
  ],
  "total": 1,
  "size": 1,
  "status": 200
}

Fix for escape characters is coming in https://github.com/Bit-Quill/opensearch-project-sql/pull/159. It is a breaking change, so it is targeted to 3.x branch.


Project Pages

[ ] Render descriptions with CommonMark
[ ] Active vs inactive/retired.
[ ] User list
[ ] Link Repo
[ ] Link PM
[ ] Tags list

Not required for 1.0.0.

[ ] Editable by PM and privileged users.

[ ] PM
[ ] Active Status
[ ] Description
[ ] Repo Link
[ ] Tag list

[ ] Join
[ ] Leave

The initial implementation should not worry too much about the join/leave functionality as that's dependent on enrollment functionality (#146)


printf printers for classes.

in addition to these stream operators, it would be nice to have an output option that would dump using printf so that we could use it on device. For instance a printRow( INDEX_TYPE const row ).
Originally posted by @rrsettgast in https://github.com/GEOSX/cxx-utilities/pull/164

And redo the input/output tests.


feat(Radio): radio button base implementation

Radio
A basic radio button.
TODO

[x] Conformance test
[x] Minimal doc site example
[ ] Stardust base theme
[x] Teams Light theme
[ ] Teams Dark theme
[ ] Teams Contrast theme
[x] Confirm RTL usage
[x] W3 accessibility check
[ ] Stardust accessibility check
[ ] Update glossary props table
[x] Update the CHANGELOG.md

API Proposal
Default

<Radio label="Make your choice" />

  <input type="radio" />
  Make your choice

Regarding the label click, it would be ideal if we can get the input to be a child of the label.  This will avoid a lot of complexity in the component that we face over at Semantic UI React.

actually, almost overlooked Levi's comment about reusing Input component - yes, absolutely agree, we should do it this way


⚠ Samsung has degraded performance

In 016a745, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2521 ms

Resolved: Samsung performance has improved in 6b91047.


Support Arrays for the Map scalar functions

Which issue does this PR close?

Closes #11436
Rationale for this change
What changes are included in this PR?
Add new function to handle support arrays for the Map scalar function.
Are these changes tested?
Existing testcases
cargo test

Are there any user-facing changes?
No breaking changes

Sorry, this change took more time than expected. The good thing is I deepened my understanding of Arrow.
@goldmedal @jayzhan211 requesting you help to review this change.

I hope to review this later today

Thanks @dharanad  and @jayzhan211

Thank You @alamb @goldmedal @jayzhan211 @Weijun-H .
I deepened my understanding on Arrow and had fun working on this issue.

Thank You @alamb @goldmedal @jayzhan211 @Weijun-H . I deepened my understanding on Arrow and had fun working on this issue.

That describes almost all of my experiences working on this project. I am glad others get the same thrill


Fix error about symbolic link

Fix the error below

$ mkdir testdir; echo "print 'hello'" > testdir/test.py; ln -s test.py testdir/test; python testdir/test
/Users/meng/.pyenv/libexec/pyenv: line 43: cd: test.py: Not a directory
hello

There is another copy of the function in python-build. It'd be better to update both.

https://github.com/yyuu/pyenv/blob/v20151210/plugins/python-build/bin/python-build#L60-L72

Please send similar fix to https://github.com/rbenv/rbenv and https://github.com/rbenv/ruby-build

@mengbo Could you rebase to latest master to fix conflict?
And, it's really helpful for both rbenv and pyenv if the abs_dirname can handle filename without / (==> a file in current directory). Do you have any idea to implement it?

This also fix https://github.com/yyuu/pyenv/issues/580

This fixes #588 as well. Thanks

Any chance this can be merged?


Error when adding second debt type: private parking charges debt

Error
<EMAIL_ADDRESS>Error
Retry
Interview has an error. There was a reference to a variable 'debt[1].emergency_dict_temp' that could not be looked up in the question file (for language 'en') or in any of the files incorporated by reference into the question file.
History
Ran initial code at 0.00208s
id: set currency
initial: True
code: |
set_locale(currency_symbol='£')
Tried to run mandatory code at 0.00209s
id: interview order
mandatory: True
code: |
user_saw_introduction
user_saw_preparation
debt_signpost
user.name.first
user.name.last
if user.dob.plus(years=18) > today():
under18_kickout
if not user.uk:
nonUKresident_kickout
if not user.uk_debt:
nonUKdebt_kickout
user.work
user.uk_debt
#user_provided_personal_info
user.house_status
if user.house_status == 'Renting':
if user.renting_status == 'Joint tenants':
user.house_status_rentshare
if user.house_status == 'Home owner':
if user.homeowner_status == 'Joint Mortgage':
user.house_status_mortshare
if user.house_status == 'Living with family':
share_housing = True
if user.house_status == 'Homeless':
share_housing = False
user.house_num = 1
user.house_aware = False
if share_housing == True:
user.house_num
else:
user.house_num = 1
if user.house_status == 'Homeless':
user_contact_information_homeless
else:
user_contact_information
user.marriage
current_insolvency
debt_info_signpost
user_identified_priority_debts
user_identified_nonpriority_debts
debt_parent_list[0]
debt.total()
finance_signpost
jobs.total()
other_income.total()
assets.total()
add_review_to_menu = True
expenses.total()
documents_uploaded
summary
signpost_bs_solutions
user_saw_standard_bs
user_saw_mentalh_bs
user_saw_debt_solutions
user_signed
predocx_variables
letter_and_end
Needed definition of debt[1].complete at 0.00223s
Tried to run block at 0.00239s
id: Debt complete order
generic object: Debt
code: |
x.type
x.value
x.creditor
x.name
if temp_other_name_exists:
x.other_name = temp_other_name
x.urgency
x.priority
x.court_action
x.first_miss_date
x.notes
x.complete = True
Needed definition of debt[1].urgency at 0.00251s
Tried to run block at 0.00266s
id: calculates .urgency from .urgency_reason if appropriate
generic object: Debt
code: |
if len(x.emergency_dict.true_values()) > 0:
x.urgency = x.emergency_dict.true_values().comma_and_list()
else:
x.urgency = "None"
Needed definition of debt[1].emergency_dict at 0.00277s
Considered using block at 0.00292s
if: number_of_emergency_choices > 1
generic object: Debt
question: Emergency information about ${ x.name }
subquestion: |
Are you at risk of any of the following?  Check any that apply.
fields:

no label: x.emergency_dict
datatype: checkboxes
code: |
x.emergency_choices
Tried to run block at 0.00295s
sets: x.emergency_dict
generic object: Debt
code: |
x.initializeAttribute('emergency_dict', DADict.using(auto_gather=False,gathered=True))
x.emergency_dict[x.emergency_choices[0]] = x.emergency_dict_temp
Needed definition of debt[1].emergency_dict_temp at 0.00311s
Show variables and values

occured again with personal loan debt

I think I fixed this by commenting out an old code block.  I need to go through and clean out the code.  I think ALKiln should be working, so hopefully that catches problems if I do that.
It seems to be working but I should check again in the morning.
In emergency_dict_type branch - 4d6ad206d6c4c1be09145c256fd827f3503c9ff4


The warning on the "Start a new React Project" page may be outdated

From the nextjs website, I think the below warning isn't true anymore.

Next.js's App Router is currently in beta and is not yet recommended for production (as of Mar 2023). To experiment with it in an existing Next.js project, follow this incremental migration guide.

Page link: https://react.dev/learn/start-a-new-react-project
File link: https://github.com/reactjs/react.dev/blob/main/src/content/learn/start-a-new-react-project.md

#6005

So why it is not solved till now ?? #6274
It is not something Big, It is just a simple Pull Request.
I sometimes feel this repo is dead.

There is a more comprehensive update needed for this page. For now https://github.com/reactjs/react.dev/pull/6318 will handle the inaccuracy as revisit the page. Thank for reporting

There is a more comprehensive update needed for this page. For now https://github.com/reactjs/react.dev/pull/6318 will handle the inaccuracy as we revisit the page. Thanks for reporting!

Thank You so much Contributor Sir 🙂


Sorting files by timeCreated

In gcloud NodeJs API client using bucket.getFiles Is there an option to sort files according to the createdDate or modifiedDate? I referred following link, Gcloud NodeJs Docs, I couldn't find any such option.

I don't believe there is, but if the upstream Storage API allowed sorting options, that would be a nice feature to have.
@Capstan -- is there any place we can make feature requests for the Storage API?

I'll have to close here, but maybe @Capstan heard your suggestion and we'll see it implemented some day! :)

And no, we don't have a public feature request location. Alternate sort orders are requested periodically, but we have no committed plans to introduce it.


Crystal dependency error

I updated crystal to the latest version ( using crystal website instruction for ubuntu https://crystal-lang.org/install/on_ubuntu/)
$ crystal -v
Crystal 1.1.0-dev [e7b46c407] (2021-03-30)

But when trying to update kcr I got the following error:
$ kakoune.cr git:(master) make install
make: llvm-config: Command not found
shards build --release --ignore-crystal-version
Resolving dependencies
Fetching https://github.com/alexherbo2/fifo.cr.git
Fetching https://github.com/alexherbo2/rsub.cr.git
Unable to satisfy the following requirements:

- `crystal (< 1.0.0)` required by `fifo 0.1.0+git.commit.a5df8025fd3ab182ae2511d525f708cbbe0ed43a`
- `crystal (~> 1.0, >= 1.0.0)` required by `rsub 0.1.0+git.commit.43c6b9836ee281328bccfdf8c669bab26448e3b3`
Failed to resolve dependencies
make: *** [Makefile:12: build] Error 1

Try make clean to clean the lock file.

Worked! Thanks
But I encountered a new issue:
Error target kcr failed to compile:
Error: Invalid option: --ignore-crystal-version

pile:
Error: Invalid option: --ignore-crystal-version

I just removed the --ignore-crystal-version and it worked, I hope it is fine.
I got so lost in the dependency spiral that I don't even remember why I had to upgrade crystal xD