helmfile-wrapped: 0.168.0 -> 0.169.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/helmfile/helmfile/releases.
meta.description for helmfile-wrapped is: Declarative spec for deploying Helm charts
meta.homepage for helmfile-wrapped is: https://helmfile.readthedocs.io/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.169.0 with grep in /nix/store/k427c96al6f5wqr0rb7zspclhlzqw01q-helmfile-0.169.0
found 0.169.0 in filename of file in /nix/store/k427c96al6f5wqr0rb7zspclhlzqw01q-helmfile-0.169.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

helmfile
helmfile-wrapped

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/k427c96al6f5wqr0rb7zspclhlzqw01q-helmfile-0.169.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A helmfile-wrapped https://github.com/r-ryantm/nixpkgs/archive/fa96f9d63c4ffbb11d678ccbe1308c49b272fc4a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/fa96f9d63c4ffbb11d678ccbe1308c49b272fc4a#helmfile-wrapped

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/k427c96al6f5wqr0rb7zspclhlzqw01q-helmfile-0.169.0
ls -la /nix/store/k427c96al6f5wqr0rb7zspclhlzqw01q-helmfile-0.169.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 2 packages built:

helmfile
helmfile-wrapped

Maintainer pings
cc @kliu128 @yurrriq for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

I think in the future we should prefer PRs like #349839 instead. Is there some way we can prevent @r-ryantm from opening PRs to upgrade helmfile-wrapped (instead of helmfile)?


Statuspage.io metrics support

guv knows the number of in-flight jobs. This could be used for populating metrics to statuspage.io, like for example Travis does:

http://doers.statuspage.io/api/v1/metrics/

This is live in guv 0.1.x. To enable metrics support you need to do two things:

Set your statuspage.io API key to STATUSPAGE_API_TOKEN env var
Add a metric key with the metric ID to those queue definitions you're interested in your guv YAML config

@bergie  Edited comment to add one more step needed


Revamp option functionality to become more intuitive middleware

Right now developers can create middleware-like functionality but it's called "options" and it requires being done inside of msngr's internal structures which isn't intuitive and the naming just doesn't make sense. So the existing adding and using of options should be transformed into the following:

[ ] Create a top level method called msngr.middleware(key, fn, force). This method is how a developer can create a middleware for msngr.

key is a string that will be used when a developer wants to enable this middleware
fn is a function that is called for the middleware. It's definition is as follows:

msngr.middleware("test", function(message, payload, next) {
    next(); // optionally accepts a new payload to replace the old payload
});

force is an optional boolean that indicates whether this middleware should be forced on all messages. By default this is false which means a developer has to explicitly tell their emits to use whatever middleware(s) they want but when this is true it will execute no matter what.

[ ] Add an msngr.unmiddleware(key) method to allow removal of a middleware.
[ ] Remove the existing way of adding options via the internal API exposed when extending
[ ] Add a msngr("topic").use(middleware) function to the message object which specifies, via a string, what middleware to use. Works almost exactly like the existing msngr("topic").option(key) except use() won't be able to accept any type of special options (those should go into configurations should they be necessary).
[ ] Update documentation

I want this to happen but I need to release 4.0.0 sooner as it has a lot of enhancements and good thing and I likely won't get to this feature for another few months.

This is in 5.0


Refactor: Replace all references from 'Register' to 'Sign Up'

This pull request addresses issue #49. All references to 'Register' have been replaced with 'Sign Up' across the project.
closes #49
Please review the changes and merge if everything looks good.

.

.


Forward method?

Is there a simple forward method one can use to forward emails parsed from a *.eml file?
If not, what are the recommended changes to make? to the Mail::Message object? I guess the to header, but maybe also some other things? :like the signatures?

There isn't a baked-in method because there's no standard way to do email forwards, especially with text/html multipart.
You'd want to create a new Mail object, set its To address to the person you're forwarding to, set its Subject to something like Fwd: #{source.subject}, and set its body to the quoted source.text_part.decoded with a "Forwarded …" message at the top.

Thanks. I am afraid though that this would miss a lot of things like attachements for example :( I'll to create something from scratch then!

👍
On Tue, Sep 19, 2017 at 13:02 Julien Genestoux<EMAIL_ADDRESS>wrote:

Thanks. I am afraid though that this would miss a lot of things like
attachements for example :( I'll to create something from scratch then!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/mikel/mail/issues/1154#issuecomment-330656515, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAx97POYHZo0sTNqXx_yiVaDndiVLqks5skB19gaJpZM4Pc493
.

I just did that and it seems to work fine (even for attachments)
transfer_message = message.dup
transfer_message[:to] = new_to
transfer_message[:subject] = "Fwd: #{transfer_message[:subject]}"
transfer_message.deliver!


Removed link that no longer exists

Article processing charge FAQ was removed. Removing link from welcome page.

will include this in faq pr


Adding 2016

Based on current support https://spmtreleasescus.blob.core.windows.net/install/default.htm
closes https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/2258

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@JoanneHendrickson PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Push.js cache making "go back" weird.

I have bar-nav and bar-tab for header and footer everypage,
sometimes when i press the back button on my phone or
a javascript link which have history.back function,
the page will have two "header" and footer".
The only 'fix' I can do now is remove push.js from javascript ,
or change the cache size down to 0 from 20.
Anyone having same issue?

Still seeing this issue. After using the browser back button or history.back(), the inner HTML of .content is being replaced with the entire inner HTML of body for the page being loaded (i.e. what we're going back to) and rendering gets weird. For example:
Initial loading of a page that has a header bar (.bar-nav), secondary header bar, and a tab bar:

"Click" a link and navigate to another page that only has a header bar:

Browse back via either browser back button or emulated by history.back():


add environment variable

Description of the feature

Backup to a S3 compatible host with https(default), like minio depends on a valid/trusted SSL certificate. There is no way currently to either add the certificate as a trusted or turn off ssl validation. The backup now fails:
CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate (_ssl.c:1129)
the underlying aws cli command line can take a parameter,  --no-verify-ssl and the backups would work to self signed S3 hosts. Best would be to configure this with env variable that if set would add that line to the command
Benftits of feature

Most non cloud deployments will have a self signed certificate for S3 backup locations. Now, its not possible to backup to a S3 compatible host that does not have a valid trusted certificate. Thus the S3 feature is useless unless only using http and that is not compatible with most other clients.
Additional context

Seems reasonable enough to do. I support it.
Tell me though - what would be a command line I would use to allow to use my own CA cert? I can build that in too and one can map a volume to the certs.

For removing cert validation the below change should work I think,
           aws ${PARAM_AWS_ENDPOINT_URL} s3 cp ${TEMP_LOCATION}/${target} s3://${S3_BUCKET}/${S3_PATH}/${target} --no-verify-ssl
That is without using a env to set the --no-verify-ssl
For the own CA validation, I'm out on a limb but from what I can read there are 2 ways:

use $ export AWS_CA_BUNDLE="/data/ca-certs/ca-bundle.pem" where /data/ca-certs would be mapped to a volume where to place the pem file
ca_bundle setting in the config file and mapping a volume to the pem file

I think option one is quick but has a security issue, but most home users I think would be ok with that. Option 2 I think the only viable version is using the export variable and volume mapping, seems not that hard and is more secure

Has been added to 3.0.0
Both options, have a peek at the README on how it works.

Cool!
Tested without adding any of the new parameters, and it worked just fine without verifying the cert.
I set S3_CERT_SKIP_VERIFY to FALSE and it still worked, without pem file.
I tried to test the certificate validation, I'm on unraid and I mapped /data/ca-certs first , then I set S3_CERT_CA_FILE to the path /data/ca-certs/ca-bundle.pem but its still not verifying the cert
There could be something different with running it on unraid with setting the variable S3_CERT_SKIP_VERIFY seems to be default value either way :)

Interesting. There is a tonne of output if you wanted to see the s3 command executing by setting DEBUG_MODE=TRUE.
I am cheating a bit here, the cert and the S3_CERT_SKIP_VERIFY option all go into the same variable like so:
          if [ -f "${S3_CERT_CA_FILE}" ] ; then
                print_debug "Using Custom CA for S3 Backups"
                s3_ssl=" --ca-bundle ${S3_CERT_CA_FILE}"
            fi
            if var_true "${S3_CERT_SKIP_VERIFY}" ; then
                print_debug "Skipping SSL verification for HTTPS S3 Hosts"
                s3_ssl="${s3_ssl} --no-verify-ssl"
            fi

I will break it apart and see if that gives you any other results - I've noticed a few other anomalies which I'll push with 3.0.2 shortly.

yeah, Im not the best to understand the code too but ill see if I can get some debugs for you o 3.0.2 when you have it :)


⚠ Bofrost.lu has degraded performance

In 0ea5a85, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1243 ms

Resolved: Bofrost.lu performance has improved in b5b0f22 after 4 hours, 42 minutes.


Build for Hyprland v0.39.1

I replaced some references of CWindow by PHLWINDOW which allows the plugin to be built for hyprland v0.39.1 again. The new build was tested on v0.39.1-113 and it worked as expected.
My c++ experience is very limited therefore let me know should something be off.
On a side note:
For my rice I'd like to have some features for this plugin which are not yet implemented. I'm willing to implement them on my own but at the moment I have a very crappy c++ setup using vscode since Clion doesn't support projects using Makefiles. Would it be possible to maybe add a CMake or meson build configuration? This would make contributing a lot better since I'd be able to use Clion as my editor of choice

About the PR
Hi, thank you very much for your contribution.
This plugin already builds against 0.39.1, so I'm assuming this PR is addressing changes which will go live in the next hyprland release.
In general, I like that hyprland is moving away from raw pointers, but I decided to support only official releases, so this PR will be reviewed and eventually merged when we get hyprland 0.40 :) but thank you very much, you saved me the effort of doing this myself.
About other changes

Which new features would you like to implement? I'm curious :)
yes, this project should move to more modern tooling like cmake or meson, the setup was just copied from another plugin available back then. For now, you can use bear to generate the compile_commands.json. Something like bear - - make all. You should be able to use this in your editor

Hi
We've already talked once about the support of the git version in the past. What's your opinion to have a dev branch which supports the git version without any guarantees and which gets updated mid releases? When there is a new release the dev branch could simply be merged to the master.
This would allow me to use a custom install script for the plugin and as soon as the plugin breaks for the git version I could make a pull request to the dev branch.
About the features I'm missing from this plugin:
For my sidebar I'd like to have a dispatcher which returns all virtual desktops with some additional information in a format like this:
[
  { "id": 1, "populated": true, "focused": true, "workspaces": [...] },
  { "id": 2, "populated": true, "focused": false, "workspaces": [...] }
]

where the populated key contains whether any windows are on this virtual desktop and the focused key contains whether the virtual desktop is currently focused. The workspaces key is simply an array of all hyprland workspace id's which are associated to this virtual desktop.
I think this dispatcher would make a lot of sense since as of my knowledge it's currently not possible to get all virtual desktops with their workspaces using one single dispatcher and this makes it difficult to implement something like a sidebar in an efficient way
And thanks for the recommendation of bear. Using this I'm now able to use CLion :)

I'll use this pull request to push any new changes should the plugin break once again until the v0.40 version is released

Hi We've already talked once about the support of the git version in the past. What's your opinion to have a dev branch which supports the git version without any guarantees and which gets updated mid releases? When there is a new release the dev branch could simply be merged to the master. This would allow me to use a custom install script for the plugin and as soon as the plugin breaks for the git version I could make a pull request to the dev branch.

I mean, theoretically I'm not against it. I'd rather keep a v$(next hyprland version) branch (which will get deleted after merge).
However, as you said, I'll give no guarantees of mantaining it. If you feel up for it, I could make you a developer of this repo and you'd be free to contribute to each branch every time, without asking for PRs.
In which case, of course, the PR will only happen once the branch is merged to main.

About the features I'm missing from this plugin: For my sidebar I'd like to have a dispatcher which returns all virtual desktops with some additional information in a format like this:
[
  { "id": 1, "populated": true, "focused": true, "workspaces": [...] },
  { "id": 2, "populated": true, "focused": false, "workspaces": [...] }
]

where the populated key contains whether any windows are on this virtual desktop and the focused key contains whether the virtual desktop is currently focused. The workspaces key is simply an array of all hyprland workspace id's which are associated to this virtual desktop. I think this dispatcher would make a lot of sense since as of my knowledge it's currently not possible to get all virtual desktops with their workspaces using one single dispatcher and this makes it difficult to implement something like a sidebar in an efficient way

This sounds good yeah. It wasn't really possible to have this until Hyprland v0.37 I think (or around there). This can be done now, feel free to contribute it whenever you have time :)

And thanks for the recommendation of bear. Using this I'm now able to use CLion :)

glad you managed!

Hi
Sorry, I forgot to answer to your previous comment.
In my opinion it would be better to have a static branch called vnext onto which changes regarding the git version can be pushed. In the time where no new major hyprland version is released this branch should be the recommended branch for hyprland-git users but it may not work for hyprland-git.
As soon as a new major version is released the changes needed for the plugin to work again should be pushed to this vnext branch and once it works it gets merged into the master. After merging the vnext branch can be reused for the next major hyprland release.
In my mind the vnext branch is something like an unstable release: When the latest version of the plugin doesn't work for the hyprland-git users they may give the vnext branch a shot since it always contains the changes of the latest plugin version + there is a change it already contains fixes for the hyprland-git version.

However, as you said, I'll give no guarantees of mantaining it. If you feel up for it, I could make you a developer of this repo and you'd be free to contribute to each branch every time, without asking for PRs.

If I find the time and the plugin isn't working for me I'm open to fixing the plugin. However, I don't want to commit myself to always fix it. Adding me as a developer would make the progress of fixing the plugin for the git version easier since I don't have to always have a fork and create pull requests.
I hope it's clear what my idea for the vnext branch is


PowerShell command will ask you for -VM and you will also need -CreateOption

Set the OSDisk to be Ephemeral (not persisted)
$VirtualMachine = New-AzVMConfig -VMName $VMName -VMSize $VMSize
$VirtualMachine = Set-AzVMOSDisk -VM $VirtualMachine -DiffDiskSetting Local -Caching ReadOnly -CreateOption FromImage

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

ID: 8064dd15-4df9-79ed-9ba2-6faceb0fc6d0
Version Independent ID: 5698d8b5-23e9-878d-fdfd-59f871dd552b
Content: Ephemeral OS disks for Azure Windows Virtual Machines - Azure Windows Virtual Machines
Content Source: articles/virtual-machines/windows/ephemeral-os-disks.md
Service: virtual-machines-windows
Sub-service: disks
GitHub Login: @cynthn
Microsoft Alias: cynthn

@kknight-valorem
Thanks for the feedback! We are currently investigating and will update you shortly.

@kknight-valorem Thats a valid feedback
I have assigned the issue to the content author to investigate further and update the document as appropriate.

@jakaruna-MSFT did you walk through the doc or get anymore details?

We are currently cleaning up old issues and closing out items that are greater than 90 days old. If an issue is still present, please open a new feedback item on the document so we can prioritize correctly.
#please-close


OpenGL error running 'Hello World' code

Hiya! I'm getting:
File "/home//.local/lib/python3.10/site-packages/OpenGL/GL/shaders.py", line 196, in compileProgram
program.check_validate()
File "/home//.local/lib/python3.10/site-packages/OpenGL/GL/shaders.py", line 105, in check_validate
raise RuntimeError(
RuntimeError: Validation failure (0): b'active samplers with a different type refer to the same texture image unit'
Error when running the Hello World code on CPU:

Any pointers on how to fix it?

I'm getting the same issue on a M1 Mac

Seeing the same thing on a rented A10

Based on the jit_render.py code, the OpenGL shader validation error on M1 Macs is occurring in the texture binding and program compilation process. Here are the key issues and required changes:

In the forward_pass function, texture units are not properly managed:

def forward_pass(self, renderer, V, P, cam_pos, flags, program_flags, screen_size, color_list=None, reflection_mat=np.identity(4, np.float32), floor_tex=0):
    # Need to track and limit texture units
    active_texture = lighting_texture  # Could exceed M1's 16 texture unit limit

In the bind_lighting function, texture units are not properly bounded:

@njit
def bind_lighting(pid, flags, light, shadow_map, light_matrix, ambient_light, gl):
    active_texture = 0
    # Need to ensure texture unit doesn't exceed 15 (0-15 range for M1)

Required changes:

Add texture unit management:

MAX_TEXTURE_UNITS = 16  # M1 Mac limit

@njit
def bind_lighting(pid, flags, light, shadow_map, light_matrix, ambient_light, gl):
    active_texture = 0
    # Track used texture units
    used_units = set()

    for i in range(n):
        if active_texture >= MAX_TEXTURE_UNITS:
            break
        # Assign texture units ensuring no conflicts
        texture_unit = active_texture % MAX_TEXTURE_UNITS
        used_units.add(texture_unit)
        active_texture += 1

Modify texture binding in forward pass:

@njit
def forward_pass(...):
    # Track sampler types per texture unit
    sampler_types = {}

    for id in idx:
        if active_texture >= MAX_TEXTURE_UNITS:
            break

        # Ensure different sampler types use different units
        texture_unit = active_texture % MAX_TEXTURE_UNITS
        if texture_unit not in sampler_types:
            sampler_types[texture_unit] = set()

Add validation checks:

def _validate_texture_units(self):
    # Ensure total texture units don't exceed M1 limit
    total_units = (
        len(self.shadow_map) +  # Shadow maps
        sum(1 for tex in self.textures if tex[0])  # Material textures
    )
    if total_units > MAX_TEXTURE_UNITS:
        raise ValueError(f"Total texture units ({total_units}) exceeds M1 Mac limit (16)")

These changes would:

Prevent texture unit overflow
Ensure different sampler types use different units
Validate texture unit allocation before rendering
Work within M1 Mac's OpenGL limitations

The error occurs because the current implementation doesn't properly manage texture units within the M1's constraints.

Same issues on a Windows 11 laptop under WSL2 with both nvidia or intel backend.

import os
os.environ['PYOPENGL_PLATFORM'] = 'glx'

add this before the genesis import

This fixed it for me, also appears to be the solution for #270


Remove outdated script mk_sdcard_image.sh

This script is replaced by the rouge utility.
Signed-off-by: Ruslan Shymkevych<EMAIL_ADDRESS>

The corresponding part of README.md ("Creating SD card image (old way)") is removed as outdated.
Reviewer is added.

Patch is ready to be merged. Reviewer is added.
@iusyk could you please merge?


Please check contains "test" for skipTestGroups, not starts with

Currently, the skipTestGroups only checks for configurations starting with androidTest or test.
But this does not match things like integTest..., systemTest..., functionalTest..., ...
I think for the default recognition it would be more appropriate to check for toLowerCase().contains("test") instead.
To do something like that manually, you need some discouraged work-arounds like in the comments of #22 that reach into the models of all other projects to get all configuration names and then be able to filter on them, or list them all in the root project and hope non get added in subprojects in the future.

Sorry about the delay, I've had a ton going on outside of my OSS work.  Can you take a look at https://github.com/dependency-check/dependency-check-gradle/pull/392 .
Does that work for you?

Sure, regex check is of course fine too.
I'd probably have done
boolean isTestConfiguration = configuration.name =~ /((^|[a-z0-9_])T|(^|_)t)est([A-Z0-9_]|$)/

instead though. :-)

My groovy code looks so much like Java... Fixed: https://github.com/dependency-check/dependency-check-gradle/pull/392/commits/be04e39d1235d1dba34b043cfb251dc8835ae2a6

slightly more correct:
boolean isTestConfiguration = configuration.name ==~ /((^|[a-z0-9_])T|(^|_)t)est([A-Z0-9_]|$)/

Well, I wouldn't use Groovy for a public plugin anyway. :-D

In hindsight I wouldn't either.


Q: How does it handle multiple remotes

Cool tool. I wonder how it behaves when I have 2 remotes e.g. upstrem and orgin?

Right now, the fetch steps pulls from all remotes, but the pull only happens from origin.

An interesting feature would be - when working on forks - that the update pulls from upstream and reapplies to remote.


Code of Conduct Plural

Love the code. One nitpicky thing, though:
I think the plural of "code of conduct" is "codes of conduct," not "code of conducts."

https://en.wiktionary.org/wiki/code_of_conduct#Noun
Good eye :+1:

Thanks for your feedback. See https://github.com/todogroup/opencodeofconduct/issues/84.


[Question] How to make Ped creation callbacks?

Hello. I'm trying to make some kind of a multiplayer mod and I need to stop peds/vehicles/props from spawning and remove the ones I don't need. So is there a way to detect when peds are created so I can check if they belong to a remote player or they should be deleted or are there better ways to make peds and vehicles server sided?

In the current version, no.
I recommend using World#GetAllEntities to compare their handles to a List of handles that you want to keep.
ex:
List<int> allowedHandles = new List<int>();
foreach(Ped p in World.GetAllPeds())
  if(!allowedHandles.Contains(p.Handle)
    p.Delete();

Don't store the Entity Object in the List, of course; only use the Handle.

Thanks for the answer. That's the idea I had in mind but I thought it could be unefficent to run it in the main loop especially that I'm going to do the same thing for vehicles and props probably

@electricman226 There's no need to use a list for that; a HashSet<int> would do the job.

@Nacorpio Thanks mate, fixed that in my reply.

I can confirm it's safe to do a loop through all entities in a script loop: I do it myself in my own project :P

What about player events like player death or shots fired? Is it also safe to detect those manually in a loop?

@TheFuseGamer I'm currently developing a wrapper doing exactly that for my current project.

@Nacorpio Great. I hope you release it soon.

@TheFuseGamer I'm probably not releasing it publically, but you can contact me if you are interested.

@TheFuseGamer Are you trying to create a mod that lets you host your server for your friends to join, by the way?

@Nacorpio I am, yeah. But I don't have any friends in mind to play with so I'm just doing it for fun.

@TheFuseGamer I don't have any serious ongoing projects currently, so I could help you if you want.

@Nacorpio Cool. I'm not familiar with github so how would I go about letting you help?

@TheFuseGamer Do you have Discord or Skype?

Skype. Not sure if it's alright for me to post it here though.

I don't think it would be too much of an issue.

what you are asking requires hooking so isn't possible with this(or .net languages in general) you would need to use C++ and a hooking library for that


Platform-Independent Build

I was getting build errors on this and packages that depend on this when trying to build on Windows because the copyfiles script wasn't compatible.
I found a simple solution by adding a devDependency called "copyfiles" and using that in the script.

Nice one! Certainly makes the project more mature. So far it's only been compiled on four machines if you include the CI server.


Remove broken build options

Sad to see that support for non-Windows was broken to save 0.1% lines of code https://github.com/julealgon/DiabloCore/pull/5
Might as well remove all of it since it's none functional at this point. So much for your complaints about not supporting OSs that the developers don't use I guess.

Sad to see that support for non-Windows was broken to save 0.1% lines of code https://github.com/julealgon/DiabloCore/pull/5

I think you are just missing the bigger picture on this. None of this code will remain as is (certainly not cmake) as I make the move to C#. I was just trying to trim as much as I could to remove anything out of the way for the migration. I didn't know upfront how many lines of code each of those things would really be, and that's a secondary concern really.

Might as well remove all of it since it's none functional at this point.

Only reason I didn't remove this was because I didn't know 100% if I could. I appreciate you doing it.

So much for your complaints about not supporting OSs that the developers don't use I guess.

I'm not quite sure what you mean here.

Glad you like the cleanup :)

I'm not quite sure what you mean here.

I just remember you complaining that Linux developers don't go out of there way to support Windows. So it seamed ironic that you actively removing Linux support.
I don't think I'm missing the big picture, just being pessimistic. But hey, maybe this will end up as the first C# application I encounter with enough build instructions that I can actually run it outside of Windows :D

I just remember you complaining that Linux developers don't go out of there way to support Windows. So it seamed ironic that you actively removing Linux support.

Well currently, I'm not going out of my way to support anything in the C++ stack really 😆 I'm completely ignorant on cmake, so I just wanted to minimize noise as much as I could so that whatever remained was easier for me to parse for the conversion later.

I don't think I'm missing the big picture, just being pessimistic. But hey, maybe this will end up as the first C# application I encounter with enough build instructions that I can actually run it outside of Windows :D

Hopefully that should be the case. I'm looking into using the Monogame engine for the C# version (doing a few tests with it at the moment), and that is portable.
C# has been portable for several years at this point so it surprises me that you've had such bad luck in finding a portable project created with it.

C# has been portable for several years at this point so it surprises me that you've had such bad luck in finding a portable project created with it.

I'm aware, C# proponents keeps telling me. But so fare no one I have med have actually used it or know how to set it up since they all just use Windows. I haven't looked in to it myself either, all I have concluded is that it isn't as simple as downloading Visual Code and clicking build which is what I was told by someone that never tried.


Reworked byte <-> LSP position conversions

I have a WIP review of this; sorry for taking so long. I'll review the new changes and submit the review today.


Fill in missing author properties for failing ckans

See https://github.com/KSP-CKAN/CKAN/pull/3111, these ckans are missing the author property.
This PR adds it to those that fail validation, there's another one for those that validate successfully (#2108).
grep -L "author" **/*.ckan
RSSTextures2048/RealSolarSystemTextures_2048-1.0.ckan
RSSTextures4096/RealSolarSystemTextures_4096-1.0.ckan
RSSTextures8192/RealSolarSystemTextures_8192-1.0.ckan
RSSTexturesDDS2048/RSSTexturesDDS2048-1.0.ckan
RSSTexturesDDS4096/RSSTexturesDDS4096-1.0.ckan
RSSTexturesDDS8192/RSSTexturesDDS8192-1.0.ckan

Maybe we should freeze these instead? Looks like @sarbian just doesn't host RSS textures anymore.

Maybe we should freeze these instead? Looks like @sarbian just doesn't host RSS textures anymore.

Good point. I had the "don't freeze modules they might be mirrored" though in mind, that's why I didn't think of freezing them. But since they aren't (otherwise the inflation wouldn't fail).
I'm gonna push another commit that freezes them, but keep the changes I already made, since it does no harm.

Uh, did I delete something from my server ??

@sarbian, the nathankell folder, probably about 5 years ago?

https://ksp.sarbian.com/nathankell/RealSolarSystem/Textures//2048.zip
https://ksp.sarbian.com/nathankell/RealSolarSystem/Textures//4096.zip
https://ksp.sarbian.com/nathankell/RealSolarSystem/Textures//8192.zip
https://ksp.sarbian.com/nathankell/RealSolarSystem/Textures//RSSTextures2048DDS.zip
https://ksp.sarbian.com/nathankell/RealSolarSystem/Textures//RSSTextures4096DDS.zip
https://ksp.sarbian.com/nathankell/RealSolarSystem/Textures//RSSTextures8192DDS.zip

No big deal, we're just tidying up some old metadata.


Remove memory  allocation in Win32 WatcherImpl::directoryChangeCompletion

In WatcherImpl::directoryChangeCompletion we allocate a Buffer::OwnedImpl to call ioHandle write.
This should be changed to writev to prevent memory allocation or we should implement and use #14244

cc @envoyproxy/windows-dev

I don't have the ability to assign this, but I went ahead and submitted a PR for this issue.


Default groups with voting rights ignored for agenda polls

When creating a agenda poll groups selected in Settings -> Elections -> Default groups with voting rights are not preselected in Entitled to vote field.

https://github.com/OpenSlides/openslides-backend/issues/1675
Did not notice this when suggesting this issue for this sprint - we can move it to the backlog again for now.


Info for adding svelte-drag?

Hi
I wanna add this library svelte-drag to this repo.
This is a single Svelte Action, about dragging. Where do I put this in the README?
Thank u

Create a new field under utilities called 'Draggable' and put svelte-drag in it. svelte-drag looks to be a very good project with great docs. Thanks.

Thanks!! I wanna extend this. I have another library coming svelte-resizable, and another called svelte-rnd, which combines these both. So should I have a category like "Draggable and Drop"?

Just "Draggable" is fine.

I linked to your neodrag repo instead (shows svelte-drag is deprecated by it).
Thanks!


count(links) in needs table columns

It would be great to be able to show the NUMBER of incoming/outgoing links in a table.
Now, it is possible to show the links itself; which gives very wide ( & unreadable) tables.
Often I'm mostly interested whether there is a (related) need or not; or in how many. The details I can see elsewhere (another table, or in the need itself)
I suggest to have/allow a syntax like:  "count( any-ColumnName-that-is-a-list )" for anyplace where
any-ColumnName-that-is-a-list  is allowed. Whenever the col isn't a list is is useless to count --either raise an exception, or (better?) return 0
This seems simple to implement -- the count functions already exist in python:-)
Note: by this proposal it will work for (the default) links and links_back, and for custom-ones.
Attention: the same syntax should also/mainly work when used as:
.. needttable:
   :colums: id, title, count(incomming), count(outgoing), outgoing
   ...

Thanks

Great idea :+1:
Maybe we can make your approach even more generic to allow the definition of python code.
Example:
.. needttable:
   :colums: id, title, code(len(incoming)), code(if outgoing "Yes" else "No"), outgoing, code(if need("timestamp") < datetime.now() - timedelta(weeks=2) "overdue" else "in time" )
   ...

Maybe we can also reuse the concept of dynamic functions for this. Not sure, but I will update this issue when the concept is in place.

Personally, I'm not a fan of allowing code in documentation.  Once others thought is was a good idea, introducing all kind of viruses. Let's learn from that! So, no, I do not support the python-body in eg needtables :-) for the same reason.
Secondly "writers" are not always good programmers-- it scares them away . And, as I like to  convince them to use "Maintainable (technical) Documentation" - like sphinx, needs, plantUML etc. anything that is counterproductive in persuading them, ...
Anyhow
Function-alike syntax, like my suggested "count()", is Oke; it not really code.
Anything that needs ``eval( and friends)`, "bad". Dynamic functions are IMHO on the edge (but didn't;'t realy need/love/study them; for the same reason


Fix for Bulky flaw

A quick fix for the Bulky flaw, which wasn't adding the extra encumbrance.

I'm ashamed of myself. It was two lines and botched one. Thanks for the fix.


Missing ? operator in Erlang

(Original issue 653 created by EddArmitage on 2011-03-29T08:20:24.242990+00:00)
I've recently had a problem with pygmentize not correctly parsing some Erlang code that used the ? operator (Full story here: http://tex.stackexchange.com/questions/14166/red-box-drawn-around-question-mark-operator-in-minted-erlang-code/).
The fix was to add an escaped ? term to the regex of line 659 in functional.py, so it now reads
{{{
operators = r'(+|-|*|/|<|>|=|==|/=|=:=|=/=|=<|>=|++|--|<-|!|?)'
}}}
This seems to fix the issue.

(Original issue 653 last updated on 2011-06-18T08:56:57.003348+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by birkenfeld on 2011-06-18T08:56:56.994399+00:00)
Fixed with <<changeset 2bc2b55c28c9>>.


Separate SlimSocket from core

It's better for SlimSocket (Base, Client, Server) to be separated from core library. It's related but core library can be used without SlimSocket. Also SlimSocket barely depends on Akka.Interfaced and also it will grow regardless of core library.

Separate SlimSocket Project: Akka.Interfaced.SlimSocket


[Bug] Broken download for ephemeral_server due to wrong CA

What are you really trying to do?
Trying to run tests in CICD using auto downloaded ephemeral_server.
Describe the bug
The pipeline where we try to run the test can access outside internet only via proxy. So downloading anything via SSL requires adding our own CA either to CA root ot by appending it via some env variable. We automatically do both and it works everywhere except in pipeline that is downloading ephemeral_server to execute unit tests.
The problem is this: https://github.com/temporalio/sdk-core/blob/c27b739e1a4c3f42c694b203ae3a269f5a8ec63f/core/Cargo.toml#L56
rustls-tls feature == rustls-tls-webpki-roots (https://github.com/rustls/webpki-roots)
That means that downloading ephemeral_server  will be validated only via Hardcoded Mozilla's trusted root certificates and nothing else. System root CA is not included and it is not possible to append any other CA via ENV variable.
I suggest switching to rustls-tls-native-roots. This feature also allows to append custom CA for validation.
Minimal Reproduction
running test via
from temporalio.testing import WorkflowEnvironment

env = await WorkflowEnvironment.start_local()

in an environment where access to outside is provided via self sign proxy
Environment/Versions

OS and processor: Linux x86_64/CICD/Jenkins
Temporal Version: latest
Are you using Docker or Kubernetes or building Temporal from source? Using provided temporal pkgs

There are many reasons why a user may not be able to use the default downloading approach (a TLS-terminating proxy is only one). In these cases, we would recommend downloading/extracting the CLI yourself and setting the executable path as dev_server_existing_path= which will prevent this default download that assumes traditional HTTP/TLS is used.

Yes, we should use system roots and we should fix (I think we incorrectly assumed this was the default).
In the meantime, you can download/extract the CLI yourself and set the executable path as dev_server_existing_path= which will prevent this default download approach.


the problem about github  Account number

When downloading github object, there is an account verification problem, I want to skip this github object, what should I do? I have tried several methods, but they all failed.

I have solved it.
https://github.com/allenai/objaverse-xl/pull/29/commits/f60e8d576912c43c349d9eb4309ef19ed37131fe


jbig2dec: Fix cross-compilation

Description of changes
Fix cross-compilation issue -- autogen.sh call ./configure without arguments.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Nah, rebasing and pushing caused the review requests. Next time mark as draft before the rebase, then ofborg/github doesn't start requesting reviews.

@Mindavi well, I found that it not correctly works on staging (but works on master, and as overlay on 23.11, investigating, marked PR as draft)

Looks like it fixed independently in f4c51495b73809304062bac1ee487297a707a4c0 (by setting $configureScript)

Oh, and I merged it...

@Mindavi I'll do backport of merged fix (not this one), to staging-23.11, would you like to review it?

Sure, sounds good to me.


Show QR code for receive address in "receive bitcoin" dialog

Here we should also include an image showing a QR code of the generated address:
https://github.com/yzernik/squeakserver/blob/e795c20f58c3b8cb0229ac40fd6d02adb878b4f5/frontend/squeak-node-frontend/src/components/ReceiveBitcoinDialog/ReceiveBitcoinDialog.js#L126-L132

Closing because not responsible for bitcoin wallet anymore.


CI flow for npm publish

I think we should use git tag to manage npm version. How you think about this idea?

I think that's a good idea!!


Fix for #2105

derive signature count from signatures not recovered keys.  Also do it the same whether in push_transaction or apply_block EOSIO/eos#2105
This resolves #2105

This pull request is verified in the source code and in the unitest case of pull request #2172


interop-reactiveStreams support conversions in both directions

I need to investigate a test that only fails on Travis and will open a new PR once it has been fixed.


add kind for Asset Pack of Play Asset Delivery

What does this PR do?
Android Studio向けに、Play Asset Delivery の Asset Pack をプロジェクトに追加するため、
kindに 'AssetPack' を追加しました。
https://developer.android.com/guide/app-bundle/asset-delivery/build-native-java?hl=ja
How does this PR change Premake's behavior?
プロジェクトの kind に 'AssetPack' を指定すると、locationで指定したプロジェクトのフォルダパスに
Asset Pack のbuild.gradleを生成し、そのフォルダパスをsettings.gradleに追加します。
Anything else we should know?
ありません
Did you check all the boxes?

[x] Focus on a single fix or feature; remove any unrelated formatting or code changes
[x] Add unit tests showing fix or feature works; all tests pass
[x] Mention any related issues (put closes #XXXX in comment to auto-close issue when PR is merged)
[x] Follow our coding conventions
[x] Minimize the number of commits

You can now support Premake on our OpenCollective. Your contributions help us spend more time responding to requests like these!

LGTM 👍


[WIP] Fix tests in aws_test.go to use secrets

fail test :AWS cloud AddAccountResourceSelector all tests:
`•! Panic in Spec Setup (BeforeEach) [0.004 seconds]
AWS cloud
/home/bangqizhu/Desktop/webhooks/testrepo/pkg/cloud-provider/cloudapi/aws/aws_test.go:42
AddAccountResourceSelector
/home/bangqizhu/Desktop/webhooks/testrepo/pkg/cloud-provider/cloudapi/aws/aws_test.go:275
VM Selector scenarios [BeforeEach]
/home/bangqizhu/Desktop/webhooks/testrepo/pkg/cloud-provider/cloudapi/aws/aws_test.go:380
Should match expected filter - single vpcID only match
/home/bangqizhu/Desktop/webhooks/testrepo/pkg/cloud-provider/cloudapi/aws/aws_test.go:381
Test Panicked
failed to add object &Secret{ObjectMeta:{account01  namespace01   999 0 0001-01-01 00:00:00 +0000 UTC   map[] map[] [] []  []},Data:map[string][]byte{credentials: [123 34 97 99 99 101 115 115 75 101 121 73 100 34 58 32 34 34 44 34 97 99 99 101 115 115 75 101 121 83 101 99 114 101 116 34 58 32 34 34 44 34 114 111 108 101 65 114 110 34 32 58 32 34 114 111 108 101 65 114 110 34 44 34 101 120 116 101 114 110 97 108 73 68 34 32 58 32 34 34 32 125],},Type:,StringData:map[string]string{},Immutable:nil,} to fake client: no kind is registered for the type v1.Secret in scheme "pkg/runtime/scheme.go:100"
`

Rahul helped me to get the successful runs for aws_test.go.
@bangqipropel : Please checkout the PR locally and run the tests.
Also fix the tests in aws_security_test.go and other files as needed.

Not needed


Submariner: add support for Baremetal agent HCP

Baremetal hosted control plane clusters are provisioned using
Agent platform, the Agent platform uses central infrastructure
management service to add worker nodes to a hosted cluster.

The content of the hostedcluster CR spec for baremetal HCP cluster should
be something like [1], and the provider should be identified [2]
as Provider.hostinventory.

This PR adds Submariner support for baremetal HCP cluster.

[1]
'
 spec: {
   platform: {
     agent: {
       agentNamespace: clusterName,
     },
     type: 'agent',
 }
'
[2]
https://github.com/stolostron/console/blob/main/frontend/src/resources/utils/get-cluster.ts#L572

/assign @aswinsuryan

/approve


bump master build version

Simply change master branch to 11.0.999

You can test this PR using the following package version. 0.10.999-cibuild0023271-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


Add codeclimate

We really should be doing codeclimate scans for pull requests. Of course, first we need to dramatically bring down the number of problems in our codebase. I've started to work on this and will be occasionally committing work in the issue/codeclimate branch.

Fixed via #1178


Prevent Android crashes

@dabit1 What sort of crashes does this prevent?

I don't know why but sometimes my app crashed when I loaded maps. Reading logcat I found that map var was null.

Besides, other methods in the same class checks if map var is null before do an actions, likely for the same reason.

I don't see this check happening anywhere else in the class. Can you show me where you see this?
You can do a check for ref in JS before calling any of these methods.

Why this pr was not merged? This crash happens many times on my HTC devices.
Now I have to do some tricky method to prevent the crash.
Pls merge the pr !
Fatal Exception: java.lang.NullPointerExceptionAttempt to invoke virtual method 'void com.google.android.gms.maps.GoogleMap.setPadding(int, int, int, int)' on a null object reference Raw Text
| com.airbnb.android.react.maps.AirMapView.fitToCoordinates (AirMapView.java:655)
  | com.airbnb.android.react.maps.AirMapManager.receiveCommand (AirMapManager.java:236)
  | com.airbnb.android.react.maps.AirMapManager.receiveCommand (AirMapManager.java:27)

Thanks for submitting this and sorry for the late response. An updated PR has been submitted here.
https://github.com/airbnb/react-native-maps/pull/1835


Roles not properly parsing

In 0.2.5, roles are not being parsed fully in the test project (BRS_LOCKED6_20210521 - 01 Opening Scene 2023-11-22 07-59-54 - #37).

I now have sync-clips parsing roles correctly, even in very complex projects. It was extremely nuanced when nested clips were involved and took some very careful fine-tuning of the parser to accommodate. MarkersExtractor will now output the roles as they are seen in Final Cut Pro (and therefore as the user expects).
Next one to tackle is multicam clips which are equally nuanced but in different ways.

While we're at it, there are two questions about how we should format data that I've had come up recently while reworking things.
1. Title Case for Default Roles
What I'm observing is that sometimes FCP exports roles as lowercase. From what I can tell, it only does that for "built-in" or "default" roles.
For example, FCP will show "Dialogue" but in the FCPXML it will output the role as "dialogue" or "dialogue.dialogue-1".
I have noticed it happen with "Dialogue" and "Music" roles, and I believe "Effects" may be lowercased as well.
(User-defined roles are always verbatim - character case is never changed when output to FCPXML.)
This means that currently, some default roles may output to the manifest file as lowercase since MarkersExtractor is taking them as-is. (The only modification currently being done is reducing the subrole when appropriate. ie: "dialogue.dialogue-1" will be reduced to "dialogue" for the manifest file.)

💡 Question: Do we want to title-case these roles so they appear as they do in FCP? Essentially there would be a process added that detects default role names and makes them title-case (capitalizes first character).

2. Matching FCP's Display of Roles: Main vs. Sub-Role
FCP only shows the sub-role in the clip inspector. This is for both audio and video roles. Video roles have an automatic sub-role generated by FCP even though it's not obvious. The audio sub-role is more obvious since it breaks them out and forces you to use them. Small technical detail, but useful to know.
Currently we output both main role and sub-role for video role when it's not reducible. ie: "VFX.Smoke". However, FCP will display this in the inspector view as just its sub-role, ie: "Smoke".

Obviously, outputting main + sub-role provides more data for us to work with.
However, with more recent refactors and bug fixes, I have made audio roles output as FCP displays them in certain cases such as sync-clips where there may be multiple audio sources.
For example, if two audio sources "Dialogue.MixL" and "Dialogue.MixR" are enabled, FCP displays it in the inspector as a comma-separated list of sub-roles, ie: "MixL, MixR". This is what MarkersExtractor will output to the manifest file for Audio Role. And this is what I suspect users will expect.
In consideration of making the manifest output consistent, do we also implement this for Video Role to only show sub-role?
If we don't, then do we make Audio Role show main + sub-role for all roles? So instead of "MixL, MixR" it would read as "Dialogue.MixL, Dialogue.MixR". The trouble with this is that it can get very long very quickly. In that complex project you sent me, sometimes there are upwards of 6 audio sources and it is far more concise to use FCP's behavior of only using sub-role.

💡 Question: Do we want to match FCP's display of Roles in the user interface for our output manifest?

Question: Do we want to title-case these roles so they appear as they do in FCP? Essentially there would be a process added that detects default role names and makes them title-case (capitalizes first character).

Cosmetically, it would look nice in the database if we can capitalises first character?

In consideration of making the manifest output consistent, do we also implement this for Video Role to only show sub-role?

I think it would be highly beneficial for users If we can have MainRole.Subrole. It becomes more consistent? Subrole would never be printed or list alone in the manifest output?
Users would have multiple VFX "Stems" for the shots.

Hence in the Database's Video Role & Subrole  column, users can have VFX, VFX.Mirror Ball, VFX.Clean Plate, VFX.Main Plate values in the database. And users can easily create a search filter where  Video Role & Subrole contains "VFX".

If we don't, then do we make Audio Role show main + sub-role for all roles? So instead of "MixL, MixR" it would read as "Dialogue.MixL, Dialogue.MixR". The trouble with this is that it can get very long very quickly. In that complex project you sent me, sometimes there are upwards of 6 audio sources and it is far more concise to use FCP's behavior of only using sub-role.

Question: Do we want to match FCP's display of Roles in the user interface for our output manifest?

Does it make sense to include all of them?  Example - Dialogue.MixL,Dialogue.MixR,Dialogue.Boom-1,Dialogue.Boom-1 etc.
That way, all the values would become a multi-select item in the database. It also gives users insight into their timeline if they have missed anything. Just some thoughts.
PS: Don't include space when adding ",".
Using Custom Column Types

csv2notion_neo --token YOUR_TOKEN_HERE --column-types "number,multi_select" test.csv

I think it would be highly beneficial for users If we can have MainRole.Subrole. It becomes more consistent? Subrole would never be printed or list alone in the manifest output?

Sure, that's doable.

Does it make sense to include all of them? Example - Dialogue.MixL,Dialogue.MixR,Dialogue.Boom-1,Dialogue.Boom-1 etc.

That would then be consistent, yes. More verbose, but no information is abbreviated or lost.

Don't include space when adding ,.

Sure. FCP uses ,  (with the space) as the delimiter when displaying multiple roles in the UI, so I was just following that convention.

all the values would become a multi-select item in the database

Clever. Makes sense to format with just a comma separator then.

One possible wrinkle is that FCP allows the , comma character to be used when the user names roles. So a consumer of the CSV parsing this field may be more complex than just separating it by comma characters. A possible solution might be to use a null character (ASCII code 0, or "\n" in Swift string literal) instad of a comma to separate multiple roles.
For example, the Audio Role & Subrole field may contain these three totally valid roles:
Dialogue.MixL
Dialogue.MixR
Some Role, With Comma.Some Role, With Comma-1

With a comma delimiter, it would be entered in the manifest as follows:
Dialogue.MixL,Dialogue.MixR,Some Role, With Comma.Some Role, With Comma-1

With a null delimiter, it would be possible to parse it. (Represented as <NULL> here for demonstration.)
Dialogue.MixL<NULL>Dialogue.MixR<NULL>Some Role, With Comma.Some Role, With Comma-1

drop csv and switch to json

It's not a bad idea, honestly. As the manifest data becomes more descriptive over time, we may outgrow the capabilities of CSV any way.

I now have sync-clips parsing roles correctly

I have now also resolved Multicam roles parsing issues, and the fix for both will be in 0.2.8.
All roles should be parsing correctly now. That will allow us to move onto other features next-in-line, like role exclusion (#57).

we can have MainRole.Subrole. It becomes more consistent
include all of them. Example - Dialogue.MixL,Dialogue.MixR,Dialogue.Boom-1,Dialogue.Boom-1

Multiple roles are now output to the manifest with full main role + sub-role, using a comma delimiter without a space (,). Will be in 0.2.8.


pipenv might be nice

pipenv might be nice. implements a locked Pipfile like yarn does, along with virtual env.
Pipfile would replace requirements.txt.

Sounds like a good idea, I will look into it.


Opening a modal triggers a click inside inputs within that modal

Short description of the problem:
If you click to open a modal, that click propagates into the modal itself and can click open an input.
What behavior are you expecting?
For the initial opening click not to affect anything inside of the modal.
Steps to reproduce:

Create a page where you can open a modal with a button where an input sits directly in the same spot inside the modal
Open a modal with the button
Watch they keyboard appear due to focus from clicking onto the input

Recording of issue
https://cl.ly/1R2n0o0U3U45
Which Ionic Version? 1.x or 2.x
RC0 V2
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 6.3.1
Ionic Framework Version: 2.0.0-rc.0
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.1.0-beta.1
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Mac OS X El Capitan
Node Version: v6.7.0
Xcode version: Xcode 8.0 Build version 8A218a

Hello, thanks for opening an issue with us! Would you mind posting a repo that i can use to reproduce this issue? Thanks!

@jgw96 probably related with https://github.com/driftyco/ionic/issues/7693
It does not happen in desktop and it is not longer an issue in iOS WKWebView.
This is a UIWebView-only issue

Confirmed that this is fixed with WKWebView.  Thanks guys, Ionic is amazing, keep it up.


Doesn't work using some code density

For struct using code density:
public struct Advert: Codable {
    public let location: Location
    public let imageId, unitId, name: String

}

it gonna create initializer like this:
public init(location: Location, imageId, unitId, name: String,
which is not legal.

The format is not supported, as per "Known limitations" in the readme. I'm willing to accept a PR that adds this though!


Can't do a fuzzy search for settings from built-in extensions

From https://github.com/Microsoft/vscode/issues/38329
Need to update or remove _settingsByName (there are two)

Verification

Open settings editor, close vscode
Open vscode (must start with settings editor persisted as open)
Search 'typescript'
Verify that you see settings from the typescript extension


Fix image paths by generating urls correctly

Previously using image-url which did not generate the correct url with the custom asset prefix. Using image-path creates the correct path, but needs the url helper which is added here.

I don't think we've had to do this for other projects.  Why is this needed here?

Ah, have just updated this PR. After renaming the css files from *.css.scss to *.scss, image-url appears to work correctly again

:+1: Good stuff.


[FEATURE]: Load the Model once

Description
Hi, for the verify function instead of passing only model_name, we can give an option to pass the whole model for both face detection and recognition, so that way we can reduce a huge amount latency.
def verify(
    img1_path: Union[str, np.ndarray, List[float]],
    img2_path: Union[str, np.ndarray, List[float]]
    model: Union[str, FacialRecognition] = "Facenet512",
    detector_backend: Union[str, Detector] = "opencv",
    distance_metric: str = "cosine",
    enforce_detection: bool = True,
    align: bool = True,
    expand_percentage: int = 0,
    normalization: str = "base",
    silent: bool = False,
    threshold: Optional[float] = None,
    anti_spoofing: bool = False,
) -> Dict[str, Any]:

model loading is singletonish. if you call build_model before verify, then you will not spend time to build a model.

_ = DeepFace.build_model("Facenet")
DeepFace.verify(img1, img2, model_name = "Facenet")


Error: failed to find libmagic

Hi,
python: version 3.11.3
o/s: Windows 10
I installed the sbom4files as instructions. Tried to run it and received the following error:
Traceback (most recent call last):
File "", line 198, in run_module_as_main
File "", line 88, in run_code
File "\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\Scripts\sbom4files.exe_main.py", line 4, in
File "\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\sbom4files\cli.py", line 16, in
from sbom4files.filescanner import FileScanner
File "\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\sbom4files\filescanner.py", line 7, in
import magic
File "\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\magic_init.py", line 209, in
libmagic = loader.load_lib()
^^^^^^^^^^^^^^^^^
File "\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\magic\loader.py", line 49, in load_lib
raise ImportError('failed to find libmagic.  Check your installation')
ImportError: failed to find libmagic.  Check your installation

Fixed in version 0.3.0

Thanks for the feedback.
I can now reproduce the issue within a python virtual environment.
It would appear that there is a clash between the python-magic library
(used by Linux systems) and python-magic-bin which is used by Windows. Both
get installed on Windows which is not required.
Uninstalling python-magic allowed the program to run (sbom4files --help)
but when I tried to create an SBOM I got another error.
I will look into this later but can you provide details of your environment
and confirm your version of python and the installed modules (pip list will
be ideal).
On Thu, 10 Aug 2023, 15:12 Sajeenth Shivauthayalingam, <
@.***> wrote:

I'm having the same issue right now.
raise ImportError('failed to find libmagic. Check your installation')
ImportError: failed to find libmagic. Check your installation
—
Reply to this email directly, view it on GitHub
https://github.com/anthonyharrison/sbom4files/issues/2#issuecomment-1673302550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAID23XE726FM7S2B7WMNTXUTT6JANCNFSM6AAAAAA2OQNGJA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

This issue highlights the problem and potential temporary workaround https://github.com/ahupp/python-magic/issues/248

Try the following workaround (seems to be related to some dependency order issues)
pip uninstall python-magic
pip uninstall python-magic-bin
pip install python-magic
pip install python-magic-bin


CI: Node client API wrapper

Motivation
Interactions with the node across our CI and tests are scattered and using different methods and tools instead of a unified method. once we'll have such a central unit for all interactions with a node, we'll be able to to upgrade this for all tests and future tests as well.
This change can also precede #2316 or even follow it. it will give us the ability to switch the method of data retrieval from the node easily and not be dependent on one method like today with the logs.
As part of #2494 we're breaking the CI into smaller modules that can be available across our tools and even individually.
Design

A client that can be created once for each node using the data extracted from the required information stored on k8s metadata.
Ability to execute code in the container
Run checks against the node using the API or logging (temporarily and with provided client)
A multi client abstraction to run something on all or a group of nodes.

Eventually we might host these services along with some other clients as an API exposed to test runners like mentioned in #2494

Note: this will probably be an abstraction over the network client in #2502 at first, but the point is to be able to easily replace the internals .

Can you please clarify a bit. Specifically, are we talking about this pattern?

If a test (or any other ci client) needs node data or to send a command to the node then it should use one of the existing api services methods, e.g. NodeService, MeshService, GlobalStateService if such a method exists.
Otherwise, add api method(s) to the DebugginService (which is designed to CI use only).
I think this is a good design pattern to follow and matches the idea that api services outside of DebuggingService are designed mainly for usage by user-facing clients (e.g. smrepl, smapp, dash, explore)

@avive , you are talking about #2316. this issue is about structure within the CI while you are referring a broad change across node, api and its clients - this is an ongoing issue and not fully related to this.
this issue is meant to make changes like these easier to do, basically breaking the dependency on a specific way of retrieving information. eventually we'll want it to access the API like you mentioned but also if other methods will be needed to perform some checks we could add it within the Node Client. (for example retrieving cpu/mem from the machine or reading a log line).


SweepSuccess: use that to finalize Bob's side

this event is not reliably being emitted, although the sweep is actually successfull

it may be important to get this to work correctly, because its the best trigger for finalizing the swap, @TheCharlatan
around the time such a log is produced a sweep success event could be sent?
[2021-11-18T19:20:47Z INFO  farcaster_node::syncerd::monero_syncer] sweep transaction hash 7ef95f20a773c6319d46f829142a6090374ef1592a7ae3446df0cc2e84be68bd

As far as I can tell, the SweepSuccess message is sent correctly. I tested it on a couple of swaps and always saw it on the logs.


How to add Schema Markup?

Description
I want to add schema markup that I've created using Google's Structued Data Markup Helper tool.
Where do I place the below code?

Any movement?

@hjvedvik
Does Gridsome not support schema?

I haven't used the Structured Data Markup Helper myself, but I guess you want to add that to your markup? Take a look at the docs for adding script tags with here vue-meta.

You can for example do this in the page component where you want the script. Just replace innerHTML with the contents of your script tag.
<template>
  <Layout>...</Layout>
</template>

@jd-fillmore closing, as your issue has been addressed above :point_up:
Feel free to re-open if you're still in need of help!

hi add this method but rich test tool not find my rich snippet http://kolaytatligrid.surge.sh/yazilar/

If you want you can do something like this if its cleaner for you.
 data() {    return {      jsonld:  {       "@context": "https://schema.org",       "@type": "FAQPage",       "mainEntity": [{         "@type": "Question",         "name": "What is the return policy?",         "acceptedAnswer": {           "@type": "Answer",           "text": "Most unopened items in new condition and returned within 90 days will receive a refund or exchange. Some items have a modified return policy noted on the receipt or packing slip. Items that are opened or damaged or do not have a receipt may be denied a refund or exchange. Items purchased online or in-store may be returned to any store.

Online purchases may be returned via a major parcel carrier.  Click here  to initiate a return.
"         }       }, {         "@type": "Question",         "name": "How long does it take to process a refund?",         "acceptedAnswer": {           "@type": "Answer",           "text": "We will reimburse you for returned items in the same way you paid for them. For example, any amounts deducted from a gift card will be credited back to a gift card. For returns by mail, once we receive your return, we will process it within 4–5 business days. It may take up to 7 days after we process the return to reflect in your account, depending on your financial institution's processing time."         }       }, {         "@type": "Question",         "name": "What is the policy for late/non-delivery of items ordered online?",         "acceptedAnswer": {           "@type": "Answer",           "text": "Our local teams work diligently to make sure that your order arrives on time, within our normaldelivery hours of 9AM to 8PM in the recipient's time zone. During  busy holiday periods like Christmas, Valentine's and Mother's Day, we may extend our delivery hours before 9AM and after 8PM to ensure that all gifts are delivered on time. If for any reason your gift does not arrive on time, our dedicated Customer Service agents will do everything they can to help successfully resolve your issue.

Click here to complete the form with your order-related question(s).
"         }       }, {         "@type": "Question",         "name": "When will my credit card be charged?",         "acceptedAnswer": {           "@type": "Answer",           "text": "We'll attempt to securely charge your credit card at the point of purchase online. If there's a problem, you'll be notified on the spot and prompted to use another card. Once we receive verification of sufficient funds, your payment will be completed and transferred securely to us. Your account will be charged in 24 to 48 hours."         }       }, {         "@type": "Question",         "name": "Will I be charged sales tax for online orders?",         "acceptedAnswer": {           "@type": "Answer",           "text":"Local and State sales tax will be collected if your recipient's mailing address is in:

Arizona

California

Colorado

"}         }]     }    }  },  metaInfo() {    return {       script: [         { innerHTML: JSON.stringify(this.jsonld), type: 'application/ld+json' }       ] } },


i can't add comments

What happened?
i add comments.html,but it did not show.
Theme version
lasted
Hugo version
hugo v0.101.0-466fa43c16709b4483689930a4f9ac8add5c9f66 windows/amd64 BuildDate=2022-06-16T07:09:16Z VendorInfo=gohugoio
What browsers are you seeing the problem on?
Chrome
Relevant Hugo log output
no everything

Where in your project have you placed the comments.html file? It needs to be in layouts/partials/comments.html in order to be picked up by the theme.
You also need to ensure showComments is set to true in your theme configuration or in the article front matter.

Where in your project have you placed the comments.html file? It needs to be in layouts/partials/comments.html in order to be picked up by the theme.
You also need to ensure showComments is set to true in your theme configuration or in the article front matter.

I'm sorry,i don't know this file should in theme folder or project folder.

comments.html should be in the project folder,i'm sorry.


Add Base setup for API projects

Why
For the API project, we need to set up some Plug, Param Validation, or Error View, we could inherit these configurations from our existing API projects.
Who Benefits?
The new API project's developer 🤞

Hi @byhbt can you share a plan on which component will you add into the API variant (make it as a TODO list) on this issue so we can keep track easily

[ ] Param Validation
[ ] API error view
[ ] Fallback controller
[ ] ....

Hi @byhbt 👋did you check the comment above :)


Delete peak elements from max heap until the last element.

💥 Proposal
You are given a max heap,
your task is to delete the peak elements from the max heap until the last element is left.
Have you read the Contributing Guidelines on Pull Requests?
Yes
I want to work in C++

/assign


Tutorial is broken

Hi,
I tried your tutorial on gitpod and here is the code that I used :
archetype hello

variable value : string = ""

entry main () {
  value := "Hello Tezos world!"
}

but after deploying it, completium-cli cant call the main with following error :

$ completium-cli call 1-hello
main is not found.

is the tutorial up to date and supposed to work ?

Hi @FreedomPrevails thanks for the report, it is fixed now.
Indeed we recently changed the way completium-cli calls contracts with only one entry point; now there is no special treatment and you need to specify the entry point anyway; hence here the command is:
$ completium-cli call 1-hello --entry main

Kind regards.

Thank you for your reply.
Sorry to bother you but now I have a problem with part 2 of tutorial.
Based on the code, it should not accept any odd numbers and should fail but I was able to set it to 15 ! is this a bug ?
Here is the contract :
https://better-call.dev/granadanet/KT1LsLmriSynuLuvkBEu5TNyHDAYdj44v8MB/operations

I found the problem. seems the code in tutorial is wrong, r2 has used value instead of v :
r2 otherwise "EXPECTED EVEN VALUE": value % 2 = 0
it should be :
r2 otherwise "EXPECTED EVEN VALUE": v % 2 = 0

I am starting to learn tezos smart contract programming, there are a few other choices like ligo and smartpy and etc.
do you think archetype is stable and production ready ? does it provide all the required features for smart contract programming right now ?

Hi @FreedomPrevails ,
Thank you for your feedback,
indeed the value should be v and not value... it is fixed now.
Yes Archetype is stable and active; it produces production quality code and provides the required features:

all Michelson primitives are available
high-level abstractions to ease contract programming (new types, state machine, assets ...)
VS code extension with syntax highlighting and compilation commands
formal specification language for formal verification with why3
dedicated JS lib for test scenarios

We are currently working on a decompiler Michelson --> Archetype, and step by step execution in VS code.
Hope this helps,
Benoit


Debug Test in vscode fails inside container

Configuration:
I am using the following version of VSCode:
Version: 1.46.1
Commit: cd9ea6488829f560dc949a8b2fb789f3cdc05f5d
Date: 2020-06-17T21:13:08.304Z
Electron: 7.3.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.3.0-59-generic snap

With the Python Extension ms-python.python version v2020.6.8. The only PIPs installed are:
Package           Version
----------------- -------
astroid           2.4.2
attrs             19.3.0
autopep8          1.5.3
isort             4.3.21
lazy-object-proxy 1.4.3
mccabe            0.6.1
more-itertools    8.4.0
packaging         20.4
pip               20.1.1
pluggy            0.13.1
py                1.8.2
pycodestyle       2.6.0
pylint            2.5.3
pyparsing         2.4.7
pytest            5.4.3
setuptools        47.1.1
six               1.15.0
toml              0.10.1
wcwidth           0.2.4
wheel             0.34.2
wrapt             1.12.1

Problem
I am trying to debug my python tests in vscode and i get the following error in the debug console:
============================= test session starts ==============================
platform linux -- Python 3.8.3, pytest-5.4.3, py-1.8.2, pluggy-0.13.1
rootdir: /workspaces/test_python_debugging
collected 0 items / 1 error

==================================== ERRORS ====================================
_________________ ERROR collecting tst/dummy/test_unit_math.py _________________
ImportError while importing test module '/workspaces/test_python_debugging/tst/dummy/test_unit_math.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
tst/dummy/test_unit_math.py:5: in <module>
    from dummy.math import add
E   ModuleNotFoundError: No module named 'dummy'
--------------- generated xml file: /tmp/tmp-188wLObs4r6II9x.xml ---------------
=========================== short test summary info ============================
ERROR tst/dummy/test_unit_math.py
=============================== 1 error in 0.19s ===============================
ERROR: not found: /workspaces/test_python_debugging/tst/dummy/test_unit_math.py::test_addition
(no name '/workspaces/test_python_debugging/tst/dummy/test_unit_math.py::test_addition' in any of [<Module tst/dummy/test_unit_math.py>])

The tests themselves run fine in the terminal. In other terms there is no issue with not being able to find the "missing" module. Example:
0f617e2a72ee [/workspaces/test_python_debugging]$ python -m pytest tst/**/te
st*.py -k test_addition
=========================== test session starts ============================
platform linux -- Python 3.8.3, pytest-5.4.3, py-1.8.2, pluggy-0.13.1
rootdir: /workspaces/test_python_debugging
collected 1 item

tst/dummy/test_unit_math.py .                                        [100%]

============================ 1 passed in 0.01s =============================
0f617e2a72ee [/workspaces/test_python_debugging]$

The way I am launching the debugger is by clicking on the Debug Test link of the given test. Below is the .devcontainer.json configuration being used:
{
  "image": "test_python_debugging",
  "extensions": [
    "ms-python.python"
  ],
  "mounts": [
    "source=${localWorkspaceFolder},target=/app,type=bind,consistency=cached"
  ],
  "settings": {
    "terminal.integrated.shell.linux": "/bin/ash",
    "terminal.integrated.shellArgs.linux": [
      "-l"
    ],
    "python.testing.pytestArgs": [
      "tst",
      "-k test_unit"
    ],
    "python.testing.unittestEnabled": false,
    "python.testing.nosetestsEnabled": false,
    "python.testing.pytestEnabled": true,
    "python.linting.enabled": true,
    "python.linting.pylintEnabled": true,
    "python.pythonPath": "/usr/local/bin/python"
  },
  "containerUser": "app",
  "containerEnv": {
    "PYTHONPATH": "/app/src"
  }
}

I think that this is most likely a bug and would like to report it. Otherwise, can someone tell what I am missing in my .devcontainer.json?

If you open the Output pane, and select Python Test Log, you should see the exact command line it's running pytest with - this might help identify any missing options.
If there's nothing there that sticks out, can you share that log?

Hi @int19h,
Please find below the output of running pytest from the output tab ("Python Test Log"):
============================= test session starts ==============================
platform linux -- Python 3.8.3, pytest-5.4.3, py-1.9.0, pluggy-0.13.1
rootdir: /workspaces/test_python_debugging
collected 1 item

tst/dummy/test_unit_math.py .                                            [100%]

--------------- generated xml file: /tmp/tmp-148FgtzeERcJjkq.xml ---------------
============================== 1 passed in 0.01s ===============================

Thanks for looking into this.

Hm, so with your new repro (with the fresh container), it looks like it's working? Or are you still seeing the error in the other pane?

Yes the problem persists. I get the same error as in the description when i debug. (But, as you asked me to run pytest which i understood to be without debug so, just run debug, i get the output i mention in the previous comment). In short my problem is still here <- I cannot break and debug code.


Don't fire mesh removal events for ztunnel (#44762)

Please provide a description of this PR:
As mentioned in https://github.com/istio/istio/issues/44762, spurious eventdeletes are fired for ztunnels during namespace reconciliation - this is incorrect (ztunnel pods are never "removed from the mesh"), functionally harmless, but confusing, and causes ztunnel reconciliation to (potentially) trigger more often than it should, which isn't ideal.

@bleggett: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
19becd7eafafcd4e222f20bdac289ef99d82a120
link
true
/test unit-tests-arm64

lint_istio
19becd7eafafcd4e222f20bdac289ef99d82a120
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
19becd7eafafcd4e222f20bdac289ef99d82a120
link
true
/test unit-tests-arm64

lint_istio
19becd7eafafcd4e222f20bdac289ef99d82a120
link
true
/test lint

gencheck_istio
19becd7eafafcd4e222f20bdac289ef99d82a120
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
19becd7eafafcd4e222f20bdac289ef99d82a120
link
true
/test unit-tests-arm64

lint_istio
19becd7eafafcd4e222f20bdac289ef99d82a120
link
true
/test lint

gencheck_istio
19becd7eafafcd4e222f20bdac289ef99d82a120
link
true
/test gencheck

unit-tests_istio
19becd7eafafcd4e222f20bdac289ef99d82a120
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Engine Summary button not working

The "Engine Summary" button on the re-index view of the control-panel page does not work properly.  It leads to a broken link.

Fixed.  See v1.2.1


Open in Gitpod buttons

I'll let @natemoo-re make it look the way he wants, but I think I have hooked up all the logic and basic elements in place to make this functional.
Each card now also has a gitpod logo svg and a link that takes you to open that specific starter, launched in dev mode, with live preview open.

Woohoo—thanks for kicking this off! I might have a few design tweaks but overall this is looking great. Really excited to expand our astro.new support. ❤

I'd like this to eventually become 1 button + 1 dropdown, similar to be below. However with Nate's design tweets this is acceptable to launch as-is, so I won't block this PR with my crazy design ideas :)
If anyone reading this wants to implement this design, I'd love to help out how I can!
Stackblitz-supported Browser:
NAME
[Stackblitz] [▼ Open in...]

Other Browsers:
NAME
[Codesandbox] [▼ Open in...]


Not working properly

Not working properly  I used it for preventing screenshot and screen recording, Flag secure,  When I entered into the screen its worked fine but when I navigate another screen and then press back button in android it's not working - it's allows screenshot and screen recording. please let know me what can i do for this?

I don't understand what you are saying


Pad domain addition?

When I'm making scatter plots (and others), I find myself padding the scale's domain by the pixel radius of my circles to avoid them overlapping with the y-axis. I specify the number of pixels I want to increase on the left and right side of the domain, use the scale's inverse to add or subtract that amount from the domain and use that result to reset it.
I made a convenience function to do this and was curious if you would want a PR if this seems like useful functionality.
The api could be something like:
scaleLinear()
  .domain([0, 100])
  .range([0, 200])
  .padDomain([10, 10])
// new domain is [-5, 105]

[edited for better order of operations in the API example]

In case it is helpful, we implemented this functionality in Vega for various scale types (symlog coming soon):

Here is the domain padding method: https://github.com/vega/vega/blob/master/packages/vega-encode/src/Scale.js#L140
Using these transform utilities: https://github.com/vega/vega/blob/master/packages/vega-util/src/transform.js

I like this idea—I needed something similar in this recent scatterplot. I ended up doing a little transformation of the domain:
function padLinear([x0, x1], k) {
  const dx = (x1 - x0) * k / 2;
  return [x0 - dx, x1 + dx];
}

y = d3.scaleLinear()
    .domain(padLinear(d3.extent(data.flatMap(d => [d.R90_10_1980, d.R90_10_2015])), 0.1))
    .rangeRound([height - margin.bottom, margin.top])

I can see it being nice to specify the padding in pixel space rather than as a proportion, though.

I'll take a crack at this as a PR for d3-scale using the Vega setup as a jumping off point. It will need to be modified to pad the min and the max separately.
I think this would be good to include as a part of d3-scale instead of building into outside libraries since there's not a simple way to distinguish between all the different types of scales and what transform function you'd need to use.

+1! adding a padding in pixels to scales could be very useful.
Any update on this feature?
Is that something that will be added to d3 scales in v5?

IMO it might be nice to just expose that padLinear function by @mbostock as part of d3-scale.
This one:
function padLinear([x0, x1], k) {
  const dx = (x1 - x0) * k / 2;
  return [x0 - dx, x1 + dx];
}

It's nice that it is isolated, not required to be a method on scales.
For the pixels case, k can be computed externally as pixels / width, so maybe no need to add any pixel-specific API.
This would set a precedent and open the door for subtle implementations for other scale types, like padLog, padPow. I wonder, though, if these would require access to the scale's range as well... Perhaps not?

I started a PR for padLinear: https://github.com/d3/d3-scale/pull/213
Please let me know if this looks like a good direction. With some encouragement I'm happy to flesh out the PR with README updates - but first I want to confirm this direction is acceptable. Thanks for reviewing!

Related for map projections: https://github.com/d3/d3-geo/issues/162

My original implementation was a bit off but this is now properly implemented in LayerCake with the Vega transform helpers so that it works on log and power scales, too. You supply the padding you want in pixels for either the min or the max.
I most likely won't have the bandwidth to do a d3-scale PR for this but someone should feel free to if they want.

Very cool! If anyone wants to branch from my PR #213 and add the implementation for log and power scales, go for it!

Stepping back a bit, could the same fundamental problem be solved instead by drawing the axes differently? For example, offsetting the tick labels, and drawing grid/domain lines such that they are slightly removed from the points that fall exactly on the min/max of the domain? Maybe with a technique like that this whole problem could be sidestepped.

Can someone shed light on the math of adding padding to a logarithmic scale? 😕
Let's say I have a log-scale Y-axis, with domain [d1, d2], and the total height in pixels is 1000. How can I adjust the domain values to add 20px of padding to the top and bottom of the axis?


Add README instructions on how to add a new entry

Copying relevant instructions from slack here, so that internal and external contributors can make suggestions (reducing our manual work).

Merging as is to have some instructions in the readme.
We can re-iterate if needed, but this is better than no instructions and people not knowing what to do :)


Address comments from PR #5251

Pre-submit checklist:

Branch

[x] Tests are provided (if possible)
[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Changelog fragments have been written (if appropriate)
[x] Relevant tickets are mentioned in commit messages
[x] Formatting, PNG optimization, etc. are updated

PR

[ ] (For external contributions) Corresponding issue exists and is linked in the description
[x] Targeting master unless this is a cherry-pick backport
[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

Unclear if you want a review since this is still marked draft?

Sorry for the confusion. Yes, it's ready for review.


[flutter_tools] disable failing gen l10n test

Description
This test started failing on an unrelated PR. skip until it can be investigated.
TBR @shihaohong

LGTM. Will take a look!


Add: Progress-bar/This is a cascading progress bar component based on react

https://devarchy.com/react/progress (react-yan-progress)
github: https://github.com/Yangfan2016/react-yan-progress
npm: https://www.npmjs.com/package/react-yan-progress

Please PR and we will review.


Register hackerteam.is-a-fullstack.dev

Added hackerteam.is-a-fullstack.dev using the CLI.

For valid record, value 1
On Fri, Feb 2, 2024, 9:01 AM William Harrison @.***>
wrote:

@.**** requested changes on this pull request.
In domains/hackerteam.is-a-fullstack.dev.json
https://github.com/free-domains/register/pull/406#discussion_r1475635011
:

@@ -0,0 +1,14 @@
+{

"domain": "is-a-fullstack.dev",
"subdomain": "hackerteam",

"owner": {

   "email"<EMAIL_ADDRESS>

},

"records": {

   "AAAA": [""]

Please use a valid record.
—
Reply to this email directly, view it on GitHub
https://github.com/free-domains/register/pull/406#pullrequestreview-1858502956,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFY4RUXRTBUGO7QRRKSSLTTYRSFNLAVCNFSM6AAAAABCUPWB3OVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQNJYGUYDEOJVGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


"46 times faster" Link in Readme is broken

This does not exist anymore: https://gist.github.com/substack/2790507#gistcomment-3099862
Please update the link, or provide a comparison in the Readme directly without a link indirection.

Here's the content: https://web.archive.org/web/20211006063112/https://gist.github.com/substack/2790507
However, due to #76, and the fact that performance isn't that important compared to correctness, which this package provides for universal code, I'll rework that section of the readme instead.


Module tags for Google (they use the word "labels")

Describe the bug
Looks like yor will not populate labels for GCP modules with "labels" for example:
module "project" {
  source = "git@github.com:my-org/terraform-google-project?ref=v2.1.3"

  prefix            = var.app_abbrev
  system            = var.system
  env               = var.env
  random_project_id = var.random_project_id
  labels = {
  }
}

Changing labels to tags populates fine. For consistency I'd like to see tags = labels with modules in Google.
Thanks!

Was fixed in #164


Git LFS and GitHub Pages

GitHub Pages "doesn't support" Git LFS.[^1][^2]

Because it always first tar all the files and upload artifact[^3], then download artifact and deploy site[^4].
At the tar step, there should be no symlinks nor hard links[^3].

But you don't have to give up using LFS. The key is to pull LFS, by (verified by branch checkout-lfs).

either use actions/checkout with input lfs: true (default is false)[^5] to checkout all
or run git lfs pull --include=<paths> to checkout selectively[^6].

This works if the GitHub Pages is published by

a custom workflow[^7], or
GitHub Pages bot, after the sources are prepared by previous workflow (hence one can config it to checkout LFS). Note the bot always run a jekyll build[^2].

Remaining cases[^8] can be solved by adding a custom workflow to checkout and push to gh-pages branch, hence reduce to case 2.
[^2]: A typical GitHub Pages workflow run triggered by pushing to gh-pages branch: https://github.com/muzimuzhi/arara/actions/runs/3621158739
[^3]: actions/upload-pages-artifact https://github.com/actions/upload-pages-artifact
[^4]: actions/deploy-pages https://github.com/actions/deploy-pages
[^5]: actions/checkout https://github.com/actions/checkout
[^6]: git lfs pull doc https://github.com/git-lfs/git-lfs/blob/main/docs/man/git-lfs-pull.adoc
[^7]: https://docs.github.com/en/pages/getting-started-with-github-pages/configuring-a-publishing-source-for-your-github-pages-site#publishing-with-a-custom-github-actions-workflow
[^8]: For the specific workflow run in [^2], it's the default lfs: false input of checkout step in job build to blame.

I have a question: Will this always download ALL of the data stored in LFS and push ALL of it to pages, even though part of it is already there?

@SubNerd I think the main barrier is, currently the deployment step executed by actions/deploy-pages only accepts an artifact as source of Pages files. Thus an extra pair of uploading and downloading (not sure) steps is not skippable.
If one continues to use actions/deploy-pages, the only optimization is to only fetch LFS for latest git ref, and only pack needed files in an artifact.
If you're equipped with Javascript and Web knowledge, maybe it's possible to learn from source code of actions/deploy-pages to implement your own deploy-pages action that can use files in hand to deploy, not from an artifact.

see https://github.com/orgs/community/discussions/30113.

Will do.
Thanks for the reply!


perlPackages.MojoSQLite: 3.002 -> 3.003

Motivation for this change
Updates the Mojo::SQLite Perl module to 3.003
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @volth

@GrahamcOfBorg build perl528Packages.MojoSQLite perl530Packages.MojoSQLite


travis ci build - upgrade pypy version to be 2.9 or greater

Our travis ci build is failing on the two pypy settings because the default version of pypy for trusty is too old - it's 2.8 and needs to be at least 2.9.x in order to install pandas correctly. See https://travis-ci.org/intermine/intermine-ws-python/jobs/580760283 for an example of the failure.
This is a link of different python and pypy versions: https://github.com/travis-ci/docs-travis-ci-com/pull/1249#issuecomment-307741238
To test this, you could enable travis ci for your fork of this repo, or create a WIP (work in progress) PR - either is fine.
if you want to pick up this task

Take a look at the InterMine contributing guidelines
Comment on this issue stating that you intend to work on the task
When you're ready, add your work to the repo and create a pull request.

What to do if you need help
Mention @yochannah, tweet @yoyehudi, pop by to say hi on chat or if needed email<EMAIL_ADDRESS>Don't forget we're usually only available during uk office hours and will not be able to respond at other times :)

quick additional note - if we can't get pypy 2.7 working it is probably not the end of the world, as 2.7 support is discontinued from the end of this year.

@yochannah Can't we go for xenial linux distri. on travis?

@KVGarg that's another option, but then it requires a nontrivial script upgrade as the current script expects Java 8 (which isn't easy to install on xenial)

for now I have removed pypy and I'll create an issue to add it back when we can upgrade our build machine distro. This is dependent on https://github.com/intermine/intermine/issues/2127


Extension Not Working with Signed in account

Hi
In my chrome extension of R to L Notion
R to L feature, while I'm signed in Not Working!!! ... But in Signed Out States (Like incognito) its fin

hi, this bug report is in the wrong place & doesn't contain enough information for me to help you.
please read the contributing guidelines and then try again: https://notion-enhancer.github.io/about/contributing/


Raytrace example won't compile (wasm32::atomic_notify)

I'm trying to run the raytrace example and I'm hitting a compile error.
Compiling wasm-bindgen-futures v0.4.17
error[E0425]: cannot find function atomic_notify in module core::arch::wasm32
  --> C:\Users\neni\.cargo\registry\src\github.com-1ecc6299db9ec823\wasm-bindgen-futures-0.4.17\src\task\multithread.rs:39:33
   |
39 |             core::arch::wasm32::atomic_notify(
   |                                 ^^^^^^^^^^^^^ not found in core::arch::wasm32

error: aborting due to previous error

For more information about this error, try rustc --explain E0425.
error: could not compile wasm-bindgen-futures

Steps to reproduce:

Checkout the raytrace example (https://github.com/rustwasm/wasm-bindgen/tree/master/examples/raytrace-parallel)
Run ./build.sh

I'm using the Rust nighlty (nightly-x86_64-pc-windows-msvc unchanged - rustc 1.47.0-nightly (81dc88f88 2020-08-13)) and I have rust-src component added. Am I missing some steps maybe? Any help is appreciated.
Thanks,
Nemanja

Thanks for this! This is because the intrinsic has been renamed to memory_atomic_notify in nightly. I'll work on getting this fixed next I'm back at a computer.


Colors?

Is there a way to change the NormalTextColor? I saw a few modpacks do it, but now sure how to get the colors.

Did you look through the properties of a label in the pages?

What does that even mean? I'm sorry I don't understand.

CurseForge Pages

No I didn't, is there a way to find TextColor?

All properties a label (text) can have are listed on the Label page  changing color will change the text color. Use an rgb to int converter like this to get the rgb integer from your color.

Ahh, thank you


New Emulator updates

Fix emulator message filtering
Add tests for Path variables
Fix event query for newly created account
Add ability to automatically resolve imports, when deploying contracts
Add automatic resolver for contract name

@MaxStalker test plan is missing
can we consider having separate PRs if changes are not highly related to each other? benefits:

clear change history tracking
making testing easier
make reviewers's lives easier
(and ignore this comment if all listed changes are highly related)

@kerrywei Good call! I will prepare separate issues and PRs now.