False positive on NoNameShadowing with unused Unit as outer lambda parameter

Expected Behavior
Not to report NoNameShadowing, when outer lambda implicit parameter has type Unit and is unused.
Observed Behavior
test.kt:20:26: Name shadowed: implicit lambda parameter 'it' [NoNameShadowing]
Steps to Reproduce
Minimal (theoretical) example:
fun String.test(a: String.(Unit) -> Unit) {
    // ...
}
"".test { // it is Unit and unused
    chars().forEach { print(it) } // Name shadowed: implicit lambda parameter 'it' [NoNameShadowing]
}

Context
The issue seems to be that Unit is explicitely defined. When lambda parameter type is empty, the rule works fine.
fun String.test(a: String.() -> Unit) {
    // ...
}
"".test { // it is Unit and unused
    chars().forEach { print(it) } // No error
}

Your Environment

Version of detekt used: 1.20.0-RC2
Version of Gradle used (if applicable): 7.4.2

I'm not sure whether this is really a bug or not. If you explicitly define a parameter as Unit, there is an implicit lambda it with the value Unit and you are able to work with it. So there is a name shadowing.
In the other case, if you don't define a parameter, there will be no implicit lambda. So the following code should not work:
fun String.test(a: String.() -> Unit) {
    // ...
}
fun main() {
    "".test {
        println(it)
    }
}

I agree with @kvn-stgl  here. Why is this a false positive? You are shadowing the it of the outer lambda so it should be flagged. If you don't want to shadow it add _ - >.
And why do you have unit as a parameter? Could you provide an use case?

In the other case, if you don't define a parameter, there will be no implicit lambda. So the following code should not work:

Oh, I thought missing parameter and Unit meant practically the same thing, but apparently not.

I agree with @kvn-stgl here. Why is this a false positive? You are shadowing the it of the outer lambda so it should be flagged. If you don't want to shadow it add _ - >.

That is an option. I understand now that technically the parameter is shadowed, but as the outer parameter is unused, I would prefer that it wouldn't result in NoNameShadowing.

And why do you have unit as a parameter? Could you provide an use case?

The Unit parameter might be in an external library. Another use case is with Ktor routes, for example get. Simplified example from my application:
suspend fun wrapCall(call: ApplicationCall, callback: suspend (CallWrapper) -> Unit) {
    callback(CallWrapper(call))
}

routing {
    get("/path") {
        wrapCall(call) {
            processPath(it)
        }
    }
}

IntelliJ IDEA inspection NestedLambdaShadowedImplicitParameter doesn't flag this case. It shows warning only when the outer lambda parameter is used:
routing {
    get("/path") {
        println(it)
        wrapCall(call) {
            processPath(it) // "Implicit parameter 'it' of enclosing lambda is shadowed"
        }
    }
}

I would like to know what other maintainers think about this but to me even if you are not using it you are shadowing that value and it's not that clear what it you are using.

I agree with @BraisGabin . If it is of type unit I think it should not raise an issue, but in any other case it should.

Related: The corresponding IDEA inspection was changed to not report if outer 'it' is not used. I find this to be the better approach.

I too agree that this is a false positive. In facts this rule forces us to define a name when there's not an explicit clash among its.


Questions about gpg smard card functionalities and yubikey configured with agent forwarding

Hi Drdhuh,
Thanks for you fantastic works, it really help me to understand and configure my YUBIKEY to work as i want.
I have some difficulties with this part :
https://github.com/drduh/YubiKey-Guide#remote-machines-agent-forwarding
SSH connection using gpg-agent and key created and stored into yubikey (as smartcard) works, so typing ssh-add -l on remote after connection works and return 4096 SHA256:xxx cardno:xxx (RSA) . Thanks for that.
BUT anything linked to detection of smartcard, like gpg --smart-status, gpg --card-edit don’t work remotly, this is strange or is it a normal behavior ?
So, a second question, linked to this first, do you think it could be possible to forward a challenge/response configured on slot 2 by ssh using agent forwarding ?
Connecting using :
ssh -A -R /run/user/0/gnupg/S.gpg-agent:/run/user/1000/gnupg/S.gpg-agent.extra xxx

or
ssh -A xxx

My ~/.ssh/config for xxx :
Host xxx
  Hostname xxx
  User root
  ForwardAgent yes
  IdentityFile ~/.ssh/id_rsa_yubikey.pub

I'm using :
locally : gpg (GnuPG) 2.2.12
remotly : gpg (GnuPG) 2.2.20
LOCAL
local agent socket
socketdir:/run/user/1000/gnupg
dirmngr-socket:/run/user/1000/gnupg/S.dirmngr
agent-ssh-socket:/run/user/1000/gnupg/S.gpg-agent.ssh
agent-extra-socket:/run/user/1000/gnupg/S.gpg-agent.extra
agent-browser-socket:/run/user/1000/gnupg/S.gpg-agent.browser
agent-socket:/run/user/1000/gnupg/S.gpg-agent
homedir:/home/reyman/.gnupg

gpg-agent.conf
enable-ssh-support
default-cache-ttl 300
max-cache-ttl 999999
debug-level guru
debug-all
log-file /home/reyman/.gnupg/gpg-agent.log

REMOTE
agent sockets :
socketdir:/run/user/0/gnupg
dirmngr-socket:/run/user/0/gnupg/S.dirmngr
agent-ssh-socket:/run/user/0/gnupg/S.gpg-agent.ssh
agent-extra-socket:/run/user/0/gnupg/S.gpg-agent.extra
agent-browser-socket:/run/user/0/gnupg/S.gpg-agent.browser
agent-socket:/run/user/0/gnupg/S.gpg-agent
homedir:/root/.gnupg

gpg.conf
use-agent

Remote Nix config
        services.openssh.enable = true;
        services.openssh.permitRootLogin = "prohibit-password";
        services.udev.packages = [ pkgs.yubikey-personalization ];
        programs.ssh.startAgent = false;
        programs.gnupg.agent.pinentryFlavor = "curses";
        programs.gnupg.agent = { enable = true; enableSSHSupport = true; };

        services.openssh.extraConfig = ''StreamLocalBindUnlink yes'';

I also post this problem on nix forum :
https://discourse.nixos.org/t/yubikey-smartcard-challenge-mode-usable-on-remote-ssh/8936/11
Thanks !

Hi.
That is a normal behavior because the socket S.gpg-agent.extra is restricted to increase security.
If you want to use all commands, try using the local S.gpg-agent socket.
Kind regards.

@Nepochal Hi!
Ok, hum, i'm not sure to correctly config this thing.
I need to remove all the "agent-extra-socket" in local and remote conf,
or i need to reroute agent-extra-socket on /run/user/1000/gnupg/S.gpg-agent ?
Rest regards.

Try using
ssh -A -R /run/user/0/gnupg/S.gpg-agent:/run/user/1000/gnupg/S.gpg-agent xxx
to connect to your server. Your local S.gpg-agent socket is capable of all commands.
Nevertheless, you should be aware that using the S.gpg-agent socket is less safe, precisely because the commands are not restricted there.

I don't use nor generally recommend agent forwarding, so please send a PR yourself if there's useful information to add to the guide.


(#1322) Fix crashes in test case PushTests.test__008__LocalDevice__has_a_device_method_that_returns_a_LocalDevice (4f837671-6233-41f1-94e8-01174d1da7b8)

What does this do?
Introduces various concurrency-related fixes related to the local device. Together, I believe these fix various crashes that we've observed in the test mentioned in the title (#1322). See commit messages for more details.
How do I know it's fixed the test?
These are the results of the tests running continuously with these fixes applied. At time of writing, there are 97 test runs with no failures in this test, but I'll keep it running for a while more before merging.

D'oh, put the wrong issue number. Gonna open a new PR.


can't found ggml-vocoder-model.bin

after install and built, run with command line, then exit with no ggml-vocoder-model.bin file found error. where can download this file?

It’s on huggingface (there’s an link in the readme)

I havent uploaded the vocoder model to hugging face yet, but I'm happy to upload it if you want to try out the current version of tue code. Though keep in mind I'm still working on the logic for the vocoder.

I havent uploaded the vocoder model to hugging face yet, but I'm happy to upload it if you want to try out the current version of the code. Though keep in mind I'm still working on the logic for the vocoder.

Waiting for your great work.

Alright the models in the hugging face should be up to date :^)


added cereal, expanded drop, added filter between columns

Looks good. merge when ready.

You forgot to add pinguins

Once you have a approval, feel free to squash and merge


NoC RTL  simulaton

Hi sir, your work is really amazing. I also completed a NoC with chisel, and the basic unit passed the unit test of chiseltest, but I don't know how to do RTL simulation test for the complete NoC written in chisel language. I would like to ask you, how did you test when you completed the NOC that was not integrated into the chipyard framework.

Integration with Chipyard has already been completed, and will be released in the next Chipyard release before October 1st.

I realize I misunderstood your question.
I wrote Chisel testbenches for the entire network, these were basic traffic injectors that would load the network.

Thank you！
-----原始邮件-----
发件人:"Jerry Zhao" @.>
发送时间:2022-09-20 05:57:50 (星期二)
收件人: ucb-bar/constellation @.>
抄送: Elena @.>, Author @.>
主题: Re: [ucb-bar/constellation] NoC RTL simulaton (Issue #26)
I realize I misunderstood your question.
I wrote Chisel testbenches for the entire network, these were basic traffic injectors that would load the network.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


wsl Terminal does not accept copying from windows history prompt directly

Environment
Windows build number: 10.0.18363.0
Windows Terminal version: 0.11.1121.0

Steps to reproduce
Start a wsl Terminal instance having "commandline" : "wsl.exe -d Ubuntu-18.04" configured in the profile settings. (same happens for cmd.exe)
Press Win+V. Click on an entry of the appearing clipboard history popup.
Expected behavior
It should paste the contents on the shell, like it already happens for powershell.
Actual behavior
In wsl: nothing is pasted. The contents are transferred to the internal clipboard, but not pasted.
In powershell: the contents are pasted (as expected).
In cmd: ^V is displayed at the current cursor position. The contents are transferred to the internal clipboard, but not pasted.
Obviously, pressing right click to paste it afterwards works as expected. This only affects direct pasting from the windows history.

Self-closing it as duplicate of #322 - Did not find it at first :-/


Future updates like the PyTorch version?

Really hoping that this repo would have an update like the PyTorch version :(
Really love your work.

Agreed, great and valuable work! Please, updates like the Pytorch version will be extremely helpful.


Introduce csslint and htmlhint as dev packages

This will introduce csslint and htmlhint as development packages and run as pretest.
ONE-vscode-DCO-1.0-Signed-off-by: SaeHie Park<EMAIL_ADDRESS>

From draft #113

csslint tool fails if there is no .css file exist in the  directory...


Make cachier optional

Fixes #100

I've rebased this on top of #102 so that we can see test results here.

Cachier still works for me. It's not getting knowledge of new package types, obviously, but I haven't had any issues with it.

I can't see the logs from the ansible/check failures, but I'm happy to address them if I can.

The check failure is on zuul and it was a timeout and I bet it was some kind of "glitch" so nothing to do except running the checks again to see if it happens again

Cachier still works for me. It's not getting knowledge of new package types, obviously, but I haven't had any issues with it.

ok. nice. So, if it's still working, I'm happy with the current PR, as long as the checks are happy too

recheck

recheck

This time the Ubuntu test passed, but the linter timed out. Resource contention issues? :laughing:

I would call it zuul-gambling... let me remove the linters at least, two failures in a single day is far more than I am willing to deal with.


Pass Container's rect down when calling RequestRelayout.

This allow Relayoutable children to know more about their bounds so that they can better control their layout when rendering. Update ScrollContainer to make use of it. GridsLayout can be nested well enough that this additional information and swapping some containers from a RowLayout(Vertical) to a GridLayout(Col=1) allows some improvement without fully implementing a solution that would require significant more work.
Included updated demo to demonstrate it working which also required messing with some List internals.

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

Still reviewing this one. It'll take a little time to work through

Still reviewing this one. It'll take a little time to work through

OK, let me know if you want to discuss. For convenience, here is the overall gist copied from discord.

I pass down the parent Rect through RequestRelayout()... then in the container's iteration over its kids, it calls a new method called CalcLayout() and tries to let its kids calc the layout for the parent window it is in. To get CalcLayout, I reused the RowLayout pattern of passing in a LocationFunction to a doLayout() method... CalcLayout calls it with an accumulator and Layout calls it with an actual SetLocation call... After that it was pretty straightforward  just cleaning up cases where there wasn't a check for bounds and stuff.  RowLayout was tricky because a Relayoutable() thing may need to shrink if there is non-relayoutable things after it. GridLayout probably deals with this too. but it works for my case, except for some edge cases where something that only wants a MaxPixel that lands right on a boundary.

I left in my debugging code... I simply don't have the chops to handle all these nesting containers without some live info, so maybe that's useful to incorporate somehow.

Interested in what you're take is. It ended up being less disruptive than I thought to get it to a decent place (again for me for my usage.) And I was able to largely undo any changes I made to the List... pretty much all the changes are localized to the Layout classes and little one-liners in the widgets.


[Federation] Decide whether or not to reintroduce support for non-RBAC Federated clusters in kubefed join

 Issue by perotinus
Wednesday Mar 01, 2017 at 20:20 GMT
Originally opened as https://github.com/kubernetes/kubernetes/issues/42352

This is a follow-up to #41267.
#42042 removed support in kubefed join for clusters that don't support RBAC. Based on a discussion between me, @csbell and @madhusudancs, we need to decide Kubernetes cluster versions Federation will support. If we want to support pre-RBAC clusters, we need to reintroduce the old mechanism, with noisy warnings that the user should update for better security; if not, than this issue can be quickly closed.

 Comment by madhusudancs
Wednesday Mar 01, 2017 at 20:47 GMT

cc @kubernetes/sig-federation-bugs

 Comment by nikhiljindal
Friday Mar 03, 2017 at 20:13 GMT

Yes we do need to support pre-RBAC clusters.
Generally at most places in kubernetes, we support version skews of atleast one.

 Comment by perotinus
Friday Mar 03, 2017 at 20:39 GMT

Some clarifications based on a chat with @nikhiljindal:

If #42042 does not go in, there is no problem here
Once #42042 goes in, in re this feature the FCP will still support all cluster versions that it did in 1.5 (though there may be other changes that would affect this)
Once #41830 goes in, without fixing this issue, kubefed join will support clusters back to 1.4, when RBAC was introduced

 Comment by madhusudancs
Friday Mar 03, 2017 at 21:16 GMT

Also, note PR #41830 won't go in, in its current state. It should include code to handle clusters without RBAC. @irfanurrehman isn't that the plan?

 Comment by madhusudancs
Friday Mar 03, 2017 at 21:20 GMT

Also, there won't be any regressions in kubefed join until PR #42042 merges. We will continue uploading admin user credentials and that is supported in all the clusters. The current regression that PR #41830 is trying to address is in kubefed init and that same fix can be reused, although in a slightly modified way, in kubefed join as well.

 Comment by nikhiljindal
Friday Mar 03, 2017 at 21:42 GMT

Great thanks for clarification!

 Comment by irfanurrehman
Saturday Mar 04, 2017 at 06:53 GMT

@madhusudancs ack about the impl for https://github.com/kubernetes/kubernetes/pull/41830
I was under the impression that we support only those clusters now, which support RBAC. Thanks for the clarification. Will update it accordingly.

cc @perotinus


"ReferenceError: TextEncoder is not defined" when npm test on a Create-React-App

Search terms you've used

TextEncoder

Bug description

Running npm test results in the a TextEncoder error.
To Reproduce

Create a new Create-React-App
Install dependencies
Run npm test

Expected result

Succesful test

Actual result

 FAIL  src/components/App/index.test.js
  ● Test suite failed to run

    ReferenceError: TextEncoder is not defined

      13 | //* Imports
      14 | // Package Imports
    > 15 | import { useSession } from "@inrupt/solid-ui-react";
         | ^
      16 | import { Image, Nav, Navbar } from "react-bootstrap";
      17 | import { Link, NavLink } from "react-router-dom";
      18 | // Project Imports

      at Object.<anonymous> (node_modules/jose/dist/node/cjs/lib/buffer_utils.js:5:23)
      at Object.<anonymous> (node_modules/jose/dist/node/cjs/runtime/base64url.js:5:27)
      at Object.<anonymous> (node_modules/jose/dist/node/cjs/jwe/flattened/decrypt.js:4:24)
      at Object.<anonymous> (node_modules/jose/dist/node/cjs/jwe/compact/decrypt.js:4:22)
      at Object.<anonymous> (node_modules/jose/dist/node/cjs/index.js:4:20)
      at Object.<anonymous> (node_modules/@inrupt/solid-client-authn-core/dist/index.js:4:12)
      at Object.<anonymous> (node_modules/@inrupt/solid-client-authn-browser/src/Session.ts:26:1)
      at Object.<anonymous> (node_modules/@inrupt/solid-client-authn-browser/src/index.ts:22:1)
      at webpackUniversalModuleDefinition (node_modules/@inrupt/solid-ui-react/dist/webpack:/@inrupt/solid-ui-react/webpack/universalModuleDefinition:3:1)
      at Object.<anonymous> (node_modules/@inrupt/solid-ui-react/dist/webpack:/@inrupt/solid-ui-react/webpack/universalModuleDefinition:10:2)
      at Object.<anonymous> (src/components/Page/content/NavBar/index.jsx:15:1)
      at Object.<anonymous> (src/components/Page/content/Header/index.jsx:21:1)
      at Object.<anonymous> (src/components/Page/index.jsx:16:1)
      at Object.<anonymous> (src/components/Router/index.jsx:17:1)
      at Object.<anonymous> (src/components/App/index.jsx:17:1)
      at Object.<anonymous> (src/components/App/index.test.js:14:1)
      at TestScheduler.scheduleTests (node_modules/@jest/core/build/TestScheduler.js:333:13)
      at runJest (node_modules/@jest/core/build/runJest.js:404:19)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        0.745 s
Ran all test suites related to changed files.

Environment
  System:
    OS: macOS 13.1
    CPU: (8) arm64 Apple M1 Pro
    Memory: 177.02 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.19.0 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 8.19.4 - ~/Documents/Profesional/Semantyk/Projects/Semantyk/client/node_modules/.bin/npm
  Browsers:
    Chrome: 110.0.5481.77
    Safari: 16.2
  npmPackages:
    @comunica/query-sparql-solid: ^2.3.0 => 2.4.0
    @inrupt/solid-client: ^1.23.3 => 1.25.2
    @inrupt/solid-ui-react: ^2.8.2 => 2.8.2
    @inrupt/vocab-common-rdf: ^1.0.5 => 1.0.5
    @react-three/drei: ^9.40.0 => 9.56.24
    @react-three/fiber: ^8.9.1 => 8.11.1
    @testing-library/jest-dom: ^5.16.5 => 5.16.5
    @testing-library/react: ^13.4.0 => 13.4.0
    @testing-library/user-event: ^13.5.0 => 13.5.0
    bootstrap: ^5.2.2 => 5.2.3
    lamina: ^1.1.23 => 1.1.23
    node-fetch: ^3.2.10 => 3.3.0
    npm: ^8.19.3 => 8.19.4
    rdflib: ^2.2.20 => 2.2.26
    react: ^18.2.0 => 18.2.0
    react-bootstrap: ^2.5.0 => 2.7.2
    react-dom: ^18.2.0 => 18.2.0
    react-force-graph: ^1.41.15 => 1.42.0
    react-helmet-async: ^1.3.0 => 1.3.0
    react-router-dom: ^6.4.3 => 6.8.1
    react-scripts: 5.0.1 => 5.0.1
    react-sizeme: ^3.0.2 => 3.0.2
    serve: ^14.2.0 => 14.2.0
    stream-to-string: ^1.2.0 => 1.2.1
    three: ^0.146.0 => 0.146.0
    three-forcegraph: ^1.39.7 => 1.41.4
    url: ^0.11.0 => 0.11.0
    web-vitals: ^2.1.4 => 2.1.4
  npmGlobalPackages:
    @solid/community-server: 5.1.0
    corepack: 0.15.1
    npm: 8.19.3

Hi there! Any updates on this? Still having the issue. I tried with node v14 and tried reduced my deps to these, but still not working:
  "dependencies": {
    "@inrupt/solid-client": "^1.29.0",
    "@inrupt/solid-ui-react": "^2.8.2",
    "@react-three/fiber": "^8.13.4",
    "@testing-library/react": "^14.0.0",
    "bootstrap": "^5.3.0",
    "rdflib": "^2.2.32",
    "react": "^18.2.0",
    "react-bootstrap": "^2.8.0",
    "react-dom": "^18.2.0",
    "react-force-graph": "^1.43.0",
    "react-helmet-async": "^1.3.0",
    "react-router-dom": "^6.14.1",
    "three": "^0.154.0",
    "url": "^0.11.1"
  },
  "devDependencies": {
    "depcheck": "^1.4.3",
    "react-scripts": "^5.0.1"
  },

Thanks! Looking forward to your answer

The TextEncoder class is globally available in the browser; but appears not to be available in your testing environment.
This is a known issue in the jest JSdom environment; and we use this polyfill to make it globally available before running our jsdom tests (this is how we set it up in solid-client-js).
At a cursory glance create-react-app appears to use the same jsdom environment in which case the solution for you would be to polyfill the TextEncoder in your testing environment.

Thanks @jeswr. Adding these did the trick:
import { TextDecoder, TextEncoder } from "text-encoding";
global.TextEncoder = TextEncoder;
global.TextDecoder = TextDecoder;

Now I have a different issue, unrelated to Solid.
Cheers!


feat(CategoryTheory): a biproduct of epimorphisms is an epimorphism, etc

This is used in #18497 to prove that light condensed modules satisfy countable AB4*

Thanks!
bors merge


operation failed please check your inputs

I get this error message every time I try to send Tezos off my wallet.

Is it possible to update wallet without losing my wallet because I lost my private keys so I am in a tight situation please assist.

Please help I am scared I lost my funds forever.

I'm in the same situation

I had a similar problem when sending from a KT address to a tz1.
Try the following:

try adding some XTZ to the tz1 address you are sending to
try to set the destination tz1 address to "undelegated"  ( it needs xtz for this operation )
try to set the sender fee to "medium"

I am having all sorts of problems as well suddenly.  All past transactions no longer are showing, trying to re-delegate fails, and I can not transfer funds between my accounts.   My balances show up, but besides that everything appears frozen.

Quick update: I had been using a prior version of TezBox.  Updating to to 6.0 solved all the problems (naturally).

Quick update: I had been using a prior version of TezBox. Updating to to 6.0 solved all the problems (naturally).

@tw2ocean what platform are you on? I'm on Mac, and though I downloaded 6.0.2, when I open it, it shows v5.0.2 at bottom left

i found way to fix this error restore your wallet via tezbox chrome / firefox
https://tezoswebwallet.com/#!/restore
then you should switch node to
https://teznode.letzbake.com/

i found a fix to this isusse go to Tezbox wallet with Chrome or Firefox browser:
https://www.tezbox.io
and restore your wallet then go to setting and find better node


change no. of pixels calculation from 32 to 64-bit

Description
Fixes integer overflow bug when numpy calculates number of pixels in a series of pyramidal images using np.prod, which defaulted to a np.int32 type rather than the needed np.uint64 type.
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

References
closes #691
How has this been tested?
Large pyramids now work.
Final checklist:

[x] My PR is the minimum possible work for the desired functionality

This looks good to me, but I agree that there should be a test. It should be pretty simple, something like:
import dask.array as da

def test_data_size_calc():
    s = 8192
    data = [da.ones((s,) * 3), da.ones((s//2,) * 3), da.ones((s//4,) * 3)]
    assert is_pyramid(data)

Thank you @NHPatterson!

I've added the following test to test_is_pyramid():
    s = int(np.sqrt(2**32) + 2)
    data = [np.zeros((s,) * 2,dtype=np.bool_),np.zeros((s//2,) * 2,dtype=np.bool_),np.zeros((s//4,) * 2,dtype=np.bool_)]
    assert is_pyramid(data)

I avoided using dask arrays as @jni suggested because it would be another import in the test sequence but unfortunately with this test, the numpy array does involve making a large matrix, which I've given a np.bool_ data type to use fewer bytes. Hope it is useful.

@NHPatterson I think using numpy here is fine, though we do try and keep our tests to using small arrays where possible, and adding a dask import in the test isn't a big deal as we already depend on the package. @jni is this ok with you or would you prefer to see the test with dask?
@NHPatterson if possible can you install our precommit hooks using pre-commit install and then reformat those two files as the are failing our autoformatting tests of black and flake8. You can see more info in this part of our contributing guide. If that doesn't work out then we should be able to make a reformatting commit to your PR before merging

I agree with @sofroniewn. We depend on dask and I think it makes a lot of sense to use it here. Making a >2GB object during our test sequence seems wasteful. It's a substantial chunk of available RAM in many computers out there. (Note that np.bool_ arrays still take up one byte per element.)

Yes, no problem, I'll reconfigure the pull request when I have a moment. As another topic, I'm curious if just checking the number of pixels is the best test for pyramidal images.. Would it not also be wise to check each image's dimensions to make sure they decrease sequentially by a factor of 2 or 3 (not sure how many don't fall within this regime) and possibly warn if they don't?

Thanks! We can actually support decreases that are not factor of 2 or uniform, estimating the best place to swap the resolution based on the size of the tiles and current zoom level, though in practice factors of 2x are probably best and switching for other factors might not work so great

@NHPatterson we're close to making our next release and I'd love to get this in for that, if possible can you finish it up, otherwise I'm happy to take it over if you enable edits by maintainers on the PR. Thanks so much again for your contribution!

Ok I've added the dask based pyramid check and corrected the flake8 / black issues. I think editing by maintainers is allowed if there are any supplemental changes you'd like. Really excited to see where this project can go. After seeing #736, it's clear people are very hungry for a great microscopy data viewer in Python, myself included.

Hi @NHPatterson - thanks so much for your change - I was just fixing some of the " formatting changes, but have inadvertently updated your PR will all the recent changes from master too! I will revert my commit, but and then just fix the formatting, apologies and thanks so much again for the change!

Ok, following some git jiu jitsu I've now restored this PR to it's proper state and removed the " that had crept in through the auto-formatting. I will wait till tests pass and then approve and merge!! The git squash will forgive all my git sins, and only the diff in this PR will be captured as we want


link menu with css

menu is not linked with css

As a contributor of GSSOC'24 i would like to work on this issue.Could you please assign it to me.

I can the style the menu if you want


feat(nuxt): support custom keyed composables

🔗 Linked issue
resolves https://github.com/nuxt/nuxt/issues/12369
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This adds support for creating custom keyed composables. Keys will be injected as a final argument as long as the number of arguments is less than the maximum argumentLength.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

I really need this feature, especially for the composable function which wrapper the useAsyncData . THX!


Fix/remove components from deprecated folder

PR for issues:

https://quantum-machines.atlassian.net/browse/QUAL-669
https://quantum-machines.atlassian.net/browse/QUAL-670
https://quantum-machines.atlassian.net/browse/QUAL-673

@nulinspiratie Done.

Looks good now, there are some merge conflicts because I merged the other PR. Could you fix these and then merge PR @manas-qm ?


Stock Market Prediction Using LSTM

Stock Market Prediction using LSTM
This repository contains code for predicting stock market prices using Long Short-Term Memory (LSTM) neural networks. LSTM is a type of recurrent neural network (RNN) known for its effectiveness in handling sequential data, making it suitable for time series forecasting tasks such as stock price prediction.
Requirements

Python 3.x
Libraries: numpy, matplotlib, pandas, pandas_datareader, scikit-learn, tensorflow, yfinance

Instructions

Fetching Data: Historical stock market data is fetched using the Yahoo Finance API through the yfinance library.

Data Preprocessing: The fetched data is preprocessed by scaling it using Min-Max scaling to ensure all values are between 0 and 1. The data is split into training and testing sets, and further processed to be suitable for LSTM input.

Model Building: An LSTM neural network model is built using TensorFlow's Keras API. The model architecture consists of multiple LSTM layers followed by dropout layers to prevent overfitting.

Model Training: The LSTM model is trained using the training data with a specified number of epochs and batch size.

Prediction: After training, the model is used to predict future stock market prices. Predictions are made based on the last few days of historical data.

Visualization: Predicted prices are plotted alongside actual prices using matplotlib to visualize the model's performance.

Note

Ensure you have sufficient historical data for accurate prediction.
The model performance may vary based on factors such as market dynamics and the chosen hyperparameters.

Mention the issue no

@RacharlaSanjana it is already done by someone and it does not even assigned you
Please work on it after it assigned to you


Documentation should mention installing with npm option --legacy-peer-deps

Documentation Issue
An app won't work if it was installed from scratch and dependencies were installed with npm install command.
Payload admin page will go nuts with React errors.
--legacy-peer-deps seems to be required as seen in util.js from create-payload-app module.
Best would be to get rid of --legacy-peer-deps but docs should at least mention the option when installing payload package.
Additional Details
I'm using Node 14.16.1 and NPM 7.17

Before we go ahead and update the documentation, I would like to see if there are some dependency changes that need to happen to correct the underlying problem so we don't need to use --legacy-peer-deps. Ideally we sort it out and can remove the flag from  util.js of create-payload-app.
If somebody can take a look that would be greatly appreciated, otherwise I'll do so when I have time.
It has to do with this: https://twitter.com/wesbos/status/1359173446740172810?s=19

Use yarn 😅?

Use yarn 😅?

Preach!

Use yarn 😅?

Sure, but as much as I love yarn, this should at least be documented.
At the moment, the "From scratch" section in the documentation gives instructions that doesn't work.

I would offer to help but every time I've tried to fix peerDependencies I've ended up making things worse, so I just leave them. If anyone could enlighten me as to how you should approach fixing them I'd happily submit a pull request.

I've just gone through this and unfortunately Payload relies on a few packages that need to properly add support for React 17, but everything works properly.
So, I've just gone through and updated the docs to describe that you need to use --legacy-peer-deps with npm for now. I've requested that the offending packages add proper support and once these packages update, we'll remove this new mention from docs.
👍


Feature/add inspect brk support

Inspect Break Support (--inspect-brk)
This PR adds support for --inspect-brk which is especially useful when you want to attach to the  server before any code execution using your debugger ie. vscode.
Example Workflow:

Start the server with razzle start --inspect-brk and wait for the compilation.
You will be prompted with the following  message when ready to attach a debugger:

Debugger listening on ws://<IP_ADDRESS>:9230/1a901ec9-ce71-4236-89a5-39107dcca3d9
For help see https://nodejs.org/en/docs/inspector

With Visual Studio Code you can use the following configuration in launch.json to attach to your server:

    {
      "name": "Attach to Server",
      "type": "node",
      "request": "attach",
      "port": 9230,
      "restart": true
    }

Cool

@jaredpalmer Do you think you will merge this?


Sort configs by creation time before returning

This reduces impact of conflicting virtual services, as the first created one
is respected. In addition, fix the EnvoyFilter implementation to merge multiple
configs into one big array of configs instead of arbitrarily picking a matching
workload config.
Signed-off-by: Shriram Rajagopalan<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [rshriram]

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

@rshriram: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
f367e4ef707d628470ae6a2a480a2a18764332ed
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
f367e4ef707d628470ae6a2a480a2a18764332ed
link
/test istio-unit-tests

prow/istio-presubmit.sh
f367e4ef707d628470ae6a2a480a2a18764332ed
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Lookit! I improved code coverage by 1% 72% (+1%) compared to 4203552 !!!


added own segment control plugin

Added https://pub.dartlang.org/packages/cupertino_segment_control to the flutter base.

@googlebot I signed it!

This will need documentation and tests.

@theobouwman Do you know if you will have time to work on this patch? It's fine if not, we can take it over.

@theobouwman I'm going to close this PR for now. If you have time to work on it please don't hesitate to reopen it or submit a new PR. We will hopefully get to this in due course if you don't have time. Thanks for the contribution so far!

@theobouwman – Looks like this was fixed! ^


Update examples

Description
Update examples to use latest tag.
Types of changes

[x] Documentation (add or update README or docs)

Checklist:

[ ] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Description
Update examples to use latest tag.
Types of changes

[x]  Documentation (add or update README or docs)

Checklist:

[ ]  My code follows the code style of this project.
[x]  My change requires a change to the documentation.
[x]  I have updated the documentation accordingly.
[ ]  I have read the CONTRIBUTING document.
[ ]  I have added tests to cover my changes.
[ ]  All new and existing tests passed.


'bottomLayoutGuide' was deprecated in iOS 11.0

Please consider submitting the following information (if relevant):

Library setup method: CocoaPods
Version of the library. Example: 10.0.0
Xcode version. Example: 9.0
OS version. Example: iOS 11

I have no idea to resolve this issue. sorry. 😣

Hi, thanks for reporting. We need to fix this. For the time being, the workaround is to choose a deployment target to iOS 10 or earlier.

Fixed in version 11, which introduced some breaking changes.


Allow a controller to exist in multiple areas

At present a controller may only exist in a single area using the AreaAttribute; it would be good to allow a controller to exist in multiple areas and/or all areas, instead of having to create a Controller that derives from a base class for each area.

I don't believe we have any plans to support this, it hasn't been a common request. Usually people dislike the old MVC1-5 behavior where by mistake a controller might end up in multiple areas!
@rynowak thoughts on this?

@Eilon if you could decorate a controller explicitly with multiple areas that would be great, or would it be better to have multiple route for the same controller with a prefix that is the same as a area or a preminant redirect?

@grahamehorner I'll discuss with the team and we'll get back to you.

You can do this btw with a convention. /cc @jbagga - relevant to your work

Ah, if we have a neat snippet of code that does this, I think that's ok for now. This isn't an area (ahem) that we've gotten a lot of feedback on, so a sample might be sufficient in the meantime.

Assigning to @jbagga to come up with a sample (if possible!).

Yep. My hope is that we can make a quick sample and then use it as a test. @jbagga - I'm on the hook to help out with this if you get stuck

Working on a sample

@grahamehorner Here is the sample https://github.com/aspnet/Entropy/tree/dev/samples/Mvc.ControllerForMultipleAreasSample
Closing for now since we have a sample that addresses the issue.


Build Order Ideas

Add some more functionality to build orders
Scrap Output
Add the ability to "scrap" an output

The build output still exists in stock (but with status 'destroyed' etc)
Does not count towards the "completed" quantity
Any parts allocated to that build are still "used up"

It would be a great feature for prototype builds!


FFerror: Failed to download videos, maybe the network is not available!

How do I solve this error?
FFerror: Failed to download videos, maybe the network is not available!

You can access the Pexels API yourself.


refine() breaks deepPartial()

deepPartial() is not actually deep, if nested object has refine() on it.
Example 1, works:
const test1 = z
  .object({
    inner: z.object({
      data: z.string(),
    }),
  })
  .deepPartial();

type Test1 = z.infer<typeof test1>;

/*
type Test1 = {
    inner?: {
        data?: string | undefined;
    } | undefined;
}
*/

Example 2, doesn't work, innerWithRefine is not partial:
const test2 = z
  .object({
    innerWithRefine: z
      .object({
        data: z.string(),
      })
      .refine((value) => true),
  })
  .deepPartial();

type Test2 = z.infer<typeof test2>;

/*
type Test2 = {
    innerWithRefine?: {
        data: string; // is not optional, but it is supposed to be
    } | undefined;
}
*/

confirmed

Not a bug, this is behaving as documented.

@colinhacks the screenshot in your reply didn't work 👀


Add support for rich text snippets.

Snippets should support rich text (i.e. with formatting, as opposed to  plain text).

Nobody seems to be missing this. Closing.

I was actually going to suggest this feature, would it be hard to implement? i think simple bold/italic, etc would go a long way towards making the app useful in more use cases.

i agree!


Browser zoom causes grid row misalignment

Short description
When browser zoom is not 100% (could be less, could be more), the grid rows can become misaligned when using frozen columns or select column or row number column. I should mention that our w2ui 1.5 code pull was over a year ago.
What is current behavior
If either row number column, row selection column, or frozen columns are enabled and browser zoom deviates from 100%, the left side columns become misaligned with the remaining columns of grid.
What is desired behavior
The desired behavior would be grid row is uniform between every column of the grid despite browser zoom factor.
Link to jsfidle/codepan with sample code
...
Steps to reproduce or sample
Fill grid with more than 20 rows
Ensure line numbers and select column is enabled
zoom browser to 125%

Do you have any screenshots? Any clues how to fix it? I am afraid that we have to move from tables to div (display: grid) to fix it and it is a big move.


Add Debezium

This PR introduces debezium support.

Thanks for raising the PR @TFMV!
A couple of points:

Can we add in debezium support with minimal configuration via environment variables?
Can we connect to the existing kafka or postgres instance?
Need to update the README to show debezium support
Create a new service type called change data capture (CDC) in the README and docker-compose.yaml
I see you have also added in redpanda, can you include that under messaging systems in the docker-compose.yaml and in the README

Sure thing. I've closed the existing PR and will address these items prior
to resubmitting.
Thomas F. McGeehan V
On Sat, Jun 8, 2024 at 10:02 PM Peter Flook @.***>
wrote:

Thanks for raising the PR @TFMV https://github.com/TFMV!
A couple of points:

Can we add in debezium support with minimal configuration via
environment variables?
Can we connect to the existing kafka or postgres instance?
Need to update the README to show debezium support
Create a new service type called change data capture (CDC) in the
README and docker-compose.yaml
I see you have also added in redpanda, can you include that under messaging
systems in the docker-compose.yaml and in the README

—
Reply to this email directly, view it on GitHub
https://github.com/data-catering/insta-infra/pull/17#issuecomment-2156287411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYLI2N4S2VRYHOBXZ5MTI3ZGPASTAVCNFSM6AAAAABI77DNNKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGI4DONBRGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


[JENKINS-29537] Take advantage of Run.getEnvironment calling ParametersAction.buildEnvironment

Downstream of https://github.com/jenkinsci/jenkins/pull/2993.
@reviewbybees

https://repo.azure.jenkins.io/public/ service outage?

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@reviewbybees done

@abayer assuming CI passes, can this just be merged, since we are already basically using newer baselines in master for this plugin anyway, and if we want to backport something we could just make a 2.73.x branch?

@jglick Once you resolve the merge conflict, yup.

Is there something holding this up?


POC Point subclass

This makes pygeos.Geometry subclassable and uses tp_base to quickly check if an object is a Geometry subclass.

This now also creates the Points from the ufuncs. I didn't see any performance degradation (but more testing is needed still)

Some better benchmarks:
import pygeos
import numpy as np
c = np.random.random((10000, 2))
%timeit pygeos.points(c)
2.72 ms ± 15.9 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)  # master
3.06 ms ± 29.5 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)  # PR
g = box(0, 0, 1, 1)
%timeit pygeos.intersection(g, p)
139 ms ± 419 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)  # master
139 ms ± 464 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)  # PR

So, you can measure the difference, but it is only 0.033 nanoseconds of additional overhead per geometry construction. I think we an take that performance hit, as the benefit to the pygeos API would be great.

This is really cool. Will try to test it a bit more myself as well the coming days!

I also did some similar timings with my laptop tuned for performance checking with pyperf, and with the basic pygeos.points as you did above (which should have little other overhead), I can confirm similar results:
For me, it gives around 33% overhead (338ms instead of 253ms for 1 million points, but again, constructing points is very cheap, in actual operations when calculating eg an intersection, the relative cost will be much smaller). For me this gives 85ns overhead per geometry (compared to your 33ns, the 0.033ns you noted should be 0.033µs).
So I agree that, although measurable, this limited slowdown is certainly something that seems worth it.

And as additional test, I quickly added a LineString subclass, to test something more "costly". Creating linestrings of each 100 coordinates pairs from a numpy array, I get a similar overhead per geometry (~67ns), but now this means only a relative performance hit of 1.3%

Ok so let’s do this! I think we should copy as much of the shapely API as we can.
Maybe 1 PR per type?

I was experimenting a bit more (also in Shapely), and ran into the following issue: if we have an additional class  in the hierarchy (which we will want, as a large amount of the properties and methods are shared accross all geometry types), you get errors.
Eg dummy example:
class BaseGeometry(Geometry):
    @property
    def type_id(self):
        return get_type_id(self)

class Point(BaseGeometry):
    @property
    def x(self):
        return get_x(self)

    @property
    def y(self):
        return get_y(self)

class LineString(BaseGeometry):
    @property
    def num_points(self):
        return get_num_points(self)

lib.registry[0] = Point
lib.registry[1] = LineString

Then the geometry class recognition does not work:
In [2]: p = pygeos.points(0, 1)

In [3]: p.x
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-3-7d725ae8e4e9> in <module>
----> 1 p.x

~/scipy/repos/pygeos/pygeos/geometry.py in x(self)
     47     @property
     48     def x(self):
---> 49         return get_x(self)
     50
     51     @property

~/scipy/repos/pygeos/pygeos/geometry.py in get_x(point)
    239     nan
    240     """
--> 241     return lib.get_x(point)
    242
    243

TypeError: One of the arguments is of incorrect type. Please provide only Geometry objects.

I suppose some check with tp_base needs to adjusted for this, but didn't yet look in detail at it.

Found this in the cython generated code:
int __Pyx_InBases(PyTypeObject *a, PyTypeObject *b) {
    while (a) {
        a = a->tp_base;
        if (a == b)
            return 1;
    }
    return b == &PyBaseObject_Type;
}

so using this logic instead of type->tp_base != &GeometryType in get_geom fixes it (if we know the exact class hierarchy, we can of course also simplify it)

Ok so let’s do this! I think we should copy as much of the shapely API as we can.
Maybe 1 PR per type?

I am not sure what is best, but I am currently experimenting with directly doing this in Shapely (https://github.com/Toblerity/Shapely/compare/master...jorisvandenbossche:geometry-subclasses?expand=1).
We need to decide in Shapely how we want to approach this (start somewhat fresh inside the repo, as you proposed), or more gradually. But in any case, I would personally mostly focus on Shapely when it comes to the subclasses.


Article headlines breaking at strange points

Need to address headline wrapping of articles heds on mobile (tested on iPhone)

Should be checked site-wide. May be related to this CSS.

I've sort of addressed this in #135 and things are looking a little better. I have some ideas for making things look better on small screens that I'd like to discuss.

@pfulton - I'm guess the change you made here was making the side wider is that the case? If so, I added issue #161 to address additional margin adjustments with Jos.


Revert cast platform polling mode

Description:
We used to rely on the cast device notifying us of all media status changed via callbacks. However, some apps like Netflix sometimes didn't send those callbacks even though the state had changed. In #13275, I introduced a semi-polling mode for the cast platform: we would still respond to all callbacks, but in addition we would poll the chromecast device every 30 seconds for the current state to resolve the Netflix media status issue.
However, it turned out that some other apps on the other hand don't like the polling mode at all :(. When polled for the current state via a GET_STATUS request, they responded with the media status of the most recent callback media status, not the actual media status (but only for a some apps 🤬). As there's no information in the MEDIA_STATUS messages to which timestamp the currentTime attribute corresponds, there's no good way to solve this IMHO.
This PR reverts #13770 and partially reverts #13275 to go back to the old callback-only method. It has its own issues, but is a lot better than the issues mentioned in #13770.
I must say, the cast protocol is extremely weird with lots of these things. I seriously would have expected a better quality out of one of google's products.
Related issue (if applicable): fixes #
Example entry for configuration.yaml (if applicable):
media_player:
  - platform: cast

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Do we know if there are any apps that will send MEDIA_STATUS updates periodically even if polling isn't used? If updates are received then the new_media_status() calculations can be kept to avoid unnecessary state attribute updates. But that code is not needed if we do not get MEDIA_STATUS updates.

All apps I've seen only send MEDIA_STATUS if something has actually changed. Though some (like Netflix) don't even send a MEDIA_STATUS message even if something changed (hence the half-polling mode).

@dersger I have YouTube and the default media receiver do the same during my testing... And it's true there is this old current_time callback but that only seems to happen when connecting (AFAIK pychromecast sends a GET_STATUS request when a connection is established)

I didn't get any problem when I used YouTube from an Android phone. But I do get the polling problem when using screen mirroring from the phone, or if using mirroring from a Chrome browser in Win10. I get media_position 0.0 and state playing repeatedly during mirroring.

I think that if we want to re-introduce this behavior again in the future, we should limit it to apps that are known broken, like netflix.


Mismatch of information

Under Deploying the Collector, it says The operating system is Windows Server 2016 (64 bit).
In OS License, it says The collector comes with a Windows Server 2012 R2 evaluation license which is valid for 180 days.
In OS license, shouldn't it be 2016 ?

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

ID: 089d527e-1632-9d34-1c40-756e30eb2ac6
Version Independent ID: ee2d12cd-e7ee-a50b-9ce1-496a6053e243
Content: About the Collector appliance in Azure Migrate
Content Source: articles/migrate/concepts-collector.md
Service: azure-migrate
GitHub Login: @snehaamicrosoft
Microsoft Alias: snehaa

@Daya-Patil, Thanks for the question! We are investigating and will update you shortly.

@snehaamicrosoft , any thoughts on this?

Yes, thanks for highlighting it, I will get this fixed.

Have checked in the fix, it will go live today. #please-close


ReactiveController: adding/removing controllers in lifecycle methods affects other controllers

Which package(s) are affected?
Lit Core (lit / lit-html / lit-element / reactive-element)
Description
If a ReactiveController removes itself or adds another controller during a lifecycle method like hostUpdate or hostUpdated, this can prevent other controllers from properly running.
Reproduction
The initial text of the button in this repro should be the following and then clicking the button should not change the text.
Expected:
controller1.count: 1
controller2.count: 1

Actual:
controller1.count: 1
controller2.count: 0

(after clicking once)
controller1.count: 1
controller2.count: 1

Workaround
The issue is that the controllers are stored in an Array and to process the controllers' lifecycle, that array is forEach'd. Mutating the array while it's iterating unexpectedly alters the iteration.
The only workaround is not to alter the controllers list inside a controller lifecycle method.
This could possibly be addressed by (1) copying the controllers array before iterating, or (2) making the controllers a Set.
Is this a regression?
No or unsure. This never worked, or I haven't tried before.
Affected versions
All
Browser/OS/Node environment
Browser: All

Ouch. Yeah, it's a concurrent modification bug. Set seems like the right way to go.


comment out container

See https://github.com/emberjs/ember.js/issues/15322#issuecomment-334132107
This will buy us some time with ember-model and should unblock our upgrade to 2.13.4. We're going to have either address this in ember-model or migrate fully to Ember Data before Ember 3.0.

Should we just delete it?

As we're deviating from upstream, I think I'd rather leave a clearer paper trail in code.


Add missing link to Semaphore X account

Related Issue
See #477

Does this introduce a breaking change?

[ ] Yes
[x] No

Yes, I agree, I think it is better in the footer. Maybe before Discord.


Create development branch environment

I created a newClient branch. All features should be developed on sub-branches and merged with PR.


Cannot create dns ClusterIssuer with DigitalOcean provider

Describe the bug:
While creating ClusterIssuer with this config:
apiVersion: certmanager.k8s.io/v1alpha1
kind: ClusterIssuer
metadata:
  name: letsencrypt
spec:
  acme:
    server: https://acme-v02.api.letsencrypt.org/directory
    email<EMAIL_ADDRESS>    privateKeySecretRef:
      name: letsencrypt
    dns01:
      providers:
        - name: dns
          digitalocean:
            tokenSecretRef:
              name: digitalocean-dns
              key: access-token

....

kubectl create secret generic digitalocean-dns -n=kube-system --from-file=access-token
kubectl create -f cluster-issuer.yml

cert-manager logs:
1 controller.go:140] clusterissuers controller: syncing item 'letsencrypt'
1 helpers.go:147] Setting lastTransitionTime for ClusterIssuer "letsencrypt" condition "Ready" to 2018-11-26 02:08:23.445923986 +0000 UTC m=+10271.676525736
1 controller.go:154] clusterissuers controller: Finished processing work item "letsencrypt"
1 controller.go:171] certificates controller: syncing item 'rdk/phpmyadmin-reportingdokapsy-cz-cert'
1 sync.go:120] Issuer letsencrypt not ready

ClusterIssuer description:
Name:         letsencrypt
Namespace:
Labels:       <none>
Annotations:  <none>
API Version:  certmanager.k8s.io/v1alpha1
Kind:         ClusterIssuer
Metadata:
  Creation Timestamp:  2018-11-26T02:08:21Z
  Generation:          1
  Resource Version:    80905
  Self Link:           /apis/certmanager.k8s.io/v1alpha1/clusterissuers/letsencrypt
  UID:                 25f73e31-f120-11e8-9375-de1a20202003
Spec:
  Acme:
    Dns 01:
      Providers:
        Name:  dns
    Email<EMAIL_ADDRESS>    Private Key Secret Ref:
      Key:
      Name:  letsencrypt
    Server:  https://acme-v02.api.letsencrypt.org/directory
Status:
  Conditions:
    Last Transition Time:  2018-11-26T02:08:23Z
    Message:               Resource validation failed: spec.acme.dns01.providers[0]: Required value: at least one provider must be configured
    Reason:                ConfigError
    Status:                False
    Type:                  Ready
Events:                    <none>

The same ClusterIssuer works ok with route53 provider.
Expected behaviour:
Creating ClusterIssuer should work.
Steps to reproduce the bug:

install cert-manager 0.5.2
create DO secret
create ClusterIssuer

Anything else we need to know?:
no
Environment details::

Kubernetes version (e.g. v1.10.2): 1.11.4
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): scaleway
cert-manager version (e.g. v0.4.0): 0.5.2
Install method (e.g. helm or static manifests): helm (pharos-cluster)

/kind bug

I think cert-manager version 0.5.2 doesn't support DigitalOcean. I had the same issue but upgrading to the latest dev version (using the with-rbac.yaml file from here) fixed it for me.

Ah, I thought I saw it in release notes, but maybe I misread the information.


Replace vast::directory with std::filesystem in lsvast tool

:notebook_with_decorative_cover: Description

Problem:

lsvast tool uses vast::directory which is going away soon.

Solution:

Use std::filesystem::directory_iterator instead.
Replace uses of vast::path with std::filesystem::path as well.

:memo: Checklist

[ ] All user-facing changes have changelog entries.
[ ] The changes are reflected on docs.tenzir.com/vast, if necessary.
[ ] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions

File-by-file.

Haven't quite figured out why there is linker errors when building the lsvast binary. I can repro it locally from what CI is seeing:
Undefined symbols for architecture x86_64:
  "print_index(std::__1::__fs::filesystem::path const&, indentation&, formatting_options const&)", referenced from:
      print_vast_db(std::__1::__fs::filesystem::path const&, indentation&, formatting_options const&) in lsvast.cpp.o
      l_constinit in lsvast.cpp.o
  "print_partition(std::__1::__fs::filesystem::path const&, indentation&, formatting_options const&)", referenced from:
      print_vast_db(std::__1::__fs::filesystem::path const&, indentation&, formatting_options const&) in lsvast.cpp.o
      l_constinit in lsvast.cpp.o
ld: symbol(s) not found for architecture x86_64
clang-11: error: linker command failed with exit code 1 (use -v to see invocation)

Any ideas, @dominiklohmann? Not specific to GCC as I can repro it using Clang 11.1.0.

Nice work!
For your linker error, you have a signature mismatch for all printer functions:
-void print_index(std::filesystem::path path, indentation& indent,
+void print_index(const std::filesystem::path& path, indentation& indent,

Thanks for the extra set of eyes. Definitely didn't see that obvious error last night. Just fixed it up.

ping @dominiklohmann -- pipeline should pass now. Thanks for the review.


How to handle socket connection on  Server

I am building Socket TCP Server for laravel .
I want to keep all the connections alive till i don't do connection->close(). But when i do connectionWrite->write ("Hello world") the message is depicted on client screen but connection is also closed. I don't want the connection to die. I want to reuse the connection in future transaction with server. I don't want to close the connection as it does on its own. I am using Postman like Nightingale for testing( as a client).

Hey @ChintanStratAero, thanks for bringing this up 👍
It looks like what you're seeing is normal HTTP behavior. After the client requests something and the server responds there's no need for the client to leave the connection open.  After receiving its answer the client may close the connection.
If you're using postman for testing purposes you're most likely speaking HTTP. You can take a look at https://github.com/reactphp/http, this may help!

I believe this has been answered, so I'm closing this for now. Please come back with more details if this problem persists and we can always reopen this :+1:

Yes sure thanks once again.
On Fri, 18 Mar, 2022, 1:20 AM Christian Lück, @.***>
wrote:

I believe this has been answered, so I'm closing this for now. Please come
back with more details if this problem persists and we can always reopen
this 👍
—
Reply to this email directly, view it on GitHub
https://github.com/reactphp/socket/issues/287#issuecomment-1071301190,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFMBVZXHIS7PZ5PPWFLHLHLVAOEGTANCNFSM5QY23RMA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Simon,
Thank you your response helped me. I created client in php which is
connected for ever and it working.
Thanks one more time..
On Thu, 17 Mar, 2022, 4:31 PM Simon Frings, @.***>
wrote:

Hey @ChintanStratAero https://github.com/ChintanStratAero, thanks for
bringing this up 👍
It looks like what you're seeing is normal HTTP behavior. After the client
requests something and the server responds there's no need for the client
to leave the connection open. After receiving its answer the client may
close the connection.
If you're using postman for testing purposes you're most likely speaking
HTTP. You can take a look at https://github.com/reactphp/http, this may
help!
—
Reply to this email directly, view it on GitHub
https://github.com/reactphp/socket/issues/287#issuecomment-1070792161,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFMBVZSUDX5WKOWSCVVZ3CLVAMGG5ANCNFSM5QY23RMA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


limit?

do you know of any limit as to how many requests per minute is allowed by Whatsapp? I just dont like to try myself and risk getting blocked... I don't mean sending messages but just check online status.

@thebeautiful checking online status there isnt a risk, but yes requesting last seen time, but you can calculate the last seen time by comparing when was the last time the user was online :smile:


specify transport

I am not sure if it is the right place to ask a question here:
Assume that on my server side I support only polling, (nodejs)
var io = require('socket.io')(server,{transports:(['polling'])});
how can I specify in the socket-io-client-cpp that I would like to connect to server using "polling"?
When I use
Sio::Client h;
h.connect("http://<IP_ADDRESS>:80");
This seems to use "websocket". In this case an underlying transport error is reported.

This is old issue, but I also came with idea to use socket.io c++ library. I need http fallback supporting when Websockets are blocked by firewall. Is it possible? does this library support fallbacks?

Closing as duplicate of #251


Suggestion: use e.g. with warnings.catch_warnings() in favor of module-level warnings suppression

https://github.com/ziatdinovmax/pyroVED/blob/185404796d00a2efe3063d6ab84b002496b7befd/pyroved/nets/conv.py#L20
I'd recommend against module-level warnings suppression in favor of
with warnings.catch_warnings():
    warnings.simplefilter('ignore')

at the precise spot where the warnings are thrown.
Thoughts?

Yes, this would be a more proper way of doing this. I think here we are suppressing the annoying warning about the align_corners from torch.nn.functional.interpolate.

Yeah this is definitely more nitpicky an issue for sure... I have had my experiences with a crazy number of warnings from nn.functional too. If at some point I can pin down exactly where the errors come from (and ensure I don't miss any), I will make this change. For now I'd say this is low priority. After all, it's only suppressing warnings, not errors, which by definition shouldn't really impact the code.


Add integration test for ingress feature

We need to add some integration tests after the ingress integration feature is merged.
See https://github.com/canonical/glauth-k8s-operator/pull/78

@wood-push-melon what do you see these integration tests looking like? IIRC it's not very straightforward to deploy both a microk8s and LXD cloud within an action runner, so testing against SSSD might be difficult. We could just use a netcat scan to make sure that the ports are actually open on the load balancer.


Updating to set storage performance to Standard for imported works

coverage: 84.116% (+0.03%) from 84.083%
when pulling 067ebb56b035d2437cb5b1d0c9b690e333b9316c on import-storage
into 34f34eb607a115807c213db9325cc80579e8a028 on main.


fix(website): enable crawling of description on the new layout

Proposed Changes
Setup data-coveo-field attributes on elements of the page just for the new layout. I figured we could create field mapping that map multiple metadata to the same field, making it easy to have different selectors for the old and the new layout.
The new webscraping configuration will look like this
"metadata": {
    "description": {
        "type": "CSS",
        "path": "[data-coveo-field='description']"
    },
    "title": {
        "type": "CSS",
        "path": "[data-coveo-field='title']"
    },
    "legacy_description": {
        "type": "CSS",
        "path": "h4.admin-description::text"
    },
    "legacy_title": {
      "type": "CSS",
      "path": ".panel-header h4.bolder::text"
    }
}

Potential Breaking Changes
None
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Good thinking!! :clap: :tada:


WRLDAEMS-62: Footer feedback

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://main--world-bank--aemsites.hlx.live/
After: https://WRLDAEMS-62--world-bank--aemsites.hlx.live/

All changes are done. Kindly verify.


Add more apple types

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

cc @JohnTitor

@bors r+

:pushpin: Commit 59652cf92b3a460d998a8934019aa023f279aadc has been approved by Amanieu

:hourglass: Testing commit 59652cf92b3a460d998a8934019aa023f279aadc with merge e87db9325681cd1df2659b7fd51da1330a2533d6...

:broken_heart: Test failed - checks-actions

@bors retry

:hourglass: Testing commit 59652cf92b3a460d998a8934019aa023f279aadc with merge 04b02a5a5e37239826e9e0c88a35a8140083795c...

:broken_heart: Test failed - checks-actions

The failure seems legit since I'm computing since inside a const. What would be the correct way to do so otherwise?

Silly me: the answer was simply to use macros. So I created a new macro to handle const definition. :)

@bors r+

:pushpin: Commit bc7cc8266c5a96a0d90edb377468311e4af0f4d9 has been approved by Amanieu

:hourglass: Testing commit bc7cc8266c5a96a0d90edb377468311e4af0f4d9 with merge 8fa1d6c4e9638c13574f0d4a30a661eb811bd648...

:broken_heart: Test failed - checks-actions

So macros don't compute the value directly. Sadness...

Updated!

@bors r+

:pushpin: Commit aa56cfe49fdde722404794d7f0ae15e55bd8f510 has been approved by Amanieu

:hourglass: Testing commit aa56cfe49fdde722404794d7f0ae15e55bd8f510 with merge a50d9e87b1b9c8f6dda88218b3a4a010b4952de9...


[RESTinio] Update to v.<IP_ADDRESS>

RESTinio updated to the latest version with several fixes.

/azp run

/azp run

/azp run

Oh, oops, I was looking at the wrong tests -- this passed all modern tests.


e2pg: db migrations

Today you have to supply the -reset flag on first use for e2pg to initialize it's schema. This is unsavory for several reasons:

You may accidentally reset your schema when you don't want to
You may have an existing database that you want e2pg to write to
Upgrades are not possible

The safest thing to do at this point is to manually migrate the database in a SQL console. Which is terrible. E2PG ought to have a -migrate flag that compares the schema in the target database against the schema in the E2PG binary and then updates the target database to be current with the binary.

No longer an issue with #142


checking on status of posting to Rogue with Gateway

BUG
Current Behavior
When submitted a reportback to Rogue, we currently check the code of the response that we get back from Gateway. However, what gets returned back to us is not the full response but is just the reportback. So, when we try to get the response code, we get nothing, which makes us think that the task failed. The reportback ends up in the normal table AND the failed task table, and the user does not end up on the permalink page, even though their reportback was successfully posted.
Desired Behavior
We don't want reportbacks that were posted successfully in the failed task log, and we want users to hit the permalink page if their reportback was successfully posted.
Steps to Replicate
Turn on Rogue and submit a reportback.

 As a developer, I want to check for Rogue failures in a way that works with the Gateway response.


Process received messages in queue

Creating early PR to gather feedback. Want to hear what  @thelounge/maintainers think.
Connecting to ZNC with history playback no longer freezes my browser, and appends messages in small queues instead of as-soon-as-received.
What I want to achieve is to process multiple messages for one channel at once, and append them at once (document fragments). Thus reducing DOM manipulations and style recalculations. This should also prioritise currently active channel.

[x] Sort messages per channel
[ ] Prioritize currently active channel

[ ] How does this handle for PMs/whois which force a new window to appear?

[x] Use document fragments to append multiple messages at once
[x] Trigger events (notifications, sticky scroll) once per processed channel

[ ] Look more into notifications

[x] #96 Remove 10 second interval timer to hide messages in hidden channels
[x] Limit notification sound to play every 500 milliseconds
[ ] Fix condensed stuff

@xPaw, adding the do not merge label for now, feel free to assign folks and remove the label when you're done with it.

Did another optimization pass, and removed that ugly 10 second interval timer to hide messages in non visible channels (#96).

Rebased with master. Now back to boarding a plane 😄

Alright, so I started looking at the code (not tested yet) and I am so confused 😅.
It's some very convoluted/complex code. I'm not criticizing, I know it's for good reasons, but it makes reviewing rather difficult and pretty sure within 2 months after merging we'll be wondering why this or that change :)
Any way you could comment this code, @xPaw? I know it takes some efforts to do so, but I don't think I can fully understand the changes right now.

The main idea of this PR is to append messages to DOM on browser idle callback (giving it time to breathe), and while it might be waiting for the idle event to callback, multiple messages can be received, and if they are, as such, they are processed in a single go. This gives a very nice boost for multiple messages in a single channel, as they get appended to actual DOM with single operation.

Does not work on iOS Safari:
ReferenceError: Can't find variable: requestIdleCallback

@kode54 Fixed. However, since Safari does not have requestIdleCallback, it will not benefit from this PR.

@xPaw, I see some empty checkbox in here, do you still want to work on this? I wouldn't like to start reviewing 300+ lines of changes if you happen to still edit it :)

@astorije these checkbox might not get even done (the code is fine as is).

I have rebased this somewhat, but this certainly breaks condensed messages due to the use of document fragments.

@xPaw, what do you think for this PR? Do you need something else done with condensed message so this PR is compatible? I understand you want your other PRs merged before taking up any more work, which is totally fair, do you want to close this PR until condensed stuff makes it possible again?
I really want this and your other work merged too, not sure how you want to proceed.

@astorije No reason to close this.


[ISSUE #XXX] [[ not found, when starting namesrv

[[ is bash buildin. use [ in sh when comparing numbers.
Make sure set the target branch to develop
What is the purpose of the change
[[ not found, when starting namesrv
Brief changelog
change [[ to [
Verifying this change

Coverage decreased (-0.08%) to 51.777% when pulling 208f099ec6700756db44b3dd74e0f22694f50814 on Jack0005:patch-1 into 39bb9386f10d5d8dfe81183c172a3a86f6d313bd on apache:master.

@Jack0005 there's conflict in this PR, could you resolve these conflicts first?

it is already fixed in branch: develop.


Fix for #481 - PlatformService should only work with systemInfo part, not with whole UserAgent string

Or else we will have some weird bugs, like detecting "axios/0.19.2" UserAgent as iOS device.

Can you rebase?

Seems to be fixed in #492 differently, so this PR is not needed anymore

Thank you for the update @rufanov


[Bugs?] webconsole [404 page not found]

Describe the bug
Hello, I follow the installation instructions https://free5gc.org/guide/3-install-free5gc/ to install according to the steps, the version is 3.3.0, there is no error in the process, except that there will be warnings during the webconsole installation process, such as As shown in the first picture, but I don’t think this is the reason. When I execute the webconsole browser after the installation is complete, the browser will return 404

To Reproduce
I follow the installation instruction https://free5gc.org/guide/3-install-free5gc/ Follow the steps to install until the last step，In order to execute the webconsole I execute the following command
cd /webconsole/bin ./webconsole
But it will report an error

So I copy a copy from /free5gc/config/webuicfg.yaml to webconsole/bin/config/webuicfg.yaml, config is a folder created by myself
execute it again
show

Looks like it was executed successfully，Then I try to go to the browser and enter http://<IP_ADDRESS>:5000/, <IP_ADDRESS> is the machine ip for my deployment service. After inputting, an error 404 page not found will be reported, as shown in the figure below

Environment:

free5GC Version:3.3.0
OS: Ubuntu 20.04 Server
Kernel version: 5.4.0-153-generic
go version: go version go1.17.8 linux/amd64

Please let me know if you need any other information, thank you

By the way, I recently had a need to add a packet filtering function to UPF and maybe modify it to ULCL.  how do you usually modify the code and test it? Is there a SOP? Thanks for answering

@mark6969
It's better to execute go run server.go in free5gc/webconsole.

@mark6969
Packet filtering is not implemented in free5gc yet. If you set the IP filter in webconsole, it's effectless now.

@mark6969 It's better to execute go run server.go in free5gc/webconsole. See https://free5gc.org/guide/5-install-ueransim/?h=uera#4-use-webconsole-to-add-an-ue

It is possible to implement according to your suggestion, I should not have noticed, thank you

@mark6969
I checked again and found that IP filter will exist in PFCP Session Establishment Request -> PDI -> SDF Filter. But we don't write relevant test yet. I use the webconsole PR #47

@brianchennn
Thanks for the reply, I have a question, whether the function of SDF FILTER has the function of filtering GTP-U, such as judging whether it is legal traffic based on the source and destination IP of the traffic

@mark6969
For IP Filter test, you can:

Config DNN in webconsole, setting IP Filter to : permit out ip from <IP_ADDRESS> to <IP_ADDRESS>

make sure <IP_ADDRESS> can ping from UPF, you can change yourself
make sure your UE's IP is <IP_ADDRESS>

In free5gc machine,

cd free5gc/test/go-gtp5gnl/bin;
sudo su
./gtp5g-tunnel list far
You will find that there are 4 FAR rules:

{
"ID": 3,
"Action": 2,
"Param": null,
"PDRIDs": [
3
],
"BARID": null,
"SEID": 1
},
{
"ID": 1,
"Action": 1,
"Param": null,
"PDRIDs": [
1
],
"BARID": null,
"SEID": 1
},
{
"ID": 2,
"Action": 1,
"Param": {
"Creation": {
"Desc": 256,
"TEID": 1,
"PeerAddr": "<IP_ADDRESS>",
"Port": 2152
},
"Policy": null
},
"PDRIDs": [
2
],
"BARID": null,
"SEID": 1
},
{
"ID": 4,
"Action": 2,
"Param": {
"Creation": {
"Desc": 256,
"TEID": 1,
"PeerAddr": "<IP_ADDRESS>",
"Port": 2152
},
"Policy": null
},
"PDRIDs": [
4
],
"BARID": null,
"SEID": 1
}
    - In UERANSIM, connect to free5gc:
        - (terminal 1): `build/nr-gnb -c config/free5gc-gnb.yaml`
        - (terminal 2): `sudo build/nr-ue -c config/free5gc-ue.yaml`
        - (terminal 3): `ping -I uesimtun0 <IP_ADDRESS>` **(success)**
        - (terminal 4): `ping -I uesimtun0 <IP_ADDRESS>` (success)
    - In free5gc, Drop the default FAR 1,2: format `./gtp5g-tunnel [get/del/add/mod] [PDR/FAR/QER] [interface_name] [seid]:[id] [option]`
        - `./gtp5g-tunnel mod far upfgtp 1:1 --action 1` (`1` means drop, `2` means forward)
        - `./gtp5g-tunnel mod far upfgtp 1:2 --action 1`
        -  - (terminal 3): `ping -I uesimtun0 <IP_ADDRESS>` **(failed)**
        - (terminal 4): `ping -I uesimtun0 <IP_ADDRESS>` (success)
   - You will find that you can't ping to <IP_ADDRESS> anymore, but still can ping to <IP_ADDRESS>. It means that IP Filter Rule is successfully installed in upf.

ok, i'm trying it out, thanks


Countdown in hero #66

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #66
Test URLs:

Before: https://main--vg-volvotrucks-us--hlxsites.hlx.page/
After: https://66-countdown-in-hero--vg-volvotrucks-us-rd--netcentric.hlx.page/drafts/taimur/hero

I also fixed the height issue with the hero container. On devices with smaller height like iphone SE the text was overflowing the container.

@cogniSyb you can see hero with in-page navigation here https://66-countdown-in-hero--vg-volvotrucks-us-rd--netcentric.hlx.page/drafts/marko/pdp

@cogniSyb Updated the class names to left-center and center-bottom. This change will affect the hero which used previous class names. Those class names have to updated in the content.


docs(test): Expand documentation of cargo-test-support

What does this PR try to resolve?
In wanting to document #14039, I felt it would be good to put that documentation in cargo-test-support.  To do so, I wanted a baseline of existing documentation for it to build on top of.
I was tempted to move more of "Writing tests" contrib documentation here, as its more about using cargo-test-support but I decided to hold off for now as most of that was long-form documentation and this is mostly focused on reference documentation.
How should we test and review this PR?
Additional information

Thanks!
@bors r+

:pushpin: Commit 83cbca01419f5b0cedd3b95ad46ddba9f76a6e76 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 83cbca01419f5b0cedd3b95ad46ddba9f76a6e76 with merge ea14e861c78cf5bb67e68ddeeea7ae9219a920a2...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing ea14e861c78cf5bb67e68ddeeea7ae9219a920a2 to master...


Update grad_cam resize method with TF V2.

Use TF V2 resize for GradCam, because TF V1 bilinear resize have artifact of shifting images.
https://github.com/tensorflow/tensorflow/issues/6720
https://hackernoon.com/how-tensorflows-tf-image-resize-stole-60-days-of-my-life-aba5eb093f35

Closed since we are now using skimage.transform.resize.


How to modify code to have more than two banner images, which line of code we need to add and while line of code we need to comment to have 3 dots and 3 header_images??

How did you fix this issue ?


Upload 2.0.0 to Visual Studio Extensions gallery

@JonathanWilkins  Can you please update the .vsix extension on the visual studio gallery with the one attached here: https://github.com/machine-visualstudio/machine.vstestadapter/releases/tag/v2.0.0
Release notes in the link :) Thanks!

No problem will sort it out.
On Fri, Mar 18, 2016 at 8:54 AM -0700, "Ivan Zlatev"<EMAIL_ADDRESS>wrote:
@JonathanWilkins  Can you please update the .vsix extension on the visual studio gallery with the one attached here: https://github.com/machine-visualstudio/machine.vstestadapter/releases/tag/v2.0.0
Release notes in the link :) Thanks!
—
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub

Done.
Released Version 2.0.78 as I had to fix the ID in the manifest to get the gallery to accept it.

Thanks! I have replaced the .vsix in the GitHub release with the one you uploaded to vsix gallery.


[E000291] Rep. Renee Ellmers

Rep. Renee Ellmers:
http://ellmers.house.gov
Likely contact form URL to start with:
http://ellmers.house.gov/contact-us/

YMAL file not found. I am working on it.


doesn't open some images

i've run the html page and its working for well
but then came to know doesn't open some images

this is an example

So am I , hope this BUG can be resolved.

Same here!
Any update on this issue?

After a little debugging, i found the cause of this issue.
The method onPreImageLoaded validates the selected image based on some parameters like, the container's width and height, the preview size, max zoom, etc. If the image doesn't pass these criteria, it throws an error and the image isn't displayed.
As a temporary fix for my issue, i commented the reject image if clause in the code:
{
	key: 'onPreImageLoaded',
	value: function onPreImageLoaded()
	{
		// if (this.shouldRejectImage({
		// 	imageWidth: this.preImage.width,
		// 	imageHeight: this.preImage.height,
		// 	previewSize: this.previewSize,
		// 	maxZoom: this.options.maxZoom,
		// 	exportZoom: this.options.exportZoom,
		// 	smallImage: this.options.smallImage
		// })) {
		// 	this.onImageError(_constants.ERRORS.SMALL_IMAGE);
		// 	if (this.image.src) {
		// 		this.setImageLoadedClass();
		// 	}
		// 	return;
		// }

		this.image.src = this.preImage.src;
	}
},

Hope this helps.

Got same here. Hope it will be fixed

I am also having this issue.  It seems to happen with only certain png files for me.
Even happens on the cropit site using the first demo.  I have attached two png photos (stock photos from windows). iis.png works find but the w-brand.png does not load.
I am gathering from the images I have uploaded and ricky2k7's post that the w-brand.png file fails a size constraint?
Would be great to see a fix for this.

This can be solved using the smallImage: 'allow' option:
$('.image-editor').cropit({ smallImage: 'allow' });
It still keeps the image size specified by the cropit box but you cannot zoom in on the small image.  This makes sense because it will become pixelated.
See issue #102.

After a little debugging, i found the cause of this issue. The method onPreImageLoaded validates the selected image based on some parameters like, the container's width and height, the preview size, max zoom, etc. If the image doesn't pass these criteria, it throws an error and the image isn't displayed.
As a temporary fix for my issue, i commented the reject image if clause in the code:
{
	key: 'onPreImageLoaded',
	value: function onPreImageLoaded()
	{
		// if (this.shouldRejectImage({
		// 	imageWidth: this.preImage.width,
		// 	imageHeight: this.preImage.height,
		// 	previewSize: this.previewSize,
		// 	maxZoom: this.options.maxZoom,
		// 	exportZoom: this.options.exportZoom,
		// 	smallImage: this.options.smallImage
		// })) {
		// 	this.onImageError(_constants.ERRORS.SMALL_IMAGE);
		// 	if (this.image.src) {
		// 		this.setImageLoadedClass();
		// 	}
		// 	return;
		// }

		this.image.src = this.preImage.src;
	}
},

Hope this helps.

It works!


beginners friendly "import everything"

Hello,
sorry for asking this, but I have been struggling with documentation and various imports, but can't seem to find a mapping configuration to simply import everything from osm.pbf file into PostGIS.
It is really hard for a newcomer to even know what exactly to select as a source for mapping. I just want all.
I found any : [any ] and "load_all", but the example for that kind of mapping.json would be so helpful.
BR,
Tomaz

I'm using this mapping:
tags:
  load_all: true
  exclude: [created_by, source]

tables:
  all:
    type: geometry
    columns:
      - name: osm_id
        type: id
      - name: tags
        type: hstore_tags
      - name: geometry
        type: geometry
    type_mappings:
      points:
        __any__: [__any__]
      linestrings:
        __any__: [__any__]
      polygons:
        __any__: [__any__]


[QST]colcon build embeddedRTPS outputs error: template with C linkage.

Issue template

Hardware description: stm32f767zi
RTOS: freertos + lwip + embeddedRTPS + rmw_embeddedrtps
Installation type: micro_ros_setup
Version or commit hash: galactic

Steps to reproduce the issue

generate stm32 project code

in https://github.com/ZhenshengLee/ros2_mcu/tree/master/libs/custom_stm32f767zi_ertps
generate code according to https://github.com/micro-ROS/micro_ros_stm32cubemx_utils/blob/galactic/embeddedrtps.md and
https://github.com/embedded-software-laboratory/embeddedRTPS-STM32/blob/master/readme.md
the change is with https://github.com/ZhenshengLee/ros2_mcu/commit/782a2558bd85b2524703554ab86e2c46ecb52f05

add uros extra source files.

according to https://github.com/micro-ROS/micro_ros_stm32cubemx_utils/tree/galactic/extra_sources
the mcu_ws srcfile is including embedded_rtps and rmw_embeddedrtps
https://github.com/ZhenshengLee/ros2_mcu/blob/master/ertps_ws/target_uros_packages.repos
 uros/rmw_embeddedrtps:
    type: git
    url: https://hub.fastgit.xyz/micro-ROS/rmw_embeddedrtps.git
    version: galactic
  uros/embeddedRTPS:
    type: git
    url: https://hub.fastgit.xyz/micro-ROS/embeddedRTPS.git
    version: galactic

the change is in https://github.com/ZhenshengLee/ros2_mcu/commit/8953000ea74d8006b80b7d48619c6624234f97f1

using make and compile libmicroros

https://github.com/ZhenshengLee/ros2_mcu/blob/master/libs/custom_stm32f767zi_ertps/Core/Src/main.cpp is according to https://github.com/micro-ROS/micro_ros_stm32cubemx_utils/blob/galactic/sample_main_embeddedrtps.c
https://github.com/ZhenshengLee/ros2_mcu/blob/master/libs/custom_stm32f767zi_ertps/Makefile is revisioned to support cpp programming according to https://github.com/embedded-software-laboratory/embeddedRTPS-STM32/blob/master/readme.md#code-generation-using-cubemx
make libmicroros
# firstly calling colcon build

the colcon build script is in https://github.com/ZhenshengLee/ros2_mcu/blob/8953000ea74d8006b80b7d48619c6624234f97f1/libs/custom_stm32f767zi_ertps/Makefile#L396
colcon_compile: arm_toolchain
	cd $(UROS_DIR); \
	colcon build \
		--packages-ignore-regex=.*_cpp \
		--metas $(UROS_DIR)/colcon.meta \
		--cmake-args \
		"--no-warn-unused-cli" \
		-DCMAKE_POSITION_INDEPENDENT_CODE=OFF \
		-DTHIRDPARTY=ON \
		-DBUILD_SHARED_LIBS=OFF \
		-DBUILD_TESTING=OFF \
		-DCMAKE_BUILD_TYPE=$(BUILD_TYPE) \
		-DCMAKE_TOOLCHAIN_FILE=$(EXTENSIONS_DIR)/arm_toolchain.cmake \
		-DCMAKE_VERBOSE_MAKEFILE=ON; \

Expected behavior
colcon build success, and libmicroros.o is generated.
Actual behavior
It seems that c/cpp mixed programming is broken.
some of errors are below
/home/zs/zs_ws/ros2_mcu/ertps_ws/install/include/rtps/entities/StatefulWriter.tpp:261:1: error: template with C linkage
 template <class NetworkDriver>
 ^~~~~~~~
/home/zs/zs_ws/ros2_mcu/ertps_ws/install/include/rtps/entities/StatefulWriter.tpp:333:1: error: template with C linkage
 template <class NetworkDriver>
 ^~~~~~~~
/home/zs/zs_ws/ros2_mcu/ertps_ws/install/include/rtps/entities/StatefulWriter.tpp:373:1: error: template with C linkage
 template <class NetworkDriver>
 ^~~~~~~~
/home/zs/zs_ws/ros2_mcu/ertps_ws/install/include/rtps/entities/StatefulWriter.tpp:424:1: error: template with C linkage
 template <class NetworkDriver>
 ^~~~~~~~
/home/zs/zs_ws/ros2_mcu/ertps_ws/install/include/rtps/entities/StatefulWriter.tpp:430:1: error: template with C linkage
 template <class NetworkDriver>
 ^~~~~~~~
/home/zs/zs_ws/ros2_mcu/ertps_ws/install/include/rtps/entities/StatefulWriter.tpp:444:1: error: template with C linkage
 template <class NetworkDriver>
 ^~~~~~~~
make[2]: *** [CMakeFiles/rmw_embeddedrtps.dir/build.make:499: CMakeFiles/rmw_embeddedrtps.dir/src/rmw_wait.cpp.obj] Error 1
make[2]: *** [CMakeFiles/rmw_embeddedrtps.dir/build.make:457: CMakeFiles/rmw_embeddedrtps.dir/src/rmw_take.cpp.obj] Error 1
make[1]: *** [CMakeFiles/Makefile2:88: CMakeFiles/rmw_embeddedrtps.dir/all] Error 2
make: *** [Makefile:149: all] Error 2
---
Failed   <<< rmw_embeddedrtps [11.0s, exited with code 2]
Aborted  <<< lifecycle_msgs [11.5s]
Aborted  <<< rosidl_typesupport_microxrcedds_test_msg [2.15s]
Aborted  <<< action_msgs [2.31s]
Aborted  <<< statistics_msgs [2.11s]
Aborted  <<< rosgraph_msgs [2.14s]
Aborted  <<< std_msgs [2.43s]
Aborted  <<< rcl_interfaces [2.25s]

Summary: 32 packages finished [21.2s]
  1 package failed: rmw_embeddedrtps
  7 packages aborted: action_msgs lifecycle_msgs rcl_interfaces rosgraph_msgs rosidl_typesupport_microxrcedds_test_msg statistics_msgs std_msgs
  21 packages had stderr output: action_msgs builtin_interfaces embeddedrtps libyaml_vendor lifecycle_msgs micro_ros_msgs micro_ros_utilities rcl_interfaces rcl_logging_interface rcl_logging_noop rcutils rmw rmw_embeddedrtps rosgraph_msgs rosidl_runtime_c rosidl_typesupport_c rosidl_typesupport_microxrcedds_c statistics_msgs std_msgs std_srvs unique_identifier_msgs
  27 packages not processed

Additional information
the toolchain.cmake is below
include(CMakeForceCompiler)

set(CMAKE_SYSTEM_NAME Generic)
set(CMAKE_SYSTEM_PROCESSOR arm)
set(CMAKE_CROSSCOMPILING 1)
set(CMAKE_TRY_COMPILE_TARGET_TYPE STATIC_LIBRARY)
set(PLATFORM_NAME "LwIP")

# Makefile flags
set(ARCH_CPU_FLAGS "-mcpu=cortex-m7 -mthumb -mfpu=fpv5-d16 -mfloat-abi=hard -DUSE_HAL_DRIVER -DSTM32F767xx -D_TIMEVAL_DEFINED -DSTM32CUBEIDE -DENOTSUP=1 -DECANCELED=1 -DEOWNERDEAD=1 -DENOTRECOVERABLE=1  -O0 -Wall -fdata-sections -ffunction-sections")
set(ARCH_OPT_FLAGS "")

set(CMAKE_C_COMPILER /home/zs/zs_ws/ros2_mcu/libs/custom_stm32f767zi_ertps/../../toolchain/gcc-arm-none-eabi-8-2019-q3-update/bin/arm-none-eabi-gcc)
set(CMAKE_CXX_COMPILER /home/zs/zs_ws/ros2_mcu/libs/custom_stm32f767zi_ertps/../../toolchain/gcc-arm-none-eabi-8-2019-q3-update/bin/arm-none-eabi-g++)

set(CMAKE_C_FLAGS_INIT "-std=c11 ${ARCH_CPU_FLAGS} ${ARCH_OPT_FLAGS} -DCLOCK_MONOTONIC=0 -D'__attribute__(x)='" CACHE STRING "" FORCE)
set(CMAKE_CXX_FLAGS_INIT "-std=c++14 ${ARCH_CPU_FLAGS} ${ARCH_OPT_FLAGS} -DCLOCK_MONOTONIC=0 -D'__attribute__(x)='" CACHE STRING "" FORCE)

include_directories(SYSTEM
[encoded data, 4399 characters]

    )

set(__BIG_ENDIAN__ 0)

Changing toolchain from gcc-arm-none-eabi-8-2019-q3-update to gcc-arm-none-eabi-9-2020-q2-update solves this.
Closing this.


Azure Repos extension icons not rendered in Status Bar

VSCode Version: 1.40.2
OS Version: Win 10 x64

After update VSCode to 1.40.2 version, the Azure repos icons are not rendered. The Azure Repos extension version is 1.149.2.

Steps to Reproduce:

Update to 1.40.2 version
Open VS Code

Does this issue occur when all extensions are disabled?: No

The extension version I had was 1.149.2. I updated to 1.161.0 and the problem has gone


Need for Training on New Videos

Does the model need to be trained separately for each arbitrary video?
I just tried reversing the order of 'im1.png'~'im7.png' provided in the repo and attempted to perform SR on 'im7.png' from the HR image of 'im1.png', but the result turned out blurry.

Oh, now I see, the currently provided checkpoint seems to be an overfitted model based on a specific example video from the demo, and to ensure a more general performance across other videos, it would require training with a larger dataset and more training epochs.
I'll close my issue :)


Correction for raw affiliation Capilano University, British Columbia, CA

Correction needed for raw affiliation Capilano University, British Columbia, CA
raw_affiliation_name: Capilano University, British Columbia, CA
new_rors: 046nfbs12
previous_rors:
works_examples: W4229568571
contact: a187ce88a5a89732b5df12c62a333dc8:6f986959 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Assorted issues before first run

1. Update the description in readme.md after you verify them:
You might need to update the nltk corpuses to properly run the newspaper library
python -m nltk.downloader all
2. Update the description in readme.md after you verify it:
To encounter the lxml error while installing newspaper using pip in ubuntu:
sudo apt-get install python-dev libxml2-dev libxslt1-dev zlib1g-dev
http://stackoverflow.com/questions/5178416/pip-install-lxml-error
3. Update the description in readme.md after you verify it:
To encounter jpeg is required error while installing  indicoio
sudo apt-get install libtiff5-dev libjpeg8-dev zlib1g-dev libfreetype6-dev liblcms2-dev libwebp-dev tcl8.6-dev tk8.6-dev python-tk

4. Update the description in readme.md after you verify it:
You need to update the default nltk 2.x to 3.x so that nltk.tag works
pip install -U nltk

5. Update the description in readme.md after you verify it:
You forgot to add this in the requirements
pip install wget

6. Update the description in readme.md after you verify it:
make sure adding a single path name would suffice
here is another place Newcrawler/newscralwer/credentials where you need the pathname
7. Update the description in readme.md after you verify it:
Adding the path and api key in the root folder does not make any difference, it seems that the actual configurations files are. the files in  NewsCrawler/NewsCrawler/credentials_and_configs directory
There is problem with indentation, in all the crawler files, it was resolved when I used sublime to convert all indentation to tabs then it is resolved
8. You forgot to  import re,
but used re somewhere in the middle of the codes in the dailystar and prothomalo file, but it was imported in the dtribune crawler.
9. where are you saving the jpeg files?
10. DhakaTribune crawler not working
11. Most importantly you saved the published date as a string, it should be a date, not a string, and rename date variable to date_crawled and published  date to date_published**
12. Change ner_list_location to ner_locations and change ner_location to ner_unique_locations.**
13. what does shoulder keyword stand for inside the data inserted in elasticsearch?

NLTK
Downloading Language Models and Corpus
python -m nltk.downloader all

or,
>>> import nltk
>>> nltk.download()

Downloading Image
I am saving the downloaded files at
NewsCrawler /

Updating NLTK
pip install -U nltk

To use StanfordNERTagger nltk 3.x.x version is required

Python string to date conversion example:
from dateutil import parser
dt = parser.parse("Aug 28 1999 12:00AM")

Now, you will insert this date object to the database, not the previous string object.

Shoulder keyword:
There is a special section/keyword used in dhakatribune website which is named shoulder

From the following figure, it can be seen that the INSIGHT keyword is the shoulder of that article.

I think most of the issues are resolved by @manashmndl


Add Melantonic playlist

Thanks for making this project, I'd like to add the following playlist, Melantonic by Spotify:

https://open.spotify.com/playlist/37i9dQZF1DX39ATYW02fre?si=IADasvD5QJmPQvwPfVOBRQ

The Spotify URI is spotify:playlist:37i9dQZF1DX39ATYW02fre

Looks good, thanks for contributing!