Binding in kv does not work in certain cases

If the widget in this example is added from kv, center is updated when the parent width changes:
from kivy.app import App
from kivy.lang import Builder
from kivy.factory import Factory

kv = """
<NewB@Button>:
    size_hint: None, None
    size: 20, 20
    center: (self.parent.width - 40, 40) if self.parent else (0, 0)

Widget:
    NewB
"""

class TestApp(App):

    def build(self):
        return Builder.load_string(kv)

if __name__ == '__main__':
    TestApp().run()

But if we instantiate it and add it from python, center is not updated when the parent width changes:
from kivy.app import App
from kivy.lang import Builder
from kivy.factory import Factory

kv = """
<NewB@Button>:
    size_hint: None, None
    size: 20, 20
    center: (self.parent.width - 40, 40) if self.parent else (0, 0)

Widget:
"""

class TestApp(App):

    def on_start(self, *args):
        b = Factory.NewB()
        self.root.add_widget(b)

    def build(self):
        return Builder.load_string(kv)

if __name__ == '__main__':
    TestApp().run()

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/24841022-binding-in-kv-does-not-work-in-certain-cases?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github).

You can turn on rebind:
from kivy.uix.widget import Widget
Widget.parent.rebind = True

Though I'd suggest that maybe the parent property should have rebind=True by default.

@kived thanks ;), i agree, in this case it might be warranted to do it by default.

@dessant, we'll need to get it in writing from @akshayaurora that we can change to rebind=True by default before we can do that. I'm sure he won't like that, though.

@matham why would he not like it? :P It is backward compat, and it would only affect a couple of kivy props, it would be good to have it for children too.

Why would I not like that...cause it's not needed by default,  case in point @dessant came across this issue after using kivy for 2+ years.
Adding rebind = True to widget's parent/children/anyObjPropThatHoldsAnotherWidget would mean rebinds for each of them even when they are properly bound == redundant.
vs Properly documenting this and hilting this in the docs.
I am in favor of option2.

it would only affect a couple of kivy props, it would be good to have it for children too.
Widget.parent and Widget.children are part of the base class so you would have rebinds for every widget's parent and every child, to me this seems over kill.
This would be like using a Tzar bomb where a surgical strike is needed.
Documenting this properly and allowing the programmer to set rebind = True for the widget where he needs this is more like a surgical strike that is needed.

@akshayaurora, do you have performance tests to show that the benefit of enabling it is not worth it?
Closing the issue before  option2 was done is also like using a bomb. :D

There is not a pr this is a issue,  I am open for a pr that shows no significant performance issues with a significantly sized UI, like in showcase.
For this issue closing it was a mistake,  this should be documented then this issue closed.


Switch to pytest and add description of test cases

Signed-off-by: nhan.trong.nguyen<EMAIL_ADDRESS>

@StevenL2015 I create pull request to switch all test cases to pytest and add their descriptions. Please review it for us.
Thank you so much.


Control pusher appears to not unmap teams.

New robots map fine, but it appears that old controls are never unmapped.

Resolved by a347ac0.
json.Unmarshal merges maps implicitly, so we re-initialize the map prior to accepting the payload from mqtt.


Corrected typo in media-analysis-deploy.yaml

Issue #, if available:
Description of changes:
Corrected typo in media-analysis-deploy.yaml "youre" to "your"
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Duplicate pull request


feat: remove write_account_bytecode and write_account_nonce entrypoints

These entrypoints are no longer relevant.

I will like to take care of this issue


Create fmg-brk.txt

@fmgos I am sorry for a delay with processing your request. Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


Open stage domain when mapped for up url --open

Maybe? If you have gh-polls.com mapped for example, there's not much reason to copy the API Gateway url

To clarify... what I would expect: If I mapped "www.xyz.com" I would want that (https://www.xyz.com/thepath) as the URL returned from up url

Yep definitely, agreed!


Add a possibility to log all actions

I am starting to work on remx module, that was not written by me. It would be really helpful to see all actions being logged to understand the general flow better.

as actions are just regular functions that you call, I can't inject anything.
But maybe a possibility is to add logging on the state itself? for example every mapStateToProps, or even every setter or getter


adding openebs nfs docs to sidebar.json

Signed-off-by: Raj<EMAIL_ADDRESS>

can you also add in versioned docs sidebar?


Adopt files utility and turn on windows clustered testing

@chrisdennis @cljohnso : it would seem that you probably have the same issue in Ehcache as we have in tc-platform (https://github.com/Terracotta-OSS/terracotta-platform/pull/645) with the port chooser problem (https://github.com/Terracotta-OSS/galvan/issues/215) ? The Windows build are also reaching the build time limit and get canceled ?

Close in favor of #2762


Restore token context in multiline blocks

When using state.md.block.parse it loses all context as to where the tokens are placed within a document. Notably, level and map which represent token nesting level and source line number respectively.
This PR stores the created tokens temporarily so that the context can be added back manually before pushing the tokens onto the state.
I did my best to follow the projects style, but please let me know if you have any requested changes, I'd be more than happy to change this to how you see fit. I also didn't see how this change could fit into the current testing framework, so there are no tests for now.

Sorry for the late reply, I was away from my laptop for a while.
The coding style is nice. I think the testing framework can hardly check these open/close tags, and I'd just leave the relevant tests to-dos for now.
It seems state are overriden in parse function. This PR does work, but I wonder if it can create a temporary state and call tokenize function instead, so there's no need to mutate each block tokens?

It seems state are overriden in parse function. This PR does work, but I wonder if it can create a temporary state and call tokenize function instead, so there's no need to mutate each block token?

That's a good idea, I'll give it a try. I suspect the tokens might still need to be mutated to get the correct map.
I'm away from my dev environment for the next 2 weeks, but I'll look into this again in the new year!
Thanks.

@redbug312 I've got my dev machine back and this solution based on your suggestion should be a bit cleaner than before. No more mutation, but I did have to add some padding to the text in order to get the line mapping to work correctly. Let me know what you think!

Thanks for the trying, and the revision looks great!
I'd merge the PR as v4.1.2.


Sign sdk init failed for release build apk

release build init failed
debug is ok,when i build release  SignClient.initialize failed
SDK Version

Client: Kotlin wallet
Version  android-core:1.4.0 & com.walletconnect:sign:2.2.0

To Reproduce
Steps to reproduce the behavior:

SignClient.initialize
SignClient.initialize

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Device (please complete the following information):

Device: [e.g. Pixel]
OS: [e.g. Android 12]
Browser [e.g. stock browser, chrome]
Version [e.g. 22]

Additional context

locked <0x03ecea1a> (a org.koin.core.instance.SingleInstanceFactory)
at org.koin.core.instance.SingleInstanceFactory.get(SingleInstanceFactory.kt:51)
at org.koin.core.registry.InstanceRegistry.resolveInstance$koin_core(InstanceRegistry.kt:111)
at org.koin.core.scope.Scope.resolveValue(Scope.kt:255)
at org.koin.core.scope.Scope.resolveInstance(Scope.kt:242)
at org.koin.core.scope.Scope.get(Scope.kt:205)
at com.walletconnect.sign.client.SignProtocol.initialize(SignProtocol.kt:131)
at com.walletconnect.sign.client.SignClient.initialize(unavailable:12)
at com.coinhub.wallet.CodeApplication.initSignClient(CodeApplication.java:133)
at com.coinhub.wallet.CodeApplication.b(CodeApplication.java:35)

Hey! May I suggest using our latest release? Also instead of using a standalone Sign SDK it's advised to use our Web3Wallet SDK to have more integrations out of the box.
Currently latest version is
dependencies {
    implementation(platform("com.walletconnect:android-bom:1.4.1"))
    implementation("com.walletconnect:android-core")
    implementation("com.walletconnect:web3wallet")
}

Let me know if issue persist after upgrade

@Elyniss    ok, i integrate from u self demo, demo show the integratation with sign, so  I want to know the difference between these two   Thanks

You can see an example of the Web3Wallet integration in our new sample application. Here is the link for the same https://github.com/WalletConnect/WalletConnectKotlinV2/tree/develop/samples/web3wallet

You can see an example of the Web3Wallet integration in our new sample application. Here is the link for the same https://github.com/WalletConnect/WalletConnectKotlinV2/tree/develop/samples/web3wallet

thanks


feat: add Promise support

verify and hash returns Promise when callback is not provided
closes: #44

@ericelliott spelling mistake fixed


[WIP] Everybody gets a tracer

This PR has been dragging on for .. a while .. but I think it's probably worth merging now. The punchline is that every request (in the request-evaluator sense) now gets traced/profiled automatically,   and a fair quantity of their arguments do too. I also sprinkled a few others around places I thought might be interesting, expanded the entity vocabulary, and improved the formatting of entity names where possible. The trace logs and profiles are roughly readable/informative now.
(I tried to do quite a lot of template-metaprogram tricks to cut down on the number of explicit overloads and specializations relative to the vocabulary here, all of them either failed in some resolution-ordering corner case I couldn't figure out, or else broke the layering-violation-avoidance tricks, so wouldn't link. I'm happy enough with this state.)

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 9896b1b4526a510ffe9ea85584a5f1e8ebdecd24

Build failed
Swift Test OS X Platform
Git Sha - 9896b1b4526a510ffe9ea85584a5f1e8ebdecd24

@swift-ci please test

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 9896b1b4526a510ffe9ea85584a5f1e8ebdecd24

Build failed
Swift Test OS X Platform
Git Sha - 9896b1b4526a510ffe9ea85584a5f1e8ebdecd24

@swift-ci please test and merge


Align Repo Structure with AutoRest

PrivateDNS service = network
ResourceGraph service = resources
OperationalInsights = loganalytics
imageBuilder = compute
The discussion can happen whether these changes align with rest of the repos, but automation needs to not be blocked by the mis-alignment.

Can one of the admins verify this patch?


Enabling APEX causes link errors

I get errors of this kind related to boost regex when I turn on APEX support in HPX
https://gist.githubusercontent.com/biddisco/5e081113df14bc76ed61afdfd47ebae7/raw/745baf6ddada9e71757a304bdac87282d4a025f1/gistfile1.txt
I suspect @khuck is the person to ask about this.

John -  Can you post the Cmake options you used? I haven't seen this problem.
Also, are you using the latest HPX? There was a recent change to pull the most recent code from APEX that includes OTF2 support, which has been tested with HPX - but without the communication events.

I'm using master from today. Rechecking everything because I turned off APEX and the link errors didn't go away, so maybe some other option I used is the problem.

@biddisco Please verify that libhpx.so is being linked against the correct version of the Boost.Regex library.

Please accept my apologies - it turns out to be a problem that is caused when you allow HPX to set the build flags, if I explicity force CMAKE_CXX_FLAGS=-std=c++11 then things build ok, but if I do not use that flag, HPX must set some c++14 flags and the link problems appear - I would guess that there is some #define in boost that is causing problems because I tried building boost with c++14 and hpx with c++14, but that didn't work.
I will investigate this further when I have more time, for now I will close this issue.


[flaky test] test_permissions_can_be_removed

https://drone-publish.rancher.io/rancher/rancher/2223/2/2

Appears to be fixed by https://github.com/rancher/rancher/pull/25457/commits/df6255822f3737cab806478ebbb93b37f839eb7b


header: gh907 fix vulkan.hpp detection of 64-bit arch

The generator that produced vulkan.hpp for 1.0.25 seemed to ignore the 64-bit architecture test, thereby breaking 64-bit builds.
The macro VK_DEFINE_NON_DISPATCHABLE_HANDLE is always defined according to vulkan.h.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

I've created https://github.com/KhronosGroup/Vulkan-Hpp/pull/29 to solve the issue in the generator.

This pull request was merged by cherry-picking commit 30f71a34992974494f66a6f939ca0e220005b127 to master.


Abuse Case Model

[ ] Create abuse case models based on those identified in the use case definition
[ ] Check them into source control in enpm613/docs

I added everything from the abuse case doc Wei Lun and I made in one commit.


The emcee start position generator fallback is a bad choice

As a way to generate start positions for the various walkers of the emcee sampler I find the maximum likelihood of the model, and then use the maximum likelihood parameters and perturb them by some random amounts drawn from a gaussian (on the same scale as the parameter, so a parameter that was 70kpc might be perturbed by 10-100kpc, and a parameter with value 0.01 would be perturbed on that scale).
This seems to work really really well, but I do have to do checks to make sure that all the start values are within the priors, otherwise the sampler gets quite upset. At the moment, those parameters that fall outside the priors are just redrawn from the uniform prior, without any reference to the maximum likelihood value.
I don't know why I chose to do it like this, but it was a bad decision and needs correcting/refining. I suspect its the cause of some of the bimodal posteriors I've been seeing, as well as long autocorrelation times (see issue #429)
What I should do is just draw random perturbed values for the parameters until they are all within the priors.

I have made the changes that I mentioned here already, and have also changed the scale on which the start positions are perturbed to be a factor of ten smaller than they were before, which has also really helped.
I'm going to close this issue now, although I think I might be able to make the start positions even more reliable by using a curve-fit fit rather than max likelihood, and also making sure there is no radius = 0 point, that is really screwing a lot of things up unfortunately.


can't figure out even a basic project...

trying to setup a basic project with latest jspm/builder/systemjs and it just won't work.
https://github.com/lookfirst/broken
npm install
gulp

@guybedford I gave you commit access. I'd love it if you could figure this out.

You too? simply going new Builder() in gulp blows up for me. You're much farther along.
var Builder = require('systemjs-builder')
gulp.task('build', function () {
  var builder = new Builder()
})

results in:
[16:59:48] Using gulpfile ~/work/clean-portal/client/gulpfile.js
[16:59:48] Starting 'build'...
[16:59:48] 'build' errored after 91 ms
[16:59:48] ModuleEvaluationError: undefined is not a function
    at needsPolyfill (/Users/nblack/work/clean-portal/client/node_modules/systemjs-builder/node_modules/traceur/bin/traceur.js:1879:34)
    at polyfillMap (/Users/nblack/work/clean-portal/client/node_modules/systemjs-builder/node_modules/traceur/bin/traceur.js:1889:9)
    at /Users/nblack/work/clean-portal/client/node_modules/systemjs-builder/node_modules/traceur/bin/traceur.js:1553:14
    at Array.forEach (native)
    at polyfillAll (/Users/nblack/work/clean-portal/client/node_modules/systemjs-builder/node_modules/traceur/bin/traceur.js:1552:15)
    at /Users/nblack/work/clean-portal/client/node_modules/systemjs-builder/node_modules/traceur/bin/traceur.js:3494:3

I cloned your project and tried ./ as the baseURL as I had to do that to get mine working (outside of gulp) but no dice for your issue. Also tried .. btw thanks for the angularjs-systemjs-seed project - I used that a lot.

That exception looks like some weirdness with a traceur default? I've also noticed that traceur gets added to my jspm devDeps... which seems kind of weird given that I don't want to use it and specify babel for things.

Yes Traceur is used for builder transformations.
@lookfirst as mentioned in gitter, using paths for setting extensions is no longer supported as in https://github.com/lookfirst/broken/blob/master/gulpfile.babel.js#L38. Rather remove the default extensions here.
@nathanboktae are you able to just require Traceur? Like require('systemjs-builder/node_modules/traceur/bin/traceur.js)?

If I use the jspm builder, then I get the "[10:42:06] TypeError: baseURL should only be configured once and must be configured first.", but if I use the systemjs-builder builder, then things are ok. I've updated the broken project mentioned above to show you.
There is also an issue where the source map urls that are generated at the top level are based at the wrong url root. it goes up one level of ../ too much.

@nathanboktae are you able to just require Traceur? Like require('systemjs-builder/node_modules/traceur/bin/traceur.js)?

Nope I'm not - which leads to some odd interaction between gulp and Traceur, so I didn't file an issue here. Also systemjs-builder still needs it even though you specify babel as the transpiler, or don't use ES6 code.

@lookfirst thanks, can you share a test case by any chance with the source map URL issue?
@nathanboktae that's really strange you can't even require Traceur in Node. It sounds like an interaction unrelated to jspm, but if you can share the code I can try to take a look. As mentioned in the previous comment we always use Traceur for additional code transformations during builds, even when transpiling via Babel.

@guybedford clone the broken project that I mentioned before, npm install and run gulp and look at the generated map file.

On Jul 12, 2015, at 3:14 AM, Guy Bedford<EMAIL_ADDRESS>wrote:
@lookfirst thanks, can you share a test case by any chance with the source map URL issue?
@nathanboktae that's really strange you can't even require Traceur in Node. It sounds like an interaction unrelated to jspm, but if you can share the code I can try to take a look. As mentioned in the previous comment we always use Traceur for additional code transformations during builds, even when transpiling via Babel.
—
Reply to this email directly or view it on GitHub.

@lookfirst I've sent https://github.com/lookfirst/broken/pull/1 with your fixes.

Closing to follow-up separate issues as described in https://github.com/lookfirst/broken/pull/1#issuecomment-120992280.


package_info_plus still on version 4

According to #164 the package_info_plus version should be 5, but it's still on 4: https://github.com/zeshuaro/google_api_headers/blob/34d974f9586590021a21232fb4e3a5c404379c7d/pubspec.yaml#L13
Why? I maintain a package that uses google_api_headers, but I cannot update package_info_plus and my users want to use package_info_plus version 5, see https://github.com/martin-braun-net/google_maps_place_picker_mb/issues/73
Could you please release a new version with package_info_plus at ^5.0.0?

The PR you linked was reverted in #170 as the released version 5.0.0 was retracted a couple of days later. You can also see this from the changelog: https://pub.dev/packages/package_info_plus/changelog
Since pub.dev has a policy to allow package to be retracted within 7 days, this repo won't update packages until they have been released for 7 days.
But the latest version 5.0.1 was released 7 days ago, the Renovate bot (used to manage dependencies in this repo) should raise a PR soon to bump the version, which you can also see from the dashboard in #46.

Closed by #217


tus client for Go

Hi @Acconut, as mentioned I made a version of tus client for go lang.
It was tested with 1.6, 1.7 and 1.8 (tip).
The code can be found in github.com/eventials/go-tus.
Feel free to check out, comment, or even send a PR. The lib was based a little in the JS version, but I want to add more features soon.

Congratulations, this looks really, really well. However, I noticed that you require require a filename to determine what it going to be uploaded. In some cases, one would wish to upload even if it's not stored on disk, e.g. the body of a HTTP request. The tus-js-client, for example, allows you to pass a readable stream to it (see https://github.com/tus/tus-js-client#new-tusuploadfile-options).

Oh, I see. Well, I could take a look and use an io Reader or even a byte buffer.

@Acconut I changed the library, it's almost a new one. Take a look.

I like your outdated version even more, congratulations on it :+1: I can see a few similarities to the Java library :) Do you want me to feature it on the list of implementations (http://tus.io/implementations.html)?

@Acconut It's similar the Java version :) and also the Python version. I took the inspiration from they this time.
Feel free to list in the tus.io site.

Just added your project to the list (see Client section on http://tus.io/implementations.html). Let me know if it's ok that I used the name of the Github organization as the author :)

@Acconut Thanks.
@gfronza is it ok to use Eventials name right?


ci: Fix CES Migration workflow failing when skipped

The CES migration workflow was recently modified in commit ffb8443 to only execute if the changes in the tested commit are relevant to the tests executed in the workflow. If the setup-and-test job is skipped based on this precondition, the commit-status-final job will fail to execute properly. This is because the status of the setup-and-test job is piped directly into the status parameter of the set-commit-status action, which does not support the value "skipped".
This commit removes the commit-status-final job from the workflow, as this job is only needed if the workflow is executed via a workflow_dispatch trigger that takes a SHA as an input. Additionally, the workflow does not contain a corresponding commit-status-start job, indicating that the commit-status-final job may not be needed.
An example failure can be found here:
https://github.com/cilium/cilium/actions/runs/9601333957/job/26479710471?pr=33288. The logs for the failed commit-status-final job are:
Error: state must be one of "error", "failure", "pending", "success"

Fix bug in CES migration workflow causing it to fail when it should be skipped.

cc @jshr-w

/test


React Hooks: Flickering in exercise 2

Link to exercise
In this exercise, it states:
  🚨 If you implement useEffect but see the trivia
  🚨 flickering on the screen, there is still more
  🚨 work that you need to do!

However, the solution also flickers. Was the solution meant not to flicker? Would love to see how this would be done.

For record keeping, this issue will be resolved by #22 and #23
Thank you for addressing this issue! We didn't hit the API rate limit when testing locally, but it sounds like there was a rate limit after all.
Thank you for putting up those two pull requests to fix this issue, I'll merge them in!


Ignore autogenerated test scenes

Reasons for making this change:
Test scenes is auto-generated. These files should ignore.
Links to documentation supporting these rule changes:
TODO
Sorry, I don't understant this section.

Thanks for the PR, but you failed to follow the template provided. See the project's README for help.


fold enrichment

When building signal track with MACS2, the FE signal in genome range where the coverage is very low will become very significant, because the denominator is near zero in that range. Could you assign a weight to the low coverage range  to avoid this artifact ?

When building signal track with MACS2, the FE signal in genome range where the coverage is very low will become very significant, because the denominator is near zero in that range. Could you assign a weight to the low coverage range to avoid this artifact ?

Sorry, logLR signal has already done that.

When building signal tracks, if you used 'bdgcmp' command, you can assign a bigger pseudocount to overcome such issue -- "-p PSEUDOCOUNT".

I have my own bedGraph files for control and treatment. There are zeros in control file and I would like to generate ppois signal track (-m ppois).
So I applied "-p PSEUDOCOUNT" and tried a few pseudocount 0.00001, 0.05, 0.1... But none of them actually works. The following error always appears:
Exception AssertionError: 'Lambda must > 0, however we got 0' in 'MACS2.Prob.poisson_cdf' ignored
Could you please help? Thanks!

@YiqunCao See my reply at #265


App builds on Windows but not Mac OS

Ionic version:  (check one with "x")
[ ] 1.x
[ x] 2.x
I'm submitting a ...  (check one with "x")
[ x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
When building an Ionic project in Windows, it builds and runs fine using ionic serve, but the same project on Mac OS causes a missing main.js using sudo ionic serve. No build errors showing whatsoever.
Expected behavior:
main.js should be present.
Ionic info - Windows:
Your system information:

Cordova CLI: 6.3.1
Ionic Framework Version: 2.0.0-rc.1
Ionic CLI Version: 2.1.1
Ionic App Lib Version: 2.1.1
Ionic App Scripts Version: 0.0.38
OS:
Node Version: v6.9.1

Ionic info - Mac OS:
Your system information:

Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.1
Ionic CLI Version: 2.1.13
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.38
Ios-deploy version: Not installed
Ios-sim version: Not installed
OS: macOS Sierra
Node Version: v6.9.1
Xcode version: Xcode 8.1 Build version 8B62

Note that you shouldn't need to sudo. And depending on your config this is a question for https://github.com/driftyco/ionic-app-scripts.

This issue was moved to driftyco/ionic-app-scripts#498


Add a bitset newtype wrapper

These would function the same as other set instances or n-tupled bool instances but efficiently via Bits and FiniteBits.
newtype BitSet a = BitSet a
    deriving (Eq, Bits, FiniteBits)
--  Should probably show/read as binary or a set of set bits (latter doesn't have read . show = id since it can re-order)
instance Bits a => PartialOrd (BitSet a) where
--  Avoiding complement on purpose (see finitebits note), there may be a more concise solution
    a `leq` b = (zeroBits == (a .&. (a `xor` b)))
instance Bits a => Lattice (BitSet a) where
    (\/) = (.|.)
    (/\) = (.&.)
instance Bits a => BoundedJoinSemiLattice (BitSet a) where
    bottom = zeroBits
-- see oneBits note in base 4.16 on ghc's gitlab
-- https://gitlab.haskell.org/ghc/ghc/-/blob/master/libraries/base/Data/Bits.hs#L79
instance FiniteBits a => BoundedMeetSemiLattice (BitSet a) where
--  top = oneBits -- 4.16
    top = complement zeroBits
instance FiniteBits a => Bounded (BitSet a) where
    maxBound = top
    minBound = bottom

(Natural is an example of a Bits that is not a FiniteBits, and is undefined for complement)

Makes sense. PR welcome.

FWIW, I wanted to clean up Bits and FiniteBits such that complement wouldn't be in Bits, so Natural can be an instance without miss-behaving methods: https://mail.haskell.org/pipermail/libraries/2019-December/030132.html

How about Bitwise? I'm not sure if there might be some other useful Bits-based lattice other than wrappers applied to this however (like something acting as a nested Lexicographic on bits), and it doesn't really indicate the idea of a set.
I just added Bounded as they're bounds for the partial order, the docs on Bounded themselves say
"Ord is not a superclass of Bounded since types that are not totally ordered may also have upper and lower bounds"
I have not had much luck with a show instance only requiring Bits. What I really need is a bit size that's argument dependent rather than a constant (and for read a function to check an argument can get another bit). Basically a partial isomorphism with [Bool]. The instance is possibly more hassle than it's worth over using stock show/read (either way a show instance is required by the tests).


Thread safety

The PG gem on it's own is not thread safe, so when calling this backend from concurrent threads (e.g. Sidekiq or forking in Ruote), you'll get strange exceptions like this:

PG::UnableToSend: extraneous data in "T" message
PG::UnableToSend: connection not open

This adds a mutex around all PG#exec calls to make it thread safe. The Sequel backend uses a connection pool which will be more performant, but at least in our use case that won't be an issue.
All the specs here pass, but I'd like to run it against the Ruote test suite too. However on my setup with the default in-memory backend a lot of the Ruote tests fail. Any ideas @vjt @lleirborras?

Thaaaaaaanks


Adding a typescript pack

Adding a default pack for Typescript. I tried to assume as little as I could of the typescript configuration. Assumptions I had to make:

That the codebase can be built with tsc, and doesn't need some higher level tool (I didn't add support for ts-node)
That the user has all their dependencies installable with npm (i.e. in their package.json)
That the user has a complete tsconfig.json that doesn't need to be generated

@arschles,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@arschles just an FYI that if you want to pick this up again, the code for the default packs are now stored here and are generated thanks to go-bindata.

ping @arschles, any time you might have to take a look at this again? If not no worries, I can pick this up at some point. I'm looking to add more language packs to draft in the next release.

@bacongobbler yep, I'm back at it right now - sorry for the delay 😄

Codecov Report

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

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   32.97%   32.97%
=======================================
  Files          18       18
  Lines        1107     1107
=======================================
  Hits          365      365
  Misses        693      693
  Partials       49       49

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 9b4b1b8...03adcc9. Read the comment docs.

Except for RUN npm install -g typescript in the Dockerfile, this pack should be similar / identical to the node one. Wouldn't it make sense to have a single pack that covers both and include the typescript example?
Adding something like the following to the node pack Dockerfile could do IMHO:
RUN if [ -f tsconfig.json ]; then npm install -g typescript; fi

Adding to the comment above, since Visual Studio 2017 does not generate a tsconfig.json in the typescript node.js projects, the Dockerfile needs quite a few changes to address that as well. After playing with it a bit, I got:
FROM node:6-onbuild
RUN npm install -g typescript
RUN npm install --save @types/node
RUN npm install
RUN find . -name "*.ts" -not -path "./Scripts/*" | xargs tsc
EXPOSE 8080
ENV port 8080
ENV PORT 8080
CMD ["npm", "start"]

An alternative approach consists in running msbuild (requiring a container image which includes it).
Considering that this PR clearly states that a tsconfig.json needs to be present, should we create a separate pack or should we try to address both use cases in a single pack?

TypeScript is a different language that eventually compiles down into javascript but has a completely different toolchain. The same could be said for CoffeeScript, Dart, and Elm. I think it's fine to have a completely separate language pack for a superset of Javascript.
TypeScript itself documents that "The presence of a tsconfig.json file in a directory indicates that the directory is the root of a TypeScript project.". I think @arschles is fine to assume that if there exists a tsconfig.json in the root directory, then it is a TypeScript app.

Adding to the comment above, since Visual Studio 2017 does not generate a tsconfig.json in the typescript node.js projects, the Dockerfile needs quite a few changes to address that as well.

I would argue that it's not our responsibility to comply 1:1 with Visual Studio's project structure for a TypeScript app. I would say that it is our responsibility to comply to what the actual TypeScript community defines the project structure.
Now saying that, I do realize that there are a fair few TypeScript trending repos on GitHub that do not define a tsconfig.json. Those that do not define one usually have some other file that we can check for:

tsfmt.json
tslint.json

If we check for the presence of one of those three files, I think we should be able to deploy most TypeScript apps.
If there's a better way to check for TypeScript apps, please let me know.

About detecting the language, since the heuristics involve include a lot of potentially complex guesswork, what about using Linguist, which is what Github uses? There's a Go porting which, if on par with the Ruby one, IMO would fit pretty well here.

About the build tooling, since tsconfig.json is not mandatory, there's also quite a lot of guesswork involved if we want to automate it (gulp, visual studio, custom build.sh, etc).
For the time being, if typescript is detected but a tsconfig.json is not present, an option could be to just insert a note in the Dockerfile for the user to add their specific build commands, another one could be to just build every ts file in it: RUN find . -name "*.ts" -not -path "./Scripts/*" | xargs tsc

There's a Go porting which, if on par with the Ruby one, IMO would fit pretty well here.

That's a good thought, but it's a CLI port rather than a client library. That would require some overhaul on the project side. However, that would be a good enhancement to the project. I'll file a new ticket for that.

#205 for tracking that discussion.

I'll leave it to @arschles to discuss implementation details with you over https://github.com/Azure/draft/pull/92#issuecomment-311227653. I'm not sure what's the best path forward on this one.

Since I unfortunately have not had time to look at this in months, I am going to close this. I have added https://github.com/Azure/draft/issues/448 to track further discussion.


Difference in Output Between PDF and DOC File Upload Methods

Description of the bug:
I'm trying two methods for file uploads:
PDF File Upload:
DOC File Upload:
Despite both files containing the same content, I'm noticing differences in the output. Specifically, the output response from the DOC file upload method lacks a lot of information and steps, while the PDF method works great.
Actual vs expected behavior:
Here's what I'm doing for each method:
1. PDF File Upload:
Since there is no support for direct PDF file upload, I'm converting each PDF page to an image, extracting text from the PDF, and then doing the following:
`files.append(genai.upload_file(img))
for page, (text, image) in enumerate(zip(texts, files)):
instruction = f"{instruction}\nPage {page+1}\n------------\n{text}\n{image}"
chat_session = model.start_chat(
history=[
{"role": "user", "parts": [instruction]},
]
)
response = chat_session.send_message(prompt)
print(response.text)
`
2. DOC File Upload:
For the DOC file, I'm uploading it directly to Gemini:
files.append(genai.upload_file("test.docx")) instruction = f"Usage document:\n{file}" chat_session = model.start_chat(     history=[         {"role": "user", "parts": [instruction]},     ] ) response = chat_session.send_message(prompt) print(response.text)
Any other information you'd like to share?
Can anyone help me understand why there is a difference in the output between these two methods? The content in both files is the same, but the response from the DOC file upload lacks information and steps compared to the PDF method. Any insights would be appreciated!

@anusonawane
Thank you reporting this issue.
This repository is for issues related to Gemini cookbook Examples/Quickstarts bugs or improvements. For Issues/Questions related to Gemini API, we would suggest you to use "Send Feedback" option in Gemini docs. Ref: Screenshot below.
You can also post this issue on Google AI forum.

It sounds like you're encountering a discrepancy in the output when uploading PDF and DOC files, even though the content is identical. Here are a few potential reasons for the differences and some suggestions for troubleshooting:
Possible Reasons for Output Differences

File Parsing Differences:

The libraries or methods used to extract text from PDFs and DOC files might handle formatting and content extraction differently. PDFs can often contain more complex layouts, which might be better processed when converted to images.

Content Extraction Method:

In your PDF method, you're converting pages to images and explicitly extracting text, which may allow for better control over what information gets included in the final output. The DOC upload might not be extracting all the necessary details due to how the content is structured.

Input Structure:

The way you structure your input for the chat session might affect how the model interprets the content. Ensure that the instruction format is consistent between both methods.

Model Limitations:

The model might have different capabilities or settings when processing different file types. It could be optimized for one format over another.

Suggestions for Troubleshooting

Check the Extraction Process:

Verify that the text extraction from the DOC file is capturing all relevant content. You could print the extracted text before sending it to the chat session to compare it with the PDF output.

Standardize Input Format:

Make sure that the instruction format is as similar as possible between the two methods. This includes how you concatenate instructions and how you present the files.

Debugging Output:

Print the responses from both methods at each step to identify where the information starts to differ. This can help pinpoint if the issue lies in the extraction or the chat session.

Consult Documentation:

Look into the documentation for the libraries or APIs you are using to see if there are any notes on limitations or differences in handling various file types.

Experiment with Different Formats:

If possible, try different formats for the DOC upload (like DOCX vs. DOC) or use different libraries for text extraction to see if that changes the output.

By following these steps, you should be able to identify the source of the discrepancy and potentially improve the output consistency between the two file upload methods. If the issue persists, consider reaching out to support for the specific tools or libraries you’re using for further assistance.


Integrate LLVM at llvm/llvm-project@30e6033b455b

Update BUILD files and submodule for
30e6033b455b

I'm adopting LLVM's post-submit review system :-D

=/ Sorry I was a bit slow, but LGTM.

Oh no worries. I didn't wait long, just looked at your calendar and decided it was better to keep the build green :-D


[BUG] Can not run php with xdebug or any arguments for php

I tried this code:
rpc:
  listen: tcp://<IP_ADDRESS>:6001

server:
  command: "php -dzend_extension=xdebug.so bin/console baldinof:roadrunner:worker"
  relay: "unix://var/roadrunner.sock"

http:
  address: "<IP_ADDRESS>:8080"
  pool:
    num_workers: 4

<code>

We have basic image with xdebug. We disable it by default for performance reason.
I expected to see this happen: run php with xdebug extension, in earlier version < 2.0.0 it was running as expected
Instead, this happened: server_plugin_cmd_factory: stat : no such file or directory
The version of RR used: 2.0.0
Errortrace, Backtrace or Panictrace
2021/03/04 12:29:18 handle_serve_command: Serve error:
        endure_start:
        endure_serve_internal: Function call error:
        endure_call_serve_fn: got initial serve error from the Vertex http.Plugin, stopping execution, error: http_plugin_serve:
        server_plugin_new_worker_pool:
        server_plugin_cmd_factory: stat : no such file or directory

<backtrace>

Mb it is no sense of checking for file existing?

@sudevva Yeah, I guess I'll lay on the provided command without limitation.

Thank you!

You are welcome :+1:


chore: create initial draft of documentation in README.md

This PR adds some basic documentation that explains how to use the functions provided by this library.

Here is a link to a rendered version of the markdown: https://hackmd.io/@jmPpL1ukSO6T9klrhuvriA/ryW_ldCqm


how to obtain input.json

Hey, I’m new to zkML. I want to try compiling my Keras model into Circom, and I found this repository. Great work!
I’ve successfully obtained circuit.circom, circuit.json, circuit.wasm, and generate_witness.js. However, I’m now struggling with creating the witness because I don’t understand what input.json should look like.
My model is LeNet-5, with the following configuration:
def build_lenet5_relu(input_shape=(32, 32, 1), num_classes=4):
    model = models.Sequential()

    # Input layer
    model.add(layers.Input(shape=input_shape))

    # First Convolutional Layer
    model.add(layers.Conv2D(6, kernel_size=(5, 5), strides=(1, 1), padding='same', activation='relu'))
    model.add(layers.AveragePooling2D(pool_size=(2, 2), strides=(2, 2)))

    # Second Convolutional Layer
    model.add(layers.Conv2D(16, kernel_size=(5, 5), strides=(1, 1), activation='relu'))
    model.add(layers.AveragePooling2D(pool_size=(2, 2), strides=(2, 2)))

    # Third Convolutional Layer
    model.add(layers.Conv2D(120, kernel_size=(5, 5), strides=(1, 1), activation='relu'))

    # Flattening Layer
    model.add(layers.Flatten())

    # Second Fully Connected Layer
    model.add(layers.Dense(num_classes, activation='softmax'))

    model.output_names=['output']

    return model

I assumed that I should merge your circom.json with "in" as the input, so I did:
image = Image.open(<input-image-path>)
image_array = np.array(image)

x = np.expand_dims(image_array, axis=0)

data_array = x.reshape([32, 32, 1]).tolist()

with open(<path-to-circuit-json>, 'r') as circuit_file:
    input_json = json.load(circuit_file)

input_json["in"] = data_array

with open(<path-to-input-json>, 'w') as input_file:
    json.dump(input_json, input_file, indent=4)

But when I generate the witness using generate_witness.js, I get this error:
Error: Not all inputs have been set. Only 53640 out of 85137

I suspect there might be an issue with the signals in my input.json, probably wrong format?. Do you have any clue?
Thanks for your response.

Hey, I managed to find the solution.
It turns out I need to first create output.json by running circuit.py, then merge circuit.json, output.json, and the input data under the key "in" to create input.json
Thanks for this amazing work, I'm closing this now


I wanna lint lint lint

Description:

Slowly adding missing rules
not all the necessary rules are already here, it's a process

Thank you! 🚀

For contributor:

[ ] Added entries in CHANGELOG.md with issue # and GitHub username
[ ] Reviewed Files changed in the github PR explorer
[ ] Attach screenshots of the UI components on the PR description (if applicable)
[ ] Scope of work approved for big PRs

For reviewer:

[ ] Manually tested the changes on the UI

That's funny XD coverage missing after linting

@sabau is your plan to fix and revive this PR or should we close for now?

@sabau ping ping ping

closing because no response


config cluster for odfe

Hi Guys. Currently, I'm having a problem that I install odfe in 3 nodes. But when I do curl a node in of 3. It only shows 2.
Here is my config:
cluster.name: odfe-test
node.name: "node-4854" # I use different node name for this on each node
node.master: true
node.data: true
node.ingest: true
path.data: /data/odfe/data
path.logs: /data/odfe/log
bootstrap.memory_lock: false
network.host: <IP_ADDRESS> # I use different ip for this config
http.port: 9200
discovery.seed_hosts: ["<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>"]
cluster.initial_master_nodes: 2

######## Start OpenDistro for Elasticsearch Security Demo Configuration ########
# WARNING: revise all the lines below before you go into production
opendistro_security.ssl.transport.pemcert_filepath: esnode.pem
opendistro_security.ssl.transport.pemkey_filepath: esnode-key.pem
opendistro_security.ssl.transport.pemtrustedcas_filepath: root-ca.pem
opendistro_security.ssl.transport.enforce_hostname_verification: false
opendistro_security.ssl.http.enabled: true
opendistro_security.ssl.http.pemcert_filepath: esnode.pem
opendistro_security.ssl.http.pemkey_filepath: esnode-key.pem
opendistro_security.ssl.http.pemtrustedcas_filepath: root-ca.pem
opendistro_security.allow_unsafe_democertificates: true
opendistro_security.allow_default_init_securityindex: true
opendistro_security.authcz.admin_dn:
  - CN=kirk,OU=client,O=client,L=test, C=de

opendistro_security.audit.type: internal_elasticsearch
opendistro_security.enable_snapshot_restore_privilege: true
opendistro_security.check_snapshot_restore_write_privileges: true
opendistro_security.restapi.roles_enabled: ["all_access", "security_rest_api_access"]
cluster.routing.allocation.disk.threshold_enabled: false
node.max_local_storage_nodes: 3
######## End OpenDistro for Elasticsearch Security Demo Configuration ########

My curl result: https://<IP_ADDRESS>:9200/_cat/nodes?v
ip         heap.percent ram.percent cpu load_1m load_5m load_15m node.role master name
<IP_ADDRESS>           17          81   3    0.99    0.45     0.37 dim       -      node-4854
<IP_ADDRESS>           15          49   2    1.81    0.82     0.52 dim       *      node-4856

Your cluster is splitting into two because there are three eligible masters and one is voting for itself to be the master of a new cluster on initialisation.
This is a pretty well documented issue, which can usually be solved by explicitly setting one as the leader at initialisation (use cluster.initial_master_nodes: ["node-names",...]) or using minimum_master_nodes: 3 so that the cluster doesn't initialise until all three are booted and discovered.
Check that link for some caveats to the first approach as it can cause issues if applied to a running cluster.

Yes. Thank you so much. Also, i have to delete the data folder and do follow what you said.


docs: developers generally mean appdevs

Rename "Developer Guide" to "Contributor Guide" because developers generally mean appdevs. They have different meaning for audiences.
Move "Contributor Guide" to follow "Reference". We talk everything user aspect and then for contributors.

cc @killme2008 @alili

Thanks and appreciate to @tisonkun for the great idea of this PR. We plan to change the Developer Guide to Contributor Guide in version 0.5 of the docs. I will create a PR which contains this change in the next few days. So maybe we can close this PR 😉 。

@nicecui Thanks for your feedback! Then I'll understand that you will include the rename and reorder in the next version when you cut it.
I'll close this PR as won't do in 0.4 and start a new PR includes other fixups (better links majorly)


[Feature] Sampler Selection Field

Users can now choose their desired sampler from the main settings page with our new Sampler Selection Field.
p.s: now it uses Euler A

Sweet! Great work on this and, seriously, way to keep it simply. Would love to see: simple rotation option, simple drag n drop audio track option so we can throw on a .mp3/.aac for a small soundtrack on the export.
A bit more creative but a really cool thing would be (simple, ha) ChatGPT API key plugin where I can just have it fill in both frame number + prompt row, either entirely from nothing or perhaps just 'picking up' from whatever your first prompt says. Haven't seen too many straightforward ChatGPT/OpenAI API key implementations in SD plugins but could work so well.

Hi @jet3004,
Thank you for your valuable input and suggestions. Your ideas are great and align with our long-term goals. However, currently, we are occupied with more pressing features that need to be implemented first. I suggest you write your idea in the Discussion section, where it can be reviewed and prioritized at the right time.


Not able to change the state of switcher

Hi,
I need to tap on switcher to change the state, i tried the below code
WebElement element2 = driver.findElement(By.xpath("//Switcher[@id='preferences_play_ringtone_switcher']"));

Actions action = new Actions(driver);
action.moveToElement(element2).moveByOffset(150, 170).click().perform();

But its throwing the error selendroid doesn't support this.
Is there any way to change the state of the switcher, i have many use cases to test based on switcher state change.
Please provide me some solution for this.

I'm closing this issue due to inactivity.
If is a question rather than an issue, please send questions to the selendroid user group.
If you think it's an actual problem with selendroid and still experience the issue described on the latest release of selendroid feel free to file a new issue. If you do please provide a concise reproducible test case and describe what results you are seeing and what results you expect.


Fix "No object to concat" Bug and support Some corner cases

For both ANN and KNN:
(1) Support query_df is an empty dataframe
For ANN:
(1) Support item_df is an empty dataframe (root cause of triggering "no object to concat" error from pd.concat)
For KNN:
(1) Select relevant columns (id, features) before calling union on query_df and item_df to avoid schema mismatch error

build


Create a Preact component

What is this PR for?
This PR is for making a Preact component by converting the existing React component.
Tasks

[x] Switch react dependencies over to preact
[x] Reconfigure Typescript, Rollup, and Vite to use preact
[x] Fix imports to use preact
[x] Convert react component to preact. The hope is not to use preact/compat
[x] Update development Vite app
[ ] Update development next-app
[ ] Test everything and make sure it works

Sounds like a plan! And hey I am also on a train right now 👀

Sorry for the delays! I have removed the alias for react and reduced the usage of preact/compat. I am unsure what createPortal is, but it is the only thing using preact/compat. I also ensured the code builds, fixed the development Vite app to use preact, and tested running it.

Sounds great 👏
I'll have a look today. createPortal is used to inject custom components into empty placeholder-divs, to override the markup for things like the events, the modal or parts of the header:
https://schedule-x.dev/docs/calendar/react#custom-components

I noticed now that I ran it, that we need to solve this error:

Probably in the useCalendarApp hook, we need to use @preact/signals, since in Preact we can't use hooks outside of the component. I think also we can remove the useNextCalendarApp

Of course! Changing that to a signal should be pretty easy.
I believe the persistence and double rendering comes from the hook being used outside of a component. I have found that sometimes things will have persistence issues and double (or triple) render when a breaking but non-fatal error occurs (like using a hook outside of a component). I'll have to take a look though to be sure though.

Making it a signal fixes the issue, however, it means only one calendar can be created (all new calendars will be exactly the same)
Having the signal inside the useCalendar function still creates the error and must be outside of it.


Alarms cannot be set

Version: 20170314.23 (TestFlight)
Tried using both WiFi and cellular connections, failed on both occasions, however, may just be an isolated problem. Logs show nothing out of the ordinary, just basic HTTP stuff. See attached screenshots.

Fixed with https://github.com/OneBusAway/onebusaway-deep-links/pull/22

Also, I've opened up #1000 to improve the messaging in this alert. @ualch9, if you have any free time, this should be an easy enhancement to knock out. Thanks for your bug report!

@aaronbrethorst sure thing, i'll take a look when i get the chance.


refactor(lint): migrate eslint-plugin-jest(expec-expect)

Current dependencies on/for this PR:

main

PR #1208

PR #1225   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 11, 10:15 PM: @Boshen started a stack merge that includes this pull request via Graphite.


Add option to change Side Panel Position

I would like an option to choose the side panel position in the configuration of the watchdog plugin. It would be a number input box, 0 would put the side panel icon on top of all other icons, every number after would put the icon under the next icon.
This feature already is in the 'Bossing Info' plugin.
Example: https://imgur.com/a/LN2KmQc

I could probably do something like that. I've been wanting that to be in core for so long. Wouldn't it be nice to just be able to drag them around?

Looks like it's being theoretically being worked on here https://github.com/runelite/runelite/pull/18406


Create zakir-siddiqi-2014.md

creating zakir data science transcript

Closing until you have a completed transcript, at which point you should reopen.


Spacedrive

A tracking list for moving Spacedrive back onto rspc:
Required

[ ] Finish breaking jsonrpc executor out of the core
[ ] New procedure syntax backported
[ ] Specta v2 support (+ remove Result from it)
[ ] Ensure no blocking in batches, or Axum websocket or Tauri links cause these bugs have been far too common.

Nice to haves

[ ] Subscription start/complete messages (would help with the streamed queries mess)
[ ] Streamed queries
[ ] Streaming within batches. (with websockets specifically).
[ ] Invalidation system
[ ] Tracing
[ ] File streaming
[ ] File uploads

Spacedrive is no longer my problem.


Sometimes clipboard fails to be read in Chromium browsers

If after trying to get the clipboard using one tab we get an error I should try with a different tab, if available. If we fail to read the clipboard altogether we should notify the user.

I managed to find a more robust solution using offscreen documents.


swc-windows-installer.py: Allow R-3.2.4revised\bin

On Thu, Apr 14, 2016 at 12:41:01PM -0700, Ethan White wrote:

At a workshop I just ran we ran into issues with R not being in the
path on a number of Windows computers. In the past this has been
caused because R likes to change the path where it installs itself
and we've needed to change the function that searches for R. The new
path that is causing issues is:
C:\Program Files\R\R-3.2.4revised\bin

Fixes #44.

Is it truly necessary to be even this specific / strict in regexp?  Why not just R-.*/bin?  Note: I know next to nothing about R and don't know if there are other paths we don't want that could conceivably match that regexp.

On Fri, Apr 15, 2016 at 05:23:07AM -0700, Erik Bray wrote:

Is it truly necessary to be even this specific / strict in regexp?
Why not just R-.*/bin?  Note: I know next to nothing about R and
don't know if there are other paths we don't want that could
conceivably match that regexp.

I'm in the same boat on R.  A more relaxed regexp would probably be
fine, but staying stricter means:

We're less likely to pull in the wrong thing, although that's
unlikely since we require one path component that's just ‘R’.
We can add version comparisons (since all currently-reported paths
contain version information) if we want to pick the newest R, or
some such.

The downside is that we sometimes hit situations like #44.  This is
the first bump to that regexp since it landed in 58b7183ea
(swc-windows-installer.py: Look for an R bin directory, 2014-06-09),
so I don't think this is a high risk.

Yeah, looking again at your new regexp, it seems pretty unlikely to fail again, and pulling out the version number could be useful :)

@wking Do you have merge access on this repo?

On Mon, Jun 13, 2016 at 11:31:16AM -0700, Erik Bray wrote:

@wking Do you have merge access on this repo?

I do, but I don't like to merge my own PRs ;).  If you don't, we
should get it for you too.

I do, but I don't like to merge my own PRs ;).

Yep. same.

If you don't, we should get it for you too.

Please do!

On Mon, Jun 13, 2016 at 12:58:59PM -0700, Erik Bray wrote:

If you don't, we should get it for you too.

Please do!

@gvwilson, or other SWC admin: can we get write access to this repo
for @embray?

Done!


freeipa: init at 4.4.3

libverto: init at 0.2.6
pythonPackages.nss: init at 1.0.0
pythonPackages.pki: init at 10.3.5
pythonPackages.lesscpy: init at 0.12.0
pythonPackages.python-yubico: init at 1.3.2
Changed bind to optionally build with Kerberos support.
Changed krb5 to build with system libverto by default.
Motivation for this change
This changeset contains all packages and other changes to turn NixOS into a fully capable, well-behaving FreeIPA client and domain member. While only the client portion of FreeIPA is fully implemented, some of the work for building the server component has also been done to actually make the client build succeed.
FreeIPA needs several packages to be build with MIT Kerberos, which in turn needs to use a newer version of libverto than the one bundled with krb5. Because of transitive dependencies, Samba 4 also needs to be build with that same version of MIT Kerberos. The changeset takes a stance that krb5 should much rather always be built with system libverto, which will trigger a mass-rebuild but save time and space in the long run because no duplicate versions of krb5 dependents are needed.
Building with libverto is explicitly switched off for fetchurl's curl derivation to prevent a circular dependency.
The usual way to join a host to IPA is via freeipa-client-install, which normally modifies and backs up many different files in /etc, which is not possible on NixOS, for obvious reasons. Therefore, I created a nix module that does the same job idiomatically. The only manual steps for a user is to prefetch the certificate from the server and specify the resulting hash in the configuration, as seen below. After activating the configuration, they need to join the domain and obtain the keytab file manually, for which multiple different methods exist, with the most common one presented in the build output.
A typical, a valid set of FreeIPA configuration looks like this:
{
  networking.hostName = "nixos.in.foo.com";

  ipa = {
    enable = true;
    domain = "in.foo.com";
    realm  = "IN.FOO.COM";
    server = "ipa.in.foo.com";
    basedn = "dc=in,dc=foo,dc=com";
    certificate = pkgs.fetchurl {
      url = http://ipa.in.foo.com/ipa/config/ca.crt;
      sha256 = "xxx";
    };
  };

  services.sssd.enable = true;
  services.ntp.enable = true;
  security.pam.services.login.makeHomeDir = true;
  security.pam.services.su.makeHomeDir = true;
}

The easiest way to test this is to create and set up a FreeIPA instance in a Fedora VM or docker container and build a NixOS version via nixos-container, bind-mounting nixpkgs like so:
sudo nixos-container create ipa-test
sudo mkdir -p /nix/var/nix/profiles/per-container/ipa-test/per-user/root/channels/nixos
sudo mount --bind /path/to/nixpkgs /nix/var/nix/profiles/per-container/ipa-test/per-user/root/channels/nixos
sudo nixos-container start ipa-test
ln -s /path/to/nixpkgs /path/to/nixpkgs/nixpkgs # Yes, this is required!
sudo nixos-container root-login ipa-test
ipa-test> nix-prefetch-url http://ipa.in.foo.com/ipa/config/ca.crt
ipa-test> cat > /etc/nixos/configuration.nix <<'EOF'
{ config, lib, pkgs, ... }:

with lib;

{ boot.isContainer = true;
  networking.hostName = "ipa-test.in.foo.com";
  networking.useDHCP = false;

  ipa = {
    enable = true;
    domain = "in.foo.com";
    realm  = "IN.FOO.COM";
    server = "ipa.in.foo.com";
    basedn = "dc=in,dc=foo,dc=com";
    certificate = pkgs.fetchurl {
      url = http://ipa.in.foo.com/ipa/config/ca.crt;
      sha256 = "xxx";
    };
  };

  services.sssd.enable = true;
  services.ntp.enable = true;
  security.pam.services.login.makeHomeDir = true;
  security.pam.services.su.makeHomeDir = true;
}
EOF

ipa-test> nixos-rebuild switch

I'm well aware this is not a trivial change but I believe the benefit makes up for it and I'm grateful for reviewers who feel like they could tackle this.
Things done

[X] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

tests of nss and pki are not run

I've addressed all of the issues you pointed out, @FRidh. pki-core doesn't have any tests for Python. All nss tests are now being run, but for one of them, an SSL server is started on port 1234, and I'm not sure what NixOS' policies are on tests like that.

Ping @FRidh

We have a local freeipa server. I hope I can get around to test this soon. Thanks for your work on this!

Will this lead to allowing NixOS being a freeipa server as well?

I think not (from first post):
While only the client portion of FreeIPA is fully implemented, some of the work for building the server component has also been done to actually make the client build succeed.

I'd be interested in getting FreeIPA integrated as well!
@e-user: How can I help? It would be a shame to loose all your hard work and effort!
@FRidh, @fpletz: What are your current thoughts and comment about it? Should we pick it up?

@ts468 I've only reviewed the Python expressions for which only minor changes are needed. Eventually separate commits are needed as well.

(triage) @e-user, are you still willing to push this forward? :)

Actually, I am interested again, @Ekleog. I'll look into this again over the next days.

I, too, am interested here. Anything I can assist with?

OK so the push above only takes care of the obvious merge conflicts and @FRidh's requested changes, but I haven't tested the actual implementation after the rebase yet and am going to do so within the next days.

OK, so, there's a circular dependency because of curl now depending on libkrb5, which this changeset needs to be built on a proper (as opposed to bundled) version of libverto. Testing will take me a while because of the amounts of rebuilding involved.

@FRidh Could you please check whether all of your previous remarks in the review have been addressed?

What is the status of this pull request?

@mmahut Good question. I've been trying to get this one in literally for years now.

We are very much interested in getting this in

So yeah, what's left to do? All issues had already been resolved before master moved on, again. I will happily resolve the conflicts, but I would appreciate if that could resolve in an actual merge. Alternatively, if anyone (@poelzi?) would prefer to run a real-life test against an actual server, I'll gladly provide guidance on how to do this, if so desired.

I'm open to test this if you resolve the conflicts.

Apart from the merge conflicts, is there anything that is keeping this from being merged? Would be great to be able to have NixOS machines as FreeIPA clients

ping ^^

@FRidh I hope I am not asking for too much, but since you seem to have merge rights and are already familiar with the pull request, would it be possible that you coordinate the inclusion of this package? There hasn't been any activity on this pull request since half a year and it seems as if there are no missing required changes, apart from the conflicts of course

Closing since author did not respond in the last months. If this is still important to you please reopen the discussion here.

Closing since author did not respond in the last months. If this is still important to you please reopen the discussion here.

Still important to me.

Still important to me.

@bjornfor yeah to you but not to @e-user because he did not respond in 18 months to this. Feel free to pick up the work.

@bjornfor yeah to you but not to @e-user because he did not respond in 18 months to this. Feel free to pick up the work.

Yeah sorry guys. I don't even have a NixOS installation anymore, at all, so I can't really work on it. If anyone needs help finishing this, let me know.

Yeah sorry guys. I don't even have a NixOS installation anymore, at all, so I can't really work on it. If anyone needs help finishing this, let me know.

I'm vaguely considering picking this up.  Before I dive in, does anyone know if there has been other related work that would significantly affect this one?

Almost two years. Just suppose that no one knows.

This is a package database. It will always be interesting to include a package to it.

I'm picking this up. I will open a new PR once I get things working...

Please see #207115 which supersedes this.


⚠ Bofrost.be has degraded performance

In 77371e7, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1743 ms

Resolved: Bofrost.be performance has improved in 243727d.


Create a new path_utils library in kythe/cxx/common that doesn't depend on llvm

I'll need these utility functions for the proto indexer.

May I humbly request a more descriptive commit log for this change? What's there makes sense, but doesn't give much context. A line or two about why we're pulling this out into our own library would make future code archaeologists less likely to curse our names. :)

@creachadair, sure thing - I'll add some more detail to the squash/merge commit message.


ZcashMiner::parseJob(): Invalid or unsupported block header version

It now happens a lot:
[16:20:16][0x00007fd408b11700] stratum | Connecting to stratum server equihash.eu.nicehash.com:3357
[16:20:16][0x00007fd408b11700] stratum | Connected!
[16:20:16][0x00007fd408b11700] stratum | Subscribed to stratum server
[16:20:16][0x00007fd408b11700] miner | Extranonce is<PHONE_NUMBER>
[16:20:16][0x00007fd408b11700] stratum | Authorized worker 1BgEDmdXTsUBgo4ejKXWrFoAdctNc....

Do someone knows how to resolve this error?

It seems that the best way to solve it is to wait. Somebody is sending a wrong block (with a wrong header).

What do you mean?


Twitter uses old icon

Great package, thanks! Just mentioning that the Twitter logo used is the old one (beak pointing right). Your screenshot shows the new one (beak pointing up-right) but that isn't what outputs when I use this package. Thanks :-)

Hi @GitHubGreg, thanks for the issue! I will update the twitter icon and make a new release as soon as I can.

Hi @GitHubGreg, I have just published a new version of react-social-login-buttons, which should contain the new Twitter Logo.
Thanks for contributing!

Thanks for the quick turnaround!


Update API type

Learn Build status updates of commit 7ba09ee:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/cosmos-db/monitor-resource-logs.md
:white_check_mark:Succeeded

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

@seesharprun,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Holy exclamation mark :tada:

The problem registry_labels and registry_taxonomies did not work in Post Model.
The bug because have an exclamation mark wrong in Bridget.php/_models and I fixed it.
Your project is perfect.

The functionality works, but it needs to be public not protected inside the model.

The solution is incorrect, the first if needs to be empty the second one as well.


"load_index_from_storage" equivelant method in LLamaSharp?

Trying to convert below Python code using llama_index to C# version using LLamaSharp.
Any help would be greatly appreciated, thanks 🙏
      from llama_index import (
          StorageContext,
          load_index_from_storage
      )

    # Rebuild storage context
    storage_context = StorageContext.from_defaults(persist_dir="data/storage/vedastro/")

    # Load index (heavy computation)
    index = load_index_from_storage(storage_context, index_id="vedastro_index")

    # Initialize query engine and retriever
    retriever = index.as_retriever()

    response_synthesizer = llama_index.response_synthesizers.get_response_synthesizer(
                                          response_mode="refine",
                                          text_qa_template=text_qa_template,
                                          refine_template=refine_template,
                                          service_context=llm_context_query__service_context,
                                          use_async=False,
                                          streaming=False,
                                          )

    query_engine = llama_index.query_engine.retriever_query_engine.RetrieverQueryEngine(
        retriever=retriever,
        response_synthesizer=response_synthesizer,
    )

   response = query_engine.query("How was your day sweetheart?")

Hi, could you please provide some more contexts about the python code? I'd like to ask about what is supposed to do by this code.

Quite simple, once you have the retriever, you can run queries on it.
There is already an embeddings vector index in file, we need to load it.
I've updated the code for a more complete example

It looks like load_index_from_storage is part of llama index, not part of llama.cpp? I don't know much about this area, but I think something like Semantic Kernel (which we have an integration with) provides things like that.

Thanks for your reply 🙏
Do you have any sample code using Semantic Kernel that loads a vector index or something similar?

I think that's what this demonstrates?

@martindevans Thank for the quick reply.
so in this case modelPath will be path to the vector index right?
I saw this code before posting this issue, but was not sure since it said modelPath and not vector path, will test the code and see how..
Console.Write("Please input your model path: ");
            var modelPath = Console.ReadLine();

            var seed = 1337u;
            // Load weights into memory
            var parameters = new ModelParams(modelPath)
            {
                Seed = seed,
                EmbeddingMode = true
            };

            using var model = LLamaWeights.LoadFromFile(parameters);
            var embedding = new LLamaEmbedder(model, parameters);

ModelPath would be the path to the LLM (i.e. a gguf file) which is used to generate embedding vectors.

I'll close this one since it seems like your question has been answered, but please feel free to re-open if you've still got questions :)


docs: update readme

@mason-org/triage notify(Reason::NewPullRequest)
This comment ensures a notification is delivered to maintainers.
 🤖 This is an automated comment.    📖 Source code


Publish Gap App through GitHub Actions

@ElizabethThorner As the owner of the repo, you will need to create a secret with the name of

@ElizabethThorner Can you create a token with the following permissions in the FrontendGapApp repo?

Can you then create a secret called ELECTRON_FORGE_PUBLISH_AUTH_TOKEN with the value of the token. This is required for the build pipeline.
I can extract the app and run it from the published zip folder on Mac OS. It's not working on Linux, and I haven't tried Windows yet. This is something I'll investigate next week.


Can BERT run on RTX 2070 * 2

In doc, the author suggest bert-base can run on RAM > 12 G GPU like 1080 or titan... But can BERT run on two RTX2070 which RAM is just 8G?  Did any friend had tried this? OR anybody can have other suggest ?

THX for your help , but ..emm... the question i asked is not the distributed training, I want to know , can multi GPUs RAM can work for one model?  bert-base need >12G RAM GPU, but if i have two 8G GPUs, can it run on just one GPU, but use total 16G RAM of both GPUs?  not compute in both GPUs, just use them RAM, can it do this ?

you can apply with tf.device for different layers to achieve model parallsim with data parallism using horovod. i had implemented model parallism for en large base model on 4-gpus

you can apply with tf.device for different layers to achieve model parallsim with data parallism using horovod. i had implemented model parallism for en large base model on 4-gpus

Thanks for your reply.  Did you modify the core code of BERT? Make layers distribute in different GPUs , and can you success to load the pre-train data?
You run BERT-Big model in 4-gpus? How many RAMs per GPU have?

for serving a pretrained/fine-tuned BERT model for feature extraction, you may try bert-as-service. It is optimized for efficiency, scalability as well as small memory footprint. The GPU memory consumption is around 700MB (idle) ~ 1.2GB (serving).

How does 1080 8g fine-turning bert small locally?

How does 1080 8g fine-turning bert small locally?
batch_size = 8, max_seq_length = 320, does a full fine-tuning on french wikipedia in 6 hours.


[Bug]: Github command should be asynchronous

Contact Details
<EMAIL_ADDRESS>What happened?
The github command fetches the github api to get a user's details. In the current code, this fetch is not done asynchronously. So we are getting the output after a new line has been created in the terminal.
Here is the screenshot of the bug.

What should have happened?
The github user details should be prefetched. When github command is entered, the only activity should be printing the prefetched details since new line creation is implemented with fixed time delay. This is the implementation style used in contributors command that does have this issue.
What browsers are you seeing the problem on?
No response
Version
No response
Relevant log output
No response

Hey @TechSpiritSS please assign this issue to me.

Sure👍


New Maintainer

Me and @MrBrain295 are leaving @Sideflipsa-ga we are needing a new maintainer. Apply by emailing<EMAIL_ADDRESS>

@mrdoob lol

Never mind this has changed.


How to get the clicked measure according to the xy coordinates of the mouse click.

I am going to implement the function of  touch into the scorescreen to switch to a defined position in the score and play along from there, and and I found the issue#233.
Thank you very much. But there are two questions to ask

How to get the clicked measure according to the xy coordinates of the mouse click.
How to highlight the current measure when clicked

Very interesting question!
Using the measure number to point to a certain place in the score is tricky. Although for most common scores, just providing the measure number does the job, it is necessary to take into account that for polymetric music (music in which not all instruments have the same time signature), the measure number is not an absolute value, common to all score instruments (score parts), but it is relative to an instrument.
Due to this, I didn't I coded in Lomse methods using the measure number. But I'm  observing that Lomse users ask for measure number as parameter so I recently started to code methods using measures. Currently there are very few methods using measures and most Lomse methods receiving as parameter a measure number also receives the instrument number to which the measure refers, with a default value of 0 (first instrument). The first measure (anacruxis or not) is always measure=0.
And most methods have an alternative signature using instead a  MeasureLocator, an struct for describing any location on the score:
struct MeasureLocator
{
    int iInstr;             //instrument number (0..n)
    int iMeasure;           //measure number (0..m), for the instrument
    TimeUnits location;     //TimeUnits from start of measure

    MeasureLocator() : iInstr(0), iMeasure(0), location(0.0) {}
    MeasureLocator(int i, int m, TimeUnits l) : iInstr(i), iMeasure(m), location(l) {}

};

Now, returning to your questions:
How to get the clicked measure according to the xy coordinates of the mouse click
Currently, there is no a simple way of getting the measure number. A possible way of doing it would be:
//get mouse click point (pixels)
int xm = mouse_x;
int ym = mouse_y;

//get clicked object (e.g. staffObj, auxObj, staff, whatever)
ImoObj* pImo = spInteractor->find_object_at(Pixels(xm), Pixels(ym));

//get score page
int iPage = pGView->page_at_screen_point(double(xm), double(ym));

//get information for positioning a caret at that point
LUnits x(xm);
LUnits y(ym);
spInteractor->screen_point_to_page_point(&x, &y);
DocCursorState state = pGView->click_event_to_cursor_state(iPage, LUnits(x), LUnits(y), pImo, pGmo);

//get measure number and instrument from caret state
SpElementCursorState elmState = state.get_delegate_state();
ScoreCursorState* pState = static_cast<ScoreCursorState*>( elmState.get() );
int iMeasure = pState->measure();
int iInstr = pState->instrument();

Certainly, this is not an acceptable API and it forces to access and to understand some internal objects. I'm going to fix this by including specific API methods, (suggestions welcome!)  probably:
//get mouse click point
Pixels x = Pixels( mouse_x );
Pixels y = Pixels( mouse_y );

//get clicked instrument and measure
int iMeasure = spInteractor->find_measure_at(x, y);
int iInstr = spInteractor->find_instrument_at(x, y);
MeasureLocator ml = spInteractor->find_locator_for(x, y);

I'm currently busy with other tasks but hope to have this done along next week.
How to highlight the clicked measure
Currently there are no methods for highlighting one or more measures. The only available option is to place a fragment mark (See https://lenmus.github.io/lomse/classFragmentMark.html). You will find there an example for placing a mark at the desired measure.
My first impression is that adding methods for highlighting one or more measures should be simple. So I'm going to add this to the list of things to do in the coming days.
How to play from a given measure
ScorePlayer object has method ScorePlayer::play_from_measure(iMeasure, ...). See https://lenmus.github.io/lomse/classScorePlayer.html
But parameter iMeasure does not start in 0, but goes from 1 to n. And the method does not receive the instrument number, so it is not appropriate for polymetric scores. I have to fix this.

Thanks a lot for such a quick reply.
I invoke the find_object_at method according to your commit, but the information in data.ml (MeasureLocator) is all 0.
That is to say, both pImo->is_scoreobj() and pImo->is_instrument() are false.
In addition, I thought of a method, when parsing xml to imo, record the corresponding measure value, and then pass it when the corresponding gmo is generated, and the corresponding measure can be obtained according to the gmo

The method in previous commit to my personal repo is not yet ready. Please wait for a PR to lenmus/lomse repo that will include also tests and documentation. I'm really busy this week and probably this will not be ready until the weekend.

In addition, I thought of a method, when parsing xml to imo, record the corresponding measure value, and then pass it when the corresponding gmo is generated, and the corresponding measure can be obtained according to the gmo

Thanks for the suggestion. Many music programs are focused on measures and this works for 99% of music. But these programs have problems to deal with music without measures (e.g many Eric Satie piano pieces) or polymetric music. Lomse treats measures in a different way and the measure number is not data associated neither to each Imo object nor to each Gmo object.

thank you very much

PR #355 , already merged, adds method to find measure number. See details and example in the description of this PR.
Now I will implement a method to highlight one or more measures.

I ran several test cases from lomse_test_interacotr.cpp
For example TEST_FIXTURE(InteractorTestFixture, click_info_604)
But the obtained MeasureLocator.iMeasure =-1 , MeasureLocator.ilnst=-1, MeasureLocator.location=0

First
Rebuild Lomse library and the test runner. If after doing this the tests still fail, then

Modify and run the unit tests as follows
  bool check_click_data2(ClickPointData& data, int iInstr, int iStaff, int iMeasure,
        TimeUnits location, const string& pImoName, const char* name)
        {
            bool fTestOk = data.ml.iInstr == iInstr
                           && data.iStaff == iStaff
                           && data.ml.iMeasure == iMeasure
                           && is_equal_time(data.ml.location, location)
                           && (pImoName=="nullptr" ? data.pImo==nullptr
                                                   : data.pImo->get_name() == pImoName);

//            if (!fTestOk) //      -->Comment out this judgment in order to output the log
            {
                cout << "Error in test " << name << endl
                     << "    Result: "
                     << "instr=" << data.ml.iInstr << ", staff=" << data.iStaff
                     << ", measure=" << data.ml.iMeasure << ", location=" << data.ml.location
                     << ", pImo is " << (data.pImo==nullptr ? "nullptr"
                                                            : data.pImo->get_name()) << endl
                     << "  Expected: "
                     << "instr=" << iInstr << ", staff=" << iStaff
                     << ", measure=" << iMeasure << ", location=" << location
                     << ", pImo is " << pImoName <<endl;
            }
            return fTestOk;
        }

    TEST_FIXTURE(InteractorTestFixture, click_info_604)
    {
        //@604. Click point on instrument brace

        LomseDoorway doorway;
        doorway.init_library(k_pix_format_rgb24, 82);
        LibraryScope libraryScope(cout, &doorway);
        libraryScope.set_default_fonts_path(TESTLIB_FONTS_PATH);
        Presenter* pPresenter = doorway.open_document(k_view_vertical_book,
            m_scores_path + "07012-two-instruments-four-staves.lmd");
        Interactor* pIntor = pPresenter->get_interactor_raw_ptr(0);

        //get graphic model clicked object
        LUnits x = 1634.16;
        LUnits y = 7208.22;
        GraphicModel* pGM = pIntor->get_graphic_model(); //this also forces to engrave the score
        GmoObj* pGmo = pGM->hit_test(0, x, y);

        //get clicked point info
        ClickPointData data = GModelAlgorithms::find_info_for_point(x, y, pGmo);

        CHECK( check_click_data2(data, -1, -1, -1, 0, "instrument", test_name()) );  //--> invoke check_click_data2
    }

The result of running is as follows:
Running all tests
Error in test click_info_604
Result: instr=-1, staff=-1, measure=-1, location=0, pImo is instrument
Expected: instr=-1, staff=-1, measure=-1, location=0, pImo is instrument
Success: 2134 tests passed.
Second
On a real test, when clicking on an score, have you observed any bad behavior (bad ClickPointData info)?

I tested it with wxwidget(viewTypes-wx.cpp) and found that what I got was also bad ClickPointData info( instr=-1, staff=-1, measure=-1, location=0).  details as follows:
void MyCanvas::on_mouse_event(wxMouseEvent& event)
{
    if (!m_pPresenter) return;

    if (SpInteractor spInteractor = m_pPresenter->get_interactor(0).lock())
    {
        wxEventType nEventType = event.GetEventType();
        wxPoint pos = event.GetPosition();
        unsigned flags = get_mouse_flags(event);

        if (nEventType==wxEVT_LEFT_DOWN)
        {
            flags |= k_mouse_left;
            spInteractor->on_mouse_button_down(pos.x, pos.y, flags);

            // --> test find_info_for_point     start-----
            Interactor* pIntor = m_pPresenter->get_interactor_raw_ptr(0);

            LUnits x = pos.x;
            LUnits y = pos.y;
            GraphicModel* pGM = pIntor->get_graphic_model(); //this also forces to engrave the score
            GmoObj* pGmo = pGM->hit_test(0, x, y);

            //get clicked point info
            ClickPointData data = GModelAlgorithms::find_info_for_point(x, y, pGmo);
            ImoObj *pObj = data.pImo;
            MeasureLocator ml = data.ml;
            if (ml.is_valid())
            {
                //clicked point is a valid location on an score
                int iMeasure = ml.iMeasure;
                int iInstr = ml.iInstr;
                int iStaff = data.iStaff;

                //do whatever you like with this information

                cout << "iMeasure:"<<iMeasure<<" iInstr:"<<iInstr<< "iStaff:"<<iStaff <<endl;
            }
        }
      // ---> test find_info_for_point    end-----
        ...
   }

Analyzing more in detail test  click_info_604 I observe that the output you sent to me  it no longer fails. After rebuilding the library it seems that all test passed ok. Could you please confirm this?
As to the real app test, I'm assuming that after you rebuild Lomse, you also re-installed it and then you re-build the application (viewTypes-wx.cpp). Is this correct?

Analyzing more in detail test click_info_604 I observe that the output you sent to me it no longer fails.
After rebuilding the library it seems that all test passed ok. Could you please confirm this?

Yes, the lomse unit tests ran successfully
As to the real app test, I'm assuming that after you rebuild Lomse,
 you also re-installed it and then you re-build the application (viewTypes-wx.cpp). Is this correct?

According to your example above, Use spInteractor->find_click_info_at(x, y);
get valid ClickPointData correctly.
thank you very much

You are welcome!  Can I close this issue?

highlight one or more measures.

Any suggestions or references for this?

Closed in PR #356. Usage example:
ClickPointData data = pInteractor->find_click_info_at(pos.x, pos.y);
Document* pDoc = pPresenter->get_document_raw_ptr();
ImoScore* pScore = dynamic_cast<ImoScore*>( pDoc->get_content_item(0) );

if (data.ml.is_valid() && pScore)
{
    ImoId scoreId = pScore->get_id();

    MeasureHighlight* mark = pInteractor->add_measure_highlight(scoreId, data.ml);

    .. now refresh the window so that the added marker is displayed
}

See API documentation:
https://lenmus.github.io/lomse/classMeasureHighlight.html
https://lenmus.github.io/lomse/classInteractor.html#add8f800d6e000146c0888b5c1160fe7e


Feature: better pager for kubectl output

Is your feature request related to a problem? Please describe.
When I ask for the 'D'escription of a resource, if the description is less than one window in height, the screen will flicker and return to the kube-commander interface.  There's no time to read the output of kubectl... especially I get a one-line error message like "context does not exist" (see #55)
Describe the solution you'd like
When launching kubectl to show the human something, wait for the human to press a key before dismissing the kubectl output and returning to the kube-commander UI.  A blank line between sequential uses would be nice, just to hint at where the output of the latest command starts.
Describe alternatives you've considered

Tried setting the PAGER envvar, in the hope that kubectl would pipe output to /usr/bin/less.
Alias the 'kubectl' command to kubectl "$@" |less -- didn't work because kube-commander doesn't launch kubectl in the same shell.
maybe I should install a kubectl plugin that pipes all output to $PAGER ?

You can define your pager now with PAGER env of --pager flag. Thanks for the idea!


shouldHideTweet should check config.retweets first

if I set config.retweets = 'ignore', it still hides retweets.

Looks like an issue with config.hideWhoToFollowEtc triggering RT hiding rather than the value of config.retweets
With config.hideWhoToFollowEtc false then RTs appear again with config.retweets set to ignore

Looks like an issue with config.hideWhoToFollowEtc triggering RT hiding rather than the value of config.retweets
With config.hideWhoToFollowEtc false then RTs appear again with config.retweets set to ignore


Data required under Components in documentation

Issue is related to documentation,
Pipeline snapshot is missing for few components under configuration section for DIETClassifier, FallbackClassifier etc.
Pipeline snapshot will improve understanding of how to write hyperparameters and flags like entity_recognition/intent_classification in the pipeline under DIETClassifier and similar for FallbackClassifier.
Example for pipeline snapshot image of KeywordIntentClassifier:

Thanks for raising this issue, @erohmensing will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

Hey @nikhilcheke, I totally agree, this would be a great addition! Would you be willing to add snippets for the components you mentioned?


Add myself to reviewers

Hi! I'd like to be a reviewer, and was invited by TAG-CS (Josh Berkus and co) to participate.
Signed-off-by: Tasha<EMAIL_ADDRESS>

Appointed by TAG-CS by vote of the chairs and tech leads.

Josh Berkus
Dawn Foster
Carolyn Van Slyck

FYI: Tasha was appointed by SIG-CS during the original call for reviewers, and was omitted from the initial repo due to a clerical error.  This is not a new appointment.

welcome @tashimi

Thank yooooou!! 🥳🥳🥳


doubles 1.5.0 breaks automatic teardown of top-level expectations between test cases

with doubles 1.4.0, the following test case passes:
file foo:
def bar(x):
    pass

file test_doubles:
from unittest import TestCase
import foo

class TestSomething(TestCase):

    def test_one(self):
        expect(foo).bar.with_args(1)
        foo.bar(1)

    def test_two(self):
        foo.bar(2)

With 1.5.0, it fails because the top-level expectation leaks between the tests.
______________________________________________ TestSomething.test_two ______________________________________________
test_doubles.py:27: in test_two
    foo.bar(2)
env/lib/python2.7/site-packages/doubles/proxy_method.py:75: in __call__
    self._raise_exception(args, kwargs)
env/lib/python2.7/site-packages/doubles/proxy_method.py:160: in _raise_exception
    build_argument_repr_string(args, kwargs)
E   UnallowedMethodCallError: Received unexpected call to 'bar' on <module 'foo' from '<path>/foo.py'>.  The supplied arguments (2) do not match any available allowances.
======================================== 1 failed, 1 passed in 1.08 seconds ========================================

This was caused by: https://github.com/uber/doubles/commit/0900dc9dd75e6c84cad73fbeae303c5248924b98

@toddsifleet can we release version 1.5.1?


Fix docker!

Docker is now fixed!

Coverage decreased (-8.4%) to 64.113% when pulling ade29388b82622857442dc2f5f95da0fe7fe9ba9 on fixdocker into a3f00a67d558c59e21a136c37cdea9626b8c3491 on master.


N49: August 2023

Newsletter 49: August 2023
tags: newsletter
Editors: @ozkriff, @scirin, @AngelOnFira
Another month has gone by, so it's time to put together the Rust Gamedev newsletter with August's news!
Current Schedule
The deadline for all section PRs is 2023.09.15.
We may still accept PRs that are submitted later than this, as long as they're ready before the newsletter's release, but this isn't guaranteed. If you want your section to be included, don't leave it till the last minute!
Current Structure & Status
Below is our current planned structure for the newsletter, and the status of each PR (which we'll try to keep updated).
This is not an exhaustive list - if you have your own project that you want to write about, just make a comment on this issue and open a PR!

Rust Gamedev Meetup

[ ] Details of this month's meetup - 🆓 free (@AngelOnFira @ozkriff?)

Game Updates

[ ] Veloren - 🚧 WIP by @AngelOnFira
[ ] Tiny Glade (1, 2, 3, 4) - 🆓 free (@anopara @h3r2tic?)
[ ] Tunnet (1, 2) - 🆓 free (@puzzled-squid?)
[ ] HackeRPG (1, 2, 3, 4) - 🆓 free (@username?)
[ ] Mage Rush (1) - 🆓 free (@schweller?)
[ ] Combine And Conquer (1, 2) - 🆓 free (@I3ck?)
[ ] exocave (1) - 🆓 free (@bcarruthers?)
[ ] Tribes (1) - 🆓 free (@uvizhe?)
[ ] nullableengineer’s mmo (1) - 🆓 free
[ ] terrybrash's top-down arena shooter (1) - 🆓 free (@terrybrash?)
[ ] Ars Militaris (1, 2, 3) - 🆓 free (@arsmilitaris?)
[ ] Digital Extinction (1, 2) - 🆓 free (@Indy2222?)
[ ] 8bit Duels (1, 2) - 🆓 free (@thousandthstar?)
[ ] Bare Metal Space Invaders (1) - 🆓 free (@FedericoPonzi?)
[ ] Graviton (1) - 🆓 free (@hakolao?)

Learning Material Updates

[ ] faking pixel art (1, 2) - 🆓 free (@davjcosby?)
[ ] PhaestusFox’s Bevy videos (1) - 🆓 free (@PhaestusFox?)
[ ] Failed to build multiplayer pong (1) - 🆓 free (@ChristopherBiscardi?)
[ ] Bevy's android GH (1) - 🆓 free (@NiklasEi?)
[ ] Bevy game in SvelteKit (1) - 🆓 free (@sneakycrow?)

Engine Updates

[ ] Bevy (1) - 🆓 free (@cart?)
[ ] goku (1) - 🆓 free (@ladroid?)
[ ] Rezcraft (1) - 🆓 free

Tooling Updates

[ ] mimiron (1) - 🆓 free (@asibahi?)
[ ] space_editor (1) - 🆓 free (@rewin123?)
[ ] Bevy + Overture Maps (1, 2) - 🆓 free (@alexichepura?)

Library Updates

[ ] rivet (1) - 🆓 free (@NathanFlurry?)
[ ] Oceanman (1) - 🆓 free (@tech0tron?)
[ ] dexterous_developer (1) - 🆓 free (@lee-orr?)
[ ] unrust (1, 2) - 🆓 free (@gamedolphin?)
[ ] bevy_mod_raycast (1) - 🆓 free (@aevyrie?)

Other News

[ ] Bullet points of any interesting news that doesn't have its own section - by @ozkrif

Discussions

[ ] "Someday, maybe, we will be game" (1)
[ ] "Do you wirte performance tests" (1)

Publishing Steps

[ ] Final review - by everyone
[ ] Publish - by @ozkriff
[ ] Post on /r/rust, /r/rust_gamedev, /r/gamedev, URLO, twitter.com/rust_gamedev - by @ozkriff
[ ] Pin thread on Twitter - by @ozkriff
[ ] Add comment links - by @ozkriff
[ ] Add a draft of next month's newsletter - by @ozkriff

How to Contribute
If you want to help writing the newsletter:

Read CONTRIBUTING.md.
Choose one or more of the "🆓 free" sections listed below, and leave a comment letting us know you want to work on them.

The links in brackets (like "1, 2, 3") are suggestions of links to include in the section. Feel free to add more!
The username listed next to the section (like "@ozkriff?") is a suggestion of who may want to pick up the work (usually the project's developer, or someone who has expressed interest in the past).
You are not obligated to write a section if you're tagged or your project is listed! You're welcome to ask someone else to write the section, or to ask for your project to be excluded from this month's post.
Extra sections not listed in the plan are welcomed - just leave a comment and open a PR!

Write a short overview in the newsletter's Markdown file, making sure to follow the style guidelines (see below).
Send a PR to the source branch (example: #336).
Mention this issue in your PR's description to link it all together.

Style Guidelines
The full style guide is in CONTRIBUTING.md, but here are the most important rules:

Write in third-person perspective.
Each line must be 80 characters or less, for ease of reviewing/diffing.
Only one image per section is allowed.

The maximum size is 300kb for static images and 2.5mb for GIFs.
The image should come before the text, and must have alt text for accessibility.
Prefer static images to GIFs, to keep the page load times down.

Each section should be under 1000 characters, and under 6 paragraphs.

This only applies to the rendered text, not the markup.

Keep formatting minimal - no bold/italics/etc.
Avoid long/nested bullet point lists - no changelogs!

Please use these templates as a starting point:
Games/apps/libraries:
### [Game name]

![alt text](img)
_optional image label_

[Game name] ([GitHub], [Discord], [Twitter]) by [@nickname]
is... {short project description in one sentence}.

{An overview of the recent updates with links to more details}.

_Discussions: [/r/rust_gamedev](link), [Twitter](link), [etc](link)_

[Game name]: http://example.com

Articles/blog posts/videos/etc:
### [Article name]

![alt text](img)
_optional image label_

[@nickname] published an [article] about...
{overview what the resource is about}.

_Discussions: [/r/rust_gamedev](link), [Twiter](link), [etc](link)_

[Article name]: http://example.com

Just got a ping about this -- do you need anything from me on this? Would love to contribute!

Thanks for the interest in dexterous developer! I gave writing the section a try here: https://github.com/rust-gamedev/rust-gamedev.github.io/pull/1441 - any feedback is welcome :)

dropped a PR for my video 👍 thanks for the ping

Thanks for the mention! I will write a section about space_editor:)

<...> do you need anything from me on this? Would love to contribute!

@NathanFlurry I'd appreciate it if you could submit a section about rivet - something similar to https://github.com/rust-gamedev/rust-gamedev.github.io/pull/1408 or any other library/tool description from https://gamedev.rs/news issues. :)

Thanks, added mine!


chore: scaffold changesets

WHY are these changes introduced?
Fixes ENG-6342
WHAT is this pull request doing?

Scaffolds changesets

Notes
The only option that was adjusted within changesets config.json was to flip "access": "public".  Seems like the recommended setting for public repos.  I'm not sure that we need to update any of the other default settings.  Open to feedback.

Looks good! Might be worth discussing the changelog setting as a group before finalizing. But whatever the decision there, we should be in a good position either way.

Makes sense.  This was really just an initial stab to get us unblocked on the epic.  Definitely worth discussing as a group