Support BYOS feature in api version 2021-09-01-preview

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

spring cloud

Please add some tests for this new change

/azp run


zprintrc with comments fails with 1.0.0 native image on linux

Downloaded zprintl for the 1.0.0 release off GitHub. Following zprintrc:
{:map
 {:comma? false
  ;; lift-ns? will make deps.edn entries with just {:mvn/version "..."}
  ;; turn into #:mvn{:version "..."}, which is less legible.
  :lift-ns? false
  :key-order [:type :mvn/version :git/sha]}}

results in:
$ zprint
Unable to read configuration from file /home/lvh/.zprintrc because java.lang.RuntimeException: EOF while reading, starting at line 1, compiling:(null:1:1)

Removing the comments, but otherwise keeping the zprintrc identical, works as expected. In order to check I'm not missing the obvious, I tried to use jet < ~/.zprintrc to parse the file and it works fine:
{:map {:comma? false, :lift-ns? false, :key-order [:type :mvn/version :git/sha]}}

Maybe this is because we're still trying read-string on Graal and we're missing an exception case?

Thanks for noticing this. Near as I can tell, comments in a .zprintrc have never worked.   This was masked by a bug which I fixed in 0.5.4, issue #126.
It was caused by the way that zprint reads the lines from the file and then reassembles them into a string. It left out the new lines during reassembly.
This will be fixed in 1.0.1.
Thanks again for letting me know about this!

It took me some time to figure out why my custom settings weren't taking effect. Eventually, I realized it was the comments in the config that were causing the problem.
It might be worth mentioning in the readme that .zprintrc and .zprint.edn should not contain comments. I don't think it's mentioned anywhere, although I could be blind.
Besides that small issue, though, zprint is awesome. You've done an incredible job, thanks for sharing it! 🍻

Sorry this caused a problem for you too.  I never documented it because I didn't realize comments didn't work, never having tried it myself. I have fixed it so that comments are now supported in .zprintrc files and the fix will ship in the next release, which should be in the next few weeks.
Thanks for the positive feedback.  I never really know if people are actually using zprint or how they feel about it except for the comments like yours.  It means a lot, so thanks again.

I have fixed it so that comments are now supported in .zprintrc files and the fix will ship in the next release, which should be in the next few weeks.

That is great news! Thank you!

I never really know if people are actually using zprint or how they feel about it

Yeah, I think open source projects could do with more positive feedback. People should know their work is appreciated.

This is now available in 1.0.1.  Thanks for letting me know about it!


[Bug]: Release does not load without clearing the browser cache

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
https://release.app.appsmith.com is working only after clearing the browser cache
{"responseMeta":{"status":500,"success":false,"error":{"code":500,"message":"Internal Server Error"}}}
Steps To Reproduce
Try to load https://release.app.appsmith.com without clearing cache
Environment
Release
Version
Cloud

This should be fixed after the migration ran on release, closing this till we have any further issues reported.


Using API resources from a Gen 1 Project to a newly created Gen 2

Environment information
System:
  OS: macOS 14.6.1
  CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
  Memory: 1.91 GB / 64.00 GB
  Shell: /bin/zsh
Binaries:
  Node: 20.11.1 - ~/.nvm/versions/node/v20.11.1/bin/node
  Yarn: undefined - undefined
  npm: 10.2.4 - ~/.nvm/versions/node/v20.11.1/bin/npm
  pnpm: undefined - undefined
NPM Packages:
  @aws-amplify/auth-construct: 1.2.0
  @aws-amplify/backend: 1.0.4
  @aws-amplify/backend-auth: 1.1.0
  @aws-amplify/backend-cli: 1.2.1
  @aws-amplify/backend-data: 1.1.0
  @aws-amplify/backend-deployer: 1.0.2
  @aws-amplify/backend-function: 1.3.0
  @aws-amplify/backend-output-schemas: 1.1.0
  @aws-amplify/backend-output-storage: 1.0.2
  @aws-amplify/backend-secret: 1.0.0
  @aws-amplify/backend-storage: 1.0.4
  @aws-amplify/cli-core: 1.1.1
  @aws-amplify/client-config: 1.1.1
  @aws-amplify/deployed-backend-client: 1.1.0
  @aws-amplify/form-generator: 1.0.0
  @aws-amplify/model-generator: 1.0.2
  @aws-amplify/platform-core: 1.0.3
  @aws-amplify/plugin-types: 1.1.0
  @aws-amplify/sandbox: 1.1.1
  @aws-amplify/schema-generator: 1.2.0
  aws-amplify: 6.4.3
  aws-cdk: 2.150.0
  aws-cdk-lib: 2.150.0
  typescript: 5.5.4
AWS environment variables:
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables

Description
I have this project where I am reconstructing this React app into a Flutter one for hardware performance:
https://aws.amazon.com/blogs/apn/how-to-build-a-fintech-app-on-aws-using-the-plaid-api/
which was made in 2022 that uses Gen1 stuff.
I need help taking the API and Lambda functions made by that project and adept them to my Flutter code, so I basically copied the same backend files into my amplify folder and then pushed it to amplify on the CLI.
Thing is, I started building my project following the Gen2 instructions for Typescript backend manipulation, but when trying to merge the tutorial's backend into my project, it started using Gen1 standards.
so now I have an
amplify_outputs.dart

and
amplifyconfigurations.dart

In my amplifyconfigurations.dart file, I tried adding an api section and filling in the blanks with info from the already made resource I have.
Can someone please guide me on how to merge these resources or how to properly build it again for my code to call?

Hey @bebopinbebop, thank you for reaching out. Amplify Gen 1 and Gen 2 follow different architectural and tooling paradigms. I would not suggest moving the moving your Amplify Gen 1 files into a Gen 2 application or vice versa.
refer to this document providing this information: https://docs.amplify.aws/react/how-amplify-works/faq/

Closing the issue due to inactivity. Do reach out to us if you are still experiencing this issue


Please provide help, thanks - pending more details and example of mouse scroll

win10系统在utf-8环境下(Active code page: 65001)如果执行python代码文件的路径名包含中文则click，type这些函数无法执行(执行后没反应，换成非中文路径正常)，希望能修复，谢谢。

希望后续能提供滚动鼠标的功能函数

Hi @lixiao888 for 1, can you share with me a sample script that can replicate the error and what is the error message? Can you also set r.debug(True) when you run the Python script to see if there are any other errors?
For 2, you can use vision() and send the custom command to scroll the mouse. Take note to use visual automation to click on the window you want first to make sure the OS focus is on that window.
r.init(True)
r.vision('Mouse.wheel(WHEEL_UP, 1)')
r.vision('Mouse.wheel(WHEEL_DOWN, 5)')

The 1 and 5 above means the number of steps to scroll the mouse.

问题1代码如下：
import rpa as r
r.init()
r.url('https://duckduckgo.com')
r.type('//*[@name="q"]', 'decentralisation[enter]')
r.wait() # ensure results are fully loaded
r.snap('page', 'results.png')
r.close()
错误日志如下：
START - automation started - Wed Apr 19 2023 16:18:35 GMT+0800 (中国标准时间)
LIVE MODE - type done to quit
[RPA][STARTED]
[RPA][0] - listening for inputs
[RPA][1] - https://duckduckgo.com
[RPA][1] - listening for inputs
ERROR - inconsistent quotes in text
ERROR - can't find variable: exist
[RPA][2] - listening for inputs
[RPA][3] - dump exist_result to rpa_python.txt
ERROR - can't find variable: exist_result
[RPA][3] - listening for inputs

@lixiao888 the error messages from issue 3 looks to suggest that you are not using the TagUI that comes with the Python package but the TagUI human language version. Because the error is syntax differences.
Check this location print(r.tagui_location()) and in that folder, delete the tagui folder. Then run again to let rpa package auto-downloads the right TagUI engine to use in the background.
Do you have TagUI installed separately?

感谢，问题3已解决，问题1还是不行

@lixiao888,
I think you haven't share the run log for problem 1 after fixing problem 3. Can you share the run log and a sample script that creates the problem?

问题1错误日志如下：
START - automation started - Wed Apr 19 2023 16:18:35 GMT+0800 (中国标准时间)
LIVE MODE - type done to quit
�[RPA][STARTED]
[RPA][0] - listening for inputs
[RPA][1] - https://duckduckgo.com/
[RPA][1] - listening for inputs
ERROR - inconsistent quotes in text
ERROR - can't find variable: exist
[RPA][2] - listening for inputs
[RPA][3] - dump exist_result to rpa_python.txt
ERROR - can't find variable: exist_result
[RPA][3] - listening for inputs
代码如下：
import rpa as r
r.init()
r.url('https://duckduckgo.com')
r.type('//*[@name="q"]', 'decentralisation[enter]')
r.wait()
r.snap('page', 'results.png')
r.close()

@lixiao888 the error messages from above looks to suggest that you are not using the TagUI that comes with the Python package but the TagUI human language version. Because the error is syntax differences.
Do you have TagUI installed separately?
Check this location print(r.tagui_location()) and in that folder, delete the tagui folder. Then run again to let rpa package auto-downloads the right TagUI engine to use in the background.

Closing for time being until further inputs


💡 Add "JavaScript" Complete Resources

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
Hey, @jfmartinz
Add javascript complete resources in the programming languages folder js.md
Explain your solution
I will try to include:

Roadmap
Basics Syntax and Basic Constructs
Control Structures: Conditionals (if, else if, else, switch) Loops (for, while, do-while)
Functions
Tools and IDEs
Intermediate JavaScript
Object-Oriented Programming (OOP)
Asynchronous JavaScript: Callbacks, Promises, Async/Await
Error Handling
DOM Manipulation
Advanced JavaScript Topics
Frameworks and Libraries
Back-End Development with JavaScript
Projects
Courses
Youtube channel
And Many more (If any)

Any alternative approaches/features
No response
Additional Context
No response

Hye please take note the instruction I made in your other issue which is about c++, thanks.

@jfmartinz Please assign this issue to me under GSSoC24. I will do this work


Add maxWidth/maxHeight options for legend

Closes: #2770
Related: #3761

some chore with the helpers/constants

its doesn't work guys

Any info on a fix or workaround for maxWidth?


Cannot create new game in r887 - Servers and Cores not listed

Upon logging in to the beta server running build 435 release 887 and selecting "Create Game" on the Games List screen, the correct "Create A New Game" window launches but is not populated with server and core choices.
Imported from trac ticket #199,  created by Bard on 08-03-2010 at 20:47, last modified: 08-03-2010 at 20:52

the maps.txt, cores.txt and ranks.txt and balance.txt madness has been assimilated to allrv.ini which lives in the appdir of allsrv.exe (along side).  See http://trac.alleg.net/attachment/ticket/192/allsrv.ini or get it from the latest installer.
Trac comment by Imago on 08-03-2010 at 20:49

I've added the allsrv.ini to CDN, Saisoma or whoever can update BETA?   It's not going to be me, I'd rather have the actual server-op deal with it to learn.
Trac comment by Imago on 08-03-2010 at 20:52


FreeBSD support

This completes the implementation for FreeBSD. I can't really tell if it's accurate: I don't know enough about my own hardware / ZFS pools to say if the output is plausible. While compiling something else it briefly looks like:
 zroot                                         0.00       0.00       0.00       0.00       0.00       0.00
      default                                  0.00       0.00    7647.04       6.81       0.00       0.89
   tmp                                       992.28       0.22     496.14       0.22       0.23       0.45

went with @Freaky 's implementation instead after asking both parties for input.


add enable gpu convolution tests flag in py

@rkimballn1 good point, let me change it.

updated with a skip list, just add the backend name that need to skip in the list. it will create a string to skip the test.

@diyessi add a flag to make comments.
"TODO: Need to implement this op for GPU later;"

TESTNOW


Not crawling anything

This software doesnt crawling anything for me

@dubems without application log, server log, or anything else that describe the issue on your machine, i don't see what we can do...

Same here?

How can we resolve it?

How can we resolve it?

By providing backend and frontend error logs, screenshots, context, etc...


⚠ Spryr has degraded performance

In 0fa98b5, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 6359 ms

Resolved: Spryr performance has improved in b507216 after 11 minutes.


Add Geoffrey Wong to Contributors list

@GeoffreyHW, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


Specimen terms request to be added into CIDO

Below is a list of verbatims copied from the EUA documents with limited redundant check. Those terms include synonyms, alternative terms and acronyms. Please sort it out with correct hierarchies before adding into CIDO.


Add check for LP1904580

Check for 0644 permission error in nova logs.
closes #383

lgtm thanks rodrigo!


value not used in self attention

Hi, thanks for porting the official tf code to pytorch.
I read this self attention code and found that value is not used. I think this will likely affect performance, though perhaps quite subtly, and likely undetected w/o evaluating log-likelihoods and FID.

Dear Xuechen,
Thanks for reporting this issue. You're absolutely right, and I'll correct and test it soon.
Yeah, model selection using FID, as in the original paper, is very costly and cannot be done with my limited computational resources.
Thanks again,
Hadi.


Add options to ignore element's ID

This might be a particular need of our team, but we need to be able to ignore the IDs of the elements in the dom.
This PR adds an optional argument options to getXPath, enabling to ignore the node's ID:
getXPath(element) // returns '//*[@id="foo"]'
getXPath(element, { ignoreId: true }) // returns '/html/body/span'

It was designed so that if new options are needed, this should be fairly easy to extend.

Hello @vincent-psarga !
Thank you for the contribution.


Provide user/password for pushing via pom.xml

We have a build process where the user/password for the deployment are passed in on the build server via -Ddocker.user=XXX -Ddocker.pass=XXXX
We're currently using the fabric8 plugin, which allows auth configuration like
<configuration>
   <pushRegistry>${docker.push.registry}</pushRegistry>
   <authConfig>
      <username>${docker.user}</username>
      <password>${docker.pass}</password>
   </authConfig>
</configuration>

Would you consider this as a PR? Our build servers are docker in dockers. They're 100% transient with nothing on the filesystem. The settings.xml and the docker-cli options won't work there.
Thanks

I've deployed a snapshot copy of this to our local maven mirror, and it worked as expected.  The PR has no unit tests, but there didn't seem to be any existing push tests to augment.

This was merged. Closing


Custom exception for invalid inputs

Fixes #70
Defines the exception GMTInvalidInput and uses it throughout the
library.
Include a section in CONTRIBUTING.md about checking inputs and outputs and
which exceptions to raise.

@seisman just a heads-up about this change.

Got it. The change seems good to me.


Hovering over any CPU graph in System Information crashes with 0xC0000005

Brief description of your issue
app crashes with 0xC0000005 (Invalid access to memory location.) whenever hovering over CPU graphs

Steps to reproduce (optional)
Steps to reproduce this app crash:

open System Informer
open System Information
enable "Show one graph per CPU"
move mouse cursor into any of the graph boxes
System Informer has crashed :( 0xC0000005 (Invalid access to memory location.)

Expected behavior (optional)
app kindly not to crash :)
Actual behavior (optional)
meeting black hole faster than lightspeed similar to disappearance of MH370 thoroughly analyzed by Ashton Forbes
Environment (optional)
System Informer 3.1.24298.0 Release
Windows 11 Version 23H2 (OS Build 22631.4317)

@StanleyChristmas could you email me the dump file<EMAIL_ADDRESS>

This bug was introduced in the following commit inside of PhIsCoreParked: https://github.com/winsiderss/systeminformer/commit/ec374c3c42f0316702c541478f54a1fda5767181
The number of cores aligns with the initialBufferSize boundary of 4098... (128 * 32) = 4096. So we never add the necessary extra RTL_SIZEOF_THROUGH_FIELD(SYSTEM_CPU_SET_INFORMATION, Size) and the loop reads past the bounds of the buffer.

Fixed here: https://github.com/winsiderss/systeminformer/commit/84a0a3161d1afe11c4f8dd98ee27833eb63b68e4
Will be in next build 👍

Does the mentioned fix cover a crash with the same code ( 0xC0000005 ) but achieved differently? I can make it happen by double-clicking a process in the tray icons (I have ones for both memory and CPU).


Fix-ups prior to "Module Resolution Strategies"

Some fix-ups to #181. @mhegazy

No more comments for me. off to @sandersn

I will hold off on comments in #181 to make the merge easier.


Can it slide in from left?

Could you please make so it supports sliding in from left? Thanks

Yes you can ... find the following function

(void)swipeLeftHandle:(UIScreenEdgePanGestureRecognizer *)recognizer{

and replace the following code
if (_slideMenu.ll_isOpen) {
return;
}
with this
if (_slideMenu.ll_isOpen) {
[_slideMenu ll_closeSlideMenu];
return;
}

Great! I'll test it as soon as possible and will let you know :)

yeah sure :)

Hi, my bad! I think it defaults sliding in from the left, but how do I make it slide in from the right side? Thanks.

Sorry, the current version does not support the slide from the right, if you want to, you can modify the direction of the LLDisplayLayer drawing and coordinate positions, as well as touch gestures listening :)

If I want to contribute to this repo, do I create a fork and then do all the changes locally, and then create a pull request?

Yes, of course!   :)

Great! I'll try to start working on it asap :)


bblayers.conf.sample: add meta-perl

Perl is required for building efitools used for
EFI boot entry configuration and secure boot,
Changelog-entry: add meta-perl to bblayers.conf
Signed-off-by: Alex Gonzalez<EMAIL_ADDRESS>

lgtm


Skip test_extractor only for zstd param if zstandard not installed

Currently, if zstandard is not installed, test_extractor is skipped for all compression format parameters.
This PR fixes test_extractor so that if zstandard is not installed, test_extractor is skipped only for the zstd compression parameter, that is, it is not skipped for all the other compression parameters (gzip, xz,...).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Adding 'attachments' mapping for BotMessages

Adding mapping for existing payload: "attachments" in (incoming) BotMessages

Merged. Sorry for the long delay! I'll be doing some additional verification testing, a minor version bump, and then I plan on releasing to maven central shortly.


Make Listener work with Browse Result

First of all thanks for your great work with this plugin. I love it.
Problem: If you want to transfer huge amounts of variables to a sink with limited afford, it's working great with Browse & Read. The drawback is, that you spam the sink with lot of near-static values (with changes less then once a day) with the given polling rate.
Solution: Make the Listener node work with Browse results, like the Read node does already today.
Right, I could code a function transforming a Browse result to a msg-style the Listener is able to read by myself. But subscribe is such a great function for minimizing data from opc ua that this might be a nice enhancement for other users as well.

Thank you!
It is not clearly, what are you want to do. In normal case you send one subscription to all variables to listen for. Then the Listener works as your settings set the options (publish, sample interval ...) - means you do not need a next inject to the listener from here if you just want to listen to the same variables all the time.
Please, describe more in detail, what is expected!
cya
Klaus

The flow to poll many variables at once looks like that:

The flow for subscribe many variables at once should look like that (in my opinion):

But that does not work. Therefore i have to enter all the UA-nodeIDs manually in the inject node and do it like this:

To generate the variable-list for Listening automatically and keep it on date (when add or remove UA-Nodes from the server) the approach with Inject -> Browse -> Listen would be great. In other words: The Listener should be able to deal with the output message of the Browser, like the Read-Node does.

like that?

please test v2.0.12 or above

Take a look to the whole msg object. There you will find more information.
Sent with GitHawk

It is the whole msg object. There is just the _msgid outside of the payload.
Like I said: In the upper output message there is an array "addressSpaceItem" with the field "nodeID". But this message contains also huge arrays input.addressSpaceItem and input.nodesToRead.
In my opinion the "Send just values" property in the "Listener"-Node makes no sense. For me it should be like:

"Send just values" is ticked: The Listener just sends the light-weight message I screenshoted above (with added field containing the node-ID).
"Send just values" is not ticked: The Listener sends the big message with all the information.
Reason: I'm not sure if there is an influence on performance if you spam this huge message hundred times a second. On a big machine there will be no problem.  I have the fear to book out the device with that hundreds of huge messages.

Therefore my suggest: One option for big machines with that huge all-containing message (you provide already at the upper output) and one option for light-weight on-the-edge devices with a minimum footprint message (you provide already at the lower output+nodeID). And the user can choose with the "Send just values" property.

This is the message of the upper output.

If you subscribe 1000 variables it generates at least on start-up 1000 messages with the content:

And all the 1000 messages will have two arrays with again 1000 objects each (yellow highlighted in screenshot). This means 2 Millions objects are generated at once. For small devices this could be a problem.
So my point is:

Property to switch on/off this upper output. But not just the output itself, the creation of those objects internal as well.
Give the lower output a new field named node-ID, because the lower output is really fine. Enough for 99% of all users and still very small. But there is no way to identify the variable, yet.

Yes, you are right and I know that. I am working on the structures step by step. The API of node-opcua becomes more easy so it will become less spam in the future in communication, but I need time to do all the work ;)

next version event msg

next version monitor msg 1 & 2

please test with v2.1.0


Refactor code to place constant for 'client' URI in common place

@pavolloffay First of the hopefully simple PRs indirectly related to the zipkin PR.

LGTM :+1:

@pavolloffay Thanks, will merge once build complete.


Fixup CI.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@64254be). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   84.61%
=======================================
  Files           ?        1
  Lines           ?       39
  Branches        ?        0
=======================================
  Hits            ?       33
  Misses          ?        6
  Partials        ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Can we get the rest of the (non-english) glyphs?

I would really like the ability to use emojicons in future EarthBound Text gifs.
Thanks.

Hey @w4ffl35, thanks for the suggestion. I can't imagine that standard emoji would feel very "EarthBound-y" in the generator. I'm pulling the glyphs for the text generator from glyphs programmed directly into the game, and I think I'd like to support more of those before I go on adding new glyphs.
Also, given the detail of emoji, I don't know if it would look out of place given the pixelation of EB text boxes in general.
I'll keep this open in case I get the urge to put any non-canon glyphs in the generator. Thanks.

"I'm pulling the glyphs for the text generator from glyphs programmed directly into the game, and I think I'd like to support more of those before I go on adding new glyphs." aaah this was my intention rather than requesting 'non-canon glyphs'. Adding those seems to be an entirely different beast than just utilizing the the rest of the glyphs in flavored/plain.png!

Yeah, adding the existing glyphs in flavored/plain.png would be easier and more logical. I have received requests for the saturn font portion as well. It's just a matter of coding in the offsets, mapping in the inputs that correspond to these, and probably creating a switch for the Saturn font.
One other idea I've had is to create a WYSIWYG editor for this so you can see the formatting of what you'll turn into a gif text box. That way, extra characters could be placed in the editor without having some weird, complex entity system. That way having something like the SMAAAAASH graphic would be easy to do.
I need to go through and turn this whole beast from a weekend project into something real.


do not use generic names for receiver variable, like "this"

Reference:

https://github.com/golang/go/wiki/CodeReviewComments#receiver-names
https://github.com/hailiang/gosweep

This should already be fixed in https://github.com/stealthly/siesta/tree/refactor-and-lint
All these checks should pass there.
Not merged yet though. Will probably be merged together with producer.

One more note for why it's not merged yet - this branch contains some breaking changes because of Kafka naming conventions so we decided to wait a little bit before merging that.
Thanks


Voice Bonnet drivers not installed (HACKING.md instructions)

The AIY Projects docs list the following drivers for the Voice Bonnet (v2) module:

MCU driver: modinfo aiy-io-i2c
MCU PWM driver: modinfo pwm-aiy-io
MCU GPIO driver: modinfo gpio-aiy-io
MCU ADC driver: modinfo aiy-adc

However, none of these are installed after following the instructions in the latest HACKING.md. Instructions of some sort should be added to cover this setup step, as sound does not work properly without it.
(The lack of these drivers was verified with the modinfo command, as shown in the quote above, and also by comparing dmesg output with a fresh install of the official AIY Projects OS release, where sound does work as expected.)

Update: Installing the aiy-dkms package (via apt) does provide the proper drivers, and dmesg output matches that of the AIY OS image. This should be added to HACKING.md.


action fails if packer already exists on the machine

If the setup-packer action is run and the agent already has packer installed, the step fails without providing any useful context.
Expected Behavior
I would expect the step to detect that the end result of the action has been acheived (i.e. the requested packer version is present on the agent) and the step succeeds
Current Behavior
Run hashicorp/setup-packer@v3
Installing packer:1.11.1 and adding it to GitHub Actions Path

## Steps to Reproduce

1. Create a github action that targets a runner with Packer installed
1. Call the setup-packer@v2 action
1. Execute your action

## Environment

Hi @gustavlasko, thanks for reporting this!
Do you have a link to an Actions log where you encounter this? Alternatively, a workflow .yml would also work!


Provide a simple platform-independent thread_pool.

This is in preparation to using it for processing files in parallel, initially for large file-lists to symbolize in the language server.

Codecov Report
Base: 92.93% // Head: 92.95% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (ac82f17) compared to base (598dd12).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1657      +/-   ##
==========================================
+ Coverage   92.93%   92.95%   +0.01%
==========================================
  Files         350      351       +1
  Lines       25028    25066      +38
==========================================
+ Hits        23261    23299      +38
  Misses       1767     1767

Impacted Files
Coverage Δ

common/util/thread_pool.h
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Monitor][Ingestion] Update samples

Added a samples README to give users more information regarding on how to set up environment variables and run the samples.
Additional samples for uploading file contents and uploading data from a pandas DataFrame were added.
To-do;

[x] Add sample for aborting on specific error

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

/check-enforcer evaluate

Thanks @kristapratico for the helpful review!


Update to dotnet 5

Update csproj files contents and update required nuget packeges as well as minor change in call to ValidationContext(request) and change it to ValidationContext(request) in RequestValidationBehavior.cs file. Also change in global.json

Thank you for your interest in this project. This repository has been archived and is no longer actively maintained or supported. We appreciate your understanding. Feel free to explore the codebase and adapt it to your own needs if it serves as a useful reference. If you have any further questions or concerns, please refer to the README for more information.


logging confusions

Frequent stumbling blocks during the last couple of roadshows.

@jorgemoralespou Any reason we are not accepting pull reqs like this one? I see the travis-ci is failed but it looks like an auth error which should not prevent merging the content.

@jbrannst no. Just only time. And we need to make sure these are also merged in ocp-4pre branch, as updated content for the upcoming OCP 4 goes there.


dnu restore fails on Linux VM when building a docker image

Hi,
I have problem with starting  project on Linux VM with docker installed, and I'm having difficult time in building image for aspnet.
here is my project.json :
{
"webroot": "wwwroot",
"version": "1.0.0-*",
"dependencies": {
"Microsoft.AspNet.Server.IIS": "1.0.0-rc1-final",
"Microsoft.AspNet.Server.WebListener": "1.0.0-rc1-final",
"Microsoft.AspNet.Mvc": "6.0.0-rc1-final",
"Microsoft.AspNet.Server.Kestrel": "1.0.0-rc1-final"
},
"commands": {
"web": "Microsoft.AspNet.Hosting --server Microsoft.AspNet.Server.WebListener --server.urls http://localhost:5001",
"kestrel" : "Microsoft.AspNet.Server.Kestrel --server Kestrel --server.urls http://localhost:5004"
},
"frameworks": {
"dnx451": { },
"dnxcore50": { }
},
"publishExclude": [
"node_modules",
"bower_components",
".xproj",
".user",
"**.vspscc"
],
"exclude": [
"wwwroot",
"node_modules",
"bower_components"
]
}
And my Dockerfile :
FROM microsoft/aspnet:1.0.0-rc1-final-coreclr
ADD /src /app
WORKDIR /app
RUN ["dnu", "restore"]
EXPOSE 5004
ENV LASTREFRESH 2015-06-02
the error i get when i start this command ( docker build -t aspnetbase . ) is  :
Error: FindPackagesById: Microsoft.AspNet.Server.IIS
HTTP request timed out. Exiting.
Restore failed
A task was canceled.
Feeds used:
https://api.nuget.org/v3/index.json
The command 'dnu restore' returned a non-zero code: 1
help please !!
@davidfowl

No longer supported, see https://github.com/aspnet/Announcements/issues/205.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


Got weird error with multiple ORM's in master branch

I got The 'truncation' strategy does not exist for the mongoid ORM!  Available strategies: truncation (DatabaseCleaner::UnknownStrategySpecified)
in my spec_helper
config.before(:suite) do
    DatabaseCleaner.clean_with :truncation
    DatabaseCleaner.strategy = :truncation
    DatabaseCleaner[:mongoid].strategy = :truncation
end

Then I changed my gem file
gem 'database_cleaner', github: 'DatabaseCleaner/database_cleaner' to gem 'database_cleaner', '~> 1.3.0',  the error never occurred.
Is there any bug in master branch?

duplicate of #299

@takashi version 1.4.1 should fix this.

@etagwerker thank you for fixing!


Dependency gvfs missing from swaynotificationcenter

Describe the bug
swaynotificationcenter (still as of 0.10.1 at the time of writing, but ever since 0.10.0) is missing the runtime dependency gvfs which, quoting the maintainer of the project, is required to fix MPRIS covers not showing (which is the issue I am experiencing).
Steps to reproduce the behaviour, Expected behaviour, Actual behaviour & Screenshots
Please take a look at the original issue on the upstream repository linked prior, as everything mentioned there is identical (to the behaviour on my machine).
Additional context
I have read on the discussion on 0.9.0 -> 0.10.0 PR and leon-erd's question which I saw was left unresponded. This issue was opened to hopefully clear up the need for this missing dependency.
This is my first time contributing to nixpkgs, so I want to apologise in advance in case I am doing anything wrong here.
Notify maintainers
@teto
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.28, NixOS, 24.05 (Uakari), 24.05pre615148.6143fc5eeb9c`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos

Add a :+1: reaction to issues you find important.

feel free to patch the package, I can't look into it :(


Can I Restrict Generated Methods?

I have created a view and granted a role SELECT privileges access the view. OpenAPI has generated docs including GET, POST, DELETE, and PUT methods for this endpoint. Only GET works (executing from swagger), the others are denied due to lack of permissions.
Is this the expected behavior? The docs seemed to suggest that only methods that are permitted will be generated. I'm wondering if this is working as expected, or if I haven't quite configured permissions correctly.
I'm using v11.2.

Hi! This project is not for the current OpenAPI implementation that PostgREST uses, that is still working in the core repository: https://gitub.com/PostgREST/postgrest. Regarding to your issue, it has been reported here: https://github.com/PostgREST/postgrest/issues/1870, so I'm closing this in favor of that issue.


aws-alias.sh: strip quotes from returned instance id and IP address

aws-ip currently stores the IP in quotes to $instanceIp, which messes up aws-ssh. I've trimmed the quotes prior to storing in $instanceIp

I noticed this too.  The problem is happening because your AWS CLI output type is JSON.
Another solution is to add "--output text" to the aws commands.

Yes, that is certainly another solution that I came across just after
submitting these PRs. However, this implementation doesn't hurt the folks
who forgot that flag, and helps those who did. So I think it's still worth
incorporating.
On Jan 19, 2017 3:54 AM, "James Puderer"<EMAIL_ADDRESS>wrote:

I noticed this too. The problem is happening because your AWS CLI output
type is JSON.
Another solution is to add "--output text" to the aws commands.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fastai/courses/pull/34#issuecomment-273649968, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABZ9VYHZVNEIAvPPcqePfxr1Q9qMBYHPks5rTrQ3gaJpZM4LmqLP
.

Our setup guide says to use text output mode in aws configure.  We'd prefer if someone forgot to do this that they found out sooner (otherwise, we think this would cause issues for them later on).

Fair enough, thanks for the explanation!
On Jan 20, 2017 7:21 PM, "Rachel Thomas"<EMAIL_ADDRESS>wrote:

Our setup guide says to use text output mode in aws configure. We'd prefer
if someone forgot to do this that they found out sooner (otherwise, we
think this would cause issues for them later on).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fastai/courses/pull/34#issuecomment-274113513, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABZ9VYchCg0HEtBSh5rmCwQcXrxkNgT0ks5rUN8XgaJpZM4LmqLP
.


fix: omit timestamps from event messages

As per https://github.com/Health-Education-England/tis-trainee-credentials/pull/51
TIS21-4147
TIS21-4347

Sorry about the code smells, Sonar is having none of it 🤷

Happy to ignore the code smells for now, might be an outdated rule in checkstyle not handling the newer enhanced switch statements.


Users with spaces

Created a few mock users with spaces in their user name. This is allowed in OpenFire, and to make sure it wasn't just a bug with OpenFire, I used their client Spark to test them out. It is a valid username. Candy does not allow logging in with a user name with any spaces; at least not from my testing. It would be a nice feature to be able to do so.

The PMs works!!! :D Thanks a lot, ill be customizing candy so i will have a lot of time to check it all. Looks good at first sight.


identify2 uses the user cache if possible

update tests to check that the user cache is being properly accessed

r? @patrickxb

How's performance?

I couldn't really test performance, so I don't know....
The win of this PR is that after identify the user, we won't need to reload the user again further down the stack. But that might require @mlsteele's PR to go in, which uses the cacheable user load to verify signatures.

The commits 95c8904e5ac1...0ce76ebbe685 introduce visual changes on linux. :mag_right: 1 changed

Changed: Paper_Key-Normal (before) (after)


Interface with all tools present and other major and minor code changes.

Updated a bunch of the functions, and now users can load an interface with all the tools in one place that they can select from.

Journal: I needed to load library(shiny) manually in order to make the shiny app work, but then it looked fine. However, the pdf downloading does not work for me.
In R I couldn't see a help page (I get the error message: Error in .helpForCall(topicExpr, parent.frame()) :
no methods for ‘pomodoro’ and no documentation for it as a function) , but I see in your man file that you have some sort of documentation. It would be nice if you include here, whether this shiny app also works without any input (cause this worked for me).
Pomodoro: Everything works fine for me! Just a personal thing, if you have time for this, maybe you can play a bit with the interface, e.g., putting the box in the center, using colors etc. . I have the same problem with the help page. So maybe it is just me :D i really don't know.
ToDolist: Basically everything that i said about the pomodoro app :)

Journal: I needed to load library(shiny) manually in order to make the shiny app work, but then it looked fine. However, the pdf downloading does not work for me. In R I couldn't see a help page (I get the error message: Error in .helpForCall(topicExpr, parent.frame()) : no methods for ‘pomodoro’ and no documentation for it as a function) , but I see in your man file that you have some sort of documentation. It would be nice if you include here, whether this shiny app also works without any input (cause this worked for me). Pomodoro: Everything works fine for me! Just a personal thing, if you have time for this, maybe you can play a bit with the interface, e.g., putting the box in the center, using colors etc. . I have the same problem with the help page. So maybe it is just me :D i really don't know. ToDolist: Basically everything that i said about the pomodoro app :)

@jessiwitti Thank you for the great feedback! Indeed, I double checked the shiny app, and it will crash unless the library(shinyjs) is also loaded. I marked the necessary package as such shinyjs::enable, for example, but one part of the code was missing this.
I fixed this below:
`    shiny::observeEvent(input$pause, {                                                               # when the pause button is clicked, then
shinyjs::disable("pause")                                                                                # disable the pause button from being clicked
values$playing <- FALSE                                                                        # pause the quotes generation
shinyjs::enable("play")                                                                                # allow the play button to be clicked
})
shiny::observeEvent(input$play, {                                                                # when the play button is clicked, then
  shinyjs::disable("play")                                                                                # disable play button from being clicked
  values$playing <- TRUE                                                                         # generate a quote every 10 seconds
  shinyjs::enable("pause")                                                                                # allow pause button to be clicked
}) `

I made a new pull request afterwards with the corrected NAMESPACE, but right now the merged one is the older version.
The report was sent to the course lecturers, as I did not know we should upload it to GitHub.

Thank you @brunykrijgsman for the feedback. Indeed, my Shiny app also requires the package "shinyjs" in order for the interface to run. I appreciate the comment about the x parameter for the quote function, as this was incorrectly updated: so good catch!
Indeed, unfortunately for the report I misunderstood and sent it to the course lecturers and not on GitHub.
Thanks again for the comments!


[Feature Request] TLS support for postgreSQL

Is your feature request related to a problem? Please describe.
If a PostgreSQL server is enforced SSL connection, temporal-sql-tool should connect with --tls=true.
Currently, the connection from temporal to DB is not a SSL connection. Please find the error below when connecting to a SSL enforced database.
ERROR   Unable to create SQL database.  {"error": "unable to connect to DB, tried default DB names: postgres,defaultdb, errors: [pq: SSL connection is required. Please specify SSL options and retry. pq: SSL connection is required. Please specify SSL options and retry.]", "logging-call-at": "handler.go:100"}

Describe the solution you'd like
Add --tls=${SQL_TLS_ENABLED} for queries in setup_postgres_schema method
Additional context

@temporal-data @feedmeapples PR is attached.

I just stumbled over this while deploying Airbyte.
This is related to https://github.com/airbytehq/airbyte/issues/9342

@temporal-data any updates there ?

somehow, tls is still not working with these env:
POSTGRES_TLS_ENABLED="true"
POSTGRES_TLS_DISABLE_HOST_VERIFICATION="true"

I need to additionally add the env from here to make it work:
https://github.com/airbytehq/airbyte/issues/26089#issuecomment-1716663378

This is more for future people who come here.
Hi @NikoKS I guess in your case, you are using an old temporalio/auto-setup Docker image version. I can confirm Airbyte v0.63.1, Helm Chart v0.199.0 which by default using temporalio/auto-setup 1.23.0
are picking up these env values https://github.com/temporalio/docker-builds/blob/fdf6808ec70b6263f69b78300620eeb1a4d8bcd0/docker/auto-setup.sh#L38-L43
😃


Filemonitor does not work for git repositories with file:// url

I configured spring.cloud.config.server.git.uri: file://c:/test/, with C:/test/ being a local git repo.
I expected the file monitor to watch this git repo for changes.
In https://github.com/spring-cloud/spring-cloud-config/blob/master/spring-cloud-config-monitor/src/main/java/org/springframework/cloud/config/monitor/FileMonitorConfiguration.java#L197 the repository URL gets loaded as a Resource. File watching will only be enabled when resource instanceof FileSystemResource, but the Default ResourceLoader will never return a FileSystemResource, only a FileUrlResource, so file monitoring does not work for a file:// url. As long as the default ResourceLoader is used, the instanceof check can never be true. In case of the native Repositories, the searchlocations URL(s) are loaded as resources but only checked if they exist (https://github.com/spring-cloud/spring-cloud-config/blob/master/spring-cloud-config-monitor/src/main/java/org/springframework/cloud/config/monitor/FileMonitorConfiguration.java#L211).
I could only fix that by providing a ProtocolResolver that uses a FileSystemResourceLoader for URLs starting with file://, thus returning a FileSystemResource for such URLs as described here: https://stackoverflow.com/a/51420272/727373
I think, either the check on the SCM repository URLs should be made less restrictive (why is this check more strict as the check for the native repository URL?), or at least documentation should be improved to help providing the necessary ProtocolResolver.

Yeah, I'm sure it's original intent was NativeEnvironmentRepository not git.
Not sure I'm comfortable with a custom ProtocolResolver, but maybe there's something else that could be done.

I am also not happy with the custom ProtocolResolver because it is global and might have unintended side-effects, but i found no other way to fix the behaviour. IMHO relaxing the instanceof check to resource instanceof FileSystemResource || resource instanceof FileUrlResource would fix it. The check for FileSystemResource could even be removed, because a SCM repo URL can never be a FileSystemResource, only a FileUrlResource at best.  But for the purpose of the file monitoring, both resource types should work, shouldn't they? The monitoring for the native repositories also works with file:// URLs.

Should i take a shot and allow an instance of FileUrlResource in addition to FileSystemResource to activate the file monitor on at https://github.com/spring-cloud/spring-cloud-config/blob/master/spring-cloud-config-monitor/src/main/java/org/springframework/cloud/config/monitor/FileMonitorConfiguration.java#L197 ? I see no obvious downside.

PRs welcome


Netifly deployment not working

The deployment is marked as malicious by Google Chrome and clicking proceed opens a page saying "Not found".

Hey @ajeje93,
I accidentally forgot to remove the Netlify link from the README after moving the deployment to a custom domain.
Thanks a lot for spotting it! :)
Th3Wall


Support multiple namespaces with GitOps

Multi-tenancy Planning

Layer
Component
Namespace

Infrastructure
1. Sources  2. Cluster Issuer  3. Istio CRD and Control Plane
Shared Namespace

Dependency
1. Istio service  2. CA, certificate and key  3. PostgreSQL database
Per workload  Per environment

Application
1. Orthanc Application  2. Orthanc Ingress Gateway, Virtual Service and Destination Rules
Per workload  Per environment

Implementation Steps:

Move PostgreSQL to dependency layer
Move Istio Service to dependency layer
Move Cluster Issuer to Infrastructure layer
Configure namespace on dependency layer
Verify namespace on application layer
Configure healthcheck for each layer
Remove hard-coded namespace reference
Introduce second namespace
Add resource quota to namespace configuration
Namespace isolation mechanism

all addressed except for #10 , which goes into a separate issue.


Update teslacan.py

I've changed the import of MS_TO_KPH , JERK_LIMIT_MIN, JERK_LIMIT_MAX and crcmod.mkCrcFun to be more readable. I've also changed the if acc_state == 1 to if acc_state to make the code more readable. I've changed the if cancel to if cancel is True to make the code more readable. I've changed the for loop for packer, bus in [(self.packer, CANBUS.chassis), (self.pt_packer, CANBUS.powertrain)]: to for packer, bus in [self.packer, self.pt_packer], [CANBUS.chassis, CANBUS.powertrain]: to make the code more readable. This change eliminates the need for creating a tuple for every iteration of the loop and it also makes the code more readable as the packer and bus are on the same level of indentation.
Additionally, I've changed the use of messages.append(packer.make_can_msg("DAS_control", bus, values)) to messages.append(packer.make_can_msg("DAS_control", bus, values)) to make the code more readable and to avoid unnecessary function calls. This change ensures that the function is only called once.
It is also a good practice to add proper documentation(in form of comments) to the code, to explain the purpose and functionality of the functions, variables, and other important parts of the code.
It's also a good idea to run some test cases on the code to make sure that all the changes made did not break the functionality of the code.

Use from common.conversions import MS_TO_KPH instead of CV.MS_TO_KPH to make the code more readable and to avoid unnecessary imports.
Use from selfdrive.car.tesla.values import JERK_LIMIT_MIN, JERK_LIMIT_MAX instead of CarControllerParams.JERK_LIMIT_MIN, CarControllerParams.JERK_LIMIT_MAX to make the code more readable and to avoid unnecessary imports.
Use from crcmod import mkCrcFun instead of crcmod.mkCrcFun to make the code more readable and to avoid unnecessary imports.
Use crc = mkCrcFun(0x11d, initCrc=0x00, rev=False, xorOut=0xff) instead of crc = crcmod.mkCrcFun(0x11d, initCrc=0x00, rev=False, xorOut=0xff) to make the code more readable and to avoid unnecessary imports.
Use if acc_state: instead of if acc_state == 1: to make the code more readable and to avoid hard-coding the comparison value.
Use if cancel is True: instead of if cancel: to make the code more readable and to avoid implicit boolean evaluations.
Use for packer, bus in [self.packer, self.pt_packer], [CANBUS.chassis, CANBUS.powertrain]: instead of for packer, bus in [(self.packer, CANBUS.chassis), (self.pt_packer, CANBUS.powertrain)]: to make the code more readable and to avoid unnecessary tuple creation.
Use messages.append(packer.make_can_msg("DAS_control", bus, values)) instead of messages.append(packer.make_can_msg("DAS_control", bus, values)) to make the code more readable and to avoid unnecessary function calls.


Add @sys.description capability to variables

It would be great if we were able to provide description metadata to variables, especially now with the support of user-defined data types and functions.

If this is what you mean it is already supported:
@description('My description here')
var test = 'hello'

output test string = test

Yes, sorry I wasn't aware.
When i build a bicep file into an ARM template, the description is not included in the JSON file. Is that something desirable?
For example:
@sys.description('test var')
var test_var = 'test'

@sys.description('test param')
param test_param string

produces:
{
  "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "metadata": {
    "_generator": {
      "name": "bicep",
      "version": "<IP_ADDRESS>96",
      "templateHash": "14918382223189280707"
    }
  },
  "parameters": {
    "test_param": {
      "type": "string",
      "metadata": {
        "description": "test param"
      }
    }
  },
  "variables": {
    "test_var": "test"
  },
  "resources": []
}

OK, I guess there is no native support for description on variables in ARM-templates. For me this would not provide any value, but if building the ARM-template is part of your workflow then I understand that it would make sense.
Not sure if it is the Bicep-team that is also adding features to ARM? @stephaniezyen? 😄

The main reason is that I have created a tool called bicep-docs that generates Markdown documentation from a Bicep module (https://github.com/christosgalano/bicep-docs).
What I do, is essentially parse the ARM JSON for the metadata description. It works fine for parameters and outputs; for resources and modules, I have created a separate solution that parses the description from the Bicep file.
The approach for resources and modules works just fine with variables (now in v1.0.3 https://github.com/christosgalano/bicep-docs/releases/tag/v1.0.3), but it is more of a question for uniformity.
Since parameters, outputs, user-defined types, and functions support this; perhaps it would be useful to also have it in variables.

TBH, I don't think this will be any use for variables. when you compile to bicep, you do it for a purpose - and this is to either deploy to Azure or publish to registry to use in another module.
You have no way of referencing variables outside of a particular module, so variables can be treated as only inner elements.
Can you give some use case how would you use variables descriptions after compiling bicep file?

@christosgalano FYI this type of analysis is something you can do with the JSONRPC functionality.
See https://github.com/anthony-c-martin/bicep-toolkit for example, which uses it.


IOS Compile Issue

Im developing app with ios 12. But I got below error in xcode. Also I got error react-native . I examined issue and saw libstdc lib not supportted for ios 12. But i wanna use react-native-text-detector.  How to solve this issue ?
ld: building for iOS Simulator, but linking in dylib built for macOS, file '/usr/lib/libstdc++.6.0.9.dylib' for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

I have the same issue, did someone found a solution?

Im developing app with ios 12. But I got below error in xcode. Also I got error react-native . I examined issue and saw libstdc lib not supportted for ios 12. But i wanna use react-native-text-detector. How to solve this issue ?
ld: building for iOS Simulator, but linking in dylib built for macOS, file '/usr/lib/libstdc++.6.0.9.dylib' for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

I had the same issue just now, and for anyone searching to resolve this, add the pod:
pod 'TesseractOCRiOS', :git => 'git://github.com/parallaxe/Tesseract-OCR-iOS.git', :branch => 'macos-support'
This pod doesn't use the libstdc++ library. This library is now long deprecated and support for it was dropped in iOS 12. Add this pod to your Podfile and run:
cd ios && rm Podifle.lock && rm -rf Pods && pod install --repo-update
You now no longer should see this error.


EventEmitter@trigger does not catch parameter properly.

https://github.com/TomHsiao1260/three-starter/blob/87ab8ea0a7ac1d391e1fc7f913d435e3a9d840a4/src/javascript/Utils/EventEmitter.js#L44
This should return false  after the console warning. Will throw error when trying to resolve names.

This bug has been fixed. Thank you very much for checking carefully and making this starter better!


Fix: Center Gantt chunks(fixes #31)

What is the purpose of this pull request? (Check any that's applicable)

[ ] Feature
[x] Bug fix
[ ] Dependency updates
[ ] Documentation updates
[ ] Build related changes
[ ] Changes an existing functionality
[ ] Other, please explain:

Does this introduce a breaking change?

[ ] Yes
[x] No

Dev site path example

Please replace shape in action legend with diamond, rhombus, cross, vertical_bar, tear_alt, and tear_drop to see the changes
https://engineering.cerner.com/carbon-graphs/#/line/simple

What changes did you make? (Give an overview)

Centered all Gantt chart chunks which are not correctly aligned with the reference line.

Add any screenshots
Before diamond:

After diamond:

Before triangle:

After triangle:

Before rhombus:

After rhombus:

Before cross:

After cross:

Before vertical_bar:

After vertical_bar:

Before tear_alt:

After tear_alt:

Before tear_drop:

After tear_drop:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


website using tailwind css

it is basically a website structure made by tailwind CSS framework.

Hi abhrajit117, thanks for taking the time to make an example site.
Unfortunately this example is not using the plugin this repo is for so I can't accept it. I have instructions for setting up an example in the CONTRIBUTING.md file. It is probably easiest to duplicate a example folder, rename, and have your file there vs in the root of the project. If you can update your example to use some classes generated from this plugin I'd be happy to get your example merged. Let me know if there's any issues with the make commands. 👍


UVAtlasPartition: Non-manifold mesh，"The mesh output by Triposr does not match."

File "text2texture.py", line 329, in
tmesh = text2texture(
File "text2texture.py", line 229, in text2texture
tmesh.compute_uvatlas(size, parallel_partitions=2)
RuntimeError: [Open3D Error] (void open3d::t::geometry::kernel::uvunwrapping::{anonymous}::ComputeUVAtlasPartition(open3d::t::geometry::TriangleMesh, float, bool, open3d::t::geometry::kernel::uvunwrapping::{anonymous}::UVAtlasPartitionOutput&)) /root/Open3D/cpp/open3d/t/geometry/kernel/UVUnwrapping.cpp:124: UVAtlasPartition: Non-manifold mesh
how to solve it?thanks!

+1

UVAtlasPartition: Non-manifold mesh

I try  remove this line,but this error still exits.Do you have other solutions?

Hey, do you mind uploading the model file you're running it on?


project_name-bridging-header.h？

Import RNVideoProcessing.h into your project_name-bridging-header.h.
I didn't find it, maybe you can tell me where it is?

@yz-xlame hey, where did you find the project_name-bridging-header.h?


Failed to parse chardata.

I have an xml file, and a part of the string is as follows:

    <userdatatype>module
        <dataTypeEnum schemanode="" description="" declareType="ExportAll" name="ISOLATIONMODE"/>
    </userdatatype>

When I parse the xml above using the following code ，the code is as follows:
	ele,err:=xmltree.Parse(data)
	xmltree.Unmarshal(ele,v)
	if err != nil {
		fmt.Printf("error: %v", err)
		return
	}

The corresponding structure is as follows:
type Cuserdatatype struct {
	XMLName       xml.Name       `sxml:"userdatatype,omitempty" json:"userdatatype,omitempty"`
	CdataTypeEnum *CdataTypeEnum `xml:"dataTypeEnum,omitempty" json:"dataTypeEnum,omitempty"`
	CString        string         `xml:",chardata" json:",omitempty"`
}

The parsed result "module" is lost！！

ps:but when I use the native parser. "moudle" is not lost.
 xml.Unmarshal(data, v)

Why?

ps:v := &Cmodule{}

ps:v := &Cmodule{}

It's a bug. This piece of code in marshal.go
if len(el.Children) == 0 {
	if len(el.Content) > 0 {
		e.w.Write(el.Content)
	} else {
		return nil
	}
}

The Marshal function only writes out .Content (which includes the chardata) if there are no child elements. I could write out .Content unconditionally, but the problem is that .Content also contains the XML for child elements, and it will not reflect any changes made to the tree after Parse.
I'm not sure what's best here. I guess we could include xml.CharData in the decode switch here, add a .CharData field to Element, and work that into the encoder. The Marshal output would look different from the source document, but I think it should be good enough for the example here.

It's a bug. This piece of code in marshal.go
if len(el.Children) == 0 {
	if len(el.Content) > 0 {
		e.w.Write(el.Content)
	} else {
		return nil
	}
}

The Marshal function only writes out .Content (which includes the chardata) if there are no child elements. I could write out .Content unconditionally, but the problem is that .Content also contains the XML for child elements, and it will not reflect any changes made to the tree after Parse.
I'm not sure what's best here. I guess we could include xml.CharData in the decode switch here, add a .CharData field to Element, and work that into the encoder. The Marshal output would look different from the source document, but I think it should be good enough for the example here.


restores ability to point to custom config file with --config-file

Looks sensible to me.  I changed the base branch since it's a hotfix, will merge and release as soon as automated checks pass so we can test in out on the Drupal end of things asap.


ephemeral containers are not accounted for Guaranteed QoS class

/sig node
/kind cleanup
/priority backlog
small note based on this question: https://github.com/kubernetes/kubernetes/pull/115458/files#r1093894333

@SergeyKanzhelev Just to clarify ... is this a docs only fix, or a docs fix that comes along source code/behavior change?

Just a clarification, no code change.

Thanks.
/lgtm
/approve


Usage of decorators in raw functions

Suggestion
Decorators are a very useful feature when we have to encapsulate some behaviors and enhance our code with such.
But, a game changer for those decorators would be if we could use them in raw functions as well, not only in class based scenarios.
🔍 Search Terms

decorators

✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[ ] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

I would like to be able to decorate a raw/plain JS function with decorators. So I won't need to every time I need a decorator wrap my function inside a class to be able to use it.
📃 Motivating Example

Today, if you need to create a decorator, let's say to wrap a Lambda handler function, you would need to somehow wrap it into a class and initialize this class and expose the handler of it.
But,  if you could have decorators directly in raw JS function it would be more smooth and less "boilerplate".
💻 Use Cases

For example:
export function handler(@Param('pathParamId') pathParamId: string) {}

But rather, in order to have something similar, you would need to have this code:
@Handler
class MyHandler implements LambdaHandler {
    handler(@Param('pathParamId') pathParamId: string) {}
}

Related: #7318
In the end this depends on TC39, so you should take a suggestion there. Please also note that the current decorator support in TypeScript is based on an old and outdated proposal. The team made clear that they don't do any more work on decorator support until the proposal is finished.

[ ] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)

is a non-negotiable checkbox, I’m afraid.

I was afraid that I should mark all tue boxes. But I was pretty sure that such feature would fall in this category.
But it makes sense to at least wait for TC39 before having it in TS.
Pity, but understandable.
@andrewbranch You could close for comments as well to avoid people finding this issue and start another big thread.
Thanks for fast reply also @MartinJohns :)

This is my solution:

this simple decorator wrap any function and log its execution time in console

execution-time.decorator.ts
`
export function ExecutionTime() {
return (target: Object, key: string, descriptor: PropertyDescriptor) => {
const { value: origin } = descriptor;
descriptor.value = (...args: any[]) => {
  const label = 'target.constructor.name'+'.'+'key+'(...)';
  console.time(label);
  const result = origin(...args);

  if (result instanceof Promise) {
    return result.finally(console.timeEnd.bind({}, label));
  }

  console.timeEnd(label);

  return result;
};

};
}
`

Because we can decorate only methods, i make target function as a static in some class and then import const that equals this static method

some-function.ts
`
import { ExecutionTime } from './execution-time.decorator';
class WrappedClass {
@ExecutionTime()
static async originSomeFunction() {
console.log('done');
}
}
`
export const someFunction = WrappedClass.originSomeFunction;

Final usage in client code:
index.ts
`
import { someFunction } from './some-function';
someFunction();
`
P. S.
Of course this is not a raw function. But its interface, i mean its usage looks like usage of raw function. And if you want wrap it with decorators, you should go to its wrapped class and in it do it with origin static method.


Add Config::add_root_certificate for trusting custom ca

Support connecting to registries with self-signed certificates and certificates signed by CAs that are not in the client system trust store.
Simply passes thru the certificate along to a reqwest builder method by the same name.
Adds two tests:

Verify the client fails with an SSL error when the CA is not provided
Verify the client connects without an SSL error when the CA is given

Both tests are a part of the test-mock feature.

PR needs rebase

Surprised, what did you notice that made it look like it needed a rebase? It was rebased about 5 months ago.
It should be current with the default branch now in any case.

When this PR was originally authored 2 years ago, hyper 1.x had not been released.
Do you want me to upgrade this PR to hyper 1.x, or leave it on version 0.14?

can you upgrade to hyper 1.x and it'll be good to go!

Done.

looks like we need to move to v0.14.28
sorry about that

Back to hyper 0.14
Fixed cargo fmt issues


Medium and long text support for mysql

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

Fixed issues
https://github.com/yiisoft/db-mysql/issues/2

Now it's draft. Need add recognize leghth for tiny, medum, long and other types of text for mysql only

Need other solution base on length of TYPE_TEXT
Without crutches. Need refactoring - \Yiisoft\Db\QueryBuilder\AbstractQueryBuilder::getColumnType
For information:
https://github.com/yiisoft/yii2/issues/9585
https://dev.mysql.com/doc/refman/8.0/en/storage-requirements.html
https://github.com/nkovacs/yii2-mysql/blob/master/Schema.php

#370


Nextflow version

In my initial tests the pipeline failed.  As our global install of Nextflow is ~ version 21, I tried a new version (22.10.1 which I was using for something else) that then elicited the error that a minimum version of 23.10.1 is required. I tried then version 24.10.3 (the most current release) and that also failed.  So tried with the minimum pinned version of 23.10.1.
Suggest to update readme with Nextflow version requirements. Perhaps also additionally make a note if Nextflow Version 24 does not work?

Thanks @JamesZlosnik for the report! I can add that in for sure!! For failing on the 24.10.3 version it might have to do with a script permission issue I was alerted to and fixing in #2

Thanks @DarianHole!!
24.10.3 might be a local issue, I'll try again with #2 as I also encountered a permissions issue on the CREATE_ABUNDANCE_FILTER step and was just about to submit an issue for that one too :)


Further investigate CI pipeline errors

tests are still failing, trying some more things to fix it...

@naveen521kk two interesting observations regarding the 3.10 pipeline failures:

I can reproduce them locally (but didn't manage to debug them in a way here it would help)
after 39b34fd, I could at least doctest all individual submodules successfully (i.e., pytest --doctest-modules manim/mobject) -- but not the main module itself, it still segfaults there.

In case you have any ideas what this could be, let me know and I'll try it out!

I can reproduce them locally (but didn't manage to debug them in a way here it would help)

Can you post the output when setting the environment variable PYTHONVERBOSE=1? I can't reproduce locally with WSL on my Windows machine.

Also, try adding a ctx.release() in the pytest_report_header function. Let's see if releasing the context fixes anything.

Also, try adding a ctx.release() in the pytest_report_header function. Let's see if releasing the context fixes anything.

That one I did try: releasing the context doesn't fix testing the submodules, pytest --doctest-modules manim/mobject then segfaults too.

I can reproduce them locally (but didn't manage to debug them in a way here it would help)

Can you post the output when setting the environment variable PYTHONVERBOSE=1? I can't reproduce locally with WSL on my Windows machine.

Sure: https://pastebin.com/Lz9HXfcm

Wait, what 👀👀👀

removed custom pytest_report_header, fixes testing submodules (????)

https://github.com/moderngl/moderngl/issues/517: either an issue in moderngl or readline module.

I'll be waiting for the pipeline to pass, then this should be merged (and the fix should be "stresstested" by updating the other PRs that are open).


Major rewrite of Solana crate which switches the programs over to

Anchor. Adds support for a Verifier Router which can support multiple routers. Supports Emergency Stop calls.
Work In Progress: Still Requires Rewrite of Code Comments, Documentation, Some Examples, and User Scripts.
Add support for Program Verifier Routing and Emergency Stops.

[X] Groth16 Refactors
[X] Solana Verifier Router
[X] Solana Emeregency Stop
[X] Utility crates for use with these contracts
[x] A simple contract that verifies a RiscV proof using the main contracts.
[x] Deployment scripts and management scripts for Solana contracts that can run on testnet.
[x] A TransactionSigner class that supports Fireblocks for use with Deployment Scripts

The main functionality is complete. The final step is documenting the code, adding comments, and cleaning up.

Code cleanup complete; Only waiting on Documentation of Readmes and updating/adding comments.
The example program is not working as the verifier is returning a VerificationError on the run; I tested the example code against the original Solana verifier program and got the same error; @hans, am I passing in the correct arguments to the groth16 verifier?

Documentation, code comments, and readmes have been added. The verifier still throws a verification error, but the router is complete. This is ready for code review.


Does it work for 2.90?

Hi. I unpacked the .zip, unpacked the "addon" folder, and re .zip it. Targeted it back in Blender>addons, but it doesn't show the VSE addon to activate it.
I also downloaded the ktba master.zip, installed through addons, but the VSE addon is not listed (I guess it is because it is 2.79 version?)
How do I install this correctly?
Thanks.

You only need to download and install parallel_render.py and install that: https://raw.githubusercontent.com/elmopl/ktba/master/scripts/addons/parallel_render.py

Fantastic! thank you very much! I tested and it works. Drops rendering time by half. I wish it could be faster, but for now (or until the VSE refactor) this is great to work with.

I also just bumped the tests to use 2.91. It looks like no changes in API so it seems to work fine with newer versions of Blender.


jupyter-notebook illustrating API capabilities?

Author @dhruvbatra:
Should we consider creating a notebook like this:
https://github.com/Unity-Technologies/obstacle-tower-env/blob/master/examples/basic_usage.ipynb

https://github.com/facebookresearch/habitat-api/pull/69

Related PR: https://github.com/facebookresearch/habitat-api/pull/69.


Clear unnessesary %2A, Add missing enum fields in EventWaitHandle class document.

Summary
Clear unnessesary %2A, Add missing enum fields in EventWaitHandle class document.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@huww98 Thanks for helping with this. The updates where you removed the asterisk/wildcard (%2A) created a few invalid cross-references. Revert the %2A changes on these, and it should bring the cross-reference links back ...
<xref:System.Threading.EventWaitHandle.SetAccessControl>
<xref:System.Threading.EventWaitHandle.%23ctor>
<xref:System.Threading.WaitHandle.SignalAndWait?displayProperty=fullName>
<xref:System.Threading.WaitHandle.WaitAll?displayProperty=fullName>
<xref:System.Threading.WaitHandle.WaitAny?displayProperty=fullName>
<xref:System.Threading.EventWaitHandle.OpenExisting>
<System.Threading.EventWaitHandle.%23ctor>

I accidentally saw these %2A as %20(writespace)

@GuardRex I have revert these changes, and the cross-reference links seem OK now.

In the NOTE on line 587 ...

There is no guarantee that every call to the xref:System.Threading.EventWaitHandle.Set* method will release a thread from an xref:System.Threading.EventWaitHandle whose reset mode is xref:System.Threading.EventResetMode?displayProperty=fullName. If two calls are too close together, so that the second call occurs before a thread has been released, only one thread is released. It is as if the second call did not happen. Also, if xref:System.Threading.EventWaitHandle.Set* is called when there are no threads waiting and the xref:System.Threading.EventWaitHandle is already signaled, the call has no effect.

Should the "AutoReset" reset mode also be specified there? ...

... whose reset mode is xref:System.Threading.EventResetMode.AutoReset?displayProperty=fullName.

@huww98 Thanks ... looks great! :rocket: We'll get this merged after the build finishes.

@GuardRex Now, it finished😉


deps update

Coverage increased (+18.5%) to 100.0% when pulling 62215fceb209a49816dd0cb41539da9071fd9183 on tooling-update into a448aa0538abac614da2b442121b4c5cb6b8f6bb on master.


Highlight/show TODO’s as an info diagnostic or in a different panel

Description:
To mark parts of the code for future reference, add support for TODO comments as an info diagnostic or listed in different panel.

We are closing this issue due to inactivity. If you need further assistance or have additional input, feel free to reopen it. Thank you for helping us maintain a relevant and focused issue list!


Uncaught TypeError: undefined is not a function

[Enter steps to reproduce below:]

...
...

Atom Version: 0.165.0
System: linux 3.13.0-44-generic
Thrown From: merge-conflicts package, v1.2.9
Stack Trace
Uncaught TypeError: undefined is not a function
At /home/guillaume/.atom/packages/merge-conflicts/lib/resolver-view.coffee:35

TypeError: undefined is not a function
  at ResolverView.module.exports.ResolverView.relativePath (/home/guillaume/.atom/packages/merge-conflicts/lib/resolver-view.coffee:35:73)
  at ResolverView.module.exports.ResolverView.refresh (/home/guillaume/.atom/packages/merge-conflicts/lib/resolver-view.coffee:38:25)
  at ResolverView.module.exports.ResolverView.initialize (/home/guillaume/.atom/packages/merge-conflicts/lib/resolver-view.coffee:26:6)
  at ResolverView.View (/home/guillaume/.atom/packages/merge-conflicts/node_modules/space-pen/lib/space-pen.js:184:25)
  at new ResolverView (/home/guillaume/.atom/packages/merge-conflicts/lib/resolver-view.coffee:9:3)
  at ConflictMarker.module.exports.ConflictMarker.conflictsResolved (/home/guillaume/.atom/packages/merge-conflicts/lib/conflict-marker.coffee:70:42)
  at /home/guillaume/.atom/packages/merge-conflicts/lib/conflict-marker.coffee:61:10
  at Emitter.module.exports.Emitter.emit (/usr/share/atom/resources/app/node_modules/event-kit/lib/emitter.js:82:11)
  at Object.pkgEmitter.didResolveConflict (/home/guillaume/.atom/packages/merge-conflicts/lib/merge-conflicts.coffee:14:47)
  at /home/guillaume/.atom/packages/merge-conflicts/lib/conflict-marker.coffee:27:14
  at Emitter.module.exports.Emitter.emit (/usr/share/atom/resources/app/node_modules/event-kit/lib/emitter.js:82:11)
  at Conflict.module.exports.Conflict.resolveAs (/home/guillaume/.atom/packages/merge-conflicts/lib/conflict.coffee:115:14)
  at TheirSide.Side.resolve (/home/guillaume/.atom/packages/merge-conflicts/lib/side.coffee:7:25)
  at SideView.module.exports.SideView.useMe (/home/guillaume/.atom/packages/merge-conflicts/lib/side-view.coffee:53:11)
  at HTMLButtonElement.<anonymous> (/home/guillaume/.atom/packages/merge-conflicts/node_modules/space-pen/lib/space-pen.js:220:36)
  at HTMLButtonElement.jQuery.event.dispatch (/home/guillaume/.atom/packages/merge-conflicts/node_modules/space-pen/node_modules/jquery/dist/jquery.js:4430:9)
  at HTMLButtonElement.elemData.handle (/home/guillaume/.atom/packages/merge-conflicts/node_modules/space-pen/node_modules/jquery/dist/jquery.js:4116:28)

Commands
     -4:11.7 settings-view:open (input.hidden-input)
     -4:01.2 core:confirm (input.hidden-input)
     -3:23.3 core:backspace (input.hidden-input)
     -1:48.8 merge-conflicts:detect (input.hidden-input)
     -0:13.0 core:copy (input.hidden-input)
     -0:08.6 core:paste (input.hidden-input)
     -0:02.9 core:backspace (input.hidden-input)
 25x -0:01.9 core:move-left (input.hidden-input)
  2x -0:00.0 core:move-right (input.hidden-input)

Config
{
  "core": {
    "themes": [
      "seti-ui",
      "seti-syntax"
    ],
    "disabledPackages": [
      "minimap"
    ]
  }
}

Installed Packages
# User
linter, v0.9.0
linter-pylint, v0.2.0
merge-conflicts, v1.2.9
seti-syntax, v0.2.1
seti-ui, v0.5.1
wheel-zoom, v0.6.0

# Dev
No dev packages

/cc @atom/core

Occurred for me when accepting changes during merge conflict resolution.

Part of the work in #165 might have corrected this. Can you replicate on 1.3.3?

I Just got this in 1.3.3
For me it happened when I clicked the Quit-Button on the merge-conflict panel at the bottom

Every time I click it this happens and it won't disappear.
Atom Version: 1.0.2
System: Mac OS X 10.10.4
Thrown From: merge-conflicts package, v1.3.3
Stack Trace
Uncaught TypeError: undefined is not a function
At /Users/egon/.atom/packages/merge-conflicts/lib/view/merge-conflicts-view.coffee:122

TypeError: undefined is not a function
  at MergeConflictsView.finish (/Users/egon/.atom/packages/merge-conflicts/lib/view/merge-conflicts-view.coffee:122:7)
  at MergeConflictsView.quit (/Users/egon/.atom/packages/merge-conflicts/lib/view/merge-conflicts-view.coffee:86:6)
  at HTMLButtonElement.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-space-pen-views/node_modules/space-pen/lib/space-pen.js:220:36)
  at HTMLButtonElement.jQuery.event.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/jquery/dist/jquery.js:4435:9)
  at HTMLButtonElement.elemData.handle (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/jquery/dist/jquery.js:4121:28)

Commands
     -4:55.5.0 command-palette:toggle (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -4:53.7.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -4:53.7.0 merge-conflicts:detect (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
     -4:31.3.0 core:save (atom-text-editor.editor.conflicted.is-focused)
 12x -4:16.8.0 core:backspace (atom-text-editor.editor.is-focused)
     -4:02.4.0 snippets:previous-tab-stop (atom-text-editor.editor.is-focused)
     -4:02.4.0 editor:outdent-selected-rows (atom-text-editor.editor.is-focused)
     -4:00.6.0 snippets:previous-tab-stop (atom-text-editor.editor.is-focused)
     -4:00.6.0 editor:outdent-selected-rows (atom-text-editor.editor.is-focused)
     -3:59.3.0 core:save (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "one-dark-ui",
      "solarized-dark-syntax"
    ]
  }
}

Installed Packages
# User
atom-handlebars, v1.0.0
atom-ternjs, v0.6.8
linter, v1.2.4
linter-clojure, v0.0.5
linter-csslint, v0.0.14
linter-jscs, v1.12.0
linter-jshint, v1.0.5
linter-rubocop, v0.2.7
linter-xmllint, v0.0.7
merge-conflicts, v1.3.3
pretty-json, v0.4.1

# Dev
No dev packages

Confirmed with 1.3.3 here as well.
The "Stage" label present as well on the file causing trouble.

Comparing the stack traces I realised that this is probably a completely different error should I open another issue?

@Eschon Ah, yes, I think I see what's going on in that one, and it is a different issue. If you see this before I get to a fix, a new issue would be fine :smile:

Confirmed with 1.3.3 here as well.
The "Stage" label present as well on the file causing trouble.

@georgiana-gligor Can you double-check which stack trace you're seeing - @Eschon's, or @gmeral's? Basically look for MergeConflictsView or ResolverView in the top few stack frames.

Looking at the original stack, #165 (released as part of v1.3.3) should indeed deal with this. If you can still replicate on 1.3.4, let me know :grin:


Niemytzki plane is semimetrizable

S74 (Niemytzki plane) is semimetrizable.
Following @pzjp's https://github.com/pi-base/data/pull/989#issuecomment-2526279974

@Moniker1998 Any update?

@prabau I'm waiting on your opinion about the possible change. See above

You mean the typo fix?  Yeah, thanks for fixing it.

No, I mean the suggested change

Hmm, sorry but I don't see a suggested change anywhere?

You don't see this? That's odd

Refresh the page on your browser and see if it's there.  There is no such suggestion on my page.

Anyway, I can see the suggestion from what you pasted above and I don't have an objection.  I'll make the change, and even better, write the balls as $B_d(u,\epsilon)$.

Now I can see it.

I think I understand what happened. I didn't click "add review", and that's why it said it was a pending review. Now you should see it, I believe


good practice for gateway peer loadbalancing

We should provide guidance on this as part of samples/documentation some considerations are

load balancers in a K8s environment
client handling the balancing itself

Please feel free to add more thoughts, info for capturing plus where appropriate artifacts demonstrating this should be created

eg samples, docs

@mbwhite @jkneubuh This might be a good place to capture any relevant info

On the Kubernetes front there are a few different approaches that we can employ to shape traffic between the gateway client and peers.  All of these will provide some level of HA, failover, and traffic distribution across a set of peers.
Gateway load balancing within Kubernetes can be accomplished by establishing a Service instance matching node selectors for multiple peer Deployments.  Provided that the TLS certificates for the peers have been initialized with a common SAN in the signing request, a single k8s Service can act as a front-end to multiple peers using a common DNS name.  When establishing a TCP connection from the client to the gateway peer, Kubernetes will use the Service interface to dynamically resolve the address of one of the Pods bound to the Service.
One downside of using the Kubernetes Service routing is that any finer-grain message routing, e.g. at the gRPC message layer, is not possible.  Kubernetes can help with the initial assignment of a TCP connection to one of the backing peers, but once a client connection is established it will be maintained for the duration of the socket.
By default a Kubernetes Service will use iptables to bind a client connection to a peer pod using random assignment.  The pods backing a Service instance can be monitored via Readiness Probes, ensuring that only "ready" services receive gRPC handshakes from the gateway client SDK.
Building on the iptables routing, it looks like a Kube service can also use IPVS mode to further shape the IP resolution:

IPVS provides more options for balancing traffic to backend Pods; these are:

rr: round-robin
lc: least connection (smallest number of open connections)
dh: destination hashing
sh: source hashing
sed: shortest expected delay
nq: never queue
In addition, the IPVS routing mode includes a sessionAffinity attribute which can be set to "ClientIP", ensuring that connections from a particular client are resolved to the same peer pod.
Building on the Kubernetes defaults, we could consider an additional layer of traffic shaping by co-deploying a Fabric network within a service mesh, as provided by Ambassador, Istio, or Linkerd.  This approach may be a little more involved and not generally applicable to all environments running a Fabric network on K8s.
In addition to finding a home on the Fabric docs site, I like the idea of including a reference deployment within the Kubernetes test network.  Do we have a reference example app showing the best practices for authoring a Fabric application using the Gateway SDK?  (e.g. fabric-rest-sample, but using the Gateway SDK)

@jkneubuh I can create a PR on the test-network-k8s that includes the changes along these lines.  Ironically they are relatively minor changes, but I guess that shows the power of K8S.
I believe there is an example planned for the gateway, meanwhile I'll refer you to the LedgerMessaging IBM example.

@mbwhite @jkneubuh Have we done all we need (or realistically plan to do for now) on this issue? If so, I'll close it. If not, what needs doing and what's the outlook on that?

@bestbeforetoday I believe we've done all we can at the moment.

I still have a draft PR (lingering) open for the peer load balancing in k8s.  It's got some good info but is still too "kube specific" in the context where it's currently anchored in the docs.
Mark please leave this one open.  I will connect with Josh (H) on finding the correct page / site for the doc content.

Another potentially useful snippet of information on how to configure client-side load balancing over a set of IP addresses using the Node gRPC client: https://github.com/grpc/grpc-node/issues/1307#issuecomment-601312421

Best-practice recommendations for production deployment, which discusses load balancing / fail-over, is published in the full-stack-asset-transfer-guide sample.


Add options that define when preview builds get destroyed

Is your feature request related to a problem? Please describe.
We have a process were a certain pr is merged and then processed by the QA team.
The problem is that after a pr is merged you can no longer see the preview link so the QA team cannot access it.
Describe the solution you'd like
I would like to have the option to keep the preview link after the pr merges. Or better have the option to decide when preview links should be destroyed. Some options could be:

Destroy preview link when pr merges
Destroy preview link when branch is deleted
Destroy preview link when pr is closed
Never destroy preview link. (You can always destroy the links manually from the console)

Additional context
Add any other context or screenshots about the feature request here.

Preview link posted as a separate comment to the pull request would solve the visibility issue?
The rest is somewhat duplicate with #24? – Deleting preview deployment only in the case that source branch is deleted would allow you to keep it even when the pull request is merged is closed :)


Use a form of scene partitioning

A form of scene paritioning like an octree, BSP, PVS or tiles would improve rendering and physics performance.

The lacking engine now uses an octree, as long as the scene is properly partitioned.


libevent: Add port

Awesome to hear, holding until freedesktop is back again.

With freedesktop back, and the approval from Leo, merging this.


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
f6f565f -> e06c45b

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
minor
v1.25.7 -> v1.26.1

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.26.1
Compare Source
Bug Fixes

deps: update gomod (#726) (a42f8cf)

v1.26.0
Compare Source
Features

expose checkConsistency with public function (#724) (f9ab5b8)

v1.25.8
Compare Source
Bug Fixes

deps: update gomod (#723) (7596e10)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

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


Made a change in Cpp.cpp by adding endl and removing \n

My first project on opensource. So, it was only a try.