Provide easier way to access the theme outside 'styled' components

It's easy to get the theme from components that are created using the styled way:
const StyledView = styled.View`
    color: ${({ theme }) => theme.color};
`;

But how to get from normal components or apply it for different properties? Example:
<ThemeProvider theme={{ color: 'red' }}>
    <Main />
</ThemeProvider>

<View>
    <Card aCustomColorProperty={PASS COLOR FROM THEME HERE} />
</View>

Notice how the property that needs the theme is called aCustomColorProperty, not style

We can use this issue to discuss the topic and provide different solutions.
I'll post mine in the next comment.
This was originally posted on stackoverflow and asked by @mxstbr to be posted here

This is the current solution I'm using to solve this:
Create a Higher Order Component (HoC) that will be responsable to subscribe to the theme changes pass a theme prop to any component:
import React from 'react';
import { CHANNEL } from 'styled-components/lib/models/ThemeProvider';

export default Component => class extends React.Component {
  static contextTypes = {
    [CHANNEL]: React.PropTypes.func,
  };

  state = {
    theme: undefined,
  };

  componentWillMount() {
    const subscribe = this.context[CHANNEL];
    this.unsubscribe = subscribe(theme => {
        this.setState({ theme })
  });
  }

  componentWillUnmount() {
    if (typeof this.unsubscribe === 'function') this.unsubscribe();
  }

  render() {
    const { theme } = this.state;

    return <Component theme={theme} {...this.props} />
  }
}

Then, call it on the component you need to access the theme:
import Themable from './Themable.js'

const Component = ({ theme }) => <Card color={theme.color} />

export default Themable(Component);

Styled-components could provide something like this inside the library.

@brunolemos awesome, I'm stealing this! Thanks :D

I had to create another wrapper to pass the theme around in the context. I don't think this HOC is needed but a clear naming convention for the theme provider that's in use would do it and we'd use react's contexType to get it etc

This could be a decorator, e.g.
import {withTheme} from 'styled-components'

@withTheme
class ... {
  render() {
    const {theme} = this.props
    ...
  }
}

Yeah I use it as a decorator (@Themable), it's great.
I was thinking, if the styled also passed the theme props, that could work. But, all components would rerender when the theme updates, even if it's not using the theme. So it's probably not worth it.
About the use of context, this is a good read about its problems: https://medium.com/@mweststrate/how-to-safely-use-react-context-b7e343eff076
Waiting for more opinions! I could do a PR if you think this solution is fine.

About the use of context, this is a good read about its problems: https://medium.com/@mweststrate/how-to-safely-use-react-context-b7e343eff076

We fix all of those already, don't worry 😉

I could do a PR if you think this solution is fine.

I think a withTheme wrapper would be helpful, please do submit a PR!
Don't use decorator notation in the docs though, as that's just unnecessarily confusing – not everybody knows how they work, and this works just the same but is clearer:
import { withTheme } from 'styled-components'

class Component extends React.Component {
  render() {
    const {theme} = this.props
  }
}

export default withTheme(Component)

Now people who know decorators know they can use it as a decorator, and people who don't know decorators also know how to use it without! Win-win 🎉

Was looking for an example like this on the docs/website and couldn't find it.  Eventually the right google search led me here, but this needs to get surfaced up further into the docs for advanced/themes

@mgan59 The "Advanced" section of the docs discusses how to do this with useContext, although it's admittedly buried in there. I think it should have its own header in the sidebar, because I almost missed it myself.

@zhaluza It's under: "Theming" -> "Getting the theme without styled components." I believe that that can't get clearer if we consider that any given sub-topic isn't more important than another. Please do consider that the site has a search, issues do, and if that fails maybe we can avoid your search next time by getting you a quicker community answer on Spectrum 🙌 If all else fails, although issues aren't the right place for questions most of the time, it would guarantee a quicker answer.
Anyway, I'll have to lock this, since this is four years old, so this pings a lot of unrelated people, on an issue that implemented part of this sub-topic, but not even all of it. Sorry!


Remove console.log from an article test

There seems to be an extra console.log laying around in one of the Article Tests. Probably got left behind by someone during debugging.

Thanks

Good catch @codydaig, thanks.


Option to specify location/disk

Could you add an option to specify the location for testing the disk?

The script uses whichever disk you are actively in when you call the script (essentially your pwd). So before you run the script, simply cd into a directory that belongs to the disk that you wish to test.


Include, exclude days no longer in integration setup

Before you submit a new bug report, please check that

[X] I have read frequently asked questions and limitations

Write your question here
Hi,
Include/exclude are no longer in integration setup or I am missing something ?
I've setup collection days at the begining of the year and I've just noticed that all my exceptions has gone.
Do I have to add exceptions via blueprints now ?
Mike.

You have confirmed that you have read the FAQ. Did you?

Ok, I've read it now. For me simple things got way to complicated.


Angular project can be served on a busy port

🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑
The issue occurs when using angular CLI command "ng s --port {portnumber}"
I have an API application running on the same machine at port 3000, and when I've used the same port with angular it was running successfully, but I can't access the angular app through the browser.
🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
If the problem persists, please file a new issue and ensure you provide all of the required information when filling out the issue template.

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
If the problem persists, please file a new issue and ensure you provide all of the required information when filling out the issue template.


Pr position derivative

based on this pr: https://github.com/PX4/Firmware/pull/7144
Mainly to test if it is beneficial to incorporate the rate of position estimate
TODO:
Verify that land detection and controller tracking improved

I flew the branch on a small 250 quad with badly calibrated Z accel bias to test the effect and did notice some discontinuities when commanding small height changes in alt control mode. I reworked the changes you made to blend between the two vertical velocity estimates here:
https://github.com/priseborough/Firmware/tree/position-derivative-wip
Based on a single flight It seemed to improve the smoothness, but I have run out of time today to test it further so will be interested to see what change it makes on your setup.

@priseborough:
Was it a discontinuity when entering altitude hold or when exiting it or both?
Do you have a log by any chance?

It was both ways. I will send you the log.

i don't think this is necessary anymore.


[AzureMonitorDistro] remove vendored inststrumentation libraries AspNetCore and Http

Supersedes #41370
OpenTelemetry.Instrumentation.AspNetCore and OpenTelemetry.Instrumentation.Http have both released stable versions. We can now remove the internal vendored copies of the beta libraries.
In a follow up PR, I'll upgrade OTel and add package references to the new stable libraries
Changes

remove vendored copy of OpenTelemetry.Instrumentation.AspNetCore beta
remove vendored copy of OpenTelemetry.Instrumentation.Http beta
remove reflection code from OpenTelemetryBuilderExtensions.cs

API change check
API changes are not detected in this pull request.


VSCode is quite slow to start

VSCode Version: Code 1.9.0 (27240e71ef390bf2d66307e677c2a333cebf75af, 2017-02-02T08:23:30.856Z)
OS Version: Darwin x64 15.6.0
Extensions:

Extension
Author
Version

vscode-docker
PeterJausovec
0.0.7

scala
daltonjorge
0.0.4

githistory
donjayamanne
0.1.0

javadebugger
donjayamanne
0.0.2

python
donjayamanne
0.5.2

vscode-npm-script
eg2
0.0.21

json-tools
eriklynd
1.0.2

vscode-npm
fknop
3.3.0

code-runner
formulahendry
0.5.0

vscode-javac
georgewfraser
0.0.23

PowerShell
ms-vscode
0.7.2

csharp
ms-vscode
1.5.2

Ruby
rebornix
0.10.1

java
redhat
0.0.7

startanyshell
remcoros
0.3.1

vscode-icons
robertohuertasm
4.0.1

HOCON
sabieber
0.0.1

I set VSCode as the default editor for many file types, so that I can double click and have them open in VSCode :-) I noticed quite soon the impact on my workflow, i.e. the time spent waiting for files to open. For comparison I reverted back to Sublime Text as the default editor and the difference is very visible.
From a quick test using time <command> it appears that VSCode is at least twice as slow, and perhaps slower when the default editor is already open (i.e. the open instance could be reused to open the new file)

@dluc when you feel a slow start you can use our new command to report a performance issue and that includes all the times spend. I am closing this one and wait for your report. You can do it via Cmd+Shift+P > Report Performance Issue

@bpasero

VSCode Version: Code 1.9.0 (27240e71ef390bf2d66307e677c2a333cebf75af, 2017-02-02T08:23:30.856Z)
OS Version: Darwin x64 15.6.0
CPUs: Intel(R) Core(TM) i7-4960HQ CPU @ 2.60GHz (8 x 2600)
Memory (System): 16.00GB (0.02GB free)
Memory (Process): 163.79MB working set (0.00MB peak, 111.86MB private, 74.14MB shared)
Load (avg): 1, 1, 1
VM: 0%
Initial Startup: no
Screen Reader: no
Empty Workspace: yes
Timings:

Component
Task
Time (ms)

renderer
window.loadUrl() => begin to require(workbench.main.js)
267

renderer
require(workbench.main.js)
1067

renderer
create extension host => extensions onReady()
752

renderer
restore viewlet
0

renderer
restore editor view state
411

renderer
overall workbench load
637

main + renderer
start => extensions ready
2185

main + renderer
start => workbench ready
1983

@dluc thanks, those timings look quite good to me. Note that we still look into performance via https://github.com/Microsoft/vscode/issues/15455


Update vulnerability definitions

Hello,
I can see in System Health that the update vulnerability definitions are ou of date on a fresh stackrox install.
How can I update them manually? For information the last update is 2023-08-12T00:02:22.748570390Z.
Thank you for your help.

@MrWazaby The database is initially filled with data from the time of the build, but will update after a while. Can you please check if this problem has persisted?

Indeed, I was not patient enough, sorry about that and thank you for your help!


Revert command not working

Getting
2019-05-17 14:43:51 There aren't any non-finalized blocks to revert.

But when running the node, I can see  best: #76913 (0x92fa…0741), finalized #4964 (0x22c7…44a7) so it should be able to revert
 Node name: red-structure-7006
2019-05-17 14:44:07 Roles: FULL
2019-05-17 14:44:07 Best block: #76913
2019-05-17 14:44:07 Local node identity is: QmREJHRyGAcYidPqpgYdzBD1fRDc2MKahSXSMd9VtR2xmS
2019-05-17 14:44:07 Listening for new connections on <IP_ADDRESS>:9944.
2019-05-17 14:44:09 Discovered external node address: /ip4/<IP_ADDRESS>/tcp/30333/p2p/QmREJHRyGAcYidPqpgYdzBD1fRDc2MKahSXSMd9VtR2xmS
2019-05-17 14:44:09 Discovered external node address: /ip4/<IP_ADDRESS>/tcp/30333/p2p/QmREJHRyGAcYidPqpgYdzBD1fRDc2MKahSXSMd9VtR2xmS
2019-05-17 14:44:09 Discovered external node address: /ip4/<IP_ADDRESS>/tcp/30333/p2p/QmREJHRyGAcYidPqpgYdzBD1fRDc2MKahSXSMd9VtR2xmS
2019-05-17 14:44:12 Discovered external node address: /ip4/<IP_ADDRESS>/tcp/30333/p2p/QmREJHRyGAcYidPqpgYdzBD1fRDc2MKahSXSMd9VtR2xmS
2019-05-17 14:44:17 Syncing, target=#90931 (3 peers), best: #76913 (0x92fa…0741), finalized #4964 (0x22c7…44a7), ⬇ 349.8kiB/s ⬆ 9.3kiB/s
2019-05-17 14:44:22 Syncing, target=#90932 (1 peers), best: #76913 (0x92fa…0741), finalized #4964 (0x22c7…44a7), ⬇ 1.1MiB/s ⬆ 10.3kiB/s

It works for me, are you selecting the right database using --chain and --base-path?

Tried with latest substrate and Flaming Fir and it is working and not able to reproduce it with my chain when sync scratch. Could be something wrong with the old db.


Possible missing dashboard dependency for icon setup

When installing icons for the dashboard, I am missing the inquirer dependency: https://github.com/open-rmf/rmf-web/blob/main/packages/dashboard/scripts/setup/setup.js#L2 . I don't think this is specified in the dashboard package.json file: https://github.com/open-rmf/rmf-web/blob/main/packages/dashboard/package.json

fixed by #462


Clean up LFS

When a new user tries to clone the entire repository onto their machine, we run into a git lfs bandwidth error. We need to remove any HDF file that is not used for unit testing.
@aufdenkampe - can you link to the steps you took to delete certain large files from the entire git history when splitting this repository from the original Clearwater repo?

@sjordan29, to split this repo from the original ClearWater repo that contained everything, I followed the approach outline in:

https://github.com/EcohydrologyTeam/ClearWater/issues/40

That resulted in a new repo (this one) in which I deleted everything else.
Do we want to separate out the Ohio River example and it's git history?
If not, a more streamlined approach might be to use the git-rm command to delete out everything we don't want.

@sjordan29, I just ran the following steps and commands locally, then committed ec7b797 back to origin. Let's see if this solves the problem.
From @ptomasula.

a nice little trick I came across to untrack files after you've already committed them to a repo, and before you added them to your .gitignore file. For example, in the allocations update, I had committed a bunch binaries and machine specific build files to the repository. As I slowly shift the repo towards a more finalized version, I wanted to remove these files to make it easier for others to work on in the future.

Develop your .gitignore file and commit all changes.
Clean the repo on your local machine using the following command

git rm -r --cached .

Add everything back to the repo

git add .

Commit the cleaned repo.

@sjordan29, when trying to reclone, the above mostly appeared to work, but I'm still getting "Clone failed" response at the very end because:
Error downloading object: examples/data/ohio_river/OhioRiver_m.p22.hdf (90a9d86)
This repository is over its data quota. Account responsible for LFS bandwidth should purchase more data packs to restore access.

So the good news is that the repo is no longer tracking all the other old HDF files that were committed.
The bad news is that the Git LFS Objects still exist in our remote storage and are counting against our quota. Unfortunately, these can't be deleted without deleting and recreating the repo, which will delete issues, stars, and forks. https://docs.github.com/en/repositories/working-with-files/managing-large-files/removing-files-from-git-large-file-storage#git-lfs-objects-in-your-repository  :(

I just removed LFS with 053dd819200236e20cc5076ec0fe95a40abcbefa

@sjordan29 and @jrutyna, this should now all be working, with LFS turned off. The one issue is that large files connected to the Ohio River and Sumwere Creek need to be fetched from our servers (as a temporary solution). I mentioned these in new readme files with 532e86cbe15ac5cf96b1832be9909fd2e936c56d


pytest failure at example notebooks,

fresh pull, env and install (using latest mamba instructions):
running pytest fails at smoke tests, due to not finding kernels, eg:
____________________________________________________________________________________________________ /Users/fred.shone/Projects/pam/examples/01_PAM_Getting_Started.ipynb ____________________________________________________________________________________________________
[gw1] darwin -- Python 3.9.16 /Users/fred.shone/mambaforge/envs/pam/bin/python3.9
Error - No such kernel: 'pam'
----------------------------------------------------------------------------------------------------------------------------- Captured log call ------------------------------------------------------------------------------------------------------------------------------
WARNING  traitlets:kernelspec.py:286 Kernelspec name pam cannot be found!
ERROR    traitlets:manager.py:92 No such kernel named pam
Traceback (most recent call last):
  File "/Users/fred.shone/mambaforge/envs/pam/lib/python3.9/site-packages/jupyter_client/manager.py", line 85, in wrapper
    out = await method(self, *args, **kwargs)
  File "/Users/fred.shone/mambaforge/envs/pam/lib/python3.9/site-packages/jupyter_client/manager.py", line 397, in _async_start_kernel
    kernel_cmd, kw = await self._async_pre_start_kernel(**kw)
  File "/Users/fred.shone/mambaforge/envs/pam/lib/python3.9/site-packages/jupyter_client/manager.py", line 359, in _async_pre_start_kernel
    self.kernel_spec,
  File "/Users/fred.shone/mambaforge/envs/pam/lib/python3.9/site-packages/jupyter_client/manager.py", line 182, in kernel_spec
    self._kernel_spec = self.kernel_spec_manager.get_kernel_spec(self.kernel_name)
  File "/Users/fred.shone/mambaforge/envs/pam/lib/python3.9/site-packages/jupyter_client/kernelspec.py", line 287, in get_kernel_spec
    raise NoSuchKernel(kernel_name)
jupyter_client.kernelspec.NoSuchKernel: No such kernel named pam

I can run notebooks fine after identifying the correct kernel. But pytest error persists.
I can fix by explictly installing kernel within env: python3 -m ipykernel install --user --name pam. This command is also used in the github build pipeline. But not sure how to include in local install? maybe this.

Separate complaint. I would rather not run the smoke tests on pytest - they are relatively slow which is annoying during dev.
Running a single module, eg pytest tests/test_17_vehicles.py, addresses this but then coverage reports as failing. 🤷🏻 . No biggy - just a niggle. Also just discover no coverage option: pytest tests/test_17_vehicles.py --no-cov.

The notebook tests are effectively integration tests, so I can understand why you would want to avoid them.
You can just run pytests tests/ to run the unit test suite.

Adding the kernel is the same as you would expect, just stop using python3: python -m ipykernel install --user --name pam or ipython kernel install --user --name=pam.
I'll add this info to the dev docs.


Support partial merge

Is your feature request related to a problem? Please describe.
I am making use of dataclasses and structured to build a structured config. Part of the dataclass from which the structured configuration is built has to be generated dynamically (using make_dataclass) at run time. This means that sometimes users can end up specifying excess configuration. Currently, this will result in an error when I would prefer to be able to merge what I can and then log the excess config.
Describe the solution you'd like
In addition to merge, it would be useful if a merge_known existed. This would provide similar functionality to argparse.parse_known_args, returning one object with the merged configuration and a second with config that could not be merged. This would be particularly useful for from_cli, in the event that not all the arguments on the cli belong to a single config.
Describe alternatives you've considered
I could likely go through each piece of configuration (multiple .yaml files and cli) and try and purge excess keys, but that doesn't appeal to me.
Additional context
I am relatively new to OmegaConf, so if there is a better way to do this, I am all ears.

In general, merge is very complex and I am not interested in doubling it's surface area by introducing a new variant.
Can you give an example snippet with your inputs and expected output?

Here is a simplified (relative to my use-case) example of what I am doing:
from dataclasses import dataclass, make_dataclass
from omegaconf import OmegaConf as oc
from typing import List

@dataclass
class BaseConfig:
    fields: List[str] = "???"

@dataclass
class Field:
    name: str = ""

def finalize_structure(user_config):

    fields = oc.select(user_config, "fields")

    FinalConfig = make_dataclass(
        "FinalConfig",
        [(f, Field, Field()) for f in fields],
        bases=(BaseConfig,)
    )

    return FinalConfig

if __name__ == "__main__":

    user_config = oc.from_dotlist(["fields=[field1,field2]",
                                   "field1.name=user1",
                                   "field2.name=user2"])

    FinalConfig = finalize_structure(user_config)

    config = oc.structured(FinalConfig)
    config = oc.merge(config, user_config)

    print(config)  # Works perfectly.

    user_config = oc.from_dotlist(["fields=[field1]",
                                   "field1.name=user1",
                                   "field2.name=user2"])

    FinalConfig = finalize_structure(user_config)

    config = oc.structured(FinalConfig)
    config = oc.merge(config, user_config)  # Error.

    print(config)

In the second case (the one which produces an error), a user has provided excess config i.e. they have specified only a single field but have provided config for a second field. I would really like the ability to do something like:
    user_config = oc.from_dotlist(["fields=[field1]",
                                   "field1.name=user1",
                                   "field2.name=user2"])

    FinalConfig = finalize_structure(user_config)

    config = oc.structured(FinalConfig)
    merged, unmerged = oc.merge_known(config, user_config)

where unmerged is a omegaconf.dictconfig.DictConfig containing all the parameters which could not be merged. This is very useful in my opinion as it means that if a user wanted to disable a field (as in my example), they wouldn't have to delete every mention of it across all their configuration. Instead, one could merge where possible and log the unused config.

Thanks for the additional context.
This is a very unusual usage of OmegaConf and is not  functionality I am interested in adding to OmegaConf.
One thing you can try is to merge the configs together as dicts without using the dataclass(es), and then sanitize the resulting config object from any extra fields, and finally merge the sanitized config on top of your schema dataclass.

One thing you can try is to merge the configs together as dicts without using the dataclass(es), and then sanitize the resulting config object from any extra fields, and finally merge the sanitized config on top of your schema dataclass.

This is likely the approach I will end up taking. Thanks for your swift responses.


Add Better Order of Parts: Numbered  (BOPN) from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of Spyros777, to add Better Order of Parts: Numbered  (BOPN) to CKAN.
Mod details:
name = /mod/2072/Better%20Order%20of%20Parts:%20Numbered%20%20%28BOPN%29
author = Spyros777
abstract = Sorts all parts (including DLC) within their original categories in a logical and elegant way (by adding a two digits in front of each part's name). Never again will you have to search for ANY part (fuel tanks, wings, adaptors, engines), as you will instantly find it without wasting any time. Never again you will have to scroll down to find a similar part as it will already be next to the one you used.
license = MIT
Homepage =
description = Sorts all parts (including DLC) within their original categories in a logical and elegant way (by adding a two digits in front of each part's name).
Never again will you have to search for ANY part (fuel tanks, wings, adaptors, engines), as you will instantly find it without wasting any time.
Never again you will have to scroll down to find a similar part as it will already be next to the one you used.
I used some hours of my time to make this mod, so that you don't have to use any of your time to find the parts you are looking for anymore.
NOTE:  Module manager is required.
Please direct questions about this pull request to Spyros777.

This was added in #7007.


Step_per_epoch variable gets overwritten in fit_generator

The correct custom value gets displayed in the progress bar, but is silently overwritten by the size of the training dataset divided by the batch size.

Can you show an example?

I'm seeing this as well. I downgraded to 2.0.9 for now. The provided steps_per_epoch seems to be overwritten by the number of default images (len(generaror)?)

I believe this was fixed in https://github.com/fchollet/keras/commit/2f3edf96078d78450b985bdf3bfffe7e0c627169. You can either install the latest Keras from GitHub to get the fix or as you suggested downgrade to 2.0.9.

Thanks for the response!


Regular slashes in UNC paths does not work

Windows (Power Shell, ...) also supports UNC paths using regular slashes, //wsl$/Ubuntu-18.04/home, but they are currently not considered as UNC paths by wslgit so no wslpath conversion is done.

This is now fixed in the develop branch. Thanks for your PR.


Expose dockstore.model.ts info in webservice API

Feature Request
As a first step for the implementation of #2247, expose all the info in dockstore.model.ts in a web service API.
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.7
┆Sprint: Seabright Sprint 8 Houseboat
┆Issue Number: DOCK-678
┆Epic: Serve UI from s3/CloudFront

Running for quite some time in staging.dockstore.org, considered verified


Multi-source project handling in the Lab interface

A problem with the vague way we handle defining inputs to a Tool is exposed by multi-source projects. If you have a Planet scene and a Landsat scene in a project, and you run that project through an NDVI Tool, the bands assigned to each spectral category (e.g. Red, NIR) will be different. I think we could skirt this with the ability to choose scenes within a project like in https://github.com/azavea/raster-foundry/issues/1948, but there may be a better way.

Another potential approach would be to detect the potential issue and alert the user, "hey, you've got scenes from a few different datasources here, and they may have different bands, please choose the band from each data source that you want."
At another level though, I wonder whether this kind of thing makes sense at all; Landsat 8 has a single NIR band while Sentinel 2 has four, none of which line up particularly well with the Landsat 8 NIR band in terms of either wavelength or resolution. So I'm not sure how valuable it would be to even let people do this; do we have a use-case?

@ddohler The use case in my mind is a monitoring project where you're trying to detect something and you'd rather have 100% recall with lower accuracy/confidence than 90% recall and perfect accuracy/confidence (e.g. an algal bloom in a water reservoir or a deadly disease in a corn field). You want as much coverage as possible, so you combine data sources, but you're applying your "detection" algorithm to both.
After a conversation about this with @designmatty, I am not convinced that in fact it's worth the overhead to separate projects out by source and not allow multi-source projects for the time being. You can then create a single tool that takes in multiple single-source projects as inputs and applies tailored algorithms to each source before producing parallel, asynchronous outputs. If you wanted to, you could compare those outputs as apples to oranges and later on we could solve for the problem of co-registering/resampling them to attempt an apples-to-apples comparison.


docs: add CI integration sections

Relateds #4

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

v0.0.5
GitHub release

Your semantic-release bot :package::rocket:


feature:add netstandard2.0 support

feature:add netstandard2.0 support  …
Microsoft.Extensions.Configuration.IConfigurationBuilder.Sources on netstandard1.3 is IEnumerable,and on netstandard2.0 is IList.

@laoshancun Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@laoshancun Thank you for signing the Contributor License Agreement!

Hi @laoshancun
Thanks for your PR, we appreciate the contribution, but if you have a look at our dev branch we've already got netstandard2.0 supported there. Packages from the dev branch are available in they MyGet feed if you would like to develop with them now https://www.myget.org/gallery/steeltoedev

ok ,thx


[Constants-Phase1] CFE Issue for Constants Implementation

Request: As per implementation plan,
***Analyzer and CFE
**The analyzer and CFE implements support for the new constant and potentially constant expressions behind the experimental flag.
The CFE plans to implement constant evaluation itself, rather than deferring it to the backends. The new constant features will be included in this implementation, so back-ends should not need to implement the new behavior.
If the implementation requires changes to the Kernel format, then backends may need to adapt to that, just as with any other Kernel format change.
Phase 1 ETA: please see issue [tracker] (https://github.com/dart-lang/language/issues/60)

The CFE scanner / parser has been updated to support the >>> operator.

@mit should the milestone for this issue be 2.3, I believe we are using 2.3 to denote all issues that need to be fixed for the next release.

@ajcbik

Moving to 2.3 (I missed the comment above as it was to @mit, not @mit-mit

Constant evaluation has landed in the CFE behind the experimental feature flag constant-update-2018.


Log is not created for quorum node

Hi Team,
I am trying to create goquorum network using quorum-dev-quickstart.I am using below command.
./network.sh up
But getting following issues.

Log is not created for quorum node in /log/quorum directory.
Tessara node is started in unhealthy status.

Kindly help out.
Regards!!

Hi Shweta,
I have tried with Ubuntu 18.04 OS and its working fine for me. So my question from the ConsenSys team, do you have recommendation for operating system where this setup works properly?

Hi @Shweta-hlf There are several containers that have exited in your screenshots and can't tell why. What version of the quickstart are you using and what are you system specs please?
@m0kr the quickstart works regardless of OS and targets MacOS, Linux, and Windows (with WSL2). Any OS of choice is fine per the user, however recommend that the specs of the machine allow the many containers to run depending on client of choice.


Ensure that the Ada amount in the mint field in Mary is 0

This is how it will be decoded:
instance
  (Era era, Typeable (Core.Script era)) =>
  DecodeMint (Value era)
  where
  decodeMint = Value 0 <$> decodeMultiAssetMaps decodeInteger

If the Integer is randomly generated, we won't preserve equality after a
serialisation roundtrip.

Don't merge this yet, there are some more problems with the generators that will need to be fixed.

Ok, with the current changes in this PR, the serialisation roundtrip tests in consensus seem to pass. Ready for review: @nc6 @redxaxder @uroboros
One of the GitHub Actions builds is failing with:
total amount of Ada is preserved (Chain):                                                           FAIL (430.85s)
      *** Failed! (after 286 tests):
      Exception while generating shrink-list:
        illegal value in txout: Coin (-11)
        CallStack (from HasCallStack):
          error, called at src/Shelley/Spec/Ledger/TxBody.hs:488:21 in shelley-spec-ledger-<IP_ADDRESS>-inplace:Shelley.Spec.Ledger.TxBody
      Use --quickcheck-replay=692496 to reproduce.


Heishamon breaking change FW3.2 --> Energy to Power

The following Topics will change their naming from Energy to Power after Heishamon 3.2 firmware upgrade. Are we already catching this breaking change?
TOP15	Heat_Energy_Production
TOP16	Heat_Energy_Consumption
TOP38	Cool_Energy_Production
TOP39	Cool_Energy_Consumption
TOP40	DHW_Energy_Production
TOP41	DHW_Energy_Consumption
Br
bucky2k

Yes, tried 3.3 and it breaks the integration. To add to the above list, also the heat pump model is not transmitted correctly, TOP92

Hello, I'll close this issue since Energy/Power is supported in latest pre-release (https://github.com/kamaradclimber/heishamon-homeassistant/releases/tag/1.3.0) of this integration.
@MEKadan could you open a separated issue with more details so I can investigate?

Hello, I'll close this issue since Energy/Power is supported in latest pre-release (https://github.com/kamaradclimber/heishamon-homeassistant/releases/tag/1.3.0) of this integration. @MEKadan could you open a separated issue with more details so I can investigate?

It could be that heishamon recently added more pumps to it's own code.. integration models.py is not updated 28 models vs 37 in heishamon.

Hello, I'll close this issue since Energy/Power is supported in latest pre-release (https://github.com/kamaradclimber/heishamon-homeassistant/releases/tag/1.3.0) of this integration. @MEKadan could you open a separated issue with more details so I can investigate?

It could be that heishamon recently added more pumps to it's own code.. integration models.py is not updated 28 models vs 37 in heishamon.

Yes, that's the case.
I could not get the updater script presented in models.py to work since I'm on win, but no big deal, it's easy to add your model manually when you know your model code e.g. from MQTT

https://github.com/kamaradclimber/heishamon-homeassistant/releases/tag/1.3.1 contains all models. Thanks


Add better documentation for how to use the -runQuery argument

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://github.com/snowplow/snowplow/wiki/CLA to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

Nudge @NielsKSchjoedt


Exception handling

Handle exceptions thrown from the API.
⚫ Accept codes 200 - 300 returned from API.
⚫ Handle codes 400, 403, 422 and 429 with custom exceptions.
⚫ Handle null player and guild objects with custom exceptions.
⚫ Handle unknown exceptions with UnknownAPIException.
Refactor PineappleException to NovopixelException.
Refactor toString() of NovopixelException (may want to change in the future.)

Looks good, except we haven't decided on a name yet. Novopixel isn't sticking as far as I know.

I'm in agreement 😄

@unpluggedsam update:
I believe the name we're using is Neopixel, as far as the votes go.

For future reference, the refactoring of the name should've been put in another PR.


[Android] Use GCM as default

Since c2dm is deprecated, maybe it is reasonable to use gcm as default?

Agreed - I was waiting for the final end-of-life for C2DM, which was June 2015, to pass - so makes sense to make that change now :)

https://developers.google.com/android/c2dm/


String match matcher with '?' doesn't work well

Subject of the issue
Hi, I encountered a case of strange failure.
describe 'String match' do
  it 'no problem case' do
    expect('abc?def').to match('c?d')
  end

  it 'problem case' do
    expect('abc?def').to match('bc?de')
  end
end

I expect both cases to be successful. But second case's result is failure.
Is this expected results for you?
Your environment

Ruby version:
ruby 2.5.0p0 (2017-12-25 revision 61468) [x86_64-darwin17]
rspec-expectations version:
RSpec 3.7

rspec-core 3.7.1
rspec-expectations 3.7.0
rspec-mocks 3.7.0
rspec-support 3.7.1

Steps to reproduce
describe 'String match' do
  it 'no problem case' do
    expect('abc?def').to match('c?d')
  end

  it 'problem case' do
    expect('abc?def').to match('bc?de')
  end
end

Expected behavior
Both cases will succeed
Actual behavior
Failures:

  1) String match problem case
     Failure/Error: expect('abc?def').to match('bc?de')
       expected "abc?def" to match "bc?de"
     # ./te.rb:7:in `block (2 levels) in <top (required)>'

Finished in 0.02429 seconds (files took 0.23092 seconds to load)
2 examples, 1 failure

Failed examples:

rspec ./te.rb:6 # String match problem case

Sorry. It seems that ruby's problem.


[ci] Build CW310 hyperdebug bitstream for master

Add job to build the CW310 hyperdebug bitstream for merged PRs, and
include the hyperdebug bitstream in uploads to the bitstream cache. Use
the fusesoc core name for a subdirectory in which to place the bitstream
and MMI files.

Does anyone have any idea why the CW310 Tests job was skipped? I am confused!

Does anyone have any idea why the CW310 Tests job was skipped? I am confused!

It would appear that the default condition is to traverse the entire dependency graph for the job and require that every node be 'Succeeded' or 'SucceededWithIssues'. That means that an internal node that is 'Skipped' will fail the condition.
I'm double-checking this by specifying a succeeded() condition against just the neighbors, but if true, it would be kind of an odd default.

Merging, as the failing tests are known problems.


Track deleted images in edit box

Hello.
I have one trouble and i do not have any ideas how to solve it (i checked up docs as well).
I use delete method to delete files from the server, like:
  afterRemoveImage: function (e, editor, $img) {
     ***
  }

and now i have question: how could i delete images only after i click save button for example?
i try so:
after every image delete do not delete this image, but store url (or id of image) in some array then after click on save button delete all images from this array, but! there is one bad case - if user press ctrl-z or pressed undo button: image will return back, but it's id is stored in array to delete.
is it real with froala to handle undo event's on images?
maybe you could give some ideas...
thank you.

You could check in the undo/redo events if the image got back in the editor and remove it from your array.

Docs provide only html before and html after. You propose to compare this two values?

You could check the content inside the editor and see if it contains any image that is in your array with deleted images.
The purpose of this repo is not offering support. If you have a license that includes support, please contact us via http://editor.froala.com/contact. Thank you.

hm, could i somehow on your page test this event?
on my app i see that it is not fired


Email Signup banner

position under spotlight
have a look at the variable sizes, this is how the banner should scale down.

updated in handoff file: https://www.figma.com/file/JXNtryaNYhY0XJNArzvPQx/Handoff?type=design&node-id=4287%3A10287&mode=design&t=6F2GCCKDxMPHLQU8-1
please, after the user fills out the signup, do not show this banner to him. Only to new user that did not fill up yet.
under there is also euro version - more accurate - use this one instead

email copy will be done - please don't merge the pr till it is finished.

👋

@exezbcz is this also using substack ? I had a quick look and the substack api widget that we using is done in a way that only allows one widget instance per page , i'll have a deeper look tomorrow if it's possible to get both footer and signup voucher to work at the same time ,  but if that is the case what do we do ,

can we hide the footer one if the top one is available ?

@Jarsen136 any thoughts

@exezbcz is this also using substack ?

no, beehiiv

@roiLeo seems we need a server from a server , do we have a sever or something to run this ?
https://support.beehiiv.com/hc/en-us/articles/12428276693015-Why-am-I-getting-a-CORS-error-when-trying-to-implement-my-form-

@roiLeo seems we need a server from a server , do we have a sever or something to run this ?

where do you have CORS? what's the issue here?

@roiLeo linked the pr https://github.com/kodadot/nft-gallery/pull/7622
trying to subscribe the user , using this api

and getting


Sectioned adapter + horizontal Adapter inside Vertical

Hello
Can you please make a good practice of how to integrated a solid sectioned list.
Adding an Horizontal scrolling inside a vertical adapter. (Make it smooth)
thank you
br,

Check out the demo app in the develop branch :wink:

Hello
I didn't anything related to horizontal scrolling

@AnthonyKoueik Clone the develop branch. Look at this activity: https://github.com/gotev/recycler-adapter/blob/develop/app/demo/src/main/java/net/gotev/recycleradapterdemo/activities/Carousels.kt

This has now been merged and released in 2.4.0. Docs: https://github.com/gotev/recycler-adapter#carousels


Minimum requirements for llama2-13b and llama2-70b fine-tuning

Hello!How many gpu I need for llama2-13b and llama2-70b to fine-tune ?

Transferring this to Llama recipe repo where we are a lot of fine tuning examples.

Hi! It seems that the question in the issue has been solved. Closing this issue now but feel free to reopen it if there are more questions.


ERROR. This operation couldn't be completed. Operation not permitted.

BEFORE SUBMITTING YOUR ISSUE, PLEASE LOOK AT THE PINNED ISSUES AND USE THE SEARCH FUNCTION TO MAKE SURE IT IS NOT ALREADY REPORTED. ALWAYS COMMENT ON AN EXISTING ISSUE INSTEAD OF MAKING A NEW ONE.
Describe the issue
Hello, I have followed the instructions to make the virtual machine for Windows 10 and Windows 11 on UTM, but every time I try to run the virtual machine, I get the following error: "ERROR. This operation couldn't be completed. Operation not permitted.", I am not sure what I am doing wrong here? I used CrystalFetch to download the ISO images as it says in the guide, I have tried uninstalling and reinstalling UTM, and moving it from my external hard drive to my local disk. I am using the ARM64 architecture, with the QEMU 7.2 ARM Virtual Machine (alias of virt-7.2) (virt) machine, and I have experimented with other architectures and machines to try to make it work, but nothing. It won't let me export the dubug log either.

Configuration

UTM Version: 4.3.5 (87)
macOS Version: Sonoma 14.0
Mac Chip (Intel, M1, ...): Silicon Chip

+1 Same issue.

UTM Version: 4.3.5
macOS version: Monterrey (12.7)
Intel i7 MBP

Found my issue:
"exec of /Applications/UTM.app/Contents/XPCServices/QEMUHelper.xpc/Contents/MacOS/QEMULauncher.app/Contents/MacOS/QEMULauncher denied since it was quarantined by Safari;4258FD59-63DD-4FC4-80C7-8C9B5471B8E and not approved by Gatekeeper, qtn-flags was 0x00000002"
navigated to the "QEMULauncher.app" and opened it manually, thus approving the quarantine exception.


ISSUE ABOUT  CPU USAGE RATE

HI, @gergondet
I attempted to use the  BaselineWalkingController to control a 48-dof robot with mc_rtc_ticker(200hz control loop frequency) . However I observed that the both the calculation speed of mc and the CPU usage rate are  low , are there any solutions available to make full use of CPU?（Or perhaps this is a normal phenomenon?）

hope to receive your answer

Hi @EthanZengg
This is not normal for computation times at least. Have you built the controller with CMAKE_BUILD_TYPE set to RelWithDebInfo or Release?

Hi, @gergondet,
Thank you for your response. Yes, I built it using the following command.
catkin build baseline_walking_controller -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_OSQP=ON --catkin-make-args all tests

I'm not sure this is the correct way to pass the build type, could you try:
catkin build baseline_walking_controller --cmake-args -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_OSQP=ON --catkin-make-args all tests

I thank it should be correct .However I rebuilt it using the new method ,and  the problem still persists. This is result displayed by mc_bin_perf when using the JVRC1


MINOR: Fix unit test failures from last commit, cherry-pick fix for SASL principal

@ijuma Can you take review please? It fixes PlainSaslServerTest which uses a class not present in 0.10.2. Also cherry-picked commit 9934d28a34351a63738f8e9bac6b53fd699a4905 to 0.10.2 since that is required for multiple mechanism support (SaslMultiMechanismConsumerTest was failing with the changes without this). Thank you!

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.10/3776/
Test PASSed (JDK 7 and Scala 2.10).

Merged to 0.10.2.


Focus on File After Open File Location Command

Is your feature request related to a problem?
Many people often use Recent items feature to open the folder or file they recently open, sometimes people also wanted to open file location - where the file is located. And the problem is, it will be difficult for users to find which file it is when the folder consists of many files and /.or folders.
Describe the solution
Try to focus the files when users click on Open file location.
Describe alternatives you've considered
A clear description of any alternative solutions or features you've considered.
Scope

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

System Information

OS Version: Windows 10 21H1 19043.1055
App version: <IP_ADDRESS>

Additional comment
Add any other comment or screenshots about the feature request here. What device type/UI layout and app version?

This issue should be resolved with #5282


[Bug]: Selecting Contact/Slashtag to Send Funds to Fails

After adding a couple contacts, I was able to send one on-chain payment. After putting half my funds on Lightning, I was not able to make any further payments to a Slashtag/contact by any method. After entering a payment amount, I would click on a contact and it would do nothing. Pasting a slashtag in also failed in enable continuing. Ordinary addresses do not have issues.
It is also not possible to pay a contact by going to the contact directly and hitting any of the icons.
Just added another contact that fails differently: it now throws "unable to pay invoice" error when trying to send funds.
Weirdly, when I enter my own slashtag, I am able to hit continue and slide to confirm a transaction, which then fails by informing me I cannot generate a route to myself.
I have attempted adjusting in settings whether Ln or onchain is the first choice and this does not seem to fix things.
I am on Android. A couple other people had similar issues, one of whom is on iOS, but was eventually able to get one transaction through to me. Overall, reliability is far too low as this makes the wallet largely unusable when it comes to the major feature innovation of push payments with nice UI.

Closing in favor of https://github.com/synonymdev/bitkit/issues/744


Alias keybindings in command palette

You can access user shortcuts in the command palette by typing "Keyboard Shortcuts":

The actual persisted file is named keybindings.json:

But typing "keybindings" doesn't turn up any suggestions:

If it's possible to alias some commands, it might be helpful to include other common terms for keyboard shortcuts like "keybindings" or "hotkeys"

@bpasero Is there any way for an action to define keywords that would be matched in the command palette?

Yes you can set an alias:
https://github.com/microsoft/vscode/blob/7edc0a4edb51ca4cfdda86a6e47a621c51802d8d/src/vs/workbench/common/actions.ts#L26
However it will appear in the picker (we use it to show english form of a command for translated UIs). If we feel that we need to tweak the alias to include more keywords, we could add it there but it would mean that you see this item with 2 lines (the original command name and the alias below).
Maybe it would be better to tweak just the label.

@bpasero Is not that the alias is used to show English terms in other languages?

As I said:

However it will appear in the picker (we use it to show english form of a command for translated UIs).

@sandy081 for the new quick access commands provider I pushed a change so that you can filter by alias but the alias will not show unless you use a non-english locale.

You could do this by adding the word Keybinding here:
https://github.com/microsoft/vscode/blob/da3a711f356e133162085d580c7d8b14f290cad8/src/vs/workbench/contrib/preferences/browser/preferences.contribution.ts#L227
This is not a solution for cross-language keywords though but I think not bad.

Just to confirm, does it mean that if I type Keyboard in non English locale, I do not find this command right?

@sandy081 visually if you are in non english locale the command palette looks like this:

We always allow you to find a result if you type the english command.
What I changed for the new quick access is that you can now add more words to the alias of the command and even though it will not show up in english locale you can still search for it.

Would be nice if this worked out of the box, but also might be accomplished via #50836

/duplicate
#50836

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Bump GitHub Actions Extensions

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 57.15%. Comparing base (1e90f3b) to head (719d3a3).
Report is 19 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #731      +/-   ##
==========================================
- Coverage   66.27%   57.15%   -9.13%
==========================================
  Files          22       30       +8
  Lines        2150     2978     +828
==========================================
+ Hits         1425     1702     +277
- Misses        591     1141     +550
- Partials      134      135       +1

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


Fixed URL out of link

Signed-off-by: TakaoShimizu<EMAIL_ADDRESS>What type of PR is this?
/kind bug
/kind content
/kind transalation
Any specific area of the project related to this PR?
/area documentation
What this PR does / why we need it:
Needed fix for URL out of link.
Translated running.md to Japanese.
Translated examples.md to Japanese
Translated configuration.md to Japanese
Special notes for your reviewer:
/assign @leodido
/assign @fntlnz

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign fntlnz
You can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Translated running.md to Japanese.

LGTM label has been added.  Git tree hash: 9bf60cc80d9d7cf0c5d58d87d7f645d5fe62ce9e

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

Needs approval from an approver in each of these files:

OWNERS [leodido]

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

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

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leodido]

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


Redesign Footer Section

Hi @Yash-Ainapure,
I’ve noticed that the footer section on your site appears misaligned and is taking up too much space. I can fix this and enhance its overall look. Once you assign me the issue, I’ll provide the new design within 24 hours of approval.
Let me know if you’d like me to proceed!

Hi @Yash-Ainapure,
I’ve noticed that the footer section on your site appears misaligned and is taking up too much space. I can fix this and enhance its overall look. Once you assign me the issue, I’ll provide the new design within 24 hours of approval.
Let me know if you’d like me to proceed!

Its good at present


direction of normal for meta file

and yours:

The normals I see both in your tutorial and mine don't have negative sign behind them. Why do you put a negative sign behind them in meta file?

got the reason for negative normals from https://github.com/liuyuan-pal/Gen6D/issues/7#issuecomment-1191033327
The normal is shown in the left-bottom panel. Note that the normal is opposite to the Z+ direction (up), so the Z+ direction for the object is (-0.04148,-0.13305,-0.99024)


Better usage of Home/End keys

I'd be nice if Micro supported using SHIFT+arrows for text selection like GUI editors (Gedit, etc...) do.


Database

Can I download the complete bundesanzeiger.de database somewhere?
I would like to be able to view the data for all companies.
Or is only the following way possible:
Get a list of all companies and then fetch the data using a for loop?

About the database, I do not think so but you could contact the support directly and ask.
Your second suggestion should work.


Make CodeEval respect device_eval_batch_size

What does this PR do?
Re-implementation of our CodeEval so that it respects the device eval batch size
What issue(s) does this change relate to?
Since the previous implementation silently overrode device_eval_batch_size to be generations_per_sample, there are two clear benefits of the rewrite:

Better utilization when generations_per_sample < device_eval_batch_size
Prevents OOM when generations_per_sample >> device_eval_batch_size

Regression testing
These are some regression tests using the PR code and the previous implementation (plus a patch to use temperature 0.2) which was not possible before @maxisawesome foundry PR (which was not merged at the time I ran these experiments). This experiments are with generations_per_sample=20 so some variance is expected.

Model
This PR
Before

meta-llama/Llama-2-7b-hf
12.47
13.69

codellama/CodeLlama-7b-hf
27.99
27.68

codellama/CodeLlama-13b-hf
30.73
30.37

deepseek-ai/deepseek-coder-6.7b-base
39.82
40.3

codellama/CodeLlama-34b-hf
41.31
42.29

deepseek-ai/deepseek-coder-33b-base
47.96
47.56

Phind/Phind-CodeLlama-34B-v2
65.12
65.27

Before submitting

[x] Have you read the contributor guidelines?
[ ] Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
[ ] Did you update any related docs and document your change?
[ ] Did you update any related tests and add any new tests related to your change? (see testing)
[x] Did you run the tests locally to make sure they pass?
[x] Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

Could you include in the PR description evidence that before and after this PR, results don't change (assuming you carefully set the hparams to be the same). Or if that isn't possible for some reason, at least a number on a popular model that we can match to something. Otherwise, LGTM and will approve once you have that

Added experiments on public models to the PR description

I think we have a couple of tests that have the right intention but are testing the wrong thing:

https://github.com/mosaicml/composer/blob/dev/tests/datasets/test_in_context_learning_datasets.py#L1622
https://github.com/mosaicml/composer/blob/dev/tests/datasets/test_in_context_learning_datasets.py#L1556

The idea is to make sure that inputs aren't overly left padded. The issue is that the ICL dataloader maps using dataset.map and decides the left padding based on the entire dataset. Before we were not catching that because we were only looking at all the examples.
To be fair, the test is testing for the ideal behaviour, the issue is that our ICL code doesn't do that and that contribution feels out of scope for this PR.

Reworked some of the tests. However, I'd like to know whether we can remove unnecessary left padding in a per batch basis.


Scene stacking with camera fade throws WebGL buffer overflow

Version

Phaser Version: 3.20.1
Operating system: macOS Catalina 10.15.0
Browser: Chrome 78

Description
Stacking a Scene above another Scene which uses a BitmapMask and then fading the top Scene camera generates WebGL buffer overflow indefinitely.
Example Test Code
class Scene1 extends Phaser.Scene {
    constructor() {
        super({ key: 'Scene1', active: true });
    }

    preload() {
        this.load.image('bunny', 'assets/sprites/bunny.png');
        this.load.image('phaser2', 'assets/sprites/phaser2.png');
        this.load.image('checker', 'assets/pics/checker.png');
    }

    create() {
        var bunny = this.make.tileSprite({
            x: game.config.width / 2,
            y: game.config.height / 2,
            width: 400,
            height: 500,
            key: 'bunny',
            add: true
        });

        bunny.blendMode = Phaser.BlendModes.ADD;

        var phaser2 = this.make.image({
            x: game.config.width / 2,
            y: game.config.height / 2,
            key: 'phaser2',
            add: false
        });

        bunny.mask = new Phaser.Display.Masks.BitmapMask(this, phaser2);

        this.input.on('pointermove', function (pointer) {
            phaser2.x = pointer.x;
            phaser2.y = pointer.y;
        });
    }
}

class Fader extends Phaser.Scene {
  constructor() {
    super({ key: 'Fader', active: true });
  }

  create() {
    this.cameras.main.transparent = true;
    this.cameras.main.fadeOut(0);
  }
}

var config = {
    type: Phaser.WEBGL,
    parent: 'phaser-example',
    width: 640,
    height: 480,
    scene: [
        Scene1,
        Fader,
    ]
};

var game = new Phaser.Game(config);

Additional Information

If you try to flash the camera, the same error is thrown and the flash takes no effect.
If you don't fade the camera, the error doesn't happen.
If the fading camera Scene is below the Scene with BitmapMask, the error also doesn't happen.

Interesting and oddly specific discover: Drawing a transparent single pixel at a Graphics GameObject, which has the highest depth at the Scene, at the last instruction inside the update method, seems to resolve this issue.

Just to say this was fixed in the 3.23 release, sorry for not updating this issue.


Support for log level

It would be awesome if you could add support for custom logging frameworks.
For example, I use the library loglevel because it allows me to configure what logs show up on the console with specific levels you can even change at runtime.
My use case is that I would love to see these logs in production to help diagnose an issue (where I likely don't have redux-devtools available), but I don't want them always showing up on the console without opting-in (I have a lot of actions happening all the time and it would clutter things up).
So, ultimately, what I would like to do is have the logs coming in as log.debug, or some level that is not shown by default, then I can turn them on at runtime with log.setLevel('debug') on the console -- suddenly I can see all my debug spew!

This would be great

I agree. I think we can implement applyLogger function (#1 will be solved too) which accepts logger implementation as argument. Then we pass applyLogger to applyMiddleware and now we see redux's middleware-like logger :)
Will release on this or next week.

Released!
https://github.com/fcomb/redux-logger/releases/tag/1.0.0

@theaqua This is great! but unfortunately it still does not quite provide the functionality I am looking for. Hard-coding console means that I cannot use something like loglevel (the library linked), which wraps the console methods.
I'll add a quick (non-breaking) PR to fix this if you don't mind :smile:

@idolize oh, right, I forgot implement this. Big thanks for PR!

Awesome!! Thanks @theaqua

I tried with loglevel like this:
import * as log from "loglevel";
import * as createLogger from "redux-logger";
const logger = createLogger({ collapsed: true, logger: log });

It works but the entries are not collapsed anymore. Anyone has a pointer how to configure this?


[hot-fix][CI]Disable mypy for ethosu

Fix for failing CI on ethosu mypy sanity checks
@tqchen @mehrdadh

cc @manupa-arm

Thanks for catching this! Any idea why it did not fail before?
cc: @mbaret @ashutosh-arm

Thanks @mikepapadim!  but I believe it passed CI with those checks. Did anything change on the CI ?

Ignoring missing stub warnings can be fixed with this:
mypy  --check-untyped-defs python/tvm/relay/backend/contrib/ethosu/ --ignore-missing-imports

this hit #9013, we also need to fix that.

Can we set up a big red button that merges something without CI passing?

this hit #9013, we also need to fix that.

This bug hits also stagging Jenkins.

@areusch I think this PR can be closed if #9052 get through.

i dont think committers have permission to merge a pr without CI green. maybe it's a merge conflict?

i think we don't need this anymore. closing, feel free to reopen if we do need it.


Vulkan: GridMap is not included in SDFGI baking

Godot version:

4.0, master branch, Commit Hash: 0cd98ec7e13038d09a77cf821e930be79026f943
OS/device including version:

Windows 10 v. 2004, RTX 2060, NVidia GeForce Game-Ready Driver v. 451.67, Vulkan backend.
Issue description:
My Grid Map is not being shown in SDFGI, despite being set to be used for baked lighting. Mesh Instances are able to use SDFGI with the same settings, but if it's in a Grid Map, SDFGI doesn't work at all. The expected behavior would be the Grid Map producing the same sort of lighting as the Mesh Instances.

Steps to reproduce:
Create a Grid Map and place objects with it, and then enable SDFGI. If you construct the same thing with Mesh Instances, you will notice that they affect the lighting, but the Grid map doesn't.
Minimal reproduction project:

GridMapSDFGI.zip

Does this mean all non-node-based mesh instances aren't included as well? (when using VisualServer only, that is) Or is it only a configuration issue?

Does this mean all non-node-based mesh instances aren't included as well? (when using VisualServer only, that is) Or is it only a configuration issue?

@Zylann, I lack the knowledge of the Godot rendering pipeline to properly answer that question. However, I can confirm that SDFGI Baking does include MultiMeshInstance nodes and CSG nodes with or without a combiner.

This seems to be working for me. The lighting doesn't always update right away, but in general it works.

I can confirm SDFGI works with GridMap cells on 4.0.stable. The glowing cubes are GridMap cells, they both receive and contribute to GI:


Go back

What problem does this feature solve?
Can i go back programmatically in history like in react-router?
What does the proposed API look like?
.

For the moment you can do window.history.back. I need to do an RFC for back/forward/go methods on the router to make them more useful than just calling the same function on window.history


Version bumps and fixes

Updated to newer web-view + others and fixed upcoming problems.

#87

Sorting my PR, maybe mark it for you later, for me it's inactive.


Can you provide the download link for your RSNA dataset

Thank you very much for your contribution. I think your work is very meaningful and excellent. I would like to cite your article, but I am currently facing an issue where there is no RSNA dataset available. Can you provide a download connection for the RSNA dataset that you have processed.

Thank you for having interest in my work.
As shown in my paper, you can have access to the dataset through Kaggle.
https://www.kaggle.com/competitions/rsna-pneumonia-detection-challenge/data

Thank you very much for your prompt reply. This dataset is unprocessed and not divided into training, validation, and testing sets. After processing, I found that there is an imbalance between abnormal and normal data in the dataset, which makes it impossible to make a fair comparison with your model. Therefore, we kindly request that you provide the dataset that has been processed by you.

The competition dataset is not allowed to redistribute to people who don't participate in the competition.
So could you show me that you have participated in the competition through Kaggle plattform?
https://www.kaggle.com/junyasato
I will send you the data using Kaggle.

Yes, I participated in the Kaggle plattform. I was allowed to get the data, thank you very much, I will contact you on Kaggle

I'm very sorry, I just followed you on Kaggle, but I haven't become a contributor to Kaggle and I can't send you messages. Can you send me the dataset via email. My email is<EMAIL_ADDRESS>. I'm very sorry to trouble you

I can contact you on Kaggle, so please tell me your username.

Thank you very much for your help.My kaggle is https://www.kaggle.com/shimishilei

I have shared the RSNA dataset file list with you on Kaggle.

Thank you very much, I have downloaded the CSV file of the dataset you provided! Thank you very much for your enthusiastic help to me！


docs: Update README.md

Added an overview of the portfolio project, including features and live URL.
Included a project structure section to give an overview of the repository's layout.
Listed technologies used (React, AWS, Terragrunt, etc.).
Included license and contact information.

This PR is included in version 1.0.0 :tada:


Refactor DatasetItem.vue

This pulls out the metadata and actions sections of DatasetItem.vue into their own (composition API) components. This refactor is needed so that I can make toggleable changes to this component for the Dataset Overview Page. I decided to put this refactor in its own PR so that it'll be clear what has actually changed vs what was just moved when the Dataset Overview Page PR lands.
There are 3 things worthy of note:

I extracted FilterLink into a separate component to reduce the amount that gets rerendered whenever the filters change. These links' URLs are updated on every keystroke into the search box. Pulling these into their own component, so that those updates were limited to just the links, and not the full DatasetItem, reduced rerender time by 40%. I was hoping for more, but it's still a noticeable improvement.
I extracted VisibilityBadge into its own component. It doesn't make much sense here, but I use it in the Dataset Overview Page as well.
The parentComponent: { store, router} option passed to mount.
This a workaround to some @vue/test-utils weirdness. When you mount(MyComponent, { router, store }), @vue/test-utils creates MyComponent with the {router, store} options, but it also creates a root component to hold it, which doesn't use those options. The issue is that Composition API code accesses store via ctx.root.$store, and $store isn't defined on that root component. The root component options are taken from that parentComponent option.

The first commit switches from using expect(wrapper).toMatchSnapshot() to expect(wrapper.element).toMatchSnapshot(), which is much better for diffing. For a much easier time comparing the snapshots, skip the first commit and disable whitespace comparison

@richardgoater

Did you measure the render performance in the dev tools btw?

The "40% render time reduction" figure is from comparing this branch vs master in Chrome dev tools. I tried using Vue dev tools, but its results were all weird - it reported ~25s of total CPU usage per keystroke in the free-text search box, for a ~300ms re-render.
The Chrome dev tools' Performance tab also shows the currently re-rendering component in the "Timings" section, so I'm not sure if the Vue dev tools Performance tab actually adds any value...


Aquatech Heat pump

Describe the bug
Wanted to ask about another device but thought I can make it quick if ok. I am not sure how these things work so bear with me.
I have a new heat pump. I got it installed as it is tuya compatible. I have managed to add it in HA and using a tile card since don't have any options. Not sure if there are any other options
https://www.aquatechheatpumps.com.au/dynamic
The  issue is the current water temp in the tank seems to indicate for example 17deg C which is not the current water temperature. Tuya shows me correct values. I have added screens of tuya app and also another screen in tuya which has other diagnostic temps etc.
Added HA screen showing the incorrect temperature as well.
So basically trying to figure out if the heat pump developer has stuffed up or is the data being read incorrectly by the integration. If the issue is the developer of the heat pump I can try and chase it up but the tuya app shows the info correctly, so was thinking something to do with integration and how it gets the data. I could be wrong...
I also note the tile card does not correctly show the heat pump modes. They should be Stand Water, Eco Water, Hyb Water, Hyb1 Water, ELE Water as per tuya but does have different names.
Expected behavior
Correct temp in the tank shown.
Additional context
Add any other context about the problem here.
If the bug involves a device, then please include device diagnostics from
config_entry-tuya_local-f5914d7e4059e1d693c31106ad430b9a.json
Please check if there are messages from Tuya Local in the Home
Assistant log (Settings / System / Logs), and include those too. None
If applicable, add screenshots to help explain your problem.

This seems to be caused by PR #1686, which switched the displayed temperature from outlet to inlet temperature, because it matched what was displayed in their app, and they were getting garbage for the outlet temperature.
If the value in that case is always -50, then maybe it can be made conditional.

Outlet temperature is reported as -50, same as the original report, but your device seems to be reporting an additional dp 16 with data matching the current temperature in your screenshots.  I have added this as the primary current_temperature, with a redirect to inlet temperature when it is missing (as in the original report for this device).
The mode naming is based on predefined modes in Home Assistant. My preference is to fit in with HA rather than strictly following Tuya.
Stand (abbreviation for Standard) = Heat pump
HYB = High demand
HYB1 = Performance
ELE = Electric
The decision of which Hybrid option was which was based on the description in the manual, but this was for the Gen5 device.

The modes I am ok with I guess as its not too much of an issue but the temperature I have no idea what you mean. So water temperature data is missing?

I also notice the same issue on mine, I can see the bug fixed in the code, do you know when it is going to be merged?

It is merged already

sorry, I meant released

It is merged already

I was wondering why mine wasn't showing. I had to again go to hacs for the update to show. Curious when was it released?
BTW thank you so much for fixing this.


TDMSValue from_path method do not exist

Hi,
I am starting with rust and labview and I am trying out this library, thanks for the work men!
Just a question about how to extract a value from a TDMSValue. In this particular case I need to extract the date to send it to a .csv file. I have this function:
fn extract_date(tdms_file: &TDMSFile, file: &mut File) -> std::io::Result<()> {
    let name = tdms_file.segments.get(0).unwrap().metadata.clone().unwrap();
    let obj = name
        .objects
        .iter()
        .find(|obj| obj.object_path == "/")
        .unwrap();
    // .collect();
    let date = obj
        .properties
        .iter()
        .find(|n| n.name == "DateTime")
        .unwrap()
        .value
        .clone();
    println!("extract date ---------->>> {:?}", date);
    Ok(())
}

date is of type TDMSValue, reading the docs I found that this struct has a method pub fn from_reader(...)
that I supose that is to extract that value from value depending on each type, but It seems that this function does not exists. I am not able to call this function.

Do you know what can be happening? any help is welcome. 😃

Sorry I missed this.
I think it's a misunderstanding what what I made public. from_reader is supposed to take the underlying Read + Seek from when I open the TDMS file and read the correct amount of bytes that represent that TDMS data type, and return that data type.  In the metadata there are a few fields who's type are the TDMS DateTime - but if you're looking at pulling a channel's data who's value type is that TDMS Timestamp, then you'll need to get the iterator for that channel like we do here https://github.com/DnOberon/tdms/blob/main/examples/segments.rs#L24.
Hopefully that helps? TDMS is confusing enough and I'm sorry if I made it worse


Error with JSZip import while building

I'm trying to write a Firebase cloud function with docx-templates, but when I try to run tsc on it and try to deploy it, I get this error.
Could you suggest where to look to fix this?

these are my tsconfig:
{
  "compilerOptions": {
    "module": "commonjs",
    "noImplicitReturns": true,
    "noUnusedLocals": true,
    "outDir": "lib",
    "sourceMap": true,
    "strict": true,
    "target": "es2017",
    "baseUrl": "./types"
  },
  "compileOnSave": true,
  "include": ["src"]
}

adding  "allowSyntheticDefaultImports": true helped.


lang: add gen new_from_array for cpi::accounts::*

Close #2538

[ ] It remains to find the best place to test this functionality
[ ] Make sure that it really will be useful to the project

If the solution via AccounstCount seems too general for such a small function, then I think it is quite possible to get by with simply specifying the size only within the signature of the corresponding function, minimizing the necessary additions.


Some new client-specific (signature-) strings I've become aware of...

... since this is Outlook, probably worth to add.
EN: Get Outlook for Android
EN: Get Outlook for iOS
DE: Outlook für iOS  beziehen
DE: Outlook für Android beziehen

Hi @mstroeck please could you add some email with full context?
(i.e. containing email it is being replied to and a reply with one of those strings)

OK!

Done.
Commits:
https://github.com/hellogustav/elixir_email_reply_parser/commit/bc1a3920b3280d20b3d0917f9b157d1a06a934b2
https://github.com/hellogustav/elixir_email_reply_parser/commit/ae704daed7f593997d339f60713fb4d53ee5ecdc


Wikipedia API content stopped

Hi there,
I have encountered a problem. I have made use of the api with a  for loop to extract contents based on a  list of valid wikipedia artcle. But one thing I noticed is that after a while, the api encountered a connection lost and I understand that it can be fixed with adding headers  " [('User-agent', 'Mozilla/5.0')] ". But the problem is I have no idea where to add header in this case.
In the past I have just did this:
url = ' some url '
opener = urllib2.build_opener()
opener.addheaders = [('User-agent', 'Mozilla/5.0')]
but i have no idea where to put this code for the python wikipdia library? Pleas help me.  Thank you.

did you mean wikipedia.set_user_agent('Mozilla/5.0') ?


Remove slice allocator completion callbacks from global combiner

These were never intended to be under that lock (that they were was a mistake in the closure unification work earlier this year), and doing so can (sometimes) flatten our concurrency back to a single thread.
Improves throughput from 1.3mqps --> 1.8mqps

Known issues: #9225 #9398


Only Generate Jacoco Aggregate for Client SDK Types

This PR adds an additional clause of SDKType == "client" before attempting to generate the Jacoco aggregate. Other SDK types won't have Jacoco configurations and won't produce code coverage.

Closes for #18430

Closes for #18430


SslSocketFactory on httpClient

Describe the issue
Hi I am trying to somehow set sslcontext with my own JavaKeyStore to s3 client. Sofar I haven't found a direct way apart from creating my own implementation of HttpClientEngine
Steps to Reproduce
 S3Client {
   httpClient {
     //missing config options?
   }

Current behavior
--
AWS Kotlin SDK version used
0.32.0-beta
Platform (JVM/JS/Native)
JVM
Operating System and version
macOS Ventura 13.5

Thanks for submitting an issue @jimarek. You are correct, there is currently no way to configure a Java KeyStore directly in the bundled HTTP engines. We have backlog item https://github.com/awslabs/smithy-kotlin/issues/820 to add more TLS configuration options. It doesn't explicitly mention Java KeyStores but that's included in the scope.
Writing your own HttpClientEngine wrapper is a way forward for now. Additionally, the default OkHttp client engine should also use the KeyStore configured for the JVM so perhaps setting the JVM system properties javax.net.ssl.keyStore and javax.net.ssl.keyStorePassword may work for your use case.
Generally setting a KeyStore is not necessary for a client to communicate with AWS services. May I ask what you're trying to do?

We are limiting cert root authorities as per company policy.

Resolving this issue as we have a backlog item (https://github.com/smithy-lang/smithy-kotlin/issues/820) to implement the requested functionality.


Adds support for returning values from a transaction

Hey again, as I'm starting to build out more tests I hit a minor speedbump and improved support for $transaction
I also gave the README a more thorough look over, happy to drop those if you'd prefer a more focused PR

Greet update @orta, thank you!


Requirements for SODA Crystal - Add your pain points OR use cases OR  feature requirements

Issue/Feature Description:
You can add all your pain points OR use cases OR  feature requirements for SODA Crystal project
Project Focus : Unstructured Metadata management
You can add all your inputs in the comments. We will brainstorm to bring the first list
Reference
You can refer to some of the basic information collected or prepared for SODA Crystal here
Example 1:
We are struggling with metadata search for s3, especially the performance. Please find our concern details here  OR attach the information.
Example 2:
I found a project  which handles unstructured metadata management. Can we take inputs from there?
Example 3:
Feature request : Storing huge amount of IOT data in a common format can be a good feature? <More info here - link can be added or attach>
Example 4:
Do we consider data lake pain points like ?

Adding the input from @thatsdone here #8

(Possible) Feature Request
Add OCI (Oracle Cloud Infrastructure) Object Storage Service support.
As I think we need to discuss support matrix topic with Strato, I filed an issue (https://github.com/sodafoundation/strato/issues/1425) there too.

Comments from Rakesh, IBM:

Comments from Rakesh, IBM (SODA TOC) - Apache Parquet file management for large data set, schema, - there are challenges. Lake House solution solves (Apache Iceberg, Hudi). However if we can provide a specific solution then many organizations, it will be useful where they are not able to deploy lake house kind of solution.

From the competitive analysis, it looks like there could be two different focus areas for Crystal - (1) intelligent search of data sets and (2) data management using metadata. These two would have different requirements.
Searching inside unstructured data could be an area to explore (Subhankar)

We should keep track of old metadata for objects as well as new metadata. This will allow us to answer queries such as (1) what is the rate at which unstructured data is growing (2) Which are the fastest growing datasets (3) which department is adding data at the fastest rate (4) what should be the backup strategy for a dataset based upon how fast it is growing

Hi, I want to add one use case here:
For foundation models training, a lot of data is collected and is usually stored in cloud object store (COS). The data is normally in parquet files. If a Lakehouse kind of solution is not used, these files are very difficult to manage. Some issues are:

Get me the list of all the datasets we have (a bunch of parquet files in a specific folder represent a dataset, for example)
When the dataset was last updated
What is the size of each dataset, how many files are there. What is the folder hierarchy.
What is the name and size of each file, size distribution (how many files in size ranges 0-100MB, 100-200MB and so on).
Now, if we want we can go further - at folder level, get schema of parquet file (perhaps only first file) and show all the fields available in the parquet.
There is lot more information available in parquet metadata, but that is relevant for each file, and that might be an extensive retrieval and storage.
Once all this information is available, one should be able to search for folder name (which represent dataset name) etc.

To provide compelling features against the competition, Crystal should be able to support semantic understanding of the metadata

To provide compelling features against the competition, Crystal should be extensible to support multiple query languages including natural language


aiohttp.client_exceptions.ServerDisconnectedError: Server disconnected

Getting the following error
ERROR:__main__:Unexpected error
Traceback (most recent call last):
  File "/home/jan/homeassistant-satellite/homeassistant_satellite/__main__.py", line 246, in main
    async for _timestamp, event_type, event_data in stream(
  File "/home/jan/homeassistant-satellite/homeassistant_satellite/remote.py", line 28, in stream
    async with session.ws_connect(url) as websocket:
  File "/home/jan/homeassistant-satellite/.venv/lib/python3.9/site-packages/aiohttp/client.py", line 1141, in __aenter__
    self._resp = await self._coro
  File "/home/jan/homeassistant-satellite/.venv/lib/python3.9/site-packages/aiohttp/client.py", line 779, in _ws_connect
    resp = await self.request(
  File "/home/jan/homeassistant-satellite/.venv/lib/python3.9/site-packages/aiohttp/client.py", line 560, in _request
    await resp.start(conn)
  File "/home/jan/homeassistant-satellite/.venv/lib/python3.9/site-packages/aiohttp/client_reqrep.py", line 899, in start
    message, payload = await protocol.read()  # type: ignore[union-attr]
  File "/home/jan/homeassistant-satellite/.venv/lib/python3.9/site-packages/aiohttp/streams.py", line 616, in read
    await self._waiter
aiohttp.client_exceptions.ServerDisconnectedError: Server disconnected

The IP and token are definitely correct. Any ideas?

I found when I only provided script/run --host <IP> --token <TOKEN>, I got the same error message.
For me, the issue was that I didn't read the full README file.
My server has https certificates in front of it, so (per the README) I needed to add --protocol https.
Furthermore, because the https cert is validated against the hostname, the --host <IP> had to be updated to be a <HOSTNAME> instead of an IP address.
After these two additions, I got passed this error message.
The final command for me was:
script/run --protocol https --host <HOSTNAME> --token <TOKEN>


Added Fix: Preventing to delete the Chapter Section.

This fix restores the deleted YouTube video chapters, allowing users to navigate specific segments of the video that were previously inaccessible.

Good work :)


I can reproduce the error when using it in VitePress rc10:

This is still an issue with latest version of Vitepress.
I can reproduce the error when using it in VitePress rc10:

Originally posted by @userquin in https://github.com/unocss/unocss/issues/2977#issuecomment-1712538958

We temporarily close this due to the lack of enough information.
Please provide a minimal reproduction to reopen the issue.
Thanks.
Why reproduction is required


Update logging to include a more extreme debugging level

We primarily use info and debug for our two logging levels. However, there are some log statements that occur each trade, which could result in massive amounts of output and might not be desired for by a user trying to debug a problem. Let's add an additional level (name tbd... extreme debug? verbose debug? idk) that's below debug and is recommended to only be used when you're running a controlled setting with small numbers of trades, blocks per day, and days.

Crash reporting uses critical logging level in https://github.com/delvtech/elf-simulations/pull/937


Quest 2 export in 4.2.dev2 not working

Godot version
4.2.dev2
System information
MacOS 13.5 - Godot 4.2.dev2 - Mobile
Issue description
UPDATE: It is a Quest2 specific issue. I tried it using a regular renderer on a Android Samsung phone and it works fine in 4.2.dev2
Dear developers,
I was happy to see the XR action map working in 4.2.dev2. Unfortunately when I try to export to the Quest 2, the executable crashes on the Quest 2. Enabling debugging on the Quest 2 yields more information in version 4.1.1 of Godot than in 4.2.dev2 (see enclosed screenshots).

Debug messages from the quest 2 in 4.1.1

Debug messages from the quest 2 in 4.2.dev2
I tried several things to find more information:

Renamed the export templates from 4.1.1 to 4.2dev2. Prevents crashing, but shows a black windowed screen on the quest 2.
Used the changed files from 4.2dev2 (SCsub, config.py and OpenXR_api.cpp) to get the XR action map editor working in 4.1.1.
Recompiled 4.2dev2. Didn't do anything.
Tried different Android versions (30, 33, 34), which didn't work either.

I am not sure whether it is a MacOS specific issue. I haven't been able to test this on a Windows machine.
Steps to reproduce
Just make a simple XR project, using a tutorial:

A Node3D as root.
XR Origin
XRCamera3D pointing at the plane and capsule
MeshInstance3D - Plane (10x10)
MeshInstance3D - Capsule
A simple OmniLight3D to light the scene.

Use standard white materials for the plane and capsule.
Loaded: XRTools, XR Loaders, Android build templates (need to change the version in gradle-wrapper.properties to 7.6 in the URL to get a working compile)
Settings: OpenXR Enabled in project settings, ASTC texture compression enabled
Export Settings: Android Export - Gradle build - Android target platform 33 - Godot OpenXR for Meta - XR Mode: OpenXR
Connect Quest 2 headset via USB
Enable "deploy with remote debug" in the editor
Use the run remote to run the apk on the Oculus 2.
In 4.1.1 this yields a basic scene you can navigate.  In 4.2.dev2 this crashes on the headset.
Minimal reproduction project
Just extract the following files in a OpenXR42 directory and follow the steps written earlier.
minimal project.zip

Could it have some something to do with the following commit?
https://github.com/godotengine/godot/commit/29bbc17b48b6afed20c43967e9631b4e8cef04c2 , which has the following description:
Godot Android re-architecture
Decouples the Godot java entry point from the Android Fragment component. This enables the Godot component to be more easily reused across different types of Android components including Activities and Services.
Regards,
Eelco

Cc @m4gr3d

@eelcor This was indeed a regression introduced by the Android refactor. The issue has been resolved in https://github.com/godotengine/godot/pull/79681 and will be available in Godot 4.2 dev3.

Thanks for the update!
I recompiled Godot and unfortunately I am still not able to run the application on the Quest2. It still crashes on startup. Any changes in the export templates as well?
Any ideas?
Kind regards,
Eelco

Thank you!

I recompiled both the android_release.apk and android_debug.apk and still crashes on the quest. I also see an android.source in the export templates, where can I find or generate that file?


move AzureResource CRD to helm chart and move docs to docs/

may want to consider moving it charts/ in its own chart/ and add that as a dependency requirement in the osm helm chart only when a parameter is passed to enable it.

AzureResource was removed from the repo

AzureResource was removed from the repo


Migrating to futures::Stream for all iter_* method

In this PR I plan to rewrite all iter_* methods to use futures::Stream instead of manual iterator implementation.
Using futures::Stream could give us some very good advantages, for example there are tons of useful traits in futures that we can then use with our Stream's and its also overall a more standard way then what we are doing right now.
Todo:

[x] iter_download => stream_download
[x] iter_dialogs => stream_dialogs
[x] iter_participants => stream_participants
[x] iter_profile_photos => stream_profile_photos
[x] iter_messages => stream_messages
[x] search_messages => search_messages
[ ] search_all_messages => search_all_messages
[ ] Improve the overall code

Questions:

should we really change the name of methods to stream_*? or is there better names
should we re-export futures crate?
using pin_mut macro everywhere doesn't seems like a good idea, what else should we do? should we implement custom future for lower level types?

Closing since all commits are now in master. Thanks.


clickhouse-http-client: ignore link local addresses in getLocalhost

Summary
This a possible fix for #1729.
As far as I can tell link-local addresses are useless for the purpose of getLocalhost since it's used to get a referrer address, so I ignored them.
This solves my issue on my macOS system: for all others addresses the getCanonicalHostName is almost instantaneous.
Additionally, the comment says "use the first non-loopback address" but the code didn't actually do that because the break doesn't apply to the outer loop, I fixed that too.
Checklist
Delete items not relevant to your PR:

 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.

Good day, @vrischmann ! Thank you for the PR. We will review shortly.

@vrischmann Thanks for the PR. Did you test it also on the Linux box? Maybe we should use it only macOS systems

@vrischmann Thanks for the PR. Did you test it also on the Linux box? Maybe we should use it only macOS systems

Sorry no, I didn't think of it.

@vrischmann
Would you be able to test on Linux, please?

@vrischmann
Would you be able to test on Linux, please?

Yes I can, I'll do it next week.

Hi. I've tested the fix on a Debian server and it works fine, no blocking occurs.

Thank you, @vrischmann!
Will merge shortly


[WFLY-14963] Remove redundant boolean literals

Fixes https://issues.redhat.com/browse/WFLY-14963

Personally, I am not in favor of this kind of thing.
A change like this means dozens of possible bugs, and these changes lines don't fit on the screen when doing code review so the odds of mistakes not being noticed are quite high.

@bstansberry Hello Brian, thanks for your feedback. I assume I cannot gain acceptance by splitting this and withdraw the PR.


DOCS-#22: Redo README and docs for modin-spreadsheet

Remove most Qgrid content and rewrite content for modin-spreadsheet so it's presentable for new users.

Replaces main demo with new one including history
Still relies on some Qgrid documentation and demos
Removes mentions of support for JupyterLab
Removes readthedocs build

Documentation still needs for future PR:

Live demo notebook for public i.e. binder
In-depth descriptions of core functionality

@richardlin047 Looks great, should we also change the repo name to modin-spreadsheet?

Yes, I can do that and also make any changes to the docs as needed in this PR.


Update requirements for test

Coverage remained the same at 76.923% when pulling 4e62d8da91c034e5ff4cdbc4f6e63132375b20eb on dev/update_requirements into 4f60d83157bbc1d2a2d2c082b18cd428af29d4ac on master.


shortmess+=F prevent some message for autocmd event to be shown

Describe the bug
When F flag is present in shortmes, some autocmd does not work properly (the echom command does not display messages)
To Reproduce

create a file text.txt  which is not encoded in UTF-8 format.
test.txt
open vim vim -u init.vim test.txt  with the following setting:

 set shortmess+=F
 autocmd BufRead * if &fileencoding != 'utf-8' | echomsg 'file is not utf-8 encoded!' |endif

Expected behavior
A message should be displayed. But there is none.
Environment (please complete the following information):

Vim version:

VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Apr 10 2018 21:31:58)
Included patches: 1-1453

OS:  Ubuntu on Windows 18.04
Terminal: mintty

Additional context
If I change BufRead to BufEnter or BufWinEnter, the message can be shown.  Is this the expected behavior or a bug? Why does it only affect BufRead events?

That happens, because when Vim notices the F flag for the shortmess option, it will set the flag msg_silent=1 here.
Now that will influence BufReadCmd, BufReadPost and FileReadCmd  (and possibly other) autocommands that are called from readfile() and read_buffer().
The simple solution would be to simply use :unsilent  for your autocommand.

Thanks for your advice~

Just for reference: I've reported this for Neovim a while ago, when Neovim enabled shortmess+=F by default: https://github.com/neovim/neovim/issues/8675.

To add a bit of context I'm seeing this problem when trying to use echom from within a syntax file.
I'm following the tutorial here - http://learnvimscriptthehardway.stevelosh.com/chapters/45.html
I have the following directory structure:
potion-vim
├── ftdetect
│   └── potion.vim
└── syntax
    └── potion.vim

The file in ftdetect contains
au BufNewFile,BufRead *.pn setfiletype potion

And file in syntax contains
if exists("b:current_syntax")
    finish
endif

echom "Our syntax highlighting code will go here."

let b:current_syntax = "potion"

With the following vimrc no message is displayed/recorded.
shortmess+=F
set rtp+=/home/piers/.config/nvim/plugged/potion-vim
filetype plugin indent on
syntax on

If I remove the  shortmess+=F line then I see the message as expected.


Keyboard control updates to ChefPhatRadio and ChefStatusFilterGroup elements

:nut_and_bolt: Description: What code changed, and why?
Filters in a few different areas of the site were found to not be keyboard accessible.  This branch adds keyboard control support to any elements using chef-phat-radio and chef-status-filter-group.
These elements can be found used in
applications/service-groups
compliance/reports/nodes
compliance/reports/profiles
settings/node-integrations/add
:chains: Related Resources
Fixes: https://github.com/chef/automate/issues/3375
:+1: Definition of Done
User can keyboard tab through each of the options on the pages listed above, and can hit enter to select that option.
:athletic_shoe: How to Build and Test the Change
in hab studio build components/automate-ui-devproxy && start_all_services
in a separate terminal: cd into components/automate-ui
run make update-ui-lib to update the chef-ui-library
run make serve
navigate to the following routes and tab through the filters.  Use enter to select that filter:
applications/service-groups
compliance/reports/nodes
compliance/reports/profiles
settings/node-integrations/add
:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable
Example After:

thank you for doing this!! noticed a few more things that'd it be rad to fix too if possible

[x]  for both sets on applications, if we can add a few pixels of padding to the top and reduce it from the margin above we can get the focus ring to not be cut off on the top
[x]  think we can get the ones on infrastructure > client runs > {node} > attributes too?

@susan - updated we Resources and Run list inside that infrastructure view as well.  Found some toggles along the way that need updates but will require way more work so will make a separate card for those.