Using private docker repository for app

An example in the docs on how to use 'uris' in marathon application deploy configuration file would be helpful.

@ramukima The title seems to indicate how to deploy a containerized app from a private docker repository. If that is the intent, when defining your json payload to submit to the marathon REST API, the image name would look like the following:
registry.example.com/sample_ns/sample_repo:tag
That would result in pulling the image from the private registry located at registry.example.com.
Your comment references 'uris'. Are you referring to configuration files that the container requires?
If that is the case, I found the following docs on marathon to be very helpful:
https://mesosphere.github.io/marathon/docs/application-basics.html
https://mesosphere.github.io/marathon/docs/native-docker.html
I believe there is another issue currently open about a more complete example application and documentation related to it. But hopefully this will help in the interim.


Move voting from snapshot

Fixes #398  .
Changes proposed in this pull request:

Move helpers/voting from snapshot into snapshot.js

I also moved getNumberWithOrdinal from snapshot into snapshot.js because it's only imported from voting/rankedChoice.ts
See this PR too

Can you resolve conflicts @aiinkiestism

@aiinkiestism can you fix the build error?

@samuveth
Didn't notice, thanks!
That was a typescript error.

Looks good thanks @aiinkiestism !


[Android] fix inverted image (front camera)

Fix for #328

Thanks for this! manually added it to the Android branch I was working on (or we would have had a conflict anyway).  This will get merged into master shortly.


app run --local is not generating the .env with local credentials anymore

aio app init --no-login --standalone-app
app run --local
check .env => missing namespace,auth and apihost

my bad the file is accessible under dist/<my-extension>/.env.local


Cutscenes freezing

I don't know if this mod will ever get updated again, but the update for Bendy and the Ink Machine in October broke the mod to where you can't move at all after any cutscene plays. The mod refuses to open when I revert back to another version of the game, making it almost impossible to play. The mod is really good but the new Bendy update messed it up basically. I hope this can get fixed.

I just read the other issue under this one I typed this without realizing that someone else already posted about this issue, Sorry

Duplicate of #31


restructured protobuf generated code

abstract
This PR goes towards #243 and #244.
checklist

[ ] Make sure you read and understood the CONTRIBUTING.md.
[ ] Make sure you applied sufficient labels, milestone and assignee.
[ ] Make sure all TODOs are addressed.
[ ] Make sure there is sufficient documentation.
[ ] Make sure there are sufficient tests.
[ ] Make sure all commits are squashed before merging.
[ ] Make sure to delete the feature branch after merging.

Current coverage is 91.99% (diff: 100%)

Merging #256 into master will not change coverage

Powered by Codecov. Last update f8df89d...2d9a517


Inference on marked text

Can you please add functionality of marking text as an input to the model?

Could you explain what you're asking in more detail please?

Sure.
I would like the context for the generation to be either on the whole code if no code is marked. If the user marks a part of the code, I would like the marked code to serve as context for the generation.

Exactly.
בתאריך יום ה׳, 28 בספט׳ 2023, 17:03, מאת Luc Georges <
@.***>:

What do you mean by marked? Selected/highlighted?
—
Reply to this email directly, view it on GitHub
https://github.com/huggingface/llm-vscode/issues/47#issuecomment-1739286368,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AICB3XIKLCW5MZYSV4KZSZTX4V7SHANCNFSM6AAAAAAZ3NVEHU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Support Substitutions

IE: sudo chown -R $(whoami) $(npm config get prefix)/{lib/node_modules,bin,share}

this has 3 great examples of cases to support ^

Can be done relatively easily by integrating tree-sitter-bash. I'm planning to fork and add this (along with a more fully featured Fig spec parser I'm releasing soon 🤫)

Can be done relatively easily by integrating tree-sitter-bash. I'm planning to fork and add this (along with a more fully featured Fig spec parser I'm releasing soon 🤫)

That would be 🔥


Capture Page Screenshot not correctly linking in log when using a custom root directory

When using the new keyword Set Screenshot Directory in 1.7.2, the screenshot that's appended to the log isn't pointing to the correct location.

#460 addresses this.


bug report

There is an error in
https://github.com/IBM/Autozoom-Attack/blob/d9e94f3b7ed67eb7db4b375344297bed0a391534/setup_codec.py#L142
TypeError: unhashable type: 'slice'
Maybe this code should change to
config2 = config['layers'][1::]

thanks!!


parent dir items shake when redraw

What system are you running Yazi on?
Linux X11
What terminal are you running Yazi in?
st 0.8.5
yazi --debug output
Yazi
    Version: 0.3.3 (5eabd7d 2024-10-29)
    Debug  : false
    OS     : linux-x86_64 (unix)

Ya
    Version: 0.3.3 (Arch Linux 2024-09-05)

Emulator
    Emulator.via_env: ("st-256color", "")
    Emulator.via_csi: Ok(Unknown([]))
    Emulator.detect : Unknown([])

Adapter
    Adapter.matches: X11

Desktop
    XDG_SESSION_TYPE           : Some("tty")
    WAYLAND_DISPLAY            : None
    DISPLAY                    : Some(":0")
    SWAYSOCK                   : None
    HYPRLAND_INSTANCE_SIGNATURE: None
    WAYFIRE_SOCKET             : None

SSH
    shared.in_ssh_connection: false

WSL
    WSL: false

Variables
    SHELL              : Some("/usr/bin/zsh")
    EDITOR             : Some("nvim")
    VISUAL             : Some("nvim")
    YAZI_FILE_ONE      : None
    YAZI_CONFIG_HOME   : None

Text Opener
    default     : Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block-create: Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block-rename: Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

Multiplexers
    TMUX               : false
    tmux version       : tmux 3.5a
    tmux build flags   : enable-sixel=Unknown
    ZELLIJ_SESSION_NAME: None
    Zellij version     : No such file or directory (os error 2)

Dependencies
    file             : 5.45
    ueberzugpp       : 2.9.6
    ffmpegthumbnailer: 2.2.3
    magick           : 7.1.1-39
    fzf              : 0.55.0
    fd               : 10.2.0
    rg               : 14.1.1
    chafa            : 1.14.2
    zoxide           : 0.9.6
    7z               : 17.05
    7zz              : No such file or directory (os error 2)
    jq               : 1.7.1

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "/home/aidan/.local/state/yazi" directory.

Describe the bug
parent dir items shake when redraw
https://github.com/user-attachments/assets/6254c693-4140-43f9-af32-b35c9f6bd7a6
keys in the video above:

super + space: open terminal
super + 0: close window
ctrl + d: quit shell

note the different behavior of "close window" and "quit shell"

Minimal reproducer
I can only reproduce it in my home directory
Anything else?
no
Validations

[X] I tried the latest nightly build, and the issue is still reproducible
[X] I updated the debug information (yazi --debug) input box to the nightly that I tried
[X] I can reproduce it after disabling all custom configs/plugins (mv ~/.config/yazi ~/.config/yazi-backup)

I can't reproduce it:
https://github.com/user-attachments/assets/8b65c011-59d2-4381-9c5b-a230029ca82e
yazi --version
Yazi 0.3.3 (a8bd21ad 2024-10-30)

kitty -v
kitty 0.36.4

Yazi 0.3.3 (0baccdc 2024-11-01)
More detailed information:

This only happens when the home directory (~) is the parent directory, I even copied the home directory but the problem didn't happen:

https://github.com/user-attachments/assets/b2d05c2d-991b-4735-b429-780e16d3375f

When I open/close the window, the parent directory re-renders as if it were the current directory, but it shouldn't

Please compare:
https://github.com/user-attachments/assets/cef5be12-c91c-429b-bce7-e380a26b06f2
scrolloff = 0
and
https://github.com/user-attachments/assets/80d7ff9b-9eea-4d8b-8701-013adab929c4
scrolloff = 0

Please compare:
https://github.com/user-attachments/assets/5bbc4a3e-6f6f-4f5a-ac40-8e7f11fc01bf
scrolloff = 2
and
https://github.com/user-attachments/assets/f9a69529-87e9-4163-acb4-ab703793c5bf
scrolloff = 2

Unfortunately I still can't reproduce it even when I set the parent directory to $HOME (~), and I also tried setting scrolloff = 0, scrolloff = 2, scrolloff = 5 (default) in turn doesn't make any difference either
Can you try to debug it a bit?

okay
finally...

this issue is about zsh and HISTFILE
to reproduce it, please append this line to your .zshrc file
HISTFILE=$HOME/history

HISTFILE=$HOME/history:
https://github.com/user-attachments/assets/89531659-9fbf-4e4e-8217-a676d2a83f17
HISTFILE=$HOME/Downloads/history:
https://github.com/user-attachments/assets/b41dec44-4e49-4e52-aba7-c25d776988a9


Ctrl+H in interactive window input box results in warning popup

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

VS Code Version: 1.60-insider
OS Version: Windows 10

Steps to Reproduce:

Jupyter: Create Interactive Window
Place cursor in input box, hit Ctrl+H
🐛 Popup appears with message 'cannot read property replace of null'

Ctrl+F causes a 'cannot read property show of null' message.

Close duplicate in vscode-jupyter linked above - same problem with ctrl+f as @joyceerhl  mentioned above.


ValueError: output array is read-only when calling preprocess_adata

I am trying to run ikarus from R with reticulate. To prepare the data I used this line following the tutorial:
adata = ikarus$data$preprocess_adata(adata)

This give the error:
Error in py_call_impl(callable, dots$args, dots$keywords) :
  ValueError: output array is read-only

BTW, I also wonder what preprocess_adata does internally. If not using this function, would a regular log-normalization (i.e. normalize by total UMI counts, multiplied by a scaling factor, then log-transform with a pseudocount) do the job?
I am using R 4.2.1, python 3.10.13. Since this seems to be an issue happening in python, below I list the python module versions installed:
Package                 Version
----------------------- ------------
aiohttp                 3.8.5
aiosignal               1.3.1
anndata                 0.9.2
arboreto                0.1.6
async-timeout           4.0.3
attrs                   23.1.0
BitVector               3.5.0
bokeh                   3.2.2
boltons                 23.0.0
certifi                 2023.7.22
charset-normalizer      3.2.0
click                   8.1.7
cloudpickle             2.2.1
contourpy               1.1.0
ctxcore                 0.2.0
cycler                  0.11.0
cytoolz                 0.12.2
dask                    2023.9.0
dill                    0.3.7
distributed             2023.9.0
docutils                0.20.1
fonttools               4.42.1
frozendict              2.3.8
frozenlist              1.4.0
fsspec                  2023.9.0
GMM-Demux               <IP_ADDRESS>
h5py                    3.9.0
idna                    3.4
igraph                  0.10.6
ikarus                  0.0.3
importlib-metadata      6.8.0
interlap                0.2.7
Jinja2                  3.1.2
joblib                  1.3.2
kiwisolver              1.4.5
leidenalg               0.10.1
llvmlite                0.40.1
locket                  1.0.0
loompy                  3.0.7
lz4                     4.3.2
MarkupSafe              2.1.3
matplotlib              3.7.2
msgpack                 1.0.5
multidict               6.0.4
multiprocessing-on-dill 3.5.0a4
natsort                 8.4.0
networkx                3.1
numba                   0.57.1
numexpr                 2.8.5
numpy                   1.24.4
numpy-groupies          0.9.22
packaging               23.1
pandas                  2.1.0
partd                   1.4.0
patsy                   0.5.3
Pillow                  10.0.0
pip                     23.2.1
psutil                  5.9.5
pyarrow                 13.0.0
pynndescent             0.5.10
pyparsing               3.0.9
pyscenic                0.12.1
python-dateutil         2.8.2
pytz                    2023.3.post1
PyYAML                  6.0.1
requests                2.31.0
scanpy                  1.9.4
scikit-learn            1.3.0
scipy                   1.11.2
seaborn                 0.12.2
session-info            1.0.0
setuptools              68.1.2
six                     1.16.0
sklearn                 0.0.post7
sortedcontainers        2.4.0
statistics              <IP_ADDRESS>
statsmodels             0.14.0
stdlib-list             0.9.0
tabulate                0.9.0
tblib                   2.0.0
texttable               1.6.7
threadpoolctl           3.2.0
toolz                   0.12.0
tornado                 6.3.3
tqdm                    4.66.1
tzdata                  2023.3
umap-learn              0.5.3
urllib3                 2.0.4
wheel                   0.41.2
xyzservices             2023.7.0
yarl                    1.9.2
zict                    3.0.0
zipp                    3.16.2

Hi, I was having a simular issue! Have you tried using  the copy method on the passed object?  like this… adata = ikarus$data$preprocess_adata(adata$copy())  . It should fix the issue.

@CarloManenti Thank you for the comment.
I will be closing this issue for now. Feel free to re-open if the suggested approach doesn't work.


The svg file is not created

Detailed description
Hello guys. I need your input please. The problem is that the svg file is not created.
Βelow you will see what I have done.
I would be grateful for any help.
webpack.config.js
module.exports = {
    resolve: {
        alias: {
            '@createSvg': path.resolve(__dirname, './resources/img/icons')
        }
    }
};

webpack.mix.js
require('laravel-mix-svg-sprite');

Config.svgSprite = {
    loaderOptions: {
        extract: true
    },
    pluginOptions: {
        plainSprite: true
    }
};

mix.js('resources/js/index.js', 'public/js')
    .sass('resources/sass/app.scss', 'public/css')
    .svgSprite(
        '@createSvg',
        'public/img/sprite.svg'
    )
    .extract();

index.js (my app.js file)
function requireAll(r) {
    r.keys().forEach(r);
}

requireAll(require.context('@createSvg', true, /\.svg$/));

Your environment

Version used (e.g. Node 10 and Laravel Mix 5.0.1):
Operating system and version Windows 10:

This package currently doesn't support aliases for the directory containing your SVG files. Feel free to open a PR to support this!


Minor doc: Remove the extra blank line in data types javadoc.

The extra blank line is preventing the first lines from showing up in the package summary page.

Test build #28409 has started for   PR 4955 at commit 1621114.

This patch merges cleanly.

Test build #28409 has finished for   PR 4955 at commit 1621114.

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

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

Obviously, LGTM, go ahead if you want to make this change.

Thanks!  Merged to master and 1.3


Add Valerie Thomas to Project API

Title: Valerie Thomas added to Project API
Description: Adds Valerie Thomas' bio to the project API
🤔 Reason for PR & what it fixes/accomplishes: To add another tech pioneer to the API

Thanks for contributing. PR #27  👍


Specify 'extend_existing=True' to redefine options and columns on an existing Table object.

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from datetime import datetime
from typing import Optional, List

import sqlmodel
from fastapi_amis_admin.amis.components import InputRichText, InputImage, ColumnImage
from fastapi_amis_admin.models.enums import IntegerChoices
from fastapi_amis_admin.models.fields import Field
from fastapi_user_auth.auth.models import User
from sqlalchemy import Column, String

class VideoStatus(IntegerChoices):
    unpublished = 0, '未发布'
    published = 1, '已发布'
    inspection = 2, '审核中'
    disabled = 3, '已禁用'

# Create your models here.

class BaseSQLModel(sqlmodel.SQLModel):
    id: int = Field(default=None, primary_key=True, nullable=False)

    class Config:
        use_enum_values = True

class VideoPlatformLink(sqlmodel.SQLModel, table=True):
    __tablename__ = 'publish_video_platforms'
    platform_id: Optional[int] = Field(
        default=None, foreign_key="publish_platform.id", primary_key=True
    )
    video_id: Optional[int] = Field(
        default=None, foreign_key="publish_video.id", primary_key=True
    )

class VideoChannelLink(sqlmodel.SQLModel, table=True):
    __tablename__ = 'publish_video_channels'
    channel_id: Optional[int] = Field(
        default=None, foreign_key="publish_channel.id", primary_key=True
    )
    video_id: Optional[int] = Field(
        default=None, foreign_key="publish_video.id", primary_key=True
    )

class Platform(BaseSQLModel, table=True):
    __tablename__ = 'publish_platform'
    name: str = Field(..., title='PlatformName', sa_column=Column(String(255), unique=True, index=True, nullable=False))
    videos: List["Video"] = sqlmodel.Relationship(back_populates="platforms",link_model=VideoPlatformLink)
    channels: List["Channel"] = sqlmodel.Relationship(back_populates="platform")

class Channel(BaseSQLModel, table=True):
    __tablename__ = 'publish_channel'
    extend_existing=True
    name: str = Field(title='ChannelName', sa_column=Column(String(100), unique=True, index=True, nullable=False))
    description: str = Field(default='', title='Description', amis_form_item='textarea')
    status: bool = Field(None, title='status')
    videos: List["Video"] = sqlmodel.Relationship(back_populates="channels",link_model=VideoChannelLink)
    platform: Optional[Platform] = sqlmodel.Relationship(back_populates="channels")

class Video(BaseSQLModel, table=True):
    __tablename__ = 'publish_video'
    title: str = Field(title='VideoTitle', max_length=200)
    img: str = Field(None, title='VideoImage', max_length=300,
                     amis_form_item=InputImage(maxLength=1, maxSize=2 * 1024 * 1024,
                                               receiver='post:/admin/file/upload'),
                     amis_table_column=ColumnImage(width=100, height=60, enlargeAble=True))
    description: str = Field(default='', title='VideoDescription', amis_form_item='textarea')
    status: VideoStatus = Field(VideoStatus.unpublished, title='status')
    content: str = Field(..., title='VideoContent', amis_form_item=InputRichText())
    create_time: Optional[datetime] = Field(default_factory=datetime.utcnow, title='CreateTime')
    source: str = Field(default='', title='VideoSource', max_length=200)

    # channel_id: Optional[int] = Field(default=None, foreign_key="publish_channel.id", title='ChannelId')
    channels: List[Channel] = sqlmodel.Relationship(back_populates="videos",link_model=VideoChannelLink)

    platforms: List[Platform] = sqlmodel.Relationship(back_populates="videos", link_model=VideoPlatformLink)

    user_id: int = Field(default=None, foreign_key="auth_user.id", title='UserId')
    user: User = sqlmodel.Relationship()

Description
I want to achieve

one Platform has many videos ,one video has many platform
so VideoPlatformLink to many-many link
one Platform has many channel,one channel has one platform
so i use

    channels: List["Channel"] = sqlmodel.Relationship(back_populates="platform")

3.one channel has many videos, one video has many channels
so VideoChannelLink to many2many
it tell me
sqlalchemy.exc.InvalidRequestError: Table 'publish_channel' is already defined for this MetaData instance.  Specify 'extend_existing=True' to redefine options and columns on an existing Table object.

Operating System
Windows
Operating System Details
No response
SQLModel Version
0.0.6
Python Version
3.8.0
Additional Context
I have add  **    extend_existing=True**
but it still tell me
sqlalchemy.exc.InvalidRequestError: Table 'publish_channel' is already defined for this MetaData instance.  Specify 'extend_existing=True' to redefine options and columns on an existing Table object.

The example you gave has quite a lot of things going on, and a few external dependencies, so it's not too clear to me what the issue is.
Could you:

Post a minimum example that reproduces the issue - e.g. a script like those shown in the documentation which can be ran that generates the table and attempts to add some entries into it all in one go. Something that can just be executed with python3 ./issue.py makes it much easier to help out.
Post the output of python3 -m pip freeze - there have been some issues recently with newer versions of SQLAlchemy so this might be related. You can try pinning SQLAlchemy to a lower version (poetry install "sqlalchemy<=1.4.35") and see if that helps.

@wanghaisheng did you found solution for this issue and why it is occurring ?

@lokeeshbharathi never

@wanghaisheng It is occurring due to giving same __tablename__ = 'publish_channel in some other Class or Model.

@lokeeshbharathi   I  struggle a little time and give up . I will look my old code again to check that.thank you anyway

@lokeeshbharathi   I  struggle a little time and give up . I will look my old code again to check that.thank you anyway


Issue with responsive option and width 100%

The slider is not responsive (i.e. it doesn't scale) if the options are set as follows:
width: '100%'
height: 400
responsive: true (default)
autoHeight: false (default)
However if I change the width to fixed value then it works.
It also works correctly if I set the autoHeight option to true however in my case I can't do that because of another issue: #125

If you use a percentage width you need to indicate an aspect ratio manually by using the aspectRatio property.
Cheers.


Feature/306

https://github.com/OAGr/foretold/issues/306
Please, consider this PR after all previous.

This looks pretty good. Kudos with adding the slider.
2 things:

When I try to submit a measurement, I get an error.
The value should go from 0 to 100. Right now the lowest allowed number is 1.


chore: add test-fmt make target

This is a small addition to Makefile. Since the output can be verbose I'd prefer to separate the main build (make test) from this make test-fmt. Anyone who wants to look into the local test failures can run make test-fmt.

Release Note
NONE

As it's a side addition it is fine but you may want to check that gotestfmt is locally installed before trying to run it.

Actually the test-fmt target depends on do-build and do-build is where installing gotestfmt if it's absent. So it's no need to check.


Add links to community articles in the help docs

Done. http://snozbot.github.io/articles/index.html


Lots of extension errors when opening a WSL folder

I'm trying to open a folder in WSL and at startup I see lots of errors in the console, and the scrolling in the extensions view is broken.
I have disabled the built-in version of js debug, and have the nightly version installed an enabled.
I've tracked it down to this following code returning undefined for the ms-vscode.js-debug extension:
https://github.com/microsoft/vscode/blob/d04df068bdaf096b8bca742ef60065d2ef36e24e/src/vs/workbench/contrib/extensions/browser/extensionsWorkbenchService.ts#L664-L674
Specifically, it looks like this line: https://github.com/microsoft/vscode/blob/d04df068bdaf096b8bca742ef60065d2ef36e24e/src/vs/workbench/contrib/extensions/browser/extensionsWorkbenchService.ts#L668-L669
Returns an empty array, so that causes an undefined later -- which then causes the cascade of issues.
If I re-enable the built-in js debug, the issue goes away.

Sorry for breaking this.
To verify:

Disable a builtin extension for eg: js debug
Open a remote window (SSH / Dev Container / WSL)
Make sure there are no error in the console while browsing extensions.


golang filter: loaded_ may be overwrite unexpectly

Only the last one take affects.
https://github.com/envoyproxy/envoy/blob/c256b948683b8bd800df80f69f470e1202510242/contrib/golang/filters/http/source/common/dso/dso.cc#L59-L70
It's not a serious bug, better fix it after #25677 is merged, otherwise, we may have to resolve conflicts.

/milestone 48

/assign @doujiang24
I'll fix it soon, since https://github.com/envoyproxy/envoy/pull/25677 is already merged.


CoreCLR support

With CoreCLR 2.0 it is possible to run .NET Framework assemblies with CoreCLR, if they do not use anything not implemented in CoreCLR,
It means, that there is a chance that ILRepack will work on CoreCLR without recompilation, and I've tested it. For test, in ILRepack.exe folder I've created ILRepack.runtimeconfig.json with next content:
{
  "runtimeOptions": {
    "tfm": "netcoreapp2.1",
    "framework": {
      "name": "Microsoft.NETCore.App",
      "version": "2.1.0"
    }
  }
}

After it, I run it using dotnet ILRepack.exe ... command. And per my tests, it works! Unfortunately now it works on windows - as on unix environment it does Syscall.stat / Syscall.chmod, which are provided by mono and not exists in CoreCLR and crash after it.
To make it experimental runnable on Linux CoreCLR, we should call Mono API only when we are executed on mono.

We have now released 2.0.22 which should support Unit and .NET 6/7/8:
https://github.com/gluck/il-repack/releases/tag/2.0.22
You can now run either dotnet ILRepack.exe or mono ILRepack.exe on Unix.
I have removed the dependency on Mono.Posix and shelling out to stat and chmod instead. Mono.Posix.Netstandard wouldn't have helped as it requires shipping a bunch of native platform binaries, and we are rewriting to ship a single standalone ILRepack.exe.
We are skipping source server info on non-Windows and emitting a warning.


adding the docker file

will need a docker-compose
will need bind mounts to make sure next dev works as expected
closing for now, feel free to re-open

Yep, I tried implementing the bind mounts. For some reason, I kept getting new errors when I tried to implement the bind mounts on my Windows machine. I think my docker is corrupted will try on a Mac machine and try to add the docker-compose file.

Hey @hkirat!
Just a quick update – I've wrapped up the Docker integration with bind mounts using Docker Compose. Essentially, I've set up separate compose files for both production and development.
For the production environment, I've implemented multi-stage builds, ensuring optimized staging for quicker deployment. On the other hand, for development, I've utilized bind mounts, which have significantly streamlined our local development workflows.
Please review my changes,
Thanks


Platform/ferienapidotde

Description:
Binary sensor to indicate if today is a german vacation day or not
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#9021
Example entry for configuration.yaml (if applicable):
binary_sensor:
  - platform: ferienapidotde
    name: vaction_hh  # Optional
    state_code: HH

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

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

Hi @dennismuth-collectai,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


Update gamingkit feature guide

📝 Changed routes:

/solutions/gaming

(view full size)

Commit 7ae02261ced5970334f2c616f76f128bd4ccf6cc (https://thirdweb-9xmjgcv09.thirdweb-preview.com).


add minorEdit and message

With the current implementation each update on a page informs all watchers on changes.
According to the confluence developer the api supports a minorEdit flag to not trigger notifications when changed.
https://jira.atlassian.com/browse/CONFSERVER-38804?focusedCommentId=1555704&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-1555704
This PR adds these additional fields to the Version struct.

LGTM


Practicing pull

LGTM. thanks for the contribution!

Also check out this guide for hints on fancy README formatting https://help.github.com/articles/basic-writing-and-formatting-syntax/

Links to resume workshop
Day 1
Day 2


[refactor] #3822, #3737, #2437: Refactor iroha_data_model_derive

Description
Introduces various improvements to iroha_data_model_derive:

updated to use syn 2.0
add tests (where possible, see [1])
use darling derives and traits to parse macro inputs
add support for Self type in serde partially tagged enums (#3737)
address problems in derive(Filter) outlined in #2437

[1] I am a bit unhappy with the testing of derive(Filter), but it seems so intertwined with all the parts of iroha_data_model that it's really hard to test. Additionally, testing the generated Filter impl requires the transparent_api feature
Linked issue
Closes #3882, #3737, #2437
Checklist

[ ] make CI pass

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

360 of 448 (80.36%)  changed or added relevant lines in 11 files are covered.
5835 unchanged lines in 99 files lost coverage.
Overall coverage decreased (-1.2%) to 58.218%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

data_model/derive/src/has_origin.rs
21
22
95.45%

data_model/derive/src/partially_tagged/resolve_self.rs
34
36
94.44%

data_model/src/predicate.rs
0
3
0.0%

data_model/derive/src/partially_tagged/mod.rs
39
43
90.7%

macro/utils/src/emitter.rs
2
6
33.33%

data_model/derive/src/lib.rs
36
41
87.8%

data_model/derive/src/filter.rs
116
124
93.55%

data_model/derive/src/model.rs
19
27
70.37%

data_model/derive/src/id.rs
60
85
70.59%

macro/utils/src/lib.rs
31
59
52.54%

Files with Coverage Reduction
New Missed Lines
%

cli/src/main.rs
1
0.0%

cli/src/torii/pagination.rs
1
98.9%

config/base/src/runtime_upgrades.rs
1
51.72%

config/src/torii.rs
1
96.67%

core/src/smartcontracts/isi/block.rs
1
87.5%

crypto/src/merkle.rs
1
96.23%

data_model/derive/src/id.rs
1
79.2%

data_model/src/domain.rs
1
47.37%

logger/src/lib.rs
1
95.15%

config/src/lib.rs
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.2%

Covered Lines:
21490

Relevant Lines:
36913

💛  - Coveralls


Improve OnSystem RuboCop

This PR updates the existing on_{macos,linux} RuboCop checks to work for all on_{system} blocks. This checks for if statements in the top-level and correct them to on_{system}, and also checks for on_{system} blocks in def install, test do, and others to convert them to if statements.
brew style --fix should (ideally) be able to fix most of these issues on its own. The only time that it intentionally does not autocorrect (that I know of) is when there is an else statement after if MacOS.version .... In that case, it's much harder to figure out what the correct on_{system} block for the else should be, so it just points out the error and doesn't correct.
I've also updated the tests to be more extensive and test the auto-correct ability.
Of course, this PR can't be merged until the style issues in our taps are fixed, and that can't happen until we have a new Homebrew/brew tag (which will happen soon once the new on_{system} blocks have been around for a bit longer).

Review period will end on 2022-07-01 at 03:02:24 UTC.

Review period ended.

Now that I think about it, we will probably have to separate these two cops into individual PRs. We need the updated component order cop before making changes to formulae since it allows for the necessary methods to be used in the correct places. Then, once we've updated everything, we'll be able to enable the cop that requires that on_{system} is used instead of if. However, I'm going to sleep so that is something I'll do tomorrow.

I've extracted the components order cop to https://github.com/Homebrew/brew/pull/13503 so it can be merged first.

Now that we have on_system, this will correct if MacOS.version <= :monterey to on_system :linux, macos: :monterey_or_older. It will still correct all other if conditions to the correct on_{system} methods. Also, it will now correct on_system to the appropriate if statement if it's used inside a forbidden place

🎉 great work here!


[Feature]: function arguments tooltip multi-line

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
this is a follow-up to #4789 - with that issue, the user will be able to see the tooltip if they are writing and completing the function call in a single line. However, the tooltip does not appear for multiple lines, example:

With this issue, the user should be able to see the function argument tooltip just the same, whether they are using single or several lines.
Why should this be worked on?
this is an incremental change to #4789 , and improves the user experience

tracking this under the main feature https://github.com/appsmithorg/appsmith/issues/11899


Added support in AsyncGroup for dispatch_group_notify()

While AsyncGroup currently supports calling wait(), that doesn't always work when dealing with a group. An example is trying to call a series of nested functions that eventually perform an asynchronous network call. The group seems to finish, but it appears to complete immediately, leading to no action within the async method and no actual 'waiting'. Thus, if you were performing a segue after waiting and were setting up data for the segue in the group, the segue would complete without any of the data actually being passed along.
Creating a dispatch_group_notify call lets you wait until something is done and calling the action that needs to be done after the group is complete.  For instance, you can create a group that calls various async methods within the group and trigger a segue after the group has completed its tasks.
I created this in the AsyncGroup struct. notify() is private and called by public notify<QueueName>() functions. For my purposes, I'm only calling items on the main thread in this response, so notifyMain is what I need. But if you want to trigger something in another queue, you can do so.

I just saw the older pull request for something similar. I created this for our needs and decided to share it before looking at an existing request.


Orientation change state is lost on Login screen

Upon entering a server url and rotating your device you will lose your state at the login, you will be kicked back to setting your login url. Seems like an unintended behavior.

Yeah that seems like a bug.

I will fix this later tonight.


Proposal for dynamic query cache control

Is your feature request related to a problem? Please describe.
When running an Internet-facing graph at scale, caching is an essential tool to ensure that systems and infrastructure are only dealing with requests that they absolutely have to.
I have Hot Chocolate deployed behind an edge-cache, however the TTLs being provided to our edge are currently static and it’s essential for my use case that the cache configuration can be created dynamically based on the query and potentially other factors such as downstream API responses
Describe the solution you'd like
Apollo Server has a well thought out approach to this problem in the form of cache hints that I think we could learn from.
Take the following code-first example for an ASP.NET app:
public class Query
{
    [CacheControl(1200)]
    public IEnumerable<Product> GetProductList()
    {
        // fetch and return product list
    }
}

public class Startup
{
    public void ConfigureServices(IServiceCollection services)
    {
        services
            .AddGraphQL(
                provider => SchemaBuilder.New()
                    .AddServices(provider)
                    .AddQueryType<Query>()
                    .AddCacheControl() // Field middleware to collate the CacheControl attribute values
                    .Create(),
                builder => builder
                    .UseCacheControl()); // Query middleware to chose the lowest of the collated CacheControl attribute values
    }
}

This approach could also leverage resolver context internally to provide dynamic cache control like so:
public class Query
{
    public IEnumerable<Product> GetProductList(
        IResolverContext ctx,
        [Service] ProductDataLoader loader)
    {
        var response = loader.LoadAsync();

        ctx.CacheControl.Add(response.MaxAge);

        // any other logic and return products
    }
}

Finer grain control over the header key and value could be achieved like this:
public class Startup
{
    public void ConfigureServices(IServiceCollection services)
    {
        services
            .AddGraphQL(
                // create schema
                , builder => builder
                    .UseCacheControl("Edge-Control", value => $"!no-store, max-age={value}")); // Without these arguments sensible defaults will be used, for example "Cache-Control" and "max-age={value}"
    }
}

Describe alternatives you've considered
I haven't considered any other approaches but I'm open to suggestions!
Additional context
This implementation is based upon a proof-of-concept that I've been working on which has the following unresolved issues:

I still haven't figured out how to get the values from the UseCacheControl query middleware into the ASP.NET response header (any help here would be really appreciated!)
I currently use "vanilla" .NET attributes for CacheControl and filter in the field middleware on context.Field.ClrType but I'm interested in whether there is a better approach, for example using custom directives.
The CacheControl attribute currently only accepts an integer but there could be other arguments that it should accept for use-cases that I haven't considered.
The current implementation only deals with pure code-first scenarios.

I'm happy to complete the necessary work for this if it's a feature that the community is interested in and welcome any help or support!

We need to detail a bit more all the approaches, not only pure code-first. In essence, I think this will make a great addition.
Apollo also lines out a dynamic API approach where they can interact with the caching within the resolver.
What we need to work out:

schema first => directive
code-first => fluent API
pure code-first => descriptor attribute

Necessarily all those different approaches will translate into the same mechanism.

https://www.apollographql.com/docs/apollo-server/performance/caching/#defining-cache-hints

@rstaib fyi

Also what should we use underneath? the standard Microsoft caching APIs?

Thanks for the feedback!
I thought it would be worth writing the documentation for this first in order to better flesh out the API (and later, get free documentation 🥳)
Rather than clutter up the comments here's a link to the draft: https://demo.codimd.org/csYj21OST6eHQJHIfAvvsg?view
I definitely think we should use MSFT APIs underneath as they're quite comprehensive.

Perhaps we should also elaborate other approaches to see which suites more, and to see what are the pros and cons. I think the approach which apollo is following is not bad by the way.

@tomphilbin documentation looks already good. I have given you write access on the hc repo. So you can create your own branches on this repo. Rafi can help you integrating the documentation into the hc documentation on this repo. Great work so far.

@tomphilbin shall we discuss the on the next working group meeting? I am keen to move forward on this but want to discuss the how a bit.

Yeah sounds good, is it on Tuesday?

great work @tomphilbin ! I wish every feature request would look like this!
This got me thinking today. Together with execution plan this could become even more powerful.
If we annotate fields we could even cache whole queries and offload them to a cache.
The execution engine could work out what the best caching strategy for a query is and generate a cash key based on variables and query. The whole cache could be abstracted with a simple api for storing, fetching and invalidating these keys. It would be easy to create providers for this API. This way we could also integrate things like redis easily.
Maybe we could even invalidate the cash with mutations.

Thanks @PascalSenn! I think your suggestion sounds great and I'm super keen to get started on this. Is there any chance you and/or others would be available for a quick call to discuss an approach in more depth? I'll document any outcomes in this thread.

I'm really interested in adding HTTP caching capabilities, so I would like to give this a jab in the future. I really like the Apollo approach in this regard and would try to port their @cacheControl functionality to Hot Chocolate.

Sure, shall we do a zoom?

I have put it on the backlog for 13.


Fuel Default Connector not returning ChainId

As shown in this forum post the current default fuel connector doesn't return the chainId thus it's undefined for consumers calling the currentNetwork getter.

Closing this as it's been resolved in fuel wallet


Chamfer Distance Loss

Hello,
thanks for sharing the great work..!
When I look at the code for colab surface reconstruction,
it seems that the normal discrepancy mentioned in the paper is not used in the chamfer distance loss.
I'm curious about the reason for this.
Additionally, did you experimentally perform normal estimation using Poisson solvers as mentioned in the appendix?

Hi @create7859,
thanks for your interest in the project!
We use the normal discrepancy loss in the first Colab for the surface reconstruction. It is the cosine_normal_loss in the point cloud optimisation loop.
We did not experiment with the differentiable Poisson solvers in conjunction with our pipeline. However, we do perform normal estimation from our deformed clouds as a post-processing step in the single scan animation example. Here, you can search for the get_normals_pymeshlab function in the code.
Hope this helps,
~Sergey


Exclude optional fields from database schema

We have several optional database columns, which we may not fill in, depending on the configuration. Currently, we have 2 mechanisms to deal with that:

Course grained schema variants
Fill in null.

We should explore more general ways to solve this.

Persistent: Optional field handling
Introduction
We have several optional database columns, which we may not fill in, depending on the
configuration. In the case that we do not fill in the optional column, we usually
fill in null.
There are potentially two problems with this:

Null storage is not free (explained below)
Tables with large number of number columns can be difficult to read

The Effect of NULL in PostgreSQL
In PostgreSQL, table row is physically represented as specified in the manual:

All table rows are structured in the same way. There is a fixed-size header (occupying 23
bytes on most machines), followed by an optional null bitmap, an optional object ID field,
and the user data.

The per-row null bitmap is present only if the row contains least one null value.  If it
is not present, all values are assumed to be non-null. The null bitmap occupies enough
bytes to have one bit per data column.
In effect, there is no storage overhead for tables with 8 columns or less (because 8 bits
equals the padding of one byte). Otherwise, the per-row overhead will most likely use 8
bytes (enough for up to 64 columns).
Solutions
Wait for Persistent v3
There is a proposed redesign that
would solve this. It would generate separate, extensible types for inserting.
Unfortunately, there has been no update for at least 4 years.
Course Grained Schema Variants
Another approach, which is already in
use,
is to define the schema in small pieces, and then stitch them together at runtime.
This works quite well in practice, but it may be a bit heavyhanded for dealing with one or
two columns. Common fields have to be duplicated across each variant. This also means
combining options will be difficult.
Write NULLs
The easiest solution would be to ignore all the problems above and continue to write
nullss. There likely wouldn't be any improvement in records that have ANY nulls.
Consider the case tx_out, where most records don't contain an inline_datum_id (about
2% by my count).
Custom Backend
Yet another option would be extend the PostgreSQL backend, to override its insert
implementation. Then we could redact any column we shouldn't be writing to.


Standardize how our CI creates debug logs

Context
The ways charm repo CI generates logging for debugging is not standardized across all the charm repos.  Some repos use the dump-charm-debug-artifacts action, others define their own logging, etc.  These should be standardized
The cost here is that, for most repos, CI failures do not come with enough logging to debug the issue.  This means engineers burn time having to reproduce CI errors locally, slowing down how quickly they can triage and fix issues.
However this problem is solved, a requirement is that this be a general solution that can be applied to each repo.  No (or at worst, minimal) repo-level configuration should be necessary.  This way we cannot forget to add a particular application, check a particular log, etc, in a specific repo.  We should have a cookiecutter way to put CI logging in any charm repo from a template
What needs to get done

define a consistent way all charms should print debug logs (either with dump-charm-debug-artifacts or something else
deploy this across all charmed kubeflow repos

Definition of Done

all charmed kubeflow repos have a common debug log mechanism

Referencing this commit that adds loggging to the CI https://github.com/canonical/oidc-gatekeeper-operator/pull/134/commits/b7821c0888679f93c4325a077c2c36b3543f702c

Extending on the context suggested by @ca-scribner, we also need a standardized way of providing workload logs when the CI fails. There is the Dump logs action, but it has its limitations.
Dump logs action limitations

It is a huge clustered dump of:

juju debug logs
kubernetes pods descriptions in all namespaces
kubernetes pods logs in all namespaces for all containers
kubernetes deployments description in all namespaces
kubernetes replicaset description in all namespaces
kubernetes nodes information
charmcraft logs

The above contains many logs which are almost never relevant to the failure (for example from the kube-system namespace). This makes it very difficult to parse the output of the action, where the root cause of failure gets lost.
This also means that there's a duplication of the charm container logs, shown as part of the Juju debug logs and the kubernetes logs.
It would be nicer to have the different logs separated from each other in the GH CI i.e. have categories of logs (Kubernetes, Juju, Charmcraft) so we can immediately access what we need.

It often misses the logs we actually need. Let's take kserve-controller as an example. If the integration tests fail, what I would like to see highlighted is:

the logs of kserve-controller pod, for both charm AND workload container
the details of the ConfigMap created by the controller
the details of the InferenceServices resources in the cluster
the logs from the InferenceService Pods created in the tests
the relation data between kserve-controller and resource-dispatcher charm

It can be challenging to optimize the logs of each repo this way, because every charm applies different manifests and creates different CRDs. Potentially, we could use something like this to view all resources of all kinds rather than what we get with kubectl get all, and filter it with only in the namespaces we are interested in (test namespace + user namespace if applicable)


LOW: indexable and automatically assignable event handlers

locale classes and read their schema (constant) and automatically assign handlers to dispatcher, slow option.
faster option - provide static gate to dispatcher initialization and handle creation using custom class or function. but in this case we have global scope problem.

not about internal events but about jobs like stuff


Menus contain a mix of English and another language

Describe the Issue (If applicable)
Menus accessed by pressing START in the main interface contain a mix of English and another language (Portuguese I think).
How can the issue be reproduced? (If applicable)
From the main UI where you select which system you want to go into, press START. Notice that some menu items are in English and some are not. This issue also exists when you press START inside a list of games for a specific system.
What device are you using?
RG353M
What version of ArkOS are you on?
2.0 (01252023)
Anything else you'd like to include that may help to review this issue or feature request?
I updated my device over wifi on 1/25/2023. This issue did not exist before I performed this update. I have not tried to do a fresh install of ArkOS by writing it directly onto my microSD.

I can't replicate this issue on my end.  Did you change languages in the start menu>Advanced Settings>Language?

I did not. Thanks for giving me the path to the language settings. The language was set to BR. I switched it back to EN and that resolved the issue. I tried switching back to BR, and that worked as expected too. There are still a couple items that are in English (I assume that is the fallback when a translation string is missing), but before it was about 50% English 50% Portuguese. I'm going to chalk it up to an anomaly or user error on my part. Thanks for the support!


I have an iisue

Hello I want to use the managementApiSecret but I don't know what to enter in there. -> what do I enter in there. So, I tried enter a normal password but when I did an error occur to me it said that: I did't create a header. Could you explain how I can handle this?... Please

Read: https://v1.prisma.io/docs/1.34/faq/service-secret-vs-management-api-secret-fq01/
Also, if you are using Prisma 1 cloud heroku integration, you can read this FAQ: https://v1.prisma.io/docs/1.34/faq/how-to-reveal-management-api-secret-with-heroku-integration-fq03/


Account for supplemental cell type report in readme

Issue Number
#457
Purpose/Implementation Notes

Here I added a line for the inclusion of the supplemental cell type report in all of the README's for downloads that should include this file. This applies to every download except for the spatial download.
Note that I branched this off feature/community-contributed-data since cell types will be launched at the same time.
Types of changes
What types of changes does your code introduce?

New feature (non-breaking change which adds functionality)

Functional tests
Not applicable
Checklist

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

This looks good to me, I'm going to let @arkid15r approve so he get's a chance to see it before potentially starting on #556


add notes on installing with argocd

This PR add notes on installing with argocd.

@chipzoller I addressed your comments.


Memory Management Blog Post

Write a blog post based on findings from #184

What the issue was
How to fix it (potential financial implications)
Impact of issue

I think this has already been done here: https://developer.ibm.com/technologies/node-js/articles/nodejs-memory-management-in-container-environments


Update add command to include timezones with fuzzy keywords

Currently we need to specify a proper timezone names like "Asia/Kathmandu"  to add it to the config file.

Requirements

make add command work like search command to use fuzzy keywords to look for timezone.
user needs to select or confirm if the timezone results from fuzzy search matches their intention.
once confirmed, add the timezone to the config file.

Added on the latest release:
https://github.com/yankeexe/timezones-cli/releases/tag/0.3.0


I try to build on my localhost..but something wrong

I try to build my own computer(window),,
I launched backend with docker...then I clone the frontend project, walllet,issue,verify, but when i try to make add credential.. there is error.

this 500 error, http://localhost:3000/api/wallet/siopv2/issuer/metadata?issuerId=walt.id

I would like to know how to fix this ..

I get the same error (with Windows as well) when running the full stack with Docker compose

I get the same error with Docker compose

I fixed the docker-compose setup with rev: e8ad8d9d5ec8c834d9f9d580c04632b00e78d81c
@bigrender did you run the backend using the docker-compose up command, or did you run it individually using docker run?
In any case, I managed to make it work in both cases, so if I know more details about how you're running it, I may be able to tell what's wrong.

fixed docker-compose configuration on windows: fe076fef0434dd2e8222c06f53cc7fa850a9e5bb

Thanks
2022년 3월 1일 (화) 오후 7:39, Severin Stampler @.***>님이 작성:

Closed #13 https://github.com/walt-id/waltid-wallet-backend/issues/13.
—
Reply to this email directly, view it on GitHub
https://github.com/walt-id/waltid-wallet-backend/issues/13#event-6161076600,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADLRFO3TNMTLSXBJLYZJZLU5XXWTANCNFSM5NVHNMLA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


correct ios version

Uses correct version and short version for ios builds
ping @laurafitzgerald

@odra This also needs to be cherrypicked to the FH-v4.5 branch.


Move Section should render new schedule

while moving -> rather than trying to generate the schedule on the client side, do it server side

Still need to fix snapping, but most edgecases work. Try it out!
https://github.com/kryali/classwhole/commit/9eece1b80b53261c292108863fd6a0f1f2371be7


Resolved Bug 885 : Design System > Components > Payment Tenure > Default > After submitting field not clear and data is not display in action

Description

Resolve the issues on Components Payment Tenure:

** Payment Tenuren**

Screenshots :

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

comments resolved


Remove LoggerInterface from constructor

Since it is optional, we could just remove it from the constructor and let the user set it by a setter (LoggerAwareInterface).

:+1:


Add back check for access list factory.

Fixes # .
Changes proposed in this PR:
From this discussion https://github.com/oceanprotocol/ocean.js/pull/1868#discussion_r1824334821, I reopened the topic, that assignement does not check if  the accessListFactory exists and throws this error for not EVM chains (for example in barge):
Error publishing asset: Cannot read properties of undefined (reading 'accessListFactory')

this is definitely NOT OK
This:
if ('accessListFactory' in contractAddressesConfig) {
      config.accessListFactory = contractAddressesConfig.accessListFactory
}

Will break if contractAddressesConfig is null or undefined
we cannot search for a property in something that does not exist, fyi:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Errors/in_operator_no_object
You can simply, and safely, do as i suggested above
if(contractAddressesConfig) {
  config.accessListFactory = contractAddressesConfig.accessListFactory
}

you really like to complicate @mariacarmina , not sure why :-)
This:
if (contractAddressesConfig && 'accessListFactory' in contractAddressesConfig) {
      config.accessListFactory = contractAddressesConfig.accessListFactory
 }

Has exactly the same effect as my suggested change (bellow), except that you made you more complicate (2 checks instead of 1). Can't really understand why but anyway
if(contractAddressesConfig) {
  config.accessListFactory = contractAddressesConfig.accessListFactory
}

I do not like to complicate @paulo-ocean :) but as I mentioned previously, the assignment is related to 'accessListFactory' -> so it is good practice to check as well if 'accessListFactory' is present in contractAddressesConfig, because not all the chains have accessListFactory key, because they are not confidential EVM. For me your check is incomplete, we surely need to check if contractAddressesConfig exists, but also if accessListFactory is present because that's what we want to assign. Please check https://github.com/oceanprotocol/contracts/blob/main/addresses/address.json and search 'AccessListFactory' and you'll see that only for oasis that key exists.
I need to modify the key name from contractAddressesConfig, it is not 'accessListFactory', it's 'AccessListFactory'.

you really like to complicate @mariacarmina , not sure why :-)
This:
if (contractAddressesConfig && 'accessListFactory' in contractAddressesConfig) {
      config.accessListFactory = contractAddressesConfig.accessListFactory
 }

Has exactly the same effect as my suggested change (bellow), except that you made you more complicate (2 checks instead of 1). Can't really understand why but anyway
if(contractAddressesConfig) {
  config.accessListFactory = contractAddressesConfig.accessListFactory
}

I do not like to complicate @paulo-ocean :) but as I mentioned previously, the assignment is related to 'accessListFactory' -> so it is good practice to check as well if 'accessListFactory' is present in contractAddressesConfig, because not all the chains have accessListFactory key, because they are not confidential EVM. For me your check is incomplete, we surely need to check if contractAddressesConfig exists, but also if accessListFactory is present because that's what we want to assign. Please check https://github.com/oceanprotocol/contracts/blob/main/addresses/address.json and search 'AccessListFactory' and you'll see that only for oasis that key exists.

its exactly the same @mariacarmina ... but keep your ideas, i know nothing, and should not be trying to help


Example Crashes on iOS

I just cloned this repo and without making any changes built it for iOS and ran it. It crashes on the StartScanningAsync method call.
I am running on macOS latest on a 2018 macbook pro with Visual Studio 2019 release version.
I checked out the 5.10 branch so that the nuget packages could be installed. the 5.11 doesn't work because nuget only has 5.10 and earlier.
How do I fix this crashing?

Hi @tpitman
We are looking into the issue currently and will get back to you about this crash as soon as we know more.
The issue you had with the 5.11 branch is because we accidentally already bumped the samples to 5.11.3, we undid this and it should work fine again now.

Thank you for submitting this report. We are investigating the issue.

@tpitman, could you help us out by pointing out the sample that did crash for you?


Anonymous RAT hash (EXPOSUREAPP-7186)

Hash for anonymous RAT codes are also checked.
Old anonymous RAT codes do not work anymore, please use the codes below:
Valid hash

Invalid hash

Scanning Invalid hash QrCode  shows pending screen , should not it fail ?


Localisation used as per Xcode 13.1 version.

Added [ENG] and [RU] localisation.
All strings are automatically added to the translation files when exported for localisation.

How it looks without changes in string constants:


add document for chainer.config.lazy_grad_sum

Documentation for #4136.

Please fix this error:

chainer/docs/source/reference/configuration.rst:173: ERROR: Malformed table.

Thanks, fixed.

Maybe you forgot to push the change?

Oh, sorry. Pushed.

jenkins, test this please

LGTM!


Request to update flash_attention in deepspeed inference

triton flash_attention used by deepspeed inference is compatible with 6 month old pre release version of triton( 2.0.0.dev20221202 ) .
triton dot , trans operators are rewritten in Complete rewrite of the backend from scratch
flash_attention is optimized in the Improved flash attention forward pass performance
Could you kindly update flash_attention with the current triton release.

I get this error when running deepspeed inference , with triton release version (2.0.0)
TypeError: dot() got an unexpected keyword argument 'trans_b'
With qk += tl.dot(q, k, trans_b=True)  to tl.dot(q,tl.trans(k))  , deepspeed inference hangs

@loadams  I am running stable diffusion with deepspeed inference.

Related #4008.
Am I correct in assuming that the sparse attention and stable diffusion are mutually exclusive when building DS right now? Is there somewhere in the documentation I can add an explicit mention of this?
Any plans on bumping triton dependency of sparse attention to 2.x ?


Back Button on Recycle Request Page Causes Blank Screen #68

Fixed back button working on navigation bar on Recycle Request Page:
Details of changes:-
On the Request Pickup Button instead of simply Navigator.Push( )code contains Navigator.Pushreplacement( ) which is causing the removal of previous screen in stack resulting in white screen..
@sanika391  I have also suggested some changes in color theme of navigation bar on same screen do also consider it..

Thank you, @akshayjadhav2002, for addressing the issue with the back button on the Recycle Request Page. The fix you've implemented using Navigator.PushReplacement() instead of Navigator.Push() to resolve the blank screen issue looks good. I also appreciate your suggestions regarding the color theme of the navigation bar. We will review the changes and consider them accordingly.
Great work! 😊
Best,
Sanika Chavan
[https://www.linkedin.com/in/sanika-chavan-52457b236]


What do we want done before release?

What do we need to add/fix before an official release?
i would like to get a jenkins-like setup working. XCorrosionX was working on a script on a server that downloads steam rust updates, gets latest from git, compiles, patches and outputs the Managed folder to a zip file, so GSPs and users can update very easily.

Pluton has been released


Sending attachments fails

Every time I add an attachment to outgoing mails K-9 doesn't send it. I rechecked the SMTPs, but regardless of what account I use - mails without attachments get sent, mails with attachmentd won't.

diesocke wrote:

Every time I add an attachment to outgoing mails K-9 doesn't send it. I rechecked the SMTPs, but regardless of what account I use - mails without attachments get sent, mails with attachmentd won't.

Could not reproduce it.

@diesocke: Please fill out the issue template and record a debug log.

Version number ist 5.6000. Will post the debug log, once I find time to connect my phone to a windows PC.

Please create a new issue if this is still a problem with K-9 Mail 5.7xx.


Run tests every night

coverage: 68.571%. remained the same
when pulling 648a15578bd216dea8f81b945a128e0c42dbb22d on eiriksm-patch-3
into 0c3e82488c3fba15f28539f7e3342d1e6ca137ea on main.


server/camlistored: keep track of the Let's Encrypt issue

golang.org/x/crypto/acme/autocert is broken at the moment, because of https://community.letsencrypt.org/t/2018-01-09-issue-with-tls-sni-01-and-shared-hosting-infrastructure/49996
This affects camlistored, as it relies on golang.org/x/crypto/acme/autocert to obtain an HTTPS certificate.

https://camlistore-review.googlesource.com/#/c/camlistore/+/14286

camnetdns restarted with the http-01 Let's Encrypt challenge


Cst order

It seems like this can cause a severe performance regression (when enabled) in complex grammars use cases as much as halves the performance...
I am uncertain this feature is worth the potential performance regression because:

Most of the time the ordering information is not needed.
When it is needed it can sometimes be resolved with refactoring/labeling
When it is needed the upcoming feature for CST location information would make it very easy to compute the order during the CST Visitor.


token usage tracking (Issue #268)

why
being able to track input/output/total tokens when using stagehand can be helpful for controlling costs, picking which model to use, etc.
See Issue #268 (https://github.com/browserbase/stagehand/issues/268)
what changed

Added LLMUsageEntry interface to track tokens per function call
Implemented usage recording in OpenAI and Anthropic clients
Added recordUsage() and getUsage() methods to Stagehand class
Added getTotalTokensUsed() for aggregating total token usage

Works for act(), observe(), and extract()
12 files edited, added tests and and example (hackernews.ts) demonstrating how to log tokens used. Please see diff
test plan
i wrote a few test cases and it seems to work but someone on the browserbase team should definitely test as well before merging!

whoa!! thanks @andrewgcodes! i'll take a more detailed look at this in a bit, but thank you so much for taking the time to do this!

Hey Anirudh, thanks for the quick review and kind words! I'll ping you on
the Slack

this is amazing! looking for the discussion in Slack. Also I see some Devin in here @andrewgcodes  :D


std::optional::value unavailable on earlier macOS versions

@octobanana That problem is fixed, but the earlier MacOS versions face another problem:
error: call to unavailable member function 'value': introduced in macOS 10.14
A possible workaround is to remove the optional regex matches from string.cc and string.hh. Something like that:
-std::optional<std::vector<std::string>> match(std::string const& str, std::regex rx);
+std::vector<std::string> match(std::string const& str, std::regex rx);

The logic of the optional regex matches in peacock.cc should be refactored then, as there will be no more match_opt's.
I have tried it locally just now, but the result seems a bit strange. There were no errors during the compilation and the clock runs just fine, the digital numbers are changing, but the binary ones are all enabled. Maybe something else went wrong, or my changes were incorrect.
Originally posted by @Dluzhnevsky in https://github.com/octobanana/peaclock/issues/1#issuecomment-453676936

I am using latest available clang 11.0.3 and running on Catalina 15.5 with all latest updates to comand line tools and I am getting the same error:
/third_party/stduuid/include/stduuid/uuid.h:900:47: error: 'value' is unavailable: introduced in
      macOS 10.14
            memcpy(&data[10], &device_address.value()[0], 6);
                                              ^
/Library/Developer/CommandLineTools/usr/bin/../include/c++/v1/optional:952:27: note: 'value' has been explicitly marked
      unavailable here
    constexpr value_type& value() &

I am building with base SDK 10.15 on latest Catalina with newest clang, however I have set Target SDK to 10.10, because my application needs to run on older OSes.
Should I create a new issue?


Enhance the parameters for plugin named "Naginator"

@daspilker hi, can you help me to review this PR?
Thank you very much!!!

These options are already supported by Dynamic DSL. I am no longer merger pull requests for features that are supported by Dynamic DSL.
job('example') {
    publishers {
        naginatorPublisher {
            regexpForRerun('test')
            rerunIfUnstable(true)
            checkRegexp(true)
            maxSchedule(12)
            rerunMatrixPart(false)
            delay {
                fixedDelay {
                    delay(400)
                }
            }
        }
    }
}

@daspilker  How to get the information about the "Naginator" plugin has already supported the Dynamic DSL?


Column Headers Need More Space

Hi again,
So, we noticed another thing about the heatmap links (awesome heatmaps though!) generated and returned by the API. There isn't a lot of room for column headers, although a context blurb does appear nicely when the cursor hovers over it. But if the header is over several characters long, it gets truncated. Like so:

The usual header is something like "3 Mouse Lung Cancer - specimen grey mouse 33".
Is there perhaps an argument that could be included to allow for space above the table during the Post?
Cheers,
Grant

Hi Grant,
The column label size can not currently be increased through the web application. The column label size can be increased using the col_label_scale argument if you are building your own heatmaps using Clustergrammer-JS. For information on adding the visualization to your page please see the docs.
However, adding this as an argument to the post request is something that we can do in the future.
Let us know if that helps and thanks for the feedback.
Best,
Nick

Grant,
I was able to make mine better by setting the 2 following parameters in the initial args dictionary.
1). 'col_label_scale' : 1.4   //This seems to increase the available height allowed for the labels
2). 'max_allow_fs' : 14  // This sets the maximum font size (in px) that at least the column and row labels can be.  The default is 20 which can be too big when using long names.
Hope this helps.  I also wonder if there is something that could be done (feature request) where the column label will rap to two lines if enough height is available.

Hi @JoshuaFortriede, we are planning on implementing a user resizable column/row label area (e.g. dragging). This will be implemented for Clustergrammer-GL first https://github.com/ismms-himc/clustergrammer-gl.


Show error message when loading scores/vp fails

When getResults fails or scores_state is invalid:

(Votes won't show either then.)
Live example: https://bafybeiasuxsq7l5m5spiitzakx6z5qwverwu5im5gpovaiiimc5o5kqto4.on.fleek.co/#/brokenclub.eth/proposal/0xe5865374158054a4cc59021a4788e9d506f3b62fea3458fcdb32b81b7b4f4c79
But this triggered by scores_state is invalid. Here is how it looks when getResults really fails:

Maybe this can be improved a bit but better then nothing.
About the Block component: Seems this icon prop isn't used anywhere (only the counter), so I made the color class another property to make it red for this case.
The changes in snapshot.ts are just removing the try-catch from there. I think errors should be caught as late as possible/reasonable. Otherwise you'll have to go up and up and up and up to debug/handle errors. And if a dependency (like snapshot.js) obfuscates it's errors, debugging becomes painful.
UPDATE Added error message for voting power too:

The help button in the results box is now only shown for admins of the space.
Changed messages a bit, but they are the same for everyone
Would like to have a final approval from you @metrox-eth for the below (anything beyond that is not worth the effort now imho, we're going from 0 to 100 in this anyway :D let's better think a bout where else a little message can improve things)

https://user-images.githubusercontent.com/6792578/153395151-2dc2044f-46a7-4641-8382-134f564a4b7b.mp4


[BUG] Crashes rainmeter since one of last MS update

Rainmeter version
OS
DriveList version

<IP_ADDRESS>46
Win 10 x64 pro (french) 14393
<IP_ADDRESS> (2014.06.28)

Steps to reproduce
Load a skin that use the plugin
Results
Rainmeter crashes instantly
Causes
It happens just after a Microsoft Update that involved a PC reboot.
One of the plausible responsable update:

KB4023834: Servicing Stack Update
KB3186568 .NET Framework 4.7

I hope you can reproduce and fix it. It is a valuable plugin that I've used for years.

It seems it is fixed now with the KB4022715

@freMea thank you for the thorough and well-organized bug report! I'm glad a later update seems to have fixed the problem for you. Do let me know if it becomes an issue again. I haven't had much Rainmeter time recently, but I will try to take a look if needed.


License should be documented also outside the Cargo.toml files.

The license of this projects in only described in Rust's Cargo.toml files. It should be noted in the root of the projects.

Thank you for pointing this out! Addressed in https://github.com/rust-shogi-crates/shogi_official_kifu/pull/9.


light version of HttpClient

i removed HttpClient from 0.12, but i think we need some lightweight version of it,
something like
    client = aiohttp.HttpClient('http://...', keep_alive=True)
    yield from client.request('GET', '/path1')
    yield from client.request('POST', '/path2')

@asvetlov @kxepal @popravich ?

Fixed by introducing ClientSession #328


[Android] Fix spacing in ChoiceSets with wrappjng

Related Issue
Fixes #3554
Description
When having a choice set with wrap set to false, the options appear one after the other with no spacing between them, while wrapping options appear with the previously mentioned spacing
How Verified
The test cases were used in the sample application and it yielded this result. As can be seen the spacing is kept while the text is truncated as expected.

Microsoft Reviewers: Open in CodeFlow

@Almedina-ms this needs to be made against release/1.2 bits to ensure it lands in the 1.2.5 branch - can you send out a PR for that as well? (either a CP after you commit this to master OR manually editing the 1.2 version of the code - whichever is easier). Thanks


quickstart cannot running three containers

Describe the bug
No errors but three containers cannot running and keep creating

Desktop (please complete the following information):
My datahub version
DataHub CLI version: <IP_ADDRESS>
Python version: 3.8.12 (default, Jun 26 2023, 15:09:04)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-44)]

@Peryy I believe this is a duplicate of https://github.com/datahub-project/datahub/issues/8133
Could you try with a slightly more recent cli version, and don't specify the --docker-compose file manually (we'll automatically download it)

Have the same trouble here. I cannot up docker broker.


feat(savers): don't sendMax utxo sendMaxes

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #3688

PR #3689

PR #3690   👈

This comment was auto-generated by Graphite.


Catch src import issues on pre-commit hook

Expected Behavior
Sometimes the IDE can trick you and auto-import a @celo dependency from the src folder instead of the lib folder.
For example, in ContractKit we had this in a file:
import { isValidAddress } from '@celo/utils/src/address'

Nothing in CI complains about this, but it continues to reference src after it's published to NPM which causes a runtime error as the src folder isn't published to NPM. See #7073
This can cause sizeable disruption when it comes to the NPM publish cycle. But it's easy to catch earlier, potentially in a pre-commit hook by looking at .ts files for<EMAIL_ADDRESS>this regexp, or something of that nature.
Current Behavior
Everything works until you try to use the package from NPM :(

Before publishing to contractkit/cli to NPM, we should check they do not have src dependencies


Fix #3704 - Remove values from RelationalCommand construction

Allow providing parameter values as arguments or cached property value
Cleans up RelationalCommand:

When the RelationalCommand is built, no parameter values are referenced (query scenario)
Values can be added to the command after building (other scenarios)
Cached values will be used by default
If values are passed in, they will override the cached values (all or none)

These changes are an incremental step for #3115. Changes for that issue will include command and parameter building API consistency and extensibility cleanup.
@divega @maumar @anpete

Closing this PR after talking with @divega
Too much of what changes here depends on / relates to what happens for #3115 and it would be better to review all of the changes together.


Backup folder issue

When I created a backup, the file is in the "October-CMS" folder instead of "panakour-backup".
I use October build 430.

Did you change the name inside the config file?

I use the default settings and the latest version of plugin.
The plugin use the app name (https://github.com/octobercms/october/blob/master/config/app.php#L31) as a folder name. I do not understand why this is.

what is your environment?

Linux server, PHP 7.0.25

in a production server or local?

I run the website in a production server.

Are the backups created without any problem?

Yes, the backup zip files are fine.

Unfortunately I can't reproduce this issue, I have tried with various system local and remote and it works as expected.

I see. I didn't find the problem.

Closing as it has been over a year since any activity on this occurred


fix mysql debug time

调试模式时间格式改为毫秒,原来是微秒.

Codecov Report

Merging #463 into master will increase coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #463      +/-   ##
==========================================
+ Coverage   70.24%   70.29%   +0.04%
==========================================
  Files         279      279
  Lines       24487    24487
==========================================
+ Hits        17201    17212      +11
+ Misses       6392     6382      -10
+ Partials      894      893       -1

Impacted Files
Coverage Δ

database/gdb/gdb_base.go
70.44% <0%> (ø)
:arrow_up:

container/gtree/gtree_redblacktree.go
89.38% <0%> (-0.36%)
:arrow_down:

container/gtree/gtree_btree.go
90.01% <0%> (+0.59%)
:arrow_up:

net/gtcp/gtcp_conn.go
33.11% <0%> (+1.32%)
:arrow_up:

net/gtcp/gtcp_pool.go
40.78% <0%> (+10.52%)
:arrow_up:

Continue to review full report at Codecov.

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


Make sure pointer local vars have AliasedPointer decoration.

Fixes #4111.

That's right. We can have pointer types take additional attributes like alignment and alias/restrict and coherent const etc.


Installation instructions point to the wrong package

README currently instruct to install wagtailmodelchooser when it should say wagtail-modelchooser.

Thanks for pointing this out!


Output Size

Hi Raj,
First of all, thank you so much for sharing your code, it is great!
Do you know what modifications are needed so that the output image has the same height and width as the input image?
I tried changing the crop size in eval.py but it caused some reshaping issues in the deconvolution layers. For instance, when trying to change the crop size from 480x480 to 500x375 I get this error:

Traceback (most recent call last):
File "U:\Python36\site-packages\tensorflow\python\framework\ops.py", line 1551, in _create_c_op
c_op = c_api.TF_FinishOperation(op_desc)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Cannot reshape a tensor with 758016 elements to shape [786432,1] (786432 elements) for 'unpool_3/Reshape_2' (op: 'Reshape') with input shapes: [1,63,47,256], [2] and with input tensors computed as partial shapes: input[1] = [786432,1].

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "U:\ManWin\My Documents\Dissertation\TensorFlow 2_0 Migration CNN\eval.py", line 52, in
pred = build_model(norm_image)
File "U:\ManWin\My Documents\Dissertation\TensorFlow 2_0 Migration CNN\model_contour.py", line 61, in build_model
unpool_3 = unpool_with_argmax(bn_deconv4, ind=pool3_indices, name="unpool_3")
File "U:\ManWin\My Documents\Dissertation\TensorFlow 2_0 Migration CNN\ops.py", line 94, in unpool_with_argmax
b = tf.reshape(b, [flat_input_size, 1])
File "U:\Python36\site-packages\tensorflow\python\ops\gen_array_ops.py", line 9093, in reshape
"Reshape", tensor=tensor, shape=shape, name=name)
File "U:\Python36\site-packages\tensorflow\python\framework\op_def_library.py", line 788, in _apply_op_helper
op_def=op_def)
File "U:\Python36\site-packages\tensorflow\python\util\deprecation.py", line 507, in new_func
return func(*args, **kwargs)
File "U:\Python36\site-packages\tensorflow\python\framework\ops.py", line 3296, in create_op
op_def=op_def)
File "U:\Python36\site-packages\tensorflow\python\framework\ops.py", line 1714, in init
control_input_ops)
File "U:\Python36\site-packages\tensorflow\python\framework\ops.py", line 1554, in _create_c_op
raise ValueError(str(e))
ValueError: Cannot reshape a tensor with 758016 elements to shape [786432,1] (786432 elements) for 'unpool_3/Reshape_2' (op: 'Reshape') with input shapes: [1,63,47,256], [2] and with input tensors computed as partial shapes: input[1] = [786432,1].

Thank you in advance!

I think the crop size has to be dividable by 32 otherwise the output shape of the skip connections will not fit with the shape of there target layers.


[BUG] delitem triggers -Wunused-variable

Description
Defining a cdef class with a __delitem__ method triggers -Wunused-variable.
To Reproduce
delitem_test.pyx:
cdef class K:
	cdef list a
	def __cinit__(self, list a):                   self.a = list(a)
	def __getitem__(self, Py_ssize_t i):           return self.a[i]
	def __setitem__(self, Py_ssize_t i, object v): self.a[i] = v
	def __delitem__(self, Py_ssize_t i):           self.a[i] = None

Compilation:
$ cython -3 delitem_test.pyx
$ clang -c delitem_test.c -Wunused-variable -I$(python3.7 -c'import sysconfig;print(sysconfig.get_path("include"))')
delitem_test.c:2287:18: warning: unused variable 'o_type_name' [-Wunused-variable]
  __Pyx_TypeName o_type_name;
                 ^
1 warning generated.
$ gcc-10 -c delitem_test.c -Wunused-variable -I$(python3.7 -c'import sysconfig;print(sysconfig.get_path("include"))')
delitem_test.c: In function '__pyx_mp_ass_subscript_12delitem_test_K':
delitem_test.c:2287:18: warning: unused variable 'o_type_name' [-Wunused-variable]
 2287 |   __Pyx_TypeName o_type_name;
      |                  ^~~~~~~~~~~

Expected behavior
The problematic code in the generated delitem_test.c is this, starting on line 2286:
static int __pyx_mp_ass_subscript_12delitem_test_K(PyObject *o, PyObject *i, PyObject *v) {
  __Pyx_TypeName o_type_name;
  if (v) {
    return __pyx_pw_12delitem_test_1K_5__setitem__(o, i, v);
  }
  else {
    return __pyx_pw_12delitem_test_1K_7__delitem__(o, i);
  }
}

Simply not generating the function's first line declaring o_type_name would resolve the warning.
Environment

OS: macOS 10.15
Python: CPython 3.7.10
Cython: 3.0a6
clang: 12.0.0
GCC: 10.2.0

It looks like this is fixed somewhere between 3.0a6 and the current master. But I'm not sure exactly where.

(For what it's worth it was https://github.com/cython/cython/commit/206adb437d78333f2876da6ffd0dca7258b5ab11#diff-1c3b7395c96b4ecec571dc9f3fb5f1689faa4be6006abe7c3f01145f1010cf9c that fixed it)


add support for Certificate Transparency Monitoring API

Description
adds support for Certificate Transparency Monitoring to SDK
cloudflare terraform provider issue
cloudflare community post
cloudflare docs
Has your change been tested?
sorta, open to suggestions on how to improve
Screenshots (if appropriate):
Types of changes
What sort of change does your code introduce/modify?

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[ ] This change is using publicly documented in cloudflare/api-schemas
and relies on stable APIs.

thanks for the PR here! however, as this doesn't have API schemas defined, we won't be able to merge this since the public API contract hasn't been established. i'll label this for now and raise it with the internal service team to look into.

great thanks!

closing this out since v2 will handle this automatically once the schemas are added.


Cannot run Freesurfer Regression

Describe the bug
When I tried to run Freesurfer Regression, first the pipeline would not start. I clicked on it three times. I checked, and Docker was on but not running anything. Then I closed and reopened COINSTAC and the three pipelines started. However, after 10 minutes, they were still in the first iteration of the local side. They should take a very short amount of time to run. I stopped all three pipelines and restarted COINSTAC again. The three pipelines started up again. I stopped them and started a new one. I saw the same behavior with it. I stopped all four pipelines and restarted COINSTAC. When I reopened the application, I saw that four pipelines open, none of which were making any progress.
To Reproduce
Steps to reproduce the behavior:

Create Freesurfer Regression pipeline with 5 ROIs for a single user.
Map data.
Run pipeline.

Expected behavior
I expect the pipeline to complete in a few minutes.
Desktop (please complete the following information):

OS: OSX 10.14.6
Version 4.7.5

Additional context
Did not see any errors in the logs.

Pipeline run bugs that should be fixed via the listener rewrite, closing


How to import data from csv or json?

Hello,
what is best way to import data? I want to upload data to existing db and automatically create key-values pairs from my json file. Json file contain about 200 objects and each object contain about 30 properties. Is it possible to upload data straight from csv? Thank you

Hi! There's a demo project i've done which loads a csv and queries it. It's here: https://github.com/techfort/csv-loader let me know if it's of any help, it should be simple enough.

Thank you for your answer but in my case it's not suitable because it require nodejs. I decide to use library papa parse http://papaparse.com/. Here my piece of code
document.getElementById("upload").addEventListener('click', function () {
            parseMe(document.getElementById("file").files[0]);
        });

        function parseMe(file) {
            Papa.parse(file, {
                header: true,
                complete: function (results) {
                    renderDataset(results.data);
                }
            });
        }

        function renderDataset(dataset) {
            dataset.forEach(function (el) {
               users.insert(el);
            });
            db.saveDatabase();
        }

cool - thank you. Closing this so.


Public use of the status page

Hello, I have a few questions about the status page.
Are there any plans to disable the status page?
Is it acceptable to regularly pull the status page to monitor for new streams?
If yes, is there preferred frequency?
Would the project be open to a PR that would add a  isActive  key that would indicate if the stream key is in use?
I made a simple python script that checks the status page every ~5 minutes to look for new streams, and it has been quite entertaining. I've seen many interesting game streams, security cams and many people testing they are streaming setups, it's quite random and reminds me of the early days of Twitch.
I would like to create a simple script that will send a discord notification when a new stream is active, without being a burden on the system.
Thanks!
Chase

Go for it! Build some cool hacks :)
https://b.siobud.com is a public resource. I plan to run it as long as it doesn't cause problems!
Would you be interested in running your script in the 'Real-time Broadcasting' discord? Would enjoy watching it myself :)


Plugin update badger

Fixes #1543.

Thanks for the feedback - I added a bunch of comments that hopefully clarify things, but of course please let me know if it's still cryptic.

@kate-goldenring for some reason I can't reply to the comment on lines 44-46

I think this should reuse default_data_dir from data_dir.rs. Otherwise it will also mess up any colocation of data for package managers

I chose cache_dir because this is "does not matter if the user blows it away" material - it's purely advisory.  But yeah it did feel a bit odd having it in a different place from the plugins.  Open to opinion.

@kate-goldenring for some reason I can't reply to the comment on lines 44-46

I think this should reuse default_data_dir from data_dir.rs. Otherwise it will also mess up any colocation of data for package managers

I chose cache_dir because this is "does not matter if the user blows it away" material - it's purely advisory. But yeah it did feel a bit odd having it in a different place from the plugins. Open to opinion.

@itowlson that seems like a reasonable distinction to me. I am just thinking of when people uninstall Spin we have multiple directories that need cleanup but registry uses it anyways so SGTM

@lann added a comment on the error swallowing, and changed to terminal::einfo!.  I think that covers all your remaining comments - do you want to re-review, or are you happy for me to merge on that basis?

Merge away!


Update README.md

.

@wxfil
p3 is a const pointer, cannot assign again. check following:
int main()
{
    int i = 0;
    const int ic = 2;
    const int *const p3 = &i;
    p3 =  // error. Cannot assign to readonly type `int const * const`
}

Agree with @pezy

Remember to close this PR, when no more discussion is needed.
@pezy  @wxfil


Sometimes, terminate a VM leaving garbage resources

After terminating a VM, sometimes used public ip or ephemeral disk are left in the CSP.
especially Azure
reported by @seokho-son

https://cloud-barista.slack.com/archives/CLFCLNFTJ/p1628776964014600

연관 이슈: https://github.com/cloud-barista/cb-spider/issues/232

● Fixed in https://github.com/cloud-barista/cb-spider/commit/c78120f1914054af95acb74a13e01a16373b0774, v0.5.11


Instructions for mac only

Recommend you include CTRL+ number as alternative for Linux and Windows users?

I've not personally tested it on Windows or Linux.. what is the current combination of keys used?
I can setup an option for either mod key (current) or control. Won't be an issue.

Implemented a setting as well as key press tester in v1.1.4.


Trying redux

Not even sure if this is redux ...
Still going to work on it, but I want to change the format a little. Will probably start over.

Very nice commit messages!


refactor into named functions

refactor into two named functions:
function handler(request, response) { ... }
function main() { ... }
and then a one-line invocation:
main();

OK


Call member function is slow than property.

Call member function is slow than property.
with sprite:GetPosition, sprite:SetPosition get fps 54
with sprite.position get fps 125

I’m sorry. Can you define KAGUYA_RETURN_VALUE_OPTIMIZATION 0 or change in kaguya/config.hpp ?
If faster change. i remove it.

#define KAGUYA_RETURN_VALUE_OPTIMIZATION  0 will faster than 1.
but the function call remain slow than property call.

OK. When I get home,will check it.

Bug. This function not push v to Lua.
namespace nativefunction
	{
#if KAGUYA_RETURN_VALUE_OPTIMIZATION
		template<typename T>
		struct is_return_reference_optimize :traits::integral_constant<bool,
			is_usertype<typename traits::decay<T>::type>::value	&&
			(traits::is_pointer<T>::value	||
			traits::is_lvalue_reference<T>::value)>{};

		template<typename T>
		inline int optimized_push_return_value(lua_State* state,T* v)
		{
			int top = lua_gettop(state);
			for (int i = 1; i <= top; ++i)
			{
				if (lua_type_traits<T*>::get(state, i))
				{
					lua_pushvalue(state, i);
					return 1;
				}
			}
			return util::push_args(state, v);
		};

Not bug (but slow). If argument and return value(v) is same, push stack valuelua_pushvalue(state, i);  , It is no newuserdata(not heap allocation).

I known what your mean. but only check argument type not argument value. eg.
Node* Node::CreateChild(const char* name);

will not push the return value to stack.

Omg! you are right.

Forgot ==v ,  but already remove that.

Thanks. By the way, Where are you?

at home now.

USA?

No. it's seacret.

Sorry. Kaguya is cool.  If you have time, I want to invite you work on Urho3D.

Thanks.


fix: add semicolon to fix highlight bug

Fixed css/scss syntax by adding some missing ; for highlight background to show
Resolved https://github.com/selfire1/obsidian-you-theme/issues/16

Before
After

Note: I really enjoyed this Obsidian theme! I rely on search/highlight a lot so wanted to make it better for other users too.

Merged! Thank you for your contribution and kind words :)