Test for proper serialization/deserialization of Hedera protobuf objects

Description:
Introducing a new tests suits to verify serialization & deserialization of Hedera protobuf objects.
Testing only methods toProtobuf & fromProtobuf in the following order:

Related issue(s): Issue #163

Fixes #
Added the following implementation files for each test suit

SerializationTests.cc - testing only toProtobuf method:

class ECDSAPublicKey - test ECDSAPrivateKeyToProtobufTest

DeserializationTests.cc - testing only fromProtobuf method:

class ExchangeRate - test DeserializeExchangeRateFromProtobufTest
class ExchangeRateSet - test DeserializeExchangeRateSetFromProtobufTest
class TransactionReceipt - test DeserializeTransactionReceiptFromProtobufTest
class TransactionRecord - test DeserializeTransactionRecordFromProtobufTest
class TransactionResponse - test DeserializeTransactionResponseFromProtobufTest
class AccountBalance - test DeserializeAccountBalanceFromProtobufTest
class PublicKey - test PublicKeyFromProtobufTest

SerializationAndDeserializationTests.cc - testing both toProtobuf & fromProtobuf methods:

class AccountId - test SerializeAccountIdToProtobufTest & DeserializeAccountIdFromProtobufTest
class ContractId - test SerializeContractIdТoProtobufTest & DeserializeContractIdFromProtobufTest
class TransactionId - test SerializeTransactionIdToProtobufTest & DeserializeTransactionIdFromProtobufTest
class Transfer - test SerializeTransferToProtobufTest & DeserializeTransferFromProtobufTest

The following classes couldn't be tested, because all of them seem to be internal classes:

class Endpoint
class NodeAddress
class NodeAddressBook
class DurationConverter
class TimestampConverter

Notes for reviewer:
These serialization / deserialization tests could be moved in the previously added test files to avoid adding new test suites named like SerializationTests, DeserializationTests and SerializationAndDeserializationTests.
For example:

tests SerializeAccountIdToProtobufTest & DeserializeAccountIdFromProtobufTest could be moved into AccountIdTest as they are testing only class AccountId.
test DeserializeTransactionRecordFromProtobufTest could be moved into TransactionRecordTest as they are testing only TransactionRecord.

Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

It seems like a lot of the tests that you have written here are more or less copies of tests that already existed. I don't see a point in replicating those tests; any additional testing that should happen with those objects should just be put into those tests that already exist. Also, I think it may be better to have tests involving a specific object to be contained in a test file for that object as opposed to dividing the suites up by function (e.g. serialization, deserialization, etc.).

I was my fault I didn't communicate well that approach with you in the beginning.
I removed the 3 files for separate testing (SerializationAndDeserializationTests.cc, SerializationTests.cc and DeserializationTests.cc) and moved all of the tests in the corresponding *Test.cc files.
I agree it's better to stick tests involving a specific object to be contained in a test file for that object, so I did it already. Thanks for the advice!


Refactoring required by page Table extract

child of https://github.com/uberfire/uberfire-extensions/pull/242

+1, merging this as parent PR has been merged.

master: 654fd01422747ce2d23dcfbed9c331336c73f062


Added warnings this module has been merged into another module.

Added warnings this module has been merged into another module and this repo will no longer be maintained.

This change is

Hi @brywang-msft, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Bryan Wang (PSHCT)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

rejected


WIP: Select panel announcements

Built on top of

https://github.com/primer/react/pull/4920

Fixes https://github.com/github/primer/issues/3409
Closes (completes) https://github.com/github/primer/issues/3183
Only available behind feature flag (primer_react_select_panel_with_modern_action_list)

TODO:

[ ] Add detailed change log of announcements here

Rollout strategy

[x] Patch release
[ ] Minor release
[ ] Major release; if selected, include a written rollout or migration plan
[ ] None; if selected, include a brief description as to why

Testing & Reviewing

Merge checklist

[x] Added/updated tests
[ ] Added/updated documentation
[ ] Added/updated previews (Storybook)
[x] Changes are SSR compatible
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge
[ ] (GitHub staff only) Integration tests pass at github/github (Learn more about how to run integration tests)

Close in favor of https://github.com/primer/react/pull/4968


Aborting fn install process leaves partially copied fn cli tool in target directory

curl -LSs https://raw.githubusercontent.com/fnproject/cli/master/install | sh
If you are just upgrading your functions cli client, ignore this and wait a few seconds.
You may press Ctrl+C now to abort this process.

(wait for download to start)
(hit Ctrl+C)

fn
[1]    4299 bus error (core dumped)  fn

installer should download to a temp location and move over fn when complete

What do you think of removing that few second sleep?  I'm pretty sure most people will want the upgrade and this delay is just a waste for the most part. Is there any case where someone would run this installer and not want it upgraded? cc @rdallman

seems fine to remove sleep, I can think of cases but it's not worth enumerating. at a minimum, we should just warn that we found a pre-existing install.
I think it's pretty standard to download to /tmp and move the file, we should do that anyway. (thanks for raising issue)


TiTv: rowsums are not 100%

Hi @PoisonAlien ,
In some cases, I found titv plots are like:

By reading the source code and the detailed data, I have found the problems are that some indel mutations were classified as SNP:

The trouble is located in annovarToMaf.R :

variations as GTGAG>TTGTA or CTA>TTC will be classified as SNPs when treated with the function annovarToMaf.
Do you have any suggestion about this?
Thank you very much.

Ah, I see..
Thanks for tracking the issue. I'm not sure why annovartomaf misclassified them. I will look into it soon.

Hello,
Is it possible to share an example file ?

Hi,
Your files helped me a lot! It has been a while since I dealt with annovartomaf and I realised author has changed the annotations a bit. I had to make significant changes to the code, but I think its quite stable to use now.
Could you please try it on your files and let me know if it works without any issues ? You will have to install from GitHub.
Thanks again for the files.

I have tested the new version annovarToMaf() on abount 10 ANNOVAR text files. It works perfect!
Thanks a lot for the excellent work!

Thanks for confirming. I am closing this issue for now. Please feel free to re-open if necessary.


fix: retain context timeouts in ServerStreamingAttemptCallable

Addresses callable overwrite of timeout && streamWaitTimeout mentioned in #1144.

It has been a while since I've touched Java, so please let me know if the tests are wonky.

Discussed offline: there might be similar changes to AttemptCallable (for Unary RPCs) necessary, but we are punting on that right now while we investigate how to move forward with changes to timeout/deadline calculation in general. The context timeout is handled slightly differently for Unary RPCs than for Server Streaming. We need to untangle some things.

FYI @chingor13: this shouldn't have any impact on standard client use. This only comes into affect when a caller is explicitly setting a timeout on the context given to a retryable ServerStreaming call via the call(request, context) API. This change honors that caller-provided timeout, rather than silently overwriting it w/totalTimeout as it has been doing.


declare a better human way for building app

at the moment we have a only a cli for nerds maybe its better to add something for beginners

Lol? ^


Suggestion: Add unused bf-pixel 'hey' animation

Hi guys,
So as stated in the title I would think it would be cool to add the unused hey sprite from pixel bf.
I think it would  be a good idea since there is already a noteType that makes the hey animation play.
If you want to add it I have already made the spritesheet, the xml and the json.
(Also it isn't from a random person. It is from Moawling himself)
I got the sprite from here
Bf Pixel.zip

Sure, you can have the assets. Just click on Bf Pixel.zip

There's no place in the week 6 where this could be used.
On Roses you can hear "Hey"s, but they happen during Boyfriend sections and so the animation would have been cut by a note sing


adding dynamic route to zuul

not sure if this one is currently supported.
need to add (maybe also remove, or update) existing routes to a running zuul proxy.
is this currently possible? if not - any ideas on how we might achieve the same without really restarting your app and updating mapping in a properties file?
thanks

https://github.com/spring-cloud/spring-cloud-netflix/issues/706

@aivans
how to dynamic route and add route info not start app
@RequestMapping("/reDirect")
public String reDirect(HttpServletRequest request) {
ZuulRoute route_cy=new ZuulRoute();
route_cy.setId("erp_cy");
route_cy.setPath("/erp_cy/");
route_cy.setUrl("http://<IP_ADDRESS>:8081");
discoveryClientRouteLocator.addRoute(route_cy);
ZuulRoute route_gwzx=new ZuulRoute();
route_gwzx.setId("erp_gwzx");
route_gwzx.setPath("/erp_gwzx/");
route_gwzx.setUrl(" http://localhost:9000");
discoveryClientRouteLocator.addRoute(route_gwzx);
String id=request.getSession().getId();
return "redirect:/erp_gwzx/";

@aivans
first ,i use ui gateway to offer one point endpoint services ,at the same time ,the backends service machine can't be access by the Internet, when users open the browser the ui server route to the sso system,when login success,the sso server redirect to the ui gateway ,the ui gateway according to the session id get user info from redis ,now the requirements is according to the userID, the ui gateway route to different pc get ui pages.
the first problem is how i get zuul route config info from database
the second is how ui gateway according to userid add route to zuul and zuul route to diff pc,(because some service such as http://<IP_ADDRESS>, there not the context, at the same time can't use http Re-direct 302, ) ,dynamic add the zuul route and bypass to the diff pc get ui page

Duplicates #706 & #705

Is there any way to add dynamic path variable routing. Scenario:
application.properties
zuul.routes.books.url=http://localhost:8090
zuul.routes.books.path=/mybook/*
Zuul proxy is running on localhost:8080.
For incoming request http://localhost:8080/mybook/avaiable
I'd like to send it to url http://localhost:8090/avaiable/123
All I need is to add path variable at the end of url.


PF spawns laikamilter

I set PF to talk to Laika but I have fixed that and updated this comment, look at psaux | gerp laikamilter ,this was trying to spawn laikamilter.py from master.cf and that was a failure.
smtpd_milters = inet:<my hosts IP>:7226
If there is any detailed documents on setup, examples, etc, I would be interested in seeing those.

I seen similar thread, any ideas?

Hi @howtotalktogirlsatparties,
You have to remember that in header/body content, Postfix may rewrite or reject header/body content before it stores that content in the queue file; Postfix stores the same header/body content as when no Milters are configured. If Postfix does not reject the header/body content, Postfix passes it to the first configured Milter which may modify or reject that content or may modify the stored envelope. If the first Milter does not reject the header/body content, Postfix passes it to the second configured Milter, and so on.
In Lockheed Martin's case, Postfix talks to laikamilter via the milter protocol (you configured this above), which keeps allowing call backs all the way until the end of the message.
Try setting up this line:
smtpd_milters = inet:<my hosts IP>:(hostname)

Cheers,
Montana Mendy

this makes more sense


Added Math Sprint Game

Description 📜
This game checks your IQ speed, how fast you can calculate maths solutions with different rounds

1st round 10 questions
2nd round 25 questions
3rd round 50 questions
4th round 99 questions

Fixes #698

Type of change 📝
Please delete options that are not relevant. And in order to tick the check box just put x inside them for example [x] like this

[x] Add a new project

Domain of Contribution 📊
Please delete options that are not relevant. And in order to tick the check box just put x inside them for example [x] like this

[x] Project using HTML, CSS & JS

Checklist ✅
Please delete options that are not relevant. And in order to tick the check box just put x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots

Can you add labels again, I removed them by mistake

@Mrjoy832


DataDeps warning when running get

GADM.get("JPN") # first run

yields a warning:
┌ Warning: Checksum not provided, add to the Datadep Registration the following hash line
│   hash = "a7d0955dfd9215d6b21cff3f5b7bf09b02c3b64290e7b6ac1b357b7789f45e03"
└ @ DataDeps ~/.julia/packages/DataDeps/Y2lje/src/verification.jl:44

I get similar errors for every other country.  This should probably be resolved in the registration block here:
https://github.com/JuliaGeo/GADM.jl/blob/606b61a9ce6fa7b3e4258ea12e60aa18bf30c3ba/src/GADM.jl#L64-L69

If the hash is fixed per ID, then a dictionary can be introduced in the code for lookup. PRs are welcome.


Add metals

Hello, I added a recipe for metals. Cheers!

@fgborges
Install is succeeded.
  metals = {
    cmd = { "/home/h-michael/.cache/nvim/nvim_lsp/metals/metals" },
    install_dir = "/home/h-michael/.cache/nvim/nvim_lsp/metals",
    is_installed = "file"
  },

BTW I'm not familiar with Scala, but is there always a build.sbt in the project root?

@h-michael I used scala, and yeah basically, any Scala project will have a build. It's much like Rust where you could technically use it outside of that, but most people use Cargo.

I'm going to test it on one of my scala projects and then merge. We really should try to set up automated tests for things which have LspInstall.

I have applied every suggestion to my request. Please check it.


chore: add containerfile virt-v2v

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 16.33%. Comparing base (c43e8b2) to head (06f755d).
Report is 10 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1065      +/-   ##
==========================================
+ Coverage   16.28%   16.33%   +0.04%
==========================================
  Files         112      112
  Lines       19771    19771
==========================================
+ Hits         3220     3229       +9
+ Misses      16267    16256      -11
- Partials      284      286       +2

Flag
Coverage Δ

unittests
16.33% <ø> (+0.04%)
:arrow_up:

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

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


Store whether h2 is in use in RequestContext.

A system test would be nice, but I wouldn't know how to implement it today without putting in an unreasonable effort. LGTM.

Well, right now it is especially hard to test since nothing looks at this bit at all. I hand-tested for adding a debug statement.
Eventually we will probably have some tests that expect nghttp to be in $PATH or something, and disable if it's not there --- sadly I don't think command-line h2 clients are yet widely available. Hmm, even our google-side test environment will only build it when also building Apache 2.4; I should probably modify that to build the nghttp command-line client even if Apache doesn't have h2 support so it's available for testing of nginx.


failing benchmark "max_ndvi_composite"

last run at the time: https://github.com/ESA-APEx/apex_algorithms/actions/runs/11684670042/job/32536394554
benchmark failure due to job failure:
E               openeo.rest.OpenEoApiError: [500] Internal: Server error: TypeError("'NoneType' object is not iterable") (Upstream ref: 'r-24110514497740198b2a6f53bb71cd0a') (ref: r-241105360b86482286aa860831ed76ea)

from logs of r-24110514497740198b2a6f53bb71cd0a:

Traceback (most recent call last):
  File "/opt/openeo/lib/python3.8/site-packages/flask/app.py", line 880, in full_dispatch_request
    rv = self.dispatch_request()
  File "/opt/openeo/lib/python3.8/site-packages/flask/app.py", line 865, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)  # type: ignore[no-any-return]
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/users/auth.py", line 95, in decorated
    return f(*args, **kwargs)
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/views.py", line 939, in start_job
    backend_implementation.batch_jobs.start_job(job_id=job_id, user=user)
  File "/opt/openeo/lib/python3.8/site-packages/openeogeotrellis/backend.py", line 1634, in start_job
    self._start_job(job_id, user, _get_vault_token)
  File "/opt/openeo/lib/python3.8/site-packages/openeogeotrellis/backend.py", line 1683, in _start_job
    job_dependencies = self._schedule_and_get_dependencies(
  File "/opt/openeo/lib/python3.8/site-packages/openeogeotrellis/backend.py", line 2202, in _schedule_and_get_dependencies
    convert_node(result_node, env=env.push({
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 456, in convert_node
    process_result = apply_process(process_id=process_id, args=processGraph.get('arguments', {}),
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1706, in apply_process
    return evaluate_process_from_url(
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1989, in evaluate_process_from_url
    return _evaluate_process_graph_process(
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1962, in _evaluate_process_graph_process
    return evaluate(process_graph, env=env, do_dry_run=False)
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 431, in evaluate
    result = convert_node(top_level_node, env=env)
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 456, in convert_node
    process_result = apply_process(process_id=process_id, args=processGraph.get('arguments', {}),
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1698, in apply_process
    args = {name: convert_node(expr, env=env) for (name, expr) in sorted(args.items())}
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1698, in <dictcomp>
    args = {name: convert_node(expr, env=env) for (name, expr) in sorted(args.items())}
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 488, in convert_node
    return [convert_node(x, env=env) for x in processGraph]
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 488, in <listcomp>
    return [convert_node(x, env=env) for x in processGraph]
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 456, in convert_node
    process_result = apply_process(process_id=process_id, args=processGraph.get('arguments', {}),
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1732, in apply_process
    return process_function(args=ProcessArgs(args, process_id=process_id), env=env)
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1381, in filter_bbox
    spatial_extent = _extract_bbox_extent(args, "extent", process_id="filter_bbox")
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1363, in _extract_bbox_extent
    d = {
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1364, in <dictcomp>
    k: extract_arg(extent, name=k, process_id=process_id)
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 503, in extract_arg
    return _as_process_args(args, process_id=process_id).get_required(name=name)
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 495, in _as_process_args
    args = ProcessArgs(args, process_id=process_id)
  File "/opt/openeo/lib/python3.8/site-packages/openeo_driver/processes.py", line 287, in __init__
    super().__init__(args)
TypeError: 'NoneType' object is not iterable

found the issue:
benchmark scenario uses bbox as parameter name to pass the spatial extent:
https://github.com/ESA-APEx/apex_algorithms/blob/43bf8146ce43d848040f7eb67c2e7e9c8411e1d9/benchmark_scenarios/max_ndvi_composite.json#L11-L17
but the UDP expects a spatial_extent parameter:
https://github.com/ESA-APEx/apex_algorithms/blob/43bf8146ce43d848040f7eb67c2e7e9c8411e1d9/openeo_udp/examples/max_ndvi_composite/max_ndvi_composite.json#L383-L385
It was quite confusing to figure this out, beause the spatial_extent was made an optional parameter:
https://github.com/ESA-APEx/apex_algorithms/blob/43bf8146ce43d848040f7eb67c2e7e9c8411e1d9/openeo_udp/examples/max_ndvi_composite/generate.py#L17-L19
I guess it would have failed with a much cleaner error if that parameter wasn't optional
@jdries  you seemed to make it optional intentionally with https://github.com/ESA-APEx/apex_algorithms/commit/3e14e1330ff0c9ef2f03ae9f31c1717e2c5f350d#diff-0465fb74b5eed7572e8e0e73471b227c6ed5a472441522fc64c1a026878d809dL17  is there a particular reason for that?

Yes, intentional, I actually even want to use:
https://open-eo.github.io/openeo-python-client/api.html#openeo.api.process.Parameter.spatial_extent
See:
https://esa-apex.github.io/apex_documentation/guides/udp_writer_guide.html#spatial-filtering

@jdries  I guess you commented on the bbox/spatial extent rename, which I fully understand.
My question was about making the parameter optional with a default value None/null (which doesn't work)

it's optional because a user may want to leave it empty, and specify some sort of spatial filtering later on, when using the udp in a larger pipeline, this is the same reasoning as for load_collection.

ok makes sense, but the problem is then that that spatial_extent in the UDP is currently used with filter_bbox, which does not allow the extent to be null/None. It's probably  better to move that parameter to load_collection where it is allowed to be null. Or is there a reason to have an explicit filter_bbox process instead of spatial_extent in load_collection?


v2022.11.01 Android studio Release

Please Let us know if there are any changes you wanted to include. Thanks!

Internal cleanup.


vtgate: support for /debug/health

@alainjobart

This change is

LGTM as the base. I think we have to check if we start the server after or before we get the initial endpoint set.

Yeah. I expect many iterations based on feedback.


fix(freespace_planner, mission_planner): null checking

Signed-off-by: Shumpei Wakabayashi<EMAIL_ADDRESS>Description

Resolve https://github.com/autowarefoundation/autoware.universe/issues/1251.
This PR reverts some commits in this PR.
As for freespace_planner, I remove unnecessary nullptr checking.
As for mission_planner, it waits until route_handler is ready.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

https://github.com/autowarefoundation/autoware.universe/blob/86101edd278bd572124995866d7fecf534830d21/planning/mission_planner/lib/mission_planner_base.cpp#L137
It's called here so

@taikitanaka3 Right, thank you and I fixed it.


macos-12 is dead long live arm64

This effectively reverts https://github.com/OpenAstronomy/github-actions-workflows/pull/195 and https://github.com/OpenAstronomy/github-actions-workflows/pull/186 and moves us to arm64 macos by default. (Using https://github.com/OpenAstronomy/github-actions-workflows/pull/247 it's techincally now possible to force a macos-13 native build for x86_64).
This fixes https://github.com/OpenAstronomy/github-actions-workflows/issues/256 which is the removal of macos-12.

oh lol it seems like it's quicker to build x86_64 macos wheels on arm64 than it is on the native x86_64 macos-13 runners.


Porting functions to the modern Puppet 4.x API

We noticed that your Puppet module includes Puppet 3.x function(s) and created
this pull request to help you get started porting them to the new API.
The Puppet function API has a long history and many people in the ecosystem,
including you, took advantage of this API to extend Puppet to do some really
cool things. Unfortunately the original function API had many critical
limitations, and so Puppet introduced a new and improved API with Puppet 4.x.
Each modern Puppet 4.x function is just a little faster and just a little safer
to use, meaning that as we all port our functions over, compilation times across
the ecosystem will get more and more performant. Modern Puppet 4.x functions
have improved thread safety, memory management, and load time. Even more
importantly, they’re isolated to the environment they’re loaded from.
And as a developer, you'll see benefits like namespaced function signatures,
automatic data type checking, multiple dispatches allowing you to easily handle
different kinds of function invocations, and vastly improved code reuse.

This pull request was autogenerated and the ported functions do not yet take
advantage of the data type checking or multiple dispatches. You will want to
finish the port by making those changes and adding any documentation updates.
See this tutorial for more information.
While you're at it, double check to see if there are any functions that couldn't
be automatically ported and update them yourself.
To be extra cautious about losing data, this pull request did NOT delete the
legacy function files, so when you're satisfied with the new functions, don't
forget to delete the legacy code!
If this repo is a fork, when you're all done, you might consider contributing
your work upstream.

If you have any questions, feel free to ping me in this PR, or stop by the
Puppet Community Slack to chat. And thanks so much
for being part of our vibrant community of developers!

He, thanks
this module is superseeded and replaced by https://github.com/biemond/biemond-orawls/ that was already years ago. is someone this using this one?

Probably not? My automation just scraped for open repositories that appeared to have Puppet 3.x functions. Maybe this could be archived. Fwiw, the function updater failed on the orawls module due to the embedded function definitions. Those are easier and safer in Puppet 4.x functions. I've got a little tutorial up on porting if you're interested.


Channel/Server loading visual bug

When loading different channels the normal discord ui shows up for a second, looks strange. It hasn't done this before. If the channel loads faster it doesn't show up/doesn't show for long. It goes back to normal once the channel fully finishes loading.
Screenshots

This is the wrong repository.


feat: fast iavl for v4

Brief Changelog

use an sdk branch supporting fast iavl
use go 1.18
use tendermint v0.34.24
use iavl v0.19.4
update & cleanup .golangci.yml
run golangci-lint run ./... --fix

this caused gofmt -s to run

Testing and Verifying
This change added tests and can be verified as follows:
go mod edit -replace github.com/tendermint/tm-db=github.com/baabeetaa/tm-db@pebble
go mod tidy
go install -ldflags '-w -s -X github.com/cosmos/cosmos-sdk/types.DBBackend=pebbledb -X github.com/tendermint/tm-db.ForceSync=1' -tags pebbledb ./...
osmosisd init test
cp networks/osmosis-1/genesis.json ~/.osmosisd/config/genesis.json
time osmosisd start --db_backend pebbledb

Awesome, ty for takign this on! Can we merge the SDK branch into v4? https://github.com/osmosis-labs/cosmos-sdk/pull/392
(That LGTM, aside from one small comment + CI saying some go/mod update needed)

Unfortunately, it wasn't in the end very small. I might try to reapproach the whole thing with @catShaark tonight, if he is at the office.


intermittent failures in tests/net/socket/select/ on qemu_x86

Describe the bug
I'm intermittently seeing a problem with test_select on x86 where an assertion is failing. This happens often for me.
This seems to be a timing issue with QEMU if we get unlucky with a tick boundary. It depends on how much the workstation is loaded, unfortunately we don't have -icount working and QEMU tries to sync the target clock to the host clock.
Can we relax the zassert_true(tstamp <= 1, "") checks? In the failure case, tstamp has a value of 10. Are we trying to prove that the call isn't blocking?
Observed Behavior
SeaBIOS (version rel-1.11.2-0-gf9626ccb91-prebuilt.qemu-project.org)
Booting from ROM..***** Booting Zephyr OS zephyr-v1.13.0-3961-gf2b6fca9b1 *****
Running test suite socket_select
===================================================================
starting test - test_fd_set
PASS - test_fd_set
===================================================================
starting test - test_select

    Assertion failed at /home/apboie/projects/zephyr3/tests/net/socket/select/src/main.c:97: test_select: (tstamp <= 1 is false)
got tstamp 10
FAIL - test_select
===================================================================
Test suite socket_select failed.
===================================================================
PROJECT EXECUTION FAILED

Expected behavior
SeaBIOS (version rel-1.11.2-0-gf9626ccb91-prebuilt.qemu-project.org)
Booting from ROM..***** Booting Zephyr OS zephyr-v1.13.0-3961-gf2b6fca9b1 *****
Running test suite socket_select
===================================================================
starting test - test_fd_set
PASS - test_fd_set
===================================================================
starting test - test_select
PASS - test_select
===================================================================
Test suite socket_select succeeded
===================================================================
PROJECT EXECUTION SUCCESSFUL

Are we trying to prove that the call isn't blocking?

Yes, that was the idea. For me, this issue doesn't seem to be reproducible, though while writing, I had to add + FUZZ for a case below the one which fails for you. Of course, I can believe it may fail, and we should do something about it. I have following ideas:

Would you be interested to try CONFIG_QEMU_TICKLESS_WORKAROUND=y? It was in some other configs for timing-related tests, and I started with it too, but as I didn't see any visible effects, and wasn't sure what exactly it does, I decided to remove it.
We multiple existing timeouts in the test say by 3, to allow to still reliable distinguish "blocking" vs "non-blocking" behavior, and add + FUZZ to "non-blocking" cases.
Can do whatever else quick patch you may suggest, e.g. just add + FUZZ to the failing case.

I'd say add a system tick's worth of fuzz to each

Ok, then with adjusting timeouts for blocking calls, as then required to properly tell non-blocking from blocking, it's effectively p.3, and posted as https://github.com/zephyrproject-rtos/zephyr/pull/13028


Added new shapes

#27

#27

you need to add in the functionality of each shape, so that the user can draw them on the canvas


Code coverage reports

Is it worth adding code coverage reports to the repos?
I see the interledger-js repos make use of https://codecov.io/

Yes! Now that we have some tests I think we should definitely start tracking this

closed, see PR #39


Redis Support for FedScale

Why are these changes needed?
To integrate Redis into aggregator for saving aggregation data.

Related issue number

N/A
Checks

[x] I've included any doc changes needed for https://fedscale.readthedocs.io/en/latest/
[x] I've made sure the following tests are passing.
Testing Configurations

[x] Dry Run (20 training rounds & 1 evaluation round)
[x] Cifar 10 (20 training rounds & 1 evaluation round)
[x] Femnist (20 training rounds & 1 evaluation round)

Note:

All tests are run on cpu.
Met with the following KeyError bug on the same line in Femnist once with Redis, and also once without Redis (i.e. original code). Sample error output:

File "/users/xuyehe/FedScale-rd/fedscale/core/aggregation/aggregator.py", line 386, in client_completion_handler
    duration=self.virtual_client_clock[results['clientId']]['computation'] +
KeyError: 665

Thanks a lot for your contribution! Let's try to run an e2e evaluation to validate the learning curve and Redis overhead first.

Thanks for your contribution. Can you please

remove all prints or replace them with logging.info
Summarize the results of the overhead analysis

We really appreciate your help.


MovePiece() causes an error despite never being called for. (SOLVED)don't make a global Move

It gives a null reference exception despite never being called. THis occurs after I play the first move agianst my bot.

Update to latest version of the repo 😁

Can you post the stacktrace of the exception?

@Moonwalker316 if it's solved please press the close issue button at the bottom part of the page!

Closing since solved


How to communicate with specific Connected Client

I have connected multiple client and continuously read data on server.Now How can I communicate with specific client from connected clients?

    public void Broadcast(byte[] data)
    {
        foreach(var client in _listeners.SelectMany(x => x.ConnectedClients))
        {
            client.GetStream().Write(data, 0, data.Length);
        }
    }

    public void Broadcast(byte[] data)
    {
        foreach(var client in _listeners.SelectMany(x => x.ConnectedClients))
        {
            client.GetStream().Write(data, 0, data.Length);
        }
    }


GH-24: Enable > 1 Partitions in @TopicPartition

Resolves #24

Travis + Checkstyle is our "foe" :smile:

I say "friend" not "foe".; pushed

Pushed

Merged as https://github.com/spring-projects/spring-kafka/commit/a6a71a44f54ba65233e095db786048721cd66248


local and global

When run the "local" and "global" code, I found these problems.
In the following code changes only when running to "user", the rest of kw_prompt[:-1] does not remove prompt from result. Lead to the result = "{" + result. Split (" {") [1]. The split ("}") [0] + "} "keywords in the running result is example1.
result = (
result.replace(kw_prompt[:-1], "")
.replace("user", "")
.replace("model", "")
.strip()
)

When run the "local" and "global" code, I found these problems. In the following code changes only when running to "user", the rest of kw_prompt[:-1] does not remove prompt from result. Lead to the result = "{" + result. Split (" {") [1]. The split ("}") [0] + "} "keywords in the running result is example1. result = ( result.replace(kw_prompt[:-1], "") .replace("user", "") .replace("model", "") .strip() )


Added some rails cops

Will now validate that

You are using rails logger for output
You are using proper rails date format
You are using find_by over where().first
You are using find_each instead of all.each
You are using proper rails pluralizations 1.day.ago instead of 1.days.ago

The code looks fine. So :shipit:. But why points 3 and 4?

@jahammo2 i think they are performance related.

@dacur is correct :)


Encourage users by automatically creating (or adding) the category: photos by [username]?

I received a suggestion that adding a category "Photos by [username]" when a user uploads a photo, could potentially encourage more uploads. A lot of power-users already have a category with their name, so I wonder if extending it to other users would be okay? Or would it create too much clutter? @nicolas-raoul @VojtechDostal

Excuse my ignorance, but how is adding a "Photos by [username]" category to every photo uploaded by an user different from what Special:ListFiles does (listing all files uploaded by the user)?
I just don't see how this would encourage users to upload more images 🤔

I sometimes see these categories, and I always think it is redundant and wasteful lol
This could encourage people, as it makes it a bit easier to jump to other pictures by this user, but I think that solving this problem via categories is not the best solution. How about starting to solve this problem via https://github.com/commons-app/apps-android-commons/issues/3389 instead? :-)

Makes sense. I will close this, and I agree that focusing on profiles is a better idea. :)


Problem getting drag drop to work with multiple hierarchy bound treeview

Hello, i'm hoping someone can help me as as i've been tearing my hair out over this for some hours now : P
I have a treeview, as such;
<TreeView ItemsSource="{Binding Path=TreeViewOverviewViewModel.Categories}" Name="treeViewAssets">
    <TreeView.ItemContainerStyle>
        <Style TargetType="{x:Type TreeViewItem}">
            <Setter Property="IsExpanded" Value="{Binding IsExpanded, Mode=TwoWay}"/>
            <Setter Property="IsSelected" Value="{Binding IsSelected, Mode=TwoWay}"/>
            <Setter Property="FontWeight" Value="Normal"/>
        </Style>
    </TreeView.ItemContainerStyle>
    <TreeView.Resources>
        <HierarchicalDataTemplate DataType="{x:Type WPFFM:AssetCategoryViewModel}" ItemsSource="{Binding Path= Children}">
            <StackPanel Orientation="Horizontal">
                <TextBlock ContextMenu="{StaticResource assetOverviewContextMenu}" Text="{Binding Description}"/>
            </StackPanel>
        </HierarchicalDataTemplate>
        <HierarchicalDataTemplate DataType="{x:Type WPFFM:AssetViewModel}" ItemsSource="{Binding Children}">
            <StackPanel Orientation="Horizontal">
                <TextBlock ContextMenu="{StaticResource assetContextMenu}" HorizontalAlignment="Stretch" Text="{Binding Name}"/>
            </StackPanel>
        </HierarchicalDataTemplate>
    </TreeView.Resources>
</TreeView>

However if i set the treeview to be a dragsource and droptarget, this lets me drag categories into subcategories and such, and i really only want to be able to drag the children assets between the categories. Is this possible?
Kind regards,
Shane.

@shanefirecrackr the solution from @claudekennilol should be help, so i lose this issue now...


Raw HTML injection does not for client only app

Description
I am not seeing a very clear and obvious way to directly inject HTML into the DOM. Basically I just want to take a string and have it be placed in the DOM at a specific location, without any sort of manipulation from jaspr.
My expectation is that this would be the way to do it:
text('<img src="https://ui-avatars.com/api/?name=jaspr">',rawHtml: true)

However, when trying to use this on simple or complicating raw HTML strings I keep seeing all sorts of side effects. The rawHtml flag is doing something odd to the resulting HTML and interfering with other elements.
Steps To Reproduce
See the repo below, on the raw-html branch for a complete working example and a README with other context:
https://github.com/walsha2/jaspr-issues/tree/raw-html
Feedback
Injecting raw HTML should have first class support. Sometimes there are chucks of raw HTML code that we just want to copy (e.g. an svg with many path components) and insert into the DOM without needing to fist manually convert in the jaspr native components, which can be tedious. I had to do this for an svg and it was not fun.
Using text('...',rawHtml: true) seems a bit like a second class feature? I know it sounds trivial, but something like raw('...') seems to make more sense. Maybe this even bypasses the entire Component processing to ensure that there are no unintended side effects to the raw code. Im sure that is what the intent of rawHtml flag is doing, but it seems like that might not be the case as evidenced by this bug.
The docs for text state "Utility method to create a text component when using jaspr html methods" and that does not sounds like what I want to do necessarily. It could just be a matter of semantics, but this isn't a "text component", it is a raw DOM element and I do not want it to use any "jaspr html methods".

Update - as with seemingly all of my recent issues, this problem is isolated to client only apps.

Thanks for this issue.
You are right that right now rawHtml is not well working on the client, especially when used along sibling components.
I like the proposal of using a separate raw() method and removing the rawHtml flag from the text component.


Hydrating hasMany with correct ID

I am using UUIDs in a 'supplemental' manner - i.e. they are what is output and accepted by by the API, but standard auto-incrementing integers are used in the database joins.
hasOne relationships seem to populate fine when a UUID is supplied, for example:
{
    "data": {
        "type": "locums",
        "attributes": {
            "name": "Test Account",
            "email"<EMAIL_ADDRESS>            "password": "abcd1234",
            "password-confirmation": "abcd1234"
        },
        "relationships": {
        	"country": {
        		"data": { "type": "countries", "id": "GB" }
        	},
        	"region": {
        		"data": { "type": "regions", "id": "6F0073EC-3C98-4831-9595-890DF79E6E7D" }
        	},
        	"locum-type": {
        		"data": { "type": "locum-types", "id": "14C179EA-363F-4585-8879-6111CD162D62" }
        	}
        }
    }
}

In the instance above, the 'region_id' and 'locum_type_id' fields in the 'locums' table is correctly populated with the integers instead of the supplied UUIDs.
However, attempting to populate a hasMany relationship (employees below):
{
    "data": {
        "type": "practices",
        "attributes": {
            "name": "Test Veterinary Practice"
        },
        "relationships": {
            "country": {
                "data": { "type": "countries", "id": "GB" }
            },
            "employees": {
            	"data": [
                	{ "type": "employees", "id": "DDF08296-995A-42B5-847B-FD55E8F7C813" }
            	]
            }
        }
    }
}

doesn't work, as it tries to populate the pivot table ('employee_practice') with the UUID instead of the integer.
Is there something else I need to configure in any of the API code to make this work properly? I'm willing to supply any other information needed to get this working.
Cheers.

Hi. Yes, this seems to be a problem with the different ways its hydrating relationships. For has-one, it is retrieving the actual related model from the database and then calling associate on the relationship. However for has-many, it's just reading the IDs out of the JSON API request and calling sync.
It makes more sense to retrieve all of the models and pass those into sync, because then the model key will get used properly.
I'm planning on refactoring the hydrators at some point for v1.0, but can probably fix this via a bug fix now. I'll see if I can get to it this week but I am completely overrun with work at the moment.
As a quick note to myself, this is the line that needs changing:
https://github.com/cloudcreativity/laravel-json-api/blob/master/src/Hydrator/EloquentHydrator.php#L344

Yep, that would work as a temp fix

FYI - here's what I've gone with:
    protected function syncHasMany($resourceKey, RelationshipInterface $relationship, Model $model)
    {
        $relation = $this->getRelation($resourceKey, $model);

        if (!$relation instanceof BelongsToMany) {
            return false;
        }

        foreach ($relationship->getIdentifiers() as $identifier) {
             $relation->attach($this->findRelated($identifier));
        };

        return true;
    }

Yeah, that's basically what I'm going to have to change it to. You might want to use sync though rather than attach, as you need to detach any already attached models that are not in the list of identifiers.
I need to add an interface method to the store so that you can convert the ids in a single query rather than looping like that. But will have to do that in a future release as it'll be breaking as I'll need to change the interfaces.

Closing this as it is implemented on the 1.x-dev branch.


Num.shiftRight can produce undefined behaviour

Num.shiftRight can produce undefined behaviour when "over-shifting", i.e. by more than the number of bits in the operand.
num.shiftRightBy 12 0b1000_0000i8 returns 0b1111_1111/-1 in non-optimized builds and 0 in optimized builds, for the LLVM backend.
For the moment we have disabled some unit tests for LLVM release mode.
Unit tests

Thanks for looking into it!
But no, I'm afraid that error does not close this issue.
0b1000_0000i8 is intended to represent -128, which is a valid i8 value.
This issue is about numShiftRight not behaving correctly with the value -128. If we can't create that value as a binary literal, we should rewrite the test to create the same value some other way. Maybe with a decimal literal -128, or by subtracting two other values, or something.

It's weird to me that this is an error now. It's a correct binary representation of a valid i8 value.
I guess someone decided that the number literal itself can only represent positive numbers and then the - in front of it does the negation.
Which presumably means that we now have no way to write a literal that is the most negative value of any signed integer type. 😢
This is either a bug, or a decision someone made (that I don't agree with based on the info I have right now!)

OK thanks for the explanation, I misunderstood the issue.


Added a new function that return the current viewport value

return the current viewport value

Hi @dnlgmzddr!
Thanks for taking the time to bring new ideas into the project. I highly appreciate it. I've introduced a similar functionality to the one you proposed, but in a bit different way.
Cheers,
Maciej


make features unitful?

Really just a "nice to have" thing and not crucial at all...leaving here as a note to self, to be eventually categorized properly once I decide on a list of issue tags.

@rkurchin what exactly do you have in mind? Some details/examples might help.

It's sort of in the general spirit of interpretability/readability of features. Just the idea of using Unitful.jl to attach units to all the quantities that get encoded. It wouldn't change anything about how the models work or anything.

Hmm, I understand. Using a decorator for implementing this might be an elegant solution imo.


Automated cherry pick of #4895: fix(notify): Update template

Cherry pick of #4895 on release/3.0.
#4895: fix(notify): Update template

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.0@30a4d16). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             release/3.0   #4898   +/-   ##
=============================================
  Coverage               ?   6.84%
=============================================
  Files                  ?     524
  Lines                  ?   89951
  Branches               ?       0
=============================================
  Hits                   ?    6160
  Misses                 ?   83280
  Partials               ?     511

Flag
Coverage Δ

#aFlag
6.84% <ø> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 30a4d16...11ccfdd. Read the comment docs.


[Refactoring] Remove redundant qualifier names

Issue

n/a

Overview (Required)
In this PR, I removed the redundant qualifier names to fix lint warnings.
Links

n/a

Screenshot
n/a

Please check the warning 🙏
> The following files had format violations:
See https://docs.gradle.org/7.5.1/userguide/command_line_interface.html#sec:command_line_warnings
      src/commonMain/kotlin/io/github/droidkaigi/confsched2022/strings/Strings.kt
98 actionable tasks: 7 executed, 46 from cache, 45 up-to-date
          @@ -1,6 +1,5 @@
           package·io.github.droidkaigi.confsched2022.strings

          -import·io.github.droidkaigi.confsched2022.model.Res
           import·io.github.droidkaigi.confsched2022.model.Res.strings

           public·val·Strings:·strings
  Run './gradlew :core:model:spotlessApply' to fix these violations.


Bump version to 0.13.0

https://github.com/alphagov/digitalmarketplace-frameworks/compare/v0.12.6...bump-version-0.13.0

:+1:


Fixes #970: Events#off() can now deregister multiple, space-separated events at once

Fixes #970

Great, thanks @cspotcode.


⚠ POST to /search has degraded performance

In cd84db2, POST to /search (https://dev.factiverse.no/v1/search) experienced degraded performance:

HTTP code: 200
Response time: 4856 ms

Resolved: POST to /search performance has improved in a35fbdb.


add miras-tech.com to adapters

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1333      +/-   ##
==========================================
+ Coverage   85.79%   86.06%   +0.27%
==========================================
  Files          74       74
  Lines        2534     2534
  Branches      108      108
==========================================
+ Hits         2174     2181       +7
+ Misses        360      353       -7

Impacted Files
Coverage Δ

.../core/shared/src/main/scala/io/circe/Decoder.scala
85.25% <0%> (+0.26%)
:arrow_up:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
86.54% <0%> (+1.16%)
:arrow_up:

...les/core/shared/src/main/scala/io/circe/Json.scala
77.57% <0%> (+1.21%)
:arrow_up:

...re/shared/src/main/scala/io/circe/JsonNumber.scala
96.38% <0%> (+2.4%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b40e52...4f1483b. Read the comment docs.

Thanks much!


Reports of Habits and complete visualization beyond 14 days

Hey! thanks for the awesome plugin. I've been trying to achieve something like this in Logseq for a while, so I'm really happy to have a nice way to visualize my habits.
I'd love to know if there's any way to see beyond the current 2 weeks display

Like shown in the readme screenshot, there are 14 days at any given time.
Would it be possible to have a way to navigate beyond the 14 day limits into the past? or to have ways to know how many days has a habit been done
Or for daily habits, how long have I've kept a perfect streak without missing a day.
Being able to visualize this data or perhaps export it so that it can be visualized in a .csv spreadsheet would be incredible.
Thanks a lot, really appreciate the hard work you've put into this!

Hey, I just realized that making 2 requests on the same issue was a bad idea, sorry 🙃
considering that now it's possible to go back beyond the previous 14 limits that part of the issue could be considered done.
The aspect of exporting to .csv I think it's pretty important and probably others might like to contribute to it, however since I put it in this same issue it might be hard to find the request for .csv
with that said, let me know if you'd like me to create a new issue for .csv so that you can close this one. or feel free of course to organize things as you prefer and feel free to tag me.
Thank you, love this plugin and I really hope .csv export becomes possible, since that's the only missing piece I have to move my habit tracker to this plugin
thank you!

Sorry for the delay.
Separate issues may have been better. Though, there is no need for it anymore. I think I can implement CSV export within a day. Stay tuned. 😉

@santiyounger check out v0.3.1. If there is any problem, please let me know.

hey thanks a lot for this, seems like I lost track of this notification! I just gave it a quick try! incredible work! thanks a lot for working on this
Have a great day!


Support for Stream Markers

Summary
There is a new feature on Twitch, Stream Markers. See help.twitch.tv for an overview.
The Marker can be set via chat/command /marker which we could start supporting.
Thoughts @joshuaskelly?

Lets do it! :)


[ IDEA ] - Search bar

Add a search bar compatible with multiple search engines.
Will be supporting:

Google
DuckDuckGo
Bing!

In the settings panel we can configure:

Toggle on and off the search bar
Search engine to use (see above)

We already have the chrome search bar, so it's useless


[OpOptimization]add TVM BroadcastAdd cpu gpu version

implemented CPU and GPU versions for benchmarking optimized BroadcastAdd  in TVM.

ok,thank you


diagnostic: silence outdated Travis Xcode warning.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

This doesn't add any value and stops us from testing brew doctor and others from running brew doctor on Travis CI.

ENV["CI"] may have been better? Other CI tools where users can't update Xcode set that, as does Travis.

I'd rather we add specific cases for each of those on request. We currently only really "support" Travis and Jenkins as CI platforms.

Fair enough. Just recalled similar issues when I/we implemented the Xcode version check for 10.12 and initially only implemented an opt-out for Travis, but that was a hard block rather than a simple warning so more problematic.

Appreciate the comment nonetheless, thanks. In this case it's going to be scoped just to people using brew test-bot on Circle CI rather than the much more widespread use of Homebrew itself.


arch/arm64: merge serial_pl011.c and qemu_serial.c

Summary
At present, the serial drivers qemu_serial.c and serial_pl011.c on the fvp-v8r and qemu platforms in arm64 are duplicated and need to be merged. The plan is to place them under the drivers\serial directory to create a common code module, so that both fvp-v8r and qemu can use the same code. In the future, if new platforms use pl011 serial ports, they can also be directly reused
Impact
no
Testing
ostest

@hujun260 I think you forgot to update the defconfigs after this modification, please do it:
$ ./tools/refresh.sh --silent --defaults all


Lets encrypt, unable to add Alt Name domain

Hello,
This issue happens after upgrading to latest version few days ago. I am running 19.7.5
It's unable to add more sub domain under Alt Name. In addition, there is no Alt Name domains appears when i clicked in edit mode.
Please have a look. Thank you.

Please update to 19.7.5_5.

Likely related to https://github.com/opnsense/core/commit/7a5a2712cdb8dc5cb4e2b7fae46430bd8849799f, should be fixed by updating as stated by @fichtner.


Deprecate/remove the uiClipboard directive in favor of cdkCopyToClipboard

we should kill it now that we have a proper replacement integrated in the cdk

@toxik @mhagape bump? :trollface:

we have a release branch for next major release, so it might be the right timing for this 😊


Support for ansible_ssh_private_key_file

Hi yioda,
I just copied your "user" part and changed it to "identityfile".
In Ansible the variable is called "ansible_ssh_private_key_file" and in ssh config its called "IdentityFile".
Thank you for the nice tool, it helps alot!
Greetings Tim

Hi @timd98,
Great suggestion!
I would merge this PR first, then would release a new version to Pypi as long as I have free time.
BTW,
since these hosts' information are updated from Ansible playbook, I think it should not affect the original ~/.ssh/config.
So I'm planning to change the default path to ~/.ssh/config.d/ansible.config instead of the original ~/.ssh/config.
Then include it in ~/.ssh/config by:
# Include just this config
Include config.d/ansible.config

# or include all in config.d
Include config.d/*

But some  older system seems doesn't support Include option in ssh config, I'm keeping to figure out a way to handle this,
Still, thanks for improving this project!

I've already build a new version pkg to pypi
https://pypi.org/project/ansible-inventory-to-ssh-config/1.1.0/
and here's the release note's: https://github.com/yioda/ansible-inventory-to-ssh-config/releases/tag/1.1.0


Update dependencies and devDependencies

npm v3 tries to dedupe the dependencies by default, and keeping dependencies up-to-date helps better deduplication.
I could update almost all the dependencies except for install v0.2.6, because its API seems different from the one documented on its README.

Out of curiosity, what's your need for an update? We aren't using this anymore so have no impetus to keep pushing releases.
@benjamn - not sure if you're using this for anything else but feel free to keep pushing this out if you want.

@zpao
$ npm ls commoner
<EMAIL_ADDRESS>/Users/shinnn/src/myapp
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>
Oh, though, I should not use jsx-loader since it's deprecated! I'm sorry for bothering you. Now I don't care about whether this PR will be merged or not. Feel free to close this.

https://github.com/facebook/regenerator still uses Commoner for its CLI, for what it's worth.
I should remove the https://www.npmjs.com/package/install package altogether, especially since the API Commoner was using has been deprecated/removed.


Added PoGO-Awesome

PoGO-Awesome is similar to font-awesome. With this CSS library you can easily show Pokemons or Pokemons shadows.

Hi thanks for this, please can you add this in alphabetically.
Cheers!

I will, but first I make a decent README.md for my resource, so people have a nice introduction.

Sure, will close this for now, but as soon as you are ready will re-open.


Feature requrest: setting to turn off missed golden cookies statistic

Let me turn off missed golden cookies statistic. I don't like to be able to see this statistic as it is messing with my ocd. I know my experience is not shared, but mere presence of this number is enough to cause annoyance and irritation.
I'd like a setting to turn this statistic off while keeping all the others on.

Well, this feature was originally implemented into Cookie Clicker to be then removed by the same reasons of yours. I can't talk by the development team but since this feature was re-implemented with CM, regardless of its initial reasons to be removed, I reckon this will be something of low priority to be delved into it.
I believe that implementing a turn off setting just for a particular statistic is rather arbitrary and, thus, unnecessary for most people. However, something like a "statistics hider" would something more interesting to be implemented since it would allow the user to pick what statistic suits their needs.
However, there's something one could do in to tackle your issue which is to hide such information by using a adblock's (e.g. uBlock Origin) "Element Picker Mode" feature to hide such information. I've just tried on my CC and it worked. :D

Thanks, will try

It will be fairly easy to implement this into CookieMonster.
However, I am in favor of creating a system to let the user hide certain stats without cluttering the options page. Something like @ScriptForAll describes.
Therefore, at the moment this will not be one of our higher priorities, but I will keep the issue open and return to it once we have cleared some of our backlog.


Handle use cases where the repo name/owner contains dots

Thanks a lot for this simple & awesome plugin! I've tried it with Gitea & Drone setup. (I will probably switch to woodpecker when cron PR https://github.com/woodpecker-ci/woodpecker/pull/934 is merged)
I currently get this SSL error due to my repo containing dots in the name:
Websites prove their identity via certificates. Firefox does not trust this site because it uses a certificate that is not valid for REPO_OWNER-docs.app.com-pr-21.surge.sh. The certificate is only valid for the following names: *.surge.sh, surge.sh

Error code: SSL_ERROR_BAD_CERT_DOMAIN

Would you accept a PR to handle my use case and replace dots in the name with - to pass this URL to surge?
I have 0 experience with building go apps, but I can give it a try & see how far I get if you are too busy to fix it.

That should be quite easy to fix. The corresponding line would be: https://github.com/woodpecker-ci/plugin-surge-preview/blob/3ef567c0a3e47651c56f1e38433cf7902274503d/plugin.go#L120
Some code like the following should do the trick:
pattern := regexp.MustCompile(`[^a-z0-9\-]`)
owner := pattern.ReplaceAllString(p.RepoOwner, "-")

Looking forward to your PR.


Trying to sign in crashes and gives me: ''Your app is missing support for the following URL schemes'

I'm running the following code versions on iOS 9:
Installing Google (1.1.0)
Installing GoogleAppUtilities (1.0.0)
Installing GoogleAuthUtilities (1.0.1)
Installing GoogleInterchangeUtilities (1.0.0)
Installing GoogleNetworkingUtilities (1.0.0)
Installing GoogleSignIn (2.3.1)
Installing GoogleSymbolUtilities (1.0.0)
Installing GoogleUtilities (1.1.0)

Here is the crash
 *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Your app is missing support for the following URL schemes: com.googleusercontent.apps.514797035410-uksdcup8ju2g0frgqq3f88ieqnvmr2t9'
*** First throw call stack:
(
    0   CoreFoundation                      0x0000000106d32f65 __exceptionPreprocess + 165
    1   libobjc.A.dylib                     0x000000010aa48deb objc_exception_throw + 48
    2   CoreFoundation                      0x0000000106d32e9d +[NSException raise:format:] + 205
    3   SlideMail                           0x0000000104d2d678 -[GIDSignIn signInWithOptions:] + 326
    4   SlideMail                           0x0000000104d319b8 -[GIDSignInButton pressed] + 331
    5   UIKit                               0x00000001086681fa -[UIApplication sendAction:to:from:forEvent:] + 92
    6   SlideMail                           0x0000000104b3792f -[UIApplication(EventInterceptor) heap_sendAction:to:from:forEvent:] + 337
    7   UIKit                               0x00000001087cc504 -[UIControl sendAction:to:forEvent:] + 67
    8   UIKit                               0x00000001087cc7d0 -[UIControl _sendActionsForEvents:withEvent:] + 311
    9   UIKit                               0x00000001087cb906 -[UIControl touchesEnded:withEvent:] + 601
    10  UIKit                               0x00000001086d2aa3 -[UIWindow _sendTouchesForEvent:] + 835
    11  UIKit                               0x00000001086d3691 -[UIWindow sendEvent:] + 865
    12  UIKit                               0x0000000108685752 -[UIApplication sendEvent:] + 263
    13  SlideMail                           0x0000000104b377b1 -[UIApplication(EventInterceptor) heap_sendEvent:] + 435
    14  UIKit                               0x0000000108660fcc _UIApplicationHandleEventQueue + 6693
    15  CoreFoundation                      0x0000000106c5f0a1 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 17
    16  CoreFoundation                      0x0000000106c54fcc __CFRunLoopDoSources0 + 556
    17  CoreFoundation                      0x0000000106c54483 __CFRunLoopRun + 867
    18  CoreFoundation                      0x0000000106c53e98 CFRunLoopRunSpecific + 488
    19  GraphicsServices                    0x000000010be0ead2 GSEventRunModal + 161
    20  UIKit                               0x0000000108666676 UIApplicationMain + 171
    21  SlideMail                           0x00000001048bcb22 main + 162
    22  libdyld.dylib                       0x000000010b88592d start + 1
    23  ???                                 0x0000000000000001 0x0 + 1
)
libc++abi.dylib: terminating with uncaught exception of type NSException

I added the following to my .plist and it's still crashing.
    <key>LSApplicationQueriesSchemes</key>
    <array>
        <string>com.googleusercontent.apps.514797035410-uksdcup8ju2g0frgqq3f88ieqnvmr2t9</string>
    </array>

i got the same trouble

up up

D'oh. Forgot to add the URL schemes shown at the bottom of the tutorial:
https://developers.google.com/identity/sign-in/ios/start-integrating

I have the same problem, but I am not using that scheme

I've faced the same problem. My mistake was to use the "Client ID" created in the google developer console when creating the "Web Application" login credentials and assuming they are the same to be used in iOS. You need to use the client ID in the GoogleService-Info.plist which is different!

what if i already have a GoogleService-Info.plist s for fBSDKLogin , and know am trying to make it work with GoogleSignIn??

I think this could solve your problem:

@elbuenyuz Great question, I just took a shot in the dark to get Facebook Auth and Google Auth working in the same app and it actually worked, you add two URL Schemes, both with the same value for the  REVERSED_CLIENT_ID key in the GoogleService-Info.plist (that URL value I will call RCURL).

Facebook SDK: [Facebook AppId],[RCURL]
Google SDK: [BundleIdentifier],[RCURL]

Noticed how I put that comma in bold, the url scheme entry is a key value pair and ends up in Info.plist in a particular key/value format. Here is a screenshot to help:

If anybody else had this error even after setting the URL Type on Info tab, GOOGLE_SIGNIN_URL_SCHEME on Build Settings tab and adding GoogleService-Info.plist to your project, here's a tip: when you add the plist file to your project, be sure to add to ALL freaking targets like the docs tell you to...
I had cloned a project from github and it had tvOS targets... since I'm not doing anything for apple tv, I didn't bother to check those combo boxes, nor remove those tvOS targets... and got this error on iphone simulator, which I thought was completely unrelated... it drove me crazy for two days...

If perhaps meanwhile you change your App ID and you have your API key restricted to that, don't forget to update it!
My mistake was that even that I had created a new App ID restricted to my new App ID, I had not updated it on the app URL scheme.

My issue was solved by adding this lines in "viewDidAppear"
GIDSignIn* signIn = [GIDSignIn sharedInstance];
[GlobalObj sharedInstance].delegate=self;
signIn.shouldFetchBasicProfile = YES;
signIn.clientID= @"XXXXXXXXXXXXX";
signIn.scopes = @[ @"profile", @"email" ];
signIn.delegate = self;
signIn.uiDelegate = self;

Add MAIN info.plist add new URL type

I have this problem too, and I have solved it .
The key in developer background your apply all in used in IOS , you can look at the example
clientID  = "xxxxx-xxxx.apps.googleusercontent.com"
url schemes  = "com.googleusercontent.apps.xxxx-xxxxxx

The solution for this problem is:

First you need to open you Xcode project run open ios/Runner.xcworkspace

In Xcode project find GoogleServiceInfo.plist, click and copy REVERSED_CLIENT_ID (right side)

Go to Runner -> Info-> URL Types and click to + .

Past the  REVERSED_CLIENT_ID in URL SCHEME

The solution for this problem is:
1. First you need to open you Xcode project run open ios/Runner.xcworkspace

2. In Xcode project find GoogleServiceInfo.plist, click and copy REVERSED_CLIENT_ID (right side)

1. Go to Runner -> Info-> URL Types and click to + .

2. Past the  REVERSED_CLIENT_ID in URL SCHEME

This worked for me on Flutter as well 👍

What should I do? Can you modify the support?

in my case i did'nt configure the firebase.

what worked for me was to change the webClientId key to clientID

For me I have to add these value of URL Scheme in the "Property List" not in "Code" mode of info.plist.
Why? After two hours I don't care.

In my case, I forgot to add Items for both Google and Facebook OAuth.

In my case, I forgot to add Items for both Google and Facebook OAuth.

I have already added Firebase cloud messaging. I downloaded the GoogleService-info.plist, which is needed for the Cloudmessaging. I added the google sign in for ios without using the firebase. I added the reversed client id, but i get an error that "your app is missing support for the following URL schemes"... The urlScheme is the reverse client id used by firebase and not the google sign in. How can i handle this?

The solution for this problem is:

First you need to open you Xcode project run open ios/Runner.xcworkspace
In Xcode project find GoogleServiceInfo.plist, click and copy REVERSED_CLIENT_ID (right side)

Go to Runner -> Info-> URL Types and click to + .
Past the  REVERSED_CLIENT_ID in URL SCHEME

Right! but i had issue after trying this also so make sure webClientId or iosClientId should be ClientId
GoogleSignin.configure({
      webClientId:"#id.apps.googleusercontent.com",
      ClientId: '#id.apps.googleusercontent.com',
      offlineAccess: false
    }

i also faced the same, followed all the methods once i click save (i just open it for url and client id and forgot to save it agin) in google console and rebuild its work

I had this issue for iOS, using Android Studio, so that any X-Code-related explicit troubleshooting steps are not applicable.
What I did is this:
    flutter create cscexams --org ca.tokenizer
    cd cscexams
    firebase login
    dart pub global activate flutterfire_cli
    flutterfire configure

After that, develop application, including Google Authentication (enable it in Google Console); Android works just fine, but iOS crashes with exact same message.
Explicitly adding  this section to ios/Runner/Info.plist fixed the issue:
		<key>CFBundleURLTypes</key>
		<array>
			<dict>
				<key>CFBundleTypeRole</key>
				<string>Editor</string>
				<key>CFBundleURLSchemes</key>
				<array>
					<!-- Copied from GoogleService-Info.plist key REVERSED_CLIENT_ID -->
					<string>com.googleusercontent.apps.123456-REPLACE_ME</string>
				</array>
			</dict>
		</array>

Unfortunately, flutterfire configure command doesn't do it for us.

Thanks man

thanks mwaqas01

@FuadEfendi You are a life saver, your approach worked for me. Thank you and God bless you.


is it anyway to change the chainid to other than goerli?

is it anyway to change the chainid to other than goerli?
such as change the id=5  to id=9000
could you suggest all the change I should make it happen?

@9cat sorry for the late reply. I have been flooded with notifications. Unfortunately, you cannot do this through configuration but it is sort of exposed it on the code level here - https://github.com/mateuszsokola/eth-faucet/blob/main/utils/bootstrapEthereum.ts#L9 . So if you fork this repository and change chainId to 9000 it might work.


Free clipboard respond to fix bug

Fixes #5997 (clipboard stops responding in the middle of a session)
If you can reproduce the bug you will be able to reproduce the fix.

@freerdp-bot test


Consistently use size_t while counting choices

Since we can have SIZE_T_MAX many choices, don't use plain signed
integers.

Make choices_lines unsigned to keep it consistent with
tty_{columns,lines} since commit b5fe77f ("Make tty_{columns,lines}
unsigned").

Drop (by now) redundant casts to size_t

Rework some logic in order to prevent doing subtraction on unsigned
integers since it could result in wrapping.

Rework the complex loop in print_choices() by keeping the upper bound
in the loop declaration. The other loop condition is better suited
inside the body of the loop. This convention is in general a good rule
of thumb while constructing complex loops.

Testing would be much appreciated! Especially Page-{Down,Up}.

Looks good on Debian testing & NetBSD 7.1 ...

Thanks @DBOTW, merged.


[YamlCreate] Remove white spaces at the end of each line

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

This PR fixes the issue that YamlCreate does not remove the white space before the special character when commenting out some lines in the locale manifests, leaving these comments ending with a white space, which contradicts with editorconfig rules:
https://github.com/microsoft/winget-pkgs/blob/8ce267285884b7601b93355571a648f61668ed21/.editorconfig#L9
Microsoft Reviewers: Open in CodeFlow

@stephengillie @Trenly

@ stephengillie @ Trenly

I haven't had a chance to review it, sorry.


Sample code snippet and introduction for the fslab website

Hi there, awesome work on this library!
As I will be talking about FsLab next week at .NET Conf: Focus on F#, there might be some new traffic on that page, therefore I extended the introduction parts (click on read more) of the libraries I maintain with some more introductory text and simple code snippets.
The packages page contains flips as well as a part of the fslab toolbox. If you would like to add some text/snippets, the markdown file that renders the flips card is located here (it has just been the project summary for the last months). Here is the same for Plotly.NET as reference.
I would say i am proficient enough in using Deedle to come up with some simple code myself, but i first wanted to ask if maybe a maintainer would like to control what exactly is presented on the website.
Cheers!

Thanks for the great efforts promoting Deedle via FsLab. Please go ahead with a getting-started sample. I trust your judgements. Users will be redirected here for more docs.
I'm more worried about how to maintain the current doc generation. Migrating to latest FSharp.Formatting is not trivial. Tomas put in various type specific formats for Frame and Series. It won't work with the latest version as fsdocs gets rid of doc generating scripts. And there is separate docs for C# samples. Anyway, I need to find time to work on it. If you have any spare time, feel free to take a look too. Thanks!

Cool, i added a very simple snippet.
Regarding the docs: I think as long as custom fsi evaluators are used, Deedle docs still have to use FSharp.Formatting as a library instead of a dotnet tool, but that does not mean that it can't use the new styling and functions.
Depending on the kind of formatting functions it might be worth looking into the way we solved this in Plotly.NET: Adding html formatters to the library which can then be used in both the formatting for dotnet interactive as well as the documentation with the fsdocs tool without custom evaluators. Maybe something like a ToHTMLTable member additionally to Print(). I will take a look at the current documentation after the conference and maybe we can find a way to port them with this approach.


A* for shortest path

I have added an implementation for the A* algorithm.
The implementation is very similar to the Dijkstra implementation (since Dijkstra is a specialization of A*).
The constructor of the A* shortest path takes another callback called heuristic that should return a heuristic cost estimate for the node.

The corporate contributors license agreement for<EMAIL_ADDRESS>is signed. Can one trigger the google bot to do another check?
Cheers,
Wolf

I am the author.
2016-04-25 13:48 GMT+02:00 googlebot<EMAIL_ADDRESS>
We found a Contributor License Agreement for you (the sender of this pull
request) and all commit authors, but as best as we can tell these commits
were authored by someone else. If that's the case, please add them to this
pull request and have them confirm that they're okay with these commits
being contributed to Google. If we're mistaken and you did author these
commits, just reply here to confirm.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/google/or-tools/pull/171#issuecomment-214279004

I did author the commits and I confirm that I am okay with the contribution :)

I merged it.
Thanks for the contribution.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2016-04-25 18:27 GMT+02:00 Wolf Vollprecht<EMAIL_ADDRESS>
I did author the commits and I confirm that I am okay with the
contribution :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/google/or-tools/pull/171#issuecomment-214429844


use epoll_wait() in na_sm even with timeout 0

This slightly alters the logic in na_sm_progress() so that it calls epoll_wait() (by way of hg_poll_wait()) even when the timeout is zero as long as a poll_set is present.
The reason is for performance; if the poll_wait() is not called, then na_sm must call both recvmsg() and read() in case unexpected messages are present.  The former in particular is more expensive than epoll_wait() with a zero timeout.
Output below shows the average response time difference (dropped from 3.3 usec to 2.6 usec) for a Mochi sdskv benchmark with a single process.  This example is not actually doing any na communication at all (all operations are "self" RPCs), but there is a performance difference anyway because mercury must check the active na+sm transport on every iteration.
test case:

[carns@carns-x1-7g null]$ cat baseline-64k-puts-sm.json
{
    "protocol" : "na+sm",
    "seed" : 0,
    "server" : {
        "use-progress-thread" : false,
        "rpc-thread-count" : 0,
        "database" : {
            "type" : "null",
            "name" : "benchmark-db",
            "path" : "/tmp/sdskv"
        }
    },
    "benchmarks" : [
        {
            "type" : "put",
            "repetitions" : 65536,
            "num-entries" : 1,
            "key-sizes" : [ 8, 32 ],
            "val-sizes" : [ 24, 48 ],
            "erase-on-teardown" : true
        }
    ]
}

single-process performance before change:

[carns@carns-x1-7g null]$ sdskv-benchmark baseline-64k-puts-sm.json
================ put ================
{
	"erase-on-teardown" : true,
	"key-sizes" : [ 8, 32 ],
	"num-entries" : 1,
	"repetitions" : 65536,
	"type" : "put",
	"val-sizes" : [ 24, 48 ]
}
-------------------------------------
Samples         : 65536
Average(sec)    : 0.000003345
Variance(sec^2) : 0.000000000
StdDev(sec)     : 0.000005673
Minimum(sec)    : 0.000002861
Q1(sec)         : 0.000003099
Median(sec)     : 0.000003099
Q3(sec)         : 0.000003338
Maximum(sec)    : 0.000670195

after change:

[carns@carns-x1-7g null]$ sdskv-benchmark baseline-64k-puts-sm.json
================ put ================
{
	"erase-on-teardown" : true,
	"key-sizes" : [ 8, 32 ],
	"num-entries" : 1,
	"repetitions" : 65536,
	"type" : "put",
	"val-sizes" : [ 24, 48 ]
}
-------------------------------------
Samples         : 65536
Average(sec)    : 0.000002660
Variance(sec^2) : 0.000000000
StdDev(sec)     : 0.000004737
Minimum(sec)    : 0.000002146
Q1(sec)         : 0.000002384
Median(sec)     : 0.000002384
Q3(sec)         : 0.000002623
Maximum(sec)    : 0.000395060

FWIW I was looking at this path in an attempt to figure out where the outliers (the "Maximum" field in the output).  This PR does not address that, though; still looking.

Great thanks! I have an idea about the max if you're using sm. Currently the first message to the peer will allocate the source address if it does not exist yet (see https://github.com/mercury-hpc/mercury/blob/master/src/na/na_sm.c#L2754). What we could do to optimize that is pre-allocate a pool of addresses I guess.

I guess maybe to confirm what I just said, you could check if the first message/RPC is the outlier or if it's another one?

I guess maybe to confirm what I just said, you could check if the first message/RPC is the outlier or if it's another one?

No, unfortunately they are scattered throughout the run time.  They seem to be semi-periodic, though, not totally random.


[FEATURE] : Imp of to-do-list

.We can add importance of this its significance.so user get to know about this properly. for example:
1.Enhanced Organization:
Keeps tasks organized in one place, reducing clutter and confusion.
Allows users to categorize tasks by projects, deadlines, or priority levels.
like this,Plzz assign me this issue

We can create a Seperate page for this.I want contribute to ur project.


TestNG suite with parallel DataProvider and random order of methods

I am not sure if it is a bug, but my question did not find any answer on stackoverflow for quite a long time, so I decided to post it here. Please check the following description below:
http://stackoverflow.com/questions/15265771/testng-suite-with-parallel-dataprovider-and-random-order-of-methods-how-to-ach

AFAIK currently TestNG has 3 sets of thread pools

One threadpool which is controlled by thread-count attribute of <suite|test> tag.(this is applicable only to regular methods)  and
Another threadpool which is controlled by data-provider-thread-count attribute of <suite|test> tag (this is applicable only for data driven tests) and
Another threadpool which is controlled by threadPoolSize attribute of @Test annotation (this is applicable only when you have an invocationCount of more than 1 for the @Test method)

Currently its not possible to blend all of them together such that they all work using the same common threadpool.
There's already an issue https://github.com/cbeust/testng/issues/2019 which talks about merging the thread pool for (1) and (2).
Closing this issue as a duplicate of 2019 (We could have done the other way too, but I don't think it really matters as to which becomes a duplicate of which one, as long as the requirement of the issue gets addressed.


Merge branch 'develop' into master

Description of Change
This PR contains the merge from develop to master, with the following changes:
#109: Request of permissions in the plugin
#112: Adding of permissions to plugin
cleanup and reformatting of code
@jfversluis Please remove the "-beta" suffix from the version in AppVeyor before applying this PR - this version should be our latest non-beta version. Afterwards you can add it again, as I will work on the multi-targeting. Thanks!
Issues Resolved

fixes #109
fixes #112

Platforms Affected

Android

Hm, I'll remove the tag. Maybe we should come up with something else for this. Maybe adding a tag if we want to release a version or something? This is not ideal ;)

Hmm no it isn't. Maybe just add the -beta suffix when building on develop branch, and master is always a non-beta build.
Thanks for merging!

Yeah, I thought about that. The problem is, as far as I can see in AppVeyor then it need to be two separate build definitions. The version numbers would go out of sync for the beta packages and actual ones. Not sure if it would really matter though

Maybe you could set a ${suffix} value using pretzel Powershell based on what git branch you are and add that to the version. Only works when the appveyor.yml is in the repository, though, I guess. Or maybe go with an Azure build...


Call for other Datasets

Hello, I found that there are more datasets in the paper than in this repository, I wonder if it is convenient for you to provide links to other datasets? Thank you very much if you can.

@CreaterLL here's a link to the dataset. The images weren't made available by the authors of the original paper from Bloomberg, so they had to be downloaded from Twitter. Some of the tweets were no longer available, so not all the images could be downloaded. There's a .tsv which has the content of the original tweet and the labels supplied by the dataset authors. Each row of the TSV has a tweet id field. If we were able to download the image within the tweet, they are located in images-from-serialized-tweets/{tweet_id}.jpg
https://www.dropbox.com/s/ry5hel1frlj7yxm/BloombergTwitterDSExport.tgz?dl=0


Issue 2742 new

issue#2742

Leaving the commit message as is would be confusing for anyone browsing the commit history in the future.

I agree, because funny enough, it took me some effort to figure out what the difference was between the two commits.


Message: invalid arguments Stack: cmykcs_getRgbBuffer: leboulanger.com menu

STR:

Load chrome://pdf.js/content/web/viewer.html?file=https%3A%2F%2Fwww.leboulangeronlineorder.com%2Fusercontent%2FXImages%2F%2FCafe%2520Menu.pdf
After a few seconds, the following error message pops up:

An error occurred while rendering the page.
PDF.JS Build: efd82ab Message: invalid arguments Stack: cmykcs_getRgbBuffer([object Uint8Array],8)@chrome://pdf.js/content/build/pdf.js:10712 fillRgbaBuffer([object Uint8Array],[object Array])@chrome://pdf.js/content/build/pdf.js:20390 buildPaintImageXObject([object Object],false)@chrome://pdf.js/content/build/pdf.js:11550 partialEvaluatorGetIRQueue([object Object],[object Object],[object Object],[object Array])@chrome://pdf.js/content/build/pdf.js:11686 pageGetIRQueue([object Object],[object Array])@chrome://pdf.js/content/build/pdf.js:225 wphSetupPageRequest(1)@chrome://pdf.js/content/build/pdf.js:26515 messageHandlerComObjOnMessage([object MessageEvent])@chrome://pdf.js/content/build/pdf.js:26454

@cpeterso First look in the console, there was a quite easier-to-understand error report :)
Warning: filter "JPXDecode" not supported yet
This is addressed in PR#927 (https://github.com/mozilla/pdf.js/pull/927)

Thanks @saebekassebil. I didn't think to look in the console. It seems like these error messages' locations should be switched: display "JPXDecode not supported yet" in the viewer toolbar's "More Information" dialog and then dump the stacktrace to the console. :)


Rework map and range example

Slide Iterable data types and Range Maps example has a few India specific references:

Map: https://goplay.space/#lMZT_vCRchI
Range: https://goplay.space/#S3KObilbh7H
Func
Update sample links in slides!

Please also add source files to the sample directory.
Also add the 3 embedding examples from a previous PR.


Update the lockfile

Just a maintenance lockfile update after cargo update to prune the dependabot PRs.

No longer applicable.


Returning English characters in which-key

As far as I understood, the README section on which-key contains code that should return English characters back to which-key. I have rewritten this code for myself, but still have Cyrillic in which-key.
How can I make sure that the which-key does not have Cyrillic characters?
Which-key output:

langmapper configuration:
return {
    "Wansmer/langmapper.nvim",
    lazy = false,
    priority = 1,
    config = function()
        local langmapper = require("langmapper")

        langmapper.setup({})
    end,
}

Which-key configuration:
return {
    "folke/which-key.nvim",
    enabled = true,
    dependencies = { "Wansmer/langmapper.nvim" },
    config = function()
        vim.o.timeout = true
        vim.o.timeoutlen = 300

        local lmu = require("langmapper.utils")
        local view = require("which-key.view")
        local execute = view.execute

        view.execute = function(prefix_i, mode, buf)
            prefix_i = lmu.translate_keycode(prefix_i, "default", "ru")
            execute(prefix_i, mode, buf)
        end

        require("which-key").setup()
    end,
}

Also, I didn't quite understand how to set up mapping for other keyboard layouts other than ru.
I defined the langmap option to the layout I need and called the langmapper plugin simply as langmapper.setup({}) and it seems to work.
But in the standard configuration of the plugin there is a section that configures it, it seems, only for Russian.
  layouts = {
    ---@type table Fallback layout item. Name of key is a name of language
    ru = {
      ---@type string Name of your second keyboard layout in system.
      ---It should be the same as result string of `get_current_layout_id()`
      id = 'com.apple.keylayout.RussianWin',
      ---@type string Fallback layout to translate. Should be same length as default layout
      layout = 'ФИСВУАПРШОЛДЬТЩЗЙКЫЕГМЦЧНЯБЮЖЭХЪËфисвуапршолдьтщзйкыегмцчнябюжэхъё',
      ---@type string if you need to specify default layout for this fallback layout
      default_layout = nil,
    },
  },

So I'm a little confused whether I'm using the plugin correctly.

I looked at the other issues and saw that I need to add hack_get_keymap() after calling setup() so that the which-key does not contain Cyrillic.
Now the question of keyboard layouts for other languages remains.

Now the question of keyboard layouts for other languages remains.

To add another language, you need to add a new layout to layouts (with all fields as in Russian layout), e.g:
require("langmapper").setup({
  layouts = {
    ---@type table Fallback layout item. Name of key is a name of language
    ru = { --[[layout preset]] },
    fr = { --[[layout preset]] },
    gr = { --[[layout preset]] },
  },
})

Also, don't forget to set vim.opt.langmap for the additional input method.

you need to add a new layout to layouts

I did, but I kept getting errors, so I asked about it because I thought I was doing something wrong.
Errors:

I was able to fix it by setting hack_keymap to false. This seemed to fix the problem, but I haven't used Neovim long enough after that to try all the keyboard shortcuts I use, so I can't say for sure.
Now my settings look like this.
langmapper configuration:
return {
    "Wansmer/langmapper.nvim",
    lazy = false,
    priority = 1,
    config = function()
        local langmapper = require("langmapper")

        langmapper.setup({
            hack_keymap = false,
            layouts = {
                ua = {
                    id = "ua",
                    layout = "ФИСВУАПРШОЛДЬТЩЗЙКІЕГМЦЧНЯБЮЖЄХЇʼ,фисвуапршолдьтщзйкіегмцчнябюжєхї'.",
                    default_layout = [[ABCDEFGHIJKLMNOPQRSTUVWXYZ<>:"{}~?abcdefghijklmnopqrstuvwxyz,.;'[]`/]],
                }
            },
            os = {
                Linux = {
                    get_current_layout_id = function()
                        local cmd = "gsettings"
                        if vim.fn.executable(cmd) then
                            local output = vim.fn.system(
                                cmd .. " get org.gnome.desktop.input-sources mru-sources | sed -r \"s/\\S*\\s'([^']+).*/\\1/\""
                            )
                            return output
                        end
                    end,
                },
            },
        })

        langmapper.hack_get_keymap()
    end,
}

options.lua:
local function escape(str)
    local escape_chars = [[;,."|\]]
    return vim.fn.escape(str, escape_chars)
end

local en = [[abcdefghijklmnopqrstuvwxyz,.;'[]`/]]
local ua = [[фисвуапршолдьтщзйкіегмцчнябюжєхї'.]]
local en_shift = [[ABCDEFGHIJKLMNOPQRSTUVWXYZ<>:"{}~?]]
local ua_shift = [[ФИСВУАПРШОЛДЬТЩЗЙКІЕГМЦЧНЯБЮЖЄХЇʼ,]]
vim.opt.langmap = vim.fn.join({ escape(ua_shift) .. ";" .. escape(en_shift), escape(ua) .. ";" .. escape(en) }, ",")

I also added require("langmapper").automapping({ global = true, buffer = true }) to the end of init.lua.


Having to do Promise<FrozenArray<DOMString>> is a pain. <xmp> helps, but code highlighters don't have great support for it.

<xmp>
  [Exposed=(Window,Worker)]
  interface BackgroundFetchManager {
    Promise<BackgroundFetchRegistration> fetch(DOMString id, (RequestInfo or sequence<RequestInfo>) requests, optional BackgroundFetchOptions options);
    Promise<BackgroundFetchRegistration?> get(DOMString id);
    Promise<FrozenArray<DOMString>> getIds();
  };
</xmp>

Whereas

Your highlighter will apply arbitrary/configurable highlighting to script elements?

I'm not sure, but it won't highlight it as HTML. Similar to what Github is doing above.

(I'm going ahead and adding this now, but also I just noticed: you can just do <xmp class=idl>...</xmp>, you know? No need to have a wrapping
.)

Ohh that's cool! Doesn't fix the highlighting issue, but good to know!
Thanks for making this change.

Thanks!


Can't drag

When i drag it drops the items and it's not possible to craft on survival mode. Pls fix it

Hi @4ykhxn, this issue is a duplicate of #21. If you have any additional information you wish to provide about this topic, please post it in #21.
Thanks.


fix(background): better detection of out-of-funds on Pay now button

refactor(background): use Array methods for cleaner code

Context

Closes https://github.com/interledger/web-monetization-extension/issues/387
Part of https://github.com/interledger/web-monetization-extension/issues/147

Changes proposed in this pull request

Use isOutOfFundsError (extracted from https://github.com/interledger/web-monetization-extension/pull/426)
Refactor code a bit.

Extension builds preview

Name
Link

Latest commit
0cbd8b4709ac496d61ec2ec6a8df8f04651b1595

Latest job logs
Run #9955375640

DownloadDownload


spectest index command standard outputs the index markdown file path after generated it.

Is your feature request related to a problem? Please describe.
After generating the index markdown, spectest index does not output anything. I think it is difficult for the user to understand what happened with this specification. Therefore, the generated index file path will be outputted to the standard output.
Describe the solution you'd like
If markdown.GenerateIndex() completes successfully, it will output the index file path to the standard output. If an error occurs, it will directly return the error to the higher-level function.
https://github.com/go-spectest/spectest/blob/26de85c47a740f596035376cf4b84e09ac9c7fc2/cmd/spectest/sub/index.go#L59-L62

Done https://github.com/go-spectest/spectest/pull/210


fix check load timeout before close wait

Proposed changes
close wait timeout = load timeout - elapsed time
If remaining time is not positive, return timeout
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Select2 single select inside modal in safari does not work

Select2 single select dropdown does not work inside modal in Safari. Clicking on the selection does not show the selection in the display. I am also using select2-bootstrap.css. It works perfectly fine in other browsers.

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


ConnectionRefusedError

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Hello @Butzz01, Welcome to colab-ssh community.
Thank your submitting this issue.
We highly recommend that supply more information than the title "ConnectionRefusedError". Can you please share with us the snippet that resulted in such error.
Thank you


Support button role on links that do not have href

Having a valid role on links without href will help the screen readers understand their purpose correctly.

@neeleshb looking at the PR this issue happens on CodeLens, right?
I think your fix makes sense and we can merge it in once Tyler is happy with the quick pick changes.


Add optgroup support

Copied optgroup code from aldeed/meteor-autoform.
Tested that select2 selectbox works with optgroups. Further testing maybe needed, but everything seems to be working.

Thanks!


Ops recipes

Created four ops-recipes for the prometheus-harbor alerts.

@giantswarm/team-honeybadger Can you PTAL here? Just someone who does oncall can give it read from your POV

@piontec can you re-review on monday? We need to start getting these merged.


Fix jade bugs

Adds missing spaces introduced by Jade conversion
Fixes a small typo
Closes #163

Looks good to me, going to ask Jorge to give it a once over when he gets here though to be safe.


Duplicating many_many relationships looses the extra fields in 3.6

Affected Version
SilverStripe 3.6.5
Description
When you duplicate a DataObject with a many_many relationship with extra fields defined for the relationship extra fields are lost due to how DataObject::duplicateManyManyRelations() and DataObject::duplicateRelations() function. In short when the new items are added to the many_many relationship the extra fields are not included or even retrieved when DataObject::duplicateRelations() adds the item to the list. This issue also seems to affect 4.0 but appears to have been addressed in 4.1.
Steps to Reproduce

Create a DataObject with a many_many relationship and many_many_extraFields for that relationship to another object
Add a few items to that relationship with the extra fields populated
Call duplicate on your original object which duplicates the relationship's items
The duplicated items in the ManyManyList will have lost all of their extra field values

If it's a bugfix we could fix it in a change/patch

I think it would be, it looks like the intent was to have this functionality work as expected. However it does seem to be a bit of a borderline API change especially if it's been this way throughout 3.x. I'm curious what others think on that though.

So I've come up with a change that would address this in 3.6 it's literally almost a complete backport (copy paste job) of the DataObject::duplicateManyManyRelation() method from 4.1. See webbuilders-group/silverstripe-framework@aa29d06c52a1e86c89ba57705a2e92b69a0cf86c if there's agreement that this should be addressed in 3.6 then I'll open a pull request. I've also amended the duplicate many many test to include checking that this is working. It should also be easy to backport this further to 3.5 if requested.
Also it's worth noting that looking at the core for 4.0 it look's like it has a very similar issue though the implementation is a bit different it still has the same issue (see here).