full rebuild if a class contains a constant

org.gradle.api.internal.tasks.compile.incremental.JavaChangeProcessor is full rebuild if the class contains a constant, because
       DependentsSet actualDependents = this.previousCompilation.getDependents(className, Collections.emptySet());
the new constant is Collections.emptySet(), so the new previous constant is compare to the empty, it always true.
Expected Behavior
if the class constant is not changed, it will not full rebuild.
Current Behavior
the class rebuild if the class contains a constant.

Thanks for reporting, but I'm not sure I understand your question. Are you saying if a class contains a constant field, e.g.
class A{
    public static String CONSTANT="";
}

the class will always be compiled in every build?
What's your environment (OS, JDK, Gradle version etc.)? It would be great if you could provide us a code snippet to reproduce this issue.

yes,
class A{
    public static String CONSTANT="";

    public void a() {
        .... <<<<change something
    }
}

first, I make my java compilation incremental through options.incremental = true.
then If I change the method body code of a class contains a constant field, it will always full rebuild the entire source.
My JDK is 1.8.
I debug the gradle source code JavaChangeProcessor#line39
previousCompilation.getDependents(className, Collections.<Integer>emptySet())  the second paramter is Collections.<Integer>emptySet() which is the current class constant value, but it does not analysis the current source file to fill the Collection. It always is empty. when it compare to the previous constant value, actualDependents.isDependencyToAll() is always true.
        if (actualDependents.isDependencyToAll()) { <<< always true
            spec.setFullRebuildCause(actualDependents.getDescription(), input.getFile()); <<<
            return;
        }

first compile code
package com.androidapi23;

import android.nfc.Tag;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;

public class MainActivity extends AppCompatActivity {
public static final String Tag = "mainActivity";
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
    }
}

import static com.androidapi23.MainActivity.Tag;

/**
 * Created by chenhjohn on 2017/8/22.
 */

public class MainTest {
    public void a() {
        System.out.println(Tag);
    }
}

second compile

package com.androidapi23;
import android.nfc.Tag;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
public class MainActivity extends AppCompatActivity {
public static final String Tag = "mainActivity";
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
system.out.println("hello") <<< add code this.
}
}

run gradle build with --info. it will output the log

Full recompilation is required because 'MainActivity.java' was changed. Analysis took 0.014 secs.

I'm sorry, perhaps I misunderstand you, but if you add that line into MainActivity.java, it is definitely changed... what's the relationship between this change and the constant field? I mean, it's recompiled because you add the line system.out.println("hello"), and it has nothing to do with the constant field.

This is expected, because we don't analyze the source code, so we don't know that the constant didn't change. We only have finer grained detection for bytecode changes on the classpath.
Possible workarounds:

Make that constant private and expose a static method instead
Put it in a class that changes less often

Cool, I think I misunderstand the incremental thing. Thanks for clarifying.

This is a special case for constants. Everything else can be analysed properly through the bytecode. But constant refernces aren't visible in the byte code. So we stay on the safe side and just assume that every other class uses that constant.

@jiqimaogou , I think oehme's explanation is pretty clear, so I'm to close this issue. Please feel free to reopen it if you still have questions.

Thanks for replay. For a large project, recompilation of javac cost more than 30s with instant run build, this due to two reason:
1.annotation; It will be solved at 4.6 #1320
2.constant; Hundreds of people work in the project, sometime many of the classes will have constant fields. Developer may confused why building need so much time event they actually didn't change any constants.

sometime many of the classes will have constant fields.

Note that this is only about public constants. In most cases you can avoid this by using static methods instead:
Replace this: public static final int MAGIC = 23
With this: public static int magic() { return 23; }

https://blog.gradle.org/incremental-compiler-avoidance
In the above blog post, it is explained: In Gradle 3.4, we made that behavior much smarter, and only recompile classes which could potentially be affected by the change. In other words, if the class is changed, but the constant is not, we don’t need to recompile.
Is it still the situation in Gradle 6.1? Our project has a class with public constants, and each time the class is changed, a full rebuild is executed (the constants does NOT change). If I change the constants to be private, incremental compilation works fine.

https://docs.gradle.org/current/userguide/java_plugin.html#sec:incremental_compile implies it must have been reverted or regressed since https://blog.gradle.org/incremental-compiler-avoidance was published. It would be nice to know if this is true and why it changed back.

Is it still the situation in Gradle 6.1? Our project has a class with public constants, and each time the class is changed, a full rebuild is executed (the constants do NOT change). If I change the constants to be private, incremental compilation works fine.

Same problem here, gradle 6.8-rc4

Same problem with Gradle 7.0.2.  Blank line triggers full rebuild even if the constant is not changed.

I haven't tested it yet, but I think this has been fixed in the just released 7.1-RC-1 . See https://github.com/gradle/gradle/pull/16495


Attempt to unlock mutex that was not locked

When running blockify installed from AUR on Arch Linux I encounter the following error before the process ends:
Attempt to unlock mutex that was not locked
Apparently the problem is triggered by the upgrade from glib2 and can be resolved by removing the line gtk.gdk.threads_init() from some packages but I have been unable to find it in any of the files from this package.

i'm using arch, too and i cannot reproduce this.
However, the code you're looking for is here:
https://github.com/mikar/blockify/blob/master/blockify/blockify.py#L123

Thank you for your prompt response Mikar.
I commented out the line you pointed to in both blockify.py and blockify-ui.py.
Both are subsequently launching perfectly.

I've tested the software and it's actually blocking ads unlike some other scripts I tried earlier.
Brilliant! Thank you for making this mikar, really great work :)


Add initial image support

Fixes #12 to add support for specifying an initial image when gallery component is mounted

@codematix I'm curious why this is needed? You can just change the initial image by reordering the items prop.

@linxtion I had a use case for image gallery where a page was showing thumbnail images of products. The use case required that when the user clicks on any of the thumbnails on the page, a modal is displayed with a gallery which displays a zoomed-in version of the product image and also provide arrows to scroll.
Now a product could have more than one image. Therefore, if the user clicks on the 3rd thumbnail on the page, I open the modal with the image gallery with the full version of the 3rd image and then allow scrolling to the remaining image.
If I re-order my items prop array, I will have to deal with affecting the model object shared by both the parent page and the image gallery component. Hence I felt it would be simpler to modify the image gallery component to give it an initialImage prop.

@codematix fair enough, I left some comments in the code if you don't mind adjusting I can merge. Thanks.

@linxtion the review comments are implemented. Please consider merging. Thanks!


upgrade dataprocessing python version to 3.11.5

Before release the v0.2.0 ,we need to use same python version in our python projects

data-processign
ragas_once

@ggservice007 @wangxinbiao

#699

@ggservice007 any progress?


Child Collections

A few people have asked now about the ability to have child collections, ie a collection underneath another collection so thought I'd start a post to discuss it.
In UI-O-Matic, this was possible by having a "hidden" collections, or collections that don't appear in the tree, and then they had a "Listview" property editor that you can add to your editor. This kinda worked, but did get a little messy as you could ultimately nest as deeply as you wanted to go, and the system tried to keep track of the history of where you came from and make sure you navigated back to the specific tab when you exited the child collection editor.
It might be that we look to inject a list view dynamically, rather than requiring the use of a dummy property, and simply infer "hidden" collections due to the fact they appear below an existing collection.
When looking into this though we do need to take care with paths, as I think there are areas that infer context based on the structure of the path / id.
Lastly, we need to think about the actual relationship and how we tie these two collections together, potentially requiring child collections to be linked to the parent by a property and they must provide a lambda to request that property as part of the config?
Anywho, this likely doesn't make a lot of sense at the moment as it's more of a brain dump in order to raise the issue and get peoples input. If you do have some suggests, please comment, however please do refrain from just saying "me too". I'm looking for implementation suggestions.

@mattbrailsford I think I get what is meant here. Although I might be slightly off.
We're running into a similar situation where we have an collection of machines which then has an error history powered by another table. Visually they're nested in a property stub for now but DB wise they're in separate tables linked by a FK.
I've just added in a blank "ErrorHistory" property and created a viewer data type which calls our API to populate the view via an API call. This works for us because the data is view only but I could imagine with CRUD actions this wouldn't work as well.
If Fluidity were to support adding in List Views like this out of the box for child collections that might make our life easier because then we can leverage the power of list views (and by extension custom list views!)
As I'm typing this out I'm liking it more and more! I'm happy to talk over any complexities or solutions if you need an ear. :)

@mattbrailsford I think that the implementation should either be something like a list view on a tab to allow for adding, navigating to, and deleting children, or be something like nested content for more simple children. In that case, you can edit a child directly from the parents page, without having to navigate to and from a child page.
With the list view option, it would be possible to implement the "infinite editing" they plan to include in Umbraco 8. From the list view you can open a new panel which allows you to edit the child, and quickly return to the parent.

@ThomasBleijendaal I've been thinking about this recently myself and the listview is the way I was thinking too along with v8 infinate editing. It sounds like it could really hit the spot, but might mean waiting for v8.

@mattbrailsford List views are probably the best way of handling child collections. But I don't think we have to wait for v8. If you look at how they demonstrate infinite editing, what really changes is that instead of navigating to a new page and lose context, a new pane appears when someone clicks on "open".
The list view for a child collection will be similar in v7 or v8: it must be possible to configure such child list view from the bootstrap, have a way of pre-filling some values when creating a new child (like the ID of the parent in some FK), or be able to delete children from the list view. The "only" thing that really changes is how to navigate to the child entity.

@ThomasBleijendaal sure, but the biggest issue is we'd have to code a custom dialog experience as there is nothing in v7 atm that opens a wide panel like shown in v8 and it's also whether there are any special styling rules introduced to make dialog content show in the regular layout (right now it has a dialog mode which stacks the labels / fields).
It's worth looking into, but I'd hate to spend a load of time coding a custom solution for it to be redundant within a year.

@mattbrailsford but should we build such a wide panel in v7? Currently, the open button on a fluidity entity picker navigates to the corresponding fluidity page, just like everything else in v7. This new child collection can do the same, which will be less then ideal, but its similar to how Umbraco v7 currently works.
When v8 becomes available, infinite editing support can be build in to this child collection, and be only supported on v8. It would be way too much work to try to get this into v7. But I think having a start with child collections in v7 without infinite editing / custom dialogs would be better than having no child collections at all.

@ThomasBleijendaal sure, we can do navigating to an editor, this just brings in some funky things to deal with too. 1) Keeping track of history so you can go back the way you came + breadrcrumb and 2) when you do notivate back, you'll probably want to get back to the same editor tabe the listview was on.
We managed this in ui-o-matic, but I reallyw asn't happy with the way we had to do it. I believe we had to pass in a path of ID's on the querystring and if it was a collection of ID's render the editor differently / parse the IDs differently. And for navigating back to the right tab, we had to use some aweful JS to wait for the editor to load, then jump them to the tab which wasn't very smooth.
If we do look at it sooner rather than later though, I'd like to come up with some better ways than this.
On a related note, I believe when you have a list view you can now choose which tab position you want it to appear. I'm not sure if this means some code exists to jump you to a tab when you open the editor, or whether editing a child node and going back to parent just sends you to the first tab regardless. Either way, something worth looking into to see if something does exist.


Can we get the type of variable written in the script editor?

monaco-editor version: 0.18.0
Browser: : Chrome
OS: : Windows
Steps or JS usage snippet reproducing the issue:
Hi team. Always appreciate for your help.
I should know the type of written word or variable in script editor.
In the monaco Editor, I can know the type using mouse hover Event.

For Example Above picture, I will Get local(Type) from aaa (Variable)
through hovering the mouse specific word.
but Can i get the type of word or variable written in monaco-editor??(not using mouse hover)
I want to know some API or Access Way.
Thx

That information is provided by the typescript smarts (via the HoverProvider API). We cannot expose that information and you would need to use the typescript smarts yourself.


Songs playing simultaneously

great work i loved it. but i found an issue (im testing it in firefox btw) when i click Play Sample File let the music play (not untill the end) and add another song with the input button, they both play simultaneously. So i added another button to stop all songs using a function: function stopallaudio(){ sourceNode.stop(); } just for testing of course. ill try again with a play/pause button. im still a noob in .js

Alright, for now I simply disabled the buttons while the music is playing. I'll further look into this some time later. These buttons would still need a redesign anyway.
Best regards,
Dominik


No rename

[x] tests and linter show no problems (npm t)
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/metawatch/commit/a59cc9a3841ddfa5e17af9bfbda7bd974a385bc9 and https://github.com/metarhia/metawatch/commit/2ef58e6e58bbe51978d8c820d859802d973b107f


[BUG] - Unable to attach or mount volumes on GCP ocassionally

OS system and architecture in which you are running QHub
linux
Expected behavior
QHub should by default be able to spawn a user server based on qhub configmaps and user settings.
Actual behavior
The user spawning process is actually timing out because of the following error
2022-03-09T22:29:19Z [Warning] Unable to attach or mount volumes: unmounted volumes=[conda-store], unattached volumes=[server-idle-culling home conda-store dask-etc]: timed out waiting for the condition

We were able to find a workaround that consists of launching a dashboard then re-launching the user pod. As it seems that only the first deployment fails with the mount issue. Another way to fix seems to condone the current nodes (for the user and general) and wait for the next user spawn to scale a new node.
How to Reproduce the problem?
we still don`t have a good way to reproduce the behavior, the error seems to have uncertainty attached to it. Future situations where this error shows up might help to deduce the reproducibility circumstances.
Command output
No response
Versions and dependencies used.
pip install qhub from main, and docker images set to sha-3470504.
Compute environment
GCP
Integrations
No response
Anything else?
No response

For now, due to some uncertainty of the problem. We will add some docs on the user FAQ to address this issue and add a workaround for that.

A short term work around we found (even though we weren't able to reproduce this issue on other clusters) was to launch a dashboard and then launch the user jupyterlab. This ensured that the user wasn't the first to launch.

This seems to be probably related to a Kubernetes version update in the cloud providers side, as a recent issue encountered the same problem with qhub-3.14. #1163

Ran into this issue again this time when dask-gateway pod was trying to start.
   Normal   Scheduled    2m20s  default-scheduler  Successfully assigned dev/qhub-daskgateway-gateway-7b98cb65d-6w4z4 to gke-quansightbeta-dev-general-8bd8c8ae-hslb                       │
│   Warning  FailedMount  18s    kubelet            Unable to attach or mount volumes: unmounted volumes=[conda-store], unattached volumes=[configmap secret conda-store kube-api-access-dh │
│ 4r[]: timed out waiting for the condition

I ended up restarting the general node (vm) that the pods were running on. While this is not a great fix it did get around the issue.

We are going to leave this issue open though we do not have any great ways to debug this issue. I have found that restarting the node that has the issue solves the problem for at least a day.
In order to properly solve this we will need to monitor the kubernetes node and see logs on if/when the mount fails and how this is happening.
Help would be appreciated on this issue!

This issue happened to me recently. When I couldn't launch my jlab environment, I checked k9s and somehow the qhub-conda-store-minio pod was in a bad state. I can't confirm that they are related but how I got around this issues suggests that it might.
After unsuccessfully debugging the issue, I ended up deleting the helm chart for minio and redeploying qhub altogether. This fixed whatever was causing the qhub-conda-store-minio deployment to fail.  And I no longer received volume mount errors, and launched jlab successfully.
Besides conda-store trying to build an environment, I'm not sure what was tripping minio up.

@costrouc @iameskild this error doesn't seem to show up anymore,  has anyone spotted this recently?

This happened again this week. Not sure what caused but the conda-store mounts were affecting the dask-gateway pod spawning this time. I followed the fix suggested by @costrouc of resetting the node, which worked, but we will need to check this more deeply once the migration ends.
I downloaded some logs from the node, wich had some error messages showing on. I will look at then later this week to see if I find anything useful.

For now, this will need to be added to the FAQ of Nebari with instructions on how to reset the general node

I ran into this issue again today and although I spent awhile trying to understand what the root cause of the issue might be, it still alludes me...
The only error message I've seen is the following:
MESSAGE: "I0209 21:45:13.260321    1785 kubelet_pods.go:891] "Unable to retrieve pull secret, the image pull may not succeed." pod="dev/proxy-7795cd8fd8-xkbwq" secret="" err="secret \"extcrcreds\" not found""

This may or may not be relevant but at this point, I'm grasping at straws trying to understand what is going on.

I ran into this issue today on AWS and I was able to capture an error message from the general node that might be slightly more helpful:
Warning	SystemOOM	10 minutes	kubelet	System OOM encountered, victim process: conda-store-wor, pid: 25039

Each of the times I've encountered this issue, it's always the conda-store-related PV that has trouble being mounted. The above error message seems to be pointing to a conda-store-worker process running on the general node that is causing it trouble.


invite people to this organization

There are already 4 failed invitations. Not sure how to handle that.

add me please

Please add me to the group

Would you please add me at the group?


how to set activity bar size？how to export config file

how set activity bar size? how to export config file, I think this function is more important, when i use  a new computer to work.
I useing webstorm.   export config file is cool.

Check out settings sync: https://code.visualstudio.com/docs/editor/settings-sync


Real 3d

This fake 3d approach is really good for simple stuff . I have seen it extended to allow google earth style apps.
A few weeks ago I was looking into if I could embed a real 3d system into Fyne or gio .
filament looks like a good candidate . It’s designed for speed and physical lighting. You can use it for wireframe or hidden wireframes too.
The repo produces binaries that can be used on web, desktop and mobile from a golang app.
not sure if your interested in this though

Thanks a lot for the suggestion. This is however just a bit of a showcase for https://github.com/Jacalz/linalg and how math can be used for this line drawing (especially within Fyne). The idea was based on an exercise that I did for my linear algebra course, so it is more of a proof of concept rather than serving an actual need.


Add language tag to Logo in SAML metadata

The class MetadataConfiguration.UIInfoConfig.UIInfoLogo should be extended with a lang-tag so that we can publish logotypes for different languages (required by Swamid).

Fixed in PR https://github.com/swedenconnect/signservice/pull/135.


Do you support oss ?

$ aliyun oss
ERROR: 'oss' is not a valid command or product. See `aliyun help`.

did you mean:
  ess

oss is supported in 0.70 version,
please try with aliyun oss ...

$ aliyun help | grep -i oss
$ aliyun version
3.0.69

But still missing in aliyun help even in 3.0.69.


Code generation features required to GEMM on Hopper GPU

I would like to expand on this project by breaking it down into clear and actionable steps.
Feature 1: Implementing wgmma.mma_async
To leverage the asynchronous nature of wgmma.mma_async, this warp-level gemm instruction must be grouped, committed, and waited upon. It provides the ability to perform larger gemm operations such as 64x128x16. It differs from mma_sync, which is a synchronous warp-level gemm instruction. wgmma.mma_async can read from shared memory. This step marks the initial progress towards targeting new generation tensor cores.
See here for more details about the instruction
Feature 2: Swizzling modes of wgmma.mma_async
wgmma.mma_async supports four swizzling modes, including the option for no swizzling. It is essential for performance support for all these swizzling modes.
See here to see 3 swizzling modes
Feature 3: Supporting TMA
TMA (tensor memory accelerator) is a memory engine that enables asynchronous copying of a layout (tile) from global memory to shared memory. This operation significantly saves registers. In this step, our focus will be on supporting TMA in the iree+mlir framework and establishing its connection with wgmma.mma_async.
IREE host needs to build TMA descriptor and send it to the kernel. In the kernel, a single thread could submit TMA request.
Supporting split/arrive: TMA requires split/arrive barriers (aka mbarriers). We don't have it in mlir. See for examples
Feature 4: CTA Clustering
CTA Clustering is a notable feature of the H100 architecture, allowing the clustering of Cooperative Thread Arrays (CTAs) and access to their shared memory. This step is crucial to enable warp specialization. See for examples
Feature 5: Warp specialization
Splitting consumer/producer warps. Producer fetches data from TMA, consumer does GEMM. See for examples
There is multiple ways of doing codegen here. CUTLASS has three warp specialization, they require different codegen.
Feature 7: Optimized epilogue via TMA
By leveraging TMA, we can optimize the epilogue phase of the computation. This step focuses on enhancing the performance of the epilogue through the effective utilization of TMA.
### Tasks

CUTLASS has 5 GEMM flavors for H100
| Required Features
-- | --
A100 Multistage GEMM |
KernelMultistage | 1
KernelTma | 1,3
KernelTmaWarpSpecialized | 1,3,4,5
KernelTmaWarpSpecializedCooperative | 1,3,4,5, 7 (Persistent threads, different codegen)
KernelTmaWarpSpecializedPingpong | 1,3,4,5, 7 (different codegen)
Feature 2 is required for every step to avoid bank conflicts. We need it for every GEMM flavor.
IREE Codegen
KernelTma will perhaps give decent performance for limited shapes. This can be a good start for implementing codegen.
KernelMultistage won’t give performance, it doesn't leverage TMA and CTA clusters. However, it  can be also a good start for codegen since it requires only feature-1.
To cover all shapes, we need KernelTmaWarpSpecialized, KernelTmaWarpSpecializedCooperative and KernelTmaWarpSpecializedPingpong.

Created #14126 to track Feature 3: Supporting TMA


fix state_tests filled env section format hex

added this Transaction().with_signature_and_sender_and_private_key()
which is the same but adds a private_key to the structure
        secret_key = self.secret_key
        tx = self.with_signature_and_sender()
        tx.secret_key_test = secret_key

this one is called to construct state test transactions

No changelog/summary?

No changelog/summary?

You're right, sorry about this, I'll be more careful when merging in the future :raised_hands:


Add ability to override testify properties from the command line

When running tests against alternate configurations, properties files need to be updated accordingly so that a test can be run. Having a method that allowed for properties overrides from the command line would make testing alternative systems easier

As part of designing the new cli a properties option has been added that should allow easy property input. The first approach will be to simply have the cli properties added in after the properties read from files. This will effectively allow them to "overrride" file based properties


Understanding Flakes

I'm trying to migrate to the new Flake world. The problem is, this template is so complicated it's almost impossible understand how it actually works.
So I started to build my configuration from scratch: https://gist.github.com/suhr/4bb1f8434d0622588b23f9fe13e79973
But there's no documentation for the new stuff. So here I will ask questions about how to do things using Flakes.

Is there a way to get the home-manager command back?
I tried to do the following:
{
  inputs = {
    nixpkgs.url = "github:NixOS/nixpkgs-channels/nixos-unstable";
    nixpkgs-stable = {
      url = "github:NixOS/nixpkgs/nixos-20.03";
      flake = false;
    };
    home.url = github:rycee/home-manager/bqv-flakes;
    nur.url = github:nix-community/NUR;
  };

  outputs = { nixpkgs, nix, self, ... }@inputs: {
    nixosConfigurations.nixos = nixpkgs.lib.nixosSystem {
      system = "x86_64-linux";
      modules = [
        (import ./configuration.nix)
        inputs.home.nixosModules.home-manager
      ];
      specialArgs = { inherit inputs; };
    };
    legacyPackages.x86_64-linux = self.nixosConfigurations.nixos.pkgs;
  };
}

Then I added home-manager.users.suhr.programs.home-manager.enable = true; to the system configuration.
nixos-rebuild switch ran without problems. But there's no home-manager at the user side.

nixos-rebuild switch ran without problems. But there's no home-manager at the user side.

It seems that this is intended behaviour from home-manager. See the setting for home-manager.users.<name?>.programs.home-manager at <home-manager/modules/programs/home-manager.nix>. This checks not only that home-manager is enabled but also that submoduleSupport is disabled (and in this case it's not). This seems to come from the first ever commit with module support (rycee/home-manager@1bc59f729047886b845ffd9162d40593fad5c7f0).
As far as I understand (and take this with a grain of salt), you should probably just manage the home-manager generations as part of the whole system generations, and therefore there should be no need to use the home-manager CLI tool.

Yeah, I figured out that home-manager does not support cli with flakes. I use a flake to manage user packages now. It would be nice to have something to manage some configuration as well though.

I'm going to close this as it really isn't related to the project. However, I am hoping to update the documentation to be a bit more thorough in the near future.


fn-fix_mdm_metadata_edit_error

Description

fix edit name and description not working if type is stagedb or database
Related Issue :

METATRON-3022
How Has This Been Tested?

1.Management-> Explore data -> Metadata
2. click list item
3. check edit name and description is working whatever metadata data type is
Need additional checks?
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

run build
deploy to 6

run build
deploy to 2

@thinkcsww
The height and width of the textarea do not match the right button.

@ufocsw
2. what text should be displayed when there is no description value ? No description?
I check planning book but there is no guide.

@thinkcsww
If description is undefined, the same message as for empty value is displayed.
"No Description"

@ufoscw
Sorry, i misunderstood
Modified

run build
deploy to 2

@thinkcsww
Please check Korean resource bundle

@minhyun2
Modified


Create a BasicMathNode that can evaluate expressions given as a string

Creates a new BasicMathNode that will evaluate basic math functions given as a string.  Also translates math functions into the correct backend (math, numpy, or jax).
This will allow us to write functions nodes (for basic math) as:
x0_node = BasicMathNode(
    "pow(10.0, -0.4 * (distmod - alpha * x1 + beta * c + m_abs - 10.635))",
    distmod = host.distmod,
    alpha=...
    ...
)

Thanks @hombit, I filed some issues for some of those:

I need to think about how to provide a string representation, because I currently use __str__ for some of the node labeling.
Added issue #186
Added issue #187
Agreed. I don't have a great solution here, but I can try adding better debugging (#188)
5a) I don't think jit compiling is possible with this node, so we would need to be careful.
5b) You can use a random variable as input the same way as within any other node.


Figure out why npm fails on windows

What the issue is about
Currently it is not possible to get going with the codelab on windows using npm (yarn works).
See analysis by @jacekk:
https://github.com/codelab-fun/codelab/pull/1160#issuecomment-544174093
The goal of this task is to identify which packages are causing the issue, and investigate getting rid of them and replacing
Where to start
Check out this repo in windows and run npm install

I checked yesterday. On the last version of macOS npm install didn't work too. Yarn works perfectly.

Can you check with node 10?

Can you check with node 10?

Okay, I will try to do it.

Today I tried to use node js v10.17.0 (macOS Catalina) and all packages installed without any errors (with npm i comand).


Disallow chore Commit Type Across Org

Right! We should aim for consistency. Therefore, we should disallow chore in the core repo as well. Right?
Originally posted by @jginsburgn in https://github.com/karma-runner/karma-chrome-launcher/pull/249#discussion_r819012769

@devoto13 WDYT?

I think it is already disallowed, we just need to change the Semantic Release to use different scope in the commit or something.


Zoom GUI is not shown

Issue description
Zoom GUI is not shown on current master (https://github.com/NixOS/nixpkgs/commit/1539167d215841a966b8395c1025d66812d63d31)
This started to happen after package refactoring (https://github.com/NixOS/nixpkgs/pull/42742, https://github.com/NixOS/nixpkgs/pull/42922, https://github.com/NixOS/nixpkgs/pull/43149), and not due to package upgrade (I've checked that).
I've documented some findings already https://github.com/NixOS/nixpkgs/pull/43149#issuecomment-403320716
ping @matthewbauer @tadfisher @shlevy @srhb @domenkozar and other Zoom users. Am I alone with this issue?
Steps to reproduce
$ nix-env -if https://github.com/nixos/nixpkgs/archive/master.tar.gz -A zoom-us -p ~/zoom-us-nix-profile
$ ~/zoom-us-nix-profile/bin/zoom-us
zoom started.
[CZPClientLogMgr::LogClientEnvironment] [MacAddr: 54:13:79:91:A6:8F][client: Linux][OS: NixOS 18.09.pre.5da8543 (Jellyfish)][Hardware: CPU Core:2 Frenquency:2.5 G Memory size:15951MB CPU Brand:Intel(R) Core(TM) i5-7200U CPU @ 2.50GHz GPU Brand:][Req ID: ]
Linux Client Version is 2.2.128200.0702
QSG_RENDER_LOOP is
XDG_CURRENT_DESKTOP = GNOME;   GDMSESSION = gnome3
Graphics Card Info:: 00:02.0 VGA compatible controller: Intel Corporation HD Graphics 620 (rev 02)
Zoom package arch is 64bit, runing OS arch is x86_64
AppIconMgr::systemDesktopName log Desktop Name: gnome3
^C

At this moment no GUI (no greeting window) is shown.
Technical details

system: "x86_64-linux"
host os: Linux 4.16.10, NixOS, 18.09.pre.5da8543 (Jellyfish)
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.1pre6148_a4aac7f
channels(root): "nixos-18.09pre142796.4b649a99d84, nixpkgs-18.09pre142651.95a8cb3ade1"
channels(danbst): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixpkgs

I at least cannot reproduce this with nix-env -iA nixpkgs.zoom-us on nixpkgs 1539167

@danbst I installed to a profile using your reproduction steps, and it works fine here.
What is in your LD_LIBRARY_PATH in your shell?

This also reproduces the issue:
nix run -f https://github.com/nixos/nixpkgs/archive/master.tar.gz -u PATH zoom-us -c zoom-us

Currently investigating what zoom-us needs on the $PATH.

Thanks for your efforts, but when writing a command can you (and everyone else) please be complete? I had time to quickly confirm what you wanted to say, but here I am telling you that it is not a valid test case.
error: Package ‘zoom-us-2.2.128200.0702’ in /nix/store/442xsjbqwld3dmddbjm0303xvlg4l3dg-master.tar.gz/pkgs/applications/networking/instant-messengers/zoom-us/default.nix:94 has an unfree license (‘unfree’), refusing to evaluate.

I wrote an example of a complete command in closed thread with zoom in the title started by myself, so it's not like I don't practice this myself.

@coretemp Sorry about that. As the package is unfree, you'll need allowUnfree = true; in ~/.config/nixpkgs/config.nix or NIXPKGS_ALLOW_UNFREE=1 in your environment. So a complete standalone test case would be:
NIXPKGS_ALLOW_UNFREE=1 nix run -f https://github.com/nixos/nixpkgs/archive/master.tar.gz -u PATH zoom-us -c zoom-us

@tadfisher does this reproduce issue too?
### bash
$ NIXPKGS_ALLOW_UNFREE=1 \
  nix-shell --pure -I nixpkgs=https://github.com/nixos/nixpkgs/archive/1539167.tar.gz \
            -p zoom-us \
            --run "
               export XDG_RUNTIME_DIR=$XDG_RUNTIME_DIR
               export LD_LIBRARY_PATH=/run/opengl-driver/lib
               zoom-us
            "

I've checked that commit 66e18c3 (last revision of old zoom-us expression) launches fine for me with this bash command.

It sounds like it was caused by 81c2f270e535fbc2b7dc83c555b0d5625af004f4 using autoPatchelf? It could be that there is some incompatibility between what zoom-us is built with and what we have locally. Maybe we should just revert that part (but leave the version updates). I would much prefer to use Nixpkgs Qt here but there is a lot of unknown in how zoom-us is built. It might just be safest to revert to the old expression.

Turns out the issue was missing QML2 imports. This was discovered by comparing:
NIXPKGS_ALLOW_UNFREE=1 QT_DEBUG_PLUGINS=1 nix run -f  https://github.com/nixos/nixpkgs/archive/master.tar.gz \
-u PATH -c zoom-us

to
NIXPKGS_ALLOW_UNFREE=1 QT_DEBUG_PLUGINS=1 nix run -f  https://github.com/nixos/nixpkgs/archive/master.tar.gz \
-c zoom-us

and looking for libraries loaded by the latter but not the former.
One fix was to add a QML2_IMPORT_PATHS environment variable to the wrapper, but I found it simpler to re-use libsForQt5.env to build an environment with all the Qt dependencies symlinked in place and using its qt.conf. The package should be easier to maintain as a result since you don't have to know what feature each qt5 package is meant to provide.


Completed Issue-970 by adding code and Readme.md

Description

Added script for Converting PDF to Word document-
Added REa

Fixes #(970)
Replace issue_no with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x ] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

Input:

Output:

Completed ISSUE 970 - PDF to word converter


Closed-loop position control mode jumping along a trajectory

Hello again Tobit,
I have been trying to use closed-loop position control commands (using your repo installed as a python package) to get a motor to follow a smooth position curve: as a simple test, a sin(t) curve with a typical time step of 1 ms (though I have tried various step sizes up to 0.1 s and down to 0.1 ms). Like this:
startup_pos = {}
for id in motor_IDs:
    startup_pos[id] = motor[id].getMultiTurnAngle()

t = 0
dt = 0.001

def sin_pos(t):
    max_angle = 30 # degrees
    freq = 0.5 # Hz
    return startup_pos[5] + max_angle * math.sin(2 * math.pi * freq * t)

while True:
    fr_knee.sendPositionAbsoluteSetpoint(sin_pos(t), 500)
    time.sleep(dt)
    t += dt

However, this results in my motor jumping to various positions along the expected curve and stopping at each for a moment (maybe about a half-second), like it is only receiving some of the commands and ignoring the others. The frequency of the jumps does not seem to vary with time step size that I can tell. Have you experienced similar? Do you know if there is a substantial delay between when these motors will accept new closed-loop position commands? I'm using several RMD-X8-Pro-H V3, and each I've tested has done the same.
I have also tried various max velocity inputs, which as expected changes the speed at which the motor jumps from one position to the other, but otherwise is the same.
I'm aware this may be an issue with the motor rather than with your repo, so I apologize if this is an inappropriate forum for this discussion. I just figured you may be knowledgeable, so it was worth asking. I have reached out to myactuator for comment also. If this is just a behaviour of the motor for this control mode, I would like to try Motion Control Mode as an alternative, thus #9.
Thanks again,
Stefan

Hi again Stefan,
I did some initial tests with a very simple code similar to yours and I observed the very same behaviour that you are describing when setting new set points repeatedly. It works fine though when I send it a single target command and wait for it to complete.
There is not much information about the inner workings of this command online. What I expect to be happening is that the actuator itself has its own (secondary) closed loop control with its own gains that in an example like above you are interfacing with your primary loop in an open loop fashion (You never check the position that the actuator returns you and do not adjust your commanded value accordingly). I assume the actuator will try to reach the given set-point at its own speed limited by the max speed you set which might result in a choppy start-stop motion if the gains of the inner loop are not well tuned. You should be able to improve this behaviour by playing around with the max speed parameter though. I was not too interested in position control and so I have not looked much more into it. If I remember I got it reasonably smooth by choosing appropriate max velocity values.
Several comments under this Youtube video suggest that it might be a potential firmware issue with the actuator and that the manufacturer could supply you with an updated firmware that allows you to change the acceleration limits to 0 which should fix this behaviour:

UPDATE: it works great with Arduino, if we set the “position plan max acc” = 0 and “position plan max dec” = 0. In this case the motion is smooth (no steps).

I have not come around to test this though. If you try it out (or MyActuator comes back with an explanation), please let me know!

Hi Tobit,
Setting the position planning acceleration and deceleration limits to 0 seems to have worked - each motor I've tested can now follow a smooth curve on position control. Good find on that video. I wonder what setting these parameters to 0 actually does though... Maybe it unlimits the acceleration?
FYI MyActuator did get back to me as well. They said:

The time between sending and receiving the motor command of X8-Pro-H is about 2ms.
Sending and receiving the command too quickly will cause CAN message blocking

Which I take to mean a time step of either 2 ms or 4 ms (depending on your interpretation) is required. I'm using 2 ms now and so far its working well. However, prior to changing the acceleration limits the jumping behaviour occurred regardless of time step.
For future reference: I have motors of two RMD models and both exhibited the same behaviour (and are fixed by the same method): X8-Pro-H V3 1:6 and X8-Pro V2 1:9.
Thank you also for your thoughts re: adjusting the max velocity value as a potential improvement. Doing so changed the speed at which the motor would jump from one position to another but it would still stop at each position for a moment. I wasn't able to find a value that resulted in sufficiently smooth movement.
Cheers and thanks again!
P.S. This fix is helpful, but I am also still interested in an implementation of Motion Mode Control as #9. Thank you for taking a look at that - I look forward to hearing your thoughts.

Just to add a couple details: I was able to change the acceleration limits to 0 without a firmware update. I did so with a manually sent (can-utils cansend) CAN frame though, since I think there is a lower bound on the allowable values in your code (though I didn't try to use it). Presumably I could have also done so in their software connecting through the UART module.

It is likely that you can't send these values through my code. I implemented it according to their protocol documentation. Setting the limits to 0 is not something that is discussed there. I will update the code accordingly and add a comment.
Sure, I will have a look at how to implement this and respond to your other issue in the following days.
Yes, I have seen the 2 ms (and therefore 500 Hz update rate) as well in previous tests, see https://github.com/2b-t/myactuator_rmd_ros/pull/1#issuecomment-2156685589.

Thanks. Yes it may be helpful to have something in the code to change acceleration limit to 0: in some further testing today it does seem like setting the acceleration limit to 0 unlocks the acceleration, so when I send a single position control command with a max velocity e.g., of 500 I get a very high acceleration on motion start and some overshoot at the destination. For my application that's likely fine, as I expect to mostly use position commands at each time step over a path rather than a single command for a large position change. It may be helpful to be able to update the acceleration limit to and from 0 in the code depending on whether you're sending repeated or single commands though.


Feature request: Support HEIC

I am using 2.1.1 via Flatpak on Fedora 39. I'd like to be able to edit HEIC files.

It looks like there is a GDK pixbuf loader for those formats (e.g. https://github.com/linuxmint/pix/issues/134), so I think this should just work if that's installed (although we'd need to verify that)
If it does work, this would need to be added for:

[ ] Windows installer
[ ] macOS installer
[ ] snap package
[ ] flatpak package

Testing on Ubuntu 23.10, installing heif-gdk-pixbuf allows a HEIC image to be loaded successfully in Pinta. But the pixbuf loader doesn't seem to have implemented support for saving, so any edits would need to be saved to another file format.

For the snap, it'd be best to treat it the same as the webp pix-buf which was added to the Gnome extension so all other Gnome snaps could benefit from it too. Similarly I feel this is the kind of thing that would make sense in the Flatpak runtime rather than the Pinta Flatpak itself, but I'm less confident there if/how that would be done.

Thanks! Yeah I agree it would be handy to have this in the runtime by default so that other apps can benefit easily

For the snap side of this, I've seen this pop up in the news recently: https://www.omgubuntu.co.uk/2024/04/ubuntu-now-supports-heif-images
If HEIC support is in the default 24.04 install, it suggests that it's in the main repositories and could likely end up being trivial to get into Pinta's snap when it's repackaged for Core24 (which may be before of after 2.2 depending on timelines involved). Of course, this is really assuming it'll end up in the Gnome snap for Core24 since that's where it needs to be to work conveniently & for other apps to benefit.


generic qrect

Pull Request Test Coverage Report for Build 942822936

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 80.38%

Files with Coverage Reduction
New Missed Lines
%

operators/src/engine/query_processor.rs
1
39.47%

Totals

Change from base Build 936043945:
0.0%

Covered Lines:
17301

Relevant Lines:
21524

💛  - Coveralls


Add REST API

Or should it be GraphQL now?

As stated in the README, this project is no longer maintained and has been retired. Because of this, I am archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


Issue #18: CREATE A PAGE ON QATAR FOOTBALL AND LIFESTYLE

Closes: Issue #18
Issue Title : CREATE A PAGE ON QATAR FOOTBALL AND LIFESTYLE

Info about the related issue (Aim of the project) : Created a page for qatar football and lifestyle
Name: Alvin Okafor
GitHub ID: https://github.com/alvinokafor
Idenitfy yourself: Hacktoberfest Contributor

Closes: Issue #18
Add-ons📃
Used HTML and CSS to create an informative simple page on football and lifestyle in qatar

@alvinokafor please write Closes : # in pr description

Don't write the word issue @alvinokafor

@Nirvik07 sorry about the mistake, i've made the correction

@alvinokafor check everything and if you face  any problem with Hacktoberfest point system related to my repo, ping me in discord. Discord Id : babe07#7187


Problem using 'balance' with CLI

If a 'weight' column already exists in a cool file (i.e. the file has already been balanced), running the balance command causes an error.
Using /tests/data/GM12878-MboI-matrix.2000kb.cool,
cooler balance -p 10 tests/data/GM12878-MboI-matrix.2000kb.cool
followed by
cooler balance -p 10 tests/data/GM12878-MboI-matrix.2000kb.cool --force
causes the following error:
KeyError: "Couldn't delete link (No write intent on file)".
Ran across this testing CLI tools on a .cool file created by a .hic to .cool converter I wrote and thought you all should know.

Thanks, Carl! I can't reproduce the error on your example, though. Which version are you using?

Hi Nezar, I'm using cooler 0.5.3 and h5py 2.6.0.

Ah yes, that's patched in 0.6.1.


Adding 6 to see if it compiles...

This should be updated to use node_js key, but I really just want to see if this is building on these versions and I'm too lazy to do it manually

Elsewhere... https://travis-ci.org/tessel/t2-cli/jobs/125957491

Going to close this.


Support for new feature: Dynamic Overlay API

Is your feature request related to a problem? Please describe.
Ability to set the dynamic overlay via the VAPIX API
Describe the solution you'd like
Support to set Dynamic Overlay options as indicated in the VAPIX API documentation here: https://www.axis.com/vapix-library/subjects/t10175981/section/t10007638/display?section=t10007638-t10119641
Describe alternatives you've considered
I can make manual calls to the API but it would be great to have it supported in axis-js as we are using the library in other places of the application
Additional context
There are quite a few features offered thru the VAPIX API, but I think adding/updating/listing current overlays would be a great start for axis-js

Hi @juventus18, is this something you would like to contribute?

@FantasticFiasco yeah, I think I'll do a subset of the VAPIX API and focus only on the text overlays.


[QUERY] Service Bus - OperationCanceledException on RenewLock

Query/Question
Sometimes I got "The operation was canceled." in my SubscriptionClient with SessionHandler.
I could not find any reasons for that. Could you please describe the case?
Please let me know if I need provide more info

Microsoft.Azure.ServiceBus.ServiceBusException:
at Microsoft.Azure.ServiceBus.MessageSession+d__14.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.MessageSession+d__18.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.SessionReceivePump+d__24.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
Inner exception System.OperationCanceledException handled at Microsoft.Azure.ServiceBus.MessageSession+d__14.MoveNext:
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.Amqp.AsyncResult.End (Microsoft.Azure.Amqp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)
at Microsoft.Azure.ServiceBus.Core.MessageReceiver+<>c__DisplayClass86_0.b__1 (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Threading.Tasks.TaskFactory`1.FromAsyncCoreLogic (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.Core.MessageReceiver+d__86.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.MessageSession+d__14.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)

There is similar issue with MessagePumpTaskAsync. It was fixed by ignoring the OperationCanceledException. Can I also ignore such exceptions or is it critical for RenewLock?
Environment:

Name and version of the Library package used: Microsoft.Azure.ServiceBus 5.1.3
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects):  5.0.203 Windows
IDE and version : Visual Studio 16.9.6

//cc: @shankarsama

Thank you for your feedback.  Tagging and routing to the team best able to assist.

Hi @shankarsama could you please help?

Hi @shankarsama could you please help?

Can anyone answer me? Please

Hi @Marusyk.   Yes;  @shankarsama's team owns the Service Bus service and the generation of the SDK that you're asking about.    Looping in a few of the other team members in case they're available to assist.
//cc: @DorothySun216, @bainian12345,  @vinaysurya, @yvgopal

@Marusyk Apologies for the delay here.
Please note that the Microsoft.Azure.ServiceBus package is now deprecated. A newer package Azure.Messaging.ServiceBus is available as of November 2020. While the older package will continue to receive critical bug fixes, we strongly encourage you to upgrade. Read the getting started guide and  migration guide for more details.
If you still have issues using the older Microsoft.Azure.ServiceBus package, please use your Azure support plan to open an Azure support ticket.

Thanks @ramya-rao-a very useful and in time :(


Make carrier specification comply with javascript frontend supported styles

Specifying multiple carriers as a single string is currently not supported; knit(..., "CA CB") will fail even if CA and CB have been specified in the initial carrier list. This is probably slightly different from the options the js frontend allows.

Opinion: My feeling on this is that specifying a list as a space-separated string is a bridge too far even for dynamically-typed languages. (In general, I'd prefer if we chose one right way to specify things instead of trying to support all possible ways.)

I am totally okay with that, I think then the js-frontend shouldn't support the arbitrary strings which it seems to. (They should probably try to be as similar as possible in formats they support although I suspect only a few people would switch between the two frontends)


Re-orient top-level documentation toward new developers

Purpose
Make the project friendlier to newcomers.
Change summary
Reorganize and lightly rewrite the top-level README.md and CONTRIBUTING.md so that they are aimed more at a newcomer audience -- developers who have heard of DSNP but doesn't know much about it yet.  The README now has a table of contents and starts out by stating the purpose of the Example Client and who the intended audience is.

Do we want to link to https://libertydsnp.github.io/example-client/ from the README, by the way?

A lot of great changes!

Do we want to link to https://libertydsnp.github.io/example-client/ from the README, by the way?
I think that's a good idea.

Suggestion: Even without the changes suggested above, this PR is an improvement (I think) on the docs there currently.  How about we merge to main, and then I do a separate PR to address what I'm able to from the commentary above.
The reason I suggest doing it that way is that most of the comments above are about further improvements; they're not bugs in the PR so much as things we could do in addition to this PR.  When that's the case, I think merge-and-improve is the best course, because that way the perfect is not being the enemy of the good and we get the stuff that's already done into the mainline of development.
Thoughts?

Also, I just said this to @shannonwells in another channel:
"But if you'd like to tweak the doc changes -- which you are far more qualified to do than I am anyway -- +1 and just put the tweaks directly on the branch before merging!"

I'm good to approve if we only remove or change the info about static-server, which I'm fine doing myself just so we can get it merged.

Ah, you're right -- that's an actual technical mistake in my change that does make things worse :-), good point.  Yes, please go for it.

Ah, I merged locally but I can't push to main:
$ git branch
* main
  newcomer-doc-updates
$ git merge newcomer-doc-updates
hint: Waiting for your editor to close the file...
Waiting for Emacs...
Merge made by the 'recursive' strategy.
 CONTRIBUTING.md |  97 +++++++++++++++++++++++++++++++++++++++++++++++----------------
 README.md       | 168 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----------------------------------------------
 2 files changed, 169 insertions(+), 96 deletions(-)
$ git push origin main:main
Enumerating objects: 20, done.
Counting objects: 100% (16/16), done.
Delta compression using up to 4 threads
Compressing objects: 100% (5/5), done.
Writing objects: 100% (11/11), 6.10 KiB | 6.10 MiB/s, done.
Total 11 (delta 7), reused 9 (delta 6), pack-reused 0
remote: Resolving deltas: 100% (7/7), completed with 5 local objects.
remote: error: GH006: Protected branch update failed for refs/heads/main.
remote: error: 2 of 2 required status checks are expected.
To github.com:LibertyDSNP/example-client.git
 ! [remote rejected] main -> main (protected branch hook declined)
error: failed to push some refs to 'github.com:LibertyDSNP/example-client.git'
$

It merges cleanly, FWIW, so whenever you get a chance it shouldn't take hardly any time, @shannonwells or @claireolmstead.


Added 'webex' to 'webexteams'

Replaces #192.
Added webex as Label to webexteams, since those are identical.
NOTE: As of a reply I got from Cisco support, Cisco is consolidating "Cisco Webex Meetings" and "Cisco Webex Teams" into just "Webex".
The download of the Apple Silicon version of Webex Meetings does not work reliably (fails most of the time). So I will no longer consider this one.

Thanks!


Landing Page Title Slider first pass

Incorporated basic functionality of the slider. Basic functionality
should be bug-free. Reponsiveness considered.
Points for further consideration/development

Having just the verbs slide through while having the entire line
adjust slightly so that it is always centered.
Having the verbs fade in/out instead of sliding in
Visual refinements (eg: having Nova start sparkling at the end)

Preview:
https://user-images.githubusercontent.com/51963099/131238206-bbb8deb1-17e9-4fbe-8c08-e3d30825fb72.mp4

Catherine's feedback
@catherinehhu

@catherinehhu 's feedback
i feel like if we could rotate down instead of up?
also would it be possible to only scroll thru the changing word instead of the entire phrase?
and also perhaps making the transition a teeny bit slower

with some suggestions implemented
https://user-images.githubusercontent.com/51963099/131239068-f187d17f-a179-46bb-8e9f-a64c353eec0c.mp4

agree on making the transitions slower! prob even slower on the second version hmm. also think it will look even cooler w only the verb changing 8)


Style Multipoint GeoJson

When i tried to add styles to a multipoint geojson, the layer disappeared. i noted that on the dist directive at the createStyle function there is not a case for image or something like ol.style.Circle

@andresramirezmiori Is this issue still valid? There are a couple of other closed issues and PRs referencing this one. So I assume it's solved. Can you confirm?


Zoom doesn't work (in some specific conditions)

On 720p and 1080p videos , it zooms out instead of zoom in . Its doesn't do anything in 480p
It works well in 360p , but the notch (status bar of phone) is not covered when zoom in , notch area remains black.

Is this fixed?


[WFLY-12985] Provide a separate feature pack and dist for non-observability MP specs

https://issues.redhat.com/browse/WFLY-12985

retest this please

retest this please


[FEATURE] Support LAMBDA in typecheck

Several specifications in the examples repository are using the LAMBDA-syntax. For instance, SlidingPuzzles defines the operator move(p, d) as follows (the type annotations are introduced by me):
move(p, d) == "(<<Int, Int>>, <<Int, Int>>) => <<Int, Int>>" :>
              LET s == "<<Int, Int>>" :>
                    <<p[1] + d[1], p[2] + d[2]>>
                  pc == ChooseOne(board, LAMBDA pc : s \in pc)
              IN <<pc, {<<q[1] - d[1], q[2] - d[2]>> : q \in pc}>>

The type checker produces the following error:
[SlidingPuzzlesTyped.tla:47:42-47:61]: Undefined name LAMBDA. Introduce a type annotation.

Currently, the type checker just ignores the LAMBDA syntax. It probably can be just substituted with a LET-IN definition. In this case, the type checker would automatically infer the types inside LAMBDA.

It is working already in unstable. Nothing to fix.


Integrate zcash-light wallet from a separate repo

Probably need to do some godep stuff? Not sure how you want to do that, @cpacia. Running godep get github.com/paulbellamy/zcash-light seems to do nothing for me. :/
Edit: Note, still being audited, just getting this ready in the meantime.

Coverage decreased (-0.002%) to 26.113% when pulling 2c52602e913aec5673b5003b7f4f269d6af8e16a on paulbellamy:paul/zcash-separate-repo into 1e7e9f39931b1f9e11c092bd6e06861d8bc6371d on OpenBazaar:master.

@paulbellamy Please reopen when ready to submit.


Adding doc on Kubernetes health Probe configuration with Dapr /healthz

Adding document describing /healthz health endpoint in Dapr . Liveness/Readiness in Kubernetes and its configuration in Dapr sidecar container
Issue reference
Closes #475
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[x] Extended the documentation

I would add a paragraph that explains the health endpoints are not tied to Kubernetes and can be used with any ochestrstor.

I do not agree with the words in this doc. This needs updating

I do not agree with the words in this doc. This needs updating

Lets go through this in sync up, we can revise.


docs: use backstage.io link to backend-to-backend auth docs

Hey, I just made a Pull Request!
There's rightfully been some confusion in various contexts about where to find this documentation - switching to a fully qualified docs URL should make it easier to find. I'm assuming since this PR only changes comments it's not worth a changeset, but happy to add one if needed!
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

A changeset will be needed for this change to be released in case there are no other changes to the create-app. Make a tiny one 😁

:+1: @jhaals done in 2687029a67!


Unable to parse passwords with dollar signs

If you run a command like:
python3 kijiji_repost_headless -u<EMAIL_ADDRESS>-p wiRf4f2F6Zip0DvZZ$ubls0IfuxKD5xQVl0P5VCUT show

The password is parsed as:
wiRf4f2F6Zip0DvZZ

You will likely have to use single quotes around the password string so that the shell parses it correctly.
Dollar sign characters are used for variable dereferencing in most shells, which is why it's cutting off the latter half of the password in the example. The shell variable ubls0IfuxKD5xQVl0P5VCUT is unlikely to be defined, and thus dereferences to an empty string, therefore the text wiRf4f2F6Zip0DvZZ$ubls0IfuxKD5xQVl0P5VCUT gets interpreted by the shell as just wiRf4f2F6Zip0DvZZ.
This 'problem' is due to how the shell parses input, not with this python module.

Sounds good!


layers: lx464 Add drawtime validation of bound buffer memory.

Add a drawtime check to validate that storage or uniform buffers
have valid memory bound to them at draw time before using.
Change-Id: Ia0a07f221dfcb2ea4315e4f52ee04d7cdda22cf7

+1

cherry-picked and pushed to master


datadog measuring seconds or milliseconds?

the graphs in datadog are showing that requests are taking 50-300s meaning almost 1minute to over 5 minutes.  I doubt this is correct.  Seems like it's actually measuring milliseconds.  Can someone confirm?

We stopped using datadog years ago so I can't say for sure. I know this was working correctly at the time.

We are seeing the same issue. The datadog docs uses python's time.time() which returns seconds
start_time = time.time()
results = db.query()
duration = time.time() - start_time
statsd.histogram('database.query.time', duration)

For anyone with this issue, the fix is to go into Metric -> Summary and locate the node.express.router.response_time.* metrics and change the metadata values that say seconds to milliseconds.  This should change the graphs to represent milliseconds instead of seconds.


[BUTTON IDEA]: Settings Button

Name of button: Settings Button

Description: Button to open a settings menu

Button Type (Animated, Special, etc...): Icon

Will you work on it?:Yes

Great idea! I've assigned you.

Do we have to create an icon?


[HOLD for payment 2024-07-22] [HOLD for payment 2024-07-17] [Payment card / Subscription] fix card data not showing up in the new subscription page

Problem
Right now if you add a card to your existing subscription via https://dev.new.expensify.com:8082/settings/subscription/add-payment-card it'll not show anything. There are multiple issues here causing this, but they are small enough to be combined into this one use case.
The first problem is explained in this slack thread. Basically we are not properly initializing the fundList in the Onyx db on the client. We need to fix that.
The second problem is that after this was completed we never updated App's code to make use of the new format of the data, which means it doesn't show the default card.
Solution
Fix both of the described problems above and make a newly added payment card show up in the UI.
Issue OwnerCurrent Issue Owner: @abekkala

Web-PR is in review, App PR is on hold until Web-PR is merged and deployed

Web PR is merged, waiting deploy.
App PR is in review, waiting approval and also waiting web changes to be deployed.

Web PR was deployed to staging, waiting deploy to production

App PR has merged: https://github.com/Expensify/App/pull/44361
Adding labels to get a BZ assigned, and @brunovjk as the C+ reviewer.

This is done.

Hey, this line of code was missed on the @blimpich PR - https://github.com/Expensify/App/blob/main/src/libs/SubscriptionUtils.ts#L206
Not his fault, this one was sneaky 😅
The specific line of code doesn't cause the error reported in this issue but it will not trigger card expired related UI.
I can create a PR for this if you want 😄

Ah, thank you for catching that @pac-guerreiro! I'll make a quick issue/PR for it right now 👍

Also no regression tests should be needed for this issue. @abekkala I think we need to pay @brunovjk for reviewing this PR?

Yes $250

@brunovjk offer sent!

thank you all :D

@brunovjk payment sent and contract ended - thank you! 🎉


IDENTITY-4047 Create Fine Grained Permissions for Workflow feature

Depends on https://github.com/wso2/carbon-identity/pull/1383

Need to merge https://github.com/wso2/product-is/pull/318 as well with this.


MUIC-479: Fix deprecated calls to the SDK

Also adds SwiftFormat + fixes Linter warnings about trailing whitespaces

@kuldar-daniel sorry for a long read under this simple PR, but I did some investigation and push one additional commit with swiftgen template that we could apply to this case. Please pull the latest commit 16782d6 and build the project to check the results. Perhaps it will be a better solution than the addition optionality and execution fatalError. Please ping me directly for some additional information/discussion.
Thank you.

@dukhovnyi worrks for me. Reverted style changes


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1668 ms

Resolved: Bofrost.lu performance has improved in 285d402 after 6 minutes.


Upgrade curator

Upgrades to the latest curator version and follows the inclusion/exclusion pattern that storm uses.

@Crim reading around I'm pretty sure that the issues we hit before were because curator wants to pull in zookeeper 3.5.  This should work around that. 🤞


Fix teleporting when Google API directions quota reached

Short Description:
Polyline improvements
Fixes/Resolves

Fix teleporting when API quota reached
Add unittest tests for the Polyline class (including pickles for directions and elevation responses)
Add requests-mock to requirements.txt
Remove the old ad-hoc polyline_generator_tester.py

Good style, less coding, more testing code :)


Add /c/ag and /c/pg URI

I can see that the travis ic complaints that
"HTTP request sent, awaiting response... 503 Service Temporarily Unavailable
2016-08-15 14:16:58 ERROR 503: Service Temporarily Unavailable."
https://travis-ci.org/openthread/openthread/jobs/152396657
Could I know what is the cause of the failure? Not quite sure whether the failure is caused by my change or not?

The travis error you saw was a transient one - it couldn't connect to launchpad for some reason.  I restarted the build and it passed.

Thanks for implementing the MGMT_ACTIVE_GET and MGMT_PENDING_GET commands.  Please see my comments inline.

LGTM 👍


Don't manage /etc/paths inside cpe_pathsd

/etc/paths should be managed outside of this API (imo).
This API is arbitrarily managing the system paths file for macOS but not Linux.
As system paths can change, it should be left up to the org to decide if they want to manage this or not, especially since it has no impact on cpe_pathsd itself.
In fact, Apple did change system paths in catalina:
/usr/local/bin
/usr/bin
/bin
/usr/sbin
/sbin
/Library/Apple/usr/bin
/Library/Apple/bin

The alternative would be to add to the API to allow path injection into the system paths file (at the top).

bump


Add support for multipe instagram files

closes GH-26

Added classifier for instagram comments

"Information about you" can include keys ['primary_location','inferred_email']

Just need to fix failing tests and then this can be merged


Make Migrate

Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in _run_code
File "/root/.cache/pypoetry/virtualenvs/llama-app-backend--Qk0ygDj-py3.11/lib/python3.11/site-packages/alembic/main.py", line 4, in
main(prog="alembic")
File "/root/.cache/pypoetry/virtualenvs/llama-app-backend--Qk0ygDj-py3.11/lib/python3.11/site-packages/alembic/config.py", line 630, in main
CommandLine(prog=prog).main(argv=argv)
File "/root/.cache/pypoetry/virtualenvs/llama-app-backend--Qk0ygDj-py3.11/lib/python3.11/site-packages/alembic/config.py", line 624, in main
self.run_cmd(cfg, options)
File "/root/.cache/pypoetry/virtualenvs/llama-app-backend--Qk0ygDj-py3.11/lib/python3.11/site-packages/alembic/config.py", line 601, in run_cmd
fn(
File "/root/.cache/pypoetry/virtualenvs/llama-app-backend--Qk0ygDj-py3.11/lib/python3.11/site-packages/alembic/command.py", line 398, in upgrade
script.run_env()
File "/root/.cache/pypoetry/virtualenvs/llama-app-backend--Qk0ygDj-py3.11/lib/python3.11/site-packages/alembic/script/base.py", line 579, in run_env
util.load_python_file(self.dir, "env.py")
File "/root/.cache/pypoetry/virtualenvs/llama-app-backend--Qk0ygDj-py3.11/lib/python3.11/site-packages/alembic/util/pyfiles.py", line 93, in load_python_file
module = load_module_py(module_id, path)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/.cache/pypoetry/virtualenvs/llama-app-backend--Qk0ygDj-py3.11/lib/python3.11/site-packages/alembic/util/pyfiles.py", line 109, in load_module_py
spec.loader.exec_module(module)  # type: ignore
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/workspaces/sec-insights/backend/alembic/env.py", line 9, in
from app.core.config import settings
File "/workspaces/sec-insights/backend/app/core/config.py", line 4, in
from pydantic import BaseSettings, AnyHttpUrl, EmailStr, validator
ModuleNotFoundError: No module named 'pydantic'
make: *** [Makefile:15: migrate] Error 1

Have you run poetry shell then poetry install?

even after running these, still same error


Improve faiss by using updated MKL

@MostAwesomeDude thanks! I actually pushed htis up last night when I hit the wall so to speak. I have a version that builds now.
The real goal with these changes is to achieve semver+epoch, ie: 1.7.4+0.

Just curious, for the Dockerfile, is there a specific reason that you broke
out every package into its own layer (e.g. RUN apt install X; RUN apt
install Y;...)
Robert
On Mon, Dec 4, 2023 at 2:55 PM Corbin Simpson @.***>
wrote:

@.**** approved this pull request.
I'm still waiting for step 58 to complete (compiling FAISS) but this looks
likely to succeed. Thanks for improving the readability with those long
option names!
—
Reply to this email directly, view it on GitHub
https://github.com/artificialwisdomai/origin/pull/130#pullrequestreview-1763592148,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHDOZR3X7ML6VWCRGAKSUDYHZIGJAVCNFSM6AAAAABAFHKR62VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONRTGU4TEMJUHA
.
You are receiving this because your review was requested.Message ID:
@.***>

Just curious, for the Dockerfile, is there a specific reason that you broke out every package into its own layer (e.g. RUN apt install X; RUN apt install Y;...) Robert
…
On Mon, Dec 4, 2023 at 2:55 PM Corbin Simpson @.> wrote: @.* approved this pull request. I'm still waiting for step 58 to complete (compiling FAISS) but this looks likely to succeed. Thanks for improving the readability with those long option names! — Reply to this email directly, view it on GitHub <#130 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHDOZR3X7ML6VWCRGAKSUDYHZIGJAVCNFSM6AAAAABAFHKR62VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONRTGU4TEMJUHA . You are receiving this because your review was requested.Message ID: @.***>

There are many approaches to installing packaged tools within Linux.
The two obvious alternatives:
B. RUN apt -y install build-essential python3 python3-full python3-venv python3-pip swig ninja-build git  cmake gpg curl zstd
C. RUN apt -y install build-essential && apt -y install python3 && apt -y install python3-full && apt install python3-venv && p apt -y install python3-pip && apt -y install swig && apt -y install ninja-build && apt-install git && apt -uy install cmake && apt -y install gpg && apt -y curl && apt -y zstd
A from this PR:
RUN apt install -y build-essential
RUN apt install -y python3
RUN apt install -y python3-full
RUN apt install -y python3-venv
RUN apt install -y python3-pip
RUN apt install -y swig
RUN apt install -y ninja-build
RUN apt install -y git
RUN apt install -y cmake
RUN apt install -y gpg
RUN apt install -y curl
RUN apt install -y zstd

Can you take a look at the approaches, A, B, C and define which one offers the best software quality?
I would encourage ignoring layers, because, as you can see, this PR only spits out a binary in the host filesystem in target/.... If you wish to evaluate with layers, can you tell me why layer minimization matters?
Thanks
-steve

@rstarmer @MostAwesomeDude this latest patch concludes this patch series. The goal of this patch series was to ensure an updated faiss is installed, rather than a downrev version which happens today.


ENH: Create the login page

In this step we will add a login as well as a registration page to the project and connect it to the user management.

@gianlucafr Hi, I have created a branch called WIP-Index-Page for you. You can use it to implement the index.html. The file already exists in the folder ISD22/pacmanapp/templates.
If you want, you can use the bootstrap framework for designing the page. It is already added to the project so there is no additional setup needed. Information about that can be found on Bootstrap.
About the index.html file: There is this block content and endblock in the file. Between those two lines, you can write your "code".


feat: create a single images overlay

With this change all of the images that we use across our charts will now be defined in a single place. This will better allow an automated pipeline to control our image usage and ensure that we're maintaining a unified set of images for a given release, branch, or deployment.

All images are using the builtin kustomize image replacement solution.
The default base image set is now defined as 2024.1
The helm values are defining the image-<name> string within them, permitting the override to take place
The kustomization files have been updated to all use the updated resources defined in the image overlay

due to upstream feature gap, this can't go in for now. But we can circle back on this at a later date.


Not working for Behat tests in Drupal 8

I wonder if anyone has this library working for writing behat tests in Drupal 8? I'm using the version 1.4.4 of this library and my own fork of the [https://github.com/bkosborne/behat-vcr-extension](behat vcr extension) (so I could incorporate a bug fix and allow a version bump of this library).
I just get blank cassette files, which I think indicates that this library's replacement curl functions are not being called. I'm trying to test some pages on my site that make Guzzle requests, which it seems like should be fine, since Guzzle is using curl under the hood.
I read in another issue that the VCR must be 'started' right away after the autoloader, but I'm not sure if that's really the case anymore? In the various VCR extension libraries I've seen, none of them do that, they just turn it on when it's needed.
In any case, when I try to turn it on in my index.php right after the autoloader, I get a Drupal error about it being unable to locate another class that it needs for database interactions (not sure why).

Hey @bkosborne ,
Your issue might be related to an Opcache issue I've been facing.
Basically, if your PHP files are loaded into Opcache and if you enable PHP-VCR after that, PHP-VCR will not be able to "instrument" the PHP files and injects its code in those.
#254 will fix this when merged. It might be a good idea to retest PHP-VCR when this is merged.

Please retest when 1.5.0 is released, as it will contain the opcode fix.


Uncaught ImportError in _types.py line 10

Aider version: 0.57.1
Python version: 3.11.5
Platform: macOS-14.6.1-arm64-arm-64bit
Python implementation: CPython
Virtual environment: No
OS: Darwin 23.6.0 (64bit)
Git version: git version 2.42.0
An uncaught exception occurred:
Traceback (most recent call last):
  File "base_coder.py", line 1115, in send_message
    yield from self.send(messages, functions=self.functions)
  File "base_coder.py", line 1392, in send
    hash_object, completion = send_completion(
                              ^^^^^^^^^^^^^^^^
  File "sendchat.py", line 87, in send_completion
    res = litellm.completion(**kwargs)
          ^^^^^^^^^^^^^^^^^^
  File "llm.py", line 23, in __getattr__
    self._load_litellm()
  File "llm.py", line 30, in _load_litellm
    self._lazy_module = importlib.import_module("litellm")
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "__init__.py", line 20, in <module>
    from litellm.proxy._types import (
  File "_types.py", line 10, in <module>
    from pydantic import BaseModel, ConfigDict, Extra, Field, Json, model_validator
ImportError: cannot import name 'model_validator' from 'pydantic' (/Users/arthur/anaconda3/lib/python3.11/site-packages/pydantic/__init__.cpython-311-darwin.so)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "aider", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "main.py", line 709, in main
    coder.run()
  File "base_coder.py", line 723, in run
    self.run_one(user_message, preproc)
  File "base_coder.py", line 766, in run_one
    list(self.send_message(message))
  File "base_coder.py", line 1117, in send_message
    except retry_exceptions() as err:
           ^^^^^^^^^^^^^^^^^^
  File "sendchat.py", line 24, in retry_exceptions
    litellm.exceptions.APIConnectionError,
    ^^^^^^^^^^^^^^^^^^
  File "llm.py", line 23, in __getattr__
    self._load_litellm()
  File "llm.py", line 30, in _load_litellm
    self._lazy_module = importlib.import_module("litellm")
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "__init__.py", line 20, in <module>
    from litellm.proxy._types import (
  File "_types.py", line 10, in <module>
    from pydantic import BaseModel, ConfigDict, Extra, Field, Json, model_validator
ImportError: cannot import name 'model_validator' from 'pydantic' (/Users/arthur/anaconda3/lib/python3.11/site-packages/pydantic/__init__.cpython-311-darwin.so)

Thank you for filing this issue.
At first glance this looks like you installed aider in an environment with many other Python modules and aider's dependencies got mixed up. Can you please try to install aider in a separate Python environment? Either with anaconda (which you seem to be using) or venv or even pipx?
This document may be helpful -> https://aider.chat/docs/troubleshooting/imports.html

Thanks for trying aider and filing this issue.
This looks like a duplicate of #1437. Please see the comments there for more information, and feel free to continue the discussion within that issue.
I'm going to close this issue for now. But please let me know if you think this is actually a distinct issue and I will reopen this issue.


Resize header images for responsive

We've created header images, these need to be resized to serve appropriate sizes for smaller and larger screens

@robisaacnz
I've replaced the reports images in each folder with correct NZSL version. When you get a mo, could you please resize appropriately?

These are all done now afaik.


fit fix + simulation fix

neh, time_out gave issues when I wanted to generate multiple participants, I believe, or multiple datasets – something like that.


Window class does not match expected

It looks like something broke in my installation of this. I have it in my Program Files on my HDD, and I've tried running it as admin, but neither fix this issue.
I started up the executable today after realising it was failing to startup on boot (I was in the process of fixing that), but when I tried to use it, the window isn't appearing. I enabled debug and got these three lines:
Hotkey Pressed: ~Media_Play_Pause

Window ID: 47123948
Class: CabinetWClass

Window class does not match expected: CabinetWClass

I updated from the initial version I had - 1.1.0 - to 1.2.5 (latest) and the issue is still persisting.

I Have The Same Issue

Hm what version of Windows are you running and can you specify an example of what program it doesn't work with? There isn't really an installation, it's literally just one file, so there isn't really anything that can 'break' with the exe itself unless something with windows itself changed.

Wait are you trying to use it on regular windows explorer? It's only meant to show up for 'Save As' or 'Open' type windows, not Windows Explorer in general

Wait are you trying to use it on regular windows explorer? It's only meant to show up for 'Save As' or 'Open' type windows, not Windows Explorer in general

Windows 11, I only have ExplorerPatcher, it worked fine when I first downloaded it (I say installation but I mean just download, lol), I've tried resetting the settings, I haven't tried removing the tilde.
What I mean by the window not appearing is I mean that pressing the hotkey doesn't bring up the context menu.
Also, yes I was trying to use it on regular Windows Explorer, I've completely forgotten what I was trying to do when I created this issue because I have a terrible memory (I might've been trying to move downloaded files). Thusly, also forgot this isn't meant to be used on Windows Explorer. My bad!


Don't install Cloud SDK when importing Debian 8.

The Cloud SDK is currently broken for Debian 8 imports; the SDK dropped support for Python 3.4, which is Debian 8's version.
This change skips installing the Cloud SDK for Debian 8, and skips testing for it.
Testing
Ran the following:

daisy -project edens-test -zone=us-central1-b daisy_integration_tests/debian_8_translate.wf.json
daisy -project edens-test -zone=us-central1-b daisy_integration_tests/debian_9_translate.wf.json

/test all


Add Apollo config

This config allows using the Apollo GraphQL extension for VSCode which enables schema introspection and validation within gql tags.

Pull Request Checklist

[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).

Couldn't we add the same thing in package.json to limit the number of files in the repository root? Awesome change nonetheless! 🔥🔥🔥

I used apollo.config.js because I wanted to take advantage of BACKEND_URL variable.

Wouldn't it be more practical to use a named service instead of an environment variable? This way we could publish our schema updates to Apollo Engine and automatically update it in the client without access to a running server instance.

True, I haven't thought yet about publishing schema to Apollo Engine, but that's what we will do anyway at some point. Maybe we could merge this PR now so that devs can start using VSCode plugin right away and we will move things to package.json later?


UI filling fix

I was having a problem where the picker was filling the screen:

after using useSafeArea as true and adding a top margin of 10, my problem was fixed

@adnanjpg  Thank you for your contribution. Released in v2.0.20.


add copy button to submenu, universal copypaste component

solves#1474.
Should run Universal( Electron + Browser Figma)
Uses token.name as path value.

Thank you for this.
Could you remove any codestyle changes? This makes it super hard to see what changed if everything changed.

I am using the TSConf of the project in VSCode. Think its based on my Prettier Config. Sorry. Try to adopt your codestyles.


Reset error

When I run grecaptcha.reset(id) there is an error

am I doing something wrong? it is not caused by wrong id because I have only one captcha and the id is 0 is checked - thats what it is supposed to be for first captcha render right?

Thanks for the report - please give us some more information to be able to reproduce (ideally the url where this is happening, browser, etc).

Tried to recreate it but the error is not shown. probably issues with my site also so i guess false alarm. but still. if I reset the captcha is the callback function called? I want to remove the styling on the outer div of the captcha as you can see here. https://honzaik.xyz/captcha/ it works only onload but when reset (2s after completing captcha) the callback is not called.

Closing super old issues. Please re-raise if still relevant.


perf(robot-server): Handle analysis requests serially to take more advantage of cache

Overview
When an HTTP client requests a protocol analysis, the server:

Retrieves a binary blob of the analysis from the database.
Parses and validates the blob into a Python CompletedAnalysisResource objects.
Serializes the CompletedAnalysisResource down to JSON and returns it to the client.

Step 2 is a known major compute bottleneck. PR #12648 put it behind a cache, which helps a lot. But that implementation has a hole: if there are two simultaneous requests for the same uncached analysis, they'll both parse the analysis from scratch. Ideally, one of the requests would wait for the other one to complete, and just reuse its result. This PR makes it do that.
This was an attempt to treat RSS-328. For unknown reasons, it doesn't improve the overall user-facing time, even though our benchmarks do show an improvement.
Test plan
Behaviorally, I think we're adequately covered by existing integration and unit tests.
Performance-wise, I've tested this by running this benchmarking script from the discussion in RSS-328. I've confirmed that when running it in parallel 3 times, the "getting each protocol analysis" phase is ~3x faster: ~160s, down from ~500s.
Changelog

If an analysis request might be cached or add to the cache, handle it serially with respect to other such requests.

Add an asyncio.Lock to CompletedAnalysisStore.
Acquire that lock every time something calls .get_by_id() or .get_by_protocol().

Review requests

Does this fix make sense?
Can the comments be made more clear or concise?
This implementation has a limitation where small requests will get blocked behind big ones. This doesn't affect overall throughput, but it does affect latency. There are ways around this, but they're significantly more complicated, so I left them out. Does this sound acceptable?

Risk assessment
Low, but see the note above about small requests getting blocked behind big ones.

Discussed with RSS people in-person.


support customisable externalHelpersModuleNameText for --importHelpers flag

TypeScript Version:  2.1.1
Currently we can use --noEmitHelpers with --importHelpers to mitigate generation of bloated code in ever file which uses TS helpers ( extends, decorate ... etc ).
To make this work, currently only official tslib is supported which is hardcoded here
if I wanna override any of these helpers I cannot use  --importHelpers, instead I need to import whole tslib and override helpers that I need before any code in app is executed, which is not very convenient, like here:
import 'tslib';

// Babel implementation of _inherits
function _inherits(subClass, superClass) {
  if (typeof superClass !== "function" && superClass !== null) {
    throw new TypeError("Super expression must either be null or a function, not " + typeof superClass);
  }
  subClass.prototype = Object.create(superClass && superClass.prototype, {
    constructor: {
      value: subClass,
      enumerable: false,
      writable: true,
      configurable: true
    }
  });
  if (superClass) Object.setPrototypeOf ? Object.setPrototypeOf(subClass, superClass) : subClass.__proto__ = superClass;
}

(window as any).__extends = _inherits;

What I would like to have is flag in compilerOptions for setting node_module library name which should be used for --importHelpers.
Something like:
npm install --save my-tslib
{
  "compilerOptions": {
     "module": "es2105",
     "target": "es5",
     "noEmitHelpers": true,
     "importHelpers": true,
     "importHelpersLibrary": "my-tslib"
  }
}

// Hello.tsx
import * as React from 'react';
class Hello extends React.Component {}

will compile to:
// Hello.jsx
import * as tslib_1 from 'my-tslib';
import * as React from 'react';

var Hello = (function (_super) {
  tslib_1.__extends(Counter, _super);
  function Counter() {
        var _this = _super.apply(this, arguments) || this;
        return _this;
  }
  return Hello;
}(React.Component));

This will also help to support libraries like skate.js and use for instance Babel extends behaviour to make it work https://github.com/skatejs/skatejs/issues/936

Aurelia is a framework made of lots of independent modules and distributed in compiled form to several platforms (e.g. NPM, Bower, stand-alone). It is currently compiled with Babel but there is momentum to switch to TS.
One outstanding issue is that within this context, we don't want to take an external dependency (on tslib) but of course we don't want to repeat helpers all over the place either.
If we could specify the helpers module name in an option, then we could package them inside one of our core modules and have all bundles import from that.
This way helpers are not repeated and we still have no external dependencies.

I know the issue is rather old but, does anyone from the TypeScript maintainers team approve this feature?
I'd be happy to try implementing it, but as mentioned in the CONTRIBUTING, it would be great to have an approval first 🙂
FYI, in the project I'm currently working on (embedded in thousands of websites), we would like to provide our own implementation for some of these helpers, in order to make sure we don't use some objects/functions (such as Object.assign) that could be overridden by a 3rd-party script loaded in the same page as our script.


Release: 1.4.0beta1: Error running command mina client status: Failure "message length (13094) did not match expected length (13089)"

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues: https://github.com/MinaProtocol/mina/issues
[X] This issue is not a question, feature request, RFC, or anything other than a bug report. Please post those things in GitHub Discussions: https://github.com/MinaProtocol/mina/discussions

Description
I have installed the new beta using the commands here https://github.com/MinaProtocol/mina/releases/tag/1.4.0beta1.
After installing the beta I get the following error when running the command mina client status.
 Failed to get status
 ((rpc_error
   (Bin_io_exn
    ((location "client-side rpc response un-bin-io'ing")
     (exn
      (Failure "message length (13094) did not match expected length (13089)")))))
  (connection_description <created-directly>) (rpc_tag Get_status)
  (rpc_version 0))

If I check the logs it looks like the node is operating normally, it's just this command that isn't working.
Steps to Reproduce

Install new beta
Run command mina client status
Error message is displayed

Expected Result
Expect to get the usual mina client status output
Actual Result
Failed to get status
((rpc_error
  (Bin_io_exn
   ((location "client-side rpc response un-bin-io'ing")
    (exn
     (Failure "message length (13094) did not match expected length (13089)")))))
 (connection_description <created-directly>) (rpc_tag Get_status)
 (rpc_version 0))

How frequently do you see this issue?
Always
What is the impact of this issue on your ability to run a node?
High
Status
Failed to get status
((rpc_error
  (Bin_io_exn
   ((location "client-side rpc response un-bin-io'ing")
    (exn
     (Failure "message length (13094) did not match expected length (13089)")))))
 (connection_description <created-directly>) (rpc_tag Get_status)
 (rpc_version 0))

Additional information
No response

This issue was fixed by restarting the server. Closing the issue.


Added support for rollbar.js when SecureHeaders 2.x installed. Fixes #442

As a original author of the PR, I feel obliged to provide the fix.
However, SecureHeaders 2.x is nothing more than Rails controller hack, while Rollbar is a Rack Middleware. This means (and please correct me I'm wrong), the rollbar.gem doesn't have access to controller (or more exactly, controller instance variable @content_security_policy_nonce which holds the nonce) at the point when it invokes the script_tag method.
So what I did, I simply disabled the nonce fix for secure headers < 3 (or in more general, a non-middleware SecureHeaders version) - it's not perfect, but at least script tag is rendered properly again.

Hey @ajgon, thank you for this. I'll try it.
We've added a plugins infrastructure to the gem and now some things have been moved. So now we have support for other libraries in lib/rollbar/plugins/. This is not released yet, but the new code should work with that.
Yesterday I found a problem with rollbar.js & secure headers integration, so I'll need to fix that. But I'll use your commit to add support for 2.0.
Thanks.