Reactive UI (Razor / JSX)

Mostly a pipe dream, but it would be nice if you could use razor pages from csharp to build out reactive rmlui. For lua a custom jsx-like language would be really nice.

Hi @trevis,
I'd love to help implement reactive UI for Chorizite using Razor pages or a custom JSX-like language for Lua. This sounds like an exciting feature!
Before I dive in, I have a few questions to make sure I'm aligned with the current implementation and requirements:
Razor Pages with RMLUI:
    Are you looking to integrate Razor pages directly with RmlUI to build the UI reactively from C#?
    Will the UI updates be bidirectional (i.e., changes in the UI reflected in C# logic, and vice versa)?
    Is there any existing framework or component in place that could be extended for this, or would I need to start from scratch?

JSX-like Language for Lua:
    Do you have any specific ideas or requirements for how this custom JSX-like language should work?
    Will this language be used in conjunction with RmlUI to generate UI components, or is it meant for other tasks as well?

Reactive Model:
    Do you already have a reactive model in place for handling state changes and UI updates, or would this need to be implemented as part of this feature?
    How do you envision the binding between the data model and UI elements to be structured?

UI Rendering:
    Will this new reactive UI approach need to support complex elements (e.g., animations, dynamic updates) that aren’t fully supported by the current system?
    How should the integration with the existing UI system (RmlUI) work to ensure backward compatibility?

Once I have a better understanding of these points, I can start planning the implementation and contribute to this feature.
Looking forward to collaborating on this!
Best,
Gerson Jr.


Fixes for rc4

Fixes for rc4

The second commit undoes the work to test e3sm_to_cmip in its own environment. the implementation was enough for ad-hoc testing, but is not robust enough for production use. In particular, e3sm_to_cmip_environment_commands wasn't set to the ts environment_commands by default, meaning it would always have to be set.
A more robust solution is giving e3sm_to_cmip its own zppy task, as described in #467.


Mutagen sync issues when running more than one DDEV site at the same time

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
No response
Expected Behavior
That I can cleanly run two DDEV instances with ddev config global --mutagen-enabled without Mutagen sync issues
Actual Behavior
Mutagen reports errors when I start a second DDEV site when one is already running.
Steps To Reproduce
No response
Anything else?
I will attempt to provide more details and get this to fail again soon.

Hi @davereid-pfg - thanks. But everybody is running multiple projects with mutagen. So please do add the link to the gist and give specific instructions.
Also it would be kind if you would test with current HEAD, because there are several mutagen changes there. brew unlink ddev && brew install --HEAD --fetch-head ddev

Closing this for now. When you come back with info, we can reopen.
Also note that there is significant work in current HEAD on mutagen, including a new version. Would love to have you try it out. brew unlink ddev && brew install --HEAD --fetch-head ddev.


Expose Indexes::positions to C/C++/Python

This is required to efficiently deal with gradients/values and might help writing new calculators in Python

This was not needed for what I wanted to do, so I'll close the PR but keep the code around in case it is needed later


Apple App Store guidelines.

The lib is cool. However, I'm still concern about Apple approval for this library when submitting app for review. As described In License agreement and section 2.5.2 of AppStore Review Guidelines, an interpreted code may be downloaded to application but may not change the primarily purpose of the app nor changing the whole app binary structure.

Yep, this lib operates in a bit of a gray area within the App Store guidelines, you should use it at your own risk.


Longest Common subsequence in Python

Is your feature request related to a problem? Please describe.
Yes, I would like to work on this algorithm.
Describe the solution you'd like
As it is still not coded in Python so I would like to code it in python.
@Kumar-laxmi  Kindly assign me with this task, I will do my best .
Under SSOC 23

Assigned!
@Niissh : Python

Assigned! @Niissh : Python

Thank you for your kind consideration

@Kumar-laxmi i sent pull request why was it reported as spam kindly tell

so i can improve it the next time

@Niissh plz assign this to me.


Fixes #413 — Blurry pins on iOS 8.3+

The stop icon factory was being instantiated way too early in the application's lifecycle. Loading images too early caused the UIScreen's scale factor to not be properly set (it was reporting as 1.0 on a retina device). In general we shouldn't really be overriding the application delegate's init method, as this is run before the underlying infrastructure is even spun up (run loop, window, etc.). There must've been a change in the UIScreen initialization sequence in iOS 8.3.
Additionally, the OBAStopIconFactory was unnecessarily caching the UIImage objects for each icon. [UIImage imageNamed:] is already cached by the system.
There's some additional clean-up that can be done here — using enums for the different stop types, using a singleton instead of a factory (that is essentially what is happening anyway), etc. I'll submit another PR for those once this is merged.

awesome, thanks!


Use of Cyclius god affects bonus income calculation for all upgrades and buildings

Seems to be an issue with the following code:
var godLvl = Game.hasGod('ages');
if (godLvl == 1) mult *= 1 + 0.15 * Math.sin((Date.now() / 1000 / (60 * 60 * 3)) * Math.PI * 2);
else if (godLvl == 2) mult = 1 + 0.15 * Math.sin((Date.now() / 1000 / (60 * 60 * 12)) * Math.PI2);
else if (godLvl == 3) mult = 1 + 0.15 * Math.sin((Date.now() / 1000 / (60 * 60 * 24)) * Math.PI2);
The time difference between when the game calculates total cps and the sim calculates cps means every upgrade will have slightly inflated numbers. Clearly evident on upgrades that should have 0 bonus income like the elder covenenant and mouse upgrades. Hovering over the mouse shows it changing over time.

Oops, yea, I know what's wrong.

For a second, I thought I overlooked your fix on it, but nope, you have the same problem too, lol

Fixed in Version 2.0034.2

That should do it, let me know if it doesn't!

I was a about to post the same issue (detected it in kitten upgrades which showed incredibly high income increase). New release has fixed it, thanks!


Added successful flag to page publish operation

Coverage remained the same at 82.604% when pulling 20f4584a17903c78682db8d7600769d82e003e5e on czpython:feature/cleanups-for-moderation into b252f53d1ab38cf2d5947ec9a703982464c306bf on divio:release/3.4.x.

Coverage remained the same at 82.604% when pulling a8c054ea40d71bab1cd12d4405d301f754e088af on czpython:feature/cleanups-for-moderation into b252f53d1ab38cf2d5947ec9a703982464c306bf on divio:release/3.4.x.


Change the section headers

The section headers are a little opaque, so make these changes:

"Questions about you" stays the same
"Current employment" changes to "Your current employment"
"Employment history" changes to "Your employment history"
"Current job" changes to "Your working practices"
" Final questions" stays the same

done


regenerate artifacts

API changes have been detected in azure-synapse-artifacts. You can review API changes here

/check-enforcer override


3.0.0 modal components need unique segment

Short description of the problem:
Prior to ionic-app-scripts 3.0.0 lazy loaded modals just needed an IonicPage decorator. With ionic cli v3 I now have to add a segment or name for them.
What behavior are you expecting?
I would like the old ability to not have a name or segment. This allowed the url to stay the same as the current page on desktop website.
Steps to reproduce:

Create 2 modal components with the below setup.
@IonicPage()
@Component({
templateUrl: 'test.modal.html',
})

Run prod build
node --max_old_space_size=4000 ./node_modules/.bin/ionic-app-scripts build --prod

Get following error
Error: There are multiple entries in the deeplink config with the segment of
test.component

Which @ionic/app-scripts version are you using?
The above scenario worked on app-scripts ^2.1.3
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

This seems like a bug. Can you show me in a sample project?
Thanks,
Dan

Hey Dan,
Here is a small example repo.
https://github.com/ztecharoberts/ionic-modal-issue

Found this issue and this possible solution:
https://forum.ionicframework.com/t/there-are-multiple-entries-in-the-deeplink-config-with-the-segment-of-main-error-after-latest-update/107111
Any suggestions? Thanks.

having the same issue. but my use case is with list and details type pages where when clicking in the detail page and refreshing should take back to the list. this worked because they had the same segment. if this is a bad practice, what is the correct way to achieve navigation to the detail view being just "in memory"?

while this is addressed a workaround is to add a / at the end of the segment. this fools the check into thinking they're different while the router treats them as the same. so something like page-segment becomes page-segment/. but again THIS IS A HACK not a solution to the bug


Use security requirement names from spec

Pull Request Test Coverage Report for Build 131

7 of 8 (87.5%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 87.213%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

restdocs-openapi-gradle-plugin/src/main/kotlin/com/epages/restdocs/openapi/gradle/OpenApi20Generator.kt
7
8
87.5%

Totals

Change from base Build 129:
-0.03%

Covered Lines:
1064

Relevant Lines:
1220

💛  - Coveralls


Update CanKolay3499/CNvim to OxygeNvim/OxygeNvim

Repo URL:
https://github.com/OxygeNvim/OxygeNvim
Checklist:

[x] The plugin is specifically built for Neovim, or if it's a colorscheme, it supports treesitter syntax.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] The title of the pull request is Add/Update/Remove `username/repo`  (notice the backticks around `username/repo`) when adding a new plugin.
[x] The description doesn't mention that it's a Neovim plugin, it's obvious from the rest of the document. No mentions of the word plugin unless it's related to something else.
[x] The description doesn't contain emojis.
[x] Neovim is spelled as Neovim (not nvim, NeoVim or neovim), Vim is spelled as Vim (capitalized), Lua is spelled as Lua (capitalized), Tree-sitter is spelled as Tree-sitter.
[x] Acronyms should be fully capitalized, for example LSP, TS, YAML, etc.

Thanks for the PR!


Improve log-API performance

Purpose

Optimize log-API performance

Fixes #
https://github.com/ballerina-platform/ballerina-lang/issues/23461

Codecov Report

Merging #23530 into master will increase coverage by 4.42%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #23530      +/-   ##
==========================================
+ Coverage   14.59%   19.02%   +4.42%
==========================================
  Files          51       56       +5
  Lines        1411     1540     +129
  Branches      219      238      +19
==========================================
+ Hits          206      293      +87
- Misses       1189     1231      +42
  Partials       16       16

Impacted Files
Coverage Δ

composer/packages/tracing/src/DetailView.tsx
72.72% <0.00%> (ø)

composer/packages/tracing/src/TraceList.tsx
94.44% <0.00%> (ø)

composer/packages/tracing/src/TraceLogs.tsx
70.37% <0.00%> (ø)

composer/packages/tracing/src/index.ts
0.00% <0.00%> (ø)

composer/packages/tracing/src/ToolBar.tsx
71.05% <0.00%> (ø)

Continue to review full report at Codecov.

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


I got ECONNREFUSED error running from npm start despite Chrome is running

I got ECONNREFUSED error running from npm start despite Chrome is running.
Uncaught exception
{ Error: connect ECONNREFUSED <IP_ADDRESS>:34876
    at Object._errnoException (util.js:1041:11)
    at _exceptionWithHostPort (util.js:1064:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1153:14)
  code: 'ECONNREFUSED',
  errno: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 34876 }

I tried this one on my local Windows machine (it was fine), and then tried to run it on my DigitalOcean machine, but every time my express server redirects request to the Rendertron (or I request rendering from its UI) I get the error above.
Running Chrome version is:
google-chrome -version
Google Chrome 61.0.3163.100

The machine is running Ubuntu 16.04.
Maybe some setup hacks for just plain installation? Not dockerized one? Cause there are some instructions in README but for Docker image only, how to apply same fixes for non-dockerized version? Docker is heavy and my tiny machine will not survive its pressure.
Please help :) I really need this stuff :)
BTW, for some reason If I try to redirect renders to the https://render-tron.appspot.com/ instead of my local one (since I couldn't get it up and running) it doesn't render anything saying that it's unable to render URL :( Is it possible to use this endpoint from other IPs or whatever? :)  Cause I have very low demand on site, so this even non-stable instance will be enough for me.

looks like the same problem as my last reply in this post
https://github.com/GoogleChrome/rendertron/issues/90
if so, it's the problem of chrome launcher.
you can try to either

set the pollInterval to 10000 in chrome-launcher.js

or

update the chrome-launcher dependency in Rendertron to 0.8.0 (Currently is 0.3.x)

Though I tried both. and still failed. But some people are saying this fixed the problem.

@hekod777 chrome-launcher.js? Where I can find it. He is not in the repo.

Good.
We are now on the same boat.
I posted this issue on chrome-launcher github page 4 days.

PS: if the performance issue of dockerized Rendertron can be fixed, it will be the best. then I don't have to use this npm start work around ~

@RIP21: add your site to https://github.com/GoogleChrome/rendertron/issues/113. It can be used for other sites, but I think I'm seeing some issues with the running instance right now.

It sounds like chrome-launcher only supports some environments and maybe not the DigitalOcean one that you mentioned? The only real dependency is Chrome, so if you have Chrome, maybe we can configure chrome-launcher to point to exactly where it is.
npm start should be fine as long as it can find Chrome.

It can find Chrome and it even says in which port it was launched, but then failed to succeed.

@samuelli about my site that was unable to be loaded via your instance, it was a no detection of the page load for some reason.  I fixed that using code that is in the Chrome using stupi 5 second timeout to be sure it loaded :)

the problem of chrome launcher is the debugger
isDebuggerReady(){}
you can find more details about this issue here
https://github.com/GoogleChrome/chrome-launcher/issues/30

Found something that might help us to get rid of chrome launcher
linke:  https://developers.google.com/web/updates/2017/06/headless-karma-mocha-chai

Creating your own Headless Chrome launcher
The ChromeHeadless launcher is great because it works out of the box for testing on Headless Chrome. It includes the appropriate Chrome flags for you and launches a remote debugging version of Chrome on port 9222.
However, sometimes you may want to pass custom flags to Chrome or change the remote debugging port the launcher uses. To do that, create a customLaunchers field that extends the base ChromeHeadless launcher:
karma.conf.js
module.exports = function(config) {
...
config.set({
browsers: ['Chrome', 'ChromeHeadless', 'MyHeadlessChrome'],
customLaunchers: {
  MyHeadlessChrome: {
    base: 'ChromeHeadless',
    flags: ['--disable-translate', '--disable-extensions', '--remote-debugging-port=9223']
  }
},

}
};

Actually with the newest version of Rendertron, I am able to run Rendertron on an EC2 instance.

@hekod777: is that without the docker container then?

@samuelli
Yes, without docker container.

I was having this issue running with npm start and now I solved it.
I've tried the modified version of isDebuggerReady() made in chrome-launcher.js by @hekod777 at https://github.com/GoogleChrome/chrome-launcher/issues/30 and I can see that the Port is being assigned.
Also i have: "chrome-launcher": "^0.8.0",
net.createConnection(this.port) is rejecting the connection. WHY?
Also, I've tried to increase the pollInterval, with no success.
It's a dedicated server with Ubuntu 16.04 and Plesk and Google Chrome 63.0.3239.108
What I did to solve it is to add the '--no-sandbox' to the chromeFlags at main.js, I mean:
const appPromise = chromeLauncher.launch({
chromeFlags: ['--headless', '--disable-gpu', '--remote-debugging-address=<IP_ADDRESS>', '--no-sandbox'],
And solved!!!

It's not fair to say no sandbox is bad and not provide a solution


event.json should really be script.yml

Confusing to use event.json for people not familiar with Lambda/Serverless. Use script.yml so comments can be included.

We're now using yaml.


Release v9.0.0

Tracking issue for the v9.0.0 release. Now that v1.12 has been released, we should release client-go v9.0.0 too.
TODO:

[x] Update CHANGELOG and README: https://github.com/kubernetes/client-go/pull/466
[x] Smoke test with https://gist.github.com/nikhita/183c20c8dd1a34b4ba04ef38b244d67b
[x] Create the tag/release.
[x] Link the changelog on the releases page: https://github.com/kubernetes/client-go/releases/tag/v9.0.0.
[x]  Fix the tag title using GitHub UI

Retagging:

[x] Smoke test with https://gist.github.com/nikhita/183c20c8dd1a34b4ba04ef38b244d67b
[x] Create the tag/release.
[x]  Fix the tag title using GitHub UI

/cc @sttts @caesarxuchao

There were some commits missing in the tag earlier. Now,

https://github.com/kubernetes/publishing-bot/pull/116 fixed the commits and the tag has been published again.

Ran the smoke tests using https://gist.github.com/nikhita/183c20c8dd1a34b4ba04ef38b244d67b. Logs of the test: https://gist.github.com/nikhita/75fde17c537349815cb1cc2980b01086

Manually verified that commits are fine.

Also manually verified that git checkout v9.0.0 && godep restore && go build ./... runs successfully after a fresh install. (here, v9.0.0 points to the tag on my fork)

https://github.com/kubernetes/client-go/releases/tag/v9.0.0

This is finally done! :tada:


[Feat]Terms page for the website.

Description
We need to implement a "Terms" page for the website to ensure users are informed about our terms of service and policies. This page will detail the terms and conditions for using the site, including usage rights, legal disclaimers, and data privacy agreements.
Proposed Solution:
Create a dedicated "Terms" page that can be easily accessed from the website footer. This page should:
Display the terms of service in a clear and organized manner.
Include sections for:
General terms and conditions
User responsibilities
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hello @Vaishh18, this is a very good feature, but we are converting the entire code into the Next.js framework for easier backend integration. Once the Next.js conversion is complete, I will let you know so you can work on that issue.

Actually we're getting very close to 100 stars!  Would you mind giving our repository a star?  ⭐


aarch64 builds fail: extremely laggy filesystem

https://ci.adoptopenjdk.net/view/Failed Builds/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-linux-aarch64-openj9-linuxXL/397/console
09:07:25  [ 59%] Building CXX object runtime/gc_modron_standard/CMakeFiles/j9modronstandard.dir/StandardAccessBarrier.cpp.o
09:15:43  wrapper script does not seem to be touching the log file in /home/ubuntu/workspace/build-scripts/jobs/jdk11u/jdk11u-linux-aarch64-openj9-linuxXL@tmp/durable-63690d87
09:15:43  (JENKINS-48300: if on an extremely laggy filesystem, consider -Dorg.jenkinsci.plugins.durabletask.BourneShellScript.HEARTBEAT_CHECK_INTERVAL=86400)

Node:
08:42:28  All nodes of label ‘build&&linux&&aarch64&&dockerBuild’ are offline
08:43:32  Running on EC2 (adopt_aws) - Dynamic Linux aarch64 VM provisioned from AWS (i-0fbd7d9f07a7b18f3) in /home/ubuntu/workspace/build-scripts/jobs/jdk11u/jdk11u-linux-aarch64-openj9-linuxXL

It looks like this failure started from 17th Oct...

The common theme seems to be the job fails after running exactly 30mins ... ?

At some point in thie middle of this afternoon this seems to be ok. All affected builds during the time period appeared to be aborting after 30 minutes - no obvious reason why and we have not knowingly taken any remidial action on it. I will close for now and reopen if it recurs.

Problem still occuring: https://ci.adoptopenjdk.net/view/Failed Builds/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-linux-aarch64-openj9/

Above PR made no difference - https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/build-test/job/jobs/job/jdk/job/jdk-linux-aarch64-openj9/317/consoleFull showed the same failure.
I have added the appropriate labels to https://ci.adoptopenjdk.net/computer/docker-packet-ubuntu1604-armv8-1/ and my proposal would be that unless we can determine the cause of the failures we disable the dynamically provisioned EC2 aarch64 systems for the GA and single thread all builds through that packet machine. I am running a build at https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk/job/jdk-linux-aarch64-hotspot/285/console to verify whether it is able to run the dockerBuilds successfully, so if that works the only potential with the proposal is if we see any recurrance of https://github.com/AdoptOpenJDK/openjdk-build/issues/1804

(NOTE: @gdams had increased the capacity on the systems yesterday but that did not resolve the issue)

After seeing that in the time when the logs stopped the docker images was no longer running I wondered if we were hitting an issue with the docker subsystem on the dynamically provisioned hosts being updated and restarted while the builds were taking place. Looking at the jenkins job log, and the package logs on the host system, they seemed almost exactly an hour out at a time when the docker.io package was updated (likely just a time zone difference in how they're being prevented)

15:37:28  [ 88%] Building CXX object runtime/compiler/CMakeFiles/j9jit.dir/home/ubuntu/workspace/andrew-aarch-test/workspace/build/src/omr/compiler/optimizer/ReachingDefinitions.cpp.o
15:37:28  [ 88%] Building CXX object runtime/compiler/CMakeFiles/j9jit.dir/home/ubuntu/workspace/andrew-aarch-test/workspace/build/src/omr/compiler/optimizer/OMRRecognizedCallTransformer.cpp.o
15:37:29  [ 88%] Building CXX object runtime/gc_modron_startup/CMakeFiles/j9modronstartup.dir/mminit.cpp.o
15:37:29  [ 88%] Building CXX object runtime/compiler/CMakeFiles/j9jit.dir/home/ubuntu/workspace/andrew-aarch-test/workspace/build/src/omr/compiler/optimizer/RedundantAsyncCheckRemoval.cpp.o
15:37:29  [ 88%] Building CXX object runtime/compiler/CMakeFiles/j9jit.dir/home/ubuntu/workspace/andrew-aarch-test/workspace/build/src/omr/compiler/optimizer/RegisterCandidate.cpp.o
15:45:02  wrapper script does not seem to be touching the log file in /home/ubuntu/workspace/andrew-aarch-test@tmp/durable-90579abf
15:45:02  (JENKINS-48300: if on an extremely laggy filesystem, consider -Dorg.jenkinsci.plugins.durabletask.BourneShellScript.HEARTBEAT_CHECK_INTERVAL=86400)

vs the update logs:
2020-10-20 14:37:19 upgrade docker.io:arm64 19.03.6-0ubuntu1~18.04.1 19.03.6-0ubuntu1~18.04.2
2020-10-20 14:37:19 status half-configured docker.io:arm64 19.03.6-0ubuntu1~18.04.1
2020-10-20 14:37:31 status unpacked docker.io:arm64 19.03.6-0ubuntu1~18.04.1
2020-10-20 14:37:31 status half-installed docker.io:arm64 19.03.6-0ubuntu1~18.04.1
2020-10-20 14:37:35 status half-installed docker.io:arm64 19.03.6-0ubuntu1~18.04.1
2020-10-20 14:37:35 status unpacked docker.io:arm64 19.03.6-0ubuntu1~18.04.2
2020-10-20 14:37:35 status unpacked docker.io:arm64 19.03.6-0ubuntu1~18.04.2
2020-10-20 14:37:35 configure docker.io:arm64 19.03.6-0ubuntu1~18.04.2 <none>
2020-10-20 14:37:35 status unpacked docker.io:arm64 19.03.6-0ubuntu1~18.04.2
2020-10-20 14:37:35 status half-configured docker.io:arm64 19.03.6-0ubuntu1~18.04.2
2020-10-20 14:37:37 status installed docker.io:arm64 19.03.6-0ubuntu1~18.04.2

I therefore believe that these issues have been caused by an automatic update on the machine trying to update docker from a one-off template image used for creating the dynamic instances. If we rebuild the template with a more up to date docker.io package I believe it will resolve the problem

Looks ok after rebuilding the image (and increasing the space again as the first rebuild was too small to run a build on). Closing as I'm now reasonably confident that the issue has been identified and resolved.


Move in figures documentation

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

Project coverage is 83.05%. Comparing base (a485017) to head (9924586).
Report is 11 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #184   +/-   ##
=======================================
  Coverage   83.05%   83.05%
=======================================
  Files          57       57
  Lines        4811     4811
=======================================
  Hits         3996     3996
  Misses        815      815

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


EVA-495

Function for mapping/converting allele origins other than "somatic" and "germline" to "germline"
Tests for this function
Use these converted allele origins for evidence strings

Coverage increased (+0.1%) to 77.6% when pulling e136c7429da7a01a454c347db82b30620d7e83da on tomdcsmith:convert_allele_origins into 25061f31f3bce835afbfc03809c3784d2c71f2b8 on EBIvariation:master.

In addition to the test I have ran the pipeline with this changed and confirmed that some records allele origins that aren't "germline" or "somatic" map to "germline".

Coverage increased (+0.1%) to 77.6% when pulling 902813c0448974658ba4dcb1bda6e9b428accd2f on tomdcsmith:convert_allele_origins into 25061f31f3bce835afbfc03809c3784d2c71f2b8 on EBIvariation:master.

Coverage increased (+0.1%) to 77.6% when pulling b2471ab8d1cf021d95034c980b7396a66f3e5b9f on tomdcsmith:convert_allele_origins into 25061f31f3bce835afbfc03809c3784d2c71f2b8 on EBIvariation:master.


VarUintError(VarUintOverflow)

Seems Carp testnet users have run into the following issue:

thread '' panicked at 'VarUintError(VarUintOverflow) [149, 165, 239, 47, 17, 33, 161, 151, 252, 155, 237, 176, 57, 120, 126, 183, 35, 233, 136, 37, 180, 253, 77, 248, 108, 23, 236, 14, 63, 77, 199, 34]', tasks/src/multiera/multiera_address.rs:272:22

Seems to happen at epoch 183 on testnet

Fixed by b30c29520a4ce10ca9ddba4fcb81cf9fbfed890e


How does this even work?

Hi, sorry I know this isn't exactly an issue but I was hoping you could clear this up for me.
I tried running your example in my project but when I try to webpack winston it is going to attempt to require fs in several places and obviously that is not going to work for the browser.  How do you deal with this.  I dont see anything special in any of your configs but I must be missing something...
I get a ton of output like this:
Module not found: Error: Can't resolve 'fs' in 'C:\Users\Bass\dev\LetterCounter\node_modules\winston\dist\winston\transports'
resolve 'fs' in 'C:\Users\Bass\dev\LetterCounter\node_modules\winston\dist\winston\transports'
Parsed request is a module
using description file: C:\Users\Bass\dev\LetterCounter\node_modules\winston\package.json (relative path: ./dist/winston/transports)
Field 'browser' doesn't contain a valid alias configuration

https://github.com/winstonjs/winston/issues/287#issuecomment-457959528?


Jpark Added more functionality to buttons.

I think I just need to add the modals to add questions and answers... I need to add tests though...

Great work, Jun! Lots of functionality here.

Looks good


'Camps' for suspectFLEE

When does a suspect stop traveling? This should work slightly differently for a 'suspect-agent' than for a forced migration agent.
On their way, a suspect may encounter multiple POIs ( at which they may stop (parking garage, etc). At each of these, the suspect stops traveling with a certain probability attributed to the POI.

Remind to self that "stop travelling" may mean deleting the agent from the simulation. This would require a bit of new code.

Note: I am working on this now in a separate branch.


New package: FormFactor v0.1.0

Registering package: FormFactor
Repository: https://github.com/nuzhat07/FormFactor.jl
Created by: @nuzhat07
Version: v0.1.0
Commit: 3f7a7b2c3a61f71c7497ea25e6c1e9ca78e7b12e
Reviewed by: @nuzhat07
Reference: https://github.com/nuzhat07/FormFactor.jl/issues/1

This package is empty, there's nothing beyond the template from what I can see.


fix paper logic in samples

the logic was not wrong. added less confusing const names: https://github.com/alpacahq/alpaca-backtrader-api/commit/c6b6ad4ad1c3dd2abbd7fc366aee9c73919d0551


add favicon and correct title

add favicon and correct title
Issue #294
What is the change?

Add favicon file in images folder
Add favicon link in head of all pages
Change title of all pages based on the page

Is Development Tested?

[x] Yes

Can you add screenshot of it how it is going to look like now

Can you add screenshot of it how it is going to look like now

Can you add screenshot of it how it is going to look like now


[enhancement] up and downcrafting recipies for capacitor banks

The tiers of capacitor banks increase by a set multiple of 5 I wanted to suggest several shapeless crafting recipes
5 basic banks form a bank
5 banks form a Vibrant bank
and vice versa
a vibrant bank breaks down into 5 banks
a bank breaks down into 5 basic banks

There are already recipes to do this


Added option to partially hide market listings

When you have many items listed on the market, and you visit the market home page, it is annoying to scroll down to other content of the page. Feature disabled by default.

This is cool, can you convert your additions to use tabs though please.

Think I fixed the indentation issue. Don't understand what you want me to do with the continue in the for loop. The stuff gets appended for each list there is (My buy orders and My sell listings).

`if ( listingCount < 5 )
{
continue;
}
// rest of the code`

Do you want those extra four lines for your own preference sake, or is there a good reason for doing that?

The reason is to reduce nesting as the further code will have less indentation. Just a simple fix which you can quickly do before squash.

Added and squashed.

:>

Are you having a giggle m8? Surely you didn't add spaces intentionally :goberserk:

I blame PHPStorm.

Don't know what you talking about. :^)

Nevermind.

Check now.

Thanks!


Support for  jwilder/nginx-proxy

Since I'm playing with this setup on a shared container sandbox, I can't give it exclusive use of external port 80.
I'm planning to use https://github.com/jwilder/nginx-proxy on this box to support running fairly random things.
I'm locally modifying the commander image to support this, the two main things I need to do:

Remove the nginx container from docker-compose.yml
Get the VIRTUAL_HOST environment variable passed into the web container

The latter looks feasible using a web.env the same way grafana.env works. I can also give the grafana instance a name this way.
The former could be done with an extending docker-compose config file. I can't see a better way to override a listed service in docker-compose.yml.
I also need to attach the nginx-proxy to the relevant app_default network but I feel that's something that nginx-proxy should be able to take care of. Right now after I attach the network, I need to restart nginx-proxy, due to jwilder/docker-gen#190.

Random thought, and it might not be possible, but could you make the port an optional command-line parameter (that defaults to 80) when bringing up the server? Then you could choose your port. I'm just wondering if there is a way that does not involves modifying commander's docker-compose.yml ?

I'm not sure if that would work. I'll give it a try next time I get to this. That would mean nginx in-front of nginx in this case, but it's also generally usable for other shared-host situations beyond the automatic nginx-proxy use-case. I'd hence have an nginx.env file to pass the VIRTUAL_HOST environment variable through to NGINX.
This might have the advantage of not needing to mess with the app_default network too. The network part of nginx-proxy (and Docker Compose) is mostly magic for me right now.
The only blocker would be if the nginx-proxy setup won't proxy for published ports, and since (by my limited understanding of Docker) published ports are a subset of exposed ports, I don't imagine that's a blocking issue.
I'll probably try out both methods, since I think the non-hacky version of my current effort won't be too bad.

Appreciate you looking into this :-)
Cheers

Ah, just noticed the front-page images live in the nginx instance, not the 'web' instance. So it's serving a purpose other than reverse-proxy.

Yes. Good point.

I've got the first step up in #10. This is basically enough for now. I was considering trying to get a non-published port working but for my use-case, having people able to bypass nginx isn't a problem.
The only wart remaining is that startup-ordering is fraught with peril:

Create jwilder/nginx-proxy container
Start cyber-dojo on port other than 80
Add nginx-proxy container to app_default network
Restart cyber-dojo so nginx-proxy regenerates its configuration.

My leaning is two-fold right now:

See if I can get nginx-proxy to react to having a network added (may depend on newer Docker API per jwilder/docker-gen#190.)
Consider whether I can add another command-line option to cyber-dojo for optionally binding nginx to another network (in this case, the default 'bridge' would work).


az/Hide sensitive info on logs

Closes #1056

We might be able to fix this after all, the webView has some debug logging settings:
abstract class WebView {
  ///Debug settings used by [InAppWebView], [HeadlessInAppWebView] and [InAppBrowser].
  ///The default value excludes the [WebView.onScrollChanged], [WebView.onOverScrolled] and [WebView.onReceivedIcon] events.
  static DebugLoggingSettings debugLoggingSettings = DebugLoggingSettings(
      maxLogMessageLength: 1000,
      excludeFilter: [
        RegExp(r"onScrollChanged"),
        RegExp(r"onOverScrolled"),
        RegExp(r"onReceivedIcon")
      ]);
...
}

I would like to exclude all logs that start with this pattern:
WebView ID<PHONE_NUMBER>53145431831672362462304017140 calling "EncointerJsService" using

Because we actively log the same content in our JS-callback handler.
Can you have a look at it again?


Fixes bug in getStartIndexAcc method

This PR fixes the issue mention in #15
The changes are made in yamlParse.ts file inside this repository, so this PR is dependent on #14 . But, if maintainers decides not to use yamlParse.ts and use  node-yamlParse module for YAML parsing, same change can be made on that repo.
This PR also includes the changes made in PR #14 , merging this PR will add both features of PR 14 and Fix for issue 15

@angad-singhh can you check if these are still needed?


Chore: Run lint task in Travis

This PR simply uses nvm install and then npm run lint to run the lint tasks. It also moves the algolia to an after_success step.
This is probably not the right approach. A better approach would be to see if we can switch the Travis build language to NodeJS and remove the algolia line entirely since we are now on DocSearch. This is just a POC (to try to see if it might be a viable temporary solution, should the DocSearch questions remain open for a while).

Ack, I was really hoping this would trigger a Travis build. It must still be using the old logic (only master branch) despite the changes in this branch.
I'll dig in more later.

No we don't need Ruby anymore. DocSearch doesn't require build step. So it should be fine to switch all builds to run normal Node image.

Awesome, then I'll close this PR. Thanks!


various (small) improvements

Description
What is this PR

camera.py and cameras.py: rename old 'clippingRange' -> 'clipping_range' rename old 'focalPoint' -> 'focal_point'
render.py: remove import matplotlib.pyplot as plt, this looks like a genuine bug
correct close() method accordingly
add actor._mesh.transform = None in if isinstance(actor._mesh, VedoVolume) as future vedo release will need this
settings.py: set WHOLE_SCREEN = False as default, avoid setting size='full' as this is poorly dealt with in vtk
fix output path in examples/screenshot.py and video.py
pyproject.toml: a newer version of "morphapi>=0.2" seems to be needed

How has this PR been tested?
Tested on the examples
Is this a breaking change?
No
Does this PR require an update to the documentation?
No
Checklist:

[ ] The code has been tested locally
[ ] Tests have been added to cover all new functionality (unit & integration)
[ ] The documentation has been updated to reflect any changes

Opened #294 for test that still doesn't pass - may well be the data we regress against in that test.


polarfs has already open source

https://github.com/ApsaraDB/PolarDB-FileSystem

yes，it is. but it is a fs client . Alibaba didn't open source its polar store which is a distrubute file system.

yes，it is. but it is a fs client . Alibaba didn't open source its polar store which is a distrubute file system.

I just downloaded the code. It is really just a client. Sorry to disturb you :)


k3OS always shuts down machine after successful PXE install

Version

v0.8.0
Architecture

amd64
Describe the bug

k3OS always shuts down the machine after PXE install, even ignoring k3os.install.power_off=false
To Reproduce

PXE boot with something like:
LABEL k3os-server
    MENU LABEL K3OS Server
    KERNEL k3os/v0.8.0/k3os-vmlinuz-amd64
    INITRD k3os/v0.8.0/k3os-initrd-amd64
    APPEND k3os.mode=install k3os.install.iso_url=http://server/k3os/v0.8.0/k3os-amd64.iso k3os.install.config_url=http://server/k3os/k3os-server.yaml k3os.install.device=/dev/sda k3os.install.power_off=false

Expected behavior

Machine reboots directly into k3OS.
Actual behavior

Machine shuts down after successful install.
Additional context

It is not any strange behaviour of my hardware, I explicitly see 'shutting down' flash on the terminal after install is finished.

Possibly addressed by #143. Need to test.


Miscellaneous fixes from AL2023 testing

Add " around B64_CLUSTER_CA so a multi-line value can be saved correctly into the file
In AL2 we had ec2-net-utils which is now named amazon-ec2-net-utils so handle them separately/explicitly (so it's obvious!)
Patch MACAddressPolicy which defaults to persistent in AL2023 and causes issues

/ci build


The "Propagate Access Control Lists" option is only available in Storage Explorer version 1.28.1+

The Propagate Access Control Lists option mentioned at https://learn.microsoft.com/en-us/azure/storage/blobs/data-lake-storage-explorer-acl#apply-acls-recursively is only available for Storage Explorer version 1.28.1 and up. It would be better to add a note so that users are aware of this version requirement.

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

ID: 89b67548-fef4-4ad8-9605-ca5a16a4f1c1
Version Independent ID: 1be3381a-db1f-32e1-61e3-b4a0173ef4ce
Content: Storage Explorer: Set ACLs in Azure Data Lake Storage Gen2 - Azure Storage
Content Source: articles/storage/blobs/data-lake-storage-explorer-acl.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

@charleswang007
Thanks for your feedback! We will investigate and update as appropriate.

@charleswang007
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.

Thanks @AjayBathini-MSFT for your prompt support.


Fix calculation in Order::calculateInvoiceAmount

Description

Why is it necessary?

The amount calculated by Order/Order::calculateInvoiceAmount() differs from the basket,
this might lead to inconsistent prices calculations if the order is saved later in the backend.
(Can be tested by changing shipping costs, saving and reverting to old value)

What does it improve?

Consistency of price calculation

Does it have side effects?

No

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
This PR is a subset of #847

How to test?
Run the tests before applying fix in calculateInvoiceAmount

Detailed writeup
While we originally tried to test/fix the first issue, the second issue emerged after we finished the tests and needed to be fixed as well, even though the implications weren't as extensive as with the first issue.
First Issue:

Line 1069 in Order.php (here it's irrelevant if shop is in net mode or not)
Values rounded to 3 decimal places In s_order_details result in deviations, rounding them to two places will match the sBasket behaviour.
Solution: Extract getPrice() to $price = round($detail->getPrice(), 2); in all 3 occurrences.
Reproduce: Buy 5 x "SW10239" in Frontend (shop in net mode), change shipping costs in backend and save and revert shipping costs back to old value. The amount now has changed from 96.48 to 96.47.

Screenshot basket: amount = 96.48

After calculateInvoiceAmount was triggered by changing shipping costs.

Second Issue:

Line 1085 in Order.php
Missing rounding after calculation will result in unrounded values in the database.
Solution: round result to two decimal places.
Reproduce: Buy article in Frontend, change quantity in Backend and save, check s_order table for amount. The amount now stored in the database is not rounded at all. (Shop must not be in net mode to reproduce)

What has been tested ?

That Order/Order::calculateInvoiceAmount() calculation matches sBasket
Checkout process (via frontend controllers)

Edit: extract $detail->getPrice(), update PR message

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-18297.
Please use this issue to track the state of your pull request.

merged with ecf9ac2d5eb0dbb156e962d353e395988772bc03


distance_of_time doesn't use options

distance_of_time is defined as
def distance_of_time(seconds, options = {})
  display_time_in_words DOTIW::TimeHash.new(seconds).to_hash, options
end

display_time_in_words treats options passed to distance_of_time as include_seconds.

@avapoet: are you able to create a patch for this issue now?

Fixed by #21.


terminology: schema or structure?

I would like to know, schema or structure, which one is more clear when using a database product?

I think we should stick with schema, since milvus is using it. many thanks for mentioning it. lets change it to schema.

and we should change rows to entities as well.

@shanghaikid row count should also be changed as entity count?


Do not include filename when using multipart/form-data field that is not a file

Noticed curlconverter is converting all cURL -F as 'files'. But not all -F are files, some are actual form data, also Python 'requests' adds 'filename' field if you send as 'files'.
Only curl form data with '@' can be uploaded using 'files', remaining everything should go as 'data'

It looks like the files argument with a dict of tuples with None as the first entry is still the preferred way for sending non-file multipart/form-data through requests.

files = {
    'username': (None, 'davidwalsh'),
    'password': (None, 'something')
}

requests.post('https://httpbin.org/post', files=files)

This is pretty ugly, but it gets rid of the filename field and should help compatibility.


Initial load failing when page accessed direct from url

This may be an iron router issue, but since I can only reproduce it with google maps, I wanted to pick your brain.
I have a page with a map on it, the result is the same whether the script is loaded onStartup or onBeforeAction in the router.
If I refresh the page, everything is OK.
If I access the page via a link, everything is OK.
If I load the page by typing in the url, only the upper left tile loads. Disabling the cache has no effect. Any thoughts on the cause? Unfortunately, I don't know meteor well enough to know the difference between what it does on a refresh & what it does on a GET request...
Additionally, there's about a 10% chance that it fails to enter the callback inside GoogleMaps.ready (but touches lines before and after the .ready call). I can only reproduce this part in Chrome Canary 42, so we can ignore it for now, although I'd wager the 2 are related.

Hey @mattkrick thanks for this, would you mind creating a simple reproduction project I could clone and try out? That would help me a great deal with debugging.

Sure thing, unfortunately, it works with simple scenarios. I'm slowly adding more complexity until I can get a failure, but it may take awhile. I'm pretty sure I'm causing a race condition somewhere, just can't pin it down...

Yeah it's interesting as I've encountered something similar but very very rarely

All I can conclude is that too many front-loaded resources can sometimes cause a failure.
When I stick a bunch of junk in the lib or add a ton of link imports (polymer elements) , I can increase failure rates to near 100%. When I remove or lazy-load them, I can't get it to fail at all.
Interestingly enough, when I uninstall the google-maps package & instead clone the files into my repo, failure rates increase as well, even when I stick them in /lib. That tells me file load order definitely plays a part in this.
Two solutions:
Add a delay on map creation. It's not pretty, but you could allow the user to specify the time delay in the options object.
        setTimeout(function () {
          GoogleMaps._create(data.name, {
            instance: new google.maps.Map(canvas, data.options),
            options: data.options
          });
        }, 300)

A more elegant solution would be to append 3 lines to the end of _create to refresh the map once everything is done loading.  I think this is pretty foolproof, but given enough time, I'll probably break it somehow...
   _create: function (name, options) {
      this.maps[name] = {
        instance: options.instance,
        options: options.options
      };
      this._ready(name, this.maps[name]);
      var self = this;
      google.maps.event.addListenerOnce(this.maps[name].instance, 'idle', function() {
        google.maps.event.trigger(self.maps[name].instance, 'resize');
      })
    }

Hey thanks for digging into it a bit. I'd like to have a reliable repro setup to investigate a bit more. I'll see if I can reproduce the bug.


These images are hard to use

Whoever added this:
"Note: Multi-arch tags, such as 4.7.1, have been updated to use windowsservercore-1709 images if your host is Windows Server 1709 or higher or Windows 10 Fall Creators Update (Version 1709) or higher. You need Docker 17.10 or later to take advantage of these updated tags."
Does not understand the whole point of containers.  Regardless of where you build your container, you should get the same result!  This is utterly broken.

If you want to be tied to a specific platform, then you should be using a platform tag such as 4.7.1-windowsservercore-lstc2016.  The intent of multi-arch tags is that when you build on a specific platform, then you will get an image that will work with said platform.  The use of multi-arch tags is a practice that you see across the official Docker repos.  This allows users to pull image X from any platform and just have it work.


Fix CN docs: api/paddle/optimizer/Optimizer_cn.html

Inspired by: 飞桨快乐开源活动全新升级🔥
Issue from: https://shimo.im/sheets/2Us6FhlaVAgxfHys/ytCqF (No. 31)

英文文档对parameters的描述支持tuple，中文没有，需要补充。

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@linzeyang 如果对文档活动感兴趣，可以进群参与更多活动～

LGTM

@sunzhongkai588 谢谢您的审核，请问Docs-NEW这个失败要怎么处理呢？

@sunzhongkai588 谢谢您的审核，请问Docs-NEW这个失败要怎么处理呢？

等 CI rerun 通过即可

谢谢，现在已经通过了 :)


Please add support of Admin3

For all new adapters it is required the support of admin3.
You can just copy it from somewhere.
and add "materialize": true to io-package.json/common

Sorry, I don't get this.Google seems not to like me today, and even here https://github.com/ioBroker/ioBroker/wiki/Adapter-Development-Documentation i am not able to find anything about Admin3. Can you give me a hint?

Well, I installed it in an admin3-Environment, and it works with admin3.

Please make a index_m.html file, like in here:
https://github.com/ioBroker/ioBroker.template/blob/master/admin/index_m.html
and set this flag:
https://github.com/ioBroker/ioBroker.template/blob/master/io-package.json#L70

[X] done.
But.. I don't see any difference in my mac/raspberry environment :-)


LF-23479 switching api calls to always be https, updating tests

https://livefyre.atlassian.net/browse/LF-23479

#review


Expose GetAuthServerEndpoint and make user name optional for login

We don't need the user name for login since we grab the identity from the token internally.

@zawachte-msft : I am just curious if you still had this PR on your radars?


Use newer version of cdk-dynamo-table-viewer

When going through the CDK workshop, I noticed the referenced version of cdk-dynamo-table-viewer (3.0.2) has an earlier release of the CDK core, which causes compilation issues when going through this workshop with the newest version (1.0.0) of the CDK core. Here's the page where the npm install command is mentioned:
https://cdkworkshop.com/20-typescript/50-table-viewer/200-install.html
For me, using 3.0.5 of the table viewer made this issue go away:
npm install<EMAIL_ADDRESS>

We should be able to omit the explicit version number now that we are in 1.0.0

@eladb This issue comes back for "cdk-dynamo-table-viewer" 3.1.2 with aws-cdk version 1.35.0 while doing the typescript CDK workshop.


Update to v1.8.1

PR-URL: #53
Related: iojs/io.js#1436
Signed-off-by: Hans Kristian Flaatten<EMAIL_ADDRESS>

LGTM and thanks :+1:
Btw, for simple version bumps there is no need to wait for consensus or +1's. This is mentioned in our GOVERNANCE.md. The submitter can just merge them right away with a short note.

Thanks for the heads up @pesho


Is this suitable for use in a banner campaign?

I need a very simple database for a banner campaign to store latitude/longitude for a bunch of addresses. The client wants to be able to easily add more addresses during the campaign.
Tabletop.js + google drive works perfectly for this, but I'm worried we might have trouble with the amount of requests we're likely to generate. Has anyone done this before and can comment?
Or, if you can point me in the direction of a better (but equally simple) solution that would be very welcome!
Thanks!

Don't use it for production. Especially banners - way too much exposure.
https://www.google.com.au/url?sa=t&rct=j&q=&esrc=s&source=web&cd=2&ved=0ahUKEwiPorm-rL3QAhXEmpQKHbrBBe0QFgggMAE&url=https%3A%2F%2Fgist.github.com%2Fjsvine%2F3295633&usg=AFQjCNGl8YAQOWmqwB5FwI-azfBXTSP_aA&sig2=7Y6C3B-0GGMre_pH2JMAXQ&bvm=bv.139250283,d.dGc&cad=rja
Also note that recently Google has blocked the use of public asset from google drive for banner campaign(DoubleClick) and is forcing devs to use Asset Library instead.

Thanks, @bastole, you've def hit the nail on the head.


fixed Simultaneous accesses error, but modification requires exclusiv…

error stack
Simultaneous accesses to 0x103582e90, but modification requires exclusive access.
Previous access (a modification) started at TimedSilver`ts_observationRemoversForObject(_:) + 292 (0x103510d84).
Current access (a modification) started at:
0    libswiftCore.dylib                 0x0000000108cc3f00 swift_beginAccess + 505
1    TimedSilver                        0x0000000103511580 closure #1 in ts_observationRemoversForObject(_:) + 54
2    TimedSilver                        0x00000001035115d0 thunk for @callee_owned (@unowned UnsafePointer<UnsafeRawPointer?>) -> (@error @owned Error) + 22
3    TimedSilver                        0x0000000103512e60 partial apply for thunk for @callee_owned (@unowned UnsafePointer<UnsafeRawPointer?>) -> (@error @owned Error) + 91
4    libswiftCore.dylib                 0x0000000108a9f640 withUnsafePointer<A, B>(to:_:) + 12
5    TimedSilver                        0x0000000103510c60 ts_observationRemoversForObject(_:) + 375
6    TimedSilver                        0x0000000103510bf0 TSObservationRemover.makeRetainedBy(_:) + 38
7    TimedSilver                        0x000000010350faa0 NotificationCenter.ts_addObserver<A>(_:name:object:queue:handler:) + 923
8    TSWeChat                           0x000000010240e900 TSChatViewController.keyboardControl() + 406
9    TSWeChat                           0x000000010242ffb0 TSChatViewController.viewDidLoad() + 1877

spelling error


Update Lint/Void to recognize more nonmutating methods

Update Lint/Void to recognize more nonmutating methods when the CheckForMethodsWithNoSideEffects option is used.
This PR adds one (now)-native transform_keys method, and about a dozen common ActiveSupport methods such as stringify_keys, compact_blank and squish.
The ActiveSupport methods are only recognized when AllCops/ActiveSupportExtensionsEnabled: true (such as in a Rails project)

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

The goal seems reasonable, but it might be preferable to adjust the approach slightly. Instead of using ActiveSupportExtensionsEnabled, it might be better to provide a user config parameter like NonmutatingMethods. (There might be a better name than NonmutatingMethods.)
After that, In RuboCop Rails, configure NonmutatingMethods as follows:
# config/default.yml in the RuboCop Rails
NonmutatingMethods:
  - compact_blank
  - deep_merge
  - deep_stringify_keys
  - ...

This approach allows for the elimination of the NONMUTATING_ACTIVE_SUPPORT_METHODS_WITH_BANG_VERSION constant, ensuring that changes in Active Support do not have an impact.


[Pending] Specifying hashInputOrder

waiting for requirement of md5 confirmation
add functionality and tests for specifying inputHashOrder for algorithms:
hmac{md5, sha1, sha256, sha512} and {sha1, sha256, sha512}

This implementation makes sense to me. It also happens to match an internally approved API proposal. @bojeil-google WDYT?


docs(html-symbol-entities) improve instructions

Improve spelling of the instructions.
Improve readability of the text.

Thank you.


Make container id a required parameter for all runc(client) operations

Per the face to face discussions we decided to make --id, or similar parameter, required for all of the runc operations.
I'll generate a PR to update the runtime.md docs to specify the need for the container id for each of the runc operations.  In particular, update the lifecyle description.

On Thu, Jan 21, 2016 at 03:49:15PM -0800, Mike Brown wrote:

I'll generate a PR to update the runtimemd docs to specify the need
for the container id for each of the runc operations In particular,
update the lifecyle description

It's not my favorite approach [1,2,3], but as of 6f2665a it looks like
this is already done in #225.
 Subject: Re: removal of /run/opencontainer/containers, add --state?
 Date: Tue, 8 Dec 2015 15:49:57 -0800
 Message-ID<EMAIL_ADDRESS>

Ah I see.. been ill for a bit just getting back into it.  Thx Trevor will close.

I think a PR for runc is still needed though and we should just make it a normal arg, not a flag.
ie. runc <action> <id> ...


knope silently fails to create release tag?

Hello! 😄
Recently, we've had some issues where knope fails to create a new git tag for a release. It appears to succeed, and generates the right commit and CHANGELOG entry, but the associated tag doesn't get created. From my understanding of the documentation it shouldn't be possible to successfully create a release without an associated git tag if you have GitHub setup properly.
This seems to happen intermittently on full releases, but never on pre-releases. We're using 0.7.1, and this is our knope.toml. The CI logs for these releases look normal and don't indicate any error. Happy to try and provide other diagnostics!

It looks like your release workflow is missing the Release step (your prerelease workflow has it), so I'd expect the release workflow to never create a GitHub release (and therefore tag). I also suggest combining prerelease and release and using the --prerelease-label CLI option instead, to avoid issues like this in the future.
I opened https://github.com/rigetti/quil-rs/pull/256 as an example (that GHA syntax might not be right, though)

Oh, as a manual test, you can also run with --dry-run and it should tell you if it's going to create a GitHub Release. I like to do this automatically in CI so I can see what will happen next time I run a full release.

Doh, sorry about that. I was under the impression it had been working and misread the release workflow. I'll be using that --prerelease-label trick to simplify our workflows in other repos as well.


Migrate out of deprecated features in typespec

Replace old @service({version} with versioning lib
Missing @sharedRoute

/azp run

/azp run


Fix broken links in changelog

Provide a description of what has been changed
Checklist

[ ] A PR is opened to update the documentation on https://github.com/kedacore/keda-docs
[X] Commits are signed with Developer Certificate of Origin (DCO)
[ ] Tests have been added

@tomkerkhove looks like there was some kind of mix up in the change?

Ha messed up the PR with merging.
Will fix tomorrow, sorry!


Documentation improvements

This attempts to work towards the extend documentation around managing Helm releases task mentioned in https://github.com/fluxcd/toolkit/issues/238:
The thought here is to have a leveled documentation, increasing detail in each level, and avoiding too much duplication between levels:

https://toolkit.fluxcd.io/guides/helmreleases
https://toolkit.fluxcd.io/components/helm/helmreleases/
https://toolkit.fluxcd.io/components/helm/api/

This:

copies some of the content about HelmRelease CRD options from 1. to 2. with the idea being to remove or reduce it from 1. and replace with a more prominent link to 2. (as a follow up in the toolkit repo).
adds more documentation to 2.
removes some of the copy/pasted golang documentation from 2. since it already exists in 3. The status condition types / reasons are kept (though moved under the status section) since those are not currently included in 3. This could be potentially be resolved in the future by https://github.com/ahmetb/gen-crd-api-reference-docs/issues/21

I have to think for a bit about removing the structures from the spec docs, but do have some comments in the meantime.

I discussed this a bit with @stefanprodan, and we both think the in-documentation code blocks should stay. Reason for this is that once we get out of the "rapid development cycle", new proposals should be made against the spec, and this becomes hard without them being available. We are currently working on more governance (including documentation) for the Flux project, and I will make sure this process will be documented somewhere (cc: @scottrigby).
I also think this means that the documentation within the spec that is added in this PR should be rewritten to something that describes desired application behaviour, instead of being explanatory to a user. But this can be taken care of as a follow up action, including moving bits and pieces to the HelmRelease guide (which targets users).
I took the liberty to rebase the PR and undo the code removals from the spec so this can be merged in the meantime, as we are trying to make new controller releases today and would like to have this included.


error with artifactory setup

Hello People,
When I was setting up this artifactory in an ubuntu machine, artifactory service is not getting started.

Executed ''systemctl status artifactory.service" then showing below messages.

Followed documentation - https://www.jfrog.com/confluence/display/RTF/Installing+on+Linux+Solaris+or+Mac+OS#InstallingonLinuxSolarisorMacOS-ServiceInstallation
Thank you.
Mani

https://computingforgeeks.com/how-to-install-jfrog-artifactory-on-ubuntu/
please follow this doc
take t2.large or medium

Minimum requirement for artifactory 64 bit os, 4 cores processor, 4GB RAM
I think you took t2.micro  this machine is for 1 CPU and 1GB RAM this configuration is not enough for artifactory. Try above configuration.
If you already done this please close the issue.


switch to rehype-mermaid to make graphs SSR

This solution needs to run playwright during build and I can't get that working on netlify 😞


Cannot compile rnnoise

@Desklop Hello. I cannot run the compile_rnnoise.sh script. Here is the output of my terminal; looks like some files and directories are missing:
`
javier@DESKTOP:/mnt/c/Users/javie/Desktop/RNNoise_Wrapper$ ./compile_rnnoise.sh
Archive:  rnnoise_master_20.11.2020.zip
90ec41ef659fd82cfec2103e9bb7fc235e9ea66c
creating: rnnoise-master/
inflating: rnnoise-master/AUTHORS
inflating: rnnoise-master/COPYING
inflating: rnnoise-master/Makefile.am
inflating: rnnoise-master/README
inflating: rnnoise-master/TRAINING-README
inflating: rnnoise-master/autogen.sh
inflating: rnnoise-master/configure.ac
creating: rnnoise-master/doc/
inflating: rnnoise-master/doc/Doxyfile.in
creating: rnnoise-master/examples/
inflating: rnnoise-master/examples/rnnoise_demo.c
creating: rnnoise-master/include/
inflating: rnnoise-master/include/rnnoise.h
creating: rnnoise-master/m4/
inflating: rnnoise-master/m4/attributes.m4
inflating: rnnoise-master/rnnoise-uninstalled.pc.in
inflating: rnnoise-master/rnnoise.pc.in
creating: rnnoise-master/src/
inflating: rnnoise-master/src/_kiss_fft_guts.h
inflating: rnnoise-master/src/arch.h
inflating: rnnoise-master/src/celt_lpc.c
inflating: rnnoise-master/src/celt_lpc.h
inflating: rnnoise-master/src/common.h
inflating: rnnoise-master/src/compile.sh
inflating: rnnoise-master/src/denoise.c
inflating: rnnoise-master/src/kiss_fft.c
inflating: rnnoise-master/src/kiss_fft.h
inflating: rnnoise-master/src/opus_types.h
inflating: rnnoise-master/src/pitch.c
inflating: rnnoise-master/src/pitch.h
inflating: rnnoise-master/src/rnn.c
inflating: rnnoise-master/src/rnn.h
inflating: rnnoise-master/src/rnn_data.c
inflating: rnnoise-master/src/rnn_data.h
inflating: rnnoise-master/src/rnn_reader.c
inflating: rnnoise-master/src/rnn_train.py
inflating: rnnoise-master/src/tansig_table.h
creating: rnnoise-master/training/
inflating: rnnoise-master/training/bin2hdf5.py
inflating: rnnoise-master/training/dump_rnn.py
inflating: rnnoise-master/training/rnn_train.py
inflating: rnnoise-master/update_version
Updating build configuration files for rnnoise, please wait....
./autogen.sh: 10: ./autogen.sh: autoreconf: not found
./compile_rnnoise.sh: line 13: ./configure: No such file or directory
make: *** No targets specified and no makefile found.  Stop.
/mnt/c/Users/javie/Desktop/scratch_coding/RNNoise_Wrapper
mv: cannot stat 'rnnoise-master/.libs/librnnoise.so.0.4.1': No such file or directory
'librnnoise.so.0.4.1' has been successfully moved to 'rnnoise_wrapper/libs/librnnoise_default.so.0.4.1'
`

I have debugged further and it seems this directory and file are not being created:
rnnoise-master/.libs/librnnoise.so.0.4.1
I don't see .libs directory neither the .so file.
To fix ./autogen.sh error, I needed to install autoconf libtool
Another question @Desklop , if I try to use the pre-compiled .so, I also get problems:
/home/javier/.local/lib/python3.6/site-packages/pydub/utils.py:170: RuntimeWarning: Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work
  warn("Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work", RuntimeWarning)

I have installed ffmpeg but it did not help.

Hello. I am able to to run both ways (python and the command-line-interface)! Thanks a lot.
I don't have avconv so the error always prints out but it seems defaulting to ffmpeg is working.
But the original issue is still not resolved. I have used the pre-compiled .so for now. Please resolve/close the issue as you see fit. Thanks.


Makefile

testing file creation from within github

just move it and remove compile.sh in the PR and ill add it in n thanks for this


too many connection resets (due to end of file reached - EOFError) PersistentHTTP::Error

Hi,
My bot crashed with this error. From the look of it, it does not seem like it is coming from my code. It happened after my bot was running for 1 or 2 days.
/var/lib/gems/2.0.0/gems/persistent_http-1.0.6/lib/persistent_http.rb:335:in `rescue in block in request': too many connection resets (due to end of file reached - EOFError) after 1729 requests on 27144020 (PersistentHTTP::Error)
        from /var/lib/gems/2.0.0/gems/persistent_http-1.0.6/lib/persistent_http.rb:300:in `block in request'
        from /var/lib/gems/2.0.0/gems/gene_pool-1.4.1/lib/gene_pool.rb:137:in `with_connection'
        from /var/lib/gems/2.0.0/gems/persistent_http-1.0.6/lib/persistent_http.rb:299:in `request'
        from /var/lib/gems/2.0.0/gems/httparty-0.13.5/lib/httparty/request.rb:101:in `perform'
        from /var/lib/gems/2.0.0/gems/httparty-0.13.5/lib/httparty.rb:522:in `perform_request'
        from /var/lib/gems/2.0.0/gems/httmultiparty-0.3.16/lib/httmultiparty.rb:124:in `post'
        from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.6/lib/telegram/bot/api.rb:36:in `call'
        from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.6/lib/telegram/bot/api.rb:31:in `method_missing'
        from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.6/lib/telegram/bot/client.rb:34:in `fetch_updates'
        from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.6/lib/telegram/bot/client.rb:29:in `listen'
        from bot.rb:62:in `block in run'
        from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.6/lib/telegram/bot/client.rb:22:in `run'
        from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.6/lib/telegram/bot/client.rb:10:in `run'
        from bot.rb:61:in `run'
        from bot.rb:72:in `<main>'

Seems like the old problem (#11) still occurs. The hardest part is that i can't reproduce it (or at least it would a couple of days to check). I've some thoughts but it may take time to test it.

Let me know if I can be of any help. Right now, I am restarting the process every time it crashes so if you want me to put some debug info somewhere so that I can send it to you, I would gladly do so !

Please update gem up to 0.3.9 and let me know ;)

Thanks ! Will do asap !

Hi, after a few days trial, I can say I repeatedly get this error (pretty quickly) :
/var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.9/lib/telegram/bot/api.rb:42:in `call': Telegram API has returned the error. (error_code: "502", uri: "https://api.telegram.org/bot137205796:AAEDirj9Of1oi6_VHoO5BgT1CLLBwDzOapU/getUpdates") (Telegram::Bot::Exceptions::ResponseError)
	from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.9/lib/telegram/bot/api.rb:33:in `method_missing'
	from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.9/lib/telegram/bot/client.rb:34:in `fetch_updates'
	from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.9/lib/telegram/bot/client.rb:29:in `listen'
	from bot.rb:62:in `block in run'
	from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.9/lib/telegram/bot/client.rb:22:in `run'
	from /var/lib/gems/2.0.0/gems/telegram-bot-ruby-0.3.9/lib/telegram/bot/client.rb:10:in `run'
	from bot.rb:61:in `run'
	from bot.rb:72:in `<main>'

It looks like Telegram server error and not related to the gem.

Oops... my bad, I forgot to catch the Exception.
Conclusion: it seems good! I will close the issue and reopen it in case the pb resurfaces


Added Hungarian (magyar) localization

I followed the guide and created a Hungarian localization. I believe I haven't left out anything, it builds and works fine locally, except for the "Try it out" part, which I wasn't able to test locally.

Hey @laymonage, sorry for the ping, but could you (or someone else) please take a look at this? Thank you!


devcontainer cli failed after upgrade vscode

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

VS Code Version: 1.94.2
OS Version: Ubuntu 22.04

Steps to Reproduce:

Update the vscode from snap
From terminal I got this error:
humdalab@HUMDALAB-01W:~devcontainer
/home/humdalab/.local/bin/devcontainer: 52: /snap/code/170/usr/share/code/code: not found

How can I update the path? With the suggested method "You can quickly try out the CLI through the Dev Containers extension. Select the Dev Containers: Install devcontainer CLI command from the Command Palette (F1)." it is not working.

I got exactly the same error.
Version: 1.95.0
Commit: 912bb683695358a54ae0c670461738984cbb5b95
Date: 2024-10-28T20:16:24.561Z
Electron: 32.2.1
ElectronBuildId: 10427718
Chromium: 128.0.6613.186
Node.js: 20.18.0
V8: 12.8.374.38-electron.0
OS: Linux x64 6.8.0-47-generic snap
When I go to check /snap/code rep:
ls /snap/code
172  173  current

indeed 170 is not there

What do you get when running the following in a terminal:
readlink `which devcontainer-insiders`

This should show the script in a cli-bin folder. What other files are in this cli-bin folder?


Update try-or-buy-microsoft-365.md

Added M365 E3 nd E5 and removed o365 E3

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/2478

@JohanFreelancer9  Please copy review


Marker#and is not supported

net.logstash.logback.marker.LogstashMarker provides an and method, but it is not usable because the various derived Markers do not override the add method defined in slf4j-api. The add from org.slf4j.helpers.BasicMarker assumes there's a list of Markers, but that's not the case for the logstash markers, so add ends up adding extra fields to it's referenceList, but that list is not used when generating JSON at serialisation time.
So, either and should throw an UnsupportedException or the various add methods should be implemented. It does seem though that several of the logstash markers would have a difficult time implementing add in a consistent way.

Hi @gpremer,
Can you provide a unit test, or sample code that reproduces the problem you are having?  I just tried using the and method, and it's working for me.
I used it like this:
LOGGER.info(append("n1", "v1").and(append("n2", "v2")), "Testing marker and logging.");

and received the following log output (formatted for clarity)...
{
    "@timestamp":"2015-01-07T22:56:22.482-08:00",
    "message":"Testing marker add logging.",
    "logger":"net.logstash.logback.encoder.MarkerAndTest",
    "thread":"main",
    "level":"INFO",
    "levelVal":20000,
    "n1":"v1",
    "n2":"v2"
}

It works internally like this:

When you call and, it will add the marker provided as an argument (e.g. n2) to the referenceList  of the marker on which and is invoked (e.g. n1)
Later, at logging time, when the LogstashFormatter writes the event, the markers are written to the json output in LogstashFormatter.writeLogstashMarkerIfNecessary here.
The first marker (n1) will get written on line 300, and
then the referenced markers (n2) are recursively written on line 306

Hum, judging your test, it seems that the problem may be related to the way we test. At the moment I'm just writing unit tests and not quite ready to seem how it looks in elasticsearch/kibana.
We've used the same way of testing as in the logstash unit tests.
    @Test
    public void thatMarkerEntriesCanBeAppended() throws Exception {
        // given
        final LogstashMarker marker = Markers.appendEntries(ImmutableMap.of("k1", "v1"));

        // when
        final LogstashMarker combo = marker.and(Markers.appendEntries(ImmutableMap.of("k2", "v2")));

        // then
        final StringWriter stringWriter = new StringWriter();
        final JsonGenerator jsonGenerator = new JsonFactory().createGenerator(stringWriter);

        jsonGenerator.writeStartObject();
        combo.writeTo(jsonGenerator);
        jsonGenerator.writeEndObject();
        jsonGenerator.flush();

        assertThat(stringWriter.toString()).contains("k1");
        assertThat(stringWriter.toString()).contains("k2");
    }

    @Test
    public void thatMarkersCanBeAppended() throws Exception {
        // given
        final LogstashMarker marker = Markers.append("k1", "v1");

        // when
        final LogstashMarker combo = marker.and(Markers.append("k2", "v2"));

        // then
        final StringWriter stringWriter = new StringWriter();
        final JsonGenerator jsonGenerator = new JsonFactory().createGenerator(stringWriter);

        jsonGenerator.writeStartObject();
        combo.writeTo(jsonGenerator);
        jsonGenerator.writeEndObject();
        jsonGenerator.flush();

        assertThat(stringWriter.toString()).contains("k1");
        assertThat(stringWriter.toString()).contains("k2");
    }

What we get is  (twice):
   java.lang.AssertionError: expecting:<'{"k1":"v1"}'> to contain:<'k2'>

Since both of these unit tests fail, and in light of your comments, it would seem better to test using a LogstashFormatter instance. However, it's not clear from the API how to go about that.
The issue is not critical for us (and certainly not if it does not occur at run time) as we worked around it  appending entries to the map we supply to appendEntries instead of calling appendEntries multiple times.

Ok.  Yeah, I can see why your test fails, since it only calls marker.writeTo, which only writes this marker, and not its references.  It's the LogstashFormatter that actually does the recursive marker writes of the references.
If you really want to confirm behavior in your unit tests, you could write tests against LogstashFormatter similar to the LogstashEncoderTest
I'm going to close this issue, since logstash-logback-encoder is working as expected.

That makes sense. Thanks for your support.


Inject service into components too

Since I came across multiple situations where we needed to track things from inside the components, I think the service should be accessible by default in components too.

Agreed - merged!

@kylenathan can you please bump up the version?

Yup, just doing now.

@matixmatix Bumped version

@kylenathan thanks 🎉


category file for trained models ZOO

Hi guys, where is the category file list for the models pre-trained in the ZOO?
We will need to upload that in order to use the model, right?

This demo works with category files:
https://github.com/szagoruyko/functional-zoo/blob/master/demo.ipynb


Lax Checking for ML-DSA

This is ongoing work in https://github.com/cryspen/libcrux/tree/ml-dsa-arithmetic

Closed in #646 .


[Enhancement] (nereids)implement showPluginsCommand in nereids

showPlugins command is supported in legacy planner but not in nereids(new doris fe planner). you need implement the following showPlugins command in nereids:
unsupportedshowPluginsStatement
SHOW PLUGINS

@Vallishp pls add a comment, then I am able to assgin the issue to you

@LiBinfeng-01   pls assign to me


Fix test function call

This is small fix, correcting cargo test run.
What kind of change does this PR introduce?

[x] Fix
[ ] Feature
[ ] Codestyle
[ ] Refactor
[ ] Other

Did this PR introduce a breaking change?
A breaking change includes anything that breaks backwards compatibility either at compile or run time.

[ ] Yes, please list breaking changes
[x] No

@last-partizan thanks for the fix :)

I think one bonus added by the clippy change is that you can add a GitHub token and the clippy changes will be automatically applied @Kethku

@last-partizan not complaining but you should've checked the code style box.

@j4qfrost that's not code style, that was broken test
complaints during pre commit hook was from cargo test, not from cargo fmt or something

You made a change to the clippy actions for the GitHub CI. Was that from another branch?

@j4qfrost you mean this? https://github.com/Kethku/neovide/pull/781/files
That wasn't me :)


"Grămești" is listed in the drop-down menu.

Description:
"Grămești" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Grămești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Grămești, SUCEAVA".


Update companies.yml

Adds the following events or companies:
Rise
Checklist

[ ] I have updated the event or company counts
[x] I have put the most recent relevant date in the "Last Update" column
[x] I have linked to the article about the change, not the company's website

I didn't update company count.


Multiple alb-ingress-controller support

Hello,
I currently have applications across two namespaces. I've spun up an alb-ingress-controller with the --watch-namespace parameter pointing to the first namespace, which works as expected.
When I spin up the second alb-ingress-controller, the controller pod gets to the "running" state however, the alb itself does not get provisioned. The logs show the controller waiting for a leader lease (not sure what that is exactly).
`AWS ALB Ingress controller
Release:    v1.1.2
Build:      git-cc1c5971
Repository: https://github.com/kubernetes-sigs/aws-alb-ingress-controller.git
W0726 23:36:01.489292       1 client_config.go:549] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
I0726 23:36:01.552747       1 :0] kubebuilder/controller "level"=0 "msg"="Starting EventSource"  "controller"="alb-ingress-controller" "source"={"Type":{"metadata":{"creationTimestamp":null}}}
I0726 23:36:01.553445       1 :0] kubebuilder/controller "level"=0 "msg"="Starting EventSource"  "controller"="alb-ingress-controller" "source"={"Type":{"metadata":{"creationTimestamp":null},"spec":{},"status":{"loadBalancer":{}}}}
I0726 23:36:01.553525       1 :0] kubebuilder/controller "level"=0 "msg"="Starting EventSource"  "controller"="alb-ingress-controller" "source"=
I0726 23:36:01.553769       1 :0] kubebuilder/controller "level"=0 "msg"="Starting EventSource"  "controller"="alb-ingress-controller" "source"={"Type":{"metadata":{"creationTimestamp":null},"spec":{},"status":{"loadBalancer":{}}}}
I0726 23:36:01.553806       1 :0] kubebuilder/controller "level"=0 "msg"="Starting EventSource"  "controller"="alb-ingress-controller" "source"=
I0726 23:36:01.553955       1 :0] kubebuilder/controller "level"=0 "msg"="Starting EventSource"  "controller"="alb-ingress-controller" "source"={"Type":{"metadata":{"creationTimestamp":null}}}
I0726 23:36:01.554383       1 :0] kubebuilder/controller "level"=0 "msg"="Starting EventSource"  "controller"="alb-ingress-controller" "source"={"Type":{"metadata":{"creationTimestamp":null},"spec":{},"status":{"daemonEndpoints":{"kubeletEndpoint":{"Port":0}},"nodeInfo":{"machineID":"","systemUUID":"","bootID":"","kernelVersion":"","osImage":"","containerRuntimeVersion":"","kubeletVersion":"","kubeProxyVersion":"","operatingSystem":"","architecture":""}}}}
I0726 23:36:01.554568       1 leaderelection.go:205] attempting to acquire leader lease  kube-system/ingress-controller-leader-alb...`
How do I get the second alb-ingress-controller to cycle through the provisioning of the second alb?

Hi, do you end up being able to solve this issue?

At the time of our implementation, AWS had some limitations in regards to the alb-ingress-controller.
I ended up spinning up a single instance of the controller without the --watch-namespace switch. Each namespace then has its own ingress in which you specify the ingress-class. This is in turn provisions an alb with the ingress rules specified. Hope that helps.

For anyone who stumbles upon this, the trick is to create a new config map ( kubectl create configmap second-ingress-controller-leader-alb -n kube-system)  and specify the same in the alb-ingress-controller.yaml file with “- --election-id=second-ingress-controller-leader-alb” https://github.com/kubernetes-sigs/aws-alb-ingress-controller/blob/master/cmd/options.go also has boolean “election” and I am not sure what would  be the effect of setting “election” to false.
In my setup , I have one ALB Ingress Controller on private subnet and one ALB Ingress Controller on public subnet.


Now playing history should update existing songs' last played date

It does work but didn't the first night


How to integrate wordcount plugin with ckeditor4-react?

Hi,
Kindly let me know, How to integrate wordcount plugin with ckeditor4-react?
`import CKEditor from 'ckeditor4-react';
<CKEditor
data={content && content}
onBeforeLoad={ CKEDITOR => {
// If namespace was already loaded, this callback will be never called!
// There will be only one attempt to add external plugin.
// console.log("PPPPPP", CKEDITOR.plugins);
CKEDITOR.plugins.addExternal( 'wordcount', '/node_modules/ckeditor-wordcount-plugin/wordcount/', 'plugin.js' );
}
}
onChange={onChange}
config={{ ...ckEditorConfig }}
id= "description"
name= "description"
  />`

Sorry but i dont know how that works, but according to the documentation you would need something like this....
<CKEditor 				initData="This is a CKEditor 4 WYSIWYG editor instance created by ⚛ React." 				config={{ 					extraPlugins: 'wordcount' 			/>

i tested the code and it works with react...
function App() {
	return (

				<CKEditor
					debug={true}
					 onBeforeLoad={ CKEDITOR => {
            CKEDITOR.plugins.addExternal( 'wordcount', '/node_modules/ckeditor-wordcount-plugin/wordcount/', 'plugin.js' );
        }
    }
					config={{
					toolbar: [
						[ 'Source' ],
						[ 'Styles', 'Format', 'Font', 'FontSize' ],
						[ 'Bold', 'Italic' ],
						[ 'Undo', 'Redo' ],
						[ 'EasyImageUpload' ],
						[ 'About' ]
					],
					extraPlugins: 'wordcount'
				}}
					initData="
Hello world!
"
				/>

			{`Running React v${ React.version }`}

	);
}

export default App;

Thanks @w8tcha .
still i am  getting below error.
Line 29:14:  'e' is assigned a value but never used      no-unused-vars
printWarnings @ webpackHotDevClient.js:138
handleWarnings @ webpackHotDevClient.js:143
push../node_modules/react-dev-utils/webpackHotDevClient.js.connection.onmessage @ webpackHotDevClient.js:210
plugin.js?t=L4K9:1 Uncaught SyntaxError: Unexpected token '<'
ckeditor.js:253 Uncaught TypeError: Cannot read property 'icons' of null
at window.CKEDITOR.window.CKEDITOR.dom.CKEDITOR.resourceManager. (ckeditor.js:253)
at window.CKEDITOR.window.CKEDITOR.dom.CKEDITOR.resourceManager. (ckeditor.js:253)
at n (ckeditor.js:248)
at Array.r (ckeditor.js:248)

Sorry but then I can't help you

Okay. No problem. But i think error in your plugin. Have you integrated wordcount plugin with React and ckeditor4-react component? Please try.

Yes i tried it with the above code i posted it worked without problem

Then Kindly share me git repo.

I uploaded the react sample to ..
https://github.com/w8tcha/CKEditor-WordCount-Plugin/tree/master/wordcount/samples/ckeditor4-react/basic


my_total_turns_spent() to return the sum of all $location[] turns_spent

Keeping track of the turns in a separate Integer in AdventureSpentDatabase to prevent the need to reducing the Map values every time the function is called

This would be extremely useful to me! I do enough free fights that I can't really use total_turns_spent to determine if I've done something meaningful during my last turn (as opposed to, say, hitting a june cleaver NC or zone intro NC or whatever)
That being said, I think my_delay_turncount might be a little more natural a name. I'm mostly thinking about when it gets camelcased in js: myDelayturncount vs myDelayTurncount

ya i'll rename it when i get home

Naming-wise: how is delay related to this? It looks like it's more the total combats. If they were in a delayed zone, you'd be reducing delay, but they aren't necessarily.

Agree with Ryo - the naming sounds odd.

I'm not attached to the function name i just didn't want it anyone to confuse it with my_turncount()

changed it to my_total_turns_spent


Generate code snippets for $search queries

Snippets aren't generated for queries that include the $search query parameter. This is seen both in docs and Graph Explorer.

Doc examples where snippets aren't generated: Example 6 and Example 7 of the List users article.

Screenshot from Graph Explorer:

Desired outcome
Code snippets be generated for queries with $search. This is good for developer self-service.

Update: I see code snippets are now being generated. However, the feedback from customers is that the C# snippets aren't completely accurate, for example, Example 1 and Example 2. The snippets don't have the variable declarations.
Kindly assist to look into it @andrueastman .
Thanks.

Hi @adhiambovivian, @andrueastman; I believe the issue has also been confirmed in the Java SDK as per issue 15305. I don't have the privilege to add the Java label to this issue.


Improving Table performance by caching eloquent queries

Description
The "problem"
In a ListRecords page, every time an Action is clicked the database query is re-executed.
(In fact this happens every time Livewire "talks" to the server)
A single action with a form seems to trigger the exact same query to the DB at least 3 times (4 times if it requires confirmation).
With high number of records and/or complex queries (e.g. where the sorting is done on a related table column) this has a significant impact in the user experience.
Proposed solution
This PR aims to improve the performance by leveraging Laravel's cache and avoid extra queries.
How it works:

this feature must be enabled explicitly by setting protected bool $shouldCacheQueries = true; in any Page that extends ListRecords
cache key is defined by the query's SQL
cache is tagged using the Component's id.
queries are cached by a default of 600 seconds
cache is flushed anytime an Action executes successfully, to reflect changes in the table.

Notes:

This solution relies on a cache driver that supports tagging, such as Redis.
If not available, queries are simply not cached.
If enabled without this support, will throw an exception.

I'd love to get some feedback, even if only to kick-off a discussion on how to improve such scenarios.
Code style

[X] composer cs command has been run.

Testing

[X] Changes have been tested.

Documentation

[ Not yet ] Documentation is up-to-date.

I personally suggest leaving this in userspace since there is currently no caching strategy for Eloquent queries in filament. You do have full control to implement query caching that way.

Hey, I agree that this can be a very subjective implementation, so worth a plugin or something. Thanks anyway!

I personally suggest leaving this in userspace since there is currently no caching strategy for Eloquent queries in filament. You do have full control to implement query caching that way.

Could you specify (high-level) how to achieve this in userspace?
Can't find anything in the docs that allows me to override the query "execution"....

All the methods you implement in this PR exist on the List page class, from the InteractsWithTable trait

I'll (re)check that.
Many thanks!!


adds DMG format option

Would fix #8, tested with hdiutil
$ create-dmg -f UDZO fixture.app
$ hdiutil imageinfo fixture-0.0.1.dmg | grep Format
Format Description: UDIF read-only compressed (zlib)
Format: UDZO

I'm intentionally not exposing an option for this as the current format works for most people. No good reason to support macOS versions older than 10.11. macOS upgrades are free. If you really need this, you could always fork and publish a version with more options.

Well, at least some people seem to be using older versions of OSX/macOS, see gillesdemey/Cumulus#92.
Also, what about people having a company not to upgrade, need other software that only works on older versions of OSX, or just non-tech savvy people who have no clue upgrading is a good idea?


Strange lines in Field

This part of the code of Field:
https://github.com/TriOptima/iommi/blob/84c2cf220fd4549a7bedc028605557c6d031dc37/iommi/form.py#L718-L724
seems to be a copy of sorts of these lines in Query:
https://github.com/TriOptima/iommi/blob/84c2cf220fd4549a7bedc028605557c6d031dc37/iommi/query.py#L260-L266
And indeed self.search_fields does not seem to be used in Field anywhere else.  Bad case of copy paste?

search_fields is used by choice_queryset__extra__filter_and_sort

Doh. Missed that.  Still sad at the code duplication.  We are missing some abstraction I guess.  Also I still dislike iommi using the extra namespace.  Or more precisely I dislike the whole extra namespace ...


Get Street, address, city, state from Prediction object.

How to get street, address, city, state from Prediction object.  There is a 'terms' list field that have the information but how to know where in the list that information is present.
For reference, in one of my javascript app, I am using google autocomplete and to get specific information I did the following:
let place:Place = new Place();
    googlePlace.address_components.forEach(a =>{
      if(a.types.indexOf("street_number") > -1){
        place.streetNumber = a.long_name;
      }else if(a.types.indexOf("route") > -1){
        place.address = a.long_name;
      }else if(a.types.indexOf("locality") > -1){
        place.city = a.long_name;
      }else if(a.types.indexOf("administrative_area_level_1") > -1){
        place.state = a.short_name;
      }else if(a.types.indexOf("postal_code") > -1){
        place.zip = a.short_name;
      } else if(a.types.indexOf("country") > -1){
        place.country = a.short_name;
      }
    });

to get details about the prediction, you need to pass the place_id to the getDetails() function
https://pub.dartlang.org/documentation/google_maps_webservice/latest/google_maps_webservice.places/GoogleMapsPlaces/getDetailsByPlaceId.html
then you will received address_components


⚠ Bofrost.at has degraded performance

In 71cd520, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1061 ms

Resolved: Bofrost.at performance has improved in 9910e0f.


Preliminary BMS app, timer and clock change, CAN changes

Added additional macros. Created a basic timer (TIM6) to interrupt every 1 second (for sending the BMS charging info to the charger while charging). Created a preliminary BMS application state machine. Included a screenshot of the clock diagram used to derive the timer (HSI->PLL->SYSCLK->HCLK->PCLK1->APB1_TIM6_CLK). Created a TinyBMS Initialization function.

good job!


Add sexyLoot

sexyLoot (For Pleasure)
Etherscan link: https://etherscan.io/address/0xb13efab167751a95ec6f9844944c74aa668dded6
Website: https://opensea.io/collection/sexyloot
Twitter: https://twitter.com/LootSexy

Could you please resolve the conflict?

What is the conflict? Sorry I don't understand. I'm not a dev. I followed instructions of forking the list of projects, adding mine, and submitting a pull. It's all a very complicated way to just supply the name/link/etherscan.
I can't see any button or label or indication of a conflict, or what to click, etc.

It means people have push to the main branch of code after you submit this PR, and your code branch is outdated and have difference with the main branch.
If you don't know how Git works, maybe the simplest way to do it is to:

go to your fork https://github.com/sexyLoot/website
make sure you are on main branch (it should be)
You will see "This branch is 10 commits behind lootproject:main.", click "fetch upstream" and then "fetch and merge"
close this pull request, and open a new one like what you did before. (don't open with patch-1 branch, open with your new branch).

Another way to do it is to delete your fork, and then refork this project, and make your changes then submit a pull request.
It is way easier to do if you know how Git works, but otherwise, just follow my steps above and it should work. Just make sure you copy your few lines of changes before you delete your branch, in case you need your change.

Thanks kindly, will do! Appreciate the tips, extremely good of you to help.


Custom skull check error

In the latest version, this plugin treats all heads as incorrect items.
How to reproduce: Use the headdatabase and try to get the head
Paper version 1.21-130-master@b1b5d4c (2024-08-10T10:02:42Z) (Implementing API version 1.21-R0.1-SNAPSHOT)
Panilla version 1.12.2

Should be fixed in 1.12.3


Best practice to restricted a draggable item from being dragged into a specific droppable item?

Hi !
We are currently implementing DND within an application and need to control which draggable elements can be dragged in specific droppable elements.
The docs (section: useDroppable) show an example using the data argument with a custom "accepts" parameter. This is exactly what we need, but I am now trying to figure out how we set the restriction.
Do we need to set the droppable element to "Disabled = true" if the droppable item doesn't accept the item? Or is there a better way?
function Droppable() {
  const {setNodeRef} = useDroppable({
    id: 'droppable',
    data: {
      accepts: ['type1', 'type2'],
    },
  });

  /* ... */
}

Example: we are creating a dashboard where each element is a "block". (Nav items, nav groups, navbar etc are all blocks). The nav_grop block has a parameter: accepts:[nav_item, button, input] so we want to make sure that that only these blocks can be dragged into the nav_group block.

Answering this myself as I found it in a great post: https://github.com/clauderic/dnd-kit/discussions/809


image/draw:Draw is slow on Paletted images

I've been investigating an issue where drawing Paletted images seems much slower than other types of image conversion.
AFAIK, it's drawing the image twice?
https://github.com/golang/go/blob/f9ed2f75c43cb8745a1593ec3e4208c46419216a/src/image/draw/draw.go#L178
It executes the "fast" drawPaletted image, then falls into the default case, and executes the "slow" drawing path? I think there's supposed to be a return there?

@nigeltao?

You're absolutely right. I'll send out a fix.

thanks Nigel! that was quick :dash:


Players cannot move themselves and ridden items by dragging their token.  GMs can.

I thought this was a setting in the configuration.  And, I do have (under 'Client Settings') the "Rider Movement" set to "Move ridden Token (if owned).
However, when a player tries to move their token, I (on the GM side) see a message saying "Still riding...." but the player can't move the ridden token - and their token does not move.  When I, as the GM, move their token (the one riding) both their token and the ridden token move appropriately.
Rideable v2.6.0
Foundry v 11.307
Browser - latest Chrome

Just to make sure, does the player also have the Rider Movement set to Move ridden Token?

Ah... no, they did not.  They, and I, did not know to set that.  Sorry.
Suggestion - maybe create a setting where the DM can "default" the setting to "Move Ridden Token", or whatever they choose.
That fixed the problem for them.  Thanks!

As far as i know the GM can not actually influence the players/clients settings (without a specific module), so i can not really set a default as such. I will keep an eye on it though, maybe i find an alternative.

Ah... ok.  I was thinking it could be done similarly to how "Auto Rotate" allows tokens to be set to "Global (Default)", "On", or "Off".  With the Global (Default) being whatever the GM has set as the default in the config settings.  Hope that made sense.  :)

Thats agood idea, i will look into it in one of the next releases


Move received file to final destination only after writing is complete?

When another application polls a directory, into which the FTP server writes a file: is there a possibility to check whether the file is completly written? A simple solution would be that the FTP servers downloads the file's content to a temporary file, which is moved to the final destination only after having received the complete data. Is something like this possible with pyftpdlib?

Take a look at the docs. There are callback methods of FTPHandler class that are invoked on successful download/upload. You can override them and put logic in there.

Thanks! That will help to solve the problem.