Gem name and release on rubygems

Since the name "components" is already used/blocked on rubygems we should probably find a new name, rename the project/files and work towards an offical release on rubygems?
@jensljungblad do you have any names on your mind already?

What about simply naming it rails-components? Seems free (there are some variations with an underscore, singular etc.) and more importantly, is really easy to find. A google search for "Rails components" would eventually lead pretty straight to this project (currently ranking 7 for that query for me). Finally, all of the similar-named projects are barely used at all (and barely have any stars) so this one would pretty quickly become the headliner (e.g. be considered the proper rails-components gem).
If you want to go for something more unique I suppose you could do something like "komponents", but tbh, rails-components would get my vote any day.
Anyway, I think releasing this as a gem would make sense now since it would also help with attracting more users :)

Yeah, rails-components sound pretty good to me :) What about the namespace, should we change it or keep Components?

While it certainly looks pretty just as Components, I think it might be better to be on the safer side here and just change it to RailsComponents (maybe less confusing as well, idk). But then again, I don't have a ton of experience with working with a lot of different gems outside the standard Rails ones, so I can't say what the convention is or whether it's likely that it would clash with something.

The naming conventions for gems can be found here: https://guides.rubygems.org/name-your-gem. There are ways around them though, and people don't really stick to them. E.g. rails-components should mean there is a Rails::Components module, and not a RailsComponents one. That would rather be for a gem named rails_components. But people break these rules quite a lot, and there are ways around the autoloading issues.
Could also throw in "view" somewhere in the name perhaps, to make it clear it's view components and some other form of component.. rails_view_components is quite long though!

I see. I'm not entirely sure if it's allowed/expected but imo using Rails::Components sounds the most logical to me. This gem is a pure Rails add-on after all.
Otherwise, maybe either just stick with components + Components, or go with ViewComponents + view_components? That way it's not awfully long and I guess it follows the conventions. Maybe not crystal clear that it's for Rails, but yeah, these conventions cause sacrifices.

The naming conventions for gems can be found here: https://guides.rubygems.org/name-your-gem.

Funny - I was also looking up those naming convention the other day and started wondering if it would be wise to move into the rails namespace via Rails::Components.
Standalone gems often have a rails adapter/integration gem e.g. rspec-rails and define a RSpec::Rails namespace to prevent naming conflicts. Since the components gem is fully dependent on rails and probably will not exist as a standalone gem there is no need to split it up into two gems.
But we could follow the same naming convention and use something like our_name-rails. If I didnt miss anything the components-rails name is not used on rubygems.org yet ;)

Using the Rails namespace feels a little weird to me. It just looks like it's a part of Rails.
E.g. rails-components using a Rails::Components::Component class name.
Some alternatives:

rails-components with a RailsComponents constant (i.e. cheat a little).
rails_components with a RailsComponents constant (ask if we can take the name, since the repo is archived)
rails_view_components with a RailsViewComponents constant

Still a big fan of "invading" the Rails namespace, but if I had to pick a different option, I'd vote for number 1.

Wouldn't it make more sense to invade actionview in that case? :)

That does make more sense. Would that result in a name such as actionview-components then?

Yeah. Makes sense if seen as a pure rails/actionview extension, but doesn't exactly roll off the tongue :)

God, I hate naming stuff lol :P
Found this as well: https://rubygems.org/gems/actionview-component

@msalzburg Was your proposal to use components-rails but keep the constant as Components? That is what I've seen that pattern used for mostly. E.g. sass-rails, elasticsearch-rails etc.

@fgblomqvist Haha I know.. At least GitHub works pretty well as a gem repository!
actionview-components is available though :)

Haha always something. And yeah true :)
Alright, my final vote falls on actionview-components with ActionView::Components::Component. It makes logical sense in every way possible in my head, and I'm sure people will find it no matter what. As long as the readme contains the word "rails" it should show up in some Google searches. Maybe it'll get listed in some other spots eventually, or perhaps get featured in some article. Finally, I suppose it's good to think long-term and realize that complex ActionView rendering is probably on the way out (imo), getting replaced by either client-side rendering or just other server-side frameworks (e.g. react server-side rendering). Meaning that, I don't think we're looking at a 10-year lifespan for something like this so probably better to get it out there sooner rather than later ;)

Published to RubyGems finally: https://rubygems.org/gems/elemental_components. Closing :)


cycle/http stop requesting after getting 401 error

I make a simple login as in this gist:
https://gist.github.com/kkpoon/96ea4c5e67a173bf114e0db40750693d
it is works ok when I type correct password, below screen shot shows 3 auth requests sent when pressed 3 times Login button with correct username password

Once I have input wrong username and password, it received 401 status. and no matter how many times I press the login button with wrong username password, it just sent one request

Even I correct the username password to the good one, it does not send any more request to the server.
It looks like loginRequest$ still producing when I press login, but loginResult$ does not get any message from the subscription, no HTTP request sent.

You should correctly handle errors. This:
.mergeAll()
    .catch(err => Rx.Observable.just({error: err}))

Will create a flat stream of responses and after the error stream completes, this is how streams work, even if you catch an error. Instead you want to do something like:
.map(r$ => r$.catch(err => just({error: err})))
  .mergeAll()

thanks @whitecolor . you are correct
I changed the stream to handle the login result to
  const loginResult$ = HTTP
    .filter(res$ => res$.request.name === 'login-auth')
    .map(res$ => res$
      .catch(err => Rx.Observable.just({error: err}))
      .map(res => res.body)
    )
    .mergeAll()
    .startWith(null);

This is not the issue of cycle/http but my concept. Thanks

this thing https://github.com/whitecolor/cycle-async-driver has helpers .success() and failure() which provide flat streams, helpers maybe used with official HTTP driver.


Item count

In this PR I did the following:

I created a function to check action count
I imported and rendered it. check image below.

Hi 👋 @DannieMo,
Good job so far!
You still need to work on some issues to go to the next project, but you are almost there!
Highlights
✔ The PR title and description look good and professional.
✔ Descriptive commit messages.
Required Changes ♻
Check the comments under the review.

[ ] No linter errors.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me in your question so I can receive the notification.
**_Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Hi @DannieMo
Status: Approved ✔
Your project is complete! There is nothing else to say other than... it's time to merge it 🚢
Congratulations! 🎉
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
You can also consider:
x
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


NIPS 2018 deadline unannounced

so it shouldn't be on there.

I do see it here: https://nips.cc/Conferences/2018/Dates


[Feature] Inspectors: inspecting & extracting

Thought I provide you with two operators I used in my project.
Context
I testest a graph like structure and made up a small test graph in which I wanted to check whether properties were correctly set. Testing graphs is always annoying and a lot of work.
Problem
I needed to check several properties, leading to a lot of code similar to this one:
node1.prop1 shouldBe xxx
node1.prop2 shouldBe yyy
node1.prop3 shouldBe zzz

forOne(node1.prop4){
  it.name shouldBe "name1"
}
forOne(node1.prop4){
  it.name shouldBe "name2"
}

Solution/Helpers
I defined two very simple methods that allow a simpler to read and comprehend version.
inspecting
inspect exposes the properties and provides an assertion/mental context
inspecting(node1){
  prop1 shouldBe xxx
  prop2 shouldBe yyy
  prop3 shouldBe zzz
}

extracting
Extracting allows to pull out properties from a collection to make or assertions. forOne only allows and assertions, however, i might have missed something in your documentation.
extracting(node1.prop4){ name }
  .shouldContainAll("name1", "name2")

In sum, I think these two fit well in the kotlintest dsl and are a simple but useful addition.
Furthermore, I think all inspectors should work similar to inspecting, i.e., providing the context relative to the inspected object; however, this is another topic.

Simple and effective - great!

I moved them to their own files and fixed it for you.
https://github.com/kotlintest/kotlintest/commit/611eee578e931e0b8c075743339e279c5f3be7f2


unused variable error when declaring a function

declare function foo(): any;
this is causing a warning: "unused variable: 'foo'"

Thanks!


ring: add GetWithOptions method to adjust per call behavior

What this PR does:
This change adds a new method that accepts 0 or more Option instances that modify the behavior of the call. These options can (currently) be used to adjust the replication factor for a particular key or use buffers to avoid excessive allocation.
Which issue(s) this PR fixes:
Part of grafana/mimir#9944
Checklist

[ ] Tests updated
[ ] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

I wasn't able to avoid these functions escaping to the heap. Benchmarking the two methods shows that GetWithOptions is about 14% slower. I'm going to leave these as mostly separate methods and work on adding tests to GetWithOptions. After that I'll mark this as ready for review and we can start experimenting with higher replication factors in Mimir.
$ benchstat old.txt new.txt
goos: linux
goarch: amd64
pkg: github.com/grafana/dskit/ring
cpu: Intel(R) Core(TM) i9-10885H CPU @ 2.40GHz
                                                         │   old.txt    │                new.txt                 │
                                                         │    sec/op    │    sec/op      vs base                 │
Ring_Get/without_zone_awareness-16                         863.2n ± ∞ ¹   1009.0n ± ∞ ¹        ~ (p=1.000 n=1) ²
Ring_Get/without_zone_awareness,_not_enough_instances-16   654.2n ± ∞ ¹    758.2n ± ∞ ¹        ~ (p=1.000 n=1) ²
Ring_Get/with_zone_awareness-16                            1.138µ ± ∞ ¹    1.252µ ± ∞ ¹        ~ (p=1.000 n=1) ²
Ring_Get/one_excluded_zone-16                              855.0n ± ∞ ¹    962.0n ± ∞ ¹        ~ (p=1.000 n=1) ²
geomean                                                    861.0n          979.7n        +13.80%

After adding unit tests for this change, it turns out to not be compatible with zone-awareness. There's an assumption in the ring that only a single instance per zone is returned during quorum checks. The result of this is that there are Ring.Get() calls that should fail when not enough zones are available that would now succeed since we return multiple instances per zone and the existing code only checks the number of instances, not zones.

I'm going to close this PR. The changes to the ring required for https://github.com/grafana/mimir/issues/9944 are more complicated than expected. I'll put the various concerns into a design doc for some discussion with the Mimir team and other database teams as well.


can you provide the code？

can you provide the code of calculate results/data/DIAMOND.json

Each value in the results json corresponds to the final evaluation of a single training run (see Launching a training run in the README). The evaluation code can be found in collect_test (specifically, the value used is the final_return_mean on line 298), which is called at the end of run,  both in src/trainer.py. Hope that helps!


[$500] [Wave 6: Workspace Chats] There’s no offline next steps banner, just blank.

Slack conversation: https://expensify.slack.com/archives/C02MW39LT9N/p1703191424265509?thread_ts=1703187017.215259&cid=C02MW39LT9N
Action Performed:

As the member, go to NewDot
Go to global create (+) > Request money > enter an amount > confirm
Navigate to the expense report view
Go offline
Tap the Submit button
• Verify the report is optimistically submitted :white_check_mark:
• Verify the text in the next steps banner updates to: “The details of the next steps will update when you’re back online.” :x:
Go back online
• Verify the report is submitted successfully :white_check_mark:
• Verify the next steps banner text updates to: “Waiting for %approverName% to review these expenses” :white_check_mark:

Expected Result:
There should be an offline message about next steps for a report "The details of the next steps will update when you’re back online.”"
Actual Result:
There's no offline message
Screenshots/Videos

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014e89ba23323a2eac
Upwork Job ID:<PHONE_NUMBER>352990720
Last Price Increase: 2024-01-04

sorry, was ooo! confirmed!

Proposal
Please re-state the problem that we are trying to solve in this issue.
There are no offline next steps banner.
What is the root cause of that problem?
This is a new feature.
Currently, Next Step banner is displayed only through reportNextStep_ onyx entries sent from BE as shown below.

We do not optimistically create these when expense reports are submitted offline for collect workspaces.
What changes do you think we should make in order to solve the problem?
We can optimistically generate a minimal version of reportNextStep_ Onyx entry that contains the message “The details of the next steps will update when you’re back online.”
We can make this change within submitReport by adding this to optimisticData here
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
If we submit the expense report while offline, we don't see any next step banner.
What is the root cause of that problem?
This is more like a new feature. When we submit a report, we optimistically clear the next step,
https://github.com/Expensify/App/blob/dc3e19bdcc95023d74cd7fdd433f6208707eaaf8/src/libs/actions/IOU.js#L3139-L3144
and the BE is supposed to give us a new next step, but currently, it only happens after we reopen the report.
Now, we want to show a next step banner while offline telling the user that the next step will be shown after going back online.
What changes do you think we should make in order to solve the problem?

The strategy is, that if we are offline and the current next step is empty, we want to show the "pending" next step.
(apply below changes to MoneyReportHeader)
const shouldShowPendingNextStep = isOffline && _.isEmpty(nextStepProp.message);
const nextStep = shouldShowPendingNextStep ? NextStepUtils.getPendingNextStep() : nextStepProp;

The next step utils new function will look like this:
function getPendingNextStep(): ReportNextStep {
    return {
        type: null,
        message: [{text: "The details of the next steps will update when you’re back online."}]
    };
}

I am going to close this one in favour of more general optimistic next steps https://github.com/Expensify/App/issues/28771 issue
There is many granular things which need to be updated for the next steps so no reason to keep it separate


Notifications | Implement notifications badge |5|

https://trello.com/c/QmnFY8Kl/779-notifications-implement-notifications-badge-5
Placeholder card for a notification badge, providing number of unread notifications. Holding off on this for now, until we figure out implementation details for notifications.
Also will need to decide if this badge should refresh periodically to update in "real time" or just have it static and only update when the user refreshes the page.

 Notifications | Implement notifications badge |5|


Feature: wrap secrets in custom types to prevent them from leaking

Description of change
This aims to close #890. I looked into secrecy and tried to integrate it into the code base and found some odd trait bound issues that wouldn't allow using them in structs that implement Serialize or Deserialize. The crate provides a serde feature that should enable exactly this, but this still would lead to weird issues of standard library types not implementing traits from the secrecy crate. This file seems to be the issue. They require the wrapped type to implement SerializableSecret which they never implement for standard library types anywhere?
Anyway, instead I decided to replicate the functionality of the crate myself by creating a wrapper type that makes sure to not expose the inner value unless by calling the expose function explicitly. The underlying data will also be zeroed out in memory when the type is dropped to ensure it won't leak afterwards.
I have looked through the code base searching for places where this type could be useful and made some changes. If you know other places where secrets are being used that I missed, please let me know.

I wrapped the password in Secret, must've overlooked that the first time, thanks.
I am unsure what address_private field is, should we wrap that in Secret too?

I am unsure what address_private field is, should we wrap that in Secret too?

I don't think so. I think it's the address that can be used internally by the backend. It might save some external network calls when compared to accessing the DB through the public URL.

A user in our discord brought up that using SharedDB will print out the database connection string when running or deploying a project, which in my opinion shouldn't be the default. What do you guys think about adding a CLI flag to both run and deploy to only show secrets like connections strings on demand?

A user in our discord brought up that using SharedDB will print out the database connection string when running or deploying a project, which in my opinion shouldn't be the default. What do you guys think about adding a CLI flag to both run and deploy to only show secrets like connections strings on demand?

I would include the secrets flagging for the DB connections as well. At the same time, maybe a global flag will do better? It's gonna be accounted for both secrets and db connection strings. What do you think?

maybe a global flag will do better?

Are you talking about a flag for cargo-shuttle itself (not subcommands)? I think since this will only apply to 3 of the 15 or so subcommands, I think it wouldn't be a good fit for a global flag.
I will start implementing the functionality for run and deploy and if you have any objections, I can make it a global flag afterwards.

@AlphaKeks Do you want more feedback or help with finishing this? I can take a look if needed.

Oh yeah, kinda forgot about this, my bad! Will take a look tomorrow and get this up to date, then ask for review. I'm pretty sure this was basically finished.

Oh yeah, kinda forgot about this, my bad! Will take a look tomorrow and get this up to date, then ask for review. I'm pretty sure this was basically finished.

We've been discussing polishing this a bit before merging but it is almost there. Please take a look at my last review.

Hey @AlphaKeks, I hope you're doing alright :)
Sorry for the delay and back and forth with the review, we're still interested in getting this merged, if you're also still interested in working on it and getting this through. After discussing it with the team, there are still a few changes we would like to make:

Don't hide the connection string, only the password in it. It could be replaced by ******* (a fixed number).
Add a --show-secrets option but only to the resource list command, that would reveal the password in the connection string.
Rebase on top of main.

If you don't want to or don't have the time to continue working on it that's perfectly fine, just let us know and we'll take over. Thank you very much !

Not really interested anymore, sorry.


ES Reindexing Fixes

Required Fixes

Need to run indexing even if the error is encountered in between for some entity.
Add better logging for the Indexing
Add a language option

@mohityadav766 lets move the code to SearchResource, we don't want to main another API for this. SearchResource already has ES rest client

done


Add bar direction configuration

Add a direction configuration parameter which can be set to horizontal or vertical.
Proposed implementation for #44 which has been stuck for a while
Resolves #44

The drawing logic is too complicated for my taste (and it was complicated even before this PR). My plan is to replace it with pixman in the future.
I would suggest to drop any attempts to optimize it for performance and just make it as simple as possible for now. When I wrote that memcpy trick, it was more from a curiosity than actual performance optimization needs.

Closing in favor of https://github.com/francma/wob/pull/108


Fix swift package build

Fixes #1058

Waiting for checks to complete before merging it (hope it will work)

hmm, maybe we should disable the pods tests on regular builds, they take forever. they could be run before releasing.

Let's keep them as it for now. At least we'll be sure everything works. And when a PR is opened, we don't see it directly so it has enough time to build everything.

@jberkel Now we have to publish a new release (for xcode support and fixes mainly).
Also, DM me on Twitter @NathanFallet so that we can talk about everything easily.

@NathanFallet don't have twitter, I'll send you an email


Added message creation for pacbio batch with tag2.

Closes #2893
Changes proposed in this pull request:

...

I will not be sure if this is the correct approach until I test it out on UAT (will not push into develop until I know).
I have created a new pac bio batch message which will include tag2 and I can then create and send this message on the command line.


ignoreDependencies make bower crashes

Hello, I have an issue when using the ignoreDependencies option in .bowerrc.
Output of bower -v && npm -v && node -v:
1.8.0
3.10.10
v6.9.5

Additional environment details (proxy, private registry, etc.):
I use a proxy and a private registry in addition to the official one.
Steps to reproduce the issue:

Add ignoredDependencies field inside .bowerrc and fill it with a dependency
Run a bower install
Run a bower install --force

Describe the results you received:
usr/local/lib/node_modules/bower/lib/node_modules/q/q.js:155
                throw e;
                ^

TypeError: Cannot read property '_release' of undefined
    at StandardRenderer._tree2archy (/usr/local/lib/node_modules/bower/lib/renderers/StandardRenderer.js:439:47)
    at Array.map (native)
    at StandardRenderer._tree2archy (/usr/local/lib/node_modules/bower/lib/renderers/StandardRenderer.js:491:49)
    at StandardRenderer.<anonymous> (/usr/local/lib/node_modules/bower/lib/renderers/StandardRenderer.js:161:24)
    at /usr/local/lib/node_modules/bower/lib/node_modules/mout/object/forOwn.js:12:27
    at exec (/usr/local/lib/node_modules/bower/lib/node_modules/mout/object/forIn.js:71:19)
    at forIn (/usr/local/lib/node_modules/bower/lib/node_modules/mout/object/forIn.js:38:17)
    at Object.forOwn (/usr/local/lib/node_modules/bower/lib/node_modules/mout/object/forOwn.js:10:9)
    at StandardRenderer._install (/usr/local/lib/node_modules/bower/lib/renderers/StandardRenderer.js:149:17)
    at StandardRenderer.end (/usr/local/lib/node_modules/bower/lib/renderers/StandardRenderer.js:50:21)

It seems that bower considers that the dependency that has been put inside ignoreDependencies is installed and it crashed when trying to retrieve the installed version.
Describe the results you expected:
No crash.
Additional information:
The dependency that I put inside ignoreDependencies is a sub-dependency of one of my dependencies (not sure if it is very clear... :x).
Thank you for your work :)

The issue seems to be there:
function handleLogger(logger, renderer) {
    logger
    .on('end', function (data) {
        if (!bower.config.silent && !bower.config.quiet) {
            renderer.end(data);
        }
    })
[...]
}

The missing property is not true even if it should for the uninstalled modules.

Could you share your .bowerrc please, I'll try to reproduce it.


cmd/go: add ability to download info, mod and source zip

In the Athens project, the code that downloads code from VCS's and stores it in the database uses go get behind the scenes. We have to do some hacky things right now - like create a skeleton go.mod file and an empty func main() {} to get go get to work.
Instead of doing that, we'd like to use a command - say, go get $THING --artifacts-only - that downloads the code from the appropriate VCS (still respecting redirects, etc... as it does now), creates the source zip, and then leaves just the info, mod, and source zip in the current directory.  and all of this without requiring that there is a valid go.mod and any go code in the cwd.

Is this the same thing as #26609?

would this be in the scope of being able to do something like go list -m -versions [module]@latest from outside of a module directory?

@rsc what do you think about a go list command?
@marwan-at-work mentioned today at the athens dev meeting that we might have to duplicate internal code from cmd/go to get list working properly on the server

@arschles go list -m -versions already works. We just have to be inside a dummy go module, which is a bit hacky :)

I wonder if we can add “don’t require go list -m to be in a go module” to
the wishlist on this issue 😊
On Thu, Jul 26, 2018 at 11:52 Marwan Sulaiman<EMAIL_ADDRESS>wrote:

@arschles https://github.com/arschles go list -m -versions already
works. We just have to be inside a dummy go module, which is a bit hacky :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/26577#issuecomment-408198429, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEU0avamQSCPoUZzBDsQVDiHWFRlI1Bks5uKhALgaJpZM4VfGit
.

Should this be a release blocker just like its #26609 dup?

Also from Athen's perspective, it's okay to just do all three at the same time (info/mod/zip). As long as Go doesn't traverse the dependencies of what we're asking for and tries to download them and "go install" them. This has led to some failed builds. go get github.com/gopherjs/gopherjs v0.0.0-20180315154859-97d8e769111c for example fails, but if it's inside a mod file that doesn't need one of the dependencies that make it fail, then go build will work.


Logo

We could use a cool logo for RailsBump! :)

Once we have a logo, it should be the favicon as well.


Fix "unexpected keyword argument 'filename'"

Yum was calling the start function of pakrat/progress.py with incorrect parameters. https://github.com/ryanuber/pakrat/issues/3

@jrwesolo wow it's been a while since I've seen this code - Thanks for doing this! Just left one question but otherwise LGTM.

Replied above to your question. Thanks for taking a look!

Thanks!


Unable to connect to harbor, prompting that the certificate is invalid

When I try to connect to the built harbor and run the pod, the following error is prompted

I refer to https://github.com/canonical/microk8s/issues/2521, but it still can't solve my problem. Has anyone encountered it? It has troubled me for many days

Hi @jepson66,
Are you using the registry addon to build the image? If so, can you check if specifying the image as localhost:32000/library/mynginx:v2 solves your issue?

tanks，@neoaggelos The registry plug-in is enabled, but localhost: 32000 is not allowed. My harbor address is not localhost. Where should I configure it?

解决了


[CLI]: CLI is excessively restrictive in check for --host; does not support private/internal DNS hostnames

Describe the bug
I currently have a wandb server instance deployed internally on a network, inside a VPN service. I have an internal hostname (let's say myhost) under which I can access wandb; http://myhost:1234.
Unfortunately, when using the CLI, I can't log in to this instance:
$ wandb login --relogin --host=http://myhost:1234
wandb: You can find your API key in your browser here: http://myhost:1234/authorize
wandb: Paste an API key from your profile and hit enter, or press ctrl+c to quit:
wandb: ERROR Host must be a url in the form https://some.address.com, received http://myhost:1234

If I remove the check for normalized_host in sdk/lib/apikey.py, everything works as expected without any further changes. The check simply makes sure that either a . or localhost is in the hostname before allowing login, which is an excessively restrictive check.
Additional Files
No response
Environment
WandB version:
OS: macOS 14.1
Python version: 3.11
Versions of relevant libraries: 0.16.2
Additional Context
No response

I can make the PR for this since it's such a small change, just want to see if people are open to the change and if there should still be checks on the hostname or whether that line should be removed completely.

Hi @Jamil , thank you for reporting this. You are welcome to create a PR for this. I will flag with our SDK team for review and they will make a final decision.

Thank you @MBakirWB! I just opened up the PR at #6999, let me know when you get a chance to review it with the SDK team :)


Set up CI with Azure Pipelines

Hmm... cross doesn't seem to support Azure Pipelines yet: https://github.com/rust-embedded/cross/pull/169

There seems to be a workaround. I'm not sure if we should use it in the end, but I'll try it.

@carllerche This PR should be ready for review.
The build time seems to have been reduced from 5-6 to 3-4 minutes.
Although I did not actually test deployment, I think that it works well because ci/azure-deploy-docs.yml and ci/azure-install-rust.yml are basically the same as mio's (The only difference is that {Mio,mio} has been replaced by {Bytes,bytes}.).
Also, I removed ci/before_deploy.sh because they did not seem to be actually used.

@taiki-e Funny enough, I can't +1 this because I opened the PR... So, I am +1 and will force merge this :)

Thanks again!

Hmm... seems to deploy docs was failed: https://dev.azure.com/tokio-rs/bytes/_build/results?buildId=1699&view=logs&jobId=bbfa2fa6-a202-51ab-3559-6013e9e5f686
remote: Invalid username or password.
fatal: Authentication failed for 'https://github.com/tokio-rs/bytes/'

Ah.. I will have to fix this.


redis sentinel

使用dubbo作为组册中心时，支持 redis sentinel 模式吗

The redis registry is implemented on a really old redis version, a version neither support sentinel nor cluster. You can also see dubbo build it's own cluster solution on redis.
We have upgrade the jedis version to 2.9.0, but we still haven't use the features like sentinel or cluster.


Polymorphic this breaks valid TS 1.6 code:

This code compiles fine in 1.6 but fails in 1.7. I assume this is related to polymorphic this:
class Foo {

	root(): Foo {
		let foo = this;
		while (foo._parent) {
			foo = foo._parent;  // type 'Foo' is not assignable to type 'this'
		}
		return foo;
	}
	private _parent: Foo;
}

http://www.typescriptlang.org/Playground#src=

class Foo {

	root()%3A Foo {
		let foo %3D this%3B
		while (foo._parent) {
			foo %3D foo._parent%3B
		}
		return foo%3B
	}
	private _parent%3A Foo%3B
}

Since this is a backwards incompatibility, the current 1.7 release violates semver.

@sccolbert you're right that this is a breaking change, and we've documented it on the breaking changes list for future reference. I'm sorry about the inconvenience that this caused, but the workaround here is small enough that it shouldn't be anything more than adding a type annotation to foo in your example.
Regarding semver, you're also right in that respect, but in general we don't really follow semver. It's not something we're unaware of, but I'm probably not the right person to answer why TypeScript 1.7 isn't called TypeScript 7.

@DanielRosenwasser Thanks for the quick response.
Is following semver something that the TS team would consider, given how vital it is to the way npm and other package managers do things?


Not working in dotnet 6 on linux

It seems that in dotnet 6 there's a breaking change that drops support for System.Drawing.Common in non-windows environments. My application throws an exception with this message:
System.Drawing.Common is not supported on non-Windows platforms. See https://aka.ms/systemdrawingnonwindows for more information.
And this is the stack trace:
at System.Drawing.SafeNativeMethods.Gdip.GdipGetGenericFontFamilySansSerif(IntPtr& fontfamily)
at System.Drawing.FontFamily.GetGdipGenericSansSerif()
at System.Drawing.FontFamily.get_GenericSansSerif()
at System.Drawing.Font.CreateFont(String familyName, Single emSize, FontStyle style, GraphicsUnit unit, Byte charSet, Boolean isVertical)
at System.Drawing.Font..ctor(String familyName, Single emSize, FontStyle style, GraphicsUnit unit, Byte gdiCharSet, Boolean gdiVerticalFont)
at System.Drawing.Font..ctor(String familyName, Single emSize, FontStyle style, GraphicsUnit unit)
at NPOI.SS.Util.SheetUtil.IFont2Font(IFont font1)
at NPOI.SS.Util.SheetUtil.GetDefaultCharWidth(IWorkbook wb)
at NPOI.SS.Util.SheetUtil.GetColumnWidth(ISheet sheet, Int32 column, Boolean useMergedCells, Int32 firstRow, Int32 lastRow)
at NPOI.SS.Util.SheetUtil.GetColumnWidth(ISheet sheet, Int32 column, Boolean useMergedCells)
at NPOI.XSSF.UserModel.XSSFSheet.AutoSizeColumn(Int32 column, Boolean useMergedCells)
at NPOI.XSSF.UserModel.XSSFSheet.AutoSizeColumn(Int32 column)
Is there any workaround or upcoming fix to this problem?

We don't have plan to workaround this issue so far. Please try to use .NET 5 instead. Frankly speaking, Microsoft made a very bad decision. It's just like MSFT doesn't want to contribute to cross-platform System.Drawing anymore. And the existing projects which are using System.Drawing.Common have to reply on your own.

#656


[R] Error when trying to save a data.frame with NULL column names

Describe the bug, including details regarding any error messages, version, and platform.
df <- data.frame(a=1:3,b=4:6)
names(df) <- NULL
arrow::write_feather(x=df, sink="~/tmp/df")
# Error: Invalid input type, expected 'character' actual 'NULL'

Component(s)
R

Thanks for reporting this @OfekShilon!
I had a look at our other file writing functions to have a look at behaviour, and this isn't just restricted to those.  Interestingly, readr::write_csv() has the same behaviour.
df <- data.frame(a=1:3,b=4:6)
names(df) <- NULL
arrow::write_feather(x=df, sink="~/tmp/df")
#> Error: Invalid input type, expected 'character' actual 'NULL'
arrow::write_csv_arrow(df, "tst.csv")
#> Error in `value[[3L]]()`:
#> ! x must be an object of class 'data.frame', 'RecordBatch', 'Dataset', 'Table', or 'RecordBatchReader' not 'data.frame'.

#> Backtrace:
#>     ▆
#>  1. └─arrow::write_csv_arrow(df, "tst.csv")
#>  2.   └─base::tryCatch(...)
#>  3.     └─base (local) tryCatchList(expr, classes, parentenv, handlers)
#>  4.       └─base (local) tryCatchOne(expr, names, parentenv, handlers[[1L]])
#>  5.         └─value[[3L]](cond)
#>  6.           └─rlang::abort(...)
arrow::write_parquet(df, "tst.csv")
#> Error: Invalid input type, expected 'character' actual 'NULL'

readr::write_csv(df, "tst.csv")
#> Error: Invalid input type, expected 'character' actual 'NULL'

Can I ask a bit more about the use case here, and the circumstances in which it's desirable to have a data.frame with NULL column names?

@thisisnic We didn't have a direct need for this behavior - it was part of some complex logic that deleted col names in one part, than saved/loaded and applied new names at a different part.  We did work around it, it's just another place where the migration from native robj serialization to arrow bit us and we thought we'd report.

OK, good to know.  In that case, I think a sensible solution will be adding some validation of input df names so that the error message supplied to the user is more informative.


atom: 1.9.9 -> 1.10.1

Motivation for this change
Atom out of date, updated to latest
Things done

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

[x] NixOS
[ ] OS X
[ ] 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.

not sure what nox-review did, but it output this:
/nix/store/zb4495hajc88gbghsr30i6d7n50q7104-atom-1.10.1
Result in /run/user/1000/nox-review-_dklyhwm
total 0
lrwxrwxrwx 1 bdimchef users 55 Sep  7 09:26 result -> /nix/store/zb4495hajc88gbghsr30i6d7n50q7104-atom-1.10.1

I also doubt anything depends on atom, so maybe that's expected.


VQADataset

hello, I try to run your train.py.  However, "from dense_coattn.data import VQADataset". Maybe you forget updating your code in dense_coattn.data.

Hi,
Thanks for your comment! I already upload the new data folder. Please tell me if there is any other problem.


Switch URL to HTTPS for FastStone Image Viewer version 7.5

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


Can't see GUI in Ubuntu

I have Linux Ubuntu 16.04 LTS Xenial. It seems I cannot get the GUI even with the instructions given. Any help is appreciated.
Here is the code and the error:
sudo docker run -i --rm --privileged -e DISPLAY -v /tmp/.X11-unix:/tmp/.X11-unix -v /tmp:/scratch -v /media/egarza/INP_MRI_Backup/projects/INP/addimex_tms/data/mri/nifti:/bids_dataset -v /media/egarza/INP_MRI_Backup/projects/INP/addimex_tms/data/mri/outputs:/outputs bids/cpac /bids_dataset /outputs GUI No protocol specified Namespace(analysis_level='GUI', aws_input_creds=None, aws_output_creds=None, bids_dir='/bids_dataset', data_config_file=None, mem_gb=None, mem_mb=None, n_cpus='1', output_dir='/outputs', participant_label=None, participant_ndx=None, pipeline_file='/cpac_resources/default_pipeline.yaml', save_working_dir=False) Starting CPAC GUI Unable to access the X Display, is $DISPLAY set properly?

We will look at this. Sorry for the delay.
Sent from my iPhone

On Jul 29, 2017, at 1:15 PM, Eduardo Garza-Villarreal<EMAIL_ADDRESS>wrote:
I have Linux Ubuntu 16.04 LTS Xenial. It seems I cannot get the GUI even with the instructions given. Any help is appreciated.
Here is the code and the error:
sudo docker run -i --rm --privileged -e DISPLAY -v /tmp/.X11-unix:/tmp/.X11-unix -v /tmp:/scratch -v /media/egarza/INP_MRI_Backup/projects/INP/addimex_tms/data/mri/nifti:/bids_dataset -v /media/egarza/INP_MRI_Backup/projects/INP/addimex_tms/data/mri/outputs:/outputs bids/cpac /bids_dataset /outputs GUI No protocol specified Namespace(analysis_level='GUI', aws_input_creds=None, aws_output_creds=None, bids_dir='/bids_dataset', data_config_file=None, mem_gb=None, mem_mb=None, n_cpus='1', output_dir='/outputs', participant_label=None, participant_ndx=None, pipeline_file='/cpac_resources/default_pipeline.yaml', save_working_dir=False) Starting CPAC GUI Unable to access the X Display, is $DISPLAY set properly?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

No problem, sir. I tried several fixes such as this [http://fabiorehm.com/blog/2014/09/11/running-gui-apps-with-docker/], but no luck.
Thanks
Ed

Are you willing to try installing singilarity? It should just work
Sent from my iPhone

On Aug 1, 2017, at 11:31 AM, Eduardo Garza-Villarreal<EMAIL_ADDRESS>wrote:
No problem, sir. I tried several fixes such as this [http://fabiorehm.com/blog/2014/09/11/running-gui-apps-with-docker/], but no luck.
Thanks
Ed
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I did that already, then tried to import the docker2singularity cpac with no luck. I get an error. Let me paste that error.
Is there a way to setup cpac from singularity instead of using the converter?

sudo docker run --privileged -ti --rm -v /var/run/docker.sock:/var/run/docke r.sock -v /home/egarza/MRI/apps/singularity_images:/output singularityware/docker2singularity bids/cpac Size: 5533 MB for the singularity container (1/9) Creating an empty image... Creating a sparse image with a maximum size of 5533MiB... Using given image size of 5533 Formatting image (/sbin/mkfs.ext3) Done. Image can be found at: /tmp/bids_cpac-2017-03-29-71120146c4f4.img (2/9) Importing filesystem... (3/9) Bootstrapping... (4/9) Adding run script... (5/9) Setting ENV variables... Singularity: sexec (U=0,P=146)> Command=exec, Container=/tmp/bids_cpac-2017-03-29-71120146c4f4.img, CWD=/, Arg1=/bin/sh (6/9) Adding mount points... Singularity: sexec (U=0,P=152)> Command=exec, Container=/tmp/bids_cpac-2017-03-29-71120146c4f4.img, CWD=/, Arg1=/bin/sh (7/9) Fixing permissions... Singularity: sexec (U=0,P=158)> Command=exec, Container=/tmp/bids_cpac-2017-03-29-71120146c4f4.img, CWD=/, Arg1=/bin/sh We're running on BusyBox/Buildroot Singularity: sexec (U=0,P=201)> Command=exec, Container=/tmp/bids_cpac-2017-03-29-71120146c4f4.img, CWD=/, Arg1=/bin/sh find: /proc/2/task/2/fd/11': No such file or directory
find: /proc/2/task/2/fd/11': No such file or directory find: /proc/2/task/2/fdinfo/11': No such file or directory
find: /proc/2/task/2/fdinfo/11': No such file or directory find: /proc/2/fd/11': No such file or directory
find: /proc/2/fd/11': No such file or directory find: /proc/2/fdinfo/11': No such file or directory
find: /proc/2/fdinfo/11': No such file or directory

I have a singularity image you can download. I will send info later today.
Sent from my iPhone

On Aug 1, 2017, at 11:42 AM, Eduardo Garza-Villarreal<EMAIL_ADDRESS>wrote:
sudo docker run --privileged -ti --rm -v /var/run/docker.sock:/var/run/docke r.sock -v /home/egarza/MRI/apps/singularity_images:/output singularityware/docker2singularity bids/cpac Size: 5533 MB for the singularity container (1/9) Creating an empty image... Creating a sparse image with a maximum size of 5533MiB... Using given image size of 5533 Formatting image (/sbin/mkfs.ext3) Done. Image can be found at: /tmp/bids_cpac-2017-03-29-71120146c4f4.img (2/9) Importing filesystem... (3/9) Bootstrapping... (4/9) Adding run script... (5/9) Setting ENV variables... Singularity: sexec (U=0,P=146)> Command=exec, Container=/tmp/bids_cpac-2017-03-29-71120146c4f4.img, CWD=/, Arg1=/bin/sh (6/9) Adding mount points... Singularity: sexec (U=0,P=152)> Command=exec, Container=/tmp/bids_cpac-2017-03-29-71120146c4f4.img, CWD=/, Arg1=/bin/sh (7/9) Fixing permissions... Singularity: sexec (U=0,P=158)> Command=exec, Container=/tmp/bids_cpac-2017-03-29-71120146c4f4.img, CWD=/, Arg1=/bin/sh We're running on BusyBox/Buildroot Singularity: sexec (U=0,P=201)> Command=exec, Container=/tmp/bids_cpac-2017-03-29-71120146c4f4.img, CWD=/, Arg1=/bin/sh find:/proc/2/task/2/fd/11': No such file or directory
find: /proc/2/task/2/fd/11': No such file or directory find:/proc/2/task/2/fdinfo/11': No such file or directory
find: /proc/2/task/2/fdinfo/11': No such file or directory find:/proc/2/fd/11': No such file or directory
find: /proc/2/fd/11': No such file or directory find:/proc/2/fdinfo/11': No such file or directory
find: /proc/2/fdinfo/11': No such file or directory
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Cool, thanks.

As a follow-up, my student just installed bids/cpac in his Macbook Pro OSX El Capitan. We did all the instructions to get the Display, and we get the same error with Display.

Tried with singularity and GUI worked like a charm.
Now, I tried to make and save a subjects list and gave me the same error as with the full CPAC:
singularity run -B /home/egarza:/mnt -B /mnt:/scratch -B /media/egarza/INP_MRI_Backup/projects/INP/addimex_tms/data/mri/bids:/bids_dataset -B /media/egarza/INP_MRI_Backup/projects/INP/addimex_tms/data/mri/outputs:/output /home/egarza/MRI/apps/singularity_images/bids_cpac-2017-03-29-a12af7f5d881.img /bids_dataset /output GUI
Namespace(analysis_level='GUI', aws_input_creds=None, aws_output_creds=None, bids_dir='/bids_dataset', data_config_file=None, mem_gb=None, mem_mb=None, n_cpus='1', output_dir='/output', participant_label=None, participant_ndx=None, pipeline_file='/cpac_resources/default_pipeline.yaml', save_working_dir=False)
Starting CPAC GUI
Gtk-Message: Failed to load module "gail"
Gtk-Message: Failed to load module "atk-bridge"
Gtk-Message: Failed to load module "unity-gtk-module"
Gtk-Message: Failed to load module "canberra-gtk-module"

(run.py:30602): Gtk-WARNING **: Unable to find default local file monitor type

(run.py:30602): Gtk-WARNING **: Unable to find default local file monitor type
saving /media/egarza/INP_MRI_Backup/projects/INP/addimex_tms/data/mri/data_config_addimex_tms.yaml
Gathering anatomical files...
2017-08-01 13:39:46,657 : Found 19 files!
170801-13:39:46,657 sublist_builder INFO:
	 Found 19 files!
Gathering functional files...
2017-08-01 13:39:46,658 : Found 17 files!
170801-13:39:46,658 sublist_builder INFO:
	 Found 17 files!
2017-08-01 13:39:46,658 : Filtering anatomical files...
170801-13:39:46,658 sublist_builder INFO:
	 Filtering anatomical files...
2017-08-01 13:39:46,658 : Not filtering out any potential sites...
170801-13:39:46,658 sublist_builder INFO:
	 Not filtering out any potential sites...
2017-08-01 13:39:46,658 : Filtered down to 19 files
170801-13:39:46,658 sublist_builder INFO:
	 Filtered down to 19 files
Filtered down to 19 anatomical files
2017-08-01 13:39:46,658 : Not filtering out any potential sites...
170801-13:39:46,658 sublist_builder INFO:
	 Not filtering out any potential sites...
2017-08-01 13:39:46,658 : Filtered down to 17 files
170801-13:39:46,658 sublist_builder INFO:
	 Filtered down to 17 files

(run.py:30602): Gtk-WARNING **: Unable to find default local file monitor type
Error importing CPAC
cannot import name fsopendir

Is there a conflict between cpacs?
Thanks
Ed

Still can't save the data nor pipeline files using the GUI with singularity. I opted for editing the regular pipeline, but it would greatly help our work if we could do it from the GUI at some point, specially for teaching reasons.
Thanks
Ed

I am having the same issue with an ubuntu machine (14.04.5 LTS). The issue being: "Unable to access the X Display, is $DISPLAY set properly?" Has there been any resolution to using docker?

i have been able to use it following these instructions:
https://github.com/bids-apps/cpac#running-docker-container-on-linux
make sure that you are correctly passing the DISPLAY environment variable
into the container
On Fri, Jun 29, 2018 at 1:09 PM Zarrar Shehzad<EMAIL_ADDRESS>wrote:

I am having the same issue with an ubuntu machine (14.04.5 LTS). The issue
being: "Unable to access the X Display, is $DISPLAY set properly?" Has
there been any resolution to using docker?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BIDS-Apps/CPAC/issues/11#issuecomment-401431772, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABpTWEhBvh95yued27Dx43z91eVLh7kyks5uBm1TgaJpZM4OnaPh
.

--
Associate Professor of Diagnostic Medicine, Dell Medical School at the
University of Texas at Austin
Director, Computational NeuroImaging Lab
http://computational-neuroimaging-lab.org/
Founding Chair, Organization for Human Brain Mapping Open Science Special
Interest Group
Co-Founder, The Neuro Bureau http://neurobureau.org/
Impact Story https://impactstory.org/CameronCraddock
Google Scholar Citations
http://scholar.google.com/citations?user=lZ9hwSsAAAAJ&hl=en
GitHub https://github.com/ccraddock
SlideShare http://www.slideshare.net/CameronCraddock
Check out my recent blog Open Science Resources for Contributing to Brain
Imaging Research
http://blogs.biomedcentral.com/gigablog/2016/03/15/open-science-resources-contributing-brain-imaging-research-guest-blog-cameron-craddock/
Share that Brain @ the International Neuroimaging Datasharing Initiative
http://fcon_1000.projects.nitrc.org/!
The GigaScience Brainhack Thematic Series
http://brainhack.org/brainhack-thematic-series-call-for-papers/ is open
for your submission.
Submit a paper to the Neuroimage Special Issue on Brain Segmentation and
Parcellation
http://www.journals.elsevier.com/neuroimage/call-for-papers/brain-segmentation-and-parcellation/
!
The Preprocessed Connectomes Project
http://preprocessed-connectomes-project.org/ is now sharing preprocessed
versions of the ABIDE and ADHD200 neuroimaging datasets.
Join me at upcoming Brainhack http://brainhack.org/ Events:
2017.03.02 - 2016.03.05 Brainhack Global
http://events.brainhack.org/global2017
Remember to submit your Brainhack Project Report to the 2016 Brainhack
Proceedings http://brainhack.org/proceedings

or try singularity, it just works
On Fri, Jun 29, 2018 at 1:10 PM Cameron Craddock<EMAIL_ADDRESS>wrote:

i have been able to use it following these instructions:
https://github.com/bids-apps/cpac#running-docker-container-on-linux
make sure that you are correctly passing the DISPLAY environment variable
into the container
On Fri, Jun 29, 2018 at 1:09 PM Zarrar Shehzad<EMAIL_ADDRESS>wrote:

I am having the same issue with an ubuntu machine (14.04.5 LTS). The
issue being: "Unable to access the X Display, is $DISPLAY set properly?"
Has there been any resolution to using docker?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BIDS-Apps/CPAC/issues/11#issuecomment-401431772, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABpTWEhBvh95yued27Dx43z91eVLh7kyks5uBm1TgaJpZM4OnaPh
.

--
Associate Professor of Diagnostic Medicine, Dell Medical School at the
University of Texas at Austin
Director, Computational NeuroImaging Lab
http://computational-neuroimaging-lab.org/
Founding Chair, Organization for Human Brain Mapping Open Science Special
Interest Group
Co-Founder, The Neuro Bureau http://neurobureau.org/
Impact Story https://impactstory.org/CameronCraddock
Google Scholar Citations
http://scholar.google.com/citations?user=lZ9hwSsAAAAJ&hl=en
GitHub https://github.com/ccraddock
SlideShare http://www.slideshare.net/CameronCraddock
Check out my recent blog Open Science Resources for Contributing to Brain
Imaging Research
http://blogs.biomedcentral.com/gigablog/2016/03/15/open-science-resources-contributing-brain-imaging-research-guest-blog-cameron-craddock/
Share that Brain @ the International Neuroimaging Datasharing Initiative
http://fcon_1000.projects.nitrc.org/!
The GigaScience Brainhack Thematic Series
http://brainhack.org/brainhack-thematic-series-call-for-papers/ is open
for your submission.
Submit a paper to the Neuroimage Special Issue on Brain Segmentation and
Parcellation
http://www.journals.elsevier.com/neuroimage/call-for-papers/brain-segmentation-and-parcellation/
!
The Preprocessed Connectomes Project
http://preprocessed-connectomes-project.org/ is now sharing
preprocessed versions of the ABIDE and ADHD200 neuroimaging datasets.
Join me at upcoming Brainhack http://brainhack.org/ Events:
2017.03.02 - 2016.03.05 Brainhack Global
http://events.brainhack.org/global2017
Remember to submit your Brainhack Project Report to the 2016 Brainhack
Proceedings http://brainhack.org/proceedings

--
Associate Professor of Diagnostic Medicine, Dell Medical School at the
University of Texas at Austin
Director, Computational NeuroImaging Lab
http://computational-neuroimaging-lab.org/
Founding Chair, Organization for Human Brain Mapping Open Science Special
Interest Group
Co-Founder, The Neuro Bureau http://neurobureau.org/
Impact Story https://impactstory.org/CameronCraddock
Google Scholar Citations
http://scholar.google.com/citations?user=lZ9hwSsAAAAJ&hl=en
GitHub https://github.com/ccraddock
SlideShare http://www.slideshare.net/CameronCraddock
Check out my recent blog Open Science Resources for Contributing to Brain
Imaging Research
http://blogs.biomedcentral.com/gigablog/2016/03/15/open-science-resources-contributing-brain-imaging-research-guest-blog-cameron-craddock/
Share that Brain @ the International Neuroimaging Datasharing Initiative
http://fcon_1000.projects.nitrc.org/!
The GigaScience Brainhack Thematic Series
http://brainhack.org/brainhack-thematic-series-call-for-papers/ is open
for your submission.
Submit a paper to the Neuroimage Special Issue on Brain Segmentation and
Parcellation
http://www.journals.elsevier.com/neuroimage/call-for-papers/brain-segmentation-and-parcellation/
!
The Preprocessed Connectomes Project
http://preprocessed-connectomes-project.org/ is now sharing preprocessed
versions of the ABIDE and ADHD200 neuroimaging datasets.
Join me at upcoming Brainhack http://brainhack.org/ Events:
2017.03.02 - 2016.03.05 Brainhack Global
http://events.brainhack.org/global2017
Remember to submit your Brainhack Project Report to the 2016 Brainhack
Proceedings http://brainhack.org/proceedings

Hi Cam,
We still get the same error with Docker. In different computers with Ubuntu.
Singularity works with the GUI but we are still unable to save the YML file.

OK, i will look into this again.
On Fri, Jun 29, 2018 at 1:56 PM Eduardo Garza-Villarreal <
<EMAIL_ADDRESS>wrote:

Hi Cam,
We still get the same error with Docker. In different computers with
Ubuntu.
Singularity works with the GUI but we are still unable to save the YML
file.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BIDS-Apps/CPAC/issues/11#issuecomment-401443987, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABpTWHH4EfUB1LdgbEqDTOMH9aV39OS-ks5uBnhrgaJpZM4OnaPh
.

--
Associate Professor of Diagnostic Medicine, Dell Medical School at the
University of Texas at Austin
Director, Computational NeuroImaging Lab
http://computational-neuroimaging-lab.org/
Founding Chair, Organization for Human Brain Mapping Open Science Special
Interest Group
Co-Founder, The Neuro Bureau http://neurobureau.org/
Impact Story https://impactstory.org/CameronCraddock
Google Scholar Citations
http://scholar.google.com/citations?user=lZ9hwSsAAAAJ&hl=en
GitHub https://github.com/ccraddock
SlideShare http://www.slideshare.net/CameronCraddock
Check out my recent blog Open Science Resources for Contributing to Brain
Imaging Research
http://blogs.biomedcentral.com/gigablog/2016/03/15/open-science-resources-contributing-brain-imaging-research-guest-blog-cameron-craddock/
Share that Brain @ the International Neuroimaging Datasharing Initiative
http://fcon_1000.projects.nitrc.org/!
The GigaScience Brainhack Thematic Series
http://brainhack.org/brainhack-thematic-series-call-for-papers/ is open
for your submission.
Submit a paper to the Neuroimage Special Issue on Brain Segmentation and
Parcellation
http://www.journals.elsevier.com/neuroimage/call-for-papers/brain-segmentation-and-parcellation/
!
The Preprocessed Connectomes Project
http://preprocessed-connectomes-project.org/ is now sharing preprocessed
versions of the ABIDE and ADHD200 neuroimaging datasets.
Join me at upcoming Brainhack http://brainhack.org/ Events:
2017.03.02 - 2016.03.05 Brainhack Global
http://events.brainhack.org/global2017
Remember to submit your Brainhack Project Report to the 2016 Brainhack
Proceedings http://brainhack.org/proceedings

I have the reverse problem. I can save the YAML file with singularity but when I try to run preprocessing, it doesn't do anything. Well it shows one line (below) but then nothing is run.
strategy_entries:  [['_compcor_ncomponents_5_selector_pc10.linear0.wm0.global0.motion1.quadratic0.gm0.compcor1.csf0', '_target_angle_deg_90', '_threshold_0.2']]

Which singularity container are you using?
Sent from my iPhone

On Jun 30, 2018, at 5:55 PM, Zarrar Shehzad<EMAIL_ADDRESS>wrote:
I have the reverse problem. I can save the YAML file with singularity but when I try to run preprocessing, it doesn't do anything. Well it shows one line (below) but then nothing is run.
strategy_entries: [['_compcor_ncomponents_5_selector_pc10.linear0.wm0.global0.motion1.quadratic0.gm0.compcor1.csf0', '_target_angle_deg_90', '_threshold_0.2']]
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

The one I made by converting the CPAC docker to singularity.

Ok. What version of the docket container
Sent from my iPhone

On Jun 30, 2018, at 9:40 PM, Zarrar Shehzad<EMAIL_ADDRESS>wrote:
The one I made by converting the CPAC docker to singularity.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I was able to get the GUI to work using x11docker (https://github.com/mviereck/x11docker). Would you mind trying that? This sample worked for me: x11docker --bids/cpac /tmp /tmp GUI

I am able to run the GUI that way, I include the --homedir flag and it works.
I can save the Subject file, but when I try to Generate the Subject YAML it tells me this:

I will look into this further. In the meantime, can you generate your data config file using the 'test_config' flag to the CPAC app? Also, you appear to be using an older version of CPAC (the logo is different), perhaps you could try with a newer version?

Everything works for me now in singularity. There was a path issue for the inputs that wasn't being displayed.

@czarrar would you please elaborate so that we can fix the error reporting in future versions?

When running in the 'participant' instead of 'GUI' option, I saw that I had improper BIDS paths that was causing an issue. In the 'GUI' setting, when I ran the analysis, it didn't show this error or any of the output that comes with CPAC. However, when I fixed the path issue, then going through the 'GUI' ran fine. Hope this helps.

I downloaded the latest bids/cpac. Now it does not give me the error, but it also does not Generate the Data Config file. And still, the CPAC looks as if it crashed but it is still responsive. Very strange.
I will use the test_config and see if I can save the pipeline. Thanks

Made Singularity GUI work using the instructions. Now I did not have a problem converting the docker image to singularity and it all opened fine. I do have a problem, but I made it another Issue. It may seem this new issue is similar to the Docker issue with x11 as it has to do more with the Path Search.


Match arbitrary code blocks

Fixing #40

@izahn the fix was pretty simple. Previous behaviour was a bit restrictive and only allowed matching on a limited set of languages. Now, matches can be on anything.
With --match=matchthis we would match fenced code blocks looking like
```matchthis

and pandoc code blocks looking like
```{#id .foo .bar .matchthis}

I think this fixes your use case whilst also being more useful in general.

Good news, thanks! I commented on the commit to note that it breaks in
python 3 due to a python 2 style print statement.
On Mar 24, 2016 6:04 AM, "aaren"<EMAIL_ADDRESS>wrote:

@izahn https://github.com/izahn the fix was pretty simple. Previous
behaviour was a bit restrictive and only allowed matching on a limited set
of languages. Now, matches can be on anything.
With --match=matchthis we would match fenced code blocks looking like

and pandoc code blocks looking like

```{#id .foo .bar .matchthis}

I think this fixes your use case whilst also being more useful in general.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
<https://github.com/aaren/notedown/pull/41#issuecomment-200768470>

I think one of the travis failures is a caching / timeout problem so merging as is.


Developers workshop

@mmarsoleta, Krista is fairly free the next could of weeks so we can just put something in her calendar. Kevin is away on holidays until the 29th of August. I'm waiting to hear back from Laura Klade - she gets very busy this time of year so it might be easier to do a one-on-one with her after we have assessed the complexity of the features


[openCypher tck case] Pattern Comprehension

Unsupported.
Feature: Pattern3 - Pattern Comprehension

  Scenario: [1] Returning a pattern comprehension
    Given an empty graph
    And having executed:
      """
      CREATE (a:A)
      CREATE (a)-[:T]->(:B),
             (a)-[:T]->(:C)
      """
    When executing query:
      """
      MATCH (n)
      RETURN [p = (n)-->() | p] AS ps
      """
    Then the result should be, in any order:
      | ps                                     |
      | [<(:A)-[:T]->(:C)>, <(:A)-[:T]->(:B)>] |
      | []                                     |
      | []                                     |
    And no side effects

refer to : https://neo4j.com/docs/cypher-manual/current/syntax/lists/#cypher-pattern-comprehension

refer to : https://neo4j.com/docs/cypher-manual/current/syntax/lists/#cypher-pattern-comprehension


idea: consider no longer re-exporting anything in app

Since it is fairly common to extend the session service within an app, the re-export is both unneeded and in many build setups results in indeterminant service resolution during build when the app uses session.ts instead of session.js

Great timing for the suggestion as I'm planning to make some breaking changes.
AFAIK that's what ember-data also does in newer versions - where you're required to import it yourself, but I'd like to ask.
What about initializers?
Would you expect users to import them themselves?
Or is it a time to come up with an implementation that doesn't rely on them?

closed by #2895


修改了不通顺的文字

 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.mingyang.zeng seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you very much！


Unable to see OpenId implementation on UI

Could you please check now.

Great. It's working now. Closing the issue!


Allow contributors to npm install directly from git remote url

Add a prepare script to package.json so contributors can run their fork directly from git, by adding the url of their fork in their main project's package.json using something like "mui-datatables": "https://github.com/benoitg/mui-datatables.git".  See https://docs.npmjs.com/cli/install for reference

Coverage remained the same at 72.082% when pulling a08aa5d4ef1967cd5420d233368d79cc50dcba5f on benoitg:master into e9938b8f3ac4fa06e05d52d2596fd3411148d57c on gregnb:master.


Break reminder

Description of the new feature / enhancement
Break reminder with the following features:

reminder notifications for when it's time to take a break, stretch, drink water, etc.
customizable notifications (silent, screen overlay, sounds, ...)
customizable reminder frequency
customizable active hours
customizable active days
maybe lock the screen for 5 min. e.g.

Scenario when this would be used?
power users sit the whole day behind their screens all day & night. Day in - day out. That is not the most healthy thing to do for humans. The power user feels the consequences of this routine later on. A break reminder would protect power users from themselfs.
Supporting information
No response

Neat idea, i don't see powertoys building this out.


cmake build - required for win

What we have to change in cmake:
The shared library has to link to the executable, the ENABLE_EXPORTS target property of the executable has to be set, the executable has to export the required symbols and your shared library has to import them

Works :+1:


ci: Cleanups/optimizations for azp, Docker build and bazel

fix bazel invokation in build_setup.sh
use bazel.yml and matrix for precheck jobs
separate env -> env.yml
add cache for bazel artefacts - external directory, install dir, .cache
mount more tmpfs in self-hosted
ensure all bazel invokations use correct options to prevent creation of duplicate build environments

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Hey @phlax did this perhaps break  mobile coverage?
It's failing with gcc errors on my PR which I think are unrelated and looks like it failed the final run here
https://github.com/envoyproxy/envoy/actions/runs/4789920456/jobs/8518358939

its possible, but im struggling to think why
this seems to be the error

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
gcc: error: unrecognized command line option '-fprofile-instr-generate'; did you mean '-fprofile-generate'?
gcc: error: unrecognized command line option '-fcoverage-mapping'

is it possible something changed there somewhere ?
ill see if i can see any related settings/change

these settings seem to be mobile-specific

$ git grep coverage-mapping
mobile/third_party/rbe_configs/cc/cc_toolchain_config.bzl:                            "-fcoverage-mapping",
$ git grep profile-instr-gen
mobile/third_party/rbe_configs/cc/cc_toolchain_config.bzl:                            "-fprofile-instr-generate",
mobile/third_party/rbe_configs/cc/cc_toolchain_config.bzl:                    flag_group(flags = ["-fprofile-instr-generate"]),

trying to think why gcc would have changed or the args given to it

also mostly this is changes to azp - other than to the scripts in ci/  which i dont think mobile calls

mobile coverage calls the coverage script changed in this PR
https://github.com/envoyproxy/envoy/blob/main/.github/workflows/mobile-coverage.yml#L42
It's why I thought the PR might be related to the regression


Remove remaining --db options from cli commands

#2495 Modified the reth directory structure to use --datadir for all reth data, instead of having different possible paths for the db, config, jwt secret, etc. Previously the main way users would override a data directory was from the --db flag.
There are still a few remaining --db flags, which should not be necessary any more. For example:
https://github.com/paradigmxyz/reth/blob/ec7a10b57e3cf97ce6d12414e2f2266338d61351/bin/reth/src/chain/init.rs#L24-L27
https://github.com/paradigmxyz/reth/blob/ec7a10b57e3cf97ce6d12414e2f2266338d61351/bin/reth/src/db/mod.rs#L32-L35
These arguments should be removed and replaced with the directory set by the --datadir flag.

I'd like to work on this one :)

@leovct awesome! assigned :)

This was fixed in #2575


List submodules as well

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[x] Vim
[x] Neovim
[x] Etc.

OS

[x] Linux
[x] Mac OS X
[x] Windows
[x] Etc.

Shell

[x] bash
[x] zsh
[x] fish

Currently if you have a project with git submodules, fzf will only list files of the root project.
Submodules could easily be read from the .gitmodules file in the project root.
Implementing this in fzf could make sense because you could traverse faster than doing a bunch of piped unix commands to get the file list.

fzf is just a unix filter like grep and it's not really responsible traversing the file system. By default, it uses find command by default, but you can change it by setting FZF_DEFAULT_COMMAND.


Resistor enhancement

Please consider adding parameter to constructor of Resistor that specifies that _temp.Temperature(simulation) is executed in LoadBehavior of the resistor.
In other words: temperature behavior can be optionally executed on load.

Could you please elaborate on why that would be more advantageous than calling the temperature behavior in the BeforeLoad event or adding your own custom entities?
Each behavior now has a clearly defined function. Temperature behaviors are called when temperature changes, LoadBehavior is called for calculating a DC solution. Inserting a call to Temperature() in LoadBehavior would destroy that.

I will try :)
At the moment resistance is computed in TemperatureBehavior. I want to make a resistor which has a resistance that can change during simulation. It's not very easy to achieve now. That's way I wanted to introduce some kind of hack (e.g. calling Temperature() during load).
I feel that is not clean and not good solution. But i don't have a better and clear idea.

@svenboulanger  I will try your ideas.

@svenboulanger OK. You were right. The changes are not really needed.
However I would welcome change regarding to getting temperature behavior of specific entity:
I had to write following code:

You should be able to get it by using
var temp = simulation.EntityBehaviors[entity.Name].Get<BaseTemperatureBehavior>();


Initial impl of pytorch quant toolkit

When modify the graph for calibration, is it necessary to replace the layer to be quantized with the referenced one (one that has fake-quant for the weight). PyTorch does not do it. However, in many backends (like disc, trt, neural_engine), if there is no fake-quant op corresponding to the weight on the graph, the quantization will not be enabled.

Add copyright statement at the top of each file


Import with XCode 11 SwiftPackageManager doesn´t compile

Hi Rastislav,
thank you so much for updating Package.Swift
The import via XCode 11/Swift Packages/Add Package Dependency now works. :-)
But unfortunately, I get 104 Compiler Errors.
The first one is:
//
//  LimitedGestureViewDelegate.swift
//  FlexColorPicker
//
//  Created by Rastislav Mirek on 29/9/19.
//
//	MIT License
//  Copyright (c) 2018 Rastislav Mirek
//
//  Permission is hereby granted, free of charge, to any person obtaining a copy
//  of this software and associated documentation files (the "Software"), to deal
//  in the Software without restriction, including without limitation the rights
//  to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
//  copies of the Software, and to permit persons to whom the Software is
//  furnished to do so, subject to the following conditions:
//  The above copyright notice and this permission notice shall be included in all
//  copies or substantial portions of the Software.
 IN NO EVENT SHALL THE
//  AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
//  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
//  OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
//  SOFTWARE.
//
public protocol LimitedGestureViewDelegate {
func gestureRecognizerShouldBegin(_ gestureRecognizer: UIGestureRecognizer) -> Bool
}
Use of undeclared type 'UIGestureRecognizer'
Am I doing something wrong?

Hi Heiko, I am working on it. Give my 1 hour and try again. Let me know if it does not work after 1 hour.

Thank you.
You are doing great.
I started using FlexColorControl since yesterday and suddenly I could use SPM.
I am so lucky
Viele Grüße von Heiko

Am 08.10.2019 um 12:35 schrieb Rastislav Mirek<EMAIL_ADDRESS>
Hi Heiko, I am working on it. Give my 1 hour and try again. Let me know if it does not work after 1 hour.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Ok, I might need more than 1 hour but you should have it fix by this evening

Finally it's pushed, please try it now.

Hi,
I got another error with 1.3.1
/var/folders/03/1cwvdlhj0kl0rgytrf0r2s4r0000gn/T/TemporaryFile.uvjtFa.swift:38:5: error: argument 'platforms' must precede argument 'products'
    platforms: [
~~~~^~~~~~~~~~~~

Never ending story... Ok, fix comming right now.

Thanks. Please tag it such that the update can be picked up by SPM

Sorry for the trouble... How does it look now?

Much better. SPM trouble seems to be over now. Thanks.
But indeed there are compile errors:
import UIKit in LimitedGestureViewDelegate.swift, ColorPaleteControl.swift

I can add the explicit UIKit import to LimitedGestureViewDelegate.swift  but it is already in ColorPaleteControl... By any chance did you mean some other class?

No I see the use of undeclared type error in LimitedGestureViewDelegate.swift

If that's the only compile error you see it should be fixed now. Please try again. And your help is very apreciated.

Unfortunately it is not the only compile error: I think this should help
In RectangularHSBPaletteDelegate.swift, CGPointExtension.swift replace import Foundation by import UIKit

I do not understand why SPM would need that but it's done. Please try now, finger crossed that it finally works.

Just 6 issues left should be done by replacing import Foundation by import UIKit in CGRectExtension.swift

Changed the import. Are we there yet? Anybody else sees different errors?

It works now. No more error!! Migration to SPM is completed.
Thanks a lot @RastislavMirek

Superb! Thanks a lot.
Rastislav, if you like would you like to tell me, what you do professionally and how it you comes, you support such a nice Module for free.
Best regards from Heiko

Am 10.10.2019 um 04:40 schrieb Tim Bert<EMAIL_ADDRESS>It works now. No more error!! Migration to SPM is completed.
Thanks a lot @RastislavMirek https://github.com/RastislavMirek
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/RastislavMirek/FlexColorPicker/issues/20?email_source=notifications&email_token=AAOPYG4QDIEOFH2WCOWPPULQN2I2BA5CNFSM4I6P6ILKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA2HDOA#issuecomment-540307896, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOPYG5QNH64HGGUFCA6EUDQN2I2BANCNFSM4I6P6ILA.

Superb! Thanks a lot. Rastislav, if you like would you like to tell me, what you do professionally and how it you comes, you support such a nice Module for free. Best regards from Heiko
…

Professionally, I develop made to order iOS and React apps. FlexColorPicker is spin-off from one of my personal projects (EasyType Keyboard). I support it because it is used and because there seems to be no alternative "hackable" color picker for Swift.


remove definition of OPENSSL_VERSION_NUMBER

OPENSSL_VERSION_NUMBER is deprecated in 4+ but function in 3+.
quote from see opensslv.h
....
# if !OPENSSL_API_4
/* Synthesize OPENSSL_VERSION_NUMBER with the layout 0xMNN00PPSL */
...
#  define OPENSSL_VERSION_NUMBER        \
....

quote from cversion.c:
....
#if !OPENSSL_API_3
unsigned long OpenSSL_version_num(void)
{
    return OPENSSL_VERSION_NUMBER;
}
#endif
....

I cannot see reason for such difference.
Please synchronize versions - 3 is fine with me.

  Like this:

do this

Put your literal text inside three backticks one a line by themselves.

I agree that the difference is unnerving.  This has already been noticed during the debate in #7853 (feel free to share an opinion there)

Now I would like to change request  - from "deprecation of definition OPENSSL_VERSION_NUMBER and function OpenSSL_version_num() / master 2018-12-10" to remove definition of OPENSSL_VERSION_NUMBER .

So the reason is that format of macro definition is changed.
As result of this third party scrips(!) are not able to find  openssl version number.
Remark: Only executable could check properly version number, i.e. tools  has to perform compilation and environment to allows execution.

The similar impact of changes could be seen in script util/mktar.sh - due to change of format script does not work.

You're quite right about util/mktar.sh, thanks for the notification.  I've submitted #9745 to fix it, feel free to check it out.

Richard Levitte wrote:

You're quite right about util/mktar.sh, thanks for the notification.  I've submitted #9745 to fix it, feel free to check it out.

util/mktar.sh is not real issue at is part of mainstream code (when is
need it will adjusted to current code base).
Иmagine that this is third party software that uses definition of FOO -
change of value format requires changes in external software
nevertheless if is marked as deprecated or not.
Real case is this one:
Documentation:
https://cmake.org/cmake/help/latest/module/FindOpenSSL.html
Code:
https://gitlab.kitware.com/cmake/cmake/blob/master/Modules/FindOpenSSL.cmake#L350
Regards
Roumen Petrov

Well, I did think of external parties as well, so if you look at #9745, you will notice a new script that extracts version information in a uniform way regardless of (currently released) version.  Hopefully, programs such as cmake's FindOpenSSL will find that useful.

Richard Levitte wrote:

Well, I did think of external parties as well, so if you look at #9745, you will notice a new script that extracts version information in a uniform way regardless of (currently released) version.  Hopefully, programs such as cmake's FindOpenSSL will find that useful.

Script is not usable - for third party software it adds extra
dependencies to the script language(at build time).
In such case we could binary (executable) as openssl version outputs
required information. Unfortunately is not usable in cross-platform build.
Regards,
Roumen Petrov

On Sun, Sep 01, 2019 at 04:32:36AM -0700, Румен Петров wrote:

Script is not usable - for third party software it adds extra
dependencies to the script language(at build time).
In such case we could binary (executable) as openssl version outputs
required information. Unfortunately is not usable in cross-platform build.

To build software that wants to link to openssl, the
/usr/bin/openssl binary is not needed, and at least in Debian it's
normally not present.

Aha, gotcha.  It would be better with a generated script with just the values, created alongside openssl.pc etc.  Sure, that can be done as well

Generation of another (new) file with information will note resolve issue with third party software.
For instance CMAKE still will fail to detect OpenSSL.
Let assume that CMAKE enhance check for version to detect new format.  This is not security issue and more or less will not be back-ported. So users of "old" cmake will still impacted.
But I start to think what if is changed "direction" in hypothetical script - do not export but to import (generate).
What is a script using input from command line or file to generate include/openssl/opensslv.h and to propagate to other files if necessary. For instance "version" is used in generated pkg-config files.
Also script could update change log  (CHANGES) with version and date.

It's true that we could go that way...  A lot of other types of projects have the version number somewhere in configuration data, and generate whatever they need for presentation etc (a notable example is autoconf, where the version of the project is defined by AC_INIT), and cmake has a similar mechanism (although not as mandatory), as part of the project command.
We could simply add a VERSION statement in build.info...  or add a version number directly to $config{version}.  I'm not sure which I prefer...
Either way, we will have to tweak our release scripts...

Richard Levitte wrote:

....(a notable example is autoconf, where the version of the project is defined by AC_INIT)...

Yes, automake recommend file "version.m4" with M4-macros suitable for
AC_INIT and etc. Also it takes care for dependency and distribution of
that file.
If project uses autoheader this will produce a number of PACKAGE_*
definitions in configuration header (config.h by default).
But not everything is generated from configuration and build systems.
For instance in rpm-spec files also has version information.
At time of release configure.ac (indirectly from version.m4 if used) and
other files (for instance rpm-spec) has to be updated.
Next step is automation of release process: "prepare version <IP_ADDRESS>
....", where prepare is script.
Regards,
Roumen

Obsoleted by #10203


Random (safe) prime generation, how to go about it?

Hi @tarcieri and others. Thanks for all the hard work; I see so many changes since I last checked the repository!
For the scicrypt crate and e.g. the RSA crate it is necessary to generate (safe) primes. I also saw that this feature is on the feature wishlist.
Design considerations
I want to try my hand at implementing it but I would like to hear your opinion about a few design decisions:

What primality test to use? I suggest the Baille-PSW test, for which no pseudoprimes are currently known. It is also implemented in GMP.
Should the test run in constant time? I suggest a solution that is constant time only if the number is actually prime. I.e. if the test finds that the number is not prime, it can abort early.
How do we generate a random prime?

I want to elaborate a bit on the last point:
Given a constant-time primality test (or one that only runs in constant time when the number is actually prime), the only way I imagine being free of timing side-channels is to keep generating random potential primes until finding one that passes the test. However, this is very slow. A much faster approach (like in OpenSSL) is to sieve: We would first compute the residues of the potential prime modulo many small primes. If any equals 0, we increase all residues by 2 (or 4 for a safe prime), and try again. In this way we can weed out many composite numbers without having to run the primality test. However, this probably causes timing patterns that one might exploit to learn more about the random prime. I also don't know any papers about this that could help us.
My suggestion
I suggest that I implement:

The Baille-PWS test, which is only constant-time if the number is prime.
Random (safe) prime generation, which is not leaky by generating random numbers and testing if they are prime.
Random (safe) prime generation, which is leaky since it uses a sieve. (In this case, it might be considered wasteful that the underlying primality test runs in constant time.)

I am curious to hear your thoughts. I'm happy to help out where possible!

I believe @fjarri already has an implementation of random prime functionality although I don’t think it’s public yet

Oh that's good to know! I think it's https://github.com/entropyxyz/crypto-primes. Exciting work!

I am not sure BPSW test can be made constant time, since both of its constituents are data-dependent:

Miller-Rabin requires the number of iterations equal to the number of trailing zeros in n - 1,
Lucas (strong) requires one to do a number of checks equal to the number of trailing zeros in n + 1 (although it may be possible to make Lucas-V constant-time)

Moreover, is some usable information actually leaked this way?

It's my understanding that the first test in Baille-PSW is simply a Fermat primality test base 2, but that might be wrong. However, if this is the case, then that part sounds easy to do in constant time. For the Lucas test I was thinking about computing the right element of the Lucas sequence, similar to the Montgomery ladder. I'm curious to hear your thoughts.

No, the first test is more involved, it's Miller-Rabin test, and it has a variable number of iterations depending on n.
For the Lucas test, yes, the binary step is used to propagate the sequence, and it can be made branch-free - but it only needs to be propagated to such odd r that r * 2^k = n + 1, so again it depends on n. Although, as I mentioned, it does not apply to the Lucas-V test.
I filed https://github.com/entropyxyz/crypto-primes/issues/17 if you want to continue the discussion there.

I think this can be closed, since all the relevant discussion is in the crypto-primes issue above.


Added the --exclude flag to use while project.json file searching

Added the --exclude flag which excludes a comma-separated list of directories while searching for project.json config files
node_moduels/, .git, .svn  are already excluded in the code. developers can add other lists through --exclude flag

I assume you tested this inside and outside of a project?

I tested the following cases:
a) App with project.json file. b) App without project.json.file c)  App that has nested localization


Carthage Error:  The following build commands failed: 	CompileSwift normal x86_64 /Users

When running carthage update I get the following error:
The following build commands failed:
	CompileSwift normal x86_64 /Users

Hello. I reviewed the files of projects and it seemed to go OK, without absolute links. I personally do not use the Carthage. It would be interesting if someone that knows Carthage see this message. Maybe he/she could help best. Let us wait.

@vmartinelli very very thanks anyways! ill post this on twitte/SO to see if i can get the right person on this.

seems like this is the root reason for failures: Carthage/Carthage#417. I was able to just remove the 7.1 simulator and things were happy for me.
Probably be good to post the log from building as well though, it'll tell you where it is when you try to "carthage build". You can also "carthage build --no-skip-current" from your project's Carthage/Checkouts/AlecrimCoreData folder to attempt to build only AlecrimCoreData to get a log for only it it building.

Carthage support is removed for now. Sorry. Please see #42


Sprite3D would be called Sprite3d in Bevy

Feel free to just close this if you don't want to change it, but in Bevy the 3D types are named Foo3d rather than Foo3D:

I'm all for consistency! I'll change that, thanks for the feedback.


IC sources not attached for IU classes

I'm using this plugin with the IU version because I depend on the Javascript plugin. The IU jar files are included when I open the project in IntelliJ, but the IC source files that get downloaded alongside the IU classes are not attached properly. Whenever I refresh the Gradle model I have to attach the sources manually, which quickly gets annoying. It would be awesome if there was a way in which the plugin would handle this for me.

Should work. What version of IDEA do you use and what version of IU do you use?

https://github.com/Turbo87/intellij-emberjs/blob/master/build.gradle#L27
and I use the latest EAP of IntelliJ Ultimate (15.0.2 RC) for working on the project.

awesome, thanks!

@zolotov it seems that with a recent update to v0.2.12 of this plugin the automatic attachment/download of IC sources for IU stopped working again. any ideas how to fix this?

nevermind, I was being stupid... 🙈


CMAKE : itk Eigen3 generated cmakefiles do not inherit CMAKE_SYSTEM_VERSION

[context]
Windows 7 64b, Visual Studio 2019
fresh ITK v5.3.0
[description]
While trying to compile ITK 5.3.0 with CMake-GUI, I once again encountered  #1888
[analysis]
I finally found out that the generated ITKInternalEigen3-build\CMakeFiles\3.26.4\CMakeSystem.cmake contained the wrong set(CMAKE_SYSTEM_VERSION ...)
Actually, under Windows 7, when using CMake and Visual Studio 2019 with the latest Windows (10) SDK , I first have to set CMAKE_SYSTEM_VERSION to "10.0" before "Configuring" the CMake.
Unfortunately, it seems that the CMake files generated for Eigen3 ignore that specific tuning and resort to the auto-detected Windows environment, which is not usable in my case.
[fix]
Manually editing ITKInternalEigen3-build\CMakeFiles\3.26.4\CMakeSystem.cmake after a first pass of "Configure" to explicitely set(CMAKE_SYSTEM_VERSION "10.0") was the solution, and ITK 5.3.0 could be built and used without further problem.

I confirm that it seems to be fixed.
I tried to build the latest master branch (5.4) with my CMAKE/CMAKE_SYSTEM_VEERSION process, and it worked like a charm


cannot install on centos 7

OS
CentOS Linux release 7.6.1810 (Core)

Install error
[root@VM-4-4-centos ~]# pip3 install blinker
Looking in indexes: http://mirrors.tencentyun.com/pypi/simple
Collecting blinker
  Downloading http://mirrors.tencentyun.com/pypi/packages/1b/51/e2a9f3b757eb802f61dc1f2b09c8c99f6eb01cf06416c0671253536517b6/blinker-1.4.tar.gz (111 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 111.5/111.5 KB 7.1 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [1 lines of output]
      ERROR: Can not execute `setup.py` since setuptools is not available in the build environment.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

then I try to use yum
yum install python-blinker-1.3-2.el7.noarch.rpm

But it only can be used in python2, not have python3, I don't know what can I do.
I see https://centos.pkgs.org/7/centos-extras-x86_64/python-blinker-1.3-2.el7.noarch.rpm.html, the change log in 2014 say it support python3, but actually it not. Or I made a mistake, hope get your help.

Hi @harrylyx You do not have the setuptools library. Install it with the following command and try again.
pip install -U setuptools wheel pip


github stats not updating

The github stats for my pod JJFloatingActionButton have never been updated since I created it in November.
http://metrics.cocoapods.org/api/v1/pods/JJFloatingActionButton
I have released a couple of new versions since then.
Not sure if this is related to #43, but probably to CocoaPods/cocoadocs.org/issues/565.

my pod seems to be blocked also.
i did a clean pod lib create, pushed those files and still everything is stuck (readme does not change)

Same issue with: https://github.com/tonystone/tracelog

Same issue with mine:
https://cocoapods.org/pods/FilesProvider

This is still happening: https://cocoapods.org/pods/Criollo

My project's stars count hasn't been updated since last october (october 2017)
https://cocoapods.org/pods/PinLayout
Thanks

I have the same issue http://metrics.cocoapods.org/api/v1/pods/Localize.json

Same with http://metrics.cocoapods.org/api/v1/pods/Flowter.json
All the metrics has never been calculated at all.. :/

Same here, metrics not updated for https://cocoapods.org/pods/XMLCoder and https://cocoapods.org/pods/CoreXLSX

Same here (Fluky& Constrictor)

I never had stats for github.com/flowtoolz/SwiftObserver.
Is there anything special one has to do initially?
Like, does Cocoadocs look for sources only in specific folders?


Silent Badge Update Not Working

Here's my note:
var num_new = 5
var length = user.ios_tokens.length
var myDevices = []
for(var i = 0 ; i < length; i++) {
   myDevices.push(new apn.Device(user.ios_tokens[i]))
}

var note = new apn.Notification ();
note.badge = Number(num_new)
apnConnection.pushNotification ( note, myDevices );

I'm running with DEBUG=apn and can see the connection being established and getting drained so it appears to be sending.
My app has badges set to true in the notifications settings.
I can update the badge when sending a message with a complete payload, but not when I only send the badge number.
I've tried the following:

dropping the priority to 5 (using note.priority = 5) and that didn't work either.
setting alert, sound to null
adding an empty payload object
edit: few more things i've tried
sound=""

I figured I'd be getting some kind of feedback on failure but I'm not seeing anything - from what I can tell this should be updating the badge.
Closing edit: it looks like my ios developer might not be doing anything with alerts while the app is open - will re-open if the issue is indeed this lib.

Did you figure out the problem with this?

Looks to be iOS side. Will know for sure on Monday. You can delete if you'd like.

I'd prefer to reopen it if you're happy to provide an update when you confirm one way, or another?

Will do!

@argon the problem was how the iOS app was handling silent badge updates - your library performed as expected.
Thanks!