Add metadata to API for pagination

Add metadata to API for pagination
Expected:
Because we are limited to 100 records per page on the /GET queries, there should be some metadata to help us know how many pages or records there are to help with pagination
Actual:
Without metadata the pagination logic on the client side has to be more complicated.  Because I don't know what the last id is going to be, I have to get the first page, get the last id, then add one and get the next page with min_id=(prevPageLastId+1) and continue until I have less than 100...
Example:
[
"_metadata":
{
"page": 5,
"per_page": 100,
"page_count": 6,
"total_count": 521,
},
"Replays": [
{
"id":370026,
"filename":"3336195043",
...
}]
]
Rational:
Building an app to query the basic info about each replay (not download them) and then provide very basic stats in real time as new replays are uploaded.  Need to be able to query for replays during desired time periods and page through all records for that time period. I.E. "Top 10 Hero Win Ratios in last 24hr for Hero League"

Agreed

@poma this seems pretty important (at least to me). I know you put it on the Backlog, do you mind if I try to add this?

I agree that it is important but I didn't see a way to add this without breaking changes, some users may already rely on result being an array of replays. So before implementing please post how your API will look like. Maybe we can expose it on different url or pass some request parameters to enable pagination functionality.
I think for the first version of API we will just pile on features (with no breaking changes) without worrying too much about elegant design, and after that will release carefully designed API v2.

Making a separate API sounds like a plan. I'll get a PR going soon to get the discussion going around the design.


Media player, browse for subtitle, how to access a subtitle stored in a usb stick or other file system location?

Help request
Your Environment

Netflix add-on version: 0.15.11
Operating system version/name:LibreElec 9.+
Device model:  Raspberry-Pi 2 B+

Used Operating system:

[ ] Android
[ ] iOS
[ ] Linux
[ ] OSX
[x] Raspberry-Pi
[ ] Windows

Describe your help request

Media player, browse for subtitle, how to access a subtitle stored in a usb stick or other file system location?
Screenshots

This does not depend on the addon but is an option of the Kodi player
you have to ask in the forum


Update README for release and fix typos

What does this PR do?
Update README for release and fix typos
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)

[x] N/A

[x] Did you make sure to update the docs?

[ ] N/A

[ ] Did you write any new necessary tests?

[x] N/A

Codecov Report
Base: 87.06% // Head: 87.06% // No change to project coverage :thumbsup:

Coverage data is based on head (5fea270) compared to base (256d677).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   87.06%   87.06%
=======================================
  Files          26       26
  Lines        2033     2033
=======================================
  Hits         1770     1770
  Misses        263      263

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

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


Allignment of Navbar

Describe the feature
Hello Project Admin
I would like to Allign Navbar perfectly so that it will make our website to look Attractive
Can you assign me this issue under GSSOC'24
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Can you assign me this issue under GSSOC'24 @RitiChandak

level will be assigned after seeing your work

Yeah okk @RitiChandak


Fix date formatting in the instance history view (#2338)

Pull request opened by the merge tool on behalf of #2338

Merged in f49d83ef1f8a5bb52a53e6aa0f8bbd2a65d00abf


⚠ ClamAV has degraded performance

In be749b6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 86 ms

Resolved: ClamAV performance has improved in 6ee6e57.


Make profiling during training more informative

Currently we log metrics for a single training step, ever training.eval_freq steps. The problem with this is that there may be large variance in the metrics meaning we often don't get a representative value.
The worst case of this is the timing metric data_s which is 0 a lot of the time, but sometime non-zero and large because the dataloader is working on fetching the next set of batches. This means we don't get a good read on the data loading bottleneck. It would be better to have an aggregated metric which is the average data_s per step for the last training.eval_freq steps.
Of course, it's still useful to have the non-aggregated metric, so we need to think about how to do this without losing useful information.

Thanks, Alexander for sharing this issue with me. I'll start working on it

The logged items are. Here is my understanding of their usecase:-

smpl (Number of samples): This is the number of samples seen yet. A cumulative number, that makes sense as it is.
eo (Number of episodes): An episode represents one complete video or simulation. Yet another cumulative number, that makes sense as it is.
ep (Number of epochs): The number of times complete dataset has passed through the training. Yet another cumulative number, that makes sense as it is.
loss (Loss): The loss in the current training iteration.
lr (Learning rate): The current learning rate.
data_s (Batch loading time): Time it took for the previous training iteration to load the batch data.
updt_s (Batch updating time): Time it took to update the policy (understand this as the time taken to backpropagate)

Proposed new logging items:-
8. avg_data_s (Average batch loading time): average of all batch loading times till the completion of the current iteration.
9. avg_updt_s (Average batch updating time): average of all policy update time till the completion of the current iteration.

@MayankChaturvedi thanks for getting a start on this:
"ep" is number of episodes, and "epch" is number of epochs. I'm not sure about "eo".
"updt_s" also includes the forward pass.

Do you really mean average of all batch loading times? Or just since the last log?
Again do you mean all since the beginning of training?

For 8 and 9, I think a more useful thing would be the average since last log. It's not pretty that the variance is dependent on the log frequency, but I think that's not too bad from a usability perspective. What do you think?
cc @Cadene in case you want to chime in

Oops my bad "ep" not "eo". "p" and "o" are together in the keyboard 🙃
Thanks, @alexander-soare for your insights. You are right, average since the last log makes more sense than the overall average.
Would one last log at the end of the training, informing overall average update time and loading time help the users? (This could be separate from this issue)
I second the thought that updt_s and data_s should be removed

Yeah all items are being printed to the terminal. It's possible you'll need to cut down to keep them on one line. I think there's some redundancy in printing step, epch, and smpl. Maybe one or more of those can go.


AEM Environment Indicator config causes slow loading of model.json output of pages

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: 6.5.8
[ ] ACS AEM Commons Version: 4.8.0
[ ] Reproducible on Latest? yes

Expected Behavior
After code deployment with AemEnvironmentIndicator OSGI config, the http://localhost:4502/content/wknd/language-masters/en/faqs.model.json should continue to load in 1s or less than 1s

Actual Behavior
Config properties are applied and visible in the OSGI console  but http://localhost:4502/content/wknd/language-masters/en/faqs.model.json is now taking longer time to load (approx. 30-33 seconds). Similar load time was seen on our actual AEM project content pages

Steps to Reproduce

Install ACS AEM Commons (4.8.0v) on a clean 6.5.8 AEM instance
Install 'wknd' project or skip this step if you already have an existing project with content pages
Create an OSGI config, like
<jcr:root xmlns:sling="http://sling.apache.org/jcr/sling/1.0" xmlns:cq="http://www.day.com/jcr/cq/1.0" xmlns:jcr="http://www.jcp.org/jcr/1.0" xmlns:nt="http://www.jcp.org/jcr/nt/1.0"
jcr:primaryType="sling:OsgiConfig"
css-color="#f9e208"
browser-title-prefix="AEM Testing"
excluded-wcm-modes=[""]/>
Do a code deployment & experience the behavior

@davidjgonzalez / @joerghoh Greetings!! Please help me with your valuable inputs

@balvinder-singh hi! sorry for the delay - can you make a request to a slow model.json and review the timing for the filter using: http://localhost:4502/system/console/requests

@davidjgonzalez  no worries, thanks for your reply!!
It introduces an "allowed extensions" OSGi config that pre-emptively allows the filter to just call the filter chain; default config is to only allow extensions:

html
htm
jsp (for crxde lite or pack mgr)
 (for /miscadmin, system/console, etc...)

I think this should cover HTML request/responses. Alternatively, we should switch it to a deny list .. and enumerate: json, xml, etc... though, there's alot more not-HTML extensions than HTML-extension in AEM
acs-aem-commons-content-5.0.5-SNAPSHOT.zip
acs-aem-commons-content-5.0.5-SNAPSHOT-min.zip

@davidjgonzalez  Hi, thanks for sharing the fix. it worked & solved our issue of slowing response & truncated response in model.json
& can you please share the release date of 5.0.5 version, since this is a SNAPSHOT zip file & we always include release version within our build process.
Thank you in advance!!

@davidjgonzalez Hi, can you please share the expected release date of this 5.0.5 version ?
It would be helpful for our team to plan our Project Deliverables. Thanks!!

@balvinder-singh gonna try for thurs/fri - i'd like to get a PR review on this issue https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/2626 as well as https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/2618

@davidjgonzalez Hi, Greetings!!
can you please share the expected release date of this 5.0.5 version ?

5.0.6 is now released (and contains the fix)!

@davidjgonzalez thank you for the update!!
Besides I got the release 5.0.6 version of acs-commons & deployed on our DEV environment with following configs:

but the environment indicator is not visible for system/console/configMgr, /bundles or /inbox (non-html extensions)
can you please guide on how to show environment indicator for above extensions as well

Try adding the string  to the list as mentioned in the description. That should be in the default list anyhow, I can add that to  default for next release.

@davidjgonzalez Yes that will be helpful!! Thank you.


Increasing the fee on some bitcoin transactions errors with "Writer(): uInt: value out of bounds"

The user explained it was a transaction they created with Magic Eden to transfer some rare sats to the ordinals wallet on Leather.
I have seen one other report but no examples of affected transactions. I will add them when users share them here.
The error in the screenshot in the next comment pops up after the user clicks "submit" on the fee increase screen.

Example of another transaction that will throw this error when trying to increase the fee:
https://mempool.space/tx/ceb437c902132022dbaaea2feb754241253ddaa539b74c16114bcc619fc1fb31 = payment transcaction (I am not sure if this is relevant but the dapp creates this subsequent transaction with the first output... for the actual mint https://mempool.space/tx/ec1ad7ed5ce1f4e2a5db8367d26c4b6a9447dce53923f3c97d080fbea0ab7648)
This is from stampscan.xyz, minting SRC20, steps to reproduce

Sign in with Leather
Click mint
Add values like this (or choose another token)
PXL, 3141, 2, and set a fee rate.

Proceed
Sign transaction
Then increase the fee via the activities tab in the wallet
See this error

In this case the RBF flag is false, the wallet should apply CPFP to increase the fee. The error doesn't help users, it should be prevented. Users want the ability to increase the fee on any type of transaction.
On mobile/kiwi a user reported seeing a different error (Error: transaction size must be defined) instead. I was not able to reproduce but they said also seeing it on desktop, this error occurs on step 6 in the flow mentioned above:
https://github.com/leather-wallet/extension/assets/33360391/4d758958-1922-4e39-8850-16f097e975f4

I saw another case like this today. It was for this incoming transaction
https://mempool.space/tx/811fd8009595ae3b45034d07c37277654b2c28834350c503aa47f29a2c52e005
The user tried "increase fee" on this transaction and lead to the same error.
I do not expect the increase fee option to show on incoming transactions currently.
I think the bug was that the button, to increase fee, should not have been shown in the first place.

Another transaction that had the "increase fee" option shown to the user but the transaction doesn't support RBF so it shouldn't have. As they tried to increase the fee they saw the same "out of bound" error.

https://mempool.space/tx/521f8645c74b8ceb381954772e6de858992d59e02cbbfed9552afc10377e3324
In this case CPFP would have been an option. It would be great if users can click increase fee and the wallet can pick CPFP/RBF whatever is applicable depending on the type of transaction.
The user noted that they had trouble copying the error text, it would help users if their was some kind of error log they could access to see past error messages.

i'm having the same issue with increasing the fee. i increased the fee about 20 minutes ago just fine. but wanted to increase again by a little bit and it gave me the same error "out of bounds". no RBF support for the transaction as well. the wallet extension (chrome) shows "increase fee" button


Added wrapper for xarrays/dask arrays

Not super confident about this - I had to alter the tests so that they point at the wrapper function, but the tests pass now.

Also, we now have special logic for xarray inputs, but we have no test for them. Do you think you could add that here?

Also, we now have special logic for xarray inputs, but we have no test for them. Do you think you could add that here?

Should I do this for all of these clients: 'threaded_client', 'processes_client', 'distributed_client' or just for one case?

There is really only only one test function, which is heavily parametrized:
https://github.com/xgcm/fastjmd95/blob/074ab2f5ff07ffd1d2303b680739f1b8cd52cd67/fastjmd95/test/test_jmd95.py#L61-L67
You should try to add xarray dataarrays to that existing framework.

Is this ok?

This is perfect Spencer! Thanks so much for this PR.


typo: "resources/cpu" -> "resource/cpu"

Fixing a small typo in the example spec.

Great catch. Thank you.


enhancement: Keyword Query Language (KQL) search syntax support

Description
Introduce support for KQL syntax for search.
The functionality consists of a kql lexer and a bleve query compiler
Supported field queries:

Tag search tag:golden tag:"silver"
Filename search name:file.txt name:"file.docx"
Content search content:ahab content:"captain aha*"

Supported conjunctive normal form queries:

Boolean operators AND, OR, NOT,
Nesting ( ...SUB_QUERY... )

for example:
Query:
(name:"moby di*" OR tag:bestseller) AND tag:book NOT tag:read
Result:

Resources with name: moby di* OR tag: bestseller.
AND with tag:book.
NOT with tag:read.

AST:
&ast.Ast{
  Nodes: []ast.Node{
    &ast.GroupNode{
      Base: &ast.Base{Loc: &ast.Location{...}},
      Nodes: []ast.Node{
        &ast.StringNode{...},
        &ast.OperatorNode{...},
        &ast.StringNode{...},
      },
    },
    &ast.OperatorNode{Base: &ast.Base{Loc: &ast.Location{...}}, Value: "AND"},
    &ast.StringNode{Base: &ast.Base{Loc: &ast.Location{...}}, Key: "tag", Value: "book"},
    &ast.OperatorNode{Base: &ast.Base{Loc: &ast.Location{...}}, Value: "NOT"},
    &ast.StringNode{Base: &ast.Base{Loc: &ast.Location{...}}, Key: "tag", Value: "read"},
  },
}

Todos:

[X] KQL-STRING-QUERY to AST kql syntax subset [KQL-TO-AST-LEXER]

[X] name (ast.StringNode)
[X] tag (ast.StringNode)
[X] content (ast.StringNode)
[X] free text-keywords (ast.StringNode)
[X] conjunctive normal form (ast.GroupNode)
[X] operators AND, OR, NOT (ast.OperatorNode)

[ ] AST to BLEVE-STRING-QUERY compiler [AST-TO-BLEVE-COMPILER]

[x] name (ast.StringNode)
[x] tag (ast.StringNode)
[ ] content (ast.StringNode)
[x] free text-keywords (ast.StringNode)
[x] conjunctive normal form (ast.GroupNode)
[ ] operators AND, OR, NOT (ast.OperatorNode)

Related Issue

Fixes https://github.com/owncloud/ocis/issues/7042

How Has This Been Tested?

unit tests

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[X] Code changes
[X] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

We should discuss how to handle the implicit boolean operators and NOT together. It is not explicitly described in the documentation. The rules are different for the multiple free-text expressions, multiple instances of the same property restriction, multiple instances of the different property and grouping-property.
https://github.com/owncloud/ocis/blob/46b8e437f3f46c8f50fe3b82dcf6356e9e3a473a/services/search/pkg/query/kql/factory.go#L156-L176
Should we support and how do we represent?
For example:
 author:"John Smith" author:"Jane Smith"
This functionally is the same as using the OR Boolean operator, as follows:
author:"John Smith" OR author:"Jane Smith

Is NOT author:"John Smith" NOT author:"Jane Smith"  became NOT author:"John Smith" OR  NOT author:"Jane Smith ?
author:"John Smith" filetype:docx
This is the same as using the AND Boolean operator, as follows:
author:"John Smith" AND filetype:docx

Is NOT author:"John Smith" NOT filetype:docx"  became NOT author:"John Smith" AND NOT filetype:docx ?

author

the ms spec defines when to use a implicit AND and when to use a implicit OR, but negotiations aren't mentioned.
my 5 cents: since no description is available for those cases i would say, as soon as someone uses an operator as edge between two nodes:
┌─────────────┐                            ┌───────────────┐
│             │        ┌─────────┐         │               │
│QUERY-NODE-1 ├────────┤ OP-EDGE ├─────────┤ QUERY-NODE-2  │
│             │        └─────────┘         │               │
└─────────────┘                            └───────────────┘

we do not add any other implicit type there. cC.: @tbsbdr

I assume documentation is done as a follow up?

🤣 SURE, to be more exact, KQL is not enabled at the moment, but in the follow up we will enable it and provide a readme and with the help of @mmattel a even more fluffy docs page 💋


Unaddressed issue raised in feature/newdb, id="taille_texte" + title="Valeur des marges"

See full description / discussion in the Pull Request review:
https://github.com/readium/readium-desktop/pull/199#pullrequestreview-187578801

Still in the develop branch:
This was reported back in December 2018 ;)
https://github.com/readium/readium-desktop/blob/a39e7dfbeffa0d8c33f5d0cbe0dbe875e41d1136/src/renderer/components/reader/ReaderOptions.tsx#L280
https://github.com/readium/readium-desktop/blob/a39e7dfbeffa0d8c33f5d0cbe0dbe875e41d1136/src/renderer/components/reader/ReaderOptions.tsx#L279-L280
https://github.com/readium/readium-desktop/blob/a39e7dfbeffa0d8c33f5d0cbe0dbe875e41d1136/src/renderer/components/reader/ReaderOptions.tsx#L300-L301
https://github.com/readium/readium-desktop/blob/a39e7dfbeffa0d8c33f5d0cbe0dbe875e41d1136/src/renderer/components/reader/ReaderOptions.tsx#L342-L343

TODO: check all instances of duplicate IDs in React JSX
https://github.com/readium/readium-desktop/issues/398

Fixed via https://github.com/readium/readium-desktop/pull/383


ANC PostPartum - Next Steps - HIV [6h]

Issue
If a patient was diagnosed with HIV at some point during their pregnancy, and they are not already enrolled in an ARV services program (determined in specialty care review) then we will refer them to ARV services for continuous care post-pregnancy.
This is the referral screen for ARV services:

This is the wording/screen that the clinical team decided on.
Related Issues:

#376

To Do

Create logic
Style next steps screens per design

@jeanette-pelizzon
When patient is diagnosed with HIV, we ask if there's HIV/PMTC program at HC:

Then, at Next steps, we refer to Integration HIV/PMTC:

In this issue, instead of using Integration HIV/PMTC, we say ARV services.
Are these 2 different programs? If not, shouldn't we have common name for this?

We should have a common name. There was a large discussion around this ages ago and I believe we settled on ARV service as that's what we refer to it as in newer screens.
Let's go with ARV services.

https://nutrition-ihangane.pantheonsite.io/
QA Review
Checked and approved:

Patient is referred to ARV services, if they were diagnosed with HIV during pregnancy, and they are still not enrolled in ARV services.

@balagan73 I'm guessing here that this is a result of a different HIV diagnoses during teh preganancy.
@anvmn can you confirm?

@adamhstewart
This needs looking into.

@adamhstewart
What needs to happen is that if patient is dignosed with Discordant Partnership, the header at next steps should say so. Fixed at #475.

https://nutrition-ihangane.pantheonsite.io/
QA Review
Found issues and questions:

In first antenatal encounter partner has positive HIV test results, it is not displayed anywhere.

History
Next steps
Progress report


Is there a way to start RenetServer on an external interface?

All examples / usages use <IP_ADDRESS> which makes server only listen on loopback interface. Using <IP_ADDRESS> doesn't work as well for some weird reason related to UDP protocol itself (something to do with it being unable to tell which interface to reply on). So is there a solution to that problem without hardcoding / passing actual WAN IP address directly?

You can bind the socket to <IP_ADDRESS> (all valid IP for this computer). The problem is that <IP_ADDRESS> is not a valid destination to send packets to.
So you would need to use the public address for the server set to <IP_ADDRESS> (loopback), 192.168.0.x (local network), or your computer's public IP.
For the public IP you need to do some setup because you are probably behind a NAT, and would need to port forward. Or you can use a signaling server (what WebRTC does) to punch-through the NAT.

Well yeah, i understand all of that reasoning. So there is no way to automatically bind it to a correct IP address (because even <IP_ADDRESS> doesnt always work) and i always have to manually specify my external IP address as a command line argument.

Well yeah, i understand all of that reasoning. So there is no way to automatically bind it to a correct IP address (because even <IP_ADDRESS> doesnt always work) and i always have to manually specify my external IP address as a command line argument.

Yes, you need to specify the address somehow, if you bind an UdpSocket using <IP_ADDRESS> you will need to get the address somehow.
With #102 the host check will be skipped in unsecure connections, so it will be easier to test.


Create sitemap for places and add robots.txt

The setup is based on: https://www.sitemaps.org/protocol.html

nice! could you add a quick README as well?
btw, we might need to do it for all dcid's too: see #92

Added README. okay, i will add other nodes in follow up PR. Hopefully to check this setup can work properly first.


Shuttle doesn't open iTerm 2.9 (nightly)

Shuttle ( http://fitztrev.github.io/shuttle/ ) doesn't open iTerm 2.9 (nightly).
And all works fine with iTerm 2.1
I have set:
"terminal": "iTerm"

Applescript support has changed in 2.9 and is not backward compatible. The docs are here: https://iterm2.com/applescript.html
Should be a relatively easy change.

+1 for a new version of shuttle compatible with 2.9/3

Hello All,
On my G+ page I had a poll regarding this topic. I think it's best that sshuttle support all three versions of iTerm. I plan on supporting this by implementing a settings database. This database will contain the applescript needed for all three versions. You would select the desired version in a settings dialog. Until I can get this coded I'll build a version that works with Nightly. Feel free to contact me if you have questions. -Matt

Complete! See release v1.2.3

Thank you!!


Popover regression from 4.2.x to 4.3.x

(Mac OS Mojave 10.14.4, Chrome 74.0.3729.157)
It looks like more complex data-content for a Popover is not rendered correctly. For example, these two spans with a popover (fiddle):
<span id="span-id-1" data-toggle="popover" data-placement="right" data-html="true" title="Title" data-content="<button type='button' class='btn btn-secondary btn-sm' id='btn-id-1'>Button</button>">POP
POP

work fine with v4.2.1

but the button in the Popover disappear when I switch to v4.3.2

This seems to be the case with input elements in general, whereas text in p and span shows up ok.

Hi @jenstroeger,
That's the correct behavior and you'll find why by reading that: https://blog.getbootstrap.com/2019/02/13/bootstrap-4-3-1-and-3-4-1/
And you'll find how to fix that behavior here: https://getbootstrap.com/docs/4.3/getting-started/javascript/#sanitizer
BTW it's a good practice to read changelogs when you update any libs 👍

@Johann-S, ugh, yes I missed that. All fixed and working now again, thank you 👍🏼


Fix: allow Keycloak table input to be editable

fix: https://github.com/casdoor/casdoor/issues/3171
fix demo

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: Correct issues with userled support on the Nucleo F4x1RE.

Summary
Correct issues with userled support on the Nucleo F4x1RE.
Specifically:
The proper headers are included when CONFIG_USERLED is defined.
LED_DRIVER_PATH is set when CONFIG_USERLED_LOWER is defined and CONFIG_ARCH_LEDS is NOT defined.
board_userled() is corrected for the nucleo-f4x1re.
board_userled_getall() is implemented for the nucleo-f4x1re.
userled_lower_initialize() is called when properly configured.
some comments added.
Impact
userled support on the Nucleo F4x1RE will work properly for the Nucleo F4x1RE.  No other impacts.
Testing
tested and verified on the Nucleo F4x1RE.

Hi @ryanoabel please fix these issues:
Error: /home/runner/work/incubator-nuttx/incubator-nuttx/nuttx/boards/arm/stm32/nucleo-f4x1re/src/stm32_bringup.c:86:1: error: Missing blank line after comment
Error: /home/runner/work/incubator-nuttx/incubator-nuttx/nuttx/boards/arm/stm32/nucleo-f4x1re/src/stm32_userleds.c:167:5: error: Space follows left parenthesis

@ryanoabel could  you squash to one commit?


Unknown exception connecting to MQTT broker: [Errno 2] No such file or directory

First, thanks for your work on this project.  I've been running rtlamr on my Linux server for a while and have finally gotten to a point where I wanted to switch it over to Docker.  This project looks like it fits my needs perfectly!
As noted above, I am getting an error when trying to use the docker container.  Here's the logs:
[2022-04-08 12:35:24] Sending message to MQTT:,
[2022-04-08 12:35:24]  > topic => rtlamr/status,
[2022-04-08 12:35:24]  > payload => offline,
[2022-04-08 12:35:24]  > retain => True,
[2022-04-08 12:35:24] Unknown exception connecting to MQTT broker: [Errno 2] No such file or directory,
[2022-04-08 12:35:25] Configured MQTT sender:,
[2022-04-08 12:35:25]  > hostname => <IP_ADDRESS>,
[2022-04-08 12:35:25]  > port => 1883,
[2022-04-08 12:35:25]  > username => hass,
[2022-04-08 12:35:25]  > client_id => rtlamr2mqtt,
[2022-04-08 12:35:25]  > tls => {'ca_certs': '/etc/ssl/certs/ca-certificates.crt', 'certfile': '/etc/ssl/my_self_signed_cert.crt', 'insecure': True, 'keyfile': '/etc/ssl/my_self_signed_cert_key.key', 'tls_version': None, 'ciphers': None},
[2022-04-08 12:35:25] Sending message to MQTT:,
[2022-04-08 12:35:25]  > topic => rtlamr/status,
[2022-04-08 12:35:25]  > payload => online,
[2022-04-08 12:35:25]  > retain => True,
[2022-04-08 12:35:25] Unknown exception connecting to MQTT broker: [Errno 2] No such file or directory,
[2022-04-08 12:35:25] Trying to start RTL_TCP: /usr/bin/rtl_tcp -s 14448385,
[2022-04-08 12:35:25] RTL_TCP started with PID 7,
[2022-04-08 12:35:30] RTL_TCP is ready to receive connections!,
[2022-04-08 12:35:30] Trying to start RTLAMR: /usr/bin/rtlamr -msgtype=scm+ -format=json -filterid=14448385 -unique=true -symbollength=32,
[2022-04-08 12:35:30] RTLAMR started with PID 13,
[2022-04-08 12:35:30] 12:35:30.626183 decode.go:45: CenterFreq: 912600155,
[2022-04-08 12:35:30] 12:35:30.626264 decode.go:46: SampleRate: 1048576,
[2022-04-08 12:35:30] 12:35:30.626271 decode.go:47: DataRate: 32768,
[2022-04-08 12:35:30] 12:35:30.626276 decode.go:48: ChipLength: 32,
[2022-04-08 12:35:30] 12:35:30.626545 decode.go:49: PreambleSymbols: 16,
[2022-04-08 12:35:30] 12:35:30.626553 decode.go:50: PreambleLength: 1024,
[2022-04-08 12:35:30] 12:35:30.626557 decode.go:51: PacketSymbols: 128,
[2022-04-08 12:35:30] 12:35:30.626562 decode.go:52: PacketLength: 8192,
[2022-04-08 12:35:30] 12:35:30.626568 decode.go:59: Protocols: scm+,
[2022-04-08 12:35:30] 12:35:30.626572 decode.go:60: Preambles:<PHONE_NUMBER>100011,
[2022-04-08 12:35:30] 12:35:30.626577 main.go:124: GainCount: 29,
[2022-04-08 12:46:38] Shutdown detected, killing process.,
[2022-04-08 12:46:38] Killing RTL_TCP...,
[2022-04-08 12:46:43] Killed.,
[2022-04-08 12:46:43] Killing RTLAMR...,
[2022-04-08 12:46:43] Killed in the first attempt.,
[2022-04-08 12:46:43] Graceful shutdown.,

and here's my rtlamr2mqtt.yaml file:
general:
  sleep_for: 0
  verbosity: debug
  tickle_rtl_tcp: false
  usb_reset: '002:004'

mqtt:
  host: <IP_ADDRESS>
  # Path to CA certificate to use. Mandatory
  tls_ca: '/etc/ssl/certs/ca-certificates.crt'
  # Path to certificate file to use. Optional
  tls_cert: '/etc/ssl/my_self_signed_cert.crt'
  # TLS insecure: defaults to False
  tls_insecure: true
  # Certificate key file to use. Optional
  tls_keyfile: '/etc/ssl/my_self_signed_cert_key.key'
  user: 'hass'
  password: 'XXXXXXXXXXX'
  ha_autodiscovery: true
  ha_autodiscovery_topic: homeassistant

custom_parameters:
  rtltcp: "-s 14448385"
  rtlamr: "-unique=true -symbollength=32"

meters:
  - id: 14448385
    protocol: scm+
    name: test_meter
    format: "######.###"
    unit_of_measurement: "\u33A5"
    icon: mdi:gauge
    device_class: energy

and finally, my docker-compose entries:
  rtlamr:
    container_name: rtlamr2mqtt
    image: allangood/rtlamr2mqtt
    restart: unless-stopped
    depends_on:
      - rtl433
    devices:
      - /dev/bus/usb
    volumes:
      - /sharedfolders/appdata/rtlamr2mqtt/rtlamr2mqtt.yaml:/etc/rtlamr2mqtt.yaml:ro
      - /sharedfolders/appdata/rtlamr2mqtt/data:/var/lib/rtlamr2mqtt

It appears that the container is reading the config correctly but I'm not sure why it's having a hard time connecting to the MQTT broker.  I'm using it successfully with a handful of other containers.  This is, however, the first time I've tried using a user for connecting as my broker accepts anonymous connections.
Any help in troubleshooting this is most appreciated!

Looks like the problem is the tls configuration. Are you using a MQTT
broker with TLS encryption? If not, just remove the tls configuration. If
yes, then you must point the configuration to existing files.
On Fri., Apr. 8, 2022, 7:55 a.m. Chuck Connors, @.***>
wrote:

First, thanks for your work on this project. I've been running rtlamr on
my Linux server for a while and have finally gotten to a point where I
wanted to switch it over to Docker. This project looks like it fits my
needs perfectly!
As noted above, I am getting an error when trying to use the docker
container. Here's the logs:
[2022-04-08 12:35:24] Sending message to MQTT:,
[2022-04-08 12:35:24]  > topic => rtlamr/status,
[2022-04-08 12:35:24]  > payload => offline,
[2022-04-08 12:35:24]  > retain => True,
[2022-04-08 12:35:24] Unknown exception connecting to MQTT broker: [Errno 2] No such file or directory,
[2022-04-08 12:35:25] Configured MQTT sender:,
[2022-04-08 12:35:25]  > hostname => <IP_ADDRESS>,
[2022-04-08 12:35:25]  > port => 1883,
[2022-04-08 12:35:25]  > username => hass,
[2022-04-08 12:35:25]  > client_id => rtlamr2mqtt,
[2022-04-08 12:35:25]  > tls => {'ca_certs': '/etc/ssl/certs/ca-certificates.crt', 'certfile': '/etc/ssl/my_self_signed_cert.crt', 'insecure': True, 'keyfile': '/etc/ssl/my_self_signed_cert_key.key', 'tls_version': None, 'ciphers': None},
[2022-04-08 12:35:25] Sending message to MQTT:,
[2022-04-08 12:35:25]  > topic => rtlamr/status,
[2022-04-08 12:35:25]  > payload => online,
[2022-04-08 12:35:25]  > retain => True,
[2022-04-08 12:35:25] Unknown exception connecting to MQTT broker: [Errno 2] No such file or directory,
[2022-04-08 12:35:25] Trying to start RTL_TCP: /usr/bin/rtl_tcp -s 14448385,
[2022-04-08 12:35:25] RTL_TCP started with PID 7,
[2022-04-08 12:35:30] RTL_TCP is ready to receive connections!,
[2022-04-08 12:35:30] Trying to start RTLAMR: /usr/bin/rtlamr -msgtype=scm+ -format=json -filterid=14448385 -unique=true -symbollength=32,
[2022-04-08 12:35:30] RTLAMR started with PID 13,
[2022-04-08 12:35:30] 12:35:30.626183 decode.go:45: CenterFreq: 912600155,
[2022-04-08 12:35:30] 12:35:30.626264 decode.go:46: SampleRate: 1048576,
[2022-04-08 12:35:30] 12:35:30.626271 decode.go:47: DataRate: 32768,
[2022-04-08 12:35:30] 12:35:30.626276 decode.go:48: ChipLength: 32,
[2022-04-08 12:35:30] 12:35:30.626545 decode.go:49: PreambleSymbols: 16,
[2022-04-08 12:35:30] 12:35:30.626553 decode.go:50: PreambleLength: 1024,
[2022-04-08 12:35:30] 12:35:30.626557 decode.go:51: PacketSymbols: 128,
[2022-04-08 12:35:30] 12:35:30.626562 decode.go:52: PacketLength: 8192,
[2022-04-08 12:35:30] 12:35:30.626568 decode.go:59: Protocols: scm+,
[2022-04-08 12:35:30] 12:35:30.626572 decode.go:60: Preambles:<PHONE_NUMBER>100011,
[2022-04-08 12:35:30] 12:35:30.626577 main.go:124: GainCount: 29,
[2022-04-08 12:46:38] Shutdown detected, killing process.,
[2022-04-08 12:46:38] Killing RTL_TCP...,
[2022-04-08 12:46:43] Killed.,
[2022-04-08 12:46:43] Killing RTLAMR...,
[2022-04-08 12:46:43] Killed in the first attempt.,
[2022-04-08 12:46:43] Graceful shutdown.,
and here's my rtlamr2mqtt.yaml file:
general:
sleep_for: 0
verbosity: debug
tickle_rtl_tcp: false
usb_reset: '002:004'
mqtt:
host: <IP_ADDRESS>
Path to CA certificate to use. Mandatory
tls_ca: '/etc/ssl/certs/ca-certificates.crt'
Path to certificate file to use. Optional
tls_cert: '/etc/ssl/my_self_signed_cert.crt'
TLS insecure: defaults to False
tls_insecure: true
Certificate key file to use. Optional
tls_keyfile: '/etc/ssl/my_self_signed_cert_key.key'
user: 'hass'
password: 'XXXXXXXXXXX'
ha_autodiscovery: true
ha_autodiscovery_topic: homeassistant
custom_parameters:
rtltcp: "-s 14448385"
rtlamr: "-unique=true -symbollength=32"
meters:

id: 14448385
protocol: scm+
name: test_meter
format: "######.###"
unit_of_measurement: "\u33A5"
icon: mdi:gauge
device_class: energy

and finally, my docker-compose entries:
rtlamr:
container_name: rtlamr2mqtt
image: allangood/rtlamr2mqtt
restart: unless-stopped
depends_on:
- rtl433
devices:
- /dev/bus/usb
volumes:
- /sharedfolders/appdata/rtlamr2mqtt/rtlamr2mqtt.yaml:/etc/rtlamr2mqtt.yaml:ro
- /sharedfolders/appdata/rtlamr2mqtt/data:/var/lib/rtlamr2mqtt
It appears that the container is reading the config correctly but I'm not
sure why it's having a hard time connecting to the MQTT broker. I'm using
it successfully with a handful of other containers. This is, however, the
first time I've tried using a user for connecting as my broker accepts
anonymous connections.
Any help in troubleshooting this is most appreciated!
—
Reply to this email directly, view it on GitHub
https://github.com/allangood/rtlamr2mqtt/issues/108, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFY2XRIERUHQQ7QMRMUNDLVEAUD7ANCNFSM5S4PEGVQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I have to make some changes in the README to clarify when the tls options should be present in the config file.

I have the same issue and when I remove the tls configuration, the tls default configuration pops back up on restart.

Thanks @allangood !  That did fix it.   I think what got me is the example file states:
  # Path to CA certificate to use. Mandatory

So I thought that it was necessary.  The container is up and appears to be active.  I do get the online message in HA but I am not getting meter readings.  I did run the command one the command line and know my meter number.  From that command I see:
[2022-04-08 12:03:10] {"Time":"2022-04-08T12:03:10.643465156Z","Offset":0,"Length":0,"Type":"SCM","Message":{"ID":14448385,"Type":4,"TamperPhy":0,"TamperEnc":2,"Consumption":1126,"ChecksumVal":48424}}

This is the config I'm using.  Am I doing something incorrectly?
custom_parameters:
  rtltcp: "-s 14448385"
  rtlamr: "-unique=true -symbollength=32"

meters:
  - id: 14448385
    protocol: scm+
    name: test_meter
    format: "######.###"
    unit_of_measurement: "\u33A5"
    icon: mdi:gauge
    device_class: energy

Apologies for the constant questioning.  I'm sure I'm overlooking something obvious but could certainly use a push in the right direction if possible.

Yes, that bit it is very confuse in the README file... it is mandatory only if the tls is being used.
If you are not using TLS it is safe to remove the whole "tls_something"... I will push some changes to make this better.
Looking the output, your meter is using SCM and your configuration seems to be using SCM+

Yep.  Started working as expected @allangood  .. I guess I'll push my luck one last time and ask how I can update the name and unit of measure.  I've made the change in the config file but HA is still seeing it with the test_meter name and the default units.

Just restart the container:
docker restart <container_name>

@allangood   Thank you.  That is what I've tried and it does not work.  Is it something with the persistence in MQTT?

No, it is not related. Check the sensors again, you may have both there. The old and the new one. :)

So I cheated.  I went and turned off autodiscovery and created the device in HA sensors.  This appears to be fine.  I blindly copied the example and I'm thinking about the format config.  My meter is old.  It has five little numbered wheels with an arrow that points to each.  The reading, as you can see above, gives that value.  I'm thinking that I need to change the format to match or just get rid of the entry altogether and that will give the whole integer value rather than the float that I have now, right?

The actual daily consumption is the reading at midnight minus the current reading.  I'm guessing that's what the utility meter definitions (hourly/daily/monthly) calculate?

This is not related to the issue anymore. Could open a discussion to ask for help instead?
Thank you.

Thanks for helping get this working.  Sorry went off topic at the end.  All the best to you!


--writable in singularity command

Short Description of the issue
Allow to edit any part of the container running:

to be discussed for possible implications at the HPCs

Environment

Operating System:
Other related components versions:

Steps to reproduce

Logs, stacktrace, or other symptoms

output

Summary of proposed changes

--writable-tmpfs

out dated


Affiliate Instances Do Not Have Updated Member Count

^ on uk.dosomething.org, canada.dosomething, etc.

 # of Members Not Updated on Affiliate Instances

 # of Members Not Updated on Affiliate Instances

@angaither any thoughts on this? Thanks!

@namimody uhm, it's a variable set in each database separately. not sure if/how it gets updated automatically....
(cc @deadlybutter)

No I don't think the affiliate sites would be updated. I can update the script however to do this, would be trivial.
BUT if we're migrating everyone to one site, is there really any point?

Thanks @angaither and the @deadlybutter! Call is up to @mikefantini!

Just going to close this since we are so close to launching global. Let me know if that's the wrong call @mikefantini.

BTW they can manually update this until than


Switch to calculating adjusted max velocity from wheel base

Rather than using a tuning variable (K), calculate how fast we can go around a circle with a certain radius with the wheelbase of our drivetrain. Should be definitive and math shouldn't be too bad.

Did this, robotsquiggles had some great stuff on this. Don't know if it actually works yet lol.


Fix compatibility with Firefox and IE

I'm not sure why using location.ancestorOrigins is necessary, considering that:

It's not compatible with Firefox. (for good reason)
Will be empty & crash if you directly load the page anyway.

I assume that this will always be rendered through NR1 so I don't see any downside to using location.href.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Unable to redirect to microsoft sso login page from app url when running cypress feature file using github action

Unable to redirect to company's microsoft sso login page from app url when running cypress feature file using github action however it works fine locally.

Error log snippet:

User Profile Screen Validation
Validate User Profile Popup Screen appears for a registered sso user:@ssoUser1
**CypressError: Timed out after waiting 120000ms for your remote page to load.

Your page did not fire its load event within 120000ms.
You can try increasing the pageLoadTimeout value in __w/app/app/cypress.config.js to wait longer.
Browsers will not fire the load event until all stylesheets and scripts are done downloading.
When this load event occurs, Cypress will continue running commands.**
Redirect url: https://login.microsoftonline.com/<>.onmicrosoft.com/oauth2/v2.0/authorize?login_hint=&prompt=select_account&response_type=code&client_id=<>&redirect_uri=https%3A%2F%2Fsso-dev.<>.com%2Flogin%2Fcallback&nonce=<>&scope=openid%20profile%20email%20https%3A%2F%2Fgraph.microsoft.com%2FUser.Read%20https%3A%2F%2Fgraph.microsoft.com%2FDirectory.Read.All&state=<>&sso_reload=true

Please do let us know if you need additional inputs. Thank you

@asagarelse
Please post your GitHub Actions workflow.

Hello @MikeMcC399 , Thanks for the response, much appreciable!
Here is the GitHub Actions workflow *.yaml file
name: Run Feature Test
on:
  workflow_dispatch:
    inputs:
      ENV:
        type: choice
        description: 'Environment'
        required: true
        default: DEV
        options:
          - DEV
      RUNNER:
        type: choice
        description: runner
        required: true
        default: deploy-eks
        options:
          - self-hosted
          - ephemeral
          - deploy-eks
      FEATURE:
        type: choice
        description: feature to be run
        required: true
        options:
          - UserProfile
          - Users
      BROWSER:
        type: choice
        description: browser
        required: true
        headed: true
        default: chrome
        options:
          - chrome
          - electron
          - firefox
      StartAPP:
        type: choice
        description: Starting Application
        required: true
        default: app
        options:
          - app

jobs:
  cypress-test-run:
    name: Execute Test-${{ inputs.FEATURE }}
    runs-on: [self-hosted, ephemeral, '${{ inputs.RUNNER }}']
    container:
      image: cypress/included:13.10.0
      options: -u root -e AWS_STS_REGIONAL_ENDPOINTS -e AWS_DEFAULT_REGION -e AWS_REGION -e AWS_ROLE_ARN -e AWS_WEB_IDENTITY_TOKEN_FILE
      volumes:
        - /var/run//test/123/secrets/eks.amazonaws.com/serviceaccount:/var/run/test/123/secrets/eks.amazonaws.com/serviceaccount

    steps:
      - name: Checkout
        uses: actions/checkout@v4

      - name: Get AWS Secrets
        id: secrets
        uses: ./.github/actions/fetchAWS-Secrets

      - name: Build and Run Tests
        shell: bash
        run: |
          export $SECRETS_ENV

          npm ci
          npx cypress run --spec 'cypress/e2e/features/${{ inputs.FEATURE }}.feature' --browser ${{ inputs.BROWSER }} -e StartAPP='${{ inputs.StartAPP }}',ENV='${{ inputs.ENV }}',RunFrom='CI'

      - name: Upload json
        uses: actions/upload-artifact@v4
        if: always()
        with:
          name: json-report-${{ inputs.FEATURE }}
          path: |
            cypress/cucumber-json/${{ inputs.FEATURE }}.cucumber.json
            cypress/screenshots/

  generateReport:
    name: Generate HTML Report
    needs: cypress-test-run
    if: always()
    runs-on: ${{ inputs.RUNNER }}
    container:
      image: cypress/included:13.6.0
      options: -u root -e AWS_STS_REGIONAL_ENDPOINTS -e AWS_DEFAULT_REGION -e AWS_REGION -e AWS_ROLE_ARN -e AWS_WEB_IDENTITY_TOKEN_FILE
      volumes:
        - /var/run/secrets/eks.amazonaws.com/serviceaccount:/var/run/secrets/eks.amazonaws.com/serviceaccount

    steps:
      - name: Checkout
        uses: actions/checkout@v4

      - name: Setting Permissions
        run: |
          chmod -R a+rwx /__w/identity-phoenix/identity-phoenix
          npm ci

      - uses: actions/download-artifact@v4
        with:
          path: /__w/identity-phoenix/identity-phoenix/cypress/cucumber-json

      - name: Display structure of downloaded files
        run: ls -R

      - name: Generate Report
        uses: ./.github/actions/generate-Report

@asagarelse
Thank you for providing your GitHub Actions workflow, which shows that you are running Cypress on a self-hosted runner using a shell command. You are not using the Cypress JavaScript GitHub Action cypress-io/github-action:
runs-on: [self-hosted, ephemeral, '${{ inputs.RUNNER }}']
...
steps:
npx cypress run --spec 'cypress/e2e/features/${{ inputs.FEATURE }}.feature' --browser ${{ inputs.BROWSER }} -e StartAPP='${{ inputs.StartAPP }}',ENV='${{ inputs.ENV }}',RunFrom='CI'

This issue list is for reporting bugs in cypress-io/github-action or for enhancement requests to the action. It is not for general issues regarding using Cypress under GitHub Actions. Your problem belongs into the second category and so this issue needs to be closed.
I hope you can find somebody on Discord to help you. You might need to share the test code which is failing. Good luck!


Failed to fetch snap...

MetaMask 11.7.0 - mina-portal 0.1.2
Failed to fetch snap "npm:mina-portal": No matching versions of the snap are on the allowlist.

@U00A3 The issue has been fixed! Can you check again?

Since this has been resolved. I will close this issue.


fixed a crash that occurred when bot tried to list available commands...

… and there was a unicode pattern in one of the respond_to functions
Example pattern:
@respond_to(u'привет', re.IGNORECASE)
def hi_answer(message):
    message.send(u'И тебе привет')

The crash traceback was:
Traceback (most recent call last):
  File "/Users/datagreed/.venv/marvin/lib/python2.7/site-packages/slackbot/dispatcher.py", line 33, in dispatch_msg
    func(Message(self._client, msg), *args)
  File "/Users/datagreed/Work/personal/Marvin/plugins/love.py", line 40, in generic
    u":sweat:",
  File "/Users/datagreed/.venv/marvin/lib/python2.7/site-packages/slackbot/dispatcher.py", line 158, in reply
    text = self._gen_reply(text)
  File "/Users/datagreed/.venv/marvin/lib/python2.7/site-packages/slackbot/dispatcher.py", line 125, in _gen_reply
    return self._gen_at_message(text)
  File "/Users/datagreed/.venv/marvin/lib/python2.7/site-packages/slackbot/dispatcher.py", line 119, in _gen_at_message
    text = '<@{}>: {}'.format(self._get_user_id(), text)
UnicodeEncodeError: 'ascii' codec can't encode characters in position 0-1: ordinal not in range(128)

Um, anyone? @lins05

@dangra thanks, I've updated the pull requests
@lins05 if eveything is okay, could you please merge accept the request and update the project version? I really want to use this in production with pip-installed package :)

Thanks for the PR! I have upgraded the pypi package to the latest version (0.2.4).

Nice, thanks!


Add TimeseriesDB and integrate it to chained-function-eventing

Signed-off-by: Bora M. Alper<EMAIL_ADDRESS>

Regarding duplicated structures in the tests, see https://github.com/ease-lab/vhive/pull/286#discussion_r655206825.


[controlPlane] Failed to upgrade Control Plane: [[host k8s-m1 not ready]]

Describe your issue here

Useful
Info

Versions
Rancher v2.5.15 UI: v2.5.14

Route
undefined

一样


Account key

In the 'Configure your storage connection string' section, it would be helpful to explain or provide a link on where to find your access key. Thanks

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2449e264-13a7-ecc0-7400-b6d6bed1b029
Version Independent ID: bcd3da61-efe6-4ef5-1c79-4fa78cdf473d
Content: Get started with Azure Queue storage using .NET
Content Source: articles/storage/queues/storage-dotnet-how-to-use-queues.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

@gcopley Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@gcopley - Thanks for your feedback.
I have added information to this article to explain how to retrieve the account key. The change will go live later today.
#please-close


Rename Element class

Redefining the Element class causes functionality in rails-ujs to break. Specifically the logic that handles triggering a POST request when a link with method: :delete is clicked no longer works as expected. This change renames the class to avoid the conflict.
I'm also adding a few functions that are expected to be defined on the stripe objects:

addEventListener for input validation
confirmCardSetup

Looks good to me. Thanks!


Demo does not work

On https://niiknow.github.io/vue-datatables-net/ , attempting to sort fields such as "ID", "Name", "Email" does not work. See screenshot

Yes, because I'm really not using the right back-end that support datatables.net server-side.   As a result, paging does not work either.  I will have to switch to a different back-end and update the doc.

Can anyone confirm the example shown on the home page actually works simply using NPM to install the libs and importing all the libs as shown.  I'm not even able to instantiate.

Hello, I just tried out the demo, but I always get the error message:

These are my steps:
git clone https://github.com/niiknow/vue-datatables-net
cd vue-datatables-net
npm install
npm run watch
No files were edited. I unlocked CORS with an extension.

Stale issue.


Add support for windows

I tried to compile it with setup.py (with changed src dir) but get a "stdio.h no found" error, and I tried nmake injector, and it succeded with several obj/lib/dll, how could I use these file to make accessible pyd for pyinjector?
Thx!

Hey @AlucardNosferatu , I've added windows support and uploaded windows wheels to pypi. Please check if they work for you :)

Hey @AlucardNosferatu , I've added windows support and uploaded windows wheels to pypi. Please check if they work for you :)

It's very nice of you that the update is so fast :) but as I try to use pip install pyinjector, I still cannot install it but with a new error:
C:\Windows\system32>pip install pyinjector
Looking in indexes: https://mirrors.aliyun.com/pypi/simple/
Collecting pyinjector
  Using cached https://mirrors.aliyun.com/pypi/packages/fa/86/95190dfe1b4fae06cd0bed1b7c90715f0d1950680ec346eaad3d3b31ad14/pyinjector-0.1.4.zip (23 kB)
Building wheels for collected packages: pyinjector
  Building wheel for pyinjector (setup.py) ... error
  ERROR: Command errored out with exit status 1:
   command: 'c:\program files (x86)\microsoft visual studio\shared\python37_64\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"'; __file__='"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' bdist_wheel -d 'C:\Users\16413\AppData\Local\Temp\pip-wheel-hnh0f1q4'
       cwd: C:\Users\16413\AppData\Local\Temp\pip-install-err9tpmn\pyinjector_2b66e69df7e840489404580dc2c6cff8\
  Complete output (20 lines):
  running bdist_wheel
  running build
  running build_py
  creating build
  creating build\lib.win-amd64-3.7
  creating build\lib.win-amd64-3.7\pyinjector
  copying pyinjector\pyinjector.py -> build\lib.win-amd64-3.7\pyinjector
  copying pyinjector\__init__.py -> build\lib.win-amd64-3.7\pyinjector
  copying pyinjector\__main__.py -> build\lib.win-amd64-3.7\pyinjector
  running build_ext
  building 'pyinjector.libinjector' extension
  creating build\temp.win-amd64-3.7
  creating build\temp.win-amd64-3.7\Release
  creating build\temp.win-amd64-3.7\Release\injector
  creating build\temp.win-amd64-3.7\Release\injector\src
  creating build\temp.win-amd64-3.7\Release\injector\src\linux
  C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\bin\HostX86\x64\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -DEM_AARCH64=183 -Iinjector\include "-Ic:\program files (x86)\microsoft visual studio\shared\python37_64\include" "-Ic:\program files (x86)\microsoft visual studio\shared\python37_64\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.8\include\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\cppwinrt" /Tcinjector\src\linux\elf.c /Fobuild\temp.win-amd64-3.7\Release\injector\src\linux\elf.obj
  elf.c
  injector\src\linux\elf.c(27): fatal error C1083: 无法打开包括文件: “elf.h”: No such file or directory
  error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Community\\VC\\Tools\\MSVC\\14.27.29110\\bin\\HostX86\\x64\\cl.exe' failed with exit status 2
  ----------------------------------------
  ERROR: Failed building wheel for pyinjector
  Running setup.py clean for pyinjector
Failed to build pyinjector
Installing collected packages: pyinjector
    Running setup.py install for pyinjector ... error
    ERROR: Command errored out with exit status 1:
     command: 'c:\program files (x86)\microsoft visual studio\shared\python37_64\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"'; __file__='"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record 'C:\Users\16413\AppData\Local\Temp\pip-record-_8vt8p6p\install-record.txt' --single-version-externally-managed --compile --install-headers 'c:\program files (x86)\microsoft visual studio\shared\python37_64\Include\pyinjector'
         cwd: C:\Users\16413\AppData\Local\Temp\pip-install-err9tpmn\pyinjector_2b66e69df7e840489404580dc2c6cff8\
    Complete output (20 lines):
    running install
    running build
    running build_py
    creating build
    creating build\lib.win-amd64-3.7
    creating build\lib.win-amd64-3.7\pyinjector
    copying pyinjector\pyinjector.py -> build\lib.win-amd64-3.7\pyinjector
    copying pyinjector\__init__.py -> build\lib.win-amd64-3.7\pyinjector
    copying pyinjector\__main__.py -> build\lib.win-amd64-3.7\pyinjector
    running build_ext
    building 'pyinjector.libinjector' extension
    creating build\temp.win-amd64-3.7
    creating build\temp.win-amd64-3.7\Release
    creating build\temp.win-amd64-3.7\Release\injector
    creating build\temp.win-amd64-3.7\Release\injector\src
    creating build\temp.win-amd64-3.7\Release\injector\src\linux
    C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\bin\HostX86\x64\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -DEM_AARCH64=183 -Iinjector\include "-Ic:\program files (x86)\microsoft visual studio\shared\python37_64\include" "-Ic:\program files (x86)\microsoft visual studio\shared\python37_64\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.8\include\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\cppwinrt" /Tcinjector\src\linux\elf.c /Fobuild\temp.win-amd64-3.7\Release\injector\src\linux\elf.obj
    elf.c
    injector\src\linux\elf.c(27): fatal error C1083: 无法打开包括文件: “elf.h”: No such file or directory
    error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Community\\VC\\Tools\\MSVC\\14.27.29110\\bin\\HostX86\\x64\\cl.exe' failed with exit status 2
    ----------------------------------------
ERROR: Command errored out with exit status 1: 'c:\program files (x86)\microsoft visual studio\shared\python37_64\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"'; __file__='"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record 'C:\Users\16413\AppData\Local\Temp\pip-record-_8vt8p6p\install-record.txt' --single-version-externally-managed --compile --install-headers 'c:\program files (x86)\microsoft visual studio\shared\python37_64\Include\pyinjector' Check the logs for full command output.

from log I see the error is caused by miss-use of elf.h in linux src code, however, I checked your setup.py and see it already modified with switchable win32 src dir, maybe there is some kind of env miss-configure in my own PC, I will double-check it now.

Hey @AlucardNosferatu , I've added windows support and uploaded windows wheels to pypi. Please check if they work for you :)

It's very nice of you that the update is so fast :) but as I try to use pip install pyinjector, I still cannot install it but with a new error:
C:\Windows\system32>pip install pyinjector
Looking in indexes: https://mirrors.aliyun.com/pypi/simple/
Collecting pyinjector
  Using cached https://mirrors.aliyun.com/pypi/packages/fa/86/95190dfe1b4fae06cd0bed1b7c90715f0d1950680ec346eaad3d3b31ad14/pyinjector-0.1.4.zip (23 kB)
Building wheels for collected packages: pyinjector
  Building wheel for pyinjector (setup.py) ... error
  ERROR: Command errored out with exit status 1:
   command: 'c:\program files (x86)\microsoft visual studio\shared\python37_64\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"'; __file__='"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' bdist_wheel -d 'C:\Users\16413\AppData\Local\Temp\pip-wheel-hnh0f1q4'
       cwd: C:\Users\16413\AppData\Local\Temp\pip-install-err9tpmn\pyinjector_2b66e69df7e840489404580dc2c6cff8\
  Complete output (20 lines):
  running bdist_wheel
  running build
  running build_py
  creating build
  creating build\lib.win-amd64-3.7
  creating build\lib.win-amd64-3.7\pyinjector
  copying pyinjector\pyinjector.py -> build\lib.win-amd64-3.7\pyinjector
  copying pyinjector\__init__.py -> build\lib.win-amd64-3.7\pyinjector
  copying pyinjector\__main__.py -> build\lib.win-amd64-3.7\pyinjector
  running build_ext
  building 'pyinjector.libinjector' extension
  creating build\temp.win-amd64-3.7
  creating build\temp.win-amd64-3.7\Release
  creating build\temp.win-amd64-3.7\Release\injector
  creating build\temp.win-amd64-3.7\Release\injector\src
  creating build\temp.win-amd64-3.7\Release\injector\src\linux
  C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\bin\HostX86\x64\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -DEM_AARCH64=183 -Iinjector\include "-Ic:\program files (x86)\microsoft visual studio\shared\python37_64\include" "-Ic:\program files (x86)\microsoft visual studio\shared\python37_64\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.8\include\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\cppwinrt" /Tcinjector\src\linux\elf.c /Fobuild\temp.win-amd64-3.7\Release\injector\src\linux\elf.obj
  elf.c
  injector\src\linux\elf.c(27): fatal error C1083: 无法打开包括文件: “elf.h”: No such file or directory
  error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Community\\VC\\Tools\\MSVC\\14.27.29110\\bin\\HostX86\\x64\\cl.exe' failed with exit status 2
  ----------------------------------------
  ERROR: Failed building wheel for pyinjector
  Running setup.py clean for pyinjector
Failed to build pyinjector
Installing collected packages: pyinjector
    Running setup.py install for pyinjector ... error
    ERROR: Command errored out with exit status 1:
     command: 'c:\program files (x86)\microsoft visual studio\shared\python37_64\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"'; __file__='"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record 'C:\Users\16413\AppData\Local\Temp\pip-record-_8vt8p6p\install-record.txt' --single-version-externally-managed --compile --install-headers 'c:\program files (x86)\microsoft visual studio\shared\python37_64\Include\pyinjector'
         cwd: C:\Users\16413\AppData\Local\Temp\pip-install-err9tpmn\pyinjector_2b66e69df7e840489404580dc2c6cff8\
    Complete output (20 lines):
    running install
    running build
    running build_py
    creating build
    creating build\lib.win-amd64-3.7
    creating build\lib.win-amd64-3.7\pyinjector
    copying pyinjector\pyinjector.py -> build\lib.win-amd64-3.7\pyinjector
    copying pyinjector\__init__.py -> build\lib.win-amd64-3.7\pyinjector
    copying pyinjector\__main__.py -> build\lib.win-amd64-3.7\pyinjector
    running build_ext
    building 'pyinjector.libinjector' extension
    creating build\temp.win-amd64-3.7
    creating build\temp.win-amd64-3.7\Release
    creating build\temp.win-amd64-3.7\Release\injector
    creating build\temp.win-amd64-3.7\Release\injector\src
    creating build\temp.win-amd64-3.7\Release\injector\src\linux
    C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\bin\HostX86\x64\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -DEM_AARCH64=183 -Iinjector\include "-Ic:\program files (x86)\microsoft visual studio\shared\python37_64\include" "-Ic:\program files (x86)\microsoft visual studio\shared\python37_64\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.8\include\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\cppwinrt" /Tcinjector\src\linux\elf.c /Fobuild\temp.win-amd64-3.7\Release\injector\src\linux\elf.obj
    elf.c
    injector\src\linux\elf.c(27): fatal error C1083: 无法打开包括文件: “elf.h”: No such file or directory
    error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Community\\VC\\Tools\\MSVC\\14.27.29110\\bin\\HostX86\\x64\\cl.exe' failed with exit status 2
    ----------------------------------------
ERROR: Command errored out with exit status 1: 'c:\program files (x86)\microsoft visual studio\shared\python37_64\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"'; __file__='"'"'C:\\Users\\16413\\AppData\\Local\\Temp\\pip-install-err9tpmn\\pyinjector_2b66e69df7e840489404580dc2c6cff8\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record 'C:\Users\16413\AppData\Local\Temp\pip-record-_8vt8p6p\install-record.txt' --single-version-externally-managed --compile --install-headers 'c:\program files (x86)\microsoft visual studio\shared\python37_64\Include\pyinjector' Check the logs for full command output.

from log I see the error is caused by miss-use of elf.h in linux src code, however, I checked your setup.py and see it already modified with switchable win32 src dir, maybe there is some kind of env miss-configure in my own PC, I will double-check it now.

I can successfully install it with downloaded zip and python setup.py install, but not pip install, well that's enough for me anyway, Thank you ! @kmaork 👍 👍

The reason it failed for you is because you are using a mirror of Pypi (mirrors.aliyun.com) and not pypi itself. You can see in the logs that it used pyinjector==0.1.4, and not 1.0.1. That is an older version that does not support windows.
I don't know this mirror, but it will probably sync with Pypi soon and you will be able to install the whl files I uploaded, instead of building it youself.

I now released hypno==1.0.0 which supports Windows and uploaded windows wheels to pypi 😄


Update actions/checkout workflow to v4

Description
Updates to latest version as using v2 now shows a depreciation message.
Checklist

[x] Tested against our testing policy (Resolution, Browser & Accessibility)
[x] Follows our coding standards and style guide
[ ] CHANGELOG entry

@paulrobertlloyd good catch! Could also set up Dependabot to keep these updated automatically.

Yeah; you want to add that to the dependabot workflow config?

@paulrobertlloyd there isn’t one! (yet)

Oh yeah; how is Dependabot working then?

@paulrobertlloyd dunno, some kind of default? 🤷♂


Try Documenter v1.0 for docs

This is ready to merge.

Bump!

If you PR from a branch in this repo (not from a fork) there is a Docs preview. I am not going to build this locally. If you did, and the docs look good, please merge yourself.

Hey, I don't have write access here so unfortunately can't spin up a local branch.  There is one error I'm seeing locally that I wasn't before with the demo card theming, will take a look at that...

Ok, please ping me again when this is ready!

superseded by https://github.com/MakieOrg/AlgebraOfGraphics.jl/pull/542


Bazel go_test: populate XML_OUTPUT_FILE and utilize new data in BazelTestEngine [phab:D368]

Created by schroederc at 2015-06-26 17:19:15:

zarko accepted this revision at 2015-06-26 19:34:34

schroederc closed via commit 1c3fbcc9a1fd6b364f07d7f0c8be9512ab275808 at 2015-06-26 19:35:44


opengraph twitter previews don't work

vs. whatever matrix.org is using, i assume the default synapse media handler:

Have you enabled oEmbed? https://github.com/turt2live/matrix-media-repo/blob/master/config.sample.yaml#L343-L348
Twitter doesn't support OpenGraph previews (unless the requester user agent includes "bot", but afaik both the matrix-media-repo and synapse developers have refused to make that change)

Have you enabled oEmbed?

i have tried that: https://github.com/turt2live/matrix-media-repo/issues/277

this issue was actually related to the OpenGraph preview handler, but it looks like the commit that closed this is related to oEmbed handling.  Testing the latest code confirms that Twitter still fails to properly preview with OpenGraph, but the fixes for oEmbed do appear to be working for me.

OpenGraph for Twitter won't work - Twitter disabled that ability.


refactor(infer): consolidate two main functions

when the infer function was developed, it grew quite fast and some functions were tailor made for very specific use cases. While separation of concerns is a good practice w.r.t. maintainability, the wheel must not be re-invented for each minor deviation in behavior.
That being said, the two function inferQueryElement and attachRefLinksToArg are doing essentially the same. They both walk an arg which is - generally spoken - an expression. That might be a simple ref, func, val or a (nested) xpr.

inferQueryElement was used for the columns of a query. Each column was resolved to a query element. All elements together formed the set of query elements. The other main part of the function is the attachment of the infamous $refLinks next to each ref array. Moreover, there was a flag insertIntoQueryElements which could be set to false, if the arg must not be inserted into the queries elements. This is the case for nested xpr, the where, having, with, groupBy and orderBy clause of the query.
attachRefLinksToArg does exactly what the name implies, it attaches $refLinks to an arg but does not resolve the arg to a query element. This function was used for the from clause of the query.

It is obvious that both functions have significant shared logic. This change removes the usage of the attachRefLinksToArg function.

The stakeholder tests (cap/stakeholder-tests/actions/runs/6078819) are all green with this change, so I am confident that this can be merged.

@patricebender @stewsk this pr breaks cds ci, i'll revert for now


Add new SoapResponseStrategy to help in predetermined test cases

Thanks for your work, really amazing.
I need to write some Integration test cases depending on castlemock, so that my application will call castelmock to return different each time for the same specific web service operation.
The point is, I cannot depend on both the existing  SoapResponseStrategy strategies (RANDOM and SEQUENCE), I cannot depend on RANDOM because my test cases need to know the result ahead so I could assert my code, and I cannot depend on SEQUENCE for many reasons one of them it is not a good idea to have test cases run in a certain order, they should run independently.
The suggested solution is to return different mock responses based on different input (XPath could used here).
BTW, SoapOperationStatus.ECHO is not helpful hence my application is sending real SOAP request message, and if I changed it to send some SOAP response to get it back in the request, I then changing in actual application code.
Thanks.

Here's a quick and dirty illustration of what I mean
castlemock_mock_xpath.patch.zip If you like the idea, I can put more effort in the code and implement the Web part as well and submit a PR

Hi @mhewedy!
Thank you using Castle Mock, and more importantly, thank you your amazing suggestions. XPATH has long been on my list to do. I really like your approach. Looks very straightforward. If you want, we can do this in two ways:

You make the changes and submit a PR. The only thing I would ask of you would be to sign and submit a contribution agreement (https://www.apache.org/licenses/icla.pdf).
I can make the changes. I can have the changes in place either today or tomorrow.

What would you prefer?
Once again, thank you very much.
Best regards,
Karl Dahlgren

Give me a day or two so I could try with the first option. otherwise, we can go for option 2. I'll be glad to contribute to this great software.

Sounds great! Looking forward to see the result.
Poke me if you need any support or anything.
Best regards,
Karl Dahlgren

I like the idea of linking response with request in XPath way it's really helpful feature  , great work guys .

This new XPATH feature has been implemented and merged into the develop branch.

Thank you @mhewedy for you contribution!
Great work!
Best regards,
Karl Dahlgren


support to be required by npm

I know this is a cortex package. If set dependencies like cortex.dependencies, it is possible to be required by npm?

Yes, I has already been an npm package at the very beginning.

But you didn't set dependencies. Can you add and publish?

landed in v0.1.1

oh, no. You should not add util to npm dependencies.
I will rollback 7539522.

Sorry, I forgot they are build-in modules.

I recall why I want to add those built-in dependencies! Because I want to use this package in webpack.


🚀. Publish to GitHub Nuget Registry in Release Workflow

Is there an existing issue for this?

[X] I have searched the existing issues

Suggestion
Add the nuget package to be released on GitHub in addition to Nuget.org. This should be added to the release workflow. This could likely be done with an action just following the nuget publish.
The step should follow this step in the release:
https://github.com/microsoft/electionguard-cpp/blob/99788bc9fb76ee2293e9b131965b91b85ba6fee3/.github/workflows/release.yml#L135-L137
Helpful Docs:
https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-nuget-registry
Possible Implementation
No response
Anything else?
No response

Hello @keithrfung, I would like to work on this too.

@avinal Thanks for the followup.


Trunk/shared model json related

Description of the Change
Change all JSON serialisation/deserialisation. Includes genesis, genesis load, raw block insertion and block loader. All this is for relocation to shared_model.

Benefits
less old model

Possible Drawbacks
no

Usage Examples or Tests [optional]
no

Alternate Designs [optional]
no

You can remove optional section from the PR description.
Also DCO is missing


Update projects.md

Added square bears to the projects file.

I'm only going to merge PRs with deployed contracts, so feel free to open this when you have that

Okay thank you

On Feb 17, 2022, at 11:17 PM, cygaar @.***> wrote:
I'm only going to merge PRs with deployed contracts, so feel free to open this when you have that
—
Reply to this email directly, view it on GitHub https://github.com/chiru-labs/ERC721A/pull/102#issuecomment-1044046016, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADFD5PY5TBYXOYXRNXNOQ5LU3XW7DANCNFSM5OLCDPQA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.


Do you have the python script  for subsampling the full-resolution mesh data with semantics

Hi, you may refer here https://github.com/weiyithu/SurroundOcc/blob/736b3ada2e4dd0b912593330fd2c96bfbb223aef/tools/generate_occupancy_nuscenes/generate_occupancy_nuscenes.py#L404. We first convert mesh vertices to voxels, and then use KNN to assign semantic labels.


Fix notification issues

Resolves https://github.com/NEARBuilders/gateway/issues/297
Resolves https://github.com/NEARBuilders/gateway/issues/298
Video Demo:
https://github.com/NEARBuilders/gateway/assets/46527178/ee48d5cc-0f8f-4b94-a705-0a292391ed04

I thought about it, it's going to be lots of forking though.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Elliot Braem @.>
Sent: Friday, May 17, 2024 9:48:50 PM
To: NEARBuilders/gateway @.>
Cc: Zeeshan Ahmad @.>; Author @.>
Subject: Re: [NEARBuilders/gateway] Fix notification issues (PR #304)
@elliotBraem commented on this pull request.

In src/pages/Viewer.jshttps://github.com/NEARBuilders/gateway/pull/304#discussion_r1605310644:

@@ -51,15 +51,17 @@ function Viewer({ code, widgets }) {
}, []);

return (

<Widget

 src={!code && src}

 code={code} // prioritize code

 props={{

   path: src,

   ...passProps,

 }}

 config={{ redirectMap }}

/>

Instead of doing this so "Show post" and "show comment" show -- how about we modify these links to point to the Build DAO gateway style of posts/comments?
This will involve pulling in the full notifications from mob.near so we can modify the details, but that is okay.
—
Reply to this email directly, view it on GitHubhttps://github.com/NEARBuilders/gateway/pull/304#pullrequestreview-2063959318, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALC7FSSZNLUZESCLNJMTZFTZCYYHFAVCNFSM6AAAAABH3MXWBKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDANRTHE2TSMZRHA.
You are receiving this because you authored the thread.Message ID: @.***>


Change on Step 6

I have implemented those steps and I have found a syntax error in a command (wrong parameter name) and I have also found a mismatch error in the resource location. I have added 2 notes, one on top of the doc and another one on the step 6 where I came across the error

@sethmanheim
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
Also, the notes in the diff view will need to be converted to the standard Markdown format before we can merge. The PR review team can do that for you after you sign off and before we merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


fix(audits/server): Avoid auditing non well-formatted GraphQL-over-HTTP requests

https://github.com/graphql/graphql-over-http/pull/241

:tada: This PR is included in version 1.18.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added MS Graph Search Api usage

Q
A

Bug fix?
no

New feature?
yes

New sample?
no

What's in this Pull Request?
As promised and already demosntrated in last week's PnP community call I implemented the usage of Microsoft Graph Search Api as a configurable alternative.

Nice update - thanks so much!


Slider Does Not Work When Passing Base64 Pictures:

picture.src contains base64 Picture data. IT DOESNT WORK.
                <AwesomeSlider onTransitionRequest={(e)=>{console.log(e)}} animation="cubeAnimation">
                    {
                        pictures.map((picture, index) => {
                            return (
                                <div key={index} data-src={picture.src} />

                            )
                        })
                    }
                </AwesomeSlider>

Check This: https://github.com/rcaferati/react-awesome-slider/pull/117


JWT authentication

Related to #14. It uses solidus_jwt gem in order to authenticate users via JSON Web Token. Authentication errors are reported to the client using the 401 Unauthorized HTTP status.
It comes with a RSpec configuration refactoring, made because I was having troubles testing Spree::GraphqlController. If needed it could go into another PR.

I'm closing this one for the moment; big changes are on the way


Enhance Build Hook with AI Features: Enhanced Logging Mechanism and Error Management and Dynamic Configuration

Some major changes have been done in the CustomBuildHook script in this commit including the integration of AI and optimization of the build process for the Open Webui frontend. The first area of improvement in these updates is in the logging mechanism, secondly in the error handling mechanism and lastly in incorporation of AI in the configuration patterns.
First of all, the script has been modified to have proper logging mechanism implemented. This addition is very important for improved visibility and for debugging purposes all through the build process. New verbose messages have been added to log the start and finish of major build operations, including npm install and npm run build. This logging is useful not only to get information but also to identify the problems. In the event of any failure in the subprocess commands, the exact messages that are displayed are logged down to assist in the easy detection of the cause of the fault. This enhanced logging mechanism helps to guarantee that any exception that may arise in the course of building the project is well captured and easily dealt with.
Secondly, the process of error handling has been enhanced as compared to the previous version. Earlier, it had no provisions of how to deal with errors in the subprocess commands. Some of the enhancements include; Error checking for the subprocess. run commands to enable correct identification and reporting of any errors that may occur. This advancement guarantees that if command such as npm install or npm run build fails the script will write down the error message and throw an exception. This approach stops the process and thus does not allow any further execution with possible build problems and enables a quick focus on the errors.
The most prominent change is the addition of the dynamic configuration with the help of Artificial Intelligence. A new method apply_dynamic_configuration has been included in this paper on how AI can be used to control the build process using the build data. This is a place-holder method which can be enhanced with a form of AI which can process build data and apply modifications or corrections if necessary. At present this method is empty but it opens up a prospective for future integration of AI that can help streamline the build process.
The script also exports an environment variable, $APP_BUILD_HASH with the current build version. This variable is useful in maintaining and controlling the different builds in a project and make sure that each build will be labeled according to its version.
In conclusion, these update help to improve the script by providing a better way of navigating through the code, better handling of exceptions and also prepare the script for future integration of artificial intelligence.
Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:

[ ] Target branch: Please verify that the pull request targets the dev branch.
[ ] Description: Provide a concise description of the changes made in this pull request.
[ ] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[ ] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[ ] Testing: Have you written and run sufficient tests for validating the changes?
[ ] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[ ] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description

[Concisely describe the changes made in this pull request, including any relevant motivation and impact (e.g., fixing a bug, adding a feature, or improving performance)]

Added

[List any new features, functionalities, or additions]

Changed

[List any changes, updates, refactorings, or optimizations]

Deprecated

[List any deprecated functionality or features that have been removed]

Removed

[List any removed features, files, or functionalities]

Fixed

[List any fixes, corrections, or bug fixes]

Security

[List any new or updated security-related changes, including vulnerability fixes]

Breaking Changes

BREAKING CHANGE: [List any breaking changes affecting compatibility or functionality]

Additional Information

[Insert any additional context, notes, or explanations for the changes]

[Reference any related issues, commits, or other relevant information]

Screenshots or Videos

[Attach any relevant screenshots or videos demonstrating the changes]

This pull request brings major changes to the existing CustomBuildHook script which aims at improving the build process by incorporating intelligence in the process as well as better error handling.

Enhanced Logging:

Added logging to see the step by step build process and to identify the errors with ease. This gives a better understanding of the build process and also provides a means of identifying problems by the use of detailed error messages.

Improved Error Handling:

Implemented improved error checking for the subprocess. grep -E ‘make: *** This makes sure any problem that may occur during the npm installornpm run build` will be documented and exceptions will be thrown so as to prevent the occurrence of other problems.

AI-Driven Dynamic Configuration:

Implemented a new method called apply_dynamic_configuration that will pave way to the integration of AI based approach. This method shows that the future developments could use the AI to adapt the build process according to the build data.

Environment Variable Handling:

Make the APP_BUILD_HASH environment variable to contain the build version to ease on the version control and management.

Impact:
All these changes are intended to improve the usability of the build process by adding more log information and improving the way that errors are handled. The addition of the capability in the form of dynamic configuration is the first move towards improving the build process with the use of AI and its potential for future improvement based on analysis.


Added custom html template

I think this template should be used more, cause it fixes some major fullscreen issues, and looks great :)

That seems really great.
Thanks, @egordorichev!


Rename

Are there plans to publish a version of the gem with the new name?

There are but we're prioritising other features right now - happy to take a PR tho!

Let me know what you think about https://github.com/Flagsmith/flagsmith-ruby-client/pull/7

Thanks so much @NeoPhi  - this is really great. Will push to a new ruby gem now.

Thanks so much @NeoPhi  - this is really great. Will push to a new ruby gem now.


github: Don't sync dependency dashboard to AzDO

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Exclude "Dependency Dashboard" issue when syncing to Azure Devops.
Why should this Pull Request be merged?
Keep "Dependency Dashboard" out of bug backlog.
What testing has been done?
None

Invalid workflow file: .github/workflows/sync_github_issues_to_azdo.yml#L13
The workflow is not valid. .github/workflows/sync_github_issues_to_azdo.yml (Line: 13, Col: 9): Unexpected symbol: '"Dependency'. Located at position 65 within expression: !github.event.issue.pull_request && github.event.issue.title != "Dependency Dashboard"

Invalid workflow file: .github/workflows/sync_github_issues_to_azdo.yml#L13
The workflow is not valid. .github/workflows/sync_github_issues_to_azdo.yml (Line: 13, Col: 9): Unexpected symbol: '"Dependency'. Located at position 65 within expression: !github.event.issue.pull_request && github.event.issue.title != "Dependency Dashboard"

Fixed in #524


Update WindowsBuild.md

When building against released libxml2 versions from upstream, we need to explicitly specify the static flag.  This should help make the builds reproducible by others.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test


chore(make): extracts version from git tag

Ongoing version in the built binary will be latest-tag-next, so that we know from which stable release it evolved.

@aslakknutsen This PR will be exceptionally merged w/o the review as you are not available this and coming week. Adding you as a reviewer though so you can see what has been changed.


Splitting a polygon with a polyline that crosses the polygon multiple times sometimes does not work

me again, I thought i'd create this one a separate issue as its not to do with holes.  If I split this polygon:
{"type":"Polygon","coordinates":[[[0,0],[10,0],[10,10],[0,10],[0,0]]]}
with this polyline:
{"type":"LineString","coordinates":[[-2.5,5],[5,12.5],[12.5,5],[5,-2.5]]}
the output is this, the third polygon it returns is not correct
{"type":"Feature","properties":{},"geometry":{"type":"MultiPolygon","coordinates":[[[[0,6.011473509933775],[0,0],[6.015058430717863,0],[10,3.965083056478406],[10,5.998302828618968],[5.991633333333334,10],[4.021819699499164,10],[0,6.011473509933775]],[[6.015058430717863,0],[10,0],[10,3.965083056478406],[6.015058430717863,0]],[[10,3.965083056478406],[10,5.998302828618968],[5.991633333333334,10],[4.021819699499164,10],[0,6.011473509933775],[0,0],[6.015058430717863,0],[10,3.965083056478406]],[[10,5.998302828618968],[10,10],[5.991633333333334,10],[10,5.998302828618968]]]]}}
I could consider restricting that use case as its probably a bit complex for my needs in reality.  I have to say though I'm finding it very easy to find cases where it returns incorrect results.  If you like I can share some more

I have to say though I'm finding it very easy to find cases where it returns incorrect results. If you like I can share some more

Welcome to the joy of writing geometry algorithms when no-one is paying you to do so (even though there is a sponsor button on the repo) ;)
On a more serious note this is still v0.0.1, it hasn't been battle tested, so failing cases do help.
In this case it looks like the 3rd contour is a duplicate of the 1st (albiet with the coords in different order). I suspect at some stage the rewiring of the polygons is taking a wrong turn and so it's collecting the wrong output contour.

sorry I didn't meant to sound overly critical.  I'm happy to share more examples if they are helpful.  I'll add them as new issues

No worries @timrutter :)
I've just published 0.0.2 which fixes a bunch of issues including this issue (not the single segment issue of #2 yet) but I'd be keen for you to test it out.
Turns out setting the fill-opacity of polygons to something greater than 0 helps to spot the dodgy overlapping polygons because if there are overlapping polys they become transparent because of svg rendering rules.


gradient barrier points present in waterbodies

Some gradient barrier points are in waterbodies because the valley slopes up quickly from river/lake edge.
When running the slope calculation, remove streams of waterbody edge types.

fixed with https://github.com/smnorris/bcfishpass/commit/e1a1d68bc1c0d6237b47a225e9c72b82488a038a

fixed with https://github.com/smnorris/bcfishpass/commit/e1a1d68bc1c0d6237b47a225e9c72b82488a038a


check uniqueness fails

I cannot add a new lecture using the angular frontend.
This is likely due to the fact that lectures/check has not yet received its gql counterparts.
The whole flow should also be reconsidered. This check should be explicitly done on the server when trying to add a new card and not be a seperate call on the frontend

agree

I remember why this was implemented this way:
When adding a lecture, this lecture is first stored in localstorage and only if the user also adds a card, will the lecture be added on the server. This was to prevent people from creating lectures without adding cards.
Thus this check makes sense in my opinion^^
Anyways, it is fixed


Add SonarScanner script for CI

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Warning callout

Warning callout
Use this issue to discuss the Warning callout component.
Preview the warning callout component
Find out more about the warning callout component and when to use it in the NHS Digital service manual.
See our contributing guidelines

Closing these individual placeholder issues.
Please use the NHS digital service manual backlog to discuss the usage of this component. If you're having trouble with the code, or have found a bug, please raise an issue on the NHS.UK frontend backlog


fix: compilation

What this PR changes/adds
Fix compilation that was broken because of upstream changes
Why it does that
Briefly state why the change was necessary.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes # <-- insert Issue number if one exists

Codecov Report
Patch coverage: 50.00% and project coverage change: -0.16% :warning:

Comparison is base (4ba9dc9) 65.58% compared to head (3e9b8f5) 65.43%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
- Coverage   65.58%   65.43%   -0.16%
==========================================
  Files          25       25
  Lines         433      434       +1
  Branches       15       15
==========================================
  Hits          284      284
- Misses        134      135       +1
  Partials       15       15

Files Changed
Coverage Δ

...ache/FederatedCatalogDefaultServicesExtension.java
0.00% <0.00%> (ø)

...edc/catalog/store/InMemoryFederatedCacheStore.java
96.15% <66.66%> (-3.85%)
:arrow_down:

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


Add Response#raw

Summary
Fork of https://github.com/antaz/llm/pull/13

Add LLM::Response#raw
The parsed response body from a LLM response
Add LLM::Response::Completion
Represents a response from a completion API.
A subclass|type of LLM::Response.
Add LLM::Completion#messages
Delegates to @provider.completion_messages
Add LLM::Provider#completion_messages
Returns an array of LLM::Message objects
(After parsing a provider-specific data structure)
Adjust require logic
Our require logic was hard to follow and duplicated.
Try to standardize a different approach

@antaz What do you think about this approach ? The completion_messages method has been made private as  suggested previously. That hides it from IRB autocomplete as well.