SIGINT trap consistent with TERM behaviour

Summary
fixes #988

Details
Basically now first Ctrl+C will terminate scenario execution but will stil execute at_exit hooks. Second Ctrl+C will also interrupt that.

Motivation and Context
see #988
In the past first Ctrl+C only told Cucumber to exit after this scenario. And a second one terminated immediately without at_exit hooks. So there was no way to interrupt immediately but still execute at_exit. This could lead to tested system to remain in inconsistent state and require manual restore.

How Has This Been Tested?

Not sure how can I test this.

I editted my local copy of the gem and run a test scenario
$  cucumber features/long_feature.feature
Feature: Long running scenarios

  @clean-up
  Scenario: 100 seconds with clean-up # features/long_feature.feature:4
    Given I love you                  # features/step_definitions/gah.rb:1
      hey
^C
Exiting... Interrupt again to exit immediately.
    When I wait 100 seconds           # features/step_definitions/gah.rb:8
      exit (SystemExit)
      ./features/step_definitions/gah.rb:9:in `sleep'
      ./features/step_definitions/gah.rb:9:in `/^I wait (\d+) seconds$/'
      features/long_feature.feature:6:in `When I wait 100 seconds'
    Then I do nothing                 # features/step_definitions/gah.rb:5
     --->  I cleaned you <---

Failing Scenarios:
cucumber features/long_feature.feature:4 # Scenario: 100 seconds with clean-up

1 scenario (1 failed)
3 steps (1 failed, 1 skipped, 1 passed)
0m2.806s

Screenshots (if appropriate):
Types of changes

[ ] Refactor (code change that does not change external functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Not sure about the above.
Checklist:

[ ] I've added tests for my code
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I'll pull this down and give it a quick play

So I couldn't get the new behaviour (Adding your one line didn't affect things for me), Possibly due to me not often using at_exit hooks.
# sample.feature

Feature: Test
  Scenario: One
    Given I say hello

  Scenario: Two with hook
    Given I say hello

# sample_steps.rb
Given('I say hello') do
  puts 'hello'
end

Before do
  puts 'start of scenario'
end

After do
  puts 'I am sleeping after the scenario'
  sleep 3
end

at_exit do
  puts 'I am a slow at_exit hook'
  sleep 3
end

Running without your code yielded the following
1 (Plain run)
start of scenario
hello
.
I am sleeping after the scenario
start of scenario
hello
.
I am sleeping after the scenario

2 scenarios (2 passed)
2 steps (2 passed)
0m6.014s
I am a slow at_exit hook

2 (Ctrl+C)
start of scenario
hello
.
I am sleeping after the scenario
^C
Exiting... Interrupt again to exit immediately.

1 scenario (1 passed)
1 step (1 passed)
0m3.004s
I am a slow at_exit hook

This suggests to me that without your code, Ctrl+C does still run the global teardown hooks.
If I've got something wrong, please let me know. I've also tried with your code and didn't notice any difference.

Forgot to write I have used this project as test:
https://github.com/akostadinov/testcuke
First of all the intention of the change is when hitting Ctrl+C during scenario execution, not during After hook. But as I see from your log the behaviour still makes sense. You have interrupted what is currently going on and go directly to at_exit for clean-up.
In my project I notice that After hook is executed at_exit so if you do Ctrl+C during scenario, then After hook will still be executed. If I hit Ctrl+C during After hook, then interrupting it and going to at_exit makes sense to me.
In the past one hits Ctrl+C but nothing apparently happens. You still need to wait for things to happen as normal. Second hit will stop everything wihtout a chance for clean-up. That's what I would like to see fixed. And this will be as far as I can tell consistent with kill my_cucumber_process.
I think if you use my project with wait times of 100 seconds, it will be more apparent. Please let me know what you think, I can add release notes. But I would appreciate to tell me what text will be desired more than SIGINT trap consistent with TERM behaviour which is already the commit message.
Thank you for looking into this!

Hi There,
Yes you are right, my example was bad. I've tried a new one with a slow scenario (Not before/after), and it works exactly as you described.
Can you just add something into the Changelog, explaining the before/after. Pretty much as you just have here. I'll get this merged in once that's done.
Cc/ @tooky I've tested this locally and it works exactly as he has described. This is a good bugfix. (Beforehand it was actually broken)

Done, let me know if wording is good.

Hi @akostadinov,
Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾
In return for this generous offer we hope you will:

✅ Continue to use branches and pull requests. When someone on the core team approves a pull request (yours or someone else's), you're welcome to merge it yourself.
💚 Commit to setting a good example by following and upholding our code of conduct in your interactions with other collaborators and users.
💬 Join the community Slack channel to meet the rest of the team and make yourself at home.
ℹ Don't feel obliged to help, just do what you can if you have the time and the energy.
🙋 Ask if you need anything. We're looking for feedback about how to make the project more welcoming, so please tell us!

On behalf of the Cucumber core team,
Aslak Hellesøy
Creator of Cucumber

Merged. Thanks for your contribution


Add License

ISC looks interesting, but MIT is more popular.
Make sure to add the license.txt file and to change it in the package.json and pom.xml if needed.

MIT is more popular.  Yolo


Implement periodic resync

etcd-based felix already supports resync but it doesn't trigger it on a timer, which I think we'd like to do.  This could be as simple as aborting stopping the polling loop so that we drop out and go round the resync loop again every n seconds.

Can I suggest considering adding some jitter to the timer?  The old zeroMQ Felixes could put very high load on the controller if they all resynced at once (after the whole cluster came up at roughly the same time).

Indeed, I've got an issue to track avoiding the thundering herd too.

I'm pondering how to do the timer for this, using time.time() may bite us later because it's non-monotonic.  I usually use monotime but it's not packaged.

We can package our own dependencies if we need to, it's not that tricky.

What about sleep() (within a dedicated thread) or add_timeout().  Surely gevent provides something like those?

How much do we really care about the monotonic issue? It seems to me like it's very marginal; not much risk of a problem, and not much that could go wrong if time.time() slips backwards by a second or something.

@plwhite There is no guarantee that the slip will only be a second. We cannot trust a non-monotonic clock not to jump around wildly. Given that we don't care about what the actual wall clock time is, we definitely want a monotonic clock.

Why would anyone implement a periodic X in any other way than

either a loop that calls sleep() and then does X

or using a library call of the style of glib's g_timeout_add (https://developer.gnome.org/glib/stable/glib-The-Main-Event-Loop.html#g-timeout-add).

In both cases, dealing with possibly non-monotonic time is Someone Else's Problem.  So why are we discussing it here?

@neiljerram we already have a loop that's doing the polling so the natural way that occurred to me to structure that code is to calculate the time at which we want to stop polling and resync and then check if we're past that time each time around the loop.
You're right, we could use something like gevent.spawn_later(), which is callback function that we have simple access to, and somehow kick the etcd polling loop.  I'll probably go with something like that if monotime isn't available.

Think we need to go back to first principles on this issue - i.e. why are we doing anything?  I don't think "which I think we'd like to do" is good enough.

Added a work-in-progress monotonic-time branch.  (Just a quick copy/paste from stack overflow.)

I think the idea is that periodic resync makes us more robust to various classes of errors and bugs that lead to us being out of sync and never spotting it.  For example:

bug in our etcd polling loop so it miscalculates the next polling index under some weird corner case
bug in etcd causing same
a lost event from etcd due to client or server bug
bug in our incremental update code for all our "indexes" over the data: this code is necessarily complex; even if it's well tested there may be lurking bugs
bug in our etcd directory deletion handling, failing to clean something up
transient failure to program iptables/ipsets caused by another process holding kernel locks
transient failure in our cleanup routines: e.g failure to delete iptables chain because its still referenced by something external and out of our control

It also puts our sync-to-snapshot routine on the mainline so that it gets more test exposure.  Right now we use it at start-of-day, on a blank "database" and after that, only when we hit an error in our polling loop.

@plwhite @Lukasa.  I've been thinking this are should be refactored a bit, particularly if we're moving towards implementing periodic resync and felix status reporting.
I think it makes sense to take the etcd loop off the etcd watcher's greenlet.  It doesn't fit the Actor model (a message that just blocks the queue!).  We should probably move it to its own greenlet and have it be managed by the EtcdWatcher greenlet instead.  The EtcdWatcher creates the polling greenlet then registers a callback in case it exits.  Since it should never exit, we probably just kill the process if that happens.
Then we change EtcdWatch to EtcdAPI or some such and expose API methods to send status reports and record errors.
For the periodic resync, we could do something very simple, such as start another greenlet that just sets a flag every n minutes and then sleeps.  The polling greenlet simply checks the flag and only carries on polling while it's set.  It's not very actor-y but Python's variables are atomic and volatile anyway so a boolean flag is safe.

If the clock jumps about wildly, it'll make our resynching a bit weird in terms of timing - but I don't think we care particularly, do we? So many things on the box are likely to break if the clock goes that bad, that we are probably the least possible problem.
More generally though, I really would like to keep this simple. I'm happy to tweak the etcd thread structure (which I agree has gone a bit far from a nice clean design, and could use some changes to make it more maintainable), but "very simple" sounds good to me - the extra greenlet that @fasaxc suggests sounds like it is probably a good part of that.


Old naming Concordium ID still present.

Description

When you delete app, dialog still stays remove “Concordium ID” instead of Concordium Testnet Wallet.
In import dialog, app is also still called “Concordium ID”.

Versions

Concordium Testnet Wallet v1.0(14)
iPhone 11, iOS 14.5.1

@concordium-cl, @kristiyandobrev
Findings from testing:
It longer says Concordium ID, but instead it says ProdTestNet both in the "delete app dialog" and below the app icon. Instead it should say Concordium Testnet Wallet.

Verified as working in 1.0(16)


remove manual gzipping of request, as it's done automatically!

📝 Summary
Fixes #365 and replaces #382
http.Request automatically adds Accept-Encoding: gzip header and transparently decompresses the response. You can see this working in the test in this PR. Also see https://pkg.go.dev/net/http#Response.Uncompressed
But when the header is set manually, then it won't decode the response automatically, which is what we're seeing in #382
In fact, gzip was already support since the very beginning of mev-boost. :)

✅ I have run these commands

[x] make lint
[x] make test-race
[x] go mod tidy

Codecov Report

Merging #383 (482c050) into main (1a11894) will increase coverage by 0.63%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #383      +/-   ##
==========================================
+ Coverage   66.02%   66.66%   +0.63%
==========================================
  Files           7        7
  Lines         889      888       -1
==========================================
+ Hits          587      592       +5
+ Misses        265      262       -3
+ Partials       37       34       -3

Flag
Coverage Δ

unittests
66.66% <ø> (+0.63%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

server/utils.go
75.67% <ø> (-0.33%)
:arrow_down:

server/service.go
79.28% <0.00%> (+1.42%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Merging this quickly because getPayload on the main branch is currently broken


endpoints with same URL but different versions throws InvalidAcceptHeader on 0.11.0

I am mounting two versions v1 and v2, and I'm using the :header option. Two of the endpoints have the same URL /api/version.
In 0.10.1 I could select the correct version using the Accept header.
In 0.11.0 I can access v1(which is mounted first), but I can no longer select v2 (mounted second). It throws Grape::Exceptions::InvalidAcceptHeader.

Could you try building a test that reproduces this?

Confirmed with 2 separate apps. When upgrading from 0.9 to 0.11 we receive this error on endpoints with multiple versions.

Interesting, took one my projects where this was failing and stripped out all but the bare minimum and versioning works with both 0.9 and 0.11 using the header. See repo: https://github.com/mattsnyder/grape-versioning-example/tree/master
0.9 is the master and 0.11 is on a branch. Will try to investigate further.

@dblock @jpaas I found it!!
I began adding in behavior from my other apps that experienced this problem into this sample app and found it was when I added a rescue_from :all that the versioning broke.
So if you run the specs on the 0.11.0 branch a test will fail with:
1) GrapeVersioningExample::Endpoints::PingV1 GET /ping(version 1)
     Failure/Error: expect( async.response ).to include("version 1")
       expected "[:error, \"\\nProblem:\\n  Invalid accept header\\nResolution:\\n  API vendor or version not found.\"]" to include "version 1"
     # ./spec/api/ping_spec.rb:7:in `block (4 levels) in <top (required)>'
     # ./spec/api/ping_spec.rb:6:in `block (3 levels) in <top (required)>'

I'm guessing error handling is used for version handling??

Yes I also have a rescue_from :all. Nice find!

@jpaas Also found an interesting bug in https://github.com/mjackson/rack-accept that prohibits the use of _ in the hypermedia type. And underscore is allowed per http://tools.ietf.org/html/rfc6838#page-4 section 4.2.
We're working on a PR for rack-accept to fix it.

Interesting, care to link a bug in rack-accept?

Also, again, we want a test in Grape for this.

@dblock No issue filed with rack-accept yet. @jerrod is currently working on that end of things.
Any insight into the bug that @jpaas and I found with versioning?

Here's the pull request for Rack-Accept -> https://github.com/mjackson/rack-accept/pull/17
FWIW this appears to be separate from the rescue :all issue @mattsnyder unearthed so perhaps we should split this discussion. I'd love to contribute some better specs to grape around header versioning.

@dblock Any movement on this? Or an idea why/where this is occurring?

I don't have any additional insight into this :(

This is caused by the issue demonstrated in https://github.com/intridea/grape/pull/1046
The mere presence of a rescue_all block breaks recognition of additional versions of endpoints.
Comment out the rescue block and the specs pass.  I have been unable to fix, but it appears to be an isse with the way rescues work in the Middleware::Error.call! method.

Very interesting, thanks for the repro @jerrod.

Grape raises Grape::Exceptions::InvalidAcceptHeader with cascade:  true, when a routes does not match the requested version, to tell rack to try the next route.
Isn't rescue_from :all expected to rescue from this exception?

I feel that yes, rescue_from :all should catch this as well.

@romanlehnert @dblock But clearly we'd want the final behavior of having multiple-version-endpoints to work even if a rescue_from :all is present right? rescue_from :all shouldn't be catching expected exceptions, especially exceptions needed for functionality.
We identified the PR which caused this regression, most specifically this line
We are attempting to put together a pull request to raise a different error when specifically the version is invalid, and to pass that through so that Rack can do its thing, but we might not be able to do so while still keeping all the features introduced by the above linked PR...

Adding @croeck.

@dblock thx for adding me. I'll have a look into this during the next couple of days.

@dblock @croeck Thanks for the quick reply!
https://github.com/ruby-grape/grape/pull/1114 has been created as a proposal solution. We tried to find a way to tell "are there any mounted versions to try after this?" but found no way to do so, hence we falled back to just always letting an InvalidVersionHeader go through. Please let us know your thoughts.

OK, first my apologies to all affected by this issue which seems clearly related to my changes :disappointed:
I looked into the issue as well as the proposed changes. Grape handles multiple versions via the cascade=pass header and leaves the actual work to the parent server, e.g. Rack.
With my changes the invalid versions were handled in the application as soon as there was a rescue handler, rather than passing the exceptions on. Therefore the cascading broke.
One idea worth considering might be to build a map of alternative_versions, for instance into the options of a Route. It could be done once during the initialization of the Grape::API and would have to regard the params, path and method to identify valid alternatives.
This information could then be used when raising or processing the exception. If there are (remaining) alternatives, the exception is not rescuable. Otherwise it could be processed by the rescue handlers.


Issue with verbose set to True

In "\fw_algorithms.jl:279"
Fails to convert state.primal to Float64.
juliacall.JuliaError: MethodError: no method matching Float64(::Py)

The fix implemented is a wrapper for the objective function as suggested here : https://github.com/JuliaPy/PythonCall.jl/discussions/469


show-tick on wrong item when using Firefox 3.6.4

When using firefox 3.6.4 on Linux x64 (as per Fedora 14) and on 3.6.28 on Windows (XP)
Goto http://silviomoreto.github.io/bootstrap-select/ and search for "selectpicker show-tick" on the page, in that example, shown below, if you select Mustard, then pull-down the selector, Ketchup is ticked.

Mustard <--- select this
      Ketchup <-- but this is ticked
      Relish

This does not happen on Chome, IE or Firefox 20 in on Windows (XP).
You can download FF 3.6.x from:
http://support.mozilla.org/en-US/kb/install-older-version-of-firefox

Bootstrap doesn't support Firefox 3.6 as of August 2013. Closing.


Apple Messenger app ignores text

When I use ShareScreenshotAndText the text is ignored in Apple Messenger, Signal, and Facebook Messenger, it only sends the screenshot. When testing in Xcode I see the error "Error creating the CFMessagePort needed to communicate with PPT" It seems to work fine in Discord. I am using IOS 14.3 and building in Unity 2020.2 with Xcode 12.3

Some apps like Facebook don't allow text to be shared to be programmatically added, only images and in some cases links
As for Apple Messenger and Signal, I've not used them before, but once I get the chance I'll look into it

Ok Thank you. Also I noticed when I tested facebook messenger with just text, it works. Actually all 3 work with just text. On iOS at least.


deps: update to windows-sys 0.36

This is a separate PR from #337 because some people will probably want a dep on 0.35 and others a dep on 0.36, to avoid duplicate dependencies in their tree.
As such, #337 should be merged first, a version released, then this one and another version released

See also https://github.com/microsoft/windows-rs/issues/1720

tokio/mio just took a dependency on 0.36 so that may be a good version to rally around for the time being.
https://github.com/tokio-rs/mio/pull/1569

I'm just going to merge this one and make a single release. This version churn is quite annoying, and if it continues then I'm considering just switching back to winapi.

Sorry for the annoyance. I'll probably exclude windows-sys from the regular 3-week release cadence that we've been on to allow the current version (0.36) to establish itself.


Support CannonDebugRenderer

Add CannonDebugRenderer function for assisting debug visually.
How to use:
<a-scene physics-world="debug: true">.
debug is false by default

Thanks! By the way, I would put attention on https://github.com/donmccurdy/aframe-extras which has more physics work at this point.


Font Awesome Webfonts folder is not imported when using "academic import --assets"

Welcome to Academic's GitHub repo 👋
We use GitHub only for bug reports and feature requests - it's our project management tool.
For help and questions, please join our community chat or use the forum 🚑.
Also, you can search and browse the extensive Academic and Hugo documentation.
For questions on Blogdown, please reach out to the Blogdown community.
Describe the bug
If one uses the "academic import --assets" feature, only the css files associated with Font Awesome are extracted from the CDN repos.  That may be the way its supposed to work based on the assets.toml folder.   Unfortunately, the "search" and other Font Awesome icons are no timported because they are in the webfonts folder.  One must also fetch the webfonts folder and install the folder at the same level as the "vendor" folder (not within the "vendor folder").   At that point the "search" and related FontAwesome icons will appear.    This may be the desired behavior but there should be some documentation about needing do do this when assets are imported.
To Reproduce
Steps to reproduce the behavior:

install academic theme using git clone
install the academic admin tool
cd to the site directory
academic import --assets
hugo server --watch
browse to localhost:1313
You will not see the search (magnifying glass)
Now copy the webfonts folder over to static/css/webfonts
refresh the browser and you will see the search icon

Expected behavior
I would like to see the webfonts imported as well.
Technical details:

Academic Version: 4.6.3 (I think...)
Hugo Version: v0.60.0/extended linux/amd64
Browser/OS: Chrome 78.0.3904.108 (Official Build) (64-bit)

If applicable, add screenshots to help explain the issue.

This is not the repo for Academic, not the Academic Admin tool. Please search the issues in the relevant repo: https://github.com/sourcethemes/academic-admin


Update MonoGame dependency

Current submodule dependency is out of date. Update xamarin branch of MonoGame to sync up with develop. Fix any conflicts and run through platform-specific test suites to locate any regressions.

Closed in f9fa387


Hosed Bash after setting shell to Zsh. Blank Screen

How can I edit the files inside WSL so I can, for example, set my shell in /etc/passwd back to bash, so I can move along?

bash.exe -c "chsh"


Custom Object Name Links Don't Work

Make sure to read the troubleshooting section before creating an issue
Describe the bug
When clicking an object name via SF inspector, this should take the user to the object manager page for that object in setup. Instead the url object name is undefined and navigates the user to the object manager home page in setup.
This works for standard objects but not custom objects.
To Reproduce
1 - Open an org
2 - Open SF Inspector
3 - Search for a custom object
4 - Click Object Name (URL Link)
Expected behavior
Should take user to custom object manager page for that object.
Screenshots

Desktop (please complete the following information):

OS: MacOS Ventura 13.5.2
Browser: Chrome
Version: 119.0.6045.123

Hi @SimonKaya ,
Could you run the query
SELECT COUNT() FROM EntityDefinition and tell me how many records are returned.
I don't know how are you dev skills but you may check if everything is going fine during the retrieve of the EntityDefinition objects.

If you could share me the results via a video or gif it could be great, if not we can see if we can schedule a short meeting

Thanks for the fast response @tprouvot
I don't use the chrome inspector or debugging tool for my line of work but think I've caught what you were looking for. I seemed to have to refresh the tab to catch the debug however now sure if that makes a difference or not based on your video.
Looks like my count is 2620.
https://github.com/tprouvot/Salesforce-Inspector-reloaded/assets/83288247/a35dda0d-53e0-4ec1-af2f-b9a47634ec2a
If you need more information or example it might be easier to plan a call at.

@SimonKaya
Great thanks for this 👏
If the total number of EntityDefinition is 2620 it means you should iterate only two times in this loop.
Could you check if the object you are looking for is contained in this list ?
To be able to do that you'll have to put a conditional breakpoint by testing the record.QualifiedApiName with your custom object

@tprouvot
The expected record looks to be there from what I can see:

The potential cause is due to the following code where it expects durableId that is undefined:

In my troubleshooting it seems to be caused by durableId not being retrieved at first describe, later the durableId it is retrieved at via this line but the addEntity code does not check if the entity previously had no durableId.
So, a potentially fix is to add this check.

@tprouvot - I have created #228 based on what I understood, hopefully it helps.

Hi @SRosewood ,
Could you check if the fix of @jefersonchaves could resolve your issue ?
To do so, you can create a conditional breakpoint (based on the name of your custom object) on addEntity method from popup.js file and enter this code in the dev console entity.durableId = durableId;
After that check if the link in the popup is functional

Yes, I did the test manually on my browser before the PR. That is the same code and it fixed the issue.

I'm closing the issue since Jeferson provided a fix, please feel free to reopen it if you're issue is still running

Sorry for the delayed response @tprouvot .
Yes has worked thanks @jefersonchaves look forward to the fix making a release soon. 👍


Issue with Semantic-UI React

Hello, and thank you for the boilerplate.
I`m trying to use Semantic-UI React library, but when i import any component to my project and insert it to the code - the component in the browser loose all his styling from semantic and look like regular HTML component.
Why is this happen?

Hi @psycura,
Semantic-UI React package doesn't include the css for semantic ui, you should include the css separately either from a cdn, or you can use semantic-ui-css package to include it in your build pipeline.


[QUESTION] Can [SendTo] be used with an async listener

Question
Can [SendTo] be used with an async listener? I tried to do it like the example below but got this error:
fail: Steeltoe.Messaging.RabbitMQ.Connection.CachingConnectionFactory[0]
      Unexpected invocation of Steeltoe.Messaging.RabbitMQ.Connection.CachingConnectionFactory+DefaultChannelCloseLogger, with Channel shutdown:(null)
fail: Steeltoe.Messaging.RabbitMQ.Listener.DirectMessageListenerContainer[0]
      Consumer SimpleConsumer [queue=batch_worker, consumerTag=amq.ctag-WO-WNouiLn_SIs0eCd7ttQ identity=59817589] canceled - channel closed

Environment:

.NET Core 3.1
Steeltoe Version 3.0.1

Example
[RabbitListener(Binding = "MyQueue")]
[SendTo("AnotherQueue")]
public async Task<string> Listen(MyMessage message)
{
    return await DoSomethingAsync(message);
}

Hi TravisBoatman,
I believe so... I remember thinking about that use case ... let me dig into a bit more and get back to you.
Dave

@dtillman
I was able to figure it out. I needed to add AckMode = "Manual" to the RabbitListener attribute. Like so:
[RabbitListener("myQueue", AckMode = "MANUAL")]


Implement _check

Moved redundant parameter checking code
if (typeof x !== 'number') {     throw new TypeError(${x} is not a number);   }   if (typeof y !== 'number') {     throw new TypeError(${y} is not a number);   }
from the following functions add(), subtract(), multiply(), divide() to a common function, _check().

Hello again, @sdocy! I want to congratulate you again for completing this challenge.
 You'll still have access to your fork, but you won't be able to submit anymore pull requests to this branch.
Don't worry though!
Happy Hacking!


error when building in linux

This may be a simple one. When I build  the CUDA Utility Library in the
"common/" subdirectory on our linux workstation, I got the following error:

In file included from src/cutil.cpp:36:
./../common/inc/cutil.h:36:26: error: cuda_runtime.h: No such file or directory
src/cutil.cpp:48:27: error: builtin_types.h: No such file or directory
make: *** [obj/x86_64/release/cutil.cpp_o] Error 1

Anybody knows how to fix this? Thanks! Do I need to give more details?

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 10 Feb 2011 at 3:36

Do I need to put cudpp_src_1.1.1 at a specific location?

Original comment by<EMAIL_ADDRESS>on 10 Feb 2011 at 3:46

You can also do make cuda-install=<PATH>.  I'm closing this as invalid since
it's not a bug.

Original comment by<EMAIL_ADDRESS>on 1 Jul 2011 at 3:38

Changed state: Invalid
Added labels: Type-Other
Removed labels: Type-Defect

Hi,

I had the same issue when I built cudpp on Linux for the first time. This error
is because make doesn't know the location of cuda_runtime.h and builtin_types.h.

To solve the issue locate the directory - type 'locate cuda_runtime.h' (For me
it was in /export/apps/cuda/include/cuda_runtime.h). Once you have the
directory, take the parent directory name (In my case it is /export/apps/cuda)
and type 'make CUDA_INSTALL_PATH=<parent directory>'. This should solve your
problem.

-Ritesh

Original comment by<EMAIL_ADDRESS>on 14 Feb 2011 at 11:15

Before I built cudpp, I already build nvidia's sdk examples successfully.

Original comment by<EMAIL_ADDRESS>on 10 Feb 2011 at 4:25


Zoom in the map component does not work?

Hi, while developing the page with the map component, I found that the zoom attribute does not work if the second select statement is present. Is this intentional?
thank you

select
    'map'   as component,
    'New Delhi-1' as title,
    28.6139     as latitude,
    11     as zoom,
    77.209      as longitude;

select
    'map' as component,
    'New Delhi-2  ' as title,
    11     as zoom;
select
    28.6139     as latitude,
    77.209      as longitude;

Hello !
I think there is a confusion here. The zoom level is the same on both your screenshots. The attribute 11 as zoom did work in both cases. The only difference is in where the map is centered. In the first example, you provided latitude and longitude as top-level attributes, so the map is centered at the place you specified.

In the second example, you did not provide a latitude and longitude as top level attributes, so the map is still zoomed to zoom level 11, but centered on "lat=48,lng=3" (the default value when no top-level latitude and longitude are provided).
The second query adds pins to the map, and in the second example, if you manually center the map on New Dehli, you'll see that the point was successfully created.
In the end, what you want is probably to specify both top-level and row-level coordinates to create a pin and center the map on it:
select
    'map'   as component,
    'New Delhi-3' as title,
    11     as zoom,
    28.6139     as latitude,
    77.209      as longitude;
select
    28.6139     as latitude,
    77.209      as longitude;

I updated the default behavior to remove the default centering on "lat=48,lng=3" which was confusing. Now when top-level coordinates are omitted, the map centers on the pins that you add to it.
Now this
select
    'map'   as component,
    'New Delhi-1' as title,
    28.6139     as latitude,
    11     as zoom,
    77.209      as longitude;

select
    'map' as component,
    'New Delhi-2  ' as title,
    11     as zoom;
select
    28.6139     as latitude,
    77.209      as longitude;

select
    'map'   as component,
    'New Delhi-3' as title,
    28.6139     as latitude,
    11     as zoom,
    77.209      as longitude;
select
    28.6139     as latitude,
    77.209      as longitude;

select
    'map'   as component,
    'New Delhi-4' as title,
    28.6139     as latitude,
    77.209      as longitude;

displays like that:

awesome, thank you for the explanation


What did you use to create server-side API?

https://api.whereisdi.com/admin/login
This looks like a no-code or pre-packaged solution, possibly? What did you use to stand up this instance? Thank you.

The back-end is beyond the scope of this add-on. If you're interested in doing your own projects you can start with researching headless cms projects.


Finish setup

Changes

Added a .travis.yml file
Turned on Travis CI
Setup docker on Travis

Notes:

We will need to set up a DockerFile

Please Review: @bhowellsofi @Kuroshii

Seems good enough for now, if the travis stuff works. I tested the REST methods manually because it's easy and I wasn't 100% sure how to set up the tests.
A few things that can be improved (what to do next, before we start doing real stuff):

Add direct tests for the two API methods (probably ends up using Mockito)
Probably simplify build to use spring-web and spring-webmvc directly instead of through one of spring's starter project
Have an explicit Dockerfile and docker entrypoint (so we don't rely on travis for deploying later)

Also, I'd like to keep @bhowellsofi out of this project if I can, so personal and work stuff are separated (and my work doesn't claim intellectual ownership of this project); try not to put that account on PRs and hopefully removing SSH keys and whatnot should make it work as I intended.


factors: key value tests, doc comments, and Azure factor

Adds inline docs, tests, and the final KV Azure factor

@rylev I can rebase and add in the generic substitution for the Azure factor too once this is in: https://github.com/fermyon/spin/pull/2665/files/bbdf0aa5cc8dc22159aa6ba3b0c250d1552184c8#diff-c336678c804222fc652b2ea241955544fdc5b24f852c533fa02ec722b063c304


Wrong order of next launches

I have noticed some bugs:
https://api.spacexdata.com/v3/launches/upcoming

that the data of response  contains some launches with wrong date 01.01.19
there are launches in the response not sorted by date descending.
https://api.spacexdata.com/v3/launches/next
one of the launch contains date(upcoming) 01.12.19 but in the response of the next  launch the date is 04.12.19 .
Will wait your reply! Thank You!

See Date field FAQ's in the docs introduction for more info on this.
TLDR The flight number is the only correct way to sort these launches, because the dates can represent partial dates. Because of this, a few more context fields are provided to aid in displaying the dates correctly.


Fancy batching plug

[ ] Handle customization of pipelines
[ ] refactor the various FIXME tags.
[ ] check on whether {:error tuples need to be handled in inner def execute stuff.

I just noticed Apollo doesn't let you specify a separate URL for the batching endpoint, so it would be nice if absinthe_plug recognized incoming JSON arrays as batched queries :)

Is this in a good place, testing wise, where I can build a PR that would re-sit the processing here on top of the refactoring I've done in #53, and still make sure everything works? Or is it non-functional?

It largely works but a lot of the hackyness where will get wildly easier after the changes related to middleware goes thorugh. I'd like to wait until that/s gone


Add newline after caption (#12276)

Fixes #12270.

Pull Request Test Coverage Report for Build 256901836

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.003%) to 38.941%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/notebook/browser/cellViews/textCell.component.ts
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/contrib/notebook/browser/cellViews/textCell.component.ts
1
3.44%

Totals

Change from base Build 256585656:
-0.003%

Covered Lines:
19145

Relevant Lines:
44849

💛  - Coveralls


Add RAG 24 Test Bindings

@lintool it passed now!

Yup, go ahead and merge. (Write good commit msg please.)


Changes Cargo license field to valid SPDX

This change fixes the license field to valid SPDX.
Documentation says that SPDX license expressions support AND and OR operators to combine multiple licenses. Previously multiple licenses could be separated with a /, but that usage is deprecated.
It is currently causing dependency reviews to fail like:

Thanks! I'll add a release bump and push this out.


PVA: Composer allows selecting different bot name resulting with failure to publish

Repro

Create a bot in PVA, start Composer
In name selection, modify the default name
Create your bot
Publish
ERROR: cannot publish due to different bot dialog name expectations
MODIFY: Change dialog name to original --> it works.

Had other errors. This worked today.
Closing.
FYI @GeoffCoxMSFT


Error: req.flash() requires sessions

Hello
I get the error "Error: req.flash() requires sessions" using connect-flash, I use Express 4.17... I do not understand it seems my setup is correct.
Here is my main.js
const Session = require('express-session');
const redis = require('redis');

let RedisStore = require('connect-redis')(Session)
let redisClient = redis.createClient()

app.use(flash());
app.use(Session({
    store: new RedisStore({ client: redisClient }),
    secret: config.server.secret,
    cookie: {maxAge: Date().now + (60 * 1000 * 30) } ,
    resave: false,
    saveUninitialized: false
}));

Here is the code of the route where the error come from
const express = require('express'),
    router = express.Router(),
    passport = require('passport');

router.get('/',
    require('connect-ensure-login').ensureLoggedOut(),
    (req, res) => {
        res.render('register', {
            user : null,
            errors : {
                username : req.flash('username'), <-- error here
                email : req.flash('email')
            }
        });
    });

Redis is started up
Thanks for your help
Laurent

I fixed the problem.
Router after flash.
Please close this issue

@lbois , Please note you may close it by your self.

@lbois , Please note you may close it by your self.


Browser autoreload on file changes

Feel free to contribute if you know how to do this feature.
ng serve has it already, but I run the server using npm start

Done in https://github.com/DavideViolante/Angular2-Full-Stack/commit/8c8701837104ebeae23966f3b138919d4b1520ef


How to define a simple LSTM layer?

Click to expand!
Issue Type
Support
Source
source
Tensorflow Version
tf 2.10.0
Custom Code
No
OS Platform and Distribution
ubuntu 20.04
Mobile device
none
Python version
3.8
Bazel version
5.1.1
GCC/Compiler version
9.3.0
CUDA/cuDNN version
none
GPU model and memory
none
Current Behaviour?
I want to build a single LSTM-layer model, but got unidirectional-LSTM instead.

Standalone code to reproduce the issue
#define.py
inputs = layers.Input(shape=(5, 4),batch_size=3)
lstm = layers.LSTM(units=3,return_sequences=True,return_state=True,time_major=True)
out,_,_ = lstm(inputs)
print(out.shape)
model = keras.models.Model(inputs = inputs, outputs = out)
model.save('model')

#tflite_convert
tflite_convert --output_file /mnt/e/lstm.tflite --saved_model_dir ./model/

Relevant log output
No response

Hi @njuhang !
Could you check these link on Fused LSTM  conversion for converting Bi-directional LSTM to TFlLte model and let us know.
Thank you!

I have checked these links. But they can't help to solve my problem.
There is no bi-directional LSTM in my question, I just want to build a simple LSTM layer. But got Unidirectional LSTM instead, same as this link. As you known, they are two different layers in tensorflow lite.

Hello, Is there any help?

I have another question, in unidirectional sequence lstm, there is a input called "model/lstm/zeros". But it says "Tensor data is empty." in its description. So I don't know how this input get zero data.

BTW, I can't get a LSTM model from pytorch->onnx->tflite.

@njuhang You can convert the model from Jaxx. Also I am not sure what a simple LSTM layer is in your case. You can also build a custom layer. Can you please point me out to what LSTM layer are you looking into. Thanks!

In tensorflow/lite/kernels/lstm.cc, there is a full LSTM, it's different from UnidirectionalSequenceLSTM in tensorflow/lite/kernels/unidirectional_sequence_lstm.cc. How to build this type of layer?

@njuhang I don't understand what type of LSTM layer you need. Can you point me to pytorch where they have this LSTM layer. Thanks!

Emm, there is nothing to do with pytorch.
I need such a LSTM layer, when executing inference, it use functions defined in tensorflow/lite/kernels/lstm.cc. But currently I can only get Unidirectional LSTM layer, which use functions defined in tensorflow/lite/kernels/unidirectional_sequence_lstm.cc. My code has been attached above.
My question is how to get a .tflite model, with such a LSTM layer.

Hello? Is there any progress?

@njuhang Please post this issue in tensorflow forum for any help. This is primarily for bug/performance, bulild/install issues. Thanks!


Show inline table dependencies links in the DAG

Currently there is no dependency shown in the DAG visualisation

I think we plan to get rid of inline tables in v2, closing this issue.


Explain the role of Docker in Monterey

I expected a lot more from this attached document - information that would help our team understand what are my reasons for being excited with Docker; now, I will have to write all this myself 😏. This attached article is completely generic and basic, describing the evolution that led to Docker, but since it is already here, I do not want to delete it.
Codeship_Why_Containers_and_Docker_are_the_Future.pdf

Codeship_Why_Containers_and_Docker_are_the_Future.pdf

Case 1 - Deployment

Image 3

... to be continued


Add spherical rect query.

Correct, there is no assumption about the cell union.

Thanks for your review!


about datasets

Hi jonasricker!
I wonder if the generated face dataset is included in the dataset you provided.

Hi hsywatchingu,
currently our dataset only contains LSUN bedroom images. In a future version we might extend if to facial images.
The checkpoints used to create the images are provided in the repositories of LDM and P2.

The additional datasets are now also available at Zenodo.


TypeError when compiling

Hello tfyoung, many thanks for programming this esphome sensor integration for Junctek battery monitor. A I try to compile it it gives me following error:
INFO Reading configuration /config/esphome/yucon-battery-monitor.yaml...
ERROR Unable to load component junctek_kgf.sensor:
Traceback (most recent call last):
File "/esphome/esphome/loader.py", line 162, in _lookup_module
module = importlib.import_module(f"esphome.components.{domain}")
File "/usr/lib/python3.9/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "", line 790, in exec_module
File "", line 228, in _call_with_frames_removed
File "/config/esphome/.esphome/external_components/9c6cd628/components/junctek_kgf/sensor.py", line 54, in
cv.Optional(CONF_VOLTAGE): sensor.sensor_schema(
TypeError: sensor_schema() takes from 0 to 1 positional arguments but 5 were given
Failed config
sensor.junctek_kgf: [source /config/esphome/yucon-battery-monitor.yaml:18]
Platform not found: 'sensor.junctek_kgf'.
platform: junctek_kgf
address: 1
voltage:
name: Battery Voltage
current:
name: Battery Current
battery_level:
name: Battery Level
temperature:
name: Battery Temperature
the config I use is the one provides as example:
external_components:

source: github://tfyoung/esphome-junctek_kgf

uart:
tx_pin: 1
rx_pin: 3
baud_rate: 115200
sensor:

platform: junctek_kgf
address: 1
voltage:
name: "Battery Voltage"
current:
name: "Battery Current"
battery_level:
name: "Battery Level"
temperature:
name: "Battery Temperature"

Are you able to help me as my programming skills are not so good to fix this issue.
Many thanks
Christoph

Hi, I changed sensor.py to be working now. Just change CONFIG_SCHEMA to this
CONFIG_SCHEMA = cv.All(
cv.Schema(
{
cv.GenerateID(): cv.declare_id(JuncTekKGF),
cv.Optional(CONF_ADDRESS, default=1): cv.int_range(1, 255),
cv.Optional(CONF_VOLTAGE): sensor.sensor_schema(
unit_of_measurement=UNIT_VOLT,
icon=ICON_FLASH,
accuracy_decimals=1,
device_class=DEVICE_CLASS_VOLTAGE,
state_class=STATE_CLASS_MEASUREMENT,
),
cv.Optional(CONF_CURRENT): sensor.sensor_schema(
unit_of_measurement=UNIT_AMPERE,
icon="mdi:current-dc",
accuracy_decimals=1,
device_class=DEVICE_CLASS_CURRENT,
state_class=STATE_CLASS_MEASUREMENT,
),
cv.Optional(CONF_BATTERY_LEVEL): sensor.sensor_schema(
unit_of_measurement=UNIT_PERCENT,
icon=ICON_PERCENT,
accuracy_decimals=1,
device_class=DEVICE_CLASS_BATTERY,
state_class=STATE_CLASS_MEASUREMENT,
),
cv.Optional(CONF_TEMPERATURE): sensor.sensor_schema(
unit_of_measurement=UNIT_CELSIUS,
icon=ICON_THERMOMETER,
accuracy_decimals=0,
device_class=DEVICE_CLASS_TEMPERATURE,
state_class=STATE_CLASS_MEASUREMENT,
),
}
).extend(uart.UART_DEVICE_SCHEMA)
)

Thanks for reporting this. I've been meaning to fix this having had the issue come up on another integration i'm working on (I haven't updtated my junktek for a while to hit this for this module). I'll push a fix for it soon, though if you wanted to take your changes and make a pull request for them, I'd gladly merge.

This should be fixed now with the most recent commit to main

Many thanks tfyoung


sharp is crashing on Image conversion

These are the error
** (sharp:16896): CRITICAL **: gsf_outfile_new_child_full: assertion 'outfile != NULL' failed
** (sharp:16896): CRITICAL **: gsf_outfile_new_child_full: assertion 'outfile != NULL' failed
** (sharp:16896): CRITICAL **: gsf_output_write: assertion 'output != NULL' failed
** (sharp:16896): CRITICAL **: gsf_output_close: assertion 'GSF_IS_OUTPUT (output)' failed
** (sharp:16896): CRITICAL **: gsf_output_set_error: assertion 'GSF_IS_OUTPUT (output)' failed

Hello, please can you provide more information such as a code sample and image(s) that reproduce this error.

Hi ,Thank you for the quick response.
Here is the sample code and image
Sample Code
These is the Node js API used for image conversion
var ImagesConversion = function (data) {
return new Promise((resolve, reject) => {
var inputPath = data.inputPath;
var fileName = data.fileName;
var outputPath = data.outputPath + fileName + ".dz";
var fileId = data.fileId;
var pixels = 0;
    common.getPixels(inputPath).then(function (numberOfPixels) {
        sharp(inputPath)
            .png()
            .limitInputPixels(numberOfPixels + 1)
            .tile({size: 256})
            .toFile(outputPath, function (err, info) {
                if (err)
                    return reject(err);

                return resolve("1");
            });
    }, function (err) {
        return reject(err);
    });
});

};
Sample image URL used for the conversion
Please download the image from here
https://lumos-air.s3.eu-west-1.amazonaws.com/Image_001--Image%40%40--%40%40Blue.tif?X-Amz-Expires=265622&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIZGBI4566U6V4M4A/20180921/eu-west-1/s3/aws4_request&X-Amz-Date=20180921T044258Z&X-Amz-SignedHeaders=host&X-Amz-Signature=31b4aeb50eee37e9552242feab4cdf54f3bd7bf6f274d28c1a756b5e85219499
And Also Where i can see the sharp log files when i get exception or error?

Thanks, this looks a bit like https://github.com/libvips/libvips/issues/242 - please make sure the directory paths and permissions are correct and don't already exist.

We have taken care of the directory paths and permissions and also we are giving the extension for the input image in code (.png()).
These API is working fine for other image but only for the few image(the above reference image ) sharp is crashing.
How to Resolve this issue ?
Where can i see the sharp log files ??

Thanks, sadly I'm unable to reproduce this. Which version of sharp/libvips is this with and on what platform? Are you able to provide a standalone code example that exhibits this behaviour e.g. without implicit dependencies such as data and common.

"Where can i see the sharp log files ??"

If there's an error during processing, this is returned as the first parameter to the callback function provided to toFile, as your example code is already handling.

Which version of sharp/libvips is this with and on what platform?
"sharp": "^0.20.0" and Windows platform ,OS windows 10

Standalone code
var sharp = require('sharp');
var createDeepZoomImages = function (data) {
    return new Promise((resolve, reject) => {
        var inputPath = "E:/EDrive/ImageScan_001.tif";
        var fileName = "ImageScan_001";
        var outputPath = "E:/EDrive/Scan/ImageScan_001.dz"
        var pixels = 0;
        getPixels(inputPath).then(function (numberOfPixels) {
            sharp(inputPath)
                .png()
                .limitInputPixels(numberOfPixels + 1)
                .tile({size: 256})
                .toFile(outputPath, function (err, info) {
                    if (err)
                        return reject(err);

                    return resolve("1");
                });
        }, function (err) {
            return reject(err);
        });
    });
};

//This function will return the numberOfPixels(1920001)
var getPixels = function (inputPath) {
    return new Promise((res, rej) => {
        sharp(inputPath)
            .metadata()
            .then(function (metadata) {
                var pixels = (metadata.width * metadata.height) + 1;
                return res(pixels);
            }, function (err) {
                return rej(err);
            });
    });
};

Thanks, I'm unable to reproduce this.
Are you able to reproduce the same problem on a non-Windows machine?
Are multiple images being processed concurrently? Please note the documented tile warning: "multiple sharp instances concurrently producing tile output can expose a possible race condition in some versions of libgsf."

Feel free to re-open with more information if the problem persists.


Add option to choose what kind of method should be generated

Fixes https://github.com/yeoman/generator-jquery/issues/26

lgtm.


[Spike] [DO NOT MERGE] ON Week 2024

Summary
As part of my work for ON Week 2024, I added a Project Management feature to the Recorder. This interacts with the Synthetics CLI init feature to generate a project. The user can then access the normal Recorder functionality to record a script and save it to their project.
From the Management UI, the user can push a project to Kibana as they would do via CLI. They can also delete projects, and see the status of any monitors that are running from their projects in Kibana.
Implementation details
Adds a significant amount of backend functionality to the electron server to persist the necessary information to push to the Kibana server, interact with Synthetcs, and manage files.

Not how I would do it but good job so far

Not how I would do it

Feel free to contribute to the Recorder any time! 😃


[Improvement] Introduce Maven Modernizer plugin to Pulsar

As discussed in dev mail list: https://lists.apache.org/thread.html/re99b342765b2cfb26f3da0eedfb78e7312de158bf6c7ed0085d32781%40<dev.pulsar.apache.org>
We want to introduce Maven Modernizer plugin to Pulsar.
This plugin helps detecting uses of legacy APIs which can be replaced with modern APIs that are often more performant, safer, and idiomatic than the legacy equivalents.
We'll break down the task by sub-module so anyone interested can work on a sub-module independently.
We'll start with checking violation with Java 8. Acceptance criteria will be passing existing CI.
Help are welcome, please reply with module you want to work on so there won't be conflict. And if change for single module is too big, please break down pr to reasonable size, say < 100 change so it's easier to review.
Please don't close the master issue via comment.
<plugin>
    <groupId>org.gaul</groupId>
    <artifactId>modernizer-maven-plugin</artifactId>
    <configuration>
        <failOnViolations>true</failOnViolations>
        <javaVersion>8</javaVersion>
    </configuration>
    <executions>
        <execution>
            <id>modernizer</id>
            <goals>
                <goal>modernizer</goal>
            </goals>
        </execution>
    </executions>
</plugin>

[ ] managed-ledger
[x] pulsar-common
[x] pulsar-broker-common
[ ] pulsar-broker-auth-athenz
[ ] pulsar-broker-auth-sasl
[ ] pulsar-broker
[x] pulsar-client-api
[x] pulsar-client
[x] pulsar-client-admin-api
[x] pulsar-client-admin
[x] pulsar-client-tools
[x] pulsar-client-tools-test
[ ] pulsar-client-auth-athenz
[ ] pulsar-client-auth-sasl
[ ] pulsar-websocket
[x] pulsar-proxy
[ ] pulsar-discovery-service
[ ] pulsar-zookeeper-utils
[ ] pulsar-testclient
[ ] pulsar-config-validation
[ ] pulsar-transaction
[ ] pulsar-functions
[ ] pulsar-metadata
[ ] pulsar-package-management

Please tag with help-wanted
I'll work on pulsar-common

i will work on pulsar-transaction and pulsar-broker @MarvinCai

@MarvinCai you can create a subtask for pulsar-broker for @youzipi, ask them to comment there, and assign.
Although, pulsar-broker is a heated code path various patches modify its code. @youzipi you may find a way to separate it into more fine-grained patches to avoid conflict or contact with domain experts like @codelipenghui to cooperate on that module. I assume that applying this plugin is like a code style reformat among the whole module.

@tisonkun seems we've enabled the plugin for all modules, can close this issue?

@MarvinCai No. There're still several dangling issues:

https://github.com/apache/pulsar/pull/17275
https://github.com/apache/pulsar/issues/16991

Concretely, we turn on the plugin for pulsar-broker, but ignore several patterns now for gradually handling them. I think @youzipi is actively working on this. We can help with reviewing.

@MarvinCai @codelipenghui @nodece should be closed by #17968.


Responsive page

Global

Add global variable to handle black text color.

Homepage

Main content located in the center of the page uses tables and is not responsive.  When the window becomes smaller a horizontal scroll bar is rendered.
Fix:
Utilize Flexbox and Grid layouts
Create custom media queries to handle element colors
Blog/News/Event cards located on the right side will disappear and become inaccessible between 639px - 1200px.

Implement CSS to render Blog/News/Event cards below main content when screen is smaller than 1200px.
Write custom logic to render upcoming events. (unable to import event_card directly from source component.  When doing this it will break events_card in right side cards.

Update buttons utilize UI kit button classes.

Expression Page

Divide different components into cards
Update buttons with Ui kit button classes
Update inputs with Ui kit input classes
Incorporate mobile responsiveness using UI kit classes

SoySNP50K Page

Incorporate mobile responsiveness using UI kit classes.

Center bottom table on smaller devices.
Add horizontal scrollbar to table for smaller screens.

Go Analysis Page

Incorporate mobile responsiveness using UI kit classes.

Element will be centered on smaller screens.

Update buttons with Ui kit button classes
Create functions to handle disabling buttons if text area is empty or no file has been selected in upload text component.

Gene-name translator

Add horizontal scrollbar to table for smaller screens

Cmap-js: Genetic Map Viewer

Fix all accordion being open instead of just the first one under "…new CMap-js viewer" portion.
Add horizontal scroll to table for smaller scree

I agree - very nice work, @ChenProm-USDA.
Some minor things that I'd like to see if they could be addressed easily -
It looks like the bottom row of buttons didn't make it into this PR:
[Other Legumes] [Expression Resources] [Help & Contact]
The GO buttons: good that those are now standard uikit, but the dark green stands out visually as the most prominent feature in the page. Is it possible to set a lighter color for the buttons (home page and expression)?

@StevenCannon-USDA  Remember the 508 compliance issue is to have as high a contrast as possible in the text.  If you lighten the "Green" you also reduce the contrast.  If you make the buttons lighter, then we may want to change the text color to Black to get the contrast.  Currently, the "Active" buttons have a dark green background and the inactive buttons have a white background.  This is to make it clear which buttons are active.  Lightening the green would reduce the contrast between active and inactive  buttons.  Yes, the eye is drawn to the green buttons, but that is on purpose.  If I had reduced vision I would probably like the higher contrast between buttons.  I don't think we should change it.

@StevenCannon-USDA
Fixed: "It looks like the bottom row of buttons didn't make it into this PR:
[Other Legumes] [Expression Resources] [Help & Contact]"


Given project for action get_version_number

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
I have multiple projects in one folder. I trying to get version from one project. When I use action get_version_number it ignores project I put into arguments.
In documentation https://docs.fastlane.tools/actions/get_version_number/ is for key xcodeproj this definition:

Path to the main Xcode project to read version number from, optional. By default will use the first Xcode project found within the project root directory

I looked into source code https://github.com/fastlane/fastlane/blob/master/fastlane/lib/fastlane/actions/get_version_number.rb and I found, that action get_version_number always selects first project. This is wrong.
Command executed
get_version_number

activity

activity

activity

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest fastlane version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
Friendly reminder: contributions are always welcome! Check out CONTRIBUTING.md for more information on how to help with fastlane and feel free to tackle this issue yourself 💪

What a great and, of course, well written post. It`s so useful…

activity

activity

I investigated this a bit and it seems to be a misunderstanding of the documentation slash misleading behavior of the actual code. The documentation says:

Path to the main Xcode project to read version number from

The way it is consumed in the code is as if this path contained the containing directory of the xcodeproj file. So it will search in this given directory, looking for the first xcodeproj that it can find 😅  I agree this is totally misleading, though. We should not only update how the logic works but also the docs.
I'll add this to my to-do list 👍

I've opened a PR fixing this behavior @raptorxcz @tedgonzalez - https://github.com/fastlane/fastlane/pull/17574
Would you mind help testing it? 😃

Can we possibly accelerate this? This fix is definitely of interest to us folks with multi-project workspaces 😃

Can we possibly accelerate this? This fix is definitely of interest to us folks with multi-project workspaces 😃

@CSHawley you may test the PR #17574

@CSHawley you may test the PR #17574


[zydis] update to v4.0

Fixes #28301

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Shouldn’t this be ready? Would be great having a more recent Zydis version in vcpkg

Duplicate of #31909


Overlapping text

Example on some resolutions with large titles.

Ref #650 - This is still happening

Closing as fixed in PR #776


feat(frontend): authorization frontend

🍰 Pullrequest

Based on #403
Issues

fixes #370

Todo

[X] None

I think we should require correct values for this - maybe we want so set a default here, since the frontend only makes sense to operate when authentication is possible. Furthermore the default values for
 AUTHORITY: (import.meta.env.PUBLIC_ENV__AUTH__AUTHORITY ?? '') as string,
  AUTHORITY_SIGNUP_URI: (import.meta.env.PUBLIC_ENV__AUTH__AUTHORITY_SIGNUP_URI ?? '') as string,

make no sense anymore, since the presenter hides its buttons, not the frontend
Furthermore AUTHORITY_SIGNUP_URI is not needed in the frontend.

Fixed - I kept the AUTHORITY_SIGNUP_URI since that would cripple the authService if not present.

Looks like a build problem to me. The guard does not trigger at all all when building for production. If I remove all the code and just leave a console, no output is seen anywhere


Use slim container image of Solr

this image is enough for our tests and it divides size by 2
Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

/component-test solr


hurl: 4.1.0 -> 4.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Orange-OpenSource/hurl/releases.
meta.description for hurl is: Command line tool that performs HTTP requests defined in a simple plain text format
meta.homepage for hurl is: https://hurl.dev/
meta.changelog for hurl is: https://github.com/Orange-OpenSource/hurl/blob/4.2.0/CHANGELOG.md
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.2.0 with grep in /nix/store/g943zwjlqjyamfbkxal7n38gzy9igb9m-hurl-4.2.0
found 4.2.0 in filename of file in /nix/store/g943zwjlqjyamfbkxal7n38gzy9igb9m-hurl-4.2.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
hurl
vimPlugins.hurl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/g943zwjlqjyamfbkxal7n38gzy9igb9m-hurl-4.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A hurl https://github.com/r-ryantm/nixpkgs/archive/f11c3d08be7cf5a13e2ca8a525443c1d5a36621d.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/f11c3d08be7cf5a13e2ca8a525443c1d5a36621d#hurl

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/g943zwjlqjyamfbkxal7n38gzy9igb9m-hurl-4.2.0
ls -la /nix/store/g943zwjlqjyamfbkxal7n38gzy9igb9m-hurl-4.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

hurl
vimPlugins.hurl

Maintainer pings
cc @eonpatapon @figsoda for testing.

Result of nixpkgs-review pr 282615 run on x86_64-linux 1

2 packages built:

hurl
vimPlugins.hurl


request mentorship to fix a small bug/issue

Question: Have you read the mentoring guide? Please only submit this request for general mentoring (not GSoC).
Answer: Yes I have read the mentoring guide.
What is your background? Any experience with Go, Kubernetes, React, Typescript, etc.?
Answer: I have previous experience in Golang, Kubernetes ; but not react.
Is there any particular issue you'd like to work on?
Answer: I would like to work with issue #9224
Do you want to work with your mentor async (i.e. by Slack messaging or issue comments), or sync (e.g. using Zoom video)? Please include your time-zone if you want to work sync.
Answer: (async) (timezone/ IST)

@sarabala1979 Would you like to mentor this given that you already commented in the mentioned issue?


Update/progress button multiple clicks (and other bits)

💬 Description
A few changes related to this PR https://github.com/gathercontent/app/pull/2294

Stops the progress button callback from being called multiple times (when a user goes mental and clicks as quick as they can, previously we'd send of a bucket load of requests)
Updates top-bar cell styling
Removes incorrect docs
Calls event.preventDefault() on ConfirmationModal submit

✅ Checklist

[ ] Tests written
[ ] Browser tested
[ ] Added to documentation
[ ] Added to storybook

 UI For Approvals Shareable view


Crash in smoke tests

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=301920&view=artifacts&pathAsName=false&type=publishedArtifacts
39a0c57d-2f33-4289-9676-9b8df3e9f8fc.dmp.zip

termination from spdlog module due to exception triggered inside a noexcept block
Operating system: Mac OS X
                  13.7.0 22H123
CPU: amd64
     family 6 model 58 stepping 10
     4 CPUs

GPU: UNKNOWN

Crash reason:  0x00000000 / 0x00000000
Crash address: 0x7ff8164ce196
Process uptime: 2 seconds

Thread 0 (crashed)
 0  libsystem_kernel.dylib!__pthread_kill + 0xa
    rax = 0x0000000000000000   rdx = 0x0000000000000000
    rcx = 0x00007ff7b2561278   rbx = 0x00007ff859be2800
    rsi = 0x0000000000000006   rdi = 0x0000000000000103
    rbp = 0x00007ff7b25612a0   rsp = 0x00007ff7b2561278
     r8 = 0x00007ff7b2561140    r9 = 0x00007ff8164c36fe
    r10 = 0x0000000000000000   r11 = 0x0000000000000246
    r12 = 0x0000000000000103   r13 = 0x0000003000000008
    r14 = 0x0000000000000006   r15 = 0x0000000000000016
    rip = 0x00007ff8164ce196
    Found by: given as instruction pointer in context
 1  libsystem_pthread.dylib!pthread_kill + 0x107
    rbx = 0x00007ff859be2800   rbp = 0x00007ff7b25612a0
    rsp = 0x00007ff7b2561280   r12 = 0x0000000000000103
    r13 = 0x0000003000000008   r14 = 0x0000000000000006
    r15 = 0x0000000000000016   rip = 0x00007ff816505ee6
    Found by: call frame info
 2  libsystem_c.dylib!abort + 0x7b
    rbx = 0x0000000000000003   rbp = 0x00007ff7b25612e0
    rsp = 0x00007ff7b25612b0   r12 = 0x00007ff857e689b0
    r13 = 0x0000003000000008   r14 = 0x00007ff859be2800
    r15 = 0x00007ff7b25612b8   rip = 0x00007ff81642cb45
    Found by: call frame info
 3  libc++abi.dylib!abort_message + 0xf1
    rbx = 0x00007ff7b25613c0   rbp = 0x00007ff7b25613f0
    rsp = 0x00007ff7b25612f0   r12 = 0x00007ff857e689b0
    r13 = 0x0000003000000008   r14 = 0x00007ff8164c36d0
    r15 = 0x00007ff7b25612f0   rip = 0x00007ff8164c0282
    Found by: call frame info
 4  libc++abi.dylib!demangling_terminate_handler() + 0xf1
    rbx = 0x00007ff8164b2486   rbp = 0x00007ff7b2561430
    rsp = 0x00007ff7b2561400   r12 = 0x0000000114d3c800
    r13 = 0x0000012c00138080   r14 = 0x00007ff8164c372a
    r15 = 0x0000012c000f1700   rip = 0x00007ff8164b23e1
    Found by: call frame info
 5  libobjc.A.dylib!_objc_terminate() + 0x68
    rbx = 0x0000000000000001   rbp = 0x00007ff7b2561450
    rsp = 0x00007ff7b2561440   r12 = 0x0000000000000000
    r13 = 0x0000012c00138080   r14 = 0x00007ff7b25614d0
    r15 = 0x0000000000000000   rip = 0x00007ff8161857d2
    Found by: call frame info
 6  libc++abi.dylib!std::__terminate(void (*)()) + 0x6
    rbx = 0x0000012c000125a0   rbp = 0x00007ff7b2561460
    rsp = 0x00007ff7b2561460   r12 = 0x0000000000000000
    r13 = 0x0000012c00138080   r14 = 0x00007ff7b25614d0
    r15 = 0x0000000000000000   rip = 0x00007ff8164bf6db
    Found by: call frame info
 7  libc++abi.dylib!std::terminate() + 0x36
    rbx = 0x0000012c000125a0   rbp = 0x00007ff7b25614c0
    rsp = 0x00007ff7b2561470   r12 = 0x0000000000000000
    r13 = 0x0000012c00138080   r14 = 0x00007ff7b25614d0
    r15 = 0x0000000000000000   rip = 0x00007ff8164bf696
    Found by: call frame info
 8  spdlog.node!__clang_call_terminate + 0xb
    rbx = 0x0000012c00012600   rbp = 0x00007ff7b25614c0
    rsp = 0x00007ff7b2561480   r12 = 0x0000000000000000
    r13 = 0x0000012c00138080   r14 = 0x00007ff7b25614d0
    r15 = 0x0000000000000000   rip = 0x0000000114cf873b
    Found by: call frame info
 9  spdlog.node!Napi::details::TemplatedInstanceVoidCallback<Logger, &Logger::Log<(spdlog::level::level_enum)2>(Napi::CallbackInfo const&)>(napi_env__*, napi_callback_info__*) + 0x67
    rbx = 0x0000012c00012600   rbp = 0x00007ff7b25614c0
    rsp = 0x00007ff7b2561490   r12 = 0x0000000000000000
    r13 = 0x0000012c00138080   r14 = 0x00007ff7b25614d0
    r15 = 0x0000000000000000   rip = 0x0000000114d2a707
    Found by: call frame info
10  Electron Framework!v8impl::(anonymous namespace)::FunctionCallbackWrapper::Invoke(v8::FunctionCallbackInfo<v8::Value> const&) [js_native_api_v8.cc : 503 + 0x8]
    rbx = 0x0000012c013e1500   rbp = 0x00007ff7b2561500
    rsp = 0x00007ff7b25614d0   r12 = 0x0000000000000000
    r13 = 0x0000012c00138080   r14 = 0x00007ff7b25614d0
    r15 = 0x0000000000000000   rip = 0x000000011f3d6fe1
    Found by: call frame info
11  Electron Framework!Builtins_CallApiCallbackGeneric + 0x9e
    rbp = 0x00007ff7b2561540   rsp = 0x00007ff7b2561510
    rip = 0x00000001176ad8de
    Found by: previous frame's frame pointer
12  0x177798803
    rbp = 0x00007ff7b25615d0   rsp = 0x00007ff7b2561550
    rip = 0x0000000177798803
    Found by: previous frame's frame pointer
13  0x1777989dc
    rbp = 0x00007ff7b2561650   rsp = 0x00007ff7b25615e0
    rip = 0x00000001777989dc
    Found by: previous frame's frame pointer
14  0x177798e67
    rbp = 0x00007ff7b2561708   rsp = 0x00007ff7b2561660
    rip = 0x0000000177798e67
    Found by: previous frame's frame pointer
15  0x17768fb62
    rbp = 0x00007ff7b25617e0   rsp = 0x00007ff7b2561718
    rip = 0x000000017768fb62
    Found by: previous frame's frame pointer
16  Electron Framework!Builtins_InterpreterEntryTrampoline + 0x127
    rbp = 0x00007ff7b2561868   rsp = 0x00007ff7b25617f0
    rip = 0x00000001176ab627
    Found by: previous frame's frame pointer
17  Electron Framework!Builtins_InterpreterEntryTrampoline + 0x127
    rbp = 0x00007ff7b25618d0   rsp = 0x00007ff7b2561878
    rip = 0x00000001176ab627
    Found by: previous frame's frame pointer
18  Electron Framework!Builtins_PromiseFulfillReactionJob + 0x2e
    rbp = 0x00007ff7b2561908   rsp = 0x00007ff7b25618e0
    rip = 0x00000001177cb22e
    Found by: previous frame's frame pointer
19  Electron Framework!Builtins_RunMicrotasks + 0x2bd
    rbp = 0x00007ff7b2561978   rsp = 0x00007ff7b2561918
    rip = 0x00000001176db0bd
    Found by: previous frame's frame pointer
20  Electron Framework!Builtins_JSRunMicrotasksEntry + 0x9f
    rbp = 0x00007ff7b25619f0   rsp = 0x00007ff7b2561988
    rip = 0x00000001176a8fdf
    Found by: previous frame's frame pointer
21  Electron Framework!v8::internal::(anonymous namespace)::Invoke(v8::internal::Isolate*, v8::internal::(anonymous namespace)::InvokeParams const&) [simulator.h : 187 + 0x2]
    rbp = 0x00007ff7b2561b30   rsp = 0x00007ff7b2561a00
    rip = 0x00000001190d3bea
    Found by: previous frame's frame pointer
22  Electron Framework!v8::internal::(anonymous namespace)::InvokeWithTryCatch(v8::internal::Isolate*, v8::internal::(anonymous namespace)::InvokeParams const&) [execution.cc : 477 + 0xb]
    rbp = 0x00007ff7b2561b90   rsp = 0x00007ff7b2561b40
    rip = 0x00000001190d4a3a
    Found by: previous frame's frame pointer
23  Electron Framework!v8::internal::Execution::TryRunMicrotasks(v8::internal::Isolate*, v8::internal::MicrotaskQueue*) [execution.cc : 578 + 0x8]
    rbp = 0x00007ff7b2561bf0   rsp = 0x00007ff7b2561ba0
    rip = 0x00000001190d4bb5
    Found by: previous frame's frame pointer
24  Electron Framework!v8::internal::MicrotaskQueue::PerformCheckpoint(v8::Isolate*) [microtask-queue.cc : 185 + 0xb]
    rbp = 0x00007ff7b2561ca0   rsp = 0x00007ff7b2561c00
    rip = 0x0000000119106dc1
    Found by: previous frame's frame pointer
25  Electron Framework!node::InternalCallbackScope::Close() [callback.cc : 137 + 0xc]
    rbp = 0x00007ff7b2561cf0   rsp = 0x00007ff7b2561cb0
    rip = 0x000000011f358f43
    Found by: previous frame's frame pointer
26  Electron Framework!node::CallbackScope::~CallbackScope() [callback.cc : 92 + 0x8]
    rbp = 0x00007ff7b2561d10   rsp = 0x00007ff7b2561d00
    rip = 0x000000011f358c01
    Found by: previous frame's frame pointer
27  Electron Framework!gin_helper::internal::CallMethodWithArgs(v8::Isolate*, v8::Local<v8::Object>, char const*, std::__Cr::vector<v8::Local<v8::Value>, std::__Cr::allocator<v8::Local<v8::Value>>>*) [event_emitter_caller.cc : 42 + 0x5]
    rbp = 0x00007ff7b2561dc0   rsp = 0x00007ff7b2561d20
    rip = 0x0000000117c2d6df
    Found by: previous frame's frame pointer
28  Electron Framework!v8::Local<v8::Value> gin_helper::EmitEvent<char [6]>(v8::Isolate*, v8::Local<v8::Object>, char const (&) [6]) [event_emitter_caller.h : 53 + 0x12]
    rbp = 0x00007ff7b2561e30   rsp = 0x00007ff7b2561dd0
    rip = 0x0000000117b68fb6
    Found by: previous frame's frame pointer
29  Electron Framework!electron::MessagePort::Close() [message_port.cc : 159 + 0x12]
    rbp = 0x00007ff7b2561ee0   rsp = 0x00007ff7b2561e40
    rip = 0x0000000117b68d1d
    Found by: previous frame's frame pointer


Adding Feign to the library for use without Spring

Hello. I am coding a reactive API library for the exchange, I want to find out if it is possible to use a reactive Feign as a dependency. I'm not sure that it will work without Spring, but I would like this version of my reactive client to be able to be used both in Spring applications and in regular Java applications.

Is it possible?
Are there any features that I should take into account?
And is reactive feign compatible with java8?

thanks for the answer!

Hi. It's possible.
You should use feign-reactor-jetty or feign-reactor-java11

@kptfh Thank you! If I will add this dependency to my library, can I use it for example with spring + Netty?  I want to make a universal solution so that you can use the client both with spring and without spring.

feign-reactor-web based on Spring WebClient (based on Netty)

So there are 3 implementations of reactive feign. You may choose one that fits you best

Hmm. In the case of creating a library, I probably will have to write several implementations - for spring and for applications that do not use spring? Sorry, this is my first library xD

correct.
create core module and separate module for each implementation
I use this approach in reactive feign :)


InteractiveViewer onInteractionUpdate does not get called when panEnabled is false.

Steps to Reproduce

Add InteractiveViewer widget
Use the onInteractionUpdate property
Set the panEnabled property to false
Trigger the GestureDetector by using a pan gesture
See that onInteractionUpdate does not get called

Expected results:
Make onInteractionUpdate get called, even when panEnabled is false.
Actual results:
The onInteractionUpdate property does not get called, although this case is described in the onInteractionUpdate docs:
https://github.com/flutter/flutter/blob/c5a4b4029c0798f37c4a39b479d7cb75daa7b05c/packages/flutter/lib/src/widgets/interactive_viewer.dart#L259-L260

Details/Findings
In the InteractiveViewer source, I found this line:
https://github.com/flutter/flutter/blob/c5a4b4029c0798f37c4a39b479d7cb75daa7b05c/packages/flutter/lib/src/widgets/interactive_viewer.dart#L798-L800
The _gestureIsSupported looks like this:
https://github.com/flutter/flutter/blob/c5a4b4029c0798f37c4a39b479d7cb75daa7b05c/packages/flutter/lib/src/widgets/interactive_viewer.dart#L729-L742
So if the GestureType is pan it returns the value of the panEnabled property, which is false in this case.

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.1, on macOS 11.2.2 20D80 darwin-x64, locale
    de-AT)
    • Flutter version 2.0.1 at /Users/patrick/Applications/flutter
    • Framework revision c5a4b4029c (2 days ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/patrick/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.54.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.20.0

[✓] Connected device (2 available)
    • iPhone von Patrick (mobile) • 74fd9232bceb37fcaffe76b271f3568b95d9bbf5 •
      ios            • iOS 14.4
    • Chrome (web)                • chrome                                   •
      web-javascript • Google Chrome 88.0.4324.192

• No issues found!

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.1, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-GB)
    • Flutter version 2.0.1 at /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision c5a4b4029c (4 days ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.54.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (3 available)
    • SM A260G (mobile) • 5200763ebcfa861f • android-arm    • Android 8.1.0 (API
      27)
    • macOS (desktop)   • macos            • darwin-x64     • Mac OS X 10.15.4
      19E2269 darwin-x64
    • Chrome (web)      • chrome           • web-javascript • Google Chrome
      88.0.4324.192

! Doctor found issues in 1 category.

CC @justinmc

It's ok on master (the call to onInteractionUpdate is at the top of the _onScaleUpdate) but not in stable, beta or dev.
If I made a PR on stable to be like master, could it be considered as a hotfix?

@GP4cK Is it really ok on master?  I wasn't seeing onInteractionUpdate being called.  Are you using touch gestures or a mouse? Here is the relevant code at the time of this comment.
I've opened a quick PR to fix this, if I'm not mistaken about it not working on master: https://github.com/flutter/flutter/pull/78990

@justinmc weird. I hadn't tested, I was just reading the source code on Github and switching branches to compare.
I even forked the repo yesterday and you can see that in my version, widget.onInteractionUpdate is called at the beginning of _onScaleUpdate, before _gestureIsSupported.
Anyways, good you made a PR!


md-select not re-setting selected value after selecting new option

Bug, feature request, or proposal:
Bug
What is the expected behavior?
I am trying to reset the selected option from a md-select . Initially the default option is set to 2, however when I select a new value I am trying to reset the selected option to a different value for example here is my md-select
<md-select name="teamSize" [(ngModel)]="selectedTeamSize" (change)="updateSubscriptionOnKeyUp(company)">
        <md-option *ngFor="let sizes of teamSizes" [value]="sizes.value">{{sizes.displayValue}}</md-option>
      </md-select>

on my updateSubscriptionOnKeyUp function
if(company.subscriptions[0].type === "Free") {
      if(this.selectedTeamSize > 2) {
        this.selectedTeamSize = 2;
      }
    }

but the value this.selectedTeamSize is not being set to 2, it remains my selected value, not sure what I am doing wrong
What is the current behavior?
Selected value is set rather than the new reset value
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
@angular/cli: 1.1.2
node: 8.1.2
os: darwin x64
@angular/animations: 4.2.3
@angular/common: 4.2.3
@angular/compiler: 4.2.3
@angular/core: 4.2.3
@angular/forms: 4.2.3
@angular/http: 4.2.3
@angular/material: 2.0.0-beta.6-0aaeb69
@angular/platform-browser: 4.2.3
@angular/platform-browser-dynamic: 4.2.3
@angular/router: 4.2.3
@angular/cli: 1.1.2
@angular/compiler-cli: 4.2.3

@limpep Have you tried using the latest version of beta?

@donroyco I'm using "@angular/material": "git+https://github.com/angular/material2-builds.git",

@limpep It's stated otherwise in your list:

@angular/material: 2.0.0-beta.6-0aaeb69

beta7 is out.

@donroyco Just upgraded, no luck same issue
@angular/cli: 1.1.3
node: 8.1.2
os: darwin x64
@angular/animations: 4.2.4
@angular/common: 4.2.4
@angular/compiler: 4.2.4
@angular/core: 4.2.4
@angular/forms: 4.2.4
@angular/http: 4.2.4
@angular/material: 2.0.0-beta.7
@angular/platform-browser: 4.2.4
@angular/platform-browser-dynamic: 4.2.4
@angular/router: 4.2.4
@angular/cli: 1.1.3
@angular/compiler-cli: 4.2.4

@limpep any errors in the console? Specifically something like "change after checked"?

Try splitting up your two-way binding so you have [ngModel]=... and (ngModelChange)=...
Plunker reproduction?

Have you try surrounding updateSubscriptionOnKeyUp  code with a setTimeout?

Can you provide a reproduction in a plunker?  I was not able to reproduce this myself but might be misunderstanding what you had occuring.


Invalid ArtifactID

See: https://github.com/500px/kinesis-stream/issues/8
This PR fixes a typo in the Readme referencing artifact ID kinesis-stream when most repositories list it as kinesis-stream_2.12.

External PR issue was fixed in #7. I will update the version in the readme in #10


Simplified Chinese localization error

Lang code for Foundry's Simplified Chinese should be cn, so...please kindly change it on next release?
like this:
		{ 			"lang": "cn", 			"name": "中文（简体）", 			"path": "lang/zh_Hans.json" 		}

Fixed by #48


Update README to reflect error best practices.

This PR updates README.md, adding a section about proper error handling and updating the Response Serialization section to define and use an example of a custom ErrorType.

This looks great @jshier. I replaced some tabs with spaces and rebased. Thanks for putting this together! 🍻


Support Swift package manager source layout conventions

Make it easy for a build author to create a build that uses the Swift PM source layout conventions.
This might be by making these conventions the default, or by adding a plugin or some DSL element to allow the layout convention to be declared.
As an increment, we might first add a way for the build author to declare the source files/directories for a Swift project in a single place and have all compilation tasks and Xcode project generation honour this, rather than having to configure this in several places.

4 years ago, I had a discussion about what would be the source layout conventions to use in a native project. I was drawn by the Maven and Gradle as there seems to account for any possible use case. But why such layout? The closest I came to answer this question was in this discussion. Although it doesn't totally answer the questions, the source layout convention is the most flexible for any every use case you can think of. This is why I would recommend using Gradle source convention for Swift by default and have a plugin to apply the Swift PM source layout convention if required.


Write tests for data entry functionality

This is a good opportunity to learn some minitest, the ruby library we use for writing tests!

What are we trying to do? QA our data entry form, /data_entry!

What feature or behavior is this required for? We wrote a data entry function in a hurry for deployment; we want to have it tested so that we don't accidentally break it in a future small release.

How should we solve this issue?

We should write the following:

A controller test in patients that confirms that the methods data_entry works, and that data_entry_create works with both the success and failure conditions
A new Capybara integration test that i) fills out the form successfully and confirms that the fields show up on the patient edit page it dumps you on; and ii) fills out the form unsuccessfully and confirms that any errors are on the page.

I'd be happy to take this one!

hey @vincecampanale , that would rule! Let me know if I can assist with anything as you dig in.

Hey @colinxfleming happy to help! I'm running into some setup issues. The command rake db:seed is returning this error:
$ rake db:seed
rake aborted!
NoMethodError: undefined method `ruby_dirs=' for #<Quality::Config:0x007ff28311d050>
Did you mean?  ruby_files
/Users/vincecampanale/dev-stuff/open-source/dcaf_case_management/lib/tasks/quality.rake:8:in `block in <top (required)>'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/quality-27.3.0/lib/quality/rake/task.rb:57:in `initialize'
/Users/vincecampanale/dev-stuff/open-source/dcaf_case_management/lib/tasks/quality.rake:3:in `new'
/Users/vincecampanale/dev-stuff/open-source/dcaf_case_management/lib/tasks/quality.rake:3:in `<top (required)>'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:268:in `load'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:268:in `block in load'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:240:in `load_dependency'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:268:in `load'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/railties-<IP_ADDRESS>/lib/rails/engine.rb:658:in `block in run_tasks_blocks'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/railties-<IP_ADDRESS>/lib/rails/engine.rb:658:in `each'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/railties-<IP_ADDRESS>/lib/rails/engine.rb:658:in `run_tasks_blocks'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:452:in `run_tasks_blocks'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/railties-<IP_ADDRESS>/lib/rails/engine.rb:453:in `load_tasks'
/Users/vincecampanale/dev-stuff/open-source/dcaf_case_management/Rakefile:4:in `<top (required)>'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/gems/rake-12.0.0/exe/rake:27:in `<top (required)>'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `eval'
/Users/vincecampanale/.rvm/gems/ruby-2.3.1/bin/ruby_executable_hooks:15:in `<main>'
(See full trace by running task with --trace)

hey @vincecampanale , it looks like a gem we were using depreciated something and you're the first person to get bitten, sorry!
Go ahead and comment out the code in lib/tasks/quality.rake and that should fix.

@colinxfleming Okay, I think everything is ready to go. This is the output of rake test - is it expected?

@vincecampanale given the scope of this issue those are almost definitely false flags. I'd start up a pull request for what you did and see if it passes the CI test.

@colinxfleming I just meant that I was done with setup - I'll get on writing the tests now! I haven't done this before, so it may take a while but I hopefully I can get it done in a couple of hours.

Oh word! Yeah, go ahead and dig in without fretting about those. There are
some similar tests you can adapt for some ideas.
On Tue, Jan 24, 2017, 6:55 PM Vince Campanale<EMAIL_ADDRESS>wrote:

@colinxfleming https://github.com/colinxfleming I just meant that I was
done with setup - I'll get on writing the tests now! I haven't done this
before, so it may take a while but I hopefully I can get it done in a
couple of hours.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/colinxfleming/dcaf_case_management/issues/771#issuecomment-274977759,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADsA9PsArnSjVgsnxJCSU7dCgTvOPcOPks5rVo9fgaJpZM4LGJ29
.


Unable to locate package ...

Please can i get help ? what should i do to make it work ?

Please take a look at the installation instructions to see if that helps. We have changed to JDK 11 as the base.
Please let me know how this goes or close the issue if everything goes well.

Hi, @Kmotrus . Do you still experiencing some issues with environment setup?

@Kmotrus closing this for now. Feel free to ask questions/reopen this issue if you need help with it.


[computer-use] Correct prompt caching for tools

Following the official prompt caching documentation the current way computer use uses prompt caching does not include the tools - only the system prompt. This patch fixes this.

Nice, thank you!  Line 247 appears to be a duplicate comment of the line above it.

Oh yes, you are right that second like should say "tools" instead of "system prompt" - ah sorry, I did that one manually in the github editor and missed it.

Fixed the typo now

@Quasimondo thanks for your contribution. Per our documentation, this isn't actually necessary!

Prompt caching references the entire prompt - tools, system, and messages (in that order) up to and including the block designated with cache_control.

By caching the system prompt, we also cache tool definitons.

Thanks for clarifying that - maybe you could ask your documentation team to make that point even more clearer or add another example that shows tool + system prompt use at the same time.


[NFC] Add names of deserialized decls to pretty stack traces

Several declarations must be at least partially deserialized before their names can be determined. Add those names to pretty stack traces to make deserialization crashes easier to debug.
We don’t test pretty stack traces, so this doesn’t contain any test changes.

@swift-ci please smoke test


#748 roadmap link in README

#748 roadmap link in README
+Roadmap, future plans and other information about the project can be found at
+https://wiki.hyperledger.org/projects/burrow
+

thanks for this, didn't see it before #756 which adds it based on your issue

OK


Use poll() in BIO_socket_wait(), if available

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Fixes #24527

Also please drop the merge commit with git rebase -i

To be able to accept this we will need a CLA signed by you according to https://openssl-library.org/policies/cla/index.html

Also please drop the merge commit with git rebase -i

There are now two merge commits :(

Also please drop the merge commit with git rebase -i

There are now two merge commits :(

Sorry, no git expert here. This is the result of my efforts. Any help welcome!

 Then you need to do this on the BIO_socket_wait-use-poll branch checkout:
git rebase -i master
git push --force

git rebase -i will open an editor, where you mark the merge commits to be dropped.

 Then you need to do this on the BIO_socket_wait-use-poll branch checkout:
git rebase -i master
git push --force

git rebase -i will open an editor, where you mark the merge commits to be dropped.

Thanks! I hope it's okay now.

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This pull request is ready to merge

I wonder why it is supposed to be better to poll rather than using select()?

I wonder why it is supposed to be better to poll rather than using select()?

Searching the internet for "select versus poll" will give you answers, for example:
https://devarea.com/linux-io-multiplexing-select-vs-poll-vs-epoll/ or https://daniel.haxx.se/docs/poll-vs-select.html
select() is inefficient and limited to a maximum of FD_SETSIZE file descriptors (Linux kernel by default sets this to 1024).

Thank you for the explanation and the pointers with interesting further detail!
(I just corrected the pointer MarkDown syntax in the above comment to make them clickable.)

Pushed. Thanks for your contribution


readarr-develop: stuck on v<IP_ADDRESS>87

The readarr container don't seem to have been updated for several weeks:

readarr-develop is still on v<IP_ADDRESS>87. The latest pre-release version is v<IP_ADDRESS>34

https://github.com/onedr0p/containers/pull/1088