Use @chainlink/contracts package for dev contracts

TODO: re-deploy & update contract addresses

Why do we need to keep contracts/dev/FunctionsOracle.sol and friends? and not just reference those from the package?

Why do we need to keep contracts/dev/FunctionsOracle.sol and friends? and not just reference those from the package?

So that hardhat compiles them and holds the artifacts (ABIs) for them.
See https://ethereum.stackexchange.com/questions/114376/how-to-compile-external-contracts-using-hardhat

Why do we need to keep contracts/dev/FunctionsOracle.sol and friends? and not just reference those from the package?

So that hardhat compiles them and holds the artifacts (ABIs) for them that we use within the CLI commands. See https://ethereum.stackexchange.com/questions/114376/how-to-compile-external-contracts-using-hardhat

Why do we need to keep contracts/dev/FunctionsOracle.sol and friends? and not just reference those from the package?

So that hardhat compiles them and holds the artifacts (ABIs) for them that we use within the CLI commands. See https://ethereum.stackexchange.com/questions/114376/how-to-compile-external-contracts-using-hardhat

Why do we need to keep contracts/dev/FunctionsOracle.sol and friends? and not just reference those from the package?

So that hardhat compiles them and holds the artifacts (ABIs) for them that we use within the CLI commands. See https://ethereum.stackexchange.com/questions/114376/how-to-compile-external-contracts-using-hardhat

Just so I understand, is FunctionsOracle.sol is not included in our imported package?  Is that why it still must be provided here?

Made obsolete by https://github.com/smartcontractkit/functions-hardhat-starter-kit/pull/73


[Bug]: In Single Select Tree Widget, selected value does not clear on clicking the cross button.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When we clear a value in a single select tree widget, the selected value does not become null or empty, it defaults back to the default value.
Ideally, the selected value should become empty.
Steps To Reproduce

Drop single select tree widget
Allow clear selection and make sure you have the default value.
Now, try clearing the value, you will notice that the selected value defaults back to the default value instead of the empty value . Check the video attached to reproduce.

https://user-images.githubusercontent.com/6636360/186369618-f1fc8a7b-3084-41b0-887c-e757a4b03c23.mov
Public Sample App
No response
Version
v1.7.14

cc: @laveena-en


Audit and update styles for error, info and warning, etc.

So we need to update the the styles for the errors, warnings etc. right?

And add/update tests for these too.

در تاریخ جمعه ۱۵ شهریور ۱۳۹۸ ه.ش.، ۲۳:۱۱ Hardik Khandelwal <
<EMAIL_ADDRESS>نوشت:

So we need to update the the styles for the errors, warnings etc. right?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/36763?email_source=notifications&email_token=ADHC6UJSDKNCCLGZQTQYA73QIKP4TA5CNFSM4IUHBK22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6DWRKY#issuecomment-528967851,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADHC6UJ3U74HD6LVWT4HMPDQIKP4TANCNFSM4IUHBK2Q
.

If it is not a problem, could you please mention the directories that need to changed.

@raisedadead I can do it.

Hi Guys, thanks a lot for your interest. BUt I figured out this is not very trivial of issue to start with. I have assigned this to @ahmadabdolsaheb instead.
Please go thru some of the other help wanted issues.
Thanks and happy contributing.


Shell script on Gpio-Button and RFID Card for squeezlite integration

I would like to integrate the Phoniebox into my multiroom audio system powered by PiCorePlayer and Logitech medias server. Therefore I would like to have the option to run this batch commands when a button is pressed and/or a RFID card is read:
`sudo service mopidy stop
sudo pkill mopidy # I am not sure if I need both
squeetlite -n test -o sysdefault:CARD=Device &`
(Any improvements on this are welcome!)
Some help would be great!
Best wishes,
Don

It seems I found the solution: all the effort is not necessary, when squeezlite is running, the phoniebox still works. As soon as squeezlite starts an output, both sources are played simultaneously - nice!


Shadow endpoint doesn't support for ReadableStream

Describe the bug
RequestHandlerOutput's body accepts ReadableStream instance in general, yet it throws an error when the endpoint is a shadow endpoint.
Reproduction
The code works well if it's called as standalone endpoints, but it throws an error if it's called as shadow endpoints:
export const get: RequestHandler = async () => {
  const res = await fetch('some url')
  return { body: res.body }
}

The following code is direct cause of the error:
https://github.com/sveltejs/kit/blob/2d6f0d2d49d4505112bdc6f2f0ef1b8ce1d1fb40/packages/kit/src/runtime/server/page/load_node.js#L508-L510
If I comment-out the above code, everything works fine.
Logs
Body returned from endpoint request handler must be a plain object
Error: Body returned from endpoint request handler must be a plain object
    at validate_shadow_output (file:///Users/ak/prjs/pxd-cr/services/crapp/.svelte-kit/runtime/server/index.js:2052:9)
    at load_shadow_data (file:///Users/ak/prjs/pxd-cr/services/crapp/.svelte-kit/runtime/server/index.js:1988:38)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async load_node (file:///Users/ak/prjs/pxd-cr/services/crapp/.svelte-kit/runtime/server/index.js:1597:5)
    at async respond$1 (file:///Users/ak/prjs/pxd-cr/services/crapp/.svelte-kit/runtime/server/index.js:2243:15)
    at async resolve (file:///Users/ak/prjs/pxd-cr/services/crapp/.svelte-kit/runtime/server/index.js:2744:11)
    at async respond (file:///Users/ak/prjs/pxd-cr/services/crapp/.svelte-kit/runtime/server/index.js:2688:20)
    at async file:///Users/ak/prjs/pxd-cr/services/crapp/node_modules/@sveltejs/kit/dist/chunks/index.js:298:24

System Info
System:
    OS: macOS 12.3
    CPU: (20) x64 Intel(R) Xeon(R) W-2150B CPU @ 3.00GHz
    Memory: 501.01 MB / 32.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 17.8.0 - /usr/local/bin/node
    Yarn: 1.22.15 - /usr/local/bin/yarn
    npm: 8.5.5 - /usr/local/bin/npm
  Browsers:
    Chrome: 100.0.4896.75
    Firefox: 96.0.3
    Safari: 15.4
  npmPackages:
    @sveltejs/adapter-auto: ^1.0.0-next.33 => 1.0.0-next.33
    @sveltejs/adapter-node: ^1.0.0-next.72 => 1.0.0-next.72
    @sveltejs/kit: ^1.0.0-next.299 => 1.0.0-next.299
    svelte: ^3.46.4 => 3.46.4

Severity
blocking an upgrade
Additional Information
No response

This is deliberate - during SSR, the object is passed directly from the endpoint to the component, without any kind of buffering or deserialization. It's only when you access the endpoint via client-side navigation that the data is turned into a Response. Use res.json() in your handler

@Rich-Harris, Thank you for the explanation.
But my current application need to invoke shadow endpoint during SSR and also from browser via setInterval(), since the endpoint calls a remote API which returns varying data from call to call.
Does it look like a valid use case for streaming support for shadow endpoint?
For now, I made standalone endpoint like [blah].svelte and [blah]-stream.ts and let [blah].svelte call the standalone endpoint using old method - Load from <script context="module">. Could you recommend a better way to handle this situation?
Thank you.


Regression [Office365] unable to connect to SMTP server

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
mhh not the first time Office365 gives trouble :/
Expected behavior
I was and want to be able to send Emails with my standard unchanged settings.
Actual behavior
Error: Cannot connect to server. (Unable to open connection to SMTP server.)
Steps to reproduce

Create Office365 account
Try using official settings: smtp.office365.com:587
Setup could not finish

Environment
K-9 Mail version: November 2017
Android version: 6
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

Sounds like a network issue, not like a bug in K-9 Mail.

The issue persists and is not dependent on the network.
The Error occurs on WLAN and LTE networks.
Other (non-Office365) accounts are not affected.
Please reopen!

I was able to configure the outgoing mail using the following settings, as auto-detected by Thunderbird:
incoming: imap-mail.outlook.com, SSL/TLS
outgoing: smtp-mail.outlook.com, STARTTLS
Using K-9 Mail Version 5.403 on Android 7.1.1
Account type: IMAP
Please check if these settings work for other cases too.

I can confirm, that SwampertX's server settings work.


Navigation cut off on mobile

On a phone, the "News & Articles" link is cut off. This also results in some blank space to the right of the main page content.

Thanks @nawatts we will work this into an upcoming sprint

Thanks @nawatts we will work this into an upcoming sprint


Publish to Pub.dev

Hi, Guys
I can't find this library at:
https://pub.dev/packages?q=TheGorgeousLogin
It is there under another name? If is not, there is some reason to not publish this awesome widget?
Thank you!

No it's only a public Flutter UI design, there is no point in publishing it on pub.dev


Editor is weirdly scaled and has a lot of overlapping textures

editor scale on the settings menu

Please join the discord for support questions: https://discord.gg/mKyXdEsMZD
This site is for crash/bug reports and suggestions.


Aniwave malfunctioning (Search + Stream)

Source information
Aniwave 13.57
Source language
English
Steps to reproduce

Open Aniyomi
Go to browse tab
Open Aniwave
Try searching for an anime or watch an episode of any series
It will return an error

Expected behavior
Should return a list with anime titles matching my search query or stream the episode I'm watching
Actual behavior
Returns an error
Aniyomi version
0.14.6-6678
Android version
Android 13
Other details

Webview works fine, I've checked so it is a problem with the extension itself
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Same problem

Aniwave dosnt work at all

Aniwave dosnt work at all

Thanks for mentioning that, I've updated my issue accordingly.

Same for FMovies. Exact same error.

this should still be open as the stream isn't working because AniwaveUtils still uses 9anime.eltik.net (wich is down) to get the source

9anime.eltik.net is down

So it was also a source problem in the end, I tried visiting that site previously, it seemed weird why I only saw just an apache homepage

9anime.eltik.net is down

So it was also a source problem in the end, I tried visiting that site previously, it seemed weird why I only saw just an apache homepage

I think 9anime.eltik.net used to be a service by a fellow FOSS developer who had to stop hosting it due to a complain he received. Correct me if I'm wrong @aniyomiorg/extensions

Same it doesn't work

With the latest extension update, streams are working again, but I still see javlang errors resulting into a failed download.

Issue resolved

Issue resolved

no it's not

Issue resolved

no it's not

It's not? With aniwave 13.58, I can stream almost every anime without any issues.

Issue resolved

no it's not

It's not? With aniwave 13.58, I can stream almost every anime without any issues. The search works also just fine.

What are your settings in the aniwave extension? It still shows the 404 error to me, but perhaps you have different settings on, hence it works. Thanks in advance.

What are your settings in the aniwave extension? It still shows the 404 error to me, but perhaps you have different settings on, hence it works. Thanks in advance.

Have you tried a different domain?

What are your settings in the aniwave extension? It still shows the 404 error to me, but perhaps you have different settings on, hence it works. Thanks in advance.

Have you tried a different domain?

I tried a different domain, didn't work. I tried the settings on your photo, and it still doesn't work! But thanks for it. Interesting though.

I tried a different domain, didn't work. I tried the settings on your photo, and it still doesn't work! But thanks for it. Interesting though.
Does your selected domain load in your browser?
Have you updated aniwave to 13.58?

I tried a different domain, didn't work. I tried the settings on your photo, and it still doesn't work! But thanks for it. Interesting though.

Does your selected domain load in your browser?
Have you updated aniwave to 13.58?

Did both, still not working

Wait a min, it does work on the preview version!! But not on the stable version, which I prefer... anyway for them to update the stable version?

Wait a min, it does work on the preview version!! But not on the stable version, which I prefer... any way for them to update the stable version?

The stable version will be replaced with what is now the preview. Sorry if you don't like it but it's impossible to keep the stable version because of various bugs like that. 😅

Wait a min, it does work on the preview version!! But not on the stable version, which I prefer... any way for them to update the stable version?

The stable version will be replaced with what is now the preview. Sorry if you don't like it but it's impossible to keep the stable version because of various bugs like that. 😅

Oh, that is completely okay! That's actually what I meant, that I'd like if stable would be updated to be like the preview version (although I'm aware preview is for testing and can have bugs and whatnot, so we had to wait).
Anyway, this is such a GOATED android app, and I'll support you guys always! Thanks for the response.


[BUG] Empty error 400 when checking for an index

Describe the bug
When I try to run any command to interact with my opensearch node, I get an error 400 that has no TransportMessage, making it hard to see what's going on. I should note that this happened after I added an opensearch dashboard container, before, it was running just fine. Here is the stacktrace:
  File "/Users/skarakay/Dev/stan/app/main/opensearch/rag.py", line 54, in check_if_index_exists
    if OS_CLIENT.indices.exists(index=index):
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/client/utils.py", line 181, in _wrapped
    return func(*args, params=params, headers=headers, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/client/indices.py", line 428, in exists
    return self.transport.perform_request(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/transport.py", line 448, in perform_request
    raise e
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/transport.py", line 409, in perform_request
    status, headers_response, data = connection.perform_request(
                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/connection/http_urllib3.py", line 290, in perform_request
    self._raise_error(
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/connection/base.py", line 316, in _raise_error
    raise HTTP_EXCEPTIONS.get(status_code, TransportError)(
opensearchpy.exceptions.RequestError: RequestError(400, '')

And here is the code:
OS_CLIENT = OpenSearch(
    hosts=[OPENSEARCH_URL],
    http_auth=(OPENSEARCH_USERNAME, OPENSEARCH_PASSWORD),
    use_ssl=False,
    verify_certs=False,
)

def check_if_index_exists(index: str) -> None:
    try:
        if OS_CLIENT.indices.exists(index=index):
            OS_CLIENT.indices.delete(index=index)
        idx_settings = INDEX_SETTINGS_TEMPLATE.copy()
        idx_settings["mappings"]["properties"] = {
            index: {
                "type": "knn_vector",
                "dimension": 4096,
                "method": {"name": "hnsw", "space_type": "l2", "engine": "faiss"},
            }
        }
        OS_CLIENT.indices.create(index=index, body=idx_settings)
    except Exception as e:
        raise e

I can ping the container just fine:
{
  "name" : "d7019020e897",
  "cluster_name" : "docker-cluster",
  "cluster_uuid" : "_XG-GxzGSLyjj4l2jkHjmA",
  "version" : {
    "distribution" : "opensearch",
    "number" : "2.12.0",
    "build_type" : "tar",
    "build_hash" : "2c355ce1a427e4a528778d4054436b5c4b756221",
    "build_date" : "2024-02-20T02:18:49.874618333Z",
    "build_snapshot" : false,
    "lucene_version" : "9.9.2",
    "minimum_wire_compatibility_version" : "7.10.0",
    "minimum_index_compatibility_version" : "7.0.0"
  },
  "tagline" : "The OpenSearch Project: https://opensearch.org/"
}

There are no logs on the node side. In case, since I suspect this is where it's coming from, here is my containers in my compose:
node-opensearch:
    container_name: node-opensearch
    image: opensearchproject/opensearch:2.12.0
    env_file:
      - path: .env
        required: false
    environment:
      - discovery.type=single-node
      - plugins.security.disabled=true
    ulimits:
      memlock:
        soft: -1
        hard: -1
    ports:
      - 15821:9200
      - 15822:9600
    networks:
      - opensearch-net

  opensearch-dashboards:
    image: opensearchproject/opensearch-dashboards:2.12.0
    ports:
      - 15823:5601
    expose:
      - "5601"
    environment:
      DISABLE_SECURITY_DASHBOARDS_PLUGIN: true
      OPENSEARCH_HOSTS: '["http://node-opensearch:9200"]'
    networks:
      - opensearch-net

networks:
  opensearch-net:

Related component
Other
To Reproduce

Launch the containers just like in the docker compose
Try the python code
Fail

Expected behavior
I should be able to execute my commands correctly, with or without a dashboard running
Additional Details
Everything is in the description.

When running the container without a dashboard running on the side, the stacktrace changes and I get this:
  File "/Users/skarakay/Dev/stan/app/main/opensearch/rag.py", line 54, in check_if_index_exists
    if OS_CLIENT.indices.exists(index=index):
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/client/utils.py", line 181, in _wrapped
    return func(*args, params=params, headers=headers, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/client/indices.py", line 428, in exists
    return self.transport.perform_request(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/transport.py", line 446, in perform_request
    raise e
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/transport.py", line 409, in perform_request
    status, headers_response, data = connection.perform_request(
                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/skarakay/Dev/stan/.venv/lib/python3.12/site-packages/opensearchpy/connection/http_urllib3.py", line 279, in perform_request
    raise ConnectionError("N/A", str(e), e)
opensearchpy.exceptions.ConnectionError: ConnectionError(<urllib3.connection.HTTPConnection object at 0x13ec6aea0>: Failed to establish a new connection: [Errno 61] Connection refused) caused by: NewConnectionError(<urllib3.connection.HTTPConnection object at 0x13ec6aea0>: Failed to establish a new connection: [Errno 61] Connection refused)

This probably means that I did something wrong, but I have no idea since it was working fine before

Is the issue in the actual 400 response or the python client parsing it? Can you reproduce this with curl, what's the server returning?

[Triage - attendees 1 2 3 4 5 6 7]
@opensearch-project/admin Please transfer this issue to the python client repo


Eco add not working from console

I type in console
multicurrency eco add Yavoroshko Яволар 100
And just nothing happends
No errors no feedback nothing.

This is fixed in v1.2.3.


Update to version 4.12

Still there are compilation errors. I've created an issue in CGAL/cgal-swig-bindings with the error.

You shouldnt update boost pin yet (or update it in every feedstock that doesnt use cb3 and propose the update in conda-forge-pinning-feedstock).

@jschueller, what if we build two versions of CGAL 4.12: build number 0 with Boost 1.66 and build number 1 with Boost 1.67?

That would work, but I'd prefer all the boost pinnings of all feedstocks to be updated at the same time.

In fact, when we switch to conda-smithy 3, we wont even need to pin directly in the recipe:
https://conda-forge.org/docs/meta.html#pinning-packages

(or update it in every feedstock that doesnt use cb3 and propose the update in conda-forge-pinning-feedstock)

@jschueller, do you know a simple way to list these feedstocks?

yes, type "boost-cpp 1.66" or "boost 1.66" in the github search bar of the conda-forge org (on the top left of the page), (then click Code tab, which shows we have eg 17 recipes depending on boost-cpp):
https://github.com/conda-forge


chore(deps): update jwt [ZEND-3114]

Addressing https://github.com/contentful/node-apps-toolkit/issues/127

chore: won't release a new version
Should we make the dependency ^8.0.0 || ^9.0.0 as it's still compatible with the old version?

With v8 being vulnerable, we probably should't include it


Fix the scope to look-up the constant as the receiver type of the macro

Fixed #5343. It is another solution than #5349. I believe this PR is better than #5349.
Currently the constant in expansion inside macro is resolved by caller-scope, for example:
class Foo
  macro foo
    {{Baz}}
  end
end

class Bar
  class Baz
  end

  def self.foo
    Foo.foo
  end
end

p Bar.foo # => Bar::Baz

It is unexpected because the constant inside method is of course resolved by the receiver type of the method. And so, the second example in #5343 is caused.
This PR fixed scope to look-up the constant as the receiver type of the macro.

@MakeNowJust say in that example the macro did want to access Baz, how would that be done?

@RX14 Using Bar::Baz is the right way.

It does not make a sense that this example works only if Foo.foo is called from the scope having Bar.
class Foo
  macro foo
    {{Baz}}
  end

  class Baz
  end
end

class Bar
  class Baz
  end

  def self.foo
    Foo.foo
  end
end

class FooBar
  def self.foo
    Foo.foo
  end
end

p Bar.foo # => Bar::Baz
p FooBar.foo # compilation error: undefined constant 'Baz'

Isn't it?

@MakeNowJust but now you have to pass in the outer scope somehow? Macros can reference local variables and local methods, why not local types too.

This example works. What is problem?
class Foo
  macro foo(x)
    {{x}}
  end

  macro bar
    Baz
  end
end

class Bar
  class Baz
  end

  def self.foo
    Foo.foo Baz
  end

  def self.bar
    Foo.bar
  end
end

p Bar.foo # => Bar::Baz
p Bar.bar # => Bar::Baz

This PR fixes the semantic in only macro expansion inside macro.

@MakeNowJust ah yes never mind me I was just confused.

I think what @RX14 fears is that we may not be able to access a 'local' type from the macro system, as {% SomeType %}
module IncludeMe
  macro included
    STORAGE = [] of _ # Create a 'local' Type in the caller's (includer) scope
  end

  macro add(node)
    {% STORAGE << node %} # Access a 'local' type in the caller's scope, from the macro system
  end
end

class Foo
  include IncludeMe

  # Add some stuff from macros to a 'local' type
  add a + b
  add :bla
end

But it seems to still work, so :+1:

Can this go forward?

This fix is correct 👍

needs a rebase, sorry for the delay merging

I merged master into this on GitHub Web UI. Please do squash merge.


Find specs: parse WHATWG page

The find-spec.js script could also look at specs defined in https://spec.whatwg.org/ to identify specs that are not in the crawl.

Fixed via browser-specs integration


Flowchart for integral landing page

Wireframe this and pass it to Merci.

Oops should have made this in Integral: https://github.com/Singly/integral/issues/170


RFC: Use stable Rust

Opening this on top of #511 as an idea to switch to using stable rust. Since we don't have to chase specific nightly versions, it seems like a good idea to just use stable rust with a minimum version for specific features we want.
But, this toolchain selection seems very confusing because again rust seems like it doesn't really support cross-compiling. What we want are very separate ways to specify the toolchain for building for embedded targets from ways to specify the toolchain for running host-side tests. It remains perplexing to me that rust still doesn't really seem to take this seriously.
Specifically, I don't know how to tell cargo "use nightly, but use a nightly with miri available". Maybe it does this automatically since we are running a miri command? I don't know.

Specifically, I don't know how to tell cargo "use nightly, but use a nightly with miri available". Maybe it does this automatically since we are running a miri command? I don't know.

It won't. rustup determines the toolchain version, cargo parses the miri command. They're not the same tool, and by the time anything knows you need Miri you're already running the wrong toolchain version.

I did some digging and found the reasoning here: rust-lang/rustup#3355. It seems the underlying issue is that because user-defined toolchain names exist, rustup's argument syntax is limiting. It may still be possible, but I don't think it's worth the effort for us to pursue it.

I'm not arguing with you, but these people built an entire programming language, they could figure out how to handle host-side testing needing a different compiler than a binary for an embedded system (if they wanted to).

Improved by #513


Changing the flat button colors

How are you supposed to change the color of flat buttons? Doing something like class="blue" just colors the button blue all the time--not when hovered over.
In MaterializeCSS, you're supposed to do class="waves-effect waves-blue btn-flat"--but that doesn't work here.
What am I missing?

Can you add an example in Stackblitz please? I will be easier to help you.

Sure. Here is an example:
https://stackblitz.com/edit/ngx-materialize-z9vteo

I actually have the same issue; all flat buttons become green when the mouse is hovered over them and this color cannot be changed (in my case, I would prefer it to be either light gray or transparent).
It's curious 'cause in the documentation website (in the modal, card and buttons sections at least), none of the flat buttons have any color on hover. Trying to override the CSS rules with background-color: whatever !important doesn't help, since it applies the style to all buttons (flat, float and normal buttons alike); fixing the style for flat buttons, but breaking it for the rest.
I think this may be because the mz-button directive adds a CSS class to the element called btn even when the button is flat (in which case it also adds the btn-flat style). The flat button that you have in the documentation website (in the buttons page) only has the btn-flat class, not btn btn-flat as it happens in my code.

So far, the only workaround I can think of so far is to apply a CSS class to each float button that overrides the on-hover color.


Combine several jobs into one to save computation resources

Develop a new ticket generator: combined_tickets_generator which combines combine_n_job jobs into one by " && ".
Because the initialization of the container and environment will take some time, which is a constant offset proportional to the number of jobs. So this PR will reduce those time.

@dachengx I quickly went over the code -- just for my understanding: Isn't this the same as just reducing the n_batch parameter?

@hammannr I see your concern. Sorry, I was not clear enough in the description. Each job will have a unique nominal_value or generate_value for a parameter. Sometimes the number of combinations of different nominal_values and generate_values is very large. If I want to do a gridscan of two parameters' nominal_values, and each has 100 different choices, I will need to submit 10000 jobs. This PR mainly solves such a problem. If I set combine_n_jobs` to 100, then I will only need to submit 100 jobs.


feat: create review list view

Description

This PR adds the feature column review in list view.
These changes are done because of user feedback and because Kris just needed to see if it could be done, and ended up doing it.

Related Issues

Closes #237
Testing

[ ] Yes
[ ] No, not needed (give a reason below)
[x] No, I need help writing them

I was thinking that I would be able to re-use some of the tests from the previous page, but by
Reviewer Focus
This PR requires an in-depth review.

Checklist

For all PRs that are not general documentation

[ ] Tests accompany or reflect changes to the code
[ ] Tests ran and passed locally
[ ] Ran the linter and formatter
[ ] Build has passed locally
[ ] Relevant documentation has been updated

@lwjohnst86, @philter87 am I right in thinking that because this is branched off another branch it won't run tests and doesn't require reviews until the first branch is merged in?

@K-Beicher that's correct 😁


Add pointers to the right TSP tools

Related to #143 Need to add some more information on how to solve the TSP problem

@Beramos https://github.com/MichielStock/STMO/blob/master/src/tsp.jl


After login, if the intended redirect is the backend it does not render, instead it stays on the login page.

This might be due to the fact that the redirection is via inertia but the backend uses a different root view.

This is the case only if you use Inertia - Vue stack for your breeze installation.

This issue is not related to acacia. It is an inertia issue and the way to get around it is to use the default (blade) stack for inertia in your app and include this code on the AuthenticatedSessionController::create method:
public function create(Request $request)
    {

        if (\Request::inertia()) {
            return Inertia::location(url()->current());
        }

        return view('auth.login');
    }

If you must use the Vue-Inertia breeze stack, then instead of redirecting after login (store() function), you can do Inertia::location to force a full page load.
public function store(LoginRequest $request)
    {
        $request->authenticate();

        $request->session()->regenerate();

//        return redirect()->intended(RouteServiceProvider::HOME);
        $path = session()->pull('url.intended', RouteServiceProvider::HOME);
        return Inertia::location(url($path));
    }

The same rules apply on logout.


Implement visible portion features

Implementing features for center, rotation, region and visibleMapRect. (See #49)
Still draft as I wasn't able to fully test the rotation yet, as either storybook or my browser is a bit buggy.
@Nicolapps however, could you possible do a quick review if the general implementation seems reasonable to you?

Hi @Nicolapps,
thank you so much for this valuable feedback! Those are all valid points which on most I haven't yet have a clear answer to give. I will think more about this in detail and give an update here.
Generally speaking, the ref solution would work. I just personally don't like refs so much and I might have to introduce further useEffects for updating the values instead of just being able to pass a state. Although, yes it shouldn't make too much difference in the implementation.
I am personally working with a bigger software development team (product not involving MapKit JS) and we try to avoid refs wherever possible, as they can be a mess in maintenance and can be hard to debug but this again depends a lot on what they are used for and how. And yes, this rather applies to React than Javascript, though still it would be prettier for the code not having to use state and refs and just to rely on state. https://react.dev/learn/referencing-values-with-refs#best-practices-for-refs


[docs] CToSwift: CF types, protocols, and swift_private

Basically, all the remaining easy cases. See #26947 for what else belongs in this document.

@swift-ci Please smoke test


How to reinstall

I uninstalled OpenSSH Client. And now it is gone from the Manage optional features panel and is missing from the Add a feature panel.
I tried adding the feature using the PowerShell script, but it is added in the history, but using SSH does not work.
Is there a work around to reinstall OpenSSH client

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

ID: 07417484-bd6d-e6f4-110e-e42c2428a369
Version Independent ID: 844743a6-f67b-421f-9591-13da36b23b14
Content: Installation of OpenSSH For Windows
Content Source: WindowsServerDocs/administration/OpenSSH/OpenSSH_Install_FirstUse.md
GitHub Login: @maertendMSFT

Unfortunately, Add-WindowsCapability failed. Error code = 0x800f0954

Same issue, can't remove \ install.

Hi,
I had the same isuue Error code = 0x800f0954, this issue related to bug in local network WSUS server
Fixed by next steps:

Run regedit as Administrator
Go to HKEY_LOCAL_MACHINE\SOFTWARE\Policies\Microsoft\Windows\WindowsUpdate\AU
Change UseWUServer to 0
Run CMD as Administrator and execute two commands
net stop wuauserv
net start wuauserv

After that try Install the OpenSSH Server
Add-WindowsCapability -Online -Name OpenSSH.Server~~~~<IP_ADDRESS>
After that change UseWUServer to 1
Done, hope it will help you.

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Determine what backend we plan to use

We are able to use Supabase and the PoC that Jayden has provided
We can use the C# .NET backend that Albert and Will have produced
We need to come to a decision about what backend to use.

@albert118 Could you please comment on this issue the points you raised yesterday, and then close this?

Looks good to me

A further point that crosses my mind while writing up the previous comment,
what options could be used instead of SupaBase? Are there better/simpler/more appropriate alternatives? I image a ExpressJs would be a possible choice too.


Support removing translations

Right now, removing a translation from the source translation file will cause a not implemented error.

Fixed by #7 00ccae134e01c865c9007b6a02d5ac21a91be42a


feat: add double quoting output

Resolves #19.

Coverage remained the same at 100.0% when pulling b37776182590b2f69e7ffd7d4ec02632e052e515 on add-double-quoting into 6fd0ac0abf737563f05d878d84cff82ad21d2e93 on main.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


uefi: Add get_variable_boxed

This alloc-only method is a more convenient form of get_variable.
Closes https://github.com/rust-osdev/uefi-rs/issues/776

Checklist

[ ] Sensible git history (for example, squash "typo" or "fix" commits). See the Rewriting History guide for help.
[ ] Update the changelog (if necessary)

README entry is missing.


suggetions are visible only with commented lines

Hi,
I read the issues and there similar cases to mine like, I can see the completions only in commented line. Those issues are closed but this bug or feature still exist. Is there any solution for this? Thank you!
I use spacemacs, auto-completions, lsp-mode with company. I tried with corfu, cape and it didn't work, not even with commented line. Below is my config.
;; Codeium config stats
;; we recommend using use-package to organize your init.el
(use-package codeium
;; if you use straight
;; :straight '(:type git :host github :repo "Exafunction/codeium.el")
;; otherwise, make sure that the codeium.el file is on load-path
:init
;; use globally
(add-to-list 'completion-at-point-functions #'codeium-completion-at-point)
;; or on a hook
;; (add-hook 'python-mode-hook
;;     (lambda ()
;;         (setq-local completion-at-point-functions '(codeium-completion-at-point))))

;; if you want multiple completion backends, use cape (https://github.com/minad/cape):
;; (add-hook 'python-mode-hook
;;     (lambda ()
;;         (setq-local completion-at-point-functions
;;             (list (cape-super-capf #'codeium-completion-at-point #'lsp-completion-at-point)))))
;; an async company-backend is coming soon!

;; codeium-completion-at-point is autoloaded, but you can
;; optionally set a timer, which might speed up things as the
;; codeium local language server takes ~0.2s to start up
;; (add-hook 'emacs-startup-hook
;;  (lambda () (run-with-timer 0.1 nil #'codeium-init)))

;; :defer t ;; lazy loading, if you want
:config
(setq use-dialog-box nil) ;; do not use popup boxes

;; if you don't want to use customize to save the api-key
(setq codeium/metadata/api_key "--------------------------------------------------------")

;; get codeium status in the modeline
(setq codeium-mode-line-enable
    (lambda (api) (not (memq api '(CancelRequest Heartbeat AcceptCompletion)))))
(add-to-list 'mode-line-format '(:eval (car-safe codeium-mode-line)) t)
;; alternatively for a more extensive mode-line
;; (add-to-list 'mode-line-format '(-50 "" codeium-mode-line) t)

;; use M-x codeium-diagnose to see apis/fields that would be sent to the local language server
(setq codeium-api-enabled
    (lambda (api)
        (memq api '(GetCompletions Heartbeat CancelRequest GetAuthToken RegisterUser auth-redirect AcceptCompletion))))
;; you can also set a config for a single buffer like this:
;; (add-hook 'python-mode-hook
;;     (lambda ()
;;         (setq-local codeium/editor_options/tab_size 4)))

;; You can overwrite all the codeium configs!
;; for example, we recommend limiting the string sent to codeium for better performance
(defun my-codeium/document/text ()
    (buffer-substring-no-properties (max (- (point) 3000) (point-min)) (min (+ (point) 1000) (point-max))))
;; if you change the text, you should also change the cursor_offset
;; warning: this is measured by UTF-8 encoded bytes
(defun my-codeium/document/cursor_offset ()
    (codeium-utf8-byte-length
        (buffer-substring-no-properties (max (- (point) 3000) (point-min)) (point))))
(setq codeium/document/text 'my-codeium/document/text)
(setq codeium/document/cursor_offset 'my-codeium/document/cursor_offset))
;;;
 ;;;
(use-package company
	:diminish
	:bind (("C-M-i" . company-complete)
       		:map company-mode-map
       		("<backtab>" . company-yasnippet))
     :bind (:map company-active-map
                 ("C-n" . company-select-next)
                 ("C-p" . company-select-previous)
                 ("C-s" . company-filter-candidates)
                 ("<tab>" . company-complete-selection))
     :bind (:map company-search-map
           ("C-n" . company-select-next)
           ("C-p" . company-select-previous))
	:hook (after-init . global-company-mode)
	:init
	;;:config
	(global-company-mode t)
	(setq company-tooltip-align-annotations t
      	company-tooltip-limit 12
      	company-idle-delay 0.05
      	company-echo-delay (if (display-graphic-p) nil 0)
      	company-minimum-prefix-length 0
     	company-icon-margin 3
     	company-require-match nil
      	company-dabbrev-ignore-case nil
      	company-dabbrev-downcase nil
      	;;; get only preview
        company-frontends '(company-preview-frontend)
        ;;; also get a drop down
        company-frontends '(company-pseudo-tooltip-frontend company-preview-frontend)
      	company-global-modes '(not erc-mode message-mode help-mode
                                 	gud-mode eshell-mode shell-mode)
      	company-backends '((company-capf :with company-yasnippet)
                         	''(company-dabbrev-code company-keywords company-files)
                         	company-dabbrev)))
;;;

Seeing the same in Doom Emacs. I suspect this isn't working with a language server that's running.

(defun complete-codeium ()
  "Codeium." ; Doc string.
  (interactive)         ; Specify is command.
  (setq completion-at-point-bkp completion-at-point-functions)
  (setq completion-at-point-functions '(codeium-completion-at-point t))
  (call-interactively 'company-complete)
  (setq completion-at-point-functions completion-at-point-bkp)
)

Having the same issue, it seems that configuration to add codeium completion to completion-at-point-functions does not work well - furthermore, I don't want to call codeium for each completion.
Here's a (probably very hacky) function to perform explicit codeium completion (my first elisp interactive function !)
You can then bind the function to specific key e.g. (global-set-key (kbd "s-n") 'complete-codeium)

@gg-mmill

(for some reason, a local override of completion-at-point-functions does not seem to work:

Should work... here's fixed version:
(defun complete-codeium-bis ()
  (interactive)         ; Specify is command.
  (let ((completion-at-point-functions #'codeium-completion-at-point))
    (call-interactively 'company-complete)))

You had (call-interactively 'company-complete) still in the let binding vs in the body.

Yepper, here you go...
If using corfu, I'd suggest also getting its sibling: cape.
This part matches above, if just wanted to run codeium by itself outside of completion functions. I.e. matches above. It's bound to C-c p c below.
(defun my/cape-codeium (&optional interactive)
  "Allow codeium capf to be run by itself"
  (interactive (list t))
  (when interactive
    ;; if also testing copilot, clear their overlay before showing capf popup
    (when (bound-and-true-p copilot-mode) (copilot-clear-overlay))
    (cape-interactive #'codeium-completion-at-point)))
(keymap-global-set "C-c p c" #'my/cape-codeium)

For full gory effect, this is my setup for corfu, cape and eglot...
This merges codeium, eglot, dabbrev, cape-line all together into one capf:
;; Turn on/off if want to use codeium completions
(defvar my/codeium-is-enabled t)

(defalias 'my/capf-eglot+et-al
  (cape-capf-super
   (cape-capf-silent
    (when (bound-and-true-p my/codeium-is-enabled)
      (cape-capf-properties #'codeium-completion-at-point
                            :annotation-function
                            #'(lambda (_) (propertize "  Codeium"
                                                      'face font-lock-comment-face))
                            :company-kind (lambda (_) 'magic))))
   (cape-capf-buster #'eglot-completion-at-point 'equal)
   (cape-capf-properties #'cape-dabbrev
                         :annotation-function
                         #'(lambda (_) (propertize " Dabbrev"
                                                   'face font-lock-comment-face)))
   ;; if line gets too chatty
   (cape-capf-silent
    (cape-capf-properties #'cape-line
                          :annotation-function
                          #'(lambda (_) (propertize " Line"
                                                    'face font-lock-comment-face))))))

;; eglot, just to validate with just eglot
(defun my/cape-eglot(&optional interactive)
  (interactive (list t))
  (when interactive
    ;; if also testing copilot, clear their overlay before showing capf popup
    (when (bound-and-true-p copilot-mode) (copilot-clear-overlay))
    (cape-capf-buster (cape-interactive #'eglot-completion-at-point) 'equal)))
(keymap-set eglot-mode-map "C-c p g" #'my/cape-eglot)

;; Since merging eglot and others, assign a key binding to test
;; can remove if feel good
(defun my/cape-eglot+et-al(&optional interactive)
  (interactive (list t))
  (when interactive
    ;; if also testing copilot, clear their overlay before showing capf popup
    (when (bound-and-true-p copilot-mode) (copilot-clear-overlay))
    (cape-interactive #'my/capf-eglot+et-al)))
(keymap-set eglot-mode-map "C-c p z" #'my/cape-eglot+et-al)

(defun my/eglot-capf-config()
  (setq-local completion-category-overrides '((file (styles partial-completion)))
              completion-category-defaults nil
              completion-styles '(orderless partial-completion basic)
              ;; add in file first, capf will know when in file mode
              completion-at-point-functions (list #'cape-file #'my/capf-eglot+et-al)))

(add-hook 'eglot-managed-mode-hook #'my/eglot-capf-config)

You can do about the same for elisp, lsp-mode, org-mode, etc. different mode completions.  I'll leave that up to you.
Also, cape has a bunch of other completions, like tempel, keywords, emoji's, etc. that could also be combined in like above.

Awesome! Yeah, actually my setup is with Corfu + Cape but not with Eglot. Thank you so much!

Now that kind of error when run codeium init "Doing vfork: Exec format error: I have API key already in my config. No completions appear. No error in codeium diagnose. Just basic config as it's adviced in readme. I am confused.

Good-o, glad it's working.  Probably should close this issue.

Hi, I reopened this as this is not a bug and related to my first question, Corfu, cape etc.
I would like to see the suggestions as inline not in the box. I disabled the wivew in box feature, now I don't see anything. I am aware of that I could do this with company-frontends variable (setq company-frontends '(company-pseudo-tooltip-unless-just-one-frontend company-preciew-frontend).
How I can do that with Corfu, Cape, lsp-mode?
Could you help please? Thank you!

Since nothing to do with Codeium, you should probably start by looking at Corfu's repo for info and help, googling, etc. 🙂
With that said, I believe what you are looking for is corfu-candidate-overlay:
https://code.bsdgeek.org/adam/corfu-candidate-overlay

It is not codeium bug or faulty , but it is related to codeium . Codeium gives brief instructions to work with company but not with Corfu and Cape . That would be really useful for people if they write into readme  how to integrate as many folks use Corfu and Cape . Anyway, thank you so much!

I close now. Thank you!

..


GOOD_STRINGS addition novalidate

novalidate should be added to GOOD_STRINGS

thanks for your interest, however I haven't done any django work in years, and hence I am unable to develop this library further


Detected empty functions JS-0321

Unexpected empty method 'click'
src/app/modules/shopping-list/pages/add-products-shopping-list/add-products-shopping-list.component.ts
src/app/layout/navbar/navbar.component.ts
src/app/services/layout/sidebar.service.ts
can you assign me under hacktoberfest

@Srijan25 It's all yours 👍


root is recreated when setting fields in a new bloblang processor

Hey
I just came across some weird bloblang behavior which I don't fully understand
With input being {"message": "input message"}
pipeline:
  processors:
  - bloblang: |
      root.foo.bar = this.message
      root.foo."buz me".baz = "I like mapping"

I get back the expected
{"foo":{"bar":"input message","buz me":{"baz":"I like mapping"}}}
But this pipeline
pipeline:
  processors:
  - bloblang: |
      root.foo.bar = this.message
  - bloblang: |
      root.foo."buz me".baz = "I like mapping"

gives {"foo":{"buz me":{"baz":"I like mapping"}}}
Not sure why root seems to be recreated on the latter?
Bests & thanks

Hey @wirtsi, with the bloblang processor a mapping expresses how to create a new document by extracting data from an existing input document (https://www.benthos.dev/docs/guides/bloblang/about#assignment), which means when you assign fields to root you start off with an empty object. If you want to start off with all the existing fields by default then start your mapping with root = this:
pipeline:
  processors:
  - bloblang: |
      root.foo.bar = this.message
  - bloblang: |
      root = this
      root.foo."buz me".baz = "I like mapping"

Thanks Ash ... much appreciated, got it now.
But that would also mean that root = {} (https://www.benthos.dev/docs/guides/bloblang/walkthrough#your-first-assignment) is actually not needed because root is wiped on assignments anyways, right?


An Introduction to PyTorch Lightning

This closes #4642

Plagiarism check done ✅

@Bayler good job introducing PyTorch Lightning. I'm satisfied with the tutorial as is.

@collins-a Thank you for your review.

@Bayler the gif seems longs. Could you upload it to YouTube and then embed it in the article.

@Bayler the gif seems longs. Could you upload it to YouTube and then embed it in the article.

@WanjaMIKE  No worries. I'll do that then ping you when done

@Bayler the gif seems longs. Could you upload it to YouTube and then embed it in the article.

@WanjaMIKE No worries. I'll do that then ping you when done

@WanjaMIKE done. I hope I've embedded it correctly.
cc: @collins-a

@Bayler Please refrain from creating a pull request to close it.
We close PR based on how long they have been opened - so this PR was going to be closed (if I had not read the comments).

@Bayler Please refrain from creating a pull request to close it.
We close PR based on how long they have been opened - so this PR was going to be closed (if I had not read the comments).

Noted.

Nice tutorial @Bayler I like your style

Merged #4816 into master


[WIP] Flytectl get started added for running example

TL;DR

Added Get started guide for registering first workflow

For testing examples are available at https://github.com/flyteorg/flytesnacks/releases/tag/v0.2.89
Type

[ ] Bug Fix
[X] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
Tracking Issue
https://github.com/lyft/flyte/issues/
Follow-up issue
https://github.com/flyteorg/flytesnacks/pull/166
https://github.com/flyteorg/flyte/pull/942

@evalsocket You would need to sign the commits using the GPG keys for this repo. Hence the merge is blocked.
Whenver you get time you can setup this for your account . Follow the instructions from this page https://docs.github.com/en/github/authenticating-to-github/about-commit-signature-verification
which is the same one from merging is blocked is label above.
Also if possible once you are able to do this , Can you add this to Getting started page so the contributors are aware of this new change.

@pmahindrakar-oss let's merge it after flytesnack release

Ping me or @kumare3 to merge it once the aritifacts are available for flytesnacks. And also instead of going of a static version can this pick whats the latest available.

@pmahindrakar-oss @kumare3 flytesnacks examples are available in new release, We can merge it

this did not work for me
flytectl register files https://github.com/flyteorg/flytesnacks/release/releases/download/v0.2.89/flytesnacks-core.tgz -d development -p flytesnacks --archive
INFO[0000] Using config file: [/Users/ketanumare/.flyte/config.yaml]
INFO[0000] Config section [root] updated. No update handler registered.  src="viper.go:349"
INFO[0000] Config section [logger] updated. Firing updated event.  src="viper.go:351"
Error: unexpected EOF

➜ flytectl version
INFO[0000] Using config file: [/Users/ketanumare/.flyte/config.yaml]
INFO[0000] Config section [admin] updated. Firing updated event.  src="viper.go:351"
INFO[0000] Config section [storage] updated. No update handler registered.  src="viper.go:349"
INFO[0000] Config section [adminutils] updated. No update handler registered.  src="viper.go:349"
INFO[0000] Config section [root] updated. No update handler registered.  src="viper.go:349"
INFO[0000] Config section [logger] updated. Firing updated event.  src="viper.go:351"
{
  "App": "flytectl",
  "Build": "cdc1938",
  "Version": "0.1.8",
  "BuildTime": "2021-05-26 15:26:36.030606 -0700 PDT m=+0.013616840"
}{
  "App": "controlPlane",
  "Build": "db32bb3",
  "Version": "v0.5.4",
  "BuildTime": "2021-05-21 00:11:58.498302524 +0000 UTC m=+0.028839574"
}

The reason for the failure - bad path @evalsocket -
flytectl register files -d development -p flytesnacks --archive https://github.com/flyteorg/flytesnacks/releases/download/v0.2.89/flytesnacks-core.tgz

worked!


Feature request: use 'dark' instead of 'dark-theme'

Would be a nice little feature!

You can change the dark and light mode selectors in the config:
  /**
   * Customize the selector used to apply the dark versions of the color palette
   * @default ".dark-theme"
   */
  darkSelector?: string;

  /**
   * Customize the selector used to apply the light versions of the color palette
   * @default ":root, .light-theme"
   */
  lightSelector?: string;

Ah I see, didn't realize that was a setting. Thanks!


Can useReduxState be used when mapStateToProps depends on component's own props?

@epeli many thanks for the lib! I've been playing with it today, and noticed this issue:
Example scenario: there's a cache of whatever in redux and we want to select data based on some id passed to component.
Like so: https://codesandbox.io/s/1qzzp84rq4 The User component gets an userId prop passed to it (which changes on button clicks) and pulls that user's name from redux. Currently it gets stuck with data for initial userId and ignores changes.
Am I doing something wrong? Is this by design?

Yes there were a bug related to this case. I ported your codesandbox example to a test case and made a fix for it.
The fix is now in the 0.1.1 version on npm.


Bug: "Add Selection To Find Next Match" - Selection not reset when using cursor to select a new word

VSCode Version:

1.25.1
1.26.0-insider

OS Version:

Mac OS X 10.13.6

Steps to Reproduce:

Open find dialog
Click word and press command-D to highlight full word
Select another word with duplicates (this should collapse the selection)
Press command-D once.
Expect to highlight only current word, but now two cursors appear.

See .gif for demonstration. This does not happen when using the keyboard to move the cursor.
Does this issue occur when all extensions are disabled?: Yes

Good catch, thanks!

Played with it a bit and find that it's not related to the find widget. It's about how we handle ctrl+d.
If the selection is empty, and we press ctrl+d, we will select current word.
If we already select one word, we will search for the next same word in the document, and then select it, if we press ctrl+d. This behaves the same as we put the cursor in the word (no selection), and press ctrl+d twice.
The query in the find widget reflects what's being searched.

I think there's a little more to it than that. The Ctrl+D behavior becomes "stuck" when it selects the second version, and pressing Ctrl+D again does not select the next instance of the word as you'd otherwise expect:

Alternatively, if this is working as designed, do you have any suggestions for how to restore the behavior from previous versions (1.24 and earlier)? Here's an example:

@misenhower thanks, the gif you showed above is definitely a bug, we need to fix that.

@rebornix I think this issue was closed by accident. There was an additional bug I noticed in my last screen recording but the original issue still exists.


Is this project till supported??

 "@angular/core": "12.2.6",
Project dead?

RIP :-)

Yes,
Expect new releases soon.


Player Movement break the game

Tried implementing player movement but after trying to draw the rectangle in a different way it stopped rendering things at all. Here is my code:
struct MainState {
    // position player bottom of the screen
    player_x: f32,
    player_y: f32,
}

impl MainState {
    fn new() -> Self {
        MainState {
            player_x: 0.0,
            player_y: 0.0,
        }
    }
}

impl event::EventHandler for MainState {
    fn update(&mut self, ctx: &mut Context) -> GameResult {
        // player movement
        if keyboard::is_key_pressed(ctx, KeyCode::A) {
            self.player_x -= 10.0;
        }
        if keyboard::is_key_pressed(ctx, KeyCode::D) {
            self.player_x += 10.0;
        }
        Ok(())
    }

    fn draw(&mut self, ctx: &mut Context) -> GameResult<()> {
        // make background black
        graphics::clear(ctx, Color::from_rgb(0, 0, 0));

        // draw player at the bottom of the screen with player movement
        let player_rect = Rect::new(self.player_x, self.player_y, 50.0, 50.0);
        let player_rect_mesh = graphics::Mesh::new_rectangle(
            ctx,
            DrawMode::fill(),
            player_rect,
            Color::from_rgb(255, 255, 255),
        )?;
        graphics::draw(
            ctx,
            &player_rect_mesh,
            (ggez::mint::Point2 { x: 0.0, y: 0.0 },),
        )?;

        Ok(())
    }
}

that doesn't work @NafizTech


add docker-compose for development and production

https://github.com/Invasor-de-Fronteiras/Erupe/commit/46bdce9908f9950bfa7340ca59adb872175b44a4


TwitterRipper: Linter warning: probably incorrect Long.equals(String)

[Java] Unlikely argument type for equals(): String seems to be unrelated to Long
        Long lastMaxID = 0L;
...
            if (tweets.size() == 1 &&
                    lastMaxID.equals(tweets.get(0).getString("id_str"))
                    ) {

Its comparing a Long object with string. Merging the fix.

Looks like https://github.com/4pr0n/ripme/pull/567 is the attempted fix, but I don't understand why the whole file changed. @Avinash13iitkgp can you provide more information about your change?


DOM doesn't sync with data in a rare case

Version
2.6.11
Reproduction link
https://codesandbox.io/s/adoring-framework-lx5k7
Steps to reproduce
Following the link.
What is expected?
I think dom should be updated when input something.
What is actually happening?
DOM is not updated.

If you add v-if="true" on <template v-slot:debugger>, or simply remove <template v-slot:debugger>, or change { scopedSlots: this.$scopedSlots } to { scopedSlots: { ...this.$scopedSlots} }, the example will work well. I think it is related to some diff operations.

The scopedSlots cannot be passed down to children just like that because it contains information that links it to the current component, creating a new object is necessary:
scopedSlots: {
        default: this.$scopedSlots.default,
        debugger: this.$scopedSlots.debugger,
      }
// or as you mentioned
{ ...this.$scopedSlots }

Otherwise the slot isn't rendered when wrapper changes


need to remove reference to ltupin repo

isnt't the ltupin repo now moved here:
$ git clone https://github.com/ltupin/ansible-aws.git

Resolved with the commit https://github.com/apachecloudstack/k8s/commit/d8486645f331296fe854afbb06cddc391d2dccfb
(commit lag)


fix: Status bar icon colors on iOS

Will fix #3123
https://user-images.githubusercontent.com/246838/195414838-e6de1eb9-692b-4f79-9cea-2846cbfa4add.mp4

Yes, but what about Dynamic Island support 🏝  ?

Yes, but what about Dynamic Island support 🏝 ?

Mmm I don't really understand the link. Dynamic Island is when you have a notification

Me trying to make humor, based on the screenshot 😭


Pull request

add circle.yml
remove .DS_Store
add a test case in testGetPost.py
solve warnings about unclosed files in testGetPost.py
unzip test data

@chrisfilo Thanks! Should I create another 3 PRs or just leave it.

It's fine for now.

Closes #3.

LGTM - merge when ready

I'm not sure whether CircleCI execute the test case
https://circleci.com/gh/frank-24/mriqcwebapi/15

It did, if you expand the section titled "$ python test/testGetPost.py"
You see:
..
----------------------------------------------------------------------
Ran 2 tests in 0.000s

OK

Yeah. While the execution time is 0.000s, and I tried to print out some info in the test cases which was missing in the output.

Yeah something is not quite right. Try:

increasing verbosity of the unittest
trying a test that should fail
debugging this interactively by SSHing to CircleCI

I'm debugging via SSH and here's the error message:
ConnectionError: HTTPConnectionPool(host='localhost', port=80): Max retries exceeded with url: /scenarios (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x7f1ce418c810>: Failed to establish a new connection: [Errno 111] Connection refused',))
Maybe the issue is related to an invalid URL address?

Did you run remember to spin up the API via docker-compose?
On Thu, May 4, 2017 at 2:03 PM, frank-24<EMAIL_ADDRESS>wrote:

I'm debugging via SSH and here's the error message:
ConnectionError: HTTPConnectionPool(host='localhost', port=80): Max
retries exceeded with url: /scenarios (Caused by
NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection
object at 0x7f1ce418c810>: Failed to establish a new connection: [Errno
111] Connection refused',))
Maybe the issue is related to an invalid URL address?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/mriqcwebapi/pull/5#issuecomment-299308430,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkpxbjOtZ7SKYlW4enWuhLw_-5rel1ks5r2j0NgaJpZM4NQFo_
.

Oh, I forgot.
I execute docker-compose up -d and then run python test/testGetPost.py:
it shows
----------------------------------------------------------------------
Ran 2 tests in 2.592s

OK

and the info I tried to print out

http://stackoverflow.com/questions/284043/outputting-data-from-unit-test-in-python
On Thu, May 4, 2017 at 2:27 PM, frank-24<EMAIL_ADDRESS>wrote:

Oh, I forgot.
I execute docker-compose up -d and then run python test/testGetPost.py:
it shows

Ran 2 tests in 2.592s
OK
and the info I tried to print out
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/mriqcwebapi/pull/5#issuecomment-299314110,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkp34cmsDxTvBjit0ZA6NMRiidDSQXks5r2kLRgaJpZM4NQFo_
.

https://circleci.com/gh/frank-24/mriqcwebapi/25
It seems to be working now.

Nice work!


Test cases

Would like a review of the code for the pyyest framework. I installed one simple test case that uses all defaults and it. From my perspective it is a pass.

Because I accidentally pushed without a sign-off I was forced to "force push" a  commit to correct that problem.
ptdropper force-pushed the test_cases branch from a4c6e79 to fbe4744


Added webserver start

Now it shows the map provided that you input your config and user
settings

Next thing to do would have the login page edit the config settings

Good, to improve productivity by not just can be merged by me, I sent you the invitation to desktop-dev team. Thanks a lot for joining us.


Export-DbaScript - Clean file even if no prefix is used

Type of Change

[x] Bug fix (non-breaking change, fixes #7451 )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Unit test is included
[ ] Documentation
[ ] Build system

When -NoPrefix is used, the file is not "initialized". Because of a possible prefix, every export is done with "append" to not lose the prefix. So the "initialization" of the file has to be done even if $prefix is $null.

Thank you so much 🙇🏼  Will release momentarily.


Assign merge candidate in order to be reviewed

As NHS admin reviewer of manual merge requests I want to be able to assign merge candidate to review.
In order to do that we need to implement mutation base on logic and spec

[ ] implement

[ ] test

please, see the comments on test cases page

[ ] try to assign new merge candidate and receive an error

{
  "data": {
    "assignMergeCandidate": null
  },
  "errors": [
    {
      "errors": [
        {
          "$.manual_merge_candidate_id": {
            "description": "has already been taken",
            "rule": ""
          }
        }
      ],
      "extensions": {
        "code": "UNPROCESSABLE_ENTITY"
      },
      "locations": [
        {
          "column": 0,
          "line": 2
        }
      ],
      "message": "Validation error",
      "path": [
        "assignMergeCandidate"
      ]
    }
  ]
}


docs(guide/expression): replace tt by code

Replace tag <tt> by <code>.

This element is obsolete. Use a more appropriate element, such as <code> or <span> with CSS, instead.

lgtm


[FEA] Audit WindowExec

Is your feature request related to a problem? Please describe.

Ensure Spark and Rapids plugin version of the exec match functionality.

Verify Config specific to the operator match.

Verify API is consistent and fully translated.

Port relevant tests.

There is difference in API. Will sync up with @mythrocks to understand the missing parameters.
case class WindowExec(     windowExpression: Seq[NamedExpression],     partitionSpec: Seq[Expression],     orderSpec: Seq[SortOrder],     child: SparkPlan)
These configs are in the tests of Spark test Suite-
Not required:
SQLConf.OPTIMIZER_EXCLUDED_RULES
SQLConf.WINDOW_EXEC_BUFFER_IN_MEMORY_THRESHOLD

Apologies for the delay in responding on this issue.
The reason for this variance is that these parameters were redundant. The partitionSpec, orderSpec, etc. were available from the Apache Spark WindowExpression class, via the WindowSpecDefinition. It is therefore available in GpuWindowExpressions, where the evaluation happens. I found that in GpuWindowExec was dropping those expressions on the floor, and extracting them in GpuWindowExpression.
Examining the code again, I see that Apache Spark uses it in the definition of WindowExec::requiredChildOrdering(). It doesn't seem to affect the execution of WindowExec, since this information is available in GpuWindowExpression.
@tgravescs, I'm not too familiar with requiredChildOrdering(). Would it be alright to ignore this API variance for now, until such a time as when the output column ordering is affected?

Thanks @mythrocks  for  detailed explanation. I did not know that the information is available from WindowExpression.

so I think the default requiredChildOrdering is going to be empty and the when Spark goes to make sure its requirements are fullfilled will always be true. It means that if the ordering got messed with at all during shuffle it won't fix it. I'm guessing that is never happening right now and I'm not sure off the top of my head a condition that would cause that.
I think it would be good for us to at least file an issue to investigate it further.  If we can add requiredChildOrdering back I think it would be good in case it does hit some case. The only issue might be if databricks is different.
override def requiredChildOrdering: Seq[Seq[SortOrder]] =
Seq(partitionSpec.map(SortOrder(_, Ascending)) ++ orderSpec)

If we can add requiredChildOrdering back I think it would be good in case it does hit some case.

I have filed #486. I'll try add this back in the next sprint.

Thanks @mythrocks and @tgravescs . I am closing this as other things are audited and follow on issues are filed.


Use .java-version file for local version

To make the tool better interoperable between the windows version and the Linux/Mac version it would be great if the .java-version file would be used to determine what java version to use locally.
This would greatly help when using this in teams with developers on different operating systems, but all using JEnv.

Roadmap:

Whenever jenv-getjava.psm1 is called it should check for an active jenv use
After that, it will check if a .java-version file is present and get the version
Then it will check if there is a jenv local specified

If so it will get the path, execute path/bin/java -version  and parse the version
If the retrieved version does not match with the .java-version file the user is warned

In case there is no local specified it will parse the version of the global and compare them
If they don't match or there is no global it will loop over every jenv and check if it finds a matching version and suggests creating a local

Here is a function to get the Java Version from the path to an executable
https://github.com/FelixSelter/JEnv-for-Windows/blob/main/src/util.psm1#L18
I need to know if you want to keep working on this or if I need to do it?

Refer to #37 for further implementation details

I like this, 'pyenv' also supports this. And its a nice documentation for git repos which JRE/JDK to use.


Remove Python 3.8 support.

This PR removes support for Python 3.8, effectively making Numba's new minimum supported Python version 3.9. Commits can be reviewed individually if preferred. This also fixes a bug in error handling in numba.tests.support.print_azure_matrix and augments that function with a print out of the "reverse" map such that the Azure information is available grouped by Python and also by NumPy version.

I have given the PR a look, current changes largely look good without any major issues/changes. There are a few instance still for 3.8 specific code that remain which the PR may have missed:
https://github.com/numba/numba/blob/51fce0235e8b7ba1dc9ae41657c546a608b02847/numba/core/transforms.py#L114
https://github.com/numba/numba/blob/51fce0235e8b7ba1dc9ae41657c546a608b02847/numba/core/transforms.py#L188
And a few questions I've highlighted below

Thanks for the review. RE the changes above, I left those in as it was not at all clear that the condition can only occur for 3.8 apart from what was in a comment from 7 years ago suggesting the case (there's no version guard in the code that will execute to prevent one or the other branch from running and there's nothing to suggest anyone has assessed this region with respect to other newer Python versions). I'm happy to try and remove it and then see if anything breaks on the Azure build, what do you think?

RE: https://github.com/numba/numba/pull/9310#issuecomment-1831893159
I ran the looplifting tests (numba.tests.test_looplifting) on Python 3.12 and printed the line number and which condition was hit, result is this:
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS > 1
188: LOOP EXISTS > 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS > 1
188: LOOP EXISTS > 1
114: LOOP EXISTS == 1
114: LOOP EXISTS > 1
188: LOOP EXISTS > 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1
114: LOOP EXISTS == 1

it looks like at least on 3.12 there are loops with exit count > 1, i.e. both branches are still in use. Can you replicate @kc611 ?

Note: py311_np126 failure is because I forgot to update the test count in the azure matrix config, it's now 18, not 21. Will fix.

e6eac03 removes the Python 3.8 specific commentary from numba.core.transforms.
19fc868 fixes the issue with the wrong number of test configurations declared in the Azure config.

it looks like at least on 3.12 there are loops with exit count > 1, i.e. both branches are still in use. Can you replicate @kc611 ?

Yes, this can be replicated locally. As a result, the correct course of action is to simply leave the branches as-is and just modify the comments. (which has been done in e6eac03)


Centered styles

Adding .center and .middle positioning styles for logs to support positioning like top center, middle right, bottom center and middle left as described/requested in #104.
I also updated the website documentation and demo script to demonstrate this functionality.
cc: @grafa, @bogal

I left the generated /website/css/styles.min.css and /website/js/all.js as well as the src .css and diet files out, but can commit those if needed. just let me know.

Merge?


Individual rdf4j jars could be turned into bundles

Currently, the only bundle information in any of the rdf4j artifacts is in the aggregated osgi-runtime bundle.  It can be useful to have each artifact be a bundle so they can be deployed separately.
I am currently rejiggering the OWLAPI osgi distribution as part of setting up a move to gradle, and I've put together some code to do some basic bundle wrapping of any non-bundle dependencies, but that's just using the jar versions as the package versions, which is not ideal.
Also, naive packaging doesn't distinguish between packages that are part of the public API, and packages that are internal to the module. This distinction is also relevant for jigsaw  (which also decides visibility at the package level, with the extra nuance that modules can limit which other modules have access to their contents).

Would using rdf4j-runtime-osgi help in the short term:
https://search.maven.org/#search|ga|1|a%3A"rdf4j-runtime-osgi"
Unfortunately, none of the regular RDF4J developers work on OSGI-reliant projects, so it doesn't get much love.

The big bundle can be dependency mangled by consumers.  It's still nice to have the individual modules bundled, and not too much extra work to do it the lazy way. I'm surprised it's not mandatory for Eclipse :)


Update README.md

Added Table of contents, PR welcome and License badges

Why h2 instead of ##?

I aligned the headings to center using h2 tags first, then i changed the alignment to left, so i used h2 tags instead of ##.

@amaljose02 can you fix the last h2 on contributing?
Thanks for the PR!
Can we show you some appreciation for the contribution?
Also, we're giving away some Hacktoberfest bonuses and goodies for this month. We'll get it all figured out after you shoot us an email after this is reviewed/merged.

@tzarebczan sorry, i didn't noticed the h2 at contribution. I've now changed the h2 to ##.

Please rebase and squash this.

closing in favor of #339


⚠ ClamAV has degraded performance

In b2db116, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in 6441039 after 19 minutes.


Change alt text to "close sidebar"

Improve accessibility (?). Not sure if AMP is already doing something here.

makes sense - LGTM


HHH-18675. Ignore generic, synthetic property in JPA-model

Hibernate should not map synthetic properties. However by copying the synthetic property the synthetic aspect is not copied. To circumvent this create a SyntheticProperty on copying. And ignore the property in the jpa-model.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

https://hibernate.atlassian.net/browse/HHH-18675

@beikov Thank you, I have changed your suggestions.
Is it possible to backport? Shall I then make another pull-request?

Yeah, please create a PR against the 6.6 branch. Please note that our license headers are slightly different on that branch though.


Map authorizer custom keys

Custom authorizers have a serious flaw in that they can not return headers. This means that the CORS header can not be set for 401's.
To work around this I will "allow" every request, but include the authorization status and reason(missing token, invalid token, expired, etc) in the authorizer context. However serverless-wsgi does not currently extract those keys.
I would like to setup a way to extract these authorizer keys...

Zappa has a simple solution for this issue: https://github.com/Miserlou/Zappa/blob/master/zappa/wsgi.py#L111
It will extract the principal ID and user ARN from the request context and pass it through WSGI using a custom header. I suggest adopting the exact same convention for this library...

I have ended up wrapping wsgi.handler with my own entrypoint. It extracts all of the authorizer keys and adds them to the headers before calling wsgi.handler.
If anyone wants to do this be careful because lambda has a system path setup with wsgi.handler in it! I used imp.load_source to get around this.


174422943 — Cohorts Controller: nested routes

This PR simply adds basic nested routes under project for the cohort actions.

Change routes file
add before_filter to limit projects

[#174422943]
https://www.pivotaltracker.com/story/show/174422943

@scytacki the nested routes piece of this wasn't hard to fit in.
I will add the links to the projects page when I get there.  This can be tested by manually accessing routes in the browsers address bar.


chore: change vxfield autoFocus api to autofocus

PR needs rebase.

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-sigs/prow repository.


chore: update launch screen text

:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Grocy docker patch fix

Fixes "blank white page after running docker".
Actually it fixes:
Warning: Use of undefined constant GROCY_USER_ID - assumed 'GROCY_USER_ID' (this will throw an Error in a future version of PHP) in /www/controllers/BaseController.php on line 47

Thanks - but I don't think this caused any problems, as it is only a WARNING for now...

I agree, but that's the only thing coming up on my end, when I do a clean clone and build. What's happening on your end?

You had mentioned (on the other pr), that you got white pages when trying to use it. The only thing I found was this warning - that when fixed, showed the app. Keep me posted if anyone says it isn't working with the docker version.

Sure - but was not me, was @cmantsch in https://github.com/berrnd/grocy/pull/78#issuecomment-430835742.

Sure - but was not me, was @cmantsch in #78 (comment).

yep it was me, and it seems like the changes fixed it. I now get the interface of grocy instead of a white page with only a warning.


Migration Guide or Examples

Hi there,
Thank you so much for the work you've done here.
It's a little vague on what has changed. On the surface it looks like a lot but it really doesn't seem like too much.
Would you be able to add some examples or a migration guide for common use cases?
The one I'm stuck on at the moment is running inside of https://github.com/cycjimmy/semantic-release-action

It's a little vague on what has changed. On the surface it looks like a lot but it really doesn't seem like too much.

That's a good assessment of it :sweat_smile:
The binary has the same interface it did before. The behavior of some of the subcommands has been improved slightly, but nothing significant.
The addition of the npm module is a convenience for publishing to a cargo registry like crates.io.
This section of the readme describes how to configure semantic-release-cargo to publish your package to crates.io when you invoke semantic-release.

I also feel very puzzled on how to use this action.
It would be nice if you could provide an example on how to use it with semantic-release-action as mentioned in the doc link you posted above?
Also, how is next-release-version possible to figure out what is the next version, by looking at the Cargo.toml?
My Cargo.toml is a workspace file and I wish to only cargo publish a subcomponent, is there a way to set it up? Thank you :)