Android Support

Any plans for Android implementation, or recommendations of other modules?

:+1:

Android does not have any control which will open a message compose window within your own app - if you want a view to keep you in your own app you will need to design & code it manually.
The other option is to use the android intent system - this will allow the user to launch any app which supports sending sms messages, but this is launched outside of your own app. If you use the startActivityForResult method it will return you to your app.
If I get the time I will take a look and see how feasible it is to add that support to this module.
In the meantime, maybe the following module will help? https://github.com/lucasferreira/react-native-send-intent

React Native has built-in support for intents with Linking. See https://github.com/anarchicknight/react-native-communications for an easy-to-use wrapper that supports this.

This module was made for a specific reason of opening a message view within your own app.
This is not possible on Android as discussed above.
There are other modules already discussed which have support for launching a message view outside of your app.
Going to close this

Can any one suggest any package for this support. Need to work for both iOS and android.


build a docker image from a CI pipeline

Set up a Gitlab pipeline to build and publish a Docker image with the dependencies, output, and data (src, test, docs, etc can be skipped).

Published to https://hub.docker.com/repository/docker/ssube/textual-engine from https://git.apextoaster.com/ssube/textual-engine/-/pipelines


document inference

hi,
could you provide example for predicting new document topics or vectors from trained model?
thanks.

My guess is you would use fit_partial here: https://github.com/cemoody/lda2vec/blob/master/examples/twenty_newsgroups/lda2vec/lda2vec_model.py#L37
There is an update_only_docs options that freezes the word embedding matrix and topic matrix.
Then in the mixture, you fit a new w for the new document and replace it with the w here: https://github.com/cemoody/lda2vec/blob/master/lda2vec/embed_mixture.py#L108
It would be great to have this functionality in official capacity.

I randomly initialized a topic distribution vector for the new document and appended it to the document matrix from the training. Then I used fit_partial with update_only_docs set to True. After a few hundred epochs, I was getting decent topic distribution of the new document. The problem is that the new document vector is not as sparse as the document vectors from the training. Maybe increasing the number of epochs can solve the problem, but then it would take too long for prediction.

@yg37 have you done any additional thinking on this?


input type="time"

Hello, first of all I wanted to congratulate you for the great work you have done. Regarding the question I wanted to ask you, I have not seen any input type time, yes it is true that you have a input type datetime and from this I can obtain only the time, but it is not the same, the question is if you are considering including it in a future.

@abrahammarco7 Thank you for submitting feature request. We will look into adding time type.


dependencies attribute doesn't work inside repeatable field

Discussed in https://github.com/Laravel-Backpack/CRUD/discussions/4097

Originally posted by rickyviz January 19, 2022
dependencies attribute doesn't work inside repeatable field
i using  a select2 field that depends on another field, but cant get dependencies, using dd($form = backpack_form_input()); but repeatable request still null, need this for using in query parameter

PR https://github.com/Laravel-Backpack/CRUD/pull/4211 treats it in backpack/crud.
PR https://github.com/DigitallyHappy/backpack-pro/pull/8#pullrequestreview-896082535 treats it in backpack/pro.
Let's keep this open until both are merged and released.

@promatik can you please review this too?

Almost ready. @pxpm please fix the merge conflicts in https://github.com/DigitallyHappy/backpack-pro/pull/8

Merged https://github.com/DigitallyHappy/backpack-pro/pull/8 - 12 days ago. Will be tagged and released later today 🎉


SNAP_FAST_db_dir database error running in comprehensive mode

Hi, I am running surpi on a dataset using comprehensive mode. I do not have a SNAP_FAST_db_dir database, and i a getting an error message in the generated log file as a result. Do I need to have the SNAP_FAST_db_dir database even if I am not running in fast mode?

Hi,
You do not need a SNAP_FAST database when running in comprehensive mode, though currently SURPI does require it. In order to change this, you can change the code in the following way:
In the SURPI.sh script, remove the following lines (Lines 563-576). That will remove the FAST check...
echo -e "SNAP FAST Mode database"
for f in $SNAP_FAST_db_dir/*
do
if [ -f $f/Genome ]
then
echo -e "\t$f: ${green}OK${endColor}"
else
echo -e "\t$f: ${red}BAD${endColor}"
if [ "$run_mode" = "Fast" ]
then
reference_check="FAIL"
fi
fi
done

Ahh thank you very much
Barry
On Jan 30, 2017, at 5:31 PM, Scot Federman <notifications<EMAIL_ADDRESS>wrote:
Hi,
You do not need a SNAP_FAST database when running in comprehensive mode, though currently SURPI does require it. In order to change this, you can change the code in the following way:
In the SURPI.sh script, remove the following lines (Lines 563-576). That will remove the FAST check...
echo -e "SNAP FAST Mode database"
for f in $SNAP_FAST_db_dir/*
do
if [ -f $f/Genome ]
then
echo -e "\t$f: ${green}OK${endColor}"
else
echo -e "\t$f: ${red}BAD${endColor}"
if [ "$run_mode" = "Fast" ]
then
reference_check="FAIL"
fi
fi
done

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/chiulab/surpi/issues/28#issuecomment-276226291, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYKMN4pLCQ9r8ARIXfeHfFlPqpXPtJ7Jks5rXnLlgaJpZM4Lx9FA.

The materials in this message are private and may contain Protected Healthcare Information or other information of a sensitive nature. If you are not the intended recipient, be advised that any unauthorized use, disclosure, copying or the taking of any action in reliance on the contents of this information is strictly prohibited. If you have received this email in error, please immediately notify the sender via telephone or return mail.


feat: emptyStringAsUndefined option

Closes #109
I also unified JSDoc between docs and code, because I noticed that they were out of sync.

@juliusmarminge Can you take a look at this PR? It has been sitting here for a while.

Sorry bout that, it wasnt in my review pile so didnt see it. I'll have a look tomorrow when I'm at my computer


Use visibility overriding to resolve text node issues

Use visibility with CSS overriding to resolve text node issues
Add multiple run check
Use id #mainonly instead of class .mainonly
Update README minified

Still that's a great PR and truly solves the text node problem, so I will accept it gladly!


"Major change" highlighting not correct

In the "Update edit page" section, there is one line highlighted as a major change in the OnPostAsync method but in fact the entire method is different from what the scaffolding provided, including the method signature.

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

ID: 8f592059-3328-88a8-5c6c-43121ac8a0a5
Version Independent ID: f8a63abf-824b-5b80-e03d-abcb2c162b95
Content: Razor Pages with EF Core in ASP.NET Core - CRUD - 2 of 8
Content Source: aspnetcore/data/ef-rp/crud.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

We'll look at these in the next doc sweep.


Correct regressions towards apktool.yml generation

fixes: #3169

[x]  mApkFileName should return to apkFileName
[x]  mSdkInfo should return to sdkInfo
[ ]  resourcesAreCompressed was what compressionType, fall back for the old
[x]  sdkInfo.* missing is because minSdkVersion will overwrite all

compressionType is just hard-coded to false. It appears that broke between 2.0.0 and 2.1.0 when we removed the Zip64 stuff, which meant we no longer parsed the resources.arsc to check for stored/deflate.
So not sure if I should bring that back or just keep status quo for keep it as false.

compressionType is just hard-coded to false. It appears that broke between 2.0.0 and 2.1.0 when we removed the Zip64 stuff, which meant we no longer parsed the resources.arsc to check for stored/deflate.
So not sure if I should bring that back or just keep status quo for keep it as false.
Regression occurred back here for compressionType: v2.0.1...v2.0.2

In the master branch decode - build worked without exception.
I have not found an opportunity to make a synonym for the field in the snakeyaml.
I think we may make custom serializer to yaml without using any library - structure very simple.

In the master branch decode - build for this apk worked without exception.

Yeah its tough to notice since a newly decoded application would generate a apktool.yml that the build understood. Probably more noticable for these various tools that swap versions or rebuild versions from previously decoded files. I don't think there is a bug with build - just a regression of the property names.

I have not found an opportunity to make a synonym for the field in the snakeyaml. I think we may make custom serializer to yaml without using any library - structure very simple.

Not sure I follow - I was going to try and prefer resourcesAreCompressed if its set and fall back to compressionType if its not found. However, the typing of the properties probably makes that difficult since neither would be nullable.

Not going to make any changes at the moment for the compression ones. They've been broken for years, but I'll add it back even though its false 100% of the time.


Need highlighting to the symbol there gets cursor

When Quick Open for Find in A File get invoked using by Ctlr+Shift+O , In my opinion, when the highlighting point to the symbol which owns the cursor in the editor that is more useful. So I move easily the cursor to a symbol nearby using by Up or Down arrow key.
Most some tool such as Sublime Text or Eclipse do that.
my environment is
Code Insiders latest
Primary tool is c/c++
OS is Ubuntu Gnome 16.04

I don't think I understand. When you focus a symbol, the editor should scroll to that symbol. Is that not what you see?

My point is that the focus always place on top instead of the  current symbol of the editor on start up.
The editor scrolls good whenever the  focus moves. That is not problem. I hope you understand what I mean.
Thanks.

@eypsume sorry I do not understand what you mean, maybe draw a picture of what you mean to illustrate your idea?

@eypsume sorry I do not understand what you mean, maybe draw a picture of what you mean to illustrate your idea?


security/acme-client: Add NSUPDATE_ZONE support to nsupdate DNS-01 Service

This change is to update nsupdate (RFC 2136) DNS Service for the Let's Encrypt module to support a Zone declaration.  As described in PR#1963 for acme.sh:

Some DNS servers for which dns_nsupdate.sh is applicable (such as dyn.com's 'Standard DNS' TSIG update mechanism), require that the zone be set during the nsupdate transaction.  Therefore we add a new environment variable NSUPDATE_ZONE which is used to set the zone for the DNS TSIG transaction.

Currently OPNSense is behind what acme.sh supports for the nsupdate DNSAPI. See the acme.sh pull request for more detailed information.

This has been tested on:

Versions      OPNsense 20.1.7-amd64
              FreeBSD 11.2-RELEASE-p20-HBSD
              OpenSSL 1.1.1g 21 Apr 2020

Using nsupdate DNS Service with nsupdate table something like:

Server (FQDN)    update.dyndns.com
Zone             myzone.tld
Secret Key       key "xxxxxxxxxxxxxxx" {
                    algorithm hmac-md5;
                    secret "xxxxxxxxxxxxxxxxxxxxxxxxxxxx";
                 };

Which resulted in a valid cert issued by Let's Encrypt.

@fraenki
Ready for review and comment. If I've screwed something up - please let me know what and I will correct straight away.

Merged, thanks!

@fraenki Glad to be of some small help.


grammar

https://github.com/romainpanno/liber/blob/096561c7262a6b8a38790913550fd5758f4f5290/README.md?plain=1#L26
waring -> warning
https://github.com/romainpanno/liber/blob/096561c7262a6b8a38790913550fd5758f4f5290/README.md?plain=1#L85
for update -> to update
sorry 😢

Thanks for the issue 😭


AMP app mention? :)

Awesome - I checked the theme out.  Nice :)
I am the lead developer of the AMP for Shopify app which, while we don't AMP the homepage yet, we are quickly becoming an incredibly robust solution, powering thousands of shops and millions of unique Shopify pages worldwide.  If possible, a mention would be awesome - I'm happy to chat about routes to maybe collaborate on an update to this or other themes.
You can download the app from Shopify, as well.
Cheers

Sorry, I did not get your point. Not sure what you mean by "mention"

Linling Zhang
Mobile: +44(0)7922505235
Email<EMAIL_ADDRESS>------------------ Original ------------------
From:  "Stephen<EMAIL_ADDRESS>Date:  Thu, Sep 20, 2018 04:23 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject:  [DrLina/Shopify-AMP-Theme] AMP app mention? :) (#2)
Awesome - I checked the theme out.  Nice :)
I am the lead developer of the AMP by Shop Sheriff app which, while we don't AMP the homepage yet, we are quickly becoming an incredibly robust solution, powering thousands of shops and millions of unique Shopify pages worldwide.  If possible, a mention would be awesome - I'm happy to chat about routes to maybe collaborate on an update to this or other themes.
Cheers
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hello.
DrLina, could I get your email? Have a question, maybe you can help me.. Its will be good.. God bless ya if you still there..


CONFIG_SOC_MCIMX7D_M4 is undefined but referenced

Referenced in ext/hal/nxp/imx/devices/MCIMX6X/device_imx.h.

@ulfalizer ,
How does the bug manifest itself? It asks if the symbol is defined, which isn't, therefore it just skips some code.
The file is in ext folder, coming from FreeRTOS BSP for i.MX processors.
The files in ext folder are usually not modified, but in this case CPU_MCIMX6X_M4 has been changed to CONFIG_SOC_MCIMX6X_M4 and similarly for CPU_MCIMX7D_M4.
The original file:
/*
 * Include the cpu specific register header files.
 *
 * The CPU macro should be declared in the project or makefile.
 */
#if defined(CPU_MCIMX6X_M4)

    /* CMSIS-style register definitions */
    #include "MCIMX6X/include/MCIMX6X_M4.h"
    #define RDC_SEMAPHORE_MASTER_SELF   (5)
    #define SEMA4_PROCESSOR_SELF        (1)

#elif defined(CPU_MCIMX7D_M4)

    /* CMSIS-style register definitions */
    #include "MCIMX7D/include/MCIMX7D_M4.h"

    #define RDC_SEMAPHORE_MASTER_SELF   (6)
    #define SEMA4_PROCESSOR_SELF        (1)

#else
    #error "No valid CPU defined!"
#endif

Perhaps the #if directives could be removed and only the relevant branch could stay there.
But I wanted to make only minimal modifications to the external code.

@stanislav-poboril
IMO, dead code is a bug in itself, because people normally assume that all code is used (and might need to be updated).
Could leave it in if you think it'd be more work than it's worth to maintain a "fork" though.

I understand. The legacy FreeRTOS BSP and this file is probably not going to be updated, or updated frequently.
Bigger issue I think is that this file (and possibly others) is duplicated in ext for i.MX6 and i.MX7. If it would not duplicated but shared, there would not be a dead code.  I will have a look if some of those files in ext could be shared between i.MX6/7 instead of removing the dead code first.

Fixed in pr #13818


Use github actions instead of bash commands

Codecov Report

Merging #360 (498c8bd) into main (64a98cc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #360   +/-   ##
=======================================
  Coverage   53.58%   53.58%
=======================================
  Files          59       59
  Lines        4893     4893
=======================================
  Hits         2622     2622
  Misses       1973     1973
  Partials      298      298

Continue to review full report at Codecov.

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


Vue3: Virtualisation Management page is blank

Setup

Rancher version: dev of 2.10.0

Describe the bug

Blank page is shown instead of harvester cluster list
Code lives in harvester-manager built-in extension

To Reproduce

spin up head
Nav to Virtualisation Management

Result

blank page
warnings and error in console

Expected Result

empty list

Screenshots

Additional context

The error [Vue warn]: Invalid VNode type: undefined (undefined) leads me to believe that this error is a result of a Vue2/Vue3 version mismatch. Seeing as how harvesterhci.io.management.cluster^1 is defined in the harvester/dashboard repo, I think that we will have to migrate Harvester to Vue3 before we will be able to get this functioning.

Worth noting that the harvester extension is only used in the cluster context, and is loaded when a user navigates to one. This page works with the built-in harvester-manager extension.
Separate issue, the linked file should be in their extension rather than the forked dashboard code.

Validated on Rancher version: v2.10-5d66d49d7a72b4c439a33043007e9bb0223c8693-head
UI: master cf8c6d1
Navigation to the Virtualization Manager is no longer broken. No errors were thrown on console either.


Use stylis-rule-sheet plugin

Replace our plugin to split rules with stylis-rule-sheet

@thysultan do you mind reviewing this branch?


Signer crash?

I can't create invoice and I found this on the logs:
[BreezSdk] {WARNING} (2024-01-21T20:34:41.700656Z) : policy failed: policy-other get_current_holder_commitment_info: invalid next holder commitment number: 831 != 832
[BreezSdk] {SEVERE} (2024-01-21T20:34:41.700688Z) : FAILED PRECONDITION: policy failure: get_current_holder_commitment_info: invalid next holder commitment number: 831 != 832
[BreezSdk] {SEVERE} (2024-01-21T20:34:41.703995Z) : BACKTRACE:
0:
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33: _ZL15__pthread_startPv
34: __start_thread

we've been working on addressing this - see https://gitlab.com/lightning-signer/validating-lightning-signer/-/issues/207 and https://github.com/ElementsProject/lightning/pull/7010

This has been fixed by the recent CLN release. Closing this now as we deploy the new versions on the server.


Allow for stream=true on CWL location attributes

Only useful if presign is also set to true, but does not "MOUNT" the presigned url, and only adds the presigned URL to the input json and not the list of mount paths.

More complex than originally thought, removing from milestone.

With the 'externalData' attribute now available, it is worth considering using stream=True as a default and renaming this instead to 'stage', set to false by default when presign is set to True. For 'non-icav2' inputs that must be presigned, this would also be set to true.
Because presigned urls do not accept any additional parameters without affecting the presigned url itself, it would not be possible to download presigned urls specified by the user, instead these would need to remain as strings

I think the options should be

For ICAv2 inputs, have the option to 'presign', this means location attributes are converted to presigned urls for location attributes. ICAv2 inputs without the presign option will be downloaded with a corresponding 'mountPath' attribute.

For non ICAv2 inputs, users would have the option to 'download', set to true by default, if set to false, the location attribute would be converted into a presigned url.

Update:
Use 'stage' as the parameter. By default for all inputs, icav2, gds, presigned urls.., stage is set to true
Given presigned urls cannot accept an additional query parameter, a user can specify stage as a boolean value adjacent to the input. So both the following methods would be accepted
class: File
location: icav2://playground-proj/path/to/fastq.gz?stage=true

class: File
location: icav2://playground-proj/path/to/fastq.gz
stage: false

And
class: File
location: https://s3.amazonaws.com..../presigned.url
stage: false

This would also be acceptable for directories
class: Directory
location: icav2://playground-proj/to/directory
stage: false

class: Directory
location: gds://proj/path/to/directory/
stage: false

Where presigned urls would then be generated for each file inside the directory.


Antidepressant drug

Add a drug that boosts mood, suitable for "treating" the Pessimist and Depressive traits (permanent -6 and -12 to mood respectively). Should have side-effects: maybe increased rest need (drowsiness), vomiting (nausea), increased food need (weight gain)?
No tolerance, but the drug should create a dependency with withdrawal symptoms on discontinuation: maybe reduced consciousness/movement, decreased need for sleep, vomiting and severely depressed mood? Can we integrate with Common Ailments to cause headaches as a symptom of withdrawal?
Possible names: paracitalopram, estalopram, elopram, lopram? Yeah, maybe lopram.

Added the antidepressant lorpram (name subject to change).

requires lorpram production research project
synthesized at a drug lab from 1 neutroamine per 2 doses
mood +12 for 24 hours
50/50 chance of addiction per dose taken
withdrawal is mood -35 for 3 days (plus pain, vomiting, increased chance of social fights)

So basically don't start on it unless you have a steady supply of it.


Bugfix: throw error if utxos is null or undefined

I think I caught the "utxo is undefined" bug.
https://github.com/dashevo/dashcore-node/pull/62
But just to be sure let's throw an error if utxo is undefined.

@Cofresi please resolve conflicts.

@Cofresi please resolve conflicts.

done @shumkov 👍


LRP minor improvements

No need to run the full CI pipeline since the PR only updates log messages, and documentation.


Revert "Ensure CI pipelines run against vscode stable (#11720)"

This reverts commit 903b39e2e7422092aecf82d6a4d515a701046008.

Codecov Report

Merging #11723 (6dafd9f) into main (903b39e) will decrease coverage by 15%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            main   #11723      +/-   ##
=========================================
- Coverage     59%      44%     -16%
=========================================
  Files        483      300     -183
  Lines      34447    23431   -11016
  Branches    5601     4061    -1540
=========================================
- Hits       20668    10315   -10353
- Misses     11827    11971     +144
+ Partials    1952     1145     -807

Impacted Files
Coverage Δ

src/platform/common/utils/stopWatch.ts
16% <0%> (-84%)
:arrow_down:

src/kernels/execution/cellExecutionCreator.ts
5% <0%> (-76%)
:arrow_down:

...ls/raw/session/rawNotebookSupportedService.node.ts
17% <0%> (-74%)
:arrow_down:

src/kernels/execution/cellExecutionQueue.ts
6% <0%> (-74%)
:arrow_down:

src/kernels/common/kernelSocketWrapper.ts
7% <0%> (-74%)
:arrow_down:

src/kernels/common/kernelConnectionWrapper.ts
4% <0%> (-70%)
:arrow_down:

.../notebooks/controllers/vscodeNotebookController.ts
9% <0%> (-69%)
:arrow_down:

src/platform/webviews/webviewPanelProvider.ts
12% <0%> (-68%)
:arrow_down:

...ls/jupyter/session/jupyterSessionManagerFactory.ts
33% <0%> (-67%)
:arrow_down:

src/platform/common/application/workspace.base.ts
11% <0%> (-66%)
:arrow_down:

... and 335 more


Migration from gogs turns SSH keys unusable

If gitea doesn't update those keys somehow then somewhere should be documented that keys will stop working unless you update them.
The keys are stored in ~/.ssh/authorized_keys with references to gogs command so they must be updated to reference gitea command.
Maybe it is even better to duplicate them instead of removing the keys just in case someone wants to have a gogs and a gitea installation.
Well, don't know which is the best approach but makes no sense to have the keys listed in user profile cause they are stored in database (a reference I suppose and not the keys themselves) but are not correctly configured in the system.

We will document how to migrate from gogs to gitea before we release v0.7. github.com/go-gitea/docs#1


Create finer wasDerivedFrom distinctions

This enhancement to the ontology allows us to distinguish the different kinds of derivation.

@davearcher, if this can be approved by tomorrow, then we can include it in v4.1.


Terminal Popup overflows

Instructions: Replace the template text and remove irrelevant text (including this line)
Describe the bug
A clear and concise description of what the bug is.
(Issues related to the runtime files should be reported to their maintainer, check the file header.)
To Reproduce
fu!  OnTermExit(job, message)
    close
    " TODO: add some code to confirm that current window is a popup.
    " TODO: prevent close other window by accident.
endfunction

:call Flt_term_win('bash',0.9,0.6,'Todo')<CR>

Expected behavior
A clear and concise description of what you expected to happen.
Output is redrawn when it reaches the edge(It is only redrawn when you go into normal mode and back)
Screenshots
If applicable, copy/paste the text or add screenshots to help explain your problem.

Environment (please complete the following information):

Vim version [e.g. 8.1.1234] (Or paste the result of vim --version.)
OS: [e.g. Ubuntu 18.04, Windows 10 1809, macOS 10.14]
Terminal: [e.g. GNOME Terminal, mintty, iTerm2, tmux, GNU screen] (Use GUI if you use the GUI.)

Windows 10
Vim 8.2

You need to set the maxheight entry to the same value as minheight.
Why are you generating the borders manually in the first place? Vim does it for you automatically, see the border, borderhighlight and borderchars entries under :h popup-usage. There is no need to open two popup windows as above.

I think the following does exactly what you want:
function! Pop_term_win(cmd, width, height, border_highlight) abort
    let width = float2nr(&columns * a:width)
    let height = float2nr(&lines * a:height)
    let bufnr = term_start(a:cmd, {'hidden': 1, 'term_finish': 'close'})

    let winid = popup_create(bufnr, {
            \ 'minwidth': width,
            \ 'maxwidth': width,
            \ 'minheight': height,
            \ 'maxheight': height,
            \ 'border': [],
            \ 'borderchars': ['─', '│', '─', '│', '┌', '┐', '┘', '└'],
            \ 'borderhighlight': [a:border_highlight],
            \ 'padding': [0,1,0,1],
            \ 'highlight': a:border_highlight
            \ })

    " Optionally set the 'Normal' color for the terminal buffer
    " call setwinvar(winid, '&wincolor', 'Special')

    return winid
endfunction

call Pop_term_win('bash', 0.9, 0.6, 'Todo')

Thanks

Why are you generating the borders manually in the first place? Vim does it for you automatically, see the border, borderhighlight and borderchars entries under :h popup-usage. There is no need to open two popup windows as above.

I could be wrong, but I think it's based on some code I wrote in the past.
In the context of fzf, I think there was no need for maxheight.
As for the borders, I didn't want to re-invent a second mechanism; I based the code on an existing implementation for Nvim, which doesn't support borders around floating windows.
I did try to submit a PR to make fzf use Vim's native borders but it was not merged, because the maintainer was reluctant to handle 2 different mechanisms for no visible gain to the user.

FWIW, I wrote a small recipe to create a popup terminal in Vim, and a gist to make it togglable.

Why are you generating the borders manually in the first place? Vim does it for you automatically, see the border, borderhighlight and borderchars entries under :h popup-usage. There is no need to open two popup windows as above.

I could be wrong, but I think it's based on some code I wrote in the past.
In the context of fzf, I think there was no need for maxheight.
As for the borders, I didn't want to add a second mechanism; I based the code on an existing implementation for Nvim, which doesn't support borders around floating windows.
I did try to submit a PR to make fzf use Vim's native borders but it was not merged, because the maintainer was reluctant to handle 2 different mechanisms for no visible gain to the user.
FWIW, I wrote a small recipe to create a popup terminal in Vim, and a gist to make it togglable.

Totally my fault, I know that's not what the code was initially meant to be used for


Dropbox Google Drive option

Does it show dropbox google drive option by default or something has to be done for that as in ios only browse option is coming for me and in android it opens file manager.
I did not find the options as shown in pic of repo.

Document providers on both platforms are based on the apps the user has installed.


Fail subgraph if contract calls are not supported by the Ethereum node

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
When graph-node runs against a non-archive Ethereum node, contract calls are not supported. However, when making contract calls from a subgraph on such a node, it simply retries forever. All it logs are debug messages that are easy to miss if you don't have debug logs enabled with e.g. --debug or GRAPH_LOG=debug.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.

Connect graph-node to a non-archive Ethereum node.
Deploy a subgraph that makes contract calls.
Observe that it retries making the first contract call forever with no chance of recovering.

What is the expected behavior?
When a subgraph on such a node makes a contract call and it fails with an error like
Nov 14 14:18:17.159 DEBG Trying again after eth_call RPC call failed (attempt #1)
with result Err(Web3Error(Rpc(Error { code: ServerError(-32000), message: "This request
is not supported because your node is running with state pruning. Run with --pruning=archive.",
data: None }))), block_hash: 0xb4d3413066344d8f76368c0bd98c4a52184b9366e401215ffd0bf2e73e369b59,
block_number: 118842, subgraph_id: Qmaf6AFkpPWkULhyza2GT68NotJVL671JK2zjiLcR2Uczd,
component: SubgraphInstanceManager

we should immediately fail the subgraph, because there is no chance for it to ever get past this. Not all developers have the graph-node started with e.g. --debug and might think the call just hangs forever (which it kinda does).

We currently get that response routinely when connected to Alchemy, it goes away after retrying a few times. This looks like an issue on their end but for as long as that's happening we don't want to fail the subgraph on this case.

This is expected to only happen with non-archive nodes (e.g. full nodes), which is why it was reported originally. But indeed, we're also seeing this against Alchemy where it is unexpected.

So, Alchemy has clarified that this is not particular to their service, because Parity will return that if you make an eth_call to a non-existing block. This can happen if the node that gets the eth_call is behind or if there was a reorg. This ties in with https://github.com/graphprotocol/graph-node/issues/400, because if we did eth_call by hash and a reorg happened, this would retry forever! Possibly if this retries too much, we should start over the block processing.

Now that we have the capabilities feature, this is solved.


uBlock blocks feedback on tutorials

https://dotnet.microsoft.com/learn/aspnet/hello-world-tutorial/install#
I am using Windows 10 21H1 (OS Build 19043.1348) with Edge Version 96.0.1054.43 (Official build) (64-bit)
If you click "I ran into an issue" button, and click "Provide Feedback" button, the page just scrolls up and down and nothing happens.
The button works normally only when the "Feedback" button on the right side next to the vertical scroll bar appears.
I think the correct behavior should be that "Provide Feedback" button being disabled until loading or the new component with loading placeholder shows up to load Feedback module.

Related to #4

We have since implemented some logic to display a link for feedback if there's an ad blocker.


mkdocs issue

CdaFactory --
Normally, the following statement would include pydocs API information
::: src.oncoexporter.cda.CdaFactory

but this is throwing an error so it is commented out for now.

seems to be working now


Add more information about the system to dart:io

This issue was originally filed by @kaendfinger

NodeJS has the 'os' module with a lot of methods for getting information about the system. Equivalent methods should be implemented in Dart.
Currently Platform provides a getter for the number of processors. This should be extended to provide more information about the processors.
See https://nodejs.org/api/os.html#os_os_cpus for an example of the extent of the information that NodeJS provides.
Endianness: https://nodejs.org/api/os.html#os_os_endianness
Free Memory: https://nodejs.org/api/os.html#os_os_freemem
Load Average: https://nodejs.org/api/os.html#os_os_loadavg
Uptime: https://nodejs.org/api/os.html#os_os_uptime
Total Memory: https://nodejs.org/api/os.html#os_os_totalmem
Justification:
Some people rely on this type of information in their NodeJS code. What if these people want to port their code to Dart? They will be turned away when they see we don't provide this information.
This is general system information, it has a lot of useful uses such as a system uptime tracker or a memory usage recorder.
Why it should not be in a package:
These things require native code to access this on some platforms, or require external processes to get this information.

Barring specific customers and requests I can't see this happening anytime soon.


Fix range exceeds data length crash

If the data returned on to the TextToSpeech synthesize function is
empty, a crash would occur because there was no length checking.
I have signed the Contributor License Agreement.

 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.Ryan Mitchell seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

All outstanding items have been addressed. What else do I need to do to get this approved and merged?

@tigersoft thanks for catching and fixing this error. This look good to me, I'll go ahead and merge it in.

@tigersoft - Thanks for bringing up this issue! While this PR fixes your specific problem of receiving data less than 12 bytes, this check you have:
 guard data.count >= riffHeaderSize else {
             return false
         }

may not actually cover other cases. For example, what happens when a WAV file received is only 13 bytes long? In this cases, and if this case can exist, it might be best to reject the file as a valid WAV file. Plus, as @vherrin was suggesting, the size of the data should technically be checked when riffChunkID and riffFormat are assigned. Taking a look at the dataToUTF8String function, it looks like when converting the buffer to a string, the range contains unchecked bounds, so we could instead add a check there that data is non-empty, and/or it contains the string marked at offset of length.
I've opened up another issue referencing this one, to continue further investigating alternative solutions. I'm interested in hearing what you think.
Lastly, could you explain what error messages you were receiving, and the context of what the problem you ran into was?

@schen22 - You have very good concerns. Thank you for creating issue #556.

Lastly, could you explain what error messages you were receiving, and the context of what the problem you ran into was?

A crash occurred possibly when there was a network connectivity issue. Here is a redacted version of the crash:
2016-12-23 11:54:01.140 XYZAPP[49972:296938] *** Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[_NSZeroData subdataWithRange:]: range {0, 4} exceeds data length 0'
*** First throw call stack:
(
  0   CoreFoundation                      0x0721b746 __exceptionPreprocess + 182
  1   libobjc.A.dylib                     0x06ea4a97 objc_exception_throw + 44
  2   CoreFoundation                      0x0721b66d +[NSException raise:format:] + 141
  3   Foundation                          0x03b3ac22 -[NSData(NSData) subdataWithRange:] + 284
  4   libswiftFoundation.dylib            0x07e807ca _TFV10Foundation4Data7subdatafT2inGVs5RangeSi__S0_ + 74
  5   TextToSpeechV1                      0x03a8b19d _TTSf4g_d___TZFC14TextToSpeechV112TextToSpeechP33_15CA90B9E8F4B57B04C464D277FD8DAF9isWAVFilefT4dataV10Foundation4Data_Sb + 45
  6   TextToSpeechV1                      0x03a8b7de _TTSf4gs_g_g_n_g___TFFC14TextToSpeechV112TextToSpeech10synthesizeFTSS5voiceGSqSS_15customizationIDGSqSS_11audioFormatOS_11AudioFormat7failureGSqFPs5Error_T__7successFV10Foundation4DataT__T_U_FGV7RestKit12RestResponseS4__T_ + 286
  7   TextToSpeechV1                      0x03a81dac _TFFC14TextToSpeechV112TextToSpeech10synthesizeFTSS5voiceGSqSS_15customizationIDGSqSS_11audioFormatOS_11AudioFormat7failureGSqFPs5Error_T__7successFV10Foundation4DataT__T_U_FGV7RestKit12RestResponseS4__T_ + 92
  8   TextToSpeechV1                      0x03a8925d _TPA__TFFC14TextToSpeechV112TextToSpeech10synthesizeFTSS5voiceGSqSS_15customizationIDGSqSS_11audioFormatOS_11AudioFormat7failureGSqFPs5Error_T__7successFV10Foundation4DataT__T_U_FGV7RestKit12RestResponseS4__T_ + 125
  9   RestKit                             0x032cb2a4 _TTSf4g_g_d_gs_n___TFFV7RestKit11RestRequest12responseDataFT17completionHandlerFGVS_12RestResponseV10Foundation4Data_T__T_U_FTGSqS3__GSqCSo15HTTPURLResponse_GSqPs5Error___T_ + 180
  10  RestKit                             0x032c44dc _TFFV7RestKit11RestRequest12responseDataFT17completionHandlerFGVS_12RestResponseV10Foundation4Data_T__T_U_FTGSqS3__GSqCSo15HTTPURLResponse_GSqPs5Error___T_ + 60
  11  RestKit                             0x032c818d _TPA__TFFV7RestKit11RestRequest12responseDataFT17completionHandlerFGVS_12RestResponseV10Foundation4Data_T__T_U_FTGSqS3__GSqCSo15HTTPURLResponse_GSqPs5Error___T_ + 109
  12  RestKit                             0x032c427e _TFFV7RestKit11RestRequest8responseFT17completionHandlerFTGSqV10Foundation4Data_GSqCSo15HTTPURLResponse_GSqPs5Error___T__T_U_FTGSqS2__GSqCSo11URLResponse_GSqPS4____T_ + 110
  13  RestKit                             0x032c81e3 _TPA__TFFV7RestKit11RestRequest8responseFT17completionHandlerFTGSqV10Foundation4Data_GSqCSo15HTTPURLResponse_GSqPs5Error___T__T_U_FTGSqS2__GSqCSo11URLResponse_GSqPS4____T_ + 67
  14  RestKit                             0x032c6d15 _TTRXFo_oGSqV10Foundation4Data_oGSqCSo11URLResponse_oGSqPs5Error____XFdCb_dGSqCSo6NSData_dGSqS1__dGSqCSo7NSError____merged + 101
  15  RestKit                             0x032c42f8 _TTRXFo_oGSqV10Foundation4Data_oGSqCSo11URLResponse_oGSqPs5Error____XFdCb_dGSqCSo6NSData_dGSqS1__dGSqCSo7NSError___ + 56
  16  CFNetwork                           0x02b70a63 __49-[__NSCFLocalSessionTask _task_onqueue_didFinish]_block_invoke + 343


remove sections for manually creating the test database and building with stack-in-docker

https://github.com/PostgREST/postgrest/pull/1665 removes test/destroy_test_db and test/Dockerfile.test, so this removes the sections in the docs that reference it.
Of course this has to be replaced with documentation for our awesome nix dev environment (hi @monacoremo :D), but first I wanted to make sure that we're not pointing to tools that are not available anymore.

@monacoremo I guess you would be suited for the task to write some stuff about the nix dev env. Should we just point to the nix/Readme.md in the meantime? Your call.

Should we just point to the nix/Readme.md in the meantime?

Yes, let's link to the nix/readme for now!
Not sure yet what the focus in the docs should be vs the readme. Happy to discuss if you have thoughts on that!

Should we just point to the nix/Readme.md in the meantime?

Yes, let's link to the nix/readme for now!
Not sure yet what the focus in the docs should be vs the readme. Happy to discuss if you have thoughts on that!

I thought more about that. I feel like we shouldn't replicate the nix docs in both places. I also think they need to be in the main repo, for a random contributor to come along and just be able to work with it, before hitting the docs for it. The readme is really good.
So... maybe we should just replace the whole development.rst section in the docs with a short contributing.rst that links to the nix-readme and mentions a couple of other resources (docs repo, commit guidelines and stuff). Then in the main repo we could add a small section Contributing to the main readme as well - probably the same content. And all the nix stuff stays right where it is put.
WDYT?

Fully agreed - it should be in the main repo and we should not duplicate, so what you propose is the best way to go!

First step: I had a look at what was left over in development.rst after removing the obsolete parts here. While we don't need the two sections "Build from Source" and "Running the Test Suite" for our development env anymore, I thought it would still be nice to keep them - for those folks:

When a pre-built binary does not exist for your system you can build the project from source.

I took those two sections and integrated them into the "Installation" chapter.
Next step will be to write that "Contributing" part.

Got a bit side-tracked here, because I noticed some links were outdated. So I added postgrest-docs-linkcheck which uses sphinx linkcheck to do exactly that. Did a major cleanup of all external links with that. The tool could be added to CI, but currently it's hitting rate limits at github.com, because we have so many of those links. Support for rate limiting is already included upstream, but unreleased - so maybe we can add that later to CI.

So, this should be good to merge right now. Could you have a look before, @steve-chavez?
Here's a summary what I did:

Removed development.rst, which was heavily outdated after PostgREST/postgrest#1665.
Moved a part of the development chapter ("Building from Source") to install.rst as another way of installing PostgREST.
Left all the "how to run the tests" out - this is replaced by the docs on our nix dev environment.
Added a "Contributing" part to index.rst that just links to our Contributing Guidelines on GitHub. From there we will need to link to the nix Readme (right now, we're linking back to the removed docs page).
Bonus: Added postgrest-docs-linkcheck, ran it and fixed all kinds of outdated links. Reformatted a bit here and there.

Steve, I'm mostly looking for feedback on 2-4. No need to go through all the changed links ;)
The goal was to reduce duplication of content going forward, that's why the docs now only link to the main repo for everything related to the development/contributing.

2 and 4 LGTM! I've left some comments above, feel free to merge after you're done with those.
:+1: for linkcheck


Add COMPLETE pragma for FallingEdge pattern synonym

Tiny follow-up to #3752, specifically 396949c5344a263c04377fa4d9b60b46098b7101

bors merge

bors retry

bors retry


If model has incrementing id "save()" won't invalidate

Describe the bug
If you have table migrations for your models that use $table->increments('id'); rather than just a $table->integer('id')->unsigned(); the library fails to invalidate the cache on save().
This is because a code path is followed which falls back to the base query. The code path is: Illuminate\Database\Eloquent\Model > save() > performInsert() > insertAndSetId() > Illuminate\Database\Eloquent\Builder::insertGetId() > __call > toBase().
The problem happens when the Eloquent model returns true for getIncrementing() in the performInsert() function of the Eloquent Model. Because CachedBuilder from the library doesn't override insertAndSetId() to perform a checkCooldownAndFlushAfterPersisting() the cache is never flushed and this results in dirty reads from the cache.
Eloquent Query
For some reason I could only replicate this in testing. My test is pretty much the below structure. The library was actually behaving when I'm using my development environment but when I run my tests in the testing environment any model which used incremental ids would fail if it did multiple writes and then tested the returned results.
        $this->assertEquals(0, SomeModel::get()->count());
        $this->somePostWhichCreates($data);
        $this->assertEquals(1, SomeModel::get()->count());

Environment

PHP: 7.2
Laravel: v5.8.15
Model Caching: 0.4.14

Work around
At the moment the work around I have found to fix this issue is to use the "BaseModel" setup method and then add the following method to the base model:
    /**
     * Insert the given attributes and set the ID on the model.
     *
     * @param  \Illuminate\Database\Eloquent\Builder  $query
     * @param  array  $attributes
     * @return void
     */
    protected function insertAndSetId(Builder $query, $attributes)
    {
        $this->checkCooldownAndFlushAfterPersisting($this);
        parent::insertAndSetId($query, $attributes);
    }

Unfortunately I cannot leave a link to the code base as it's a private repo and my company wouldn't be happy.
For some reason I've only noticed this in testing. Though when I look at the code it seems to look like this may be a bug. I might need someone with better knowledge of the code to deny or confirm this.

@saernz Thanks for reporting this. I will look into this soon.

Likely related to #225

@saernz Are you overriding your model's created/deleted/saved static methods? That's what triggers cache invalidation. I have not been able to replicate your issue as described.

Ahh I have found out what I have done, and I must apologise if I've caused any wasted time.
I did not realise that by using Event::fake() in my tests that this also stops model created/deleted/saved events as well! Some of my tests contain:
    /**
     * Setup the test environment.
     *
     * @return void
     */
    protected function setUp(): void
    {
        parent::setUp();
        Event::fake();
    }

which is causing the problem. Very sorry for wasting your time. When I remove Event::fake() it seems to fix the problem. I was also confused when looking at the code as I didn't realise Eloquent also boots it's traits and I thought the bootCachable was dead code, which it obviously isn't.

No worries, @saernz, glad it's working for you now! :)


linux-x64.node load fail on Centos8

this error cased NATIVE be NULL, so the voicePlayer didnt work:

Error: node-loader:
Error: /lib64/libm.so.6: version `GLIBC_2.29' not found (required by /.vscode/extensions/kaiyi.qwerty-learner-0.3.7/dist/0499bc3cb3e2b70114d4091e8eb506dd.node)

may be its because the linux-x64.node so is compiled on Ubuntu which has newer version of glibc lib.
my Centos only support to GLIBC_2.28, maybe such nativemod should be based on lower version of glibc to support more linux release version? or maybe we can add a new .node binary so to satisfied support from users like me.
by the way one can check if the same err occur to you by doing this:
ldd [your home path mostly]/.vscode/extensions/kaiyi.qwerty-learner-0.3.7/dist/0499bc3cb3e2b70114d4091e8eb506dd.node
you will find error like i quote above if you have the same problem.

我也一样的情况，src/resource/voice/rodio/linux-x64.node 不能支持Deepin系统
我用的是Deepin 20.3，不能播放单词声音，检查后发现NATIVE=null，应该是编译好的linux的node文件不兼容Deepin
pudding@pudding-PC:~/Desktop$ cat /etc/os-version
[Version]
SystemName=Deepin
SystemName[zh_CN]=深度操作系统
ProductType=Desktop
ProductType[zh_CN]=桌面
EditionName=Community
EditionName[zh_CN]=社区版
MajorVersion=20
MinorVersion=20.7
OsBuild=11038.007
pudding@pudding-PC:~/Desktop$ cat /etc/os-release
PRETTY_NAME="Deepin 20.7"
NAME="Deepin"
VERSION_ID="20.7"
VERSION="20.7"
VERSION_CODENAME="apricot"
ID=Deepin
HOME_URL="https://www.deepin.org/"
BUG_REPORT_URL="https://bbs.deepin.org/"


Add Relevance Queries

By doing some post-processing of results it's possible to determine what inputs remain relevant to a query, when some of them are abducible, and others are known. It's worth investigating whether there is a good way to generalize that logic and allow the user to make relevance queries separately, so the post-processing happens on the server.

Closing as duplicate to #234, which has more details.


⚠ Samsung has degraded performance

In cb2aa95, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3311 ms

Resolved: Samsung performance has improved in b595e26.


Unauthorized error when running eas build from CI

Build/Submit details page URL
No response
Summary
I am attempting to run eas build from an AWS CodePipeline. I run the following:
export EXPO_TOKEN=$(aws secretsmanager get-secret-value --secret-id "$EXPO_TOKEN_SECRET_NAME" | jq '.SecretString')

eas build --platform ios --non-interactive

and then I see the error
CombinedError: [Network] Unauthorized
    at makeErrorResult (/usr/local/
lib/node_modules/eas-cli/node_modules/@urql/core/dist/52541c06.js:231:10)
    at /usr/local/lib/node_modules/eas-cli/node_modules/@urql/core/dist/52541c06.js:393:20
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  graphQLErrors: [],
  networkError: Error: Unauthorized
      at /usr/local/lib/node_modules/eas-cli/node_modules/@urql/core/dist/52541c06.js:393:83
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5),
  response: Response {
    size: 0,
    timeout: 0,
    [Symbol(Body internals)]: { body: [PassThrough], disturbed: true, error: null },
    [Symbol(Response internals)]: {
      url: 'https://api.expo.dev/graphql',
      status: 401,
      statusText: 'Unauthorized',
      headers: [Headers],
      counter: 0
    }
  }
}

    Error: GraphQL request failed.

Managed or bare?
Bare
Environment
expo-env-info 1.0.5 environment info:
System:
OS: Linux 4.14 Ubuntu 22.04.1 LTS 22.04.1 LTS (Jammy Jellyfish)
Shell: 5.1.16 - /bin/bash
Binaries:
Node: 19.6.0 - ~/.nvm/versions/node/v19.6.0/bin/node
Yarn: 1.22.19 - /usr/bin/yarn
npm: 9.4.0 - ~/.nvm/versions/node/v19.6.0/bin/npm
npmGlobalPackages:
expo-cli: 6.3.2
Expo Workflow: managed
Error output
CombinedError: [Network] Unauthorized
    at makeErrorResult (/usr/local/
lib/node_modules/eas-cli/node_modules/@urql/core/dist/52541c06.js:231:10)
    at /usr/local/lib/node_modules/eas-cli/node_modules/@urql/core/dist/52541c06.js:393:20
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  graphQLErrors: [],
  networkError: Error: Unauthorized
      at /usr/local/lib/node_modules/eas-cli/node_modules/@urql/core/dist/52541c06.js:393:83
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5),
  response: Response {
    size: 0,
    timeout: 0,
    [Symbol(Body internals)]: { body: [PassThrough], disturbed: true, error: null },
    [Symbol(Response internals)]: {
      url: 'https://api.expo.dev/graphql',
      status: 401,
      statusText: 'Unauthorized',
      headers: [Headers],
      counter: 0
    }
  }
}

    Error: GraphQL request failed.

Reproducible demo or steps to reproduce from a blank project
Can't provide a simple repro without spinning up a whole AWS account.

Figured it out -- the secrets manager command was adding quotes around the EXPO_TOKEN.


Add persistence for the Exchange.

At the moment it is all in memory, so the last update tokens are not persisted. This causes a full reload on launch. Have it persist.

This is now done, but not tested.


Temporary fix: check import_package before writing

This pull request tries to fix invalid import path: "" (related issues: #222 #220 )
I've noticed this behavior recently in my build pipline and tried caddyplug package alone: it fails for the first one or two times and works all the following try.
I've checked the source code, but found nothing could result this behavior, so I decide to make this temporary fix, hope this will work for everyone else.

Just FYI, this didn't work for me. Fails in the same way that master does.
    plugins="git,cors,realip,expires,cache,googlecloud" \
    github.com/abiosoft/caddy-docker.git#pull/223/head

...

can't load package: package caddy:
cors.go:2:10: invalid import path: ""
error at 'building caddy'
The command '/bin/sh -c VERSION=${version} PLUGINS=${plugins} ENABLE_TELEMETRY=${enable_telemetry} /bin/sh /usr/bin/builder.sh' returned a non-zero code: 1

@gabegundy you have to rebuild abiosoft/caddy:builder first, the builder.sh is bundled inside it.

Hello, can you make an advice please
is there a way to use your pull in DockerFile?
process wrapper
RUN go get -v github.com/abiosoft/parent
As i understand builder.sh is downloading here
but how can be downloaded your pull instead?

@jffin just replace builder.sh for the abiosoft/caddy:builder, if you are using the default dockerfile in this repo
#
# Builder
#
FROM abiosoft/caddy:builder

# add this line before you run `/bin/sh /usr/bin/builder.sh`
ADD https://raw.githubusercontent.com/jeffreystoke/caddy-docker/master/builder/builder.sh /usr/bin/builder.sh

# ...

#
# Final stage
#
FROM alpine:3.10

# ...

it works
thanks a lot

@tobiasmuehl that could happen since there is no limit to the retry, can you make sure your plugin is properly annotated with package here: https://github.com/caddyserver/caddy/blob/96579b97f687c95aeb85f09029d48122d84c668e/caddyhttp/httpserver/plugin.go#L626
Note: change the file version to match yours.

@jeffreystoke That explains it, the redis plugin is missing from the list. Using the build service from caddyserver.com for now but would prefer to self-build.

Redis is a clustering plugin so it probably belongs somewhere else, not in directives.
// directives is the list of all directives known to exist for the
// http server type, including non-standard (3rd-party) directives.

@tobiasmuehl I see that now. Thanks. I reworked the build to be muti-stage (seems cleaner anyway) and it works great. Thanks.

Unfortunately it doesn't seem to work anymore. I just tried it with the Dockerfile snippet and didn't have any luck.

@tobiasmuehl that could happen since there is no limit to the retry, can you make sure your plugin is properly annotated with package here: https://github.com/caddyserver/caddy/blob/96579b97f687c95aeb85f09029d48122d84c668e/caddyhttp/httpserver/plugin.go#L626
Note: change the file version to match yours.

Redis is a clustering plugin so it probably belongs somewhere else, not in directives.
// directives is the list of all directives known to exist for the
// http server type, including non-standard (3rd-party) directives.

Hi,
I am also running into the same issue. Can you please elaborate on the steps that you followed?

Anyone managed to solve?

@sparampalli I'm not self building, using the commercial installer provided. Here's my dockerfile:
FROM alpine

RUN apk update && apk add bash curl
RUN curl https://getcaddy.com | bash -s personal redis
ADD Caddyfile /etc/Caddyfile
ENTRYPOINT ["caddy"]
CMD ["--conf", "/etc/Caddyfile", "--agree=true"]

Someone managed to solve this problem.
I can't build.
I've tried using the script above but it freezes when loading the plugins.
It only works with the "cloudflare" plugin, but I need to include other plugins.
Does anyone find a solution to this error?

@oslah212 i found no way. I solved it bundling the pre-compiled Caddy executable in the docker container. Dirty fix but needed.

hi @Guglio95 . How can I solve it this way? My goal is to use it with some additional plugins.

When I want to use plugin namesilo, i build caddy in this way. Hope it would help.

FROM abiosoft/caddy:builder as builder1
ARG version="1.0.3"
ARG plugins="namesilo"
process wrapper
RUN go get -v github.com/abiosoft/parent
RUN VERSION=${version} PLUGINS=${plugins} ENABLE_TELEMETRY=false /bin/sh /usr/bin/builder.sh

@oslah212 you should get the Caddy v1 binary, or compile it by yourself if you need additional plugins.
Then you should create a docker container bundling in it the compiled executable.
This is a bad trick, but I needed to get it working; then I migrated to Caddy v2 :smile:
FROM alpine:3.10

ADD caddy /usr/bin/caddy
COPY Caddyfile /etc/Caddyfile

VOLUME /root/.caddy
VOLUME /var/log

ENTRYPOINT ["/bin/sh"]
CMD ["/usr/bin/caddy", "--conf", "/etc/Caddyfile", "--log", "stdout", "--agree=false"]


getEventDriver failed when running from a C++ thread

@s-ludwig help would be much appreciated:
I have a C++ application that dlopen's mylib.so and calls a function myfun from that mylib.so which uses vibe ; everything works fine when the function is called from the main C++ thread, but when its run from a thread it segfaults when calling:
client=new TCPClient(Endpoint(port, host));

further inspection shows it's assert'ing here:
nothrow @safe vibe.core.driver.EventDriver vibe.core.driver.getEventDriver(bool)
inside:
assert(ignore_unloaded || s_driver !is null, "No event driver loaded. Did the vibe.core.core module constructor run?");
How would i fix this? I tried without success to call from myfun(from the non-main thread):
import vibe.core.core;
setupDriver;

but vibe.core.core.setupDriver is not public;
Also tried calling runEventLoop from myfun but assert's with same error as it calls getEventDriver()

You need to initialize (and shut down) druntime manually:
import core.runtime;
Runtime.initialize();
// ...
Runtime.terminate();

That should properly run the module constructors and cause the driver to be created.

@s-ludwig I believe I already have, from the main thread, as follows:
main.cpp:
C++ starts
dlopen mydlib
call Runtime.initialize from mydlib

Thread T;
while(true){
 in some iterations:
      T=a new thread T spawned(by C++) // no control over this part, pat of a bigger application

  on thread T:
    call myfun from mydlib
}

mydlib.d:
myfun:
// either will assert [1]
auto client =new TCPClient(Endpoint(host,port));
auto api = new RestInterfaceClient!Foo("http://<IP_ADDRESS>:someport");

[1]
I also tried wrapping this inside:
thread_attach; (rt_moduleTlsCtor);...;thread_detachThis; (rt_moduleTlsDtor);
this causes SEGFAULT after a few calls

depending on what i try stacktrace may occur:
    frame #0: 0x00007fffb4bb2639 libmydlib.so`_D9invariant12_d_invariantFC6ObjectZv + 29
    frame #1: 0x00007fffb4ba5052 libmydlib.so`_D4vibe5utils6memory21AutoFreeListAllocator5allocMFNbmZAv + 90 at memory.d:332
    frame #2: 0x00007fffb4b0cfdf libmydlib.so`lev_alloc + 47 at libevent2.d:1280
    frame #3: 0x00007fffb40f1999 libevent-2.1.so.6`event_mm_calloc_ + 56
    frame #4: 0x00007fffb40ef46c libevent-2.1.so.6`bufferevent_socket_new + 30
    frame #5: 0x00007fffb4b07606 libmydlib.so`_D4vibe4core7drivers9libevent215Libevent2Driver10connectTCPMFS4vibe4core3net14NetworkAddressS4vibe4core3net14NetworkAddressZC4vibe4core7drivers13libevent2_tcp22Libevent2TCPConnection + 546 at libevent2.d:293
    frame #6: 0x00007fffb4b26ad8 libmydlib.so`_D4vibe4core3net10connectTCPFAyatAyatZC4vibe4core3net13TCPConnection + 436 at driver.d:31
    frame #7: 0x00007fffb4a12356 libmydlib.so`_D10msgpackrpc9transport3tcp91__T15ClientTransportTC10msgpackrpc6client40__T6ClientS2610msgpackrpc9transport3tcpZ6ClientZ15ClientTransport6__ctorMFC10msgpackrpc6client40__T6ClientS2610msgpackrpc9transport3tcpZ6ClientS10msgpackrpc6common8EndpointZC10msgpackrpc9transport3tcp91__T15ClientTransportTC10msgpackrpc6client40__T6ClientS2610msgpackrpc9transport3tcpZ6ClientZ15ClientTransport + 98 at tcp.d:158
    frame #8: 0x00007fffb4a11449 libmydlib.so`_D10msgpackrpc6client40__T6ClientS2610msgpackrpc9transport3tcpZ6Client6__ctorMFS10msgpackrpc6common8EndpointZC10msgpackrpc6client40__T6ClientS2610msgpackrpc9transport3tcpZ6Client + 53 at client.d:33
    frame #9: 0x00007fffb49cb639 libmydlib.so`_D2...myfun...sgpackrpc6client40__T6ClientS2610msgpackrpc9transport3tcpZ6Client + 501 at myfun.d:213

Sorry, thread_attachThis was actually the function that I meant to reference. Initializing/shutting down repeatedly on the same thread is currently dangerous, because that could render certain global state invalid. I can add a test case for this in the future, but currently it is hard for me to say where there might be such hidden state that survives the shutdown.
If there is any possibility to get notified before the C++ threads terminate, then that would surely be the best approach (i.e. initialize in a singleton fashion and shutdown before the thread gets notified). If the threads are long-lived, just leaking resources for the OS to reclaim might be a viable option, too.

@s-ludwig  Thanks; will get back to you;
Just so I don't explore an exponential number of options:
can you confirm that rt_moduleTlsCtor (after thread_attachThis) is needed?

Yes, that's needed, too.

Looking at the conversation, the issue seems fixed. This is the kind of issue which would fit well in a discussion as mentioned here. @timotheecour : Feel free to reopen if that is not the case.


rsocket client is not completed

This library is not completed. Is someone working on it because it's working only on TCP and only request response that are working. I will like to know if you will do any update on this library soon to include the request stream and request channel using websocket

see - #5

WIP websocket implementation in the websocket branch.
example usage in the examples folder.
feedback appreciated.


Service logs filled with 'ps' related error

I'm getting lots of instances of this error in the logs for my postgresql service based on postgres:9.6.9-alpine:
data_postgresql.1.ft2gof5jci25@IS-57436    | ps: bad -o argument 'command', supported arguments: user,group,comm,args,pid,ppid,pgid,etime,nice,rgroup,ruser,time,tty,vsz,stat,rss
The errors aren't generated by any other containers using Postgres services, as they occur when the other containers aren't running. Googling around, I believe this may be fixed by using an enhanced version of 'ps' as follows because Alpine has a more limited version:
apk --no-cache add procps
Could this be included in future builds?
This might be related to an excess CPU consumption issue I've been having as the CPU is thrashed as it continually tries to execute the 'ps' command.
https://dba.stackexchange.com/questions/207589/excessive-postgres-docker-cpu-consumption
I haven't finished fully investigating the issue yet, so will update with my conclusions when done.

As noted on stack overflow, you've been hacked: https://dba.stackexchange.com/a/207636


PgQuery::Node: Add inner and inner= helpers to modify inner object

Because of how oneof messages work in protobuf, its a bit tedious to make modifications to a Node object, in particular when changing the type of the inner object held within the Node in a generic code path (i.e. with a dynamic inner type), which required first knowing the name of the new inner object type, and then using public_send to set the new value.
To help, introduce the new "inner" and "inner=" methods for PgQuery::Node, that get/set the inner object directly, avoiding the use of public_send.
This will be of particular help for anyone utilizing the walk! API to make modifications to the query tree. To illustrate, an example is added to the treewalker spec showing how to utilize this.

Additionally, this makes utilizing the walk! treewalker easier for the common use case of working with the node directly (without using the parent or location information).

Per request (in a different channel), I've run a quick benchmark showing that this doesn't impact the time the treewalker takes negatively, which it does not:

Before (6 longest SQL statements in fingerprint test file):
Benchmarking time for parse+walk...
                       user     system      total        real
4a6db94fbada8341  52.937340   0.678991  53.616331 ( 53.624522)
b7dbf54ce62af0ca   0.241984   0.002727   0.244711 (  0.244712)
18e71bc17baea13b   0.006921   0.000028   0.006949 (  0.006943)
3a5494404465d0f9   0.014623   0.000034   0.014657 (  0.014658)
1cca3f304295181c   0.009612   0.000013   0.009625 (  0.009623)
f936eab75b8c1b90   0.000537   0.000001   0.000538 (  0.000538)

Benchmarking memory for parse+walk...
Calculating -------------------------------------
    4a6db94fbada8341    13.806B memsize (     0.000  retained)
                        49.838M objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    b7dbf54ce62af0ca    76.937M memsize (     0.000  retained)
                        88.861k objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    18e71bc17baea13b     2.268M memsize (     0.000  retained)
                        10.540k objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    3a5494404465d0f9     4.793M memsize (     0.000  retained)
                        17.635k objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    1cca3f304295181c     3.159M memsize (     0.000  retained)
                        12.182k objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    f936eab75b8c1b90   158.198k memsize (     0.000  retained)
                       926.000  objects (     0.000  retained)
                        50.000  strings (     0.000  retained)

After (6 longest SQL statements in fingerprint test file):
Benchmarking time for parse+walk...
                       user     system      total        real
4a6db94fbada8341  50.811444   0.857045  51.668489 ( 51.743262)
b7dbf54ce62af0ca   0.241765   0.002221   0.243986 (  0.244304)
18e71bc17baea13b   0.006827   0.000019   0.006846 (  0.006843)
3a5494404465d0f9   0.014364   0.000009   0.014373 (  0.014371)
1cca3f304295181c   0.009489   0.000013   0.009502 (  0.009501)
f936eab75b8c1b90   0.000513   0.000004   0.000517 (  0.000515)

Benchmarking memory for parse+walk...
    4a6db94fbada8341    13.806B memsize (     0.000  retained)
                        49.838M objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    b7dbf54ce62af0ca    76.937M memsize (     0.000  retained)
                        88.862k objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    18e71bc17baea13b     2.268M memsize (     0.000  retained)
                        10.541k objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    3a5494404465d0f9     4.793M memsize (     0.000  retained)
                        17.636k objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    1cca3f304295181c     3.159M memsize (     0.000  retained)
                        12.183k objects (     0.000  retained)
                        50.000  strings (     0.000  retained)
    f936eab75b8c1b90   158.278k memsize (     0.000  retained)
                       927.000  objects (     0.000  retained)
                        50.000  strings (     0.000  retained)

Benchmark script:
# spec/benchmark.rb
#
# Needs "s.add_development_dependency 'benchmark-memory', '~> 0'" in the gemspec

require 'pg_query'
require 'json'
require 'benchmark'
require 'benchmark/memory'

def fingerprint_defs
  # Sort input SQL by size (largest first), since those would be most interesting usually
  @fingerprint_defs ||= JSON.parse(File.read(File.join(__dir__, 'files/fingerprint.json'))).sort_by { |d| -d['input'].size }
end

puts 'Benchmarking time for parse+walk...'

Benchmark.bmbm do |x|
  fingerprint_defs.each do |testdef|
    x.report(testdef['expectedHash']) { PgQuery.parse(testdef['input']).walk! { |_, _, _, _| } }
  end
end

puts 'Benchmarking memory for parse+walk...'

Benchmark.memory do |x|
  fingerprint_defs.each do |testdef|
    x.report(testdef['expectedHash']) { PgQuery.parse(testdef['input']).walk! { |_, _, _, _| } }
  end
end


refactor: graphql-codegen with apollo without swr

Using graphql-codegen and apollo/client to implement loading and saving data on the Team page.
Code generation is performed using two commands:
1. Schema generation
npm run generate:graphql-schema

Connects to Hasura and pulls data from it by writing it to the graphql/graphql-schema.json file. Must be called after changing the database structure or changing the metadata. Requires a connection to Hasura.
2. Type generation
npm run generate:graphql-types

Uses the prepared schema. Generates generic types and writes them to graphql/graphql.ts file. Then searches for .graphql files and creates .generated.ts files next to them. Must be called after creating or modifying .graphql files. Does not require a connection to Hasura.

@michaelfurutin Lgtm! But, seems like generate:graphql-schema command requires HASURA_GRAPHQL_ADMIN_SECRET & NEXT_PUBLIC_GRAPHQL_API_URL vars to be in the .env file. Is it possible to make it read .env.local too?

now we can use .env.local


4.9.0 package.json version seems to be 4.9.0-rc5

Versions
4.9.0
Describe the bug
If you look at the package.json you will see the below which is incorrect.  I verfied with adaptive-expressions and botbuilder-lg.
"version": "4.9.0-rc5"
Expected behavior
package.json should match published version.
[bug]

I pull down the package:
npm i adaptive-expressions

When I look at the package.json, I see:
"_from": "adaptive-expressions",
  "_id"<EMAIL_ADDRESS>  "_inBundle": false,

and further down in the file:
  "typings": "./lib/index.d.ts",
  "version": "4.9.0"

So I think this is good to go.


[FLINK-16299] Release containers recovered from previous attempt in w…

…hich TaskExecutor is not started
What is the purpose of the change
Release containers recovered from previous attempt in which TaskExecutor is not started.
Brief change log

Call nodeManagerClient#getContainerStatusAsync when getting containers from previous attpempt. When received container with state NEW, release it.
Add TestingNMClientAsync.class in test infra, mock the behavior of NMClientAsync.

Verifying this change
This change added tests and can be verified as follows:

YarnResourceManagerTest#testStartWithContainerFromPreviousAttempt

Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no

cc @tillrohrmann @xintongsong

Thanks for the review @xintongsong . I've addressed your comment.

@xintongsong Thanks for the review. PR updated.

FYI: Add empty increaseContainerResourceAsync method in TestingNMClientAsync in case using Hadoop 2.8.0+.

Thanks @xintongsong @TaoYang526 . I've updated the PR.
The major change is:

Move the release logic from onContainerStatusReceived to onGetContainerStatusError.
Change the getContainerStatusAsync in TestingNMClientAsync, from function to consumer.
Change the testHandleContainersFromPreviousAttempt test case.

Closed since it is not a problem. For details, see FLINK-16299.


Links in Cards

Hello,
Using your example in https://www.neoteroi.dev/mkdocs-plugins/cards/#options, I'm not able to get the cards to generate the link.
Here is what I have in the markdown file
- title: Zeus
  content: |
    Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor
    incididunt ut labore et dolore magna aliqua.
  image: ./img/icons/001-zeus.png
  url: https://en.wikipedia.org/wiki/Zeus

i know i do not have the image, so i expect that to 404, but when i hover over the card, it displays the path to the image. When clicked, it tries to open the image in a preview as opposed to the wikipedia link. I even tried using an image i do have and tried with the image property removed altogether.
When I check dev options, i do see the 'a href' element wrapping the nt-card class, but it doesn't manifest on the page.

Hi,
I cannot reproduce the issue you describe. I took this screenshot using the latest version of Material for MkDocs and documentation site for my plugins:

Doesn´t matter if a picture of a card can be found or not, the card renders properly with an anchor tag to the configured URL.

Thanks for your time! I think I see the problem, just not sure what could be causing it.
The link shows its nested three times in a single nt-card class.

I commented out all of my mkdocs extenstions as I thought something was conflicting. But even with just 'neoteroi.cards' active, it still produces the same triplicate output.

That's weird 👀
Can you please share your markdown and versions of dependencies so I can try to replicate the issue?

Sure! I'm on mkdocs-material-9.5.3 and I'm using neoteroi-mkdocs 1.0.4.
The markdown is
::cards::

- title: ChatBot
  content: Lorem ipsum dolor sit amet.
  image: ./icons/032-chatbot.png
  url: https://en.wikipedia.org/wiki/Zeus

::/cards::

If you were referring to python dependencies, I'm not sure how to get that. I'm using a custom mkdocs-material docker image thats setup to sync with the latest official image, and pulls the plugins via PIP everytime there's a new release  --
pip install --no-cache-dir neoteroi-mkdocs
more plugins\
That build pipeline is hidden from me. I apologize for being annoying.

No worries @carried-away, you are not annoying. And thanks, I meant the versions of dependencies you are using, the two you gave are perfect. I tried replicating your issue with the same version of libraries (when I tried previously I only had a small difference in material for mkdocs because I was using the previous version), and I cannot reproduce the issue. The only difference is that I am not using the custom mkdocs-material docker image. I cannot really think of something in the source code of my library that might cause this issue.

Thank you for checking. I really appreciate your time especially so close to the holidays.
I was able to get the link property to work by using the 'image-bg' option, but it jacks up the icon image.

So I'll see if I can follow the logic and see what in my environment is messing it up. Thank you again and Merry Christmas!

Thank You, @carried-away
I wish the same to you, Merry Christmas!


Pass WorkspaceConfiguration from convenience constructor

Motivation:
A convenience constructor of Workspace received some arguments. However, they are ignored.
It may be a simple mistake.
Modifications:
I passed some arguments to the default constructor from arguments.
Result:
[After your change, what will change.]

@swift-ci smoke test


cleanup

this should be ready to merge after the tests pass..

Pull Request Test Coverage Report for Build 592677343

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/utils.jl
0
2
0.0%

Totals

Change from base Build 592262638:
0.0%

Covered Lines:
0

Relevant Lines:
163

💛  - Coveralls

the new commits addressed  your comments, also fixing the conflict warnings. i also fixed the naming convention for the models. they are now sorted as they appear in the arch-comp article.

There are problems in the docs for the Airplane model at the bottom.


singularity: 4.1.2 -> 4.1.3

Description of changes
Changelog: https://github.com/sylabs/singularity/releases/tag/v4.1.3
cc @ShamrockLee
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

singularity.tests builds fine on both x86_64-linux and aarch64-linux.

Result of nixpkgs-review pr 310186 run on aarch64-linux 1

4 packages built:

singularity
singularity-overriden-nixos
udocker
udocker.dist

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

4 packages built:

singularity
singularity-overriden-nixos
udocker
udocker.dist

Result of nixpkgs-review pr 310186 run on aarch64-linux 1

4 packages built:

singularity
singularity-overriden-nixos
udocker
udocker.dist

The source hash matches the one fetched using nix-prefetch-github. (This manual check is important against a feasible security hazard of Hydra poisoning via PR https://github.com/NixOS/nix/issues/969#issuecomment-2096302974)
$ nix-prefetch-github sylabs singularity --rev v4.1.3
{
    "owner": "sylabs",
    "repo": "singularity",
    "rev": "53e19ae1f4dce4777de981defa334ed6419cb334",
    "hash": "sha256-pR8zyMr23wcbDCXAysVEgGUDHkrfhLoVF3fjMLgZFYs="
}

The name of the goModules FOD contains the version of the main package, and should be automatically re-fetched upon version changes. This guarantees that the relevance of vendorHash as long as the packages build successfully and the source hash is relevant.
Both singularity and singularity.tests.image-hello-cowsay builds on x86_64-linux NixOS, showing that the updated SingularityCE builds images alright.
The image running functionality would still be affected by #295809, so I didn't test it.
@GaetanLepage Could you help run nixpkgs-review for this PR? My current NixOS laptop doesn't have enough RAM to run it, and it would take a lot of time on my phone (NixOnDroid).


Tucker/fix deletes and messages

What
Fixed delete commands (they were referencing incorrect urls). Also improved the messages for successful deletes.
Note
PR #73 must be merged first, and then this PR should be retargeted to main.
Peer Review
Please follow the upstream omnivector documentation concerning
peer-review guidelines.

I guess it's necessary to change the unit tests as well, doesn't it?

It's the linter that's failing right now. The next PR fixes that. There are very few unit tests in jobbergate-cli, and the coverage is really sparse.
We plan to provide complete coverage with unit tests in the coming re-write of Jobbegate CLI.


popup() with SSO authentication scheme - pending checks and more details

Hello everyone,
I'm currently working on a script to download contents from the enterprise mail automatically since we need to generate reports from the downloaded contents.
One of our mailbox uses SSO scheme for authorisation purpose and it runs as stated in the following:

I use r.click() in the main tab opened by r.init() after entering my credentials to log in
It pop-ups a new tab with its url ended with /sso and then I redirect to the mailbox page within the tab automatically
Then I tired to use r.popup('url to the mailbox page') to tell tagUI reseting the tab to the newly opened tab
It ended up with error message saying:

[RPA][ERROR] - cannot find popup tab containing URL string 'url to the mailbox page (ended up with .jsp)'

Alternative trial: I've tried r.vision() to send command visually but because of the computer was reinstalled by IT dept. following specific requirements, complex SikuliX command would shut the engine down leaving the whole process wait endlessly...so negativ.
Is there a solution to this? Any help would be appreciated.
Best,
Lamarche

Hi @Lamarche-Lam it sounds like you want to automate on the popup tab with URL ending with /sso. Can you share what is the full URL of the popup tab and main tab? From your description, the step should be below. Also, check this example 2 here.
r.popup('sso')

Hi @Lamarche-Lam it sounds like you want to automate on the popup tab with URL ending with /sso. Can you share what is the full URL of the popup tab and main tab? From your description, the step should be below. Also, check this example 2 here.
r.popup('sso')

Hi @kensoh ,
Not really...
Sorry that I found a typo in the process 2 of my question at

"It pop-ups a new tab with its url ended with /sso and then I redirect to the mailbox page within the tab automatically"

What I actually want to say was

"It pop-ups a new tab with its url ended with /sso and then this newly opened tab redirects to the mailbox page within the tab automatically". The mailbox page has the url with ".jsp" ending.

Can you share what is the full URL of the popup tab and main tab?

Sure, the main tab was:

http://123.456.12.34:3455/hub/

and the main tab opened one popup tab but with two urls in order:

http://123.456.12.34:3455/hub/account/sso
I checked the JavaServer Pages of http://123.456.12.34:3455/hub/ and I found there was an action when I clicked the button, which was something like:

var form = "form if='form' action='/hub/account/sso' method='post' target='blank'>";

http://123.456.12.44:3499/company/page/main.jsp

the urs were made up but the behaviour of them were well reflected.
Thank you.

I see... Then usage would be like the following -
If you mean after loading http://123.456.12.34:3455/hub/account/sso the webpage automatically redirects to http://123.456.12.44:3499/company/page/main.jsp, then use r.popup('main.jsp').
If you require some manual 2FA action on http://123.456.12.34:3455/hub/account/sso before going to main.jsp, you may want to do a r.wait() for some time to allow you to manually click.
Lastly, if you are planning to automate the clicking to go from sso to main.jsp page, then you can use r.popup('sso') to automate on that page.

It meets the description of the first scenario but RPA returns error saying it cannot find the popup tab with the url ended with 'main.jsp'.
I used exactly the same command r.popup('main.jsp') as you did but it returned negative.

Oh I see... Can you check this, print(r.tagui_location()). In that folder, there is a tagui or .tagui folder depending on your OS. Then inside there is src\tagui_chrome.log. Can you attach the log file here? It will show the messages from Chrome.

Oh I see... Can you check this, print(r.tagui_location()). In that folder, there is a tagui or .tagui folder depending on your OS. Then inside there is src\tagui_chrome.log. Can you attach the log file here? It will show the messages from Chrome.

Ohoh, I forgot to mention this. It seemed that the getTarget method not function properly.
When r.popup() works, it would look like:

[tagui] INPUT  -
[19] {"id":19,"method":"Target.getTargets","params":{}}
[tagui] OUTPUT -
[19] {"id":19,"result":{"targetInfos":[{"targetId":"0CABB68EA2F1CA7A896DCB11A938C702","type":"page","title":"","url":"https://passport.bilibili.com/login?from_spm_id=333.1007.top_bar.login","attached":false,"openerId":"82BCB9E2D0924FF8ED29176E3C66FED0","canAccessOpener":true,"openerFrameId":"82BCB9E2D0924FF8ED29176E3C66FED0","browserContextId":"F5E9D045F593BF7E5B2981F7321B4507"},{"targetId":"82BCB9E2D0924FF8ED29176E3C66FED0","type":"page","title":"\u54d4\u54e9\u54d4\u54e9 (\u309c-\u309c)\u3064\u30ed \u5e72\u676f~-bilibili","url":"https://www.bilibili.com/","attached":true,"canAccessOpener":false,"browserContextId":"F5E9D045F593BF7E5B2981F7321B4507"}]}}

It showed the url of popup tab first and then the url of main tab. However, the urls of popup and main tabs was arranged reversely, it looked like:

[tagui] INPUT  -
[19] {"id":19,"method":"Target.getTargets","params":{}}
[tagui] OUTPUT -
[19] {"id":19,"result":{"targetInfos":[{"targetId":"0CABB68EA2F1CA7A896DCB11A938C702","type":"page","title":"the unicode of the tab title","url":"https://www.bilibili.com/"},{"targetId":"82BCB9E2D0924FF8ED29176E3C66FED0","type":"page","title":"the unicode of the tab title","url":"https://passport.bilibili.com/login?from_spm_id=333.1007.top_bar.login"}]}}

In addition, strings like "attached", "openderId", "canAccessOpener", "openerFrameId", "browserContextId" were missing out.
the urls were made up but the behaviour of them were well reflected.

Not sure if above are the real URLs or dummy URLs because no mention of sso or jsp string in URLs.
If the popup URL is https://passport.bilibili.com/login?from_spm_id=333.1007.top_bar.login from above, you can try connecting using r.popup('top_bar.login') to see if it works, since this string does not exist in the main URL.

Adding on to above suggestion, are you able to tell what difference this popup compared to other popups?
Is it look different or the way it appear is special? I don't understand why Chrome responds differently in your logs.

Not sure if above are the real URLs or dummy URLs because no mention of sso or jsp string in URLs.
If the popup URL is https://passport.bilibili.com/login?from_spm_id=333.1007.top_bar.login from above, you can try connecting using r.popup('top_bar.login') to see if it works, since this string does not exist in the main URL.

Hi @kensoh , These urls from the chrome log were made up, they were not the real urls I encounted with. I just used them as an example to illustrate the problems.
The whole process could be demostrated in the following annomation:

The main tab was the "join github" page.
When i click on Terms of Service, it opened a new tab with url as "github.com/site.terms" in a new tab first and then the url changed into "docs.github.com/...." in the same tab.
but in my case, the main tab was http://123.456.12.34:3455/hub/
and it opened a new tab at http://123.456.12.34:3455/hub/account/sso
and the new tab redirected to http://123.456.12.44:3499/company/page/main.jsp automatically.

Adding on to above suggestion, are you able to tell what difference this popup compared to other popups?
Is it look different or the way it appear is special? I don't understand why Chrome responds differently in your logs.

Adding on to above suggestion, are you able to tell what difference this popup compared to other popups?
Is it look different or the way it appear is special? I don't understand why Chrome responds differently in your logs.

The only difference, I think, the urls of the main tab and the popups in my case was that thery were all internal urls (I worked for a financial entity so everthing i did was in an isolated enviroment) and they were all constructed as some string like ip:port/xxx/xxx/.

I see.. Can you share the full tagui_chrome.log for r.popup('main.jsp')?

I see.. Can you share the full tagui_chrome.log for r.popup('main.jsp')?
Without the full logs to see what is Chrome's response, and without access to your system, I can't think of ways to troubleshoot the problem. Alternatively you can use visual automation and keyboard automation to automate on the new tab, see here - https://github.com/tebelorg/RPA-Python#visual-automationsee_no_evil

Sorry… I cannot provide you the log file at the moment since due to company’s policy. I would try something else based on you suggestion. Thank you for helping me.

Oh yes, I suspect that is the case. Sure, I hope visual and keyboard automation works for you.
It is very easy to use, just that native web automation is more powerful (especially for reading data).

Do see this section on element identifiers to understand more on visual automation -
https://github.com/tebelorg/RPA-Python#element-identifiers

Oh yes, I suspect that is the case. Sure, I hope visual and keyboard automation works for you.
It is very easy to use, just that native web automation is more powerful (especially for reading data).

Definitely will try :)


Support product ID 884{0|2}

As discussed in #60.

Also please update README with new supported models


ensure custom properties propagation

Didn't see where the custom properties collection will be kept to propagate by geo2topo into final topojson. This change should ensure such custom properties propagation.

This isn’t a generic script, so there’s no concept of “custom properties” here. If you want to generate a different TopoJSON file with custom properties, you are welcome to fork this repo and modify the script.


Add default Accept header and port number to Host

Kitura-net adds a default Accept header and appends port number to host name if
non-default ports(443/80) are used. In order to maintain parity default Accept
header is added and port name is appended if non default port is used.

 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.Harish Kumar S seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

The changes have been tested with:

Kitura-WebSocket-NIO
Kitura-Sample
Kitura-CouchDB
SwiftMetrics


Advance to "Use latest db_connection with transitive allow" commit.

Coverage remained the same at 85.783% when pulling 6593f22f390f810fdd52bb941ff6f344c2676c27 on transitive-allow into 442a14f51432806da760625bfc9f47f0adb24ee5 on master.


[DO NOT MERGE] SEO edits whats-business-voice

SEO edits for the description, lengthening and adding keywords. Added a section per the recommendations to get keywords related to small business into the article. Content is essentially marketing info to emphasize the small business potential.
Made minor edits for MS standards in body.
@DaniHalfin

I'm reviewing. Thanks.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@DaniHalfin Hi! Is this PR going to be closed? Thanks!

@dstrome will you review the Business Voice blurb at the bottom of this PR please? Dani issued this PR with SEO updates. But I don't feel qualified to TR the BV section. Thanks!


Localization for problems doesn't seem to work.

Hi, thank you for this awesome project. I'm currently trying to add localization support to our validations. But it doesn't seem to work.
I've created a custom ProblemHandler to handle problems and return errors to our users. In the problem handler I'm retrieving messages for each problem in a similar way to this:
@Override
public void handleProblems(List<Problem> problems) {
    problems.forEach(problem -> LOGGER.error(problem.getMessage(locale)));
}

locale being a field in my ProblemHandler implementation.
I accept the Locale through Accept-Language header in my requests.
It seems that whichever locale I'm trying to send in I always get back english. Am I doing something wrong? I'm using 0.15.0 version of justify.
Thank you for your response.

@adreviko  Thank you for trying this library.
I have added a small test in the commit 4f6e322 as below, and it seems to work fine.
    @Test
    public void getMessage_shouldReturnDifferentMessageByLocale() {
        Problem problem = createProblem(SCHEMA, INSTANCE);
        String message1 = problem.getMessage(Locale.ENGLISH);
        String message2 = problem.getMessage(Locale.JAPANESE);

        assertThat(message1).isNotEmpty();
        assertThat(message2).isNotEmpty();
        assertThat(message1).isNotEqualTo(message2);
    }

The current implementation includes messages only for English and Japanese. If you need other languages, I must add translated messages as resource files of this library.
Thank you.

Hi @leadpony, yes you are correct I've tried with japanese as specified language and it works correctly. Thank you for your help. At the moment we still don't know which languages will be supported in our product.


Question: How to convert ps to pdf?

Hello,
I just started using this docker image and have a question. I am not able to figure out how to convert the ps to pdf. Online search tells me to use ps2pdf but it looks like this program is not installed in the image. If I try to install anything using apt-get I get errors.
For example, this is what happens when I try to install ghostscript:
root@eea737608448:~# apt-get install ghostscript
Reading package lists... Done
Building dependency tree
Reading state information...   They have been ignored, or old ones used instead.

Btw using pdflatex is not an option for me due to the work I am doing. I need to convert a PS to PDF.

It seems the issue is that this image is not able to resolve any domain names. You can try calling wget with any domain and it will fail.

The problem here was I used --net none when I created my Docker container.


Memory Game with Vanilla JavaScript

Hello @jainSiddhantt,
It will be really great if you put your project in a folder named your USERNAME, and kindly please work on a sub-branch instead of master branch. (refer to README.md).
So, the correct directory will look like: MemoryGame/JainSiddhantt/(...your Files)
Thank You.


netclient: No support for DNS due to requiring modification of /etc/hosts

Describe the bug
When using netclient with a netmaker server configured with DNS, the domains are unable to populate the /etc/hosts file, rendering the feature unable to be used. This happens because the hosts file is symlinked into the nix store, and as such cannot be modified.
Steps To Reproduce
Steps to reproduce the behavior:

Connect to a netmaker server with DNS enabled using netclient.
Enable services.netclient and join the server.
Check the journalctl logs for netclient to find issues modifying /etc/hosts due to being readonly.
Try pinging one of the defined domains in netmaker, and see that the ping fails. However, when pinging the ip, it will succeed.

Expected behavior
Pinging the domain should succeed
Additional context
The service will likely need to be able to modify the hosts file dynamically in order to work correctly. I'm not sure what the solution would be in this case though.
Notify maintainers
@wexder

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.00 MiB download, 0.00 MiB unpacked):
  /nix/store/wfsbr2vjijkr5nax29y83vizz36ym34a-nix-info
copying path '/nix/store/wfsbr2vjijkr5nax29y83vizz36ym34a-nix-info' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.54-rt15, NixOS, 23.11 (Tapir), 23.11pre-git`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@purepani oh I've not been using the dns so I didn't noticed this. I can reproduce it easily, however I'm not sure how I can fix it. Will give it a shot in next couple of weeks

@purepani I don't think there's anything we can do to allow netclient to write to /etc/hosts

I am pretty sure this can be closed now that Netmaker abandoned the idea of using hosts file: https://github.com/gravitl/netclient/pull/655

I'm just not sure what to configure instead.

Current documentation suggests pointing your client machine to the CoreDNS' IP address. I have confirmed this with Netmaker developers on Discord.

@nazarewk thanks for finding this.

FYI to solved DNS completely server-side without any client-side config (wired up CoreDNS as NS record)
https://github.com/NixOS/nixpkgs/pull/283768#issuecomment-1912175838

FYI: solved DNS completely server-side without any client-side config (wired up CoreDNS as NS record) #283768 (comment)

Hello there, would you be able to expand how or what did you do specifically?
I did 1, 2 and 3 from this: https://docs.netmaker.io/architecture.html#coredns
but where did you wire the NS record? an actual domain pointing (which one?) to the netmaker server public address or how?