Update jsoniter-scala-core, ... to 2.6.3

Updates

com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-core
com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-macros

from 2.6.0 to 2.6.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala" } ]

labels: library-update, semver-patch

Codecov Report

Merging #183 (53d1b68) into master (e31fc88) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #183   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          24      24
  Lines         234     234
  Branches        9       9
======================================
  Misses        234     234

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 e31fc88...53d1b68. Read the comment docs.

Codecov Report

Merging #183 (53d1b68) into master (e31fc88) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #183   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          24      24
  Lines         234     234
  Branches        9       9
======================================
  Misses        234     234

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 e31fc88...53d1b68. Read the comment docs.

Superseded by #190.


Buildsystem update

Fixed building errors for windows
Added the option to build for debian systems

The WASM build is failed is failed, could you please check it?

Thank you for the changes!


Fix code example in remote-config.rst

Fixing code example With Dependency Injection

Ooops, damn you copy & paste 😅
Thank you! 🌺


Torrentio stream list not interlocked

Torrentio addon stream list

MediaFusion addon stream list
I understand that MediaFusion is linked to the stream list of Torrentio.
However, after installing Torrentio addon as shown in the picture, the stream list and the MediaFusion stream list are different.
Because of this, I'm having a hard time using it.

Hi @hsp1020. As a discussion with Torrentio dev, I've decided to disable scraping streams from Torrentio on the official user-facing website https://mediafusion.fun.  The previously scraped data will remain as-is, but will be gradually replaced by Prowlarr indexers.
The good news is that we've integrated Prowlarr indexers, except for TamilBlasters and TamilMV. The Prowlarr indexer data will be available.
Alternatively, you can set up the MediaFusion addon completely locally with Torrentio stream scraping enabled. (It's up to you for not overloading the Torrentio 😅😅😅)

If it's right to set Torrentio stream scraping enabled like a screenshot, I've already set it up that way and I'm using the media fusion add-on. Still, the stream list doesn't pop up.

If it's right to set Torrentio stream scraping enabled like a screenshot, I've already set it up that way and I'm using the media fusion add-on. Still, the stream list doesn't pop up.

I've already set it up that way and I'm using the media fusion add-on. Still, the stream list doesn't pop up.

Have you setup the mediafusion addon on local?  not using the https://mediafusion.fun/

Just accessed https://mediafusion.fun/ and Torrentio Stream enabled and reinstalled. But things are the same. What am I supposed to do?

"As a discussion with Torrentio dev, I've decided to disable scraping streams from Torrentio on the official user-facing website https://mediafusion.fun/. The previously scraped data will remain as-is, but will be gradually replaced by Prowlarr indexers."

as i mention earlier, the https://mediafusion.fun/ site won't give any new torrentio links. That feature is disabled by default. You have to set up this Add-on completely in your local to configure as your wish.
To set up the add-on locally, please wait until i release the detail documentation.

When are you going to release the detail documentation?

here is the detailed guideline https://github.com/mhdzumair/MediaFusion/blob/develop/deployment/README.md 🤖

I just saw the detail guidelines you posted. but I'm an Android version user. Is there no way to enabled Torrentio link on Android at all?

You can set up the add-on on any VPS server. After talking, I decided not to use the Torrentio scrap feature on https://mediafusion.fun because it makes torrentio extra load. The prowlarr scrapper adding more torrent by every seconds.


[ShallowMount] Cannot click on Vuetify icon when stubbing VIcon

Version
1.0.0-beta.24
Reproduction link
https://codesandbox.io/s/61jzxnovnk
Steps to reproduce
Trying to test click when using VIcon
I reproduce some scenarii :
it works when shallowMounting without using Vuetify, it works when mounting using Vuetify
I doesn't work when shallowMounting with Vuetify when selecting Component, by ref, by name
What is expected?
a way to trigger click on stub when shallowMount
What is actually happening?
it doesn't work

A temporary workaround is to stub the component manually:
const Stub = {
  template: '

}

const wrapper = shallowMount(TestComponent, {
  stubs: {
    'VIcon': Stub
  }
})

wrapper.find(Stub)

@eddyerburgh is this what you were thinking for a workaround? https://codesandbox.io/s/42rxjjvz44
Not sure if I implemented your suggestion how you intended.

@eddyerburgh : to me, the issue is not fixed in beta 25.
On my CodeSandbox, I update VueTestUtils version 25, it doesn't solve anything
Are you sure, this issue is fixed?

Confirming. v25 didn't fix stubs problem

@eddyerburgh is it possible to reopen this issue?

@eddyerburgh , thank you for your suggestion, but it does not help in the case, when there is a click handler passed as prop to parent component:

    I am parent
    <v-btn @click="callbackFromProp"></v-btn>

export default {
    props: {
        callbackFromProp: Function
    }
}

@p1pchenk0  In that case you need to supply a stub that calls the listener on the root element:
const wrapper = shallowMount(TestComponent, {
  stubs: {
    'v-icon': {
      template: '
'
    }
  }
})

wrapper.find('.icon').trigger('click')

I don't think we should add all listeners to the auto stubs. With normal components we use the on value of the component that's being stubbed, but not all components expose this info.

I'm going to close this issue, since it's not a bug but a question. But let me clear up some misconceptions.
shallowMount only stubs components that are registered. This is to avoid false positives where components are stubbed and rendered in tests, but can't render in production.
The only guaranteed way to select components that have been stubbed is to use a component selector:
const wrapper = shallowMount(TestComponent)
wrapper.find(Icon)

Or a ref selector:
const wrapper = shallowMount(TestComponent)
wrapper.find({ ref: 'icon' })

If you don't have access to the component, you must use the stubs option:
const Stub = { template: '
' }

const wrapper = shallowMount(TestComponent, {
  stubs: { icon: Stub }
})
wrapper.find(Stub)

If anyone is still trying to figure this out and is confused by the conversation in this issue, I found the solution in the documentation. In short, instead of triggering through the wrapper API like this:
wrapper.find(ChildComponent).trigger('click')
you should call the vue instance $emit function like this:
wrapper.find(ChildComponent).vm.$emit('click')
Hope that helps.

If anyone is still trying to figure this out and is confused by the conversation in this issue, I found the solution in the documentation. In short, instead of triggering through the wrapper API like this:
wrapper.find(ChildComponent).trigger('click')
you should call the vue instance $emit function like this:
wrapper.find(ChildComponent).vm.$emit('click')
Hope that helps.

I've used this solution before, but it doesn't seem to work now. On further inspection I noted the following:
// Default stub of a v-text-field (Vuetify) child component
const vTextFieldStub = wrapper.find('v-text-field-stub[name="password"]')

// Output: `Wrapper { isFunctionalComponent: true }`
console.log(vTextFieldStub)

// Output: `undefined`
console.log(vTextFieldStub.vm)

I can't emit the click event because vm is undefined.  How can I emit an event on functional components?

I think this problem is not solved. could you reopen ?

in my case, I use @click.native Instead of @click it works.

I have a similar issue with .trigger('click') not dispatching a click on a VBtn in a fully mounted component. So instead of wrapper.find(selector).trigger('click') and then asserting that the click called a method, I am dispatching a classic browser click event, eg:
document.querySelector(selector).dispatchEvent(new Event('click'))
Doing it this way makes my assertion pass as expected.

I have the same issue as well (in beta.24)
In my component, the child component I'm targetting is a functional component, and so it can't trigger its custom event by using vm.$emit.
const wrapper = shallowMount(MyComponent);

// ChildComp is a functional component
const childWrapper = wrapper.find({ name: 'ChildComp' });

// cannot trigger with this method, as `vm` is undefined
childWrapper.vm.$emit('customEvt'); // ERROR: cannot read property $emit of undefined

I am running beta 34 and I can't reproduce any of the solutions.
https://vue-test-utils.vuejs.org/guides/#emitting-event-from-child-component
This does not work, the parent component doesn't receive any events and the event handlers don't fire at all.
@lukaVarga document.querySelector(selector) returns undefined for me..
My test:
    it('increments the internal counter when the + button is clicked', () => {
        const wrap = mount(NumericSpinner)
        wrap.find(SpinnerValueChangeButton).vm.$emit('click')
        console.log(wrap.emitted())
    })

The console.log() outputs [Object: null prototype] {}

@ccarstens beta 34? Do you mean beta 31 (which is the latest version atm), or beta 24?
Regardless, for document.querySelector(foo) to return something, you have to pass the attachToDocument option (see https://vue-test-utils.vuejs.org/api/options.html#attachtodocument) when mounting the component.
I'm currently running beta 31 and I can do wrapper.find(foo).trigger('click') and it works fine for me for all use-cases (including triggering it and checking that the handler was invoked on eg. a Vuetify component, so <v-btn @click="doSth"></v-btn>, or on a classic html element, like  for instance).
After having upgraded from beta 29, I in fact had to adjust tests to not use the  method I mentioned in my previous comment (document.querySelector(selector).dispatchEvent(new Event('click'))) anymore and had to change it to wrapper.find(selector).trigger('click')

Usually i have a lot of same child components, like buttons, so the implementation wrapper.find(ChildComponent).trigger('click') is not good for that case

If anyone else stumbles on this thread looking for a way to use shallowMount yet still interact with simple ui-components like buttons, this worked for me:
            import ComponentBeingTested from 'Component.vue';
            import ButtonComponent from 'ButtonComponent.vue.';

            ....

            // In a test:
            const wrapper = shallowMount(ComponentBeingTested, {
                stubs: { ButtonComponent } // use real implementation for this "stub"
            });
            wrapper.find('.first-button-component').trigger('click');
            await wrapper.vm.$nextTick();


Add ability to specify headers on introspection requests

The GraphQL API I'm introspecting requires an API Key to be provided via the request headers. Is there a way to do this currently?

I see there's some options to provide an Authorization header, but in our case, we'd need to be able to specify the header key arbitrarily
https://github.com/anvilco/spectaql/blob/8943fe336beb602a6dbc34df9a83e732358f41ed/app/spectaql/graphql-loaders.js#L46-L64

I'd noticed that it was limited support in DociQL, and was waiting for the day when someone asked for more. This makes sense to me and I will add it ASAP.

Added in v0.2.2 which is now on NPM.


Querybuilder does not resolve relationship names correctly (camelCase instead of snake_case)

The following code
location = new Location({
          locationType: locationType,
          types: types,
          placeId: result.place_id,
          user: userContext?.user,
          addressComponents: addressComponents,
          formattedAddress: result.formatted_address,
          locationGeometry: result.geometry,
          jsonGeocodeResults: results,
          plusCode: objectKeysToCamelCase(result.plus_code),
        });
        const qb = em.createQueryBuilder(Location);
        try {
          await qb.insert(location).onConflict().ignore().execute<Location>();
          await em.flush();
        } catch (e) {
          console.log('ERROR inserting location', e);
          await em.rollback();
          em.clear();
          em.flush();
          console.log('Ignoring duplicate placeId');
        }

produces the following query (values omitted here)
insert into "location"
("address_components", "created_at", "data_language", "deviceLocations", "formatted_address",
 "id", "json_geocode_results", "location_geometry", "location_type", "place_id", "plus_code",
 "postcode_localities", "types", "updated_at", "user_id") values (.........)

where deviceLocations is a 1:n relationship.
Why is queryBuilder producing anything for deviceLocations where it is not even supplied and why is it using CamelCase in the name?
Could you provide a fix please soon.
We would like to use MikroORM in production soon so this is super urgent!

Dependency
Version

node
?

typescript
?

mikro-orm
4.5.9

your-driver
postgres

where deviceLocations is a 1:n relationship.

You can't select 1:m relation, it is an inverse side. Just drop it from the qb.inset() parameter.


Include Microsoft.Diagnostics.FastSerialization in the TraceEvent package

Currently the Microsoft.Diagnostics.Tracing.TraceEvent assembly depends on Microsoft.Diagnostics.FastSerialization, but no dependency was added to the NuGet package. Rather than deal with separate NuGet packages at this time, this change simply includes the dependency in the original NuGet package.
:memo: If we ever need to split these in the future, we can trivially do so and add a dependency from the current NuGet package to the new one without breaking upgrade scenarios. Since this makes distribution a little bit more complex (managing multiple NuGet releases from one repository) and didn't seem essential at this time, I chose the simplest route to packaging for my initial proposed approach.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #273   +/-   ##
=======================================
  Coverage   24.35%   24.35%
=======================================
  Files         197      197
  Lines      112394   112394
  Branches    11210    11210
=======================================
  Hits        27373    27373
  Misses      84994    84994
  Partials       27       27

Flag
Coverage Δ

#2015
24.35% <ø> (ø)
:arrow_up:

#2017
24.35% <ø> (ø)
:arrow_up:

#Debug
15.81% <ø> (ø)
:arrow_up:

#Release
14.9% <ø> (ø)
: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 3844164...78487b6. Read the comment docs.

This is reasonable, but I am mulling over whether we event want a separate DLL for the FastSerialization code (what makes this different from the other utility code that happens to be in TraceEvent.dll (e.g. PEFile  ...).    This was originally spit out to allow reuse by Visual Studio, but in retrospect, that was not a clear win.   Hmm...

@vancem If you aren't sure, we could file that as a follow-up issue to take on separately.

I have been mulling this too long.   It is now blocking someone, and so we will do it this way and we can change it later as a separate issue.


Further improved the above index.html part

Kindly add screenshots/ screen recording for ease of assessment.


Any chance of getting a new release?

#40 was merged after the current 0.16.0 release. Any chance of getting an updated release with this fix in?

Hi Kenneth,
Release 0.17.0 is now available - https://github.com/caskdata/netty-http/releases/tag/v0.17.0


[api-major] Enable editing by default

Note that this patch at least shouldn't land until both bug 1784272 and bug 1785248 are fixed.

/botio integrationtest

/botio integrationtest

/botio-linux preview


Response buffering & manipulation

I would like to add full support for response buffering and manipulation. My needs are to fully buffer particular responses dependent on findings in headers (for example: Content-Type) and possibly modify, or in some cases fully replace the entire response. Due to the nature of responses, I'd like to add (perhaps pluggable) support for encodings, compression (gzip/brotli) and chunking at a minimum.
I see in the LUA branch that there is the start of something like this but it seems very basic & in the issue tracker there is talk of needs for buffering there as well.
Was this work ever completed in any way? If not, can you provide some pointers as to how you would like to see this fit into the system?

Yes, your analysis is right, the LUA branch was a contributed PoC that is too basic to be generally useful. It lacks all the needed infrastructure changes: more complete and general HTTP parsing, request and response caching. I never did the conceptual work on how these should fit into sslsplit so this is green field. Happy to discuss specific proposals. Looking into the future, the solution should also be able to support HTTP/2.


ENH:  Initial commit.

This isn't necessarily for inclusion.  I had done some work integrating the new MeasureImageSimilarity into ANTsR thinking that this would be useful for a particular project but I decided to not work through ANTsR for now.  However, I didn't want to delete what I had done so I'm putting it here.  This commit is not going to pass because MeasureImageSimilarity not included in the ANTs core apps.

@ntustison .... is this just making a ants commit to include MeasureImageSimilarity in the ants core apps? would be very useful to have around.

I was putting together an R interface for MeasureImageSimilarity.cxx but that would require putting it in the ants core apps.  I figured that I would let people use it a bit before pulling the trigger on the necessary changes.  Are you up for making those changes now?

not up for it it now - let's give it a month, roughly

Sounds good.

superceded by jeff's recent imageSimilarity commit.


execute torq.q with q

I needed this fix to make it start from my Linux install. I don't see how launching a nonexecutable script could work anywhere, but I'm probably missing something.

The q executable is added to the start line $sline on line 62 of torq.sh based on the contents of config/process.csv. In you case it seems that qcmd in process.csv is not populated for the specific process you are trying to start and there is no fall back to a default behaviour. A fix for this could be to revert your change then add the following between lines 61 + 62:
if [ -z $qcmd ]; then
  qcmd = "q"
fi

This will default to q if no qcmd is specified in config/process.csv.
Can you try the above and let us know if you have any success?

Done. Indeed I did not specify qcmd in process.csv. I added the default you suggested. Now I also needed a fix for the kill command, my second patch. Maybe something was wrong here, but in my case the additional $ causes a problem.

I see you undid the second change with the additional $, was this issue resolved?

Yes, when using your config file template in the form localhost,{KDBBASEPORT}+2... but those scripts are fragile.


DetachAll should deal with cyclic links

It should be possible to deal with DetachAll without causing StackOverflow exceptions. One possibility: DetachAll can keep a (thread) local hash map of objects traversed so that it can prevent going into a cycle on previously detached objects. This will also ensure that each persisted object in the object graph is instantiated only once and linked to wherever it is linked to in the object graph.

I think there is a little problem with the map containing the already detached objects. The key to find the already detached object is based on ODocument hashCode(). The result of this method is not unique. This leads to some big problems, the map will return the wrong already detached object, if an other object has the same hash ->  in most cases we will have an ClassCastException.
I suggest to use getIdentity() with an ORID as key ?

I you can make an example testcase, I'm happy to see if I can fix it.

I don't think it's about a testcase. The codomain of a hash function is finite, so there might be collisions. ORID seems the best solution as @thewall13 suggested.

Every change should have a testcase. It is to prevent regressions.
Op 30 jan. 2015 09:41 schreef "Vitor Enes Duarte"<EMAIL_ADDRESS>
:

I don't think it's about a testcase. The codomain of a hash function is
finite, so there might be collisions. ORID seems the best solution as
@thewall13 https://github.com/thewall13 suggested.
—
Reply to this email directly or view it on GitHub
https://github.com/orientechnologies/orientdb/issues/2525#issuecomment-72170193
.

Yes, there should be a test-case for this function.
I don't no why there is no one or maybe I don't find him.
But in fact I agree to @vitorenesduarte that the missing test-case is an other problem. Of course it can also be fixed together with the key problem.

I think you misunderstood me. There is a testcase for detachAll with cycles, which works fine. See commit https://github.com/orientechnologies/orientdb/commit/de09f08d16a419f57b5fa44d2ed3e4cb88d1adbd
What I am asking is an additional testcase showing the issues you are having. So give us some class definitions and object instantations which cause the ClassCastException you are experiencing, as I am not having such problems.

It's not about cyclic links but about two different records having the same hashCode().
Look at hashCode() implementation in this link. It's unlikely, but as there are more inputs than outputs, collisions may happen.

So if i don't totally missunderstood than are the hashValues of cluster X on position 31 the same as the hash of Cluster X+1 on position 0.      And so    X:32   ==  X+1:1  ; X:33   ==  X+1:2   -> etc.
This seems not very unlikely. The hashCode method seems very strange for me. For every cluster position number with an size of max 32bit the hash code will be:    (31 * clusterId) +  clusterPosition.
So and for every position id bigger 32 bit it wil be something else, but this case is most of the time irrelevant.

@wouterv : Sorry that I didn't see the test. I only looked into the test-folder of orientdb-object.

https://github.com/orientechnologies/orientdb/issues/3561


Non responsive Console

I have orientdb running on aws vm.  The console.sh does not do anything after the prompt no matter what I type ( I have rebooted the orientdb server and even the vm).  The web interface seems to just run fine.  Below are the screenshots

I was able to resolve the issue by running console.sh as a superuser (sudo)


Following Location only for 201, 202 and 3xx status codes

Some domains are sending the Location header even for e.g. 200 status codes. For example the domain minutouno.com or mlssoccer.com
I think that jsoup should follow the Location header only on 201, 202 and 3xx status codes.

(Closing, outdated; only 3xx with Location are redirected)


Redeferral fix for cross-site functions

Don't let redeferral update the entry point on a function object's type if the entry point is already a cross-site thunk.

:shipit:


[docs] 段落変える

ちょっと見やすくする
https://github.com/anatawa12/AvatarOptimizer/blob/386a56b1741be09a02c003465cb1014021fd72ea/.docs/content/docs/reference/remove-mesh-by-blendshape/index.ja.md?plain=1#L15
等

ついでに: You’ll see list of blend shapes and check to select blendShapes. If some vertices in your mesh is moved more than Tolerance by selected BlendShape, this component will remove the vertices. You'll see the list ofの方が良さそう(by maskの方がそうなってた)


Where are the docs?

What options are supported by brewfile?  I can't find anything after extensive googling.  Even the main README mentions unless without describing it....
Can you point me to docs on the syntax of your DSL?

The DSL is defined at https://github.com/Homebrew/homebrew-bundle/blob/677f39d62a8aefdcbc091f193af1f7ddbee0052d/lib/bundle/dsl.rb
The individual components (cask, brew) are documented in their own repositories as bundle should be passing through any arguments or parameters as is.

Those aren't docs :(

Where are the docs?

In the README and brew bundle --help.

Even the main README mentions unless without describing it....

unless is a Ruby keyword. Brewfile is a Ruby DSL.

I think you should leave this issue open.

No, sorry, we don't leave issues open unless there's actionable work in them. As-is: there is not.

Yes, so where's the description of the DSL.  I can write a DSL in python too, that doesn't mean it's going to have the same syntax.
This is highly actionable.  There are zero docs, by your own admission.  If you'd like to keep it that way, I guess that's up to you.

Here is the output of brew bundle --help in case you were curious:
Usage: brew bundle subcommand

Bundler for non-Ruby dependencies from Homebrew, Homebrew Cask and the Mac App
Store.

        --file=                      Read the Brewfile from this file. Use
                                     --file=- to pipe to stdin/stdout.
        --global                     Read the Brewfile from ~/.Brewfile.

brew bundle [install] [-v|--verbose] [--no-upgrade]
[--file=path|--global]

Install or upgrade all dependencies in a Brewfile.

    -v, --verbose                    Print the output from commands as they are
                                     run.
        --no-upgrade                 Don't run brew upgrade on outdated
                                     dependencies. Note they may still be
                                     upgraded by brew install if needed.

brew bundle dump [--force] [--describe] [--no-restart]
[--file=path|--global]

Write all installed casks/formulae/taps into a Brewfile.

        --force                      Overwrite an existing Brewfile.
        --describe                   Include a description comment above each
                                     line, unless the dependency does not have a
                                     description.
        --no-restart                 Do not add restart_service to formula
                                     lines.

brew bundle cleanup [--force] [--zap] [--file=path|--global]

Uninstall all dependencies not listed in a Brewfile.

        --force                      Actually perform the cleanup operations.
        --zap                        Remove casks using the zap command
                                     instead of uninstall.

brew bundle check [--verbose] [--no-upgrade] [--file=path|--global]

Check if all dependencies are installed in a Brewfile.

    -v, --verbose                    Print and check for all missing
                                     dependencies.
        --no-upgrade                 Ignore outdated dependencies.

brew bundle exec command

Run an external command in an isolated build environment.

brew bundle list [--all|--brews|--casks|--taps|--mas]
[--file=path|--global]

List all dependencies present in a Brewfile. By default, only Homebrew
dependencies are listed.

        --all                        List all dependencies.
        --brews                      List Homebrew dependencies.
        --casks                      List Homebrew Cask dependencies.
        --taps                       List tap dependencies.
        --mas                        List Mac App Store dependencies.

No mention of the DSL, not even the subcommands allowed.   I'm a big fan of brew-bundle, but it's not super useful for complex cases without docs.  It's especially confusing because there are a number of outdated docs on the net for old iterations of brew bundle.

But I guess I'll start learning about Ruby DSLs since documentation doesn't seem to be a priority for you sigh.

Perhaps you can answer this at least.
Is the following section of the man pages applicable to brew-bundle?
i.e. Can I define a custom external command and call it in a Brewfile?
CUSTOM EXTERNAL COMMANDS
       Homebrew, like git(1), supports external commands. These are executable
       scripts  that  reside  somewhere  in  the  PATH,  named brew-cmdname or
       brew-cmdname.rb, which can be invoked like brew  cmdname.  This  allows
       you to create your own commands without modifying Homebrew's internals.

       Instructions for creating your own commands can be found in  the  docs:
       https://docs.brew.sh/External-Commands

Please reread our Code of Conduct and adjust your communication accordingly: https://github.com/Homebrew/.github/blob/master/CODE_OF_CONDUCT.md#code-of-conduct

How have I violated that code of conduct?  I'm sorry if I have.

I came here with an issue that you've acknowledged exists.  I've done nothing but work with you to try to resolve that problem, respectfully and openly.
I will leave now.  Good luck with whatever's bugging you.

I'm sorry if I have.

I appreciate the apology.

I've done nothing but try to work with you to try to resolve that problem, respectfully and openly.

I have not found your communication to be respectful. For example:

Good luck with whatever's bugging you.

Yes, I agree leaving is probably for the best, sorry.


Quotes to wrap XpathExpression should be kept as what it is after serialization

We recently found that all Quotes which used to wrap XpathExpression will be removed after serialization. For example, original,
leaf lf2 {
type uint8;
must '. <= ../lf1';
}
After serialization,
leaf lf2 {
type uint8;
must . <= ../lf1;
}
We double checked with RFC 7950, actually, it is valid according to the 'must' and 'when' definition.
But we still propose to wrap XpathExpression with Quotes, or at least keep as what it is. There are a few reasons,

The example in RFC 7950, (ex, must statement and when statement) all wrap with quotes. That means according to the spec, they recommend to have the quotes.
If there are WS in XpathExpression, Quotes can avoid some parse errors. Actually, pyang doesn't accept 'must' or 'when' expression without quotes if there are WS in expression. Pyang will report it as 'error: unterminated statement definition for keyword "must", looking at...'
XpathExpression with quotes has better readability

Could you elaborate on the serialization usecase? Do you read in an existing model with an existing XPath or do you create a new UnparsedXpath? Is it in the SerializationIntegrationTest?

Hi Jan,
You can refer to the test case and data in #119.
From Yuwen's iPhone
在 2018年5月17日，下午4:33，Jan Koehnlein <notifications<EMAIL_ADDRESS>写道：
Could you elaborate on the serialization usecase? Do you read in an existing model with an existing XPath or do you create a new UnparsedXpath? Is it in the SerializationIntegrationTest?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/theia-ide/yang-lsp/issues/118#issuecomment-389887897, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAxaKXTPr9zu4E5P2mkW_rzTJ3Nats9cks5tzYoogaJpZM4UC_zs.

Depends on https://github.com/eclipse/xtext-core/pull/747

Fixed

Unfortuantely, we found some other strange behaviors.

'/' position is the end of a segament. there will be an unnecessary space ' ' append into that segament. See output example.
Input: leaf lfb2 { type leafref { path "/ytest:c1/" + "ytest:l1"; } }
Output: leaf lfb2 { type leafref { path "/ytest:c1/ " + "ytest:l1"; } }
segament break at ':', whatever it is at the end or at the beginning of a segament, there will be unnecessary space ' ' append into the segament
Input: leaf lfb2 { type leafref { path "/ytest:" + "c1/ytest" + ":l1"; } }
Output:  leaf lfb2 { type leafref { path "/ytest: " + "c1/ytest " + ":l1"; } }

The space will make xpath totally wrong.

Never mind. I've added the missing case and more tests.

Hopefully fixed now. Don't hesitate to reopen if you encounter further pecularities.


Check Balanced Paranthesis added

Excellent work @punit3299
Please star this repository to show your support, in the meantime I am reviewing the code.

Thanks @amritansh22
Waiting for your review.

Excellent work  @punit3299 . Thanks a lot for making this PR.


other blocks

https://github.com/carloslfu/blocklify

https://blockly.games/music?lang=en&level=3


GWT module 'traccar' may need to be (re)compiled

I was getting license error, because I downloaded professional version of wrapper. Then I packaged again with wrapper community version. I got my new packages and server has been started. But as I load localhost:8082 into browser I got this error in my browser.
How to recompile this?

Have you compiled the GWT project?

I just compiled the project at the beginning in eclipse, I didn't compile it again after downloading new wrapper tar.gz file.

How did you compile it in Eclipse? Did you click GWT compile?

No sir! I didn't click on this option.

Well, you have to click on it to compile web interface.


tidb-lightning insert wrong value to boolean(tinyint) type with tidb backend

Bug Report
Please answer these questions before submitting your issue. Thanks!

What did you do? If possible, provide a recipe for reproducing the error.

create table t2, the column type is boolean.
mysql>use test;
mysql> show create table t2;
+-------+---------------------------------------------------------------------------------------------------------------+
| Table | Create Table                                                                                |
+-------+---------------------------------------------------------------------------------------------------------------+
| t2    | CREATE TABLE `t2` (
  `t` tinyint(1) DEFAULT NULL
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_bin |
+-------+---------------------------------------------------------------------------------------------------------------+
1 row in set (0.00 sec)

prepare import csv, with true value.
vi test.t2.csv
true
1
false
0

use tidb-lightning import the csv

What did you expect to see?

1
1
0
0

What did you see instead?

0
1
0
0

Versions of the cluster

TiDB-Lightning version (run tidb-lightning -V):

[root@10 bin]# ./tidb-lightning -V
Release Version: v4.0.6
Git Commit Hash: 12fe8b151119becc4cac0501806504e720e4e61d
Git Branch: heads/refs/tags/v4.0.6
UTC Build Time: 2020-09-16 03:45:37
Go Version: go version go1.13 linux/amd64

- TiKV-Importer version (run `tikv-importer -V`)

        use tidb backend

- TiKV version (run `tikv-server -V`):

[root@10 bin]# ./tikv-server -V
TiKV
Release Version:   4.0.1
Edition:           Community
Git Commit Hash:   78d7a854026962669ceb2ee0ac343a5e88faa310
Git Commit Branch: heads/refs/tags/v4.0.1
UTC Build Time:    2020-06-12 09:13:05
Rust Version:      rustc 1.42.0-nightly (0de96d37f 2019-12-19)
Enable Features:   jemalloc portable sse protobuf-codec
Profile:           dist_release

- TiDB cluster version (execute `SELECT tidb_version();` in a MySQL client):

| Release Version: v4.0.1
Edition: Community
Git Commit Hash: 689a6b6439ae7835947fcaccf329a3fc303986cb
Git Branch: heads/v4.0.1
UTC Build Time: 2020-06-15 03:54:18
GoVersion: go1.13
Race Enabled: false
TiKV Min Version: v3.0.0-60965b006877ca7234adaced7890d7b029ed1306
Check Table Before Drop: false |

- Other interesting information (system version, hardware config, etc):

    >
    >

Operation logs
tidb-lightning.log

Configuration of the cluster and the task

Screenshot/exported-PDF of Grafana dashboard or metrics' graph in Prometheus for TiDB-Lightning if possible
When debug the code, I found the statement is as below, but It can't be insert via mysql client. I'm not sure why it can be inserted via mysqlConn.exec.

REPLACE INTO `test`.`t2` VALUES('true');

Looks the issue is caused by the default case of switch when running tidbEncoder.appendSQL.

By default Lightning runs in a non-strict SQL_Mode, which would reduce those errors to only warnings. You could reproduce as:
mysql> set sql_mode = '';
Query OK, 0 rows affected (0.02 sec)

mysql> REPLACE INTO `test`.`t2` VALUES('true');
Query OK, 1 row affected, 1 warning (0.01 sec)

mysql> show warnings;
+---------+------+---------------------------------------------------------+
| Level   | Code | Message                                                 |
+---------+------+---------------------------------------------------------+
| Warning | 1366 | Incorrect integer value: 'true' for column 'a' at row 1 |
+---------+------+---------------------------------------------------------+
1 row in set (0.00 sec)

mysql> select * from test.t2;
+------+
| a    |
+------+
|    0 |
+------+
1 row in set (0.00 sec)

CSV does not distinguish between strings and keywords, unlike SQL. In CSV, true and false are always considered as the literal strings 'true' and 'false'. This is also the behavior of MySQL's LOAD DATA. So the behavior may be undesirable but is totally in line with the standard.
We may consider adding keyword specialization for true and false, similar to null, if this is a pressing need.

Many thanks for you elaboration, learn a lot from you.
By reference to https://dba.stackexchange.com/questions/80727/converting-falsestring-to-falseboolean-during-load-data-local-infile, it propose below solution. This is just for your reference.
LOAD DATA LOCAL INFILE 'file1' INTO table n1 FIELDS TERMINATED BY ',' enclosed by '\"' LINES TERMINATED BY '\n' (@var1,c2) SET c1 = IF(@var='True',1,0);

And if tidb-lightning could't support the boolean field, I suggest to give a workaround such as replace the boolean value with 0 or 1 on official user guide.


run-vcpkg@v11 does not support prepended/appendedCacheKey

We use prepended/appendedCacheKey for two main reasons:

We have two different VS solutions in the repository, in two folders. One has a strict superset of packages compared to the other. We run two different CI jobs, one which builds both solutions and one which only builds the smaller. Using appendedCacheKey we create two different vcpkg 'bundles', bespoke for each build. If they share a cache key, without any other changes, the build with the smaller set of deps may 'win' and write the cache entry. The other build, being slower, will fail to write its dependencies to cache, and future hits will pull down the smaller package. This is unfortunate for us because the expanded set of deps include qt, which takes a very long time to bootstrap. Having the ability to generate two separate cache keys lets us avoid this issue and keep the cache bundle size for the faster job, smaller.
Because we have multiple vcpkg.json, we get the output Warning: The file vcpkg.json was found multiple times with glob expression '**/vcpkg.json'. which historically meant all of the vcpkg.json files were ignored for cache key calculations.
We have custom triplet and port files in our repository. We used to use appendedCacheKey with the hashfiles feature to track changes to these files and automatically invalidate cache.

We would appreciate if support for these was reimplemented but understand that you likely dropped this support for a reason :)

Not needed with the new builtin-to-vcpkg caching mechanism.

@akrieger sorry for the late, I was going to answer sooner or later. And you are right, the caching of built packages is delegated to vcpkg that is doing the perfect caching run-vcpkg could not be doing by itself. The good news is that the user manipulation of the cache's key is not necessary anymore.


Will FreeType be supported ?

Hi,
I would like to make my font looks better. One way as the imgui C++ does, is to make use of freetype lib. I wonder when will pyimgui support this functionality, also for now, what is the suggested way to achieve the same goal.
Thanks!

This is definitely possible right now. I have tested that already with TTF fonts and documentation of core ImGui lib suggests it should work with OpenType fonts too. I will test it myself and provide some examples & documentation on the readthedocs to show how to do this.

I am using freetype to get a good looking for the fonts.


Document Micropython compatiblity

Hey guys!
This awesome library can be made to work with micropython easily by using the micropython port of datetime and using the micropython versions of time and requests:
@@ -1,8 +1,8 @@
 # coding=utf-8

-import requests
+import urequests as requests
 import datetime
-from time import mktime
+import utime as time

Thats it, everything I tried worked.
What do you think, just a section in the README, an already patched file in a subfolder or a edited fork in a new repo?
XOX

Thanks for the praise!
Ohh, that's actually pretty cool, thanks for trying that out :D
It appears we could also check sys.implementation.name to see whether we're running on micropython and import the right modules depending on that.
I'm not 100% sure what I think about that but it would have the advantage of not having to maintain multiple different variants.
Unfortunately, I don't have a micropython board handy right now, but I'll try around in the unix micropython variant.

Hey there! I found this awesome package a couple of days ago and wanted to make it run on my ESP8266 as well.
But unfortunately any urequests.get command to mvg raises a OS Error -40 . @peterfroehlich if your still around do you have any idea where this could be coming from and how to fix it?

Hey everyone! I'm also currently working on that topic.
@s6adhimm I am not sure if I can help you, but I'll try. If you still have this problem, could you please try the following minimal example code:
import urequests
response = urequests.get(url='https://www.mvg.de/api/fahrinfo/departure/de:09184:490?footway=0')
print(response.json())
I don't have the same microcontroller like you, I have a Raspberry Pi Pico W, and on this board the example code above works, it is supposed to display the departures from Garching.
Could you please tell me if this code works for you? :)


package version update to compatible version of native-web and native-mobile

Description
Style's npm packages need to be updated to be compatible with updated native-web, native-mobile and core.

worked upon no further reports


Eledger-107: Get list wallets

create API for getting all the list of wallets.

Merged the request file.
Issue resolved


Using newrelic with next.js

Bug report
Describe the bug
next run build fails with Module not found: Error: Can't resolve 'aws-sdk' when require('newrelic') is present in a page.
To Reproduce
See minimally reproducible example: https://github.com/zeit/next.js/compare/canary...djskinner:newrelic
Expected behavior
A next.js app can easily be instrumented with newrelic both on the server and on the browser.
Additional context
require('newrelic') in server.js is fine and server-side instrumentation works without issue. The problem is only with browser instrumentation.
newrelic package details and usage instructions: https://www.npmjs.com/package/newrelic

It looks like this is a server-only module, so you can't use it inside pages/index.js, instead, I guess you can use _document.js as it's only rendered on the server side.


DVS128Gesture训练以及验证精度问题

你好，我使用了你们给出的DVS128Gesture_sampled_train.txt文件以及对应的验证txt文件，用你们给出的模型checkpoint验证时，发现精度并不是很好，并且我发现你给出的sampled_train.txt文件与实际预处理出来的png文件对应不上，为啥间隔是4或者2呀

请你能给出对应正确的sampled_train.txt文件吗

Do you change the config settings when conducting the AFE representation?
 self.sample_event_num_min = 15000
 self.sample_event_threshold = 30

Can you show me the event frame after the AFE representation?

Due to the variability in the number of resulting event frames, a large number of event frames in a single sequence within a mini-batch can lead to GPU memory overload. Therefore, we sample the resulting event frames to maintain training stability.

这是我经过AFE处理过后的DVS128Gesture，其中每个照片都是连续的，而不是你仓库里面的间隔为2或者4

Yes, so I downsample the preprocessed event frames every 2 or 4 or 8 invertal to ensure the batch size won't be too large.

看看你最后生成的event frame长什么样子？

I downsample the preprocessed event frames at intervals of 2,4, or 8 to keep the batch size manageable. What does your final event frame look like?


Gradle 3.0 in gradle-wrapper.properties file causing error while loading the project in Android Studio 3.0

While importing the repository in Android Studio 3.0.1 , got this error
10:31 AM	Gradle sync failed: Could not initialize class org.jetbrains.kotlin.gradle.KotlinGradleModelBuilder Consult IDE log for more details (Help | Show Log) (354ms)
Update gradle distribution url to use gradle-4.1-all.zip, causes the error to go away.

#114 pr "Using latest Gradle release" merged


Csse layout input_data

Description

~currently atop #458~
Changelog description

[x] implement AtomicResult.input_data = AtomicInput in all the harnesses and adjust some output checks as needed. Check extras is getting separated btwn in/out as formerly merged.

Status

[x] Code base linted
[x] Ready to go

Requesting review @davidbrownell, @krachwal, @rfievet, @ketanbj, @varun646, @jyoung3131


Refactor HeaderView component

Details

Replace withWindowDimensions and withLocalize with the hook accordingly and subscribe current policy of the report instead of get all policies on HeaderView to prevent unnecessary re-renders
Fixed Issues

$ https://github.com/Expensify/App/issues/30724
PROPOSAL: https://github.com/Expensify/App/issues/30724#issuecomment-1789356040
Tests

Create a new workspace
Change the avatar of the workspace
Go to admin room
Click on header > workspace name
Delete workspace
Verify that the avatar in the header of admin room is reverted to the default avatar

[x] Verify that no errors appear in the JS console

Offline tests

Same as above
QA Steps

Create a new workspace
Change the avatar of the workspace
Go to admin room
Click on header > workspace name
Delete workspace
Verify that the avatar in the header of admin room is reverted to the default avatar

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/129500732/264e9e6d-71cc-4c7b-8aa4-df9672f51ebf

Android: mWeb Chrome
https://github.com/Expensify/App/assets/129500732/28ac7449-e90f-4983-909d-c594a69f3583

iOS: Native
https://github.com/Expensify/App/assets/129500732/80f31f40-22d2-45bf-9a3c-d094e93d5fe8

iOS: mWeb Safari
https://github.com/Expensify/App/assets/129500732/e5911c8c-d46a-4a8c-8423-62ed83222ea3

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/129500732/55971f9a-6100-4285-9d45-6f4b47baf888

MacOS: Desktop
https://github.com/Expensify/App/assets/129500732/5aeaa476-18b4-4f05-be61-5154529e2c76

Reviewer Checklist

[X] I have verified the author checklist is complete (all boxes are checked off).
[X] I verified the correct issue is linked in the ### Fixed Issues section above
[X] I verified testing steps are clear and they cover the changes made in this PR

[X] I verified the steps for local testing are in the Tests section
[X] I verified the steps for Staging and/or Production testing are in the QA steps section
[X] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[X] I checked that screenshots or videos are included for tests on all platforms
[X] I included screenshots or videos for tests on all platforms
[X] I verified tests pass on all platforms & I tested again on:

[X] Android: Native
[X] Android: mWeb Chrome
[X] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[X] MacOS: Desktop

[X] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[X] I verified proper code patterns were followed (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I verified that this PR follows the guidelines as stated in the Review Guidelines
[X] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/61146594/e1c04573-2eca-407f-804a-02702f372f5c

Android: mWeb Chrome
https://github.com/Expensify/App/assets/61146594/1bf1921a-c536-4359-ad92-ad2fbcf6ac58

iOS: Native
https://github.com/Expensify/App/assets/61146594/17f5d224-6f20-4df0-b8df-3efc8a36e4c9

iOS: mWeb Safari
https://github.com/Expensify/App/assets/61146594/ff33868d-0e0c-4767-9359-33f53125193b

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/61146594/7a7e0cf7-caab-46bd-8823-fdd12b49b98f

MacOS: Desktop
https://github.com/Expensify/App/assets/61146594/eedbb264-f6af-4c9b-8cd2-85eacb9aea4b

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by https://github.com/mountiny in version: 1.4.5-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


File/Folder enhancement propositions

File/Folder picker builder could have additional two optional methods :
setMinialPath(String path) for limiting user going too far up in the folder system tree.
shouldDisplayDotStartingItems() for removing invisible files from display.
EDIT :
A filter object could be passed to allow personalized filtering (i.e. replacing the dot thing and allowing extended filtering instead just mime type)
This filter could be passed to the folder picker also as only file picker allow mime filtering for now.

You should create a custom dialog for something like that.


feat: add setVariables function

Adding a new function for mock contracts. With setVariables mocks will
be able to edit multiple internal variables at once.

Fixes #111

any chance of this being accepted?

yo @tonykogias, there's an active bounty on this task and is assigned to you, so that other devs don't overlap your work.
please confirm if you'll be continue working on it so that we keep that assignment,
again, thanks for your contribution! <3

I will resolve the comments later today

@wei3erHase I pushed the fixes please let me know if the changes work for you.

please add yourself as contributor here: https://app.dework.xyz/i/0FDGv5pCT7TJXZVki1wJ0Z

please add yourself as contributor here: https://app.dework.xyz/i/0FDGv5pCT7TJXZVki1wJ0Z

removed the test and assigned myself on the link.

cannary release at<EMAIL_ADDRESS><3


Card Details Not Resetting Properly. Able To Submit With Empty CardField.

Issue
While I was doing some error validation for my applications checkout screen, I noticed that after entering information into a CardField component, I could then erase all of that information, and still submit and get a successful payment. If this isn't clear, the steps to reproduce listed below should clarify things.
Steps To Reproduce
I know most of the Stripe test information can be chosen at random, but ill make it easy on you, and leave what I entered into the component. Since you guys are on the ones that built this library, I would assume you would have the following setup before trying to reproduce:

useConfirmPayment and CardField imported
A button to trigger the payment
A method that fetches a clientSecret, which will then be used in confirmPayment

Fill out a single CardView component with the following details:

#:<PHONE_NUMBER>424242
mm/dd: 09/29
cvc: 359

Erase all of the information you have just entered.
Submit the payment.
It should process successfully.

Code
Here is a snippet of what I am using to process a payment. I have a button that when pressed, triggers the submitPayment function. From speaking to someone previously on here, it is my understanding that somehow behind the scenes the card details are extracted from the CardField component.
Don't be confused by our API route /payment/token, behind the scenes its calling stripe.paymentIntents.create, and passing back the client secret.
/**
 * @function getClientSecret
 */
const getClientSecret = async () => {
  try {
    const res = await axiosClient.post('/payment/token', {
      productId: route.params.productId,
      saveCard: false,
    });

    return res.data.result.clientSecret;
  } catch (e) {
    console.log('Stripe getClientSecret Error', e);
  }
};

/**
 * @function submitPayment
 */
const submitPayment = async () => {
  try {
    const clientSecret = await getClientSecret();

    setApiError('');

    const { paymentIntent, error } = await confirmPayment(clientSecret, {
      type: 'Card',
    });

    if (error) {
      handleStripeError(error);
    } else if (paymentIntent) {
      handleSuccess(paymentIntent);
    }
  } catch (e) {
    console.log('Stripe submitPayment Error', e);
  }
};

...other code

<CardField
  postalCodeEnabled={true}
  cardStyle={cardFieldStyles}
  style={{ height: 45, marginTop: 10, marginBottom: 10 }}
/>

...other code

Other Notes
By logging the cardDetails via the CardField's prop, onCardChange, I see that after filling the CardField, and then erasing the information, some of the properties are not reset. Here is a console.log of what onCardChange shows after everything has been erased.
{
  last4: '4242',
  expiryMonth: '',
  expiryYear: '',
  postalCode: '',
  brand: 'Visa',
  complete: true
}

The expiration month and year do in fact reset, but last4, brand, and complete do not.

This has been fixed in https://github.com/stripe/stripe-react-native/releases/tag/v0.1.2 please update to that version


Syntax operations also need to ensure project is present for the open script infos since update could be pending to make sure open script info has project

Also convert one relevant test case to baseline
Fixes #50131

So what exactly happened before? Was it something like this?

Open a file - a default project is automatically made.
Close the file - the default project is automatically closed because it has no files open.
Open another file - the default project is assumed not to be closed (?)

So what exactly happened before?

Open a file say src/file.ts. Opens configured project say tsconfig.json
Close this file. Configured project is not collected because the removal of projects and scriptinfos that are orphan happens only when opening file
open another file say tests/file.ts  It finds a tsconfig file tests/tsconfig.json but that does not include this file. But existing tsconfig.json has this file so we dont create inferred project. After this collecting orphan project collects tsconfig.json because there is no open file which will open this project (tests/file.ts does not count because any file in tests folder would open tests/tsconfig.json and not tsconfig.json) So now tests/file.ts is not part of any project
Call outlinging span and we were throwin error before the fix. With the fix it sees there is no project for this file so it creates inferred project for this file. (just like it would have if we didnt do first two steps)

I'm still missing why we'd need a project at all to handle this request.  Is that just an unfortunate consequence of our design?

I'm still missing why we'd need a project at all to handle this request. Is that just an unfortunate consequence of our design?

Thats our design for now.. in most cases the open files are already part of the project (the scenario we are fixing is just unfortunate) so it was never priority to experiment separate project for syntax only operations. (To get hold of LS on which we can forward those requests)


Add unit tests

The previous version of Catspeak had some sense of unit tests, this version should too.

Added unit testing module and a buncha tests to version 2.2.0.


show the Run... button the Services / Controllers pages too?

might be nice to let folks run stuff from more places

+1


Fix for ELE-699, None default for --group-by

To address issue #816 I've set the default value of --group-by option to None, since later on when the Config is instantiated, the self.slack_group_alerts_by gets alert as the last non-null value.
https://github.com/elementary-data/elementary/blob/2bc0204ca304f9205bfa6964a41937294e7ec3c8/elementary/config/config.py#L111-L115

@dimoschi does it now work for you in the config that caused the original issue?
I ran the automation on it anyway.

@dimoschi does it now work for you in the config that caused the original issue? I ran the automation on it anyway.

Yes, it works perfectly fine with the config. FYI the config is the following:
slack:
  token: "definitely-not-a-token"
  channel_name: "data-ops"
  group_alerts_by: "table"
  alerts_config:
        alert_fields: ["table", "column", "result_message", "test_parameters", "test_query", "test_results_sample", "description","subscribers"]

and the slack post was the following:

Tests failed though 😞

Copied the changes to this PR
https://github.com/elementary-data/elementary/pull/823
In order for the automation to be able to run.

@RoiTabach I guess it was because the Pr comes from a forked repo? FYI, I wasn't able to push a new branch, that's why I forked it.

@dimoschi it's merged

(@dimoschi Sending you a Slack message in the elementary community slack space)


Doubt Regarding fine tuning of detection model

@faustomorales, Thanks for great work.
I have question regarding fine tuning of a detection model. As per repository keras-ocr internally using CRAFT model to localise words. As per my understanding during fine tuning, weights of original  CRAFT model will get updated /fine tuned (Correct me if I am wrong).
You have already provided example  (https://keras-ocr.readthedocs.io/en/latest/examples/fine_tuning_detector.html), In this case "ICDAR 2013" data set is used for training, So after it, Can I expect new model will be having better (or equal) localization accuracy compared to original CRAFT model ?
Thanks in Advance.

The original CRAFT model was trained using a weakly supervised for which source code was unavailable. The weakly supervised approach allows them to use much larger datasets because it doesn't require character-level labels. Please see the paper for more information. We have an open issue for writing the weakly supervised version implementation (https://github.com/faustomorales/keras-ocr/issues/40) but I'm not sure if/when I will be able to get to it.
All that said, when you fine tune a detector for a specific task, I would expect the fine-tuned detector to function better for that task than the general CRAFT model trained in a weakly supervised manner.
Since this is not a bug, I'm closing this issue for the moment. Thanks!


Editing an llm-content text part removes focus

To repro:

go to Specialist Testing Grounds
click on Outline Writer
try to edit the text part
the text part will loses focus on every input

I tried to spelunk and I can't figure out easily why that happens.
I thought it might be because I added markdown rendering, but it still reproduces without it.

Oh interesting! it looks like this happens with all the inputs in the Node Details. When I try to edit the input schema, I get the same result.
Let me look real quick....

I think it's because the schema update triggers a re-render. It is likely to do with the guard in the UI controller and/or the shouldUpdate in the node info

It's really tricky to get right because we don't want it to re-render when you type, but we do when you change edges or add/remove properties. In fact the problem is probably the @breadboardnodeupdate listener I added today in the UI controller. It probably needs to be a bit more discerning about whether it qualifies for a re-render.

https://github.com/breadboard-ai/breadboard/commit/4e181089ef87e6a67dd51ee2f81797a86e77f897 this one is what made it go kablooey.

I wonder if the Editor API "graphchange" event could help with this somehow.

I'll take a fresh pass at this now

I realised that we can listen directly for schema changes. I don't think we can use the graphchange event because it responds to all changes, and here we only want to know when edges and schema changes occur. I suppose we could consider expanding on the GraphChangeEvent to let us know exactly what changed internally?

This sounds like a really good feature. Basically, a flag on the event that is true when the event changes the shape of the graph? WDYT?


[ 🚧 WIP ] Create PrintReverseOfANumberByRecursion.c

Fixes #5555
Why:
Closes #5555
What's being changed:
Add C program for "Print reverse of a number by recursion".

Actually, your program does not use recursion to solve the problem, it would fix #3469, which is already fixed, and not #5555. If you want, you can adapt your code to use recursion and fix that, and then open another PR. Regards.


Backport of #12187 to release/v1.12.

#6048 introduced a small regression switching the API to return git commit message summary not the whole message.
This PR causes the API to return the full message once again – in release/v1.12.

Ah, it seems @techknowlogick already did the backport.


Prototype of a more aggressive parallel scanner

This is a work-in-progress changeset for a more aggressive parallel scanner.
The current parallel scanner wastes time waiting for thread pool results when
it could have been more aggressively continuing to gather more tasks to
dispatch. The idea here is that if we have dispatched the maximum number of
jobs and none are done yet, we should gather more tasks instead of doing a
blocking wait on the thread pool. In my testing, I have seen a 4-5% improvement
in some VMware builds.
One upside of this change is that it paves the way for us to be able to
implement remote caching because we now ask the cache for groups of tasks
instead of one-at-a-time.
At VMware we don't have a good mechanism for testing upstream SCons changes so
this is a blind application of a patch I wrote. It passes some tests but
may have some bugs compared to the version I applied internally. More work
needed here; for now I just want to get people's thoughts on whether this is heading in the right direction.
Contributor Checklist:

[ ] I have created a new test or updated the unit tests to cover the new/changed functionality.
[ ] I have updated master/src/CHANGES.txt directory (and read the README.txt in that directory)
[ ] I have updated the appropriate documentation

@grossag - Improvements here seem to be of interest recently. Have you had a look at my proposed #3386?

@grossag  - What version of SCons are you running internally?

Looks like you have some code errors?
NameError: global name 'ParallelV2' is not defined:
  File "/home/travis/build/SCons/scons/src/engine/SCons/Script/Main.py", line 1374:
    _exec_main(parser, values)
  File "/home/travis/build/SCons/scons/src/engine/SCons/Script/Main.py", line 1337:
    _main(parser)
  File "/home/travis/build/SCons/scons/src/engine/SCons/Script/Main.py", line 1112:
    nodes = _build_targets(fs, options, targets, target_top)
  File "/home/travis/build/SCons/scons/src/engine/SCons/Script/Main.py", line 1273:
    jobs = SCons.Job.Jobs(num_jobs, taskmaster, options.parallel_v2)
  File "/home/travis/build/SCons/scons/src/engine/SCons/Job.py", line 113:
    taskmaster.set_tm_owns_caching(not isinstance(self.job, ParallelV2))

Looks like you have some code errors?
NameError: global name 'ParallelV2' is not defined:
  File "/home/travis/build/SCons/scons/src/engine/SCons/Script/Main.py", line 1374:
    _exec_main(parser, values)
  File "/home/travis/build/SCons/scons/src/engine/SCons/Script/Main.py", line 1337:
    _main(parser)
  File "/home/travis/build/SCons/scons/src/engine/SCons/Script/Main.py", line 1112:
    nodes = _build_targets(fs, options, targets, target_top)
  File "/home/travis/build/SCons/scons/src/engine/SCons/Script/Main.py", line 1273:
    jobs = SCons.Job.Jobs(num_jobs, taskmaster, options.parallel_v2)
  File "/home/travis/build/SCons/scons/src/engine/SCons/Job.py", line 113:
    taskmaster.set_tm_owns_caching(not isinstance(self.job, ParallelV2))

Good catch. I haven't tested it a lot yet; mostly just wanted to get a conversation started. That said, I did attempt to resolve the issue with another commit just now.

@grossag - What version of SCons are you running internally?

We are running SCons 2.0.1 with a few custom patches on top and a lot of VMware-specific overrides. We are working internally on updating to 2.5 and then to 3.0 and hope to have that done in the next few months.

@grossag - Improvements here seem to be of interest recently. Have you had a look at my proposed #3386?

Thanks for linking to this @acmorrow. I looked just now but I think that your change does not attempt to address the main issue I'm concerned about. Specifically, your change still waits on a job to be ready if jobs==self.maxjobs, while I think the big performance improvements will come if we only wait if task==None.

@grossag - What version of SCons are you running internally?

We are running SCons 2.0.1 with a few custom patches on top and a lot of VMware-specific overrides. We are working internally on updating to 2.5 and then to 3.0 and hope to have that done in the next few months.

Skip 3.0.0 and go to either 3.0.5, or 3.1 (to be released soon) Some performance improvements in 3.1. 3.0.0 had minor issues with print() and py2/py3. so definitely skip that.

@grossag - What version of SCons are you running internally?

We are running SCons 2.0.1 with a few custom patches on top and a lot of VMware-specific overrides. We are working internally on updating to 2.5 and then to 3.0 and hope to have that done in the next few months.

Skip 3.0.0 and go to either 3.0.5, or 3.1 (to be released soon) Some performance improvements in 3.1. 3.0.0 had minor issues with print() and py2/py3. so definitely skip that.

Can you push your current scons code to github repo somewhere?
Be interested to see what changes you've made and if any would be useful to the rest of the scons community

@grossag - Improvements here seem to be of interest recently. Have you had a look at my proposed #3386?

Thanks for linking to this @acmorrow. I looked just now but I think that your change does not attempt to address the main issue I'm concerned about. Specifically, your change still waits on a job to be ready if jobs==self.maxjobs, while I think the big performance improvements will come if we only wait if task==None.

That is true, but I would think of that as basically equivalent to just running with -j 1000000 or some huge number under my model. My model came into being when I was looking at how SCons behaved with a large -j number in an icecream cluster. It would spend basically all of its time trying to achieve the target concurrency, building up a huge backlog of uncollected tasks until it literally couldn't find another, and only then collect more. Essentially, SCons just couldn't find enough parallel work in our tree to come close to saturating the clusters compute capacity.
The idea in my change is to try to find a balance, where you consume jobs when they are ready such that you are more likely to make it easy to find new jobs. So the target concurrency becomes aspirational, not required.
If you don't mind giving it a try, I'd be interested to know how much (if any) of your observed 4-5% you see trying my approach with a large -j value.

One upside of this change is that it paves the way for us to be able to
implement remote caching because we now ask the cache for groups of tasks
instead of one-at-a-time.

I'd like to understand a little better what you have changed and what motivated it. We are currently using remote shared caching on both Linux and Windows with some success, simply via a shared network file-system. But we are certainly interested in improvements!

Going to work on another version that will integrate with Bazel remote cache server. Will post another pull request in a few weeks to solicit feedback.


asset parameter for savings_project_position() should be optional

Issue subject
The asset parameter for savings_project_position() should be optional but it is not.
https://github.com/binance/binance-connector-python/blob/5c2afe36c300214b20c9ee7ccb7b1ef1841d2cb2/binance/spot/savings.py#L181
API doc https://binance-docs.github.io/apidocs/spot/en/#get-fixed-activity-project-position-user_data
Environment

binance-connector-python 1.15.0
Python 3.9

Thank you for spotting this. We will fix it accordingly.

This is fixed in #164


474907D1 - Bass Pro Shops - The Strike (USA)

Validation

[X] I've read the FAQ.
[X] The Xenia build used is from a Canary branch. (not UWP/MLBS/AlexVS/master/pull requests, etc)
[X] This issue isn't for help (tech support) with Xenia, or a game.
[X] I've checked if this game doesn't already have a compatibility report.

Wiki page (optional)
No response
Xbox 360 Marketplace link (optional)
No response
Xenia Canary version
https://github.com/xenia-canary/xenia-canary/commit/764f230dd9883bfb63e3aea7642aac26f010e1bb
Issues
Crashes when loading level. Does not crash in mainline Xenia.
https://github.com/xenia-project/game-compatibility/issues/313
Log
xenia.log
Screenshots

Labels
regression

Tested on: https://github.com/xenia-canary/xenia-canary/commit/a3bd34122be24b5e9868cbe00d937751df428571
xenia.zip
Requires protect_zero = false to reach gameplay.

backwards-compatibility-unsupported | requires_protect_zero_false | state-gameplay


Reintroducing deployment with Fabric/Invoke

Invoke (by the same author as Fabric) seems to be a library for leveraging task executions, not particularly with system administration and deploying but about all-around task running. ([refer] (http://www.fabfile.org/roadmap.html#invoke-fabric-2-x-and-patchwork).) Thus, I think it would be better to utilize Fabric 2.0 when it's launched instead of using "raw" Invoke.

+1 for Invoke.

Invoke is already Python 3 compatible, due to being a new codebase with few dependencies.

Invoke seems better

@miaekim: No offenses, but to clear up Invoke is a module that makes up Fabric 2.0 (which will become 3000 compatible, but has no launch date yet.) I was concerned about that Invoke is too broad (or abstract) than Fabric API.

@tkiapril: :+1: You are using and can use both for right purpose.


Add  kisshome-research 1.0.4 to stable

see https://github.com/ioBroker/ioBroker.repositories/pull/4060

RE-CHECK!

merging anyway as this adpater is for special use only


Add a cron monitore

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
New Monitor, UI Feature
🔖 Feature description
To monitore cron, it's easier to send OK/NOK with curl (like healthcheck.io) or if cron don't execute after specific time it raise an alert.
I see push monitore and I think it can have a little improvement : set Heartbeat interval in second or cron execution format.
For exemple I want to monitore a cron who execute each monday at 3AM, I can define 604800 in interval, or define 0 3 * * 1 in interface to monitore the cron
✔ Solution
Add in push monitor a way to select interval or cron execution time format.
❓ Alternatives
Healtcheck.io can work (it work) but multiplying the tools can be a bit of a headache
📝 Additional Context
No response

I don't think this will be a thing for uptime kuma. Its more for web things like websites, dns records and so one but not for cronjobs.

I think this is a great idea. I don't see why uptime-kuma couldn't be used to monitor cronjobs, it would do the job just fine.
If there are push monitors, why not time range push.

That's what I think too. The time interval alone is not sufficient for monitoring the status of the task.

It would be great to have an api that would allow to send POSTs to know if our cronjob was successful or not.
A bit like betteruptime does

I think it's a good feature for Kuma, web services often have cron jobs hidden they rely on.

I'm also interested in this functionality, similiar to what cronitor does: https://cronitor.io/cron-job-monitoring
I the feature itself would be interesting, I'm willing to create a PR for that

Changing the monitor sheduler is a lot of work.
Current plan is here if you want to contribute to this PR (via targeting the queue instead of the master branch):

https://github.com/louislam/uptime-kuma/pull/3447


Importing into Jupyter notebook raises error

Hi
This is a really useful  looking project for supporting novice coders.
I just tried installing the current version from PyPi (0.4.1) and running in a Jupyter notebook (classic and in JupyterLab) but I get an error on the initial import:
%pip install friendly
from friendly.jupyter import *

Generates the error:
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-1-612f8b1979f1> in <module>
----> 1 from friendly.jupyter import *
      2
      3
      4

~/.local/lib/python3.8/site-packages/friendly/jupyter.py in <module>
      1 from friendly_traceback import session  # noqa
----> 2 from .ipython import *  # noqa
      3 from .ipython import helpers
      4 from friendly import rich_formatters
      5

~/.local/lib/python3.8/site-packages/friendly/ipython.py in <module>
     34 # so as to shorten name of code block "files" for SyntaxError
     35 # cases - since SyntaxErrors do not generate frames.
---> 36 frames = inspect.getouterframes(inspect.currentframe())
     37 session.ipython_frame = None
     38 for frame_info in frames:

/usr/lib/python3.8/inspect.py in getouterframes(frame, context)
   1489     framelist = []
   1490     while frame:
-> 1491         frameinfo = (frame,) + getframeinfo(frame, context)
   1492         framelist.append(FrameInfo(*frameinfo))
   1493         frame = frame.f_back

/usr/lib/python3.8/inspect.py in getframeinfo(frame, context)
   1463         start = lineno - 1 - context//2
   1464         try:
-> 1465             lines, lnum = findsource(frame)
   1466         except OSError:
   1467             lines = index = None

/usr/lib/python3.8/inspect.py in findsource(object)
    838         pat = re.compile(r'^(\s*def\s)|(\s*async\s+def\s)|(.*(?<!\w)lambda(:|\s))|^(\s*@)')
    839         while lnum > 0:
--> 840             if pat.match(lines[lnum]): break
    841             lnum = lnum - 1
    842         return lines, lnum

IndexError: list index out of range

In passing, I note that from friendly.jupyter import * is a really dodgy (and bad practice) thing to inflict onto novices. They donlt really have any idea what's in the namespace or how it works anyway, so importing who knows what via *  could create all sorts of inexplicable errors and misunderstandings for them.

Thanks for the report, I will look at it right away.
I understand about the * import suggestion. I have tried to explain why here: https://friendly-traceback.github.io/docs/import_all.html

@psychemedia Could you tell me which version of Jupyter notebook/lab you are using?  I cannot reproduce the error here, but I am using the latest version of jupyter:
> jupyter --version
jupyter core     : 4.7.1
jupyter-notebook : 6.4.0
qtconsole        : not installed
ipython          : 7.25.0
ipykernel        : 6.0.1
jupyter client   : 6.1.12
jupyter lab      : 3.0.16
nbconvert        : 6.1.0
ipywidgets       : not installed
nbformat         : 5.1.3
traitlets        : 5.0.5

Looking at the traceback you included, I see that code blocks "file names" are of the form "<ipython-input-1-...>" which indicates to me that this is an older version than the one I am using.  I really should support older versions as well...

Version 0.4.2 uploaded to PyPI should work.

Seems to import okay with that latest version, thanks (though reports an older version?)
from friendly.jupyter import *
> friendly_traceback 0.4.0; friendly 0.4.1.

Re: import *: I just noticed your explanation in the docs:-) I think it would be useful to add just a minimal import statement that shows how to load in just what's required to support the richer traceback messages. Eg at the moment, I'm simply interested in the richer traceback/error message part of the package, but not any of the interactive commands which is perhaps more just the friendly-traceback components?

Thanks.
The reported version number not being updated is weird: it should be automatically extracted from the code. Here's what I see here:

However, I think I saw the old version number like you did when I ran it first. Anyway, as long as it works, it should be fine.
Regarding your other comment:
My recommendation would be to do
from friendly.jupyter import Friendly

so that, if ever you needed, you could type in Friendly.why(), etc.

I would possibly also import the functions light and dark as they can provide nicer output, depending on the theme used by Jupyter.
Alternatively, if you really aren't interested in anything but the absolute minimum, the following should work.
from friendly import jupyter

I really need to document all of this ...
Thanks again for your report and comments, and for confirming that it seems to work correctly for you.

Hi, thanks... from friendly import jupyter looks interesting.
Just in passing, not sure if you've seen this PR to IPython that adds the cell run number to error messages: https://github.com/ipython/ipython/pull/13043

I had not see that.  I wonder if it was motivated by a suggestion I made:
https://github.com/ipython/ipykernel/issues/716
(also https://github.com/ipython/ipython/issues/12755).
With this change, friendly-traceback loses one of its advantages! ;-) ;-)

For teaching novices, I think helpful error messages could be really useful.
Another related example I'm aware of is the pseudo text-from-code generator and narrative error reports used in BlockPy: https://blog.ouseful.info/2016/02/18/blockpy-python-blockly-environment/

Thanks for the latest link; I'm reopening this issue as a reminder to look at this in more details later.

RE: blockpy - this appears to to be the helpful bit: https://pedal-edu.github.io/pedal/

@psychemedia I have created a new issue, #5, that might be of interest to you. Feel free to add your comments.

This issue was kept open for reference to some other projects. One of these (pedal) was already linked in an old issue from the previous friendly-traceback [1].  This information properly belongs to friendly-traceback and not friendly (this project); it has now been added as an issue to friendly-traceback [2].
One of the projects mentioned is Pedal. That project had mentioned using Friendly-traceback in one of its own issues [3] which I commented upon.
With the new issue [2] created, there is no need to keep this issue open anymore, since everything has been taken care of.
[1] https://github.com/aroberge/friendly/issues/100
[2] https://github.com/friendly-traceback/friendly-traceback/issues/30
[3] https://github.com/pedal-edu/pedal/issues/79


Remove pillow as a dependency

Pillow is a dependency to use ImageField columns on a database model- which we use to store the logo field in apps/civic_pulse/models.py Agency model.


Add VirtualResizableDirectMemory, that uses virtual memory API

https://github.com/linksplatform/Memory/blob/51d78fb523cb55c0784950db95d3e746e6617443/HeapResizableDirectMemory.cs#L12

Looks like it is not available in .NET yet. So the only way is to create wrapper around native API at Windows, Linux, and macOS.


Remove unnecessary checks for API handles

Extra checks are not needed for zero API handles in a function like RenderProgram::dispose().

It may be fixed in the future when rolled up with other changes, but it's not a priority so I'm going to mark it as 'won't fix'.

Will it be more useful to annotate this issue with an other label?


[BugFix] fix refresh hive metadata

Signed-off-by: stephen<EMAIL_ADDRESS>What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
This currently results contents of cache will be clears, which is equivalent to invalidate cache. So we need to add "/" at path of PathKey to check if this partition path need to refresh.
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 0 / 2 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/connector/hive/CacheUpdateProcessor.java
0
2
00.00%
[151, 152]

@Mergifyio backport branch-2.5


LGTM.com - false positive

Description of the false positive

This regex used an r f-string with a configuration variable name being replaced with the value to search for. It is not searching for the variable names characters in the replacement field.
URL to the alert on the project page on LGTM.com
https://lgtm.com/projects/g/tensorwerk/hangar-py/snapshot/4a94812c263984239b03132c8f8f76cbaaf45a4f/files/src/hangar/backends/hdf5_00.py?sort=name&dir=ASC&mode=heatmap#xb4387382d0c2cc09:1

Thanks for reporting this!
We made a fix for this in our internal repo, and it should be deployed to LGTM.com within the next few weeks :)


dump_syms: 1.0.1 -> 2.0.0

https://github.com/mozilla/dump_syms/releases/tag/v2.0.0
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg build dump_syms

Still getting symbols on firefox 91esr and 103.0.2.


Capture group in regexp find and replace not working.

Version: 1.38.1
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.7.0
Steps to Reproduce:

Insert "(?<=")fa fa-(.*?(?="))(">)" without the wrapper quote, into the find field, enable regular expression.
Insert $1 into replace field
Press search

Observe that the matched string is replaced to '$1' instead of the actual capture group

Does this issue occur when all extensions are disabled?: Yes

Similar to this issue https://github.com/microsoft/vscode/issues/80667 with newlines in the search field and a literal $1 inserted as the replacement.  That issue was resolved in the Insider's Build.  But it also happens with lookarounds.

Should work as expected in Insiders version. Could you check using lates build https://code.visualstudio.com/insiders/ ?

 replaced to <div class="works as expected in Isiders.

Version: 1.39.0-insider (user setup)
Commit: 8b6e0efc0a98639085fabeb5226a516785424796
Date: 2019-09-19T03:15:21.720Z
Electron: 6.0.9
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.18980

@vincent-cm do you also have problem with "Find in files" instead of "Find" (in current editor)?

@vincent-cm do you also have problem with "Find in files" instead of "Find" (in current editor)?
I was able to reproduce this in Find in files, but works as expected in Find.
/cc @roblourens

I did not try find in files. But it failed when in find in project as described.

On 19 Sep 2019, at 20:43, Andrii Dieiev<EMAIL_ADDRESS>wrote:
@vincent-cm do you also have problem with "Find in files" instead of "Find" (in current editor)?
I was able to reproduce this in Find in files, but works as expected in Find.
/cc @roblourens
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@IllusionMH Yea I just mean the "find in files"

This is actually a separate issue, now that we support lookback. Thanks


style: fixing all lines over 111 chars

fixing all lines as directed from lint lll
What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

146 of 197 (74.11%)  changed or added relevant lines in 9 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 79.125%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/agent/bakerapi.go
3
4
75.0%

pkg/agent/datamodel/types.go
8
9
88.89%

pkg/agent/params.go
27
31
87.1%

pkg/agent/baker.go
69
83
83.13%

pkg/agent/datamodel/sig_config.go
4
18
22.22%

pkg/agent/utils.go
9
26
34.62%

Files with Coverage Reduction
New Missed Lines
%

pkg/agent/baker.go
2
73.15%

pkg/agent/datamodel/sig_config.go
4
57.98%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
2062

Relevant Lines:
2606

💛  - Coveralls


Can we hold on 3.0?

There are still HTTPS test regressions from 1.x to 2.x I'd like to see resolved.

(Sorry for lack of detail - I'm traveling on my phone. Will give some more details when I can sit down.)

Wanted to release a new version since there were 70+ commits on master, and 2.0.0 was released 3 months ago. 3.0.0 would have been 2.1 except a breaking change was introduced to the Sniffer API, returning a symbol, instead of a hash (and expanding the scope from CMSs to frameworks).
Opened https://github.com/benbalter/site-inspector/issues/60 to track the boogyman regressions. Have discovered one that's awaiting your input. Specific regressions called out there, pull requests, or individual issues for other regressions are likely the best way to move forward and see them resolved. :smile_cat:


I want to add a simple carousal out of html and css

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.
Add image here
After building the component please raise a PR with a screenshot of the component and add the component in path of the file.
If you need to use any icon please use it from Hero Icons
Good luck.

Could you assign it to me?

I am doing it already

I can not see an image here. @AashishKatila