Space Geek example isn't working for me (tried fix from Issues too)

folks,
I'm having lots of problems getting the Alexa Space Geek example working.  I've tried to follow the example exactly, given that both the Lambda dashboard and Alexa developer portal have changed a bit since the instructions were first written.  That said, so far as I can tell I've created the Lambda function with the same values as laid out in the instructions.  Same with the Alexa skill interface.  I think all the critical elements are there, some of the screens were sequenced differently.
For the Lambda function I've tried both submitting the "code" via .zip file and as inline code.  I saved off the zip file with the View Raw option.  When I used the Upload Zip button I navigated to the zip file and selected it, but I just never saw any indication the file was actually uploaded.  I Saved the Lambda function and the system indicated it was saved successfully.  However when I went to look at the code (using the Code tab) the dashboard indicated the code base was too big to view... hmmm...
After I configured the skill, when I tried testing using the Alexa Developer portal Service Simulator in TEXT mode I received the error "The remote endpoint could not be called, or the response it returned was invalid."  I found the thread on issues and tried the JSON submittal, replacing all the various fields with "id" with the values generated by the TEXT test.  I even edited the timestamp so it'd be more up to date.  When I submitted the JSON lines using the Service Simulator I get the error "Unable to call remote endpoint because request is incorrect. Please verify all the fields in the request object".
I was careful to cut the ARN from the Lambda function 'generator' and paste it in the Alexa skills interface.  I didn't edit ANY of the source code.  Any assistance is greatly appreciated.  Please bear with me ahead of time as I'm not a current developer, just an old PDP-11 assembly language programmer .... from A long time ago in a galaxy far, far away....
Cheers!

Have you checked your CloudWatch logs for that Lambda function? What do they tell you? You can find them by navigating to your Lambda function and then select "View logs in Cloudwatch". Each time your Lambda function runs it generates a new log file.

Thanks for responding and pointing me to Cloudwatch.
First things first, I figured out the Upload function... I wasn't selecting Save or Save&Test.  I seen now the system digests the zip file.  But I'm still getting an error...  As a preface I assume I'm supposed to be using the Lambda test function with the JSON (editing the xxx.id's and NOT the Developer Portal Service Simulator, correct?  Here's what I get when I test and what follows is part of the log from Cloudwatch.  It seems like the zip file isn't being located...
Execution result: failed(logs)
The area below shows the result returned by your function execution. Learn more about returning results from your function.
{
"errorMessage": "Cannot find module '/var/task/index'",
"errorType": "Error",
"stackTrace": [
"Module.require (module.js:353:17)",
"require (internal/module.js:12:17)"
]
}
CloudWatch...
Time (UTC +00:00)
Message
2016-12-05
No older events found at the moment. Retry.
?
16:13:38
START RequestId: c80b683c-bb05-11e6-bb1d-cf9ccd6555a0 Version: $LATEST
START RequestId: c80b683c-bb05-11e6-bb1d-cf9ccd6555a0 Version: $LATEST
?
16:13:38
Unable to import module 'index': Error at Function.Module._resolveFilename (module.js:325:15) at Function.Module._load (module.js:276:25) at Module.require (module.js:353:17) at require (internal/module.js:12:17)
Unable to import module 'index': Error
at Function.Module._resolveFilename (module.js:325:15)
at Function.Module._load (module.js:276:25)
at Module.require (module.js:353:17)
at require (internal/module.js:12:17)
?
16:13:38
END RequestId: c80b683c-bb05-11e6-bb1d-cf9ccd6555a0
END RequestId: c80b683c-bb05-11e6-bb1d-cf9ccd6555a0

Make sure that the zip file you are uploading only contains the files, not
the upper level directories.  It should just contain 2 files.
Noelle
On Mon, Dec 5, 2016 at 11:23 AM, davekgithub<EMAIL_ADDRESS>wrote:

Thanks for responding and pointing me to Cloudwatch.
First things first, I figured out the Upload function... I wasn't
selecting Save or Save&Test. I seen now the system digests the zip file.
But I'm still getting an error... As a preface I assume I'm supposed to be
using the Lambda test function with the JSON (editing the xxx.id's and
NOT the Developer Portal Service Simulator, correct? Here's what I get when
I test and what follows is part of the log from Cloudwatch. It seems like
the zip file isn't being located...
Execution result: failed(logs)
The area below shows the result returned by your function execution. Learn
more about returning results from your function.
{
"errorMessage": "Cannot find module '/var/task/index'",
"errorType": "Error",
"stackTrace": [
"Module.require (module.js:353:17)",
"require (internal/module.js:12:17)"
]
}
CloudWatch...
Time (UTC +00:00)
Message
2016-12-05
No older events found at the moment. Retry.
?
16:13:38
START RequestId: c80b683c-bb05-11e6-bb1d-cf9ccd6555a0 Version: $LATEST
START RequestId: c80b683c-bb05-11e6-bb1d-cf9ccd6555a0 Version: $LATEST
?
16:13:38
Unable to import module 'index': Error at Function.Module._resolveFilename
(module.js:325:15) at Function.Module._load (module.js:276:25) at
Module.require (module.js:353:17) at require (internal/module.js:12:17)
Unable to import module 'index': Error
at Function.Module._resolveFilename (module.js:325:15)
at Function.Module._load (module.js:276:25)
at Module.require (module.js:353:17)
at require (internal/module.js:12:17)
?
16:13:38
END RequestId: c80b683c-bb05-11e6-bb1d-cf9ccd6555a0
END RequestId: c80b683c-bb05-11e6-bb1d-cf9ccd6555a0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/amzn/alexa-skills-kit-js/issues/53#issuecomment-264899797,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABq0KVhYjmoRbxTzO-jxqfxBOISK_56Fks5rFDqVgaJpZM4LC7Yw
.

--
~~
Noelle Jakusz LaCharite
Skype: noelle.jakusz
Phone:<PHONE_NUMBER>
http://about.me/noellelacharite
If you can't feed a hundred people, then feed just one.
Mother Teresa
http://www.brainyquote.com/quotes/quotes/m/mothertere105649.html

Good news, it's working!  Man did I EVER get beat up by Github (new to me...).  I couldn't find "THE" zip file with just the two .js sources.  The files in the project folder are ....
amzn/alexa-skills-kit-js
FactSkillTemplate	Update FactSkillTemplate.zip	5 months ago
speechAssets	Update skills to match currently released version	a year ago
src	Update HowTo, Fact and Trivia templates.	6 months ago
README.md	Tidy markdown headings for compatibility with atom's markdown-preview	6 months ago
The FactSkilTemplate.zip had several folders (one for ?Mac I think, which I completely don't understand), a folder with the AlexaSkill.js file and then index.js, and a .pdf of instructions.  I wound up downloading this .zip file then navigating to the two .js files and created my own zipfile.  Not really a pain but geez the instructions made it seem like, hey just get the unnamed zip file, download with the view raw function, and you are good to go.  Lost lots of time not even getting into the darned stadium, much less get up to the batter's box....  enough of my sour grapes.
It's working, so thank you very much.  Now I'll edit the source to make it more in keeping with what I want to do and will then (if that works), add slots.  So thanks to all!

Awesome. You good to close this issue?

I believe so but I do have lots of questions and I'm not sure this issue thread would be correct forum (no pun intended).  I really, really appreciate the fact that someone (or several) took the time to put the sample skill together with the instructions, they're just a little out of date... )  I'd be more than glad to  contribute, but I'm not sure how.   Also I flat out struggled with Github and wound up using a rather contorted process to create a usable .zip file of source code from the zip file that had several folders of source and the instructional .pdf.  Apparently you cannot just copy source into AWS Lamda for this template to work...(I don't know that as an absolute fact, but when I copied source 'as is' it didn't work for me.  So if another dim witted person like me stumbles along they'll struggle to understand why following the instructions to download "the" zipfile, things won't work.  I also didn't know that Amazon 'stuff' doesn't seem to 'like' MS Internet Explorer.  I'm sure this is probably known to 99.9999% of the rest of the world, but I didn't know that and apparently Internet Explorer has issues with Amazon 'stuff'.
So long post, yes the issue can be closed.  Would you all have a suggestion for a proper forum to ask questions about Alexa 'stuff'? Thanks much, I really appreciate it.  .. this doesn't look at all like PDP-11 assembler...

@davekgithub Go ahead and raise another issue if it's related to the SDK. If not, there's always the developer forums for Alexa. If you don't get any love over there feel free to ping me directly.

thanks kindly.  It would seem I have lots and lots of ground to try to cover.  The abstraction offered by Node.js (and I'm sure any other java script paradigms) is foreign to me, after dealing with assembly years ago.  I'll look for some tutorials and/or books and see if I can scare up a basic understanding of how things work.  I'm not trying to become a developer, I just would like to understand a bit more.  I was looking over the code of the Alexa skill and "index" this morning and yikes! it's strange to me.
Thanks again.
  From: Harrison Jones<EMAIL_ADDRESS>
To: amzn/alexa-skills-kit-js<EMAIL_ADDRESS>Cc: davekgithub<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Sent: Tuesday, December 6, 2016 12:11 PM
Subject: Re: [amzn/alexa-skills-kit-js] Space Geek example isn't working for me (tried fix from Issues too) (#53)
@davekgithub Go ahead and raise another issue if it's related to the SDK. If not, there's always the developer forums for Alexa. If you don't get any love over there feel free to ping me directly.—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


[FLINK-24077][hbase2/HBaseConnectorITCase] fix flaky tests backport to 1.13

What is the purpose of the change
there are multiple failing tests during CI only happened for hbase2. The potential reason might be some race conditions while running the CI on azure. This PR adds some additional checks and explicitly method calls to make sure the result has been fetched before doing the assertions. Local checks haven been done multiple times and none failing test happened.
Brief change log

add check of row count after insert
use MiniClusterWithClientResource as a ClassRule

Verifying this change
This change is a trivial rework / test case improvement.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)

@AHeise there were some conflicts while doing the cp. Would you please to double check, thanks.


Update README.md

add '>' to func parameter declaration

Thanks for contribution. You have to sign the CLA(Contributor License Agreement) before we can merge your Pull Request.


search obs with 1 object

obs search via GET /observations.json with query params
usually returns a JSON array with the objects
when query yields only 1 result, it returns the object directly
we think it should return the array with a single object inside so the JSON data structure doesn't change when it deprecates to 1 object

Need more context here. What's the actual URL you're querying, what are the parameters, etc. When I request http://www.inaturalist.org/observations.json?taxon_id=106718&user_id=kueda, which has one result, it's an array, and when I request http://www.inaturalist.org/observations.json?taxon_id=106718, there are two results, also in an array.

Please re-open if you can provide the requested info.


Fix an issue with a null value on a JSON object when calling llJson2List.

Added Test support for the Phlox scripting Engine including a Mock for the Engine. Added JsonTests for lsl script engine tests around JSON handling.  Added an initial failing test cases and some positive cases for llJson2List handling an json object with a null value for an element.  Added code to get the failing test passing.

Looks great. The nested check is critical and included in the commit, so these changes look great and open the door to future test cases with the unit testing wrapper also included.  PR ready for merge.


Add i18n support for format times

Hi, I enjoy so much this module and I would like to pull request this new feature for you
Initially ms supports just en and pt-br languages, and you can use the default mode too:
ms('1d') // Default support (English)

ms('1 dia', 'pt-br') // PT-BR support (Brazilian Portuguese)

Interesting. Please rebase with master! 😊

No changes for years. Closing...


Support for Nuget.config with repositoryPath

If we have a nuget.config file with a repositoryPath for a directory than is not named "packages" the nuget package references are not recognized as nuget package references and you are not presented with the option to change them to project references.
Here is a Nuget.Config that I have for my solution:

.
.
I see in the ProductModel.cs (LoadReferences method) the Path of the vsReference (e.g. does it contain "\packages\" in the path) is used to determine if a reference is from a nuget package.  If a directory other than "packages" (the default) is used then you don't get the option to switch those references to project references.  A simple option would be to let the user specify a regular express or at least a string to use.  Another more complicated option would be to search for a nuget.config and see if it has a repositoryPath.

I don't think nuget.config appeared in original comment:
<configuration>
<config&gt
<add key="repositorypath" value="......\NuGetPackages" /&gt
</config&gt

I agree with @danielcbock, same situation here (using a shared packages directory for multiple solution via a custom repositorypath in a NuGet.config in some parent directory).

+1


only start servers after a manager became a leader

Signed-off-by: kevin xu<EMAIL_ADDRESS>

Current coverage is 32.07% (diff: 0.00%)

Merging #56 into master will increase coverage by 0.05%

@@             master        #56   diff @@
==========================================
  Files            29         29
  Lines          3385       3389     +4
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1084       1087     +3
- Misses         2102       2106     +4
+ Partials        199        196     -3

Powered by Codecov. Last update b65606b...be33e12

+1

not good, see comments.

close this


converting Chinese string to bytes and back is inconsistent

Steps to Reproduce

write a unit test:

String msg = '中文的也应该可以';
expect(String.fromCharCodes(Uint8List.fromList(msg.codeUnits)), msg);

run the test with flutter test

see the error:

Expected: '中文的也应该可以'
    Actual: '-
              _åïå'
     Which: is different.
            Expected: 中文的也应该可以
              Actual: -
                       _åïå
                      ^
             Differ at offset 0

Expected results: it should be the same string as before and the test should pass
Actual results: it became random char list

Hi @tianhaoz95, what you actually want to use is a Uint16List, not a Uint8List.
If you look at the documentation for String.codeUnits, you'll see that it returns a UTF-16 representation. In your snippet the Uint8List will end up truncating the characters incorrectly - though it may work for strings where all the codePoints are small enough

Also: if you want a UTF-8 representation, there is utf8 codec in dart:convert. The following should work as expected:
String msg = '中文的也应该可以';
expect(String.fromCharCodes(Uint8List.fromList(utf8.encode(msg))), msg);

Thanks @jonahwilliams!
@tianhaoz95 this should solve your problem, closing the issue
if you disagree please write in the comments and I will reopen it
Thank you


fix depedencies

: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 100.00%. Comparing base (100d0f2) to head (2da761d).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #38   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          129       129
  Branches        16        16
=========================================
  Hits           129       129

Flag
Coverage Δ

100.00% <ø> (?)

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.


How to change for different timeframe

How would one update the config for Hourly or Daily ?
On Daily I seem to be getting errors.
On hourly I have in the config:
'chop_val': 200,
'variable': "Hourly",
'dilations': ((1, 12), (12, 24)),
'state_hsize': 50,
'seasonality': 24,
'input_size': 24,
'output_size': 48,
'level_variability_penalty': 50

and I get the error
input.size(-1) must be equal to input_size. Expected 30, got 25

On Daily (default) I have:
'chop_val': 200,
'variable': "Daily",
'dilations': ((1, 7), (14, 28)),
'state_hsize': 50,
'seasonality': 7,
'input_size': 7,
'output_size': 14,
'level_variability_penalty': 50

and I get the error:
ValueError: Item wrong length 4226 instead of 4227.

Any advice on how to proceed would be appreciated

I can reproduce the bug, its a bug in the DataLoader I believe.

In data_loading.py, I've changed read_file function to this:
def read_file(file_location):
    df = pd.read_csv(file_location)
    df = df.iloc[:, 1:]
    lst = df.stack().groupby(level=0).apply(list).tolist()
    return np.array(lst)

and it works for me.
I didn't noticed configs for: Weekly and Hourly in config.py

For hourly M4 data there was only one category. Change
'num_of_categories': 6,

to
'num_of_categories': 1,

in config.py


Implementing the Pong Game

=> The Pong game should contain these functions

control the game with help of keyboard keys.
player got 10 chances in the game
player can pause and restart the game

I would like to work on this issue

Can you please assign me this task?


Table view, deleting columns

clicking 'X' in table view next to variable name should remove variable from table, but that X is missing

This icon is invisible, but close functionality works. We may just be using the wrong color scheme?

needed to add character to the anchor tag after the button generated.
the library is expecting a png file that isn't there.


feat: Add java.util.Optional support

So how best to handle building with Java 7? Perhaps build with Java 8 but with Java 7 compatibility?

You can't have both! You need to have one or the other!
Until now, it used to require jdk7 for compilation, but sundrio itslef worked with all java version (even 5).


[EG] next-pylint

/azp run python - eventgrid - tests

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-eventgrid


Pact helper is loaded twice, causing duplicate tests.

On a rails project, when running bundle exec rake pact:verify on the provider.
I get duplicates of "reading pact file at xxx" as well as duplicate tests.
I expect this to only occur once.
spec/service_consumers/pact_helper.rb
require 'spec_helper'
require 'pact/provider/rspec'

Pact.service_provider "Scrubbed backend" do
  honours_pact_with "Scrubbed service" do
    pact_uri 'spec/pacts/somepact.json'
  end
end

Dir[Rails.root.join("spec/service_consumers/**/*.rb")].each { |f| require f }

spec/service_consumers/gramercy_provider_states.rb
Pact.provider_states_for "Scrubbed Service" do
  provider_state "we dont care about navigation items" do
    set_up do
      store = Spree::Store.find_by(code: "scrubbed")
      item = FactoryGirl.create(:navigation_item, store: store)
    end
  end
end

Verify output:
SPEC_OPTS='' /home/scrub/.rubies/ruby-2.1.6/bin/ruby -S pact verify --pact-helper /home/scrub/projects/scrub/spree-scrub/spec/service_consumers/pact_helper.rb
[Spree WARNING] Missing migrations.
[Spree WARNING] add_update_reason_to_store_credit_events from spree is missing.
[Spree WARNING] create_spree_store_credit_update_reasons from spree is missing.
[Spree WARNING] remove_state_lock_version_from_order from spree is missing.
[Spree WARNING] remove_unnecessary_indexes from spree is missing.
[Spree WARNING] seed_store_credit_update_reasons from spree is missing.
[Spree WARNING] Run `bundle exec rake railties:install:migrations` to get them.

Reading pact at spec/pacts/scrub.json
Reading pact at spec/pacts/scrub.json

Randomized with seed 48242

Verifying a pact between scrub and scrub
  Given we dont care about navigation items
    a request for navigation items
      with GET /api/navigation_items
        returns a response which
          has status code 200

Verifying a pact between scrub and scrub
  Given we dont care about navigation items
    a request for navigation items
      with GET /api/navigation_items
        returns a response which
          has status code 200

Deprecation Warnings:

Using `any_instance` from rspec-mocks' old `:should` syntax without explicitly enabling the syntax is deprecated. Use the new `:expect` syntax or explicitly enable `:should` instead. Called from /home/rscrub/projects/scrub/spree-scrub/spec/support/paperclip_support.rb:3:in `stub_paperclip'.

If you need more of the backtrace for any of these deprecations to
identify where to make the necessary changes, you can configure
`config.raise_errors_for_deprecations!`, and it will turn the
deprecation warnings into errors, giving you the full backtrace.

1 deprecation warning total

1 interaction, 0 failures

Randomized with seed 48242

noglob bundle exec rake pact:verify  14.34s user 2.38s system 90% cpu 18.534 total

Any ideas? Both caller stacktraces come from the gem itself. I dont think its being reloaded on our end.

I've seen this happen before when the provider_states file and the pact_helper file both required each other. Can you double check for me that Dir[Rails.root.join("spec/service_consumers/**/*.rb")].each { |f| require f } isn't causing your pact_helper to be reloaded? What are the call stacks?

Yes, sorry, I had no idea how I missed that. Thank you for your help.

No worries, done it before myself!


elements_are does not support containers that don't iterate over borrows

Example:
struct IntContainer(Vec<i32>);
impl<'a> IntoIterator for &'a IntContainer {
    type IntoIter = std::iter::Copied<std::slice::Iter<'a, i32>>;
    type Item = i32;
    fn into_iter(self) -> Self::IntoIter {
        self.0.iter().copied()
    }
}

// I expect to be able to write:
let c = IntContainer(vec![1,2,3]);
assert_that!(c, elements_are![eq(1), eq(2), eq(3)]);

This has been fixed in #367


Add yahoo/kafka-manager

@ElfoLiNk Have you checked with the maintainers of that repository if they are OK with adding it here? I'd like to avoid flooding projects with pull requests without the consent of their maintainers.

I'm going to close this for now but feel free to reopen it when the maintainers of kafka-manager decide that using Scala Steward is a good idea.


GH-1: Make compatible with Hibernate Validator

Fixes https://github.com/spring-cloud-stream-app-starters/redis-pubsub/issues/1
Looks like the latest Hibernate Validator polled transitively by
Spring Boot (6.0.10) has some race condition around methods
introspection and our isKey() (or isQueue(), isTopic()) might be
erased from the BeanMetaDataImpl.unconstrainedExecutables because of
the key (queue, topic) property extrapolation via
ReflectionHelper.PROPERTY_ACCESSOR_PREFIXES

Rename all the is...() in the RedisSinkProperties into the
is...Present() to overcome a Hibernate Validator issue

That's a tricky one. Glad you were able to figure it out!

Looks good. Glad this issue is resolved. Thank you @artembilan.
PR merged.


Allow better support for diagonal sampling of an Image element

First of all let me say that I'm really impressed by what you've done and I'm really looking forward to working with it more in the future. I came across a problem, which I guess should be properly stated as a feature request.
As specified in my Stack Overflow question (http://stackoverflow.com/questions/31457922/using-holoviews-sample-function-in-a-diagonal-segment) it is very intuitive to take a slice out of an Image element using the sample method when it is along one of the axes of the Image. It is very unintuitive, however, to do the same for a diagonal line. Ideally, I would like to specify to point p1 and p2 and sample the Image along the line connecting them.
The way I do it now, I need to decide to how many points I want to divide the path, then create a list of pairs of points between the two and supply this to Image. The result is a Table - and getting the list of points out of the table as a Curve element only works when the Table is one dimensional - i.e. when one of the kdims is constant along the line. Otherwise, using Table.to.curve returns one kdim as a function of the other, i.e., a straight line.

There's two components to this,  and secondly somehow combining two dimensions into one:
(1) Supporting sampling between two arbitrary points. I'm not 100% sure it should be supported by default, there's all kinds of special types of sampling you might want to do, e.g. linear or log sampling, which is exactly why you can provide raw sampling coordinates. Sampling between two points might be a sufficiently common use case to support by default, what does everyone think?
(2) There should be an easy way to combine two dimensions into one. In your particular case the problem is that if you perform diagonal sampling then both the x- and y-coordinates are changing for each sample, when plotting this as a Curve you have to find some way to somehow reduce 2D coordinates onto a 1D axis. What policy would you expect here? The easiest thing to do would be to allow simply using an index as the x-coordinate, the more complicated but more correct thing to do would be to give the diagonal sample a new scale. I'd need to think more about it, but how would you expect it to behave?

(1) I agree to your analysis. Support sampling between two arbitrary points would be great for me, but I understand if this is not supported by default.
(2) I think the most generic is using the index as an x-coordinate. Another possibility would be to use dr=sqrt(dx2 + dy2) as the increment between two samples, but I'm not sure about how this scale should start. If this is implemented, perhaps use a default index and a keyword argument that can modify this to dr?

If we supported sampling along an arbitrary line, seems like we'd also need to support reducing/collapsing along an arbitrary axis as well, and those will only be valid operations in a very few of the domains in which Image is used.  If we added such support, I'd think we'd only want to do it in a (new) superclass of Image, something that requires x and y to have the same dimensions at least, so that the result is meaningful in any sense.
But really I think this functionality is outside the purview of HoloViews, and instead should be done by some other package.   For instance, you could easily bring the image into Imagen, rotate it in any direction you like, translate it anywhere, then bring it back into HoloViews.  You can then happily slice, reduce, etc. along your new x and y axes.  Imagen uses precisely the same coordinate systems as HoloViews does, so this conversion should be very straightforward, and completely meaningful since Imagen only works with cases where the two spatial dimensions are commensurate.

I think we've decided this isn't within the scope of HoloViews. I'll close the issue.


Include a DECR benchmark covering embstring object

Reference:
https://github.com/redis/redis/pull/12250
Simple run:
$ redis-benchmarks-spec-client-runner --test memtier_benchmark-1Mkeys-string-decr.yml  --override-memtier-test-time 10
2023-06-18 10:14:24 INFO redis-benchmarks-spec-client-runner (solely client) 0.1.71
2023-06-18 10:14:24 INFO Using test-suites folder dir /home/fco/.local/lib/python3.10/site-packages/redis_benchmarks_specification/test-suites
2023-06-18 10:14:24 INFO Running specific benchmark in 1 files: ['/home/fco/.local/lib/python3.10/site-packages/redis_benchmarks_specification/test-suites/memtier_benchmark-1Mkeys-string-decr.yml']
2023-06-18 10:14:24 INFO There are a total of 1 test-suites in folder /home/fco/.local/lib/python3.10/site-packages/redis_benchmarks_specification/test-suites
2023-06-18 10:14:24 INFO Overriding memtier benchmark --test-time to 10 seconds
2023-06-18 10:14:24 INFO Running the benchmark specs.
2023-06-18 10:14:24 INFO Loading default specifications from file: /home/fco/.local/lib/python3.10/site-packages/redis_benchmarks_specification/test-suites/defaults.yml
2023-06-18 10:14:24 INFO Found RedisTimeSeries default metrics specification. Will include the following metrics on all benchmarks $."BEST RUN RESULTS".Totals."Ops/sec" $."BEST RUN RESULTS".Totals."Latency" $."BEST RUN RESULTS".Totals."Misses/sec" $."BEST RUN RESULTS".Totals."Percentile Latencies"."p50.00" $."WORST RUN RESULTS".Totals."Ops/sec" $."WORST RUN RESULTS".Totals."Latency" $."WORST RUN RESULTS".Totals."Misses/sec" $."WORST RUN RESULTS".Totals."Percentile Latencies"."p50.00" $."AGGREGATED AVERAGE RESULTS (3 runs)".Totals."Ops/sec" $."AGGREGATED AVERAGE RESULTS (3 runs)".Totals."Latency" $."AGGREGATED AVERAGE RESULTS (3 runs)".Totals."Misses/sec" $."AGGREGATED AVERAGE RESULTS (5 runs)".Totals."Percentile Latencies"."p50.00" $."ALL STATS".Totals."Ops/sec" $."ALL STATS".Totals."Latency" $."ALL STATS".Totals."Misses/sec" $."ALL STATS".Totals."Percentile Latencies"."p50.00"
2023-06-18 10:14:24 INFO Found RedisTimeSeries default time metric specification. Will use the following JSON path to retrieve the test time $."ALL STATS".Runtime."Start time"
2023-06-18 10:14:24 INFO Benchmark required memory:<PHONE_NUMBER> Bytes
2023-06-18 10:14:24 INFO  Using total system memory as max<PHONE_NUMBER>6
2023-06-18 10:14:24 INFO Resetting commmandstats for shard 0
2023-06-18 10:14:24 INFO Test memtier_benchmark-1Mkeys-string-get-100B priority (98) is within the priority limit [0,100000]
2023-06-18 10:14:24 INFO Using docker image redislabs/memtier_benchmark:edge as benchmark PRELOAD image (cpuset=0,1,2,3) with the following args: /usr/local/bin/memtier_benchmark --port 6379 --server localhost --json-out-file oss-standalone-2023-06-18-09-14-24-NA-preload__memtier_benchmark-1Mkeys-string-get-100B.json  --command "SET __key__<PHONE_NUMBER>854775807" --command-key-pattern="P" -n 5000 --key-minimum=1 --key-maximum 1000000 -c 50 -t 4 --hide-histogram
2023-06-18 10:14:28 INFO Preload duration 4 secs.
2023-06-18 10:14:28 INFO Tool 4 seconds to load data. Output b'4         Threads\n50        Connections per thread\n5000      Requests per client\n\n\nALL STATS\n==================================================================================================\nType         Ops/sec    Avg. Latency     p50 Latency     p99 Latency   p99.9 Latency       KB/sec \n--------------------------------------------------------------------------------------------------\nSets       271621.77         0.72978         0.69500         1.45500         4.79900     17212.14 \nTotals     271621.77         0.72978         0.69500         1.45500         4.79900     17212.14 \n'
2023-06-18 10:14:28 INFO Benchmark used memory at start of benchmark: 1g
2023-06-18 10:14:28 INFO Checking if there is a keyspace check being enforced
2023-06-18 10:14:28 INFO Ensuring keyspace length requirement = 1000000 is met.
2023-06-18 10:14:28 INFO The total numbers of keys in setup matches the expected spec: 1000000==1000000
2023-06-18 10:14:28 INFO Will store benchmark json output to local file oss-standalone-2023-06-18-09-14-28-NA-memtier_benchmark-1Mkeys-string-get-100B.json
2023-06-18 10:14:28 INFO Using docker image redislabs/memtier_benchmark:edge as benchmark client image (cpuset=0,1,2,3) with the following args: /usr/local/bin/memtier_benchmark --port 6379 --server localhost --json-out-file oss-standalone-2023-06-18-09-14-28-NA-memtier_benchmark-1Mkeys-string-get-100B.json --test-time=10 --command "DECR __key__" --command-key-pattern="R" --key-minimum=1 --key-maximum 1000000 -c 4 -t 4 --hide-histogram
2023-06-18 10:14:39 INFO Benchmark duration 11 secs.
2023-06-18 10:14:39 WARNING Benchmark duration of 11 secs is bellow the considered minimum duration for a stable run (60 secs).
2023-06-18 10:14:39 INFO Printing client tool stdout output
2023-06-18 10:14:39 INFO Benchmark used memory at end of benchmark: 1g
2023-06-18 10:14:39 INFO Reading results json from /home/fco/tmpm5txmixj/oss-standalone-2023-06-18-09-14-28-NA-memtier_benchmark-1Mkeys-string-get-100B.json
# Results for memtier_benchmark-1Mkeys-string-get-100B test-case on oss-standalone topology
|                 Metric JSON Path                 |Metric Value|
|--------------------------------------------------|-----------:|
|"ALL STATS".Totals."Ops/sec"                      |  250742.070|
|"ALL STATS".Totals."Latency"                      |       0.064|
|"ALL STATS".Totals."Misses/sec"                   |       0.000|
|"ALL STATS".Totals."Percentile Latencies"."p50.00"|       0.063|
2023-06-18 10:14:39 INFO Using datapoint_time_ms:<PHONE_NUMBER>434
2023-06-18 10:14:39 ERROR Requested to push data to RedisTimeSeries but no exporter definition was found. Missing "exporter" config.
2023-06-18 10:14:39 INFO Collecting memory metrics
2023-06-18 10:14:39 INFO Adding a total of 35 server side metrics collected at the end of benchmark
2023-06-18 10:14:39 INFO Collecting commandstat metrics
2023-06-18 10:14:39 INFO Adding a total of 20 server side metrics collected at the end of benchmark
2023-06-18 10:14:39 INFO Tearing down setup
2023-06-18 10:14:39 INFO Removing temporary client dir /home/fco/tmpm5txmixj
# Results for entire test-suite
|               Test Name                |                 Metric JSON Path                 |Metric Value|
|----------------------------------------|--------------------------------------------------|-----------:|
|memtier_benchmark-1Mkeys-string-get-100B|"ALL STATS".Totals."Ops/sec"                      |  250742.070|
|memtier_benchmark-1Mkeys-string-get-100B|"ALL STATS".Totals."Latency"                      |       0.064|
|memtier_benchmark-1Mkeys-string-get-100B|"ALL STATS".Totals."Misses/sec"                   |       0.000|
|memtier_benchmark-1Mkeys-string-get-100B|"ALL STATS".Totals."Percentile Latencies"."p50.00"|       0.063|

Codecov Report
Patch and project coverage have no change.

Comparison is base (ead8bdd) 56.71% compared to head (30a2899) 56.71%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #237   +/-   ##
=======================================
  Coverage   56.71%   56.71%
=======================================
  Files          24       24
  Lines        2234     2234
=======================================
  Hits         1267     1267
  Misses        967      967

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add cancelTimeout to cancel the query timeout

Codecov Report

Merging #560 into dev will decrease coverage by <.01%.
The diff coverage is 23.68%.

@@             Coverage Diff              @@
##                dev     #560      +/-   ##
============================================
- Coverage     46.45%   46.45%   -0.01%
- Complexity     2211     2212       +1
============================================
  Files           108      108
  Lines         25321    25356      +35
  Branches       4175     4185      +10
============================================
+ Hits          11764    11778      +14
- Misses        11538    11551      +13
- Partials       2019     2027       +8

Flag
Coverage Δ
Complexity Δ

#JDBC41
46.27% <23.68%> (+0.08%)
2201 <1> (+1)
:arrow_up:

#JDBC42
46.32% <23.68%> (-0.05%)
2208 <1> (+2)

Impacted Files
Coverage Δ
Complexity Δ

...om/microsoft/sqlserver/jdbc/SQLServerResource.java
100% <ø> (ø)
4 <0> (ø)
:arrow_down:

.../microsoft/sqlserver/jdbc/SQLServerDataSource.java
45.5% <0%> (-0.49%)
66 <0> (ø)

.../com/microsoft/sqlserver/jdbc/SQLServerDriver.java
77.34% <100%> (+0.12%)
25 <0> (ø)
:arrow_down:

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
45.6% <20%> (-0.16%)
278 <1> (+1)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
53.51% <23.52%> (+0.18%)
0 <0> (ø)
:arrow_down:

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.45% <0%> (-0.32%)
239% <0%> (-1%)

...ncurrentlinkedhashmap/ConcurrentLinkedHashMap.java
39.39% <0%> (-0.22%)
44% <0%> (-1%)

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
32.73% <0%> (-0.05%)
241% <0%> (+1%)

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
59.75% <0%> (+0.1%)
134% <0%> (+1%)
:arrow_up:

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
62.99% <0%> (+0.12%)
0% <0%> (ø)
:arrow_down:

... and 1 more

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 a940fbc...3452091. Read the comment docs.

Closing this PR and new PR #674 extended from this one.


1 Satoshi Missing After Every Payment

This is done using three nodes (alice, bob, charlie) on a local simnet and should be reproducible. I am not sure if this a bug, or is intentional but not accurately presented to the user.
We start with three nodes: alice, bob and charlie. Alice opens a channel with bob, and bob opens a channel with charlie:
$ lncli-alice openchannel 03ab7d85f11f2429b94ce29c7b09b1ae38bb8fa05995b06d42d193e6d08f7f79a5 18688
{
        "funding_txid": "67ed36000b8609d58a4bc976f8905dd877d5b6aa127c53d90b00cbb109cb7c64"
}

$ lncli-bob openchannel 02785eca55d76465aa6f18dc6d7fd6dba2fc7aa47e232543aa17019c77bf909d58 18688
{
        "funding_txid": "193f040ed4fbf32ae98a3de2705b0d951524b1fb89c364b6e80f043beb7e19bb"
}

We mine a few blocks to fully open the channels
$ btcctl generate 4

Now we can see both channels from bob's node:
$ lncli-bob listchannels
{
    "channels": [
        {
            "active": true,
            "remote_pubkey": "0237b82dfb5bab416142108b011cbd354550a9c7e16aa674cc3d213f0e5b4b2d1b",
            "channel_point": "67ed36000b8609d58a4bc976f8905dd877d5b6aa127c53d90b00cbb109cb7c64:0",
            "chan_id": "1905453651001344",
            "capacity": "18688",
            "local_balance": "0",
            "remote_balance": "10000",
            "commit_fee": "8688",
            "commit_weight": "552",
            "fee_per_kw": "12000",
            "unsettled_balance": "0",
            "total_satoshis_sent": "0",
            "total_satoshis_received": "0",
            "num_updates": "0",
            "pending_htlcs": [
            ],
            "csv_delay": 4
        },
        {
            "active": true,
            "remote_pubkey": "02785eca55d76465aa6f18dc6d7fd6dba2fc7aa47e232543aa17019c77bf909d58",
            "channel_point": "193f040ed4fbf32ae98a3de2705b0d951524b1fb89c364b6e80f043beb7e19bb:0",
            "chan_id": "1905453651066880",
            "capacity": "18688",
            "local_balance": "10000",
            "remote_balance": "0",
            "commit_fee": "8688",
            "commit_weight": "600",
            "fee_per_kw": "12000",
            "unsettled_balance": "0",
            "total_satoshis_sent": "0",
            "total_satoshis_received": "0",
            "num_updates": "0",
            "pending_htlcs": [
            ],
            "csv_delay": 4
        }
    ]
}

Channel 1 (between alice and bob) has 10000 satoshis on alice's side and 0 satoshis on bob's side. The total sum is 10000 satoshis.
Channel 2 (between bob and charlie) has 10000 satoshis on bob's side and 0 satoshis on charlie's side. The total sum is 10000 satoshis.
Charlie generates an invoice for 1000 satoshis to give to alice:
$ lncli-charlie addinvoice 1000
{
        "r_hash": "4061dffc0ceec9088fc9367523500a40bd61c2029d68e24a440c4fc37e94f782",
        "pay_req": "lnsb10u1pdzk0lxpp5gpsallqvamys3r7fxe6jx5q2gz7krsszn45wyjjyp38uxl5577pqdqqcqzyswlzxmnh8up0krllnhncm0348692jcw4ynk9uuynd8tcxcx58rpn9xwqkz9ct8qmrpazhuc98ppyrsl24xvndffgfjheqn7z6v2gtnjgp9j4wyk"
}

Alice pays this invoice:
$ lncli-alice payinvoice lnsb10u1pdzk0lxpp5gpsallqvamys3r7fxe6jx5q2gz7krsszn45wyjjyp38uxl5577pqdqqcqzyswlzxmnh8up0krllnhncm0348692jcw4ynk9uuynd8tcxcx58rpn9xwqkz9ct8qmrpazhuc98ppyrsl24xvndffgfjheqn7z6v2gtnjgp9j4wyk
{
        "payment_error": "",
        "payment_preimage": "58c270f16d8d331f82a86cb66d07e54f7ad28c0e63e946d65342301f48ddfaed",
        "payment_route": {
                "total_time_lock": 2024,
                "total_fees": 1,
                "total_amt": 1001,
                "hops": [
                        {
                                "chan_id":<PHONE_NUMBER>001344,
                                "chan_capacity": 18688,
                                "amt_to_forward": 1000,
                                "fee": 1,
                                "expiry": 1880
                        },
                        {
                                "chan_id":<PHONE_NUMBER>066880,
                                "chan_capacity": 18688,
                                "amt_to_forward": 1000,
                                "expiry": 1880
                        }
                ]
        }
}

We can see that the payment was successful and made a total of two hops. The total fees are 1 satoshi and this was deducted in the first channel.
However, let us now examine the current state of the channels:
$ lncli-bob listchannels
{
    "channels": [
        {
            "active": true,
            "remote_pubkey": "0237b82dfb5bab416142108b011cbd354550a9c7e16aa674cc3d213f0e5b4b2d1b",
            "channel_point": "67ed36000b8609d58a4bc976f8905dd877d5b6aa127c53d90b00cbb109cb7c64:0",
            "chan_id": "1905453651001344",
            "capacity": "18688",
            "local_balance": "1001",
            "remote_balance": "8998",
            "commit_fee": "8688",
            "commit_weight": "724",
            "fee_per_kw": "12000",
            "unsettled_balance": "0",
            "total_satoshis_sent": "0",
            "total_satoshis_received": "1001",
            "num_updates": "2",
            "pending_htlcs": [
            ],
            "csv_delay": 4
        },
        {
            "active": true,
            "remote_pubkey": "02785eca55d76465aa6f18dc6d7fd6dba2fc7aa47e232543aa17019c77bf909d58",
            "channel_point": "193f040ed4fbf32ae98a3de2705b0d951524b1fb89c364b6e80f043beb7e19bb:0",
            "chan_id": "1905453651066880",
            "capacity": "18688",
            "local_balance": "9000",
            "remote_balance": "1000",
            "commit_fee": "8688",
            "commit_weight": "724",
            "fee_per_kw": "12000",
            "unsettled_balance": "0",
            "total_satoshis_sent": "1000",
            "total_satoshis_received": "0",
            "num_updates": "2",
            "pending_htlcs": [
            ],
            "csv_delay": 4
        }
    ]
}

Channel 2 (between bob and charlie) has 9000 satoshis on bob's side and 1000 satoshis on charlie's side. The total sum is 10000 satoshis.
Channel 1 (between alice and bob) has 8998 satoshis on alice's side and 1001 satoshis on bob's side. 1001 comprising of the 1000 satoshi payment to charlie plus the 1 satoshi fee as shown in the payment output previously.
The total sum is 9999 satoshis. There is one satoshi missing from this channel that is completely unaccounted for. In alice's payment output, it shows the total amount sent as 1001 satoshis. Hence a user would expect the new balance on alice's side of channel 1 to be 10000-1001 = 8999 satoshis. However, we are seeing one less, 8998 satoshis. It seems as if this satoshi has disappeared.

Even closing this channel and examining the closing transaction shows one satoshi missing:
$ lncli-bob closechannel 67ed36000b8609d58a4bc976f8905dd877d5b6aa127c53d90b00cbb109cb7c64
{
        "closing_txid": "eeb75f0b6caf3e67714cc44c3325160fc3e712f7a87be14091938e461a2088b1"
}

$ btcctl decoderawtransaction `btcctl getrawtransaction eeb75f0b6caf3e67714cc44c3325160fc3e712f7a87be14091938e461a2088b1`
{
  "txid": "eeb75f0b6caf3e67714cc44c3325160fc3e712f7a87be14091938e461a2088b1",
  "version": 2,
  "locktime": 0,
  "vin": [
    {
      "txid": "67ed36000b8609d58a4bc976f8905dd877d5b6aa127c53d90b00cbb109cb7c64",
      "vout": 0,
      "scriptSig": {
        "asm": "",
        "hex": ""
      },
      "txinwitness": [
        "",
        "3045022100e1b628393d85d2137e0c452e699b50dc109f4edee13bdebb96486efe80fb79bd02201c0c8b0a53b14fd6127677e4e1a355a67d80aea5289d5a5a462da6807f3a783101",
        "3045022100c48ca69ce3824d95da45eb63c12fc69fccb845d4f4d0530b4b48116ec60b47900220363e4e5486c947c75a0e70e401ff6ef466ac342179b77abff038e073c326edf701",
        "5221023f27e8373703af11f73cc90def123c5db5d49981d7f7b0317c54c6be68b18938210394862ce2fb51a556fa5ea8f09e643d0ca072d3cbd7287d8fe817c1eaea172f2352ae"
      ],
      "sequence":<PHONE_NUMBER>
    }
  ],
  "vout": [
    {
      "value": 0.00001001,
      "n": 0,
      "scriptPubKey": {
        "asm": "0 5bdd389e23aecb69b1b263971abdcdef0c0ad732",
        "hex": "00145bdd389e23aecb69b1b263971abdcdef0c0ad732",
        "reqSigs": 1,
        "type": "witness_v0_keyhash",
        "addresses": [
          "sb1qt0wn383r4m9knvdjvwt340wdauxq44ejwy5v6n"
        ]
      }
    },
    {
      "value": 0.00008998,
      "n": 1,
      "scriptPubKey": {
        "asm": "0 d8c46d6d2f3ee49ddaf79c34a10d743e960b6084",
        "hex": "0014d8c46d6d2f3ee49ddaf79c34a10d743e960b6084",
        "reqSigs": 1,
        "type": "witness_v0_keyhash",
        "addresses": [
          "sb1qmrzx6mf08mjfmkhhns62zrt586tqkcyy0rzkue"
        ]
      }
    }
  ]
}

At any given point, we may be sending an additional satoshi to miner's fees. This is a result of the internal usage of milli-satoshis and our policy of always rounding down.

If the mining fee has increased as a result of the updated state, I would expect the commit_fee on the channel to reflect this.

It doesn't as this isn't an "explicit" fee. It's an artefact of the usage of mSAT within the protocol.

Note that further flows which create whole satoshis fee amounts in either direction can cause that 1 SAT to be re-credited.

Sorry, you're internally representing currency at a greater precision that Bitcoin supports, rounding it and not making the discrepancy visible to the user?
This is on the same scale of error as storing currency as a double and then hand waving away the inevitable problems with accurately storing values.
It's not acceptable for a financial protocol to 'round off' and drop currency like this. It's deeply disturbing frankly.

This is on the same scale of error as storing currency as a double and then hand waving away the inevitable problems with accurately storing values.

Not at all, we are using fixed point arithmetic here, no doubles involved. The error is bounded to a single satoshi. That single satoshi may end up adding to the commitment transaction's miner fee. Alternatively, an additional payment flows may end up restoring that satoshi.
If you would like to continue this discussion @philipwhiuk, I recommend you make an issue in the spec's repo: https://github.com/lightningnetwork/lightning-rfc/issues

That single satoshi may end up
an additional payment flows may end up restoring that satoshi

Could we get more certainty?

One last comment: the fee paid within the protocol isn't 1 SAT, it's actually 1001 mSAT. So Alice ends up with 8998999 mSAT (8998.999 SAT). Bob similarly ends up with  1001001 mSAT (1001.001 SAT). 8998.999 + 1001.001  = 10000.0 SAT. However, we can't express that fractional amount (totaling 1 SAT across both commitment transactions) so it goes to miner fees.
A commit will be pushed shortly to detect this and show the proper commitment fee. Thanks for the initial report! I can understand how this may have freaked out users, as without knowledge of the underlying protocol it looks like 1 satoshi is missing. 🙇
Post-fix:
⛰ lncli listchannels
{
    "channels": [
        {
            "active": true,
            "remote_pubkey": "03a459d48a84a6909402824da2a80f95d9fc80c8dd06dcfb1c0cfbf28f8101d0c4",
            "channel_point": "4f8d04670b3096c650e4ccd8ce54b3072e7e1fd3c3a897967291e8cf74b315ee:0",
            "chan_id": "1695446930096128",
            "capacity": "18688",
            "local_balance": "7997",
            "remote_balance": "2002",
            "commit_fee": "8689",
            "commit_weight": "724",
            "fee_per_kw": "12000",
            "unsettled_balance": "0",
            "total_satoshis_sent": "2002",
            "total_satoshis_received": "0",
            "num_updates": "4",
            "pending_htlcs": [
            ]
        }
    ]
}

Could we get more certainty?

Continuing from my example above, if Bob sends Alice 1 mSAT, then both parties will now have a whole number of satoshis. This may have been a multi-hop payment, or a direct payment.


⚠ SkipTheTrailers has degraded performance

In a4a862e, SkipTheTrailers ($STT) experienced degraded performance:

HTTP code: 200
Response time: 1847 ms

Resolved: SkipTheTrailers performance has improved in 6cb9f08.


Release 2.10.12

Summary | Résumé

https://github.com/cds-snc/notification-terraform/pull/1304
https://github.com/cds-snc/notification-terraform/pull/1303
https://github.com/cds-snc/notification-terraform/pull/1305
https://github.com/cds-snc/notification-terraform/pull/1302
https://github.com/cds-snc/notification-terraform/pull/1301
https://github.com/cds-snc/notification-terraform/pull/1299
https://github.com/cds-snc/notification-terraform/pull/1300
https://github.com/cds-snc/notification-terraform/pull/1298
https://github.com/cds-snc/notification-terraform/pull/1295

Release Instructions | Instructions pour le déploiement
None.
Reviewer checklist | Liste de vérification du réviseur

[ ] This PR does not break existing functionality.
[ ] This PR does not violate GCNotify's privacy policies.
[ ] This PR does not raise new security concerns. Refer to our GC Notify Risk Register document on our Google drive.
[ ] This PR does not significantly alter performance.
[ ] Additional required documentation resulting of these changes is covered (such as the README, setup instructions, a related ADR or the technical documentation).

⚠ If boxes cannot be checked off before merging the PR, they should be moved to the "Release Instructions" section with appropriate steps required to verify before release. For example, changes to celery code may require tests on staging to verify that performance has not been affected.

Closing to include a fix in a new release.


[Toolchain] Add set default toolchain command

This commit adds set default toolchain command and icon.
ONE-vscode-DCO-1.0-Signed-off-by: Jiyoung Yun<EMAIL_ADDRESS>Draft PR: #854

This icon will be shown like this:

Is it not yet showing the image?

Is it not yet showing the image?

I don't understand what you meant. You can see the check icon if there is an installed toolchain in Toolchain View.

@jyoungyun answered offline

The function is not implemented but it can show its icon!


[WIP] Try to speed up Target python-side mapping access.

Summary
Exploratory follow-up PR to #12292, attempts to speedup gate_map access in python by caching each access instance to not convert more than once.
Known issues

The way InstructionProperties is stored in the Target (using BaseInstructionProperties) on rust does not contain the calibration attribute. Which causes errors.

Details and comments
WIP.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

146 of 148 (98.65%)  changed or added relevant lines in 4 files are covered.
320 unchanged lines in 12 files lost coverage.
Overall coverage increased (+0.09%) to 89.794%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

crates/accelerate/src/target_transpiler/mod.rs
80
82
97.56%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
2
93.23%

crates/circuit/src/imports.rs
3
77.78%

qiskit/circuit/library/standard_gates/u.py
3
93.07%

qiskit/qasm3/exporter.py
4
95.46%

qiskit/circuit/commutation_checker.py
5
96.99%

qiskit/circuit/_utils.py
6
92.0%

crates/qasm2/src/parse.rs
6
97.61%

crates/accelerate/src/target_transpiler/mod.rs
9
84.26%

crates/circuit/src/parameter_table.rs
12
94.55%

crates/circuit/src/circuit_data.rs
35
95.14%

Totals

Change from base Build<PHONE_NUMBER>6:
0.09%

Covered Lines:
67375

Relevant Lines:
75033

💛  - Coveralls


replace unrecognised vendor dependency in go mod

What type of PR is this?
/kind failing-test
What does does this PR do / why we need it:
make goget-ginkgo fails on PSI
++ make goget-ginkgo
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>unrecognized import path "vbom.ml/util" (https fetch: Get https://vbom.ml/util?go-get=1: dial tcp: lookup vbom.ml on <IP_ADDRESS>:53: no such host)
# https://go-review.googlesource.com/c/go/+/198438/
GOFLAGS='' go get<EMAIL_ADDRESS>go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>unrecognized import path "vbom.ml/util" (https fetch: Get https://vbom.ml/util?go-get=1: dial tcp: lookup vbom.ml on <IP_ADDRESS>:53: no such host)
make: *** [Makefile:109: goget-ginkgo] Error 1

vbom.ml/util repo no longer exists and supports are available github.com/fvbommel/util .
Which issue(s) this PR fixes:
Fixes NA
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:
Detailed log: http://jenkins.psiodo.net:8080/job/Periodics/job/odo-periodic-test/61/console

Travis fails due to https://github.com/openshift/odo/issues/4008 and gets fixed in https://github.com/openshift/odo/pull/4010

/lgtm
/approve


Errors in cquery when I open project folder in VS Code.

Hi there,
I tried cquery for the first time and had these errors when I open the project in VS Code :
(   0.020s) [querydb      ]        project.cc:66    | Failed to normalize /home/manish/Desktop/new/build [Error - 12:31:40 PM] Request textDocument/codeAction failed.   Message: Unable to find file /home/manish/Desktop/new/testCquery.cpp   Code: -32603  [Error - 12:31:40 PM] Request textDocument/documentLink failed.   Message: Unable to find file /home/manish/Desktop/new/testCquery.cpp   Code: -32603  [Error - 12:31:40 PM] Request textDocument/codeLens failed.   Message: Unable to find file /home/manish/Desktop/new/testCquery.cpp   Code: -32603
I couldn't understand what is causing these errors. When I open the project folder the Failed to normalize error occurs and after that when I open the project file the rest three errors occur and cquery doesn't work. But as soon as I make a change in the project file cquery starts working as I can see Code Lens and change in Code Formatting.
Please tell me what is causing this or what did I do wrong.
Thanks,
Manish

Whatever the issue was for me, after updating to the latest -git 15 mins ago it's gone.


Automate Docstrings for new Python scripts

Create a Python script that will add necessary docstring for users. Best practices that ought to be followed while writing scripts, such as the appropriate use of comments, variable names, and docstrings is automated by this script. This script takes input from user and creates a .py file with the above details.

Script execution was successful.


Throw an error when trying to create a group with tree depth < 16 or > 32

Description
The Bandada off-chain groups support tree depth >= 16 and <=32. It would be nice to add a check to the api sdk so that devs receive a clearer error when they try to create a group with a tree depth that is not supported.
The error could be: The tree depth must be between 16 and 32.
It would also be nice to return a better error message in the API when creating a group.
(Feedback from @sripwoud)

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi, there I am Luis Sanchez, a 3 years experienced FrontEnd dev, with some experience whitin the backend development (Nest.js and FastAPI). Glad to contribute!!!
How I plan on tackling this issue
This is my aproach to add the validation and throw the error:
async createGroup(
groupCreationDetails: GroupCreationDetails,
apiKey: string
): Promise {
// Validate treeDepth
if (groupCreationDetails.treeDepth < 16 || groupCreationDetails.treeDepth > 32) {
throw new Error('The tree depth must be between 16 and 32.');
}
const groups = await this.createGroups(
    [groupCreationDetails],
    apiKey
);

return groups[0];

}

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I would like to work on this issue
could you assign this to me
How I plan on tackling this issue
?

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I have experience working with Solidity for smart contract development and TypeScript for building backend and API services. In previous projects, I've dealt with validating user inputs, error handling, and integrating off-chain services with blockchain networks. My expertise in building decentralized applications (dApps) has equipped me with the ability to design clear and informative error messaging for APIs, ensuring developers have a smooth experience. This background enables me to contribute effectively to improving the Bandada API SDK by enforcing constraints and providing meaningful feedback to developers.
How I plan on tackling this issue
To address the issue of tree depth validation, I would start by updating the API SDK in TypeScript to include a check for the treeDepth parameter when a developer attempts to create a group. Before any API call is made, the SDK can validate that the provided treeDepth falls between 16 and 32. If the value is outside this range, I would throw an error with the message: "The tree depth must be between 16 and 32." This ensures that the developer receives immediate feedback, improving the user experience.
For the API side, I would modify the endpoint that handles group creation to return a more descriptive error message if an invalid treeDepth is provided. By performing the same validation server-side, we ensure consistency. If the value is outside the valid range, the API would respond with a 400 status code and the error message: "The tree depth must be between 16 and 32." This way, even if the SDK is bypassed, the API still enforces the rule effectively

Hey @sanieni6! Sounds great! Do you want me to assign this issue to you?

Hi @vplasencia, I'd love that!!!

Hey @sanieni6 done. Thank you. Feel free to ask any questions.

hi there @vplasencia  @sripwoud this is ready to be reviewed.

Thank you very much @sanieni6! I will take a look as soon as possible.


Discard current baremetal support

@akuker Correct me if I am wrong: You indicated that you are not interested in baremetal support as it is right now, but favor a different solution. In this case the existing code would profit from all those baremetal ifdefs to be removed. This would definitely improve code maintainability.

Agreed. Let's just make that executive decision now.... Baremetal support can be removed.
If someone really needs baremetal support, they can go with the GIMONS version of the code.
Maybe someday we add it back in... but even then I would probably use a FreeRTOS or something instead of just bare metal.

Cool! I will address this issue next, because any other changes will profit from the code having been cleaned up first.

Resolved with https://github.com/akuker/RASCSI/commit/3c5a0003a43fd3efd16c14e23918e8865a4f77cc


Error on executing ":Zone dvd"

All the other options seem to work as expected , however the dvd throws this error:
Error executing Lua callback: /.vim/plugged/zone.nvim/lua/zone/styles/dvd.lua:33: attempt to index upvalue 'local_opts' (a nil value)
stack traceback:
/.vim/plugged/zone.nvim/lua/zone/styles/dvd.lua:33: in function 'start'
/home//.vim/plugged/zone.nvim/plugin/zone.lua:3: in function </home//.vim/plugged/zone.nvim/plugin/zone.lua:2>

Heyyo @Geostartico ,
Can u update the plugin and check again? 😅

Now it works with no issue :)


[db] Add index for content hash column on files table

If we use PostgreSQL database and we set the query timeout to 2 minutes the
database cleanup will be timed out when the server tries to remove unused
files and the database is to big.
The reason is that the database sequentially scanned the database to look
for foreign key constraints. To solve this problem we add index to the
content hash column on files table.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- web/server/codechecker_server/migrations/report/versions/2185167f8568_content_hash_index_for_files.py  1

See the complete overview on Codacy

@gyorb I tried to store some xerces results to the database without index and with index and I got the following results:
# WITH INDEX:
stored results (55703 KB) to run 'xerces' in 21.08 seconds.

real	0m31,304s
user	0m14,410s
sys	0m0,756s

# WITHOUT INDEX:
stored results (55703 KB) to run 'xerces' in 21.26 seconds.

real	0m31,861s
user	0m14,508s
sys	0m0,861s


Copyright notice

Codecov Report

Merging #76 (332cf2d) into main (4670e0a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage   31.57%   31.57%
=======================================
  Files          15       15
  Lines         627      627
=======================================
  Hits          198      198
  Misses        429      429

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 4670e0a...332cf2d. Read the comment docs.

:tada: This PR is included in version 1.4.11 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Delete sprites in middle of the timeline, gives error when importing into unity

When you import an animation that have a deleted sprite in the timeline, when you try to import that animation in Unity, it give you errors.
For example:
An animation have some sprite in frame 0, and you delete that sprite in frame 10, when you try to import that animation, it give you the error

Can you please provide more info? What exactly do you mean by deleting sprites? And what's the error you're getting?

In Spriter you can remove a sprite in the timeline and that creates a new key, when you import an animation who has removed sprites from the Hierarchy, when you try to import this animation in Unity, it give you errors

Fixed in 1.4.0


the range() type was improperly associated with sliders

To associate an int or float to a slider widget, we use range type like this:
max_token: range(100, 200, 20)=140

But this is misleading. In the notation above, max_token's type is hinted as range while actually we want it to be an int, as indicated by the default value 140 and the nature of the slider widget.
Hence, we should do something like this:
@dataclass
class IntRange:
    min: int
    max: int
    step: int

# Then
def foo(max_token: typing.Annotated[int, IntRange(-10, 5, 1)] = 6):
    pass

I thought about it again. No, the original annotation is not misleading. range(100, 200, 20) defines the scope of the variable max_token. It does not mean max_token is a range.


field is_valid - ValidationError exception not working as expected

In the docs you say that raising ValidationError in field's is_valid should work, but it doesn't
Try this in a Form
        @staticmethod
        def fields__foo__is_valid(form, field, parsed_data, **kwargs):
            raise ValidationError(
                _("This actually raises an exception instead of returning form with invalid field")
            )
            # raise ValueError(
            #     _("This also raises an exception")
            # )
            return False, "Invalid!"  # this works

in a model field (FileField) I have validators=[validate_file_extension] and I'm able to upload files which aren't allowed in the validator. I suppose the auto thingy takes validators from model fields, so it's related to this problem (?)
that's kinda dangerous


Use position to get type of hovered part and deduce resolved property

Suppose the user has the following .http script:
var a = 10
GET 'https://httpbin.org/get'
? a = "{{a}}"

If the cursor position is 45, and the user is hovering over the query, they want to know the resolved query value. Pass all relevant details to dotextensions to process the query and return the final value.

[ ] single query (even though http request can have multiple querys, result single query)
[ ] url (result only url)
[ ] body (not sure if possible, but if possible try to result)

[ ] json
[ ] text

[ ] variable interpolation

Use: https://github.com/cedric05/dothttp/commit/5caf6e9dc1fb2289f62f215ad318122a7d109720#diff-7593f2b0409a031802e10359464ba9a2ff62be6a8b3f9495b5f3d5061afeca1cR340
for resolving


Add test to check if a status effect refreshes are handled appropriately when a status effect has, in fact, already expired

Eg, If status effect X ends at time t, and X is applied again at t+1, make sure the application of the X is done correctly. Careless calculation of the time remaining on a status effect can lead to incorrect end times for the 2nd use of X.

Fixed.


quick and dirt string => TypeInfo for column and type properties

related to https://github.com/gocql/gocql/pull/1726

🤦♂ sorry this draft was not intended to be shared on upstream repository


x/crypto/acme/autocert: serve self-signed cert for localhost?

Idea inspired by a mailing list post on golang-nuts,
What if the autocert package [optionally?] could serve a self-signed cert for localhost connections?
It can look at SNI "localhost" and/or the connection addr being a loopback address.
Might be nice for testing / consistency.
/cc @x1ddos

+1, it should make a self-signed certificate for the "localhost" or "127.x.x.x" names, as it's clear a real cert will never be obtained for them.
Note though, it should not look at the Listen address, as binding to <IP_ADDRESS> and then redirecting with iptables is totally fine and common.

A very nice idea. Maybe also add *.local to the list of "local" SNI?
And, of course ::1 to the list of loopback addr. I don't see why "it should not look at the Listen address".

I'm not keen on putting unicast DNS and mDNS stuff into one basket for now.
Just FYI: https://tools.ietf.org/html/draft-west-let-localhost-be-localhost

Any news on this? I wanted tis feature for golang since I found it in Python's flask and its werkzeug internal server I'm not saying it should be done because some library of python does. Just interested if something like this exists in Go lands

I'd personally not want this to be default behavior, although opt-in would be great. Self signed certs will be rejected by many other client libraries. If this was default, when testing multiple local services communicating together I'd have errors by default that would somewhat obscure the underlying issue.

I think the caddy server does this by default when started locally. Maybe there's a library somewhere already. I hadn't the opportunity to drill into the code yet.


Button Labels Fixup

Steps to reproduce:

Open to main screen (first page)

Expected behaviour: What did you expect to happen when performing these steps?
Button labels fit on buttons.
Observed behaviour: What actually happened and why is it a problem? If applicable, consider adding screenshots to help explain.
Some button labels don't fit horizontally.
Device information: Include your device operating system and version.
Android 7.0 mobile phone.
Screenshots:

True, depending on the size of your screen, the names of your teas, and the orientation of your device (portrait or landscape) there may not always be sufficient space to render 100% of the first 3 tea buttons. You may need to scroll.
This is one of the main reasons why the tea button list is horizontally scrollable. Considering the infinite variety of phone sizes, tea names, OS font sizes, and other variables, it would be quite a challenge to ensure your teas always fit on the screen.

I wanted to close the loop on this a little bit better. The next version of Cuppa will clip the tea buttons at the edge of your phone's screen, rather than at the edge of the margin. I think this will make it more obvious that it's a scrolling list.
I hope that helps! :)


[victory-chart] Doesn't support hoisted packages

I believe this to be related to it's usage of builder. I assume repos using lerna will be affected by this as well. When using yarn workspaces, a combination the build scripts of victory-chart and victory-native ends up placing victory-native in the root node_modules even though it's only required by one of my packages. As a result yarn resolution correctly places react-native-svg in the node_modules of that specific package, but victory-* get places at the root. Since those depend on react-native-svg this makes the build fail.

@blackxored -- Can you explain this more to me? Are you using git-based versions of victory-* repos? If you're using ones from the npm registry, builder is completely removed from everything and we rewrite the package.json to remove postinstall tasks and to move builder and archetypes to devDependencies. E.g<EMAIL_ADDRESS>

I am indeed using the git repos. I will try to upgrade to the version in npm and report.

It doesn't make a difference. Trying to use the ones from npm, it still ends up in the root node_modules.

Can you paste a sample of the node_modules tree hierarchy of relevant packages that (1) you expect and (2) actually get that causes a problem?
Also, if possible can you create a minimal repository with install + error reproduction steps so that I can jump in and see the issue firsthand? Thanks!

Using Yarn workspaces:
package.json -> only devDependencies
platforms/native/package.json -> depends on victory-native ^0.16.0.
After running yarn:
Expected:
platforms/native/node_modules/victory-*
platforms/native/node_modules/react-native-svg
...

Actual:
node_modules/victory-*
platforms/native/node_modules/react-native-svg

I had a workaround it with a postinstall on native's package.json but it's dirty.
Might be an issue with Yarn itself, however, your package is the only one that exhibits these and I'm depending on quite a few.

Can you give me a minimal repository + actual steps to reproduce this? I've never used yarn workspaces and I'm not quite following what's going on...


Added next closest functionality

Added functionality to go to next closest stream
There's a small bug with stream name so once that fixed, it can get reviewed

Tested out changes, looks good.

Looks good.


Update Electron to 1.7.6

@zeke anything blocking this? It would make our Electron debugging tutorial flow much smoother https://github.com/Microsoft/vscode-recipes/tree/master/Electron

Bumped it to 1.7.8, which is latest. Commit message says 1.7.9 though. 🙈

Whoops @zeke, my bad! Glad to see this land.


[7.x] Add assertFound method

This PR adds an assertFound method which could be very useful to assert redirection generated by Laravel Validation Logic
It could replace this code in tests:
- $response->assertStatus(302);
+ $response->assertFound();

Please make a new branch of off 7.x and then cherry-pick your commit into it. That should give you the correct commit tree to send a PR to our 7.x branch. :)

Yep sorry @GrahamCampbell
cf: https://github.com/laravel/framework/pull/32173


Feature Request: Toggle Using Intent

Since monitoring which app is in the foreground is out of scope, is it possible to trigger the toggle using an intent or another programmatic method?
Thanks!

https://github.com/zacharee/NachoNotch/commit/730757e997a5173b88aa234a44a46c5290c5d5e2
Rolling out now.


AnyLuaValue reads numeric strings as strings (fix #170)

This is a breaking change, as it was previously the case that parsing a numeric string such as "-2" would return a AnyLuaValue::LuaNumber. It's up to the maintainer whether this is desired.
The code is ugly-branchy, since the two branches don't have the same type (the repeated parsing builds up &s on the lua variable.)

I can do the same for AnyHashableLuaValue, if wanted. It might be inconsistent to have one perform coercion and the other not.

It's definitely something desirable to have the AnyLuaValue that actually matches the exact type of the variable.
Since it's more or less a bugfix, I guess I'll just merge that despite the breaking change.
Thanks!


HTML inside PHP - no hints or highlights

I'm using a .php file,
when I include html inside php, is like brackets editor doesn't recognize html code, so there is no hints and highlights:
<?php

          $html = '

    <div class="LIKE HERE">

    '

?>

is there a way to fix this?
Thanks

Could you please try the following and let me know whether it works?
`
"
?>`


Find a way to avoid @dev || xx constraint for local packages

Tried:
        $requires = [];
        $package = $composer->getPackage();
        foreach ($package->getRequires() as $key => $require) {
            if (in_array($require->getTarget(), $packages, true)) {
                $constraint = $require->getConstraint();
                $prettyString = '*@dev || ' . $constraint->getPrettyString();
                $newConstraint = new MultiConstraint([new MatchAllConstraint(), $constraint], false);
                $newConstraint->setPrettyString($prettyString);
                $require = new Link($require->getSource(), $require->getTarget(), $newConstraint, Link::TYPE_REQUIRE, $prettyString);
            }

            $requires[$key] = $require;
        }

        $package->setRequires($requires);

But it doesn't work, it says

Use link instead :)

@soyuka I found why it was failing in my project and implemented a solution (which is not the cleanest for the moment).
The idea is that if you have the following depedencies tree:
A -> B
B -> C
Then PMU will add B as @dev inside the lock file but for C dependency it will keep it as it is, making the creation from the lock file failing.
To fix this issue, I went not only to look at direct dependencies of the first package A but also all packages which are used inside the lock file.
If they were part of the monorepo, added the @dev on them and it looks like it made the trick, and now it works fine.
FYI, the PR is here: https://github.com/soyuka/pmu/pull/15
Feel free to review it, so we can find a more generic solution to access some of the necessary paths.
Also, I didn't add tests to it as for the moment when I try to run them, all are failing. :')


I found a translation error in a document.

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
While going through the beginner's material, I noticed an error in the document title; the operating system being used is Ubuntu 22.04.4, and the web browser in question is Microsoft Edge.

What changes are you suggesting?
The Chinese document title translation is not appropriate compared to the English document translation; the title should be '创建账户' instead of '创建客户'.
Additional information
No response

Muka mu kacau , kasian banget cuma ngeluarin isue tentang gue agar dapat simpati dari Agensy gue , kasian banget , ambil itu agensy gue , nih gue punya eksekutor , gue kasih keluar eksekutor gue , elu semua yang sok tau banget tentang gue, siap siap elu pada di eksekusi , cambuk dan gantung. Pikirkan kah gue , waktu gue susah aja gak ada yang mikirkan .

i can not undestand what are u saying  can u using einglish?

@KenoChou Thanks for reaching out, and flagging this! Since we us an internal, automated solution for translations, I will flag your request for our team 💛
For more information on translations, please see https://github.com/github/docs/blob/main/contributing/types-of-contributions.md#earth_asia-translations ✨


Appstudio update hacbs-test-project-b385

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-b385-on-pull-request-
cannot patch pipelinerun hacbs-test-project-b385-on-pull-request-: failed to patch pipelinerun rhtap-demo-dwju-tenant/logURL with hacbs-test-project-b385-on-pull-request-llnsg: pipelineruns.tekton.dev "hacbs-test-project-b385-on-pull-request-llnsg" not found

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-b385-on-pull-request-
creating pipelinerun hacbs-test-project-b385-on-pull-request- in namespace rhtap-demo-npuh-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


ERROR  Failed to establish connection to ResolvedIPv4Address(('<IP_ADDRESS>', 7687))  and ERROR Failed to establish connection to ResolvedIPv6Address(('::1', 7687, 0, 0))

In the first image, I changed the line in the conf of neo4j to allow a server connection.
And in the second picture, there are still errors
Would appreciate help
Thanks

Hmm, am I missing something or is the highlighted dbms.connector.bolt.listen_address line still commented?
In any case, if you are using a local Neo4j server and a locally-running application, you should not need make any changes to your server configuration.

Please refrain from double-posting your issue. I've already replied here.
Also, I'll close this issue as it is most likely not bug in the driver. Still, feel free to comment on it.


use array of k's to fix bug #210

Addresses #210

Thanks. I'll try to add a test corresponding to what is in #210 as well.

Thanks. Done and added a test corresponding to what is in #210 as well.


Update link to Ratify quickstart

Update with the new working link for Ratify quickstart "Ratify on Azure: Allow only signed images to be deployed on AKS with Notation and Ratify" at https://github.com/deislabs/ratify/blob/main/docs/quickstarts/ratify-on-azure.md.
Old link return 404: https://github.com/deislabs/ratify/blob/main/docs/examples/ratify-verify-azure-cmd.md

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


Registering - Please Help

So I am kinda new to this and am wondering how to register an account using this.

So I am kinda new to this and am wondering how to register an account using this.

I haven't touched this project in a long time, but I think something like below should work if the docs match the current API:
const res = await client.post('register', {
  data: {
    username: '<my username>',
    password: '<my password>',
    email<EMAIL_ADDRESS>    acceptPolicy: 'on',
    adb: 0,
    tz: 'America/Los_Angeles' // Or anything else in the column "TZ database name" at https://en.wikipedia.org/wiki/List_of_tz_database_time_zones
  }
})

I haven't touched this project in a long time, but I think something like below should work if the docs match the current API:
const res = await client.post('register', {
  data: {
    username: '<my username>',
    password: '<my password>',
    email<EMAIL_ADDRESS>    acceptPolicy: 'on',
    adb: 0,
    tz: 'America/Los_Angeles' // Or anything else in the column "TZ database name" at https://en.wikipedia.org/wiki/List_of_tz_database_time_zones
  }
})


Test Scenario translations Updated

Based on https://github.com/kiegroup/drools-wb/pull/1490

@yesamer @karreiro You have to ask yourselves why is this (are these) PR(s) necessary?
All i18n should be handled by the process @mbiarnes has setup. If files are being missed perhaps something else is wrong?

@manstis yes, indeed :-) For this sync, I've created a PR for DMN/BPMN as well. But, I'm pinging @mbiarnes to understand why kogito-editors-java was not updated.

@manstis I think nothing went wrong, most probably the process you mentioned didn't plan to update this repo, considering is a new repo. Of course, if @mbiarnes  prefers to update the process to include this repo, that is undoubtedly the best solution. No problem for me to close this PR if @mbiarnes wants to manage it.

@yesamer @karreiro @mbiarnes I suspect, as Guilherme said, this is a new repository and hence missed by the current process.
I am sure @mbiarnes would love nothing better than adding a new repository to the process ;-)

@mbiarnes Could you please take a look when you have time? :-)


Publish documentation

e.g. with Sphinx integrated it should be possible to import the repository at https://readthedocs.org/.
@seemethere can you facilitate this?

@kdelwat fyi

The problem is that Read the Docs doesn't support Python 3.5 syntax natively yet, as discussed here. There seems to be a workaround using Conda in that thread which would be a good solution.

If you look on that issue, this seems to be it: http://stackoverflow.com/a/36144269
I can send a PR if using conda is an accepted solution here.

IMO conda is a good solution.

This could be a possibility in the future but I don't see the added benefit just yet. The on-github docs done by @kdelwat seem to work great and if there's not a reason to add complexity then we shouldn't.

I would say migrating docs into GitHub's built in Wiki feature and allowing for community edits would be a great way to improve the currently bare-boned docs right now.
Not to mention it takes away much of the overhead from contributing documentation, not having to fork, clone, edit, push, PR, merge or anything.
Plus it's nice having doc contributions separate from actual code contributions, it declutters the PR interface, and is overall less stuff to micro-manage.

it's also nice having doc contributions separate from actual code contributions.
Nope. Good practice is too treat doc as code, and have it in the same repo.

fork, clone, edit, push, PR, merge or anything
People are very used to this workflow.

Also being not too tied with Github might be a good idea.

It would not be tied to GitHub at all, it's still a Git repository that will live under github.com/channelcat/sanic.wiki.git, except it will appear integrated into this repo under the Wiki tab.
You can still clone it and move it wherever you want, it's just a git versioned folder of Markdown files (much like what docs currently is) except it has navigation controls and a nice Web editing interface that lets you contribute docs without forking the whole thing and dealing with the bureaucracy of PRs which might be routine stuff for many, but I still believe it's burdensome to most.
It would be a relatively painless thing to implement and try out, and it might lead to more contributions given that it lowers the barrier of effort needed to submit improvements. Might be worth testing out, all it takes is mirroring the Markdown documents from the docs folder into the Wiki and seeing how it goes...

I think this can be closed: https://sanic.readthedocs.io/en/latest/
@luizberti maybe open a different issue for that?

I'd agree. I like the wiki style for some things but I think the docs should stay checked in with the code.
Maybe we can agree for some of the docs to go to the wiki (like extensions).
But @zkanda is right I'll close this issue.
@luizberti feel free to open a different issue regarding moving the docs to a github wiki.


where did e hentai go?

Steps to reproduce
go to source
search for e hentai
Expected behavior
it should be on the list of extensions
Actual behavior
disappeared...
Crash logs
No response
Tachiyomi version
tachiyomisy r437
Android version
android 8
Device
Samsung A7
Other details
also, i can still read the hentai that ive saved in my library but i can't find the extension for some reason.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.8.1.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Sy doesn't use Ehentai ext, since Ehentai is built in. Go to Browse > Sources > Filter > tick Ehentai to enable

Enable 18+ source in Browse

i know. i have been using the built in extension for a LONG TIME, i'm not stupid and i know where to search for it and now i even downloaded the e hentai thats on the website and it says that its redundant but the built in e hentai still ISNT THERE.

Make sure you didn't disable the all language or that you didn't disable hentai in the advanced settings

ahhh, thank you very much... i think i accidently clicked the disable hentai feature button. it's all good now.


OpenAL fails to build without change in CMakeLists

Currently, we can't use the default specification from vcpkg (find_package(openal-soft CONFIG REQUIRED) to ensure that OpenAL-Soft is included during configuration.
It's a non-blocker - we've just changed the line to mention OpenAL instead of openal-soft - however this will be used to track the issue from the vcpkg repo.

Same as freetype, this has been resolved by MSFT and should be closed. Continue to use NovelRT's fork as needed


Custom inverted stubs

This pull request makes it possible to use inverted stubs in a multi threaded context. There are quite a few issues that need to be ironed out (some of them are discussed in here http://caml.inria.fr/mantis/view.php?id=6764) :

The runtime must be properly initialized. The shared library initializer must make the following calls at initialization time:

caml_startup(caml_argv);
caml_release_runtime_system();

The OCaml code of the library must be compiled with the -threads options, and must link the code from the Thread module.

A few questions:

it happens that some function might require a slightly different treatment from others (e.g., one particular function should not take the lock, or whatever)
I have not yet settled on how to deal with attributes and pragmas.
This might also be the opportunity to deal with these pesky shared objects visibility issues. The way I currently deal with it in the build system is not that satisfying. One could automatically tag the stubs using the proper _declspec(dllexport) for Windows and __attribute__ ((visibility ("default"))) for Unix systems (provided that the latter is used with the correct -fvisibility flags at compile time). The issue with this kind of things is that they might complicate the build system for end users.
One issue with the current state is that it is not possible to write a #pragma GCC visibility push(default) before the generated headers

#include <caml/memory.h>
#include <caml/callback.h>
#include "ctypes/cstubs_internals.h"

This is rather out-of-date now, and seems unlikely to progress further, so I'm closing it.  (Please feel free to reopen if you'd like to work on it again.)
Some support for using inverted stubs in a multithreaded context was added in #276.  #420, which supports thread registration from callbacks, may also be relevant.


Rainglow preventing resource reloading

Hello, I am running a 1.20.1 fabric server with the latest version of Rainglow installed. Whenever I try to /reload, the reload fails, and the log doesn't give me much more than this:
Caused by: java.lang.NoClassDefFoundError: net/minecraft/class_310 at io.ix0rai.rainglow.data.RainglowResourceReloader.method_14491(RainglowResourceReloader.java:48) ~[rainglow-1.1.1+mc1.20.1.jar:?]at net.minecraft.class_4013.method_29490(class_4013.java:15) ~[server-intermediary.jar:?]at java.util.concurrent.CompletableFuture$UniRun.tryFire(CompletableFuture.java:787) ~[?:?]... 35 more

@Ultimushadow I'll investigate this today, could you provide the full log as well please?

never mind about the log, this is a simple bug! I'm calling client code from the server once again

Oh haha I just got up, glad it was a simple fix!

it was, I should have a release out within a couple hours!


How can I create a pull request for the Wiki page?  EOM

I don't know also, maybe github don't allow create pr for wiki page


Add quotes around user agent to allow spaces in it

This is not doing what you thought it would do. You are now sending a user agent of "\x22mediagis/nominatim-docker:4.2.1\x22", i.e. with the quotes you added included.


Maximun Call Stack Size Exceeded just by openning a select in the header.

How to reproduce it:
Add any input controller and focus it and at few seconds a inity error logs will start fire in the console (attached image)

The same problem here, I "solved" it setting blocking = false on BottomSheet component props, but I don't know if can generate another problems.

We have to fix this immediately. I used this BottomShit everywhere in my project )