expresson_is_true schema test is wrong when the expression evaluates to null

Describe the bug
the expression_is_true test generates a SQL statement like below. This is actually testing for expression is true or null.
with validation_errors as (
select
    *
from schema_name.table_name
where not(<expression>)
)

select count(*)
from validation_errors

Steps to reproduce
setup (on postgresql 13):
create table test_expression_is_true (a, b) as (
  values (1, 2),
         (2, 3),
         (3, null)
  )

generated query for expression_is_true where expression: b > a
with validation_errors as (
select
    *
from test_expression_is_true
where not(b > a)
)

select count(*)
from validation_errors

Expected results
the test should fail with 1 row
Actual results
the test passes
Screenshots and log output
System information
packages:
  - git: "https://github.com/fishtown-analytics/dbt-utils.git"
    revision: 0.1.21

Which database are you using dbt with?

[ x ] other (specify: athena)

The output of dbt --version:
0.16.1

Are you interested in contributing the fix?
yes, i think the fix is relatively simple.
instead of not (expression ) the where clause should be not(expression) or (expression) is null to test for expression_is_true

Correct me if I'm wrong, but setting condition = FALSE results in a pass every time, right? The current code looks like this:
with meet_condition as (
    select * from {{ model }} where {{ condition }}
),
validation_errors as (
    select
        *
    from meet_condition
    where not({{expression}})
)
select count(*)
from validation_errors

Setting condition = FALSE results in a zero row meet_condition CTE:
with meet_condition as (
    select * from {{ model }} where FALSE
),

Shouldn't the logic look more like this?
with meet_condition as (
    select * from {{ model }}
),
validation_errors as (
    select
        *
    from meet_condition
    where {% if condition %}not {% endif %}( {{expression}} )
)
select count(*)
from validation_errors


How to set path to other server (just like http://localhost:3000/__webpack_hmr)

In my electron project https://github.com/chentsulin/electron-react-boilerplate/tree/react-transform, I should request to http://localhost:3000/__webpack_hmr, but It will automatic polling to file:///__webpack_hmr .
GET file:///__webpack_hmr net::ERR_FILE_NOT_FOUND

I only see path can be set as options, but path=http://localhost:3000/__webpack_hmr not works. Is there any other options can get this done?

You should be able to set the full path in https://github.com/chentsulin/electron-react-boilerplate/blob/react-transform/webpack.config.development.js#L16
This has worked for others in the past, so if you're still having issues it must be electron related - I would recommend trying to create an EventSource connection manually in the console to test.

'webpack-hot-middleware/client?path=http://localhost:3000/__webpack_hmr' <- this one works now. Not sure why not works previously...Thanks your instant reply! : )

By using this line 'webpack-hot-middleware/client?path=http://localhost:3000/__webpack_hmr' I got the message [hmr] connected. But after I made some change in the code, [hmr] is not requesting to localhost to retrieve the hot-update.json and then it shows cannot find update (full reload needed) warning. How can I make react-hot-middleware to look for hot-updates in localhost ?

@tahmidsadik112 Do you have defined publicPath? See my example
https://github.com/gaearon/react-transform-boilerplate/issues/47#issuecomment-151909080


Using [fields] inside function in XSL script

Hey there.
I'm trying to create a list with two rows for each RowId, first visible and second hidden with this code:
https://pastebin.mozilla.org/9029589
However once I place the token brackets in the onClick="toggle('[UserDefinedRowId]');" and click "generate from html template" I get this error:
RawUrl:/cc/10/page/ctl/GenerateXsl/mid/614/Edit/Current
Referrer:https://mysite.pt/cc/10/farmacossu/ctl/GenerateXsl/mid/614/Edit/Current
ExceptionHash:KXfcUjiSKeSKzbjkDs5CNxWZvVE=
Message:'<', hexadecimal value 0x3C, is an invalid attribute character. Line 149, position 30.
If I do not place the brackets in html, but edit the xslt script with {udt:UserDefinedRowId} it works just fine. This only happens inside the toggle function, everywhere else works fine.
Best regards,
João

Cannot reproduce as there is no more info in the paste bin.
Just a guess: you cannot enter js in the html template. That is by design.

You can enter JS. You just can't use tokens in some places.

Ok, but that is only by chance, and I would not recommend it.
If it works for you, then perfect. If it fails like for tokens, I will not spent any time fixing it.


feat/#470/publish npm package

feat: create node script and package
chore: add generated package folder to gitignore
feat: publish npm main package

Summary:
Briefly describe the changes made in this PR.
Issue Reference(s):
Fixes #470
/claim #470
Build & Testing:

[x] I ran cargo test successfully.
[x] I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

Checklist:

[ ] I have added relevant unit & integration tests.
[ ] I have updated the documentation accordingly (if applicable).
[x] I have performed a self-review of my own code.

@ologbonowiwi Can you provide the steps to test if this is actually working.

You'll need to make manually what the ci will do.
You need to build a tailcall version, export the node variables that ci.yml is exporting, node ./npm/prepare-core-package.js and then npm publish --access public (on the package created under /packages), after that you can npm publish --access public on /npm folder

The best way to test this would be to run the ci.yml as a GitHub action; this would also work.
We can change the ci.yml trigger to be on the pull request; it'll likely ask for your permission, and when granted, it'll run, and we'll see if it works

Run a GitHub action locally it'd also work, but I'm not sure how to do that

Run a GitHub action locally it'd also work, but I'm not sure how to do that

👉 https://github.com/nektos/act

@tusharmath any success testing this?

find a way for you to test this @tusharmath:

create a fork of tailcall on your GitHub account;
clone the repo locally;
set all the SECRETS/variables needed to run the actual build. It should reflect tailcallhq/tailcall permissions to publish on NPM;
add ologbonowiwi/tailcall as a git remote;
merge feat/#470/publish-npm-package on your main branch
git push

now, the GitHub actions will trigger, and you'll see the build happening.
remember to make any necessary tweaks on package names to avoid overriding the actual tailcall stuff.

It'd be nice to get part of the bounty since it's already implemented with all requested changes/tweaks, and now you have multiple possibilities to check if it works.
I had already worked on #512, and after the job was finished, the requirements for the implementation changed, and I needed to re-implement the same thing using another approach, so it's fair to say that the bounty is well deserved.
@tusharmath

It'd be nice to get the bounty since it's already implemented with all requested changes/tweaks, and now you have multiple possibilities to check if it works.
I had already worked on https://github.com/tailcallhq/tailcall/pull/512, and after the job was finished, the requirements for the implementation changed, and I needed to re-implement the same thing using another approach, so it's fair to say that the bounty is well deserved.
@tusharmath

bump

Closing this in favour of #488

@ologbonowiwi The approach in this PR and in #488 are similar. In both cases there is additional work that needs to be done. Since #488 was the initial PR, it's only fair the bounty is delivered for the PR. Thanks once again for your contributions!

Closing this as we merged #488.

/tip 10$ @ologbonowiwi


andHaving must be similar to andWhere. And are nested

What steps will reproduce the problem?
Try to add several addHavings to the query.
What is the expected result?
 [having] => [
        0 => 'and'
        1 => [
            0 => '<'
            1 => 'miles'
            2 => 100
        ]
        2 => [
            0 => '<='
            1 => 'finalPrice * ((100 - finalDiscount) / 100)'
            2 => 'test'
        ]
        3 => [
            0 => '<='
            1 => 'finalPrice * ((100 - finalDiscount) / 100)'
            2 => 'tests'
        ]
    ]

What do you get instead?
[having] => [
        0 => 'and'
        1 => [
            0 => 'and'
            1 => [
                0 => '<'
                1 => 'miles'
                2 => 100
            ]
            2 => [
                0 => '<='
                1 => 'finalPrice * ((100 - finalDiscount) / 100)'
                2 => 'test'
            ]
        ]
        2 => [
            0 => '<='
            1 => 'finalPrice * ((100 - finalDiscount) / 100)'
            2 => 'tests'
        ]
    ]

Additional info

Q
A

Yii version
2.0.12

Solution?
andHaving must be same as andWhere. Instead of :
if ($this->having === null) {
            $this->having = $condition;
        } else {
            $this->having = ['and', $this->having, $condition];
        }
        $this->addParams($params);
        return $this;

It should be like this:
if ($this->having === null) {
            $this->having = $condition;
        } elseif (is_array($this->having) && isset($this->having[0]) && strcasecmp($this->having[0], 'and') === 0) {
            $this->having[] = $condition;
        } else {
            $this->having = ['and', $this->having, $condition];
        }
        $this->addParams($params);
        return $this;

it does not change anything in functionalilty, it just generates nicer SQL.


--babel-config-file CLI option fails if not absolute path

npx livepack src/index.js -o build --babel-config pre --babel-config-file my-babel-config.js
Throws an error Cannot find module 'my-babel-config.js'.
Path should be resolved. One of:

Allowing Babel to resolve path itself providing root / rootMode properties to Babel register
Resolving path in CLI prior to calling register(), relative to cwd
Resolving path in register()

Also fails if configFile option passed to register() programmatically, and configFile is not an absolute path, but this is less problematic - perhaps it should fail.

--babel-config-file CLI option has been removed.


feat(parser): support revert statement

Closes #696
Followed these instructions https://github.com/hyperledger-labs/solang/issues/696#issuecomment-1066666159
revert statement remains unimplemented! in sema

My suggested parser changes did not work. I'm trying to see what changes are needed, let me get back to you

@mattsse I don't think I can add commits to your PR, but here is a commit which fixes the grammar: https://github.com/seanyoung/solang/commit/16ee98629f67c94adfab6ba5970832f65f3d415c

Thanks so much!
https://github.com/seanyoung/solang/commit/16ee98629f67c94adfab6ba5970832f65f3d415c appended

There is also an issue in the solang-parser test, I had not tested that, this should fix that https://github.com/seanyoung/solang/commit/787515bf9d3f4d1a3f5fed3d863f6e784a00f932

The vscode extension needs a small fix for the test to pass, in vscode/src/test/suite/extension.test.ts

Looks great, I'll fixup the vscode test in a follow up PR


WAG Example missing parameter

Manage web traffic with Azure PowerShell
https://docs.microsoft.com/en-us/azure/application-gateway/scripts/create-vmss-powershell
The sample script, has a missing parameter:
Set-AzVmssStorageProfile $vmssConfig   -ImageReferencePublisher MicrosoftWindowsServer
-ImageReferenceOffer WindowsServer   -ImageReferenceSku 2016-Datacenter
-ImageReferenceVersion latest
This gives when creating the VMss:
New-AzVmss   -ResourceGroupName myResourceGroupAG
-Name myvmss `
-VirtualMachineScaleSet $vmssConfig
New-AzVmss: 'CreateOption' cannot be null.
The solution is to add the parameter:
-OSCreateOption FromImage

@Candelit -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

@Candelit, Thanks. all the app gateway articles got updated except this script file.
-OsDiskCreateOption FromImage parameter added to the script.
#please-close

@Candelit, Thanks. all the app gateway articles got updated except this script file.
-OsDiskCreateOption FromImage parameter added to the script.
#please-close

Thanks a lot!


Add config option to always show suggestion list

This feature is useful if we are doing the actual filtering we need via a REST service and we want to show all the results it returns. If the exact text matches, the highlight will continue to be displayed. This is useful if the user is searching for values that may exist in metadata but not always displayed on the title or subtitle.

Yeap , have same prorblem

@apasccon any update on getting this pull request merged in? I just merged in your latest changes into my repo so there are no longer any conflicts.


Python3 and GCP issues

Hi,
I've been trying to run on GCP with an Ubuntu 18.04 VM with Python3. I get cascading module issues, which I have resolved by installing the correct modules and/or changing the code to point at correct modules.
I have noticed that even though I am running on GCP, it is trying to load aws_network from aws_dpb_emr.py. I don't want or need to install AWS deps.
Finally, after working around numeous module issues, I'm stuck on perfkitbenchmarker/flag_util.py throwing an error.
PerfKitBenchmarker-1.13.0/perfkitbenchmarker/flag_util.py", line 276, in __init__     if isinstance(convertible_to, (basestring, units.Unit)): NameError: name 'basestring' is not defined
My question is, how can I run this using python3, and how can I run on GCP so that I don't have to install other providers deps.

Note, I just installed python 2.7 and pip using apt and a benchmark is now running.

Hi Andy,
Unfortunately the 1.13 release is quite dated.  As you note, running from master/head is should work completely for Python 2.7.  It should also work in most cases with Python 3.6.  See some discussion in https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/1989.
If you run into specific issues running in Python 3.6 from master/head, please feel free to open a new issue with repro steps and your pkb.log and assign it to me to take a further look.  I'm also happy to review pull requests.
Thanks,
David

As an aside, we are working towards making periodic stable releases again. I don't have a date for when we will start this, but hopefully by the end of the year.


Identify incorrect flow direction

Each stream reach show know which adjacent reaches are upstream.
Required for:

accurate segmentation (and accurate catchment delineation)
determine accurate routes
identify all upstream reaches from a selected point

This one sounds kind of fun.
We discussed directed graphs other issues like: https://github.com/SouthForkResearch/pyGNAT/issues/7
I think all we have to do here is the same kind of thing and then compare adjacent nodes to see if there's a problem then flag all the problems.

I found an interesting tool built for shapefile networks called nx_spatial.  Looks like he wrote a script related to finding flow directions. Might be worth checking out.

I think I've found a way to find stream segments (i.e. graph edges) with an incorrect flow direction.
https://github.com/SouthForkResearch/pyGNAT/commit/d2a358fc74d0aa22f89fdf10851d808f9057d8ba


Revise manual tests

Issue Number
Somewhat related to https://github.com/input-output-hk/cardano-wallet/issues/1963.
Overview

dac491dcc3a69f1cc2bce9b7fb09ff2eb2445f05
jormungandr specific tests

9b0c5513daa7478685eadfc22260e6742e723cd0
Rewrite DBMigration and SyncTolerance tests to be run on cardano-node

2db09059ba78b92c4b665046be583e3ca44dc059
Drop redundant SyncProgress and TrackBlockHeight tests (sanity tested during DBMigration)

466a43b0fc82c313174f530a0016e1654be5d008
Test for Syncing in the Byron era

143f2cb5fa8e98842a37c06e383987577334ea1e
Add $ in from of cmds and fix indentation

2c40953eae7fa0ca897efd86ef64cd6578bc51a7
Add exemplary curl commands for smoke checking wallet functionality in DBMigration test

Comments

bors r+


Creating new endpoint for retrieving all solutions of a user from all assignments in a group at once.

Codecov Report

Merging #400 (6d362da) into master (4a20951) will increase coverage by 0.03%.
The diff coverage is 92.59%.

@@            Coverage Diff             @@
##           master     #400      +/-   ##
==========================================
+ Coverage   71.67%   71.70%   +0.03%
==========================================
  Files         463      463
  Lines       15503    15529      +26
==========================================
+ Hits        11111    11135      +24
- Misses       4392     4394       +2

Impacted Files
Coverage Δ

app/V1Module/presenters/GroupsPresenter.php
83.33% <87.50%> (+0.20%)
:arrow_up:

app/V1Module/router/RouterFactory.php
100.00% <100.00%> (ø)

app/model/repository/AssignmentSolutions.php
85.14% <100.00%> (+1.45%)
:arrow_up:

app/model/view/AssignmentSolutionViewFactory.php
91.22% <100.00%> (ø)

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 4a20951...6d362da. Read the comment docs.


Fixes #701: Use pooled adaptor buffers instead of ping pong buffers i…

…n tcp adaptor. Introduced some common raw buffer read and write function in adaptor_common.c

Codecov Report

Merging #709 (d434f41) into main (dab515d) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #709      +/-   ##
==========================================
- Coverage   26.95%   26.94%   -0.01%
==========================================
  Files         132      132
  Lines       31346    31349       +3
  Branches     4997     5003       +6
==========================================
  Hits         8448     8448
- Misses      21807    21810       +3
  Partials     1091     1091

Flag
Coverage Δ

unittests
26.94% <0.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/adaptors/adaptor_buffer.c
0.00% <ø> (ø)

src/adaptors/adaptor_common.c
13.76% <0.00%> (-6.24%)
:arrow_down:

src/adaptors/http1/http1_adaptor.c
7.94% <ø> (ø)

src/adaptors/http2/http2_adaptor.c
2.16% <0.00%> (+0.05%)
:arrow_up:

src/adaptors/tcp_adaptor.c
2.22% <0.00%> (-0.01%)
:arrow_down:

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

Changed the buffer size to 16 and removed all occurrences of sys.stdout.flush()


Add completion queue impl

This will be useful for short circuiting directly to completion queue impl version.

This file is needed to create a similarly named file in net/grpc and to be able to bypass switcher as suggested by nicolasnoble@

Folded into https://github.com/grpc/grpc/pull/19446


Implement consume x operator with the accepted SE-0366 syntax.

Implement it as a contextual operator that only parses as an operator when followed by an identifier.
Matching swift-syntax changes are https://github.com/apple/swift-syntax/pull/1379

https://github.com/apple/swift-syntax/pull/1379
@swift-ci Please smoke test


raidboss: More ex6 triggers

Fix 2nd Terrastorm trigger
Add Gale Sphere triggers
Add Arctic Assault trigger
Add Flames of Eventide trigger
Not 100% sure on the arctic assault trigger yet, I was only able to find a few variations to test against.
This includes both triggers for Gale Sphere that we discussed, open to further suggestions on these. Example screenshot from raidemulator for each of the new triggers:

There are some issues with these in-instance. Trying to sort them out still.

This includes both triggers for Gale Sphere that we discussed, open to further suggestions on these.

It might be helpful to call the clone order a little before calling the safe spots so you don't get stacked info alerts.  You can determine the order the clones will fire based on the Goldez's Shadow combatants.  They each use a different ability indicating which wall they will float to.  IIRC:
844F: North
8450: East
8451: West
8452: South

There are some issues with these in-instance. Trying to sort them out still.

Yeah, the clone order was correct for me but it felt like E/W N/S directions were (always?) swapped during gale sphere.  N/S seemed ok for the meteors + ice spikes but E/W also was (always?) backwards.

This should be ready for a proper review now.

Gale Sphere safespots are wrong sometimes.

Gale Sphere safespots are wrong sometimes.

With the latest version? 6ff8031 or 744567d.

Gale Sphere safespots are wrong sometimes.

With the latest version? 6ff8031 or 744567d.

Will try actually

Okay it works with the newer commits, my bad. Good stuff

Terraform 2 safe spots are wrong, it seems to always be the opposite?

Terraform 2 safe spots are wrong, it seems to always be the opposite?

The ice spike wall is inverted sometimes during terra 2 I think. I'm investigating that.


Dartmouth: their org has two default streams

I can't replicate this in production; there's only one default stream (confirmed through console) and it's named like in the screenshot. The screenshot is odd since it seems to list the same stream twice...

There were two default streams with the same name but different identifiers. We'll need to keep an eye on this. I think the duplicate is now named 2022-03-25 - 2022-03-26

I'm going to close this for now since the issue seems to have gone away, but let's keep an eye on it.


[Question] Can you make the training log public?

Question
Can you make the training log public?

Thank you for your interest in our research! In our commitment to making our research fully transparent, we're pleased to release the wandb logs here.
We also put them in MODEL ZOO so that people can find them more easily.

Thank you for your interest in our research! In our commitment to making our research fully transparent, we're pleased to release the wandb logs here.
We also put them in MODEL ZOO so that people can find them more easily.

Thank you for doing such a great job!

@haotian-liu This url only contains the logs of the 13B model. Can you please provide the logs of the 7B model?


Intermittent CI failure in step Install Carrier. Deployment troubles, timeout in waiting for completion.

Deploying Quarks
⣾ Waiting for resource  to be running in quarks ... 💤   ...
Couldn't install carrier: failed waiting Quarks deployment to come up: no pods in quarks with selector
Error: Process completed with exit code 1.

Can happen fir gitea as well. Might be too short a wait period, i.e. when GH is loaded and slower in performing deployment.

Can happen fir gitea as well. Might be too short a wait period, i.e. when GH is loaded and slower in performing deployment.

Actually the cli is too fast, starting to wait for a pod to be running without waiting for the pod to exist.
See PR #47 (merged) and #52 (pending) for fixes.

Actually the cli is too fast, starting to wait for a pod to be running without waiting for the pod to exist.
See PR #47 (merged) and #52 (pending) for fixes.


Keyboard bindings for terminal escape sequences.

Description of the new feature/enhancement
I would like to be able to create key bindings for terminal escape sequences. For example, I would like to bind ctrl+tab to \x1B[27;5;9~ and ctrl+shift+tab to \x1B[27;6;9~.
Proposed technical implementation details (optional)

You can already achieve this with a sendInput action (see PR #7249). For example, try adding the following comands in the actions section of you settings.json file:
{ "command": { "action": "sendInput", "input": "\u001b[27;5;9~" }, "keys": "ctrl+tab" },
{ "command": { "action": "sendInput", "input": "\u001b[27;6;9~" }, "keys": "ctrl+shift+tab" }

Thank you! Sorry if I missed it in the documentation.

No worries. I also originally thought it wasn't documented, but I see now there are actually some docs here:
https://docs.microsoft.com/en-us/windows/terminal/customize-settings/actions#send-input

Thanks very much 😊


fix(card): add eyebrow to card pictogram story

Related Ticket(s)
#5795
Description
Adds missing eyebrow copy to default web component Card story.
Changelog
New

eyebrow copy added to default story.

Hey Kenny! Thanks for the fix. I'm afraid this won't fix the situ when eyebrow text is absent though. Can we restore the knobs so I can test the cases where some optional texts are absent? Left side is Web components, right side React. React card's knobs are expected.
We recently cleaned up the pictogram cards in Web component (Card : with pictogram) and I asked for some knobs to be removed. Maybe the changes were accidentally applied here in the default card as well.

@shixiedesign There is no knob for eyebrow so if you remove it via the inspector, or even a Storybook knob, the eyebrow custom element will remain. This will create the extra space we don't want. One thing we could do is have a checkbox which shows/hides the eyebrow – that would remove the element entirely if it is unchecked.


feat(coral): Topic messages: Add filter to set partition (hashtag1236)

![Image](

Closing this issue since we're not adding this (after sync with Backend).


iOS bug - unwanted scrolling when focusing an input inside the modal

You can test this issue here: https://limonte.github.io/sweetalert2/ios-bug.html with iOS device of course :) For other devices (desktop, Android) there's no such bug.
Tested in Safari and Chrome - the same result, so I think it's iOS issue.
This happens only if there's an input inside the modal and I tap that input. In the same moment, that input gets focus and native iOS keyboard become visible.
The page below modal in the same moment is automatically scrolled to 50% of its height. This behaviour is totally unwanted and I have no clue how to prevent this default iOS "feature".

PS. this issue was recreated since we have implemented different modal positioning starting from 5.0.0

Related StackOverflow question: http://stackoverflow.com/q/39626302/1331425

I just released v5.3.1 with the fix for this issue!
Many thanks to @jpattishall and his detailed research and answer on StackOverflow: http://stackoverflow.com/a/40033422/1331425


Cached repo is not removed from builder when app is deleted

How to reproduce:

Create application named foo
Push to it to make builder generate /home/git/foo.git
Destroy application with deis apps:destroy
Create application again with the same name
Try to push
Outcome: 'remote is up to date'.
Expected: proper push.
What happens is repo not being removed from the builder when application is destroyed.

this was fixed recently with https://github.com/deis/deis/pull/3752, but thanks for the report :)

Oh, nice, looking forward for 1.7 :)


exception when trying to access a value of "null"

Error Case:
In a cell i have a value, which is null. This will break the download() with the following error:
Subscriber.js:242 Uncaught TypeError: Cannot read property 'indexOf' of null     at Ng2CsvService.webpackJsonp.1016.Ng2CsvService.escapeRowValue (Ng2Csv.service.js:51)     at Ng2Csv.service.js:42     at Array.map (<anonymous>)     at Ng2CsvService.webpackJsonp.1016.Ng2CsvService.convertToCsv (Ng2Csv.service.js:42)     at Ng2CsvService.webpackJsonp.1016.Ng2CsvService.download (Ng2Csv.service.js:18)     at BatchComponent.webpackJsonp.1146.BatchComponent.startDownload (batch.component.ts:169)     at BatchComponent.webpackJsonp.1146.BatchComponent.downloadSelected (batch.component.ts:138)     at Object.eval [as handleEvent] (BatchComponent.html:113)     at handleEvent (core.es5.js:11998)     at callWithDebugContext (core.es5.js:13467)
Expected Behaviour:
I would expect that null values will be like an empty string in a cell, displays nothing.

Thanks for raising this issue. I suspect you're using the OrderedProjectionCsvRowMapper as I don't think this issue is present when using AutoCsvRowMapper. The reason being that AutoCsvRowMapper always returns a non-null and non-undefined string. To avoid this I think you could have specified in your mapping something like ['MyColumn', x => x.columnValue || ''] but I think it's sensible to make a change to the library to allow this behaviour to be configured.
I've release 0.4.0 where by default null and undefined values will be written out as empty strings. For example, it will appear as an empty column value columnOneValue,,columnThreeValue in the CSV. You can change the value to something else, e.g. config.outputValueForNull = 'NULL'; would lead to output like columnOneValue,NULL,columnTwoValue.
Thanks again. Let me know if you have any issues with the above.

So sorry for late response (projects wasn't updated since then): I updated and removed my helper method.
Looks like this is fixed. Thank you very much!


Display internal addresses even when models are external

Closes: RHOAIENG-7579
Description
This PR aims to display internal addresses even when models are external.

How Has This Been Tested?
check the inference endpoint:
when external route is enabled:

it should display both internal and external address

when external route is not enabled

it should only show internal address

Test Impact
Added cypress tests.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

@vconzola Could you please take a look at the above screenshots?

Also LGTM aside from @gitdallas 's nitpicks on capitalization @ppadti

Look into the PatternFly Capitalization guidelines and the relevant rules are:

Capitalize proper nouns, product names, acronyms, and initialisms. For example: React, PatternFly, and HTML.
Write all components in lowercase unless they start a sentence.
I guess we won't treat 'internal' and 'external' as proper nouns then I agree with @gitdallas 's comment that internal should be lowercase.
cc @kaedward for confirmation.

Do we need to display "(internal and external)" in the button text? Could we just use "Endpoint details"? This is my preference unless we absolutely need to include the types in the button text.
Alternatively, if we do need to include the types, we could say "Internal and external endpoint details". WDYT?

@kaedward I think the types are important, so a user can see if a model is accessible both internally and externally without having to open the pop-up. I like your idea of saying "Internal endpoint details" or "Internal and external endpoint details" and getting rid of the parentheses.

@ppadti @mturley I see this is already merged. Do we need to open another issue for the text change @kaedward suggested?

@ppadti if you want to open a followup PR I can review it tomorrow, but if you don't have time then yeah we should open a separate issue. Sorry @vconzola.

@ppadti if you want to open a followup PR I can review it tomorrow, but if you don't have time then yeah we should open a separate issue. Sorry @vconzola.

yeah sure, I will open a followup PR for this.

Followup PR: https://github.com/opendatahub-io/odh-dashboard/pull/3351


DBZ-762 Add support for citext data type

https://issues.jboss.org/projects/DBZ/issues/DBZ-762
I had to change debezium-parent's version back to 0.9.0-SNAPSHOT to get this to build at all.

@gunnarmorling I did the changes you requested. I also changed POM versions back to 0.8.0-SNAPSHOT since I was able to get it to build (I wonder what was up with that). Although I did have to change it in support/checkstyle/pom.xml as well, which hasn't been done in master.

Rebased and applied. Thanks a lot, @olavim!
I've squashed everything into a single commit (in order to keep the history lean) and also completed the implementation for handling citext arrays (a piece for that was still missing, as was the test). Nice change, much appreciated!

@gunnarmorling Thanks for your help.
FYI: I was testing this in a kafka connector, and encountered a hickup when the citext converter in PostgresValueConverter would receive data of type PGobject. I added the logic to convert from it, but this didn't come up during the integration tests. I think tests should be added to ensure that data can be converted from PGobject.

Hum, I saw that, but didn't think too much about it. Any idea how we can reproduce this, i.e. when would it be submitted as a PGObject? Could it depend on the DB version?


CACP-166 API Versioning

What
Link to story
Adds the versionist gem to allow for versioning of API endpoints. This is implemented at present to use the URL path to define the version - but this can easily be changed to use an HTTP header or request parameter if required.
As a result the code has been refactored to move functionality into the api namespace. Some additional refactoring was done to introduce the idea of internal and external namespaces (ie those available to other LAA applications and HMCTS applications respectively). This allows for better control of endpoints offering similar functionality (eg. we will have two hearings endpoints - one to be called by CPUI for searching, the other to be called by HMCTS to post updates. This approach allows the two to be kept clearly separate.)
The URLS for the two existing API endpoints are now
/api/internal/v1/prosecution_cases (was /api/prosecution_cases)
and
/api/external/v1/hearings (was /hearings)
This PR also removes an unnecessary route/controller and associated specs, and fixes an issue with binding.pry.
Checklist
Before you ask people to review this PR:

[x] Tests and linters should be passing
[x] Github should not be reporting conflicts; you should have recently run git rebase master.
[x] Avoid mixing whitespace changes with code changes in the same commit. These make diffs harder to read and conflicts more likely.
[x] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

Looks great 👍
We should probably also make changes to the schema before we merge this

Good point. I forgot about the schema. I'll update it and add it to this PR.


[BC Idea]:

BC Idea Link
https://experience.dynamics.com/ideas/idea/?ideaid=ec62231e-c43d-ef11-a295-000d3a059b20
Description
It seems that since ever, the Trial Balance printed out the Net Change for given period and the balance on the upper limit of the date filter provided. However, in praxis it is needed that the balance on the lower limit of date filter provided is printed out, net change and the balance on the upper limit of the date filter provided.
Old:

Desired:

Currently there is an RDL Report but also an Excel Report. This change has been approved for Excel Report.
Reference: https://github.com/microsoft/BusinessCentralApps/issues/833
I will provide the implementation for this BC Idea

[x] I will provide the implementation for this BC Idea
Internal work item: AB#540995

hey @JesperSchulz. Can you approve this change? It has already been approved but in the BusinessCentralApps repositorium.

Can anyone enter the full subject of the issue?


cleanup: correct typo in travis scripts

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

@Madhu-1 addressed the comment, ptal.  Thanks

@mergifyio backport release-v3.3


Future feature: User management. Ideas, wishes, roadmap

This is a COPY of Issue 1255: Future feature: User management. Ideas, wishes, roadmap, filed on Google Code before the project was moved on Github.

Submitted on 2012-12-10T17:53:05.000Z by<EMAIL_ADDRESS>Status: Accepted

Please review the original issue and especially its comments. Comments here on closed issues will be ignored. Thanks.
Original description
A few notes on how I see the future User Management feature

# Goals:
- storing login/pwd in the DB instead of config.php
- segregating short URL per user

# Needs:
- an UI to add/edit/delete users
- a full fledged installer that will prompt user for an admin login & password and store that in DB, instead of having users write that down in a config file, see issue 1254

# Features:
I want that feature to work like the user management in WP, that is
- roles (ie 'admin', 'user', 'statviewer', etc)
- capabilities (ie yourls_current_user_can('manage_plugins'), 'view-stats', 'use-api', etc...)
- everything filterable to create as many roles and capabilities as needed

Default roles should probably be:
stats: view stat pages only
api: use API actions only
user: capabilities = 'stats' + 'api' + shorten URL, edit/delete short URL
admin: capabilities = 'user' + manage plugins + manage users + manage options if any

# DB

To be pondered and decided. A 'user' table and a 'user_meta' table.

# Misc

During the process, attention should be paid to other areas that should benefit from these feature, such as Issue 1239 (multiple secret signatures per user)

When user management will be implimented?

In the future, no ETA, probably lowish priority anyway to be honest

I just want to add here that I think the currently existing multi-user plugin would be good to use as a reference for what user management could look like if implemented directly in YOURLS:  http://www.matbra.com/en/code/multi-user-yourls-plugin/
I've quite heavily modified my version of the above plugin.  It supports LDAP login and also our company's WebSSO login.  When a user logs in for the first time, their account will automatically be created.  Each user has their own admin page where they can see and manage their own short links.  They can also view (and search) all short links in the system.

Very interesting, thanks for the link @adigitalife!

Some thoughts on users…
The current system scales really badly because in several places it relies on cycling through the list of users. This is OK if you have half a dozen users, but isn't going to work well on a large installation. In looking at user management this needs to be eliminated.
The way cookies are generated is one place. This is fairly simple to change. Using PHP's internal session management is the obvious way.
The API is another place. For the basic signature style API a change here would require that a user's API key be stored in the database. It wouldn't need to be based on hashing the username, although it could be initially to ease migration. Verifying the API key is then simply a question of looking it up in the DB. Storing an API key in the DB, the form of which is independent of the username, also has the advantage that a compromised API key could be reset on a per user basis. Currently the only way to reset an API key is to change YOURLS_COOKIEKEY, which resets everyone's API key
The big problem is the time limited signature token API method. There is no way of verifying the call without cycling through all users and testing the received signature against each in turn. This is completely unscalable. I therefore think this API call in it current form should be deprecated and replaced. This should happen as soon as possible to allow authors of systems relying on this method time to change their code.
A replacement would need to include the username in the request. Clearly the simple way to do this is just to add a 'user' parameter to the request. Alternatively, a single paramater 'key' could be used that consists of a base64 encoded concatenation of username, timestamp and signature, separated by a suitable character, perhaps null (\0). Internally then base64 decode and split on the separator.

I like the concept of the multi-user plugin as it stands except that it doesn't seem to support the concept of roles, which is just as important when it comes to supporting multiple users. The roles as proposed above are a good set. I might suggest one extra layer of "editor", which would be like user but with the ability to see all shortened links, not just their own.
At the moment, I'm inclined to stick with https://github.com/nicwaller/yourls-authmgr-plugin because, for me, the role separation is more important than controlling who can see which links, but I really would like to see YOURLS improved to support both at the same time rather than having to choose between them.

@pcolmer how about Auth Manager Plus, I think it does everything you are mentioning here. I have yet to add admin interface and other features, but under the hood it seems to run pretty good.

Thanks, @joshp23. I've upgraded our installation to use it.


VCS support in MakeCode App

Would it be possible to use a VCS like Git in the MakeCode App? Then you wouldn't have to use the online webpage just for version control which is essential for larger projects.

There is already github support for projects in arcade which can be used as version control.

I do not see one in the app:

(I heard that it's disabled cause when you are using the app you are not expected to have internet so I was thinking of maybe doing the same thing with Git)


add GCR airdrop

gm i am doxxed leader of Golden Celestial (GCR) - contact directly on X if you have any comments or questions. thank you!
Golden Celestial Ratio (GCR) on X


[TACHYON-639] Avoid copying ByteBuf data to DataByteBuffer in RPCBlockResponse#decode()

https://tachyon.atlassian.net/browse/TACHYON-639
@gpang

Jenkins, test this please.

Jenkins, test this please.

@gpang , @calvinjia , please review this PR

@gpang Thanks for the comments. I have either fixed your comments or replied. Please check.

Thanks for this great patch @ooq ! I left a few more questions and comments.

@gpang Thanks for the timely feedback. Updated again!

@calvinjia  Thanks for the comment. Fixed it.

Thanks @ooq ! I left a few small questions/comments.
Are there any unittests or integration tests for this new code path? (You can put them in the next PR)

Thanks @gpang and @calvinjia  for the comments! Updated again.

@calvinjia Just fixed your comments.
@gpang Yeah, there should be some integration test. The test should do check if the reference count of the underlying Netty buffer goes to zero, if we use a NettyRemoteBlockReader/RemoteBlockInStream to fetch a remote block/file. To be added in a separate JIRA.

@apc999 Addressed your comments and added unit tests for DataNettyBuffer.

@ooq looks good overall, is there anything that could prevent a ByteBuf from being closed? Or any way to enforce ByteBufs are not leaked (ie. through invariants or testing)?

@calvinjia In either situation, when we actually release the ByteBuf (when closing a NettyRemoteBlockReader for example), DataNettyBuffer.release() will return false. In the case of NettyRemoteBlockReader, we throw an IOException during close(). The invariant here is that the reference count of the ByteBuf during release might be 1.

@calvinjia Thanks for the suggestion. Updated.

Fixed! @calvinjia

I added a few minor comments and questions. Thanks!

@gpang Thanks for keeping the high standards of code quality. I have fixed your comments.

LGTM

+1 LGTM


Export platform logger constructor so it can be used outside of Fx

Description of change
Checklist

[ ] Tested in playground or other setup
[ ] Screenshot (Grafana) from playground added to PR for 15+ minute run
[ ] Documentation is changed or added
[ ] Tests and/or benchmarks are included
[ ] Breaking changes

Summary by OpenAI
New Feature:

Added a constructor for creating loggers outside of the package
Refactored provideLogger function to use the new constructor
Introduced support for additional writers and their closing mechanism

🎉 A logger's tale, now more versatile,
With constructors new, and writers that compile.
Refactored with care, our code does gleam,
Closing writers with ease, like a well-oiled machine. 🚀

@kklimonda-fn I am using the PR as a test bed for openai code review tool! :)
Please ignore excessive comments.


Curtain motor - TS0601 - (Tuya cluster)

Brand: Tuya
ManufacturerName: Tuya
Product ID: TS0601
Device ID: 81
Profile ID: 260
https://www.aliexpress.com/item/4000739390813.html
Please let me know if you need more information or help testing. You can also find me in the Homey Community slack under @ Bert

Manufacturer ID is _TZE200_zah67ekd
Product ID is TS0601

"ids": {
"modelId": "TS0601",
"manufacturerName": "_TZE200_cowvfni3"
},
"endpoints": {
"endpointDescriptors": [
{
"endpointId": 1,
"applicationProfileId": 260,
"applicationDeviceId": 81,
"applicationDeviceVersion": 0,
"_reserved1": 1,
"inputClusters": [
0,
4,
5,
61184
],
"outputClusters": [
25,
10
]
}
],
"endpoints": {
"1": {
"clusters": {
"basic": {
"attributes": [
{
"acl": [
"readable",
"reportable"
],
"id": 0,
"name": "zclVersion",
"value": 3
},
{
"acl": [
"readable",
"reportable"
],
"id": 1,
"name": "appVersion",
"value": 71
},
{
"acl": [
"readable",
"reportable"
],
"id": 2,
"name": "stackVersion",
"value": 0
},
{
"acl": [
"readable",
"reportable"
],
"id": 3,
"name": "hwVersion",
"value": 1
},
{
"acl": [
"readable",
"reportable"
],
"id": 4,
"name": "manufacturerName",
"value": "_TZE200_cowvfni3"
},
{
"acl": [
"readable",
"reportable"
],
"id": 5,
"name": "modelId",
"value": "TS0601"
},
{
"acl": [
"readable",
"reportable"
],
"id": 6,
"name": "dateCode",
"value": ""
},
{
"acl": [
"readable",
"reportable"
],
"id": 7,
"name": "powerSource",
"value": "mains"
},
{
"acl": [
"readable",
"writable",
"reportable"
],
"id": 65502
},
{
"acl": [
"readable",
"reportable"
],
"id": 65533,
"name": "clusterRevision",
"value": 2
},
{
"acl": [
"readable",
"reportable"
],
"id": 65534,
"name": "attributeReportingStatus",
"value": "PENDING"
},
{
"acl": [
"readable",
"reportable"
],
"id": 65504
},
{
"acl": [
"readable",
"reportable"
],
"id": 65505
},
{
"acl": [
"readable",
"reportable"
],
"id": 65506
},
{
"acl": [
"readable",
"reportable"
],
"id": 65507
}
],
"commandsGenerated": "UNSUP_GENERAL_COMMAND",
"commandsReceived": "UNSUP_GENERAL_COMMAND"
},
"groups": {
"attributes": [
{
"acl": [
"readable",
"reportable"
],
"id": 0,
"name": "nameSupport",
"value": {
"type": "Buffer",
"data": [
0
]
}
},
{
"acl": [
"readable",
"reportable"
],
"id": 65533,
"name": "clusterRevision",
"value": 2
}
],
"commandsGenerated": "UNSUP_GENERAL_COMMAND",
"commandsReceived": "UNSUP_GENERAL_COMMAND"
},
"scenes": {
"attributes": [
{
"acl": [
"readable",
"reportable"
],
"id": 0
},
{
"acl": [
"readable",
"reportable"
],
"id": 1
},
{
"acl": [
"readable",
"reportable"
],
"id": 2
},
{
"acl": [
"readable",
"reportable"
],
"id": 3
},
{
"acl": [
"readable",
"reportable"
],
"id": 4
},
{
"acl": [
"readable",
"reportable"
],
"id": 65533,
"name": "clusterRevision",
"value": 2
}
],
"commandsGenerated": "UNSUP_GENERAL_COMMAND",
"commandsReceived": "UNSUP_GENERAL_COMMAND"
}
},
"bindings": {
"ota": {
"attributes": [],
"commandsGenerated": "UNSUP_GENERAL_COMMAND",
"commandsReceived": "UNSUP_GENERAL_COMMAND"
},
"time": {
"attributes": [],
"commandsGenerated": "UNSUP_GENERAL_COMMAND",
"commandsReceived": "UNSUP_GENERAL_COMMAND"
}
}
}
}
}

Hello, I've just recived a similar device.
It seem recognized by my homeypro as zigbee device, but not as curtain motor :-(
zb_product_id	"TS0601"
zb_manufacturer_name	"_TZE200_xu4a5rhj"
If I can help you somehow to develop the driver, please let me know how

I'm having problems with connecting my device, which is also a TS0601, but with a different manufacturer name: _TZE204_xu4a5rhj, which is not in the supported list. Can I help or do something (in code, or at the motor itself). I'm quite new to coding, but keen to learn...


Completions should be aware of infix backticks

Doing completion on
"a" `should

gives the option shouldBe, but the completion/snippet does not have the closing backtick, and is presented as requiring two more params.

I would like to take a look at this in the next week, any pointers which files are responsible for this behaviour?

@fendor This is the function you need to change https://github.com/haskell/haskell-ide-engine/blob/master/src/Haskell/Ide/Engine/Support/HieExtras.hs#L323


R improvements

To my knowledge, I just completed the first ever pre-commit.ci run with R 🎉 (https://results.pre-commit.ci/run/github/403327126/1631911523.jRvhF2_RSR-hbEgpnbu7DA). There are a few things to improve though:

Although I suggested to --without-recommended-packages, I think we should not do that because some recommended packages are hard to build from source (when initializing the environment) and have system dependencies, and it won't add much to the size of the image to include it.
I am still wondering if we can make the global renv cache work under /opt/R. Unit tests should cover this, so I might submit a PR.
After 120s, it seems like the installation of the environment is timed out. While this seems to make sense for most repos that host only one hook, mine hosts a lot of hooks and hence installation takes a bit longer (but is cached thanks to your work). There are two major R repos from which to get packages, and both only support binaries for either Linux or Windows/macOS. I assume most users will not use Linux, so I prefer to have binary R package install for them locally, which means Linux package installs will be from source. Hence, I propose to increase the timeout limit. Not sure what's needed here for my hooks, should we try to double it? The reason I managed to complete a run was because I temporarily switched to binaries for Linux.
I anticipate some problems with missing system dependencies as some hooks have additional_dependencies:. In particular, my {roxygen2} hook requires the user to specify all runtime dependencies of his package as additional_dependencies: because {roxygen2} requires them (to generate the docs). There are tools in R to install these system dependencies, but they rely on apt and hence don't put the installed packages into /opt/R. We can also postpone this for now I think.

Although I suggested to --without-recommended-packages, I think we should not do that because some recommended packages are hard to build from source (when initializing the environment) and have system dependencies, and it won't add much to the size of the image to include it.

If there's specific ones to add that seems fine, building all of them doesn't seem productive though (for instance I don't think a linter will need generalized fortran extensions, hopefully).

I am still wondering if we can make the global renv cache work under /opt/R. Unit tests should cover this, so I might submit a PR.

The only persistent state which is kept between runs is inside the environment directory.  Additionally runs are run as an unprivileged user which does not have write permission to the /opt directories (again, intentionally).  As such, I don't think that enabling /opt/R as a cache helps or is possible.

After 120s, it seems like the installation of the environment is timed out. While this seems to make sense for most repos that host only one hook, mine hosts a lot of hooks and hence installation takes a bit longer (but is cached thanks to your work). There are two major R repos from which to get packages, and both only support binaries for either Linux or Windows/macOS. I assume most users will not use Linux, so I prefer to have binary R package install for them locally, which means Linux package installs will be from source. Hence, I propose to increase the timeout limit. Not sure what's needed here for my hooks, should we try to double it? The reason I managed to complete a run was because I temporarily switched to binaries for Linux.

I don't really want to bump this up, it's already quite long and any increase here significantly increases my risk from a cost and abuse perspective.

I anticipate some problems with missing system dependencies as some hooks have additional_dependencies:. In particular, my {roxygen2} hook requires the user to specify all runtime dependencies of his package as additional_dependencies: because {roxygen2} requires them (to generate the docs). There are tools in R to install these system dependencies, but they rely on apt and hence don't put the installed packages into /opt/R. We can also postpone this for now I think.

yeahhhh that's not really going to work so well.  those'll probably just be unsupported (?).  It seems a little bit weird to build docs as part of pre-commit -- I'd usually leave that to a testing phase

Ok, thanks for the explanations. I found ways to work around these limitations except for the system dependencies.

Except one problem:
failed to create directory at path '/tmp/renv'

Traceback (most recent calls last):
19: suppressWarnings({
        old <- setwd("/pc/clone/dynByPtYQnevuTTk-fDJ5w/renv-default")
        source("renv/activate.R")
        setwd(old)
        renv::load("/pc/clone/dynByPtYQnevuTTk-fDJ5w/renv-default")
    })
18: withCallingHandlers(expr, warning = function(w) if (inherits(w,
        classes)) tryInvokeRestart("muffleWarning"))
17: source("renv/activate.R")
16: withVisible(eval(ei, envir))
15: eval(ei, envir)
14: eval(ei, envir)
13: local(...)
12: eval.parent(substitute(eval(quote(expr), envir)))
11: eval(expr, p)
10: eval(expr, p)
 9: eval(quote(...), new.env())
 8: eval(quote(...), new.env())
 7: renv_bootstrap_load(project, libpath, version)
 6: renv::load(project)
 5: renv_load_project(project)
 4: ensure_parent_directory(projects)
 3: ensure_directory(unique(dirname(path)))
 2: stopf("failed to create directory at path '%s'", rownames(info))
 1: stop(sprintf(fmt, ...), call. = call.)

E.g. in https://results.pre-commit.ci/run/github/186313720/1632590538.J-B6BzAhSkSCbs91JsIBUQ. This pops up randomly I feel, i.e. not in every run. You set RENV_PATHS_ROOT in the Dockerfile so the root of the cache will be there. And then renv probably wants to use it (I guess not a problem because during running hooks it does not want to install packages) or at least ensure the directory exists.
I think we should rather disable the cache altogether with RENV_CONFIG_CACHE_ENABLED=False as per the docs (either R option renv.config.cache.enabled or in the above form as an env variable). I can file a PR to that effect if you want.

I wonder if that's renv not being multiprocess safe (?)  does it happen with require_serial: true
disabling the cache also seems fine 👍

Seems like you are right, I need to further investigate thread/process safety with {renv}. Not sure but maybe #94 can resolve the problem, at the very least I don't think it makes anything worse.


Add NN syntax for assigments

Needed for Nancy's project
(This will cause conflicts with ThingTalk 2.0, but nothing major)

Pull Request Test Coverage Report for Build 1150

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
61 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.02%) to 80.17%

Files with Coverage Reduction
New Missed Lines
%

lib/nn-syntax/parser.js
61
68.13%

Totals

Change from base Build 1149:
0.02%

Covered Lines:
9229

Relevant Lines:
11050

💛  - Coveralls


Unexpected behaviour when using nodejs-pubsub to load balance message to subscribers

Environment details

OS: Windows
Node.js version: 10.15.3
npm version: 6.11.3
@google-cloud/pubsub version: 0.31.0

I want multiple subscribers to process messages one message at a time. From my understanding this can be achieved by using the following settings:
const subscription = pubsub.subscription(subscriptionId, {
    flowControl: {
      allowExcessMessages: false,
      maxMessages: 1
    },
    streamingOptions: {
      maxStreams: 1
    }
  });

Based on the current unacked message count further subscribers will be started/stopped to reduce overall processing time.
In case the subscription has more unacked messages than currently running subscribers i would expect any newly started subscriber to instantly pickup a message and start processing.
Instead my observation is that a newly started subscriber will never pickup a preexisting unacked message except in a situation where no subscriber was running. It seems as the previous running subscribers are putting a hold on messages even though they are not currently processing them. I think my above settings should prevent from such behaviour.
In contrast my load balancing scenario works fine if i start multiple subscribers and add multiple messages to the topic afterwards.

I think I have an idea of what is happening here. The configuration you've shared is indeed the only way to process a single message at a time, however it should be noted that PubSub doesn't actually let us configure the number of messages to send down at any given time.
I think what is happening is you are receiving a batch of messages (could be in the hundreds) and we give you the desired amount (in this case 1) while we continuously extend the ack deadline for any others until you're ready for them. This essentially puts a lock on these messages and prevents them from being redelivered.
You could try setting the flow control maxExtension option to 0, although this will result in a frequent redeliveries if you have a short ack deadline.

In my use case processing a single message can take multiple (up to ~20) minutes. I tried various scenarios to validate your guess. I'll try to illustrate it with an example where messages are published one by one to prevent batches of multiple messages:

start subscriber A
publish message 1
-> subscriber A processes message 1
publish message 2
-> subscriber A still processes message 1, message handler not called for message 2
start subscriber B while message 1 is still processed by subscriber A

I would expect subscriber B to immediately pickup message 2.  It looks like subscriber A put a lock on message 2, which from my understanding should not happen using my above configuration.
I tried the same scenario with maxExtension: 0. This seems to prevent from extending the ack deadline of message 1 while it is being processed. While i would like to have the "queued" or locked messages to be redelivered i need the ack deadline for the currently processed message to be extended as long as processing finishes or fails (in this case .nack() would be called).

@nicolasguenther by any chance are you creating (not starting) subscriber B after message 2 is published?

I create and start subscriber B after message 2 is published. I would expect subscriber B to immediately pickup message 2 when started.

@nicolasguenther I had to do a double check on the subscriber docs, but it sounds like what you're seeing is happening by design.

A message published before a given subscription was created will usually not be delivered for that subscription. Thus, a message published to a topic that has no subscription will not be delivered to any subscriber.

Thanks for pointing this out. I think it is crucial to distinguish between the terms subscription and subscriber. I'm using a single subscription with multiple subscribers as described in the same document:

Multiple subscribers can make pull calls to the same "shared" subscription. Each subscriber will receive a subset of the messages.

My subscription was created before any message was published. My subscriber B (the node program using nodejs-pubsub) in the above scenario was indeed started after message 2 was published.
When i publish a message with no running subscribers and start a subscriber afterwards, the message is delivered to the subscriber, Interestingly enough.

@nicolasguenther Thanks for clarifying and sharing all these details about your application. I think you may want to consider using the pull API.
Since your application only wants to process 1 message at a time and it can take a bit of time, it seems like the streaming API doesn't offer you very many advantages, especially since we can't control which streams/subscribers messages are actually delivered to. Using the pull API you can specify the exact amount, so any subsequent messages won't get locked in a subscriber you don't want them in.
WDYT?

Thanks again. I think obtaining a subscription with pubsub.subscription() (as i did) is implicitly using the pull API as described in  Subscription Docs:

The subscription will pull for messages automatically as long as there is at least one listener assigned for the message event. By default Subscription objects allow you to process 100 messages at the same time. You can fine tune this value by adjusting the options.flowControl.maxMessages option.

I still don't understand why using this with the above flow control settings and streaming options doesn't behave as expected. I see that using pubsub.v1.SubscriberClient() like in the example you linked could give me more fine-grained control, though. So i'll give it a try in the next days.

@nicolasguenther so we offer 2 clients, a handwritten one (pubsub.subscription()) and a generated client (pubsub.v1.SubscriberClient). We don't actually use or support the pull RPC in our handwritten client (which you appear to be using), we use a streaming version instead.
It is pretty reasonable to expect messages to stop being delivered to your subscriber once you've hit the configured limit, but currently grpc (our transport library) doesn't actually let us stop the flow of data until an unconfigurable buffer is filled. So what ends up happening is we just manage those messages behind the scenes until you're ready for them.

There hasn't been a response on this for a while, so I'm going to go ahead and close it, but please feel free to re-open and/or comment further if there is still a problem.


Url.Action return null when action method name has ~Async as suffix

Describe the bug
Url.Action return null when action method name has `Async as suffix.
To Reproduce
Steps to reproduce the behavior:

ASP.NET Core 3.0
Run this code

        [HttpGet("{id:int}")]
        public async Task<ActionResult<string>> GetByDate([FromRoute] int id)
        {
            throw new NotImplementedException();
        }

        [HttpGet("{id:int}/detail")]
        public async Task<ActionResult<string>> GetDetailByDateAsync([FromRoute] int id)
        {
            throw new NotImplementedException();
        }

        [HttpGet("url")]
        public string GetUrl()
        {
            return "The result is " + Url.Action("GetDetailByDateAsync", new { id = 123 });
        }

        [HttpGet("url2")]
        public string GetUrlDetail()
        {
            return "The result is " + Url.Action("GetByDate", new { id = 123 });
        }

in any controller

Run Server and check out about 'url', 'url2' result.

in url

in url2

target action methods of url and url2 has difference name.
Expected behavior
url is works in asp.net core 2.2, In my think, This changes affected to Url.Action.
Even if this is not bug, this give confusing to IDE or old users. IHMO, Breaking Changes document need to make detail more.
Temporary Solution for old users.
Someone experience this issue, please setup the this options to avoid mis-matching action name isuue.
// or AddControllers, Add* about MVC
services.AddMvc(options =>
    options.SuppressAsyncSuffixInActionNames = false);

Screenshots
This is my confusing point. IDE tell me about below options can be used but, actually default name strategy is changed. So It's not.

And document didn't tell me exactly change about changed Url.Action behavior.
Thank you. If you think about this is not bug. Please close this issue.
Best Regards,

@enif-lee the workaround you have listed would be the right way to go about it. We announced this breaking change early as part of the 3.0 release - https://github.com/aspnet/Announcements/issues/351 - and the change in behavior is documented as part of our migration guide - https://docs.microsoft.com/en-us/aspnet/core/migration/22-to-30?view=aspnetcore-3.0&tabs=visual-studio#async-suffix-removal-from-controller-action-names. We're planning on revisiting the experience around this in the next release -https://github.com/aspnet/AspNetCore/issues/14103 - since this has been a fairly common pain point for our users.


Upgrade to k3s v1.21.x

I need to keep an eye on k3s v1.21.x for when it reaches stable channel. Will note any possible issues once that investigation is in full force.

On 1.21.2 right now.


Inf_maze

Your checklist for this pull request
Author name
Author: Pinyuan zheng

About your game
DO THE MAZE  and hope you go though the right way

Use WASD and go though finish lines as the end  and make it to the real ending also there are softlock where you just got to restart to bring more fun and more to the time also there is only one map for the win and you could miss it so look out

Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

That how it works I made some of the maps so that you are just stuck and you have to restart  cause it is fun to restart and also there are map maps and there is only one winning map but if you go the wrong way for the map then you get to make more fun to get back to the map for the win

It a part of the game
On Fri, Dec 20, 2024 at 1:52 PM Gus Ruben @.***> wrote:

@.**** requested changes on this pull request.
Thanks for your submission! It seems clear that there was a lot of effort
put into this game, but I can't seem to figure out how to get past this
level! If this is just a mistake on my part you should let me know, though
you could consider adding clearer instructions if it's a misunderstanding
and not a puzzle. If this is a bug, though, I'd be happy to approve your
game after it's fixed!
image.png (view on web)
https://github.com/user-attachments/assets/b6abd95a-1f93-4977-aa05-d0e52db66f33
—
Reply to this email directly, view it on GitHub
https://github.com/hackclub/sprig/pull/2708#pullrequestreview-2518144318,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BNWA6TWW7NDJCWRDXRWZFJD2GRROPAVCNFSM6AAAAABTSWFB2SVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMJYGE2DIMZRHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Sub category display

upgrade NodeBB 1.0.3 to 1.1.0
v1.0.3 sub category is a block, v1.1.0 is item
This is Vanilla

This is Material

Just because vanilla theme displays subcategories as cards doesn't mean every other theme has to display them the same way. Every theme can display items on pages they way it wants.
Anyway, material theme has a settings page in the admin control panel where you have an option to display the subcategories as cards.


Create save states for version 2.8 of Bizhawk

For now, only the following versions of Bizhawk have had save states created for the original trials:

v2.9.1
v2.8
v1.13.2

The script checks the version of Bizhawk being used in a rather crude fashion, which will likely only allow these specific versions to work, and possibly other v2.9.x versions.

Resolves #3


first element of _locks field in ConcurrentDictionary points to itself in .net Core 6.0

Description
The first element of _locks field in ConcurrentDictionary.cs points to itself, resulting in infinite recursion and stackoverflow exception when traversing and taking an action such as deep-cloning etc.
We Expect that the first element should not point to itself. Similar to 4.8 where it is a simple array of objects.
Configuration
Windows although not relevant.
Regression?
It did not cause any issues in .NET framework 4.8
Other information
None

Which version are you using?

Version=<IP_ADDRESS>

Hi @mairaw Is the information provided above sufficient?

Do you have a piece of code which uses ConcurrentDictionary where you can repro the infinite recursion or stack overflow?


EthStorage Devs Meeting #74 Agenda

Meeting Info

Apr 02, 2024, 11:00 UTC+8
Duration: 60 minutes
Meeting link
Anyone can comment on the agenda where they would like this to be discussed.

Agenda

Protocol/App/Research progress

Public testnet progress track

EigenLayer integration (AVS? EigenDA?)

Added a e2e test to SoulETH pr
Analyse and discuss about FaultDisputeGame in depth with Po and Bill
Added deposit/withdraw function to the BatchInbox contract.
Added logic to handle inbox contract execution failure for batch submit here; But the deriving side still needs more investigation, as it's also called by op-programe, so need to handle preimage-oracle related logic.

EthStorage -- sync

Fix issues (https://github.com/ethstorage/es-node/issues/253, https://github.com/ethstorage/es-node/issues/262, https://github.com/ethstorage/es-node/issues/261)

Public testnet

Discord
https://github.com/ethstorage/ethstorage-doc/pull/23
Blob api for OP integration

modify the dispute game feature to 4ary, add unit tests
Verification steps


Framework Design

[ ] definition of interfaces of core
[x] definitions of components

Components defined in #7

Core(class) itself does not define an interface for other components.
Core(module) interface is defined as the set of all components in the core module.


A bunch of fixes and partial addressing #3614

Nice :D ...
Remember I blew up Display.controller so that is causing your merge conflicts.

Oh, I thought I already rebased... oh well.

I noticed you moved attachments to be exposed to triggers, there was a particular reason?

Codecov Report

Merging #3620 (59e910e) into development (7022262) will increase coverage by 6.59%.
The diff coverage is 79.06%.

@@                Coverage Diff                @@
##             development    #3620      +/-   ##
=================================================
+ Coverage          30.63%   37.23%   +6.59%
+ Complexity         15875     6924    -8951
=================================================
  Files                427      214     -213
  Lines              74268    37264   -37004
=================================================
- Hits               22755    13874    -8881
+ Misses             51513    23390   -28123

Impacted Files
Coverage Δ

sources/ElkArte/Controller/Post.php
34.69% <0.00%> (-7.07%)
:arrow_down:

sources/ElkArte/Controller/ProfileOptions.php
6.17% <0.00%> (-7.24%)
:arrow_down:

sources/ElkArte/Controller/Register.php
37.25% <ø> (+0.32%)
:arrow_up:

sources/ElkArte/IlaIntegrate.php
3.94% <ø> (+0.31%)
:arrow_up:

...urces/ElkArte/MessagesCallback/DisplayRenderer.php
88.00% <ø> (-2.20%)
:arrow_down:

sources/ElkArte/Search/Search.php
0.00% <ø> (-52.35%)
:arrow_down:

sources/subs/Attachments.subs.php
11.01% <ø> (-6.60%)
:arrow_down:

themes/default/Theme.php
68.65% <ø> (ø)

themes/default/Display.template.php
51.97% <50.00%> (-0.02%)
:arrow_down:

themes/default/Post.template.php
47.66% <50.00%> (+0.01%)
:arrow_up:

... and 355 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 8b4e18b...59e910e. Read the comment docs.

I noticed you moved attachments to be exposed to triggers, there was a particular reason?

I was working on a new feature which will add some microdata to the templates, notably display.  Having access to the attachments allows to build a more rich set of micro data.  It specifically adds open graph and schema.org style data, one for when you share a post, and one to help improve search results.  The way I did it was to tie into the display events and so needed the attachments to be a class property so I could use them.  It only really needs access if there is an attachment on the first post as that is what is used to build the data.

I forgot to mention I was working on your comment above... oh well, will open a new PR later!

Cool .. After I merged this, I was like, hey where can I get access to the attachment array 🙈
The previous display render data had the inline attachments in those results and then the display class had the below post attachments in the the attachment array that I had placed as a property to gain access.
Anyway glad you are working on it !

I had forgotten to enable the new attachment module, so now I see the attachment array in the display renderer.  The only thing that is missing really are the ILA attachments which are excluded from that array.  May just set a flag in that array, like show_inline and let the template exclude them from showing below the post vs not having the data passed at all?

hmm... if that is the case, then, since I'm facing a little issue with my original idea, a simple solution could be to, instead of "skipping" the ILA attachments from the $output['attachments'] array, to add an index like "ila" that would be then used at theme-level to ignore those that are presented in-line.
And probably, in a certain way, this is an even better solution, because it leaves the final decision of how to visualize the attachments to the theme (for example it could open the door to the possibility to present the ILA attachments in the download area as a simple link because reasons).

A separate array is good as it would keep old themes working as intended.

Ah, for some reason it was not that way on my install, I had to go and add it.
So for the rest I put a couple comments in the code, but template_load_likes_button() as a function does not exist and if (!empty($context['likes'])) is not set due to the way random is loading (or not) the integration function, which when called will properly load the likes template (_above) as defined in the GenericMessages template.  I hope that makes sense and its not that I'm going mental.

@emanuele45 Sorry to be 🐛 I'm looking at the ILA fix, where we return both an attachment data and the ila data array.  Thats easy enough but while working on that (so I can do a bit more on the micodata) I noticed a little issue.
The way the code is now set, it did not load attachments array if it was an ILA.  The problem is if its an unapproved attachment then you don't get the option to approve it from the display page  as the attachment is simply not listed. OK easy fix to not load/continue the ILA array unless its also approved, that way the attachment array still gets it, allowing easy approval.
No what should we show in the post for an unapproved ILA?  Right now I have it as our default mime image, or it could be as simple as an X mark, or maybe an empty outline box, or use a text to image or waiting approval?

Another thought is just the paperclip icon or simply nothing at all.

Spuds wrote on 4/3/22 8:33 PM:

@emanuele45 https://github.com/emanuele45 Sorry to be 🐛 I'm looking
at the ILA fix, where we return both an attachment data and the ila
data array. Thats easy enough but while working on that (so I can do a
bit more on the micodata) I noticed a little issue.
The way the code is now set, it did not load attachments array if it
was an ILA. The problem is if its an unapproved attachment then you
don't get the option to approve it from the display page as the
attachment is simply not listed. OK easy fix to not load/continue the
ILA array unless its also approved, that way the attachment array
still gets it, allowing easy approval.
No what should we show in the post for an unapproved ILA? Right now I
have it as our default mime image, or it could be as simple as an X
mark, or maybe an empty outline box, or use a text to image or waiting
approval?

hmm... good point.
Another option could be a dedicated message:
         $img = new TextImage($text);
         $img = $img->generate(200);

         $this->prepare_headers('no_image', 'no_image', 'image/png',
'inline', 'no_image.png', true, false);
         Headers::instance()->sendHeaders();
         echo $img;

         obExit(false);

Though it would imply some more server processing and it may be rather
ugly. :D
An "x" or a clip may lead thinking there is a problem?
Maybe really a simple text message injected into the post saying
something along the lines "when the image will be approved it will
appear here"?

Nods ... I was also concerned about using a default image, or an x or clip as it would likely generate the ever useful "Hey your attachment is broken" responses.  The text2image is no worse then making a thumbnail   I'll see what may work and go from there!

Spuds wrote on 4/3/22 1:10 AM:

In sources/ElkArte/Modules/Random/Display.php
https://github.com/elkarte/Elkarte/pull/3620#discussion_r841128990:

@@ -74,6 +76,23 @@ public static function followup_message_list($messages)
$context['follow_ups'] = followupTopics($messages, self::$includeUnapproved);
}

/**

Show likes.

@param int[] $messages

*/
public static function load_likes($messages)
{

global $modSettings, $context, $txt;

if (!empty($modSettings['likes_enabled']) && !empty($messages))

{

	require_once(SUBSDIR . '/Likes.subs.php');

	$context['likes'] = loadLikes($messages, true);

	theme()->getLayers()->addBefore('load_likes_button', 'body');

Which this being loaded, there should be no need to call the
template_load_likes_button_above() from display template init ?

Indeed, this is what I was looking for, I didn't check that part of the
reply before.
The part
     if (!empty($context['likes']))
     {
         template_load_likes_button();
     }

can be removed without issues.

Spuds wrote on 4/3/22 1:09 AM:

So for the rest I put a couple comments in the code, but
|template_load_likes_button()| as a function does not exist

Ohh... that's indeed another problem.
I think I see what happened here.
Let me try to fix it.
IIRC I changed my mind midway, fixed it somewhere else, but forgot tu
push the fix (or something like this).


feat(history): add sticky headers to history sections

description

sticky section headers for the history section
see GIF

This is awesome!!! I'll make sure all of your PRs get into the release tomorrow


Erreur perms

hello i'm french, i have this error :
Adding` SSH key
Creating local backup: "Automated backup 2019-02-19 16:45"
Backup created: fd4d06a9
Copying fd4d06a9.tar to /volume1/homes/MrJuJu0319/backup_hassio on <IP_ADDRESS> using SCP
Warning: Permanently added '[<IP_ADDRESS>]:5100' (ECDSA) to the list of known hosts.
Permission denied, please try again.
Permission denied, please try again.
<EMAIL_ADDRESS>Permission denied (publickey,password).
lost connection

end my config is:
{
  "ssh_host": "<IP_ADDRESS>",
  "ssh_port": 5100,
  "ssh_user": "MrJuJu0319",
  "ssh_key": [
    "-----BEGIN RSA PRIVATE KEY-----",
    "MIIEowIBAAKCAQEAq2wieTSZAgXdnVQSOJKh5JzL+YHayfiHt8bYRruzNhP2uFoJ",
    "34OKRijGmEA+Ku1Rk4epQcJnHNWpcX0Fcyv73JiLuRlK8ngtU89LDYFkCrh5YaNh",
...
    "GhuFfJD0Cn2TonfSorS9tlOXHpkOr9K4jnguiN3b9oBSXnYCgwN8CrzUrCK+eVoT",
    "smFbcafQymPFcsAJrCQg52quktDq+X3jkpJxhbxks1lIWlvt4iiV",
    "-----END RSA PRIVATE KEY-----"
  ],
  "remote_directory": "/volume1/homes/MrJuJu0319/backup_hassio",
  "zip_password": "",
  "keep_local_backup": "5"
}

I need help

exactly the same happens to me , one thing i found strange is why the ssh key needs to be segmented and separated by commas ,  will that maintain the key correctly concatenated?, do you know if it is possible use password instead of ssh key?

No for is moment
Le sam. 2 mars 2019 à 22:51, gus<EMAIL_ADDRESS>a écrit :

exactly the same happens to me , one thing i found strange is why the ssh
key needs to be segmented and separated by commas , will that maintain the
key correctly concatenated?, do you know if it is possible use password
instead of ssh key?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mr-bjerre/hassio-remote-backup/issues/18#issuecomment-468963758,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APgSL5ChIWdcD8Ism-h-QED_8C7Zmvwfks5vSvJqgaJpZM4bDJdh
.

Im having this issue as well connecting Hassio to Unraid. When I had HA in docker I could get this to work but not with this add on. Iv tried every way I could think of. Any suggestions?


Language server should use abbreviated report header for diagnostic message

Currently the diagnostic message that is sent by the language server is generated by Report::render_ci (called by into_lsp_diagnostic).
This is designed to be displayed in the console and includes a header line like
── UNUSED DEFINITION ──────────────────────────────────────────── filename.roc ─

But it looks unpleasant when displayed inline in the text editor (Neovim in my case – I don't know how other editors will display the message):
Position : { row : Index, col : Index }     ■ ── UNUSED DEFINITION ──────────────────────────────────────────── filename.roc ─

The line is too long
Including the filename is unnecessary, I already know which file I am currently editing
Using all caps should not be necessary to make it stand out

I would expect that it looks like this:
Position : { row : Index, col : Index }     ■ Unused definition

Furthermore (which I've omitted above because it currently is outside the right border of my editor window), all lines of the report (which look nice when printed to the console or when expanding the diagnostic in an overlay window) are joined into one line, which would now become visible and would look something like that:
Position : { row : Index, col : Index }     ■ Unused definition `Position` is not used anywhere in your code.    30│  Position :  […]

([…] abbreviated by me, it would actually go on)
I'm not sure if LSP supports distinguishing a "single line"/"preview" and an "expanded"/"full" diagnostic message or if it is unavoidable that the full content will be shown in the preview. Or if this is an issue with how Neovim displays multi-line diagnostics.

Thanks for bringing this to our attention @mkrieger1! We should investigate the possibility of customization here :+1:

In the meantime I have added a workaround for this in my Neovim setup, in case anyone is interested: https://gist.github.com/mkrieger1/27da5bb601d3c7ee87df224895a3db89


Updated the readme to include the latest cronjob config

fixed #206

@wombat There are merge conflicts. Can you please resolve them?

@arun-gupta Apparently another PR (#212) was merged quicker - thats why the conflict came up.


FIX checklist name

Fixed checklist name in Logic App checklist.
Thanks for the heads up @UmarMohamedUsman!

Duplicate with #675. I will take that one and close this, since it includes other changes (such as the addition of the service fields)


Use ImageKit

Install and add to the project ImageKit as image manager.
Here is the link for installation:  https://django-imagekit.readthedocs.io/en/latest/
The next step is using CDN as an image storage.

cd52d8864ae200a78e4149b149d31998315c0ca0


Timesheet - 05/05/2021 - Mr. K

Task 1
Worked on Download Wallet Page Hifi design. Updated the design according to the feedback.
Proof of Work

https://www.figma.com/file/dniKTujBGiZmRha4OrtCOT/Download-Wallet?node-id=1%3A10

Time Spent
5.0 hours
Account Number
c93ee24b89991a1438b45444cf06e5bb167ed1796d54a800174def4480f98ed0

Approved


Cannot specify SSL verify_mode or CA chain

I'm having trouble connecting to an SSL-cert-protected Splunk instance because there is no way to set the SSL verify_mode to VERIFY_PEER and pass in a CA chain. I can set my cert and key but since there is no way to pass the full CA chain which contains the required intermediate certs, then authentication fails.
The root of the problem is that the verify mode is currently hard-coded to VERIFY_NONE in lib/splunk-sdk-ruby/context.rb. This seems to be analogous to the --insecure curl flag, which doesn't just ignore the server cert but also breaks authentication when intermediate certs need to be passed in.
A potential fix to the splunk sdk would be to allow verify_mode and the path to the CA chain to be passed in, which would then be set inside context.rb (the verify_mode and ca_file attributes on Net::HTTP and ssl_context.verify_mode). I can provide a full example if that would help.
Is it likely that this could be fixed? I really can't think of a way of working around this without actually changing the sdk.

Version of project: 1.0.5
Platform version: Mac OS X and Linux
Framework version: Ruby 2.2.0
Splunk version: Splunk 6.1.3

@MatthewDooler thanks for the bug report. A couple of things:

Do you think it would be possible to get an example, as you suggested? Basically, the config on the Splunk side (with the relevant certs) and then the config and certs on the SDK side? That way it should be a lot easier to see what the fix should be.

As always, we're open to Pull Request contributions to fix the issue :)

I don't have admin access to the Splunk instance so can't pull down the exact config, but basically the setup is like this:

there's a root CA and two intermediate CAs issued by the root (call them CA1 and CA2).
the server (Splunk) has a cert issued by CA1 and only trusts clients presenting certificates issued by the CAs in its trust chain (which are CA1 and the root).
the client has a cert issued by CA2. The server has no knowledge of CA2 but will trust it if the client sends the CA2 cert when making the request (in addition to its own client certificate). This works because the server trusts the root CA, implicitly trusting CA2 or any other child of the root provided it can verify the intermediate cert was actually issued by root. It can only do this if it's actually sent a copy of the intermediate cert.

Does this help clear things up? If not, the fix should actually be quite easy so I can submit a PR if it's likely to be accepted!

@matthewdooler Can I relate this https://gist.github.com/imechemi/785f839a1f162e52096f6b4216538a41 as well over here?


Make getName() work on name tokens itself

Codecov Report

Merging #43 into master will decrease coverage by 0.21%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
- Coverage   64.85%   64.63%   -0.22%
==========================================
  Files          38       38
  Lines        2359     2367       +8
  Branches       45       45
==========================================
  Hits         1530     1530
- Misses        810      818       +8
  Partials       19       19

Impacted Files
Coverage Δ

src/tokentree/utils/TokenTreeCheckUtils.hx
7.86% <0%> (-0.29%)
:arrow_down:

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 844c99f...08a7d40. Read the comment docs.


Add test for SpatialPredicatePartitioner

Tried to implement two saptial predicates: one for "inside radius check" and other for "inside rectangle check".

Yes
On Wed, Feb 26, 2020, 20:50 Renato Aranha<EMAIL_ADDRESS>wrote:

@errearanhas commented on this pull request.
In test/partitioning.jl
https://github.com/JuliaEarth/GeoStatsBase.jl/pull/18#discussion_r384757250
:

@@ -196,7 +196,31 @@
end

@testset "SpatialPredicatePartitioner" begin

TODO

g = RegularGrid{Float64}(10,10)
R = 5.0
SPred1(x,y) = norm(x-y) < R

you meant to say p = SpatialPredicatePartitioner((x, y) -> norm(x-y) <
5.0), and then s = subsets(partition(g, p)), right?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/JuliaEarth/GeoStatsBase.jl/pull/18?email_source=notifications&email_token=AAZQW3ODDYKNOQSDV54JCJLRE3I2JA5CNFSM4K4H37FKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCXCP4VQ#discussion_r384757250,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3PBKBOYGPVQCBHYTO3RE3I2JANCNFSM4K4H37FA
.

Codecov Report

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

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage    65.6%   65.67%   +0.06%
==========================================
  Files         111      111
  Lines        1922     1923       +1
==========================================
+ Hits         1261     1263       +2
+ Misses        661      660       -1

Impacted Files
Coverage Δ

src/partitioning/spatial_predicate.jl
100% <0%> (+100%)
: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 94aaf74...6f660d0. Read the comment docs.


Manage iOS8 UIActionSheet deprecation

As exposed here:
http://nshipster.com/uialertcontroller/?utm_campaign=iOS_Dev_Weekly_Issue_166&utm_medium=email&utm_source=iOS%2BDev%2BWeekly

I'm going to close this, since I don't believe it's a practical concern. Happy to reopen if anyone feels otherwise.


[Task] Monitoring Template Yml Metrics I18n | 监控模版指标国际化任务认领 #1212

What's changed?

Checklist

[x]  I hereby agree to the terms of the HertzBeat CLA
[ ]  I have read the Contributing Guide
[ ]  I have written the necessary doc or comment.
[ ]  I have added the necessary unit tests and all cases have passed.

Add or update API

[ ] I have added the necessary e2e tests and all cases have passed.

@all-contributors please add @tslj1024 for code


Add Setter and Getter functions to results spreadsheet google creds

This PR adds a "setter" and a "getter" functions to results spreadsheet google credentials, in order to allow the use of CreateSheetsAndDriveServices func independently from using upload results-spreadsheet sub-command.
In current state, the credentials are sent using a flag to the mentioned sub command, which limits functions using this variable to run using the sub-command only.
In the operator-results-spreadsheet repo, we would like to allow the use of CreateSheetsAndDriveServices func (which uses the credentials variable) in order to add more columns to an existing sheet, and for that we need the credentials to be set manually - the "setter" and the "getter" functions allows it.

This is a way to solve it, but it doesn't look like a good approach to me, as the setters/getters use an internal global/package var. I would probably change the CreateSheetsAndDriveServices to add an extra string argument for that credentails path.

I agree - That's a better idea. I'll do the change, thanks!


v1: enable filtering by subreddit on messages endpoint

:eyeglasses: @ajacksified

minor nit, :nail_care:

:haircut:

:fish:


show session in day or track wise tabs

addressed here https://github.com/fossasia/open-event-webapp/pull/61


rust-analyzer: 2021-01-25 -> 2021-02-01

Motivation for this change
Bump version.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111582 run on x86_64-darwin 1

3 packages built:

rust-analyzer
rust-analyzer-unwrapped
vscode-extensions.matklad.rust-analyzer

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111582 run on x86_64-linux 1

3 packages built:

rust-analyzer
rust-analyzer-unwrapped
vscode-extensions.matklad.rust-analyzer


ci build should run xcop

rultor runs xcop on merge, so should CI build
Also, it would be nice to slip tests and linters in two separate jobs, as it was done in cactoos
https://github.com/yegor256/cactoos/blob/master/.github/workflows/build.yml#L30-35

@andreoss can you do this?

@yegor256 Yes, please assign to me.

@yegor256 Done.

@andreoss thanks!

@yegor256/z everybody who has role DEV is banned at #488; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)


certs: Use TWEAK domain certs if set

Web proxy extension of snikket-im/snikket-server#69

:wave:
I'm trying to piece things together so we could rebase and merge this PR prior to the upcoming release. I'm not aiming for full support of split-domain setups in this release, but I'd like to make any small safe steps that we can towards it. This seems like one!
I see that SNIKKET_TWEAK_CERT_DOMAIN is being pulled from SNIKKET_TWEAK_XMPP_DOMAIN. But in the case of a split-domain setup, we want nginx to have certs for the web domain, right?

I have forgotten some of what I wrote back then but you are correct. If SNIKKET_TWEAK_XMPP_DOMAIN is set, the jids are of the form user@SNIKKET_TWEAK_XMPP_DOMAIN and the web portal is still hosted on SNIKKET_DOMAIN. This means that certificates for SNIKKET_DOMAIN should be used.


webview_flutter doesn't open storage/gallery while pressing on "choose file" input tags

I have the same problem~~

I have the same problem~~

found any solution

Duplicate of #27924


Miss alignment in Appointment Page in Small Screen

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Miss alignment in Appointment Page in Small Screen
Expected behavior
Proper alignment without overflow
Add ScreenShots

What browsers are you seeing the problem on?
Chrome
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@varshith257 please assign me this issue.

/assign
On Thu, 1 Aug, 2024, 9:03 pm Lakshmi Pavananjali, @.***>
wrote:

@varshith257 https://github.com/varshith257 please assign me this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/issues/1107#issuecomment-2263368396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4BF3HDDT66P2AU2LRF4AKDZPJIMXAVCNFSM6AAAAABL22C42OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRTGM3DQMZZGY
.
You are receiving this because you were mentioned.Message ID:
@.***
com>


Recipe book

After playing a many of the stages, I realized that it would be really nice for each player to be able to have a recipe book, where you could just call the functions you have created, to avoid having to copy paste from one to the next all your functionalities

Linked to https://github.com/codecombat/codecombat/issues/19, https://github.com/codecombat/codecombat/issues/2444, and https://github.com/codecombat/codecombat/issues/2561


twitterbot oauth_token

Hi,
StupidQuickQuestion -
I've got a ~/.twitter_oath after authorising the application/etc when you run 'twitter'
However it seems like twitterbot does not like using the ~/.twitter_oath file and fails to authenticate (whilst 'twitter' works as it should)
What am I doing wrong?
Thanks!

Self-Answer: Specify a oauth file in your twitterbot ini (or simply dont make such a configuration line) that is empty/unused, run twitterbot and it will run through the process just like with 'twitter'


Internal error when trying to start movie on RPi-Kodi-Netflix

Since last week I encounter this issue when trying to watch a movie on LE 9.2.8 Kodi 18.9 on a rpi3. Before all went smooth.
Any suggestions or workaround. Couldn't find any updates with fixes so far.
Thanks.

Same here on ODROID XU4, Kodi Leia

Probably duplicate of #495
Please try the testbuilds from here: https://github.com/emilsvennesson/script.module.inputstreamhelper/issues/497#issuecomment-1068239609

Brillant! Thanks!