Add a match timer

The match gamestage should have a timer that will reset the game. There are some systems already in place to support this,
To do:
1.) Add a new cVar to the gamemode in dominion.txt
2.) Add a duration based on the new cVar to the Match gamestage
3.) Write logic to reset the match if the timer expires

Rather than reset the match, it might be better to make adjustments to factors that could increase the likelihood of pushing a game to completion. Like in TF2 or any other competitive shooter, respawn times are often doubled, income value is increased to tip the scales, etc.

Overtime mechanics added with commit e1d0f89


Add InstanceProfile CRD

What resource do you need?
AWS cli parameter is:
create-instance-profile and add-role-to-instance-profile
What is your use case?

We are building VM which runs multiple commands against multiple endpoints where credentials are needed. Instead of passing these secrets directly into userdata field we can use Instance Profile to read credentials from AWS SecretManager.
Would you be willing to contribute it using code generator?
Sure, but might take some time at this point as we just started to use crossplane.

we also interested in instanceProfile since karpenter needs one ;)
  KarpenterNodeInstanceProfile:
    Type: "AWS::IAM::InstanceProfile"
    Properties:
      InstanceProfileName: !Sub "KarpenterNodeInstanceProfile-${ClusterName}"
      Path: "/"
      Roles:
        - Ref: "KarpenterNodeRole"

@ytsarev is going to take this issue on! I'll assign him once he's a member of the Crossplane org 😊

@ytsarev  take care that code-generator for iam is possible but region is injected check https://github.com/aws-controllers-k8s/community/issues/1150

@haarchri great, thanks for heads up :+1:


demo not working

Thank you for the release.
Demo in http://<IP_ADDRESS>:7861/ is not working. When I click on examples or upload an image, nothing happens.
Would be great if you could fix it.
Bests

It's back to work now


KAFKA-4722: Add application.id to StreamThread name

Add application.id to StreamThread name

please retest this

retest this please

This fixes KAFKA:4722

One meta comment:
Instead of making this change, could we just modify line 258 to be
this.logPrefix = String.format("stream-thread [%s]", threadClientId);

The reason I was a bit concerned about this approach is that it complicates the logic of generating the threadClientId, which is now depending whether users set the clientId in their config or not. cc @sharad-develop @mjsax

I am not sure. Thread#getName() is called on multiple places in the the code, and each should encode the application.id IMHO.
Furthermore, I am not sure what you mean by "threadClientId, which is now depending whether users set the clientId in their config or not"? Isn't it depended on the config anyway, ie, even before this change?

My understanding of the motivation was to distinguish threads across different JVMs within the same machine on the log4j outputs; but I understand your argument now that there are other places than prefix where threadName is used (e.g. outside class StreamThread).
In this case could use just use clientId + "-StreamThread-" + STREAM_THREAD_ID_SEQUENCE.getAndIncrement() as the thread name? In this case it works for both when users specify clientId or not; the current approach achieves similar result but with a more complex logic.

Sounds good to me!
@sharad-develop Can you please update the code according to @guozhangwang suggestion.


Rewrite to composite action

Rewrite this action to GitHub Composite Action
https://docs.github.com/en/actions/creating-actions/creating-a-composite-action
#35

Marking as draft. Missed that one test also needs an update.

Have you tested that the action still runs OK?

Didn't have time to test it yet. I'll let you know once I do.
Apologies for the force push, I wanted to fix my mismatched gpg signature.

Hi Sondre,
sorry, I was on a long vacation :) I Didn't have time to take a look. Now I need to catch up so probably won't have time for this either.
Feel free to close this PR, I'll reopen or create new when I find the time.

No problem! I might jump in and finish it for you 👍

@sondrelg Added some fixes and tested with #44. Looks good IMO.
Can't test .github/workflows/clean_packages.yml so leaving this one to you.
BTW I would recommend doing major release to avoid any surprises for v1 users.

Nice one :+1: I'll review this tomorrow!

Tested the main branch a few repos and the runtime has dropped from ~40 seconds to ~30 :partying_face: And no issues

@sondrelg

Re your comment about a major bump; what makes you think that's necessary?

setup python and installs pip package inline, not isolated in docker. May impact rest of the workflow. Not sure why would anyone use it like that though.
Fairly major change in terms of how this action operates.

So nothing necessary, just as a user I would prefer clear distinction.


Have you considered moving to https://github.com/python-lsp

Hi @paradoxxxzero . Thanks for the great tool and it has been super useful. A quick question. Have you considered moving this repo to https://github.com/python-lsp?

@paradoxxxzero have you given this suggestion any thought? Thanks.


ansi-wl-pprint hash correction

This ansi-wl-pprint bug just keeps biting us! Thanks guys!

fails because gcc is missing. works on my machine. anyone else?

Yet another spurious Travis error, this patch looks good to me. :+1: Will merge in soon

closed?

Yes. Commit was pushed to master
https://github.com/NixOS/nixpkgs/commit/3c7a9194317aa68e64baaab78dd6fb3ff4309495

Interesting, never seen it done that way! Thanks!


Min/max zoom, min/max radius for circle

The native API has max/min zoom level, but I couldn't find anything for circles. I think it would be a nice idea though. Thoughts?
I thought a simple [maxZoom] and [minZoom] inputs for the map (and maybe similar for a circle if it's a good idea).

Yeah, good idea. We should add that!

PR for min/max zoom was merged a while ago; I forgot to link it to the issue: https://github.com/SebastianM/angular2-google-maps/pull/764
As for circle min/max, I couldn't find anything in Google Maps native JS API, so I scratched the idea. It's trivial to implement a setter with bounds for the radius and then pass that into the map.


feat(cdk): add initial support for external secrets

What this PR does

Adds support for external secrets in MedplumInfraConfig during CDK deployments
Closes #2788

TODO:

[x] Add tests
[x] Refactor logic to make it less hard to read
[ ] Update docs for CDK config?
[ ] Consider implications of serially fetching secrets vs. parallelization

@ThatOneBro  Nice, thanks for jumping on this!

coverage: 93.928% (+0.01%) from 93.914% when pulling 9e180bd484a017815e675ac7845b606e57f02aee on derrick-support-cdk-external-secrets into aba5660ff1314965f246046bc7bdda97f0d733ef on main.

@codyebberson Awesome, so I moved over to the GetParameterCommand, and made region only a string in MedplumSourceInfraConfig. Let me know if this looks good.
I considered batching all the requests after walking the config and collecting which keys need to be fetched, but I think that's probably an over-optimization and dumped that idea for the current setup.


For a 0.1.10 release

[x] extend compat for Flux to include 0.12
[x] extend compat for MLJModelInterface to include 1.0

Codecov Report

Merging #127 (591b849) into master (77b4f58) will decrease coverage by 0.69%.
The diff coverage is 33.33%.

:exclamation: Current head 591b849 differs from pull request most recent head 6eb628b. Consider uploading reports for the commit 6eb628b to get more accurate results

@@            Coverage Diff             @@
##           master     #127      +/-   ##
==========================================
- Coverage   92.80%   92.10%   -0.70%
==========================================
  Files           7        6       -1
  Lines         264      266       +2
==========================================
  Hits          245      245
- Misses         19       21       +2

Impacted Files
Coverage Δ

src/common.jl
91.30% <33.33%> (-8.70%)
:arrow_down:

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 77b4f58...6eb628b. Read the comment docs.


use correct port depending on tls status

Provided a way for the prometheus port to be configured based on the status of management tls.
The template test is failing at the moment.
Need to end-to-end test this, and also configure the scrape CA cert here https://github.com/cloudfoundry/loggregator-agent-release/blob/2d916a5f3766753a7a78e21393d9b9b92e0b21f4/jobs/prom_scraper/spec#L40-L41

As I was testing this end-to-end today. I noticed that an a service instance that has tls configured, the prom scraper config file on disk as the correct port 15691, but netstat reports that 15692 is open
rabbitmq-server/e65f962d-a94e-4954-9435-9f272933675f:/var/vcap/jobs$ netstat -ltpn
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:15692           <IP_ADDRESS>:*               LISTEN      -
tcp        0      0 <IP_ADDRESS>:4369            <IP_ADDRESS>:*               LISTEN      -
rabbitmq-server/e65f962d-a94e-4954-9435-9f272933675f:/var/vcap/jobs$ grep -rnw 'rabbitmq-server' -e 'port'
rabbitmq-server/etc/conf.d/12-managementConfig.conf:1:management.ssl.port = 15671
rabbitmq-server/config/prom_scraper_config.yml:2:port: 15691


[discussion] Move public keys from experiences to resources

looking over the KB the public keys somehow feel out of place in experiences - I think they should be moved to resources. What do you think?

Go ahead!

top - will do after #69 is merged as it is touching the same file


Add support for Rails 4.1

Any suggestion for an addition to the SurveyorController to permit responses?

The NUBIC/rails4 branch (last commit @f2328b4) has already been merged in. I am planning to fork the repo, get latest master to pass on Travis, and then decide how to proceed further.

I would suggest starting with the simple_form branch


refactor(frontend): Make dashboard search box the first filter

SUMMARY
Searching is the most natural filter so most people are looking for it first before checking others. This PR moves Search to the beginning of the dashboard filters in an attempt to provide easier accessibility.
Feel free to close this PR if this is not suitable for core team.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
** BEFORE **

AFTER

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@kasiazjc @jess-dillard any objections to this? If not, we can probably get it through quickly. @cemremengu (or anyone, really) would just need to resolve the conflicts

@rusackas No objections here, let's do it!

@cemremengu let me know if you can fix the merge conflict, and we'll get this thing through!

Could this be done for Charts view also?  And I suppose Dataset view as well.


"AzureChatOpenAI" object has no field "langchain_llm" error when trying to use Azure OpenAI

Hi, when trying to use Azure OpenAI according to the guide here, I ran into this error:
ValueError: "AzureChatOpenAI" object has no field "langchain_llm"

My code looks like this according to the guide, please let me know if I'm doing anything wrong:
from ragas.metrics import (
    context_precision,
    answer_relevancy,
    faithfulness,
    context_recall,
)
from ragas.metrics.critique import harmfulness
from datasets import load_dataset
from ragas import evaluate
from langchain.chat_models import AzureChatOpenAI
from langchain.embeddings import OpenAIEmbeddings

import os

os.environ["OPENAI_API_TYPE"] = "azure"
os.environ["OPENAI_API_VERSION"] = "2023-05-15"
os.environ["OPENAI_API_BASE"] = "[my base]"
os.environ["OPENAI_API_KEY"] = "[my api key]"

# list of metrics we're going to use
metrics = [
    faithfulness,
    answer_relevancy,
    context_recall,
    context_precision,
    harmfulness,
]

azure_model = AzureChatOpenAI(
    deployment="[my deployment]",
    model="[my-model-name]",
    openai_api_base="[my base]"",
    openai_api_type="azure",
)

answer_relevancy.llm = azure_model

# init and change the embeddings
# only for answer_relevancy
azure_embeddings = OpenAIEmbeddings(
    deployment="[my deployment]",
    model="[my-model-name]",
    openai_api_base="[my base]",
    openai_api_type="azure",
)
answer_relevancy.embeddings = azure_embeddings

for m in metrics:
    m.__setattr__("llm", azure_model)

# data
fiqa_eval = load_dataset("explodinggradients/fiqa", "ragas_eval")

result = evaluate(
    fiqa_eval["baseline"],
    metrics=metrics,
)

df = result.to_pandas()
df.head()

Hey thanks for reporting this  - this was because we didn't update the docs correctly. This is the updated doc https://github.com/explodinggradients/ragas/blob/7d96554d6fdbed4fa434ea67d8b15e0484d515a1/docs/howtos/customisations/azure-openai.ipynb could you check with this and confirm if it is working for you ?

@jjmachan Thank you for this. 2 things I noticed:

Small typo in this code,

# wrapper around azure_model
ragas_azure_model = LangchainLLM(azure_model)
# patch the new RagasLLM instance
- answer_relevancy.llm = ragas_auzre_model
+ answer_relevancy.llm = ragas_auzre_model

# init and change the embeddings
# only for answer_relevancy

According to Langchain's documentation, we should use deployment_name instead

azure_model = AzureChatOpenAI(
-    deployment="your-deployment-name",
+    deployment_name="your-deployment-name",
    model="your-model-name",
    openai_api_base="https://your-endpoint.openai.azure.com/",
    openai_api_type="azure",
)

Those were the only 2 things I had to fix to make it work, thanks again!

thank you so much @nmklong for taking the time to test it and suggest the changes.
merging the docs in and closing this issue now :)


Best workflow for viewing PR in VSCode?

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:13:05.928Z)
OS Version: Darwin x64 16.6.0
Extensions:

Extension
Author
Version

theme-sapphire
Tyriar
0.2.1

quokka-vscode
WallabyJs
1.0.36

nord-visual-studio-code
arcticicestudio
0.3.0

vscode-eslint
dbaeumer
1.2.11

tslint
eg2
0.15.0

prettier-vscode
esbenp
0.19.1

Go
lukehoban
0.6.62

debugger-for-chrome
msjsdiag
3.1.5

vetur
octref
0.8.5

java
redhat
0.6.0

vscode-icons
robertohuertasm
7.9.0

sass-indented
robinbentley
1.4.1

language-stylus
sysoev
1.7.3

vim
vscodevim
0.9.0

@bpasero
I remember previously you talked about how you views large PR in VSCode instead of GitHub for diff and easier jumping around. Wondering if you mind sharing your workflow, or is there any extension you are using.
I was trying to achieve that by checking out PR in a branch, git reset HEAD~10 if the PR has 10 commits. But that's problematic if the PR merges in master or do updates. Also if I want to push changes to PR, I can't do a commit again and has to reclone that branch.
Thinking it'd be awesome to add a Custom View to Git Viewlet for reviewing / switching-between PR seamlessly.
/cc @Microsoft/vscode Also would like to learn how you are reviewing PRs!

I'm interested into this, too!

@octref I am using the diff feature from GH pull requests:

add .diff to the URL (e.g. https://github.com/Microsoft/vscode/pull/12400.diff)
download the file
from the command line: git apply file.diff

Of course this does not allow you to do changes on the PR, it is just good for reviewing the change.
In case the diff does not apply because the PR is out of sync with master, do this:

git clone repository of PR (master)
git remote add upstream https://github.com/Microsoft/vscode.git
git fetch upstream
git co <branch of PR>
git merge upstream/master
git push

@bpasero Thanks. I'll try to put together an extension to simplify this a bit...Doing this everyday is a pain.

@chrmarti It would be great if something like this flow was integrated into the new GitHub Issues & PR extension

@eamodio: @rebornix has something in the works for that: https://github.com/Microsoft/vscode-github-issues-prs/pull/3

@chrmarti @rebornix awesome -- looking forward to it


[release] 1.x

Stuff to do:

[x] first version
[ ] cluster
[x] ship toString package
[ ] app.use for middleware

lol nah

[ ] pull-stream module (separate)
[ ] benchmarks
[ ] tests using cucumber spec stuff

cluster is bad; app.use() is not needed yet; we don't need pull-streams; cucumber I don't really like anymore;
and as for benchmarks: I'm quite sure we're fast


Strict modal typings

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[X] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
Using the method open of dialogService with custom components, we have to manually put the generics "Data" and "Result" to get a typed interface like the example above:
// dialog component

import { DialogService, DialogRef } from '@ngneat/dialog';

export interface Input {
  value: string;
  object: {};
  num: number;
  sym: symbol;
}

export interface Output {
  outputData: string[]
}

@Component({
  template: `

Hello World

    <button (click)="ref.close()">Close</button>
  `
  changeDetection: ChangeDetectionStrategy.OnPush
})
export class HelloWorldComponent {
  constructor(public ref: DialogRef<Input, Output>) {}
}

// app component

import { DialogService } from '@ngneat/dialog';

@Component({
  template: `
    <button (click)="open()">Open</button>
  `
})
export class AppComponent implements OnInit {
  constructor(private dialog: DialogService) {}

  ngOnInit() {
    const dialogRef = this.dialog.open(HelloWorldComponent, {
    // no errors - data should be "Input" type
    // I expect: Type string is not assignable to Type Input
     data: 'i can pass any value'
    });
    // no errors - data should be "Output" type
    // I expect: Type "any" is not assignable to Type Output
    dialogRef.afterClosed$.susbcribe(data => data.no.typing.inference)
  }
}

What is the motivation / use case for changing the behavior?
Auto typing inference when we pass a component with a DialogRef in the constructor onto DialogService could be useful
Since this library uses the newest version for Angular and Typescript, this could be done using Conditional and recursive types. If it's ok I could create a simple pr to work on this.
Since I realize that not everyone might like it, we could make this feature optional, still allowing you to pass a generic Data and Result without typing checking. This is also needed to continue to support other type of parameters like TemplateRef.
For example:
this.dialog.open<string, number>(HelloComponent, {
  // data is string
  data: 'string'
}).afterClosed$.subscribe(result => // result is number )

but if no generic are provided
this.dialog.open(HelloComponent, {
  // data will be input of HelloComponent DialogRef<Input, Output>
 // type-error
  data: 'string'
}).afterClosed$.subscribe(result => // result is DialogRef output )

Environment

Angular version: X.Y.Z

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

You're welcome to create a PR

Done :smile:


Use the new infrastructure type taxonomy

and colours

Relevant code with agreed colours:
https://github.com/nptscot/osmactive/blob/b08d91b310187c6b344d3682e040e47ce2519be1/R/osmactive.R#L437-L446
Relevant website comment: https://github.com/nptscot/nptscot.github.io/issues/243#issuecomment-2419535793

Thank you! I'm assuming we're still modelling "mixed traffic" -- roads considered parts of a route but without anything else, except maybe traffic calming or being part of an LTN. And I'm still going to retain an "unknown" category, to indicate "some kind of provision here beyond mixed traffic, but not sure what yet"

Done

Awesome


Omit platform libs which are testOnly scoped from balo & Refactor balo writer to have the correct platform label

Purpose

Added following changes
https://github.com/ballerina-platform/ballerina-lang/pull/27463
https://github.com/ballerina-platform/ballerina-lang/pull/27330
Fixes #

Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@pramodya1994 Let's wait on this for now

Closing as this was not added to the release


New GitHub account can't create projects

Creating a new account on GitHub for working with MakeCode leads to account suspension.
Any teacher who starts with MakeCode and wants his students to register an account on GitHub for that might run into this issue.
To Reproduce
Steps to reproduce the behavior:

Register a new account on GitHub
Do NOT create any repository manually on GitHub (otherwise it will work)
Got to MakeCode and create a new project
Connect your project to GitHub
It will take longer then expected, but it seem to work
The connection to github will be lost after a while
If you check on GitHub your account might be suspended

Expected behavior
The GitHub connection should work and be stable.
Screenshots

Additional context
This issue was first experienced by some teachers in makecode.calliope.cc and I just reproduced it. When they reported it, it sounds like the connection just didn't work, but the account also won't be suspended. So I'm not sure if in my case the suspension had some other reason. But they all agreed, that a virgin GitHub account cant create projects via makecode.
In my test the github project was created, since I got an email for the page build failure on that makecode project.

@pelikhan fyi

This has occurred to multiple of my students when working with arcade makecode please let me know if this is fixed.


adjust profile display

Changes Unknown when pulling e9a92dbda7319a5558b7070c67ee9c565d1d462b on make-graphs-responsive into ** on master**.


buffer_protocol for vectors and matrices in python bindings

This is an extension/takeover of https://github.com/gazebosim/gz-math/pull/498
Summary
This change enables the buffer_protocol for VectorX and MatrixYxY so that memoryiew and numpy.array can be used on them
np.array(math7.Vector3())
memoryview(math7.Matrix3d())

list(math7.Vector2d())
>>> math7.Quaterniond().xyzw() # == [0.0, 0.0, 0.0, 1.0]

Additionally:

IndexError is raised on invalid vector getitem/setitem access. This way list(vector) works correctly instead of spinning
A convenience Quaternion.xyzw() -> list is added

Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] Consider updating Python bindings (if the library has them)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.
🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

Tests are failing because of lack of numpy. Do you prefer to add numpy to the test environment or introducing this dependency is undesirable and I should rewrite the test to do away with numpy?

you can add the dependency here https://github.com/gazebosim/gz-math/blob/gz-math7/.github/ci/packages.apt

Done, thanks for the pointer :pray:

@ahcorde do you have additional pointers on why tests are failing? In particular, homebrew is missing numpy

do you have additional pointers on why tests are failing? In particular, homebrew is missing numpy

We generally don't add dependencies to stable releases.  If you could, retarget this to main, and we can work on landing it there first with the additional dependencies.

We generally don't add dependencies to stable releases. If you could, retarget this to main, and we can work on landing it there first with the additional dependencies.

Agreed. Hopefully we can backport  to gz-math7 without the numpy tests once it lands on main.

Cool, thanks for taking this over.
Perhaps the comments could help explain a bit more what's going on?
  /// \brief Underlying data pointer
  /// \remarks This method is intended for python bindings (numpy).
  /// \remarks The bounds-checking array subscript operator is much preferred for element access from C++.
  /// \return A pointer to the underlying data array.

Hi @unjambonakap , was wondering what the status on this was. Do we intend shipping this change with ionic (code freeze 26/8)? If we can't come to a consensus by then I will remove the beta label. For starters would it be possible for you to resolve the merge conflicts.

Also as this PR breaks API can we plese target main and not gz-math7

@unjambonakap any chance you can respond to the feedback in the next couple days? Since gz-math is a lower level library in the Gazebo stack, we'd like to wrap up PRs earlier to make sure there is enough time to test them before the code freeze (August 28th).

By Monday I should get it done
On Thu, Aug 22, 2024, 6:13 PM Addisu Z. Taddese @.***>
wrote:

@unjambonakap https://github.com/unjambonakap any chance you can
respond to the feedback in the next couple days? Since gz-math is a lower
level library in the Gazebo stack, we'd like to wrap up PRs earlier to make
sure there is enough time to test them before the code freeze (August 28th).
—
Reply to this email directly, view it on GitHub
https://github.com/gazebosim/gz-math/pull/524#issuecomment-2305149159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6XYYD54UMW37YDE7L62I3ZSYE3LAVCNFSM6AAAAABMFTYQRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBVGE2DSMJVHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

F. Sorry about that I was afk
Thanks for doing it
On Fri, Aug 23, 2024, 10:55 PM Benoit Maurin @.***> wrote:

By Monday I should get it done
On Thu, Aug 22, 2024, 6:13 PM Addisu Z. Taddese @.***>
wrote:

@unjambonakap https://github.com/unjambonakap any chance you can
respond to the feedback in the next couple days? Since gz-math is a lower
level library in the Gazebo stack, we'd like to wrap up PRs earlier to make
sure there is enough time to test them before the code freeze (August 28th).
—
Reply to this email directly, view it on GitHub
https://github.com/gazebosim/gz-math/pull/524#issuecomment-2305149159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6XYYD54UMW37YDE7L62I3ZSYE3LAVCNFSM6AAAAABMFTYQRCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBVGE2DSMJVHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I didn't address some of the feedback. Mainly, I think it would be good to add c++ tests.


Test for Calculator in Front-End Libraries projects is broken

Describe your problem and how to reproduce it:
The tests from the script that is pre-loaded in the Codepen template are wrong for the Front-End Libraries calculator project. The first few are fine, but from the middle of the list, several are repeats of each other and several from the project page are missing.
I’ve attached a screenshot so you can see that tests 7 and 8 describe exactly the same test, but one passes and one doesn’t.
All up, tests 7 and 8 in the feedback both describe user story 7, as shown on the project page. US8 is missing.
Tests 9, 10 and 11 describe US9. US10 and 11 are missing.
Tests 12, 13 and 14 describe US12 with 13 and 14 missing
Tests 15, 16 and 17 describe US15 - there is no corresponding US 16 and 17.
Add a Link to the page with the problem:
https://learn.freecodecamp.org/front-end-libraries/front-end-libraries-projects/build-a-javascript-calculator/
https://codepen.io/freeCodeCamp/pen/MJjpwO
Tell us about your browser and operating system:

Browser Name:  Chrome
Browser Version: Version 72.0.3626.121 (Official Build) (64-bit)
Operating System:  Windows 10

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

I've just gone and done some more digging. I ran the tests on a blank page and they look right. So, perhaps there's something in the code that I've written so far that is running interference.
So I checked my code again - thinking I could copy that here so you could see where the interference was BUT the tests look like they're working properly again.
The last thing I was fiddling with before I went to bed last night was a regex to test for an operator at the end of the input.
So, I figure at the very least you have a problem here with code that is vulnerable to being manipulated by the code it is testing. Hope that helps.

@tbushman Is this an issue with the tests or the camper's code?

Can you provide any more details about this @Beachhermit? - like what code you were using that was causing the issue?

Not really. I was having the problem, asked about it on the forum, was told to report it here, reported it, then it came good. The only clue I had was that I'd been working on a regex at the time to parse the input to the calculator and pick out operators from numbers. In the end, I went with a completely different system that processed input as it was entered.
I've been doing the d3 projects (which have their own problems with the test suite, that I just work around) for the last month or so, so all this is pretty distant memory and I don't save intermediate versions or anything super organised like that.

Thanks for reporting this but since we can't figure out what caused the problem or reproduce it - I think we can close it.


Regulation terms request for 'GO:0043131 erythrocyte enucleation'

Hi ,
Could I please ask for regulation terms ( particularly the positive one) for 'GO:0043131 erythrocyte enucleation' based on PMID: 25241935?
In Figure 5, the authors shown that Trim58 regulates nuclear extrusion during erythropoiesis.
Suggested definition:
Any process that modulates the frequency, rate or extent of nucleated precursor cells losing their nucleus during erythrocyte maturation.
Thanks,
Penelope

Trim58 depletion causes selective defects specifically during late-stage erythropoiesis, including reduced enucleation and increased formation of multinucleated cells.
'Figure 1. Trim58 Is Expressed during Late-Stage Erythropoiesis and Regulates Erythroid Maturation'


[draft]Handle guest logs

Fetch log content and save it to the workdir/provision/ List the log dir in guests.yaml
Pull Request Checklist

[ ] implement the feature
[ ] write the documentation
[ ] extend the test coverage
[ ] update the specification
[ ] adjust plugin docstring
[ ] modify the json schema
[ ] mention the version
[ ] include a release note

Updated^^,added an Optional log_path param to handle_guest_logs,in case we
want to let users specify the log dir in the future,though users can
specify --workdir-root.
On Tue, Jan 30, 2024 at 12:08 AM Miloš Prchlík @.***>
wrote:

@.**** commented on this pull request.
In tmt/steps/provision/local.py
https://github.com/teemtee/tmt/pull/2647#discussion_r1469827525:

@@ -133,6 +133,43 @@ def pull(
extend_options: Optional[list[str]] = None) -> None:
""" Nothing to be done to pull workdir """

def acquire_log(self, log_name: str) -> Optional[str]:

   """fetch and return content of a requested log"""

   if log_name == 'dmesg':

       return self.execute(Command('dmesg')).stdout

   return None

def store_log(

       self,

       log_name: Optional[str],

       log_path: Optional[Path],

Got it, I was thinking maybe users want to store the logs to dir they ran
tmt command:)
Maybe, that sounds like something users might do, but then again, it would
not be something for store_log() to decide, but for other parts of tmt,
e.g tmt.base.Run which implements tmt run: this class would decide where
to put logs, and it would task provision step with the right directory.
—
Reply to this email directly, view it on GitHub
https://github.com/teemtee/tmt/pull/2647#discussion_r1469827525, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKFR23AR6H3AJ5E723H3M2DYQ7CNXAVCNFSM6AAAAABCL3RX52VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQNBZGA2TOMRUGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


feat: TendstoUniformly.tendsto_of_eventually_tendsto

From PrimeNumberTheoremAnd.
Co-authored-by: Vincent Beffara<EMAIL_ADDRESS>

bors merge


Responsive pivot table not working in examples

When I view the example optimized for mobile devices, it does not work. On an iPhone it just shows the desktop version with pinch to zoom.

Oh nothing here is mobile-optimized or responsive in terms of layout... the
only mobile-related thing in the examples is the inclusion of a library to
handle touch interaction.
On Wed, Jun 28, 2017 at 01:11 Mark Traynor<EMAIL_ADDRESS>wrote:

When I view the example optimized for mobile devices, it does not work. On
an iPhone it just shows the desktop version with pinch to zoom.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nicolaskruchten/pivottable/issues/734, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAMbA3jGcvh3simVN3qrYKtgvjwan6otks5sIeB8gaJpZM4OHf3E
.


Test: running integration tests in a VM without ringbuffer

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

Project coverage is 36.85%. Comparing base (3c693b2) to head (da76a05).

:exclamation:  There is a different number of reports uploaded between BASE (3c693b2) and HEAD (da76a05). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (3c693b2)
HEAD (da76a05)

unittests
1
0

k8s-integration-test
1
0

integration-test
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1140       +/-   ##
===========================================
- Coverage   82.02%   36.85%   -45.18%
===========================================
  Files         141      140        -1
  Lines       11595    11586        -9
===========================================
- Hits         9511     4270     -5241
- Misses       1558     6888     +5330
+ Partials      526      428       -98

Flag
Coverage Δ

integration-test
?

k8s-integration-test
?

oats-test
36.85% <ø> (ø)

unittests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

This was an experiment and the 4.8 support task has been successfully completed by @rafaelroquetto .
Closing


alpha is not running

It appears that uwsgi is not running on the alpha server.
http://alpha.learningregistry.org/status
Returns 502 Bad Gateway error.

this is now fixed, it was a problem with the fabric script not working with the init.d scripts

Thanks Walt.


Upgrade react-native to 0.76

Problem
The new react-native version 0.76 just was released 🎉
https://github.com/facebook/react-native/releases/tag/v0.76.0-rc.0
It contains lots of bug fixes, improvements, and upgrades to the fundamental framework we are using.
Solution
Upgrade to react-native 0.76
cc @staszekscp @j-piasecki @WoLewicki - Per discussion in Slack, can you comment here so I can assign you to work on this? Thanks!

Sure! We’ll take care of that!

I'll be helping with the upgrade

I've asked people from Expo and SDK 51 is not compatible with RN 0.76 (it won't compile due to underlying changes). Given that, I believe it may be best to wait for the first beta release of SDK 52 (targeted at the first half of October) before we start the upgrade process. This way we don't have to hack around making SDK 51 compile on the current React Native version, which would have to be discarded anyway. What do you think?

This makes sense to me. cc @roryabraham @mountiny @luacmartins

Same

Agreed

agreed

Self-assigning as I am happy to help push this ahead in EU timezone

In progress


Migrate Create React App to Vite

@likui628 Awesome! Thank you for great PR! Could you tell me which source did you followed for setup swc + jest?

https://hung.dev/jest-vite

@likui628 Thanks!

@likui628 It's nice to know that my article helps.
Previously, I migrated CRA app to Vite for https://bestofjs.org. If you are interested, you can see more at the PR
Increase Developer Experience by replacing Create React App with Vite. I noted all the work I need to do to migrate from CRA to Vite in the PR description.
@ryota-murakami Hi Ryota, here we met again 😝.

@nvh95 Hey! That's your article LMAO 😄


PressListener's click() can call press without an event

Discovered in https://github.com/phetsims/phet-core/issues/103, the pressListener call in click() breaks the typing we've established. I'd like to discuss with @jessegreenberg how to handle, since our system may be more resilient to null events on presses than I anticipated.

We are currently relying on the ability to call PressListener.click() externally without an event in a couple of places. Perhaps we can revisit those cases if the PressListenerCallback requires an event. Or maybe we can create a fake SceneryEvent to pass through in those use cases.


Red Knot - Add symbol flags

Adds Symbol.flag bitfield. Populates it from (the three renamed) add_or_update_symbol* methods.
Currently there are these flags supported:

IS_DEFINED is set in a scope where a variable is defined.
IS_USED is set in a scope where a variable is referenced. (To have both this and IS_DEFINED would require two separate appearances of a variable in the same scope-- one def and one use.)
MARKED_GLOBAL and MARKED_NONLOCAL are not yet implemented. (TODO: While traversing, if you find these declarations, add these flags to the variable.)

Adds Symbol.kind field (commented) and the data structure which will populate it: Kind which is an enum of freevar, cellvar, implicit_global, and implicit_local. Not yet populated. (TODO: a second pass over the scope (or the ast?) will observe the MARKED_GLOBAL and MARKED_NONLOCAL flags to populate this field. When that's added, we'll uncomment the field.)
Adds a few tests that the IS_DEFINED and IS_USED fields are correctly set and/or merged:

Unit test that subsequent calls to add_or_update_symbol will merge the flag arguments.
Unit test that in the statement x = foo, the variable foo is considered used but not defined.
Unit test that in the statement from bar import foo, the variable foo is considered defined but not used.

It says three flags are supported, but it seems like just two?

That was a typo of "these". Sorry.
I've added a commit for the other variant of Kind.


Database migration error

-- Starting database migration from AdonisJS v4 to v5 --
Updating value from 'database/migrations/database/migrations/database/migrations/1503250034279_user' to 'database/migrations/database/migrations/database/migrations/database/migrations/1503250034279_user'
Updating value from 'database/migrations/database/migrations/database/migrations/1503250034280_token' to 'database/migrations/database/migrations/database/migrations/database/migrations/1503250034280_token'
Updating value from 'database/migrations/database/migrations/database/migrations/1566385379883_service_schema' to 'database/migrations/database/migrations/database/migrations/database/migrations/1566385379883_service_schema'
Updating value from 'database/migrations/database/migrations/database/migrations/1566554231482_recipe_schema' to 'database/migrations/database/migrations/database/migrations/database/migrations/1566554231482_recipe_schema'
Updating value from 'database/migrations/database/migrations/database/migrations/1566554359294_workspace_schema' to 'database/migrations/database/migrations/database/migrations/database/migrations/1566554359294_workspace_schema'
Updating value from 'database/migrations/database/migrations/database/migrations/1612629845398_users_update_schema' to 'database/migrations/database/migrations/database/migrations/database/migrations/1612629845398_users_update_schema'
❯ error database/migrations/1503250034279_user
❯ migrating database/migrations/1696110557648_jwt_tokens
[ error ]  create table "users" ("id" serial primary key, "username" varchar(80) not null, "email" varchar(254) not null, "password" varchar(60) not null, "settings" json, "created_at" timestamptz, "updated_at" timestamptz) - relation "users" already exists
    at Parser.parseErrorMessage (/app/node_modules/.pnpm/pg-protocol@1.6.0/node_modules/pg-protocol/src/parser.ts:369:69)
    at Parser.handlePacket (/app/node_modules/.pnpm/pg-protocol@1.6.0/node_modules/pg-protocol/src/parser.ts:188:21)
    at Parser.parse (/app/node_modules/.pnpm/pg-protocol@1.6.0/node_modules/pg-protocol/src/parser.ts:103:30)
    at Socket.<anonymous> (/app/node_modules/.pnpm/pg-protocol@1.6.0/node_modules/pg-protocol/src/index.ts:7:48)
    at Socket.emit (node:events:517:28)
    at addChunk (node:internal/streams/readable:335:12)
    at readableAddChunk (node:internal/streams/readable:308:9)
    at Socket.Readable.push (node:internal/streams/readable:245:10)
    at TCP.onStreamRead (node:internal/stream_base_commons:190:23)

Using Ferdium-Server via Docker with tag latest

Hello 👋 which tag were you using before updating?

Can you manually access the database and delete the database/migrations/database/migrations/database/migrations/database/migrations/ on all rows on the adonis migration table ? That seems to be the issue.
It should be just<PHONE_NUMBER>279_user and the script would add database/migrations/ by itself
If this is too technical and you don't mind DM me (on discord perhaps) the sqlite database I can fix it for you!

hmm 🤔 Looks like I don't have that table:
I'm using Ferdium with a PostgreSQL database.

It should be the adonis_schema table (I think)


Southworks/cosmosdb mocks

Added mocks callers to CosmosDBStorage tests

Pull Request Test Coverage Report for Build 155

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.9%) to 75.777%

Totals

Change from base Build 153:
1.9%

Covered Lines:
2393

Relevant Lines:
3055

💛  - Coveralls


EL-509 accessibility statement

https://dsdmoj.atlassian.net/browse/EL-509
Accessibility statement for launch - with text to indicate that the accessibility audit will take place just after our soft launch

Oh bugger I've done it again, I didn't spot you'd un-requested review


Use normal build script

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Closes #1

Pyproject can be quite temperamental so let's try this.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender


Re create landing page in React

This PR will

Move Assets from outside folder to assets folder
Remove Old Landing Page content
Add New Landing Page component
Fix Styling due to migration

Fixes Issue #16

I need some help regarding the font awesome icon.
Please check the attached screenshot.
At the bottom, there is a toggle button to toggle the theme dark/light.
When I click on the button the icon doesn't appear. Seems like it is not loading the icon on toggle. Though, on initial page load the icon appears. There is nothing wrong in the code as I tried inverting the logic so the Sun icon appeared.
So on initial page render, the default theme is light, hence Moon icon is visible but after toggle, the sun icon doesn't appear.

#16

@srivastavaritik, As per the previous implementation whenever a user visits the site. By default, user can see login page. I need help to understand what should be default route for landing page then? If user visits the site for the first time, I belive the user should be redirected to landing page and not login page.
Please let me know your requirements and thoughts 😄

hi @immayurpanchal , I will help you with both the problems that you faced. But at first please see that what styling is causing this

And also have a look at all the buttons on your pr and the ones at https://extraspace-cloud.netlify.app there is some slight issue. Please fix that

I need some help regarding the font awesome icon.
Please check the attached screenshot.
At the bottom, there is a toggle button to toggle the theme dark/light.
When I click on the button the icon doesn't appear. Seems like it is not loading the icon on toggle. Though, on initial page load the icon appears. There is nothing wrong in the code as I tried inverting the logic so the Sun icon appeared.
So on initial page render, the default theme is light, hence Moon icon is visible but after toggle, the sun icon doesn't appear.

This is Fixed!

hi @immayurpanchal , I will help you with both the problems that you faced. But at first please see that what styling is causing this  And also have a look at all the buttons on your pr and the ones at https://extraspace-cloud.netlify.app there is some slight issue. Please fix that

All styling related issues are fixed.

@srivastavaritik , Please let me know if I'm still missing anything as without having a concrete design it difficult to understand exact desired output of the home page. Though, I've tried my best to make it as good as possible. But I believe you're the right person to point out the mistakes 😃 . I would love to contribute more.

The issue was created because of the class names applied in the earlier were generic to bootstrap and bootstrap's default style was overwritten by the same class name. I've updated the class name so it won't clash with the default one.

@srivastavaritik , Please assign this PR to me. I would be really helpful if hacktoberfest & hacktoberfest-accepted label will be added.

@srivastavaritik , I'm able to create the build on the local. It's work fine for me. Nelify shows that module not found. Would you please check once on the Vercel?

The only remaining part in this PR is to finalise landing page route. Please let me know your thoughts and I'll update the logic accordingly.

The only remaining part in this PR is to finalise landing page route. Please let me know your thoughts and I'll update the logic accordingly.

for now, just let it be at /home route, don't add it anywhere
once I finalize it with proper content, I'll add it

ping me if you want this to be merged

ping me if you want this to be merged

Yeah, please.

@immayurpanchal please update this pr with latest commits

@immayurpanchal please update this pr

anyone wants to work on this?


Remove catkin from the project

@wxmerkt I have made this first proposal. If you need to make some other changes, you can push on this branch.

Related to #822

I'm just waiting for https://gepgitlab.laas.fr/stack-of-tasks/pinocchio/-/jobs to be done.

It seems there is a bug on 18.04. I will fix it but this is not related to this PR.
I will merge it now.


Adds dependency on github.com/grafana/jsonnet-libs mixins-util

Related Issue:  stolostron/backlog#<ISSUE_NUMBER>
Description of changes

/lgtm


[YUNIKORN-1334] Admission controller: look up user & group and add annotation if necessary

Note: this change has to go in order to stabilize the e2e tests for YUNIKORN-1334.


create domain ignoring NumberOfFilesLimited on WebServerLog

model extract:
topology:
    Server:
        myserver:
            WebServer:
                WebServerLog:
                    FileCount: 5
                    BufferSizeKb: 0
                    NumberOfFilesLimited: true

produces config.xml extract:
      <web-server-log>
        <name>myserver</name>
        <file-count>5</file-count>
        <buffer-size-kb>0</buffer-size-kb>
      </web-server-log>

Attribute is confirmed as being false in WLST:
wls:/mydomain/serverConfig> get('Servers/myserver/WebServer/myserver/WebServerLog/myserver/NumberOfFilesLimited')
0

Tested in WLS10.3.6. Setting the attribute works on wls12.1.3

It appears that WLST offline has a default of true for NumberOfFilesLimited, and omits this attribute in the config.xml for that case.  In WLST online, the default appears to be false, and is not updated if absent in config.xml.  Checking if there is a patch for this.

Note: This seems to be fixed in 12.1.3, offline omits the line in config.xml if the value is true, but online reads it correctly as true.

For this project (WDT), it does not appear that we can do anything to workaround this issue in WLST.  Please file a support case with Oracle Support for this WLST issue and list the versions that you need to be fixed.  For this particular case, the fix for newer versions appears to already exists and the support team will need to do a backport of the existing fix to the versions you require.

There is a work-around that may help, if it fits your process. If you run updateDomain (after domain create and start) with the online option and the same model, it should set the correct values for NumberOfFilesLimited and RotateLogOnStartup.  You could do this with a stripped-down model with just those elements if that's easier.

This issue has been nagging at me.  So, I did some more testing and research.  This "bug" has not been fixed and still exists in the current WebLogic product.  WLST offline does not have a way to address the fact that an attribute has multiple defaults.  Therefore, it is not possible with WLST offline to set the value of NumberOfFilesLimited to true for a production mode domain.  For WDT, the only way to get this value to true in production mode would be to use WDT online.  Please file a bug against WebLogic Server for WLST offline to fix this bug.
@53d117460ec63d70 This is why you see the values you mention in your post:
The MBean attribute NumberOfFilesLimited has two defaults, "false" for production mode, and "true" for development mode.  WLST online determines the default value (if unset) based on whether the domain is running in production mode or development mode.  WLST offline only knows of one default, the development mode default which is "true".  Based on the data you posted, you tried to create a production mode domain and set the value for NumberOfFilesLimited to true.  WLST offline decided that "true" was the default, and leaves the config.xml attribute unset.  Therefore, the config.xml does not have the element for this attribute.  After you start the server in production mode, WLST online queries that attribute and determines the value is unset and returns the default for production mode, which is false.

The suggested workaround https://github.com/oracle/weblogic-deploy-tooling/issues/303#issuecomment-476768240 gives the following error:
Caused by: oracle.weblogic.deploy.util.PyWLSTException: WLSDPLY-00072: JRF wlst method applyJRF not loaded with the executed WLST

I am running updateDomain.sh with the same arguments as createDomain.sh:
/home/rob/build/weblogic-deploy/bin/updateDomain.sh \
        -oracle_home /home/rob/oracle_home \
        -domain_home /home/rob/domain_home \
        -model_file /home/rob/build/mydomain.yml \
        -domain_type WLS

The problem with applyJRF has also been fixed and merged to master, and will be in release 0.21.


Allow control of nullability for argument types

Same as https://github.com/opendoor/pggen/issues/20 but I would also like support for statements not just queries

I can take a crack at this, though I'm not sure how fast I'll be.

@Bastiantheone I just pushed v0.3.3 that contains this fix if you want to pick it up by updating bazel configs and whatnot.


Fix the syntax error on tt-rss config file

I have used these changes in my server for a few days now, although I have only tested the passwordFile option.
Motivation for this change
A fix for https://github.com/NixOS/nixpkgs/issues/148694 fixing the syntax error in the PHP config file.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

When using unix authentication this generates invalid php:
putenv('TTRSS_DB_PASS=' . );

How are you configuring tt-rss to get that output? Omitting the password entirely?

Indeed no password. A password is not required as postgresql allows acess to the tt-rss database to the unix user of the same name.

Should the config.php omit the entire password setting or should it be set empty?
In effect, would this do the trick?
putenv('TTRSS_DB_PASS=' . '');

I think so as my current config file contains:
 putenv('TTRSS_DB_PASS=');

Added handling for unix password:

No password at all: putenv('TTRSS_DB_PASS=' );
Password explicitly set in nix: putenv('TTRSS_DB_PASS=' . 'foo');
Using a password file: putenv('TTRSS_DB_PASS=' . file_get_contents('/etc/nixos/system/services/ttrss/db'));


Status update

First, thank you for the work.
Second, I use Google Translate.
I am using the integration with Home Assistant, and I have detected that the temperature status is not updated. It is only updated when I enter the Airzone Cloud APP or modify the set point temperature from HA.
Thank you

Hello,
Thanks for you report, I found the issue, I will fix it soon.
Max

Hello,
You can download the last release, I fixed this bug :)
Max

I've been testing it and now if it updates.  Thank you.


feat: Add Sentry

Add Sentry as a plugin that is only initialized when SENTRY_DSN is set so that it only fires in production (or it can be used with another project in dev).
Wrap Lambda Handlers to capture any Lambdas crashing with Sentry.
Capture failed to decorate or update pool errors with Sentry.
fix: Fixed a bug where pools/tokens caches were being intermingled because they had the same cacheNamespace, gave them different namespaces now.

I've created a new Sentry project api for this project and have added the DSN to this repositories secrets. You can see some of the captured test errors from my dev account in the Sentry dashboard (I've disabled sending more errors by not including the DSN).

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1203070135001261

@alter-eggo good idea, there are a lot of lint issues, I'll make a separate PR for them.


Association name ending in 's' generates inconsistent method names

Originally reported on Google Code with ID 658

The following Umple snippet ...
class Qx {}
class Xs {}
association { * Qx qx -- * Xs xs; }
... generates the following code in class Qx (in Java):
private List xs;
public Xs getX(int index)
{
Xs aX = xs.get(index);
return aX;
}
You would expect the following instead:
public Xs getXs(int index)
{
Xs aX = xs.get(index);
return aX;
}
It appears the translator strips off the trailing s when generating methods that are
considered singular as opposed to plural. Also, this appears to be independent of the
multiplicities in question.

Reported by alexwithaneye on 2014-12-04 00:34:05

The problem seems to be for two-letter classes that end in 's'. =Class names should be singular so this case will be rare. However in cases where a singular class name ends in 's'  and the class is more than two letters long all seems fine. The following code generates OK, so I am closing this issue.
class Pass {
}


Initial commit of multiple python versions

This work still needs a fair amount of re-factoring but it's a start. Commiting it now so we can get some testing going.

The windows failure on python 3.11.1 are related to this:
https://github.com/python/cpython/issues/100320
I am testing to see if it's resolved in 3.11.2

3.11.2 works for windows.. turns out it's also working everywhere else! :)

This PR helps address #75

Adresses #7


Null safety and flutter 2

Please upgrade to support null safety and flutter2

I also need

already has a null-safe version will merge it soon to one


add test for AccumulativeSwingIndex

Awesome, thanks for this! :slightly_smiling_face:

@quadramadery yes, that's quite the rabbit hole to enter :joy: great idea though. A bit of a moon-shot


unsign HEAD-only (new formula)

This little tool will become very popular soon, as using it is the only currently known way of enabling 3rd party plugins in Xcode 8 (which is about to ship to a bazillion iPhone developers).
See https://github.com/alcatraz/Alcatraz/issues/475#issuecomment-225868817 for further discussion.

We don’t support HEAD-only formulae, sorry! You may want to support the formula in your own tap.


refactor(injector): make sure we call Function.prototype.toString not an override

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
refactor
What is the current behavior? (You can also link to an open issue here)
the injector uses fn.toString to extract function arguments
What is the new behavior (if this is a feature change)?
the injector uses Function.prototype.toString.call to avoid calling an override
Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:
use Function.prototype.toString.call instead of fn.toString, because the user might override the toString function of the "fn" instance, without knowing it's being used by the injector.

Hi @a510, the change looks fine, but a test would be mostly welcome.

and this should be marked as fix not as refactor


OCPBUGS-11042: Remove explicit any type

/assign @cyril-ui-developer

/jira refresh

/lgtm
Thanks for the fix. Actually, I was thinking of what type I could use instead of any. Make sense to remove them.


[PDI-18916] Backport of PDI-18375 - Implement streaming prefetch with…

… a parameter

:white_check_mark: Build finished in 30m 32s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
engine,plugins/streaming/impls/jms,plugins/streaming/impls/mqtt

:ok_hand: All tests passed!
Tests run: 3623, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message

Once I get a go ahead from @c-ferreira and @pentaho-lmartins  I will process the PR


Should be able to reverse the sort order in TicketList

It'd be pretty easy to reverse, and this makes dealing with older tickets easier.

Fixed by #7 -- though we'll see how we go with that, it might still make sense to do the reversing in code rather than via JQL DESC/ASC


Unable to obtain accuracies as mentioned in the paper.

I used the ImageNet dataset, split the same way as mentioned in the repo. Trying to reproduce EEIL and LUCIR results for 10 tasks in the shuffled LT case.

However, the best accuracy I could obtain for LUCIR and LUCIR_2stage were 44.29% and 47.73% respectively. For EEIL and EEIL_2stage the accuracies were 40.82% and 42.49%. (Without grid search)
In the reported accuracies, was gridsearch used? Is there anything that is missing in terms of setting to obtain the mentioned accuracies?

Thanks for your interest, could you please share me the log? The reported accuracies are the average accuracy over all tasks, not the last task. It might be the reason.

The gridsearch is not used.

 This would help understand the issue. Additionally, I used seed as 1993.

The accuracy reported is the average accuracy across all tasks. For example, the average incremental accuracy of LUCIR is 46.8+56.3+60.2+ ... + 41.3 = 48.7 and LUCIR_2stage is 53.4+ 59.2+...+46.1+44.9 = 52.1


colspan more than 2 is not working

Example
| a | b | c |
| - | - | - |
| > | > | 1 |

Result
1

Expect
1

I've released a new version to fix this.
https://github.com/wataru-chocola/remark-extended-table/releases/tag/remark-extended-table%401.0.1
Could you try it?
Thanks.

Fixed, thanks


Update dependencies per Dependabot surfaced PRs

Summary
Simple find and replace in csproj files.

Bump EF Core 6.0.5 --> 6.0.6
Bump EF Core SqlServer 6.0.5 --> 6.0.6
Bump EF Core InMemory 6.0.5 --> 6.0.6
Bump Microsoft.Data.Sqlite 6.0.5 --> 6.0.6
Bump Microsoft.Data.Sqlite.Core 6.0.5 --> 6.0.6
Bump System.Text.Json 6.0.4 --> 6.0.5

Closes #29876
Closes #29877
Closes #29878
Would have closed (but they were closed one-by-one):
#29852
#29853
#29864
#29865
#29845
#29846
#29847
#29848
#29849
#29850
#29851
#29854
#29855
#29856
#29857
#29858
#29859
#29860
#29861
#29862
#29863

CC @BillWagner I think you might be closing these as one-offs. Happy to update my PR here and knock the rest out in one shot if you're OK with it.

Where you find other examples, go ahead! Let me know when this is ready for a final review and merge.

@BillWagner it's ready 👍


error while build

vagrant@homestead:~/projects_local/playground/nette-webpack-example$  node -v
v14.15.3
vagrant@homestead:~/projects_local/playground/nette-webpack-example$
vagrant@homestead:~/projects_local/playground/nette-webpack-example$     npm run build

><EMAIL_ADDRESS>build /home/vagrant/projects_local/playground/nette-webpack-example
> cross-env NODE_ENV=production webpack --mode production --progress --colors

Invalid options object. Terser Plugin has been initialized using an options object that does not match the API schema.
 - options has an unknown property 'chunkFilter'. These properties are valid:
   object { test?, include?, exclude?, cache?, cacheKeys?, parallel?, sourceMap?, minify?, terserOptions?, extractComments?, warningsFilter? }
ValidationError: Invalid options object. Terser Plugin has been initialized using an options object that does not match the API schema.
    at validate (/home/vagrant/projects_local/playground/nette-webpack-example/node_modules/terser-webpack-plugin/node_modules/schema-utils/dist/validate.js:96:11)
    at new TerserPlugin (/home/vagrant/projects_local/playground/nette-webpack-example/node_modules/terser-webpack-plugin/dist/index.js:40:30)
    at Object.<anonymous> (/home/vagrant/projects_local/playground/nette-webpack-example/webpack.config.js:313:5)
    at Module._compile (/home/vagrant/projects_local/playground/nette-webpack-example/node_modules/webpack-cli/node_modules/v8-compile-cache/v8-compile-cache.js:194:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
    at Module.load (internal/modules/cjs/loader.js:928:32)
    at Function.Module._load (internal/modules/cjs/loader.js:769:14)
    at Module.require (internal/modules/cjs/loader.js:952:19)
    at require (/home/vagrant/projects_local/playground/nette-webpack-example/node_modules/webpack-cli/node_modules/v8-compile-cache/v8-compile-cache.js:161:20)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `cross-env NODE_ENV=production webpack --mode production --progress --colors`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!

Fixed.


Metrics for when partial aggregation mode is hit

Is your feature request related to a problem or challenge?
@korowa  added "skip partial aggregation mode" in https://github.com/apache/datafusion/pull/11627 which helps with high cardinality aggregates by doing minimal work for the first phase of the aggregation. This mode is triggered dynamically based on how effective the first aggregation phase is working.
However, it is not clear from the metrics (e.g. the output of EXPLAIN ANALYZE plans) when this mode
Describe the solution you'd like
I would like some sort of metric that can be used to determine when the partial aggregation phase has been invoked
Describe alternatives you've considered
I think the number of convert_to_state rows is a good metric -- 0 means no partial aggregation was done, and more than 1 gives the number of rows that were subject to the partial aggregate.
Additional context
No response

I have a draft PR for this here; https://github.com/apache/datafusion/pull/11706


Can't use on Windows

          In the case of using Mirrord (IntelliJ Plugin) on Windows, the plugin fives an error : `Can't use mirrord on Windows without WSL` as stated in #1544 even if I have WSL rinning on my machine.

Is the error message misleading and the feature is not supported at all on windows or I'm I missing something?
Originally posted by @GMouaad in https://github.com/metalbear-co/mirrord/issues/28#issuecomment-1640107658

Hi @aviramha , thank you for the followup. I'm using the initial (unchanged) configuration of the plugin and it looks as the following:
{
    "accept_invalid_certificates": false,
    "feature": {
        "network": {
            "incoming": "mirror",
            "outgoing": true
        },
        "fs": "read",
        "env": true
    }
}

I'm still dicovering MirrorD (and I find the value proposition pretty god btw.) though I didn't find a documentation section on what should be configured when running on windows using WSL.
Any hints?

You're welcome @GMouaad.
To use mirrord on Windows in IntelliJ you'd need the "Run configuration" to be configured to "Run on" Linux or open the project from the Linux path (/mnt/c$/path-to-proj usually). Can you provide the run configuration of IntelliJ you tried using?

Can you try opening the project from the WSL Path?

I tried open the project with the wsl path, but this is prohibited by windows. It tells you that you are not allowed to open this path in the program.

@aviramha , I am a colleague of @JohnnyFasching and we are using Jetbrains Rider 2023.3.3, with .NET/F# (should be same with C#)

hi @deyanp , either of you are up for a screen share session?
can schedule time here
https://calendly.com/aviram-5/30min

thanks @aviramha! Let us double-check again with these 2 options:

https://www.jetbrains.com/remote-development/gateway/
vs code supports already starting from WLS2

and get back to you for a screen sharing next week

hey @deyanp - did you get it sorted?


v2.0.0 release

https://github.com/ZeeCoder/container-query/milestone/1

Coverage remained the same at 100.0% when pulling 77bcc66752e66fae079de406d74866e8d59fbf30 on v2.0.0 into c3c0bdc9237199cdf89896380c02c732e5d47e2b on master.

Coverage remained the same at 100.0% when pulling 93ab98fc26a285a81c91589611fce40a503e3587 on v2.0.0 into c3c0bdc9237199cdf89896380c02c732e5d47e2b on master.

Coverage remained the same at 100.0% when pulling c5a0f7b1298cf442a30f7cc3e9b3bb1e64a2a2a0 on v2.0.0 into c3c0bdc9237199cdf89896380c02c732e5d47e2b on master.

Coverage remained the same at 100.0% when pulling d1b2a0a1208521e7cffb1a38e19fb7d28e841b1b on v2.0.0 into 2e00636e9f58e4577f2d2808817aa3c713bf44a8 on master.


Moved incorrectly merged buildscript

I did not notice that after rebase (after webcommon move), the corrected build.xml contents for libs.jstestdriver still landed in root. This fix just moves the content to the appropriate place in webcommon cluster dir.

Good timing, applied this, and was able to build Apache NetBeans 'basic' distro, with ${clusters.config.php.list} added to line 29 of cluster.properties.


Unable to build tikv Docker image from scratch

I am trying to build the TiKV docker image from scratch, I was successfully able to build the rust image, but i am running into compile issues when trying to build tikv
My Rust Version is rustc 1.29.0-nightly (4f3c7a472 2018-07-17)
Rustup Version is rustup 1.13.0 (ea9259c1b 2018-07-16)
Release Version that i have tried are 2.0.5 and 2.0.4
The error trace that i get is as follows:
error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
  --> src/util/buf.rs:14:5
   |
14 | use alloc::raw_vec::RawVec;
   |     ^^^^^^^^^^^^^^^^^^^^^^
   |
   = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
  --> src/util/buf.rs:27:10
   |
27 |     buf: RawVec<u8>,
   |          ^^^^^^^^^^
   |
   = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
  --> src/util/buf.rs:36:18
   |
36 |             buf: RawVec::with_capacity(capacity + 1),
   |                  ^^^^^^^^^^^^^^^^^^^^^
   |
   = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
  --> src/util/buf.rs:45:22
   |
45 |             self.buf.cap() - self.start + self.end
   |                      ^^^
   |
   = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
  --> src/util/buf.rs:51:40
   |
51 |         slice::from_raw_parts(self.buf.ptr(), self.buf.cap())
   |                                        ^^^
   |
   = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
  --> src/util/buf.rs:51:56
   |
51 |         slice::from_raw_parts(self.buf.ptr(), self.buf.cap())
   |                                                        ^^^
   |
   = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
  --> src/util/buf.rs:56:44
   |
56 |         slice::from_raw_parts_mut(self.buf.ptr(), self.buf.cap())
   |                                            ^^^
   |
   = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
  --> src/util/buf.rs:56:60
   |
56 |         slice::from_raw_parts_mut(self.buf.ptr(), self.buf.cap())
   |                                                            ^^^
   |
   = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
  --> src/util/buf.rs:61:18
   |
61 |         self.buf.cap() - 1
   |                  ^^^
   |
   = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:119:28
    |
119 |         let cap = self.buf.cap();
    |                            ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:120:18
    |
120 |         self.buf.reserve(cap, capacity + 1 - cap);
    |                  ^^^^^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:121:32
    |
121 |         let new_cap = self.buf.cap();
    |                                ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:131:37
    |
131 |                 let left = self.buf.ptr();
    |                                     ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:132:40
    |
132 |                 let new_pos = self.buf.ptr().offset(cap as isize);
    |                                        ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:137:38
    |
137 |                 let right = self.buf.ptr().offset(self.start as isize);
    |                                      ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:139:40
    |
139 |                 let new_pos = self.buf.ptr().offset(self.start as isize);
    |                                        ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:159:37
    |
159 |                 let dest = self.buf.ptr();
    |                                     ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:162:43
    |
162 |                     let source = self.buf.ptr().offset(self.start as isize);
    |                                           ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:178:47
    |
178 |                         let source = self.buf.ptr().offset(new_cap as isize);
    |                                               ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:183:39
    |
183 |                 let source = self.buf.ptr().offset(self.start as isize);
    |                                       ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:184:42
    |
184 |                 let right_len = self.buf.cap() - self.start;
    |                                          ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:189:48
    |
189 |                     ptr::copy(source, self.buf.ptr(), to_keep);
    |                                                ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:198:34
    |
198 |                         self.buf.ptr().offset(self.start as isize),
    |                                  ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:204:18
    |
204 |         self.buf.shrink_to_fit(new_cap);
    |                  ^^^^^^^^^^^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:235:28
    |
235 |         if end == self.buf.cap() {
    |                            ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:268:30
    |
268 |         if start == self.buf.cap() {
    |                              ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error[E0658]: use of unstable library feature 'raw_vec_internals': implemention detail
   --> src/util/buf.rs:304:38
    |
304 |             let right_len = self.buf.cap() - self.start;
    |                                      ^^^
    |
    = help: add #![feature(raw_vec_internals)] to the crate attributes to enable

error: aborting due to 27 previous errors

For more information about this error, try `rustc --explain E0658`.
error: Could not compile tikv.

Can anyone help me out with this issue ?, i am sorry if the above information is not enough, i am fairly new to tidb and rust.

adding #![feature(raw_vec_internals)] to src/lib.rs enabled the unstable feature in the nightly build. Sorry for the trouble, rookie mistkake


CRD tenancy

This PR adds CRD tenancy by ensuring that logical clusters are taken in account in:

CRD-related controllers
APIServices-related controllers
Discovery + OpenAPI endpoints

Main changes:

At the client-go level:

introduce a roundtripper in the lookpack config, to correctly set the cluster name (or wildcards) in the request header according to the type, and verb of the K8S request (only for CRD and APIService resource types)
Update the default informer KeyFunc (MetaNamespaceKeyFunc) to take the resource ClusterName into account in the created key, to ensure we can distinguish resources from different clusters with the same name when listing or watching with client-go using cluster wildcards

At the etcd storage level:

extend / complete the changes made by Clayton (for watches) to extend multi-cluster searches to LIST requests, and set the ClusterName correctly in the resulting object of  all types of REST requests.

At the controller level:

In both CRD-related controllers and APIService-related controllers, ensure that they take the clustername everywhere in the code, and don't mix objects from different clusters but with the same name,

At the generic API server level :

Add a clusterName parameter to the ListedPathProvider.ListedPath() interface function to allow filtering the listed paths per clusterName
Implement this filtering mechanism in several places
Replace the GenericApiServer.openAPIServices by an OpenAPIServiceProvider that allows managing openAPIServices and associated http handlers per clusterName.

... probably some other changes I don't think of now.

@smarterclayton Please ignore all the missing comments + bad format. I still have to fix those, but would like to give you the opportunity to review this PR technically, and validate (invalidate ?) the overall idea of the hacks.


Add molecule team chain .json

Let's go!

Confirming receipt of your RPC endpoint -- once your team whitelists the IP addresses from the confirmation email, you are all set for Game of Zones. 🚀

Closing in favor of the https://github.com/cosmosdevs/GameOfZones repo.


Feat: Avoid notification scrollbar if not needed

Motivation
Do not show the scrollbar for notifications of one line.
Changes

Set a min-height to the text equal to the icon size of the notification.

Screenshots

@peterpeterparker please review


Discuss when the user needs to login

To vote he must certainly sign that.

Right, currently Sovereign allows a user to see ballots without being logged in. She's required to log in only once she wants to interact (vote/delegate) with a ballot. If this behavior is appropriate for this milestone then this may be all we need.
Leaving this open for now as there are other UX flow questions that still need to be nailed down (#371 for example)

Yep.. rule of thumb: free speech always, to interact simply proof of stake (or web2 shit).

Switching non-code-specific issues to Basecamp


Feature Request: Events

Hi,
I know the library Android Signature Pad has the onStartSigning(), onSigned() and onClear() events.
Having this events will allow us to have more control over the user interactions.
I was wondering if you could implement this feature, at least for android users.
Thanks.

This would be really useful!


iOS 16.4

I'm not sure whether palen1x supports iOS 16 in the public beta yet. Could someone just tell me if it is?

16.4 is not a public beta anymore. If you mean 16.5, iirc it is not currently supported but 16.4-16.4.1 are since palera1n 2.0.0~beta6/palen1x 1.0.9.

Ok, thank you!

16.5 is supported i think


NPE in SpriteBox.handleEvent

I/flutter : ------------------------------------------------------------------------
I/flutter : -- EXCEPTION CAUGHT BY GESTURE LIBRARY ---------------------------------
I/flutter : The following exception was raised while dispatching a pointer event:
I/flutter : The null object does not have a getter 'iterator'.
I/flutter : NoSuchMethodError: method not found: 'iterator'
I/flutter : Receiver: null
I/flutter : Arguments: []
I/flutter : Event:
I/flutter : PointerMoveEvent(Point(744.5, 162.5))
I/flutter : Target:
I/flutter : SpriteBox
I/flutter : Stack trace:
I/flutter : #0      Object._noSuchMethod (dart:core-patch/object_patch.dart:42)
I/flutter : #1      Object.noSuchMethod (dart:core-patch/object_patch.dart:45)
I/flutter : #2      SpriteBox.handleEvent (<>/flutter/packages/flutter_sprites/lib/src/sprite_box.dart:227)
I/flutter : #3      BindingBase&Scheduler&Gesturer.dispatchEvent (<>/flutter/packages/flutter/lib/src/gestures/binding.dart:98)
I/flutter : #4      BindingBase&Scheduler&Gesturer._handlePointerEvent (<>/flutter/packages/flutter/lib/src/gestures/binding.dart:72)
I/flutter : #5      BindingBase&Scheduler&Gesturer._handlePointerPacket (<>/flutter/packages/flutter/lib/src/gestures/binding.dart:44)
I/flutter : #6      _dispatchPointerPacket (file:///b/build/slave/Linux_Engine/build/src/out/android_Release/gen/sky/bindings/hooks.dart:43)
I/flutter : ------------------------------------------------------------------------

@vlidholt

flutter_sprites is no longer in master.


⚠ GitLab Server has degraded performance

In fb25109, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6685 ms

Resolved: GitLab Server performance has improved in e578e28.


store camera position in url

thus it will remains the same with the url

did some work on this

currently save camera.position/camera.rotation

##issue

camera.position is fine
but controls copy the camera rotation in its own... so i need to save those phi/teta/etc... instead of camera.rotation


Add Extensions to ERC-1

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md
We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

The PR edits only existing draft PRs.
The build passes.
Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
If matching on email address, the email address is the one publicly listed on your GitHub profile.

you should add yourself in the author section of the header

you should add yourself in the author section of the header

done :)


Adds main to other-modules

I am using hspc-discovery and the file Spec.hs has the main function and is not a module. cabal-gild adds Spec to other-modules causing the error
tests/Spec.hs:3:8: error:
    File name does not match module name:
    Saw: _Main_
    Expected: _Spec_

Non-modules should not be added to other modules especially if the file is listed with main-is: Spec.hs

Can you exclude the Spec module from Gild's discovery?
-- cabal-gild: discover tests --exclude=tests/Spec.hs

Thanks for the tip of using the exact path.
I had tried exclude with glob patterns as specified in the README but the patterns do not seem to work.
-- cabal-gild: discover tests --exclude=**/Spec.hs
-- cabal-gild: discover tests --exclude=*/Spec.hs

<IP_ADDRESS>


Cool job! Add it to FasterLivePortrait to support Audio-driven video-to-video

Hello, I really appreciate your work. It's very interesting and fast. I have incorporated it into FasterLivePortrait, supporting audio-driven video-to-video, which is very cool.
https://github.com/user-attachments/assets/cc51e2a4-abf0-40a3-8849-2672bffa8abb

Hello, I really appreciate your work. It's very interesting and fast. I have incorporated it into FasterLivePortrait, supporting audio-driven video-to-video, which is very cool.
12.16.1.mp4

Cool! Thanks~


Grafana dashboard enhancement

Enhancement Proposal
Out of the listed (apache metrics)[https://github.com/canonical/wordpress-k8s-operator/pull/43#issuecomment-1427567340], it would be nice to have a Grafana dashboard that the SREs may need or rank the metrics in the order of importance.

@yanksyoon is this issue still relevant?

Created https://warthogs.atlassian.net/browse/ISD-2804 (ISD-2804) to keep track of this enhancement feature. @varshigupta12 @jdkandersson Please assign this to relevant person as discussed.


Embedded View Controllers

Hey guys, for the past week I’ve been having a real issue trying to get an embedded view controller to have a property injected into it by the container.
All the VCs exist in the storyboard. They are subclasses of UIVC.

I have the initial VC initialised programmatically in AppDelegate. This is great as it is resolved from the container.
The container view with an embed segue to VC2 does not automatically have its properties resolved.

Can the embedded VC2 have its properties resolved as well, even if it is also designed in the storyboard? I must be d

Sorry for cutting off - on mobile!
I must be doing something wrong. The examples provided are very simplistic - is there any chance you could provide an example bit of code for this registration working, if possible, for the embedded VC?

Correct me if I'm wrong: you have two view controllers - main and embedded one - both designed in the storyboard. Main one you registered and resolved using basic Swinject approach, and second one you expect to be injected via SwinjectStoryboard's storyboardInitCompleted?
If that is the case it is possible that there is some unexpected behaviour / bug, as I don't think such setup is very common. Can you provide some extract from the registration / resolution code, and maybe a storyboard screen shot?

Hey @jakubvano, this is correct, I've just create a simple project to test it and it looks like it doesn't work when the first view controller is registered by myself manually.
Here's two view controllers:
class VC1: UIViewController {
    public var test: Test?

    override func viewDidLoad() {
        super.viewDidLoad()
        print(test?.getX())
    }
}

class VC2: UIViewController {
    public var test: Test?

    override func viewDidLoad() {
        super.viewDidLoad()
        print(test?.getX())
    }
}

struct Test {
    let x = arc4random()

    func getX() -> Any {
        return self.x
    }
}

Here's the container registration and programmatic initialisation of the initial view controller in AppDelegate:
let c = Container()

        c.autoregister(Test.self, initializer: Test.init)

        c.storyboardInitCompleted(VC1.self) { r, c in c.test = r.resolve(Test.self) }
        c.storyboardInitCompleted(VC2.self) { r, c in c.test = r.resolve(Test.self) }

        self.window = self.window ?? UIWindow(frame: UIScreen.main.bounds)

        let vc1 = UIStoryboard(name: "Main", bundle: nil).instantiateViewController(withIdentifier: "VC1") as! VC1

        self.window?.rootViewController = vc1
        self.window?.makeKeyAndVisible()

        return true


Dimlet Workbench: Infuse Status lost when broken while extracting dimlets

In the Infinity 1.2.0 modpack, when breaking a Dimlet Workbench with a pickaxe while it is extracting a dimlet, it loses all infused shards (it returns as an item without the infused tooltip, when you put it into the infuser it then shows 0%).

This is not really a bug and happens with all rftools machines that are infused. But perhaps I should make sure that this does not happen as it can be somewhat annoying.

Fixed, in the next version of rftools all machines will remember ALL their settings (including infusing bonus) when breaking them with a pickaxe even.


trivial misprint

Thanks for this list. Here I fix a trivial misprint in a section title.

Thanks!!


Get error "undefined method `tags' for nil:NilClass (NoMethodError)" on ubuntu when run command: cucumber BROWSER='chrome-desktop-headless'

👓 What did you see?
Get error "undefined method `tags' for nil:NilClass (NoMethodError)" on ubuntu when run command: cucumber BROWSER='chrome-desktop-headless'
📦 Which version are you using?
source "https://rubygems.org"
gem 'allure-cucumber', '2.13.6'
gem 'browser'
gem 'cucumber', '4.0'
gem 'parallel_tests', '3.0'
gem 'rake'
gem 'require_all'
gem 'rspec'
gem 'rspec-expectations', '~> 3.3.0'
gem 'rubocop'
gem 'selenium-webdriver',         '3.141'
gem 'selenium_remote_control',    '~> 1.0', '>= 1.0.4'
gem 'rest-client',                '~> 2.0'
gem 'net-http-persistent',        '~> 3.0'
gem 'faker',    '2.11'
gem 'cuke_modeler', '3.0'
Steps to reproduce the behavior:

Trigger github actions
actions start run and execute the command in actions: cucumber BROWSER='chrome-desktop-headless'

Well cucumber BROWSER='chrome-desktop-headless' seems not to be a valid command.
If you want to run cucumber with the BROWSER env variable set with chrome-desktop-headless, you should run the following instead: BROWSER='chrome-desktop-headless' cucumber
And actually I see in your screenshot that you already have an BROWSER: chrome-headless env in your job.
What are you trying to achieve? Could you make sure your workflow and your command are right?

Well cucumber BROWSER='chrome-desktop-headless' seems not to be a valid command.
If you want to run cucumber with the BROWSER env variable set with chrome-desktop-headless, you should run the following instead: BROWSER='chrome-desktop-headless' cucumber
And actually I see in your screenshot that you already have an BROWSER: chrome-headless env in your job.
What are you trying to achieve? Could you make sure your workflow and your command are right?

Thank you for reply.
I just tried your command : BROWSER='chrome-desktop-headless' cucumber
still get the same error, this error should not be caused by the command format, since it runs pass on ubuntu one month ago, as you can see below.
And no matter which command i run on local, both works fine; in addition, BROWSER: chrome-headless env can be deleted, it doesn't matter.
I guess it should be related to the cucumber version installed on ubuntu, since I see someone else met this issue, and say this is a cucumber bug: https://community.smartbear.com/t5/Cucumber-Open/feature-is-coming-up-nil-in-query-rb/td-p/202827

Unfortunately this information isn't enough for us to reproduce this. Could you create an SSCCE / MVCE, so we can reproduce this.
The best form is in the form of a github repo, as then we just pull down the code and run it.

In adition to cucumber 4, I noticed in the link you submit to the smartbear community forum that it seems your are still using ruby 2.3.
ruby 2.3 and cucumber 4 are very old version for ruby and cucumber and are not maintain anymore for a long time now. You should consider upgrading ruby and cucumber to latest version before anything else.

Thank you for reply. I just tried your command : BROWSER='chrome-desktop-headless' cucumber still get the same error

Do you have more info? Did you tried it on your own computer or in the CI?

this error should not be caused by the command format, since it runs pass on ubuntu one month ago, as you can see below.

If a command is not well formed, it can cause issues 🤔
And the picture you submit does not show the same command as you are trying to use today.

And no matter which command i run on local, both works fine; in addition, BROWSER: chrome-headless env can be deleted, it doesn't matter.

So, please make sure to submit a decent issue with a minimal reproducible example. If the env is not relevant, then do not make it part of the issue

I guess it should be related to the cucumber version installed on ubuntu,

cucumber is not installed on ubuntu. This is not a "ubuntu" thing. This is a ruby gem. It depends on your project, and on your version of ruby. Not on ubuntu

since I see someone else met this issue, and say this is a cucumber bug: https://community.smartbear.com/t5/Cucumber-Open/feature-is-coming-up-nil-in-query-rb/td-p/202827

This is 2 years old and was based on really old version of ruby and cucumber. This is certainly not relevant anymore.
As you are yourself using a pretty old version of cucumber, first thing would be to update it. If a bug has been found 2 years ago, it may have been fixed in more recent version of cucumber.
As @luke-hill said: please try to update cucumber and ruby, and to submit a clear, neat and comprehensive minimal reproducible example. At the moment there is nothing we can do to help.

yep, I thought it's due to cucumber's version too old, and tried 5.0, 6.0, 7.0 and so on, but keeps get below error when the version is not 4.0. So I guess maybe it is limitation on ubuntu, that cannot support enough versions of cucumber.

This is not related to ubuntu.
Please look at the error message: here it says that it has not been able to find a compatible version due to non compatible dependencies in your gemfile. Nothing to do with cucumber here.

Also, your "step to reproduce" actually do not help us to reproduce your issue.
We have not the capability to trigger your own github workflow.
Are you a ruby developer?
What are you trying to achieve here? Are you responsible for your project tests? Or maintenance?

New learner on ruby and cucumber.
The ruby- 2.7.5 cucumber-4.0.0 version works fine on my local, see below:

Now I have design about 20 automation cases locally, so I would like to integrate with CI.
The cucumber command run fail on ubuntu after CI triggered

Stop putting BROWSER='chrome-desktop-headless' after your cucumber command!
This is not save and error-prone. If you need to set that env variable, put it before the cucumber command, or in your environment, or in a .env file, or whatever, but it has nothing to do after the cucumber command
Also, please upgrade to the last version of cucumber. cucumber 4 is old and not maintained anymore.
We won't be able to help you more until you use a proper command line instruction, and you use a more recent cucumber.

Thank you all the the reply.
Update ruby to 3.0 and cucumber to 7.1. solve this problem.

Just to prevent other from being misled: no, this is not a compatibility issue between cucumber-ruby and ubuntu.
@IrisLiu001 I don't know how you conclude such compatibility issue but I assure you this is not the case. Cucumber-ruby is a ruby gem in plain ruby. There may be issues based on operating systems, but that was definitely not the case here, and nothing point in that direction.
It was most likely an issue in a previous cucumber version that has been fixed in more recent versions.


Microbosh examples should mention HM user

e.g. http://docs.cloudfoundry.org/bosh/deploy-microbosh-to-aws.html
The manifest on this page is missing a section to configure HM to talk to the director. A bosh user needs to be created manually for this to work; i.e. using bosh create user hm.
This needs to be configured correctly for the resurrector and agent metrics reporting to work correctly.
This process is similar to replacing the built in an admin account, which also doesn't appear to be described anywhere at this time.

Since we are moving towards bosh-init, I've update those docs (e.g. http://bosh.io/docs/init-aws.html).