Need to be able to use custom HK2 ServiceLocator for Spring integration

We have been one of the early adopters of DW and been using it in production for over 3 years.
However, we are stuck on 0.7 and cannot move to a higher version.
The reason for this is the ******** HK2 DI that Jersey 2 introduced (and DW adopted in 0.8). It just refuses to play nice with the our Spring context that we use everywhere for DI.
We've had 2 senior architects spend 3 weeks trying to get it to work and HK2 fails to play nicely with Spring no matter what. It wants to @Inject its own beans into our JAX-RS classes, even though those are existing Spring beans and have already been injected with existing Spring beans.
The only solution seems to write a custom HK2 service locator that just fetches the beans directly from Spring and hope that works.
However, Dropwizard seems to offer no API to allow defining a custom ServiceLocator in HK2 to override the default one.
If we cannot get this to work, we cannot be stuck on 0.7 and will be faced with the difficult choice of moving to a different platform...which we would hate, since we all love Dropwizard.
Help please :-/

Thank you for being such a long time user of Dropwizard! Stories of such usage is very heartwarming.
Forewarning, I'm not expert in HK2 or Spring, but it looks like the dropwizard-guice project accomplishes what you want (ob. for guice not Spring). I'd recommend examining the project. These lines and these lines should be a good start 😃

I had a look at it and frankly speaking, it's not very clear how he accomplishes it (since I am not a Guice expert either).
From what I read on HK2 at the core of its is the ServiceLocator object. It seems that if we were able to implement our own version of that interface and just configure HK2 within Jersey to use it (instead of the one created by default by Dropwizard), that should do the trick.
However, there seems to be no way to customize this within the current DW API....

Basically, all I need to tell that bloody HK2 (which I curse Sun every day for introducing it and ruining Jersey for everybody) is to fetch an existing Spring bean from our existing context instead of trying to create it itself and inject it.

In DW 0.8 I am using Spring to @Inject into my jax-rs classes without
problems. I have written a Spring bundle that does most of the heavy
lifting. HK2 does not get in the way. I am planning a blog post on this
soon. Feel free to ask if you have any specific questions.
Den torsdag 22. september 2016 skrev Jacek Furmankiewicz <
<EMAIL_ADDRESS>følgende:

We have been one of the early adopters of DW and been using it in
production for over 3 years.
However, we are stuck on 0.7 and cannot move to a higher version.
The reason for this is the ******** HK2 DI that Jersey 2 introduced (and
DW adopted in 0.8). It just refuses to play nice with the our Spring
context that we use everywhere for DI.
We've had 2 senior architects spend 3 weeks trying to get it to work and
HK2 fails to play nicely with Spring no matter what. It wants to @Inject
https://github.com/Inject its own beans into our JAX-RS classes, even
though those are existing Spring beans and have already been injected with
existing Spring beans.
The only solution seems to write a custom HK2 service locator that just
fetches the beans directly from Spring and hope that works.
However, Dropwizard seems to offer no API to allow defining a custom
ServiceLocator in HK2 to override the default one.
If we cannot get this to work, we cannot be stuck on 0.7 and will be faced
with the difficult choice of moving to a different platform...which we
would hate, since we all love Dropwizard.
Help please :-/
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dropwizard/dropwizard/issues/1736, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFP3oeE9YUjSRu7zPls5notAKFUtH1Sks5qspvDgaJpZM4KEA7s
.

We have the opposite. All of our classes (including JAX-RS resources) were Spring beans from the beginning and we @Inject other Spring beans (e.g. DAOs) into them.
Where the bloody HK2 gets in the way is that now wants to also implement the @Inject itself (even though Spring has already done it), so HK2 gives an an error like "unable to find bean of type SomeEntityDao" (as that bean is a Spring bean, not an HK2 bean).
Basically, HK2 seems to want to manage JAX-RS resources by itself and does not handle the case at all where it's already part of another DI engine context.
This worked perfectly fine and was working great prior to Jersey 2, we had a DW app and Spring to manage all the beans inside it. Then came Jersey 2 with HK2 and ruined everything.

All my resources are managed by Spring through a @Component annotation an
package scanning. They have services injected which themselves have daos
injected etc. Works fine, never had hk2 getting in the way
Den torsdag 22. september 2016 skrev Jacek Furmankiewicz <
<EMAIL_ADDRESS>følgende:

We have the opposite. All of our classes (including JAX-RS resources) were
Spring beans from the beginning and we @Inject https://github.com/Inject
other Spring beans (e.g. DAOs) into them.
Where the bloody HK2 gets in the way is that now wants to also implement
the @Inject https://github.com/Inject itself (even though Spring has
already done it), so HK2 gives an an error like "unable to find bean of
type SomeEntityDao" (as that bean is a Spring bean, not an HK2 bean).
Basically, HK2 seems to want to manage JAX-RS resources by itself and does
not handle the case at all where it's already part of another DI engine
context.
This worked perfectly fine and was working great prior to Jersey 2, we had
a DW app and Spring to manage all the beans inside it. Then came Jersey 2
with HK2 and ruined everything.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dropwizard/dropwizard/issues/1736#issuecomment-249011334,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFP3my1AVjEDtSZEyNHjr-bxa58__dRks5qst4UgaJpZM4KEA7s
.

Are your JAX-RS resources Spring beans as well? When you have @Inject on those, who does the injection? HK2 or Spring?
Every time HK2 sees an @Inject in a JAX-RS resource, it wants to inject that resource itself. We can't stop it from doing that.
Simple example;
``
@Service    // Spring Service bean itself
@Path("/mealService")
@Produces(MediaType.APPLICATION_JSON)
@Consumes(MediaType.APPLICATION_FORM_URLENCODED)
public class MealServiceResource {
 @Inject
 private SomeSpringDao dao;

``

Yes they are, Spring does the injection. Have you tried constructor
injection? I am not using field injection the way you do, otherwise my
resources  are similar to your example
Den torsdag 22. september 2016 skrev Jacek Furmankiewicz <
<EMAIL_ADDRESS>følgende:

Are your JAX-RS resources Spring beans as well? When you have @Inject
https://github.com/Inject on those, who does the injection? HK2 or
Spring?
Every time HK2 sees an @Inject https://github.com/Inject in a JAX-RS
resource, it wants to inject that resource itself. We can't stop it from
doing that.
Simple example;
``
@Service https://github.com/Service // Spring Service bean itself
@Path https://github.com/Path("/mealService")
@Produces(MediaType.APPLICATION_JSON)
@Consumes(MediaType.APPLICATION_FORM_URLENCODED)
public class MealServiceResource {
@Inject
private SomeSpringDao dao;
``
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dropwizard/dropwizard/issues/1736#issuecomment-249014171,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFP3rTKWF9Sfw5dbvXLmBeHcgmH5F-eks5qsuDIgaJpZM4KEA7s
.

So how do you stop HK2 from trying to @Inject inject that SomeSpringDao bean?
Switching to contructor injection would be ugly and require rewriting hundreds of JAX-RS resources across multiple apps that use the same base framework....

I don't stop it. It never tries to.
Sendt fra min iPad

Den 22. sep. 2016 kl. 22.22 skrev Jacek Furmankiewicz<EMAIL_ADDRESS>So how do you stop HK2 from trying to @Inject inject that SomeSpringDao bean?
Switching to contructor injection would be ugly and require rewriting hundreds of JAX-RS resources across multiple apps that use the same base framework....
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Ah, sorry, just read your comment. So you do not use @Inject for Spring beans on JAX-RS resources, versus we do.
So your solution would not work for us.

Yes I do
@component on jax-rs resource, constructor injection and getting beans with
annotation @path from app context and registrering these with dw
Den torsdag 22. september 2016 skrev Jacek Furmankiewicz <
<EMAIL_ADDRESS>følgende:

Ah, sorry, just read your comment. So you do not use @Inject
https://github.com/Inject for Spring beans on JAX-RS resources, versus
we do.
So your solution would not work for us.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dropwizard/dropwizard/issues/1736#issuecomment-249018218,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFP3gU3Y9-ZnH2U2RKtZkW7GbkW6nSVks5qsuS4gaJpZM4KEA7s
.

Well, I am glad it works, but we do not use constructor injection anywhere in our, we use @Inject everywhere for consistency (and the code has much less boilerplate).
A cleaner IMHO solution would still be to drive HK2 with a pass-through ServiceLocator that just gives it any bean it wants from an already existing Spring context.
But we need DW to give us the ability to override the ServiveLocator within Jersey, which right now the API does not allow us to.

Just to be clear, i do use  @inject on the constructor, but if you cannot
change that i guess you are out of options  for the time being. spring boot
next?
Den torsdag 22. september 2016 skrev Jacek Furmankiewicz <
<EMAIL_ADDRESS>følgende:

Well, I am glad it works, but we do not use constructor injection anywhere
in our, we use @Inject https://github.com/Inject everywhere for
consistency (and the code has much less boilerplate).
A cleaner IMHO solution would still be to drive HK2 with a pass-through
ServiceLocator that just gives it any bean it wants from an already
existing Spring context.
But we need DW to give us the ability to override the ServiveLocator
within Jersey, which right now the API does not allow us to.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dropwizard/dropwizard/issues/1736#issuecomment-249019855,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFP3hE2wv2AQahlSwRXGOwHWf0---Emks5qsuY4gaJpZM4KEA7s
.

Hence, the initial request in this ticket.
HK2 allows you to use your own ServiceLocator, it's part of its design:
https://hk2.java.net/2.5.0-b07/apidocs/org/glassfish/hk2/api/ServiceLocator.html
They provide a default implementation:
https://hk2.java.net/2.5.0-b16/getting-started.html
but you are supposed to be able to use your own.
Unfortunately, Dropwizard does not allow us to use our own, as it initializes Jersey and HK2 without any hooks to allow injecting your own ServiceLocator.
That is what we would love to get fixed.

Have you tried looking at Spring/HK2 Bridge, like how dropwizard-guicey uses guice-bridge in their GuiceFeature to modify the service locator? It certainly looks like one can inject their own service locator.

Hm we tried it before, a few months ago, but it did not seem to work for us.
According to the docs, it was supposed to just pick it up if it was in the classpath.
The docs you are pointing to seem to have some extra instructions on how to wire the two together.
Let me try that and see if that works. Maybe that could do it.
Will try and let you know if we hit any snags or unknowns. HK2 is somewhat of an undocumented beast.

I see the same behaviour as you describe with field injection, However, if
I change @Inject to @Autowired,  field injection also works without H2K
getting in the way. Maybe an alternative if the bridge approach does not
work.
On 22 September 2016 at 23:02, Jacek Furmankiewicz<EMAIL_ADDRESS>
wrote:

Hm we tried it before, a few months ago, but it did not seem to work for
us.
According to the docs, it was supposed to just pick it up if it was in the
classpath.
The docs you are pointing to seem to have some extra instructions on how
to wire the two together.
Let me try that and see if that works. Maybe that could do it.
Will try and let you know if we hit any snags or unknowns. HK2 is somewhat
of an undocumented beast.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dropwizard/dropwizard/issues/1736#issuecomment-249026865,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFP3lvv55Onn6FAU9BuQ4bV-pi6p32nks5qsuzlgaJpZM4KEA7s
.

Has this been resolved?

Sorry, have not had a chance to look at it again, other work-related issues swamped me.
Hope to have another go at a 1.0 attempt within a month

Hi Nick, unfortunately we cannot get the Spring/HK2 bridge to work.
The reason for that is that the ServiceLocator does not seem to be initialized yet when the run() is executed.
Here is my code in the run() that is executed after our Spring context starts:
        ServletContainer sc = (ServletContainer) environment.getJerseyServletContainer();
        ApplicationHandler ah = sc.getApplicationHandler();
        ServiceLocator hk2 = ah.getServiceLocator();

It fails with a NullPointerException on
        ApplicationHandler ah = sc.getApplicationHandler();

when I debug the code I see in Dropwiizard"
    @Override
    public ApplicationHandler getApplicationHandler() {
        return webComponent.appHandler;
    }

but unfortunately...webComponent = null at this point so it throws an NPE.
Any suggestions are welcome...how can I get a valid instance of the HK2 ServiceLocator at startup.

Finally got it working, but it was a trip. Probably worth documenting in a Wiki or the webpage, so that everyone does not have to go through this obscure discovery process.
First, we need to define a Jersey Feature that is also a ServiceLocator provider and in there we enable the Spring/HK2 bridge on the alread created service locator:
import com.google.common.base.Preconditions;
import org.glassfish.hk2.api.ServiceLocator;
import org.glassfish.jersey.ServiceLocatorProvider;
import org.jvnet.hk2.internal.ServiceLocatorImpl;
import org.jvnet.hk2.spring.bridge.api.SpringBridge;
import org.jvnet.hk2.spring.bridge.api.SpringIntoHK2Bridge;
import org.springframework.context.annotation.AnnotationConfigApplicationContext;

import javax.inject.Provider;
import javax.ws.rs.core.Feature;
import javax.ws.rs.core.FeatureContext;

/**
 * Integrates the default Jersey HK2 ServiceLocator with Spring via the
 * Spring/HK2 bridge from Oracle
 *
 * @author Jacek Furmankewicz
 */
public class SpringAwareHK2ServiceLocatorFeature implements Feature, Provider<ServiceLocator> {

    private AnnotationConfigApplicationContext springContext;
    private ServiceLocator locator;

    public SpringAwareHK2ServiceLocatorFeature(AnnotationConfigApplicationContext springContext) {
        this.springContext = springContext;
    }

    @Override
    public ServiceLocator get() {
        return Preconditions.checkNotNull(locator, "Service locator is not yet available");
    }

    @Override
    public boolean configure(FeatureContext context) {
        locator =  ServiceLocatorProvider.getServiceLocator(context);

        SpringBridge.getSpringBridge().initializeSpringBridge(locator);
        SpringIntoHK2Bridge springBridge = locator.getService(SpringIntoHK2Bridge.class);
        springBridge.bridgeSpringBeanFactory(springContext.getBeanFactory());

        return true;
    }
}

Then we need to register it manually at startup, already initialzed with an existing Spring context:
        // hook up Spring context and Jersey HK2 together
        environment.jersey().register(new SpringAwareHK2ServiceLocatorFeature(context));

That finally did it, without any further modifications to Dropwizard.
Now Jersey/HK2 properly gets Spring beans and does not complain that they don't exist.
Thanks for all your help. The key code was in the Guice / Dropwizard module and I adapted it from there for Spring.
Once again, I think it's worthwhile adding a  "Dropwizard & Spring" section on the main site and documenting this there.

@jacek99 Thank you so much for documenting this!
We'd love to include this into the documentation. Do you have time to create a small PR for it?

Sure, just point you to where i could do this.

@jacek99 It would be great if you could include a paragraph in the dropwizard-core documentation at https://github.com/dropwizard/dropwizard/blob/master/docs/source/manual/core.rst

Sounds good. Will try to do this one of the evenings this week.

Any particular section you think it belongs to?
Or just add it to the end maybe as a separate section like

Integration with 3rd party libraries
** Spring

etc

@jacek99 Yes, please just add a separate section at the end of the file.

The reason @Component and @Autowired work is that the Spring/Jersey library specifically looks for those and thunks them in. If you actually use Spring without classpath scanning or @Component or old-school auto-wired, you have to provide your own service locator.

Closing this issue as it seems to have gone stale. We'd love to see Spring interop documented but we're not avid spring users so the community will need to step up. Jersey 2.26 may have changed the interop scenario as well. If discussion continues on this issue, we're happy to open it back up.


Update client wrapper

https://keboola.atlassian.net/browse/PS-3098
Depends on:

https://github.com/keboola/storage-api-php-client-branch-wrapper/pull/9
https://github.com/keboola/input-mapping/pull/117
https://github.com/keboola/output-mapping/pull/109
https://github.com/keboola/staging-provider/pull/17

To je slusny dependency hell


Error while creating default machine by start.sh

System: Windows 8.1 Pro
Docker Toolbox: 1.8.2
Docker Machine: 0.4.1
VirtualBox (installed by Docker Toolbox): 5.0.4
Git (installed by Docker Toolbox): 2.5.0
Run start.sh and get
Creating Machine default...
Creating VirtualBox VM...
Creating SSH key...
Starting VirtualBox VM...
  Error creating machine: exit status 1
You will want to check the provider to make sure the machine and associated resources were properly removed.
Looks like something went wrong... Press any key to continue...

VBoxStartup.log: https://gist.github.com/mif007/7ac45b8e13d225cddc06

I had a similar issue I fixed by deinstalling virtualbox and installing the latest Virtualbox testbuild. See https://github.com/docker/machine/issues/1716 for a detailed description.

Testbuild didn't help. And now virtualbox crashes when launched

It's a duplicate of other issues which have to do with VirtualBox start -- e.g. https://github.com/docker/machine/issues/2047 please post info there.


CSS

**User Story
As a boot camp student
I want the pre-work notes to be structured on a webpage
So that I can easily find and read the information
**Acceptance Criteria
GIVEN a pre-work Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Glayout LLM + Docs + Notebooks

Documentation has been updated to include links to Glayout notebooks
Added the Via notebook
All notebooks have been fixed (they broke after we refactored the Glayout package)

Glayout LLM v1
We are getting decent results with the 7b model. It is highly versatile on 2-4 transistor circuits. Current model config:

8 bit quantization
Computing loss on prompt completions only
Context is included in prompts
padding with [PAD] token
Current training dataset: 32 unique circuits, 54 examples
Current evaluation dataset: 2 unique circuits (currently being expanded)

Please add my comments as issues to fix later.


Create a beautiful page for users to receive a token for their app

#17 partially fixed this.

It's nice enough.


ADAPT-1256: Close Default Loophole for fields - By Abhishek & Aman

@irynabiletskaps, @mnikulesko, The failing unit test has passed. Could you please review again?

Hi @irynabiletskaps, @mnikulesko, Could you please review this again? Thank you.


Add color support for gen 3 A19 bulbs

It seems that the new model name of the third generation LIFX bulbs is LIFX A19.
Just added the string at the relevant places where the color characteristic is set.
Reference to #16

Closing this PR, as I have addressed it in version 0.1.8


marquee added with the recent changes

Included the changes from the main branch and resolved the conflicts .
added the marquee .

@anuragverma108 could you please review the changes  and merge the PR

Please note that this is not in line with our code of conduct. To ensure clarity and proper assignment tracking, please attach the issue number to your PR description. This helps us confirm that the issue is assigned to you. Kindly update your PR with the issue number and resubmit.
Example:
Fixes #123 - Brief description of the fix.

@rahulkhandait-sde #227 was the issue that I opened up a week ago then @anuragverma108 asked me to make changes and closed the issue today . So do I need to create a new issue for this and then raise a PR .
Could you please guide me .


[AWS/S3] AssumeRoleProvider credentials limited to 15m

With some assume-role EKS configurations, assumed credentials are limited to 15m.  This caps lifetime of presigned URLs to the same value.  Allow configuring credential lifetimes.
Analysis
The actual provider used is ProviderName:AssumeRoleProvider.  I verified this by emitting the ProviderName field of credentials in s3Client.Refresh.

@Isan-Rivkin you were right all along, the relevant IRSA configuration ends up assuming a different role.  That role is configured using something like this in ~/.aws/credentials:
[client]
source_profile = lakefs-runtime-profile
role_session_name = "23571113171923"
role_arn = arn:aws:iam::123456:role/sample-repo-role-us-east-01020304050607
external_id = f00-b4r-b4z-quux

(nothing secret there but most strings elided, of course).

According to the Boto configuration guide, we can probably do this by setting up ~/.aws/config to be something like
[client]
duration_seconds = 43200

Done, this works!


Multiple speakers at the same Proposal

It should be nice if we have Two speakers at the same proposal.
This reflects on the following pages:

Models
Event Detail page
Scheduled embedded page

Maybe even more pages.

Hi @luanfonceca, can I get this issue?

Hi, @victorhos, are you actually working in this issue?


fix interpretation of default bytes values

Fixes #426

@dsnet or @cybrcodr: any chance this can get a review?

Latest changes LGTM. Should I submit?

@dsnet, if LGTY, merge away!
Thanks for the quick review!


sidebar  bgcolor not change with class name

[ ] I have searched the Issues to see if this bug has already been reported)
[x] I have tested the latest version

<Sidebar collapsed={isCollapsed} className="bg-gray-400" >
    <Sidebar.Items className="bg-gray-400">
        <Sidebar.ItemGroup>
            <Sidebar.Item className="" href="#" icon={HiChartPie}  onClick={() => {toggleCollapse()}}>
                 Dashboard
            </Sidebar.Item>
            <Sidebar.Item href="#" icon={HiViewBoards} >
                Kanban
            </Sidebar.Item>
            <Sidebar.Item href="#" icon={HiInbox} >
                Inbox
            </Sidebar.Item>
            <Sidebar.Item href="#" icon={HiUser}>
                Users
            </Sidebar.Item>
            <Sidebar.Item href="#" icon={HiShoppingBag}>
                Products
            </Sidebar.Item>
            <Sidebar.Item href="#" icon={HiArrowSmRight}>
                Sign In
            </Sidebar.Item>
            <Sidebar.Item href="#" icon={HiTable}>
                Sign Up
            </Sidebar.Item>
        </Sidebar.ItemGroup>
    </Sidebar.Items>
</Sidebar>

Steps to reproduce

Go to '...'
Click on '....'
Scroll down to '....'

Current behavior

Describe what is currently happening and why it's a problem.
Expected behavior
color set in side bar

expected
Context
What are you trying to accomplish? Does this only happen on a specific browser, screen size, or operating system?
If possible, provide a live example URL, screenshot, video, or a repository with the minimal reproduction of the issue.

Have you checked whether the CSS bg-gray-400 is applied using developer tools? Share your tailwind.config.js - my guess is you are missing step 2 of the install guide https://github.com/themesberg/flowbite-react#install-flowbite-react

Have you checked whether the CSS bg-gray-400 is applied using developer tools? Share your tailwind.config.js - my guess is you are missing step 2 of the install guide https://github.com/themesberg/flowbite-react#install-flowbite-react

install properly   apply but not chage background color becuase defult color not remove

@nishant-vagh The best approach to customize complex elements such as Sidebar is to use the theme system. Some of the classes/styles that you are trying to change are part of the inner component's objects.

@nishant-vagh The best approach to customize complex elements such as Sidebar is to use the theme system. Some of the classes/styles that you are trying to change are part of the inner component's objects.

<Sidebar.Items className="bg-blue-500">
<Sidebar.ItemGroup className="bg-blue-500" >
<Sidebar.Item className="bg-blue-500" href="#" icon={HiChartPie}  onClick={() => {toggleCollapse()}}>
Dashboard
</Sidebar.Item>
<Sidebar.Item href="#" icon={HiViewBoards} >
Kanban
</Sidebar.Item>
<Sidebar.Item href="#" icon={HiInbox} >
Inbox
</Sidebar.Item>
<Sidebar.Item href="#" icon={HiUser}>
Users
</Sidebar.Item>
<Sidebar.Item href="#" icon={HiShoppingBag}>
Products
</Sidebar.Item>
<Sidebar.Item href="#" icon={HiArrowSmRight}>
Sign In
</Sidebar.Item>
<Sidebar.Item href="#" icon={HiTable}>
Sign Up
</Sidebar.Item>
</Sidebar.ItemGroup>
</Sidebar.Items>

@nishant-vagh The best approach to customize complex elements such as Sidebar is to use the theme system. Some of the classes/styles that you are trying to change are part of the inner component's objects.

i have also try but

@nishant-vagh The best approach to customize complex elements such as Sidebar is to use the theme system. Some of the classes/styles that you are trying to change are part of the inner component's objects.

Have you tried to use this approach? https://www.flowbite-react.com/docs/customize/theme

Have you tried to use this approach? https://www.flowbite-react.com/docs/customize/theme#option-3-create-a-reusable-component-with-a-custom-theme

yes i have try it
but  still not  work

@nishant-vagh could you please create an example at codesandbox so someone can investigate the issue?

@nishant-vagh could you please create an example at codesandbox so someone can investigate the issue?

https://github.com/niks200/sandbox.git

Your sandbox isn't accessible. Let me know if you can still reproduce this.

Your sandbox isn't accessible. Let me know if you can still reproduce this.
https://codesandbox.io/p/github/niks200/sandbox/main?layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522clq3mq1yf0008356irzjfqcu5%2522%252C%2522sizes%2522%253A%255B70.63979525302304%252C29.36020474697696%255D%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522EDITOR%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522id%2522%253A%2522clq3mq1yf0002356igfb08ozs%2522%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522SHELLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522id%2522%253A%2522clq3mq1yf0005356ivg6vf1w5%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522DEVTOOLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522id%2522%253A%2522clq3mq1yf0007356ifd1eg1wx%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%252C%2522sizes%2522%253A%255B50%252C50%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522clq3mq1yf0002356igfb08ozs%2522%253A%257B%2522id%2522%253A%2522clq3mq1yf0002356igfb08ozs%2522%252C%2522tabs%2522%253A%255B%255D%257D%252C%2522clq3mq1yf0007356ifd1eg1wx%2522%253A%257B%2522id%2522%253A%2522clq3mq1yf0007356ifd1eg1wx%2522%252C%2522activeTabId%2522%253A%2522clqw6wi89004e356i54rpzd35%2522%252C%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clq3mq1yf0006356ix3898s3s%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522TASK_PORT%2522%252C%2522taskId%2522%253A%2522dev%2522%252C%2522port%2522%253A5173%252C%2522path%2522%253A%2522%2522%257D%252C%257B%2522type%2522%253A%2522SETUP_TASKS%2522%252C%2522id%2522%253A%2522clqw6wi89004e356i54rpzd35%2522%252C%2522mode%2522%253A%2522permanent%2522%257D%255D%257D%252C%2522clq3mq1yf0005356ivg6vf1w5%2522%253A%257B%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clq3mq1yf0003356ijb7ngadz%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522TASK_LOG%2522%252C%2522taskId%2522%253A%2522dev%2522%257D%252C%257B%2522id%2522%253A%2522clq3mq1yf0004356iwn5nm0ee%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522TERMINAL%2522%252C%2522shellId%2522%253A%2522clq3lnf7o0012egfuhdjp3epd%2522%257D%255D%252C%2522id%2522%253A%2522clq3mq1yf0005356ivg6vf1w5%2522%252C%2522activeTabId%2522%253A%2522clq3mq1yf0003356ijb7ngadz%2522%257D%257D%252C%2522showDevtools%2522%253Atrue%252C%2522showShells%2522%253Atrue%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D

It looks like all you need to do is apply the color to the root.inner theme specifically, since a bg-gray-50 is currently supplied there.
That looks like:
import { Sidebar, theme } from 'flowbite-react';
import { twMerge } from 'tailwind-merge';

<Sidebar theme={{
  inner: twMerge(theme.sidebar.root.inner, "bg-pink-800")
}}>
 ..
</Sidebar>

This obviously isn't optimal - we should definitely consider moving all or most of the styling out of this inner key to make it more intuitive. E.g., you should be able to just do this with className without hacks. You can actually do it with className as well by specifying the inner <div> as the target - e.g., <Sidebar className="[&>div]:bg-pink-800">
I tested on your Code Sandbox and each of those works just fine.


Multiple Workspaces: Linting doesn't work with multi-root workspaces

Although it styles the code and performs the compilation, this extension doesn't seem to lint properly when using multi-root workspaces with either Solhint or Solium.
It works fine when you open a single project folder, just not multi-root

Yes, correct. All the extension works around a single workspace.

It seems the linter doesn't pickup .solhint.json configs inside a subdir. Is it configurable in the extension settings? It would be nice to let the extension pickup .solhint.json inside a subdir, or configure it where to look for .solhint.json in a monorepo.

same issue for me, I guess it is linked to this: https://forum.openzeppelin.com/t/trouble-importing-openzeppelin-contracts-into-vs-code/5230/7

Was anybody able to solve this ever? Quite annoying behavior...

nope

I am looking at it now

nice 😄 thanks!

It is checked in now.
@sebastiantf  regarding subdirectories and mono repos, if it is at root the file will work as usual (single workspace), although yes with multiple workspaces the root is in each workspace, that will need copy and paste. The solution will be to have an absolute location for the config file, which will add complications in the future for web support.


Crash with 'TypeLoadException'

System.TypeLoadException: 'Method 'add_IsEnabledChanged' in type 'SadRogue.Integration.Maps.RogueLikeMap' from assembly 'TheSadRogue.Integration, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have an implementation.'
When using the integration library (1.0.0Beta04-debug) on top of the current SadConsole example code (from the SadConsole GitHub repository, the project crashes out with the above exception at 'Game.Instance.Run()'

I believe this is an issue when the integration library is used with the v10 code because the current integration library version does not support v10.
This should be fixed when v1.0.0 is released (which will suppport v10), which should happen in a matter of days; we'll double check  at that time, then close this ticket as needed.

Closed with #76


failed to allocate 11.90G CUDA_ERROR_OUT_OF_MEMORY

When i try the WMT'16 EN-DE sample, encountered the following CUDA_ERROR_OUT_OF_MEMORY:
name: TITAN X (Pascal)
major: 6 minor: 1 memoryClockRate (GHz) 1.531
pciBusID 0000:01:00.0
Total memory: 11.90GiB
Free memory: 11.39GiB
I tensorflow/core/common_runtime/gpu/gpu_device.cc:906] DMA: 0
I tensorflow/core/common_runtime/gpu/gpu_device.cc:916] 0:   Y
I tensorflow/core/common_runtime/gpu/gpu_device.cc:975] Creating TensorFlow device (/gpu:0) -> (device: 0, name: TITAN X (Pascal), pci bus id: 0000:01:00.0)
E tensorflow/stream_executor/cuda/cuda_driver.cc:1002] failed to allocate 11.90G<PHONE_NUMBER>8 bytes) from device: CUDA_ERROR_OUT_OF_MEMORY
Traceback (most recent call last):
File "/usr/lib/python3.4/runpy.py", line 170, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.4/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/media/sbai/7A9C9BED9C9BA1E5/DL/seq2seq/bin/train.py", line 251, in
tf.app.run()
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/python/platform/app.py", line 44, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "/media/sbai/7A9C9BED9C9BA1E5/DL/seq2seq/bin/train.py", line 246, in main
schedule=FLAGS.schedule)
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/contrib/learn/python/learn/learn_runner.py", line 106, in run
return task()
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/contrib/learn/python/learn/experiment.py", line 459, in train_and_evaluate
self.train(delay_secs=0)
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/contrib/learn/python/learn/experiment.py", line 281, in train
monitors=self._train_monitors + extra_hooks)
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/python/util/deprecation.py", line 280, in new_func
return func(*args, **kwargs)
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py", line 426, in fit
loss = self._train_model(input_fn=input_fn, hooks=hooks)
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py", line 984, in _train_model
_, loss = mon_sess.run([model_fn_ops.train_op, model_fn_ops.loss])
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/python/training/monitored_session.py", line 462, in run
run_metadata=run_metadata)
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/python/training/monitored_session.py", line 786, in run
run_metadata=run_metadata)
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/python/training/monitored_session.py", line 744, in run
return self._sess.run(*args, **kwargs)
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/python/training/monitored_session.py", line 883, in run
feed_dict, options)
File "/home/sbai/tf134/lib/python3.4/site-packages/tensorflow/python/training/monitored_session.py", line 909, in _call_hook_before_run
request = hook.before_run(run_context)
File "/media/sbai/7A9C9BED9C9BA1E5/DL/seq2seq/seq2seq/training/hooks.py", line 239, in before_run
"predicted_tokens": self._pred_dict["predicted_tokens"],
KeyError: 'predicted_tokens'
Env: TF1.0 GPU & Python3.4 & ubuntu14.04
I changed the batch size and the num_units into a smaller number, but still encountered the same error.
I tried toy data, met the same error.
Is it because I am using python3.4?
############################### update ###############
I tried it on Python3.5, got the same error at the first try, and got following error when i tried again:
WARNING:tensorflow:From /usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/monitors.py:267: BaseMonitor.init (from tensorflow.contrib.learn.python.learn.monitors) is deprecated and will be removed after 2016-12-05.
Instructions for updating:
Monitors are deprecated. Please use tf.train.SessionRunHook.
*** Error in `python3.5': double free or corruption (!prev): 0x0000000002870d90 ***
Aborted (core dumped)

Same problem, but I solve it by ignoring the file "train_seq2seq.yml" and just using the "nmt_xxxx.yml"
I guess there is some trouble in the configuration about hooks, but I'm not sure.

Same problem, but I solve it by ignoring the file "train_seq2seq.yml" and just using the "nmt_xxxx.yml"
I guess there is some trouble in the configuration about hooks, but I'm not sure.

@DaoD Thanks for you reply!
However, I tried your method, still got CUDA_ERROR_OUT_OF_MEMORY.
Anyone succeeded to run the sample?
Could you tell us your configuration and env?

Yes the error will still happen but the training process could be continued.
I don't know the reason.

I read some other posts [1], [2] about this issue for other projects. It appears that such an error doesn't matter, and the GPU still gets used (just with some growth factor).
You can check whether your GPU is being used with the nvidia-smi command-line tool.
[1] http://stackoverflow.com/questions/39465503/cuda-error-out-of-memory-in-tensorflow
[2] https://github.com/tensorflow/tensorflow/issues/6048

I think this is a different error and not related to the GPU:
"predicted_tokens": self._pred_dict["predicted_tokens"],
KeyError: 'predicted_tokens'

This is probably the same as #43. I haven't been able to reproduce this and not sure what is wrong here.

I'm closing this one because it seems like a duplicate of #43 - please discuss there or re-open the issue if it's not a duplicate.

@dennybritz @papajohn thanks for your reply!
I updated the new seq2seq, got some new errors.
however, i  tried @DaoD 's method again, it still CUDA_ERROR_OUT_OF_MEMORY but the training process is continued.
new errors as following (python3.4):
I tensorflow/core/common_runtime/gpu/pool_allocator.cc:247] PoolAllocator: After 6810 get requests, put_count=8013 evicted_count=2000 eviction_rate=0.249594 and unsatisfied allocation rate=0.119824
I tensorflow/core/common_runtime/gpu/pool_allocator.cc:259] Raising pool_size_limit_ from 212 to 233
INFO:tensorflow:Performing full trace on next step.
I tensorflow/stream_executor/dso_loader.cc:126] Couldn't open CUDA library libcupti.so.8.0. LD_LIBRARY_PATH: :/usr/local/cuda/lib64:/usr/local/cudnn/lib64
F tensorflow/core/platform/default/gpu/cupti_wrapper.cc:59] Check failed: ::tensorflow::Status::OK() == (::tensorflow::Env::Default()->GetSymbolFromLibrary( GetDsoHandle(), kName, &f)) (OK vs. Not found: /home/sbai/tf134/lib/python3.4/site-packages/tensorflow/python/_pywrap_tensorflow.so: undefined symbol: cuptiActivityRegisterCallbacks)could not find cuptiActivityRegisterCallbacksin libcupti DSO
Aborted (core dumped)

I also run into this. Any solution?

Have you checked your operating processes on GPU ? It may arise by multiple opened environments that cause GPU overloaded. I recommend to close all terminals and open again. I hope this works for you.


Update worldmap.js to allow "flying" to new bounds

When the bounds command is used, give the option to "fly" (a Leaflet term meaning to animate the change of bounds: zoom out, pan, zoom in) to the new bounds rather than simply repositioning the map.  This is done very simply by providing a "fly" flag that, with a truthy value, calls map.flyToBounds instead of map.fitBounds.
There are 10 other places where map.fitBounds is used, but for my purposes, I only want to "fly" to the new bounds when I explicitly set the bounds through this command.  So perhaps these changes could/should be used in other scenarios too.

Interesting thought. I think for the wider usage - many of the others are triggered by a property fit being present - so maybe add fly as the option in those cases (rather than require both fit and fly).

I agree, that makes sense.  Is this something I should change in my patch...?  (It looks like I can still edit it.)
One possible issue I noticed just now is that one fitBounds call is immediately followed by a panTo call (which, now I think about it, seems like a strange sequence).
Testing might be an issue for me as most instances of fit are on features I don't use, but presumably there are automated tests you run before each release?

I've changed my patch and it now seems to be checking them for this pull request, so perhaps that answers my question!
As mentioned in the comment on the change I just made, it looks like a minor bug existed in a few places: it was checking whether fit was a property but then didn't check its value, so if someone used fit: false it would still fit it.  I've changed those to ensure fit is true.  I also changed my original patch so that fly needs to be true not just truthy, as that aligns better with the existing code.


Add all the strings/texts in the strings.en.yaml file and replace all the occurences with getString() in frontend

Currently, there are many text occurrences in UI which are not added to strings.en.yaml for translation. We need to identify them, add them to strings.en.yaml and replace the occurrences with getString('example') in chaoscenter/web.

@Saranya-jena
I would love to work on this issue.

@Saranya-jena can i work on this

.take


[Bug Report] DatePicker 'default-time'-attribute isnt working correctly

Element UI version
2.3.4
OS/Browsers version
Windows 10 / Chrome / Firefox
Vue version
2.5.17-beta.0
Reproduction Link
https://jsfiddle.net/7x4k1d16/2/
Steps to reproduce

set the 'default-time'-attribute
enter the daterange

(if the jsfiddle link isnt working just use the fiddle of your example under https://element.eleme.io/#/en-US/component/date-picker#default-time-for-start-date-and-end-date)
What is Expected?
e.g. :default-time="['00:00:00', '23:59:59']" will lead to [ "2018-04-04T00:00:00.000Z", "2018-04-05T23:59:59.000Z" ] when the daterange is set to '2018-04-04 - 2018-04-05'
What is actually happening?
The example will lead to [ "2018-04-03T22:00:00.000Z", "2018-04-05T21:59:59.000Z" ].
This seems to happen because the default-default-time is '22:00:00' and the difference of 24:00:00 to the set default-time is subtracted.

DatePicker always returns date in your time zone. https://jsfiddle.net/7x4k1d16/8/

Thank you. Forgot about that.


GitHub container registry typos

Typos of ghcr.io.
Tested with my test.py script.
I did GitHub code searches to confirm that people make all of these mistakes.
gcr.io (dropping the h), hcr.io (dropping the g) are both different container registries.
hrc.io - is a political org. It's unlikely to be intentionally in Enterprise code, but I didn't include it.

I need to generate the README.md for this one


Nothing is installed

Installation: npm install --global alfred-fkill. This runs and then absolutely nothing happens. No new workflow is installed, nothing changes on alfred, no .alfredworkflow file is generated.

Which version of Alfred are you using?

Alfred 3.6.2

This might be due to the fact that you need to list alfy as a required dependency.

Would it be possible to try this again? We made some changes recently to Alfy. Hope it resolves this issue.


[Improvement] Possible bug in BaseTable.java with proxyPlugin

What would you like to be improved?
proxyPlugin (line 68) hides the proxyPlugin field - is this intended?
How should we improve?
Rename the local variuable?

What would you like to be improved?
proxyPlugin (line 68) hides the proxyPlugin field - is this intended?
How should we improve?
Rename the local variuable?

I will rename the local variable.


translations:find -> Integrity constraint violation: 1062 Duplicate entry

running translations:find works great the first time I run it, but if I add new keys in my code and run it again I get this error
if I truncate the table and start fresh then it works fine again.. this seems related to the unique indexing on the table but.. how come it fails to see there are already items with the key "category" in the case below and tries to insert them anyway?
pa translations:find

   Illuminate\Database\QueryException  : SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'json-JSON-category' for key 'ixk_ltm_translations_locale_group_key' (SQL: insert into `ltm_translations` (`locale`, `group`, `key`, `value`, `saved_value`, `status`, `updated_at`, `created_at`) values (json, JSON, category, category, category, 1, 2018-10-21 13:24:58, 2018-10-21 13:24:58))

  at api.mydomain.com/vendor/laravel/framework/src/Illuminate/Database/Connection.php:664
    660|         // If an exception occurs when attempting to run a query, we'll format the error
    661|         // message to include the bindings with SQL, which will make this exception a
    662|         // lot more helpful to the developer instead of just the database's errors.
    663|         catch (Exception $e) {
  > 664|             throw new QueryException(
    665|                 $query, $this->prepareBindings($bindings), $e
    666|             );
    667|         }
    668|

  Exception trace:

  1   Doctrine\DBAL\Driver\PDOException::("SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'json-JSON-category' for key 'ixk_ltm_translations_locale_group_key'")
      api.mydomain.com/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php:144

  2   PDOException::("SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'json-JSON-category' for key 'ixk_ltm_translations_locale_group_key'")
      api.mydomain.com/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php:142

  Please use the argument -v to see more details.

@vesper8, this could be related to #123 if the keys have different case. The queries do case sensitive queries but the index is built as not case sensitive.
I will need to see how this can be added to a migration in database independent way.

in my case however all keys are always all lowercase.. so not sure if related to that.. hrmm.. will try to drill down further

ok so I've been debugging the issue and think I found out what's going on.. at least this should help you come up with a fix
so I'm tracing the process by adding a bunch of debugging inside the firstOrNewTranslation method
I run translations:find and it finds a key called "JSON.category" (which already exists and has already been translated)
it runs through firstOrNewTranslation
and it looks for an existing database entry with these attributes:
array (3) [
    'locale' => string (2) "en"
    'group' => string (4) "JSON"
    'key' => string (8) "category"
]

it correctly finds one.. and moves on
then it repeats this, this time it looks for these attributes:
array (6) [
    'locale' => string (4) "json"
    'group' => string (4) "JSON"
    'key' => string (8) "category"
    'value' => string (8) "category"
    'saved_value' => string (8) "category"
    'status' => integer 1
]

Here it will fail to find the existing entry since the "saved_value" in the database happens to be "Category" with a capital.. however this is the translated string so it could have been anything.. it could have been "A Category" for example.
It fails to find the existing entry, it then tries to create a new entry and runs into the integrity issue above
I feel that the firstOrNewTranslation should probably not be checking for the "saved_value" attribute ever.. right ? I think that's the problem here.. the firstOrNewTranslation method should only ever be checking for the locale, group and key to determine if a key already exists.. not also the value, saved_value and status
I added these unsets here
        if ($checkDB) {
            $query = $this->translation->on($this->getConnectionName());

            unset($attributes['value']);
            unset($attributes['saved_value']);
            unset($attributes['status']);

            foreach ($attributes as $attribute => $value) {
                $query = $query->where($attribute, $value);
            }

            $translation = $query->first();
        }

and it makes the problem go away for me

on a separate but not entirely unrelated note, I'm still confused why this 'json' locale even exists and why the database rows for the 'json' locale contain the same 'value' and 'saved_value' as the 'en' locale rows

@vesper8, you found a bug. Only locale, group and key should be used for the check. The fix is only using these columns. Reseting other values will cause issues for the following code which uses the attributes to set a new translation if one is not found.
        if ($checkDB) {
            $query = $this->translation->on($this->getConnectionName());

            foreach ($attributes as $attribute => $value) {
                if ($attribute === 'locale' || $attribute === 'group' || $attribute === 'key') {
                    $query = $query->where($attribute, $value);
                }
            }

            $translation = $query->first();
        }

saved_value refers to the value which is in the translation files. When the translation is first read from files the saved value is the same as value. Later if the value is modified then LTM will display this translation as unpublished.

@vesper8, the json locale stores keys used in the JSON translation files. Since these are not compatible with keys used by LTM (and the rest of translation files) LTM generates unique keys but in order to map them back to original values needs to save them in the DB. json locale is used to identify the saved mapping.

thanks for the quick fix!
I just ran a composer update and it didn't get the new version even though you pushed it 2 hours ago
  - Updating vsch/laravel-translation-manager (2.6.38 => 2.6.44): Loading from cache

Maybe just a cache issue.. I know github is having problems today

@vesper8, updated packagist.org manually. There seems to be some glitches on that site and I don't know if it is github's fault or packagist but managed to get myself locked out of my account trying to update github hooks.  ;)


Component: 'Flavor Suggestions'

Overview
The user can add suggested flavors. The user can remove the flavor they just added.
Action Items

[ ]  randomly populate 3 suggested flavors
[ ] on hover, display flavor name
[ ] on click, add the suggested flavor to cart

Resources/Instructions
Refer to the single product page wireframe

@eyeriss42 thanks for clarifying the issue, appreciate the detailed info! But it seems like this issue was already closed and the PR that solved it #22 with scaffolding for the flavor suggestions was already merged to staging.
The added tasks sound like a functionality, especially because the component will have new functions on hover/click and it will be hitting the backend to get 3 flavors. So, I would suggest for us here to create a new issue with these tasks and functionality instead of re-opening the old issue.


[Chore] buildDev and add --release

This PR add buildDev as explained here https://esy.sh/docs/en/configuration.html#esybuilddev.
It add --release to prevent bad dependencies. One example is re-web @tcoopman tried to build it as dependencies but alcotest was required. This could be catched with --release in the CI. Also the json of re-web is very similar to this repo that is why I PR here.
Also when grepping esy build I have found 1 useless file.
Also this project pull rely but is is not used. Should I add a small function/test to show example in this PR ?

I'm slightly concerned that we need that install to make it work since this is a very simple project. I wonder if we need to do changes somewhere to just have the default install work.


fix(index): correct loader sourcemaps usage

I was wrong about the sourcemap behaviour, turns out webpack does actually pass the sourcemap as the second arg to loaders, this PR restores the previous behaviour (with a perf boost by not trying to parse for an inline sourcemap if there is already a source map available)
Depends on https://github.com/webpack-contrib/istanbul-instrumenter-loader/pull/66

@mattlewis92 - Just landed #66
I'll take a look at appveyor and figure out why it's not reporting.

@d3viant0ne done!

Thanks @d3viant0ne 😄 Gave the RC a spin in our app and its all working smoothly now. We probably want to land https://github.com/webpack-contrib/istanbul-instrumenter-loader/pull/65 before cutting 3.0.0 final, as that PR is breaking.


tools/replay: set NavDestination from the route

I was thinking it would be better if we got rid of the param reading in map.cc and just have navRoute publish its navDestination. Similar thing to controlsd with experimentalMode, and plannerd with longitudinalPersonality.

I agree with @sshane . otherwise we also have to prevent the ui from fetching the NavDestination from the CommaApi during replay.

I agree with @sshane .this can simplifies the synchronizations, replay also works without modification.

Good point, this is definitely the right solution. Not sure why that's only broadcasted on change.

@deanlee can you check how big navRoute is and make a PR to send it at a fixed frequency? I think there were some concerns about the size of the service inflating rlogs/qlogs


Clarify status code URL

@wesinator Hi Wes, thank you for the contribution, unfortunately it's unlikely to be accepted. Please refer to the following comment from February: https://github.com/rmaake1/httpstatuses/pull/64#issuecomment-466788195

@wesinator Hi Wes, thank you for the contribution, unfortunately it's unlikely to be accepted. Please refer to the following comment from February: #64 (comment)

Ok, could you archive this repo then ?

@wesinator I no longer have any control over the repository nor do I believe the repository owner monitors these issues, you'll need to reach out to the company to make that request.

cc @rmaake1


Add register and login page

What this PR does?

instead of toggling between component. we now toggling between register page and login page.
So i remove the authentication page and make make two different page.
I check the functionality of backend signup functionality after making my change it work. But i am not sure if login functionality is working so  please review the functionality first. i didn't make any change to backend logic. But it still good to check it.
I didn't remove the components Login and Register currently, so we have option to compare the thing.
So let me know if you need some change. i'm glad to do it.
Solve the issue no. #26


Initial content for designing

This is the initial text content I got for design from what is already online, Emily is working on getting us some specific stuff.
I've put some of the images in the figma already
content from : https://www.emily-jenkins.com/#/movedancefeel/
MOVE DANCE FEEL
An innovative dance project helping women affected by cancer to reconnect with their bodies and access group support.
Emily created Move Dance Feel in 2016 and has since been working in partnership with  leading cancer support organisations across London to integrate dance into their activity programmes; Bromley by Bow Centre with Macmillan Social Prescribing, Maggie’s Barts and Paul’s Cancer Support Centre. The project is also launching in Bristol with Penny Brohn UK in 2019/2020.
The project offers dance sessions centred around artistic practice, where women come together to dance instead of talk about their cancer experience.
“Move Dance Feel has provided a wonderful outlet for self expression and creativity, which I have found to be very healing.”

participant

Research has revealed significant benefits of dance in this context, helping participants to feel more positive, confident, active and able.
Physical expression and creative exchange gives rise to strong social relationships, which helps participants manage challenges and reduces feelings of isolation and anxiety.
Evidence shows that Move Dance Feel achieves greater wellbeing and can  combat some of the negative side effects of cancer treatment, such as alleviating feelings of fatigue and improving body confidence.
“I come feeling tired and I leave feeling like I have more energy!”

participant

Click HERE to view the April 2019 Service Evaluation report, detailing research findings.
In January 2018 an article about Move Dance Feel was published by People Dancing: Animated Magazine which you can read HERE. People Dancing is the UK’s development organisation for Community Dance.
 It becomes unfamiliar - even worse, it starts to feel as if it is an enemy. For me, dancing started to bring me back to my own body and its energy, strength and basic joyfulness.”

participant

Follow Move Dance Feel on twitter and visit the facebook page for session times, dates and locations.

link:
https://www.facebook.com/MoveDanceFeel/
https://twitter.com/movedancefeel?lang=en-gb
articles
https://www.trinitylaban.ac.uk/news/community-dance-sessions-for-women-affected-by-cancer-help-address-gap-in-support
https://www.cancercaremap.org/article/guest-blog-move-dance-feel-innovative-dance-project-women-affected-cancer
https://shinecancersupport.wordpress.com/2018/11/07/dancing-through-cancer-how-a-new-organisation-is-helping-women-with-cancer/
https://www.communitydance.org.uk/DB/animated-library/move-dance-feel
https://www.rsph.org.uk/about-us/news/guest-blog-dance-wellbeing-and-cancer-recovery.html
https://shinecancersupport.wordpress.com/2018/11/07/dancing-through-cancer-how-a-new-organisation-is-helping-women-with-cancer/
https://www.pauls.org.uk/news/move-dance-feel/
crowd funding
https://www.crowdfunder.co.uk/move-dance-feel
https://www.spacehive.com/movedancefeel
Report :
https://www.flipsnack.com/movedancefeel/move-dance-feel-service-evaluation-april-2019.html
mentions:

https://www.whittington.nhs.uk/default.asp?c=28554

https://www.pauls.org.uk/news/move-dance-feel/

http://www.lahf.org.uk/move-dance-feel-weekly-sessions-–-24-september-2018

A-guide-to-reducing-variation-and-improving-outcomes-in-cancer-rehabilitation-in-London.pdf
Extracts :
Take a look at Phase Two on PAGE 10 - to implement improvements to ensure provision of comprehensive cancer rehabilitation at the appropriate level for all cancer patients, across all tumour groups and at every stage of the pathway (akin to Move Dance Feel!)
Take a look at stats and predictions on PAGE 17
The traditional medical model is not sufficient to fully meet the personalised care agenda, and a fundamental shift in thinking and behaviours will be needed to deliver better outcomes. The principles of personalised care involving a holistic, patient centred approach are inherent to the delivery of high quality rehabilitation PAGE 18
See PAGE 20
In the UK, NHS England commissioning guidance for rehabilitation has highlighted the vital role that rehabilitation plays in delivering better outcomes for patients,
‘A modern healthcare system must do more than just stop people dying.
NHS England (2016) Commissioning Guidance for Rehabilitation

My motivation for setting up the project was in finding out that despite cancer survivorship rates improving, approximately 70% of people in the UK report negative side effects between one and ten years after treatment. [https://www.macmillan.org.uk/documents/aboutus/commissioners/movemorereport.pdf#page=7]


Unsubscribe with optout USSD

Please make sure my logic is correct here, The problem is that when a user does a optout(and doesn't say yes to loss messages) the change is never created in the registration hub, leaving the subscription active, If the user decides to register again there will be two active subscriptions.
I think this issue only surfaced now because we never allowed the users to reregister.
Voice PR to follow.

This looks good to me, :+1:
I'm just wondering if this is something we should be doing in the jsbox app, or if we should be using webhooks system to do this (have optouts call the registration service, which does the unsubbing)? I'm not quite sure what the best solution is.
I think for now this is okay though, and it seems to be the strategy that we're following in momconnect for the casepro pods (the pods do a optout + unsubscribe).


Problem with Geocoding Function

Authors of ggmap package have made changes to geocode().  The default service is now datasciencetoolkit.org.  This produces errors when using location names like White House.

Code in presentation script needs to be updated to specify Google as the source when geocoding locations like White House.
May need to update presentation slides.

Updated the presentation code to change the source to Google on all geocode() calls.  No changes needed to the presentation slides.


Update strings.json

for better Chinese description
What
change text for : share screen and end call
Why
original locale text is not proper translated, cause confuse.
the changed text is based on Microsoft Teams, to provide consistance of experience.
How Tested
just text changing , no need to test.
Process & policy checklist

[X] I have updated the project documentation to reflect my changes if necessary.
[X] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

nothing could break for text change

@yiming Thank you for finding the issue with this translation! These files are generated automatically by a translation service, we will file the issue with them and will see the strings updated soon!

Thank you for filing the issue @yiming we have filed an issue with the localization service. we will close this PR when the strings have been updated, they will then be in the following release

Hi @yiming, we now have your string corrections in this pull request in version 1.19.0. We had to relay your corrections to our translation teams so that translation they provide us periodically are now updated. Thank you and please let us know if there are other incorrect translations.


Refactor log origin string

Use externally passed variable to validate log origin string.

@AlCutter let me know if it is ok now.

Will merge this to allow us continue setting up the FT log environment. @AlCutter let me know if there is any other issue with this PR.


Add hunters scans

Closes #187
Checklist:

[ ] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[x] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[x] Have removed web_hi_res_512.png when adding a new extension

uh is the check failing because the okhttp3 imports are before the java imports? and is it relevant? every other extension had the imports ordered like that.


Is there a way of using blurkit with jetpack compose?

I'm trying to blur composable views. Is there a way of using Blurkit to achieve this?

+1 I'd like to know how to use this with Jetpack Compose. Right now, most Blur libraries don't support Jetpack Compose and the solutions out of the box such as the Blur.Modifier does not truly blur layers below. If we can see a Jetpack Compose example, it will be super helpful for the Android community!

I am able to Do it!!! I used another View Blur Library and used it in a Compose Library and the results are crazy! I'll probably name it compose blur view or something, keep an eye on my github, I am publishing it


Add the option to output JSON Table Schema v2 instead of Redshift DDL

@alexanderdean I think all Table Schema tickets can be closed as we had slight misunderstanding of it?

Agree!


Documentation Missing Sample SQL Query for kolide_k2_package Table

Describe the bug
The documentation for the kolide_k2_package table does not include a sample SQL query. Users looking to understand how to interact with the kolide_k2_package table via SQL queries would benefit from a concrete example. This omission makes it challenging for new users to quickly get started with querying the table.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://github.com/grendel-consulting/steampipe-plugin-kolide/blob/main/docs/tables/kolide_k2_package.md'
Scroll down to the section where a sample query might typically be found.
Notice the absence of a 'Sample Queries' section or any SQL example.

Expected behavior
I expected to find at least one sample SQL query demonstrating how to select data from the kolide_k2_package table. For example, a simple query to fetch all columns for all records, or a more complex query illustrating how to filter records based on certain conditions.
Screenshots
N/A
Desktop (please complete the following information):
N/A
Smartphone (please complete the following information):
N/A
Additional context
Adding sample queries to the documentation would significantly improve the usability of the documentation for new users and serve as a quick reference for more experienced users. Providing examples of both simple and complex queries could help demonstrate the table's capabilities and the types of insights users can gain from the data.

Do the examples in #60 help clarify @toliver38 ?

Looks great!


View User: Update password

Updating a user with a new password value results to an error:
baddata: "fdfgdsfgf"
message: "property password is immutable!"
but we need this

Updating the password is implemented using the endpoint
PATCH /rest/profile  {data: {...       password    : "oldPassword",       new_password: "newPassword"     } }
for the user to change the password himself, using the fields password & new_password.
Or, for users with admin role, using the endpoint
PUT|PATCH /rest/users/{id} {data: {displayName     : "DisplayName",                            password        : "secr3t",                            email           : "nobody@localhost",                            curatoryGroupIds: [cg.id],                            enabled         : true,                            accountExpired  : false,                            accountLocked   : false,                            passwordExpired : false,                            defaultPageSize : 15,                            roleIds         : [2, 3, 4, 6, 7]     } }
to set a new password directly

Yes it is better now. The service /rest/users throws no error and the password can be changed by an administrator, but when I login in with this user here test and same password, it gets an 403 requesting the profile.


3D models updates, various fixes

New 3D models for the PS, PMU1 and VM (fixing some minor symmetry issues, mesh issues, etc)
New texture to go along the new models
Added a GT.Country list to some 300PS units missing it

Obsolete.
A new PR will be made with the same, and additionnal, changes.

Obsolete.
A new PR will be made with the same, and additionnal, changes.


Installation fails on Android based Smartphones bought in China

Avoid duplicates

[ ] Bug is not mentioned in the FAQ
[ ] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[ ] Bug is not already reported in another issue

Describe the bug

The App couldn't be installed on several Android based Smartphones bought in China, such as Xiaomi. But it appear it has no problem with those Chinese Smartphone which are available and purchased in Germany.
A example: on a Redmi smartphone the user received a note that he should "activate Exposure Logging", but the app showed no reaction after the option was confirmed.
Expected behaviour
Explanation or fix for this Problem

Steps to reproduce the issue

Technical details

Mobile device:
Android version:

Possible Fix

Additional context

@amijoern

It sounds like you are reported this problem on behalf of another user. Do you have access to the phone which has an issue?
Was the CWA app installed directly from
Google Play Store https://play.google.com/store/apps/details?id=de.rki.coronawarnapp&hl=en or via some other method?
What version of Android is installed on the phone with the problem?
Is Google Play Services installed on the phone? If yes, what version?
What version of COVID-19 Exposure Notification Framework is installed on the phone? See FAQ help on determining ENF version.

See also the FAQ note [Google/Android]: Why are Android 6 and Google Play Services minimum requirements?

Hello everyone,
the questions from @MikeMcC399 are on point, could you please provide additional information regarding your issue? The Contact Tracing API is a part of Google Play Services. Consequently, the app will not work on devices without enabled play services.
Best regards,
SG
Corona-Warn-App Open Source Team

This was answered a while ago. See https://github.com/corona-warn-app/cwa-app-android/issues/1444#issuecomment-715284139

Corona-Warn-App Open Source Team


Jd/move structs to is

Companion PR to https://github.com/NREL-Sienna/InfrastructureSystems.jl/pull/335

All tests are now passing here (except for Test AGC, which is broken in psy4 for unrelated reasons)

Here is the test environment on which tests are passing locally:


Logging needed.

We still do not have any logging, which always makes me anxious.

We've just added the logging feature.


How to run this?

Hi.
After running the colab I get:

hi @tralala87 👋 the solution is in the image please read the green text


Feature/2024 02 bug fixes

Critical Changes

Updated the Flexi Pages to avoid errors on MetaDeploy

Changes

Updated sample metadata to utilize the Text Operator
Updated ReadMe file
Ordered getBundles method by label
Updated CCI file to add English names for MetaDeploy

Issues Closed

#163
#162

Thank you!


Adding support for H100

What does this PR do?
Fixes #61.
The get_cuda_kernel_launch_stats function returns a dataframe with 31315 rows before this fix and a dataframe of 32835 rows after the fix. This shows that we are tracking cudaLaunchKernelExC events now.
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)

[ ] N/A

[x] Did you write any new necessary tests?

[ ] N/A

[ ] Did you make sure to update the docs?

[x] N/A

[x] Did you update the changelog?

[ ] N/A

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.05% :tada:

Comparison is base (1db4a3f) 90.45% compared to head (23a2559) 90.50%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
+ Coverage   90.45%   90.50%   +0.05%
==========================================
  Files          29       29
  Lines        2148     2160      +12
==========================================
+ Hits         1943     1955      +12
  Misses        205      205

Files Changed
Coverage Δ

hta/analyzers/cuda_kernel_analysis.py
90.97% <100.00%> (+0.06%)
:arrow_up:

hta/analyzers/trace_counters.py
96.59% <100.00%> (+0.03%)
:arrow_up:

tests/test_trace_analysis.py
100.00% <100.00%> (ø)

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


Crashing when opening local favorite tab

The app keeps crashing when the favorite tab is selected.
I think the reason for this error is that some time ago I favorited a gallery that is now deleted and I cannot remove the favorite. I tried searching the code of the gallery through a browser and entering directly to it from the search bar then the app crash again.
The code of the gallery causing this problem is 288395.
I need at least a way to save or export my local favorites.

Should be fixed in 2.0.7, otherwise install the debug version and enable send crash report


[SPARK-5154] [PySpark] [Streaming] Kafka streaming support in Python

This PR brings the Python API for Spark Streaming Kafka data source.
    class KafkaUtils(__builtin__.object)
     |  Static methods defined here:
     |
     |  createStream(ssc, zkQuorum, groupId, topics, storageLevel=StorageLevel(True, True, False, False,
2), keyDecoder=<function utf8_decoder>, valueDecoder=<function utf8_decoder>)
     |      Create an input stream that pulls messages from a Kafka Broker.
     |
     |      :param ssc:  StreamingContext object
     |      :param zkQuorum:  Zookeeper quorum (hostname:port,hostname:port,..).
     |      :param groupId:  The group id for this consumer.
     |      :param topics:  Dict of (topic_name -> numPartitions) to consume.
     |                      Each partition is consumed in its own thread.
     |      :param storageLevel:  RDD storage level.
     |      :param keyDecoder:  A function used to decode key
     |      :param valueDecoder:  A function used to decode value
     |      :return: A DStream object

run the example:
bin/spark-submit --driver-class-path external/kafka-assembly/target/scala-*/spark-streaming-kafka-assembly-*.jar examples/src/main/python/streaming/kafka_wordcount.py localhost:2181 test

Test build #25267 has started for   PR 3715 at commit eea16a7.

This patch merges cleanly.

Test build #25267 has finished for   PR 3715 at commit eea16a7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Test build #25951 has started for   PR 3715 at commit adeeb38.

This patch does not merge cleanly.

Test build #25956 has started for   PR 3715 at commit 2c567a5.

This patch merges cleanly.

@tdas I think this PR is almost ready, please follow the example to double check it, thanks!

Test build #25958 has started for   PR 3715 at commit 97386b3.

This patch merges cleanly.

Test build #25951 has finished for   PR 3715 at commit adeeb38.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Test build #25956 has finished for   PR 3715 at commit 2c567a5.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Test build #25958 has finished for   PR 3715 at commit 97386b3.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Hello,
I made a fork of the main Spark repo and pulled this pull request into it. I was able to build it successfully but I get the followng error when calling KafkaUtils.createStream( ... )
The line:
kvs = KafkaUtils.createStream(streaming_context, ZOOKEEPER, GROUP, {TOPIC: 1})

...gives the following error message:
No kafka package, please put the assembly jar into classpath:
 $ bin/submit --driver-class-path external/kafka-assembly/target/scala-*/spark-streaming-kafka-assembly-*.jar

When I try to run that 'submit" command with the specified parameters, I get:
$ bin/spark-submit --driver-class-path external/kafka-assembly/target/scala-2.10/spark-streaming-kafka-assembly-1.3.0-SNAPSHOT.jar

Error: Must specify a primary resource (JAR or Python file)
Run with --help for usage help or --verbose for debug output
Using Spark's default log4j profile: org/apache/spark/log4j-defaults.properties
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/home/myerske/spark/external/kafka-assembly/target/scala-2.10/spark-streaming-kafka-assembly-1.3.0-SNAPSHOT.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/home/myerske/spark/assembly/target/scala-2.10/spark-assembly-1.3.0-SNAPSHOT-hadoop1.0.4.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [org.slf4j.impl.Log4jLoggerFactory]

I've verified that the JAR file is there.
Has anybody had the same issue or can offer any help?
Thanks

Also, I believe the warning message on line 77 of python/pyspark/streaming/kafka.py should read
$ bin/spark-submit
rather than
$ /bin/submit

@kennethmyers fixed, thanks!

Test build #26106 has started for   PR 3715 at commit 370ba61.

This patch merges cleanly.

@kennethmyers I think you should put the path of your scripts after spark-submit options, such as:
bin/spark-submit --driver-class-path xxxxx path/to/your_script

@davies Thanks! That worked! I'm new to Spark and misunderstood the function of the "spark-submit" command.

Test build #26106 has finished for   PR 3715 at commit 370ba61.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Test build #26177 has started for   PR 3715 at commit 31e2317.

This patch merges cleanly.

Test build #26180 has started for   PR 3715 at commit dc1eed0.

This patch merges cleanly.

Test build #26180 has finished for   PR 3715 at commit dc1eed0.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Test build #26177 has finished for   PR 3715 at commit 31e2317.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

@JoshRosen Can you take brief look at the python stuff?

Should we have tests for this?  Do we have tests for the other Python streaming sources?

Test build #26209 has started for   PR 3715 at commit 23b039a.

This patch merges cleanly.

Test build #26209 has finished for   PR 3715 at commit 23b039a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Jenkins, retest this please.

Test build #569 has started for   PR 3715 at commit 23b039a.

This patch merges cleanly.

Test build #569 has finished for   PR 3715 at commit 23b039a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Jenkins, test this please.

Test build #26324 has started for   PR 3715 at commit 23b039a.

This patch merges cleanly.

Test build #26324 has finished for   PR 3715 at commit 23b039a.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Test build #26331 has started for   PR 3715 at commit f257071.

This patch does not merge cleanly.

Test build #26331 has finished for   PR 3715 at commit f257071.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Test build #26339 has started for   PR 3715 at commit e6d0427.

This patch merges cleanly.

Test build #26339 has finished for   PR 3715 at commit e6d0427.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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

Test build #26361 has started for   PR 3715 at commit 4280d04.

This patch merges cleanly.

Test build #26361 has finished for   PR 3715 at commit 4280d04.

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

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

Test build #26578 has started for   PR 3715 at commit d93bfe0.

This patch merges cleanly.

@tdas  @pwendell done

I am merging despite a small comment from me.

Test build #26578 has finished for   PR 3715 at commit d93bfe0.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class KafkaUtils(object):

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


Basic Ingredient schema

Based on discussion the current ingredient schema looks like it will be:
ingredients: [
  name: "Potatoes"
  quantity: "1KG"
  prep: "diced"
  notes: "watch out for slippery potato pieces!"
,
  name: "Onions"
  quantity: "2"
]

Where prep and notes are optional. Frontend formatting would be displayed as:
$quantity $name, $prep ($notes)
e.g.
1KG Potatoes, diced (watch out for slippery potato pieces!)
The source of ingredient knowledge, added later, would have an array of terms to map to an ingredient. This moves the issue of plurals/alternative names to a single source of truth while keeping the recipes themselves simple and readable.

To all for recipes to start being added with the new schema sooner rather than later I'll focus in upgrading ingestion then worry about database schema later. For now it can flatten the ingredient object as shown above and the frontend can remain agnostic to ingestion/schema changes.

A lot of recipes have optional ingredients. While this is noted in the notes section, it might be worth having a separate optional field that defaults to false.


fix syntax: markdown syntax for heading

fix syntax: markdown syntax for heading
The Jaguar part didn't conform to Markdown syntax, resulting in failure in displaying heading style after rendering to webpage.

I signed it


Simplification of subdomain search

Checklist

[x] only relevant code is changed (make a diff before you submit the PR)
[x] run tests npm run test
[x] tests are included
[x] commit message and code follows the Developer's Certification of Origin

Checklist (for documentation change)

[ ] only relevant documentation part is changed (make a diff before you submit the PR)
[ ] motivation/reason is provided
[ ] commit message and code follows the Developer's Certification of Origin

After thinking about it for a while, I wonder if a RegEx search is needed at all.
The task of the subdomain detection is not to verify the domain view.
The string "hostname" is already verified by the browser window.location.
The detection should split the string and output the partial string by index (can be specified by the user).
This is much faster and more robust with String.split.
What do you think?

lgtm, what about you @terhuerne ?


Bijectors, reverse mode hangs

MWE:
module MWE
import Bijectors, Enzyme
b = Bijectors.CorrBijector()
binv = Bijectors.inverse(b)
f(x) = sum(b(binv(x)))
x = randn((0, 0))
Enzyme.gradient(Enzyme.Reverse, f, x)
end

Output:
WARNING: replacing module MWE.
 ** On entry to DTRMM  parameter number  9 had an illegal value
 ** On entry to DLASCL parameter number  9 had an illegal value
 ** On entry to DTRSM  parameter number  9 had an illegal value
 ** On entry to DTRSM  parameter number  9 had an illegal value

It seems to just hang after that. I ran out of patience after 5 minutes.
Enzyme current main branch.

Okay this one requires minimization and isn't resolved by current dev branches

@penelopeysm and also would you be able to help reduce this one too?

Giving this a shot now

It's an issue with Cholesky on a zero-dim matrix:
module MWE
import Enzyme, LinearAlgebra

function cholesky!(A)
    C, info = LinearAlgebra._chol!(A, LinearAlgebra.UpperTriangular)
    return C
end

f(x) = sum(cholesky!(copy(x)))
x = zeros((0, 0))
Enzyme.gradient(Enzyme.Reverse, f, x)
end

fixed on main


Nim potentially unwanted software Windows Defender

Installed nim 1.4.8 with scoop. Windows defender says there might be a potentially unwanted software in the package.

Anyone has this issue as well? Is it a false positive?

Seems to be a known issue with this nim version
https://github.com/nim-lang/Nim/issues/17886


Added play_media function to allow channel change functionality

Added play_media function to allow channel change functionality useful with alexa, google or from the homeassistant app as well as play,pause.
Channels can be selected either by a channel number or by a channel name which is looked up in the channels list to retrieve the channel number.
Added attribute "volume_step_default" that can be used by code in Alexa to set the default volume step increment when an Alexa volume up command is received. The current Alexa default is 10 which is too high.
The Alexa smart home code was also modified to add stepSpeaker function to allow volume to be set in variable increments.

This works so well for me, only tweak I needed was increasing delay between IR commands from 0.5 to 1 because my set-top box is a bit slow so that's a suggestion for a config parameter. Now I'm a step closer to hands-free TV, changing cable TV channels with Alexa is awesome.


[CI] Add GitHub action for labeler

Signed-off-by: Aadhitya A<EMAIL_ADDRESS>Description
This PR is all about a new action to comment on labels...
This PR fixes #2669
Notes for Reviewers
CI Maintainers/Contributors, please test if this action works or not...
Signed commits

[x] Yes, I signed my commits.

Hmm.... How we can test this?....

@alphaX86 I added this same workflow to the meshkit repo as a test. Here's an issue with the issue/design required label: https://github.com/layer5io/meshkit/issues/9#issuecomment-831546883. What do you think? Room for improvement in the message?
Given the nature of the design being required prior to PR, I commented out that portion. On second thought, it is probably valid to include it to the extent that someone submits a PR that in fact does need a design spec written.

@leecalcote I saw the test comment. Seems its great! And yeah, as you said for PR it may need such. Anyway, it works!

Nice work, @alphaX86. Next, we could use auto-labeling of PRs based on changes to particular filepaths:

area/docs for changes in /docs/*
component/mesheryctl for changes in /mesheryctl/*

Nice! Will make an issue and PR on this soon! 👍


[FEAT] useFormContext is not exported via @pankod/refine-react-hook-form

Is your feature request related to a problem? Please describe.
useFormContext  is not available in pankod/refine-react-hook-form.
Describe alternatives you've considered
I have not seen it included in https://github.com/refinedev/refine/pull/3209
import {  useFormContext } from "react-hook-form"
Additional context
No response
Describe the thing to improve
thank you

This might be discussed if there is a usecase to use  useFormContext over e.g. simply calling useForm in deeply nested components.

Hey @capsloq
useFormContext is already exported. I don't quite understand what the issue is, can you elaborate a bit?

Typescript tricked me you are right @yildirayunlu it's correctly exported. Sorry for the inconvenience and thank you for checking.


User option for consistend height of minibuffer

When there are less actual candidates than selectrum-num-candidates-displayed specifies the minibuffer pops up at a lesser height than usual. I think that is annoying because my eyes are used to see the prompt at a higher position. I would like to be able to always give it the same height regardless how many candidates are available. This could be by default or by adding a user option, what do you think?

I just stumbled about a related problem: The variable resize-mini-windows controls how minibuffer should resize, when set to nil selectrum is not able to show it's candidates at all so it should probably be let bound to grow-only.

I would like to be able to always give it the same height regardless how many candidates are available.

Yep, sounds reasonable. It should be a new user option, disabled by default.

it should probably be let bound to grow-only

Also true. I didn't know about this variable before, thanks for pointing it out.

Let me know what you think.

Looks good, alternatively we could check for window-height and use enlarge-window but I guess there is no problem to insert empty lines.

I like this solution as it means there's only one way that we set the window height, which is by inserting text. Thanks.


BlockChain and How it works

Checklist

Please note that in case of below checklist not updated accordingly, the maintainers have the right to immediately close your pull request.

[x] You created a branch from dev branch.
[x] You PR is raised on dev branch and not onmaster.
[x] Blog follows Blog Guidelines.
[x] You have read the Contribution Guidlines before creating this PR.
[x] Checked for any duplicate blog entry.
[x] Checked for all grammatical errors in the blog.
[x] You have not published the blog content anywhere else on the internet on platforms like medium, dev.to etc.
[x] The content is not copy-pasted from anywhere else on the Internet.

Description of change

Author Details for Blog Post Entry

id: Rishav Raj
github: https://github.com/Rishav139
linkedin: https://www.linkedin.com/in/rishav-raj-1a5505198/

@puneetsingh24 Thanks for the reply. I went through the mentioned blog, I could not find the technical key terms explanation. I have tried to explain every term in detailed way. Also, my blog was inclined towards Bitcoin. Would it be correct if I change the blog title from 'Blockchain and How it works' to 'Bitcoin and How it works'.
Please let me know if you have any further suggestion. Thanks in advance.


Box doesn't disappear when X is clicked

In the blue box there is an X, which indicates that the box will go away when the X is clicked, but nothing happens.

Not a huge deal, but maybe can be fixed?

Good catch, just addressed this, thanks @jeanwalshie !


Cannot pull hourly/minute data from Climacell

I've recently transitioned to v4; updated the integration in HACS, got the API and udpated my code. However, although I'm able to obtain daily data, I'm unable to pull data from either hourly or per-minute forecasts.
Logs:
021-04-20 02:29:53 ERROR (SyncWorker_3) [custom_components.climacell.data_provider] ClimacellTimelineDataProvider._retrieve_data error status_code 400
2021-04-20 02:29:55 WARNING (SyncWorker_0) [custom_components.climacell.sensor] TimelineSensor.update - Provider has no data for: cc climacell Hourly Temperature 0h
2021-04-20 02:29:55 WARNING (SyncWorker_5) [custom_components.climacell.sensor] TimelineSensor.update - Provider has no data for: cc climacell Hourly Temperature 1h
2021-04-20 02:29:55 WARNING (SyncWorker_2) [custom_components.climacell.sensor] TimelineSensor.update - Provider has no data for: cc climacell Hourly Temperature 2h
2021-04-20 02:29:55 WARNING (SyncWorker_1) [custom_components.climacell.sensor] TimelineSensor.update - Provider has no data for: cc climacell Hourly Temperature 3h
etc.

OK, can ignore this now, sorry. I was requesting moonPhase hourly when only daily data was available. Once I removed that request, all my other hourly data worked again.
On another note though, it seems "observation_str" is no longer giving me a date format for my timezone but for UTC. It's not a big deal but I was relying on that to draw a meteogram in my timezone.


路表->路标

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Reviews welcome - GUVNOR-3193 Change Project from a folder in a repo to a repository

Please do not merge.
This PR does the following:

Introduces the Project and Module classes. The old Project that was the maven project is now called Module, just to avoid confusion with the new Project. The new Project is the active project that the workbench is presenting in the UI or the Project that we can later build ( these changes still only support building single Modules ). The Project has an active branch and branch changes require a recreation of the Project ( We can change this later, but after spending months with the code it is quite clear that keeping this step will save us from an extra month of code changes and bug fixes. )
Introduces a Branch class. This is again to make our code a bit easier and safer to use. We used to just store the branches as strings, when a branch is a location in a repository that has a name and a path.
Removes asset-mgmt, this is no longer used
Renames org.guvnor.common.services.project.model.Repository to org.guvnor.common.services.project.model.MavenRepository so it is easier to know the difference between MavenReposity = used where we have dependencies and Repository = GIT repository that stores code.
Changes the REST API. We now hide repositories and access projects directly.
Structural changes in RepositoryService and ConfiguredRepositories to fix fired events (See GUVNOR-3555)

Just using the build servers to run the tests since they take few hours locally.
https://issues.jboss.org/browse/AF-648
The related PRs:

https://github.com/kiegroup/kie-soup/pull/7
https://github.com/AppFormer/uberfire/pull/814
https://github.com/kiegroup/drools/pull/1402
https://github.com/kiegroup/kie-wb-common/pull/1015
https://github.com/kiegroup/drools-wb/pull/566
https://github.com/kiegroup/jbpm-designer/pull/643
https://github.com/kiegroup/jbpm-wb/pull/832
https://github.com/kiegroup/kie-wb-distributions/pull/573

Build finished. 3508 tests run, 7 skipped, 0 failed.

Can you please move this PR for https://github.com/kiegroup/appformer ?


TST: corrected URL for newly added Excel test data file

There was one small thing overlooked when merging PR #10964: the url of the renamed test file was still pointing to my repo instead of pydata/pandas.

This is already fixed in PR #10967.


Plugin API to communicate with other plugins

Leader
User story
Users sometimes want to link data between plug-ins. For example, when a button in one plugin is pressed, it triggers an event in another plugin.
Requirements
type PluginExtensionInstance = {
  id: string; // widget id, block id
  pluginId: string; // includes version
  name: string; // excludes version
  extensionId: string;
  extensionType: "widget" | "block";
};

reearth.plugins.instances: PluginExtensionInstance[]
reearth.plugins.postMessage(id: string; message: any)
reearth.on("pluginmessage", (e: { data: any, sender: string }) => {})

Examples
Testing list

[ ]

We already have that API. I will close this issue.


Add some missing tasks to development/getting-started.md

Closes: N/A
What's Changed
I noticed that the development/getting-started.md documentation was missing a number of our more recently added testing/linting commands. This PR attempts to address that gap.

While you're here, would you mind leaving a note in the start/stop sections around being able to specify the name of the service to start when calling invoke start, probably in this section: https://nautobot.readthedocs.io/en/stable/development/getting-started/#using-docker-with-invoke
I know there are a lot of options for all commands but these can be super helpful with not having to restart the DB, Selenium, or Celery every time.


feat: add QueryCache.fetchQuery method

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Enforce numeric data in signals

Which issue(s) are closed by this pull request?
Closes #431, requires #436
Changes proposed in this pull request:

Check for non-numerical data in Signal.__init__

I did not find a way to check for boolean data, because it is converted to float when calling np.atleast_2d(data). Any ideas?

Good point, I added it to the freq and freq_raw setter so far. I think the time setter is refactored in #396. Should we rather add this as todo there to avoid conflicts? The check is now a static class method so it would be easy to include.

We decided to postpone this until #396 is merged

Closed in favor of #505 which was done for the new Audio classes that can handle complex time data.