License

Can you license the code for forking http://choosealicense.com/

Did it. You are good to go.

Great


Is it possible to customize group view ?

I mean I want to have image and title, how do I do that ?
With flatPosition I won't get the exact index when expanded to fetch the right item.

hey have  you get any solution for same ...

Unfortunately no I didn't.


Set version v0.6.4

Codecov Report
Attention: Patch coverage is 16.66667% with 15 lines in your changes missing coverage. Please review.

Project coverage is 73.04%. Comparing base (79725c8) to head (323ec1f).
Report is 55 commits behind head on develop.

Files with missing lines
Patch %
Lines

...phelel/velph/cli/transport/plot/plot_selfenergy.py
0.00%
6 Missing :warning:

.../phelel/velph/cli/transport/plot/plot_transport.py
0.00%
4 Missing :warning:

...helel/velph/cli/transport/plot/plot_eigenvalues.py
40.00%
3 Missing :warning:

src/phelel/velph/cli/el_bands/plot.py
0.00%
1 Missing :warning:

src/phelel/velph/cli/ph_bands/plot.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #35      +/-   ##
===========================================
- Coverage    73.42%   73.04%   -0.39%
===========================================
  Files           74       77       +3
  Lines         5223     5483     +260
===========================================
+ Hits          3835     4005     +170
- Misses        1388     1478      +90

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


Custom configuration name text box prevents input of text with length less than 50

Type of issue

[X] Bug
[ ] New feature
[ ] Enhancement

Description

...
Steps to reproduce

Add packages of custom configuration type with a name less than 50 chars

Expected behavior
Should allow the name of less than 50 chars
Current behavior <!-- The actual behavior observed --
Does not allow the name of less than 30 chars
Known workarounds
No
Possible solution
...
Context and Environment Standard java deployment

Operating System: ... Ubuntu
GitHub branch/tag: ...

Screenshot

completed https://github.com/Azure/pcs-remote-monitoring-webui/pull/1175


MacOS Klayout infinite window loop crashes computer

Hello,
I have recently installed klayout and open pdk to my Mac. When I add the library to klayout on Mac. the python script launches infinite amount of klayout windows as like this:

Do you know how I can fix this ? I suspect it's because of circular import loop in the script ?
Refik

Leo Moser (@mole99) made a pull request to the upstream library that will likely solve this problem:  https://github.com/efabless/sky130_klayout_pdk/pull/25.  The pull request is under review.

Hi @Revenantx86, as Tim has said, there is now a new method to install the sky130 KLayout PDK from Efabless.
You should be able to install the package directly from KLayout's package manager:

Let me know how that goes for you.


Stipple.@kwdef: Fixed incorrect doc string

Sorry, I did not see that this has already been fixed in master.


CMake Error attempting to build latest develop

Trying to build the latest bitshares develop branch and seeing this when running cmake
CMake Error at libraries/fc/CMakeLists.txt:193 (add_subdirectory):
  The source directory

   ~/Documents/repos/bitshares/libraries/fc/vendor/websocketpp

I checked out the master branch in fc and was able to build.

Same thing here ... fc not up-to-date in branch bitshares?

IMO such inconsistencies in the develop branch are to be expected and shouldn't be reported as an issue.
If you're using the develop branch you're expected to be able to handle such "issues" yourself (as you clearly are). ;-)

@pmconrad Good point. I'll close the issue.

It's reasonable to expect that the build is never broken on any public branch.
In this case, the build is not broken, but there is still a bug in that a submodule was added to fc--and will cause build failure if missing--without being documented anywhere.
I've updated the build instruction files to reflect that you should do git submodule update --init --recursive to properly checkout all submodules now.


Replace use of hits[].projects[].contributorMatrices with contributedAnalyses

After https://github.com/DataBiosphere/azul/issues/3420 has been deployed to prod, please replace usage of the deprecated response field with the replacement. There are no changes to the content of the field, only the name of the field has changed.
Deprecated : hits[].projects[].contributorMatrices
Replacement: hits[].projects[].contributedAnalyses

Hi, @danielsotirhos @theathorn this update is deployed to prod. You can remove the deprecated fields in dev and all should still be good. Thanks! D


polygon-fill galleruy

http://riccardoscalco.github.io/textures/

Done in 08-map-patterns/patt.sketch

Ready to implement :)

@viddo we should put these there at the same time we rebuild the modal to select this

Which modal is this for?

Image selector for polygon fiill
On May 7, 2015 11:45 AM, "Nicklas Gummesson"<EMAIL_ADDRESS>wrote:

Which modal is this for?
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb/issues/2839#issuecomment-99796625.

Reassigned to @javierarce then, since he started on this modal :)

Some previews:

super nice

+1 :)

Reviewing this comment, I have a doubt… should this patterns be available for the old modal too?

Hey, one doubt. When we select the category wizard we are currently showing just markers (so the assets we decided that are markers). But in that case I think it makes sense to show both. In general, I don't know if the distinction of patterns and markers make actual sense… what do you think?

yes it does, right? It depends on the type of data


Remove shims project

The year is 2022; I would hope that these are defined by now in all of our supported platforms. I think it's already the case that we have required a newer lib anyway via #49623 using copyWithin, and nobody's complained yet.
This removes the shims project and makes us throw if we can't find Map or Set.
It'd ultimately be nice to be able to just use the "real" Map and Set, but this is a good step.

Let's roll it out in 4.9 and see what happens


geos-config call is skipped if NO_GEOS_CONFIG env is set

In this case, CFLAGS and LDFLAGS must be set.
Resolves #322

Coverage remained the same at 78.033% when pulling a74dcfb2021fd248eff9ecf80d0dd60b8631f315 on sans-geos-config into 7b09eb777f688d9c404b0f20b80864c8e51c446e on master.

Coverage remained the same at 78.033% when pulling a74dcfb2021fd248eff9ecf80d0dd60b8631f315 on sans-geos-config into 7b09eb777f688d9c404b0f20b80864c8e51c446e on master.

Coverage remained the same at 78.033% when pulling a74dcfb2021fd248eff9ecf80d0dd60b8631f315 on sans-geos-config into 7b09eb777f688d9c404b0f20b80864c8e51c446e on master.


NIFI-10577 Fix Sensitive Checkbox rendering for Fetch Parameters

Summary
NIFI-10577 Fixes the Sensitive status checkbox rendering on Firefox in the Fetch Parameters dialog for Parameter Providers.
Changes include replacing the input elements and :before pseudo-element selectors with the nf-checkbox styling approached used in other components.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[ ] JDK 8
[X] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Reviewing...

Reviewing now


Symfony autowiring monolog channels

Hi everyone,
While looking for a way to autowire monolog channels, and I got stumbled upon StackOverflow thread where guy named "DrafFter" suggests adding LoggerFactory.
https://stackoverflow.com/questions/43788114/symfony-autowiring-monolog-channels
Maybe this can be implemented within this bundle? Or is there a better way to do autowiring?

In Symfony 4.2, https://github.com/symfony/symfony/pull/28234 will allow this. (will require a PR here to use registerAliasForArgument())

@MaksSlesarenko @nicolas-grekas @Seldaek Hi there!
Recently I was implement single point access to the all registered loggers by MonologBundle.
And also I tried to do some better solution - and did auto-generated logger decorators. Each class decorates one object of one of the registered monolog channel.
Sounds cool and it's works, but i have doubt  about my implementation because auto-generated classes places in the vendor folder under my bundle.
When you'll have time, please, tell me is this OK?
P.S.: Link to the bundle https://github.com/adrenalinkin/monolog-autowire-bundle

Do someone want to submit a PR since Symfony 4.2 is released?

@lyrixx Hello Grégoire!
I am interested and will be proud submit Pull Request. But first of all, please, can you review my implementation -  monolog-autowire-bundle.
Is that conception can be applied by Symfony?

Hey @adrenalinkin!
Your implementation is great, but in 4.2, this is now much easier - should be basically one line of new code - see https://github.com/symfony/symfony/blob/4.4/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php#L684 for an example
This would be awesome to have :)

Hello, @weaverryan!
Thanks you for your opinion, nice to hear that.
Correct me if I miss-understand you. Instead of creation different logger decorator classes we will focuses on the parameter names.
Now time for example:

Some channels:

monolog:
    handlers:
        doctrine:
            type:   stream
            path:   "%kernel.logs_dir%/%kernel.environment%.doctrine.log"
            level:  info
            channels:
                - "doctrine"

        acme:
            type:   stream
            path:   "%kernel.logs_dir%/%kernel.environment%.acme_channel.log"
            level:  info
            channels:
                - "acme_channel"

And some specific naming for the parameters to auto-wire these channels:

<?php declare(strict_types=1);

use Psr\Log\LoggerInterface;

class AcmeLoggerAware
{
    /**
     * @var LoggerInterface
     */
    private $acmeLogLogger;

    /**
     * @var LoggerInterface
     */
    private $doctrineLogger;

    public function __construct(LoggerInterface $monologChannelDoctrine, LoggerInterface $monologChannelAcme)
    {
        $this->acmeLogLogger = $monologChannelAcme;
        $this->doctrineLogger = $monologChannelDoctrine;
    }
}

First Question
Am I right understand main idea?

And one drawback of the implementation described above, in my opinion, - we don't have clear auto-complete in the modern IDE. All programmers should keep in mind specific naming rule.
On other side we can generate different logger decorator classes and then we will have useful and clear auto-complete. But drawback, I think, - more difficult realization with code generation.
<?php declare(strict_types=1);

use Linkin\Bundle\MonologAutowireBundle\Logger\ChannelAcmeLogLogger;
use Linkin\Bundle\MonologAutowireBundle\Logger\ChannelDoctrineLogger;
use Psr\Log\LoggerInterface;

class AcmeLoggerAware
{
    /**
     * @var ChannelDoctrineLogger
     */
    private $acmeLogLogger;

    /**
     * @var ChannelDoctrineLogger
     */
    private $doctrineLogger;

    /**
     * @var LoggerInterface
     */
    private $logger;

    /**
     * @param ChannelAcmeLogLogger $acmeLogLogger
     * @param ChannelDoctrineLogger $doctrineLogger
     * @param LoggerInterface $logger
     */
    public function __construct(
        ChannelAcmeLogLogger $acmeLogLogger,
        ChannelDoctrineLogger $doctrineLogger,
        LoggerInterface $logger
    ) {
        $this->acmeLogLogger = $acmeLogLogger;
        $this->doctrineLogger = $doctrineLogger;
        $this->logger = $logger;
    }
}

Second Question
What do you think about drawbacks of each implementations and what implementation should we apply?


Fixed some links in Tutorials overview page

Tag #891

In this PR I have fixed some links in tutorials overview page as mentioned under #1032

@Rohitesh-Kumar-Jain and @proudofsimin please add this PR to the merging list.
Thank you :)


Issue with assets

Hi, I am using the pot asset contained in this repo in writing my own environment: "mjcf/pot/mobility.urdf". However, I am having trouble in the reset phase.
in the initialization of environment
get root state tensor, useful for obtaining state information about the object
actor_root_state_tensor = self.gym.acquire_actor_root_state_tensor(self.sim)
self.root_state_tensor = gymtorch.wrap_tensor(actor_root_state_tensor)
get dof state tensor

_dof_states = self.gym.acquire_dof_state_tensor(self.sim)
self.dof_states = gymtorch.wrap_tensor(_dof_states)

self.gym.refresh_actor_root_state_tensor(self.sim)
self.gym.refresh_dof_state_tensor(self.sim)
self.saved_dof_states = self.dof_states.clone()
self.saved_root_state_tensor = self.root_state_tensor.clone()

and then when resetting:
self.gym.set_dof_state_tensor(self.sim, gymtorch.unwrap_tensor(self.saved_dof_states))
self.gym.set_actor_root_state_tensor(self.sim, gymtorch.unwrap_tensor(self.saved_root_state_tensor))

However, this leads to very weird physics/simulation errors in which all the pots suddenly fly around/glitching out upon reset. I verified that this is not a problem of my environment because everything loads correctly when I don’t reset things. But when I reset the tensors the environment just glitches crazily. Is there a reason why this might be happening?
As another test, I used another pot asset (the xml file): mjcf/pot/pot.xml, and the issue immediately disappears and everything works correctly. I am wondering what might be the difference between the mobility.urdf vs. pot.xml file, and why one of them leads to the glitch?
Thank you!

Dear @livermoreee ,
thanks for your sharing!
in the initialization of environment
get root state tensor, useful for obtaining state information about the object
...
and then when resetting:
...

This part of the code is really to ensure that there is no strange reset, so we save the state tensor at init and use it during reset.
However, this leads to very weird physics/simulation errors in which all the pots suddenly fly around/glitching out upon reset. I verified that this is not a problem of my environment because everything loads correctly when I don’t reset things. But when I reset the tensors the environment just glitches crazily. Is there a reason why this might be happening?

Can you tell me if you have changed anything based on the source code? Because this is not the case when we use it, as follows:
https://user-images.githubusercontent.com/55881363/198129890-ac659ce4-7608-487b-8bda-71fb36bbb22c.mp4
As another test, I used another pot asset (the xml file): mjcf/pot/pot.xml, and the issue immediately disappears and everything works correctly. I am wondering what might be the difference between the mobility.urdf vs. pot.xml file, and why one of them leads to the glitch?

This other pot is the version we tried during the development phase. It is a hand-written mjcf object, as shown below. It is not the same as the pot in our benchmark, so the reward shaping will be different from the source code, which has been abandoned.

Anyway, since Isaac Gym is still in development, empirically he will have kinds of weird issues, especially with the physics engine. I guess it may be a bug caused by collision, so you can try to modify the pot's init pos to make it a little higher from the table at the beginning and see if it can be solved.

Thank you for the detailed reply! I wrote a script below that demonstrates the problem that I am seeing:

Thank you for the detailed reply! I wrote a short script that demonstrates the problem that I have been seeing. This script can be placed inside the bi-dexhands directory and directly run. Here I am initializing only a table and the pot to be dropped from 1 meter above the table, and reset the environment every 200 steps. By observing the resets, it seems like the pot sometimes falls straight through the table, and sometimes glitches out and even flies away. However, if the asset is replaced with pot.xml instead of mobility.urdf, the issue disappears. Could you help me take a look at what I might be doing wrong? Thank you!
from isaacgym import gymapi
from isaacgym import gymutil
from isaacgym import gymtorch
from isaacgym.torch_utils import *

import math
import numpy as np
import torch
import random
import time
gym = gymapi.acquire_gym()
device = 'cuda:0'

sim_params = gymapi.SimParams()
sim_params.up_axis = gymapi.UP_AXIS_Z
sim_params.gravity = gymapi.Vec3(0.0, 0.0, -9.8)
sim_params.dt = 1.0 / 60.0
sim_params.substeps = 2
sim_params.use_gpu_pipeline = True
sim_params.physx.solver_type = 1
sim_params.physx.num_position_iterations = 8
sim_params.physx.num_velocity_iterations = 1
sim_params.physx.rest_offset = 0.0
sim_params.physx.contact_offset = 0.001
sim_params.physx.friction_offset_threshold = 0.001
sim_params.physx.friction_correlation_distance = 0.0005
sim_params.physx.num_threads = 0
sim_params.physx.use_gpu = True

compute_device_id = 0
graphics_device_id = 0
physics_engine = gymapi.SIM_PHYSX
sim = gym.create_sim(compute_device_id, graphics_device_id, physics_engine, sim_params)

viewer = gym.create_viewer(sim, gymapi.CameraProperties())

asset_root = "../assets"

table_dims = gymapi.Vec3(0.6, 1.0, 0.4)
asset_options = gymapi.AssetOptions()
asset_options.fix_base_link = True
table_asset = gym.create_box(sim, table_dims.x, table_dims.y, table_dims.z, asset_options)

object_asset_file = "mjcf/pot/mobility.urdf"
object_asset_options = gymapi.AssetOptions()
object_asset_options.density = 1000
box_asset = gym.load_asset(sim, asset_root, object_asset_file, object_asset_options)

num_envs = 256
num_per_row = int(math.sqrt(num_envs))
spacing = 1.0
env_lower = gymapi.Vec3(-spacing, -spacing, 0.0)
env_upper = gymapi.Vec3(spacing, spacing, spacing)
print("Creating %d environments" % num_envs)

table_pose = gymapi.Transform()
table_pose.p = gymapi.Vec3(0.5, 0.0, 0.5 * table_dims.z)

box_pose = gymapi.Transform()

envs = []

# add ground plane
plane_params = gymapi.PlaneParams()
plane_params.normal = gymapi.Vec3(0, 0, 1)
gym.add_ground(sim, plane_params)

for i in range(num_envs):
    env = gym.create_env(sim, env_lower, env_upper, num_per_row)
    envs.append(env)

    table_handle = gym.create_actor(env, table_asset, table_pose, "table", i, 0)

    box_pose.p.x = table_pose.p.x + np.random.uniform(-0.2, 0.1)
    box_pose.p.y = table_pose.p.y + np.random.uniform(-0.3, 0.3)
    box_pose.p.z = table_dims.z + 1
    box_pose.r = gymapi.Quat.from_axis_angle(gymapi.Vec3(0, 0, 1), np.random.uniform(-math.pi, math.pi))
    box_handle = gym.create_actor(env, box_asset, box_pose, "box", i, 0)
    color = gymapi.Vec3(np.random.uniform(0, 1), np.random.uniform(0, 1), np.random.uniform(0, 1))
    gym.set_rigid_body_color(env, box_handle, 0, gymapi.MESH_VISUAL_AND_COLLISION, color)

cam_pos = gymapi.Vec3(4, 3, 2)
cam_target = gymapi.Vec3(-4, -3, 0)
middle_env = envs[num_envs // 2 + num_per_row // 2]
gym.viewer_camera_look_at(viewer, middle_env, cam_pos, cam_target)

gym.prepare_sim(sim)

actor_root_state_tensor = gym.acquire_actor_root_state_tensor(sim)
root_state_tensor = gymtorch.wrap_tensor(actor_root_state_tensor)

_dof_states = gym.acquire_dof_state_tensor(sim)
dof_states = gymtorch.wrap_tensor(_dof_states) # (self.num_envs * total DOF, 2)

_rb_states = gym.acquire_rigid_body_state_tensor(sim)
rb_states = gymtorch.wrap_tensor(_rb_states)

gym.refresh_actor_root_state_tensor(sim)
gym.refresh_dof_state_tensor(sim)
saved_dof_states = dof_states.clone()
saved_root_state_tensor = root_state_tensor.clone()

step = 0
while not gym.query_viewer_has_closed(viewer):

    if step % 200 == 0:
        gym.set_dof_state_tensor(sim, gymtorch.unwrap_tensor(saved_dof_states))
        gym.set_actor_root_state_tensor(sim, gymtorch.unwrap_tensor(saved_root_state_tensor))

    step += 1

    gym.simulate(sim)
    gym.fetch_results(sim, True)

    gym.refresh_dof_state_tensor(sim)
    gym.refresh_actor_root_state_tensor(sim)

    gym.step_graphics(sim)
    gym.draw_viewer(viewer, sim, False)
    gym.sync_frame_time(sim)

gym.destroy_viewer(viewer)
gym.destroy_sim(sim)

Hi @livermoreee ,
Thank you for your detailed sharing! There is nothing wrong with the main part of your code, the problem is in the parameter of the physics engine.
After I modified it into:
sim_params.physx.solver_type = 1
sim_params.physx.num_position_iterations = 8
sim_params.physx.num_velocity_iterations = 0
sim_params.physx.rest_offset = 0.0
sim_params.physx.contact_offset = 0.001
sim_params.physx.friction_offset_threshold = 0.001
sim_params.physx.friction_correlation_distance = 0.0005
sim_params.physx.num_threads = 16
sim_params.physx.use_gpu = True

it could simulate normally, as follows:
https://user-images.githubusercontent.com/55881363/198270112-b5d95659-c4b6-48e0-88d7-b43e5a577b55.mp4
I only adjusted these two parameters:
sim_params.physx.num_velocity_iterations = 0
sim_params.physx.num_threads = 16

For why I modified sim_params.physx.num_threads, the official documentation of it is described as follows:
Number of CPU threads used by PhysX. Should be set before the simulation is created. Setting this to 0 will run the simulation on the thread that calls PxScene::simulate(). A value greater than 0 will spawn numCores-1 worker threads.

Therefore, it can be set according to the number of CPUs you want to use, maybe the bigger the better.
For why I modified sim_params.physx.num_velocity_iterations, You can take a look at this post on the isaacgym official forum, it said:

Therefore, in general, this parameter can be set to 0.
To be honest, most of the bugs in isaacgym currently come from the physics engine, especially when it comes to collisions. Sometimes I also look for answers on the official forums I mentioned above, which are very helpful.
Hope this can help you.

That helped, thank you!


Codegen adds references to both TorchSharp CPU and GPU packages. Should only be one.

System Information (please complete the following information):

Model Builder Version (available in Manage Extensions dialog): <IP_ADDRESS>5703
Visual Studio Version: 17.3.6

Describe the bug
Can't build project machine learning model is added to because CPU and GPU TorchSharp packages are added as references to the csproj file. It should only be either or based on the environment used to train.
To Reproduce
Train model for Text Classification scenario using Local GPU environment.
Expected behavior
Only one package reference is added depending on the environment selected.

Local CPU - TorchSharp-cpu
Local GPU - Torchsharp-cuda-windows

Screenshots

Additional context
.csproj file
<ItemGroup>
  <PackageReference Include="Microsoft.ML" Version="2.0.0" />
  <PackageReference Include="Microsoft.ML.TorchSharp" Version="0.20.0" />
  <PackageReference Include="TorchSharp-cpu" Version="0.98.1" />
  <PackageReference Include="TorchSharp-cuda-windows" Version="0.98.1" />
</ItemGroup>

Error message
<Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">

  <Target Name="CheckOneTorchSharpRuntime"
    AfterTargets="ResolveReferences"
    BeforeTargets="PrepareForBuild">
    <Error Text="Two TorchSharp runtime packages have been referenced (both libtorch-cpu and libtorch-cuda)" Condition="'$(TorchSharpCudaPackage)' == 'true' AND '$(TorchSharpCpuPackage)' == 'true'" />
  </Target>
</Project>

Workaround: Comment out one of the packages in csproj file -> Restart VS ->  Rebuild project


Change the modules to use lncm's fork

Change the modules to use LNCM's fork

Have you actually tested this one works as well?
The reason I haven't merged it in yet, is because I didn't have time to replace pi-init2 and make sure it works the same.

Also, I think there might be some Makefile changes needed, but I don't recall the exact changes I've done to pi-init3

Yes I'm using it

I just had a moment to look a bit deeper into this PR, and this change doesn't affect the actual process at all, and doesn't cause pi-init3 to be used. So if it worked, it worked because pi-init2 worked :).
To make sure it works, some extra changes in Makefile are also necessary, you can either try your luck with that, or I'll be on it probably around tomorrow.

Closing in favour of #14


Added Reward Backpropogation Support

Hi,
I have added support for AlignProp (https://align-prop.github.io/) for finetuning Stable Diffusion model using reward gradients.
AlignProp directly backpropagate gradients from the reward model to the diffusion weights. Thus is about 25x more sample and compute efficient than policy gradient based methods like DDPO.
The current implementation seems to train effectively, almost within an hour on a single A100 while using Aesthetic reward model. Please find the attached loss and reward curves  + some qualitative results after training.
https://github.com/huggingface/diffusers/issues/7312
Difference between DDPO and AlignProp:

DDPO uses PPO, which is a policy gradient method for aligning diffusion models.  AlignProp doesn't use policy gradients instead it directly backpropagates gradients from the reward function to diffusion denoising process, to maximize reward.

AlignProp can only work when the reward function is differentiable, DDPO on other hand can handle non-differentiable reward functions, as it never backpropagates gradients from the reward function weights.

As AlignProp takes benefit of the differentiability of the reward function as it backpropagates gradient. It is significantly more sample efficient than DDPO.

The loss function in AlignProp is simply the negative of the reward value outputed by the reward function, while in DDPO it's the PPO loss function.

As the reward function is sitting on the RGB images. AlignProp requires to do the full denoising chain from Noise to RGB during training, while DDPO can instead sample random denoising timesteps, similar to diffusion training.

DDPO and AlignProp both use LoRA and gradient checkpointing.

CC: @parthos86 @sayakpaul  @lvwerra @younesbelkada

Image Generations post training:

Thanks a lot @mihirp1998 for your hardwork ! In principle this looks good ! I just have few questions with respect to the differences between this method and DDPO, could you clearly highlight either on the documentation or in this PR what are the major differences between DDPO and this algorithm ? 🙏 I would also like to have a review from @sayakpaul if possible, what do you think of Reward Backpropagation ? Thanks !

I have added the differences in the pull request, let me know if u have some doubts or think something is missing.

Thanks for your contributions. I left a couple of comments.
I would love to see some concrete comparisons to DDPO (training time, reward dynamics, convergence of the validation samples, etc.).

I have made concrete comparisions with DDPO here. I ran the DDPO default code in TRL with batch size 128, while AlignProp also uses the same batch size. As can be seen AlignProp is significantly more sample efficient, here i train both the models for a few hours. Here x-axis is the epochs and y-axis is the reward achieved.

Although i ran the above experiments for a few hours, AlignProp only takes about 30 minutes to converge to a good solution. So i early stopped at the 8th epoch in training. Below is the comparision with DDPO after training both models for 30 minutes. Here x-axis is the training time and y-axis is the reward achieved.

Both the curves are similar to the curves in the paper.
The above curves were with the same set of prompts during training/testing. In the curve below i show AlignProp results on unseen prompts. As can be seen there is not much gap in results between seen/unseen prompts. Here dotted lines is the unseen prompts while solid line is the seen prompts.

Finally here are some generated images from AlignProp, for seen/unseen animals after training.

CC: @sayakpaul @younesbelkada

Hi @sayakpaul @younesbelkada ,
Any update on this? Happy to fix things if something's missing.

cc @sayakpaul could you give a final look ?

@mihirp1998 can you run the styling checks? make precommit

Will do. Allow me today as today is the SD3 release :)

Sure, no rush

@mihirp1998 can you also add alignprop_trainer in the documentation _toctree.yaml file?

Great work. I really like the sample efficiency gains :)

Thanks! Is something left from my side? As it still says: 2 workflows awaiting approval

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Great work. I really like the sample efficiency gains :)

Thanks! Is something left from my side? As it still says: 2 workflows awaiting approval

Oh nvm, seems like everything is approved! Thanks!

cc @vwxyzjn for a final look, if all looks great to you, feel free to merge!

Hi @vwxyzjn ,
Any update on this? Happy to fix things in case something's missing.


Introduce conditionally disabling tests

Right now you can only disable a test statically, sometimes you might want to disable a test only if certain conditions are met. For example, if the code under test is optional for your application and requires a certain extension the test suite should disable any tests that require that extension if it is not available. This is currently not possible. We should avoid introducing a TestCase::markDisabled... it requires all of the hooks and test processing to still be invoked when we could, and should, be skipping past these potentially costly, error-triggering procedures. Instead we should introduce a #[DisabledIf($method, $reason)] Attribute. The $method should be a public method on the class it was annotated on. If it returns true the test will be disabled otherwise it will run as normal.

I decided to move this out of the 0.6 release because getting this implemented is going to require changing around a fair amount of how we think of disabled tests. Statistics for pre running will need to be adjusted because we no longer can reliably tell whether or not a given test/case/suite is disable until runtime. I believe that this feature is important enough and has enough impact on the existing codebase that the implementation needs more thought.


Create Documentation for any SDK Implemented in our Developer API docs

Summary: Generate comprehensive documentation for any SDK that has been implemented here to assist developers in using it effectively.
Idea Flow:

Write clear and user-friendly documentation, including installation instructions and code examples.

Hi @phyleria, I would like this to be assigned to me to work on.

Sure, go ahead.

Which SDK would you like to work on?

The JavaScript SDK

Unfortunately that issue has been taken already here
I'll be closing this now.
You can pick another issue or create one on documentation and I'll assign it to you.


appending:  scalar,table    behaving incorrectly when table has keys

Keys on 2nd table should only be relevant if first arg has keys - which cannot here since is a scalar.
Get error when table has keys on appending dimension, e.g. 5, ('a' << 6) or 5,,(6.keyC: 'a')
When, have key on non-appending dimension (which must be a singleton dimension since appending to a scalar), the key of the table is used. E.g. 5,,('a'<<6) or 5,(6.keyC: 'a')
Problem does not occur with 1-entry table, e.g. 5::, ('a' << 6). Probably need to fix _appendSc() method in core.js; this is used for appending  scalar,table  and   table,scalar  but evidently always acting like table is first wrt keys.

Fixed for v0.3.1


Update instructions and clean up formatting

Instructions still don't work correctly, but look better :)  (See Issue #165)

Coverage remained the same at 86.951% when pulling bf028050f6cb8dfcd9b2b3c46df9d7832a680846 on dbkinder:patch-1 into b46fb9709eca287dbd46e9c143dbd243cceac096 on otcshare:master.


Parent Component Rollover when Matching

I want to apply a layout to some pages, and a variant of the layout another page. However, all these pages exist at the top-level.
If I have a <Layout navBarHidden={true | false}>. Should it be possible to add this component twice under Router under the same path, and then continue to the next component if no matches were found? For example,
<Router>
  <Layout path="/">
    <Page1 path="page1" />
    <Page2 path="page2" />
  </Layout>
  <Layout path="/" navBarHidden>
    <Page3 path="page3" />
  </Layout>
  <Page4 path="/page4" />
  <NotFound default />
</Router>;

Or is there a better way to achieve this?

Another suggestion would be to wrap <Layout/> so that it took in props passed to the different pages.  That would lend itself a bit better to the component model and clean up the code  😊
Hope that helps and thanks for using reach router!


Add Skills support

Create test + React component.
Example syntax:
 {
    "skills": [
        {
            "name": "Web Development",
            "level": "Master",
            "keywords": [ "HTML", "CSS", "Javascript" ]
        },
        {
            "name": "Compression",
            "level": "Master",
            "keywords": ["Mpeg", "MP4", "GIF"]
        }
    ]
 }

Not sure what "level" is for. Might ignore that. Basically grouped arrays of strings.

Finished ages ago.


⚠ ClamAV has degraded performance

In 3a4cac8, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 197 ms

Resolved: ClamAV performance has improved in 61f310e.


Add client details to order and ticket email template variables

Fixed issue where when there is a new order or ticket, email templates don't receive client's details in the template variables.

@samapraku very thank you for that.
We will review that and approve if everything is alright.


Add optional possibility to generate captcha partial with HAML

Hi!
Here is small change that adds possibility to generate default captcha partial with HAML, additionally to default ERB:
rails generate simple_captcha haml
I believe that HAML is quite popular among developers and this option could save a bit of time..
Few covering tests for generators was added :innocent:
Also I've added dummy application.css file because without it there was two failed Capybara tests.

thanks for the contribution!


How to reproduce results in the arxiv paper ?

Hi,
Do you guys have a trained model to support claimed numbers in the paper ?
or can you provide details on how to reproduce the training strategy to get the same.?

Please read the paper arxiv paper. What result are you getting?
On Thu, Sep 10, 2020, 2:52 AM Saleem Ahmed<EMAIL_ADDRESS>wrote:

Hi,
Do you guys have a trained model to support claimed numbers in the paper ?
or can you provide details on how to reproduce the training strategy to
get the same.?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/VIROBO-15/UDBNET/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANQL5YHOWY6QKWLBT34MOZTSE7WZLANCNFSM4RDKGXNA
.

Haven't tried benchmarking it yet, wanted to use it on our own dataset for summarizing whiteboard lectures.
Access to a trained model checkpoint would be helpful.


email bug

a@a

/^[a-zA-Z0-9!#$%&'+\/=?^_{|}~-]+(?:\.[a-zA-Z0-9!#$%&\'*+\\/=?^_{|}~-]+)@(?:a-zA-Z0-9?.)+a-zA-Z0-9?$/

From README:

IMPORTANT NOTE ABOUT EMAIL VALIDATION. As of version 1.12.0 this plugin is using the same regular expression that the HTML5 specification suggests for browsers to use. We will follow their lead and use the same check. If you think the specification is wrong, please report the issue to them. If you have different requirements, consider using a custom method.
In case you need to adjust the built-in validation regular expression patterns, please follow the documentation.


Add test framework

This pulls in the code from #2 as well so we should probably not merge this as-is.

Rebased after #2 merged so it should be clean now


Allow access only from a whitelisted host

allow localhost always and the application docker container
env variable per container the client is used in

closed by #15


Component resolution unexpected when only partially specified (updated)

Describe the bug
This component is incorrectly resolved for this project: Hello.zip
    - component: ARM::CMSIS Driver:USART

Use this command
csolution covert Hello.csolution.yml --context .Debug+AVH-CM0plus

The component does not exist, as for the DFP for this device only contains Keil::CMSIS Driver:USART.
The Hello.Debug.AVH+CM0plus.cbuild.yml file shows that csolution picks:
  components:
    - component: ARM::CMSIS<EMAIL_ADDRESS>      from-pack<EMAIL_ADDRESS>
However in<EMAIL_ADDRESS>this component is not marked with attribute isDefaultVariant.  This behavior is only expected for components that have isDefaultVariant set.
Expected behavior
Error message that the component cannot be resolved.
Environment (please complete the following information):

All packs installed from hello.cbuild-pack.yml
CMSIS-Toolbox 2.5.0-devint5
OS: Windows

This looks like intended behaviour rather than a bug:

The component identifier ARM::CMSIS Driver:USART:Custom corresponds to  Cvendor::Cclass:Cgroup:Csub, there is no Cvariant so the attribute isDefaultVariant is irrelevant.
Partial component identifiers are accepted and resolved to the best match.

This needs review.  I think resolving it in an IDE is great, but csolution should perhaps not auto-pick it.

The feature of allowing partially specified packs and components was introduced in the CMSIS solution project format right from the start. In my opinion removing this capability should be considered for a major version update (e.g. 3.0.0) and not for 2.6.0.

https://github.com/Open-CMSIS-Pack/cmsis-toolbox/blob/main/docs/YML-Input-Format.md#component-name-conventions
documents under Partly defined components the behaviour.  :Csub is not automatically selected for the user.
Today default implementations of device interfaces are defined without :Csub. It is therefore dangerous to add :Csub when there is not default implementation.

As just seen in the last nightly build, this impacts partial component identifiers such as CMSIS:OS Tick in
Open-CMSIS-Pack/csolution-examples, which now requires CMSIS:OS Tick:SysTick to be resolved.
Potentially many other projects may have replicated it.

It appears also in some other files within the Open-CMSIS-Pack organization:

https://github.com/Open-CMSIS-Pack/cmsis-toolbox/blob/878cd68d0d046512ed87fb7532416c4c214ce954/docs/CubeMX.md?plain=1#L198
https://github.com/Open-CMSIS-Pack/vscode-get-started/blob/a7225162af0572c5d2f935fe6a6cf722d4dd28c9/hello/hello.cproject.yml#L9
https://github.com/Open-CMSIS-Pack/RefApp-Framework/blob/1fc13e1e8e57c850b973d40a79c7e81a60c3cfe2/layer/RTOS/RTX/RTOS.clayer.yml#L17
https://github.com/Open-CMSIS-Pack/cmsis-toolbox/blob/878cd68d0d046512ed87fb7532416c4c214ce954/docs/MCUXpressoConfig.md?plain=1#L200
https://github.com/Open-CMSIS-Pack/cbuild2cmake/blob/8e45fa1f13deacf0ebca633ca1db16350d5eacdd/test/data/solutions/library-rtos/project/project.cproject.yml#L7

Should be also fixed.

fixed
fixed
fixed
fixed
fixed


Missing Scala 1.X versions artifacts

It seems only 2.X versions artifacts are published in Sonatype. I am testing the waters for a huge codebase migration from Akka to Pekko and we are using swagger-akka-http for Scala 2.13 on version 1.1.1.
It would facilitate things a great deal to have this version available to use :pray:
Thank you in advance!

That's a really old version with many versions published since - even in 1.x.y release range.
I don't want to go back through all the legacy versions of swagger-akka-http and produce Pekko versions.
I think it would be much simpler if you just copy the classes you need from swagger-akka-http and migrate them to Pekko yourself. Migrating to Pekko is pretty easy. See https://pekko.apache.org/docs/pekko/1.0.0/project/migration-guides.html
You can just embed a couple of these classes in your own project.
You can find the swagger-akka-http source at the v1.1.1 tag
https://github.com/swagger-akka-http/swagger-akka-http/tree/v1.1.1

I don't want to go back through all the legacy versions of swagger-akka-http and produce Pekko versions.

Makes perfect sense, you already have a large codebase at hand for that to be feasible.

I think it would be much simpler if you just copy the classes you need from swagger-akka-http and migrate them to Pekko yourself. Migrating to Pekko is pretty easy. See https://pekko.apache.org/docs/pekko/1.0.0/project/migration-guides.html

I am migrating a big codebase to Pekko myself thanks to find and sed, just a few extra casses is not that bad :slightly_smiling_face:

You can just embed a couple of these classes in your own project.
You can find the swagger-akka-http source at the v1.1.1 tag https://github.com/swagger-akka-http/swagger-akka-http/tree/v1.1.1

Thank you!

Give me a shout if you run into trouble. If sed is not working out for you, it might be worth looking at https://github.com/jtjeferreira/pekko-scalafix


Provide a desktop application

Small mac app wrapper that starts up and shows the web client (should be separate project).
There are issues with using SPM together with XCode in Mac apps... perhaps wait until Apple sorts this out.

One way of going about this would be to include the Catena binary in the application as-is and just run that as a process. The app wrapper would then provide a very basic browser view on the web client (or just open the browser and sit in the system tray).


Let challenge names change, without affecting portfolio

We should be able to change the name of a challenge, without having to deal with fallout from code portfolio etc.
One solution would be to store all old names in an array, and then implement logic, so only the new name would show in portfolio, and automatically replace the challenge url also.

@atjonathan Can we add some tests to make sure that for a single challenge id, the name is same both in the challenges json file and in the other json file which houses the ids and titles of challenges.
This way we don't have to worry about renaming of challenges breaking the portfolio. So, if the contributor fails to rename the challenges in both the places, our tests would report.
What do you think?

@arun1595 There is no file that has only ids and titles.

@BerkeleyTrue
I meant the json files which holds information about all the challenges in a certification.
For example, we have seed/challenges/01-front-end-development-certification/front-end-development-certification.jsonwhich contains the ids and titles of the bonfires and ziplines.

Those titles are never rendered. They are just used for use humans working on those files. It's kind of hard to tell one Object id from another.

Nevertheless if implemented this should take care of migrating the search from name based to id based and encoding the URL of the challenge that is stored in the user profile with id, which means while visiting the challenge the reverse look-up is also applicable.
Using challenge id's is a good idea, but is going to affect a ton of things (impact not broken):

The wiki which automatically loads the relevant challenge in the sidebar.
Current User portfolios
Search and Look up
npm run test?

Anyone looking into this should look into all these areas and if I have missed some.

This is fixed in current staging. Display Names are pulled directly from challenge
Happy Coding


Update pom.xml

maven-checkstyle-plugin use inputEncoding instead of encoding

Codecov Report

Merging #5115 (18a8c13) into develop (b24ce72) will increase coverage by 0.07%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #5115      +/-   ##
=============================================
+ Coverage      43.22%   43.29%   +0.07%
- Complexity      7776     7785       +9
=============================================
  Files            996      996
  Lines          69233    69233
  Branches        9159     9159
=============================================
+ Hits           29927    29976      +49
+ Misses         35539    35486      -53
- Partials        3767     3771       +4

Impacted Files
Coverage Δ

...a/org/apache/rocketmq/filter/util/BloomFilter.java
60.43% <0.00%> (-2.20%)
:arrow_down:

.../apache/rocketmq/store/ha/DefaultHAConnection.java
65.86% <0.00%> (-1.21%)
:arrow_down:

...ava/org/apache/rocketmq/store/index/IndexFile.java
73.68% <0.00%> (-0.88%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
82.38% <0.00%> (-0.48%)
:arrow_down:

...ent/impl/consumer/DefaultLitePullConsumerImpl.java
69.37% <0.00%> (-0.47%)
:arrow_down:

...org/apache/rocketmq/store/DefaultMessageStore.java
53.24% <0.00%> (-0.16%)
:arrow_down:

...apache/rocketmq/store/timer/TimerMessageStore.java
54.18% <0.00%> (-0.10%)
:arrow_down:

...ava/org/apache/rocketmq/filter/util/BitsArray.java
59.82% <0.00%> (ø)

.../org/apache/rocketmq/proxy/config/ProxyConfig.java
49.74% <0.00%> (ø)

...mq/client/impl/producer/DefaultMQProducerImpl.java
45.29% <0.00%> (+0.23%)
:arrow_up:

... and 8 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Adding namespace to dataset class.

This does not add support for namespace in _implicit_environ.
Related to #902.

As a result of #861 we will likely be removing _implicit_environ which makes this the actual resolution of #902.
/cc @jonparrott

FWIW I checked where namespace is used in the .proto API. It is only used in PartitionId (along with dataset_id) and the PartitionId message type is only used in Key and RunQueryRequest.
Accordingly, the only places we support namespace is:

Connection.run_query(dataset_id, query_pb, namespace=...)
Key(..., namespace=...)
Query(..., namespace=...)

Cool, as long as it's an option on the client class when that happens I'm satisfied.

LGTM.  I plan to rename datastore.dataset.Dataset -> datastore.client.Client after this merges.

OK SGTM


[BUG] The array or object is rendered incorrectly

🐛 Bug description
import { ref } from 'vue';
...
setup() {
  const text = ref(["123"]);
  return () => (

{text.value}

  )
}

Error: Maximum recursive updates exceeded. This means you have a reactive effect that is mutating its own dependencies and thus recursively triggering itself. Possible sources include component template, render function, updated hook or watcher source function.
Render out 123
🏞 Desired result

No error
Render out ["123"]

🚑 Other information
import { reactive } from 'vue';
...
setup() {
  const text = reactive ({a:"123"});
  return () => (

{text}

  )
}

Object will have no rendering results, And I want is { "a": "123" }

You should use JSON.stringfy

🤣


Added sahyadri.isrdc.in to the bootstrap hubs

ISRDC @ IIT Bombay is one of the four information security research centers spread across India. ISRDC @ IIT-Bombay does R&D in a wide range of topics in information security and privacy.

Thank you for your submission. It will be reviewed soon and submitted for processing in CI.

Thank you for your submission. It will be reviewed soon and submitted for processing in CI.

Jenkins Build Summary
Built from this commit
Built at 20190810 - 18:12:31
Test Results

Build Type
Log
Result
Status

msvc.Debug
logfile
1178 cases, 0 failed, t: 568s
PASS :white_check_mark:

gcc.Release -Dassert=ON, MANUAL_TESTS=true
logfile
915 cases, 0 failed, t: 4m59s
PASS :white_check_mark:

docs, TARGET=docs
logfile
1 cases, 0 failed, t: 0m1s
PASS :white_check_mark:

gcc.Debug -Dcoverage=ON, TARGET=coverage_report, SKIP_TESTS=true
logfile
1178 cases, 0 failed, t: 16m53s
PASS :white_check_mark:

clang.Debug
logfile
1178 cases, 0 failed, t: 2m56s
PASS :white_check_mark:

msvc.Debug, NINJA_BUILD=true
logfile
1178 cases, 0 failed, t: 603s
PASS :white_check_mark:

gcc.Debug
logfile
1178 cases, 0 failed, t: 2m54s
PASS :white_check_mark:

clang.Debug -Dunity=OFF
logfile
1177 cases, 0 failed, t: 6m43s
PASS :white_check_mark:

gcc.Debug -Dunity=OFF
logfile
1177 cases, 0 failed, t: 4m47s
PASS :white_check_mark:

clang.Release -Dassert=ON
logfile
1178 cases, 0 failed, t: 5m9s
PASS :white_check_mark:

gcc.Release -Dassert=ON
logfile
1178 cases, 0 failed, t: 4m57s
PASS :white_check_mark:

gcc.Debug -Dstatic=OFF
logfile
1178 cases, 0 failed, t: 2m56s
PASS :white_check_mark:

msvc.Debug -Dunity=OFF
logfile
1177 cases, 0 failed, t: 1091s
PASS :white_check_mark:

gcc.Debug -Dstatic=OFF -DBUILD_SHARED_LIBS=ON
logfile
1178 cases, 0 failed, t: 2m51s
PASS :white_check_mark:

gcc.Debug, NINJA_BUILD=true
logfile
1178 cases, 0 failed, t: 2m50s
PASS :white_check_mark:

msvc.Release
logfile
1178 cases, 0 failed, t: 402s
PASS :white_check_mark:

clang.Debug -Dunity=OFF -Dsan=address, PARALLEL_TESTS=false, DEBUGGER=false
logfile
1177 cases, 0 failed, t: 5m1s
PASS :white_check_mark:

clang.Debug -Dunity=OFF -Dsan=undefined, PARALLEL_TESTS=false
logfile
1177 cases, 0 failed, t: 8m26s
PASS :white_check_mark:

https://sahyadri.isrdc.in:51235/crawl doesn't seem to work for me...

Allowing a hub to be crawled is nice, but I don't know that we want to make that a requirement for including the hub in the bootstrap list.
I will note that this particular hub already appears on the network and, if I am right, already carries several hundred peers. If @ivishwas wishes to share general details about his system (CPU, RAM, bandwidth) that's up to him.


feat: Customizable_Fee_Vault

[x] issue #3
[x] follows contribution guide
[ ] code change includes tests
[ ] breaking change

Fee_vault.deposit_native() is invoked only if fee_vault address is set.

Nice work, humble but it'll be a useful change!


Cb check standalone

This PR takes on the important parts of #416, but without the hassle of exposing cb-schema as a public toplevel library, which it turns out completely messes up our docker setup.
Instead, it is now a library inside the pipeline, which is properly linked to and accessed when needed.
I also incorporated this comment: https://github.com/ocurrent/current-bench/pull/416#discussion_r1121924894

As the CI passes and the error reporting has been improved, I now declare cb-check to be in beta!
I'll incorporate user-suggested changes and merge when satisfied

The force-pushes were rebases and a clean-up of the commit history, without changes. As everyone seems happy, I'll merge as-is.


Fix merge when parent is null.

Fix Dart runtime issue found in https://github.com/antlr/antlr4/pull/3841#issuecomment-1229266680

@parrt This should fix the issue, tested in local and all tests pass.

awesome!

oh shoot. This is on my fork not the main. If it's OK with you I will just make the quick tweak and close this one.


Trouble opening preview in separate pane

[Enter steps to reproduce:]

...
...

Atom: 1.14.1 x64
Electron: 1.3.13
OS: Ubuntu 16.04.2
Thrown From: atom-latex package 0.7.7
Stack Trace
Uncaught TypeError: Cannot read property 'file' of undefined
At /home/james/.atom/packages/atom-latex/lib/manager.coffee:88

TypeError: Cannot read property 'file' of undefined
    at Manager.module.exports.Manager.findMainSelfMagic (/packages/atom-latex/lib/manager.coffee:88:5)
    at Manager.module.exports.Manager.findMainSequence (/packages/atom-latex/lib/manager.coffee:65:21)
    at Manager.module.exports.Manager.findMain (/packages/atom-latex/lib/manager.coffee:42:15)
    at Server.module.exports.Server.httpHandler (/packages/atom-latex/lib/server.coffee:33:26)
    at /packages/atom-latex/lib/server.coffee:12:46)
    at emitTwo (events.js:106:13)
    at Server.emit (events.js:191:7)
    at HTTPParser.parserOnIncoming [as onIncoming] (_http_server.js:543:12)
    at HTTPParser.parserOnHeadersComplete (_http_common.js:105:23)

Commands
     -0:27.6.0 atom-latex:preview-tab (input.hidden-input)

Non-Core Packages
atom-beautify 0.29.23
atom-latex 0.7.7
multi-cursor 2.1.5
pdf-view 0.57.0

Thanks for the logs - I can now reproduce the error by doing the following

Open a .tex file
Click Preview PDF  in the LaTeX panel
PDF opens in new tab pane - and PDF tab is in focus
Package → Atom-LaTeX → Preview PDF or Preview PDF in new Tab

I will look into adding a fail-safe to avoid this in the future.
But could I ask why you would like to Preveiw PDF when the PDF is already open?
The Preview PDF  is only meant to open the compiled PDF - Don't you mean Build PDF if you want to see the compiled changes to your .tex file?

On some further investigation - I can also reproduce this error by doing the following.

Add any atom project
Add another atom project and open a .tex file in the second project.
Click Preview PDF in the LaTeX panel
PDF opens in new tab in a split pane - and PDF tab is in focus
Drag the PDF tab back to the .tex pane -  collapsing two panes to one.

Result - PDF does not render with just a grey background and finally a Unexpected server response
PDF.js v1.6.414 (build: 3b3a179)
Message: Unexpected server response (0) while retrieving PDF "http://<IP_ADDRESS>:56240/preview.pdf".

There is no error toast popup - but same TypeError in the debugger.
Strangely, I can't seem to reproduce this in dev mode - need to investigate more!


Add support to build recovery flashable zip after compilation succeeds

Closing in response to adding x86 kernel build support


Problem pdf re-conversion

Hi, I found a bug in your theme for obsidian. I too love the simplicity and graphical lightness of things 3, so when I saw your theme I installed it straight away, very well done! Unfortunately though I encountered a bug and had to uninstall. I don't think it makes any difference but the language I am writing in is Italian.  Exporting the file to pdf the result is acceptable, however when I take the text of the pdf and paste it on Page mac os the result is that all the letters of the words are mirrored , as if written from right to left. I hope you can solve it soon! Good work!

Hey @SerJeamp glad you're enjoying the theme! thanks for creating this issue, however I don't think this is an issue with the theme and more so has to do with your pdf viewer and Pages handling pasting italian text 🤷
does the same issue occur with other themes?


IssueID:1919:Improve PWM and ADC driver framework. Remove old flash device driver framework.

IssueID:1911:Remove old flash device driver framework
IssueID:1919:improve PWM and ADC driver framework

Cancelled.

Reopened.

感谢提交 Pull Requests！
Thanks for submitting a pull request.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.福明 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

start build

手动触发！
感谢 Pull Requests！
Thanks for submitting a pull request.

Security Check : 5d924f86428be8c73317a1bf30f932e6719f10aa

auto build fail,number:276

start build

手动触发！
感谢 Pull Requests！
Thanks for submitting a pull request.

start build

手动触发！
感谢 Pull Requests！
Thanks for submitting a pull request.

auto build success,number:277


Still using inspect.py, which breaks agent in python 3.11.4

Description
I run the latest newrelic agent (newrelic==<IP_ADDRESS>) and it doesn't work with python 3.11.4. It appears as if the only solution to this is to downgrade to 3.10.
Expected Behavior
I expected newrelic to support python 3.11.
Troubleshooting or NR Diag results
This is the stack trace:
  File "/web/[path to my code]", line 1, in <module>
    import newrelic.agent
  File "/usr/local/lib/python3.11/site-packages/newrelic/agent.py", line 15, in <module>
    from newrelic.config import (
  File "/usr/local/lib/python3.11/site-packages/newrelic/config.py", line 32, in <module>
    import newrelic.core.agent
  File "/usr/local/lib/python3.11/site-packages/newrelic/core/agent.py", line 32, in <module>
    import newrelic.core.application
  File "/usr/local/lib/python3.11/site-packages/newrelic/core/application.py", line 34, in <module>
    from newrelic.core.data_collector import create_session
  File "/usr/local/lib/python3.11/site-packages/newrelic/core/data_collector.py", line 23, in <module>
    from newrelic.common.agent_http import (
  File "/usr/local/lib/python3.11/site-packages/newrelic/common/agent_http.py", line 15, in <module>
    from newrelic.common.object_wrapper import patch_function_wrapper
  File "/usr/local/lib/python3.11/site-packages/newrelic/common/object_wrapper.py", line 27, in <module>
    from newrelic.packages.wrapt import (ObjectProxy as _ObjectProxy,
  File "/usr/local/lib/python3.11/site-packages/newrelic/packages/wrapt/__init__.py", line 10, in <module>
    from .decorators import (adapter_factory, AdapterFactory, decorator,
  File "/usr/local/lib/python3.11/site-packages/newrelic/packages/wrapt/decorators.py", line 34, in <module>
    from inspect import ismethod, isclass, formatargspec
ImportError: cannot import name 'formatargspec' from 'inspect' (/usr/local/lib/python3.11/inspect.py)

Steps to Reproduce

Install python 3.11.4
pip install newrelic==<IP_ADDRESS>
Add import newrelic.agent any file and run it

Your Environment
Running Python 3.11.4

Hi there--the good news is that the version right after that one (v8.2.1) fixes this issue, as shown in this PR.


form.submit() to localhost server with all network adapters disabled will hang, when regular AJAX is fine

We have an app that uses a node-webkit client and tomcat server, which can be located on the same machine. We've been tracing a bug where sometimes the form.submit() just hangs forever, and finally realized that it happens if (e.g.) a laptop is set to Airplane Mode, or otherwise all network adapters are disabled (including any VMWare virtual adapters, etc).
Our app doesn't care about network adapters (all the AJAX calls to the localhost tomcat server work fine) but it's only the form.submit() that breaks. Any obvious reasons as to why this might be? It'd be nice for this to work in this particular use-case scenario, since it can be quite common for our application....

I'm not sure how FormData behaves inside node-webkit and in airplane mode :)
I'd assume that ajax calls are coming from within the browser part of your app and FormData operates in the node part. So it might be the difference. Also FormData doesn't handle anything network, it operates on top of native nodejs methods. So it must be nodejs thing.
What you can try though, is to use npmjs.org/request library (there are examples in the readme how to use it with the request lib), maybe it would produce different results.
PS. All our tests are using localhost for connectivity, but we never tried to run them in the Airplane mode (and my laptop doesn't have such mode btw).
So I'm not sure as of nature of the issue nor of the ways to solve it.

@CrackAttack I think this issue is a bit out of scope. If you are using AJAX (IE in the browser) electron or not, you should be using the native FormData implementation, not this. This module is to be used for emulating a form data like api within node js.
Let me know if you disagree, or think I missed the point.

Actually I am using it correctly in this instance - we need to use it within node.js (needing to bypass the regular FormData security with sending files without the user needing to specifically select/browse for the files). For us, we use both situations in our app. In one instance, the user will browse for a document and upload it to the (localhost) server using the native implementation, and it works. In a different case, we internally are collecting files to upload to the server without the user needing to browse for them, and we are using this API for node.js to then upload the file(s). In this case, the form-data does not behave like the native one does, and will "hang" when the server is localhost and no active network adapters are present. The native one doesn't have an issue.
All that being said, this is likely close-able, and I think it's useful just having this visible for posterity's sake in case someone else stumbles across it? For us, this specific use-case is relatively rare for our customers (to have both client and server installed on the same PC, and having no active network adapters), and we mitigated this by simply recognizing this scenario, and noting that if the files are on the same PC anyway, we don't need to "upload" them but rather can just browse the local file system for both the client and the server!


Add more strict message about M1 + system Ruby installation

There is a problem with system Ruby on M1 macs when someone tries to use cocoapod-generate gem. It looks like error:
  /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require': dlopen(/Library/Ruby/Gems/2.6.0/gems/ffi-1.15.5/lib/ffi_c.bundle, 0x0009): tried: '/Library/Ruby/Gems/2.6.0/gems/ffi-1.15.5/lib/ffi_c.bundle' (mach-o file, but is an incompatible architecture (have 'arm64', need 'x86_64')) - /Library/Ruby/Gems/2.6.0/gems/ffi-1.15.5/lib/ffi_c.bundle (LoadError)

or something like this.
Work around is to use custom Ruby installation (and reinstall cocoapods and cocoapod-generate gems): https://youtrack.jetbrains.com/issue/KT-49418#focus=Comments-27-5429773.0-0
It would be better to help kdoctor users with this case

doctor already shows an info message if system ruby installation is used:
 * System ruby is currently used
          Consider installing ruby via Homebrew, rvm or other package manager in case of issues with cocoapods installation

Do you think the output should be more detailed for M1 users? My concern is that there are few ways to install ruby and providing instructions on how to install and set up system PATH for all of them may be verbose.

Yes. For M1 we should show strict error because system Ruby on M1 is broken and Cocoapods gem doesn't work


python38Packages.awscrt: 0.12.0 -> 0.12.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.awscrt is: "Python bindings for the AWS Common Runtime"
meta.homepage for python38Packages.awscrt is: "https://github.com/awslabs/aws-crt-python"
meta.changelog for python38Packages.awscrt is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.12.2 with grep in /nix/store/8jrv7f2wqj1bykqna9llnjv3lk01fq3l-python3.8-awscrt-0.12.2

found 0.12.2 in filename of file in /nix/store/8jrv7f2wqj1bykqna9llnjv3lk01fq3l-python3.8-awscrt-0.12.2

directory tree listing: https://gist.github.com/408a7ffce8fd9545acc701a553a0f875

du listing: https://gist.github.com/2c2d1d5d2579f4546cc2a0c4f465f56e

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

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.awscrt
python39Packages.awscrt

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8jrv7f2wqj1bykqna9llnjv3lk01fq3l-python3.8-awscrt-0.12.2
ls -la /nix/store/8jrv7f2wqj1bykqna9llnjv3lk01fq3l-python3.8-awscrt-0.12.2/bin

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

Maintainer pings
cc @davegallant for testing.

@ofborg eval


pmap optimization: Don't precompute size and ndim on DeviceArrays.

We don't even look at them most of the time, and they are in the critical path for running pmap code.
Saves ~1-2ms on a pmap microbenchmark.

Without a specific benchmark showing they are faster to precompute or cache, it's hard to conclude they are worth caching/precomputing. Certainly for my benchmark, the prod call takes measurable time, and we never even call size so it's just wasted time. Do you recall the use case that motivated these? If not I'd suggest checking in this PR and readding caching when we see a use case.
To be honest, I also suspect we should just make shape, dtype, ndims, and size all just call into the C++ bindings rather than trying to optimize around something that may have been slow in some previous iteration of them.

I think the case that motivated precomputation was the microbenchmark that Dougal made 6+ months ago. But now that I think about it, maybe computing those things on-demand is a dominant strategy, i.e. would have made those benchmarks faster too.
(Also IIRC at the time of writing these things our C++ bindings for shape information were relatively slow, since they were using protos.)


CLI for multiple training tasks

As part of #126, @ziw-liu suggested:

For the future iteration (v0.3?) it might make sense to replace the manual CLI entry point (viscy [command]) with python's built-in (python -m viscy.cli.module [command]) to make supporting multiple learning tasks easier.

This makes a lot of sense and I'd vote to make this change sooner to clarify that the repo does implement multiple learning tasks. It is easy to change our virtual training scripts that launch training from viscy [command] -> python -m viscy.cli.module [command]. Are there other factors for which we should wait to refactor CLI?
@ziw-liu @edyoshikun

I'd do it after v0.2.0 is released (i.e. after #114) so that the v0.1.0 API reported in various places is kept in the release.

I am having trouble seeing what is the added benefit of using python -m viscy.cli.module [command]. What is this enabling?
Also,  #114 does not rely on the CLI so you could change it now. I would prefer if you make the changes now and we update the wiki to the latest method of calling the training/prediction than having to change this twice. The instructions will be on the wiki .

I merged #114. @ziw-liu feel free to make the changes

@ziw-liu The change in CLI and the refactor of modules that implement multiple learning tasks can be done in a single PR.


Can rehypeShikiji customize meta?

Clear and concise description of the problem
```js name="test" copy
console.log('test')
```
What if I want to get the name parameter?
Suggested solution
I hope that the parameters in the meta can be exposed
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Sure, if you have any idea about the API design, feel free to bring it up here or make a PR.


🔄 sync collections search

Thank you for your contribution to the KodaDot NFT gallery.
👇  _ Do a quick check before the merge.
PR type

[x] Bugfix

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main-nuxt and I've no conflicts
[x] I've tried respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted screenshot of demonstrated change in this PR

What's new?

[x] added collections searchbar
[x] PR closes #1687

Screenshot

Legit! Works!

However @roiLeo can we centre the buy button?

Legit! Works!
However @roiLeo can we centre the buy button?

#1685 I think it will be done here?

Or let it merge 🥇


Compare page

References User Story #92

[t:50]

Adding left and right section to Compare page, working on getting styling right

[t:20]

Finished styling, made edits to Filterbar and Data as well


gapi.client.init promise fails silently

Possible duplicate here.
importing gapi as object using this script
import {
    gapi
} from 'gapi-script';
import oauth2Keys from './oauthKeys.json';
const APIKEYS = oauth2Keys.web;
const DISCOVERY_DOCS = ['https://www.googleapis.com/discovery/v1/apis/youtube/v3/rest'];
const SCOPES = ["https://www.googleapis.com/auth/youtube.readonly", "https://www.googleapis.com/auth/youtube"];

//im building a vuejs3 application
mounted() {
    gapi.load('client:auth2', this.initClient);
},
    data() {
        return {
            GOOGLEAUTH: null
        }
    },
    methods: {
        initClient() {
            var self = this;
            console.log("initialising client...");
            gapi.client.init({
                discoveryDocs: DISCOVERY_DOCS,
                clientId: APIKEYS.client_id,
                scope: SCOPES
            }).then(function () {
                console.log(self);
                //listen for sign-in state changes
                self.GOOGLEAUTH = gapi.auth2.getAuthInstance();

                console.log("googleauth is....");
                console.log(self.GOOGLEAUTH);

                self.GOOGLEAUTH.isSignedIn.listen(self.updateSigninStatus);
                // Handle initial sign-in state. (Determine if user is already signed in.)
                self.updateSigninStatus(self.GOOGLEAUTH.currentUser.get());
            });
        }
}

Console logs in the promise callbacks not running. this.GOOGLEAUTH returning undefined in subsequent code. If I remove the clientid and scope params, the console logs run but this.GOOGLEAUTH still returning undefined.

Thanks for the issue report.  We do not support using the linked module to load GAPI since it is a snapshot of the GAPI loading code which appears to be at least 8 months old at this point which results in loading old versions of the libraries.  Please use officially documented methods of loading the libraries and try your use case again(samples: https://github.com/google/google-api-javascript-client/blob/master/docs/samples.md)


[SPARK-14004][SQL][MINOR] AttributeReference and Alias should only use the first qualifier to generate SQL strings

What changes were proposed in this pull request?
Current implementations of AttributeReference.sql and Alias.sql joins all available qualifiers, which is logically wrong. But this implementation mistake doesn't cause any real SQL generation bugs though, since there is always at most one qualifier for any given AttributeReference or Alias.
This PR fixes this issue by only picking the first qualifiers.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
Existing tests should be enough.

Test build #53531 has started for PR 11820 at commit 1a35c8a.

Test build #53531 has finished for PR 11820 at commit 1a35c8a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53531/
Test PASSed.

LGTM

Merging to master.


feat: Prefab, Duplication and naming overhaul

Alright, this is a big one:
This is a wide overhaul of the handling of map objects concerning names and fields that reference other map objects.
It's also a pretty complete rewrite of the prefabs to take that into account and simplify their handling.
The general idea is to be able to keep references correct through those operations, as much as possible. That means:
Renaming a map object also modifies every single reference to it, including on itself. This does mean that renaming needs a button to commit the changes now (and the change will be prevented if it would result in duplicate names)
Duplicating an object will keep every reference that's still available after the dupe, this mostly means that references to other objects will be emptied out if the dupe goes to another map.
Duplicating multiple objects will correctly modify references to other duplicated objects, so for example if we duplicate both Treasure and ItemPart in this example:

The new Treasure will point to the new ItemPart!

The same logic applies to prefabs ! If we prefab multiple objects and they have references to each other, those will be restored once we load the prefab again, but other references will be automatically emptied out.
Finally, I consolidated all the prefab tools to their own window, which I think works really well !

I'm putting this as a draft for two reasons:

This is a pretty huge change, I'm aware, so I'm very open to feedback and may have forgotten some details
This... needs more testing, I'll try and work with it for a few days to see if I missed anything, but I mostly work with Elden Ring. It should be pretty generic, but the devil is in the details

Looks good. I will go through and test the changes with each project type to scout for any oversights.

I did notice there were a few issues with some map object types, but they seem related to the MSB implementation and are the same with the current implementation so it doesn't matter for now.
It would be great if BTL.Light was supported too.
Any how, once you are ready for it to be merged, undraft it and I'll merge it in.

Alright, yeah haven't found any big issues either, I tweaked the UI a bit, but this should be at least relatively good to go !

Sorry, I've had to revert this PR as there were some issues and I don't want to regress Map Editor functionality:

The "Rename" button is a bit of a workflow blocker, having to press it to change a name is a regression from the previous behavior where you could freely edit the name field.
The renaming doesn't respect the 4 digits at the end of several map objects (such as c4690_9000, 9000 should match the instance ID), it instead applies a number wrapped with {}. This is a regression to the previous behavior that handled this correctly.

If those issues are fixed, make a new PR with the fixed code and I'll merge the changes back in.

Hey, alright i'll look into it ! I'd like to have your opinion on a few things first, since both of those were conscious changes, not accidents. (Maybe I should have put more focus on them in the PR, sorry about that)
For the renaming indices, it's a bit ambiguous, since the {} this is the naming schemes soulsformats uses for duplicates (and indeed the extra {} part will be stripped in the actual data), but yeah I could see the _instanceId being less surprising.
The renaming button is a harder problem to solve. The idea behind it was to make renames act more intuitively, and not breaking references to an object just by renaming it. The problem is that it makes the act of renaming fallible, since you do not want duplicate names in your map, that makes cross references ambiguous, and that's kind of impossible to avoid if your text input is a direct map to the string.


How to improve script load performance

I was thinking about how to improve the performance of the scripts in my applications.
I currently have several applications using React.
My idea is the root application load the libraries together and the loaded application only provides the app script.
Is that a valid idea? Has anyone had success or failures at this point?

this is kind of a monorepo approach


add availability zones

clusters can have multiple availability zones now. to get a better
understanding on how this affects subnet ranges and cluster sizes the
info endpoint provides a bit more information about the configured ip
ranges in the installation. clients can then calculate the number of
nodes for the users.
Please make sure in case you are changing the spec all our customers are informed beforehand.
You can throw a message in #sig-customer with the changes and the Solution Engineers will take care of
sharing the changes with the customers.

A few general notes:
If we think that we might change the API soon so that instead of a number of availability zones users should define specifically which ones a cluster should use, then we should go for a more extensible API design that allows to prevent breaking changes. In cluster creation:
"availability_zones": {
  "number": 3
}

could be later extended to allow, as an alternative, the one below, while still supporting the above:
"availability_zones": {
  "selection": ["eu-west-1a", "eu-west-1c"]
}

A style detail: Alle the little description values do not need a . in the end, as long as they are only one sentence long.

If we think that we might change the API soon so that instead of a number of availability zones users should define specifically which ones a cluster should use, then we should go for a more extensible API design that allows to prevent breaking changes

No this won't be the case.


Show separate traces for different GPUs

When browsing a perfetto trace when multiple GPUs are being considered, I see the runtime activity separated by the different threads but the GPU activity is condensed in a single trace. I think it would be preferable to have a different trace per GPU (similar to rocprof). Otherwise the the activities show truncated and is not as insightful as it could be.
Maybe there is an option to enable that that I missed? I am running the trace collection as:
omnitrace-sample --trace --wait 55 --duration 10 -c $wd/omnitrace.cfg -- python -u ...

I'm actually working on that right now. Should have a PR before the end of the day

FYI, it's an extension on top of the changes in #201 -- there will be a row per-stream + per-device.


i cant scan the card number without steel seal

if it because of the card has 19 number?

@sparkdreamstudio I do not understand what you mean by "steel seal".
But yes, if this card has 19 digits that would partially explain your problem.
card.io is designed to scan traditional embossed cards (which yours is not), of these 5 types: MasterCard, VISA, AmEx, Discover, JCB (which yours is not).


.Net: Auto Function Calling Bad Request Message Not Included in Tool Response

Describe the bug
We are using Open API style plugins with automatic function calling. When our APIs return 400 (Bad Request), we include descriptive messages in the response which can be very helpful in directing the LLM to self-correct mistakes (or more effectively prompt the user for additional clarification). This was a behavior which we observed in the SK V0 Stepwise Planer, but it appears that with automatic function calling, only the status code is included in the tool response. This means that the LLM will NOT see the message included with the bad request response, missing out on valuable details. See screenshot below.
To Reproduce
Steps to reproduce the behavior:

Import a plugin to the kernel (i.e. kernel.ImportPluginFromOpenApiAsync).
For the sake of repo, you can create simple API that returns a bad request with a message.
Invoke chat completions with automatic function calling. We happen to be using streaming chatCompletion.GetStreamingChatMessageContentsAsync(chatHistory, settings, kernel);
Observe the chat history after invocation. You will see that the bad request message is not included and generic error message has been logged in the tool response.

Expected behavior
We would expect that the full response (i.e. status code and response content/ message) would be included in the Tool response message.
Screenshots

Platform

OS: Windows
IDE: Visual Studio
Language: C#
Source: Nuget Package 1.3.1

Hi, I had the same issue. SK version 1.6. GPT-4, Auto Function calling and streaming.
I narrowed down the issue to (here)[https://github.com/microsoft/semantic-kernel/blob/7fd2ac3dad87b914eed037850593740b9e2e1685/dotnet/src/Connectors/Connectors.OpenAI/AzureSdk/ClientCore.cs#L1012] where the Id, Name, and Arguments properties could not be found. This was because I persisted the chat history into CosmosDB and then recreated the history every time. I selected "CamelCase" naming conversation which broke this parsing.
To fix this, I parsed the JsonObject myself which skipped the code linked above.
It would be good to make this parsing case insensitive if possible.

Thanks for sharing. We're currently updating our filters so that they can allow developers to override / customize the error and cancelation messages for the LLM. We expect this to land in the next week or two. At that time, we'll provide documentation on how you can address this need.

We're currently updating our filters so that they can allow developers to override / customize the error and cancelation messages for the LLM.

PR for this functionality is here: https://github.com/microsoft/semantic-kernel/pull/5671


Remove imported and not used time

Fix Quote history example, current it fails.
$ go run main.go
# command-line-arguments
./main.go:5:2: imported and not used: "time"

Codecov Report

Merging #12 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   89.69%   89.69%
=======================================
  Files          10       10
  Lines         291      291
=======================================
  Hits          261      261
  Misses         15       15
  Partials       15       15

Continue to review full report at Codecov.

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


Can I use bluetooth in the chroot?

Is there a way so i can use the bluetooth in the chroot?

What kind of bluetooth device?  Audio and input devices are already handled by Chromium OS; setting them up there should make them work in the chroot.

I want to connect my phone for data transfer.
On Oct 5, 2015 10:17 PM, "David Schneider"<EMAIL_ADDRESS>wrote:

What kind of bluetooth device? Audio and input devices are already handled
by Chromium OS; setting them up there should make them work in the chroot.
—
Reply to this email directly or view it on GitHub
https://github.com/dnschneid/crouton/issues/2154#issuecomment-145730927.

I have a similar question,  but in my case I want to use the blue-z library to connect to a Mbientlab Metawear board to do some development on it.

im trying to play audio inside crouton using my phone so using it as a bluetooth reciever...is this possible


[SPARK-27216][CORE][BACKPORT-2.4] Upgrade RoaringBitmap to 0.7.45 to fix Kryo unsafe ser/dser issue

What changes were proposed in this pull request?
Back-port of #24264 to branch-2.4.
HighlyCompressedMapStatus uses RoaringBitmap to record the empty blocks. But RoaringBitmap couldn't be ser/deser with unsafe KryoSerializer.
It's a bug of RoaringBitmap-0.5.11 and fixed in latest version.
How was this patch tested?
Add a UT

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Test build #4685 has started for PR 24290 at commit bce50c2.

Test build #4685 has finished for PR 24290 at commit bce50c2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged to 2.4


Can I use RedisSearch?

I am kinda new to Redis and surely new to RediSearch and, from what I am studying, using RediSearch is dependent on the client you are using.
As I couldn't find an example with RediSearch in the documentation, I wonder if is it possible to use RediSearch with this package?

It seems I can build the interface to use the modules using ModuleBase.
Do you know if anyone has created an interface for RediSearch?
it would be nice to have and we could also add this package to the RediSearch clients list

Yes, a module is the way to go.


Added new group header formatting sample

Q
A

Bug fix?
no

New sample?
yes

What's in this Pull Request?
Added new view formatting sample which can be used to customize the group header to remove column name from group header.

cool, thx @ganesh-sanap.

Thank you @VesaJuvonen!

Hey @ganesh-sanap that's great. But sadly it is not working for me.  The group header is "object object" for me.

Is it also possible to replace the count in brackets with a total? So for example I group by month and show the total of sales?
Thanks a lot in advance!

@1704eu What is the data type of column you are using for grouping? You have to adjust JSON based on column type.


Clarity On Subscriptions & Paths

Could you provide some clarity on how Subscription Paths work?

Are Paths available for all events or only http events?
If Paths are available for all events, are they optional?

Having Paths only work for http events is not my preference because it creates a confusing happenstance.  At the same time, Paths are just like Channels (a longstanding concept in event-driven architectures) and are useful for all events.
My preference would be to have Paths work for all Events, but having them be optionally included in the Subscription is interesting, since it reduces the requirement that people hardcode Paths into their logic, makes config simpler, and ultimately makes getting started easier.  As an application/architecture grows, allowing Paths to be optionally embraced later will help keep things organized, segregated and more governable.

Right now path parameter is required only for http subscription type. Any other subscriptions (custom event) doesn't require this parameter.
I'm marking this as a feature request.

Already done https://github.com/serverless/event-gateway/commit/311b2bfa0e8dba34a117ef6b0614de49cfc5e14f


New Feature: Spectrum Analyzer

I was working on a project and wanted to construct this- https://www.instructables.com/Raspberry-Pi-Spectrum-Analyzer-with-RGB-LED-Strip-/
This is probably one of your older projects.
I tried a lot to achieve this but was unsuccessful. Can someone probably help in creating this if it is possible using BiblioPixel?

There's already a similar animation here: https://github.com/ManiacalLabs/BiblioPixelAnimations/tree/master/BiblioPixelAnimations/matrix/spectrum


Image Component

Would be really interested in an a custom Image component. Sorta like Mantine does

Hi @Wamy-Dev ,
Sira is a design-system based on Tailwindcss which only focus on css styles now, I do not plan to realize same functions like Mantine (ex: controll image loading states...) after my reflection.
At same time, tailwindcss had some classes to control styles on img (like aspect-ratio, object Fit...) already.
So, image component will be considered as a preflight styles to add basic border-radius on img element in next release version.


Dependency "open" is flagged as critical severity vulnerability

This is a bug regarding a vulnerable dependency of this project called open.
vulnerability report
Someone posted a github issue in open's repo suggesting that this package opener is a good alternative.

thanks @fitzk
it's published here in v0.23.2


Fix build and enable tests

Removing the package-lock.json is temporary until https://github.com/npm/cli/pull/40 is merged and available (or amp-story/linter moved to a separate repo): it seems local dependencies and package-lock.json are not compatible, and since package-lock.json doesn't really work anyway this doesn't seem like much of a loss.
This PR also activates the amp-story/linter tests.

I keep losing this: documentation on how file: modules are supposed to work.


Minor corrections and short API validation how-to

Hi all,
some minor corrections for issues that came up during code generation. Also added a description how to validate the API description via Apimatic.
Sorry for creating a second pull request in such a short time, I am quite new to GitHub :smiley: .
Best regards
Jannis

👍


WeatherBitHelper API not working on Windows system

The sunrise and sunset times of the API not working for Windows systems but working for Mac.

Fixed it. Closing the issue. Added this as another branch.


SQL comment inludes next line if blank, causing syntax error

System information:

Operating system (distribution) and version Linux Debian/SID
DBeaver version Version <IP_ADDRESS>011011603
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Multiple queries in one editor separated only by empty lines.
SELECT * FROM foo

SELECT * FROM bar

Position cursor at the first query and hit [CTRL]+[ENTER]
Query is successfully executed
Add comment below the first query
SELECT * FROM foo
--

SELECT * FROM bar

Position cursor at the first query and hit [CTRL]+[ENTER]
Query fails:
SQL Error [42601]: ERROR: syntax error at or near "SELECT"
  Position: 24

Add yet another blank line after comment
SELECT * FROM foo
--

SELECT * FROM bar

Position cursor at the first query and hit [CTRL]+[ENTER]
Query is successfully executed.
Apparently the comment also masks the next blank line

thanks for the report

seems the issue is fixed in the scope of #10416
close this issue as duplicate


TimeStep classes not exported?

"If you don't mind add a fun gif or meme, but no pressure"

I know this feature was released just a few days ago so I thought I'd open this issue to help with its progress, although this could just be me overlooking something.
Instantiating a new LSTMTimeStep returns the error below, and brain.recurrent.LSTMTimeStep is undefined. I fixed this by exporting it in the compiled index.js, then instantiating with .default():
// Before

const net = new brain.recurrent.LSTMTimeStep.(options);

// const net = new brain.recurrent.LSTMTimeStep({
//             ^
// TypeError: brain.recurrent.LSTMTimeStep is not a constructor

// After

const net = new brain.recurrent.LSTMTimeStep.default(options);

// dist/index.js
var LSTMTimeStep = require('./dist/recurrent/lstm-time-step');
var brain = {
  crossValidate: crossValidate,
  likely: likely,
  lookup: lookup,
  NeuralNetwork: NeuralNetwork,
  NeuralNetworkGPU: NeuralNetworkGPU,
  TrainStream: TrainStream,
  recurrent: {
    RNN: RNN,
    LSTM: LSTM,
    GRU: GRU,
    LSTMTimeStep: LSTMTimeStep // <---
  },
  utilities: utilities
};

I'm sure that this problem would apply to the other "TimeStep" classes as well, but I've only tested it on RNNTimeStep which does indeed produce the same error.

How are you importing brain?

Using const brain = require('brain.js');

I get the same error, my code is:
const brain = require('brain.js');
const net = new brain.recurrent.LSTMTimeStep();
net.train([1,2,3]);
var output = net.run([1, 2]);  // 3
console.log(output);

error:
const net = new brain.recurrent.LSTMTimeStep();
            ^

TypeError: brain.recurrent.LSTMTimeStep is not a constructor

if I use const brain = require('brain.js').recurrent;
I get this error:
const net = new brain.recurrent.LSTMTimeStep();
                               ^

TypeError: Cannot read property 'LSTMTimeStep' of undefined

Released v1.2.1 to address this.  Ty for finding.


Update Wagtail documentation links

Fix 404 internationalisation docs link
Use wagtail.org domain
Always refer to the latest version of Wagtail in documentation links

Thanks for the review Sage!


[BUG] Unable to connect to PostgreSQL with self-signed SSL

Describe the bug
We have been setting the env variable PGSSLMODE to no-verify for connecting to our self-signed PostgreSQL database.
After upgrading from 1.4.1 to 1.4.10 we observed the following error:
2024-01-23 19:34:32 [ERROR]: ❌ [server]: Error during Data Source initialization: no pg_hba.conf entry for host "***", user "***", database "flowise", no encryption

My investigation pinpointed to this commit, which sets ssl: true/false.
Configuring the DATABASE_SSL to true yields:
2024-01-23 20:47:26 [ERROR]: ❌ [server]: Error during Data Source initialization: self-signed certificate in certificate chain
Error: self-signed certificate in certificate chain
    at TLSSocket.onConnectSecure (node:_tls_wrap:1659:34)
    at TLSSocket.emit (node:events:517:28)
    at TLSSocket._finishInit (node:_tls_wrap:1070:8)
    at ssl.onhandshakedone (node:_tls_wrap:856:12)

It seems like PGSSLMODE is now ignored.
To Reproduce
Steps to reproduce the behavior:

Set-up a postgresql with a self-signed SSL certificate
Configure the postgresql datasource with DATABASE_SSL=true
Configure PGSSLMODE=no-verify

Expected behavior
Successful datasource connection.
Screenshots
N/A
Flow
N/A
Setup

Installation: docker/k8s with flowiseai/flowise:1.4.10
Command: sh -ec "flowise start"
Flowise Version 1.4.10
OS: Linux
Browser N/A

Additional context
N/A

do we have PGSSLMODE env variable in Flowise?

do we have PGSSLMODE env variable in Flowise?

PGSSLMODE is read in by the pg npm library, and it cannot be combined when we add ssl to the connection (line 81 of the connection-parameters.js file in the pg library.
@HenryHengZJ  I'll make another PR for supporting self-signed certificates via an env var with the cert as a base64 string like we discussed in the original PR, following these guidelines.
@efthymiosh Since you have a self signed Postgres server will you be able to help test this? The setup would be that there will be an env var DATABASE_SSL_KEY_BASE64 which will need to be the self-signed client cert as a base64 string. By passing that string in, the pg client will use ssl and connecting using the self-signed certificate.

@HenryHengZJ The approach I took was to add the support for proper self signed cert, which takes priority over the simple ssl: true option (current implementation) and then defaults to adding nothing to the pg connection if neither are provided, allowing the usage of any PGSSLMODE used by the pg npm library, including the unrecommended no-verify option mentioned above and in other issues/discussions.
@efthymiosh This PR should allow for you to continue using PGSSLMODE=no-verify without having to add anything else to your env file.

Hi @HenryHengZJ
Is this issue still open? When trying to connect to PostgreSQL:
[ERROR]: [server]: Error: error: no pg_hba.conf entry for host "", user "", database ")", SSL off
Error: error: no pg_hba.conf entry for host "", user "", database ")", SSL off
at buildLangchain ($HOME/data-talk-flowise/node_modules/flowise/dist/utils/index.js:331:19)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async App.buildChatflow ($HOME/data-talk-flowise/node_modules/flowise/dist/index.js:1496:40)
at async $HOME/data-talk-flowise/node_modules/flowise/dist/index.js:1054:13
Did add:
SHOW hba_file;
hba_file
$HOME/Library/Application Support/Postgres/var-16/pg_hba.conf:
Added:
TYPE DATABASE USER CIDR-ADDRESS  METHOD
host  all  all <IP_ADDRESS>/0 scram-sha-256
In postgresql.conf:
listen_addresses = '*'
port = 5432
restarted postgreSQL

Hi @HenryHengZJ
Is this issue still open? When trying to connect to PostgreSQL:
[ERROR]: [server]: Error: error: no pg_hba.conf entry for host "", user "", database ")", SSL off Error: error: no pg_hba.conf entry for host "", user "", database ")", SSL off at buildLangchain ($HOME/data-talk-flowise/node_modules/flowise/dist/utils/index.js:331:19) at process.processTicksAndRejections (node:internal/process/task_queues:95:5) at async App.buildChatflow ($HOME/data-talk-flowise/node_modules/flowise/dist/index.js:1496:40) at async $HOME/data-talk-flowise/node_modules/flowise/dist/index.js:1054:13
Did add:
show hba_file;
hba_file
$HOME/Library/Application Support/Postgres/var-16/pg_hba.conf:
Added:
TYPE DATABASE USER CIDR-ADDRESS METHOD host all all <IP_ADDRESS>/0 scram-sha-256
In postgresql.conf:
listen_addresses = '*' port = 5432
restarted postgreSQL

The issue you're encountering is with the Sql Database Chain node whereas this issue is related to Postgres for hosting your flowise data. The Sql Database Chain uses langchain/sql_db to connect to Postgres. While that package does support connecting using ssl and a self-signed ca, there is currently nothing in the node that would allow you to add your cert to the connection. However, it is using typeorm's PostgresConnectionOptions, which defaults to using the pg library, which does afford you the use of the env variable PGSSLMODE. You can view the options here: https://github.com/brianc/node-postgres/blob/master/packages/pg/lib/connection-parameters.js
Hope that helps

Had the same issue and finally fixed it. I am using an AWS RDS and had to add a configuration group in order to override the ssl settings. See https://stackoverflow.com/a/77787495 But the important part is setting the correct configuration for the postgres vector node. Turns out it uses typeorm under the hood. Therefore you have to provide the correct configuration (see screenshot).

After this the issue was resolved.


PostgeSQL procedures: Omit header should be on for procedure under debug

"Omit header" checkbox should be on for procedure under debug to sync line numbers with server repsresentation

Now "Omit header" is on for the most cases, let's track defects with separate tickets


[DOCFIX] fix cn doc Running-Alluxio-On-Docker.md

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19356/
Test PASSed.

Merged build finished. Test PASSed.