Incomplete Node.js build environment in v4 Docker container

I deploy Verdaccio on an internal docker host; to facilitate installing third-party plugins, e.g. verdaccio-auth-gitlab, I build custom images from Verdaccio's official docker image.
My Dockerfile looks something like this:
ARG VERSION
FROM verdaccio/verdaccio:${VERSION}
RUN yarn add verdaccio-auth-gitlab

For Verdaccio v3, this works nicely, but my experiments with Verdaccio v4-alpha - e.g. docker build --build-arg VERSION=4.0.0-alpha.4 . -  have, so far, triggered two distinct problems:

The v4 containers run, by default, as user verdaccio, but several source files are owned by root. yarn fails when trying to unlink /opt/verdaccio/node_modules/.yarn-integrity, npm fails when trying to write files inside /opt/verdaccio/node_modules/@verdaccio/file-locking. This can be worked around by adding USER instructions in the Dockerfile but is still somewhat annoying.

The v4 container no longer contains a full Node.js and Python build environment: node-gyp fails with Error: Can't find Python executable "python" while NPM or Yarn is trying to install the iltorb module.

I fully realize that v4 is an alpha release, and that verdaccio-auth-gitlab does not officially support v4 anyways, but having a stable and working build environment for easy plugin installation in the Docker image should IMO be an important feature of the final v4 release. 🤔

cc: @dlouzan

In v4, two of the biggest motivators are:

Avoid running things as root, since this is a bad thing (tm)
Support environments where this no-root policy is enforced by the runtime (e.g. Kubernetes OpenShift) and integrate it into the Helm chart.

The way we foresaw is to make plugins use the docker builder pattern. You then have full freedom to use whichever image you want for building your plugin, as long as you then afterwards copy the artifacts in the final verdaccio image. You can take a look at the verdaccio-gitlab Dockerfile for a sample of this usage.
Would this solve all your concerns with v4? I agree it would make sense to make this explicit in the documentation.

Sure, since you've basically redesigned the approach to installing plugins, concerns about a build env inside the main verdaccio docker image aren't really valid anymore.
But yeah, the documentation on that is very bare, so far.


Fix build to simulators in Apple silicon Macs

Remove arm64 in excluded_archs since we can support Apple silicon Mac simulators now

This PR is related to https://github.com/starburstlabs/crm-turbolinks-ios/pull/68
Please see https://github.com/starburstlabs/crm-turbolinks-ios/pull/68#issuecomment-1168908256


Auto-complete type suggestion

I love how atom-typescript suggests the given types for undeclared variables on hover. However, it would be awesome if there was a keystroke to put in this type suggestion automatically, because they can get complicated... e.g.
const ALLOWED_METHODS = _(['post']);

is a _.LoDashImplicitArrayWrapper<string>, but the only way to remember that is to go into the lodash.d.ts file manually copy it in.
Did a quick search for this through the FAQs/commands/issues/PRs, didn't see anything, but apologies if this is a duplicate.

You mean have Keyboard shortcut for Type information currently only shown on Mouse Hover ? https://github.com/TypeStrong/atom-typescript#type-information-on-hover

Yep!

@JaKXz cool, created a clean issue for the request : https://github.com/TypeStrong/atom-typescript/issues/726 :rose:


DummyBidAdapter

Checkout the branch, the dummyBidAdapter to the modules and build prebid
Then you can create and use Prebid AdSources with the dummyAdapter

just a memo: we should add eids support, so that we could debug them as well


ILogReceiverServer ProcessLogMessages

Under the OperationContract attribute definition, IsOneWay = false (the default). I don't know if this is the intended behavior, because it prevents the ability to use MSMQ.

"If the IsOneWay property is set to false (the default), even methods that return void result in a reply message. In this case, the infrastructure creates and sends an empty message to indicate to the caller that the method has returned. (Using this approach enables the infrastructure to send SOAP faults back to the client.) Setting IsOneWay to true is the only way to cancel the creation and dispatch of a response message."

Closed by #565


Train custom data

Can you explain how to train custom data please ?

Just modify data-specific parameters (e.g., data_root, data_list num_classes) in config.yaml.

Should I adjust parameter  num_negatives in config file for custom data ?

Also, how to generate custom SegmentationClassAug for custom data? Any tips on this?

The num_negatives means the number of negative keys per query in contrastive learning. Our method is robust against this parameter. Thus, I don't think it worth tuning for custom data.
However, if you fail to get expected results, it can be ablated on your dataset.

Should I adjust parameter num_negatives in config file for custom data ?

The num_negatives means the number of negative keys per query in contrastive learning. Our method is robust against this parameter. Thus, I don't think it worth tuning for custom data. However, if you fail to get expected results, it can be ablated on your dataset.

Also, how to generate SegmentationClassAug for custom data? Any tips on this?

SegmentationClassAug is the ground-truths for each input images.
Assume we have an image with shape HxWx3, thus the corresponding ground-truth should be HxW.
And each emelent denotes the class for each pixel.


Rename Project

The name hex_crypto implies that this dependency has to do with hex directly, when in fact its that it was split off from hex. In preparation for releasing this on hex I would like to change its name to something more generic. The only idea I have come up with so far is pbcs for Password-Based Cryptography Specification from https://tools.ietf.org/html/rfc8018
@potatosalad Does this name apply / make sense?

I'd say either pbcs or pkcs5, maybe?
I can't find many examples out there of people using pbcs (outside of some Oracle documentation), but there are a whole lot of uses of pkcs5.
Possible Reasons:
The PKCS standards have numbers assigned to the kinds of operations you perform: https://en.wikipedia.org/wiki/PKCS
The naming of the PKCS#5 operations document has changed a few times in the past:

PKCS#5: Version 1.0 — Password-Based Encryption Standard
PKCS#5: Version 1.5 — Password-Based Encryption Standard
PKCS#5: Version 2.0 — Password-Based Cryptography Specification
PKCS#5: Version 2.1 — Password-Based Cryptography Specification

If there is more visibility for pkcs5 I’d say we go with that. Besides, I think hex_names_are_hard is overloaded.

Renamed project also

Fixed and published as pbcs


feat: Added feature to Create FAQ

Fixes #677
Changes:

Added option to create FAQ.
Refractored the name of FAQRepository to FaqRepository to maintain uniformity throughout the codebase.

Screenshots for the change:

@iamareebjamal Please review.

@iamareebjamal I've updated the PR, please review.


(AdotCollectorAddOn): (Error resolving version for addon)

Describe the bug
Adding AdotCollectorAddOn causes errors during synth and creation errors during stack deployment
Expected Behavior
adotaddOn should be created
Current Behavior
Adding AdotCollectorAddOn causes these errors during synth:
2024-08-12 23:56:11.342	WARN	/node_modules/@aws-quickstart/eks-blueprints/lib/addons/core-addon/index.ts:208	main	  AccessDeniedException  User is not authorized to perform this action, AccessDeniedException, client, [object Object], [object Object] error stack:   • index.js	de_AccessDeniedExceptionRes 	/node_modules/@aws-sdk/client-eks/dist-cjs/index.js:2539   • index.js	de_CommandError 	/node_modules/@aws-sdk/client-eks/dist-cjs/index.js:2512   • task_queues	processTicksAndRejections 	internal/process/task_queues:95   • index.js	 	async /node_modules/@smithy/middleware-serde/dist-cjs/index.js:35   • index.js	 	async /node_modules/@smithy/core/dist-cjs/index.js:165   • index.js	 	async /node_modules/@smithy/middleware-retry/dist-cjs/index.js:320   • index.js	 	async /node_modules/@aws-sdk/middleware-logger/dist-cjs/index.js:34   • index.ts	async AdotCollectorAddOn.provideVersion 	/node_modules/@aws-quickstart/eks-blueprints/lib/addons/core-addon/index.ts:187   • index.ts	async AdotCollectorAddOn.deploy 	/node_modules/@aws-quickstart/eks-blueprints/lib/addons/core-addon/index.ts:90
And further down:
2024-08-12 23:56:11.345	WARN	/node_modules/@aws-quickstart/eks-blueprints/lib/addons/core-addon/index.ts:210	main	Failed to retrieve add-on versions from EKS for add-on adot. 2024-08-12 23:56:11.346	WARN	/node_modules/@aws-quickstart/eks-blueprints/lib/addons/core-addon/index.ts:211	main	Possible reasons for failures - Unauthorized or Authentication failure or Network failure on the terminal. 2024-08-12 23:56:11.347	WARN	/node_modules/@aws-quickstart/eks-blueprints/lib/addons/core-addon/index.ts:212	main	 Falling back to default version. DEBUG Core add-on adot has autoselected version v0.94.1-eksbuild.1
Then, when the stack is being deployed, it fails at the creation of the adotaddOn resource with this error:
Resource handler returned message: "Addon version specified is not supported (Service: Eks, Status Code: 400, Request ID: aa780b9c-42ef-4033-a41c-78c02af6fb5d)" (RequestToken: a4b84fcf-b769-482b-b518-fb06c871e2ed, HandlerErrorCode: InvalidRequest)
Reproduction Steps
new blueprints.addons.AdotCollectorAddOn()
Possible Solution
No response
Additional Information/Context
No response
CDK CLI Version
2.146.0 (build b368c78)
EKS Blueprints Version
1.15.1
Node.js Version
v21.6.1
Environment details (OS name and version, etc.)
Mac OSX 14.5
Other information
No response

@vpopiolrccl Your terminal should have AWS profile and AWS_REGION setup so your cdk command can access to retrieve the ADOT Addon version. That should also solve your next problem of picking the right version for your Addon. Could you please tell us what version you are using K8s cluster?


Cancel Connection, and dismiss callback

This is a really simple library, but it don't have a function for cancelling direction request when activity getting paused.

I just added it in v1.1.0. Sorry for take very long time to support this feature.
DirectionTask task = GoogleDirection.withServerKey(serverKey)
        .from(origin)
        .to(destination)
        .transportMode(TransportMode.DRIVING)
        .execute(this);

...

if (!task.isFinished()) {
    task.cancel();
}

DirectionTask based from Call in Retrofit 2. So you can get Call by
Call<Direction> call = task.toRetrofitCall();


Distribute binaries for Win/Mac/Linux

The Windows one might be complicated since I can't get Bogue to install in a non-WSL2 environment. Might just have to leave it at Mac/Linux for now until that changes.

Two potential solutions for Windows:

Figure out why pkg-config fails to locate SDL2-related things so the exe can be built locally
Find a working GH action that will build an exe/msi for Windows


Chrono fixes for VS Update 2

MSVS 2015 Update 2 includes an updated chrono implementation that overlaps with our own chono code. This PR disables our chono code when compiling with MSVC 2015 Update 2.
@miguelportilla @HowardHinnant

Current coverage is 63.48%

Merging #1565 into develop will decrease coverage by -0.02% as of 0d755cb

@@            develop   #1565   diff @@
=======================================
  Files           895     895
  Stmts         63697   63697
  Branches          0       0
  Methods           0       0
=======================================
- Hit           40452   40440    -12
  Partial           0       0
- Missed        23245   23257    +12

Review entire Coverage Diff as of 0d755cb

Powered by Codecov. Updated on successful CI builds.

:+1:  Unity and non Unity Debug/Release build fine on Visual Studio 2015 v.14.0.24720.00 Update 1.

:+1:

Still :+1:

@miguelportilla @HowardHinnant I pushed another set of commits to address comments and take care of a some warnings I noticed (I see Howard already reviewed as I was testing :-) Mickey, I'd love it if you could compile on Update 1 again to make sure I didn't muck it up.

Rebased onto b10

:+1:  Changes LTGM. Unity and non Unity Debug/Release build fine on Visual Studio 2015 with Update 1. All tests pass.

Squashed


[FLINK-27907][runtime] implement disk read and write logic for hybrid shuffle

What is the purpose of the change
Implement disk read and write logic for hybrid shuffle
Brief change log

Migrate BufferReaderWriterUtilTest to junit5 and assertJ.
Introduce HsSubpartitionFileReader and HsResultPartitionReadScheduler.
Introduce HsMemoryDataSpiller.

Verifying this change
This change added unit tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector:  no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Hi @xintongsong, all comments have been resolved, please take a look~


Modal overflow on mobile

Modal is larger than screen width on mobile.

Temporary fix but still a bit weird.


Disable persisting data

Hi guys, do you have any idea on how to disable persisting data definitively ?

You must turn off the active parameter in REDUX_PERSIST

Disable lines 35 and 52 in setup-root-store.ts.

Closing this - not an issue


STORM-2764:  Only login the user if not already done

If the changes look good I am happy to put up pull requests for other branches too.

@HeartSaVioR I put it up as #2352


Implement SkeinMAC

I have already implemented HMAC because I needed it for PBKDFs.

I started working on adding Threefish and Skein to Cipher yesterday, and in the process, I have added Mac, HMac, SkeinMac, and a few other things.
If you want to see a preliminary view of it, go to my fork of cipher, and go to the branch "skein" which I branched off of 0.4.0.
Just giving you a heads up that you might not have to worry too much about this since I added HMac :P

Hi Yulian:
That's very good news :-). I saw your branch yesterday when I was
releasing cipher 0.3.1.
It's also good that you branched 0.4.0 because I'm changing all the
factories (now named registry) to be able to create things like:
     new StreamCipher("AES/SIC")

or:
     new PaddedBlockCipher("AES/CBC/PKCS7")

Anyway, don't worry too much about these changes because they don't
affect the crypto APIs (BlockCipher, StreamCipher, etc.) and your work
won't have to be changed too much (maybe none at all) to adapt to the
new factories. I'm already finishing it so it should be pretty close to
what will be the released code (in branch 0.4.0).
To finish with, I will have a look at your code during this week and if
I see something problematic with my new changes I will tell you to save
us work. Also, any suggestions you may have for mine are very welcome.
Thanks again for contributing,
Ivan


chore(main): release sinkchart 1.17.0

:robot: I have created a release beep boop
1.17.0 (2023-02-09)
Features

include license issue recommendations (f2af2f4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sandworm-hq/sinkchart/releases/tag/sinkchart-v1.17.0 :sunflower:


v0.14.0 - which iOS icons should be generated?

I was using ticons to generate splashes an icons for my project when I saw this message last week:
-----------------------------------------
Update available: 0.14.0 (current: 0.13.3)
Run npm update -g ticons to update
-----------------------------------------

So, after heeding the upgrade hint (despite not seeing that release here on GitHub?) it now generates fewer images. Is that on purpose? What about -60, -small, appicon@3x and so on? Here is a diff in the difference in output of icons -p android,iphone -m mdpi -M xdpi.
--- 0.13.3	2015-09-07 14:43:30.000000000 +0700
+++ 0.14.0	2015-09-07 14:16:28.000000000 +0700
@@ -4,16 +4,7 @@
 [INFO]  Generated: ../iTunesConnect.png
 [INFO]  Generated: ../app/assets/iphone/iTunesArtwork@2x
 [INFO]  Generated: ../app/assets/iphone/iTunesArtwork
-[INFO]  Generated<EMAIL_ADDRESS> [INFO]  Generated: ../app/assets/iphone/appicon.png
-[INFO]  Generated<EMAIL_ADDRESS>-[INFO]  Generated<EMAIL_ADDRESS>-[INFO]  Generated: ../app/assets/iphone/appicon-Small.png
-[INFO]  Generated<EMAIL_ADDRESS>-[INFO]  Generated: ../app/assets/iphone/appicon-Small-40.png
-[INFO]  Generated<EMAIL_ADDRESS>-[INFO]  Generated<EMAIL_ADDRESS>-[INFO]  Generated: ../app/assets/iphone/appicon-60.png
 [INFO]  Generated: ../app/assets/android/appicon.png
 [INFO]  Generated: ../MarketplaceArtwork.png
-[OK]    Generated 18 icons
+[OK]    Generated 9 icons

If your project is using the 5.0.0 beta SDK then it should generate the above because since 5.0.0 Titanium generates the icons for you. If if happens with earlier SDKs as well then that's an error. Please let me know.

Aha, yeah that looks like a bug then:
Titanium Command-Line Interface, CLI version 3.4.2, Titanium SDK version 4.1.0.GA

I just tested with an 4.1.0.GA projects and there it generates all of them correctly. What do you have in your project's tiapp.xml for <version /> exactly?

Really? Weird. My version is set in the app config XML: <sdk-version>4.1.0.GA</sdk-version>, and I have according to appc ti sdk only 4.1.0.GA installed. Do you perhaps have multiple ones installed and conflicting version preferences between Studio / CLI or something?

TiCons just reads the tiapp.xml and sets that. Try running it with -t to see details.

It says 5.0.0, just like in constants.js ... I had a look at the code and I think I see what's going on: isProject and the subsequent parsing of the tiapp.xml file happens only unless -d is set. I didn't realize it initially but I was actually prepending the command line args with -d ../project_root/ in my build script.
So, while it isn't exactly a bug, I was expecting the config to be found/looked for in outputDir...

On a second thought, it should probably be considered a bug since https://github.com/FokkeZB/TiCons-CLI#smart-defaults complies with that said expectation :)

Good catch! Fixed!


OS X Support

It would be really awesome to get OS X support here for jenkins slave node setup

Actually the best way to do this on Mac OS is simply homebrew_cask 'java.

Apologies, as I'm new to Chef, but what's the best/cleanest way to insert a dependency into the project? Homebrew requires "depends homebrew" in metadata.rb (no entry in Berksfile required).

Initial support in now.


Re-add Rust / Markdown parser tutorial

See https://github.com/cloudflare/cloudflare-docs-engine/pull/181

Closing this as complete based on cloudflare/cloudflare-docs-engine#181


Use $HOME as a possible storage location

If LOCALAPPDATA and USERPROFILE are both null (as is the case on
Linux/OSX) use "$HOME/.aspnet" as the root folder for data protection
keys
Fixes #76

Hi @ellismg, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Matt Ellis). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla2.msopentech.com.

TTYL, MSOTBOT;

@davidfowl @GrabYourPitchforks Thoughts on this approach?  I didn't do platform sniffing because it was not clear I could always get an IRuntimeEnvironment from the service provider and it didn't seem worth the effort to do the old pinvoke to libc::uname trick.

Looks reasonable to me. :)

Thanks @GrabYourPitchforks.  @davidfowl The Travis build is broken but it looks like it's been that way for a while.  What can I do to get this ready to ship?  I would like to get the fix on myget so I can stop doing my hacky workaround of setting LOCALAPPDATA.

/cc @stephentoub as an FYI

@Eilon Levi thought you might be a good guy to tell me if this is ready to ship or not.

:shipit: Looks legitimate, just have one small comment, then merge.

Thanks, @Eilon!


Remove jQuery dependency

jQuery is not needed for such simple plugin. It should be removed.

Work started on new repository simivar/vanillajs-checkall.
As UniformJS and iCheck both depend on jQuery it won't be supported out-of-the-box.
This repository will be deprecated (archived) and users redirected to new plugin.

simivar/vanillajs-checkall

NPM package has been deprecated. Information added to Github Pages and README.


Support establishing a new session that is initiated on the remote peer.

If the remote amqp peer tries to initiate creating a new session, this client closes out the amqp connection due to this error:
unexpected frame: &amqp.performBegin{RemoteChannel:0x0, NextOutgoingID:0x0, IncomingWindow:0x64, OutgoingWindow:0x64, HandleMax:0xffffffff, OfferedCapabilities:amqp.multiSymbol(nil), DesiredCapabilities:amqp.multiSymbol(nil), Properties:map[amqp.symbol]interface {}(nil)}

This issue got lost in my email, oops. It appears that @alanconway is looking into adding support for this in #177

closing as a dup of #177


[ODS-5584] Making SSL mount optional for AdminApp image

Context
Right now Ed-Fi Docker images are hardwired to only run if they have an SSL certificate mounted in a docker volume. This makes it incompatible to run in a cloud environment where SSL encryption is usually handled by another service like a load balancer or an API Gateway. An example of this behavior can be seen in the link below.
https://github.com/Ed-Fi-Alliance-OSS/Ed-Fi-ODS-Docker/blob/5ae5208f3dde940ae14e5fc93a79323419e77b21/Web-Ods-AdminApp/Alpine/pgsql/run.sh#L44
Solution
Modify Ed-Fi Docker images so that the SSL mount is optional

 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.

@FluentGuru just curious - why did you close this PR?
@pmcvtm can you summarize the internal discussion on how we'll handle this change?

@FluentGuru just curious - why did you close this PR?
@pmcvtm can you summarize the internal discussion on how we'll handle this change?

Hi @stephenfuqua I closed the PR cause there was an issue with a commit that I couldn't amend to verify so after much trying I decided to just recreated the PR. Here's the new one
https://github.com/Ed-Fi-Alliance-OSS/Ed-Fi-ODS-Docker/pull/127


[Bug Report] FreeBSD doesn't have native libraries for changing volume

Bug Description
java.lang.IllegalArgumentException: Unknown operating system: FreeBSD
Steps to Reproduce

play a song

Expected Result
play music
Debug Output
[10:27:20] [INFO] [NativeLibraryLoader]: Native library libmpg123-0: loading with filter com.sedmelluq.discord.lavaplayer.natives.ConnectorNativeLibLoader$$Lambda$289/0x000000081d304418@44783e26
[10:27:20] [INFO] [NativeLibraryLoader]: Native library libmpg123-0: could not detect sytem type, but system filter is com.sedmelluq.discord.lavaplayer.natives.ConnectorNativeLibLoader$$Lambda$289/0x000000081d304418@44783e26 - assuming it does not match and skipping library.
[10:27:20] [INFO] [NativeLibraryLoader]: Native library connector: loading with filter null
[10:27:20] [ERROR] [NativeLibraryLoader]: Native library connector: loading failed.
java.lang.IllegalArgumentException: Unknown operating system: FreeBSD
at com.sedmelluq.lava.common.natives.architecture.DefaultOperatingSystemTypes.detect(DefaultOperatingSystemTypes.java:46)
at com.sedmelluq.lava.common.natives.architecture.SystemType.detect(SystemType.java:44)
at com.sedmelluq.lava.common.natives.NativeLibraryLoader.detectMatchingSystemType(NativeLibraryLoader.java:180)
Additional Info
I've been using this bot on this operating system for about 5 months
FreeBSD freebsd 11.2-RELEASE-p3 FreeBSD 11.2-RELEASE-p3 #0: Thu Sep  6 07:14:16 UTC 2018     root@amd64-builder.daemonology.net:/usr/obj/usr/src/sys/GENERIC  amd64
EDIT--
root@freebsd:/home/tiago # cat serversettings.json
{
"693196163922264065": {},
"690643625524068412": {},
"690649853327310850": {},
"689485187729719450": {"volume": 45}
}root@freebsd:/home/tiago # vi serversettings.json
{
"693196163922264065": {},
"690643625524068412": {},
"690649853327310850": {},
}
after I deleted the volume directive the bot started working again
Checklist

[X] I have looked for information about this within the documentation
[X] I have searched for similar issues on the issues page
[X] I am running the latest version of the bot:

Changing the volume requires JMusicBot to perform audio manipulation. It uses native libraries for this. JMusicBot doesn't have these libraries for FreeBSD at the moment.
In the meantime, keep the bot's volume to 100% with the volume command, and use the volume slider on Discord.

Changing the volume requires JMusicBot to perform audio manipulation. It uses native libraries for this. JMusicBot doesn't have these libraries for FreeBSD at the moment.
In the meantime, keep the bot's volume to 100% with the volume command, and use the volume slider on Discord.

Nice, i keep the volume in 100% and work for me :) thx

You can't play local MP3s because of this issue as well.  Exact same error.


Added CI flag checking

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

Project coverage is 62.39%. Comparing base (e7fc05e) to head (b7a90ec).
Report is 29 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #676      +/-   ##
==========================================
- Coverage   63.93%   62.39%   -1.54%
==========================================
  Files          49       49
  Lines        1902     1912      +10
==========================================
- Hits         1216     1193      -23
- Misses        686      719      +33

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


Updated to allow deletions

Updated config/mount script to allow RW which will only allow deletions. UnionFS still will write to local_decrypt due to the order of locations in script. Tested and worked OK.
See https://github.com/madslundt/cloud-media-scripts/issues/36 for screen shots of proof for deletions.

Showing proof of DL worked without problem and loaded to local-decrypt:

Is it possible that you can make write-able as an option within config and not as default?

Yeah I can do that, just wasn't sure if that is the best route since by default it will write to local_decrypt due to unionfs priorities. Tried to "keep it simple stupid" as it wouldn't impact anyone either way unless deleting.

It's a good point. However, people who are using it now knows that it's read-only. That's why I don't want to confuse them by changing default options to writeable storage.
However, good job on your PR!

Code added, will add comments for readme when have time.

Made a another update in REAME for config instructions for GID/UID. All done now.

if [ "$delete_flag" == "1" ]; then
Should be good right? Think thats right. Does it even need a wrapping in "" or would just ==1 work? Not a bash master. I use "" because I saw you did in other sections.


Tests fail with he latest version of weblogo

weblogolib.ColorGroup does not exist anymore.

Indeed. It's now weblogolib.SymbolColor.
How can we manage that? Change it and precise the number of version supported ?

Is it just the name that changes? Is the prototype the same? If so, we could just link them both to the same name.

Prototype is the same yes. You can check my PR

fix in #134


Added evaluation of logical operators

Added evaluation of logical operators and specs.

I added type checking to the evaluation function. I think figuring out the syntax errors before runtime can be better implemented when we put in the "fast track" evaluation functions, because we'll be doing type checking there.
I wasn't sure of the error message to use when performing an operation that's undefined on certain types, so I used Operation is undefined. Is there a way to phrase this to be more informative?

Also at this point it might be useful to include the type in the Node structure, so we don't have to perform a lot of typeof's and instanceof's.

Also at this point it might be useful to include the type in the Node structure, so we don't have to perform a lot of typeof's and instanceof's.

:+1: Want to do it as part of this PR?

Want to do it as part of this PR?

Sure

I added the enum EvaluationNodeType and added a type property to the Node structure.
Currently, since the evaluation functions are more general (ie, they may be a variable or they may have to evaluate child expressions), we cannot only check node._type. However this will be useful going forward to assign evaluation functions that are more strongly typed for runtime.
I also factored out some helper function from createRuntimeAst so it's a little more readable.

Nice!


Use canonicalize_name to look for .dist-info in wheel files

wheel files contain a directory that ends with ".dist-info", but its full name may not exactly match the name of the package as written in the wheel filename. e.g. on Windows, the wheel file name may use different letter cases. Using canonicalize_name on both the package name from the wheel filename and the directories in the zip to find a match fixes the problem.
See issue #1350
The general logic was taken from the pip code: https://github.com/pypa/pip/blob/0ae0109901f63b7f133229a9fee41332dd7366b4/src/pip/_internal/wheel.py#L270

I updated the PR with the new test.

@wiggin15 Damn, that's entirely my bad. Do you still have your version of the code? Can you do a force push? I might be able to recover the original test from my git history.

Updated with force push.

LGTM.


UAR-88: Update journey protected information page

JIRA link
https://companieshouse.atlassian.net/browse/UAR-88
Change description

Created update journey protected information page

Work checklist

[ ] Tests added where applicable
[ ] UI changes meet accessibility criteria

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

Passed internally requesting CH external reviews ....


Fix: https:/ to https:// rewriting for httpproxy

Fixes the rewriting e.g.
from: https:/storage.googleapis.com/hcloud-rancher-v2-ui-driver/component.js
to: https://storage.googleapis.com/hcloud-rancher-v2-ui-driver/component.js
So that the https:/ will be correctly rewritten to https://.
I tested three test cases as the ui driver URL:

http://[...]
https://[...]
hostname.tld/component.js

all are working fine. :smile:
Currently http links will work, but https links won't work as URL. They will get an 502 internal server error with "invalid host" message. That's because he could not parse the URL correctly with https.

https://github.com/rancher/rancher/issues/12458


Params Does not Work when is pased to schedule a Job

Im Trying to create a Job with a Basic URL and schedule this Job using params to not repeat the same Job with different Params, but I check an the params are not sending like that into the schedule.
http://localhost:4040/api/job/now
This is the Body I sent:
{
"name": "SapUp",
"data": {
"params":
{
"vm_name": "NVD-PW-CA5-SAP-VT-11",
"instance": "nn",
"ticket":"TICKETTEST08",
<EMAIL_ADDRESS>}
}
}
This is the Job Definition Im Using
{
"_id" : ObjectId("5d1638e15f48de2af1aace10"),
"name" : "SapUp",
"url" : "https://webstation.cositas.com:3700/api/sap_up",
"method" : "POST"
}
the errror Im Recieving is
{
"_id" : ObjectId("5d163962bbc6042b04da0fe4"),
"name" : "SapUp",
"data" : {
"params" : {
"vm_name" : "NVD-PW-CA5-SAP-VT-11",
"instance" : "nn",
"ticket" : "TICKETTEST08",
"mail"<EMAIL_ADDRESS>}
},
"type" : "normal",
"priority" : 0,
"nextRunAt" : null,
"lastModifiedBy" : null,
"lockedAt" : null,
"lastRunAt" : ISODate("2019-06-28T15:59:32.224Z"),
"failCount" : 1,
"failReason" : "406 - {"id":406,"msg":"Debe ingresar vm_name"}", (Is Not recieving this Param)
"failedAt" : ISODate("2019-06-28T15:59:32.379Z"),
"lastFinishedAt" : ISODate("2019-06-28T15:59:32.379Z")
Is this an Issue of the agenda-rest or Im using wrong the npm.
Best regards
Andres

We should probably close this issue. I have tested this code and it works as designed. Is there another reason to keep it opened?

I should add some unit tests for this. @keyvan-m-sadeghi Is that OK?

@keyvan-m-sadeghi I added the unit tests to test this functionality. I think we can close this thicket when #73 is merged.

@andresmelian You can take a look at the relevant unit test for this feature here: https://github.com/agenda/agenda-rest/blob/62dd72e53e174a80eb891807a42f0c64db055831/src/util.js#L72-L84

Great to hear, thanks @geosp :tada:

Closing as #73 is now merged.


Integration with rendering freameworks

We may want to integrate it with react and possibly other rendering engines, do you have any plans or ideas on how to provide the glue for it? (e.g. useSignal, etc. hooks for react)
I was thinking on just copypasting what preact-signals have as a submodule here (like usignal/solid), but want to ask first

theoretically this API is almost a drop-in for preact signals so whatever they do should work here too ... should though, as I haven't seen what they do or what they have, but I am confident it should be enough. Any explicit link before I land in the wrong file around preact useSignal implementation?

I was looking here
The useSignal, useComputed, and useSignalEffect look simple enough.
The rest of that file looks confusing to me, that's what I'm not sure about, should've clarified it initially.
(I'm guessing the rest there is to be able to just pass signals into JSX and have it working automatically, if so then it's nice, but hooks should be enough imo, feel free to close in that case)

if so then it's nice, but hooks should be enough imo, feel free to close in that case

On the second thought it's not exactly clear how to make sure react rerenders the component on signal update.
Something like that I guess (if using hooks from here):
const sub = (signal) => {
  const [, rerender] = useState(signal.value)
  return [signal, effect(() => rerender(signal.value))]
}

const Component = () => {
  const [signal, unsub] = sub(useSignal(123))
  return <div>{signal.value}</div>
}

to be honest, I am not sure what is wrong with this approach, when it comes to hook based libraries:
const useSignal = value => {
  const [current, update] = useState(value);
  return {
    get value() { return current },
    set value(_) { update(_) }
  };
};
``

What you talking about? I keep getting this messages could you tell me
exactly what your seeing for I could understand.
THANK YOU
On Sun, Feb 12, 2023, 8:32 AM Andrea Giammarchi @.***>
wrote:

to be honest, I am not sure what is wrong with this approach, when it
comes to hook based libraries:
const useSignal = value => {
const [current, update] = useState(value);
return {
get value() { return current },
set value() { update() }
};};``
—
Reply to this email directly, view it on GitHub
https://github.com/WebReflection/usignal/issues/21#issuecomment-1427061590,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2ECSFZDLJE6HWE5ORXMYB3WXD7CBANCNFSM6AAAAAAUYZXWAU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

In my case we may use signals created somehwere else (e.g. shared data model module, which notifies about updates through signal value), so we need to to be albe to hook a component to a signal which could be modified from somewhere else


Redraw line chart

Apologies i am relatively new to Angularjs.
I can't seem to get the chart to redraw the data when using the http service. I am simply updating the $scope.data (this is being used as the data for the chart) but it doesn't seem to update the chart?
I am calling the service from a button, this calls the following :
$scope.searchMonthlyTransactions = function () {
$http.get('http://sas-laptop9/CashFlowService/Service1.svc/MovementTransactions/).success(function (data) {
$scope.data = data.MovementTransactionsResult;
});
}
Am i doing something stupid? Or should it update the chart with my data?
I know the data returned is ok, as if i call it "on load" the graph plots ok.
Any help would be great

Has anyone figured this one out yet? I've updated $scope.data followed with $scope.$apply() to no avail. The chart does not change until I click on the legend... then it updates.

Hi @andrewfosterrth
do you have a pluker demo to illustrate your problem (you can start modifying this one)? This would be very helpful to find the bug.
Best,
Christoph


Add permissions to workflow .yml

See: https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#permissions

See 39d1f6e


Add tests for Add vehicle

In this PR, i created integration tests for behaviors;

To create a vehicle, the user has to add a name and price.
Image upload.

Hi, @Adanzeakonobi.
Good job so far!
There are some issues that you still need to work on, but you are almost there!
Highlights:
No linter issues 👍
Clean structured code
Required Changes: ♻
the test is testing for custom code, not the associated component sadly it cannot be worked on anymore


Snap does not work on supported Debian and Ubuntu releases.

Entering /mnt/stateful_partition/crouton/chroots/stretchxfce...
crouton: version 1-20210817115015~master:d607c558
release: stretch
architecture: amd64
xmethod: xorg
targets: xfce,extension
host: version 13982.69.0 (Official Build) stable-channel snappy
kernel: Linux localhost 4.4.254-20533-gcb85bd3a0149 #1 SMP PREEMPT Thu Jul 29 21:56:25 PDT 2021 x86_64 GNU/Linux
freon: yes

Attempting to use Snap with the packages "snap" and "snapd" installed will produce this error upon entering a command to install a Snap package (that being "sudo snap install package-name").
error: cannot communicate with server: Post http://localhost/v2/snaps/package-name: dial unix /run/snapd.socket: connect: no such file or directory

Upon attempting to troubleshoot this using information acquired on various forums I found by copy and pasting that error into Google, I ran into many more errors related to "systemd" and "systemctl" which I cannot comprehend.
I replicated this on Debian Stretch KDE and XFCE4 alongside Xenial GNOME.

Duplicate of #4553.


Use yaml multiline for ConfigMap files

Is this a new chart?
No
What this PR does / why we need it:
The previous version put config files as quoted-string which makes it difficult to modify in Kubernetes manually since indentation is part of the language.
Also, while modifying that config file, I noticed one of them had the execution right, so removed it.
Which issue(s) is this PR associated with:

#Issue_Number

Special notes for your reviewer:
@tdawe
Checklist:
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Chart Version bumped
[ ] Variables are documented in the chart README.md
[x] Title of the PR starts with the chart name (e.g. [charts_dir/mychartname]) if applicable

@coulof: Thank you for submitting your pull request!
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.

We want your feedback! If you have any questions or suggestions regarding our contributing process/workflow, please reach out to us at<EMAIL_ADDRESS>

@coulof: Thank you for submitting your pull request!
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.

We want your feedback! If you have any questions or suggestions regarding our contributing process/workflow, please reach out to us at<EMAIL_ADDRESS>


Disable double colors only for Sections

Hi there,
first of all, I love this theme. I just want to know if it possible to have the sections all in black (no "Edu" in red) while keeping the Locations and Position in red!
Thank you

You can see the comment in your resume.tex file :)
% Set false if you don't want to highlight section with awesome color
\setbool{acvSectionColorHighlight}{true}

What if I want the whole first word (ex Education) to be red

What if I want the whole first word (ex Education) to be red

in awesome-cv.cls in line 155
change this code =>
\def@sectioncolor#1#2#3{%
\ifbool{acvSectionColorHighlight}{{\color{awesome}#1#2#3}}{#1#2#3}%
}
to this =>
\def@sectioncolor#1#2#3{%
\ifbool{acvSectionColorHighlight}{{\color{awesome}#1#2#3}}{\color{awesome}#1#2#3}%
}


Migrations, few indexes

We are using the NuGet packages for MembershipReboot and MembershipReboot.Ef. We see only one record in the migrations table (2015-06...InitialCreate...) , though from the reboot source it seems like we should have a bunch (from 2014 with v7 in the name).
The only indexes seem to be on the primary keys. Some load testing we are doing indicates we would get much better performance with more indexes (ex Useraccount.username.) Some of the v7 migrations in git seem like they would address this.
Is there something we should be doing to support the alternate migrations/expanded indexes? I don't see anything in the "EF Configuration" article addressing this.
Thanks!

Check in the ~/packages folder -- there's a SQL script to install the migrations. Or you can create the DB using the migrations update-database tool. I'd not trust the auto-created DB that EF gives you -- something about that creates the DB doesn't seem to work well with proper migrations.

Thanks! Will try that out.

Thanks! We worked with that and it resolved our issue.


Correct Python Quickstart link

@msmbaldwin
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi, @traveldwindling. Thank you for catching this -- will we implement your fix ASAP! #sign-off


XIVY-12892 release to market via pipeline

Looks good 👍
In the end you should look that the version number is also updated on the version raise scripts

thanks, I have changed it to ${npm_package_version:0:4}


added scrollcontainer

Adding a scroll container to the toolbar to support odd resolutions and aspect ratios (like mine)
Git added the demo scene changes. I will add a gitignore next time

merging will mess up things because toolbar got altered after the PR was created and now everything broke. i will create a new PR on the currrent 4.0


The last version release my antivirus detects it as VIRUS.

I'd like you to check it for viruses.

It has no viruses it's litterly open source...

我希望你检查它是否有病毒。

I think if you don't worry about binary distribution, you can compile it yourself

Same, trojan warning. Do all of them have viruses fr?

Window Defender detects it, but maybe it's my antivirus.

It's funny that you open an issue thread like this while the source code is here.


OR tools liner solver

@lperron
I am very new to Python an OR-Tools, so please excuse the simplicity of my question.
I installed OR-Tools on my 64-bit, Win10 machine. I am using Python 64bit. I am working on both PyCharm amd Jupyter Notebook.
I get the error No module named 'ortools'.
I had Visual Studio Professional 2017 15.9.22 downloaded but that did not fix the issue.
I reviewed the prior issue No module named 'ortools' #737, but it didn't help.
I appreciate your help.
ImportError                               Traceback (most recent call last)
 in
1 from future import print_function
----> 2 from ortools.linear_solver import pywraplp
~\AppData\Roaming\Python\Python37\site-packages\ortools\linear_solver\pywraplp.py in
11 # Import the low-level C/C++ module
12 if package or "." in name:
---> 13     from . import _pywraplp
14 else:
15     import _pywraplp
ImportError: DLL load failed: %1 is not a valid Win32 application.

did you try to install the VS 2019 redistributable ?

https://developers.google.com/optimization/install/python/windows#microsoft-visual-c++-redistributable


Fix: change WorkflowStep into an abstract class

For some reason, Typescript resolves types in a different away compared to abstract classes. As a consequence, it would not apply the overloading properly and always treats WorkflowStep.do as a 3 argument function

LGTM assuming this works. Have you tried generating the types with this and examined them or is it more of a guess-and-check situation?

Good question. To avoid thrashing, let's generate these and test locally on smoke tests @LuisDuarte1

Then no worries here either, but it looks like it can't be merged yet because "This branch has conflicts that must be resolved"


Fix OTLP exporter

Remove signal path from the URL in OTEL_EXPORTER_OTLP_ENDPOINT (ref: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/protocol/exporter.md)

#32


Setting a breakpoint at onComputedUpdated freezes the app (macOS, Android)

Not sure if this is signals.dart issue, or rather VS Code extension's but it happens to me only when playing with signals.dart example app.
Steps to reproduce:

Add LoggingSignalsObserver to drift_example in the main.dart

void main() {
  SignalsObserver.instance = LoggingSignalsObserver(); // or custom observer

  runApp(MyApp());
}

Launch drift_example either on Android or macOS in debug mode from VS Code
Play with the app by adding some tasks or switching between lists
Set a breakpoint in the onComputedUpdated of observer.dart
Try playing with the app, observe that it's not responding

https://github.com/rodydavis/signals.dart/assets/16854239/515a2f88-dd55-41f9-b1fe-f13d30e276dc
Flutter doctor (a bit funky due to fvm)
[!] Flutter (Channel stable, 3.16.0, on macOS 13.5.2 22G91 darwin-arm64, locale pl-PL)
    • Flutter version 3.16.0 on channel stable at /Users/dominik/fvm/versions/stable
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/3.2.0/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/dominik/fvm/versions/stable. Consider adding /Users/dominik/fvm/versions/stable/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (3 months ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/dominik/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.15.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] VS Code (version 1.86.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.82.0

[✓] Connected device (3 available)
    • SM G991B (mobile) • R5CR617G51T • android-arm64  • Android 14 (API 34)
    • macOS (desktop)   • macos       • darwin-arm64   • macOS 13.5.2 22G91 darwin-arm64
    • Chrome (web)      • chrome      • web-javascript • Google Chrome 121.0.6167.160
    ! Error: Browsing on the local area network for iPhone 12 mini. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

Very interesting! Looking into it

I think this is a drift issue, I have had errors through when trying to use the debugger with SQLite in transactions and streams


add api server endpoint for manually qualifying a piece of freight

Depends on #1033
This would enable hotfixes without cluttering stage DAGs with subscriptions from prod (on the far right) to a warehouse (on the far left).

Explicitly adding 2 requirements for this endpoint so they aren't forgotten:

Verify stage exists
For Stage delete endpoint, remove the entry in approvedFor for that stage


Support for perseus v0.4.0-beta.1 needed in examples/simple

Please provide a version of examples/simple that can be compiled on perseus v0.4.x and later.
Current dependencies on perseus-size-opt v0.1.7 and later cause cargo to pull two versions of certain crates when compiling using the current pre-release beta of perseus when following Docker Deployment instructions.
Thanks! 😺

I'm actually going to deprecate this plugin entirely with v0.4.0-beta.2, which removes .perseus/, and thus the need for this plugin. I would advise you leave this plugin entirely out of the v0.4.x Docker deployments. The deprecation message will go up once the new version is out (still waiting on a few things from Sycamore).
My reasoning for not updating the plugin is that its whole purpose is to simplify updating .perseus/Cargo.toml, which was hidden from the user. With the new architecture, there's just the user's own usual Cargo.toml, which means they can more easily control the optimizations they make.
Supporting the first beta version is not really worth doing if the second will deprecate it (since there's no distinction between beta versions in the Perseus docs). That said, I can still add support for the first beta if you want!

Supporting the first beta version is not really worth doing if the second will deprecate it (since there's no distinction between beta versions in the Perseus docs). That said, I can still add support for the first beta if you want!

I agree that it wouldn't make sense to resolve this issue as requested when this plugin is being deprecated soon.
My request to make the example available for 0.4.0-beta.1 was simply to standardize existing deployment docs for docker across currently available releases.
I'll make adjustments to migrate away from perseus-size-opt in 0.4.x and later.
Thanks!

No problem! Note that the next beta of Perseus will come out probably today.


fix: disabled Interaction Flag Default false for React Native Agent

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

Project coverage is 91.36%. Comparing base (bbbff35) to head (6b795f8).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #165   +/-   ##
=======================================
  Coverage   91.36%   91.36%
=======================================
  Files          11       11
  Lines         417      417
  Branches       79       79
=======================================
  Hits          381      381
  Misses         31       31
  Partials        5        5

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


Disable all dependencies for compilation-only test?

Hi thanks for the library! I am trying to make it a part of test of https://github.com/fzyzcjy/flutter_rust_bridge to test flutter_rust_bridge's ability of automatically scanning and converting third party crates. Therefore, it would be great if it is possible to disable all dependencies for compilation-only (and not to be really run) test.

Hey, thanks for the report, but I'm not sure I understand. You can't compile a rust crate without its dependencies.
We do offer the ability to use this crate without audio backend (disable the "cpal" feature) and without media decoder capabilities (the "mp3", "ogg", "flac", "wav", "m4a", "alac" features.
So compiling with default-features = false still leaves quite a few dependencies. You can use cargo tree --no-default-features --edges no-dev to inspect them.
Looking a those, I could also make creek and its transitive dependencies an optional feature (it is used for the HtmlMediaElement features) if that helps.

Yes that looks like what I need, thank you!


Cemu Appimage -> AppImage

Small capitalization fix
Was to be a part of PR for Cemu AppImage reset config but that one is in other repo (emudeck-gui-components).

Opened new one on emudeck-electron project, dev branch
https://github.com/EmuDeck/emudeck-electron/pull/22


"makisu version" broken on "go get"

If you run go get github.com/uber/makisu/bin/makisu like the instructions say, you get build-hash-2018-3-28. Works as expected in the container though.

p.s. I'm on OS X


Placeholder issue for cost anomaly detection for scratch accounts

Setup for alerts for unexpected cost changes for our scratch accounts.
Testing between the new detection service vs using a cloudwatch alarm.
Will link to best fit module once it's decided which way to go.

I don't think we are going to bother with this, we'll just keep an eye on costs through the slack notifier, usage is too all over the place for the ML.


Unit Tests for Offboard Commands

Unit Tests for Offboard Commands
Problem
Need simple unit tests to ensure the offboard movement controls the drone in the manner we intend/need.
Solution
The unit tests should test each movement individually, with different files for each axis.
Additional Information
The unit tests should call each function in a controlled environment, logging which functions fail.

Closed via #6


inking-otd: Find a way to move tablet configurations out of libraries

OpenTabletDriver.Configurations.DeviceConfigurationProvider constructor get all configurations from resources inside the OpenTabletDriver.Configurations assembly. If rd.xml can be used to remove all .json files in that assembly when using NativeAOT then I want to:

Move all configurations into a separate location, preferably right inside inking-otd.
Changes the initialize function (from .NET) to accepts the path to configurations directory.
When initializing driver (from JVM), the initializer will copies all configurations to a directory, then call the initialize function from native code with the path to configurations directory.
The function from .NET will have to use reflection (if possible?) to add configurations one by one.

DeviceConfigurationProvider.TabletConfigurations is a read-only property, and DeviceConfigurationProvider.Deserialize() is a private method, so reflection is a must here.
"But why?" - Currently, the configurations are copied to all 6 natives (windows, linux and osx with x64 and arm64 as architectures). Unifying everything into a single set of configurations would reduces storage space a lot.

It is going to take A LONG TIME to add all types from OpenTabletDriver.Configurations. I wonder if there is a way to excludes all .json instead of whitelisting all types from the assembly...


Change set is being executed if the end system version is equals to its system version

If a change set A with  its system version "1.3.4" and the end system version is "1.3.4", the A change set will be executed. Although the documentation suggests that this changeSet wouldn't be executed because the end system version is not inclusive.
I have forced the if condition on the isChangeSetWithinSystemVersionRange method of the ChangeLogService class on line 178 to:
version.compareTo(startSystemVersion) >= 0 && new DefaultArtifactVersion("1.3.4").compareTo(new DefaultArtifactVersion("1.3.4")) <= 0 and the result is true and the change method is included.
My mongock dependencies and versions are:
implementation("com.github.cloudyrock.mongock:mongock-spring-v5")
implementation("com.github.cloudyrock.mongock:mongodb-springdata-v3-driver")
mavenBom("com.github.cloudyrock.mongock:mongock-bom:4.3.4")
Is this implementation the correct behavior or is the doc just not updated?

Hello @jonaslins ,
thank you and good point. It seems that actually the error is in the documentation as now your bugfix breaks a test to specific run the range inclusively(You can see it here)
At this point, I think that we better fix the documentation, otherwise we could potentially break some projects out there already using the systemVersion range inclusively.
Feel free to make pool request to Mongock documentation project. Alternatively, if you can't or wan't, we'll do it.
Thanks again for pointing this out 😄
Error:  Tests run: 10, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.05 s <<< FAILURE! - in com.github.cloudyrock.mongock.runner.core.ChangeLogServiceTest
Error:  com.github.cloudyrock.mongock.runner.core.ChangeLogServiceTest.shouldReturnOnlyChangeSetsWithinSystemVersionRangeInclusive  Time elapsed: 0.019 s  <<< FAILURE!
java.lang.AssertionError: expected:<3> but was:<2>
	at com.github.cloudyrock.mongock.runner.core.ChangeLogServiceTest.shouldReturnOnlyChangeSetsWithinSystemVersionRangeInclusive(ChangeLogServiceTest.java:149)

[INFO]
[INFO] Results:
[INFO]
Error:  Failures:
Error:    ChangeLogServiceTest.shouldReturnOnlyChangeSetsWithinSystemVersionRangeInclusive:149 expected:<3> but was:<2>
[INFO]
Error:  Tests run: 60, Failures: 1, Errors: 0, Skipped: 0
[INFO]
[INFO] ------------------------------------------------------------------------

Cool! I'll fix the documentation then! The current doc is actually the behavior of the mongock version 3. We got this inconsistency because of our internal PR checklist when there's a new migration on the pipeline.

Anyways. I think the inclusive behavior is better and more natural than the old one. It has always caused a bit of confusion.

Yes, I also think that makes more sense to provide the range you want to include.
Thanks @jonaslins

Fixed in documentation


fromFormat as the opposite to toFormat

I can't find an easy way to parse a string that was created using toFormat. I'd like to see a fromFormat function that does that (based on current configuration) and additionally an option to configure BigNumber to automatically parse the strings passed to the constructor. Something like this:  BigNumber.config({ AUTO_PARSE: true });
Does this exist yet? In my case this is only about ignoring the groupSeparator and reading decimalSeparator as '.'.  But it would also have to handle everything else in BigNumber.Format.

Does this work for you?
function removeFormatting ({ fromNumberString, withDecimalPoint = '.' }) {
  const rx = new RegExp(`[^${withDecimalPoint}\\d]*`, 'g')
  return fromNumberString.replace(rx, '')
}

removeFormatting({ fromNumberString: '123<PHONE_NUMBER>5 6789' })
// 123456789.123456789

It works by creating a regular-expression that removes everything that isn't a digit or a decimal-point.

I don't want to enhance the BigNumber constructor so it can handle toFormat strings, but following shuckster a fromFormat function could be added using, for example
// Assumes '-' is only used as a minus sign.
BigNumber.fromFormat = (stringFromToFormat, formatObject) => {
  const sep = formatObject.decimalSeparator;
  const str = stringFromToFormat.replace(new RegExp(`[^${sep || '.'}\\d-]+`, 'g'), '');
  return new BigNumber(sep ? str.replace(sep, '.') : str);
};

(If this function was internal to the library then obviously it would have access to the global FORMAT object and so it wouldn't need to be passed in.)

probably should be called fromFormat -> parse
parse('1000.000-e', {format: '', groupSeparator, ...etc})
parse('1000.000-e')


The padding of markdown preview is broken when zooming up

VSCode Version: 1.6.1
OS Version: Windows 10 Home 64bit build 14393.321

Steps to Reproduce:

Open any Markdown file
Open Markdown preview (Ctrl + Shift + V)
Zoom up (Ctrl + ;)

Notes:

The problem occurs only when zooming up, not zooming down.

Thanks, this is #5745


Update fingerprints.yml

Adding Namron Z-Wave thermostat

 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.

Hey @steinibrodda. Thank you for your PR.
At this time we add device fingerprints as part of our Works With SmartThings program only. If you are interested in learning more about that program please check out https://developer.smartthings.com/ and please reach out to<EMAIL_ADDRESS>

Thought this was SmartThings community not SmartThings inc
man. 30. okt. 2023 kl. 20:47 skrev lelandblue @.***>:

Hey @steinibrodda https://github.com/steinibrodda. Thank you for your
PR.
At this time we add device fingerprints as part of our Works With
SmartThings program only. If you are interested in learning more about that
program please check out https://developer.smartthings.com/ and please
reach out to @.***
—
Reply to this email directly, view it on GitHub
https://github.com/SmartThingsCommunity/SmartThingsEdgeDrivers/pull/1039#issuecomment-1785926887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASLRVWP5TX2PJPG377ZQS2LYB773XAVCNFSM6AAAAAA6UFMZMOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBVHEZDMOBYG4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>


Added links to the new apps/content

Docs Build status updates of commit bc1e32d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/samples/teams-low-code-solutions.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0b1e7b7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/samples/teams-low-code-solutions.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Error when executing npm test

Hello,
Please could you help?

<EMAIL_ADDRESS>test /Users/Kong/Dev/angularjs/angular-unit-testing/example
karma start --singleRun

/Users/Joe/Dev/angularjs/angular-unit-testing/example/node_modules/socket.io/lib/store.js:35
Store.prototype.proto = EventEmitter.prototype;
^
TypeError: Cannot read property 'prototype' of undefined
at Object. (/Users/Kong/Dev/angularjs/angular-unit-testing/example/node_modules/socket.io/lib/store.js:35:42)
at Module._compile (module.js:662:30)
at Object.Module._extensions..js (module.js:673:10)
at Module.load (module.js:575:32)

Hi @chupee120!
There isn't much I can do without having a look into your codebase and just by looking at your errors. But it's related to the way you use Socket.io as you might have to mock it in your tests.
Cheers, Pablo.


ValueError

Any idea what causes this error?
ValueError: Number of classes, 21, does not match size of target_names, 19. Try specifying the labels parameter

Hi @firqaaa and thanks for using Rubrix,
Could you please send a little bit more info? Where the error occurs? The complete error trace could help

Hi, @firqaaa
Did you manage to solve the problem?

I will close the issue for now. Please feel free to reopen if you need to include some additional info.


Modified ulong_divmod from complexity O(n) to O((log n)^2)

Solves #142.
Made the ulong_divmod function tremendously faster.
New time complexity : O(log n)2
But it came with huge lines of code.
I had to do all operations in binary as it made the code easiest to understand.
So, I made a data structure binary list (referred to as bl in the code). Subsequently, I made APIs to mask ulong to bl and bl to ulong, and performed long division in binary.

I forgot to mention that I assumed ulong_divmod takes as input 2 long numbers (without sign) represented as lists. The function returns long number in list representation.
To check it that it works,
    li_a = bl_from_ulong(a)
    li_b = bl_from_ulong(b)
    q, r = bl_divide(li_a, li_b)
    print_bl(q)
    print_bl(r)
    return ulong_frombl(q), ulong_frombl(r)

You can test the function like this
q, r = ulong_divmod(ulong_fromint(2**1000)[0], ulong_fromint(3**100)[0])
print(q)
print(r)

I see that some checks have failed and I will try to get them right in some time.


Integrate Docker Support

Fixes:
Provides Docker functionality for Peering-Manager as requested under Issue #222.
There is also a docker-compose.yml supplied featuring an instance of Postgres with persistent storage which can be used to get a deployment running instantly with a set of default credentials (as defined in docker/web.env).

I'm not a docker power user so I will suppose that this PR is good to me.
However, would you mind moving it to the new dedicated repository?

I have an open pull request for this on the docker project which supports the rqworker and is based upon the netbox docker setup.

Docker related stuff are now located in the following repository: https://github.com/peering-manager/docker


Improved readme, making it more beginner friendly

Added some steps to make it more clear to the beginners trying to run the Sugarizer app locally. Added clone and change directory instruction, and some formatting fixes

That is true, I agree :)
(This PR was the result of a student (in Sugar matrix channel) struggling with setting up the Sugarizer locally)

It seems basic for a developer to know how to clone the repo. Lot of young students has already contributed to Sugarizer during Google Code In without asking for this precision.

It seems basic for a developer to know how to clone the repo. Lot of young students has already contributed to Sugarizer during Google Code In without asking for this precision.

That is true, I agree :)
(This PR was the result of a student (in Sugar matrix channel) struggling with setting up the Sugarizer locally so I thought of making it even easier, but nvm, you can close it :))


Two Hemicompact but not Weakly Locally Compact examples

See discussion at https://math.stackexchange.com/questions/4566624/can-a-hemicompact-space-fail-to-be-weakly-locally-compact

Apologies for the messy Git diff; I was working off a stale copy of master.

Very nice to add this.  I'll review in detail tonight.  But doesn't the word "compactum" mean something else in general (https://encyclopediaofmath.org/wiki/Compactum or https://ncatlab.org/nlab/show/compactum)? Maybe the property should be called "finite compact subsets"?

Depends who you ask I suppose; in my circles I've heard "compacta" used for arbitrary compact (sometimes Hausdorff compact) spaces or subspaces. https://en.wikipedia.org/wiki/Compact_space#Open_cover_definition
Maybe "Compact subsets are finite"? It's a mouthful but very clear; I'm not aware of this property being named anywhere.

Maybe "Compact subsets are finite"? It's a mouthful but very clear;

Sounds good to me. It's very clear, and not too long.

Hmm, seems the result for T291 did not format right. Double backslashes should be single ones and an "and:" was missing.


Using azure-sdk-for-python for Government security FIPS compliance

Hello,
[I was directed by Azure support case<PHONE_NUMBER>000006 to open a GitHub issue for this].
I have a question regarding the azure-sdk-for-python, we use the clients in there to fetch millions of metrics per minute from Azure for us as well as on behalf of our customers.
We're in the process of doing an audit for FIPS 140-2 compliance, and want to confirm if the Python SDK Client code makes FIPS compliant connections to Azure (e.g. TLS 1.2+, and if possible which version at a minimum we should be using in order to be FIPS 140-2 compliant.
This might be a question for the Engineering Azure Python SDK team which maintains the open-source repository https://github.com/Azure/azure-sdk-for-python.
Note that the scope is only the connection between the SDK Client and Azure endpoints is what's at interest here, not whether services within Azure are or are not FIPS 140-2 compliant.
Thanks in advance,
I.O.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Docs:0.10000186,Azure.Core:0.0611055,Compute:0.053724825'

Hi @iotmani that's a great question. @xiangyan99 can you look into this?

Are you asking if our SDK libraries can work under FIPS mode?
Could you tell us which libraries are you using?
My understanding is that FIPS mode needs to be enabled at the OpenSSL level.
Most libraries should continue to work.
But some of our libraries make OpenSSL part of the package we ship (e.g. uAMQP) hence they will not work under FIPS mode.

Hello @xiangyan99,
Thanks for the response.
Yes I'm wondering whether they'd work in FIPS mode or at the least if TLS >=1.2 is used when it's supported (which looks to be the case as requests picks the best one supported by the management.azure.com server for API calls).
To my knowledge, none of the packages we use come with their own OpenSSL libraries, but please do correct me if I'm wrong:
azure-common
azure-core
azure-identity
azure-mgmt-costmanagement
azure-mgmt-compute
azure-mgmt-monitor
azure-mgmt-network
azure-mgmt-resourcegraph
azure-mgmt-sql
azure-mgmt-storage
azure-mgmt-web
azure-storage-blob
msrestazure

Regards,
Islam.

Thank you for the information @iotmani .
You are right. None of the libraries has OpenSSL baked into the package.
They work with TLS >= 1.2. (To clarify, they don't require TLS >= 1.2. If both service and OS support TLS >= 1.2, they will use TLS 1.2 automatically).

Excellent, thanks very much!


Readme Enhancements and better setup

Nothing here

When are you gonna mergez?

If you're going to add README.md, you should probably delete README.rst

I renamed readme.rst to readme.md

When are we gonna merge bob?


[BUG]: Relations circular reference breaks query and drizzle-kit push, etc.

What version of drizzle-orm are you using?
0.28.6
What version of drizzle-kit are you using?
0.19.13
Describe the Bug
Query relations involving multiple tables (i.e., .schema files) where circular references can cause query to throw exception, e.g., ReferenceError: Cannot access [reference] before initialization, and also cause kit push operation to fail. If not already known issue please lmk and will work up a repro. Note: might take several schema files before problem surfaces, and no rhyme or reason for the one that fails, such is nature of circular refs. I've managed to work around issue by adding all my schema definitions and relations into a single file.
Please see discord channel for more details.
https://discord.com/channels/1043890932593987624/1136098683679678524/1136098683679678524
Expected behavior
No response
Environment & setup
No response

If you can please create a repro repo we can take a look faster. We would to rule out an actual circular dependency in your imports.

Thanks for taking the time to look into this. Here is a repo repro'ing the issue:
https://github.com/rolanday/drizzle-circ-dep
It's a solid-start project, but not important. Just run the package.json db:push script and you'll see output below. I didn't bother working up a query because I know then ever the push command fails so will the runtime query.
Reading config file '/Users/roland/GitHub/public/issues/drizzle-circ-dep/drizzle.config.ts'
ReferenceError: Cannot access 'account' before initialization
    at Object.account (/Users/roland/GitHub/public/issues/drizzle-circ-dep/src/schema/account.schema.ts:1:1)
    at Object.get [as account] (/Users/roland/GitHub/public/issues/drizzle-circ-dep/src/schema/account.schema.ts:14:45)
    at Object.<anonymous> (/Users/roland/GitHub/public/issues/drizzle-circ-dep/src/schema/user.schema.ts:17:43)
    at Module._compile (node:internal/modules/cjs/loader:1256:14)
    at Module._compile (/Users/roland/GitHub/public/issues/drizzle-circ-dep/node_modules/drizzle-kit/index.cjs:8623:30)
    at Module._extensions..js (node:internal/modules/cjs/loader:1310:10)
    at Object.newLoader [as .ts] (/Users/roland/GitHub/public/issues/drizzle-circ-dep/node_modules/drizzle-kit/index.cjs:8627:13)
    at Module.load (node:internal/modules/cjs/loader:1119:32)
    at Module._load (node:internal/modules/cjs/loader:960:12)
    at Module.require (node:internal/modules/cjs/loader:1143:19)

Issue resolved: user error. Details here:
https://discord.com/channels/1043890932593987624/1136098683679678524/1156821399109910569

I have similar issues, but it is not consistent.
I hit the same error for a "roles" table which has the exact same config and imports than another table but only the roles table triggers the error when I run drizzle-kit studio.
I am on:
    "drizzle-kit": "^0.20.14",
    "drizzle-orm": "^0.30.1",

It is adviced here to group all relations in the same file. While it works, it is not mentioned in the docs and on every pages, relations are colocated with the table creation.
Also I'd prefer to colocate. What should I do? Is it still a "bug" as colocated relations with table creation provoke this issue or will the docs be updated to state that we should declare relations in a separate file?

Issue resolved: user error. Details here:
https://discord.com/channels/1043890932593987624/1136098683679678524/1156821399109910569

Is this link accessible?

The link seems valid, but you have to be in the Drizzle Team Discord. In essence, the problem is not with Drizzle. It is a problem with Typescript. This issue mentions that drizzle-kit push breaks, for me its also TSC complaining.
To solve it, put your schema definition in a separate file from your relations. When you create your schema with the introspection command, that's the structure you get and here is probably the reason why they split it into two files.


oc-cluster engineer branch

Pull request for changes in oc-cluster-engineers branch

@jorgemoralespou I created a PR from your branch so I can comment on your changes

Here is my mounts-template.json:
{
  "bindMounts": [
    {
      "imagePattern": "(openshift/origin$)|(openshift/origin:.*)",
      "mounts": [
        {
          "source": "$OPENSHIFT_BINARY",
          "destination": "/usr/bin/openshift"
        }
      ]
    },
    {
      "imagePattern": "openshift/origin-deployer.*",
      "mounts": [
        {
          "source": "$OPENSHIFT_BINARY",
          "destination": "/usr/bin/openshift"
        }
      ]
    },
    {
      "imagePattern": "openshift/origin-recycler.*",
      "mounts": [
        {
          "source": "$OPENSHIFT_BINARY",
          "destination": "/usr/bin/openshift"
        }
      ]
    },
    {
      "imagePattern": "openshift/origin-docker-builder.*",
      "mounts": [
        {
          "source": "$OPENSHIFT_BINARY",
          "destination": "/usr/bin/openshift"
        }
      ]
    },
    {
      "imagePattern": "openshift/origin-sti-builder.*",
      "mounts": [
        {
          "source": "$OPENSHIFT_BINARY",
          "destination": "/usr/bin/openshift"
        }
      ]
    },
    {
      "imagePattern": "openshift/origin-f5-router.*",
      "mounts": [
        {
          "source": "$OPENSHIFT_BINARY",
          "destination": "/usr/bin/openshift"
        }
      ]
    },
    {
      "imagePattern": "openshift/node.*",
      "mounts": [
        {
          "source": "$OPENSHIFT_BINARY",
          "destination": "/usr/bin/openshift"
        }
      ]
    },
  ]
}

I get this on RHEL...
Running in developer mode, using DOCKER_HOST=tcp://<IP_ADDRESS>:2375
/root/oc-cluster-wrapper/oc-cluster: line 526: DOCKER_HOST=tcp://<IP_ADDRESS>:2375: No such file or directory
[ERROR] Cluster has not started correctly. Profile configuration will be preserved

Will submit a fix PR to your branch

@csrwng I have installed a linux box and have been testing on linux to fix the issues. They should now work, also deleting the profile works.
I think it's ready to go.

@jorgemoralespou maybe it's the bash version on my RHEL box, but I get this when running oc-cluster status:
Using client for origin v1.5.0-alpha.3
/root/oc-cluster-wrapper/oc-cluster: line 669: syntax error near unexpected token `fi'
/root/oc-cluster-wrapper/oc-cluster: line 669: `     fi'

Looks like there shouldn't be an if on line 661

@csrwng Yes, sorry for that. I tested before making the if test not use the if and forgot to remove that if statement. Now it's been removed.

@jorgemoralespou LGTM ... tested with your latest changes and things are working for me.


Index h3s in docs for search

I was searching for a specific piece of info about deployments, and I searched the term "dependency installation" in docs.
My search returned 0 results, and I had to ask someone on the project for details, and they linked me to this page -  https://www.pulumi.com/docs/pulumi-cloud/deployments/reference/#customizing-the-dependency-installation-step.  I was surprised that I didn't find that via search, given the phrase appears in the h3 there.

This is a good call out. According to the script that builds the index, we've explicitly chosen not to include H3s because they are "too noisy": https://github.com/pulumi/docs/blob/master/scripts/search/page.js#L54 ...
A decision should be made here. Should we include all H3s as well as H2s? (it's a simple change to the jQuery selector in that same location) Or accept that we won't include them and close this as wont-fix/by-design?
@interurban @sean1588

After some discussion, going to reopen and fix this.


Add system.memory.limit

Updates #127
Changes
Adds system.memory.limit which reports the total available memory in the system.
A reference implementation is available at open-telemetry/opentelemetry-collector-contrib/pull/27096
Merge requirement checklist

[ ] CONTRIBUTING.md guidelines followed.
[ ] CHANGELOG.md updated for non-trivial changes.
[ ] schema-next.yaml updated with changes to existing conventions.

After #89 the total value is listed as a valid attribute for system.memory.state, so system.memory.limit would just be system.memory.usage{system.memory.state=total} and I can close this and just implement this on the Collector.


fix(core)!: add getItems and getItemsContainer to RTI constructor

Closes #2644
What I did

add getItems and getItemContainer option functions to RTIC, update call sites
make updateItems parameter default to the result of getItems

Testing Instructions

units, manual keyboard testing for rti elements

@zeroedin merged in the linked branch, PTAL

That branch was never merged into main now so those changes were lost to the current timeline. We need to get that merged back in. I have vague recollection of wanting to side track these changes for 2.5, but I believe they need merged into main for 3.0 now.
New PR opened to merge those tabs changes in main, then we should merge main back here. #2679

Ok we'll have to take a look at the TabsController implementation of the new RTI initialization now.  Looks like you had already anticipated this change:
// TODO(bennypowers): adjust to fit, in or after #2570
this.#tabindex.initItems(this._tabs, this.#element);

If I have the opportunity tonight to take a look at this ill push some code here otherwise first in the morning.

@nikkimk definitely double check keyboard navigation here, but things seem to be working as expected for me.

checked with @nikkimk. we'll merge this even though tests in select are failing and come back to fix those in #2677


Add alex-tan/postgrest-queries client side library

Thank you :)! Already live at http://postgrest.org/en/v5.1/#client-side-libraries

👍 awesome, thanks!


Fix some outdated info

Fixes https://github.com/cncf/tag-contributor-strategy/issues/374
Fix the issues mentioned in https://github.com/cncf/tag-contributor-strategy/issues/374, except the mentoring page which is handled in https://github.com/cncf/tag-contributor-strategy/pull/379
NOTE
Please review commit-by-commit here: https://github.com/cncf/tag-contributor-strategy/pull/380/commits

@geekygirldawn Once you up date your review to approved the PR will merge.

thanks for the big website cleanup before kubecon!

No problem!
Well, if you're up to, there's more 😅 :

https://github.com/cncf/tag-contributor-strategy/pull/381 (has the most impact)
https://github.com/cncf/tag-contributor-strategy/pull/379
https://github.com/cncf/tag-contributor-strategy/pull/384
https://github.com/cncf/tag-contributor-strategy/pull/382 (updated)

My motivation: I volunteered for some booth duties for the TAG and wanted to read more about the parts of the TAG other than Mentoring WG. I saw these issues and tried to fix them.


Query track data in bulk RuntimeError: Master and Return Tracks have no 'Arm' state!

This didn't happen on my other live set but the Ableton Live Suite 11 Tutorial live set
/live/song/get/track_data 0 -1 track.arm would cause the problem
(2024-01-28 19:08:31,008) [INFO] Getting track data: ['track.name', 'track.color', 'track.arm', 'track.solo', 'track.has_midi_input', 'clip.name', 'clip.color', 'clip.is_playing', 'clip.is_recording', 'clip.is_midi_clip', 'clip_slot.has_clip', 'clip_slot.has_stop_button'] (tracks 0..-1)
(2024-01-28 19:08:31,022) [ERROR] AbletonOSC: Error handling OSC message: Master and Return Tracks have no 'Arm' state!
(2024-01-28 19:08:31,051) [WARNING] AbletonOSC: Traceback (most recent call last):
  File "C:\Users\hv\Documents\Ableton\User Library\Remote Scripts\AbletonOSC\abletonosc\osc_server.py", line 220, in process
    self.parse_bundle(data, remote_addr)
  File "C:\Users\hv\Documents\Ableton\User Library\Remote Scripts\AbletonOSC\abletonosc\osc_server.py", line 196, in parse_bundle
    self.process_message(message, remote_addr)
  File "C:\Users\hv\Documents\Ableton\User Library\Remote Scripts\AbletonOSC\abletonosc\osc_server.py", line 147, in process_message
    rv = callback(message.params)
  File "C:\Users\hv\Documents\Ableton\User Library\Remote Scripts\AbletonOSC\abletonosc\song.py", line 133, in song_get_track_data
    value = getattr(track, property_name)
RuntimeError: Master and Return Tracks have no 'Arm' state!

commit eb1dccd0505ce5d012ce2d205266561de9952ece
Ableton Live Suite 11.3.21
I confirm that this still happens on a fresh clone of the current master. All other control surface scripts were removed at the tested time.
Did you test with the Tutorial live set? i.e. Help / Built-in Lessons

Oh yes, just tested with the Tutorial set and I do indeed see the same behaviour. I've just taken a quick look, and it looks like it's because the Tutorial set has a track group within it; if I ungroup the tracks, the issue goes away. Might be to do with the way that song.tracks is counted...


kclient: apply client side filter to index result

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

I like the idea of handling the problem here. Seems like it provides a solution that is more ergonomic going forward.


net core portage

Hello,
I know that System.Runtime.Remoting is not existing in net core but did someone has alread tried to find an alternative to run legacyWrapper on net core ?
thanks !

not required anymore.


[WiP] support new packet types

Add forwarded payments and fulfillment data to the existing Interledger testnet.
This is in preparation of https://github.com/interledgerjs/amundsen/issues/18 which will also introduce much lower timeouts, as well as a new BTP version, and a new LPI version.
We could make the two networks separate, but I think the effort needed to keep the existing testnet compatible is quite minimal (this WiP-PR is a test of exactly how minimal this effort would be).

Merged as part of #22


Issues with go get github.com/moovweb/gokogiri/css

Trying to install this library and getting some error messages. Any help would be great!
$ go get github.com/moovweb/gokogiri/css
# github.com/moovweb/rubex
../github.com/moovweb/rubex/chelper.c:161:43: warning: passing 'const OnigUChar *' (aka 'const unsigned char *') to parameter of type 'const char *' converts between pointers to integer types with different sign [-Wpointer-sign]
/usr/include/secure/_string.h:119:34: note: expanded from macro 'strncpy'
# github.com/moovweb/rubex
ld: warning: ignoring file /usr/local/lib/libonig.dylib, file was built for x86_64 which is not the architecture being linked (i386): /usr/local/lib/libonig.dylib
Undefined symbols for architecture i386:
  "_OnigDefaultSyntax", referenced from:
      _NewOnigRegex in chelper.o
  "_OnigEncodingUTF8", referenced from:
      _NewOnigRegex in chelper.o
  "_onig_error_code_to_str", referenced from:
      _NewOnigRegex in chelper.o
      _SearchOnigRegex in chelper.o
  "_onig_foreach_name", referenced from:
      _GetCaptureNames in chelper.o
  "_onig_free", referenced from:
      __cgo_5dffe3ce5698_Cfunc_onig_free in regex.cgo2.o
     (maybe you meant: __cgo_5dffe3ce5698_Cfunc_onig_free)
  "_onig_match", referenced from:
      _MatchOnigRegex in chelper.o
  "_onig_name_to_backref_number", referenced from:
      _LookupOnigCaptureByName in chelper.o
  "_onig_new", referenced from:
      _NewOnigRegex in chelper.o
  "_onig_number_of_captures", referenced from:
      __cgo_5dffe3ce5698_Cfunc_onig_number_of_captures in regex.cgo2.o
     (maybe you meant: __cgo_5dffe3ce5698_Cfunc_onig_number_of_captures)
  "_onig_number_of_names", referenced from:
      __cgo_5dffe3ce5698_Cfunc_onig_number_of_names in regex.cgo2.o
     (maybe you meant: __cgo_5dffe3ce5698_Cfunc_onig_number_of_names)
  "_onig_region_free", referenced from:
      __cgo_5dffe3ce5698_Cfunc_onig_region_free in regex.cgo2.o
     (maybe you meant: __cgo_5dffe3ce5698_Cfunc_onig_region_free)
  "_onig_region_new", referenced from:
      _NewOnigRegex in chelper.o
  "_onig_search", referenced from:
      _SearchOnigRegex in chelper.o
ld: symbol(s) not found for architecture i386
clang: error: linker command failed with exit code 1 (use -v to see invocation)

What Go version/architecture are you using?
The oniguruma library you have installed on your system is x86_64
Make sure the architecture of Go and your C libraries match

This ended up being a problem with how I installed mySQL actually. Uninstalled, brew installed, everything works now magically.


Bumps azdataGraph version to 0.0.7

This PR bumps the azdataGraph version to 0.0.7

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 48.282%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14766

Relevant Lines:
29039

💛  - Coveralls


Improve headers hierarchy

I just noticed the h1 tag is used not only in the page header (for the site title), but also in the posts title and sidebar titles (like "Related posts"). Shouldn't h1 be used only in the site title, and the other ones be replaced with h2?

@ggarnier Thanks for your advice! I think you're right. In principle, we shouldn't use multiple h1 in one page. But at the same time I also think that both of SITE TITLE and POST TITLE are most important information appropriate for h1. I don't have the best idea for this problem... But at least, sidebar titles (like Related posts) have to be replaced with h2 I think 👍 Could you please some PR?

@ttskch sure, I'll work on that


Update backing-up-a-repository.md

Add Azure Blob Storage as potential backup or storage service to have more options and especially also call out a non-consumer service with SLAs.

Why:
Closes [issue link]

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@gerwaldo
Thanks so much for opening a PR! I'll get this triaged for review :zap:


Datacite DOI not working when the identifiers submission step isn't used

Describe the bug
Enable the DOI datacite registration but don't enable the "identifiers" step. Leave all options as the default. Submit an item with at least one file & look at the DOI table. It will be empty, so no DOI will be registered.
This is because the default "doi-filter" has an "is-archived_condition" BUT when the registration mechanism kicks in the item isn't archived yet !
Identifier registration code: https://github.com/DSpace/DSpace/blob/main/dspace-api/src/main/java/org/dspace/content/InstallItemServiceImpl.java#L75-L86
Setting the item to archived: https://github.com/DSpace/DSpace/blob/main/dspace-api/src/main/java/org/dspace/content/InstallItemServiceImpl.java#L216
I don't know what the best way to fix it would be, do we call on the setArchived(true) BEFORE we pass things on to the identifiers ? Because the finishItem() method expects a handle to be there.
To Reproduce
Steps to reproduce the behavior:

Enable the "org.dspace.identifier.DOIIdentifierProvider" in the identifier-service.xml
Start a new submission & archive it
Check the DOI table after, no DOI will be found in the table

Expected behavior
A DOI should be registered.
Related work
This MIGHT be related to: https://github.com/DSpace/DSpace/issues/9403 (It might not have been visible because of the pre-registration always happening there ?)

@pnbecker what are your thoughts here on Kevin's question, cfr:
I don't know what the best way to fix it would be, do we call on the setArchived(true) BEFORE we pass things on to the identifiers ? Because the finishItem() method expects a handle to be there.

@bram-atmire  you’re right. Please see @kshepherd  comment here: https://github.com/DSpace/DSpace/pull/9271#pullrequestreview-1964925474

I created a PR: https://github.com/DSpace/DSpace/pull/9619


feat: support reading OPTIONAL column in parquet_derive

Which issue does this PR close?
Closes #5716.
Rationale for this change
See details in the issue.
What changes are included in this PR?
Add definition level in macro code.
typed.read_records(num_records, None /*should use a Some(&mut Vec<i16>)*/, None, &mut vals)?;

Are there any user-facing changes?
No API change, extend parquet_derive to handle OPTIONAL columns.

Looks good to me -- thank you @double-free and sorry for the delay in review
I had one small request to update the comments. Let me know what you think

Thanks for your response! Please check my reply and see if we do need to update the code comment.

Thanks again @double-free


dashboard: Add responsiveness for smaller screens/resolutions

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
I have tried out the new dashboard while remotely accessing my firewall over VPN from an android phone. The default dashboard seems not to be responsive and adjust its widgets to smaller screens.
Describe the solution you like
The css could dynamically adjust colums or widget size based on screen size so it looks nicer. I don't know though, it's just feedback. Maybe depending on screen resolution, the widgets could align differently in the grid/columns.
Describe alternatives you considered
Manually moving all widgets into a single row and saving could work, though then it wouldn't look good in bigger screens.
Additional context
Here is a screenshot from my phone:

Unfortunately, the CSS media queries are static and do not play well with the dynamic nature of the widgets, this will have to be adjusted in Javascript.
I was wondering about making the number of columns configurable, but this prompts me to simply determine the right number of columns based on the viewport width.

@Monviech https://github.com/opnsense/core/commit/0bb13de0feca6e7468813011d2177c18d65792c8 should fix it.

Sweet I'll try it out later. Thank you.

I think it works great.
Theres only a little bit of jank when you resize the window to a smaller size, move the widgets around, save, and then resize the window a few times. Some widgets forget the row they are in and there's an empty middle row appearing.
Additionally, when you enlarge widgets over two or three rows, save and resize the window a few times, this seems to get forgotten too and the widgets snap back to one row.
This seems like it will be hard to fix though.

@Monviech I saw this during testing as well, seems to be an upstream GridStack issue. A reload of the page will always lead to consistent widths/heights though (I do believe gridstack gets confused when a lower number of columns have been saved while loading it into a page with an increased number of columns).
There is some logic being executed when widgets are resized which could potentially slow the page down - but I think the bulk of this are Chart.js internals which we have little control over. You can play around with the time limit in the debounce() function in the widget manager, which should limit the number of onWidgetResize() executions.
I've not come around to optimizing things thoroughly just yet, if you see any potential improvements let me know :)

The situation should be somewhat better with https://github.com/opnsense/core/commit/8a5789840c6a22e784ca5d443daab3f064fbc7e9. The only remaining issue I see is that reloading a page which is smaller than the original persisted one will not snap back to the original dashboard when enlarging the window to the original size. A reload will fix this, which it didn't before this commit.


Prepare for pandas extension arrays

Some background: pandas 0.23 will support 3rd party array types (think
Categorical, but defined outside of pandas). What would it take to ensure that
these also work with dask, ideally with minimal effort on the part of the
library author?
from cyberpandas import IPArray
import pandas as pd
import dask.dataframe as dd

arr = IPArray([0, 1, 2, 3])
ser = pd.Series(arr)
dser = dd.from_pandas(ser, 2)

Extension Accessors
Pandas 0.23 will have an official mechanism for 3rd party libraries to add
accessors like .dt, .cat, etc. Libraries decorate their accessor with
register_{series,frame,index}_accessor and that attaches an instance of the
accessor to the class.
In principal, dask could also attach that accessor to dd.Series, dd.DataFrame,
and dd.Index. https://github.com/pandas-dev/pandas/issues/19963 will make
discovery of the accessors easier. Given an accessor name, dask could attach
it to our classes
dd.Series.ip = object.__getattribute__(pd.Series, 'ip')

I'm not sure where this would be done. It might have to be dynamic in
_Frame.__getattribute__, since accessors can be added at any time.
Not all accessors will naively work with dask, but I think many will. All of the
ones I've implemented in cyberpandas are just transformations. They return a
Series or array of the same shape, possibly with a different dtype.
(This exact example fails because of the next issue).
Extension Types
We're allowing 3rd party libraries to define custom array types by subclassing
some pandas ABCs. As far as dask is concerned, we'll need to update blocks like
if is_datetime64tz_dtype(dtype):
    ...
elif is_categorical_dtype(dtype):
    ...
else:
    ...

To include an elif is_extension_array_dtype(dtype) (in practice, some of the
is_categorical_dtype checks can be simply replaced with
is_extension_array_dtype, since Categorical is officially an ExtensionArray).
I'm not exactly sure how things like _meta_nonempty will work. This could be
challenging, as we don't know what a sample looks like. I haven't thought
through this much, hopefully there's a decent solution. There are probably other
issues, but that's the first one I ran into.

Update, extension arrays are in the latest Pandas release.  http://pandas.pydata.org/pandas-docs/stable/extending.html#extending-extension-types


Removing outdated VS references (Part 2)

Addresses #1795: This is Part 2 of the updates, so let's call it Shark Happens!

Primarily aims to remove and harden against VS2010 references.
Also licks a handful of Visual C# 2010 references and removes a 2010 (C# 4-era) book frequently mentioned (in one doc, also drops some C# 3-era references).

@GuardRex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

One last thing

I wouldn't make any rash judgments given my performance today! :cry:


Limit secret access to plugins only

similar to the images filter for secrets, add a checkbox to only inject secret if step is exec as plugin.
this ensure only the intended entry-point do get the secrets to handle.
this does help if a plugin is not based from scratch image but do contain an shell that could be used.

I don't see how this could protect secrets? You could just create a plugin (normal docker image) that leaks the secrets, couldn't you?

this could be helpful in combination with trusted images ex. currently setting trusted image woodpeckerci/plugin-git you can still use:
image woodpeckerci/plugin-git
commands:
  - echo ${SECRET}

adding such option would prevent that

image: woodpeckerci/plugin-git
commands:
  - echo ${SECRET} | base58

-> it's a trusted image
-> secret is leaked

bounty: 50$