Jest running JS files and failing

Testes does not pass since jest is running *.test.js files.
Need to add testMatch to jest.config.js
 testMatch: [     "**/__tests__/**/*.ts",     "**/?(*.)+(spec|test).ts"   ]
LOG
`
➜  jest-examples cd ts-jest-mock-examples
➜  ts-jest-mock-examples git:(master) ✗ jest
FAIL  sound-player/SoundPlayer.test.js
● Console
console.log sound-player/SoundPlayer.js:8
  Playing sound file song.mp3

● is called with filename
TypeError: soundPlayer.playSoundFile.mockClear is not a function

   9 | const soundPlayer = utils_1.mocked(new SoundPlayer_1.default());
  10 | beforeEach(() => {
> 11 |     soundPlayer.playSoundFile.mockClear();
     |                               ^
  12 | });
  13 | it('is called with filename', () => {
  14 |     const filename = 'song.mp3';

  at Object.<anonymous>.beforeEach (sound-player/SoundPlayer.test.js:11:31)

● is called with filename
expect(received).toBeCalledWith(...expected)

Matcher error: received value must be a mock or spy function

Received has type:  function
Received has value: [Function playSoundFile]

  14 |     const filename = 'song.mp3';
  15 |     soundPlayer.playSoundFile(filename);
> 16 |     expect(soundPlayer.playSoundFile).toBeCalledWith(filename);
     |                                       ^
  17 | });
  18 |

  at Object.<anonymous>.it (sound-player/SoundPlayer.test.js:16:39)

FAIL  my-class/MyClass.test.js
● MyClass › should check the constructor has been called
TypeError: myClass.instanceMethod.mockClear is not a function

  26 |     const myClass = utils_1.mocked(new MyClass_1.default());
  27 |     beforeEach(() => {
> 28 |         myClass.instanceMethod.mockClear();
     |                                ^
  29 |         myClass.arrowFunction.mockClear();
  30 |     });
  31 |     it("should check the constructor has been called", () => {

  at Object.beforeEach (my-class/MyClass.test.js:28:32)

● MyClass › should check the constructor has been called
expect(received).toBeCalledTimes(expected)

Matcher error: received value must be a mock or spy function

Received has type:  function
Received has value: [Function MyClass]

  31 |     it("should check the constructor has been called", () => {
  32 |         new MyClass_1.default();
> 33 |         expect(MyClass_1.default).toBeCalledTimes(1);
     |                                   ^
  34 |     });
  35 |     it("should call mocked instanceMethod", () => {
  36 |         const result = myClass.instanceMethod("Peter");

  at Object.it (my-class/MyClass.test.js:33:35)

● MyClass › should call mocked instanceMethod
TypeError: myClass.instanceMethod.mockClear is not a function

  26 |     const myClass = utils_1.mocked(new MyClass_1.default());
  27 |     beforeEach(() => {
> 28 |         myClass.instanceMethod.mockClear();
     |                                ^
  29 |         myClass.arrowFunction.mockClear();
  30 |     });
  31 |     it("should check the constructor has been called", () => {

  at Object.beforeEach (my-class/MyClass.test.js:28:32)

● MyClass › should call mocked instanceMethod
expect(received).toBeCalledWith(...expected)

Matcher error: received value must be a mock or spy function

Received has type:  function
Received has value: [Function instanceMethod]

  35 |     it("should call mocked instanceMethod", () => {
  36 |         const result = myClass.instanceMethod("Peter");
> 37 |         expect(myClass.instanceMethod).toBeCalledWith("Peter");
     |                                        ^
  38 |         // result is undefined because we did not change the default mock
  39 |         // implementation of instanceMethod which is jest.fn() and simply returns
  40 |         // undefined.

  at Object.it (my-class/MyClass.test.js:37:40)

● MyClass › should change the instanceMethod mock implementation
TypeError: myClass.instanceMethod.mockClear is not a function

  26 |     const myClass = utils_1.mocked(new MyClass_1.default());
  27 |     beforeEach(() => {
> 28 |         myClass.instanceMethod.mockClear();
     |                                ^
  29 |         myClass.arrowFunction.mockClear();
  30 |     });
  31 |     it("should check the constructor has been called", () => {

  at Object.beforeEach (my-class/MyClass.test.js:28:32)

● MyClass › should change the instanceMethod mock implementation
TypeError: myClass.instanceMethod.mockImplementationOnce is not a function

  42 |     });
  43 |     it("should change the instanceMethod mock implementation", () => {
> 44 |         myClass.instanceMethod.mockImplementationOnce((s) => `Hi, ${s}`);
     |                                ^
  45 |         const result = myClass.instanceMethod("Peter");
  46 |         expect(myClass.instanceMethod).toBeCalledWith("Peter");
  47 |         expect(result).toBe("Hi, Peter");

  at Object.it (my-class/MyClass.test.js:44:32)

● MyClass › should call mocked arrowFunction
TypeError: myClass.instanceMethod.mockClear is not a function

  26 |     const myClass = utils_1.mocked(new MyClass_1.default());
  27 |     beforeEach(() => {
> 28 |         myClass.instanceMethod.mockClear();
     |                                ^
  29 |         myClass.arrowFunction.mockClear();
  30 |     });
  31 |     it("should check the constructor has been called", () => {

  at Object.beforeEach (my-class/MyClass.test.js:28:32)

● MyClass › should call mocked arrowFunction
expect(received).toBeCalledWith(...expected)

Matcher error: received value must be a mock or spy function

Received has type:  function
Received has value: [Function anonymous]

  49 |     it("should call mocked arrowFunction", () => {
  50 |         const result = myClass.arrowFunction("Joe");
> 51 |         expect(myClass.arrowFunction).toBeCalledWith("Joe");
     |                                       ^
  52 |         expect(result).toBeUndefined();
  53 |     });
  54 |     it("should change the arrowFunction mock implementation", () => {

  at Object.it (my-class/MyClass.test.js:51:39)

● MyClass › should change the arrowFunction mock implementation
TypeError: myClass.instanceMethod.mockClear is not a function

  26 |     const myClass = utils_1.mocked(new MyClass_1.default());
  27 |     beforeEach(() => {
> 28 |         myClass.instanceMethod.mockClear();
     |                                ^
  29 |         myClass.arrowFunction.mockClear();
  30 |     });
  31 |     it("should check the constructor has been called", () => {

  at Object.beforeEach (my-class/MyClass.test.js:28:32)

● MyClass › should change the arrowFunction mock implementation
TypeError: myClass.arrowFunction.mockImplementation is not a function

  53 |     });
  54 |     it("should change the arrowFunction mock implementation", () => {
> 55 |         myClass.arrowFunction.mockImplementation((s) => `Bonjour, ${s}`);
     |                               ^
  56 |         const result = myClass.arrowFunction("Joe");
  57 |         expect(myClass.arrowFunction).toBeCalledWith("Joe");
  58 |         expect(result).toBe("Bonjour, Joe");

  at Object.it (my-class/MyClass.test.js:55:31)

PASS  my-class/MyClass.test.ts
PASS  sound-player/SoundPlayer.test.ts
Test Suites: 2 failed, 2 passed, 4 total
Tests:       6 failed, 6 passed, 12 total
Snapshots:   0 total
Time:        2.955s
Ran all test suites.
➜  ts-jest-mock-examples git:(master) ✗
`

Thanks for reporting this @oriamd. Do you have some steps on how to reproduce this? If I run yarn test on my end all tests are passing.

Think I got it now.  No harm merging the PR. Thanks.


feat: Allow using either Settings or RewriteStrSettings

Closes https://github.com/cloudflare/lol-html/issues/56

@ObsidianMinor seems like the newest clippy is unhappy with the selector-related code

See #60 @inikulin


Kubernetes tools from AppsCode Inc

AppsCode has some amazing K8S tools. I believe those tools can be listed here.
https://github.com/appscode

@shihabmridha can you please open a PR?


chore(frontend): Remove Unused Svelte Files

The following files were removed because they were not used in the project anymore:
src/frontend/src/icp/components/core/IcListener.svelte

@AntonioVentilii-DFINITY you still want to wait to remove this module?

@peterpeterparker nono,  I just wanted to check that the workflow works, perfect! I will approve and merge


Turkish source

Source name
Fenrircans
Source link
fenrirscans.com
Language
Turkish
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a title with source name.
[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I will fill out all of the requested information in this form.

I have sent a question before asking why this site is not added and I have not received an answer as to why it is not added. I also did the same for a few other sites and still didn't get a response.
https://fenrirscans.com/
https://novelokur.com.tr/
https://turkcenoveloku.blogspot.com/
https://novelaozora.com/
https://www.felisnovel.com/
https://lnmakineceviri.blogspot.com/?m=1
https://hipertales.com/
Why are these sites not added?

All the people working on this project do it in their free time and don't get paid to do it. So there is no reason or responsibility to add new plugins except ones own desire. And as far as I know there are no Turkish contributors, that means there is no one who wants to add these sites.
You can only wait until there is someone who wants to add these or learn to do it yourself. If you want to learn you can join the discord where you can find help if you get stuck.

Have you ever opened the app?

Can I ask you to add these? Also, I couldn't fully understand my problem because of the translation. I don't have any intention of getting angry with you, I just wanted to ask, thank you. Also, there was a problem in the Turkish section, I will write it here and you can fix it.

This site is not working and I should have written before, this site is not a Turkish site, could you please fix this?

You should check if your requests were created before.

I dont understand Turkish so I cant fix it, Wait for someone understand Turkish


Moralis.onAccountChanged is not a function

I have cloned the project and done the yarn command successfully in order to load node_modules
Anyway, When I run the project, I met the error below.
TypeError: Moralis.onAccountChanged is not a function. (In 'Moralis.onAccountChanged(function (address) {         setWalletAddress(address[0]);       })', 'Moralis.onAccountChanged' is undefined)
When I block this code, it can't make connect to wallet successfully.
The environment, I use the default from this boilerplate.
Do you have any suggestions or I miss some part? Please guide me in advance.
Thanks

Hi can you try changing that to Moralis.onAccountsChanged in the file at frontend/providers/MoralisDappProvider/MoralisDappProvider.js.

Hi have changed to  Moralis.onAccountsChanged in the file at frontend/providers/MoralisDappProvider/MoralisDappProvider.js
but still i am getting error 404 on https://walletconnect.com/wallets and the popup with all the wallets is not shown

Hi can you post on the Moralis forum in this thread: https://forum.moralis.io/t/ethereum-react-native-boilerplate-questions/4511

guys i have the same error as mentioned above
TypeError: Moralis.onChainChanged is not a function
but i cant understand where this file exists as @alexnguyennz  mentioned.
@alexnguyennz  can you guide me on this issue, it will be a grate help to me. THANKS

Hey your error is slightly different - in your terminal, it should show you the file and line where that code is being run. You can also do a project search with your IDE or code editor.


cleaner build on ubuntu22 with boost link fix

cleans up warnings and changes needed for my Ubuntu22 boost default packages to link tests

Do you usually require two reviewers @babakpst  ?


Merge into "terraform fmt"

Hi,
I appreciate your work. It looks like a very interesting project.
However, I wonder why it is an external tool, therefore more difficult to discover, because to me it sounds like something that should be part of "terraform fmt".
Was there any previous work on inclusion in "terraform fmt" that was rejected? Or maybe it should be considered as a testing ground for POC before being included in "terraform fmt"?
Kind regards

Hi @ad-m-ss ,
I don't think this should be part of terraform fmt, but you have the same feature in tflint.


Hide horizontal scrollbar on the sidebar

layered navigation is showing a single line when increasing the price slider #4111
BUGS:

Hide overflow on the left sidebar wrapper

@praem90
changes are not coming as compilation files are missing, please run the npm command.

Resolve conflicts


For ConnectEhr, ConnectFitbit, and ConnectGarmin widgets, update hideWhenConnected flag to display connections in error

Overview
Addresses: https://github.com/CareEvolution/MyDataHelpsUI/issues/224
Adds stories to the ConnectFitbit, ConnectGarmin, and ConnectEhr to cover cases when the hideWhenConnected flag is set.  Updates the default behavior of "hideWhenConnected" to not hide the widget in the case the connection is in an unauthorized or error state.  This will allow a user to view the widget and be able to reconnect or fix a connection error.
This is a functional change to the hideWhenConnected flag.
This change can be see in the ConnectEhr->Hide Connected story, the Fitbit->Hide Connected and Garmin->Hide Connected stories.  Prior to the code change these would be hidden in the UI.

Security
REMINDER: All file contents are public.

[x] I have ensured no secure credentials or sensitive information remain in code, metadata, comments, etc.

[x] Please verify that you double checked that .storybook/preview.js does not contain your participant access key details.
[x] There are no temporary testing changes committed such as API base URLs, access tokens, print/log statements, etc.

[x] These changes do not introduce any security risks, or any such risks have been properly mitigated.

Changes to how existing connections are displayed in the user interface only.  No changes to underlying connection code.
Checklist
Testing

[ ] MyDataHelps iOS app
[ ] MyDataHelps Android app
[ ] MyDataHelps website

Testing was done via local storybook testing only
Documentation

[ ] I have added relevant Storybook updates to this PR as well.
[ ] If this feature requires a developer doc update, tag @CareEvolution/api-docs.

Consider "Squash and merge" as needed to keep the commit history reasonable on main.
Reviewers
Assign to the appropriate reviewer(s). Minimally, a second set of eyes is needed ensure no non-public information is published. Consider also including:

Subject-matter experts
Style/editing reviewers
Others requested by the content owner

Minor updates to the stories, and updates to UseEffect


Error in plot_asirra_example

The example code 'plot_asirra_dataset.py', gets the following error:
Traceback (most recent call last):
  File "plot_asirra_dataset.py", line 28, in <module>
    X, y, train_size=.6, random_state=1999)
  File "/usr/local/lib/python2.7/dist-packages/sklearn/cross_validation.py", line 1556, in train_test_split
    arrays = check_arrays(*arrays, **options)
  File "/usr/local/lib/python2.7/dist-packages/sklearn/utils/validation.py", line 287, in check_arrays
    array.ndim)
ValueError: Found array with dim 4. Expected <= 2

I got another error today when running examples/plot_asirra_dataset.py:
ValueError: The number of classes has to be greater than one.
```

Looks like that's because `np.unique(data['target'])` is `array([0.])` when `image_count` is small.

OK, we should put a check in the dataset to avoid that. PR is welcome if you have the time!

The latest commit should fix this. Try going to '~/sklearn_theano_data' and
removing the 'asirra/' directory - this should prompt a redownload of
asirra, which will take a while, but from what I could find this error is
the result of a failed download. We should probably get some kind of
checksum checking here.
On Thu, Mar 12, 2015 at 5:39 AM, Felix Lau<EMAIL_ADDRESS>wrote:

I got another error today when running examples/plot_asirra_dataset.py:
ValueError: The number of classes has to be greater than one.
Looks like that's because np.unique(data['target']) is array([0.]) when
image_count is small.
—
Reply to this email directly or view it on GitHub
https://github.com/sklearn-theano/sklearn-theano/issues/34#issuecomment-78448877
.


[.NET Native] error RHB0011: Internal error "!methodDesc->IsCanonicalMethod(CanonDefinitionTypeDiscoveryPolicy::AnyCanonLookup)"

MCVE:

Create new UWP project (min version: Fall Creators)
Add next line:

 List<(string Left, string Right)?> P2 = new List<(string Left, string Right)?>();

when I try to compile I got:

1>...\Microsoft.NetNative.targets(788,5): error : RHBIND : error RHB0011: Внутренняя ошибка: "!methodDesc->IsCanonicalMethod(CanonDefinitionTypeDiscoveryPolicy::AnyCanonLookup)" в "f:\dd\ndp\rh\src\tools\rhbind\mdilmodule.cpp:12529"
1>...\Microsoft.NetNative.targets(788,5): error : ILT0005: Задача "C:\Users\NF.nuget\packages\runtime.win10-x86.microsoft.net.native.compiler\2.1.8\tools\x86\ilc\Tools\rhbind.exe @"F:\InternP\NetNativeError-master\NetNativeError-master\NetNativeError.Control\obj\x86\Release\ilc\intermediate\rhbindargs.NetNativeError.Control.rsp"" вернула код завершения 11.

Works with value type like:
List<(int Left, int Right)?> P2 = new List<(int Left, int Right)?>();

There is no problem in debug mode.

More investigation: the problem is reproducible with local variables too:
class Test
{
    void T()
    {
        List<(string Left, string Right)?> L = null; // OK
        List<(string Left, string Right)?> L = new List<(string Left, string Right)?>(); // not OK
        _ = new List<(string Left, string Right)?>(); not OK
    }
}

Thanks for reporting the issue to us. This issue was moved to Project N bug tracker (ID: 659486). For future issues with .NET Native for Universal Windows Apps, please follow the process described here. The CoreRT repo is not tracked by people working on .NET Native for UWP apps.

For future issues with .NET Native for Universal Windows Apps, please follow the process described here.

@MichalStrehovsky Thanks. Just to clarify: There is no way to see current reports, right?

Unfortunately, no. The tool where the bug is (rhbind/nutc_driver) is closed, and so is their bug database, release schedule, etc.


Playwright docs

Working on some playwright docs with @zakiya
They playwright.md is the start of a playbook recording some info useful for writing end to end tests with playwright
The package.json and corresponding test files are there to run the samples

No, not real tests
These are docs on testing

This is internal facing documentation that is convenient to write and consume in github repo markdown. @zakiya and I thought it worked well here but happy to move it if we have a better location

Capturing some notes here for future documentation:

playwright.config.js is pretty useful. We should document and/or reference it. headless: true/false can take the place of PWDEBUG = 1
Users need to be aware of the Page.Locator class
You can store tests somewhere besides tests folder by using playwright test my/chosen/path/menus.spec.js commnad


reserved identifier violation

I would like to point out that an identifier like “orfz__scan_bytes” does eventually not fit to the expected naming convention of the C++ language standard.
Would you like to adjust your selection for unique names?

That is a name generated in c code by bison.
Le mar. 17 mars 2020 à 18:48, Markus Elfring<EMAIL_ADDRESS>a
écrit :

I would like to point out that an identifier like “orfz__scan_bytes
https://github.com/google/or-tools/blob/554cbccaa95b4c11eced13f145de1bf468e1f919/ortools/flatzinc/parser.cc#L30”
does eventually not fit
https://www.securecoding.cert.org/confluence/display/cplusplus/DCL51-CPP.+Do+not+declare+or+define+a+reserved+identifier
to the expected naming convention of the C++ language standard.
Would you like to adjust your selection for unique names?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1924, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3MMEIVZBPJVQLCUNA3RH6ZYHANCNFSM4LNVYH6A
.

How do you think about to avoid that this software depends on undefined behaviour?

Are you trying to go pedantic on me ?
This is a minizinc layer, for academic challenges.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le mar. 17 mars 2020 à 22:10, Markus Elfring<EMAIL_ADDRESS>a
écrit :

How do you think about to avoid that this software depends on undefined
behaviour
https://www.securecoding.cert.org/confluence/display/c/CC.+Undefined+Behavior#CC.UndefinedBehavior-ub_106
?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1924#issuecomment-600303067,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3NKRHWSANAW4JTJRE3RH7RMXANCNFSM4LNVYH6A
.

I suggest to reconsider a specific implementation detail.
How do you think about to reduce the tampering with the reserved name space?

FYI this issue come the prefix:
https://github.com/google/or-tools/blob/858fa626959f7e386153af82756384b79f983b5a/ortools/flatzinc/parser.yy#L2
Flex doc:

-Pprefix
changes  the default yy prefix used by flex for all globally-visible variable and  function names  to
instead  be prefix.  For example, -Pfoo changes the name of yytext to footext.   It  also  changes  the
name  of	the  default output file from lex.yy.c to lex.foo.c.
Here are all of the names affected:
yy_create_buffer
yy_delete_buffer
yy_flex_debug
yy_init_buffer
yy_flush_buffer
yy_load_buffer_state
yy_switch_to_buffer
yyin
yyleng
yylex
yylineno
yyout
yyrestart
yytext
yywrap

src: http://dinosaur.compilertools.net/flex/manpage.html


Cannot format code on Vs Code

Yes, we are working on it. In couple of weeks we will issue the updates by many things.


switched from using https to ssh on gems

Why  are farscape gem dependencies using https instead of ssh?  I found that using https requires basic auth authentication when working on farscape, locally.

screw it...closing.


DataFrame.join

First try to implement DataFrame.join function

Codecov Report

Merging #450 into master will increase coverage by 0.09%.
The diff coverage is 98.43%.

@@            Coverage Diff             @@
##           master     #450      +/-   ##
==========================================
+ Coverage   93.06%   93.15%   +0.09%
==========================================
  Files          27       28       +1
  Lines        3344     3479     +135
==========================================
+ Hits         3112     3241     +129
- Misses        232      238       +6

Impacted Files
Coverage Δ

databricks/koalas/missing/frame.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/frame.py
94.85% <98.43%> (+0.12%)
:arrow_up:

databricks/koalas/sql.py
94.73% <0%> (ø)

databricks/koalas/namespace.py
90.35% <0%> (+0.17%)
:arrow_up:

databricks/koalas/__init__.py
93.1% <0%> (+0.24%)
:arrow_up:

Continue to review full report at Codecov.

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


GetAndStoreTokens(...).CookieToken seems to always be null

Take the following example:
app.Run(async (context) =>
{
    var antiforgery = context.RequestServices.GetRequiredService<IAntiforgery>();

    // The token.CookieToken is always null
    await context.Response.WriteAsync("
Cookie Token: " + token.CookieToken + "

");
});

I'm not confident as to how many valid scenarios utilize the token.CookieToken but it's a blatant issue nevertheless.

Did a deep investigation of this and the root cause of the issue Is that we always utilize newly created cookie tokens when creating our AntiforgeryTokenSet… This means that after the first request there’s no “new” cookie token and it therefore returns null.

This issue was moved to aspnet/Home#2417


Prove all easy soundness cases

C.f. #113

ok. what are the difficult cases?

Natrec and function cases


Trying to get in touch with you regarding a security issue

Hi there,
I couldn't find a SECURITY.md in your repository and so am not sure how to best contact you privately to disclose the security issue.
Can you add a SECURITY.md file with your e-mail to your repository, so that I know who to contact? GitHub suggests that a security policy is the best way to make sure security issues are responsibly disclosed.
Once you've done that, please let me know so I can ping you the info.
Thanks! (cc @JamieSlome)

See that wiki page for info about how to contact the diaspora* security team securely.

@zidingz - just (cc) for your attention!

This is invalid as per https://huntr.dev/bounties/1624639100675-svbergerem/markdown-it-sanitizer/


Warn that we no longer support FFmpeg 3.x

I'm not sure how, but it would be really nice to do this. See #603 and #604.

Also, it would be nice to update the docs for Ubuntu >= 18.04 LTS as 18.04 comes with FFmpeg 3.4.6.

OOookay. I finally feel compelled that I need to do something. I'm thinking at a minimum adding a compile-time error that checks against the library versions to absolutely assert it is building against the version we think it is.

If it is trivial, e.g. just a couple of constants not working, I'll re-add 3x support. But I don't think it is trivial.

I'd say goodbye FFmpeg 3.x, it's hard enough maintaining the project as is..

@jlaine Thanks for the vote.
I'm just trying to figure out the macOS tests on GA, and then this warning will be rolled out.

I'm closing this, doesn't seem worth the hassle


Return "Temporary Channel Failure" for offline peer

Change Description
This fixes #6603. If a forwarding request cannot be processed because the next hop currently is offline, without this patch lnd  returns a permanent "unknown next peer" error. Instead, if the requested channel is known, with this patch lnd returns a "temporary channel failure".
Steps to Test
Send forwarding request that should go out through an inactive channel (peer is offline).
Pull Request Checklist
Testing

[ ] Your PR passes all CI checks.
[x] Tests covering the positive and negative (error paths) are included.
[x] Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

[x] The change obeys the Code Documentation and Commenting guidelines, and lines wrap at 80.
[x] Commits follow the Ideal Git Commit Structure.
[x] Any new logging statements use an appropriate subsystem and logging level.
[x]  There is a change description in the release notes, or [skip ci] in the commit message for small changes.

📝 Please see our  Contribution Guidelines for further guidance.

I don't understand enough of this code and sadly the tests also didn't help much (I'm used to tests with 10 lines max...). I'm closing this PR as I noticed the actual error is determined in another place, and I'm unable to differentiate between "offline" and "unknown".


Watermill didn't repeat effect

Also the card with the bonus needs to be revealed

The reveal shouldn't actually happen. From kahlia:

If the player decides not to return a card to activate the repeat, why should they reveal the card they drew?
If the player decides to do it, then the card is revealed when it is tucked so I don't think the reveal is really necessary.


In /projects number fields are not consistent

In the returned JSON for /projects, budget, spent, monthly_spend are returned as numbers when they are integers, and strings when they are floating point numbers.
Ideally, they'd all be numbers, as floating point numbers are perfectly acceptable in the JSON standard based on the FSM at json.org

I noticed that budget is defined as a float in the DB and was returning a JSON number regardless if the underlying value was zero or nonzero.
Based on that observation, I got project spent to return as a JSON number by changing the project spent from a decimal to a float in the DB.
Since monthly_spent comes from a method defined on the project model, I modified that method to cast the result as float which resulted in the response returning as JSON numbers for all cases.
The implementation of this can be found on this branch.

@jklarfeld - this should be merged into master now.

Thanks! Let me know when it's redeployed and I'll give it a test.

@jklarfeld - just redeployed to jellyfish dev on heroku.


Monash College added

created monashcollege.txt

@fraserjanetlee First stage of verification passed, official domain is monashcollege.edu.au as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@fraserjanetlee Pull request merged. Thank you!


fix wrong API.  fix missing error reason.

Task linked: DUPLO-10443 [Develop] TF Provider: Not able to apply settings to loadbalancer. getting error " Error applying LB settings for tenant e6dfae73-0828-4903-ac52-1e9c19506905 service 'pvc1': %!s()"


I got Cuda Error when I run test.py  ,And My soft environment is ubuntu16.04 python3.7 cpu

/home/pc/anaconda3/envs/py3.7/bin/python /home/pc/work/yq_work/im2recipe-Pytorch/test.py
test begin!
Traceback (most recent call last):
File "/home/pc/work/yq_work/im2recipe-Pytorch/test.py", line 176, in
main()
File "/home/pc/work/yq_work/im2recipe-Pytorch/test.py", line 36, in main
model = im2recipe()
File "/home/pc/work/yq_work/im2recipe-Pytorch/trijoint.py", line 102, in init
resnet = models.resnet50(pretrained=True)
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torchvision/models/resnet.py", line 261, in resnet50
**kwargs)
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torchvision/models/resnet.py", line 223, in _resnet
progress=progress)
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torch/hub.py", line 506, in load_state_dict_from_url
return torch.load(cached_file, map_location=map_location)
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torch/serialization.py", line 529, in load
return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torch/serialization.py", line 702, in _legacy_load
result = unpickler.load()
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torch/serialization.py", line 665, in persistent_load
deserialized_objects[root_key] = restore_location(obj, location)
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torch/serialization.py", line 156, in default_restore_location
result = fn(storage, location)
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torch/serialization.py", line 132, in _cuda_deserialize
device = validate_cuda_device(location)
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torch/serialization.py", line 116, in validate_cuda_device
raise RuntimeError('Attempting to deserialize object on a CUDA '
RuntimeError: Attempting to deserialize object on a CUDA device but torch.cuda.is_available() is False. If you are running on a CPU-only machine, please use torch.load with map_location=torch.device('cpu') to map your storages to the CPU.
Process finished with exit code 1

Hi,
It seems the cached model was serialized when it was in the GPU. Three options: a) You could manually remove the cached file, and let torchvision download it again; b) You could manually (not ideal) change the map_location when you call the cached file; or c) Since you are loading a trained im2recipe model, you could simply put pretrained to False, i.e. resnet50(pretrained=False).
Best,
Javier

Thanks. I changed the trijoint.py ,set pretrained = False,But ,Iget the Error
File "test.py", line 179, in
main()
File "test.py", line 36, in main
model = im2recipe()
File "/home/pc/work/yq_work/im2recipe-Pytorch/trijoint.py", line 121, in init
self.ingRNN_    = ingRNN()
File "/home/pc/work/yq_work/im2recipe-Pytorch/trijoint.py", line 65, in init
self.embs = nn.Embedding(vec.size(0), opts.ingrW2VDim, padding_idx=0) # not sure about the padding idx
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torch/nn/modules/sparse.py", line 98, in init
self.reset_parameters()
File "/home/pc/anaconda3/envs/py3.7/lib/python3.7/site-packages/torch/nn/modules/sparse.py", line 109, in reset_parameters
self.weight[self.padding_idx].fill_(0)
IndexError: index 0 is out of bounds for dimension 0 with size 0

Check where you put vocab.bin. To provide a different path (default data/vocab.bin) you can indicate it like this: --ingrW2V YOUR/NEW/PATH/vocab.bin


docker-compose: fix typo monitor_workers

flask command should be monitor-workers
Signed-off-by: Tyler Baker<EMAIL_ADDRESS>

applied. you can do a "git reset --hard HEAD~6 ; git pull" to get up-to-date


Update README to reflect that there's no builder API.

There is no builder API in QueueFile in the latest version (1.2.3).

The README is versioned with the code. GitHub doesn't let us show the latest tag automatically, unfortunately.


Where/how can I store EVN data?

My healthcare vendor would provide HL7 EVN message segment to indicate when an event such as A01 (admission), A03 (discharge), A08 (patient update), A04 (referral), and A11 (cancel admission). The event timestamp is generally different from the message header timestamp. So, I need to store the event timestamp as it occurred. Is there an FHIR entity to store EVN data?
Here is a sample of the data. Note the EVN timestamp is different from message header timestamp. Without the EVN timestamp, I would miss the event by 1 day.
MSH|^~&|J3rdsU1S/YU3pMoUwk/9ZQ==|CB99V96UUEJVQA7DK5G6APHVVG|Hospice Source||20210615165034||ADT^A08|20210615165034HEP00001905502|T|2.5||||||ASCII
EVN|A08|20210614111226|20210614111226|01||20210614111226

Recommend posting this question on chat.fhir.org


Mobile Column is Hardcoded in Making the Notifiable Model

Hello @mohammad-fouladgar,
I noticed something that didn't add up after changing my otp config, specifically after changing the mobile column:
TypeError : App\Models\SomeNotifiableModel::getMobileForOTPNotification(): Return value must be of type string, null returned
This is caused by the mobile column being hardcoded in making the notifiable model:
class OTPBroker
{
    .
    .
    .

    private function makeNotifiable(string $mobile): OTPNotifiable
    {
        return $this->userRepository->getModel()->make(['mobile' => $mobile]);
    }
}

Suggested Fix
-        return $this->userRepository->getModel()->make(['mobile' => $mobile]);
+        $mobileColumn = config('otp.mobile_column');
+        return $this->userRepository->getModel()->make([$mobileColumn => $mobile]);

Regards,
Majd

@MajdTaweel Thank you for reporting. Feel free to submit a PR to fix this issue.

This issue is closed by  PR #10.


SmoothLine can't be instantiated via python

Exception TypeError: “object of type ‘NoneType’ has no len()” in "kivy.graphics.vertex_instructions.SmoothLine.build_smooth’ ignored
Example
line = SmoothLine(points = (touch.x, touch.y), width = 2.0, group=g)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/19689732-smoothline-can-t-be-instantiated-via-python?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F42681&utm_medium=issues&utm_source=github).

Was this fixed?

Here's a patch.
index 7356a43..7b746f5 100644
--- a/kivy/graphics/vertex_instructions_line.pxi
+++ b/kivy/graphics/vertex_instructions_line.pxi
@@ -1211,7 +1211,7 @@ cdef class SmoothLine(Line):
     cdef float _owidth

     def __init__(self, **kwargs):
-        VertexInstruction.__init__(self, **kwargs)
+        Line.__init__(self, **kwargs)
         self._owidth = kwargs.get("overdraw_width") or 1.2
         self.batch.set_mode("triangles")
         self.texture = self.premultiplied_texture()

Fixed in https://github.com/kivy/kivy/pull/3545.


really need your help about skia like

Hi, first thanks for great library @ravir-simform.
I want to start show a skia-like with like state = true, but I dont know how to do it
I have tried to set
const SkiaLike = ({
  size = 180,
  onChangeValue = () => {},
}: Partial<SkisLikeProps>): React.ReactElement => {
  const value = useValue<number>(1);
  const [isLike, setIsLike] = useState(true);
  const halfSize = size / 2;
 ...

but it's not working to me, can u help? Thanks so much
P/S: I just want so show red color heart immediately without animation on first time if user liked this message

We appreciate you bringing this to our attention. We would be happy to assist you in any way we can. Please take a look here(https://github.com/SimformSolutionsPvtLtd/react-native-skia-catalog/pull/46) and let me know if you have any further concerns. Thank you!

thanks, I need modify a bit then work like charms!!!
<Paint opacity={isLike ? 1 : opacityHeart}>
        {/* <BlendColor color={heartColor} mode="srcIn" /> */}
        <BlendColor
          color={isLike ? Colors.red : heartColor}
          mode="srcIn"
        />
      </Paint>


[Auto-generated] Check & update dependencies (pyproject.toml)

Update dependencies (pyproject.toml)
Automatically created PR based on the 'CI - Check pyproject.toml dependencies' workflow in SINTEF/ci-cd.

Codecov Report

:exclamation: No coverage uploaded for pull request base (ci/dependency-updates@d3d369b). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##             ci/dependency-updates      #75   +/-   ##
========================================================
  Coverage                         ?   53.69%
========================================================
  Files                            ?        3
  Lines                            ?      311
  Branches                         ?        0
========================================================
  Hits                             ?      167
  Misses                           ?      144
  Partials                         ?        0

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


what a great documentation!

Can you type a little more on the "as follows:" section?
Add filter in the capabilities.json as follows:
The filter entry enables communication of selection-related information between the visual and the hosting application. The entry is used for both the discrete selections and the bulk selections of the Advanced Filter.

I followed this and was getting an error compiling:
TS2306: File '.../node_modules/powerbi-models/dist/models-noexports.d.ts' is not a module.
I removed this file from tsconfig.json and it worked.


Possible bug with Any (or Seq)?

Hi,
First of all - thanks for the great library! I've found a possible bug. I've tried to debug it to find the underlying cause but have struggled a bit. It can be reduced to the following case.
If we have the parser:
str := `str`
str1 := parsify.Seq(`str`, `1`)
str2 := parsify.Any(`str2`)
parser := parsify.Any(str1, str2, str)

I would expect this to be able to parse the string str (for the 3rd argument to the Any) and then try to parse the string str, I get the error goparsify.UnparsedInputError{remaining:"str"} .
My exploration of the code is guiding me to think that the Any from the str2 line is not errorring correctly but I can't see how to fix it.
Do you agree that the parser defined above should be able to parse str? And if so, could you give me a pointer of how I might go about fixing it?
Some additional information:
The exact same parser works with the library at commit 0f854720ca2bad30246020bb01cdb903a5f9406d. I noticed the main thing that has changed in between was the removal of the prediction in the Any function which my colleague suggested a couple of weeks ago. I can't see anything obvious in that commit which would trigger this new behaviour though.

I'm getting a similar error. I also don't understand it, but it seems to have something to do with how error handling interacts between nested Any parsers. I got it to run correctly and still pass all tests by ignoring the existing ps.Error in Any:
https://github.com/brandondyck/goparsify/blob/3e20a3b9244aa003fcaf26a625ef48246aef55f7/combinator.go#L36-L73
I'd submit a PR, but I'm not comfortable with applying fixes I don't understand.


user_info returning false

user_info is returning a boolean, and after authentication, it does not turn true.
running simple app in auth0

Same bug here

The logs are the following

Same bug here. user_info is false although it seems that authentication succeeded.

@charlielito @ofersabo Did you figure it out? I do have the same bug.

@charlielito @ofersabo Did you figure it out? I do have the same bug.

Sadly no, and @conradbez seems to ignore us :(

I believe these issues are fixed part of https://github.com/conradbez/streamlit-auth0/issues/1.  Can you try pip install git+https://github.com/eluzhnica/streamlit-auth0.git@fix-old-universal-login-builds#egg=streamlit-auth0-component and let me know if that works?

I believe these issues are fixed part of #1. Can you try pip install git+https://github.com/eluzhnica/streamlit-auth0.git@fix-old-universal-login-builds#egg=streamlit-auth0-component and let me know if that works?

Nope, same behavior

@charlielito Try doing pip uninstall streamlit-auth0-component first and then pip install git+https://github.com/eluzhnica/streamlit-auth0.git@fix-old-universal-login-builds#egg=streamlit-auth0-component -- that worked for me, even though just installing didn't work.

ip install git+https://github.com/eluzhnica/streamlit-auth0.git@fix-old-universal-login-builds#egg=streamlit-auth0-component

following your instructions it worked! thanks!

I think this is implemented in the latest vanilla js merge and this commit


README: add section on how to integrate with fuse

This adds a short section about how to configure fuse/uno to always pick up the checked-out fuselibs.
I'm not in love with this, but we need this information in some form. Some feedback would be greatly appreciated.
One of the big lacks, is that this doesn't really mention where to put the unoconfig-file, might be confusing to people who aren't used to dealing with unoconfig-files (which is probably most users). But I don't really know what the best advice would be.
We used to have something like this at the root of the repo, which made things work when using Stuff/uno from inside the fuselibs-repo. But that was removed by @mortend when the package-manager was introduced. We could consider adding it back...

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: kusma:x: sebbertYou have signed the CLA already but the status is still pending? Let us recheck it.

@sebbert: Pushed fixes for your input :)


Publish test-results does not find multiple test-results on a Mac when using wildcard

When running on a Mac, the publish test-results step currently does not find (multiple) test-results on the disk when using a pattern.
The step is configured with the following wildcard:
$(Common.TestResultsDirectory)/TestResults-*.xml

When executing, the step states: 2016-06-19T16:24:41.4661450Z ##[warning]No test result files matching /Users/alex/VstsBuildAgent/vsts-agent-osx.10.11-x64-2.102.0/_work/1/TestResults/TestResults-*.xml were found.

Although the files are actually on the disk in the appropriate directory:

Whereas when providing the path to (one) test-result file explicitly $(Common.TestResultsDirectory)/TestResults-UnitTest.xml, the step works as expected.

It's version 1.0.29.

I tried same thing in Mac, it worked. Can you are dropping the Test files in TestResults folder

I am using a command-line step that runs my nunit-tests:
Command: nunit-console
Arguments: $(build.binariesdirectory)/$(BuildConfiguration)/test-assembly/SightPlayer.Ui.Test.dll -xml=$(Common.TestResultsDirectory)/TestResults-IntegrationTest.xml

And as stated above, when specifying the test-results file explicitly, it does work. Only when using the wildcard and checking Merge Test Results it fails. Any way to debug this issue on my side?
Have you tried with Merge Test Results enabled?

I tried to repro this issue again. Used ubuntu agent, but it shouldn't be an issue.

It's successful. Can you try one thing please?
Try to Publish Test Report with Merge enabled from some other directory [/Users/xxx/someFolder/]`?

I've tried to publish the test-results to /users/alex/TestResults and changed the pattern for the Publish Test Results step to /Users/alex/TestResults/TestResults*.xml. However, I received the same error message and no test-results were published: 2016-07-04T16:26:54.4090870Z ##[warning]No test result files matching /Users/alex/TestResults/TestResults*.xml were found.
If the files were not present, why would the step work with the absolute paths given?

@apacha, That's weird. Can you install latest agent and try once. if not, lets have skype call and close the issue. I tried to repro, but it's working without an issue :(
you can drop email to us with skype details: devops_tools at microsoft.com

Ping. There's an alternative way to debug this.
In your build agent machine, you can find publishtestresults.js under tasks folder
Open this file and add this line before tl.ls('-R','/Users/alex/TestResults/')
You can find documentation about this lib here https://github.com/Microsoft/vsts-task-lib/blob/master/node/lib/task.ts

Closing this issue. Please reopen if not resolved


[FEATURE] Decode timestamps

What is the problem you are trying to solve?
JWTs often contain timestamps, typically iat, exp or maybe nbf. These are Unix timestamps but it would be nice if somewhere in the "JSON Web Token" tab, the values would be decoded into human-readable date/time values. Additionally, a certain highlighting could also be used to indicate when the token already expired.
How are you currently being hindered by this problem?
Currently, it is now obvious how long the token is valid or if it already expired. Manually decoding timestamps is cumbersome.
How would you like this problem to be solved?
Maybe similar to how the JWT4B extension does this. But maybe there are better ways.

Hi @exploide, thanks for the suggestion. I think it's a great idea. There's a few ways one could go about it. Some options would be:

Adding a passive scan check that raises informational issues around suspicious iat, exp or nbf values. This would only work in Pro and wouldn't apply to any JWT's within WebSocket messages.
The Proxy history highlighting could draw attention to issues around iat, exp or nbf values. This would work in Community and with WebSocket messages. I guess the downside would be that there's two 'levels' of highlighting here.
Add the info to the editor somewhere. The problem here is where to add this as there's not much real estate left on the panel. You could alternatively have an option to automatically convert the epoch seconds to a human-readable string.

Anyway those are some initial ideas. Please let me know if any of these interest you or if you have any better ones!

Not sure if this would be useful. How to decide what is suspicious. I mean an expired token is odd, but I guess this barely happens during the normal flow. It's more of an issue when you use one token in repeater for some time. That's the reason I would prefer an indication in JWT tab as in option 3. A long validity period might be an issue but I think it depends on the use case, so the pentester needs to decide anyway.
Highlighting a request with an expired token might be an option, but again I don't think this is what I'm actually looking for. The reason is that I still think the issue will mostly occur when using repeater. However, during an active scan, it might also happen that the token eventually expires. Having such a highlighting in the Logger tab (not only in regular Proxy history) would help to see that something happened and the results might be crap. But this is all somewhat optional for me.
This is what I would need the most. When considering to convert the timestamp within the editable editor area, I think you need to be careful to not confuse the reader as it might be a bit opaque that conversion is happening. I would prefer a simple non-editable info below or besides the editor area. With a red marking or something similar when the exp value already expired. But you are right that there is not much space currently. Maybe the editor needs a redesign. The JWT4B extension looks a little bit more clear for example. But I'm not a UX designer and unfortunately don't know how to do it best. :man_shrugging:

In any case, thank you for considering this feature idea :)

@exploide Thanks for your thoughts; your reasoning makes sense.
There's a few options within the editor:

On the bottom right of the panel. There should be enough room here, although it's a little separated from the payload.
You could try and fit something underneath 'Compact JSON' checkbox, although it's pretty restricted with horizontal space
One could add a label within the payload section that's only visible when there's something funky with the time claims. Obviously you lose vertical real estate here, so arguably not great on a laptop.

I guess the first one would be my choice, but I'm not a UX designer either :)

I'm also tending towards the first option.
But I'm opposed to the third option because there are use cases for viewing decoded timestamps even if there isn't anything funky at all. For example I would like to see how long the time frame is where I can reuse the token. Or whether it has an untypically long validity period and I want to report that to the customer.

OK I'll have a go at option 1.
Happy to include messaging if the token validity is unusually long as well. What would be your definition of an untypically long validity period?

Not quite sure. Might depend on some circumstances, whether it is "just" a JWT, whether it is OAuth/OIDC and there also is a refresh token in place... My focus shifted a bit away from web pentesting. Maybe I get a chance to get the opinion from our OAuth expert, don't know.

@JonasPrimbs any opinion on whether it looks feasible to decide if a JWT token's validity is unusually long or does it always "depend"? Context is if a programmatic check in the extension would make sense.

In context of an OAuth Access Token or an OIDC ID Token, a JWT should not be valid for longer than 1 hour (3600s). In case of critical environments (banking, critical infrastructure) it should be even less (about 10 minutes). But this is just a personal recommendation. Afaik, there is no official document which specifies this explicitly. They always describe the access token validity as "short-lived".
But JWT is a very universally used format.
For example: In context of OpenID for Verifiable Credentials, JWTs are used as a digital replacement for id cards which should be usable offline. In that case, JWTs would be valid for at least 24 hours, typically for a week, and sometimes even for years.
But those JWTs can be identified with their "typ" parameter in the JOSE header, e.g. "vc+sd-jwt" see here for Verifiable Credentials, while access tokens should have the "typ" "at+jwt" see here.

@exploide I've added a warning label when any of 'iat', 'exp' or 'nbf' are invalid. I'm sure it can be improved, so let me know what you think.
I've not added any logic around 'exp' dates in the future for now.

Thank you for the effort implementing this!
I built the extension from source and had a quick look. If I see correctly, you implemented a warning label but no actual decoding of the timestamps? Being able to read them in human-readable format is what I was actually asking for. Sorry if there was a misunderstanding. I just need a convenient way to see what dates and times are represented by the timestamps.
Warnings about invalid values are nice to have, but not that essential for me. However, combining it would actually be really nice, e.g. by highlighting the human-readable datetime in red if expired or something like this.
Example on how the JWT4B extension is doing it:

This is what I would need :)
Side note: During testing I stumbled upon a page delivering the claim "iat": "2020-04-24T19:14:06.967" and jwt-editor reported iat value is invalid. I think it is uncommon to not have epoch seconds in the claim but it might happen and I don't know whether this should really be considered invalid.
Another side note concerning the visual representation: I found the page https://token.dev/ where they implemented the decoding via hover tooltip over the timestamp. And if it is expired, it gets a red underlining. But I don't know if this is fun to implement in Burp.

Thanks for the feedback. It's pretty easy to decode the values and display the dates from where we are currently, but we're kind of back to the start in terms of real estate on the panel, i.e. where to put them :)
Based on the RFC (for what they are worth!),  "iat": "2020-04-24T19:14:06.967" would be considered invalid.
Tooltips and underlining for the payload panel would be a fair bit of work. I think it would require a JWT payload specific lexer instead of using a generic JSON one. I've written a few of these in the past, so it's doable, but not a five minute job.

@exploide I'm pretty familiar with the RSyntaxTextArea code so if you do want to go down the route of underlining and tooltips then I can definitely point you in the right direction.

Based on the RFC (for what they are worth!), "iat": "2020-04-24T19:14:06.967" would be considered invalid.

I also had a look on the RFC and I think it is not that clear. For the exp claim it is stated that

Its value MUST be a number containing a NumericDate value.

And then the NumericDate section states:

This is equivalent to the [...] "Seconds Since the Epoch", in
which each day is accounted for by exactly 86400 seconds, other
than that non-integer values can be represented.  See RFC 3339
[RFC3339] for details regarding date/times in general and UTC in
particular.

The unclear statement is that "other than that non-integer values can be represented" and then the linked RFC3339 gives examples of Internet date/time format like 1985-04-12T23:20:50.52Z.
But maybe I misunderstand something. And we should not overthink it, this was just a side note. Feel free to mark it as invalid as long as no more complaints are coming :D

if you do want to go down the route of underlining and tooltips then I can definitely point you in the right direction

Nah, this was just some idea, I don't really require something that fancy.

Thanks I missed the bit about non-integer values. As you suggest that can wait for now :)
So back to the main event. We don't have enough room at the bottom of the panel, so how about another checkbox in the payloads panel to convert iat, nbf, exp epoch times to date strings? One could add a global default in the config. I'd probably leave it off for starters.

We don't have enough room at the bottom of the panel, so how about another checkbox in the payloads panel to convert iat, nbf, exp epoch times to date strings? One could add a global default in the config.

I have some usability concerns regarding another checkbox/option. The less knobs the user has to bother with, the better. "Configurability is the root of all evil."
I would rather go with some layout changes. As I already mentioned, I'm not a UX designer but some ideas are:

The signature textarea currently spans the whole width but only displays 16 bytes in a row. There is space on the right hand side. So maybe shorten the width of the signature textarea and show the datetimes to the right. Similar to the JSON formatting elements above. This might already be enough to solve the problem.
Make the whole tab scrollable. Then there is arbitrarily much room available :D Whether it would look good might depend on the size of the user's screen. The datetimes could then either be displayed somewhere at the bottom, or to the right side of the payload area when that sections gains a bit more height per default.

Thanks for your thoughts. They are really useful!
I hear what you're saying about yet another config option.
The space to the right of the JWS signature is definitely an option. The only downside I can think of is that it's going to be empty if there's no dates in the payload.
I'm not mad about more scrolling. Plenty of people use Burp on laptops with small screens.
I'll have a go with adding a new panel to the right of the signature.

@exploide It's not totally finished, but if you get a chance, can you have a look and see if this is more what you're after? All feedback welcome.

Yes, this is what I had in mind :) Thank you very much for your ongoing effort.
Feel free to close this issue once you are done.

@exploide Excellent! Thanks for your patience. I'll make a new version with these changes in then email PortSwigger to get them into the BAppStore.


🍒 Cherry pick PR #7255 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/7255 to staging 🍒

🚀 Cherry-picked to staging by @roryabraham in version: 1.1.30-2 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @roryabraham in version: 1.1.30-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Issue: Overlapping QR and Waves

Description
When I use the shortener on my phone and I generate the QR Code, the text to download it is overlapped by the waves, so it's not clickable, and difficult to read.
Screenshot

Mobile:
iPhone 11 (iOS 15.6.1)
Browser:
Safari 15

Done by https://github.com/origranot/url-shortener/pull/199


tokenProviderFunction doesn't work as expected.

In order to balance application users api call, we have 5 application users.
ServiceClient is initialized with bellow constructor
var client = new ServiceClient(new Uri(_authProvider.AuthDataConfiguration.Resource),_authProvider.GetBearerToken, false, _logger);
authProvider will cache the access token in concurrentdictionary
public async Task<string> GetBearerToken(string Resource) {   var key = 0;   bool locked = false;   try   {     Monitor.TryEnter(_padlock, maxWaitForClientMs, ref locked);     if (locked)     {       _currentIndex = (_currentIndex + 1) % AuthDataConfiguration.ClientCredentials.Count;       key = _currentIndex;     }   }   finally   {     if (locked)     {       Monitor.Exit(_padlock);     }   }   if (!_tokenCache.TryGetValue(key, out var item) || item.ExpiresOn < DateTime.UtcNow.AddMinutes(5).ToUniversalTime())   {     item = await RequestToken(       AuthDataConfiguration.Authority,       AuthDataConfiguration.Resource,       AuthDataConfiguration.ClientCredentials[key]);       _tokenCache.AddOrUpdate(key, item, (key, oldValue) => item);   }   _logger.LogDebug($"Index of bearer token is {key}");   _logger.LogDebug(item.AccessToken);   return item.AccessToken; }
I sent 10 requests and from application log the token is provided evenly twice each.
But with Telerik Fillder these ten requests doesn't use those auth token evenly. only use 3 access tokens and count is 2,4,4
And because of that when I tested in staging environment with massive requests "Over 10000 limits" will appear in logs

I also tried another way, create 5 ServiceClient with different credentials and use them in turn, same result.
two credentials were heavily used and the other 3 credentials much less.

The client itself does not understand the threading scenario your using. In the scenario that you're using it in, how it's going to load balance is driven based on client instance you're using in each thread.
I would recommend taking a look at how the client is being selected for each thread

Thanks MattB,
It is nothing to do with thread, the ServiceClient factory is register as singleton and every D365 webapi call will apply for ServiceClient credentials via factory, factory will rotate the credentials to make sure it is absolute even.
We built this logic since crmsdk, from old dataverse analyst report the application users are used for nearly same count.
I have updated the sample code in github, Please follow bellow steps.

please provide multiple credentials in settings.
Start Telerik Fiddler
Run MultipleApiCall.exe
Compare the app.log and Fiddler log bearer token, you will see they are not matching( I mean count not sequence).

https://github.com/heroes3lover/MultipleApiCall/tree/users-mattb-variant1
Thank you very much.
This is critical issue for us, because even limit solved inside component by retrying, it will impact the caller and potential causes timeout. We need the call succeed at first place.

Looking at your code,
You're trying to change the auth token per request,  this is not a great thing to do here... as its going to show up as incremental data being modified by different users on the same record.  but its your choice.    As long as all the accounts are identical permission wise, it will work.
I see your incrementor,
If your trying to counter check calls to get access token vis network traffic, your not going to be able to match them up directly as the ServiceClient will access the token as part of accessing its internal service references, not only when its going to make a call, this was the same behavior as CrmServiceClient.
We just ( as in last Friday) shipped an update to change the behavior of that to be a bit more controlled, reducing the times the service client asks for the token, though it still does not match up 100% with the request going out.

Hello Matt, thanks for the response.
These 5 credentials are created only for this app service, so it doesn't matter which one is applied, I think this business logic was built together with developers from Microsoft. The reason is the service is very busy and it can reach 23k requests in 5 minutes sometime.
Correct, I can see the ServiceClient will access the token as part of accessing its internal service references,
but after you run the MultipleApiCall.exe and check app.log, you will see the function tokenProviderFunction are called 10 times exactly same as requests count. nobody else request the token and rotate the incrementor.
2023-06-15 09:31:17.656 +10:00 [DBG] Index of bearer token is 2
2023-06-15 09:31:17.657 +10:00 [DBG] Index of bearer token is 3
2023-06-15 09:31:18.347 +10:00 [DBG] Index of bearer token is 4
2023-06-15 09:31:18.411 +10:00 [DBG] Index of bearer token is 0
2023-06-15 09:31:18.428 +10:00 [DBG] Index of bearer token is 1
2023-06-15 09:31:18.466 +10:00 [DBG] Index of bearer token is 2
2023-06-15 09:31:18.495 +10:00 [DBG] Index of bearer token is 3
2023-06-15 09:31:18.512 +10:00 [DBG] Index of bearer token is 4
2023-06-15 09:31:18.548 +10:00 [DBG] Index of bearer token is 0
2023-06-15 09:31:18.615 +10:00 [DBG] Index of bearer token is 1
but for those 10 "POST /XRMServices/2011/Organization.svc/web?SDKClientVersion=<IP_ADDRESS>65 HTTP/1.1",
I can see one token was used 3 times.
The only thing I can imagine is token of some Post requests are not got via calling tokenProviderFunction  function.
Correct me if I am wrong.
Not sure how much impacts, I will compile one version and deploy to staging for testing.
The dataverse analyst report will show the balance of usage of application users.
Thanks again for performance issue I raised before.
Since I use IOrganizationServiceAsync2 of 1.0.39, the performance looks pretty good in Staging environment now.
Since 1.1.9 CPU usage a bit higher, not sure if version related. sometime hits 69%.

Morning Matt, above is one day data, if token usage like that, I would say there will be so many over limits error in staging/Production.

This is our current production, what is doing now is rotate the clientid and secret and create new crmclient for each request.

Surprised, I checked report for yesterday.

So the logic you wrote seems to be rotating the token used now?
if so,. lets close this one up.
thanks

Hello Matt, I will deploy to staging to see how it goes.
Staging is very busy one, the report will be more useful.
I will let you know the result.
Thanks for all the help.

Ok, im going to close this for now, Please reopen if there is something we can help you with.
Good luck with your rollout.


fix: Add a prompt to "forge remove" to prevent accidental loss of the…

… working tree

Motivation
Lost all my work after running "forge remove"

Solution
Add a prompt for the user to confirm action and warn of the consequences

Fixes #673


doc: Use correct href for nuscht-search

This commit also adds a subdirectory in the docs derivation, to have the same local structure than what is uploaded on github pages.

@Mergifyio queue


Qualify column name in SELECT statment??

I am trying to use the JOIN query, but I am getting an "ambiguous column name: id" error. I am guessing this is because I am selecting the "id" column of a table, and every one of my tables has an "id" column as a primary key. If I don't select the "id" field, my JOIN statement works fine. If I leave the .select() empty and let it return all of the columns, the "id" column does not have the correct ids.
Is it possible to qualify my column names in the select statement with the table name?
It is currently:
SQLite.select(MyTable_Table.id).from(MyTable.class)....
I would need to do something like this:
SQLite.select(MyTable.MyTable_Table.id).from(MyTable.class)....

You would have to use
SQLite.select(MyTable_Table.id.withTable()).from(MyTable.class)...
but the problem is that the withTable() call is currently broken as per my tests. Submitted a pull request for the same but it might take a while to be approved

Awesome, thanks! I hope this will get fixed soon.

@marwfair Its fixed in the develop branch

@shanrod220 Thank you!

IProperty prop = (MyTable_Table.id.withTable()).from(MyTable.class);
and I get this error: cannot find symbol method from(Class)]]any help?

I use DbFlow 3.1.1 .


[Export] Add support for adding shebang to the PCK files.

Fixes https://github.com/godotengine/godot-proposals/issues/9066
Not sure how useful it is for Linux distros, but can be a convenient way to run local tools.
Also, should improve embedded PCK robustness by removing global file base offset.

I'm also not convinced about the validity of this use case. It seems to be a pretty uncommon way to achieve what seems to be the use case in the proposal, I'll comment further there.

I guess I'll split the useful part (Also, should improve embedded PCK robustness by removing global file base offset.) to the separate PR. Which also allows this use case as well (just without an option in the export dialog).


feat: support aapt optimize

See https://developer.android.com/tools/aapt2#optimize, it will shrink apk size and no side effect.

Please open this to upstream if at all, and then we can merge the changes here, thanks


docs(license): update attribution

Codecov Report

Merging #15 (a3f0a19) into main (20e4fc9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   99.51%   99.51%
=======================================
  Files           3        3
  Lines         207      207
  Branches       66       66
=======================================
  Hits          206      206
  Misses          1        1

Continue to review full report at Codecov.

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


Upgrade react-grid-layout

Fixes https://github.com/STRML/react-grid-layout/issues/560

Thanks!

@STRML would it be possible to publish a new/patch version to npm?

That's not required; the newest version of react-resizable matches the old constraints. You may want to run a new install or regenerate your own lockfile.

the bit that's weird is I then need to add an artificial "react-draggable": "^2.2.6" dependency into my package.json just for this, and it'll eventually become stale.
It turns out this console warning isn't egregious, so I'll survive until you publish, but it still seems that publishing is the "right thing to do" from react-grid-layout's perspective

It's not, that's the whole point of these version ranges. Delete the entry
from your lockfile or shrinkwrap file and the package will update.
On May 23, 2017 7:14 PM, "Thomas Hasner"<EMAIL_ADDRESS>wrote:

the bit that's weird is I then need to add an artificial "react-draggable":
"^2.2.6" dependency into my package.json just for this, and it'll
eventually become stale.
It turns out this console warning isn't egregious, so I'll survive until
you publish, but it still seems that publishing is the "right thing to do"
from react-grid-layout's perspective
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/STRML/react-grid-layout/pull/561#issuecomment-303568240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJFPyEoNRIJcrDVECEuHBFgKWuu7C-dks5r827cgaJpZM4NhEs4
.

I wasn't aware that npm would default to the newest compliant version.
NPM's semver docs on Caret Ranges indicate that newer versions are "allowed". (There won't be a conflict if someone else requires a newer version in range)
Anyway, clearing yarn.lock and reinstalling did the trick. Thanks.

@STRML @thasner
I'm still getting the errors as referenced also by my issue #556
I've made a webpackbin https://www.webpackbin.com/bins/-Kjwr_o7tLT1JAfz9ujE that reproduces the error and you can check it out :+1: (you can check or change the versions on the Cog - top right)
I'm also having a perf issue because of this issue.
My lockfile:
<EMAIL_ADDRESS><EMAIL_ADDRESS>  version "2.2.6"

<EMAIL_ADDRESS>  version "1.7.0"

<EMAIL_ADDRESS>  version "0.14.6"


Question: would it be possible to share files on a ZeroTier virtual lan or is it technically not possible ?

This would be great to transfer files when you cannot connect directly to wlan but you can access it via zerotier over LTE connection, but I'm not sure if it's doable. Thanks.

It is possible

Thanks @zillydev for the good news.
I just tried activating zt on a new ZT network ID both on my pc and smartphone but I suppose I have to change the multicast IP to the one that zerotier uses.
...after reading this I suppose I should test a newer version on Windows 10 (I'm still on version 1.6.6 , last version available on Win 7). What's the version you've successfully tested the ZT use case ?
I made a second test: I have an sbc (an Orange Pi) with the latest ZT docker image and I set my lan bridge according to this link with this specific network ID. So I turned off the ZT client on the pc and I can still ping my lan devices from my android smartphones. In this second case I tried adding multicast routing <IP_ADDRESS>/24 -> <IP_ADDRESS> (MySBC_ZT_IP) in the "Managed routes" web GUI but it still didn't work.
It would be the best choice to use LocalSend's default settings and set ZT to accept them, so to avoid having to change settings when switching from LAN to ZT or viceversa.
I'm not sure if what I've done with Managed route's rule is correct and if I should (and how to) enable the multicast routing in the SBC host, or in the docker container, or in both...
Unfortunately I found a few other posts on the net on the topic, with many questions and not so many useful replies...
If you have any hint/advise to share it would be greatly appreciated.
Cheers,
V.

JFYI, after more than a year I finally made it !
...thanks to a lot of reading and some guidance from ChatGPT.
My ZT setup consists in an SBC (an OrangePI) with the ZT client and IPtables set so that I can access the LAN hosts from ZT's virtual LAN. You can find instructions about this here  and here.
The extra configs that made me able to detect the LAN hosts from my smartphone connected to ZT bridged network were writter with the following commands:
#Allow Multicast
sudo iptables -A INPUT -d <IP_ADDRESS>/4 -j ACCEPT

#Allow Multicast Discovery (UDP) on port 5353 (mDNS) to discover devices.
sudo iptables -A INPUT -p udp -m udp --dport 5353 -j ACCEPT
sudo iptables -A INPUT -p udp -m udp --sport 5353 -j ACCEPT

#Application Port 12345 for both TCP and UDP communications:
sudo iptables -A INPUT -p tcp --dport 12345 -j ACCEPT
sudo iptables -A INPUT -p udp --dport 12345 -j ACCEPT

#Broadcast Communication (UDP) for discovery traffic.
sudo iptables -A INPUT -p udp -m udp -d <IP_ADDRESS> --dport 12345 -j ACCEPT

#Forwarding Multicast need to traverse the ZeroTier network:
sudo iptables -A FORWARD -p udp --dport 5353 -j ACCEPT
sudo iptables -A FORWARD -p udp --dport 12345 -j ACCEPT

#Save iptables rules
sudo iptables-save > /etc/iptables/rules.v4


fix(versions): update Activiti/activiti-cloud-full-chart-security versions into master

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.82

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.82


Error after pulling the latest Docker container

After pulling latest, when i typed in Baysor it gives me this error:
"ERROR: Unable to find compatible target in system image."

Hm, right, there is a problem with target CPU architecture for pre-compiled julia binaries, which can't be detected on local tests. I'm working on this.

@asterwardhani , I tried to fix it. Can you please test the latest version?
docker pull vpetukhov/baysor:master

@asterwardhani , I tried to fix it. Can you please test the latest version?
docker pull vpetukhov/baysor:master

Looks like it is working now..thanks!!


Improving experimental feature handling

[x] ee stack switches for dependency management e.g. ee stack install --hhvm, ee stack install --pagespeed
[x] Delayed dependency installation. e.g. do not install hhvm till ee site create example.com --hhvm is called.
[x] A clear warning and prompt e.g "HHVM is experimental feature and it may not work with all plugins all your site. You can disable it by passing hhvm=off later. Do you wish to enable HHVM now for example.com? Type "yes" to continue. [no] __". This prompt can be skipped by adding --experimental flag to original command.

Ref: http://community.rtcamp.com/t/installation-process-for-hhvm-fails-causing-ee-installation-to-halt/4605

Fixed in EasyEngine 3.2.1


chore: bdd-spring-1587082943 to 0.0.1

chore: Promote bdd-spring-1587082943 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


rule ban:  Hot to set an option for an array?

Question

TSLint version: 5.10.0
TypeScript version: 2.7.2
Running TSLint via: (pick one) VSCode

TypeScript code being linted
function getDefaultState () {
    return {
        isLoading: false,
        page: CoreUtils.find('p1', 1)
    };
}

function get(relatives: Relative[]){
    const relative = relatives ? relatives.find(element => element.guid === Id);
}

with tslint.json configuration:
 "ban": [
            true,
            "eval",
            {"type": "array", "name": ["*", "find"], "message": "Use a regular for  instead."}
        ]

Actual behavior
getDefaultState: Calls to '.find' are not allowed. Use a regular for  instead.
get:  Calls to '.find' are not allowed. Use a regular for  instead.
Expected behavior
get:  Calls to '*.find' are not allowed. Use a regular for  instead.
Hot to set an option for an array?


OAUTH-3474 include serverPublicKey in the mobile app configuration

Heey!
I'm still getting updates on the changes there ^^
last email I received was from 8h ago (15:15 today).
It would be worth to take care of that somehow
Sz
wt., 2 cze 2020 o 15:15 Stein Welberg<EMAIL_ADDRESS>napisał(a):

@steinwelberg requested changes on this pull request.
In util/config.go
https://github.com/Onegini/onegini-sdk-configurator/pull/27#discussion_r433844601
:

@@ -71,7 +77,7 @@ type NS struct {

type OneginiPreferences struct {
RootDetectionEnabled  *bool json:"root-detection-enabled,omitempty"

DebugDetectionEnabled *bool json:"debug-detection-enabled,omitempty"

DebugDetectionEnabled *bool json:"debug-detection-enabledserver_key,omitempty"

is the stuff between the backticks generated? It seems like 2 fields have
been concatenated here: debug-detection-enabledserver_key
In util/config.go
https://github.com/Onegini/onegini-sdk-configurator/pull/27#discussion_r433845810
:

@@ -71,7 +77,7 @@ type NS struct {

type OneginiPreferences struct {
RootDetectionEnabled  *bool json:"root-detection-enabled,omitempty"

DebugDetectionEnabled *bool json:"debug-detection-enabled,omitempty"

DebugDetectionEnabled *bool json:"debug-detection-enabledserver_key,omitempty"

IMO this change was accidental and it should be reverted to:
debug-detection-enabled
In util/keystore.go
https://github.com/Onegini/onegini-sdk-configurator/pull/27#discussion_r433846812
:

@@ -41,7 +41,7 @@ func CreateKeystore(config *Config) {

keystorePassword := generateKeystorePassword(2048)
bcprovPath := restoreBcprov()

keytoolPath := findKeytool()

keytoolPath := findKeyTool()

nit: Actually keytool is a single word as it is a name, therefore it was
named findKeytool 😉 . It's fine to leave it as it is now though.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Onegini/onegini-sdk-configurator/pull/27#pullrequestreview-422635121,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWOUJB6AZTTRQHTGGFOHRLRUT3QDANCNFSM4NN65G5A
.


go test ./src/client errors

@svekars ran into these errors when trying to test src/client. I'm not sure if the new version of go catches errors that our containerized version is missing, but I wanted to document these because some of them seem like they're probably worth fixing:
go test ./src/client/
# github.com/pachyderm/pachyderm/src/client
src/client/pfs.go:585:7: github.com/pachyderm/pachyderm/src/client/pkg/grpcutil.ReaderWrapper composite literal uses unkeyed fields
src/client/pfs.go:612:7: github.com/pachyderm/pachyderm/src/client/pkg/grpcutil.ReaderWrapper composite literal uses unkeyed fields
src/client/pfs.go:865:7: github.com/pachyderm/pachyderm/src/client/pkg/grpcutil.ReaderWrapper composite literal uses unkeyed fields
src/client/pfs.go:662:2: the cancel function is not used on all paths (possible context leak)
src/client/pfs.go:668:3: this return statement may be reached without using the cancel var defined on line 662
src/client/pfs.go:782:2: the cancel function is not used on all paths (possible context leak)
src/client/pfs.go:788:3: this return statement may be reached without using the cancel var defined on line 782
src/client/pfs_new_storage.go:91:2: unreachable code
FAIL	github.com/pachyderm/pachyderm/src/client [build failed]
FAIL

Note that these are go vet errors, not go build errors, to repro them run:
go vet ./src/client

I'm a little confused why go test ran go vet, maybe that's a go version difference or something, but if looking to fix these messages you can repro them with the command above. And it may make sense as a more general project to add go vet to CI for all of our code (and fix the errors it uncovers.)


feat: add Series.__contains__

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[x] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Closes #1469

Checklist

[x] Code follows style guide (ruff)
[x] Tests added
[x] Documented the changes

If you have comments or can explain your changes, please do so below

ah perhaps this comment i left didn't help

that would still be better than the status quo

by 'status quo' i meant the main branch, not the previous commit in this pr


US-ASCII (ArgumentError) error after updating to 3.4.x

Running failing tests would raise strange error:
$ bundle exec rspec spec/user/user_spec.rb:257
3 examples, 3 failures

Failed examples:

rspec ./spec/user/user_spec.rb:257 # User ./signer should be possible to re-sign after attempt with invalid pin code
rspec ./spec/user/user_spec.rb:257 # User ./signer should be possible to re-sign after attempt with invalid pin code
rspec ./spec/user/user_spec.rb:257 # User ./signer should be possible to re-sign after attempt with invalid pin code

/Users/brbrr/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/rspec-core-3.4.3/lib/rspec/core/source.rb:23:in `split': invalid byte sequence in US-ASCII (ArgumentError)

        from /Users/brbrr/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/rspec-core-3.4.3/lib/rspec/core/source.rb:23:in `lines'
        from /Users/brbrr/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/rspec-core-3.4.3/lib/rspec/core/formatters/snippet_extractor.rb:50:in `expression_lines'

I'm running single example which have before :example & :context hooks
with rspec 3.3.x everything works smooth.

Does spec/user/user_spec.rb contain non-ascii characters?  Have you tagged the file with # encoding: utf-8 at the top of the file so Ruby knows to treat it as UTF-8?  What are Encoding.default_internal and Encoding.default_external in your environment?

[1] pry(#<RSpec::ExampleGroups::User::Signer>)> Encoding.default_internal
=> nil
[2] pry(#<RSpec::ExampleGroups::User::Signer>)> Encoding.default_external
=> #<Encoding:US-ASCII>

adding # encoding: utf-8 not fixing that issue.
but yeah, I got file(not a spec) where I had russian comment.
So it's resolved for me.

If you set Encoding.default_external = "UTF-8" it should also fix it.  I don't think there's any good reason to have your default_external encoding set to US-ASCII, especially because all valid ascii is valid UTF-8.

@myronmarston It's a bit weird but it was the default at one point, this must be an artifact of his environment, but we should probably still fix this literal.

I'm using English MAC OS 10.11.3 (I was switched from Russian language some time ago).
rbenv with
$ ruby -v
ruby 2.2.0p0 (2014-12-25 revision 49005) [x86_64-darwin14]

Not sure what more to say regarding my env.

Can you run env and list your environment vars?

$ env
MANPATH=/Users/brbrr/.nvm/versions/node/v5.1.0/share/man:/usr/local/share/man:/usr/share/man:/opt/X11/share/man:/usr/local/MacGPG2/share/man:/Library/Developer/CommandLineTools/usr/share/man
TERM_PROGRAM=Apple_Terminal
ANDROID_HOME=/Users/brbrr/Documents/Development/Utils/android-sdk-macosx
SHELL=/bin/bash
TERM=xterm-256color
TMPDIR=/var/folders/6g/r7_x32mn4456n4z_5mdl4n4r0000gn/T/
NVM_PATH=/Users/brbrr/.nvm/versions/node/v5.1.0/lib/node
Apple_PubSub_Socket_Render=/private/tmp/com.apple.launchd.DuzrhaHA41/Render
TESSDATA_PREFIX=/usr/local/share/
TERM_PROGRAM_VERSION=361.1
TERM_SESSION_ID=9FA51CF7-EF40-4B35-AC72-31AB00BB2A04
NVM_DIR=/Users/brbrr/.nvm
USER=brbrr
SSH_AUTH_SOCK=/private/tmp/com.apple.launchd.os0Kv6YwUw/Listeners
__CF_USER_TEXT_ENCODING=0x1F5:0x0:0x0
LSCOLORS=ExFxBxDxCxegedabagacad
PATH=/Users/brbrr/.nvm/versions/node/v5.1.0/bin:/Users/brbrr/.rbenv/shims:/usr/local/bin:/Users/brbrr/.rbenv/bin:/usr/local/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/usr/local/MacGPG2/bin:/usr/local/smlnj/bin:/usr/local/share/npm/bin/:/Users/brbrr/Documents/Development/Utils/android-sdk-macosx/build-tools:/Users/brbrr/Documents/Development/Utils/android-sdk-macosx/platform-tools:/Users/brbrr/Documents/Development/Utils/android-sdk-macosx/tools
NVM_NODEJS_ORG_MIRROR=https://nodejs.org/dist
PWD=/Users/brbrr
JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk1.8.0_60.jdk/Contents/Home
ANDROID_SDK=/Users/brbrr/Documents/Development/Utils/android-sdk-macosx
XPC_FLAGS=0x0
PS1=\[\033[36m\]\u\[\033[m\]@\[\033[32m\]\h:\[\033[33;1m\] \W \[\]$git_branch\[\]$git_dirty\[\]$
XPC_SERVICE_NAME=0
GITAWAREPROMPT=/Users/brbrr/.bash/git-aware-prompt
SHLVL=1
HOME=/Users/brbrr
LOGNAME=brbrr
LC_CTYPE=UTF-8
NVM_BIN=/Users/brbrr/.nvm/versions/node/v5.1.0/bin
NVM_IOJS_ORG_MIRROR=https://iojs.org/dist
_=/usr/bin/env

Does ruby default to US-ASCII if LANG is not defined?

That's exactly what's happening:
~$ ruby -e 'puts Encoding.default_external'
UTF-8
~$ echo $LANG
en_AU.UTF-8
~$ unset LANG
~$ echo $LANG

~$ ruby -e 'puts Encoding.default_external'
US-ASCII


Proper way to handle large state objects?

I was experiencing some slowness in my forward function, so I tried using the experimental visualization tool to debug it. One part of it in particular stuck out:

It goes on even further to the right. Up close, that blue line is a giant cond:

I checked my code, and I narrowed it down to this cond. Commenting it out removes that weird bit from the graph and speeds up the function's execution by ~20%.
graph_shape, current_index_shape, edge_counts_shape = graph_init_info
graph = hk.get_state('graph', shape=graph_shape, init=jnp.zeros)
current_index = hk.get_state('max_index', current_index_shape, dtype='int32', init=jnp.zeros)
edge_counts = hk.get_state('edge_counts', edge_counts_shape, dtype='int32', init=jnp.zeros)

# Reset the graph when it grows too large
graph, max_index, edge_counts = hk.cond(
    current_index > max_graph_nodes,
    lambda _: (jnp.zeros(graph_shape), 0, jnp.zeros(edge_counts_shape, dtype='int32')),
    lambda _: (graph, current_index, edge_counts),
    operand=None,
)

graph and edge_counts are very large ndarrays. current_index is a counter that's incremented each time the function's called, and max_graph_nodes is just a static int. (Basically, I'm trying to re-initialize the graph object once every few thousand times through the function.) graph stores values output by some of the transformers in my model, which I'm guessing is why you can see them feeding into the cond in the graph.
I could understand it if JAX was just compiling around these objects in a weird way, but the fact that the function slows down so much when adding a cond, which I don't expect to do much of anything too often, makes me think that I'm doing something wrong with the haiku state. Are there any special considerations that need to be taken with these objects to avoid the compiler behaving this way?

Hi Trevor, hk.cond passes all Haiku state in and out of the cond to allow module parameters and state to be created/updated inside the branches. In this case, it looks to me like your branch functions do not actually call into Haiku modules, so you can probably just use regular jax.cond (which will not have all the extra operands).
JAX made a change earlier this year known as "omnistaging", I think this may allow us to just pass state out of the cond (rather than in). It is possible we could optimise the branch functions to also only return the updated state from the cond too. I'll take a look later this week. This meant if you did have to use hk.cond (e.g. because you used Haiku modules inside the branch functions) we could make them a lot more minimal.

Thanks for taking a look, @tomhennigan . I had switched everything in the transformed function to the haiku version, but I'll avoid doing so where I can for now. Switching it over to the jax.lax cond does remove the wild appendage I saw in the graph; however, the speed remains the same: I still see about a 20% speedup when removing the cond vs. keeping the jax or haiku conds there.
I'm not sure why it should be such a bottleneck - it's at the beginning of the function, and everything after it is just passing stuff forward through some transformer modules, at the moment. Does that seem like something I should expect, or is it worth looking into more? If you think that question's a better fit for discussion on the JAX repo, I can move it over there.

That is odd, if the issue is not Haiku specific you may find better answers on the JAX repo (there are more people looking there).
One suggestion I would have is to try using a jax.lax.select(pred, true_val, false_val) instead of a cond(pred, true_fn, false_fn). We use this in our mixed precision examples (e.g. here) since we have a very cheap branch (does basically nothing) we rarely need to take and most of the time we take the expensive branch. So it ended up being faster to always compute both branches and then just select one output based on a boolean predicate.
AFAIK this is faster because a real cond (e.g. only execute one of the branches) on GPU requires a roundtrip of data from the GPU to the CPU in order to decide whether to take the LHS or RHS branch. While a select can be entirely computed on the GPU.

That's very useful info, thanks! Leaving this issue open since you mentioned looking at the JAX omnistaging option, but feel free to close it whenever you're ready.


Autodetect repository type

It'd be nice to have not to specify the repository type, and auto-detect it with seeing whether Python files or JS files are present.

Both file types can be used in any repository for any purpose (like maintenance tasks) ?

Well I guess you're right. You could still check whether there was a file with the name of the repository in dist or the root, and whether there was any content in custom_components. It would default to auto-detect, but you could override it if you wanted.

That level of guessing I'm not sure I want to do, this is something that is set 1 time pr repo, it does not change :shrug:

Makes sense.


Added batch delete test

Changes Unknown when pulling 2872ab3db1f6fefbed48c8a26ad313e9fe3ba4f3 on v0.5-Yong-Quan into ** on master**.


Some operations (hover, link...) failing for external files

Let's make sure issue is not already fixed in snapshots first.

[x] I verified I can reproduce this issue against latest snapshots build

Steps to reproduce
From a fresh installation and clean workspace:

Empty workspace

I tried

File > Open > browse to a dummy .java file (plain JDK, no dep to external libs)
Use completion to generate a hashCode method and generate it
Try to get hover documentation for this method

I expected: hover doc
But got: nothing
Here is some relevant log output form (<workspace>/.metadata/.log)
Log communication with JDT-LS mentions an error
{"jsonrpc":"2.0","id":"306","error":{"code":-32603,"message":"Internal error.","data":"java.util.concurrent.CompletionException: java.lang.NullPointerException: Cannot invoke \"org.eclipse.jdt.core.IType.getClassFile()\" because \"type\" is null\n\tat java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:315)\n\tat java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:320)\n\tat java.base/java.util.concurrent.CompletableFuture$UniApply.tryFire(CompletableFuture.java:649)\n\tat java.base/java.util.concurrent.CompletableFuture$Completion.exec(CompletableFuture.java:483)\n\tat java.base/java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:387)\n\tat java.base/java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1312)\n\tat java.base/java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1843)\n\tat java.base/java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1808)\n\tat java.base/java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:188)\nCaused by: java.lang.NullPointerException: Cannot invoke \"org.eclipse.jdt.core.IType.getClassFile()\" because \"type\" is null\n\tat org.eclipse.jdt.ls.core.internal.JDTUtils.getClassFile(JDTUtils.java:492)\n\tat org.eclipse.jdt.ls.core.internal.JDTUtils.discardClassFileWorkingCopy(JDTUtils.java:478)\n\tat org.eclipse.jdt.ls.core.internal.handlers.HoverHandler.hover(HoverHandler.java:51)\n\tat org.eclipse.jdt.ls.core.internal.handlers.JDTLanguageServer.lambda$5(JDTLanguageServer.java:663)\n\tat org.eclipse.jdt.ls.core.internal.BaseJDTLanguageServer.lambda$0(BaseJDTLanguageServer.java:87)\n\tat java.base/java.util.concurrent.CompletableFuture$UniApply.tryFire(CompletableFuture.java:646)\n\t... 6 more\n"}}

Tested under this environment:

OS & version: Fedora latest
Eclipse IDE/Platform version: Eclipse SDK latest snapshot
...

Community

[x] I understand reporting an issue to this OSS project does not mandate anyone to fix it. Other contributors may consider the issue, or not, at their own convenience. The most efficient way to get it fixed is that I fix it myself and contribute it back as a good quality patch to the project.

This was fixed on LS end with https://github.com/eclipse-jdtls/eclipse.jdt.ls/commit/4769cd9070a6761cf5a483cd06881d5fd4b5a218


(#12844) Backwards compatibility for 'puppet agent --enable'

In version 2.7.10 we changed the name of the lock file that is
used by 'puppet agent --enable/--disable'.  A side effect is
that if the lockfile was created under an older version of
puppet, the "--enable" command in the newer versions won't
delete the old lockfile and thus you can't run your agents.
Make --enable attempt to detect and resolve this situation.

OK, other than the couple of changes there this looks good to me.


Adding isSidelineStart/Stop methods

Remaining Todos:

[ ] Update changelog
[ ] Add test coverage for serialization exceptions
[x] Add test coverage for new methods on FilterChain
[ ] Add test coverage for isSidelineStart/Stop

@Crim I just realized we tightly coupled the FilterChain to sidelines by having the SidelineRequestIdentifier be the id for a step. 🤦♂ I'm thinking of adding a simple interface that requires toString() called a FilterChainStepIdentifier and a DefaultFilterChainStepIdentifier for test purposes and having SidelineRequestIdentifier implement that. What do you think?

@Crim I messed up, I should have never moved FilterChain out of the DynamicSpout package in the first place. I'm moving it back in here, and shifting getting away from the SidelineRequestIdentifier as a method of identifying steps.  I think this PR is good to go now for review.


Passes in all original options when processing media rules

filePath (set at https://github.com/gemini-testing/gemini/blob/master/lib/coverage.js#L118) is currently not passed into processRule when processing media rules. This causes an error on https://github.com/gemini-testing/gemini/blob/master/lib/coverage.js#L202 when generating coverage reports

LGTM

@benedfit what is the case, where this bug can be reproduced?

@j0tunn More details here

LGTM too
@j0tunn shall we merge?


Add active_model_serializers-0.10 gem signature

https://github.com/rails-api/active_model_serializers/blob/0-10-stable
I added signatures around the methods in the range described in the README.

Will take a look at this PR today.


drivers/drivers_initialize.c: check if only one console is selected

Summary
Check if only one driver is used as console to avoid a non-working console because a different one is selected and has higher priority than the one we wanted
Impact
catch incorect configuration early during compilation
Testing
CI

I removed from this check CONFIG_SYSLOG_CONSOLE options as currently it is highly coupled with XXX_SERIAL_CONSOLE for some architectures and fixing it is not that easy.

msys2 is failing:
Run pip3 install --root-user-action=ignore --no-cache-dir pyelftools cxxfilt kconfiglib
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try 'pacman -S
    python-xyz', where xyz is the package you are trying to
    install.

    If you wish to install a non-MSYS2-packaged Python package,
    create a virtual environment using 'python -m venv path/to/venv'.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip.

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.
Error: Process completed with exit code 1.

all tests except msys2 pass now

Any idea why msys2 is failing?


Security scheme is incorrect

As I see you Security Schemes must be under the Components Object.
Please check
https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.3.md#security-requirement-object
You must take the schema from the components. Now this is not done.
        info_spec = spec_dict_deref.get('info', {})
        servers_spec = spec_dict_deref.get('servers', [])
        paths = spec_dict_deref.get('paths', {})
        components_spec = spec_dict_deref.get('components', {})
        security_spec = spec_dict_deref.get('security', [])

I was wrong


Ahead-of-Time Compilation for production builds

More here : https://help.syncfusion.com/angular/gettingstarted/getting-started-with-webpack-aot
Why do Ahead-of-Time Compilation

Faster Rendering
Fewer asynchronous requests
Smaller Angular framework download size
Detect template errors earlier
Better security

@aka-jain @spyshiv

Thanks for pointing this out @anweshnayak. We will use AOT at the time of bundling. Bit it's not useful for development purpose. So, closing this issue for now.


Refactor the GameProfileManager and make GameProfile immutable.

API | Common
This PR refactors the GameProfile, GameProfileManager and GameProfileCache.

It's no longer possible to add entries to the GameProfileCache, only remove them to invalidate.
GameProfileCache is now just a cache, it was previously also a lookup service.
GameProfileCache is no longer replaceable.
GameProfile is now immutable, but has transformation methods.
"filled" profiles are now just "profile", and non filled ones are "basic profile", e.g. getProfile and getBasicProfile.
"filled" was previously a bit confusing, because isFilled meant that the profile has a name, although filling a profile through GameProfileManager also added properties. This is now renamed to hasName.

Manually merged, thanks!


Added metrics for insights into job and error rates

Also upgrades Spring Platform to 2.0.4 and Spring Boot to 1.3.4.

Coverage increased (+0.1%) to 88.896% when pulling f9b6da0e7a9b8b4d3b5520f1586a6867cc2ebe75 on tgianos:develop into a092119992b06bea3fbeda0494bb8c87fb176f86 on Netflix:develop.