There is a bug in the test case testing.

Challenge Name

https://www.freecodecamp.com/challenges/no-repeats-please
Issue Description

Earlier I wrote the code for permutation using Heaps https://en.wikipedia.org/wiki/Heap's_algorithm. My code worked great for test cases aab, aabb,aaa but when I tested the code for abcdefa my code showed 3300 results against FCC's 3600.
After trying in vain for getting help on the forum. I changed the algorithm from Heap to one of the other mentioned at https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/Algorithm-No-Repeats-Please. The code worked perfectly locally on my laptop using sublime text editor and chrome browser. My answers matched perfectly with the test cases. So I transferred my program from laptop to FCC bonfire. But on FCC bonfire it shows only first test case correct. I have rechecked my code many times for a bug. But couldn't find one. So I think there is a bug in the testing the test cases. Kindly do the needful.
Browser Information

Browser Name, Version: Google Chrome,Version 49.0.2623.112. Microsoft Edge
Operating System: Windows 10
Mobile, Desktop, or Tablet:  DeskTop

Your Code
`var temp = [];
var result = [] ;
function permAlone(str) {
arr = getPermutation(str);
console.log("Result: " + arr);
//console.log("Total: " + arr.length);
return arr.length;
}
function getPermutation(str){
var testStr;
var re = /(.)\1+/;
var cur;
var arr =  str.split("");
for(var x = 0; x < arr.length ; x++){
cur = arr.splice(x,1);
temp.push(cur);
if(arr.length === 0 ){
		testStr = temp.join('');
		if (!(re.test(testStr)))result.push(testStr);
	}
	else
	{
			getPermutation(arr.join(''));
	}

arr.splice(x,0,cur);
temp.pop();
}
return result;
}
var word= "aabb";
var sat = permAlone(word);
console.log("sat: " + sat);`

`
Screenshot

@dhanabh you've declared temp and result in the global scope. Tests are run after each other so these global variables will change accordingly. Please make them locally scoped and try again.

@erictleung : many thanks. I just added initialization for the global variables in the function permAlone(). The code is working ok now. thanks once again.

Many thanks. I'll make changes and test again. best regards.
On Sun, Apr 24, 2016 at 8:38 PM, Eric Leung<EMAIL_ADDRESS>wrote:

Closed #8266 https://github.com/FreeCodeCamp/FreeCodeCamp/issues/8266.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/8266#event-639074009


change import of "Zahlungen" in three columns (see Marions comments text)

It should look like this. Each of the column Zeitangabe – Zahlung – Quelle should be sortable


using same interface for IOS and ANDROID Elements

I was wondering if there is any specific reason that you separate android and ios platform's Elements from each other? why not using the same interface for both of them, with some specific functions on each of them; and using interfaces names in yaml files?
Maybe a yaml reader version 3 will be required to handle different locators for each platform in each specific languages? Usually you want to test the app in both android and ios and they are almost the same style and design, why not use the same script for testing them?

Hi @alizelzele,
SeLion's ios and android elements are modeled after Apple's support in UI Automation and Google's support in uiautomator.
See;

https://developer.android.com/reference/android/support/test/uiautomator/UiObject.html
https://developer.apple.com/library/prerelease/ios/documentation/DeveloperTools/Reference/UIAutomationRef/index.html

I understand your concerns w.r.t testing across platform's and share the desire to have a solution which minimizes the amount of PageYaml, etc that would need to be created when testing across platforms here. If you would like to propose a change to the existing code in the form of pull request here, we'd be happy to vet it / take it into consideration.
Cheers!

@alizelzele closing this github issue for now. Please re-open if you have further questions or see a need to. Please also feel free to ask questions via our Google group and Gitter channel

Hi @mach6 ,
I think this design aligns with the Appium principle and is required for testers cause they usually have to test the same application on both Android and IOS.
To implement this feature, we need to have below changes:

Create mobile elements interfaces, and make the existing elements of Android & iOS implement it.
code generator.
a) new template files support new interfaces and create correct instances on runtime.
b) yaml reader for new yaml page files

HERE is sample change for what I meant to implement.
Appreciate if you can review and give your opinion, or any change can make it better or more usable?

@kumaravel-jayakumar I would appreciate your feedback here.

To be able to support this we would have to create separate yaml reader and page implementation to support mobile. Wouldn't it be possible to support this feature through custom elements? This way it could be made into a module where the end user can decide if they want to use it or not.
In the end the fact that you need to test both apps and that the app are very similar is not necessarily true. In fact if an app follows the style guide of each platform there could already be quite a big difference.

@renelux
if we want to do this with custom elements again we will face the problem with multi platforms and separate page objects cause that is how selion behaves right now.
I was thinking the exact same way on the start about style guide of each platform and so, but after checking some of the applications, i find out that they are usually behaving the same way and there is not a big difference as i was expecting to be there.
Since i am keeping the compatibility with older version, every thing can be defined the exact same way as before. If anybody wants to use a common page object for both platforms, he can benefit from an already existing solution for that.

Hi @renelux my first inclination too was to propose a custom element. While I think is would be possible to some extent, it could get messy w.r.t the locators for the custom element in the PageYAML.. One would have to introduce their own parsing logic in each custom element to identify the part of the locator to be considered for platform=android and the part to be considered for platform=ios. So, long story short, I think @alizelzele raises a good point and it would be nice if SeLion had some level of support for this use case.
@alizelzele I'm sorry for my delay in replying. I took a brief look in your current progress and overall I feel you are heading in a good direction. There might be parts that a change is proposed once you submit a PR, but at this point I don't have any suggestions. Is there a particular part of the pending implementation that you want direct feedback on?

And BTW.. Thank you for taking this change on! :)

@mach6 I was wondering about the way the page-yaml-file should be and how to parse it (using a new reader or extending the existing one), but i think i found the best way to do it i will change it a bit more and create a PR.

@mach6 the code for this is almost done. Will you create a patch branch for testing and review? or @alizelzele  submits the pull request to develop branch?

Hi @ken-jiang the contribution approach is documented in our https://github.com/paypal/SeLion/blob/develop/README.md
Yes, in this case a pull request to the develop branch is what we need.
Once submitted, it will trigger the CI build for the pending PR and a review -- which will be conducted by myself, other project admins such as @sebady and @sundaramrajendran, and anyone else in the community that is interested.
Also, if applicable (only PayPal employees are exempt), a contributor must sign the CLA.

@alizelzele
Sorry for my delayed response. I really appreciate your work and contribution towards SeLion. To answer you initial question "I was wondering ... specific reason ... separate android and ios platform's Elements " over @mach6 reply. iOS test automation engineers (coming from Javascript) are familiar with a unique set of widgets and gestures (UIAElement, UIAButton & tap(), doubleTap(), dragInsideWithOptions()) whereas Android engineers (UiObject primarily & click(), clickBottomRight(), dragTo()). Though they have some commonalities between them they were not similar, so in order for them to easily on-board and have a seamless transition we had to give them a similar set of widgets and methods (naming and behaviors) as their native technology. This was the prime reason they were designed separate.
There could also be (now) or may be tomorrow that one technology (iOS/Android) has a separate widget that the other does not support. This can be said for gestures also. So putting them under a same hierarchical structure could void certain implementations.
The gestures may imply the same action but called tap in iOS but in Android its was named as click. So we did not want to choose either one as a common name preferring one technology over the other. So we decided to go separate ways.
Another point was what if I have a similar widget in both technology and a gesture that is possible in one technology and not the other. Like say if there is some gesture called "scratch" in iOS button and Android Button cannot be "scratched"?
These were the points thought out before designing them as separate hierarchy of models. But I understand and appreciate that there are commonality between the widgets and appreciate you work to unify them.


[embedded] Consider move_value/copy_value as removable for keypath optimization

Embedded Swift allows keypaths as long as they don't require any runtime instances to exist. This PR improves on when those are allowed, namely in @dynamicMemberLookup use of setters, where we end up having move_value or copy_value instructions on the keypath values in SIL. Let's consider those (move_value and copy_value instructions) to be removable by the keypath simplification algorithm.
rdar://126655571

@swift-ci please test

right, it should not matter


Update EmployeeId class to fix bug in max ID

Update EmployeeId class to fix the bug in max ID, fixes #96

LGTM


Substrate 8b3e5bc

This bumps Substrate to commit 8b3e5bc.
@sorpaas , this would require bumping the substrate frontier branch to this target commit.

Please pull master!

@notlesh I updated Substrate's frontier branch, so you can switch back now.
The tests are still failing though -- an spec issue.

@sorpaas This last commit points to a fix in evm for no_std issues related to funty. Would it be possible to get a new crate of evm published with this fix?

Released evm v0.25.0!

I'm not able to reproduce this CI error locally. Rust's suggestion sounds simple, though:
   = note: expected fn pointer `fn(&[u8], _) -> std::result::Result<(evm_core::error::ExitSucceed, Vec<_>), evm_core::error::ExitError>`
1871
              found fn pointer `fn(&[u8], _) -> std::result::Result<(ExitSucceed, Vec<_>), ExitError>`
1872
   = note: perhaps two different versions of crate `evm_core` are being used?

Where is this file even coming from (the one CI is complaining about)?
frame/evm/precompile/sha3fips/src/lib.rs

That clearly doesn't even exist on this branch...

@notlesh it is in https://github.com/paritytech/frontier/blob/master/frame/evm/precompile/sha3fips/src/lib.rs
You need to merge master (which is done when doing the CI)


Arithmetic failure when duplicate scalar coord present

🐛 Bug Report

Due to having done arithmetic with cubes that have similar scalar coordinates, I ended up with both on my cube, which is fine.  However, when I tried to do subsequent arithmetic, I got an exception.
How To Reproduce
Steps to reproduce the behaviour:
import iris.cube
import iris.coords

cube = iris.cube.Cube(1)
coord1 = iris.coords.DimCoord(1, standard_name="longitude")
coord2 = iris.coords.DimCoord(1, standard_name="longitude", long_name="longitude")

cube.add_aux_coord(coord1)
cube.add_aux_coord(coord2)

cube - cube

Traceback (most recent call last):
  File "[my-path]/double_scalar_coord.py", line 11, in <module>
    cube - cube
  File "[git-path]/iris/lib/iris/common/lenient.py", line 124, in lenient_client_inner
    result = func(*args, **kwargs)
  File "[git-path]/iris/lib/iris/analysis/maths.py", line 328, in subtract
    return _add_subtract_common(
  File "[git-path]/iris/lib/iris/analysis/maths.py", line 370, in _add_subtract_common
    result = _binary_op_common(
  File "[git-path]/iris/lib/iris/analysis/maths.py", line 871, in _binary_op_common
    resolver = Resolve(cube, other)
  File "[git-path]/iris/lib/iris/common/resolve.py", line 327, in __init__
    self(lhs, rhs)
  File "[git-path]/iris/lib/iris/common/resolve.py", line 387, in __call__
    self._metadata_mapping()
  File "[git-path]/iris/lib/iris/common/resolve.py", line 1243, in _metadata_mapping
    self._as_compatible_cubes()
  File "[git-path]/iris/lib/iris/common/resolve.py", line 487, in _as_compatible_cubes
    add_coord(coord)
  File "[git-path]/iris/lib/iris/common/resolve.py", line 479, in add_coord
    new_src_cube.add_aux_coord(coord, tgt_dims)
  File "[git-path]/iris/lib/iris/cube.py", line 1162, in add_aux_coord
    raise iris.exceptions.CannotAddError(
iris.exceptions.CannotAddError: Duplicate coordinates are not permitted.

Expected behaviour

No exception.
Screenshots

Environment

OS & Version: RHEL7
Iris Version: main

Thanks @rcomer 🙂
I'm surprised that these two coordinates could both be added to the same Cube. I know they're not identical, but they are where it matters, and we can see that they then break downstream assumptions. So I'd ideally want to prevent that.
What would happen to your original case if the original cube.add_aux_coord(coord2) has been disallowed?

Thanks @trexfeathers, I've come up with another minimal example that better illustrates how I got to this situation in the first place!
import iris.cube
import iris.coords

model_coord = iris.coords.DimCoord(1, standard_name="longitude",
                                   coord_system=iris.coord_systems.GeogCS(6371229.0))
obs_coord = iris.coords.DimCoord(1, standard_name="longitude")

model_cube1 = iris.cube.Cube(1, aux_coords_and_dims=[(model_coord, None)])
model_cube2 = iris.cube.Cube(1, aux_coords_and_dims=[(model_coord, None)])

obs_cube = iris.cube.Cube(1, aux_coords_and_dims=[(obs_coord, None)])

bias_cube1 = model_cube1 - obs_cube
bias_cube2 = model_cube2 - obs_cube

for i, cube in enumerate([bias_cube1, bias_cube2]):
    # Coord system disappears if you save to NetCDF and re-load.
    for coord in cube.coords():
        coord.coord_system = None

# [apply np.abs to bias cubes]

bias_diff = bias_cube1 - bias_cube2

So possibly the long_name was a red herring, although it did exist in my original case.  The coordinates from my original case look like this:
DimCoord :  longitude / (degrees)
    points: [180.]
    bounds: [[  0., 360.]]
    shape: (1,)  bounds(1, 2)
    dtype: float32
    standard_name: 'longitude'
    coord_system: GeogCS(6371229.0)
DimCoord :  longitude / (degrees)
    points: [180.]
    bounds: [[  0., 360.]]
    shape: (1,)  bounds(1, 2)
    dtype: float32
    standard_name: 'longitude'
    long_name: 'longitude'
    var_name: 'longitude'

There are various differences, but the coord_system appears to be the problem.  If I remove it from the original coord, I only get one longitude coordinate on my "bias" cube.

I should probably add that I started with global data and the scalar coords came out of the collapsed method.

I believe @pp-mo has an explanation, do you think you could write it down?!

Think this was as follows:

Two scalar coords, with the same standard_name, one with a coord_system
Saved to NetCDF
During loading, the coord_system either needs to be on both, or on neither. Can't cope with the mixture

The @SciTools/peloton thinks this is too niche to try and fix 🙈


Read files in UTF8 mode

Closes #667
Via https://github.com/alanz/ghc-mod/pull/15

Can someone please test this on a mac?
And I am worried about the non-utf8 files on windows too.

non-utf8 files on windows too

Why, tho. AFAIK, GHC assumes input files are utf-8 anyway.

I am planning on doing a release today, will merge this after, so people using the "stable" version don't get hit by any funnies, and then people using the master version will be able to provide feedback.


removing GPU cluster node creation steps

Issue #, if available: https://github.com/aws-samples/eks-workshop/issues/474
*Description of changes: removing GPU cluster node creation steps
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Looks like you have package-lock changes and deleted authors.md

Requesting putting deleted files back

Can you take a look now. I put the deleted file back again

Can we revert the package-lock.json changes also?
On Thu, Nov 7, 2019, 11:30 AM Peter Dalbhanjan<EMAIL_ADDRESS>wrote:

Requesting putting deleted files back
Can you take a look now. I put the deleted file back again
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/eks-workshop/pull/494?email_source=notifications&email_token=AAB3SEE4HKHFXMUCE7TOI33QSRGBRA5CNFSM4JKKMZG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDNF7QY#issuecomment-551182275,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB3SEECDXRY4OBJPANYCR3QSRGBRANCNFSM4JKKMZGQ
.

Can we revert the package-lock.json changes also?
…
On Thu, Nov 7, 2019, 11:30 AM Peter Dalbhanjan @.***> wrote: Requesting putting deleted files back Can you take a look now. I put the deleted file back again — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub <#494?email_source=notifications&email_token=AAB3SEE4HKHFXMUCE7TOI33QSRGBRA5CNFSM4JKKMZG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDNF7QY#issuecomment-551182275>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAB3SEECDXRY4OBJPANYCR3QSRGBRANCNFSM4JKKMZGQ .

should be fixed in latest commit. Thanks for reviewing this PR

Looks good!


Try to run injector tests with popular DI containers

To ensure Injector is fine with popular container it is a good idea to run tests against:

yiisoft/di
php-di
league container

Likely it won't reveal any issues but worth checking.

I've been using Injector in production with php-di for almost a year now. Works great.

That's good but I think we can try running tests against real containers. At least in a separate branch.


Don't supply a NEON_NODE_ABI env variable

(Separated out from #32.)

This is a backwards-incompatible change, so it'll need to be released as 0.2 alongside a new 0.2 version of neon.


Interrupting inference?

Nice work :)
I did not find a way to cleanly interrupt inference, could only supress output from the iterable loop. Is this somehow possible?

Hey @jrtp thanks for the issue, this was indeed not possible. However, it can now be done like this:
LlamaIterator iterator = model.generate(params).iterator();
while (iterator.hasNext()) {
    iterator.next();
    if (Math.random() < 0.5) {
        iterator.cancel();
    }
}

Maven version 3.1.0 should soon be available.

Awesome, thx for the quick inplementation. I just tried it, after the update the model loading interrupted with the previous llama.cpp I had compiled with GPU on windows, then after recompiling it seemed to work but for some reason inference suddenly stops in the middle. Any ideas what it could be?
        LlamaIterator iterator = model.generate(inferParams).iterator();
        while (iterator.hasNext()) {
            token = String.valueOf(iterator.next());
            Main.logLine(token);
            if (canceled) {
                iterator.cancel();
                break;
            }
        }

Same behaviour that inference just stops with the shipped llama.cpp (so no --Dde.kherud.llama.lib.path set) - just a lot slower ;)

Weired, just switched back and forth between dependencies and now it just flies without obvious change - thx again!

Ok, hopefully last question, now everything seems to work except exactly the same run.bat as previously throws this error now: Could not find or load main class .kherud.llama.lib.path=out
The cmd option used is this -Dde.kherud.llama.lib.path=out/
Without the option it works but without GPU, weiredly enough it also works with GPU started from intellij - any ideas how that could be affected? I triplechecked, with 3.0.2 this option was just working.

FYI - user error - for some reason Intellij had 3.0.2 and 3.1.0 in the artifacts, didnt know this isnt updating automatically when changing maven deps

Great, glad to hear everything works now!


Missing option to add credential.

Not listed in the options to add as a credential either.  I am on self hosted, newest version 1.42.1

Please reach out to N8N support, they might help diagnose the problem (there might be something wrong with your n8n setup).
This is a strange problem, and it might indicate something is wrong with the n8n configuration.
Additionally, try restarting your n8n service.

Same problem ... any solution?

Same here

Same problem here

Please reach out to N8N support, they might help diagnose the problem (there might be something wrong with your n8n setup).

This is a known issue. Clearing browser cache might help.
Check https://community.n8n.io/t/community-nodes-not-accessable/46347


kube-OVN/OVS Networking issue

We created single node cluster with latest release 20.12.02 as it was running fin.We deployed the workload successfully. but, after few days when we tried to deploy more workloads it gets stuck in "ContainerCreating" state showing the following CNI related failure.

 Warning  FailedCreatePodSandBox  86s   kubelet            Failed to create pod sandbox: rpc error: code = Unknown desc = [failed to set up sandbox container "8b0ce8e6589f5fbfab6b911896fc8e1b30a418ddf1b765ca8e699c9e10e405d4" network for pod "nginx-app-deployment-6cdc9c97d4-slztv": networkPlugin cni failed to set up pod "nginx-app-deployment-6cdc9c97d4-slztv_default" network: request ip return 500 configure nic failed add nic to ovs failed failed to run 'ovs-vsctl --timeout=30 --may-exist add-port br-int 8b0ce8e6589f_h -- set interface 8b0ce8e6589f_h external_ids:iface-id=nginx-app-deployment-6cdc9c97d4-slztv.default external_ids:pod_name=nginx-app-deployment-6cdc9c97d4-slztv external_ids:pod_namespace=default external_ids:ip=<IP_ADDRESS>': exit status 1
  "ovs-vsctl: unix:/var/run/openvswitch/db.sock: database connection failed (Protocol error)\n": "", failed to clean up sandbox container "8b0ce8e6589f5fbfab6b911896fc8e1b30a418ddf1b765ca8e699c9e10e405d4" network for pod "nginx-app-deployment-6cdc9c97d4-slztv": networkPlugin cni failed to teardown pod "nginx-app-deployment-6cdc9c97d4-slztv_default" network: delete ip return 500 {
 "protocol": "",
 "address": "",
 "mac_address": "",
 "cidr": "",
 "gateway": "",
 "mtu": 0,
 "error": "del nic failed failed to delete ovs port failed to run 'ovs-vsctl --timeout=30 --if-exists --with-iface del-port br-int 8b0ce8e6589f_h': exit status 1\n  \"ovs-vsctl: unix:/var/run/openvswitch/db.sock: database connection failed (Protocol error)\\n\", \"\""
}]
  Normal  SandboxChanged  9s (x7 over 85s)  kubelet  Pod sandbox changed, it will be killed and re-created.

This seems like OVN/OVS networking issue. attaching the openvswitch logs. Let me know what other information is required to root cause the issue
ovs_service_logs.zip

Could you please try after rebooting the node?


docs: Add blog post to folder mapping example readme.md

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.28%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4115

Relevant Lines:
4616

💛  - Coveralls


simple execution of arbitrary string rather than generated Operation

Use case
manual and unittest-related tests
testing interceptors and engine without really having type-safe query-mutation-subscription in mind
Describe the solution you'd like
for parsing a Map or even string itself suffice

Hi @nort3x 👋 Would HTTP level testing work?
// Enqueue HTTP responses
mockServer.enqueue("""{"data": {"random": 42}}""")

// Execute queries
assertEquals(
  42,
  apolloClient.query(GetRandomQuery()).execute().data?.random
)

hi @martinbonnin thanks for your reply,
interesting! i didn't know about mock-server but let me rephrase the issue description
suppose we have a setup for our client consist of different interceptors and even a custom http-engine
by default apollo client doesn't let us execute arbitrary input and it demands for existing *.graphql files to generate respective Query, Mutation and Subscription, by arbitrary i mean what you enter in a graphql live console (say graphiql for instance)
use cases?   unit-test and behavioral checks,
here is an example from spring-graphql
if we inspect it client is able to execute queries and inspect them using a path mechanism (this is an overkill!)
after all we are POSTing to the server why not be able to do it dynamically  without generated types

Ah, gotcha! It's not just about tests, you want to execute runtime-defined operation without generated models, right?
If that's the case, you'd have less troubles using an HTTP client like OkHttp directly. A big part of the value proposition of Apollo Kotlin is the generated models and type safety so if you don't need them, I'd say there's no need to use Apollo Kotlin? Or are you using something else (cache maybe?)?

thank you, as i mentioned above use cases are for testing interceptors and HttpEngine beneath apollo client plus why such a soft cap is put on the chain?

Hi 👋 Not sure I'm following. For HttpEngine, it doesn't know at all about GraphQL so you can test it without generated models. For interceptors, do you mind sharing an API you'd like to see? Maybe we can add it?

@nort3x anything help we can help with here?

I'm still not satisfied with current solution(which is revolving around testing HttpEngine separately from apollo client),
but i guess it's due personal preferences
so we can safely close this issue
thank you for your friendly support


Error in js/en_bcv_parser.min.js

I am using this in my Unity3d project. After importing the script into my project I get errors.
Error that i am getting shown:
Assets/en_bcv_parser.js(2,17): BCE0043: Unexpected token: ,.
I have also attached the script that I am using.
Note: This is a text file i converted into a text file so i could upload it.
en_bcv_parser.txt

If i remove 1 error i get 1 more and onwards. If anyone could help me with this that would be awesome. Or if anyone has the bug free version of the code that will be awesome.


SyntaxError: Unexpected token : on running node app.js

I'm getting an error while trying to start the node server:

"credentials": {
^
SyntaxError: Unexpected token :
at exports.runInThisContext (vm.js:73:16)
at Module._compile (module.js:443:25)
at Object.Module._extensions..js (module.js:478:10)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
at Function.Module.runMain (module.js:501:10)
at startup (node.js:129:16)
at node.js:814:3

My app.js looks like the following and the username and password field are filled in properly

{
"credentials": {
"url": "https://stream.watsonplatform.net/text-to-speech/api",
"password": "passwordhere",
"username": "usernamehere"
}
}

I'm also getting this error when I try to run the server on Bluemix. Probably I'm just confused about how the app.js file supposed to be structured.

When running locally you only need to replace username and password here with your service credentials.

I did that

Can you email me a copy of the file? nfriedly@us ibm.com

Sorry, typing on a phone, that should be @us.ibm.com


feat: add a global flag to disable styling (color and spinners)

This is for CI or scripts where you don't want hundreds of lines of spinner output..
nitric up -s <> --enable-styling=false

Codecov Report

Merging #205 (02a7d4c) into develop (683fe50) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #205      +/-   ##
===========================================
- Coverage    36.02%   35.87%   -0.15%
===========================================
  Files           48       48
  Lines         4189     4189
===========================================
- Hits          1509     1503       -6
- Misses        2540     2544       +4
- Partials       140      142       +2

Flag
Coverage Δ

unittests
35.87% <ø> (-0.15%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/output/verbose.go
0.00% <ø> (ø)

pkg/ghissue/buglink.go
90.32% <0.00%> (-9.68%)
:arrow_down:

pkg/utils/paths.go
5.55% <0.00%> (-5.56%)
:arrow_down:

Continue to review full report at Codecov.

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


Config revamp

Changes
This is moving towards a configuration based on the suggestion in #108.
Currently this is looking as follows:
services:
  auth-service:
    type: auth
    endpoint: auth-cluster
    failureMode: deny
    timeout: 10ms
  ratelimit-service:
    type: ratelimit
    endpoint: ratelimit-cluster
    failureMode: deny
actionSets:
  - name: rlp-ns-A/rlp-name-A
    routeRuleConditions:
      hostnames: [ "*.toystore.com" ]
      matches:
      - selector: request.url_path
        operator: startswith
        value: /get
      - selector: request.host
        operator: eq
        value: test.toystore.com
      - selector: request.method
        operator: eq
        value: GET
    actions:
    - service: ratelimit-service
      conditions: []
      data:
      - static:
          key: domain
          value: "rlp-ns-A/rlp-name-A"
      - selector:
          selector: request.headers.My-Custom-Header
      - static:
          key: admin
          value: "1"

I've separated all of the changes by commit to try make it easier to read the changes.
The primary change to non-config types is that we now have actions.conditions so we can conditionally perform an action - example provided in the envoy configuration that only calls the rate limiting service when user_id == 'alice':
make build && make local-setup

Port-forward envoy and watch the logs for all services:
kubectl port-forward --namespace default deployment/envoy 8000:8000
kubectl logs -f deployment/envoy
kubectl logs -f deployment/authorino
kubectl logs -f deployment/limitador

Test unauthenticated does not get rate limited:
curl -H "Host: test.b.multi.com" http://<IP_ADDRESS>:8000/get -i
# HTTP/1.1 401 Unauthorized

Bob is not rate limited (note no calls to limitador in logs):
while :; do curl --write-out '%{http_code}\n' --silent --output /dev/null -H "Authorization: APIKEY IAMBOB" -H "Host: test.b.multi.com" http://<IP_ADDRESS>:8000/get | grep -E --color "\b(429)\b|$"; sleep 1; done

Alice has 5 requests per 10 seconds:
while :; do curl --write-out '%{http_code}\n' --silent --output /dev/null -H "Authorization: APIKEY IAMALICE" -H "Host: test.b.multi.com" http://<IP_ADDRESS>:8000/get | grep -E --color "\b(429)\b|$"; sleep 1; done

Outstanding

[ ] Tests for conditional actions
[x] Retrieve the scope from action.data

For a future PR:

Convert routeRuleConditions.matches and action.conditions from PatternExpression to CEL
"Merge" subsequent actions of the same type to only make a single call for ratelimit

Verification steps working

Latest commit removes action.scope and this information is retrieved from the data section now

From data? This is unexpected. From a static data item with a key name (by convention) domain?
What if there are many, which one has preference?

I've dropped that commit fwiw. I tend to agree that right now the purpose of the data and scope is different and the way (we currently) process the data items means we need some custom logic to process a specific keyed data item, which doesn't seem right

Latest commit removes action.scope and this information is retrieved from the data section now

From data? This is unexpected. From a static data item with a key name (by convention) domain?

I agree with @eguzki. I don't like this change.
Whether as domain or as authCfg, in both cases this is a required piece of data and in both cases it has the meaning of “scope”. Inside of data, it feels out of place IMO. Unlike the rest of what goes there, scope is not as much "user-defined" data.
The way it was before, we could technically have an action without any data, which makes sense to me since we have default data that comes from the type of action (service) anyway. In fact, auth is an example of action that typically would involve zero user-defined data. And I could speculate with meaning to that for ratelimit as well (e.g. letting Limitador knows about request without activating any limit).
Anyway, I honestly liked more how it was before. Thanks for ditching the commit!


How to do the prediction with the output layer is LinerRegressionOutput

I have train a params model with the last layer in network like:
s2_fc2a = mx.symbol.FullyConnected(data=s2_relu5, num_hidden=1)
linearRegression1 = mx.symbol.LinearRegressionOutput(data=s2_fc2a, name="softmax1")
s2_fc2b = mx.symbol.FullyConnected(data=s2_relu5, num_hidden=1)
linearRegression2 = mx.symbol.LinearRegressionOutput(data=s2_fc2b, name="softmax2")
linearRegression = mx.symbol.Group([linearRegression1,linearRegression2])
return linearRegression

And I follow the the prediction of mnist write the predict programe like:
def predict(params, symbol, image):
    predictor = Predictor(open(symbol).read(), open(params).read(), {'data':(1 , 3, 256, 256)},'gpu',0) #use cpu if you don't have a gpu card
    img = io.imread(image)

    predictor.forward(data=img)
    # for first task
    prob = predictor.get_output(0)
    pred = np.argsort(prob[0])[::-1]
    # label_id (same as label if MNIST), score
    print pred, prob[0][pred[0]]
    # for second task
    prob2 = predictor.get_output(1)
    pred2 = np.argsort(prob2[0])[::-1]
    # label_id (same as label if MNIST), score
    print pred2[0],  prob2[0][pred2[0]]

symbol_file = "./net/1207-symbol.json"
params_file = "./net/1207-0020.params"
image_file = "./1.jpg"
predict(params_file, symbol_file, image_file)

But the output of this programe is like following type and the number is fixed when I change the input image.
0 11432.5
0 9855

My label is in the former like:
215541 10212

Is the code in prediction programe I have write  wrong or I have make mistake when I prepare to training?

Is your training accuracy correct? Does it work with only one output?

@winstywang I'm a new comer of mxnet.
It's is a multi-label regression task,Althought I don't know how to write the code about multi-label regression. So I follow the instruction in https://github.com/haria/mxnet-multi-task-example/blob/master/multi-task.ipynb I have change my data and path.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Build fails with msbuild 15

Build fails with msbuild 15 on Arch Linux, with and error message about "/usr/lib/mono/msbuild/15.0/bin/Roslyn/Microsoft.CSharp.Core.targets" not being found
Project "/home/user/godot/modules/mono/editor/GodotSharpTools/GodotSharpTools.sln" on node 1 (default targets).
ValidateSolutionConfiguration:
  Building solution configuration "Release|Any CPU".
Project "/home/user/godot/modules/mono/editor/GodotSharpTools/GodotSharpTools.sln" (1) is building "/home/user/godot/modules/mono/editor/GodotSharpTools/GodotSharpTools.csproj" (2) on node 1 (default targets).
/usr/lib/mono/msbuild/15.0/bin/Microsoft.CSharp.CurrentVersion.targets(331,5): error MSB4019: The imported project "/usr/lib/mono/msbuild/15.0/bin/Roslyn/Microsoft.CSharp.Core.targets" was not found. Confirm that the path in the <Import> declaration is correct, and that the file exists on disk. [/home/user/godot/modules/mono/editor/GodotSharpTools/GodotSharpTools.csproj]
Done Building Project "/home/user/godot/modules/mono/editor/GodotSharpTools/GodotSharpTools.csproj" (default targets) -- FAILED.
Done Building Project "/home/user/godot/modules/mono/editor/GodotSharpTools/GodotSharpTools.sln" (default targets) -- FAILED.

Build FAILED.

"/home/user/godot/modules/mono/editor/GodotSharpTools/GodotSharpTools.sln" (default target) (1) ->
"/home/user/godot/modules/mono/editor/GodotSharpTools/GodotSharpTools.csproj" (default target) (2) ->
  /usr/lib/mono/msbuild/15.0/bin/Microsoft.CSharp.CurrentVersion.targets(331,5): error MSB4019: The imported project "/usr/lib/mono/msbuild/15.0/bin/Roslyn/Microsoft.CSharp.Core.targets" was not found. Confirm that the path in the <Import> declaration is correct, and that the file exists on disk. [/home/user/godot/modules/mono/editor/GodotSharpTools/GodotSharpTools.csproj]

    0 Warning(s)
    1 Error(s)

To clarify, this happens with Xamarin's msbuild. Microsoft's msbuild works fine.

@tagcup I got the same error on Rider IDE of Intellij IDEA. I'll try change the msbuild.


html_base

The structure of the header, navigation menu, main content and footer has been made

first time tested pull request in reality)


install failed

sudo -H pip install android-resource-remover
Requirement already satisfied (use --upgrade to upgrade): android-resource-remover in /Library/Python/2.7/site-packages
Requirement already satisfied (use --upgrade to upgrade): lxml>=3.3.3 in /Library/Python/2.7/site-packages (from android-resource-remover)
I have no idea ,could you help me?

please check the readme


derive type info for some grandpa types

@davxy It's needed so that these types are properly shown in runtime metadata, I suppose they are doing something with them where they are now exposed from the runtime.

bot merge

bot merge


v4 and tRPC 10 support

https://trpc-nuxt-beta.vercel.app/

Hello, I have a question about how the useAsyncData under the hood works. Since it's always async, that means that after querying/mutating, the data object that gets returned could be null and doesn't immediately get the result after awaiting? If I have an onClick that calls this, will it be able to actually get the data back instead of null (also doesn't this mean I have to always to null if checks)

Also a different issue, this might be to do with upstream trpc type inference but I'm not sure... The issue is if we decide to do some ternary condition in the return value from the backend, the type is inferred as Pick<Object, never>, but if I don't do that, it's fine, and infers Object type properly

The init trpc also seems to be run on the client as well, is there any easy way to have it only run on the server? I definitely would've thought that nuxt 3 runs everything in the server folder only on the server, I guess I was wrong?


🌐 Add Polish translation

Thanks for the translation. I will test it and give a review :-)

I have sent fixes here: https://github.com/anarion80/lubelog_translations/pull/1

I approve

I approve

Approve


donate link is unavailable

Error 520 Ray ID: 4a1994e78ba55408 • 2019-01-31 04:55:01 UTC
Web server is returning an unknown error

Me too...
搭了梯子也不行

Me too...
搭了梯子也不行
链接裸连有效

patreon可能在中国电信网络难以访问

Now all the donate link was closed,please wait until the v4 version released

Is there still a donation link at the moment?


Update our automation to use new czgenepi infra.

Summary:

What: Update GitHub actions and terraform to use new staging/prod stack names (gestagingstack / geprodstack) instead of old ones.
Ticket: sc<fill_in_issue_number>
Env: <rdev link>

Demos:
Notes:
Checklist:

[ ] I merged latest <base branch>
[ ] I manually verified the change
[ ] I added labels to my PR
[ ] I tested in multiple browsers
[ ] I added relevant unit tests
[ ] I have notified others of changes they need to make locally (migrations, jobs, package updates, etc)

Looks good!
Is the plan for it to remain geprod and gestaging forever, or are we going to eventually migrate back to the old name space of prod and staging once we tear down the old infra? I'm just thinking about since we tend to speak verbally about "prod" and "staging", we might accidentally get confused and target the old namespace if it's always ge_____ from here on out.

I don't expect to change these names back anytime soon - it basically requires standing yet another set of parallel infra and redeploying. I think we'll have to do this from time to time for other types of risky infra changes, so we can take one of those opportunities to change the name back. In hindsight, I probably should have versioned these names (staging2?) instead of adding the ge prefix.


Tag 0.11.2 in git

Hi, can you please tag ef33cba1daea0a9434ea909752c8244778b78e7b as 0.11.2?

Looks like it's been done, thanks.


Clean TransactionReplacementByPriceRule

For example this method :
private Wei priceOf(final Transaction transaction, final Optional<Long> baseFee) {
   final TransactionPriceCalculator transactionPriceCalculator =
       transaction.getType().equals(TransactionType.EIP1559)
           ? EIP1559_CALCULATOR
           : FRONTIER_CALCULATOR;
   return transactionPriceCalculator.price(transaction, baseFee);
 }

The 'EIP1559_CALCULATOR' manage non EIP1559 transaction so this check is useless. But in order to have something that work before London we need to have different TransactionReplacementByPriceRule behavior for PRE and POST London
Maybe getting the valid TransactionPriceCalculator thanks to the protocolSpec in order to not have if else condition **
protocolSchedule.getByBlockNumber().getTransactionPriceCalculator()**

addressed in https://github.com/hyperledger/besu/pull/2545


Get-AutomationCertificate Doesn't Work in PowerShell 7.2

Get-AutomationCertificate works in 5.1, 7.1, but not in 7.2 in Azure Automation.

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

ID: d4f44677-19e6-c3af-1ee3-6a1f7c5055a9
Version Independent ID: 98b99258-be25-89b7-1ed6-b6e79a96977d
Content: What's new in Azure Automation
Content Source: articles/automation/whats-new.md
Service: automation
Sub-service: ****
GitHub Login: @SnehaSudhirG
Microsoft Alias: sudhirsneha

@rednender Thanks for your comment! We will review and provide an update as appropriate.

@rednender As mentioned here, currently PS 7.2 is supported in only 5 regions (West Central US, East US, South Africa North, North Europe, Australia, and Southeast). Did you try in one of these regions or not?

Yes, this is in East US.

I’ll add, other get-automation cmdlets work, e.g. get-automationvariable.

Yes.  Here are some more details about my environment, what I've tested, and what I've noticed.
First, this is all being tested within the same Azure Automation account on East US.  I do see that Powershell 7.2 is missing all the default modules when viewing them in AA.  I am guessing it's sharing its module space with PowerShell 7.1 since the cmdlets are still available in PowerShell 7.2
Second, I have three different certificates.  All three work fine in 5.1 and 7.1 runtimes, but none of them work from 7.2 runtime.  This is confirmed using a very simple command, i.e.
Get-AutomationCertificate -Name "Test".`
I have not encountered problems with the other built-in Azure Autmation cmdlets, though I have not tested them all. So far, Get-AutomationVariable and Get-AutomationPSCredential appear to work.
When using Get-AutomationCertificate, I do receive the following error (this only happens in the 7.2 runtime).
Get-AutomationCertificate:  Line |    4 |  $GraphCertificate = Get-AutomationCertificate -Name 'Test'      |                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~      | Object was not found.
If I try to use the certificate thumbprint via Connect-MgGraph, I receive the following error.
Connect-MgGraph:  Line |   16 |  Connect-MgGraph -ClientId $GraphClientId -TenantId $TenantId -Certifi …      |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~      | 'authority' should be in URI format.  (Parameter 'authority')
I will add I don't have a viable workaround at the moment other than going back to the 5.1 runtime. The 7.1 runtime has its own issues with using Connect-MGGraph.  I don't see another good way to retrieve the certificate, such as Get-AzAutomationCertificate (this requires a certificate to make the initial connection - so we'd be back at square 1).

Hi @rednender, PowerShell 7.2 runtime is in preview and is catching up on parity with PS 5.1 and PS 7.1. The support for Get-AutomationCertificate cmdlet will be rolled out in next deployment phase which is expected to be completed by end of March.

@KrishnaG-MSFT Thanks for the info!  What about the issue that I am seeing with 7.1 in my other Account on East US 2? Any insight into that problem?  One 7.1 environment works with get-automationcredentail and one does not.  Chalk it up to still being in Preview?

@rednender Strange behavior! Unfortunately, I was not able to reproduce the issue.
As it is hard to say what is causing you to hit the issue, support would be able to determine this with more information (like visibility to your environment, logs, traces, etc.) so here are the steps to raise a support ticket with Azure technical support if you already have a support plan.
But if you don't, then please send an email to<EMAIL_ADDRESS>with subject as "Attn:Krishna" and your subscription ID and a link of this Github thread for context and we will assist you with engaging Azure technical support.

@rednender This azure-docs repository deals about feedback related to particular Azure document page (like correcting doc bugs, doc enhancements, etc.) so going forward if issue is intended to product bug and not document enhancement then please share that kind of feedback here or by raising a support ticket.
At this point it appears that there's no edits or updates needed for the referred Azure documentation so closing this issue for now. If there are further questions regarding this topic, please do let us know and we will gladly continue the discussion.


Incorrect DOI on Open-OnDemand README

The title says it all. There is DOI :
http://dx.doi.org/10.1145/2949550.2949644
but this leads to an error page that says there is no such DOI. Could you check & correct this?

The proceedings for the XSEDE'16 conference have not yet been published. I left that as a placeholder until it appears.
Probably best to remove it until it appears.


describe tables cql statement is not implemented?

let describe_struct_cql = "DESCRIBE tables;";
let rows = ddb
.query(describe_struct_cql)
.expect("query")
.get_body()
.expect("get body")
.into_rows()
.expect("into rows");
I tried to get the list of tables by using describe statement as above but I got this error:
thread '' panicked at 'query: Server(CDRSError { error_code: 8192, message: CString { string: "line 1:0 no viable alternative at input 'describe' ([describe]...)" }, additional_info: Syntax(SimpleError) })', src/libcore/result.rs:997:5
note: Run with RUST_BACKTRACE=1 environment variable to display a backtrace.
Can you help me on how to get the list of tables in a particular keyspace?
Thanks

Hi @MaenMax ,
No, it's not implemented on CDRS level. Moreover, basing on the error you are receiving your Cassandra instance doesn't support it either. As you can see, it's a server error.
describe table; is a part command shell API (cqlsh) not a native CQL command.
Please check this answer on Stackoverflow https://stackoverflow.com/questions/38696316/how-to-list-all-cassandra-tables


embedding_size arg should be of type Int

Hi there,
I ran into a minor problem that would be easy to fix. In src/train_tripletloss.py:
parser.add_argument('--embedding_size', type=float, help='Dimensionality of the embedding.', default=128)

type=float should simply be replaced with type=int otherwise tensorflow will complain about it:
raise ValueError('num_outputs should be int or long, got %s.', num_outputs)

hope it helps.
cheers

You're right! Will fix this in next update. Thanks!!

Fixed in cb5c750d890949dabd990289e182e1c72c7ed17f.


jsk_recognition: 1.2.9-0 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository jsk_recognition to 1.2.9-0:

upstream repository: https://github.com/jsk-ros-pkg/jsk_recognition
release repository: https://github.com/tork-a/jsk_recognition-release.git
distro file: melodic/distribution.yaml
bloom version: 0.7.2
previous version for package: 1.2.7-2

checkerboard_detector

No changes

imagesift

No changes

jsk_pcl_ros

No changes

jsk_pcl_ros_utils

No changes

jsk_perception

No changes

jsk_recognition

No changes

jsk_recognition_msgs

No changes

jsk_recognition_utils
* add QT_LIBRARIES Qt5::Widgets to fix 'cannot find -lQt5::Widgets' error on debian stretch (#2398 <https://github.com/jsk-ros-pkg/jsk_recognition/issues/2398> )
* Contributors: Kei Okada

resized_image_transport

No changes

this will fix debian stretch build failure
http://build.ros.org/job/Mbin_ds_dS64__jsk_recognition_utils__debian_stretch_amd64__binary/7/console
https://github.com/jsk-ros-pkg/jsk_recognition/pull/2398


[SPARK-17985][CORE] Bump commons-lang3 version to 3.5.

What changes were proposed in this pull request?
SerializationUtils.clone() of commons-lang3 (<3.5) has a bug that breaks thread safety, which gets stack sometimes caused by race condition of initializing hash map.
See https://issues.apache.org/jira/browse/LANG-1251.
How was this patch tested?
Existing tests.

Test build #67107 has started for PR 15525 at commit f318dff.

Test build #67107 has finished for PR 15525 at commit f318dff.

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

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

Merged build finished. Test FAILed.

Jenkins, retest this please.

Test build #67111 has started for PR 15525 at commit f318dff.

I think this is OK. The general issue is whether it will be compatible with other transitive dependencies, but I believe it would remain so.

Test build #67111 has finished for PR 15525 at commit f318dff.

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

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

Merged build finished. Test PASSed.

Merging in master.

Actually I'm seeing the following exceptions locally as well as on Jenkins for Hadoop 2.2:
[error] /scratch/rxin/spark/core/src/main/scala/org/apache/spark/util/Utils.scala:1489: value read is not a member of object org.apache.commons.io.IOUtils
[error]       var numBytes = IOUtils.read(gzInputStream, buf)
[error]                              ^
[error] /scratch/rxin/spark/core/src/main/scala/org/apache/spark/util/Utils.scala:1492: value read is not a member of object org.apache.commons.io.IOUtils
[error]         numBytes = IOUtils.read(gzInputStream, buf)
[error]                            ^

I'm going to revert the patch for now.

@rxin I think the failure is not related to this PR.
The build for hadoop-2.2 is failing after reverted.
https://amplab.cs.berkeley.edu/jenkins/job/spark-master-test-sbt-hadoop-2.2/1948/console
Looks IOUtils is introduced by 231f39e3f6641953a90bc4c40444ede63f363b23 (#15285), and fixing is on progress at #15537.

Ah alright thanks. Can you submit a new pr so I can merge it again?

I see. I'll submit soon.

@rxin I submitted a new pr #15548. Thanks.


What should we do with the tile form of the map?

I'll take recommendations.

I think it'd be good to use VworldBase.


Don't override existing contracts

Codecov Report
Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Project coverage is 63.56%. Comparing base (5c83bfc) to head (c0ea9d7).

Files
Patch %
Lines

pypechain/render/contract.py
0.00%
8 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #123      +/-   ##
==========================================
- Coverage   63.61%   63.56%   -0.05%
==========================================
  Files          49       49
  Lines        2559     2561       +2
==========================================
  Hits         1628     1628
- Misses        931      933       +2

Flag
Coverage Δ

unittests
63.56% <0.00%> (-0.05%)
:arrow_down:

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

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


Tenant Breaks With ModelObserver

I have a mobel that uses tenancy and a model observer, but the second I try to add the observer the query stops adding the tenant conditions. I even have it to where the observer class has no actual code to run, so it appears to be just the act of adding it breaks.
// app.php#providers
// lots of others not shown, but order persists
Torzer\Awesome\Landlord\LandlordServiceProvider::class,
App\Providers\AppServiceProvider::class,
App\Providers\TenantServiceProvider::class,

// AppServiceProvider#boot
Game::Observe(PersistToObserver::class);

// TenantServiceProvider#boot
// lots of other things that are confirmed working properly
Landlord::addTenant('site_id', $site->id);
Landlord::addTenant('season_id', $season->id);

// PersistToObserver
class PersistToObserver {
     // everything is commented out
}

// Game Model
class Game extends Model implements Shareable, IPersistTo
{

    use BelongsToTenants, Event, UsesCustomCollection;

    // other model stuff
}

// Route controller query in question
$results = Game::with('location')
            ->withCount(['album', 'stats', 'updates'])
            ->results()
            ->take(4)
            ->get();

I've stripped that query down to nothing but Game::get() and it still doesn't apply. All of those withCount models have the same tenancy and the sub query for those work all the time. Its only when that Game is observed. Comment out that line in AppService#boot and tenancy works again.
On Laravel 5.6.39 and 3.1.0 of this pacakge

Landlord::applyTenantScopesToDeferredModels(); after add tenant

Landlord::applyTenantScopesToDeferredModels(); after add tenant


When I want to abort execution and send Ctrl+C, the process inside the docker container ignores the signal and keeps running until all tests are ran.

I'd expect testcafe process to react to signal sent by docker. From what I can see, SIGTERM is ignored (sent by docker stop). SIGKILL though is understood (sent by docker kill).
Step to reproduce
Start a test suite with the docker image and try Ctrl+C or docker stop
Your Environment details:

testcafe version:                   1.14.2
node.js version:                    v14.15.1
browser name and version: chromium or firefox (the one embed in the docker image)
platform and version:          MacOS 11.2.1 (20D74)
docker:                                   Docker version 20.10.6, build 370c289
docker image:                     8fca81fb890c

My bad, I forgot to run the container with -i.


Couldn't find file 'paper-button/paper-button'

I am a newbie in Polymer... but this seems more like rails problem. Did you have any idea why I can get it work?
I did it all. Installed gems, added to requires to application.js file.

Same here. Any way to fix this?

@piotrzaborow and @aravinthforever I figured it out.
Run:
rails g polymer:install

Then add this: <%= html_import_tag 'application'%> to your layout.
Finally, go to app/assets/components/application.html and add the components you want.

when you add some core or paper elements, you have to add it it application.html.erb
#app/assets/components/application.html.erb

//= require polymer/polymer
//= require paper-button/paper-button

So did this work for anybody? Would be interested in seeing a sample project where this was implemented. I'm stuck at this require step.


[FLINK-9083][Cassandra Connector] Add async backpressure support to Cassandra Connector

What is the purpose of the change
This pull requests rewrites CassandraSinkBase to use a Phaser and Semaphore to provide proper synchronization to support maxConcurrentRequests as a new configuration. This improves the reliability of the Cassandra Connector as it can currently overwhelm a weak Cassandra cluster if the upstream source has very high throughput.
Brief change log

Rewrote CassandraSinkBase to use a Phaser and Semaphore.
Expose public void setMaxConcurrentRequests(int maxConcurrentRequests, long timeout, TimeUnit unit) on CassandraSinkBase.
Modify CassandraSink with the new configuration. It currently does not support the WAL.
Updated the documentation about the new configuration.

Verifying this change
This change is already covered by existing tests, such as CassandraSinkBaseTest and CassandraConnectorITCase.
This change added tests and can be verified as follows:

Added tests for acquiring permits from the Semaphore and releasing permits from the Semaphore when a write succeeds or fails.

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

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

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs / JavaDocs

Nice work ! i ask my self if it should be push down to the driver or not ?

I think post 3.0.4 the driver doesn’t block on executeAsync() when the internal queues are full. It throws an exception instead that it is full. Thus, as the latest driver version is 3.6.0, users can easily be using a newer version of the driver that will fail fast instead of blocking for backpressure.

The PR needs rebase on current master.
@jparkie do you have ETA of addressing the comments?

My apologies; I've been quite busy with my personal life and work, but I should be able to carve time this weekend to address your comments.

I rebased my PR, and I'll be working on the new changes to address the PR comments.

🙏 hopefully get merged soon

I was waiting for some feedback, but I'll just go ahead and just commit immediate change.

@jparkie thanks. this will be very beneficial for our system. Currently, we are self-rate limiting via guava to avoid overwhelming Cassandra. Not ideal
looking forward to your merge

I came back from a long weekend vacation. I plan to address these comments over the next few days. Cheers.

@jparkie any update here?

I've been busy with work, but I've been making progress on the modifications and cleaning up the tests, and making them better since we have a queue for the TestCassandraSink.

I removed the lock stuff and updated the docs.

Will address my last comment while merging.


Allow to provide custom play.api.Configuration and play.api.Mode

so we can avoid to use deprecated Play.current object if needed.
This change is backwards compatible.

👍


Update RoR ActionController link

Updates the Ruby on Rails guide ActionController link to point to the accurate anchor for the Parameters Filtering section.

Thanks!


[WIP] Redux ui

Moving lib/ui to redux
What
I'm moving the ui to redux and adding some features in the meantime. This work is both an experiment and possibly sth we can merge later on.
Why

Will make contributions easier
Match the preview stack
This PR might give people an idea of the inner workings of storybook
Open discussions about APIs if needed :)

I'm mostly doing it to understand storybook better and see what we can improve or not ^^

@alexandrebodin Is this something you're still interested in? Do you need help?
I'm trying to get our # of PRs down a bit.

@ndelangen You can close.
I'll still be working on this anyway ;)

Awesome, because this does sounds like a good action to take. I'm really happy to have you on this! 🙇


There don't appear to be any tests for socialserver.api.legacy.bio?

maybe i'm just not seeing them, but they certainly don't seem to have their own file if they do exist.


fix './miri run --dep --target _'

@bors r+

:pushpin: Commit a3f0ca372bf55c3c662e62b2dd275642f1d75bec has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit a3f0ca372bf55c3c662e62b2dd275642f1d75bec with merge 5537a941a4a0e6cb5dfa91eded13a3895f55c9e8...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 5537a941a4a0e6cb5dfa91eded13a3895f55c9e8 to master...


32-bit build issue: Reboot takes int but LINUX_REBOOT_CMD_HALT overflows int

Tries now to build bin/rancheros like in Makefile.docker on a 32 bit machine (a Raspberry Pi, in fact)
The build stopped in cmd/power, on the 129th line.
It is this issue: golang/go#9584
I will patch this for my RPi

Closing this because its merged


[haxedevelop] "Find And Replace" - selected text is not pasted

How FD(<IP_ADDRESS> master#5549bb3615) works:

select some text in the file
press Ctrl+Shift+F
File and Replace panel pop-ups (if is still not)
label "Find what" gets the selected text

How HD(5.1.1) works:

select some text in the file
press Ctrl+Shift+F
File and Replace panel pop-ups (if is still not)
[BUG HERE] label "Find what" NOT gets the selected text (it is empty or contains old input)

Works for me just fine. Do you have any 3rd party plugins installed or other customizations? Whats are your OS details?

I have some theme plugins, but now I am using full default theme. Also I have ColorBox plugin http://www.flashdevelop.org/community/viewtopic.php?f=4&t=10339 but I disable it, restart HD and checked again - issue is still there.
Also some edit to issue:
If panel "Find and Replace in Files" is not open then Ctrl+Shift+F combination works fine for first time. The same behavior for "Find and Replace" panel and Ctrl+H combination. Both panels working only if these are not open.
OS is Win7 Ultimate SP1


feat: update cypress dependency version

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Doens't work in production

This is working fine on local development, when I run angular app on localhost:4200,
However, once I deploy the app to the vercel, it returns error.
main-6FYVCRR5.js:7 ERROR TypeError: (void 0) is not a constructor
    at e.getEventSource (main-6FYVCRR5.js:18:4128)
    at i._subscribe (main-6FYVCRR5.js:18:3856)
    at i._trySubscribe (main-6FYVCRR5.js:3:4428)
    at main-6FYVCRR5.js:3:4373
    at Ua (main-6FYVCRR5.js:3:2222)
    at i.subscribe (main-6FYVCRR5.js:3:4289)
    at e.<anonymous> (main-6FYVCRR5.js:18:11341)
    at Generator.next (<anonymous>)
    at chunk-2WQGNY7Y.js:1:1640
    at new t (polyfills-LZBJRJJE.js:2:2128)

 this is the compiled source which I can see in devtools

            getEventSource(t) {
            return new (void 0)(t,{
                headers: {
                    Authorization: "Bearer " + this.token
                },
                retry: 3e3
            })
        }

    here's my source code

import { Injectable, NgZone, inject } from '@angular/core';
import { Observable } from 'rxjs';
import { apiUrl } from '../../environment';
import { AuthRegularService } from '../services/auth-regular.service';
import { EventSource as CustomEventSource } from 'extended-eventsource';

@Injectable({
 providedIn: 'root',
})
export class SseService {
 token = inject(AuthRegularService).getToken();
 constructor(private _zone: NgZone) {}
 getServerSentEvent(url: string): Observable<any> {
   return Observable.create((observer: any) => {
     const eventSource = this.getEventSource(url);
     eventSource.onmessage = (event) => {
       this._zone.run(() => {
         // console.log("SSE event data received: ", event.data)
         if (!event.data) {
           console.log('False data received', event.data);
           // do not forward as this causes an error, "", false, undefined, null
         } else if (JSON.parse(event.data).msg === 'PROCESS_ENDED') {
           eventSource.close();
           observer.next(event);
         } else {
           observer.next(event);
         }
       });
     };
     eventSource.onerror = (error: any) => {
       console.log('Getting server sent event error:', error);
       eventSource.close();
       return;
       // this._zone.run(() => {
       //   observer.error(error);
       // });
     };
   });
 }

 private getEventSource(url: string): EventSource {
   return new CustomEventSource(url, {
     headers: {
       Authorization: 'Bearer ' + this.token,
     },
     retry: 3000,
   });
 }
}

Hey @Power-Howdy can you give some info on how you build or provide a small example?
I can not reproduce it right now.

@Power-Howdy I just published a version that should work based on @dargmuesli's change.
Would be happy to know if your issue has been fixed!


WIP: fix(user): make Directus related exception handling global

Fixes #14
Moves the Directus related exceptions handling to a new global ExceptionFilter.

BaseExceptionFilter doesn't handle exceptions as I expected to; it must be different from the actual ExceptionFilter that is handling HttpExceptions. I suggest merging this PR as it improves the situation a lot, but then turn these exception "transformers" into nest Interceptors.


Add a feature like writing a secret with the RDS connection string details.

In crossplane we have a great feature to create the secret after RDS Instance created with all the required parameter os a connection string.
writeConnectionSecretToRef:
      name: sample-db-con-string
      namespace: default

This can be included as part of another feature request #740. We have work being done on this design at the moment.

/remove-lifecycle stale
Even though #740 was closed, I have not closed this issue. Connection secrets, such as those from databases or EKS clusters, are not exposed as part of the Describe* for the resources. The logic can sometimes be simple to create these (such as interpolating a string), but often these requires additional custom code to be generated (such as an EKS Kubeconfig). I'm not sure if ACK even wants to start going down this direction, because it might be a slippery slope away from our core principal to keep things as per the AWS API.

This is currently possible using FieldExports https://aws-controllers-k8s.github.io/community/reference/common/v1alpha1/fieldexport/


hide already focused Window

no point having it in the list.

tried with CG by dropping the first window from the list (they're ordered front to back) but even though the first in the list is the front one, it doesn't mean it's the key window. so it doesn't returned for sure a focused window, just the most front one.

so, ended up doing the same than for the new matching algorithm: get the focused window, do one pass on all the visible windows and match by app, title, bounds. if none is found do second pass with only by app, bounds (coz some apps return different titles with CG and AX APIs). works well.


Command redesign problems

In Windows (Witsy 1.24.0), the new AI command redesign didn't work for inserting, replacing, and chatting, either by clicking or using the hotkeys (keys 'i' or 'r'). However, the copy command (key 'c') works.

Hi. Just tested again on Windows on my side and it is working fine. Can you provide more specifics on what is happening? Thanks!
https://github.com/user-attachments/assets/215be616-2eec-4296-9500-fdccbfdf0276

Sorry about that. Should be fixed in just released 1.24.1. Please confirm 😀

Thank you, Nic, for your quick response. The hotkeys for copy ('c'), insert ('i'), and replace ('r') work well. However, the hotkeys for read, chat, and write do not work yet, although clicking them is okay. Also, I believe the functionality of the new command should be more thoroughly documented. Additionally, the action column in the command settings is no longer needed.

Should be fixed in latest 1.26.1

Good enough. However, the new chat shortcut ('n') still does not work. Another problem: the text is not copied by the 'ask me anything' command.

Sorry shortcut is S as "save". They are so not good that even me did not remember it!

About "Ask me anything" I was able to reproduce once and then not anymore but will keep you updated. Thanks!

Here's what happened on my computer: I tried the "Explain" command first, and it copied the text as intended; however, when I tried the "Ask me Anything" command, it did not copy the text.
https://github.com/user-attachments/assets/6d0525d2-bc81-4817-b89c-a70da21a2259

A little unrelated, but many commands and experts are activated automatically when Witsy is first installed. I think the majority of users only require a maximum of five experts. Disabling many experts after installation was slightly cumbersome. Therefore, it would be beneficial to either disable the majority of them by default or provide a smaller selection of experts.

A little unrelated, but many commands and experts are activated automatically when Witsy is first installed. I think the majority of users only require a maximum of five experts. Disabling many experts after installation was slightly cumbersome. Therefore, it would be beneficial to either disable the majority of them by default or provide a smaller selection of experts.

I understand Witsy lacks documentation (looking for volunteers 😉) but did you see this?

Ah, how come I can't see it?

I understand Witsy lacks documentation (looking for volunteers 😉) but did you see this?

Which version are you running? (About Witsy in tray icon menu). Make sure you run the latest version. Download it from https://witsyai.com.

Which version are you running? (About Witsy in tray icon menu). Make sure you run the latest version. Download it from witsyai.com.

I have seen the option to unselect all. Witsy 1.26.1.

Here's what happened on my computer: I tried the "Explain" command first, and it copied the text as intended; however, when I tried the "Ask me Anything" command, it did not copy the text.
2024-11-30.14-26-42.mp4

Should be fixed (or at least occurring less often) in just released 1.26.2.


bump :lang rust

brotzeit/rustic@804ebfe -> brotzeit/rustic@eb5ca14

This breaks ligatures on -> for me. I've bisected rustic and nailed down the breaking change to this commit: https://github.com/brotzeit/rustic/commit/1441f1d4675aefbe7d32b8be3c05fac4ece43461. I guess the next step would be to depend directly on rust-mode and bisect it as well, I haven't done this yet.

@andresilva thanks for testing. The issue is that rustic was a fork for years and we must have applied some changes to either rust-mode or rustic in this time that broke or fixed this issue. If you could take a look at rust-mode would be great, I'll also take a look and try to find the commit that contains the related change.

Maybe this one  https://github.com/rust-lang/rust-mode/commit/959bfce215c342c83d72650b2b35201de263edfe ?

I was just writing that! Yes, that's the commit that broke it for me, thankfully I can just override rust-prettify-symbols-alist to nil to use the ligatures I have setup. I looked at rust-mode.el before bisecting but somehow missed this, it was obvious in hindsight 😜

=)
I guess you can't push to my branch so I'll add the fix


[wip] Fix travis builds

Fix failing Travis builds

[x] Integrate #645 which adds Ruby 2.5 and omits minor version numbers
[x] rubinius: Update Rubinius to 3.99 (https://github.com/bundler/bundler/issues/6163): Missing or uninitialized constant: Bundler::Deprecate
[x] rubinius: File does not exist: minitest/unit (builtin test/unit which is compatibility layer over minitest is removed in 3.100)
[x] ruby 1.9.3: Limit version of Rake (development dependency) to < 12.3 (12.3 no longer supports Ruby 1.9 which we target)
[ ] ruby HEAD: cannot load such file -- bundler/dep_proxy https://github.com/travis-ci/travis-ci/issues/8969

For some reason, require 'test/unit' in test_helper under Rubinius loads test-unit from standard lib (rubinius/3.99/codedb/source/rubysl/test/unit.rb), not from test-unit gem. Stdlib's Test::Unit delegates to minitest and breaks.
Stacktrace from my machine:
/usr/local/opt/rubinius/lib/rubinius/3.99/codedb/source/rubygems/core_ext/kernel_require.rb:135:in `require'
/usr/local/opt/rubinius/lib/rubinius/3.99/codedb/source/rubysl/test/unit/unit.rb:1:in `__script__'
/usr/local/opt/rubinius/lib/rubinius/3.99/codedb/source/rubygems/core_ext/kernel_require.rb:59:in `require'
/usr/local/opt/rubinius/lib/rubinius/3.99/codedb/source/rubysl/test/unit.rb:1:in `__script__'
/usr/local/opt/rubinius/lib/rubinius/3.99/codedb/source/rubygems/core_ext/kernel_require.rb:59:in `require'
/usr/local/opt/rubinius/lib/rubinius/3.99/codedb/source/test/unit.rb:1:in `__script__'
/usr/local/opt/rubinius/lib/rubinius/3.99/codedb/source/rubygems/core_ext/kernel_require.rb:59:in `require'
/Users/kolen/items/redcarpet/test/test_helper.rb:5:in `__script__'
/usr/local/opt/rubinius/lib/rubinius/3.99/codedb/source/rubygems/core_ext/kernel_require.rb:59:in `require'
/Users/kolen/items/redcarpet/test/custom_render_test.rb:2:in `__script__'
/usr/local/opt/rubinius/lib/rubinius/3.99/codedb/source/rubygems/core_ext/kernel_require.rb:59:in `require'
/usr/local/opt/rubinius/lib/rubinius/3.99/gems/gems/rake-12.2.1/lib/rake/rake_test_loader.rb:17:in `__script__'
/usr/local/opt/rubinius/lib/rubinius/3.99/gems/gems/rake-12.2.1/lib/rake/rake_test_loader.rb:5:in `__script__'

Maybe we should allow ruby-head to fail? It's unstable version and it depends on time when build started (it takes latest commit, not fixed version).
matrix:
  allow_failures:
  - rvm: ruby-head

(Also AFAIK rvm can install head version of rbx too, we can try to build against it but allow for failure)

Maybe we should allow ruby-head to fail?

we should definitely allow ruby-head to fail.

Made ruby-head allowed to fail. Now tests pass on CI. That error is still there in current head version (BTW, tests pass on my machine on ruby-head).

(Also AFAIK rvm can install head version of rbx too, we can try to build against it but allow for failure)

Seems that it is not available on Travis so I didn't add rbx-head (it requires building from sources non-standard way).

🙌 Thanks for fixing this!


Switch test suite to use hspec

Being worked on in #22.

In #22 we should have dropped deps on HUnit and test-framework, I'll do it now.


Adding French translation for Other (all remaining missions)

This PR is purely FR translation related:

Adding French translation for all remaining Other missions.
Fixing couple of typos in previously Fr translated files.

French translation should now be complete !

Wow, AWESOME work!!  I'm sure the community will appreciate your effort! 🏆

I'll be getting your other pull requests (the bigger ones with code changes) merged soon, too.


Inventory Panel Tweaks - Inventory Filtering / Customizable Sections

Add a feature that allows filtering inventory items into categories. Something by the lines of old Armor/Weapons/Components/Magic/Mundane/... panes, but per-item-type customizable.

I'll attempt this one myself, it doesn't seem particularly hard.

As a note, the devs are working on restoring the inventory item sorting directly in the game.


Erroneous "unused entry inspection" warning when file is ignored by IntelliJ

If a file or folder is ignored by IntelliJ in Editor > File Types, the plugin will fire an "unused entry inspection" warning even if the entry is used.
Steps to reproduce: Go to Editor > File Types in IntelliJ. In the field "Ignore files and folders", add *.idea, so the IDE won't show these folders in the project view. Add *.idea to .gitignore. There will be an inspection warning that this entry is unused, even though the folder does exist and will be ignored by git.

@AmsterdamFilho Thank you for your report.
The problem with this case is that if IDE ignores some file itself - they are not available in the virtual tree of the project - and I cannot even access them from the IntelliJ API - so they appear to not existing.
I don't know if it's possible (and worth?) to discover them in any other way.

@hsz What if the plugin did not fire any warning for files that are ignored by IntelliJ? If the IDE is ignoring the file, maybe the plugin should ignore it too.

Can I ask you to check if this issue is still present in v2.1.1?

Yes, it is. I am using IntelliJ 2017.2.2 and the version 2.1.1 of the plugin.


⚠ Bofrost.de has degraded performance

In bf7be59, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1079 ms

Resolved: Bofrost.de performance has improved in 7ddbda3 after 6 minutes.


WA-232 - Feature transfer tokens via multisig

Description
This PR allows transfer tokens (included ETH) via multisig operation. The list of ok tokens showed in the safe are automatically obtained from a mocked service.
Those contracts were deployed on rinkeby tesnet, so you can ask the "Safe Team" to send to your account some of them to test it out.
Tests simulating this feature are included 👍
Note for developers
This PR also includes a major refactor on tests, improving about 30% the time needed to run everything.
Screenshot

Travis automatic deployment:
https://staging-49-pr-41-safe-react-gnosis.surge.sh
Storybook book automatic deployment:
https://storybook-staging-49-pr-41-safe-react-gnosis.surge.sh

Travis automatic deployment:
https://staging-49-pr-55-safe-react-gnosis.surge.sh
Storybook book automatic deployment:
https://storybook-staging-49-pr-55-safe-react-gnosis.surge.sh

Travis automatic deployment:
https://staging-49-pr-254-safe-react-gnosis.surge.sh
Storybook book automatic deployment:
https://storybook-staging-49-pr-254-safe-react-gnosis.surge.sh

Travis automatic deployment:
https://staging-49-pr-170-safe-react-gnosis.surge.sh
Storybook book automatic deployment:
https://storybook-staging-49-pr-170-safe-react-gnosis.surge.sh

Travis automatic deployment:
https://staging-49-pr-98-safe-react-gnosis.surge.sh
Storybook book automatic deployment:
https://storybook-staging-49-pr-98-safe-react-gnosis.surge.sh

Something is broken now. Whenever I try to transfer tokens to the contract, the transaction fails 🤔
https://rinkeby.etherscan.io/tx/0x525059ad96256cbf2e3dbdba282e2f8bd7d233eaf9dca9e778be6f8b674976fa

Nevermind, it works :)


💡 Would have sense to have file content in real files instead of strings?

An idea came to my mind from this PR: https://github.com/octoherd/create-octoherd-script/pull/10/files
What if we have had this GitHub workflow as a file and checked periodically by renovate? An automatic PR would have updated it for us (and for future releases of actions we decide to us in workflows).
Checking other files we scaffold with https://github.com/octoherd/create-octoherd-script maybe some checks and logic could be applied:

Prettier / Lint
Unit tests / transpilation / compilation...
Easier to manipulate with octoherd actions (modify its content in a programmatic way maybe?)
It would open the door of scaffolding a package.json and package.lock with controlled version of packages (controlled by renovate) instead of doing an npm i (not sure if this is valuable or it complicates more the ecosystem)

What's your opinion here @gr2m

What if we have had this GitHub workflow as a file and checked periodically by renovate? An automatic PR would have updated it for us (and for future releases of actions we decide to us in workflows).

That's a good idea, I think we can do that.

Checking other files we scaffold with https://github.com/octoherd/create-octoherd-script maybe some checks and logic could be applied:

I don't want to force too much setup on people, they have their own preferences. Right now I think we should should probably sit back and see what people using it say about it


Readme and documentation improvements

Readme improvements

Centered spotdl image, header, description, badges
Changed badges style to flat-square
Added prerequisites section
Added ffmpeg installation section
Added Contributor Guide section
Added contributors image
Added License section
Removed $ from commands
Moved YT Music contry warning prerequisites section at the top

Removed dev utils
moved Code Docs and Design Docs to docs/code and docs/design respectively
updated LICENSE
updated pull request template and moved it to .github

Please pull from upstream & resolve conflicts
Tests failing - Please address

Please pull from upstream & resolve conflicts
Tests failing - Please address

Pulled from upstream
I am not sure why test were failing. They look fine now


type piracy

Note that methods like Base.inv(A::SparseMatrixCSC) = A\spdiagm(0=>ones(size(A,1))) are type piracy and make the package harder to compose with other Julia packages.
Just give the function a different name, e.g. sparseinv.

(Probably it's better to give it a different name anyway — people shouldn't use this function by accident, since for generic sparse matrices it is likely to give a dense inverse and hence be very inefficient.)

Thanks for your suggestions. I know about type piracy, I'll try to think about what I can do. Note that it is important for my purpose that things should be easily composable. In the example that I provide, the Woodbury matrices have an invert, and it is done by computing inv(A). It may be that A is a BlockDiagonal matrix, or that A is a dense matrix, or that A is a sparse matrix, or some other type of linear operator. So if one does as you suggest, the inverse of a Woodbury matrix becomes a lot more complicated.


Maintain selected color when adding to palette

Not a major change; just uses the last selected color for the new color when creating a palette.
In most programs (even ms paint), when adding a color to a palette it doesn't reset the color to white. Instead, it keeps the selected color.
See GIF of changes:

.

Here's how it was before. Note the reset to white:

Thanks!


If a signal handler is more than 1 line, bad code is generated

If a signal handler is more than 1 line, bad code is generated.  For example, this works fine:
local $SIG{ALRM} = sub { $signal_received = 1 };

but this tries to assign to signal.signal(0):
local $SIG{ALRM} = sub {
    $signal_received = 1;
    };

Fixed in v1.027


loki support?

Hi All, well done on a pretty solid stack, I like.
Do you foresee adding loki stack into it ? If you need help with that, can you write a short manual on how one would just splice something like that into it and set you a PR ? Greetings

Hi @sokoow,
Integrating Loki would be great and we would very much welcome a PR to that effect. Implementing that ourselves is not on our Q1 roadmap though. I'll get back to you on some thoughts/manual about how this could be done if you are interested in doing this PR.
Thanks,
Mat

Hi @sokoow,
Integrating Loki would be great and we would very much welcome a PR to that effect. Implementing that ourselves is not on our Q1 roadmap though. I'll get back to you on some thoughts/manual about how this could be done if you are interested in doing this PR.
Thanks,
Mat

Cool, thanks for replying @cevian - I am also thinking of bolting a full TICK stack into it, that'd be my next experimentation area. Think about guiding me into the structure of this repo as a time-saving measure and maybe write me a little howto so I can just crack on pls.

Cool, thanks for replying @cevian - I am also thinking of bolting a full TICK stack into it, that'd be my next experimentation area. Think about guiding me into the structure of this repo as a time-saving measure and maybe write me a little howto so I can just crack on pls.

For the foreseeable future we will be focusing on metrics and traces as well as features provided by promscale. As such we no longer see addition of loki to tobs on our roadmap. If that stance changes, we will reopen this issue.


avoid variable name used in upper scope (hashed)

Nice, thanks for fixing this lint warning

I think that this change unintentionally altered the behaviour of hashString and hashJSObj functions. In hashString:
hash = 31 * hash + string.charCodeAt(ii) | 0;

was changed to:
hashed = 31 * hash + string.charCodeAt(ii) | 0;

thus resulting in 31 * hash being the multiplication of an integer with a function which evaluates to NaN. This makes the hashString function return 0 for all strings.
The hashJSObj function suffers from a similar issue, and it only returns undefined.
Here is a PR that fixes both of those issues: https://github.com/facebook/immutable-js/pull/1325


Extension host service injection

This PR preps the extension host to make ExtHostFoo instances be injected into the extHost.api.impl file. The idea is that we can use different ExtHostFoo implementations for different extension host runtimes (Node.js, Web-worker) without re-implementing the extHost.api.impl - see https://github.com/microsoft/vscode/pull/73827
In essence there is /node/extHost.services.ts which registers singleton "services" that are used and there will be /browser/extHost.services.ts doing the similar thing for the web-worker world.

I have invented IExtHostContextService which is a bit of a kitchen sink (see below) and plans are to have a dedicated rpc service and ext host environment service.
export interface IExtHostContextService {
	_serviceBrand: undefined;
	readonly rpc: IMainContext;
	readonly initData: IInitData;
}

@Tyriar @weinand @alexr00 fyi what's happening here: in order to move more extension host logic into the /common/-layer I have extracted interfaces for extHost-things that require nodejs-api. E.g. there is now IExtHostDebugService, IExtHostTask etc. The api implementation (extHost.api.impl.ts) depends on those interfaces, not on the concrete implementations anymore - injection ftw.
However, that also means that clients program against IExtHost-interfaces and that you need to expose/define public members for 3rd parties in those interfaces.


Bug for calling end() of SessionRunHook

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
Linux Ubuntu 18.04
TensorFlow installed from: pip
TensorFlow version (use command below): 1.13.1
Python version: 3.6.7
CUDA/cuDNN version: CUDA 10.1, cuDNN 7.6
GPU model and memory: 2080Ti

Describe the current behavior
I define a hook class which is inherited from SessionRunHook, and add an assign operation in the begin() function. Then, I run the assign operation in the end() function. Although the value is changed, the results do not store into the checkpoint. So, If I restore the model, I get the model which is not changed.
Describe the expected behavior
Since we only know the end() is called before closing the session. I think the changed result should be store in the chekpoint. Otherwise, the document should provide the ordering of executing all hooks.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
class NetworkPruningHook(tf.train.SessionRunHook):
def init(self):
def begin(self):
  self.mask_var = []
  ### collect the masked variables
  for v in tf.global_variables():
    if v.name.find("mask:0") != -1:
      self.mask_var.append(v)

  self.check_mask = tf.assign(self.mask_var[0], np.zeros(shape=self.mask_var[0].shape))

def end(self, session):
  session.run(self.check_mask) # assign 0 to the variable
  print(session.run(self.mask_var[0])) # get [0, ..., 0]

  ### However, if restoring the checkpoint, the self.mask_var[0] is not [0, ..., 0]

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

In order to expedite the trouble-shooting process, please provide a full code snippet to reproduce the issue reported here. Thanks!

class NetworkPruningHook(tf.train.SessionRunHook):
 def __init__(self):
    super(NetworkPruningHook, self).__init__()
    self.sparsity = 0
    self.total_step = 0
    self.step = 0
    self.hasInitialize = False

def initialize(self, sparsity, total_step):
  self.step = 0
  self.total_step = (int)(total_step)
  self.sparsity = sparsity
  self.hasInitialize = True

def begin(self):
  tf.logging.info("[INFO] sparsity: %d", self.sparsity)
  self.mask_var = []
  self.masked_output = []
  self.assign_ops = []
  self.placeholders = []
  self.scores = []
  self.loss = None
  assert(self.hasInitialize == True)

  ### collect the masked variables
  for v in tf.global_variables():
    if v.name.find("mask:0") != -1:
      self.mask_var.append(v)
      p = tf.placeholder(tf.float32, shape=v.shape)
      self.placeholders.append(p)
      self.assign_ops.append(tf.assign(v, p))
      self.scores.append(np.zeros(v.shape))

  ### collect the outputs of masked layers and loss layer
  for op in tf.get_default_graph().get_operations():
    for t in op.values():
      if t.name.find("mask-output") != -1:
        self.masked_output.append(t)
      elif t.name.find("loss/Mean:0") != -1:
      	self.loss = t

  self.var_grad = tf.gradients(self.loss, self.mask_var)

def before_run(self, run_context):
  return tf.train.SessionRunArgs([self.var_grad, self.masked_output])

def after_run(self, run_context, run_values):
  for i in range(len(self.mask_var)):
    score = abs(np.sum(run_values.results[0][i] * run_values.results[1][i], axis=0))
    score_sum = np.sum(score)
    if(score_sum) != 0:
      score = score / score_sum
    self.scores[i] = self.scores[i] + score

  self.step += 1
  if self.step >= self.total_step:
    ### check the zero_density
    zero_density = [np.sum(x)*1. / np.shape(x)[0] for x in run_context.session.run(self.mask_var)]
    average = np.average(zero_density)
    tf.logging.info("zero_density of all layers before pruning: ")
    # for i in range(len(zero_density)):
    #   tf.logging.info("  zero_density of layer %d: %f", i, zero_density[i])
    tf.logging.info("The average zero_density of network: %f", average)

    ### pruning
    threshold = np.percentile([y for x in self.scores for y in x], self.sparsity)
    for i in range(len(self.mask_var)):
      mask = (self.scores[i] > threshold)
      run_context.session.run(self.assign_ops[i], feed_dict={self.placeholders[i]: mask})

    ### check the zero_density
    zero_density = [np.sum(x)*1. / np.shape(x)[0] for x in run_context.session.run(self.mask_var)]
    average = np.average(zero_density)
    tf.logging.info("zero_density of all layers: ")
    # for i in range(len(zero_density)):
    #   tf.logging.info("  zero_density of layer %d: %f", i, zero_density[i])
    tf.logging.info("The average zero_density of network: %f", average)

def end(self, session):
	self.hasInitialize = False

Above is my hook, and I add a masked layer by tf.Variable with name "xxx-mask" on the fully-connected layer. If I move the assign operation from begin() to end(), then the assign results will not be written in checkpoint file after training.

I tried reproducing the issue. But i am not able to get saved checkpoints.Please,help us to give  more information to reproduce the issue.Thanks!

I use the code of https://github.com/google-research/bert directly, I think the estimator will save checkpoints every some steps and when the training is end

Will it be possible to send me the full minimal code snippet to reproduce the issue for faster resolution.Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


404 Page Not Found from Sync is here in Figma Plugin

Bug: type - title
SUMMARY:
When launching Figma Tokens, the “Read more” link in the “Sync is here” screen leads to a 404 page error.
STEPS TO REPRODUCE:

Launch Figam Tokens
Click next on the Welcome to Figma Tokens screens until you get to “Sync is here!” Screen.
Click Read more

EXPECTED RESULT:
Expected to see the details linked to on this page https://docs.tokens.studio/sync.
ACTUAL RESULT:
Received a 404 page error.

Thanks for reporting! Should now be fixed


Allow users to create a 'collection' of tabs that will open specified command lines with different starting directories.

Description of the new feature/enhancement
I often need to open the same 4+ tabs with different starting directories.  It would be nice to have a feature similar to the 'collections' in MS Edge that will open several tabs to a specific command line and starting directory.

Proposed technical implementation details (optional)

You technically can do this now using combination of profiles with the commandline parameter set and using the wt argument parser (to get your tabs and/or panes). You could also just script it out purely using the argument parser (so you don't need to enter it every time you close Terminal).

Perhaps when the Settings UI is created, this could be an enhancement to remove the need to script it out yourself and configure it in the Settings UI instead (possibly as an extension).

Perhaps when the Settings UI is created, this could be an enhancement to remove the need to script it out yourself and configure it in the Settings UI instead (possibly as an extension).

Thanks for the response.  I figured there was a way to script it.  I was hoping to be able to do it through the UI.

They're still going over the spec https://github.com/microsoft/terminal/pull/6720. Once that's been flushed out, they'll begin implementing it. I'm sure they'd appreciate any feedback regarding this feature request.

As @WSLUser points out, you can use the wt commandline currently do do a little bit of this. I actually have a profile with the following:
{
    "name": "Good Morning",
    "commandline": "cmd.exe /k wt cmd.exe /k #work 15 ; split-pane cmd.exe /k #work 15 ; split-pane -H cmd.exe /k media-commandline ; new-tab powershell dev\\symbols.ps1 ; new-tab -p \"Ubuntu 18.04\" ; new-tab -p \"microsoft/Terminal\" ; focus-tab -t 0",
}

which opens up 4 tabs, and the first has 3 panes, for my initial dev environment. This unfortunately opens it in a new window currently, but it works for me. We've got lots of plans on how to improve this:

Opening initially with a set group of panes/tabs: #756
Creating actions / commands for performing multiple tasks:

via a wt commandline: #6537
via a list of actions: #5970
once either of the above two are done, this will be exposed via the command palette, #5400

Customize the new tab dropdown to be able to include custom actions (like "open these 4 tabs...") #1571/#5888


Custom resolver geo location

Dear all,
In amplify appsync grapqh API, I'll try to insert a custom resolver that is geo.
This guide is perfect.
My schema
`
type GeoLocation {
lat: Float
lon: Float
}
input GeoLocationInput {
lat: Float
lon: Float
}
type Address {
street: String
street_number: String
city: String
province: String
region: String
zipcode: String
nation: String
location: GeoLocation
formatted: String
UTCoffset: Int
}
type Experience
@model
@auth(rules: [
{allow: owner},
{allow: groups, groups: ["Admins"], queries: [get, list], mutations: [create, update, delete]},
{allow: groups, groups: ["Professionals"], queries: [get, list], mutations: [create]}
{allow: groups, groups: ["Users"], queries: [get, list], mutations: null}
])
@searchable {
id: ID!
type: Int!
status: Int!
name: String!
description: String
details: AWSJSON
createdAt: AWSDateTime
professional: Professional! @connection(name: "ProfessionalExperiences")
address: Address
location: Location @connection(name: "LocationExperiences")
}
`
and my resolvers
`## Query.nearbyExperiences.req.vtl
Objects of type Todo will be stored in the /todo index
#set( $indexPath = "/experience/doc/_search" )
#set( $distance = $util.defaultIfNull($ctx.args.km, 200) )
{
"version": "2017-02-28",
"operation": "GET",
"path": "$indexPath.toLowerCase()",
"params": {
"body": {
"query": {
"bool" : {
"must" : {
"match_all" : {}
},
"filter" : {
"address": {
"location" : {
"distance" : "${distance}km",
"location" : $util.toJson($ctx.args.location)
}
}
}
}
}
}
}
}`
and
`## Query.nearbyExperiences.res.vtl
#set( $items = [] )
#foreach( $entry in $context.result.hits.hits )
#if( !$foreach.hasNext )
#set( $nextToken = "$entry.sort.get(0)" )
#end
$util.qr($items.add($entry.get("_source")))
#end
$util.toJson({
"items": $items,
"total": $ctx.result.hits.total,
"nextToken": $nextToken
})`
When i try to query
`
nearbyProfessionals(lat, lon, km) {
let self = this;
return new Promise(function (resolve, reject) {
  const nearbyProfessionals = `query NearbyProfessionals {
    nearbyProfessionals(location: {
      lat: ` + lat + `
      lon: ` + lon + `
    }, km: ` + km + `) {
      items {
        id
        name
        address {
          city
          location {
            lat
            lon
          }
        }
      }
    }
  }`;
  self.amplifyService.api().graphql(graphqlOperation(nearbyProfessionals))
    .then(data => {
      if (data && data.data && data.data.nearbyProfessionals)
        resolve(data.data.nearbyProfessionals);
      else
        reject(-1);
    })
    .catch(err => reject(err));
});

}`
with my service
self.accountService.nearbyProfessionals(45.5, 7.6, 200)               .then(data => resolve(data))               .catch(err => reject(err));
It always give me
"Elasticsearch:UserIllegalArgumentException"
But, i correctly set index on Elastic Search
`
PUT /experience
PUT /experience/_mapping/doc
{
"properties": {
"address":{
"type": "nested",
"properties": {
"location": {
"type": "geo_point"
}
}
}
}
}
`
What could be the trouble? I have try to recreate all twice but no changes, always the same error...
Some other with this issuee or some idea to solve?
Thanks

@bagubits Please refer to https://github.com/aws-amplify/amplify-cli/issues/2385#issuecomment-534691122 and checkout the thread to learn more about this issue.


Handle duplicate original filenames in Transmitter

Also:

Store applicant summary as 00_applicant_summary.pdf for alphabetical ordering ease-of-use
Convert messy characters / and : into _ to avoid path traversal or other oddities
Sort user files deterministically to ensure filename ordering stays consistent
Allow IOException to bubble up (was being squelched presumably because lambdas cannot throw)

Tom noticed some non-Java-style variable names that I'll clean up.

@tdooner I'm gonna also escape \ characters, per this pull request. I'll merge once the tests pass.