[SPARK-28443][SQL] Spark sql add exception when create field type NullType

What changes were proposed in this pull request?
More detail see 25085.
This PR is to discuss details that add exception when create field type NullType
Now, it's ok to create table like:
val schema = new StructType().add("c", NullType)
spark.catalog.createTable(
          tableName = "t",
          source = "json",
          schema = schema,
          options = Map.empty[String, String])

But it's not the spark idea
How was this patch tested?
UT

cc @cloud-fan

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #107883 has started for PR 25198 at commit 791b3cb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/12999/
Test PASSed.

Test build #107883 has finished for PR 25198 at commit 791b3cb.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/107883/
Test FAILed.

Looks like a good idea to check this when creating a table with NullType

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13162/
Test PASSed.

Test build #108054 has started for PR 25198 at commit fdaee89.

Test build #108054 has finished for PR 25198 at commit fdaee89.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108054/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13240/
Test PASSed.

Test build #108135 has started for PR 25198 at commit 9791914.

Test build #108135 has finished for PR 25198 at commit 9791914.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108135/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13839/
Test PASSed.

Test build #108758 has started for PR 25198 at commit 0ba1242.

Test build #108758 has finished for PR 25198 at commit 0ba1242.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108758/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13871/
Test PASSed.

Test build #108791 has started for PR 25198 at commit 517a1e6.

Test build #108791 has finished for PR 25198 at commit 517a1e6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108791/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13891/
Test PASSed.

Test build #108811 has started for PR 25198 at commit 138ac5e.

Test build #108811 has finished for PR 25198 at commit 138ac5e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108811/
Test FAILed.

Test build #108833 has started for PR 25198 at commit 108b7f9.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13910/
Test PASSed.

Test build #108833 has finished for PR 25198 at commit 108b7f9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108833/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13922/
Test PASSed.

Test build #108848 has started for PR 25198 at commit fa22cf7.

Test build #108848 has finished for PR 25198 at commit fa22cf7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108848/
Test PASSed.

ping @cloud-fan

Can one of the admins verify this patch?

Test build #111595 has started for PR 25198 at commit 6a1b4f5.

Test build #111595 has finished for PR 25198 at commit 6a1b4f5.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111595/
Test FAILed.

Test build #111605 has started for PR 25198 at commit 358e2a2.

Test build #111605 has finished for PR 25198 at commit 358e2a2.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111605/
Test FAILed.


finish repeat shape groups etc

Priority is to get continuous movement working. All other “return” to func are not needed.


Fix error 404 on route change

When I try to access the routes watch, publish or explain I receive a 404 error. (I'm using Brave)
But it works when I update the routes to finish with .html. So here is a fix.

Thanks @wanjohiryan, I was working on fixing this myself. I didn't want to include the .html extension and even filed a parcel bug.
I just merged a "proper" fix to use a single page instead of multiple pages. It looks more professional without .html and query parameters.


Review animations

Go over animations

[x] Verify we can use them in our app
[x] for example any text needs to be dynamic so we can change it
[x] all images have to be svg

https://drive.google.com/drive/u/1/folders/1ymp8VAQpaP-dj6taXC7VgHMQcsZTHnWR

can u give access to google drive for<EMAIL_ADDRESS>

looks like all animations works fine

yes we can use them in our app
all images are png but I think it is ok. also a lot of animations don't have images at all. it is just css + js
sometimes texts are images so we can't change it dynamically

list of animations which have text as image:
set1:

Button_to_Counter (1)
Save button+Spinner
Share_Link_Recieve (1)
Share_Link_Send (2)

@AlexeyKosinski

all images are svg
all texts are as text. so we it can be dynamically change it

@AlexeyKosinski that is my new favorite gif


ref(tracing): Move TRANSACTION_SOURCE_* constants to Enum

Fixes GH-2696

Linter is failing because of:

Should I fix them in a different commit?

Linter is failing because of:

Should I fix them in a different commit?

yes, this is because of a new release of mypy. I will fix this.

This breaks a lot of tests, because the full enum ends up in the envelope that is sent to Sentry and not the .value of the enum. Can you please fix this?


[Local Azure Function] Task was canceled

Hi,
I just tried to deploy a very little Azure function, and invoke it in my AL Code.
My AL Code looks like this:
codeunit 50101 CallAzure
{
    trigger OnRun();
    begin
    end;

local procedure GetName(Name: Text): Text
var
    Client: HttpClient;
    Content: HttpContent;
    Response: HttpResponseMessage;
    Url: Text;
    Result: Text;
begin
    Content.WriteFrom(Name);
    Url := 'https://test.azurewebsites.net/api/TriggerName?name='+Name;
    //Url := 'http://localhost:7071/api/TriggerName?name='+Name;
    Client.Get(Url, Response);
    Response.Content.ReadAs(Result);
    exit(Result);
end;

//App runs on page 22
[EventSubscriber(ObjectType::Page, 22, 'OnOpenPageEvent', '', false, false)]
local procedure NameTheName()
var
    Julien: Text;
    Bill: Text;
begin
    Julien := 'Julien';
    Bill := 'Bill';

    Message('Premier prenom = %1 ', GetName(Julien));
    Message('Deuxieme prenom = %1', GetName(Bill));
end;

}

I tried to do that by two different ways: localy and by the url given by the azure portal. I have exactly the same code, the only difference resides in the url called.
When i call my azurewebsite.net/api url, it works fine.
When I call my localhost url (running in a separated terminal) it makes a very long time to show me:
Error: A task was canceled
(when i try this url in my browser, it works fine)
If I call my azurewebsite.net/api url, just after I have this error message:
The request for path /NAV/dev/apps?SchemaUpdateMode=synchronize failed with code 422. Reason: Cannot install extension TestAzureFunction because another service is currently modifying the state of extensions on tenant default.
And have to wait for about 5minutes to be able to deploy again my extension.
I think it'sprobably a port problem, and if it's really that, could someone explain how to configure that ..?
Tried on the latest Docker image (bcsandbox:12.0.21865.0-fr)

Hi @JulienDecottignies ! Are you still blocked by this?

Hi,
I didn't try to call my azure function locally since the day I posted the last message.
I call my azure function by the URL provided by azurewebsite.
Also, I found another way to do what I wanted to do with azure functions, by using controladdin. So I'm not able to try again to run azure function locally in my actual project for now.

Ok. Thanks for letting me know! In any case, it seems that the problem is related to the local Azure setup and not to the NAV runtime or compiler. I will close the issue.


Team 8 Review - VnV Report - Validation Beyond Testing

I think it may be worthwhile to add a section to your VnV report that talks about in what ways, other than programmatic testing (e.g. unit testing), can your code and models be validated. For example, your code can be validated through code walkthroughs and reviews with Dr.Mosser and/or the NLP researchers. I think adding this section will not help to strengthen your report, but also will aid in achieving a higher confidence in your systems as well.
Team 8 TA: @Lyuyang0130

This would be nice to have but it does not seem to fit nicely in any of the sections we have and would not warrant its own section. This idea is also touched upon in other documents.


pipenv installs in parent directory, instead of current

Issue description
electrum on  master
➜ pwd
/Users/will/Documents/src/electrum

➜ echo $PIPENV_VENV_IN_PROJECT
true

electrum on  master
➜ pipenv --python=/Users/will/.pyenv/versions/3.6.9/bin/python3
Creating a virtualenv for this project…
Pipfile: /Users/will/Documents/src/Pipfile
Using /Users/will/.pyenv/versions/3.6.9/bin/python3 (3.6.9) to create virtualenv…
⠼ Creating virtual environment...Already using interpreter /Users/will/.pyenv/versions/3.6.9/bin/python3
Using base prefix '/Users/will/.pyenv/versions/3.6.9'
New python executable in /Users/will/Documents/src/.venv/bin/python3
Also creating executable in /Users/will/Documents/src/.venv/bin/python
Installing setuptools, pip, wheel...
done.
Running virtualenv with interpreter /Users/will/.pyenv/versions/3.6.9/bin/python3

✔ Successfully created virtual environment!
Virtualenv location: /Users/will/Documents/src/.venv

electrum on  master took 6s 447ms

Expected result
pipenv to respect the $PIPENV_VENV_IN_PROJECT var

Please run $ pipenv --support, and paste the results here. Don't put backticks (`) around it! The output already contains Markdown formatting.
pipenv --support is throwing different errors...

Oh god. After about an hour of reinstalling pipenv, pyenv and various pythons I've rubber ducked myself!
Solution: if there is no Pipfile in the current directory, pipenv (I guess) will increase directory level until it finds one? This meant an errant (and blank) Pipfile found in my root src directory was causing the .venv to be created at that level, instead of inside my project.
Not sure if should be bug fix, or simply issue closed?

Correct, the Pipenv uses Pipfile to decide the venv location, if none is found, will look up to parent folders.

I have to ask, what is the logic behind this and why isn't this super clearly explained in the readme? I've spent countless hours trying to figure out why my envs don't get built correctly...


Typescript "Duplicate identifier" errors

After adding typescript to my preact-cli app, build has started reporting..
[at-loader] Checking finished with 4 errors
✖ ERROR [at-loader] ./node_modules/preact-router/index.d.ts:68:22
    TS2300: Duplicate identifier 'ComponentProps'.
✖ ERROR [at-loader] ./node_modules/preact-router/index.d.ts:69:22
    TS2300: Duplicate identifier 'PreactHTMLAttributes'.
✖ ERROR [at-loader] ./node_modules/preact/dist/preact.d.ts:15:7
    TS2300: Duplicate identifier 'ComponentProps'.
✖ ERROR [at-loader] ./node_modules/preact/dist/preact.d.ts:22:7
    TS2300: Duplicate identifier 'PreactHTMLAttributes'.
Build failed! null

after downgrading preact-router from 2.6.0 to 2.5.7 build no longer reports errors.

I used preact-cli-plugin-typescript to add typescript support
I found that https://github.com/mike-engel/preact-cli-typescript-sample worked without error until I updated the dependencies.
I am a verified typescript newb so won't be attempting a fix, but I'm guessing it's this change that is causing the issues.. https://github.com/developit/preact-router/commit/b9327a1c46a4d632b4cf01705b71b6634cbfc581#diff-b739f95ace6bbf2f0fd5e8cf1dab97d8

..happy to be told I'm missing something simple..

This one is covered by #274, don't worry it's not an issue on your part. The types for preact changed slightly which preact-router augments, it needs a small change that is not yet released.
I believe we're good to go for a 2.6.1 @developit

2.6.1 is now released, updating should fix this for you


Can't Compile, error because of if-statement in Marco

Hi,
this is my first time with NSIS, so I'm a total beginner.
I'm trying to make a Setup-File which checks for .Net 4.0 Full and if not found, installs it. But I get following error (I have removed the username for privacy reasons):

Processing config: c:\PROGRA~2\nsis\nsisconf.nsh
Processing script file: "c:/Users//DOCUME~1/cius4/src/setup/cius.nsi" (ACP)
Invalid command: ${If}
Error in macro CheckNetFramework on macroline 13
Error in script "c:/Users//DOCUME~1/cius4/src/setup/cius.nsi" on line 3 -- aborting creation process

If I add "!include LogicLib.nsh" at the beginning of my nsi-File, the error changes to:

Processing config: c:\PROGRA~2\nsis\nsisconf.nsh
Processing script file: "c:/Users//DOCUME~1/cius4/src/setup/cius.nsi" (ACP)
Error: command StrCmp not valid outside Section or Function
Error in macro _== on macroline 1
Error in macro _If on macroline 9
Error in macro CheckNetFramework on macroline 13
Error in script "c:/Users//DOCUME~1/cius4/src/setup/cius.nsi" on line 3 -- aborting creation process

So long,

Put the !insertmacro inside a Section or Function like it says in the second error message.  Like this.
Section "Microsoft .Net Framework v4.0 (required)"
   SectionIn RO
   !insertmacro CheckNetFramework 40Full
SectionEnd


chore(docs): update broken links in documentation styling

yamw

Hi @k3agan,
thank you for the help with the links update!

Hi @k3agan ,
you commit message is not following the linter format. May I ask you to update it so we can merge your PR?

Hi Fabiana, thanks for letting me know, I am updating it now
On Thu, Mar 16, 2023 at 4:29 PM Fabiana @.***> wrote:

Hi @k3agan https://github.com/k3agan ,
your commit message is not following the linter format. May I ask you to
update it so we can merge your PR?
—
Reply to this email directly, view it on GitHub
https://github.com/ydataai/ydata-profiling/pull/1281#issuecomment-1472893206,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOBKLODJDBWGQFESSEKH2LW4OO5JANCNFSM6AAAAAAVV2GECU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Just refreshed my branch, I can do it!
Sorry still new at this 😃

Hi @fabclmnt ,
I've just pushed the new commit that addresses the linting issues you pointed out. Apologies for any confusion my previous commit may have caused, and I appreciate your patience as I continue to learn the ropes. Thank you for your guidance in helping me get this pull request up to par.
I believe the pull request should now be in line with the defined standards. Please let me know if any additional changes are required.
Best regards,
Keagan

Codecov Report
Patch and project coverage have no change.

Comparison is base (05aaf87) 90.37% compared to head (b0820eb) 90.37%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1281   +/-   ##
========================================
  Coverage    90.37%   90.37%
========================================
  Files          185      185
  Lines         5725     5725
========================================
  Hits          5174     5174
  Misses         551      551

Flag
Coverage Δ

py3.8-ubuntu-latest-pandas
90.37% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

This is a learning moment for me 😆
From what I understood, there were two errors here.
subject and type both evaluated to empty although I'm not sure why
⧗   input: Cleanup styling **"docs: Updated 179 broken links"**
✖   subject may not be empty [subject-empty]
✖   type may not be empty [type-empty]

I don't understand how this evaluated to empty tbh 👀 so if you don't mind I will add [skip-ci] to @fabclmnt original commit message.

@k3agan any particular reason to change this link?

(don't worry too much about the commit semantic, we're squashing everything)

Hi @k3agan ,
can you please update your PR with the latest changes from develop branch?


How can I dynamically update the value of a text?

How can I dynamically update the value of a text?

@BarryZhan Can you please help me with this? I am trying to update the text dynamically but unable to do so.
Thanks

+1

var animationAPI = lottie_api.createAnimationApi(anim);
var textLayer = animationAPI.getKeyPath('textLayerName');
textLayer.setDocumentData({t: '1000%'})

This works


Staging model for Metal traces

What is it?
Continuation of https://github.com/opensource-observer/oso/issues/1193
Metal is one of the required chains for RF4

https://github.com/opensource-observer/oso/pull/1570


An error occurred while running git.exe rev-parse HEAD (exit code 128)

When I run
ionic package build android
and select release, it shows me this error::
An error occurred while running git.exe rev-parse HEAD (exit code 128)
Any help is needed thank you very much.

Thanks for the issue. If you are running into an issue with AppFlow, please contact AppFlow support. This issue tracker is dedicated to bugs and feature requests for Ionic Framework.


Error -bash: cd: ~wire-pod: No such file or directory

I'm trying to install wire-pod on windows 11

I figured it out. The instructions left out
wget -O - https://keriganc.com/setup-wire-pod.sh | bash


Use HTTPS for the apt repository

Fix for https://github.com/microsoft/vscode/issues/203832
How to test:

Build the .deb package.
dpkg -i the freshly-built package.
Look in /etc/apt/sources.list.d/vscode.list for an https:// URL.

It could also be worthwhile to upgrade existing installations similar to https://github.com/microsoft/vscode/blob/c9f0aa5b7303c9f60cae5889a1f04f0be56d333f/resources/linux/debian/postinst.template#L43-L45, but this requires building and testing and I don't have time for that right now. I suggest we leave that for a future follow-up.
This PR will at least address new installations.

This was proposed before in https://github.com/microsoft/vscode/pull/57992#issuecomment-419921020, you can see the reasoning for it being plain http there

@Tyriar That reasoning made sense 6 years ago, but if you look at https://packages.debian.org/unstable/apt-transport-https, apt-transport-https is no longer a separate package since the functionality was moved to apt directly in version 1.5.
The oldest supported version of Ubuntu (18.04) uses apt 1.6 and the oldest supported version of Debian (10) uses apt 1.8. Therefore, this will be a seamless upgrade for new installs.
I don't have the permissions to reopen this PR, so please consider this a request for re-review.

Nice, thanks for the follow up!
@rzhao271 FYI, could you take a quick look and approve this, want to make sure you're aware in case anything goes wrong.


feat: Switch to using managed Iterators

What is the feature you would like to implement?
Implement iterators as callbacks, so that the iterator is always managed by the DB class, so that we can always ensure that it is closed
Why is this feature important?
Our usage of iterators today leaves the possibility that an iterator may never be closed, which leads to a lot of downstream DB issues. Additionally, we need to code iterators very carefully, which is prone to bugs.
Will the protocol spec need to be updated??
No protocol impact
How should this feature be built? (optional)
db.forEachIterator(prefix?, ([key, value]) => {
 // do stuff with key value.
 // 1. if this method throws, we close the iterator
 // 2. If all [key, values] are processed, we close the iterator
 // 3. If the (optional) timeout is hit, we close the iterator
}, timeout?);

Notes

timeout default should be 1 min

@adityapk00 fixed several leaky iterators here which shipped in v1.4.1, but there are still more fixes needed.

https://github.com/farcasterxyz/hub-monorepo/pull/1156


Windows: BufferedNodeProcess doesnt work well

this function use "atom.exe" instead of "node"  to run node function and therefore fails for example to use linter
buffer-process.js
 module.exports = BufferedNodeProcess = (function(_super) {
   __extends(BufferedNodeProcess, _super);

function BufferedNodeProcess(_arg) {
  var args, command, exit, node, options, stderr, stdout;
  command = _arg.command, args = _arg.args, options = _arg.options, stdout = _arg.stdout, stderr = _arg.stderr, exit = _arg.exit;
  node = process.platform === 'darwin' ? path.resolve(process.resourcesPath, '..', 'Frameworks', 'Atom Helper.app', 'Contents', 'MacOS', 'Atom Helper') : process.execPath;

}

i think that this code should check if process.platfrom === "win32" and if so resolve with "node" and not with process.execPath .

@zcbenz,  I'm trying to execute a BufferedNodeProcess on windows, and it doesn't seem to work at all.  Even a simple script that doesn't use stdout doesn't work.
script.js
setInterval(function() {
  console.log('interval');
}, 1000)

Running this using atom.exe script.js with set ATOM_SHELL_INTERNAL_RUN_AS_NODE=1 will log correctly on the interval.  However, when I run script.js via BufferedNodeProcess it exits immediately with code 1, and no more information.
Additionally, running BufferedProcess and executing atom.exe will not execute.
My atom-typescript-tools package cannot work without this functionality as I need to boot the secondary process to not block the UI thread.

@zcbenz, http://strongloop.com/strongblog/whats-new-node-js-v0-12-multiple-context-execution/, out of curiosity is this going to be available in Atom?  If there is an interface you are working on, I would be glad to act as an alpha tester, sounds like it should be a much easier cross platform experience for what I'm doing.

Even a simple script that doesn't use stdout doesn't work.

I believe console.log is still stdout.

Error: EBADF, bad file descriptor

This also explains this : https://github.com/travs/atom-package-dependencies/issues/6 FYI all if you run something in shelljs you need to run it with silent : https://github.com/TypeStrong/atom-typescript/commit/52833e9884abb6df7d895777665b79632fb67ec4

@basarat, I also tried the same code without the console.log(), it fails there as well.  I also tried just running atom.exe -v which should output the node version.

@park9140 I think any form of non-error output even atom.exe -v would still go to process.stdout

@basarat, true however a long running script with continuos timeouts keeping it open would not have output of any sort.  Additionally, running the same tests directly against atom.exe in a standard console works fine.

however a long running script with continuos timeouts keeping it open would not have output of any sort.

Yes.
FYI a simple empty script also fails on spawn:
https://github.com/atom/atom/issues/5000#issuecomment-72409485

I definitely get this error on spawn but I don't get this manually running it so something weirder is happening

C:\Users\basaratsyed\AppData\Local\atom\app-0.177.0>set ATOM_SHELL_INTERNAL_RUN_AS_NODE
ATOM_SHELL_INTERNAL_RUN_AS_NODE=1

C:\Users\basaratsyed\AppData\Local\atom\app-0.177.0>cat test.js
console.log('hello world');

C:\Users\basaratsyed\AppData\Local\atom\app-0.177.0>atom test.js

C:\Users\basaratsyed\AppData\Local\atom\app-0.177.0>hello world

C:\Users\basaratsyed\AppData\Local\atom\app-0.177.0>

I had the same issue with this, doing something like the following fixes it.
remote.proc = cp.fork(path, {
	execPath: process.platform === 'win32' ?
		'c:\\PROGRAM FILES\\NODEJS\\node.exe' :
		'/usr/local/bin/node',
	silent: true
});

What is weird is that I get C:\Users\basaratsyed\AppData\Local\atom\app-0.177.0>hello world  instead of just hello world and it does not exit untill I press enter:
C:\Users\basaratsyed\AppData\Local\atom\app-0.182.0>atom test.js

C:\Users\basaratsyed\AppData\Local\atom\app-0.182.0>hello world  ///  I need to press enter for atom to exit

C:\Users\basaratsyed\AppData\Local\atom\app-0.182.0>node test.js
hello world

C:\Users\basaratsyed\AppData\Local\atom\app-0.182.0>

@csnover can you try similar commands for a mac. Does it hang after atom test.js?

Here is a bit more context. Using atom as a node REPL behaves weird compared to the standard node REPL. It seems confused about the prompt.

$ ATOM_SHELL_INTERNAL_RUN_AS_NODE=1 /Application/Atom.app/Contents/Frameworks/Atom\ Helper.app/Contents/MacOS/Atom\ Helper test.js
hello world

@zcbenz why is conversion done to utf8 only on windows: https://github.com/atom/atom-shell/blob/master/atom/app/atom_main.cc#L105-L137 (this is based on my rusty understanding of C)

I am experiencing the same issue as @basarat.  Executing js with atom requires a press enter to continue.  This is problematic because when I run using BufferedNodeProcess the process never ends.  If I look at process explorer I can see the launched atom process just sitting there doing nothing.

I suspect because it is waiting for me to hit enter.  On a related note, I am not receiving stderr or stdout callbacks, I suspect this is because the process never ends?

FWIW we did manage to spawn properly on both OSes : https://github.com/TypeStrong/atom-typescript/blob/master/lib/worker/lib/workerLib.ts#L170-L183
Important stuff is to not copy env and only set one property (ATOM_SHELL_...) and set cwd correctly:
this.child = spawn(process.execPath, [
       childJsPath
    ], { cwd: path.dirname(childJsPath),
         env: { ATOM_SHELL_INTERNAL_RUN_AS_NODE: '1' },
         stdio: ['ipc']
        });

I arrived at this fix purely but banging my head against it for long enough. Object.create(process.env) should work but doesn't.
That said be warned that there are some cases where Atom behaves weird when run like this. e.g. I got a totally reproduceable but nearly impossibly isolatable ReferenceError bug : https://github.com/Microsoft/TypeScript/pull/2522#issuecomment-87336165
YMMV.
Also if you spawn atom on your own package script ... you will run into this as well on windows: https://github.com/TypeStrong/atom-typescript/issues/195
Having these edge cases is why I didn't post this solution here early ... so use with care and :heart:

Setting the cwd manually fixed the not exiting automatically.  Setting the env to something minimal (ATOM_SHELL_INTERNAL_RUN_AS_NODE and one other I needed for my project) fixed it not logging anything (it is unclear whether before it was actually doing any work or not).

whether before it was actually doing any work or not

It wasn't. I tried :rose:

Seem to still be having problems spawning a node process in Windows with Atom 1.0.19...any new news on this? I tried setting a minimal env to no avail as suggested; process is still exiting with a 1 status immediately.

It looks like this has majorly broken with v1.7.0 on Windows, all of the linters that use BufferedNodeProcess are failing due to no contents being returned to stdout.

https://github.com/atom/atom/issues/11471 has been fixed making stdout work again. Does this mean this issue is now also addressed?

@damieng No, this issue has to do with how we launch the children processes in general, not the stdout.

@jonathandelgado or anyone else, can you give a summary of what is still currently broken? I'm pretty certain this has morphed since it was opened and I would like to get an understanding for what specifically is broken, most likely open a new Issue specifically for that, and then close this.

Looks like .kill() method still does not work properly if Atom wasn't ran from command line:

So platform is all set and the correct method of BufferedProcess is called, but somehow it fails to > correctly spawn wmic to get the child process ids to kill, despite clearly having it in the PATH.

However, when I start Atom from MSYS2 everything works - from cmd or the taskbar it exhibits the > behavior above.

Related issues: https://github.com/sadovnychyi/autocomplete-python/issues/202#issuecomment-210792599, https://github.com/sadovnychyi/autocomplete-python/issues/205#issuecomment-217151877
Can ask @mostaphaRoudsari or @maurerv for steps to reproduce.

Can we create a new issue for this specific remainder and close this one?  A lot of the other parts covered in this thread appear to have been addressed and I agree with @lee-dohm that we'd need clear steps as to what issues still remain.


Bump art-tools

Changes for doozer
141899a Merge pull request #700 from locriandev/refactor-gen-assembly
595d211 Merge pull request #698 from locriandev/optimize-get-nightlies
e806183 Merge pull request #711 from vfreex/cgit-atom-feed-better-exception-message
f2911b7 Merge pull request #704 from joepvd/fix-task-error-build-ok
Changes for elliott
d045c8d Merge pull request #479 from joepvd/robustize-tracker-summary-checker
0ab1a26 Merge pull request #475 from locriandev/fix-connection-reset-error

/lgtm


Local names

It would be nice to have the local names of each country available:
   "528":{
      ...
      "iso_3166_2":"NL",
      "iso_3166_3":"NLD",
      "name":"Netherlands",
      "local_name":"Nederland",
      "region-code":"150",
      ...
   },

It would also be nice to have some localization settings to have the country name in other languages. And also having another table for cities would be a very great addition


Browser pool integration

Integration of browser-pool package to SDK.

Hi @petrpatek , i'm a bit curious about this future feature, would it be possible to have different browser versions linked to a session? I'm looking for a solution like this. Would you mind to explain me it?
Congrats for this awesome work
Thanks in advance

Hi @drrodio, depends what you mean by session, but you'll have the ability to add various browsers and versions to the Pool. Unlike now, where PuppeteerPool uses a preinstalled version of puppeteer and that version only, you'll be able to create multiple browser configurations, such as:
Pseudo code, the API may still change a bit:
const puppeteer = require('puppeteer');
const playwright = require('playwright');
const BrowserPool = require('browser-pool');

const browserPool = new BrowserPool({
    browserPlugins: [
        new PuppeteerPlugin(puppeteer, { launchOptions: { executablePath: 'path-to-your-browser-version' }),
        new PuppeteerPlugin(puppeteer, { launchOptions: { executablePath: 'path-to-your-other-version' }),
        new PlaywrightPlugin(playwright.firefox),
        new PlaywrightPlugin(playwright.safari),
    ]
})

This will setup your pool to rotate 2 puppeteers with two different browsers and 2 playwrights with 2 different browsers.

@mnmkng it sounds really nice!
About playwright, when you think it will be available as a crawler inside apify?
Good work guys!

@drrodio It will be released in the v1.0.0 which we expect at the end of this year or early next year, depending on how much work we'll get done over Christmas :)

@mnmkng cool, thanks a lot!

Added in v1.0.0

Added in v1.0.0


one task does not run only on one core

When I train the RF model with one core, it takes 1 hour. If I set n_jobs=-1, it use all cores, and it takes 9 mins. There is very different during training.
However, when I predict, there is no difference, the predicting time is same for both trained model, and they both not only use one core (one CPU around 120%, other several CPU around 10%). In this case, the low-level parallel computing is not from RF model, but from other python functions in my opinion. Then it is a bit hard to figure out. Can we just keep it like this or we want to check every python function in my python script?

I realized that the problem is not the trained model (predict() function), but before predict() function during the array data prepare for predict() function. Because I printed the running time for array data prepare and predict() function, with two models, the running time of predict() function is different in two models, which means the model is different, but the time of array data prepare is same and takes up most of the time (e.g., when I predict 4 timesteps, predicting takes 0.1 s for every timestep, array data prepare takes 4.8 s, the total time is 19 s).


eslint.nodePath not resulting in module resolving

Hi,
My eslint is complaining about not finding the installed config to extend from:

[Error - 12:19:26 PM] Error: Failed to load config "airbnb-base" to extend from.
Referenced from: /Users/ME/Desktop/dev/MYCODEFOLDER/client/.eslintrc.js
at configMissingError (/usr/local/lib/node_modules/eslint/lib/cli-engine/config-array-factory.js:233:9)

This error shows up when opening the folder with my code in vscode, which has no node_modules, as everything is dockerized. There are .eslintrc files in that folder though.
I installed node@10 using brew, I am on macOS.
npm root -g shows: /usr/local/lib/node_modules, so in my vscode settings, I put:

"eslint.nodePath": "/usr/local/lib/node_modules/"

The contents of that folder are:
eslint
eslint-config-airbnb-base
eslint-plugin-import

This is my install config:
npm config get prefix
/usr/local

which node
/usr/local/opt/node@10/bin/node

which npm
/usr/local/opt/node@10/bin/npm

which eslint
/usr/local/bin/eslint

What else could be wrong here for eslint to not work in vscode?

@felix-voicehub does ESLint work for you in the terminal ?
The error indicates that airbnb-base plugin can't be loaded. Are you sure it is installed.

@dbaeumer eslint shows the commands, eslint FILE shows the same error:

ESLint couldn't find the config "airbnb-base" to extend from. Please check that the name of the config is correct.

Extending from airbnb-base actually means this module: https://www.npmjs.com/package/eslint-config-airbnb-base, as can be seen in its readme. This module is globally installed as shown above.

I might have found the issue:

Previously, ESLint loaded plugins relative to the location of the ESLint package itself. As a result, we suggested that users with global ESLint installations should also install plugins globally, and users with local ESLint installations should install plugins locally. However, due to a design bug, this strategy caused ESLint to randomly fail to load plugins and shareable configs under certain circumstances, particularly when using package management tools like lerna and Yarn Plug n’ Play.

As a rule of thumb: With ESLint v6, plugins should always be installed locally, even if ESLint was installed globally. More precisely, ESLint v6 resolves plugins relative to the end user’s project by default, and always resolves shareable configs and parsers relative to the location of the config file that imports them.

To address: If you use a global installation of ESLint (e.g. installed with npm install eslint --global) along with plugins, you should install those plugins locally in the projects where you run ESLint. If your config file extends shareable configs and/or parsers, you should ensure that those packages are installed as dependencies of the project containing the config file.

https://eslint.org/docs/user-guide/migrating-to-6.0.0#-plugins-and-shareable-configs-are-no-longer-affected-by-eslints-location
I was hoping to avoid this, but anyway, installing it in the working directory does solve this issue.

@felix-voicehub great to hear. I will close the issue then. The ESLint v6 migration does cause issue mainly caused by ESLint itself (not the extension). Fort details you might want to look at #696


Behat 3.0 and mink selenium extension with multiple host

Hi,
I have an application (type SAS) where my users switch from a domain to another.
I wanted to create a test suite on one domain and another one on a second however I cannot seem to make behat switch from one base url to the other.
Is there anyone who could help me please ?
This is my behat.yml. With this configuration, I would like to run all scenario with the tag club2 on club2.localhost and the other on club1.localhost. With this configuration, I can make the club2 suite work by runing the command : behat --suite=club2 --profile=club2. But I am looking for a way to run all suite in one command and automaticaly switching profile in the suite.
default:
    autoload:
        '': %paths.base%/features/bootstrap
    extensions:
        Behat\Symfony2Extension: ~
        SensioLabs\Behat\PageObjectExtension: ~
        Behat\MinkExtension:
            base_url: http://club1.localhost/
            selenium2: ~
    suites:
        default:
            contexts:
                - Context\FeatureContext
                - Context\SecurityContext
            filters:
                tags: ~@club2
        club2:
            contexts:
                - Context\FeatureContext
                - Context\SecurityContext
            filters:
                tags: @club2

club2:
    extensions:
        Behat\Symfony2Extension: ~
        SensioLabs\Behat\PageObjectExtension: ~
        Behat\MinkExtension:
            base_url: http://club2.localhost/
            selenium2: ~

Thanks in advance

I don't know if I went the right way, but I found a solution by creating a custom context which modify all context in the environment before each scenario.
<?php

namespace Context;

use Behat\Behat\Hook\Scope\BeforeScenarioScope;

/**
 * Description of MinkOverrideContext
 *
 * @author jobou
 */
class MinkOverrideContext extends Base\BaseMinkContext
{

    protected $baseUrl;

    public function __construct($base_url)
    {
        $this->baseUrl = $base_url;

    }

    /** @BeforeScenario */
    public function gatherContexts(BeforeScenarioScope $scope)
    {
        $environment = $scope->getEnvironment();

        foreach ($environment->getContexts() as $context) {
            if ($context instanceof \Behat\MinkExtension\Context\RawMinkContext) {
                $context->setMinkParameter('base_url', $this->baseUrl);
            }
        }
    }
}

And then loading it only for my club2 suite :
    suites:
        club2:
            contexts:
                - Context\FeatureContext
                - Context\SecurityContext
                - Context\MinkOverrideContext:
                    base_url: http://club2.clubor.localhost/
            filters:
                tags: @club2

It seems to work but I would like to have your feedback on good and bad practice of what I am doing. Thanks in advance

Closing as duplicate of https://github.com/Behat/MinkExtension/issues/155 which is in a much better place as Behat itself has nothing to do with Mink

Ok thank you for pointing that out. Sorry for the error.


⚠ Mobitopup has degraded performance

In 7d5e954, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 11705 ms

Resolved: Mobitopup performance has improved in ac736aa.


Type LoadBalancer and bare metal clusters

Baremetal clusters are real and the need for type LoadBalancer (esp for non-HTTP) services is widely felt. Document available solutions here so that potential users are not disappointed or sent on wild-goose chases.

@kbarnard10

/assign @kbarnard10

/lgtm

@chiradeep Thanks for the proposal. Unfortunately, this PR does not meet our standards.
This is because:

The proposed changes serve mainly as a vendor pitch
The proposed changes add no greater clarity to what @sftim rightly observes is an already long page

/close

I disagree. As @jaypipes notes, the page is full of vendor specific information. There's people looking to solve the specific issue around bare metal and service type loadbalancer. This PR proposes a couple of solutions. So what if the solution uses a (CNCF member's) product? It solves an end-user goal, which is the purpose of end-user docs.


IE7 problems

Hi,
Love Nivo, works in FF (max & win), Opera (max & win), Safari (max & win), Camino (mac), Chrome (max & win) and IE8, but....  (sigh)... none of the images display in IE7, nor does the navigation
I have tried adding "display:block" to "#slider a" css tag, and searched the forum and tried various solutions, but nothing seems to load. I am using the style package downloaded from the Nivo site.
I am currently looking for a bell tower.....

Dropped the last trailing comma and it did the trick in IE7. Thanks!

Had a tough time figuring this out until I realized I had unneeded options clogging it up such as for box animations so after I removed them and also the last comma in the row of options it worked for IE7 and IE6 (transitions wierd in IE6 but thats another story and one I wont read into ;)
$(window).load(function() {
$('#slider').nivoSlider ({
effect: 'fade', // Specify sets like: 'fold,fade,sliceDown'
pauseOnHover: 'true', // Stop animation while hovering
animSpeed: 500, // Slide transition speed
pauseTime: 5000, // How long each slide will show
startSlide: 0 // Set starting Slide (0 index)
});});


echoesOfLight-jacksonHickey

Echoes of Light
My artwork depicts light rays reflecting off walls and obstacles in a 2D space. Writing the code took around nine hours, during which I faced a few challenges. Since this was my first experience with software art and I rarely use JavaScript, I am proud of this project. I find the result really cool and I like experimenting with the settings to explore different variations!

Nice! This may be a bit busy to draw but I think you've earned your Blot.
Congrats, you can order it here.


Fixed Issue with k8s.io/docs/getting-started-guides/fedora/fedora_manual_config/

Problem:
In the node configuration the line
KUBELET_API_SERVER="--api-servers=http://fed-master:8080"
is removed in kublet v1.6, see here
Solution
Add this line instead:
KUBELET_ARGS="--cgroup-driver=systemd --kubeconfig=/etc/kubernetes/master-kubeconfig.yaml --require-kubeconfig" and contents of  master-kubeconfig.yaml.
Page updated
https://kubernetes.io/docs/getting-started-guides/fedora/fedora_manual_config/

/assign @zacharysarah

/approve

/assign @bradtopol
@bradtopol or @zacharysarah I'd love another set of eyes on this to verify the correctness of the example - it's straightforward enough that I suspect one or you two would know off the top

/lgtm


Error using Enum with List variable

Describe the bug
We ocassionally get following error when installing our app with release pipeline:
2021-02-26T02:16:49.6780002Z RulesMgt_ITMod/Cod_LNModIntegrMgt.al(82,32): error AL0133: Argument 1: cannot convert from 'Enum "PB365 LM LN Mod.
2021-02-26T02:16:49.6782329Z Rule Action"' to 'Enum "PB365 LM LN Mod. Rule Action"'

Rerunning the pipeline usually solves the problem. Sometimes we must trigger this multiple times until a run succeeds.
To Reproduce
The following code is related to the error.
An Enum variable as well as a list variable for an Enum are declared:
    var
        lRulesAction: Enum "PB365 LM LN Mod. Rule Action";
        lCheckList: List Of [Enum "PB365 LM LN Mod. Rule Action"];

Code that finally causes the error is adding the Enum variable to list:
            if lCheck then
                lCheckList.Add(lRulesAction);

Expected behavior
The error should not occur.
5. Versions:
We actually use Business Central 16.3 platform 16.0.14073.14195

Can you check if this issue reproduces on the latest version of the AL development tools? Can you provide a self-contained project and a script to reproduce the issue?

I think AL development tools are not really relevant. Issue occurs when installing the apps via a release pipeline. We have actually two pipelines running for different Business Central versions:

17.4 (Platform 17.0.21485.21516 + Application 17.4.21491.21531)
16.3 (Platform 16.0.14073.14195 + Application 16.0.12076.0)

The issue arises ocassionally and cannot be reproduced by a script. Both pipelines have 2 stages (2 instances, where the apps are installed) which depend on each other. Sometimes first stage fails, sometimes second stage. There is no pattern recognizable.

@atoader Any news on this issue?

We get same error run ALValidation via bccontainerhelper for acutal BC version . We run it for 9 different countries. Error occurs for different country version e.g. in first run error occurs for us container, all other countries succeed, in second run error occurs for nl container and all other countries succeed. I can share the log files from the pipeline if necessary.

Hi! In the last months, we've been working hard on adding new features that increase your productivity and on improving the quality of the tools. Unfortunately, this meant that we were not able to follow up on as many GitHub issues as we would have liked.
If this is still a problem in the latest insiders build, feel free to raise this issue again.


Provider functions can accept arguments as injected values.

factory.RegisterProvider(&knex.Provider{
					Type: new(typeWithNoRequires),
					Instance: func(dependency DependencyType) (interface{}, error) {
						return &typeWithValueImpl{Value: dependency}, nil
					},
				})

Ended up with a different implementation than described above.


serve.handlers is removed

Issue workflow progress

Progress of the issue based on the
Contributor Workflow

[x] 1. The issue provides a reproduction available on
Github,
Stackblitz
or
CodeSandbox

Make sure to fork this template and run yarn generate in the terminal.
Please make sure Mesh package versions under package.json matches yours.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
serve.handlers is removed - and it seems like its accidentally removed, because commit says nothing about it.
https://github.com/Urigo/graphql-mesh/commit/738e2f37814c41ce1f588d62305035e8a3fa059b#diff-bed2d9c511387a1bbf73a42f388b58679ad789ade9ae87a6d6631cb26003c71d
Also changelog says nothing about serve.handlers except in part related to webhooks, which seems to be another commit: https://github.com/Urigo/graphql-mesh/commit/a2ef35c3532408b82f4d87dd45941b9e46ba2cc0#diff-bed2d9c511387a1bbf73a42f388b58679ad789ade9ae87a6d6631cb26003c71d
To Reproduce Steps to reproduce the behavior:
Add serve.handlers config.
Expected behavior
To work as before.
Environment:
Not related.
Additional context

It is not removed by accident. GraphQL Mesh no longer uses Express and it no longer provides a Node specific HTTP server implementation. You can use a server framework like Express, Fastify or something else on your own with Mesh's HTTP handler, or you can use Mesh's plugin system to hook into the HTTP handling logic.


Error while trying to view details of Recycle Bin

Microsoft PowerToys version
0.81.0
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
Peek
Steps to reproduce

✔ Expected Behavior
I wanted to see a preview of a Recycle Bin, how much space it uses and how many files in there
❌ Actual Behavior
When i press CTRL+Space Bar it shows an error message
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/26278


Question: Would be ORM set in project milestones?

This projects is very interesting and has apparently the best arrange of documentation, frequent updates and code style among other ArangoDB Python drivers.
In order to use this project it would be good having a sense whether is part of the scope adding some ORM classes to it, or if that should be part of another project, having this as a base for a lower level access to ArangoDB.

Hi rafaelpivato,
Thank you for your interest in python-arango.
My priority is to making sure that I cover all of ArangoDB's API endpoints and that the API of python-arango itself is as minimal, efficient and clean as possible. ORM is definitely on my radar, however, and is certainly in my realm of exploration.

Hi,
@joowani @rafaelpivato - Perhaps joowani can focus on making the API complete in python and another person or group could start building an ORM layer on top of it. I agree that the usage of this package closely resembles arangosh so it has minimal learning curve. The ORM on top of it could make things a lot simpler and quicker for developers.

My personal opinion is I'd like this to remain the clean, uncluttered low
level API it is. There is pyArango if someone wants an ORM.
$0.02
On Mon, Jun 27, 2016 at 10:06, Kashif Iftikhar<EMAIL_ADDRESS>wrote:

Hi,
@joowani https://github.com/joowani @rafaelpivato
https://github.com/rafaelpivato - Perhaps joowani can focus on making
the API complete in python and another person or group could start building
an ORM layer on top of it. I agree that the usage of this package closely
resembles arangosh so it has minimal learning curve. The ORM on top of it
could make things a lot simpler and quicker for developers.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/joowani/python-arango/issues/12#issuecomment-228755607,
or mute the thread
https://github.com/notifications/unsubscribe/AF_EDGVd-m7rhRo-1jZy0Vtsy_wmkkCyks5qP9j9gaJpZM4HnWJI
.

Perhaps then we could launch a separate package for ORM that depends on this one, keeps this one clean and for those of us who'd like the ORM, they can just install an additional package.

Yea for now the scope of this project will remain at the API level without the ORM.
Thanks for your interest in python-arango (version 3 is finally out now btw).

@kashifpk FYI: I've started on something that depends on both
python-arango and marshmallow (see https://github.com/sebastiancodes/marshmallow_arango) but I wouldn't call this the best thinkable ORM implementation.

@sebastiancodes this looks good. In the example, why do we need a model and a schema? In any case it's really something to begin with :-)

@sebastiancodes @joowani @rafaelpivato - I've started working on an orm layer for arango using python-arango. Please take a look here:
https://github.com/threatify/arango-orm

Your suggestions and opinions are welcome.

@kashifpk @sebastiancodes Nice integrations you've written guys! How do your implementations handle load_only / dump_only (i. e. if we want the user to be able to set some value through the API, but not get it) though?

Or in this case two schemas should be used?

@imale - my plan was that arango-orm would be used within an API implementation layer so access rights (ready-only or write-only) would be handled by the API.


allow default-token and per-subscription tokens in client.yml

fixes #653

I realize it's a bit sloppy to include the latest commit here instead of in a second PR, but I needed it to pass the tests and eliminate the race condition in the old version of the tests here.

Codecov Report
Patch coverage: 75.60% and project coverage change: +0.38 :tada:

Comparison is base (477c9d3) 67.15% compared to head (758eb3f) 67.53%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #659      +/-   ##
==========================================
+ Coverage   67.15%   67.53%   +0.38%
==========================================
  Files          47       47
  Lines        6631     6642      +11
==========================================
+ Hits         4453     4486      +33
+ Misses       1484     1465      -19
+ Partials      694      691       -3

Impacted Files
Coverage Δ

cmd/publish.go
74.23% <63.15%> (+4.23%)
:arrow_up:

cmd/subscribe.go
44.72% <84.21%> (+14.85%)
:arrow_up:

client/config.go
75.00% <100.00%> (+1.66%)
:arrow_up:

server/server.go
62.71% <100.00%> (+0.17%)
:arrow_up:

user/manager.go
59.29% <0.00%> (-0.71%)
:arrow_down:

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

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

I just merged this 600 line PR without a single comment other than "wow i didn't know that". Those that know me know that that is pretty much an impossible thing to achieve. Great work and 100 thanks.

I didn't even realize this was 600 lines, haha. I knew it was big, but I didn't expect it was that big


Support DataAnnotations for FormView

It would be good to use DataAnnotations and FormView together. For example Devexpress, Mudblazor

That would be great. FormView can get the entire Model as parameter to check data annotations.
I'll consider it.

You can check the following documentation about using DataAnnotations with UraniumUI components:
https://enisn-projects.io/docs/en/uranium/2.0/validations/DataAnnotations
It's just released with v2.0


[WIP] consistent dev

WHY are these changes introduced?
Fixes #0000

WHAT is this pull request doing?

How to test your changes?

Post-release steps

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes
[ ] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

/snapit

Random comment with /snapit in the middle

another random comment

/snapit

/snapit

/snapit

/snapit

/snapit

/snapit

/snapit

/snapit

/snapit

/snapit

/snapit


Improve BNS collectible format

Perhaps better to incorporate orange color (per below)
Display raw BNS punycode value alongside unicode
Provide copy option for raw BNS value
Link to BNS details page somewhere (though not sure where?)
Label as "BNSx" if wrapped by BNSx

@markmhx What is the relation to the existing format displayed at: https://gamma.io/collections/bns ?
Shouldn't we show the same as in Gamma?

@markmhx What is the relation to the existing format displayed at: https://gamma.io/collections/bns ?
Shouldn't we show the same as in Gamma?

Gamma should change this format coz how it is now doesn't represent emoji correctly and can mislead people. I bought .stx emoji coz didn't pay attention coz it is messed up with code, some other people have same problem, so it need to be corrected

and we all know what orange colour this is Bitcoin colour

@markmhx @nattaja Just to get a bit of an overview for this, who would be the stakeholder for this approval? Only the wallet team or it involves a broader group for approval?

Perhaps better to incorporate orange color (per below)
Display raw BNS punycode value alongside unicode
Provide copy option for raw BNS value
Link to BNS details page somewhere (though not sure where?)
Label as "BNSx" if wrapped by BNSx

could we click at placeholder image and this will link us to domain detales?

@markmhx @nattaja Just to get a bit of an overview for this, who would be the stakeholder for this approval? Only the wallet team or it involves a broader group? for instance, Gamma?

this is Hiro wallet so i suppose Hiro team must approve this

@markmhx @nattaja Just to get a bit of an overview for this, who would be the stakeholder for this approval? Only the wallet team or it involves a broader group? for instance, Gamma?

can be the same colour as bitcoin logo colour in wallet, will look more clean

@mica000 This is a design decision we can make independently as a wallet team, though drawing inspiration from Gamma and elsewhere if helpful

Note: We can make the "copy" option here universal for all collectibles, not just BNS names

Here are BNS image assets provided by Snazzy | 8g.btc that could also be incorporated here: https://www.figma.com/file/qiXgy5AwPhzkvRL5ObkTi6/BNS?type=design&node-id=0-1&mode=design&t=KlJNMh8ocmgXrFne-0

Requesting access to view the Figma file


Mostly type-hinted helper classes for inclusion in v2

Summary
This is a direct copy-paste of the pseudo-library that wraps python-slackclient I use internally. The big advantage is obviously typed interactions - especially the newer stuff (ie, blockkit) has pretty solid type hinting throughout.
I'm offering up this PR without any adjustments having been made - if it's something y'all are interested in, I could potentially spend some time making changes to better fit it into this library. Happy to review with anybody - I'm also in the dev4slack group semi-often if you want to reach me realtime.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

@paul-griffith This is awesome! I'm still reading through it, but I'm very interested in working together to get this into a mergeable state for v2.

This is super nice! it would help me a lot!

It would be awesome for this to be merged in. What needs to be done for this PR to be accepted?

There are currently a few high priority bugs I'm working on so it's taken me a while to get to this. This is definitely still a PR we'd like to see merged! Right now the biggest blocker is Unit tests and documentation.

The question is if you want to merge this as-is, with the extra wrapper, or if you maybe want to integrate the helper classes with the existing clients. I'd say the latter is preferable, but that requires some extra work which I don't know if @paul-griffith wants to put in.

No, the extra wrapper is an area that I'd like to discuss more. Preferably, I'd love to work to get the message builder classes merged first. This could be used immediately with the existing API methods.

@paul-griffith I want to thank you again for contributing such a phenomenal PR. It's clear you've put a great deal of work into this and it's going to be provide a lot of value to the community.

If I can help put this into the desired shape, let me know @paul-griffith @RodneyU215. This addition would greatly simplify the code of my own bot framework

I've made improvements and started to add tests to my own lib since I
opened this, but before I do more specific work for integrating I want to
double check with my own boss about how much time I can dedicate to this. I
also just updated my own codebase to work with v2 of this package.
Rodney, may be worth arranging a time we can have a quick discussion
real-time to see what you would want changed, also - that would help me see
how much time would be needed.
On Tue, Jun 4, 2019, 10:52 AM Daan Debie<EMAIL_ADDRESS>wrote:

If I can help put this into the desired shape, let me know @paul-griffith
https://github.com/paul-griffith @RodneyU215
https://github.com/RodneyU215. This addition would greatly simplify the
code of my own bot framework https://github.com/DandyDev/slack-machine
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/slackapi/python-slackclient/pull/400?email_source=notifications&email_token=AJMFY3RYAJZSWSDGXHJYY3DPY2TVTA5CNFSM4HIPHII2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW5LNEQ#issuecomment-498775698,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJMFY3UAXMEKRZZZEWTSZGLPY2TVTANCNFSM4HIPHIIQ
.

@paul-griffith Happy to chat this week. I've sent you a DM in dev4slack and you can also email me at<EMAIL_ADDRESS>so we can coordinate with a calendar invite.

I don't plan on significantly changing anything in the core lib at this point - just need to add test coverage at this point. Happy for anyone else to contribute tests, or even just known-failing-cases - coverage right now is basically limited to whatever is explicitly not allowed in the API documentation, but I'm confident there's plenty of scenarios that aren't covered by the JSON validation this currently has.

@DandyDev If you could spare a moment to review this PR I'd love your feedback on this as well.

@RodneyU215 I will have a look at it tonight (GMT+2 time). Happy to help!

@DandyDev If you could spare a moment to review this PR I'd love your feedback on this as well.

Done!

Solid points, @DandyDev. I moved the dialog sub-elements into a dialog_elements.py - I feel better about that so that users are "encouraged" to use the builder via slack.web.classes.dialogs, and have to go out of their way to pull in other classes. I also moved the common JSON handling into __init__.py for the package.
Agreed on get_json, but I'm just as unsure on a better replacement. to_dict or to_object might be better?
Also fixed the use of Optional and shadowing type when appropriate.
elements.py is fairly long (with the latest commit, it's the largest file in this directory) but I can't see a good way to split it up like what was possible with dialogs. Open to suggestions.

Solid points, @DandyDev. I moved the dialog sub-elements into a dialog_elements.py - I feel better about that so that users are "encouraged" to use the builder via slack.web.classes.dialogs, and have to go out of their way to pull in other classes. I also moved the common JSON handling into __init__.py for the package.
...
Also fixed the use of Optional and shadowing type when appropriate.

Great!

Agreed on get_json, but I'm just as unsure on a better replacement. to_dict or to_object might be better?

I like to_dict!

elements.py is fairly long (with the latest commit, it's the largest file in this directory) but I can't see a good way to split it up like what was possible with dialogs. Open to suggestions.

I see a couple of element categories that could be split out:

Select-related stuff
Menu-related stuff

That's the only split I can think of.

One other option with elements.py would be moving abstract/base classes to a subfile, so that from slack.web.classes.elements import <x> stays pretty clean for actual use classes, but that feels low value to me.

I'd say, leave it for now. For me it's a minor nitpick. If stuff is added to elements.py in the future, it can be reevaluated.

Okay, @RodneyU215, I'll defer to you on decisions about:

get_json() vs to_dict vs ???
AttachmentBuilder, yea or nay?
Refactor elements.py?

Otherwise, I think this is almost ready to merge? :crossed_fingers:

@DandyDev Thanks for the review!
@paul-griffith

I agree to_dict() is a much better fit.
AttachmentBuilder, nay. I'd prefer not to invest too much into something we're actively working to replace.
elements.py is long, but the context is where it needs to be. Let's keep this as-is for now.

Thank you both for the progress on this. We're very close to getting this merged. Once the rename is complete let's get this shipped.


Add support of versioned SSM params

There are 2 major things:

Migrating to go mod. Go vendoring actually is a nightmare to maintain. Go mod should not break anything, but will make the project development much much easier.
Add support of versioned SSM params. Actually, the first point I mentioned is a result of upgrading aws-go-sdk to the latest version (that supports versioning)

https://github.com/remind101/ssm-env/pull/34

Fixed!


Add support for integer-gmp-1.0.0

The API for integer-gmp has significantly changed with integer-gmp-<IP_ADDRESS>
See https://hackage.haskell.org/package/integer-gmp-<IP_ADDRESS>/candidate/docs/GHC-Integer-GMP-Internals.html for details

I'm new maintainer of HsOpenSSL. Please reopen this issue in the new repo https://github.com/vshabanov/HsOpenSSL
As an author of integer-gmp it will be great if you could add support for it in https://github.com/vshabanov/HsOpenSSL/blob/master/OpenSSL/BN.hsc#L129


chore(main): release 1.0.3

:robot: I have created a release beep boop
1.0.3 (2022-12-19)
Bug Fixes

added registry to setup node (c544b3d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/flowcore-io/library-flowcore-microservice-ts/releases/tag/v1.0.3 :sunflower:


Reinstate implicit import of iris.analysis.interpolate with iris.analysis

Fixes issue #2116
The fix is pretty messy. Perhaps @pp-mo or someone else can suggest a better way?

Perhaps @pp-mo or someone else can suggest a better way?

I'm proposing : #2121
It's really not any less messy, but it retains the behaviour that importing 'iris.analysis.interpolate' emits a deprecation warning.

i think this is now obsolete.
@djkirkham please close this if so


Feature: BeforeFeature / AfterFeature hooks

The problem
Sometimes we need to run code before / after particular feature file. Currently it can be achieved in playwright-bdd only with some hacks.
In CucumberJS it's not possible - there were several related requests:

https://github.com/cucumber/cucumber-js/issues/914
https://github.com/cucumber/cucumber-js/issues/1284
https://github.com/cucumber/cucumber-js/issues/1447
https://github.com/cucumber/messages/issues/260

CucumberJS FAQ states that these hooks are not possible, because runner does not consider features when running scenarios.
A solution
Playwright provides a direct way to run code before particular file - test.beforeAll / test.afterAll. We can just utilize that.
SpecFlow also provides BeforeFeature / AfterFeature hooks.
Binding to features can be done by tags - the same as for scenario hooks. Example code:
const { BeforeFeature, AfterFeature } = createBdd();

BeforeFeature({ tags: '@auth' }, async () => {
  // run setup before features with @auth tag
});

This also matches the proposed tags from path approach.
You can put feature files and BeforeFeature hooks into @-prefixed directory, and they will be bound automatically:
features
└── @auth
    ├── signin.feature
    └── hooks.ts

Hooks naming
Hooks naming between Cucumber and Playwright is a bit confusing:

Playwright's BeforeEach / AfterEach = Cucumber's Before / After - run before/after each scenario
Playwright's BeforeAll / AfterAll = (no alternative in CucumberJS) - run before/after each feature
Playwright's worker-scoped auto-fixtures = Cucumber's BeforeAll / AfterAll - run before/after each worker

I like the explicit naming in SpecFlow:

BeforeScenario / AfterScenario
BeforeFeature / AfterFeature
BeforeWorker / AfterWorker

I think, we can use these aliases in playwright-bdd.
Running once
There is a request from community to run beforeFeature hooks only once, even in fullyParallel mode / in case when some tests are failing and new worker is created.
Keep in mind during implementation, see https://github.com/microsoft/playwright/issues/22520#issuecomment-2351421682
I you have any thoughts / concerns here, feel free to share in comments.
Or just add your 👍

What should be a default behavior of BeforeFeature / AfterFeature hooks?

Run once in each worker: this is how Playwright's test.beforeAll / test.afterAll currently behave. If some test fails, new worker is created and hooks re-run again.
Run once in a whole test run: this is not how Playwright's test.beforeAll / test.afterAll behave, but according to playwright#22520 it is often needed (to setup some database stuff, etc).

What do you think?

I vote for 1st option

What should be a default behavior of BeforeFeature / AfterFeature hooks?

I thought more about that question. If BeforeFeature runs in each worker, it's actually not BeforeFeature but BeforeWorker. A confusion goes from naming: while feature is related to BDD level, worker is related to runner. Mixing these names in hooks does not make it clean.
Finally, I propose the following changes for hooks:

Alias existing Before / After hooks as BeforeScenario / AfterScenario and recommend using the latter, as they are more explicit
Alias existing BeforeAll / AfterAll as BeforeWorker / AfterWorker and recommend using the latter, as they are more explicit
Allow to pass tags option to BeforeWorker / AfterWorker. If tags are passed, the hook runs only if matched features are being executed (so actually behaves like BeforeFeature / AfterFeature). If a new worker is created for matched feature, the hook runs again. If tags are not passed, the hook runs before / after each worker (current behavior).
Allow to pass once option to BeforeWorker / AfterWorker. If once: true is passed, the hook runs only once during the whole test run session (so actually behave like Playwright's global setup / teardown). If additionally filtered by tags, the hook runs once and only if matched features are being executed - this is the main benefit for setting up some data once for the particular feature.
No BeforeFeature / AfterFeature hooks

Example code:
const { BeforeWorker } = createBdd();

BeforeWorker({ tags: '@auth', once: true }, async () => {
  // heavy setup of test user in database for checking all auth stuff
});

Feature file can contain @auth tag on the top level:
@auth
Feature: check auth

  Scenario: check login

Or on the scenario level:
Feature: check auth

  @auth
  Scenario: check login

If at least one scenario of feature has the matching tag, BeforeWorker hook will run before that feature.

Playwright-BDD v8 shipped tagging support for BeforeAll / AfterAll. So the following requirement is fulfilled:

Allow to pass tags option to BeforeWorker / AfterWorker. If tags are passed, the hook runs only if matched features are being executed (so actually behaves like BeforeFeature / AfterFeature). If a new worker is created for matched feature, the hook runs again. If tags are not passed, the hook runs before / after each worker (current behavior).

So, BeforeFeature / AfterFeature can be now implemented with tags:
BeforeAll({ tags: '@my-feature' }, async () => { ... });
AfterAll({ tags: '@my-feature' }, async () => { ... });

One note here, that BeforeFeature / AfterFeature can be interpreted in different ways.
If there are two feature files with @my-feature tag, should BeforeAll run before each feature or only once?
The same for parallelization: if a feature is split across several workers, should BeforeAll in each worker or not?
In current implementation, tagged worker hooks will run in each worker - similar to Playwright beforeAll / afterAll hooks.
But in some cases this is not a desirable behavior. If I want to populate database for some feature, I'd like to do it once, no matter how many workers run my tests. This is a goal for mentioned once option, it's not implemented now. There are some ideas, how to achieve that, but it's not an easy task, will be investigated separately.
Closing for now. Feel free to open a new issue if needed.


Added C solution for Round G Combination Lock Problem

Have you read the Contribution Guidelines?
(Yes, I have read contribution guidelines)
Description
(Added Folder Deepak-C in folder Combination Lock containing C solution to round G problem named Combination Lock.
please merge my pull request under hactoberfest 2021)
Checklist

[Yes] I've read the contribution guidelines.
[Yes] I've checked the issue list before deciding what to submit.

Related Issues (#73 )
Fixes Issue # (73)

Hey @deetav , can you remove the settings.json file?


Use soroban-client 0.8.1

Upgraded @soroban-react library and soroban-client dependency to use soroban-client version 0.8.1
The stellar-base library is now pinned to a specific version so that a fresh installation (via e.g. npm i soroban-client) does not implicitly pull the latest major version (i.e. one without Soroban support) of the dependency (https://github.com/stellar/js-soroban-client/pull/100).

Closed in favor of https://github.com/stellar/soroban-example-dapp/pull/117


Add docs for gherkin-lint

While inspecting cucumber issue#450 I found that gherkin-lint seems to miss documentation.

And when we have them, link to them from here: https://github.com/cucumber/cucumber/tree/master/gherkin-lint/javascript

Sidenote: according to this README Gherkin-Lint is not ready for use yet.

There is no official gherkin-lint implementation. We should remive the docs and the experimental code from master.

Created a PR to remove the code here: https://github.com/cucumber/cucumber/pull/452
Closing this issue.
(thanks anyway @jackorp!)


[BUG] If path parameters are containing several path parameter separator, only the first value is kept

Describe the bug
If path parameters are containing several path parameter separator, only the first value is kept
To Reproduce
Steps to reproduce the behavior:

Add a step
Add a vertx-http step
In http Url, provide a value with a port specified like https://www.test.com:8080/demo
Open source code editor
click on sync code
--> the part :8080/demo is removed

Expected behavior
keep the full uri
Logs
If applicable, add logs to help explain your problem.
Environment (please complete the following information):

OS: [e.g. "Linux 5.10.0-9-amd64 #1 SMP Debian 5.10.70-1 (2021-09-30) x86_64 GNU/Linux"]
Output of the mvn -version` command
Kaoto Version [e.g. 1.0]

Additional context
follow-up on #576

I am testing on https://github.com/KaotoIO/kaoto-backend/pull/597 and I think this is already fixed there.


Support for ASP.NET Core 2.1?

What are the plans for supporting this on ASP.NET Core 2.1?
Given the takeup of ASP.NET Core 2.0 when it came out, I guess most people will be upgrading to 2.1 as soon as it is ready.

@nrandell, I think it depends on the compatibility story.
If it's easy to "Multi-Target", then supporting both seems like a great idea. If we have to have a 3rd integration library (ASP.Net WebAPI, Core 2, Core 2.1) then it's something we'll need to evaluate.
Personally, I suspect @drub0y will have an ASP.Net Core 2.1 integration PR submitted very soon after 2.1 is released.

So, this is a bit of an overloaded question. The short answer is: it should "just work".
First, .NET Core has declared they are using a "roll-forward" approach to their major versions, so whatever works on 2.0 should work on all 2.x. So that means if you were to spin up a new project and target .NET Core 2.1, you will be able to use all of our libraries. Not only that, but, since the libraries are targeting .NET Standard 2.0, they should continue to run anywhere that supports that standard subset of APIs.
Next, from a runtime perspective, we will automatically gain from any JIT/CLR optimizations in the 2.1 obviously. Additionally, if you wanted to configure your ASP.NET 2.1 host with something like the new Socket transport to benefit from it's performance gains over the libuv version, you could do that and the BFSDK wouldn't know the difference.
Finally, from a perspective of leveraging new APIs (e.g. HttpClientFactory), obviously we would need to upgrade the codebase to utilize newer APIS . This would take a little longer as it would mean bumping dependency versions and, as always, that's a delicate game that takes a good amount of consideration.

That's what I thought. I've had it running on 2.1, but found some issues running on Azure where I kept on getting 502.3 errors when sending activities at certain times.
However - and this is a big however - I think it may have been down to user error on my part, which is why I wanted to ask the question about should it work rather than a specific problem.
Now I've got my scenario working under 2.0, I can try again with 2.1 to see if the problems still exist and update this thread.

Great, let us know!

So I created a new test repository (https://github.com/nrandell/botbuilder-dotnet-issue-448) - and guess what - it all works ok!
I then switched the original version back to 2.1 and it worked ok.
This may be something transient that is down to the roll out of 2.1
I'll close the issue for now.


hardcoded path in test result

Line 7 of src/jemplpl/test/includetest.expect:
OH HAI I IS AT LINE 1 IN "/Users/chris/projects/fpl2cc/src/jemplpl/test/includetest.cc"
.. /Users/chris... nice shooting.

clarification:  the problem is not that it's hardcoded, it's that it's an absolute path.
.. actually why is the one above it not absolute?


[Importer] - Support regular expressions in filter pathname

The customer hubblehomes.com as a very well known pattern for their inventory homes, where all paths end in 999.  It would be very useful to be able to crawl the site and get all these inventory pages by providing a regular expression in the filter pathname field as part of the Crawl section.
Expected Behaviour
As a user I would like to have the Filter pathname support a regular expression option.

Actual Behaviour
It seems that only startsWith is supported on the path which is very limiting.
Reproduce Scenario (including but not limited to)
Steps to Reproduce

Navigated to the Crawl page
Enter https://www.hubblehomes.com as the host
Uncheck Show Preview
Enter the regular expression .*\/999 into the Filter pathname field

(No results)
Sample URL
https://www.hubblehomes.com/new-homes/idaho/boise-metro/caldwell/mason-creek/birch/19103-piaggio-ave/999

@kptdobe @catalan-adobe
Small change for an issue just reported.  I can't assign anyone (perhaps I need to do it before I create the PR).  Could you have a look?
Thanks.

Thanks for address this!


ctest.c : add POWERPC for PowerMac

this extra definition satisfies the gcc compiler on PowerPC Apple computers

Thanks. I suspect PowerMac support has not been looked at in a while

FYI for followup -- autodetecting the POWER compiler flavour on PPC Darwin seems to be not working, but forcing a PPCG4 target is successful on a PowerMac running Leopard, and on a PowerMac running PPC Tiger (2007 -- not bad!) if a newer version "make" is installed.
Forcing the PPC970 target even on a PowerMac with a 970 fails at present.


The float32 value 0.001 is displayed as 0.0010000000474974513

Testcase: https://people.freebsd.org/~yuri/model-containing-0.001.tflite
The test model has 0.001 in weights, but this number seems to be expanded to 'float64' when displayed.
Version 4.1.2

@yurivict can you have a look and provide a fix?

I don't have resources, sorry.
I am just a FreeBSD package maintainer and a user.

@lutzroeder and @yurivict if you load up the data as float and then display it then the internal representation of 0.001 is not accurate in float32 or float64. So the only way to accurately show it is use numeric format when loading and avoid the conversion all together

There is no such thing as the 32-bit float value 0.001. Your tflite model does not have a weight that is 0.001. It has a weight that is 0.0010000000474974513.
Do the following in Python to see this (or your favorite language):
import numpy as np
a = np.array([0.001], dtype=np.float32)
print("%.20f" % a)

It will print 0.00100000004749745131.

You are right!
libdouble-conversion (https://github.com/google/double-conversion) says that the decimal representation for 0x3a83126f is 0.0010000000474974513.

#include <stdlib.h>
#include <stdio.h>

int main(int argc, char *argv[]) {
	unsigned char byte_array[] = { 0x6f, 0x12, 0x83, 0x3a };
	float float_value = *(float *)&byte_array;
	printf("%f\n", float_value);
	double double_value = float_value;
	printf("%f\n", double_value);
}
// 0.001000
// 0.001000

class Program
{
    static void Main(string[] args)
    {
        byte[] array = new byte[] { 0x6f, 0x12, 0x83, 0x3a};
        float float_value = System.BitConverter.ToSingle(array, 0);
        System.Console.WriteLine(float_value.ToString());
        double double_value = float_value;
        System.Console.WriteLine(double_value.ToString());
    }
}
// 0.001
// 0.0010000000474974513

This shows that printf and System.Console.WriteLine have some arbitrary rounding thresholds and don't display floating point values properly.
libdouble-conversion converts to a decimal representation precisely, but it unfortunately only has a C API interface and is therefore more difficult to use.

Just to be pedantic about it :smile:, if you replace printf("%f\n", float_value); in the C version with printf("%.20f\n", float_value); it gives the 0.0010000000474974513 version.


[HOLD for payment 2024-07-22] [HOLD for payment 2024-07-17] [$250] [HOLD 36648] Wallet - Idology questions flow does not start after entering bank account details

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:  9.0.2-0
Reproducible in staging?: Y
Reproducible in production?: Unable to check
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4673461
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Open app or go to staging.new.expensify.com
Log in with any account without a wallet configured
Go to Settings > Wallet
Click on 'Enable wallet'
Follow the bank account flow until the final 'Confirm' page (using Bank: Chase / Account: 1111 / First name: Alberta / Last name: Bobbeth / SSN: 3333)
Tap 'Confirm' button

Expected Result:
Idology questions flow has started
Actual Result:
Nothing happens after pressing the 'Confirm' button. Idology questions flow does not start
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/e6b6fcfb-8732-44e1-8f79-1a2251d4896b
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014deaa2c3f332d2e4
Upwork Job ID:<PHONE_NUMBER>003619663
Last Price Increase: 2024-06-28

Issue OwnerCurrent Issue Owner: @
Issue OwnerCurrent Issue Owner: @OfstadC
Issue OwnerCurrent Issue Owner: @OfstadC

We think that this bug might be related to #vip-vsp

@thienlnam @OfstadC  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

It seems like this failed because of a non-matching SSN (logs)
I wonder if this was updated for our test bank account recently but it doesn't seem to be from any recent changes

Posted here, https://expensify.slack.com/archives/C03SSAQ7P/p1719415968231039 but don't think this is a result of changes made

Actually, after some confirmation in the thread there's a bug here and it seems to be that the KBA questions do not show up after the initial failure and resubmission

@Pujan92 Any update here?

@Pujan92 Please provide an update by EOD. Thanks!

Sorry @OfstadC, The PR merged and the code in the production. I think bcoz of Hold the automation isn't worked for this issue.

Thanks @Pujan92 ! Could you complete the BZ checklist when you have a chance? Thanks! 😃

@OfstadC I believe that will be handled in the parent issue https://github.com/Expensify/App/issues/36648

Ah okay! 😅 Thanks @Pujan92 !

Closing this out -
No payment needed per:

For reference, here are some details about the assignees on this issue:
@Pujan92 does not require payment (Contractor)
@koko57 does not require payment (Contractor)

And checklist should be covered here


JAX-RS Async

+1

hi all ,
for my understand...
when this feature enabled so, this : ejb fraction with method async is not needed anymore right ?
btw I'm not java EE expert

i mean, no need add EJB Fraction because this will be enabled by JAX-RS fraction is correct ?
thanks

Going out on a limb here, but could not this just be handled by the underlying resteasy implementation by using @Suspended AsynchronousResponse response as parameter to the rest-endpoint?
The @Suspend annotation tells Resteasy that the HTTP request/response should be detached from the currently executing thread and that the current thread should not try to automaticaly process the response.
https://docs.jboss.org/resteasy/docs/1.0.0.GA/userguide/html/Asynchronous_HTTP_Request_Processing.html
Or have I missed something very essential?

hi @thomasandersen77

but could not this just be handled by the underlying resteasy implementation by using @Suspended AsynchronousResponse response as parameter to the rest-endpoint?

yup this is the point too (1), so that we don't need to add one more jar as dependency ( the ejb fraction to get the async method ) because jax-rs fraction will have it enabled by default (2)
thanks

Are you planning to create an example or is it open for others to contribute. Was so fun to contribute with a bit with the Docker example ;)

@thomasandersen77 go for it !
Is open for others to contribute : ]
My plan (now) is to put alpha version in production on my current job

@eprogramming Thanks, I will :)
Please let me know how stable it is when you've had some traffic on it. Maybe my customer will consider it in a few months as well. As of now we use it to mock an external system in a Docker test environment

Please let me know how stable it is when you've had some traffic on it

Ok !

I think you're right. Mostly this is just to demonstrate that existing jaxrs async stuff actually works. Hopefully no net new code.
--
Sent from my iPhone

On Jul 9, 2015, at 6:04 AM, thomas andersen<EMAIL_ADDRESS>wrote:
Going out on a limb here, but could not this just be handled by the underlying resteasy implementation by using @Suspended AsynchronousResponse response as parameter to the rest-endpoint?
The @Suspend annotation tells Resteasy that the HTTP request/response should be detached from the currently executing thread and that the current thread should not try to automaticaly process the response.
https://docs.jboss.org/resteasy/docs/1.0.0.GA/userguide/html/Asynchronous_HTTP_Request_Processing.html
Or have I missed something very essential?
—
Reply to this email directly or view it on GitHub.

The @Suspend async JAX-RS stuff works as expected.
In a separate example where I'm mucking with NetflixOSS Ribbon, I see it functioning:
https://github.com/wildfly-swarm/wildfly-swarm-examples/blob/master/multiple/events/src/main/java/org/wildfly/swarm/examples/multi/events/EventsResource.java


readme edited

Make sure these points are followed before opening a pull request.

I have added my name and github url under the contributors sections
I have followed the example while adding myself as contributor

Refers #1

thanks @roony4
Merged! 🎉

thanks you.
On Sat, Oct 6, 2018 at 3:40 PM bl4ckw4sp<EMAIL_ADDRESS>wrote:

Merged #3
https://github.com/myfirstcontribution/myfirstcontribution/pull/3 into
master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/myfirstcontribution/myfirstcontribution/pull/3#event-1888510042,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjD0dNXRDy24Ce1o10921W72A1b2cjCwks5uiIGKgaJpZM4XLRt0
.


question on spark sql functions which are not mapped?

Can someone explain why this levenshtein call works. It is not mapped in the SQL mapping at https://github.com/rstudio/sparklyr/blob/master/R/dplyr_sql.R or https://github.com/rstudio/sparklyr/blob/master/R/dplyr_spark_connection.R.
Is the default behaviour of dplyr if the SQL is not mapped to use the function name?
FYI levenshtein is defined here: https://spark.apache.org/docs/latest/api/java/org/apache/spark/sql/functions.html
library(sparklyr)
library(dplyr)
sc <- spark_connect(master = "local")

x <- expand.grid(a = c("hi there", "thank you"), b = c("hi there xyz", "ank you"))
mydata <- copy_to(sc, x, overwrite = TRUE)
dplyr::mutate(mydata, lev = levenshtein(a, b))

Where can I find the code where this default behaviour if SQL is not mapped is occurring. In dplyr or sparklyr?

Indeed, the default behavior for dplyr is to simply generate SQL verbatim for functions without a known translator. In such a case, functions made available by the underlying SQL engine can be used -- in this case, Hive functions.
See https://cwiki.apache.org/confluence/display/Hive/LanguageManual+UDF for a list of functions; in particular, levenshtein is defined as one of the String functions.
The magic happens inside of dplyr::sql_build -- you can try debugging it to see how it uses the stored expressions constructed in the various mutate calls to construct SQL.
We should probably add some documentation to make this more clear / apparent.

Never mind, it's documented in ?dplyr::sql_variant, sorry for bothering.


Add: MTV Urheilu 2

Channel Name
MTV Urheilu 2
Alternative Names (optional)
MTV Urheilu 2 HD
Network (optional)
No response
Owners (optional)
No response
Country
FI
Subdivision (optional)
No response
City (optional)
No response
Broadcast Area
c/FI
Languages
fin
Categories (optional)
No response
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
No response
Logo
https://i.imgur.com/tgezMDk.png
Notes
No response

Please update your request:

link to the official website is incorrect
the size of the logo should not exceed 512x512px

https://github.com/iptv-org/database/blob/master/CONTRIBUTING.md#channels

Thank you, @freearhey. Updated the logo url and removed the website url. There is not official channel website available so I ended up cleaning website url information (https://www.mtv.fi/urheilu might be the closest one for "official" url)


Add support for assigning and unassigning tasks

To be added in next PR
Create field in task, List of person that the task is assigned to
Mark/unmark commands

It seems like the current sample data has tasks assigned to people who are not yet vendors, is it possible to assign tasks to vendors only in the sample data?

It seems like the current sample data has tasks assigned to people who are not yet vendors, is it possible to assign tasks to vendors only in the sample data?

Sure, i'll make the changes