Discord bot that posts messages from Meower

Originally posted by Melt2002 December 18, 2021
It would be cool to see a bot that pulls messages from Meower and posts them in Discord.

What Dsc bot Wrapper?
If its nextcord id be able to help

What Dsc bot Wrapper?
If its nextcord id be able to help

I would love to use Nextcord for it but it'll most liekly be Discord.js for ease of use when working with WebSockets on Node.js.
Also this thing will work a lot differently with New Meower, the Discord bot will most likely work as the chat bridge between Discord and Meower.

What Dsc bot Wrapper?
If its nextcord id be able to help

I would love to use Nextcord for it but it'll most liekly be Discord.js for ease of use when working with WebSockets on Node.js.
Also this thing will work a lot differently with New Meower, the Discord bot will most likely work as the chat bridge between Discord and Meower.

I dont know js sry
I have a small repo if  you would like to look at it

We're also writing pretty much all of New Meower in JS, so it would make sense for this bot to also be in JS.

I have implemented this into the Meower Discord, it's only available to moderators at this time.

welp I cant do nothing cuz i dont know js well

My code could also be used as an idea for the final implementation :d

My code could also be used as an idea for the final implementation :d
Also The idea i had for the bot that it was a 2 way system.
  That Means Meower -> discord
  discord -> meower

using a custom OAuth that connected your discord account to your meower account would work somewhat like this
you try to talk in meower chat channel -> bot notices a message in the channel (splits here) -> if not authed -> send link to discord auth page -> meower server gets auth request -> you sign into meower -> mower sends your connected details to a database somewhere as proof of auth

I like the idea, only problem is there is no Meower OAuth solution that allows for sending posts at this time, I will have to talk to MikeDEV about the possiblity of doing that.

I know how do do it with python
On Mon, Mar 28, 2022, 5:50 PM tnix100 @.***> wrote:

My code could also be used as an idea for the final implementation :d
Also The idea i had for the bot that it was a 2 way system.
That Means Meower -> discord
discord -> meower
using a custom OAuth that connected your discord account to your meower
account would work somewhat like this
you try to talk in meower chat channel -> bot notices a message in the
channel (splits here) -> if not authed -> send link to discord auth page ->
meower server gets auth request -> you sign into meower -> mower sends your
connected details to a database somewhere as proof of auth
I like the idea, only problem is there is no Meower OAuth solution that
allows for sending posts at this time, I will have to talk to MikeDEV about
the possiblity of doing that.
—
Reply to this email directly, view it on GitHub
https://github.com/meower-media-co/Meower-Server/issues/19#issuecomment-1081228079,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APLEXLJDH4DIQAOYL4RQGSTVCIZTRANCNFSM5NH6T7YA
.
You are receiving this because you commented.Message ID:
@.***>

I know how do do it with python
…
On Mon, Mar 28, 2022, 5:50 PM tnix100 @.> wrote: My code could also be used as an idea for the final implementation :d Also The idea i had for the bot that it was a 2 way system. That Means Meower -> discord discord -> meower using a custom OAuth that connected your discord account to your meower account would work somewhat like this you try to talk in meower chat channel -> bot notices a message in the channel (splits here) -> if not authed -> send link to discord auth page -> meower server gets auth request -> you sign into meower -> mower sends your connected details to a database somewhere as proof of auth I like the idea, only problem is there is no Meower OAuth solution that allows for sending posts at this time, I will have to talk to MikeDEV about the possiblity of doing that. — Reply to this email directly, view it on GitHub <#19 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/APLEXLJDH4DIQAOYL4RQGSTVCIZTRANCNFSM5NH6T7YA . You are receiving this because you commented.Message ID: @.>

I do too, but it'll be insecure, once an OAuth system that supports making posts on behalf of a user or webhooks becomes available then we can do it.

I'm gonna have a crack at it. I'm gonna try to finish my Meower OAuth system and make some sort of bridging bot, Idk how well it'll go but let's see!

This has mostly been done now, you can try it out in the official server or add the bot to your own server. Although the bot is currently broken, I'm going to fix tonight and hopefully it won't die at random anymore, but for the most part it works.

This has mostly been done now, you can try it out in the official server or add the bot to your own server. Although the bot is currently broken, I'm going to fix tonight and hopefully it won't die at random anymore, but for the most part it works.
EDIT: It uses Nextcord and CloudLink Python client.

bruh
also I will contribute to that

This has mostly been done now, you can try it out in the official server or add the bot to your own server. Although the bot is currently broken, I'm going to fix tonight and hopefully it won't die at random anymore, but for the most part it works.
EDIT: It uses Nextcord and CloudLink Python client.

bruh
also I will contribute to that

I will upload the source code to GitHub once I do a few touchups.
List of changes I want to make (so I don't forget):

Discord mentions of channels, roles and user should just be their name when sent to Meower
Send pmsg to all other users online with plain text message and user, this will help bots be able to handle Meower bot commands sent through Discord easily
Make CloudLink client thread auto restart when bot gets disconnected from Meower (even with pinging it gets disconnected every 6 hours or so)
Sending pings to Discord from Meower such as @everyone, specific users and specific roles should be made non-pingable when sending to Discord

This has mostly been done now, you can try it out in the official server or add the bot to your own server. Although the bot is currently broken, I'm going to fix tonight and hopefully it won't die at random anymore, but for the most part it works.
EDIT: It uses Nextcord and CloudLink Python client.

bruh
also I will contribute to that

I will upload the source code to GitHub once I do a few touchups.
List of changes I want to change before I upload to GitHub:

Discord mentions of channels, roles and users should just be their name when sent to Meower
Send pmsg to all other users online with plain text message and user, this will help bots be able to handle Meower bot commands sent through Discord easily
Make CloudLink client thread auto restart when bot gets disconnected from Meower (even with pinging it gets disconnected every 6 hours or so)
Sending pings to Discord from Meower such as @everyone, specific users and specific roles should be made non-pingable when sending to Discord

If there's any others you think I'm missing, please tell me.

for the pings in nextcord you can create an allowed mentions obj

I'll look into that, that sounds like it'll be helpful.

I'll look into that, that sounds like it'll be helpful.

https://docs.nextcord.dev/en/stable/api.html#allowedmentions

WAIT
from nextcord.ext import commands,utils

bot = commands.bot("?>")

@bot.event
async def on_msg(msg):
          cloudlink.sendPacket(utils.remove_markdown(message.clean_content))

Discord mentions of channels, roles and users should just be their name when sent to Meower

WAIT
from nextcord.ext import commands,utils

bot = commands.bot("?>")

@bot.event
async def on_msg(msg):
          cloudlink.sendPacket(utils.remove_markdown(message.clean_content))

https://docs.nextcord.dev/en/stable/api.html?highlight=message#nextcord.Message.clean_content

Thanks, that will probably help quite a lot :)

Discord mentions of channels, roles and users should just be their name when sent to Meower

WAIT
from nextcord.ext import commands,utils

bot = commands.bot("?>")

@bot.event
async def on_msg(msg):
          cloudlink.sendPacket(utils.remove_markdown(message.clean_content))

https://docs.nextcord.dev/en/stable/api.html?highlight=message#nextcord.Message.clean_content

This didn't work, I tried importing utils but it's from just plain nextcord now and it doesn't have the attribute remove_markdown.

hasn't this been done for a while now? closing this unless you want it reopened


/statuses?tx_id (all requests having query parameters) return(s) 400 BAD REQUEST when Content-Type: 'application/json` is set

With Content-Type: 'application/json' set:
{
    "message": "The browser (or proxy) sent a request that this server could not understand."
}

without:
{
    "message": "Not found",
    "status": 404
}

(or returns the status of transaction).
This happens with BDB v0.10.2
Might be related to: https://github.com/flask-restful/flask-restful/issues/510
This might affect all endpoints using query parameters.
Edit: Confirmation, it's affecting all endpoints using query params: https://github.com/bigchaindb/bigchaindb/pull/1577/commits/ea157a30bd09bb78dd8d751e5694cbc9c05d1f28

Might be related to: https://github.com/flask-restful/flask-restful/issues/510

This might affect all endpoints using query parameters.

This is a critical issue that already made users give up on driver development: https://gitter.im/bigchaindb/bigchaindb?at=589123aefba5bfea06b258ae

The only solution that I could find would be to explicitly set the location where the argument parser should check for the arguments.
So instead of:
parser = reqparse.RequestParser()
parser.add_argument('tx_id', type=str)
parser.add_argument('block_id', type=str)

Use this:
parser = reqparse.RequestParser()
parser.add_argument('tx_id', type=str, location='args')
parser.add_argument('block_id', type=str, location='args')

This will disregard the content-type and get the arguments directly from args instead of a json body if the client set the content-type to application/json
The only problem is that this is not compatible with with strict=True in parse_args

@r-marques I'm trying to understand if it's something we can do at a middleware level.


[Bug] Flags are ordered from oldest to newest

What Happened?
Flags are ordered from oldest to newest
Steps to Reproduce

Assign tables and judges
Have a few judges submit flags
View flags on Admin portal

What device(s) are you seeing this issue on?
No response
What browser are you seeing this issue on?
No response
Possible Solutions
Flip ordering so most recent flags are on top
Additional Info
No response

ideally this should be taken care of by #124, but it's still a good thing to note for the show all flags


[FEAT]: Create a 3D model of the complete ARNEIS machine

Description
As part of the documentation to be submitted to the OpenCV contest before the end of Phase 2 I believe it would be worthwile to produce a 3D model for the complete machine.
Additional Information
No response

For the records: The 3D models of the OAK-D-Lite PCB and enclosure are available under https://github.com/luxonis/depthai-hardware/tree/master/DM9095_OAK-D-LITE_DepthAI_USB3C

OAK-D-LITE_PCBA_Production.step
OAK-D-LITE_enclosure.stp

Those .STEP files may be displayed and navigated using a STEP File Viewer.
For instance https://www.emachineshop.com/free-online-step-file-viewer/

For the records: The 3D models of the OAK-D-Lite PCB and enclosure are available under https://github.com/luxonis/depthai-hardware/tree/master/DM9095_OAK-D-LITE_DepthAI_USB3C

Board: OAK-D-LITE_PCBA_Production.step
Enclosures: OAK-D-LITE_enclosure.stp

This was just confirmed by Luxonis in https://github.com/luxonis/depthai-hardware/issues/188#issuecomment-1013926254

Another valuable resource is the following thread:

How do you convert a .STEP file to use with LDraw?

@BonesWall  would you like to give it a try?

@alv67 please confirm whether #320 has fixed this issue.
If so please close it, otherwise comment clarifying what is still missing.

Close this by merging #328


Iox #484 improve posix mutex test

Pre-Review Checklist for the PR Author

[x] Branch follows the naming format (iox-#123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit messages are signed (git commit -s)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[x] All checks have passed
[x] Assign PR to reviewer

Notes for Reviewer
code contributed by @shankar-in
Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior
[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

Post-review Checklist for the Eclipse Committer

[ ] All checkboxes in the PR checklist are checked or crossed out
[ ] Merge

References

Closes #484

@dkroenke wrt https://docs.github.com/en/github/managing-your-work-on-github/managing-labels#applying-labels-to-issues-and-pull-requests, under "Applying labels to issues and pull requests point no. 3" , for me mentioned checkbox is not visible

@shankar-in did you try my suggestion?
EXPECT_DEATH({{ iox::posix::mutex mtx{false}; mtx.lock; }}, ".*");

From the test name, it should just check the destruction of a locked mutex. There is no need to use threads for this.

@elBoberido No I didnt try that. Doing that now. Will update that.

@elBoberido No I didnt try that. Doing that now. I have a clarification. The statement iox::posix::mutex mtx{false}; mtx.lock;  cause the process to die? Does the double inner block {{ }} make this significant?. Is my understanding correct? Could you pls explain?

At the end of the scope the dtor is called, which causes the process to die if the mutex is still locked. It might be sufficient to have it just in one pair of {}.

I like it! Unnecessary code removed and much more readable and on point! However, a comment would probably be nice ;)


Add some failing tests to fix

This PR is poised to eventually fix some new issues discovered in unification of circuit Nodes.

After comparing what we have to what Falstad has, it seems that the variables that handle the current across a capacitor and how much current the associated current source draw is the same in ours but different in Falstad's. This is the only difference that I am seeing between the implementations, but there's works while ours somehow doesn't. The behavior I'm currently seeing with capacitors is that they at first report "-0.0" current over them for the first step they are added, and then the same value for all subsequent steps. Voltage sources attached to capacitors report the correct current on the first step, but then don't change at all. I am unsure of how to fix this since there doesn't appear to be any errors on the surface, but its failing every test using a capacitor for this reason.

If that looks like the implementation really is in parity, I'm going to suspect the infrastructure is to blame--contributions to the matrices might not be made that ought to be. This warrants some further investigation.

@Grissess Cael said he expects you will investigate why this is broken and intends to leave it to you.
My cursory inspection shows that your capacitor code does not implement something related to the current source correctly; I think it should be separate from  the overall current across the device.

It is apparent to me that the MNA is pretty broken. At this point, the only thing that we've done is improve the code and mark with a test that our capacitor implementation is (has been) broken.


[SIP] sqlite host component

This introduces a SIP for adding a sqlite interface to Spin as well as an initial MVP implementation.
Much like the key-value interface, this would be a "batteries included" implementation where users gain access to a SQL database with no configuration required.
I have also created an example TODO application that uses this MVP implementation.

@itowlson I think this is ready for another look. The implementation pretty much matches the SIP now. That means the open questions still remain, but we have a working prototype. I guess the question now is: should we answer the open questions or merge and then figure out the open questions?
(FYI: the CI failure is because spin-componentize needs to be updated)

I am marking this PR as ready to review. While this feature is not complete, I do believe it is useful enough for folks to start experimenting with. Below I've tried to capture the current state and the open questions we need to answer before we can officially make this a stable feature:
This PR provides the following features:

Support in spin for a sqlite host implementation that provides a default sqlite database when running spin locally.

Support for runtime configuration for adding additional local sqlite databases.

Rust SDK support for targeting these databases from a Rust based spin component

The ability to run arbitrary sql statements against the default database on start up with spin up --sqlite-migration="YOUR SQL HERE"

I do not believe this feature is ready to marked stable. As such I have put sqlite support behind a feature flag in the only SDK that currently supports it: Rust. This will still allow folks to target sqlite if they create spin components manually (i.e., using wit-bindgen directly), but I believe we already consider such usage experimental and not officially supported by our backwards compatibility guarantees. If we believe that by merging this PR that we are committing to backwards compatibility of this feature in its current form, then I think we should wait and consider how we want to allow for experimentation before committing to backwards compatibility.

There are still some open questions in the SIP with regards to sqlite versioning and capacity limits. I don't believe those need to be solved before experimentation can begin.

There is no support for this feature in Fermyon Cloud. Since this is an experimental feature only, I believe that this is fine for now - though we will quickly want to change that.

By the way, what is the order of landing things for the first iteration?  I saw the PR took a dependency on a branch of spin-componentize.  Is the idea that we would land spin-componentize first?  Is that going to be the general workflow for as long as we need to continue to support modules?  (cc @dicej)

I think this is ready to merge. We'll wait until the next version of spin is released so that we have a full release cycle to let this bake.


error when val

return torch._C._nn.linear(input, weight, bias)
RuntimeError: mat1 dim 1 must match mat2 dim 0

./scripts/train_smpl_cam.sh test_3dpw configs/256x192_adam_lr1e-3-res34_smpl_3d_cam_2x_mix_w_pw3d.yaml

second problem when ./scripts/validate_smpl_cam.sh ./configs/256x192_adam_lr1e-3-res34_smpl_3d_cam_2x_mix_w_pw3d.yaml ./exp/mix2_smpl_cam/256x192_adam_lr1e-3-res34_smpl_3d_cam_2x_mix_w_pw3d.yaml-test_3dpw/model_1.pth:
\HybrIK/hybrik/models/layers/smpl/lbs.py", line 258, in lbs
pose_offsets = torch.matmul(pose_feature, posedirs)
RuntimeError: mat1 dim 1 must match mat2 dim 0

hello,i meet a same question.i send a qq mail for you.

@daixiangzi Hi, can you elaborate on how to solve this problem? I found that pose_feature, posedirs can not be multiplied.

I think i found the problem. The gt pose is already rotation matrix，so we did not do batch_rodrigues(). Just reshape the input is OK. And it should be better to add a judgment, whether the input is the representation of the axis angle.


Add a Gitter chat badge to README.md

foundersandcoders/beekeeper now has a Chat Room on Gitter
@msmichellegar has just created a chat room. You can visit it here: https://gitter.im/foundersandcoders/beekeeper.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

This was an accident guys! Do ignore.


Add readthedocs.io documentation

A readthedocs.io. documentation is needed with samples how to use the package.

When https://github.com/mara/mara-pipelines/pull/69 is merged and published, a sample should be added in README and docs how to run dbt on a separate container through a Mara pipeline. For example:
from mara_pipelines.commands.sql import ExecuteSQL
from mara_pipelines.pipelines import Pipeline, Task
from mara_dbt.commands import DbtRun

pipeline = Pipeline(
    id='dwh_dbt',
    description="Runs additional dbt transformations on the data warehouse")

pipeline.add_initial(
    Task(id='initialize_schemas',
         description="Create the schema",
         max_retries=3,
         commands=[ExecuteSQL(sql_statement="""
EXEC util.drop_schema_force 'dbt_next'
GO
CREATE SCHEMA dbt_next
GO
""")]))

pipeline.add(
    Task(id='run',
         description='Run dbt run',
         commands=[DbtRun()],
         context='dbt-project-dwh'))

pipeline.add_final(
    Task(id='replace_schema',
         description="",
         commands=[ExecuteSQL("EXEC util.replace_schema 'dbt_next', 'dbt'")]))


Fixed the ambiguous interaction mapping constructors

XRTK - Mixed Reality Toolkit Change Request
Overview
Fixed the ambiguous interaction mapping constructors.
Target of the change:
Is this enhancement for:

Core (core framework, interfaces and definitions)

Changes:
Brief list of the targeted features that are being changed.

Fixes: #345
Fixes: #336
Supersedes: #346

Breaking Changes:

Update the MixedRealityInteractonMapping constructors but afaik the Oculus package is the only one taking advantage of it atm.

This addresses only the changes for oculus, however, it also breaks all existing mappings for all other controllers.  This needs to be based off this fix branch.
Also, it will need the default assets updating with this fix.  Let's close this PR and I'll incorporate your changes in my branch.

however, it also breaks all existing mappings for all other controllers. This needs to be based off this fix branch.

I don't understand. This shouldn't break any existing mappings.

Let's close this PR and I'll incorporate your changes in my branch.

I don't think that's a good idea.

/azp run

/azp run

Not a fan of mixing up the order of parameters like this, but it works.

Any suggestions on how this could be better?
Mainly the issue is that the wrong Constructor is being called because of the order of the parameters.
One possible fix is to make the type different, but I'm not sure what would be an appropreate change.
The only other possibility is removing the inputName altogether and using the established inputAxisX and inputAxisY values that essentially function the same.

The only other possibility is removing the inputName altogether and using the established inputAxisX and inputAxisY values that essentially function for the same purpose.

No, they are defined concepts and should not be reused.
It's an acceptable fix


module-same-contents: replace jsonpointer by jmespath expressions

Describe the solution you'd like
Instead of having selectors "/test" we would have "test", which is more intuitive. Now it's a good moment to change it before we have any legacy :), and jmespath has even more capabilities.
Additional context
jmespath is simpler and its adoption is a bit higher. These days I saw AWS using it in one of its libraries too (https://awslabs.github.io/aws-lambda-powertools-python/2.1.0/utilities/idempotency/#idempotent_function-decorator).

If implementing this, it would fix #53 too. Check their requirements.


Thesis objects

@thuhoainguyen Please write here (or attach the file) about thesis objectives. I will help you with the code but I need to understand the objectives...

@anhtuduong I just shared you the thesis proposal through google doc. Please check !

@thuhoainguyen please write here so we can keep all the info in the project and we don't need to open email and find every time we need any info

Topic: Enhancing Kidney Lesion Segmentation through Histology-Specific Label Refinement and Classification
Description:
Recent advancements in medical imaging and machine learning have significantly improved the accuracy of automated lesion detection and segmentation. The KiTS challenges represents a pivotal step in this domain, focusing on the segmentation of kidney lesions from CT scans. This thesis proposal aims to build upon the challenge's foundation by introducing a novel approach that incorporates histology-specific labels for lesions, aiming to refine segmentation accuracy and serve as a precursor for detailed lesion classification.
Kidney lesion segmentation plays a crucial role in diagnosing and planning the treatment of renal diseases. The KiTS 2023 challenge has provided a substantial dataset for developing and benchmarking segmentation algorithms. However, the challenge's initial scope, which treated all lesions as a single class, presents an opportunity for further refinement. With the availability of histological data for approximately half of the cases, this work proposes to enhance the granularity of lesion segmentation and explore its implications for lesion classification.
This research aims to demonstrate that incorporating histology-specific lesion labels can significantly enhance the granularity and accuracy of kidney lesion segmentation. It is anticipated that the refined segmentations will not only serve as a more detailed map for clinical analysis but also as a robust foundation for developing precise lesion classification algorithms. Moreover, this work is expected to establish a benchmark for future research in the domain, providing a baseline against which subsequent advancements can be measured.
Building on the foundation of the KiTS 2023 challenge, this thesis proposal seeks to advance the field of kidney lesion segmentation and classification through the innovative use of histology-specific lesion labels. By enhancing the precision of segmentation models, this research holds the potential to contribute significantly to the diagnosis and treatment planning of renal diseases, marking a step forward in the application of machine learning techniques in medical imaging.
This internship requires the following skills: Medical Image Processing, Machine Learning, Python Programming, Image Diagnosing
Objectives:

To pretrain a segmentation network, specifically the nnUNet, on the KiTS 2023 challenge dataset, using the existing classification of background, healthy kidney tissue, and generic lesions.
To refine the segmentation process by introducing histology-specific lesion labels, utilizing the newly available histological data.
To evaluate the accuracy and specificity of the refined segmentation in comparison to the generic lesion segmentation.
If time permits, to modify and enhance the segmentation model based on initial outcomes and to establish a baseline for future developments in kidney lesion segmentation and classification.

The tasks include:

This study will leverage the nnUNet framework, renowned for its comprehensive and adaptive approach to medical image segmentation. nnUNet automates several crucial steps in the segmentation process, including model selection, data preprocessing, augmentation, and post-processing, making it an ideal choice for efficiently handling the complexities of medical imaging data. The project will commence with the nnUNet framework being utilized to preprocess the KiTS 2023 challenge dataset and automatically select an appropriate neural network architecture for the task of segmenting background, healthy kidney tissue, and generic lesions.

Subsequently, the project will innovate by adapting the nnUNet pipeline to incorporate histology-specific lesion labels into the segmentation process. This adaptation involves training the automatically selected network with a modified dataset that includes detailed lesion labels, leveraging nnUNet's capacity for efficient data augmentation and preprocessing tailored to the new segmentation task.

The effectiveness of introducing histology-specific lesion segmentation will be evaluated through a series of performance metrics, such as accuracy, precision, recall, and the Dice coefficient, comparing the outcomes against the original segmentation task.

Further exploration will be considered to enhance the nnUNet framework based on initial findings, including potential modifications to the network architecture, preprocessing, and data augmentation steps to improve segmentation accuracy and specificity for histology-specific lesions.

@anhtuduong  I just wrote the thesis proposal in this task so you can follow.


rotatetime was not as excepted when use default Local clock

Hi, I found my log rotate time is alway 8h late when I use default Local clock. My server time is CST.
I reviewed the code:
func (rl *RotateLogs) genFilename() string {
	now := rl.clock.Now()
	diff := time.Duration(now.UnixNano()) % rl.rotationTime
	t := now.Add(time.Duration(-1 * diff))
	return rl.pattern.FormatString(t)
}

I  test diff := time.Duration(now.UnixNano()) % rl.rotationTime local.  the diff result is not correct, its always less 8 hours.
I suppose  diff := time.Duration(now.UnixNano()) % rl.rotationTime  is UTC  unixtime diff result. And  when it convert to LOCAL CST, the result is not as excepted.
I know I can set a clock config, but I still wonder If there is a bug or it's just designed.
BTW,your library is very good. Thanks.

Hi, thanks for the report.
I'm looking at my code at e92e07d07, and genFilename() doesn't look anywhere close to what you described. Perhaps you are using an older copy? I say this because I have a feeling (sorry, it has been a while, so I forget the details) that I worked on exactly that problem before.
https://github.com/lestrrat-go/file-rotatelogs/blob/e92e07d07467f805f67f8980907087f976849393/rotatelogs.go#L93-L115

@lestrrat It's my fault. I pulled an old project https://github.com/lestrrat/go-file-rotatelogs .
You can close the issue then.
Thank u for your patience.


Medplum graphiql sample app fails to load

Repro:
go to https://graphiql.medplum.com and attempt to log in with console open

Was not able to repro


Aliases containing slashes cannot be used in annotations

Describe the bug
Per the docs, the correct way to use argocd-image-updater with Kustomize is to use the image name (e.g. argoproj/argocd) as an alias. However, Kubernetes only allows one slash in an annotation key, so I'm unable to use the alias for other argocd-image-updater options.
To Reproduce

Create a Kustomize overlay with a slash in the image name:

apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

images:
  - name: hjacobs/kube-resource-report
namespace: kube-resource-report
resources:
  - github.com/hjacobs/kube-resource-report/deploy?ref=20.7.3

Create an Application manifest pointing to the overlay, with argocd-image-updater annotations:

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  annotations:
    argocd-image-updater.argoproj.io/image-list: hjacobs/kube-resource-report=<container_repo_url>/kube-resource-report
    argocd-image-updater.argoproj.io/hjacobs/kube-resource-report.update-strategy: latest
  finalizers:
    - resources-finalizer.argocd.argoproj.io
  name: kube-resource-report
  namespace: argocd
spec:
  destination:
    namespace: kube-resource-report
    server: https://kubernetes.default.svc
  project: cluster-services
  source:
    path: kube-resource-report/
    repoURL: <config_repo_url>
    targetRevision: main
  syncPolicy:
    automated:
      prune: true
    syncOptions:
      - CreateNamespace=true

Expected behavior
Creation of an Argo CD Application that continuously deploys the latest version of the <container_repo_url>/kube-resource-report container image.
Version
0.7.0
Logs
error="Application.argoproj.io \"cluster-services-krr-bdatasf-npd-us-east4-ng\" is invalid: metadata.annotations: Invalid value: \"argocd-image-updater.argoproj.io/hjacobs/kube-resource-report.update-strategy\": a qualified name must consist of alphanumeric characters, '-', '_' or '.', and must start and end with an alphanumeric character (e.g. 'MyName',  or 'my.name',  or '123-abc', regex used for validation is '([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9]') with an optional DNS subdomain prefix and '/' (e.g. 'example.com/MyName')"
2020-10-01T20:36:15.016Z	ERROR	controller-runtime.controller	Reconciler error	{"controller": "applicationset", "name": "cluster-services-kube-resource-report", "namespace": "argocd", "error": "Application.argoproj.io \"cluster-services-krr-bdatasf-npd-us-east4-ng\" is invalid: metadata.annotations: Invalid value: \"argocd-image-updater.argoproj.io/hjacobs/kube-resource-report.update-strategy\": a qualified name must consist of alphanumeric characters, '-', '_' or '.', and must start and end with an alphanumeric character (e.g. 'MyName',  or 'my.name',  or '123-abc', regex used for validation is '([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9]') with an optional DNS subdomain prefix and '/' (e.g. 'example.com/MyName')"}

Sorry - just saw the additional detail about aliases with slashes at https://argocd-image-updater.readthedocs.io/en/stable/configuration/images/#assigning-aliases-to-images. Changed the slash to an underscore and all is well.

Thank you for creating this issue tho. Maybe this detail should be highlighted in the documentation a little, I agree that it can be hard to over-read in the running text.


README images are not visible when using Github dark theme

Perhaps they could be rendered with a white/light background

thanks for fixing it!


14000571_us - Tributary to Houston Tommy Creek

[ ] fiss site card has a note at the top saying "both us and ds for 14001106". Does this mean 14000571 and 14001106 have the same habitat data? 14001106_us has its own site cards so this doesn't really make sense. Maybe this note was a typo and it should say "both us and ds for 14000571"??

when a survey is done is a location that  is both downstream of one crossing and upstream of another (ie. in between the two crossings) the data applies to the reporting for both (in this case - 14000571_us and 14001106_ds).  For example - the gradient and channel widths, substrate, etc. downstream of 1401106 is the same as those characteristics upstream of 14000571.
Happens lots and there are quite a few examples in past reports.  Of interest there is an example of how this has been done incorrectly in Peace 2023 - which will need to be fixed.

Okay yup that makes sense. Turns out there is no 14001106_ds site so not much of an issue.


Remove shared build configurations among build.gradle in subprojects

Build configurations have been shared among subprojects by subprojects { ... }. But, it has made the build configurations complicated, and hard to understand.
As the first step to simplify, this pull request removes the share although it will cause some duplications.

Thanks!


Fixed some of the formatting errors reported by PyCharm as well as simplified some code constructs.

This is same at previous PR just with fixed merge conflicts

See my review in that last PR about a bug you introduced, because it's still present: https://github.com/ranaroussi/yfinance/pull/1130

Sorry, cannot see the review, maybe it got lost when I merged

This is what I see. no review at all.

As that PR is gone for me could you just please add the comment on this PR?

Spooky! I've copied over my comment.


feat(1.x): force re-navigate

It should be possible to re-instantiate just part of an app.
This used to work prior to the refactor to remove router-component. navigate used to take a second argument that over-rode the router's check against navigating to the same URL.
The new implementation should probably just decorate the navigation instruction for the router. Then each viewport can decide what it means to "force" a renavigation.

This has been removed in favor of the reactivation hook.


tracing? bug for full scale example in logsums branch

Ran into a tracing? bug when running the latest logsums branch on the full example.
Traceback (most recent call last):
  File "simulation.py", line 50, in <module>
    pipeline.run(models=_MODELS, resume_after=resume_after)
  File "e:\projects\asim\activitysim\activitysim\core\pipeline.py", line 496, in run
    run_model(model)
  File "e:\projects\asim\activitysim\activitysim\core\pipeline.py", line 422, in run_model
    orca.run([model_name])
  File "C:\Anaconda2\lib\site-packages\orca\orca.py", line 1876, in run
    step()
  File "C:\Anaconda2\lib\site-packages\orca\orca.py", line 780, in __call__
    return self._func(**kwargs)
  File "e:\projects\asim\activitysim\activitysim\abm\models\school_location.py", line 116, in school_location_sample
    trace_label=trace_hh_id and 'school_location_sample.%s' % school_type)
  File "e:\projects\asim\activitysim\activitysim\core\interaction_sample.py", line 380, in interaction_sample
    tracing.extend_trace_label(trace_label, 'chunk_%s' % i))
  File "e:\projects\asim\activitysim\activitysim\core\interaction_sample.py", line 287, in _interaction_sample
    pick_group = choices_df.groupby([choosers.index.name, alt_col_name])
  File "C:\Anaconda2\lib\site-packages\pandas\core\generic.py", line 3748, in groupby
    sort=sort, group_keys=group_keys, squeeze=squeeze)
  File "C:\Anaconda2\lib\site-packages\pandas\core\groupby.py", line 1428, in groupby
    return klass(obj, by, **kwds)
  File "C:\Anaconda2\lib\site-packages\pandas\core\groupby.py", line 345, in __init__
    level=level, sort=sort)
  File "C:\Anaconda2\lib\site-packages\pandas\core\groupby.py", line 2398, in _get_grouper
    in_axis, name, gpr = True, gpr, obj[gpr]
  File "C:\Anaconda2\lib\site-packages\pandas\core\frame.py", line 1992, in __getitem__
    return self._getitem_column(key)
  File "C:\Anaconda2\lib\site-packages\pandas\core\frame.py", line 1999, in _getitem_column
    return self._get_item_cache(key)
  File "C:\Anaconda2\lib\site-packages\pandas\core\generic.py", line 1345, in _get_item_cache
    values = self._data.get(item)
  File "C:\Anaconda2\lib\site-packages\pandas\core\internals.py", line 3225, in get
    loc = self.items.get_loc(item)
  File "C:\Anaconda2\lib\site-packages\pandas\indexes\base.py", line 1878, in get_loc
    return self._engine.get_loc(self._maybe_cast_indexer(key))
  File "pandas\index.pyx", line 137, in pandas.index.IndexEngine.get_loc (pandas\index.c:4027)
  File "pandas\index.pyx", line 157, in pandas.index.IndexEngine.get_loc (pandas\index.c:3891)
  File "pandas\hashtable.pyx", line 675, in pandas.hashtable.PyObjectHashTable.get_item (pandas\hashtable.c:12408)
  File "pandas\hashtable.pyx", line 683, in pandas.hashtable.PyObjectHashTable.get_item (pandas\hashtable.c:12359)
KeyError: 'PERID'
Closing remaining open files:data\skims.omx...doneoutput\pipeline.h5...donedata\mtc_asim.h5...done

Same error with tracing turned off as well

Did you reinstall the asim env  recently?
what version of pandas are you running? I am betting it is pre 0.20.1
groupby allows grouping on named index as well as columns starting with 0.20.1, which is what I was doing.
Anyway, I changed the order of code (moved set_index after groupby) so this should now work even with older versions.
Also added num_chunks arg to chunk iterators and chunk tracing per #176

It looks like we've got pandas 0.18 on the server.  Regardless, it's now running.  Thanks.


'Clip' is not a suitable structure for an OpenTrackIO sample

It has been suggested that we have a new 'Sample' type and this allows us to remove unnecessary fields from a Clip and vice versa

I am not sure I understand the "and vice versa" from the above.
For the "Sample is the result of removing unnecessary fields from a Clip" suggestion, is the idea that one has a Sample object, containing the Sampling.STATIC data from the clip for such fields that have non-None values as well as the Sampling.REGULAR data from the clip that is a tuple of non-None values?
Or is 'Sample' type more like what you would get from indexing a clip, where in the process of doing so, one would select the tuple values corresponding to that index, so that a 'Sample' was basically a single temporal sample?
If the latter, I am not sure what 'remove unnecessary fields' means. Would it suffice to have the Sample's to_json() method omit Sampling.STATIC fields that had None as a value, and Sampling.REGULAR fields that were empty tuples?
Hmmm. Does "and vice versa" mean there would be a method on a Sample that would take a JSON dict or JSON string and create a Clip with values of None for Sampling.STATIC attributes not in the Sample, and with empty tuples resulting in the Clip for Sampling.REGULAR attributes that, similarly, were not in the Sample?

But while we are speaking of clips: a freshly-created Clip has its various sampled fields set to None. Using Clip.append() or using accessors one can set the sampled fields to tuples of the appropriate type. Is it considered legitimate to have a sampled field have a zero-length tuple?

I'd like @palemieux to comment on this to be sure we agree on the definitions. I see it like this:
A Clip is a collection of metadata parameters that may be sampled at different rates. This is how CamDKit was originally designed. In this case each dynamic (or 'regular' as it is referred to in code) parameter has an array of values. A Clip is normally associated with a recording e.g. on a card in a camera and can contain static data and dynamic data of different lengths.
{
    "static_param1", 1
    "dyn_param1": [1,2,3],
    "dyn_param2": [1,2,3,4,5,6],
}

By contrast a Sample (or perhaps 'Frame'?) is sample of multiple metadata parameters at a particular time.
{
    "param1": 1,
    "param2": 2,
    "timestamp": 123456789
}

and so a sequence of samples (a 'Stream'?) would be an array of these objects [{...},{...},{...}].
A Stream is different to a Clip because it cannot have different frequency metadata inside as each Sample defines state at a particular time. (Sources requiring differing frequency data would send on separate Streams.)
Currently data is stored in a clip and then 'swizzled' out when the OpenTrackIO docs are generated. It would be better to refactor a new Sample type that uses the same metadata model and framework but handles the conceptual difference between a Clip and a Sample.

If the metadata parameters in the clip may be sampled at different rates, is there any way of determining a priori which sample of dyn_param2 "goes with" a particular sample of dyn_param1?
If not, what is the advantage (beyond simplicity of implementation) of storing parameters from various samples as lists, one per parameter, stored in a clip object, vs. having the clip object have a dictionary of samples indexed by timestamp (be it timecode or PTP or even a sequence number from the transport system)?

While we're discussing Clip, why do we need a duration parameter? I would think the duration would be the Clip'same capture_frame_rate multiplied by the number of instances of a regular attribute (though that way, the duration of a clip could vary depending on which attribute one examines).
Incidentally in response to @jamesmosys ' comment above I think Sample is to be preferred to Frame. Lens metadata comes in at many times the frame rate. Even if it comes in over a separate Stream with a higher sample rate, you still need to call the individual things you've gathered something, and Frame can't be it; multiple things would relate to the same frame of image or sound essence.


explosion-effect config description inconsistency

Discussed in https://github.com/PurpurMC/Purpur/discussions/1338

Originally posted by Blackrobe June  9, 2023
Hi, I want to report inconsistency in the documentation vs actual server value
https://purpurmc.org/docs/Configuration/explosion-effect_2
Key: end-crystal.*.explosion-effect (and possibly others like respawn anchor or bed in the nether)
It is said in the documentation

explosion-effect
default: DESTROY
description: The type of explosion effect. Available values are NONE, BREAK, and DESTROY.

But when I try to set it to "BREAK" it is not accepted by the server:
[09:05:13 ERROR]: Unknown value for blocks.end-crystal.baseless.explosion-effect! Using default of BLOCK
[09:05:13 ERROR]: Unknown value for blocks.end-crystal.base.explosion-effect! Using default of BLOCK

Also in the documentation, the "BLOCK" option is nowhere to be seen in any config.

Originally planned on switching back to the use of Explosion#BlockInteraction instead of Level#ExplosionInteraction, but with the way the enums are implemented it's easier to just continue using Level#ExplosionInteraction. The documentation still needs to be updated so closing this issue and reopening the one on PurpurDocs.


automatically name forward and inverted indexes

index = "ceeaus"

should create the directories ceeaus.inv and ceeaus.fwd as necessary. This simplifies the config settings and removes duplicate name checks.

added in 8977441f218d8825e7f2d7c19dab1e1a1ae9f85a


Add support for BinaryData values to DynamicData

By default, JsonSerializer.Serialize() will write BinaryData to JSON as an empty object, and JsonSerializer.Deserialize<BinaryData>() will throw a NotSupportedException because BinaryData does not have a public parameterless constructor.
We would like DynamicData to allow setting properties to Azure models, and some Azure model types have BinaryData properties.  We would also like to support assignment of BinaryData properties directly for some BYOM scenarios.
This PR adds a JsonConverter to DynamicData's default JsonSerializationOptions to write a BinaryData instance as a base64 encoded string value.  It also adds a cast operator to retrieve this value as a byte array.  It also throws an exception if you try to assign a Stream, as we believe the users intent not be to serialize Stream but instead to assign the content of the stream.  This allows us to address a customer need we might discover later without introducing breaking changes.
Addresses https://github.com/Azure/azure-sdk-for-net/issues/35944:

[ ]  Enable setting properties on DynamicData as BinaryData

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Core

Closing in favor of https://github.com/Azure/azure-sdk-for-net/pull/36791


[TAILORED] Twitter i-frame Pop-up and-or Login

As a user, when I click "Post", I want twitter to pop-up in an i-frame, already logged in, so I can post immediately.

@Cieloc was this delivered?

No twitter popup yet.


Roomdef not being fully saved to the file

Only the name of the room is being save, but nothing else. Mainly the tags, but also the flags, events etc...
I guess it will be kinda related to the inheriting flags in sphere.ini
// Area flags
// Flags for tuning the default behavior of regions, areas and rooms
// AREAF_RoomInheritsEvents			00000001 // ROOMDEFs inherit EVENTs from the parent AREADEF (if any)
// AREAF_RoomInheritsFlags			00000002 // ROOMDEFs inherit REGION FLAGS from the parent AREADEF (if any)
// AREAF_RoomInheritsTAGs			00000004 // ROOMDEFs inherit TAGs from the parent AREADEF (if any)
AreaFlags=02

Confirmed, if server goes down tag from rooms will be missed.


Add release page for Apache Daffodil VS Code 1.2.0

@mbeckerle If you don't mind taking another look at this, I just updated in tandem with closing of rc1 and releasing of rc2.


[WIP] Placeholder PR for In-Place Pod Vertical Scaling feature

This is a placeholder PR for In-Place Pod Vertical Scaling  feature.
A new md file for this work is justified. I might also need to modify some existing md files as well. I'll try to get to that as soon as I can.

/retitle [WIP] Placeholder PR for In-Place Pod Vertical Scaling feature
/milestone 1.19

/assign

Hello @vinaykul .
Checking whether you plan to continue to work on these changes?
Thanks!

@kbhawkey - This enhancement has been moved into 1.20 milestone. Thanks for your comment. Sorry for not clearing the milestone.
/milestone clear

Hello @vinaykul .
Checking whether you plan to continue to work on these changes?
Thanks!

yes - for 1.20, but we may be looking at changing the API a little. discussions are going on here https://github.com/kubernetes/enhancements/pull/1883/files

If this is going into 1.20, do we want to close this PR since it will need to be based against dev-1.20?

You can change the base branch for PRs. However, I would close this one and, @vinaykul, when you come to open a new PR you can reference this one to track the link between them.

@jimangel @sftim sure. i'll close this and reference it in 1.20 dev.


Infinite re-renders when using select in useInfiniteQuery

Describe the bug
When I transform data from infinite query with select option, I get infinite re-renders in my component
To Reproduce
Add following component and render it (implied that it'll be wrapped in QueryClientProvider of course):
import { useInfiniteQuery } from 'react-query';
import { nanoid } from 'nanoid';

const App = () => {
  const query = useInfiniteQuery(
    'todos',
    ({ pageParam = 0 }) => fetchTodo(pageParam),
    {
      getNextPageParam: ({ nextCursor }) => nextCursor,
      select: ({ pages, pageParams }) => ({
        pages: pages.map(({ todos }) => (
          todos.map(todo => ({
            ...todo,
            localId: nanoid(),
          }))
        )),
        pageParams: [...pageParams],
      })
    },
  );

  console.log('re-render')

  return null;
};

const todos = [];

for (let i = 0; i < 100; i++) {
  todos.push({ id: nanoid(), text: `Todo ${i + 1}` });
}

const TODOS_FETCH_PORTION = 10;

const fetchTodo = (cursor) => {
  const nextCursor = cursor + TODOS_FETCH_PORTION;
  const result = {
    todos: todos.slice(cursor, nextCursor),
  };
  if (todos.length - 1 >= nextCursor) {
    result.nextCursor = nextCursor;
  }
  return result;
};

export default App;

Expected behavior
Expected App to not re-render infinitely.
Desktop (please complete the following information):

OS: macOS 10.13.6
Browser: Chrome 96.04

Additional info
With useQuery this bug does not occur

can you please provide a codesandbox reproduction, thanks.

Sure!
https://codesandbox.io/s/epic-cloud-7q8gp?file=/src/App.js

thanks, I think I found the issue. infiniteQueryObserver overwrites setOptions of queryObserver, but doesn't forward the second parameter (notifyOptions):
https://github.com/tannerlinsley/react-query/blob/b84681fdc7d37645cd149f40e98cb18fd8bc393c/src/core/infiniteQueryObserver.ts#L64-L76
what we would need is this to match the parent signature.
  setOptions(
    options?: InfiniteQueryObserverOptions<
      TQueryFnData,
      TError,
      TData,
      TQueryData
    >,
    notifyOptions?: NotifyOptions
  ): void {
    super.setOptions(
      {
        ...options,
        behavior: infiniteQueryBehavior(),
      },
      notifyOptions
    )
  }

the missing notifyOptions will lead to the select result being calculated twice, which in your case is fatal because nanoid() will return a new value every time.
would you maybe like to contribute that change?
btw - if select is an inline function, it will still run on every render, so also on useQuery, the id is generated on every render. If you expect it to be stable, you'd have to make sure select itself is memoized (or do the transformation inside the queryFn).

Forwarding notifyOptions didn't help. I was able to fix this bug by moving local id generation to the queryFn. Didn't know select is called every render. Thank you!

Interesting. Let me keep this issue open because I still think it’s a bug in RQ. I can try to create a test case from your sandbox and take it from there :)

:tada: This issue has been resolved in version 4.0.0-alpha.2 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Detailed install guide or links

Hello,
I tried installing this repository to generate a newer version of the spritesheet, but I can't get it to work. I'm mainly using npm, but it seems for this repository it's required to use yarn. With a fresh install of yarn and different approaches with install and whatnot I wasn't able to figure out how to set it up.
I always got the error that pokesprite-gen-lib wasn't found until I added it to package.json of packages/cli.
After that I get an error that dada-cli-tools is corrupt and cannot be used. Is there anything im missing?

Yau was gehtn hier

Yau was gehtn hier

@koenigderluegner Hi, it looks like you figured this out? I saw you made a new pull request on their spritesheet. Any chance you could explain how you got it working? I've been butting heads with it for a while.

For those who come here in the future:
koenigderluegner has forked the repo (https://github.com/koenigderluegner/pokesprite-gen/tree/fix-broken-build) and made some fixes that allow it to be run locally - most importantly replacing the dada-cli-tools yarn package (which is corrupt) with a link to the git version.
If after you run npm i, you try the command it says the command does not exist, run npm link. Then, open a new terminal instance and the command should work.
To get the latest version of pokesprite generating, you must change the version of the pokesprite-images dependency in lib/package.json.

Hi, I'm trying to generate a gen7x spritesheet, but I'm not sure how to. Any help would be appreciated :)

Hi all, I made this package long ago and then kinda stopped working on it because it seemed to me that the overwhelming majority of people weren't interested in using the sprites as a spritesheet.
I haven't actually used this package myself in quite some time and I used a personal package (dada-cli-tools) for it which I shouldn't have, since that's really for personal use and I break it occasionally. So it probably needs some reworking. I'll put it on my todo list to look into when I have some time!

Thank you so much! Appreciate your response and effort!

Sorry about the trouble. In the meantime I think you could try the koenigderluegner fixes that isabroch mentioned above, although I haven't personally tried it.

@msikma For what it's worth, I'm currently working on a web project to view / edit Pokémon save files, and I'm using https://github.com/msikma/pokesprite-spritesheet for the sprites (it's made things pretty easy). I'd personally love and appreciate an updated sheet. 😄


chore: improve test coverage of CreateKafkaInstance and components

Adds tests for all stories about CreateKafkaInstance

@jgiardino I noticed we had a few gaps around the aria roles of the alerts, I tried to update the code to better follow the accessibility guidelines. I'm no expert about this, if you could give it a check it would be appreciated.

This introduces a new helper for testing dialogs, renderDialog. It adds a wrapping div around stories with an id root. We can point to that id when writing the appendTo callback to make sure that the dialog properly renders in the test container.

Not sure why actions are not being run. I'm gonna close and reopen the PR. Sorry for the noise...


[SPARK-29070][CORE] Make SparkLauncher log full spark-submit command line

Log the full spark-submit command in SparkSubmit#launchApplication
Adding .python-version (pyenv file) to RAT exclusion list
What changes were proposed in this pull request?
Original motivation here, expanded in the Jira..  In essence, we want to be able to log the full spark-submit command being constructed by SparkLauncher
Why are the changes needed?
Currently, it is not possible to directly obtain this information from the SparkLauncher instance, which makes debugging and customer support more difficult.
Does this PR introduce any user-facing change?
No
How was this patch tested?
core sbt tests were executed.  The SparkLauncherSuite (where I added assertions to an existing test) was also checked.  Within that, testSparkLauncherGetError is failing, but that appears not to have been caused by this change (failing for me even on the parent commit of c18f849d7667012597fe747448845d80a367b1bb).

Can one of the admins verify this patch?

ok to test

Merged build finished. Test PASSed.

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

Test build #111442 has started for PR 25777 at commit 047d74d.

Merged build finished. Test PASSed.

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

Test build #111444 has started for PR 25777 at commit 0af4f88.

Merged build finished. Test PASSed.

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

Test build #111444 has finished for PR 25777 at commit 0af4f88.

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

Merged build finished. Test FAILed.

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

Test build #111442 has finished for PR 25777 at commit 047d74d.

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

Merged build finished. Test PASSed.

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

retest this please

Test build #111453 has started for PR 25777 at commit 0af4f88.

Merged build finished. Test PASSed.

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

Test build #111453 has finished for PR 25777 at commit 0af4f88.

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

Merged build finished. Test PASSed.

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

Merging to master.


can you provide a multi thread tcp server  sample?  uv_write2 only works only  with multi processes ?

uv_write2 can't send file handle to pipe between threads?
I get hints from other posts:
"There is no way to accept a client on another loop. There are a number of things you can do:

create the listener handle, then create X new loops, send the handle to them using uv_write2 and listen in all of them
if you're on Linux (> 3.9 IIRC), create X new loops and bind a handle to the same IP and port using SO_REUSEPORT
have a single loop accept all incoming connections and send them to other loops using uv_write2."

but it doesn't figure out it's in multi threads or multi processes.

Can you post questions to libuv/help?  This issue tracker is only for bug reports.  Thanks.


Driver's License

Can this be configured to scan a driver's license or paper document, similar to the dropbox mobile app scanner?

This searches for barcodes and converts them to alphanumeric input.  I don't think there's any similarity to that.


link quickstart guide and t2 documentation

This is an issue to start a discussion to check if there is any benefit in connecting the quickstart guide and the t2 documentation.
The t2 documentation has an example crd which is actually applyable. If somebody wants to use the quickstart guide and would like to deploy a cluster, it would be easy to start with the t2 api. Therefore, the t2 documentation could use a working crd.

Does it make sense to include Test/Dev Cluster in the documentation? Is that a third approach or similar to quickstart / t2? If approaches differ, what are the reasons?


Particles animate per frame per camera

In Godot 2.1.2, with multiple cameras rendering at once, particles move at set speed * number of cameras. Probably related to #6578 (each viewport causes the whole rendering routine to be called each frame).
Possible solutions:
• Set particle speed for all emitters to original speed / number of cameras (can be done in GDscript)
• Make the renderer handle multiple cameras better
Anecdotal stuff:
• I'm working on a VR game which is probably going to come out before Godot 3.0 hits beta. I know related features are on the eventual to-do list. I noticed this bug because I had a button to toggle between single-viewport and splitscreen. Demonstration attached.
• I didn't see a 3D particle emitter in Godot 3.0 (at least with a build from a couple weeks ago), implying particles are under overhaul (explosiveness pl0x), possibly related to a renderer overhaul. Particles in Godot are tightly integrated with the renderer.
particlizer.zip (5.5 kB)

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about its current status here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.

Scope of particle timing is correct in Godot 3.0 rendering pipeline. Godot 2 pipeline is structured wrong, but is no longer maintained. Closing issue as it no longer applies.


The nuke's iconic detonation sound went missing.

Description
Got replaced with a fire alarm????

what

I think it was just a mistake in the fire alarms PR.


remove illegal MonadAsync super constraints

You cannot have type families in quantified constraints
https://gitlab.haskell.org/ghc/ghc/issues/17202#note_241130

Alternatively, we could do something like
data Async m t = Async
  { asyncThreadId :: ThreadId m
    -- AsyncRep is the associated type in the MonadAsync class
  , asyncRep :: AsyncRep m t
  }

instance Eq (ThreadId m) => Eq (Async m t)
instance Ord (ThreadId m) => Ord (Async m t)

bors r+


Broken Actor Framework Class Provider

When creating a new method for an actor (right click 'My Actor.lvclass' -> GOOP -> Add Method and the check box 'Create Actor Framework Message class for this actor method' set) the resulting send method (Send Foo.vi) will be broken.
I tracked down the issue and it looks like the VI \resource\Framework\Providers\Open_GDS\ClassProviders\ActorFramework\4.0.0\Message\Send Template.vi is broken. Unchecking the 'Allow debugging' solves the problem.
I'm using LabVIEW 2016 (16.0f5) and GOOP 1.2.31 with the shipped version 4.1.0 of the actor framework.

Thanks for finding it. The issue is that the VI is configured as Inline SubVI and Allow debugging.
I'll make sure the next build don't set the inline subVI check box.


How to use scopes?

Hi, can you help me with using scopes? I made a similar code to the example, but can't make scopes to work. My code is below.
use leptos::leptos_dom::ev::SubmitEvent;
use leptos::*;
use serde::{Deserialize, Serialize};
use serde_wasm_bindgen::to_value;
use wasm_bindgen::prelude::*;
use leptos::logging::log;
use leptos_hotkeys::use_hotkeys;
use leptos_hotkeys::use_hotkeys_ref;

#[wasm_bindgen]
extern "C" {
    #[wasm_bindgen(js_namespace = ["window", "__TAURI__", "core"])]
    async fn invoke(cmd: &str, args: JsValue) -> JsValue;
}

#[derive(Serialize, Deserialize)]
struct GreetArgs<'a> {
    name: &'a str,
}

#[component]
pub fn App() -> impl IntoView {
    let main_ref = leptos::create_node_ref::<leptos::html::Div>();
    leptos_hotkeys::provide_hotkeys_context(main_ref, false, leptos_hotkeys::scopes!());
    use_hotkeys!(("space") => move |_| {
        log!("Space pressed!");
    });
    use_hotkeys!(("enter", "scope_a") => move |_| {
        log!("Enter pressed in scope A!");
    });
    view! {

Scope A

    }
}

I created it by cargo create-tauri-app --rc (Tauri v2), cargo add leptos_hotkeys, and editing app.rs. Detecting the space key works, but Enter key is not detected when I click "Scope A" text and press Enter, or any other way.
Also, how should I use use_hotkeys_ref? In the example code, I see something like use_hotkeys_ref(ref, "space", scope) => fn). What are the roles of ref and scope? Does it mean "detect the space key in the referenced element, which is under the scope"?

Hi, there is a section on how to use scopes in the README?

Thanks. It helped. One things is that I had to use enable_scope.call("scope_b".to_string()); instead of enable_scope("scope_b");, for disable_scope as well.

Thanks. It helped. One things is that I had to use enable_scope.call("scope_b".to_string()); instead of enable_scope("scope_b");, for disable_scope as well.

Thanks! Do you mind updating the README to reflect the correct calling convention for enable_scope?


Move tests to test folder

This PR moves all the tests into a separate test folder to help with code clarity in the src folder.

This should wait for Michaels iframe fix first, then I'll rebase: https://github.com/microsoft/vscode-edge-devtools/pull/367/files


feat(notifications): add button to mark all notifications as read

Updated version

@lex111 what about putting the button to the bottom of the screen? Makes more sense to me: I go to the tab, scroll down while watching my notifications, then decide to mark them all read

It's rather controversial, I'm comfortable when it's on top, and that's why I made it position is fixed initially, so that it was always visible .. let's wait for the opinions of others.

I like the top, personally. It's more like what GitHub has.

Ready for review.


#WMS ID: 11116 - [Quarterly REVIEW] - "How to IGNITE your Oracle Tenancy"

#WMS ID: 11116 - [Quarterly REVIEW] - "How to IGNITE your Oracle Tenancy"
SELF QA LIST IS COMPLETED.
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

Please resolve conflicts and then we can merge this request.


Support Python versions <3.10

Currently this library requires Python 3.10, but does not specify this in setup.py.
For example with Python 3.9, I receive
Python 3.9.17 | packaged by conda-forge | (main, Aug 10 2023, 07:02:31)
[GCC 12.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import aqt.jax.v2.flax.aqt_flax as aqt
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/martin/miniconda3/envs/sd/lib/python3.9/site-packages/aqt/jax/v2/flax/aqt_flax.py", line 432
    match self.lhs_freeze_mode:
          ^
SyntaxError: invalid syntax
>>> import aqt.jax.v2.config as aqt_config
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/martin/miniconda3/envs/sd/lib/python3.9/site-packages/aqt/jax/v2/config.py", line 18, in <module>
    from typing import Literal, Optional, TypeAlias, Union
ImportError: cannot import name 'TypeAlias' from 'typing' (/home/martin/miniconda3/envs/sd/lib/python3.9/typing.py)

In general it would be nice to at least support Python 3.8 as it's still quite commonly used as a stable Python version.

Thank you!
However we will fix it by requiring 3.10
3.8 is at end of life and 3.9 is not far behind.
https://devguide.python.org/versions/#versions


[WIP][K8S] Spark operator

Hi, @jkremser . Thank you for making a PR.

Please follow the PR template. The template is for the community, but this PR intentionally ignore it. The screencast should be an additional attachment for that.
Is this the following? Please give a clear explanation if this is your own contribution.

https://github.com/GoogleCloudPlatform/spark-on-k8s-operator

Could you send an email to<EMAIL_ADDRESS>about this and why this should be inside Apache Spark project?

cc @holdenk

Yeah idk if we want this in Spark it's self. The dev@ mailing list is probably the right place to raise this discussion. There are a lot hard coded things in here we would need to change.

This is same as https://github.com/radanalyticsio/spark-operator.

Thanks, @nrchakradhar . For me, this seems to dumped from that repo by the author.
I took a look. The current status is difficult to be considered for Apache project contribution because there is no Apache license header and the remaining package name authorship radanalytics.
I'd like to recommend @jkremser to reopen this after cleaning up the PR by adding licenses and remove the authorship package name. I'm close this for now.

I think before putting in work on this PR getting buy-in from the dev@ list might be a good next first step.

Definitely, +1 for @holdenk 's comment.

Thanks for looking into this. Ok, I will open the discussion on the dev list. I wanted to have the PR open before the summit and put the link on the slides.
And yes, it's basically dump of that repo (radanalyticsio/spark-operator) with couple of modifications. I for instance aligned the versions of the libraries to be the same as those that are used by Spark.

Is this the following? Please give a clear explanation if this is your own contribution.
https://github.com/GoogleCloudPlatform/spark-on-k8s-operator

No, it's different operator written in Java and it supports three different custom resources, SparkCluster (spark running in standalone mode in the pods), SparkApplication (this has the same API as the operator from GCP, so it's using spark-on-k8s scheduling) and SparkHistoryServer. So there is some overlap, but it's a different thing. This operator is also available on operatorhub.io (link)

@dongjoon-hyun  @holdenk There are many operator projects out there but does not make sense to be part of the core project. Same for the Flink project. Btw in other PRs people complained about the code base being to large so this is another argument to avoid adding a lot of "external" stuff. Moreover the resource manager stuff logically belong outside of this project but for historical reasons they were kept here since the effort for the common  resource manager API never happened.

Agreed with what @skonto said. This doesn't feel like something that is necessarily part of core Spark.


Merge all fixes back to LTS

Description

Breaking Changes

JIRA Link

Checklist

[ ] Updated the Readme.md (if required) ?

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

npm package (@lts dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add Get-Uptime (C# code)

Add Get-Uptime cmdlet (C# code)
Modify psd1 files
Add tests

Close #2497
The PR replace #2507

Hi @iSazonov, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@PowerShell/powershell-committee reviewed the -pretty parameter and agreed that we should use formatting/output to handle that and improve formatting of timespan rather than adding capability into specific cmdlets.

@SteveL-MSFT  Thanks for conclusion! I did the fix.

@iSazonov I talked to @Francisco-Gamino offline about the forward link help URI for this Get-Uptime. We will request for a fwlink internally, and once we get it, will update the code with it. For now, it's OK to leave the HelpUri a blank string.
So now the only missing part is the update to OutputType. Thanks!

OutputType is fixed.
@daxian-dbw thanks for great comments!

LGTM. Thanks @iSazonov!

@daxian-dbw Thanks for the review and usefull comments!


Unable to locate package...

i ran this in ubuntu
sudo apt install build-essential libboost-all-dev libssl-dev libcrypto++-dev libcurl4-openssl-dev
and it spit out
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package libboost-all-dev
E: Unable to locate package libcrypto++-dev
E: Couldn't find any package by regex 'libcrypto++-dev'
what should i do

This isn't a WSL bug. Run update first
sudo apt update

you also can run command of  apt-get upgrade

i tried to run
apt-get update
then
apt-get upgrade
then
sudo apt-get install pyrit
and the same error
"E: Unable to locate package pyrit"

You didn't state your Ubuntu version, but pyrit was dropped from focal (20.04).

This isn't a WSL bug. Run update first
sudo apt update

after " sudo apt-get upgrade"


Excluding invalid/failed queries from analyzing

I see these changes mostly apply to the Analyzer's main class. Nevertheless that class has been deprecated in favor of the Parser's main class as you can see here. In other words, you're looking at the wrong main class (we will be removing this main class soon to avoid further confusion)


[Feature] Show process id on the logging output

Currently, the output shows the process number launched in order.. ex[0] or [1] ... it would be really nice if instead, the process id of the child process was displayed.
There are times I'd really like to just kill a specific process, not all of them.

Hi @tracker1, you can already do this with the --prefix pid argument.


Scroll Past End retains 3 lines instead of 1

Scroll Past End retains the last 3 lines instead of just 1.
I would like to modify the behavior of Scroll Past End where it retains only one line instead of three.
I could not find how to do this.
Any help is welcomed.
Atom 1.17.2
apm  1.17.0
npm  3.10.5
node 6.9.5 x64
python 3.6.1
git 2.13.0
OS: Arch Linux but it does the same on others, Linux Mint 17, 18, Ubuntu Mate 16.04.

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


Version 2.0.1

Codecov Report

Merging #32 (2b34b17) into master (3659a9d) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #32   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           35        36    +1
  Lines         1235      1241    +6
=========================================
+ Hits          1235      1241    +6

Impacted Files
Coverage Δ

include/Simple-Utility/functional/base.hpp
100.00% <ø> (ø)

include/Simple-Utility/nullables/base.hpp
100.00% <ø> (ø)

tests/nullables/additional.cpp
100.00% <100.00%> (ø)

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 3659a9d...2b34b17. Read the comment docs.


Usage as mergetool

Hi there, the plugin sounds promising. But it not yet clear to me how to use it in combination with git mergetool? I plan to directly jump into your plugin when using that command to resolve merge conflicts.
When only using the command Conflicted the current window is used, which looks weird when starting from the diff mode in vim. Thank you!

Hi @p-o-bcak, I'm not entirely sure what all is involved in setting up a mergetool or if this plugin would be amenable to usage as a git mergetool in its current form.
That said, I use the command via a git alias which handles everything for me, and allows me to start up from the shell. I've just added a section to the README about how to configure this.
I'm going to close this now as I don't necessarily see this as a bug or issue (although I'd be open to reviewing a PR related to proper mergetool support as an enhancement), and in the hopes that the above will cover what you need, but please feel free to comment or reopen if you still have trouble.


What are the different data augmentation techniques followed in the paper?

In the paper, accuracy for different data-augmentation techniques have been such as
1. Random-clipping
2. Multiscale-cropping
I didn't understand what they mean. I read something on Random-clipping but it wasn't enough to understand.
Can you put some light on these two?

Random clipping is selecting a random starting frame from the sequence, which is quite standard nowadays. The implementation is pretty clear:
https://github.com/gulvarol/ltc/blob/master/donkey.lua#L42
Multiscale cropping is a simple augmentation as well:
https://github.com/gulvarol/ltc/blob/master/donkey.lua#L213
If the explanations in the paper are not sufficient to understand, you can also check the references we give ([6, 23]).


Per Tab, Admin Mode

Decide per tab if I want to open it in Administrator mode (Run as Administrator)
Default setting 'Open as Administrator'

Possible dublicate of #1032. Also look here: #632

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #5000 for more discussion.
/dup #1032 #632 #5000


Authenticating IAM user on a Query or Mutation without giving permissions to AuthRole

Which Category is your question related to?
API, Function
Amplify CLI Version
5.5.0
What AWS Services are you utilizing?
AppSync, Lambda
Provide additional details e.g. code snippets
I have a lambda function which is making an API request to my AppSync GraphQL API. I have granted this lambda function permissions to access the AppSync API. To be able to access any field on the GraphQL API using IAM authentication, I need to:

add and @auth (rules: [{ allow: private, provider: iam }]) directory (like suggested here https://docs.amplify.aws/guides/functions/graphql-from-lambda/q/platform/js/#signing-a-request-from-lambda - this will underneath add @aws_iam directive to all the Queries and Mutations generated by the @model directive.

OR

add the AppSync @aws_iam directive if this is a custom mutation or query

My use case:
I want to access the Queries and Mutations generated by the @model directive from a lambda function using IAM authentication. The @auth (rules: [{ allow: private, provider: iam }]) provides this option, HOWEVER it also adds permissions to the "Authenticated Role" of users (like explained here: https://docs.amplify.aws/cli/graphql-transformer/auth/#private-authorization). This means that my cognito users, through identity pool can access these endpoints.
My question:
How to add @aws_iam directive to a Query or a Mutation created by the @model directive, without granting  "Authenticated Role" access to this operation?

I believe this was fixed in transformed v2. You now provide the permissions using the amplify cli and it adds the function name to the vtl and allows access. You no longer have to configure this as a specific @auth on the model itself and so it will not also add this to the Authenticated Role.

Closing due to inactivity

@DylanBruzenak , Could you please elaborate on your post? I am typically looking to give access to only one mutation to my frontend team . I would want them to access the mutation only by an iam role and also being private. But I don't want them to sign up with the cognito pool. How can this be achieved?

@Lekha30 Sorry, it's been too long since I've looked at this and my case was providing authorization for a lambda to access the api, not an end user. You might want to:

Use transformer v2 (probably the default now, but here is the migration guide: https://docs.amplify.aws/cli/migration/transformer-migration/)
maybe setup a lambda behind api gateway to access the mutation you want
use the cli amplify update function or while creating the function to give it access to the mutation
add some sort of auth to api gateway for your front end.

I hope that helps!


TypeError BaseActiveRecord::findAll()

What steps will reproduce the problem?
Model::findAll(['id' => $id]);

What is the expected result?
get ActiveRecordInterface
What do you get instead?
TypeError: Return value of Yiisoft\ActiveRecord\BaseActiveRecord::findAll() must implement interface Yiisoft\ActiveRecord\ActiveRecordInterface, array returned

Additional info

Q
A

Version
dev-master

PHP version
7.4.3

Operating system
Debian linux

You can write the code error, thanks.

Fixed,
thks,


ImportError: cannot import name 'InterpolationMode' from 'torchvision.transforms'

🐛 Bug
ImportError: cannot import name 'InterpolationMode' from 'torchvision.transforms' when trying to import PlantPathologyDM
To Reproduce
Steps to reproduce the behavior:

Import  PlantPathologyDM
See error

Expected behavior
InterpolationMode should be imported correctly
Additional context
Tested on torchvision==0.8.1.

I solved the same problem by upgrading to torchvison 0.9.1

I found an easy fix which will make the package compliant with torchvision=0.8.1. Just remove InterpolationMode import and code related to it. torchvision.transforms.Resize uses bilinear interpolation by default, so there's no need to specify it directly.


Bug report: Operations requires double-click but doesn't mention this

Describe the bug
I'm finding that when I click an operation (like "To Base64"), nothing happens.
I later discovered that double-clicking the operation caused the operation to be added to the recipe, but I didn't expect that to be required at first.
To Reproduce

Open CyberChef
Click on an operation (like "To Base64")
Observe that nothing appears to happen
Double-click that operation
See that after double-clicking the operation, it gets added to the recipe

Expected behaviour
At step 3 I was expecting the operation to appear (like in step 5) without having to double-click the operation.
Is this intended?
If it's not feasible to make single-click work (perhaps other people are accustomed to it not adding the operation to the recipe), it could be nice if the tooltip could mention that double-click is required.
Thanks

Perhaps it's a common understanding, which may be why it wasn't mentioned in the tooltip.

Adding an operation can be done by either double-click or drag and drop.
I use double-click if it is just a quick 1-step receipe or I need to operation added to the bottom of the recipe
Drag and drop if there are more steps and I need control where to insert the operation
I think double-click is deliberate so you don't add operations by accident.

Doubleclicks do not seem not work in Chrome on Android. And the drag and drop was not obvious to me. So I was quite puzzled...


Silence cobra usage and errors

When we face an error, cobra prints the whole command usage, and also the error again, which is noisy.
Example
→ bacalhau job describe j033
11:58:29.663 | INF ProtocolLabs/workspace/bacalhau/pkg/repo/fs.go:92 > Initializing repo at '/var/folders/vz/pk7pk9qx1c5gbktg8nhyl88h0000gn/T/tmp.t9RCAPgQtw' for environment 'production'
Error: could not get job j033: Unexpected response code: 500 ({
  "error": "Job not found. ID: j033",
  "message": "Internal Server Error"
})
Usage:
  bacalhau job describe [id] [flags]

Examples:
  # Describe a job with the full ID
  bacalhau job describe j-e3f8c209-d683-4a41-b840-f09b88d087b9

  # Describe a job with the a shortened ID
  bacalhau job describe j-47805f5c

  # Describe a job with json output
  bacalhau job describe --output json --pretty j-b6ad164a

Flags:
  -h, --help            help for describe
      --output format   The output format for the command (one of ["json" "yaml"])
      --pretty          Pretty print the output. Only applies to json and yaml output formats.

Global Flags:
      --api-host string         The host for the client and server to communicate on (via REST).
                                Ignored if BACALHAU_API_HOST environment variable is set. (default "bootstrap.production.bacalhau.org")
      --api-port int            The port for the client and server to communicate on (via REST).
                                Ignored if BACALHAU_API_PORT environment variable is set. (default 1234)
      --cacert string           The location of a CA certificate file when self-signed certificates
                                	are used by the server
  -c, --config string           config file(s) or dot separated path(s) to config values
      --insecure                Enables TLS but does not verify certificates
      --log-mode logging-mode   Log format: 'default','station','json','combined','event' (default default)
  -r, --repo string             filesystem path to bacalhau repo (default "/var/folders/vz/pk7pk9qx1c5gbktg8nhyl88h0000gn/T/tmp.t9RCAPgQtw")
      --tls                     Instructs the client to use TLS

could not get job j033: Unexpected response code: 500 ({
  "error": "Job not found. ID: j033",
  "message": "Internal Server Error"
})

Desired Behaviour
Just print the error, and make it clear that this is an error, by either prefixing with Error: or in a different color like we do when we print errors in job history events
https://github.com/bacalhau-project/bacalhau/blob/e7c052c0425d9688dee47cbcd5fe7542058476dd/cmd/util/printer/job_progress_printer.go#L70-L72

Agreed! And adding to this, we currently print the error message twice, e.g.
> bacalhau job describe j033

<the error message>

<usage>

<the error message again>

I think we might need a mixture of: https://github.com/spf13/cobra/blob/371ae25d2c82e519feb48c82d142e6a696fd06dd/command.go#L232-L236
// SilenceErrors is an option to quiet errors down stream.
	SilenceErrors bool

	// SilenceUsage is an option to silence usage when an error occurs.
	SilenceUsage bool

on each command.


fix(payload): Fix GitHub payload to send

:pencil: Description
:link: Related Issues

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


build dreamview error

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04):openEuler
Apollo installed from (source or binary):source
Apollo version (3.5, 5.0, 5.5, 6.0):8.0
Output of apollo.sh config if on master branch:

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:
bash apollo.sh build_cpu dreamview

Supporting materials (screenshots, command lines, code/script snippets):
(20:05:13) INFO: Invocation ID: b87064bb-52e5-48d7-8400-f7730f5dc6ef
(20:05:15) INFO: Current date is 2023-06-05
(20:05:16) INFO: Analyzed 159 targets (0 packages loaded, 0 targets configured).
(20:05:16) INFO: Found 159 targets...
(20:05:31) ERROR: /apollo/apollo-v8.0.0/modules/dreamview/backend/BUILD:43:10: Linking modules/dreamview/backend/libbackend.so failed: (Exit 1): gcc failed: error executing command
(cd /apollo/apollo-v8.0.0/.cache/bazel/6703445324e57066f8e9637d3f3146a3/execroot/apollo &&
exec env -
LD_LIBRARY_PATH=/usr/local/fast-rtps/lib/:/usr/local/lib64:/opt/apollo/sysroot/lib64
PATH=/apollo/bazel-bin/modules/tools/visualizer:/apollo/bazel-bin/cyber/tools/cyber_launch:/apollo/bazel-bin/cyber/tools/cyber_service:/apollo/bazel-bin/cyber/tools/cyber_node:/apollo/bazel-bin/cyber/tools/cyber_channel:/apollo/bazel-bin/cyber/tools/cyber_monitor:/apollo/bazel-bin/cyber/tools/cyber_recorder:/apollo/bazel-bin/cyber/mainboard:/home/openeuler/java/dragonwell-<IP_ADDRESS>+9-GA/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/root/bin
PWD=/proc/self/cwd
PYTHON_BIN_PATH=/usr/bin/python3
PYTHON_LIB_PATH=/usr/lib64/python3.9/site-packages
/usr/bin/gcc @bazel-out/riscv64-fastbuild/bin/modules/dreamview/backend/libbackend.so-2.params)
Execution platform: @local_config_platform//:host
/usr/bin/ld: error: /usr/lib64: read: Is a directory
collect2: error: ld returned 1 exit status
(20:05:32) INFO: Elapsed time: 19.181s, Critical Path: 13.89s
(20:05:32) INFO: 20 processes: 1 remote cache hit, 14 internal, 5 local.
(20:05:32) FAILED: Build did NOT complete successfully

@daohu527
Do you know how bazel generates the libbackend. so 2. params file? I found that there is a syntax error in this file.

it's auto generate by bazel rule, the copts will pass param to gcc
https://github.com/ApolloAuto/apollo/blob/446c9c6d481fef9853ab9f167a0e934092058244/modules/dreamview/backend/BUILD#L10-L15

it's auto generate by bazel rule, the copts will pass param to gcc
https://github.com/ApolloAuto/apollo/blob/446c9c6d481fef9853ab9f167a0e934092058244/modules/dreamview/backend/BUILD#L10-L15

Thank you for your reply. The problem has been resolved


console errors when running test

jest or yarn test will result in console errors which the CI ignores, but my git client doesn't like.
● Console

    console.log app/src/renderer/vuex/modules/notifications.js:34
      [ERROR] Error fetching validator set: Cannot read property 'validators' of undefined
    console.log app/src/renderer/vuex/modules/notifications.js:34
      [ERROR] Critical Error: Couldn't initialize the blockchain client. If the problem persists, please make an issue on GitHub.

@jbibla any feedback?

i don't think they are effecting my git client. but should these be resolved?


Documentation icons need label

Expected Behavior

Current Behavior
The triangle and explanation mark icons should have words on them in the documentation so that users know what they mean.
Context

Hey @hythloda I am starting with open source contribution can you help me how to get started


Extension wrongfully suggests Json as a valid type completion on Sqlite

We filter out the Decimal type if we detect a mongodb schema here in types/getSuggestionsForFieldTypes(). There should be another line to detect sqlite and filter out Json

Now it's a race with https://github.com/prisma/prisma/issues/3786 🐎

I already have a local patch to fix the completion lol, but thanks for the link, I'll add it in a comment 😁


When staking, must stake both DT & OCEAN at once

Motivation / Intended Effect *

Benefits: help on rug pulls, more fees to LPs, improves users' mental model, could help stabilize price.
Here's how it would play out.
-For non-publisher stakers: They need to buy DTs in 'swap', then stake both. So it's equivalent to staking in just OCEAN right now, because that buys DTs under the hood. Therefore no real effect, it's just two actions instead of one.
-For publishers: They're DT whales, so right now they're incentivized to stake in just DT. If they stake a bit of DT right now, that's good to cool price. If they stake lots, it's basically a rug pull ("DT minting attack" variant). So forcing both at once, in a way that maintains the proportions, could limit the latter happening. It forces the publisher to inject more OCEAN skin in the game
In short: it will help on rug pulls, and maybe to stabilize prices. We can make staking maintain proportions. Therefore for proportions in the pool to change, people need to do swaps.
New effect: new LPs have to swap, therefore pay fees to existing LPs. This is a good thing, because it's revenue to LPs.
New effect: it improves the UX because it improves user's mental model of the SW. Right now, just staking OCEAN doesn't help the user understand that a % of the OCEAN staked is used to purchase DTs. With this new approach, they have to first buy DTs (in the swap tab), then when they stake, they see that they stake both at once. It forces the user to have a slightly more complex mental model, but one that is a more 1:1 map to reality. It will help them to separate speculation (and normal purchases) from staking. Therefore it's a bit more safe for users too.
On implementation

If users don't stake tokens in the same ratio as the pool's OCEAN : DT ratio, they will immediately experience some IL. We can help users to address this by giving UI affordances to tell them how "off" their staking ratio is, compared to the pool's ratio. If we wanted, we could force them to stake within a given ratio (e.g. <5%), but perhaps that's too heavy-handed.

What's the status of this? It is marked as High so I would start right away. We still want this?

It's in "PM to discuss" column. That's the next step.

tinkering over in #277 and I think we are looking for something like this (prototype, numbers are hardcoded):


Errors are not displayed for inner errors with OneOrMore rules

As subject tells: In this case (and the tokens are recognized), the parsing fails, but error collection is empty.

fixed with pull-request #264 .
thanks to @aaaabcv


gRPC reflection for starknet data

It looks like grpcurl cannot pickup the starknet data schema through reflection. The node schema is working.

This was caused by tonic-reflection not implementing extensions. Fixed by patching the crate for the time being.


German translation: 1. Introduction + Contract

My part for #9

ok, just saw that Etherum uses Contract in German too. So I'll keep it here. I would prefer "Verträge" since it makes it easier to read :)


[BUG] Annotations are difficult to make use of

Describe the bug
I just ran the action for the first time with dry-run to see what it would propose to do, and the annotations (~ the output) look like this

Expected behaviour
There are a couple of problems with this that I can see

They contain shell escapes
The workflow proposed to comment on 128 issues and 21 pull requests, but I can't see that many annotations there (perhaps 10 is a cap?)
Maybe it would be good if the annotations were a summary of the actions to perform, and at the end of the full output the entire list was printed. I'd like to see what the action will do if I remove dry-run, but it's not that easy at the minute.

Thanks!

@iainlane can you give me a public URL to check where do you see those annotations?
I never tested this, I searched inside this repository if I could see this UI, but unsuccessfully.
I am used to simply check out the logs like here to see if everything works well.
I can take a look tonight.

@C0ZEN I'm sorry, I can't link to our run because it's a private repo 😭
But here's a run in this repo here which shows the bug: https://github.com/Sonia-corporation/stale/actions/runs/1717900568
Just from grepping the code I think it comes from processor.logger.notice, and some further chasing of documentation shows me it comes from here. Anything you log via these will appear as annotations I think, shouldn't use colours, and should make sense as a "summary"! 🙂

@iainlane yes, it's pretty clear to me what needs to be done.
I consider this a bug, since the documentation is not mentioning that annotations are incompatible with colours logs.
I will change how I use annotations to get rid of all sugar logs, and see how I can improve the notice to bring easy and useful insights.
Not only that, but I think being actually another way to just show some statistics is just a quick win.

In the version that is currently in deployment, every single warning and error will no longer appear as an annotation but as a simple log, keeping the current colouring.
So, no difference will appear in the logs.
On top of this, I added brand-new warnings and errors annotations to easily display those important information, but without any kind of formatting.
Moreover, I removed every single notice annotation, but I plan to bring back new ones based on the statistics and outputs.
See #429


no way to test Payable function

What version of Remix are you using?
The Latest One
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes, all in one file

Steps to Reproduce
I deployed a contract, that had a payable function with also a String parameter. I deploy contract successfully, however I cannot test it. When I click on "Transact" button, I expect MetaMask to popup or somehow to enter the value -- but it does not happen. It just says "transact to contract xyz.newXyz pending" but nothing happen.
Using Injected Provider - Bespoke (Neon EVM)
The browser is firefox.
Expected Behavior
It should ask for value to pay in the transaction call.
Actual Behavior
nothing happen

@smithit2000 I think you want the Remix Project that is here https://github.com/ethereum/remix-project

sry


$log is used in deleteMap but not injected

On this code location $log is used without injecting it. That means every time a map with event listener is destroyed the error occurs.
See this Plunk: http://plnkr.co/edit/kLGvrBQQG6GQVJ3xU8JQ?p=preview

click on "Map"
then click on "Start"

ok. I will fix it soon.

+1 annoying. I had to just comment out the line  $log.debug('clearing map events', eventName);

fixed. 1.16.6 released