Replace Bulma with TailwindCSS

I want to redesign the UI using TailwindCSS and turn the Catalogue into a official supported tool (not only a prototype). In the process I want to extract the components we need to create here and move them to surface_tailwind so they can be reused in other projects.

Since phoenix v1.6 should be released in the next few days, I believe we can start discussing our next steps. After reading https://github.com/surface-ui/surface_tailwind/issues/1 a few times and visiting the websites/docs of all suggested component suites, it seems to me that building our components on top of DaisyUI would be the safest choice.
So here are all (for now) the things I want to explore in this project:

A minimal suite of components based on DaisyUI to be used in the Catalogue and then extracted to its own project later.
Reusable application layouts - It would be great if we had a few generic app layouts that would allow users to quickly get started and although we'll not need more than one for the Catalogue, we could at least make the one we create available. Something similar to the "Application shells" in https://tailwindui.com/#product-application-ui.
Theming - just light/dark mode would be enough for the catalogue but since DaisyUI already provide a bunch of other themes, I don't see any reason to not allow developers to use them.
Use alpine.js to handle unavoidable client JS stuff - this is a tough call as it would make some of the components to have an external dependency, however, it seem like a good tradeoff given the benefits.
Explore the idea of having component templates that can hold all the component logic but don't depend on a specific CSS lib. Something similar to https://headlessui.dev/

I have already discussed some of those items with @davydog187 and I'm glad he's just as excited as I am about this project and will be helping me in this journey.
@haubie, @rmayhue, @Menkir and @gdub01 based on the previous discussion, it seems to me you've already played with the Tailwind + Surface  combo and experienced some of the challenges we're about to face here. I liked the discussion and ideas that were brought up regarding the ideal API and I think we'll certainly explore them here.
So moving on, I'll start with a couple of questions:

How do you feel about using DaisyUI as a starting point? Any downsides or concerns you have?
Should we even try to create a generic surface_tailwind with more components or just embrace DaisyUI, stick with it and create a surface_daisyui instead?

Hi @msaraiva. Very excited to see parts of your pioneering work and persistence with Surface being imported into the next Phoenix release!
I've been playing around with the ideas in the previous discussion including creating some components using them.
But I agree that if the goal is to get a rich and slick set of components using Tailwind CSS in a short period of time, wrapping DaisyUI maybe the path. (Plus adding some additional ones like you mentioned for layouts.) And using the name surface_daisyui.
It looks like they had similar to goals to what I was exploring in how to have some sensible defaults, but still allow component users to customise through applying a theme definition or overriding classes on a specific component. The way they've done it looks pretty nice :-)
I also really really like the idea of number 5 in terms of future proofing any components built and not have to rebuilt components from scratch if that new amazing CSS framework comes along!
In terms of an API, for the styling part of that, if the goal is targeting people who use Tailwind on a regular basis, I'm thinking that Tailwind naming conventions could be used for prop names.
e.g. if there was a prop for background, use bg as the prop name, which mirror the background Tailwind classes like:

bg-none
bg-gradient-to-t
bg-center
bg-clip-padding
etc.
e.g.

I guess one issue with this is that is it could create many props on a component, rather than a single class prop.  But the good thing with this way is that component user only has to override the props they want. And if DaisyUI is being used, those additional props are probably only needed for reaching into some of those deeper classes - e.g. with the DaisyUI avatar component, the top-level div would have the class prop is appended to as a general catch-all, and w, h and rounded for setting things deeper in:
prop class, :css_class
prop w, :css_class, default: "w-24"
prop h, :css_class, default: "h-24"
prop rounded, :css_class, default: "rounded-full"
prop src, :string

In the render function:

So the component user might just do:
<Avatar src="my_img.png" />
And if they only wanted to change the shape, it would be:
<Avatar src="my_img.png" rounded="rounded-md" />
Or add some padding:
<Avatar src="my_img.png" rounded="rounded-md" class="p-4" />
Another thought on API, for components that link off, like a menu, instead of a singular to prop, I'm wondering if the user can choose from href, patch or redirect, the last two being the Live View patch or redirect. E.g.:
<MenuItem title="Href link" href="https://google.com.au" opts={[target: "_blank"]} />
<MenuItem title="Patch to /"  patch="/" />
<MenuItem title="Redirect to new-liveview" redirect="/new-liveview" />

This is just opinion based my my experimentation and I'm sure others will have views as well!

I am glad that there is more movement in this project.
Grouping Tailwind classes into DaisyUi components is a very good start for an out of the box solution. There is no license constraint now and you can provide a library with strong defaults.
The question if the project should be called surface_tailwind or surface_daisyui implies for me on the question if also a standard API should be provided, so that one can write own components against this API. If not then surface_daisyui, otherwise surface_tailwind.
However, a standard API is not exactly easy to develop, since components can become indefinitely complex. The way to configure a component as @haubie  suggested works only for flat components. So rather for absolute basic components. For bigger things like tables it needs more configuration.
The problem here is that the application of different values of the same classes on different elements is possible e.g.:

        <div class={... different bg-color ...}

The tight coupling of design and form (html-divs etc.) makes it hard to define the properties.
What works better is the separation of design and form like DaisyUi does. You introduce your own classes that are configurable to a certain degree.
Example:
<MyComp type="danger">

defmodule MyComp do
    use Surface.Component
    prop type, :string, default: "default", values: ~w(default danger info success warning)

    def render(assigns) do

                    ...

    end
end

.foo data[type="danger"]{
    @apply text-red-200 border-2 border-red-500;
}

.inner-foo data[type="danger"]{
    @apply bg-red-50
}

.baz {
    @apply text-sm;
}

.baz data[type="danger"]{
    @apply text-red-900
}

Another advantage of this method is that the HTML is not bloated. Especially in the Inspector the html trees become much leaner.
@msaraiva  on point 4: What exactly do you mean by unavoidable Js stuff? I had used Alpine for exactly one reason: Animations. LiveView/Surface exists for the very reason that you need little to no JS anymore. Unless you want to link your application to a JS application. But there are hooks for that.

@haubie and @Menkir, thanks for your inputs.

if there was a prop for background, use bg as the prop name

I have mixed feelings about this. I believe one of the main reasons that drove people to create all those bg, m, p, h, w, etc. is simply because when you're not using components, as soon as you need dozens of those things (and you usually do), it just take too much space and the whole thing becomes unreadable. However, when using components, we have a change to abstract all that mess and instead of using all those utility classes directly, we can expose higher level properties with more meaningful names. We should still provide a nice way to allow users to override/add some of those utilities when necessary but it seems to me that if we need to do that every time we use a component, we probably haven't done a good job when abstracting those components in the first place.

The tight coupling of design and form (html-divs etc.) makes it hard to define the properties.

I totally agree. It's a hard problem to solve. I'm not sure we'll be able to find a solution for that but we can certainly try :)

What exactly do you mean by unavoidable Js stuff?

I guess "unavoidable" was a bad choice of word :)
What I mean it that besides the obvious cases where we need JS (animations, drag and drop, etc.), there are cases when we don't need server events to perform some basic user interactions.
Take this Tabs component, for example. Currently, whenever you click on a tab, an event is sent to the server. That would make sense if we needed to load the tab's content dynamically on each click but in many cases, we don't. We could just bring everything and use JS to show/hide the tabs. This may not be a problem for most components but depending on the amount of data that might be exchanged, it just adds unnecessary server round trips.  We can still use vanilla JS to handle that too but with alpine it's just much easier and clean.
Basically, whenever a component requires some state on the client, Alpine really shines. I still haven't decided if we should use alpine or not but I think it might be worth to explore the possibility.

Interesting discussion. Comfortable with the thoughts above including the use of AlpineJS.

We should still provide a nice way to allow users to override/add some of those utilities when necessary but it seems to me that if we need to do that every time we use a component, we probably haven't done a good job when abstracting those components in the first place.

I guess we could take this a bit more iteratively and aim for a less 'customisable' release initially, with a minimal API, until we settle on a nice way to handle it?
Although there is probably a 'common sense' test that could be applied to each component as to what parts we'd like to expose in its own prop? (e.g. just to use the basic avatar example above I'm thinking size, shape, image and any action would be the minimum and all optional?)
Let me know if I can contribute in some way. I'm happy to start wrapping some DaisyUI with Surface as an initial parse through. That may help with design decisions around the API as well.

With regards to DaisyUI.. the upside is it's faster to develop if we use it directly.
The downside is we won't be able to progress faster or differently from daisy ui if we call it daisy_ui.
Perhaps we could use a lot of their component-based tailwind classes and html structure to get a jump on things instead of using daisyui directly? They'll be pinned to a tailwind version so they'll be unlikely to change much I don't think.
I also like the idea of alpine for things like tabs and dropdowns.

I hadn't really taken a deeper look at DaisyUI until the last few days. It looks like it's a Tailwind plugin with JavaScript code running behind the scenes during build. I guess there is a concern by using DaisyUI, that we might be coupling ourselves to their specific build tooling, how they have organised their project, etc?
So your idea @gdub01 of using their classes and html but not the rest of it (with full acknowledgement) could be a good way to kick off surface_tailwind rather than surface_daisyui?

Hey yes David thanks that says it perfectly.
I think daisy lets you add a class of 'button' which ultimately ends up
being a list of tailwind classes. So in our case if we had a 'button'
component we'd avoid using daisy's 'button' class, but rather skip
directly to their list of tailwind classes with full acknowledgement as you
mentioned.
On Mon, Aug 16, 2021 at 12:21 AM David @.***> wrote:

I hadn't really taken a deeper look at DaisyUI until the last few days. It
looks like it's a Tailwind plugin with JavaScript code running behind the
scenes during build. I guess there is a concern by using DaisyUI, that we
might be coupling ourselves to their specific build tooling, how they have
organised their project, etc?
So your idea @gdub01 https://github.com/gdub01 of using their classes
and html but not the rest of it (with full acknowledgement) could be a good
way to kick off surface_tailwind rather than surface_daisyui?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/surface-ui/surface_catalogue/issues/20#issuecomment-899202867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM7F3B4WHRNQPS7Y7YBHZLT5CG4HANCNFSM5A2EMZ2A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

So there's a little more complexity than just copying a list of tailwind classes I'm learning.

@gdub01 as far as I could see, they work independent from each other and they are chosen according to the styled and utils config described here https://daisyui.com/docs/config.
So we could just pick the one we want, which I believe would be the styled one, with or without the additional utilities. Right?

Oh yes I think you're right @msaraiva
I think the utilities, in the case of the button, have aspects you could argue are better relegated to a theme. ie. if you want rounded buttons, you probably always want rounded buttons.. not sometimes round sometimes square. So the utility classes for buttons may not be as useful.
However in the case of the avatars https://github.com/saadeghi/daisyui/blob/master/src/utilities/styled/avatar.css having an online and offline version would be pretty useful. So perhaps we can pick and choose the utility modifiers we'd want to support.
The other interesting thing is their use of css variables like so:
border-width: var(--border-btn, 1px);
I think we'd have to decide how we'd want to set that up as it plays into theming. Perhaps we could have a module that just sets all the variables by passing it a theme object and it sets the variables inside

tags that are rendered on the page inline?


#806 ProfilerService: currentTimeTook should be buffered over several frames

change algorithm of counting currentTimeTook - now its average frame time of the last 30 frames.
closes #806

Codecov Report

Merging #859 into dev will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev     #859      +/-   ##
============================================
- Coverage     59.70%   59.70%   -0.01%
- Complexity     2322     2323       +1
============================================
  Files           251      251
  Lines         11340    11343       +3
  Branches       1032     1033       +1
============================================
+ Hits           6771     6772       +1
- Misses         4259     4262       +3
+ Partials        310      309       -1

Impacted Files
Coverage Δ
Complexity Δ

...n/com/almasb/fxgl/dev/profiling/ProfilerService.kt
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

.../src/main/kotlin/com/almasb/fxgl/app/LoopRunner.kt
97.61% <0.00%> (+2.38%)
12.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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

Thanks!


added custom form field

If you need custom field, then make field file, for example:
/voyager/formfields/custom/subcategories.blade.php
and paste Optional Details in BREAD
For example:
{
    "formfields_custom": "subcategories"
}

@alexeykostenko Very cool. I'll need to make sure to note this in the documentation 👍


[tokenization] preprocessing inputs and labels

Hello,
Firstly thanks for open-sourcing all components of alpaca_farm!
I'm looking into data_preprocessor.py and am wondering where/if the labels are set to the input_ids shifted by 1. (something like input_ids = input_ids[...,1:]  labels = input_ids[...,:-1]classic next token prediction)
However, it seems like they're set to input_ids without any shifting? I'm not sure what I'm missing but any clarification would be great :)

NVM, for baseline sft, since a default hugging-face trainer is used, shifting happens inside the default forward implementation :))

Yeah, you’re right. Shifting is handled internally in the trainer.


Why apiRoot created from the client requires a projectKey for every call ?

ClientBuilder has to have the projectKey to create a ctpClient, but when I create an apiRoot with using createApiBuilderFromCtpClient(ctpClient), I did not understand why do we need to pass the projectKey everytime, is not the apiRoot bound this ctpClient already 🤔 It's a bit confusing tbh. I just wonder if you have an alternative use case to explain the need.

Because we didn't added a project scoped apiRoot yet. :)

But it's fine to do something like
const projectApiRoot = apiRoot.withProjectKey(projectKey)

and then use the projectApiRoot instead

Because we didn't added a project scoped apiRoot yet like for the other SDKs. :)

Thanks for the prompt answer. I think I did not got the idea here, can you elaborate a bit, what is project scoped apiRoot ? 🙏🏽

The ApiRoot is generated and the entry point is always on the root directory level.
For the other SDKs we added a manually maintained ApiRoot which is scoped to the project key. E.g. the Java SDK one:
https://github.com/commercetools/commercetools-sdk-java-v2/blob/main/commercetools/commercetools-sdk-java-api/src/main/java/com/commercetools/api/client/ProjectApiRoot.java


Fix stdout handling in jobset.py

Still trying to get the portability tests green.
This time, I'm fixing only the point that failed rather than assuming the text encoding of the standard output of any particular test. The only assumption I'm making is that log lines for CPU usage are in ASCII.
Context:

https://github.com/grpc/grpc/pull/27415
https://github.com/grpc/grpc/pull/27464
https://github.com/grpc/grpc/pull/27485

Manually Triggered Test Runs (still running):

grpc_portability
C++ Basic Tests Debug

Unless we're also using the flag that consumes this table to optimize bin
packing of tests into CPUs we probably don't need this anymore.
On Tue, Sep 28, 2021, 11:37 AM Richard Belleville @.***>
wrote:

@.**** commented on this pull request.
In tools/run_tests/python_utils/jobset.py
https://github.com/grpc/grpc/pull/27493#discussion_r717862353:

@@ -357,7 +357,7 @@ def stdout(self=self):
if measure_cpu_costs:
m = re.search(
r'real\s+([0-9.]+)\nuser\s+([0-9.]+)\nsys\s+([0-9.]+)',

                   stdout())

This log
https://source.cloud.google.com/results/invocations/8498d457-2d58-4f4d-8587-fa0b7b0aa2f4/log
includes it:
run_tests_python_linux_dbg_native_default_python_alpine: "python tools/run_tests/run_tests.py --use_docker -t -j 16 -x run_tests/python_linux_dbg_native_default_python_alpine/sponge_log.xml --report_suite_name python_linux_dbg_native_default_python_alpine -l python -c dbg --iomgr_platform native --arch default --compiler python_alpine --bq_result_table aggregate_results --measure_cpu_costs --report_multi_target"
A quick grep indicates that it's coming from here
https://github.com/grpc/grpc/blob/91a2db9bc2f0195e649380f5e0f46d5ced8c73a6/tools/run_tests/run_tests_matrix.py#L463
.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/27493#discussion_r717862353, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACNG45N5XJBLFANHF2GQYADUEIDOPANCNFSM5E3GBXEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

The original adhoc portability run has gotten stuck, I started a new one: https://fusion2.corp.google.com/invocations/bdb9c77a-c454-4280-bcca-e3434f23e6e9/log

Apparently my assumption was a bad one:
  File "/var/local/git/grpc/tools/run_tests/python_utils/jobset.py", line 360, in state
    stdout().decode("ascii"))
UnicodeDecodeError: 'ascii' codec can't decode byte 0x88 in position 5789: ordinal not in range(128)

Trying again with non-ascii characters stripped:

grpc_portability
C++ Basic Tests Debug

Giving it another shot:

grpc_portability
C++ Basic Tests Debug

Known failures:

Artifact Build MacOS and Basic Tests Ruby MacOS
Basic Tests Python Windows


enhancement: Update Footer-UI

Related Issue

Improved Footer Component Area using Border and Box-Shadowing Properties in Footer Component

Closes: #450
Describe the changes you've made

Added border and box-shadow related properties in Footer Component
These Changes reflects on All Web Pages

Type of change
What sort of change have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update.
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)
Code of Conduct

[x] I agree to follow this project's Code of Conduct

/run

@amanjaiman1 . Please review on this PR

first of all its cool , but listen we will have to implement more good UI. if you've more design then share

first of all its cool , but listen we will have to implement more good UI. if you've more design then share

Ok  @amanjaiman1 . I will work on to improve UI More Better than Previous

first of all its cool , but listen we will have to implement more good UI. if you've more design then share

Ok @amanjaiman1 . I will work on to improve UI More Better than Previous

Hi @amanjaiman1 . I have updated Footer UI more better than previous in All Pages. My point of view , It's pleasant look in
several web pages without effecting other designs. Please Check and Review on It?

Hi @amanjaiman1 . Please check my PR?

Created new PR for development branch; temporarily locked main branch for maintenance.


Provide functionality to center the taskbar on windows 10

Description of the new feature / enhancement
Add a feature to center the taskbar when used on ultra wide monitors (32;9, 21:9)
Scenario when this would be used?
When an ultra wide monitor is used (32:9 ratio), it's really annoying to have the taskbar icons at the far left hand side. On windows 11 they are centered and gives it a better user experience and less looking to the left to find what you need.
Supporting information
No response

Windows 11 is how to enable this


[SPARK-16445][MLlib][SparkR] Multilayer Perceptron Classifier wrapper in SparkR

https://issues.apache.org/jira/browse/SPARK-16445
What changes were proposed in this pull request?
Create Multilayer Perceptron Classifier wrapper in SparkR
How was this patch tested?
Tested manually on local machine

Test build #63100 has started for PR 14447 at commit 04f7fed.

Test build #63100 has finished for PR 14447 at commit 04f7fed.

This patch fails R style tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

hi @mengxr , could you please tell me how to debug R wrapper? Thanks a lot
I tried to read documentation and google, but cannot figure it out myself. From SparkR console, the error message is too vague as below, and I tried to tail -f spark logs but no error messages, and also I tried to create a RWrapperSuite but the class is private and cannot be accessed.
> model <- spark.mlp(irisDF, ~ Sepal_Length + Sepal_Width + Petal_Length + Petal_Width, blockSize=128, initialWeights=seq(1, 9, by = 2), layers=3, solver='LBFGS', seed=1234L, maxIter=100, tol=0.5, stepSize=1)
16/08/01 17:16:38 ERROR RBackendHandler: fit on org.apache.spark.ml.r.MultilayerPerceptronClassifierWrapper failed
Error in invokeJava(isStatic = TRUE, className, methodName, ...) :
In addition: Warning message:
In if (is.na(object)) { :
  the condition has length > 1 and only the first element will be used

I think your formula (~ Sepal_Length + Sepal_Width + Petal_Length + Petal_Width) is wrong. For classification problem, the target of the formula should not empty.

hi @yanboliang I just tried formula Species ~ Sepal_Length + Sepal_Width + Petal_Length + Petal_Width, but still not working with msg
> model <- spark.mlp(irisDF, Species ~ Sepal_Length + Sepal_Width + Petal_Length + Petal_Width, blockSize=128, initialWeights=seq(1, 9, by = 2), layers=3, solver='LBFGS', seed=1234L, maxIter=100, tol=0.5, stepSize=1)
16/08/02 10:21:48 ERROR RBackendHandler: fit on org.apache.spark.ml.r.MultilayerPerceptronClassifierWrapper failed
Error in invokeJava(isStatic = TRUE, className, methodName, ...) :
In addition: Warning message:
In if (is.na(object)) { :
  the condition has length > 1 and only the first element will be used

Do you mind tell me how did you debug it? I'd like to debug by steps but don't know how, thanks a lot

Actually I don't have an excellent debug way for SparkR wrappers. Usually you can create a RWrapperSuite and test whether it works well in the Scala side (If you encounter some issues about private class, change it to public temporary for test). For your above issue, please provide more error logs, then I can help you to find some clue for the error.

Actually I've already tried to change the classes to be public, and open them to not only [r] but every module which still does not work.
And exactly, above comment msg is the only error log I have, and no more error logs I found...only got to know it failed on fit() method...
So the question here actually is, how to find more error logs, like failed at which line of which file...

Test build #63300 has started for PR 14447 at commit eb5f5af.

Test build #63300 has finished for PR 14447 at commit eb5f5af.

This patch fails R style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #63395 has started for PR 14447 at commit 1eae07b.

Test build #63395 has finished for PR 14447 at commit 1eae07b.

This patch fails R style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #63399 has started for PR 14447 at commit a3f5bb5.

Test build #63399 has finished for PR 14447 at commit a3f5bb5.

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

Merged build finished. Test FAILed.

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

Test build #63488 has started for PR 14447 at commit ea55e6a.

Test build #63488 has finished for PR 14447 at commit ea55e6a.

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

Merged build finished. Test PASSed.

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

Test build #63564 has started for PR 14447 at commit e08b7b0.

Test build #63564 has finished for PR 14447 at commit e08b7b0.

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

Merged build finished. Test PASSed.

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

Test build #63624 has started for PR 14447 at commit 151cd6a.

Test build #63624 has finished for PR 14447 at commit 151cd6a.

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

Merged build finished. Test PASSed.

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

so there are a few competing implementation in R and mlp might not be a super relevant name. @shivaram @mengxr any thought on spark.mlp here?

Looks like mlp is a standard abbreviation used by a bunch of R packages [1, 2] -- If that is the case then spark.mlp sounds fine to me.
[1] http://artax.karlin.mff.cuni.cz/r-help/library/RSNNS/html/mlp.html
[2] https://cran.r-project.org/web/packages/monmlp

Test build #63668 has started for PR 14447 at commit 5f7fd00.

Test build #63668 has finished for PR 14447 at commit 5f7fd00.

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

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

Merged build finished. Test FAILed.

Test build #63671 has started for PR 14447 at commit 230ea91.

Test build #63671 has finished for PR 14447 at commit 230ea91.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #63673 has started for PR 14447 at commit f56b805.

Test build #63673 has finished for PR 14447 at commit f56b805.

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

Merged build finished. Test PASSed.

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

Test build #63711 has started for PR 14447 at commit e7fa68f.

Test build #63711 has finished for PR 14447 at commit e7fa68f.

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

Merged build finished. Test PASSed.

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

These are the last few comments. @shivaram what do you think - with those fixed we are good to merge this?

Test build #63761 has started for PR 14447 at commit de9d0a7.

Test build #63761 has finished for PR 14447 at commit de9d0a7.

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

Merged build finished. Test PASSed.

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

Yep, this looks good @felixcheung -- Feel free to merge once you think its good to go.

Test build #63820 has started for PR 14447 at commit 7c94e2b.

Test build #63820 has finished for PR 14447 at commit 7c94e2b.

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

Merged build finished. Test PASSed.

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

@keypointt could you please rebase this to master? Only 2 more comments.
cc @mengxr

hi @felixcheung I got below lint-r error when running it on my laptop, after merging with master
Warning message:
In readLines(filename) :
  incomplete final line found on '/Users/quickmobile/workspace/spark/R/pkg/R/mllib.R'
R/context.R:270:12: warning: no visible global function definition for ‘collectRDD’
  local <- collectRDD(results)
           ^~~~~~~~~~
R/SQLContext.R:221:12: warning: no visible global function definition for ‘firstRDD’
    row <- firstRDD(rdd)
           ^~~~~~~~
lintr checks failed.

Test build #64001 has started for PR 14447 at commit 3bd1695.

Test build #64001 has finished for PR 14447 at commit 3bd1695.

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

Merged build finished. Test PASSed.

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

we are looking at establishing some guidelines in PR 14705. Let's hold on for another day or 2.

@felixcheung sure, no problem

Hi there - sorry about the delay. Could you please rebase to master and check out the refactoring here and doc pattern here
I'll do another pass when you are ready. Thanks!

sure, will rebase

Test build #64231 has started for PR 14447 at commit 3d2d87a.

hi @felixcheung, after I merge master in, I found that spark.kmeans, spark.naiveBayes, spark.survreg, spark.isoreg don't have the #' @param ... Additional parameters ... . https://github.com/apache/spark/blob/master/R/pkg/R/generics.R#L1325-L1327
Should all the mllib methods explain the @param ... ?

Test build #64233 has started for PR 14447 at commit 56b8cef.

Please see the changes I have listed above for details - you will see that they are either removed or moved to mllib.R

Test build #64231 has finished for PR 14447 at commit 3d2d87a.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
          logWarning(s\"All labels belong to a single class and fitIntercept=false. It's a \" +
   * Set thresholds in multiclass (or binary) classification to adjust the probability of
class MultinomialLogisticRegression @Since(\"2.1.0\") (
          logWarning(s\"All labels belong to a single class and fitIntercept=false. It's\" +
  /** Margin (rawPrediction) for each class label. */
  /** Score (probability) for each class label. */
  class MultinomialLogisticRegressionModelWriter(instance: MultinomialLogisticRegressionModel)
 class ALSWrapperWriter(instance: ALSWrapper) extends MLWriter
 class ALSWrapperReader extends MLReader[ALSWrapper]
class JavaClassificationModel(JavaPredictionModel):
class LogisticRegressionModel(JavaModel, JavaClassificationModel, JavaMLWritable, JavaMLReadable):
class DecisionTreeClassificationModel(DecisionTreeModel, JavaClassificationModel, JavaMLWritable,
class RandomForestClassificationModel(TreeEnsembleModel, JavaClassificationModel, JavaMLWritable,
class GBTClassificationModel(TreeEnsembleModel, JavaPredictionModel, JavaMLWritable,
class NaiveBayesModel(JavaModel, JavaClassificationModel, JavaMLWritable, JavaMLReadable):
class MultilayerPerceptronClassificationModel(JavaModel, JavaPredictionModel, JavaMLWritable,
class LinearRegressionModel(JavaModel, JavaPredictionModel, JavaMLWritable, JavaMLReadable):
class DecisionTreeModel(JavaModel, JavaPredictionModel):
class RandomForestRegressionModel(TreeEnsembleModel, JavaPredictionModel, JavaMLWritable,
class GBTRegressionModel(TreeEnsembleModel, JavaPredictionModel, JavaMLWritable, JavaMLReadable):
class GeneralizedLinearRegressionModel(JavaModel, JavaPredictionModel, JavaMLWritable,
class JavaPredictionModel():
case class UnresolvedInlineTable(
case class UnresolvedTableValuedFunction(functionName: String, functionArgs: Seq[Expression])
case class Literal (value: Any, dataType: DataType) extends LeafExpression with CodegenFallback

Merged build finished. Test PASSed.

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

Test build #64233 has finished for PR 14447 at commit 56b8cef.

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/64233/
Test PASSed.

Merged build finished. Test PASSed.

Test build #64247 has started for PR 14447 at commit 2d43f71.

Test build #64247 has finished for PR 14447 at commit 2d43f71.

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

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

Merged build finished. Test FAILed.

@param ... Should be in mllib.R

Test build #64274 has started for PR 14447 at commit 0dea0d2.

Test build #64274 has finished for PR 14447 at commit 0dea0d2.

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

Merged build finished. Test FAILed.

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

Test build #64279 has started for PR 14447 at commit 067f8fd.

Test build #64279 has finished for PR 14447 at commit 067f8fd.

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

Merged build finished. Test FAILed.

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

Test build #64291 has started for PR 14447 at commit 0449edb.

@keypointt The error is happening because of the href line in the spark.mlp documentation. If you move the opening brace to the previous line the error goes away. i.e. the code should look like
#' \href{http://spark.apache.org/docs/latest/ml-classification-regression.html}{
#'   Multilayer Perceptron}

Test build #64291 has finished for PR 14447 at commit 0449edb.

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

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

Merged build finished. Test FAILed.

Test build #64293 has started for PR 14447 at commit 7526b65.

btw, this has info on document generation so you could test this out locally:
https://github.com/apache/spark/blob/master/R/DOCUMENTATION.md

I did test locally this time, thanks Felix

Test build #64293 has finished for PR 14447 at commit 7526b65.

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

Merged build finished. Test PASSed.

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

hi @felixcheung anything else I should do here?

merged, thanks!


Anyone got this work with Counter Strike Global Offensive?

Sending commands works, but as previous Issues state the log appears to not be working

Anyone possibly knows any alternative for csgo in csharp? Or even how to fix this issue?

I used this:
https://github.com/stefanodriussi/rconsharp
But it does not feature logs.
Everything else worked fine.


feat: integration tests enabled

This PR enables integration tests https://github.com/lidofinance/lido-terra-integration-tests in contracts github actions.
The workflow is the following: after commit made in the branch A, github is looking for the branch in the tests repository, if the branch found, github runs test from tests/A. In case there is no such branch in the test repo we are falling back to the tests/main branch.
After PR merged, gh is looking for a target merge branch name in the test repo. In other words, in case you implemented specific test in branch tests/A for specific update contracts/A you have firstly to merge tests RP and only after that your merge in contracts branch will be successful
UPD:
we firstly need to merge this one https://github.com/lidofinance/lido-terra-integration-tests/pull/9 to disable tests for develop contracts branch

they kind of fail though?..

they kind of fail though?..

we firstly need to merge this one https://github.com/lidofinance/lido-terra-integration-tests/pull/9 to disable tests for develop contracts branch

oh my god. it's working. AMAZING


Locale with upper case is resolved as used language but doesn't translate

With a upper-case locale i.e. en-GB and lower case registered languages/aliases/wildcard aliases the locale negotiation is done lower-case only, on a match however the uppercase input locale is used.
On translation the exact set locale (case sensitive) is used to determine the used dictionary and as the dictionary is defined with a the lower-case key the dictionary is not found and the fallback translation is used instead.
As far as I see it there are two possible solutions:

On locale negotiation use the lower-case as the set locale
Call angular.lowercase() on each translation when getting the translation dictionary

If you tell me the preferred solution I could create a PR.

Have a look here: https://angular-translate.github.io/docs/#/api/pascalprecht.translate.$translateProvider#methods_uniformlanguagetag

Cool, didn't know about that method yet.
However if I understood it correctly this only configures the value returned from .determinePreferredLanguage(), right?
Furthermore, I think the problem I have described above is rather a general problem, since a language tag is accepted as a valid tag matching the defined languages but then does not resolve to the correct translation.

I don't get it. The locale negotiation behind the scenes (i.e. via determinePreferredLanguage()) is configurable with uniformLanguageTag(). However, if you apply a language explicitly, you are already in control. Where is the problem?

Okay, I've just quickly put together a plnkr (https://plnkr.co/edit/uCxtKi?p=preview) demonstrating the problem (note that I've just put together this language setup scenario).
Just click on English (en-US) and then English (en-GB) - what you will see is the spanish translation although the language is set to English.
This is due to https://github.com/angular-translate/angular-translate/blob/master/src/service/translate.js#L164 matching the lower-case preferred locale with the available locales successfully but then returning preferred (uppercase) instead of locale (lowercase) in L165.
Because of that the locale is set to en-US and then when translating in https://github.com/angular-translate/angular-translate/blob/master/src/service/translate.js#L1547 it tries to access $translationTables['en-GB'] which is undefined (in contrary to en-gb which would be defined). Since no translation table for the locale is found the next language form the fallback stack is taken.
In my opinion there are two solutions:

Alter L165 to return locale instead of preferred
Alter L1547 to get $translationTable[angular.lowercase(uses)]

When clicking on en-US or en-GB -- nothing is happening here and the keys are shown - and this is correct, as the system doesn't know anything about a fallback stack in your plunkr?!
Also one suggestion: Try to use the latest angular-translate library versions in your examples to cut out any other (already fixed) issues.

Odd, apparently my JS changes didn't save properly.
Also you are right about updating (I just took some random template I could find), however the issue also applies to the current version.
I have updated the plunker respectively.

just a quick fix - if you change your mapping on the registerAvailable... from "en-gb" to sth like "english" and register it accordingly - all mappings will work.
Yes, somewhere the detection/mapping fails currently in this special case due to lowercase or matcher.

I did realize that, still I expect it to be rather common to register the languages on their language tag and since any situations where the registered language matches the used language when comparing case insensitive but not when comparing case sensitive cause this problem (strangely enough no one seems to have experienced this problem before) it's probably better to fix this before anyone else runs into it.
Also as I said, the proper fix to this is really simple, however I am not absolutely sure if it has any side-effects.

@Aides359 We would welcome your contribution :)

Alter L165 to return locale instead of preferred

This would ensure not the requested but the actual used locale. I think that's fine. We have already some tests, but adding one test more covering this specific situation would make the PR more worth in addition

Alter L1547 to get $translationTable[angular.lowercase(uses)]

Too late in the flow, I think.

I just realized that this is not as easy as first expected.

This would ensure not the requested but the actual used locale.

This is actually not the case - while this is true with language keys registered in lower case, it breaks on any other configuration (however this is not due to the proposed change but also in current production code)


iframes, embeds, redirects... and custom deployments if necessary

I'm trying to get reliable twitter link previews, and I'm pretty happy with FxTwitter generally... so I thought surely there's a way to use this in my own personal app (as in an app I'm building to use myself, not my own app I plan to share or sell) to render an iframe style embedded preview... but I've been trying out user agents based on the source code regex and I'm still getting bumped to the original twitter urls which is causing issues due to twitter and the in app WebKit user agent... but all that aside... the point of this issue is to ask... If I want a stand alone HTML preview of a tweet rendered using the FxTwitter application stack, like if you used the publish.twitter.com embedded links, just not as crap and avoiding all the issues involved in using Twitter directly... what should I be doing?
I'm not opposed to modifying something and deploying my own copy of the entire stack, but I'm just not sure what (if the capability even exists, since I'm not completely sure there's an equivalent HTML rendering endpoint) I should be changing in order to mark out a custom user agent and reliably request the correct endpoint to get a HTML rendering of the tweet?
Do I need to implement some javascript or something else to 'pre-render" the html based on an endpoint? What is the step in the chain I'm missing? Do I even need a custom deployment?
original tweet link -> FxTwitter -> ??? -> HTML a browser can display.

We have our JSON API but currently don't have an iframe for tweet embeds yet. It's something I've thought about offering though.

I want this quite a bit. It's quite a bit of work to take the JSON API and use that to get some HTML and CSS that looks good. If anyone knows of a pre-existing template I can use, I would be thankful.


Every 2000 steps during DDSP-SVC learning This error occurs.

Every 2000 steps during DDSP-SVC learning This error occurs.
I reinstalled the closing torch and Xformers.
Could the above be the cause of the error?

Did you set up a validation set and perform preprocessing correctly？

Thank you for your advice.
I'll try again as you said.


Block the hot reload when the tab is opened

So the user won't be frustrated by the reload.
Look up vaadin-dev-tools in the DOM and call setActive(true/false) when the tab is active.
It requires the next Flow version for the deactivate method

Implemented in 24.2-SNAPSHOT

Implement in the beta1 version


⚠ Bofrost.fr has degraded performance

In f0e2da8, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1735 ms

Resolved: Bofrost.fr performance has improved in 89c66e6.


Username and password

What is the username and the password that the proxy server asks for after setting up everything?

How do I change username/password for security?


[BUG] fixed ColumnEnsembleClassifier handling of unequal lenght data

This PR fixes two bugs in ColumnEnsembleClassifier` related to this report https://github.com/alan-turing-institute/sktime/issues/2312:

tags were incorrectly set - ColumnEnsembleClassifier always thought it could not handle unequal length data, even if components supported that. This is now correctly inferred from the internal tags.
the internal mtype was set to numpy3D which could handle only equal length data by definition of this data type. This was changed to the pandas.DataFrame based data types.

FYI @TonyBagnall, @MatthewMiddlehurst, would appreciate a review.

although I did wonder what an anytagis is ...
_anytagis_then_set


CyberVisualizer:No data problem

Hello guys!
When I started the cyber_visualizer tool, there was no data play,but in cyber_monitor.

My nvidia driver is 375.39.
@in_dev_docker:/apollo/docs/demo_guide$ glxinfo -v | grep -i opengl
OpenGL vendor string: VMware, Inc.
OpenGL renderer string: Gallium 0.4 on llvmpipe (LLVM 3.4, 128 bits)
OpenGL version string: 2.1 Mesa 10.1.3
OpenGL shading language version string: 1.30
OpenGL extensions:
in_dev_docker:/apollo/docs/demo_guide$ uname -a
Linux in_dev_docker 4.4.0-31-generic #50~14.04.1-Ubuntu SMP Wed Jul 13 01:07:32 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Why is this, how can I solve it?
Should I upgrade opengl to version 3.3 or later? How to upgrade? I did not upgrade successfully.
please thank you very much.

Hi, @lhh0916  can you provide the record you used?
Thank you!

@gruminions I converted "apollo_2.0_camera_sample.bag"  to "apollo_2.0_camera_sample.record" by rosbag_to_record.

@gruminions I converted "apollo_2.0_camera_sample.bag" to "apollo_2.0_camera_sample.record" by rosbag_to_record.

Sorry, I failed to find "apollo_2.0_camera_sample.bag", can you share the link?
Thank you.

@gruminions I converted "apollo_2.0_camera_sample.bag" to "apollo_2.0_camera_sample.record" by rosbag_to_record.

Sorry, I failed to find "apollo_2.0_camera_sample.bag", can you share the link?
Thank you.

@gruminions  "https://github.com/ApolloAuto/apollo/releases/download/v2.0.0/apollo_2.0_camera_sample.bag"

@lhh0916 you docker don't support OpenGL shading language 3.3 or higher. Please try the following:
1: download the correct nvidia graphics driver  from "https://www.nvidia.com/Download/index.aspx?lang=en-us"
2: install this driver for your host machine and reboot your machine
3: put the downloaded driver into apollo docker
4: enter into the apollo  docker container, and then install the driver for apollo docker container like this:
" sudo bash nvidia-driver-file -a -N --ui=none --no-kernel-module". When meeting "[default: (N)o]:", press enter key.
5: try cyber_visualizer again.

Closed for now, please reopen if you still have problems with it.

@ZeusJupiter
I reinstalled the nvidia driver and solved the problem, thank you very much!

@ZeusJupiter Do you use nvidia-docker or nvidia container toolkit？When I install NVIDIA driver in docker, there are some conflicts with host driver


Feature: Open Together with Windows Starts

This feature allows users to set the application to automatically launch and open when the computer starts up. By enabling the "Open Together with Windows Starts" option, the application will seamlessly start in the background without requiring manual intervention from the user upon system boot. This feature offers convenience and ensures that the application is readily available for immediate use as soon as the user logs into their computer. Users can easily enable or disable this option in the application settings, providing them with greater control over their startup preferences. Overall, this feature enhances the user experience by streamlining access to the application and improving productivity from the moment the computer is powered on.

This feature can be implemented using auto-launch
Is anyone interested in this requirement?


Cannot npm run start main theme in README

Thank you for providing a nice hugo theme.
I tried to run the example in the README, but it gave me an error. Attached below is the detailed output. If you know the cause of the problem, I would be happy to get an answer.
Thank you in advance.
Environment

Windows 11 Home (64-bit)
node version v18.13.0
hugo v0.109.0-47b12b83e636224e5e601813ff3e6790c191e371+extended windows/amd64 BuildDate=2022-12-23T10:38:11Z VendorInfo=gohugoio

Run all command on Terminal in VSCode
Description
For main theme, I could not run npm run start.
git clone https://github.com/markdumay/hugo-theme-hinode.git my-hinode-site && cd my-hinode-site
npm install
npm run start

><EMAIL_ADDRESS>prestart
> npm run clean

><EMAIL_ADDRESS>clean
> shx rm -rf public resources

><EMAIL_ADDRESS>start
> exec-bin node_modules/.bin/hugo/hugo server --bind=<IP_ADDRESS> --disableFastRender

Start building sites …
hugo v0.108.0-a0d64a46e36dd2f503bfd5ba1a5807b900df231d+extended windows/amd64 BuildDate=2022-12-06T13:37:56Z VendorInfo=gohugoio
ERROR 2023/01/10 22:42:41 render of "section" failed: "C:\Users\nanjakorewa-win-ms\workspace\my-hinode-site\layouts\_default\list.html:9:28": execute of template failed: template: _default/list.html:9:28: executing "main" at <partial "assets/blog-card.html" $element>: error calling partial: "C:\Users\nanjakorewa-win-ms\workspace\my-hinode-site\layouts\partials\assets\blog-card.html:4:16": execute of template failed: template: partials/assets/blog-card.html:4:16: executing "partials/assets/blog-card.html" at <partial "image.html" (dict "url" .Params.thumbnail "ratio" "16x9" "outerClass" "card-img-wrap" "innerClass" "card-img-top" "title" .Params.title)>: error calling partial: "C:\Users\nanjakorewa-win-ms\workspace\my-hinode-site\layouts\partials\image.html:26:12": execute of template failed: template: partials/image.html:26:12: executing "partials/image.html" at <partial "utilities/GetImage.html" (dict "url" $url)>: error calling partial: "C:\Users\nanjakorewa-win-ms\workspace\my-hinode-site\layouts\partials\utilities\GetImage.html:12:20": execute of template failed: template: partials/utilities/GetImage.html:12:20: executing "partials/utilities/GetImage.html" at <resources.GetMatch>: error calling GetMatch: runtime error: invalid memory address or nil pointer dereference
Error: Error building site: failed to render pages: render of "page" failed: execute of template failed: template: partials/head/head.html:7:7: executing "head" at <partial "head/seo.html" .>: error calling partial: "C:\Users\nanjakorewa-win-ms\workspace\my-hinode-site\layouts\partials\head\seo.html:19:18": execute of template failed: template: partials/head/seo.html:19:18: executing "partials/head/seo.html" at <partial "utilities/GetImage.html" (dict "url" .)>: error calling partial: "C:\Users\nanjakorewa-win-ms\workspace\my-hinode-site\layouts\partials\utilities\GetImage.html:12:20": execute of template failed: template: partials/utilities/GetImage.html:12:20: executing "partials/utilities/GetImage.html" at <resources.GetMatch>: error calling GetMatch: runtime error: invalid memory address or nil pointer dereference
Built in 23963 ms

I am not familiar with hugo and could not find the detailed cause, but based on the error message, I can see that the error is inside utilities/GetImage.html. Then, it seemed that the thumbnail: https://picsum.photos/id/184/4288/2848.jpg line in emoji-support.md was the cause. Then I excluded emoji-support.md and npm run start worked fine.

Hi Nanja, thanks for raising this issue! At first glance, I suspect there’s an error with generating the file path when retrieving external images for further processing on Windows. I tested the code on macOS an Linux, but not on Windows yet.
I’ll run the commands on a Windows system to recreate the problem. Hopefully it’s as simple as using a cross-OS function to define the local file path. Keep you posted.

I created a patch in the branch image-win that should fix the issue. Care to test it?

I was able to build it, thank you so much for the quick fix!

Great! I've merged the changed into the main branch and created a new release v0.7.9.


README: Add badge URLs

Description
Add badge URL to README.

Codecov Report

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

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   71.42%   71.42%
=======================================
  Files           3        3
  Lines          21       21
=======================================
  Hits           15       15
  Misses          6        6

Continue to review full report at Codecov.

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


Cant find user

Describe the bug
Somehow I get the error that the acc doesn't exist although it does (I have scraped it before).
Error messages and tracebacks
/Users//.virtualenvs/ThesisObjectDetection/bin/python /Users//Documents/GitHub/RedDotPRJ_1/imageAnalyzer.py
Traceback (most recent call last):
File "/Users//.virtualenvs/ThesisObjectDetection/lib/python3.6/site-packages/instaloader/structures.py", line 520, in _obtain_metadata
self._rhx_gis = metadata['rhx_gis']
KeyError: 'rhx_gis'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/Users//Documents/GitHub/RedDotPRJ_1/imageAnalyzer.py", line 4, in
from InstagramScraper.instascraper import *
File "/Users//Documents/GitHub//InstagramScraper/instascraper.py", line 65, in
profile = Profile.from_username(L.context, USERNAME)
File "/Users/josefmoucachen/.virtualenvs/ThesisObjectDetection/lib/python3.6/site-packages/instaloader/structures.py", line 475, in from_username
profile._obtain_metadata()  # to raise ProfileNotExistException now in case username is invalid
File "/Users//.virtualenvs/ThesisObjectDetection/lib/python3.6/site-packages/instaloader/structures.py", line 522, in _obtain_metadata
raise ProfileNotExistsException('Profile {} does not exist.'.format(self.username)) from err
instaloader.exceptions.ProfileNotExistsException: Profile josefmoucachen does not exist.
Instaloader version
4.2.6
Additional context
Add any other context about the problem here.

similar here:
instaloader --login clickchecker -f insta.session
[Errno 2] No such file or directory: 'insta.session'
Session file does not exist yet - Logging in.
Enter Instagram password for clickchecker:
Enter 2FA verification code: xxxxxx
Logged in as clickchecker.
Saved session to insta.session.
No targets were specified, thus nothing has been downloaded.
instaloader clickchecker -f insta.session
clickchecker: Profile clickchecker does not exist.
Errors occured:
clickchecker: Profile clickchecker does not exist.
The profile exists for sure :-)

Duplicate of #297.


Task: First edition of OrbTk Book

Task

[ ] Finish section "Write your first App" (counter example, including screenshots)
[ ] Review of native English speaker

Related to
#106
#107
Assigness
@kivimango
@rzerres

Issue is closed, but sorry for question, where is the book?
(This page is first in google search about "orbtk book")

Please have a look at the
wip-book
repository. It is in very early stage. But you might get some valuable information.
Greetings
Ralf
Am 28.11.2020 06:32 schrieb incker<EMAIL_ADDRESS>Issue is closed, but sorry for question, where is the book?
(This page is first in google search about "orbtk book")
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/redox-os/orbtk/issues/108#issuecomment-735042483, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABI5LVDGMSFTHN4A5SOQLSDSSCDLFANCNFSM4JW6ZMSQ.

Please have a look at the wip-book repository
Can you give a link?

@incker https://github.com/redox-os/orbtk/tree/wip-book/src


MLP recurrent

This PR consists of two commits related to recurrent policy models.
The first commit implements the recurrent option for the MLP policy. This results in multiple methods shared by the MLP and CNN model which I extracted in a NNBase class.
The second commit doesn't add any new functionality but suggests a new naming convention for observations, current obervations and states. From my understanding, a general naming convention in RL is that states is an umbrella term for observations. They can be the same but if for example num_stack=4, one state is made up of 4 observations. With the term states the current code refers to the hidden state of the recurrent layer of a policy. As this might be confusing, I suggest a new naming convention with current_obs  to states, states to recurrent_hidden_states and obs remain obs. Inside thes policy class the recurrent hidden states are abbreviated to rnn_hxs.

Hey! Looks good! I will review it over the weekend.

Renaming states to recurrent_hidden_states sounds good. Since we don't know true states but approximate them with RNNs.
But for current_obs I would prefer to keep it as observations. Since we might not know true states. And the naming convention is already used in gym.

I was not aware of the naming convention in gym. And considering the unknown true states I agree that we should keep the obs and current_obs. But the differentiation between states and recurrent network hidden states should definitely be made.

Ok. Looks good to me.
Thanks a lot!
Could you please rename the variables? Let's discuss naming conventions in a separate PL/issue.
Also let me know if you have time to resolve the conflicts.

I removed the conflicts and obs to states renaming. Also I changed observations to obs to be consistent everywhere and pluralised the arguments of Storage.insert. I think the confusion between the plural of the Storage.obs variable and the obs argument is smaller than always pass pluralised variables and then the arguments are singular.

Ok. Looks good, I'm ready to merge.
Let me know if you have time to resolve the conflicts.

I added another commit which introduces an optional base_kwargs argument to the Policy class. This allows to easily change parameters of the base model (e.g., hidden_size) from the main.py file.

And another commit in which I introduce a is_recurrent method to the policy and it’s base. This cleans up the code base (no hasattr calls) and could make it easier to try different types of recurrent  cells, e.g. LSTMs.


Hitting the back button in wifi settings saves

Going into the wifi settings and then hitting back saves 'changes' and reboots setting the password to whatever 3 is the default in the password field. This does not appear to be an intentional userflow.

Thanks, I'll look into it in a bit
Sent from Proton Mail mobile
-------- Original Message --------
On Sep 3, 2022, 12:52 PM, Jon Xuereb wrote:

Going into the wifi settings and then hitting back saves 'changes' and reboots setting the password to whatever 3 is the default in the password field. This does not appear to be an intentional userflow.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hi,
I recently installed the owie with a new CBXR.
Just wondering should the OW app read the correct percentage while riding,
or does it correct itself when stationary? My app shows every time I stop
the board then is recharging.
I did use thinner wire than the 26 gauge recommended, would that mess with
the OWIE reporting the correct percentage to the MB?
Thanks for your help,
Shane
On Fri, 23 Dec 2022 at 12:11 PM, lolwheel @.***> wrote:

Closed #59 https://github.com/lolwheel/Owie/issues/59 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/lolwheel/Owie/issues/59#event-8099720882, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A4MU655KVNYX2HJ6T4IKEBTWOTNYNANCNFSM6AAAAAAQEAUCMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@OnewheelNZ Firstly, please create a separate issue when reporting problems instead of hijacking an already closed issue.
Second, that is normal as when batteries are no longer under load, their voltage is able to normalize (and thus increase over their voltage under load). Based on the battery calculations being done by voltage, when the pack is under load it will report that it's less full than it actually is. By stopping and letting the cells 'rest' you're giving the pack time to get a reasonable reading.
No your <26AWG wire is not the issue and the board is not recharging magically while riding, that is expected behavior.


Viewing pull requests - Add option to show all PR checks withotut scroll

Description
With a larger amount of PR checks, there is a scrolling bar

Add the option to always show all checks without scroll.
Example URLs
none

Duplicate of https://github.com/refined-github/refined-github/pull/1007
We actually had this feature but it was dropped in https://github.com/refined-github/refined-github/pull/4919
The feature is still welcome, but it needs to be implemented more smartly, see the closing comment in https://github.com/refined-github/refined-github/issues/4809#issuecomment-944061001
I think the feature would look like:
delegate(document, 'the expand link', 'click', () => {
	if (is being expanded) {
		select('the list').style.height = 'auto'
	}
})


got error: Module not found: Can't resolve './Picker'

On Windows 10 run the below commands.

npm start serve
npm start

Select Run in the Web browser

Getting below error,
Failed to compile.
C:/workspace/react/Twitter-Clone-master/Twitter-Clone-master/native-base-theme/components/ListItem.js
Module not found: Can't resolve './Picker' in 'C:\workspace\react\Twitter-Clone-master\Twitter-Clone-master\native-base-theme\components'

Same problem here. Did you manage to get the web browser running?

Not yet, looks like during commits some components are removed so I am looking into that if I can find something in the old commits. Let me know if you can fix it or how I can fix it?


Hello brother, about how to associate the components in spdx json. json packages with Layers

I am a beginner and I am not sure if the type of question I submitted is correct
I want to retrieve which Layer the component comes from in spdx JSON. Can you help me

Hi @limaonet, we are looking at how to best expose this information. Issue #435 is where we will track the work, so if you are interested in this feature, vote and follow that issue. I'm going to close this as a duplicate.


Random shuffle?

Can you implement a random shuffle?

Interesting request but I think the use case is too rare considering most libraries are episodic in nature. If this is implemented in the server as a native feature I will be happy to incorporate it.

I think it is in fact native to Ubooquity. It exists in the web interface as well.

(I use it because, I have a rather large comics library, and I occasionally want to shuffle the selection, to see something new I might not have discovered by browsing A-Z.)


refactored application environment and state handling to remove the use of resolves. (#11)(DSK-46)

I built the Desktop from a fresh clone of the repository and it gave me this:

I cloned the branch from scratch then npm install and bower install and everything worked fine. I don't see this behavior. You mind listing your steps? Anymore insight that you can provide?

I've never run grunt start before; I just went right to creating the installer. Does that create the sass/css files?

Go ahead and remove grunt-cli from the list of dev dependencies

Let's disable the develop section of the app since it's not really working.


⚠ Bofrost.de has degraded performance

In 86b0215, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1558 ms

Resolved: Bofrost.de performance has improved in 5da3d5f after 21 minutes.


How to download the dataset？

Hi,this is very helpful.Thanks for your share.But how to download the dataset?I click on the multi-speaker link and it doesn't take me to the dataset page, and where can I download the single-speaker dataset?

Hi
https://github.com/emotiontts/emotiontts_open_db/tree/master/Dataset/SpeechCorpus
This is multi-speaker dataset link
If you want to use a multi-emotion dataset, visit http://aicompanion.or.kr/kor/main/#
Join this web and login
then go to '기술공유' -> 데이터 공유 -> '감정 음성합성 데이터셋'
You can download data

Hi @freshwindy https://github.com/emotiontts/emotiontts_open_db/tree/master/Dataset/SpeechCorpus This is multi-speaker dataset link
If you want to use a multi-emotion dataset, visit http://aicompanion.or.kr/kor/main/# Join this web and login then go to '기술공유' -> 데이터 공유 -> '감정 음성합성 데이터셋' You can download data

Thanks for your answer.But I still can't jump to the second link when I click it. Here is the page after I click it. In addition, whether the above two links contain a single speaker dataset？

Um... In korea we can enter this site.
I think you'd better use VPN.
Two link contain single speaker dateset.
But in my paper, i used only single speaker in first link.

Hi,I use vpn that node is located in Korea,but i can't jump to.I don't know whether there are other sharing methods for this dataset, such as Google Cloud Disk.

I understand that there is no other way to do it because of the license.

Close due to inactivity.


Fix crash due to terrain loading race condition

Follow up from https://github.com/CesiumGS/cesium/pull/11296
Technically after that change, due to this line, it was possible to return a promise which resolves to undefined if a requestTileGeometry request gets throttled by the requestScheduler.
TODO:

[ ] Unit test?

Thanks for the pull request @ggetz!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.


PureRenderMixin, don't depend on React.addons

I think you can get rid of the React.addons dependency and just include the ReactComponentWithPureRenderMixin via:
import PureRenderMixin from 'react/lib/ReactComponentWithPureRenderMixin';

Would send a PR but I'm not entirely clear on how your build system work :)

That would require consumers to use Browserify, which we don't want. Fortunately this won't be an issue at all as I'm removing the dependency on React.addons in an upcoming release.

Ah ok, nice! Looking forward to the next release :)

This is the case now. 0.5.4 does not require React with add-ons.


Update Dockerfile

Update docker file for change to cuDF. Point to new repo name of rapidsai/cudf

This is completed, will link commit once pushed


Youtube video rendering?

Plugin version 0.4.0
Obsidian 0.13.19
Cannot render a youtube video in Anki.
[video](youtube_url) syntax doesn't work.

#anki/start
some video?
#anki/---
[test](https://www.youtube.com/watch?v=l9nh1l8ZIJQ)
#anki/end

What kind of support this plugin has for videos?

Hey Juris,
That's just a link in Obsidian, so it will be rendered as a link in Anki. This is expected and intended behaviour.
The video embedding works similar to the image and and audio embedding: it embeds a local file.


chore: bump react-syntax-highlighter to ^10.1.2

Issue:
What I did
Upgraded react-syntax-highlighter to the latest.

Hi @nadiia, thank you for taking the time to contribute, it's very appreciated.
I've delayed upgrading this particular package for some time, because we're using api within the package to make lines clickable/navigable.
You can find this code here:
https://github.com/storybooks/storybook/blob/cedc39f1a2e24f892483f43387e880178c008747/addons/storysource/src/StoryPanel.js#L7
Upgrading breaks this feature (at least last time I checked).
Could you verify this feature works after upgrading and if not perhaps look for a fix?
Thanks, let me know if you need something 👍


How to know the argument used by a crash poc?

Hello,
I tried to reproduce the crash CarpetFuzz finded. The name of poc in crashes is like "crashes/id:000000,sig:07,src:000000,argv:000334,op:argv,pos:0".
I guessed the used argv is about at line 334/335/336 in ranked_stubs.
I run the following command to extract the argv.
$ cat ranked_stubs_xmllint.txt  | head -n 334
$ cat ranked_stubs_xmllint.txt  | head -n 335
$ cat ranked_stubs_xmllint.txt  | head -n 336

I copy and run like the following picture.

But I still cannot reproduce the crash.
May I ask how to reproduce the finded crashes?
Thanks

Hi, the format of the file ranked_stubs is
1 [number of argvs]
2 argv:000000
...
336 argv:000334
...

so you should use line 336.
If you are still unable to reproduce the crash, I have two suggestions,

Try testing with an ASAN-compiled executable instead of the CarpetFuzz-compiled one
Check the memory limit, as mentioned in page,

If still irreproducible, I think it may be a false positive of AFL++.


add check-graylog-streams script

Pull Request Checklist
Is this in reference to an existing issue?  No
General

[x] Update Changelog following the conventions laid out on Keep A Changelog

[x] Update README with any necessary configuration snippets

[x] Binstubs are created if needed

[x] RuboCop passes

[x] Existing tests pass

Purpose
This adds an additional check called "check-graylog-streams.rb" that looks for and alerts on paused streams.  Graylog will automatically pause a stream when the processing time for it gets too long as a defense measure (long GC, expensive criteria, etc..).
Since streams are applied only at event index time, there's no way to retroactively apply them after the stream is unpaused, so quick notification and repair is important.
Known Compatibility Issues
None known, this is new script.

gentle nudge for review

Derp, yes, thank you.  This came from a CentOS 5 system on ruby 1.8.7 and I breezed past that when porting.  Upstreaming is hard.

nudge for re-review

LGTM. Thanks!

Thank you!


Chrome made some serious changes on its latest version v19. unfortunately our app is not working correctly now.

I made some changes in flutter app, Now flutter web is working fine but apps have same issue.
I think webview lib is pointing some other version of chome, so that it is not working for apps.
Please let me know how can i  update the latest chrome in this lib.

Hi @Flaming19
webview_flutter doesn't call a specific Chrome version on a device, whenever default webview is available on your mobile system, is used. This is not related to webview_flutter
Closing, as this isn't an issue with webview_flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


chore: upgrade chrono to 0.4.38

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
As titled.
Some methods in NativeDateTime are deprecated in chrono and this PR adopts the new methods accordingly by following chrono's doc.
This currently blocks OpenDAL upgrade (#17198)
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Copy paste the doc in chrono 0.4.38 here for references:
    /// Makes a new `NaiveDateTime` corresponding to a UTC date and time,
    /// from the number of non-leap seconds
    /// since the midnight UTC on January 1, 1970 (aka "UNIX timestamp")
    /// and the number of nanoseconds since the last whole non-leap second.
    ///
    /// The nanosecond part can exceed 1,000,000,000 in order to represent a
    /// [leap second](NaiveTime#leap-second-handling), but only when `secs % 60 == 59`.
    /// (The true "UNIX timestamp" cannot represent a leap second unambiguously.)
    ///
    /// # Errors
    ///
    /// Returns `None` if the number of seconds would be out of range for a `NaiveDateTime` (more
    /// than ca. 262,000 years away from common era), and panics on an invalid nanosecond
    /// (2 seconds or more).
    #[deprecated(since = "0.4.35", note = "use `DateTime::from_timestamp` instead")]
    #[inline]
    #[must_use]
    pub const fn from_timestamp_opt(secs: i64, nsecs: u32) -> Option<NaiveDateTime> {
        Some(try_opt!(DateTime::from_timestamp(secs, nsecs)).naive_utc())
    }

    /// Returns the number of non-leap seconds since the midnight on January 1, 1970.
    ///
    /// Note that this does *not* account for the timezone!
    /// The true "UNIX timestamp" would count seconds since the midnight *UTC* on the epoch.
    #[deprecated(since = "0.4.35", note = "use `.and_utc().timestamp()` instead")]
    #[inline]
    #[must_use]
    pub const fn timestamp(&self) -> i64 {
        self.and_utc().timestamp()
    }

    /// Returns the number of non-leap *milliseconds* since midnight on January 1, 1970.
    ///
    /// Note that this does *not* account for the timezone!
    /// The true "UNIX timestamp" would count seconds since the midnight *UTC* on the epoch.
    #[deprecated(since = "0.4.35", note = "use `.and_utc().timestamp_millis()` instead")]
    #[inline]
    #[must_use]
    pub const fn timestamp_millis(&self) -> i64 {
        self.and_utc().timestamp_millis()
    }


Added resource width and height information to the onFastImageLoad an…

…d onFastImageLoadEnd events
Instead of an empty map, it may be useful to have some information about the newly loaded resource.
Since a user doesn't know the dimensions of a remote resource at load time, providing the dimensions on load end allows the user to adjust the view accordingly if required.

@DylanVann any chance to have this PR merged soon?

@jeremyclee I believe you forgot to improve the documentation and a iOS implementation for this feature, that way the PR will have more chances to get merged, imho.

@jeremyclee I'm getting an error with your changes, look:
                                                   Process: com.powerhrg.connect, PID: 26188
                                                   com.facebook.react.bridge.ObjectAlreadyConsumedException: Map already consumed
                                                       at com.facebook.react.bridge.WritableNativeArray.pushNativeMap(Native Method)
                                                       at com.facebook.react.bridge.WritableNativeArray.pushMap(WritableNativeArray.java:55)
                                                       at com.facebook.react.bridge.Arguments.fromJavaArgs(Arguments.java:189)
                                                       at com.facebook.react.bridge.JavaScriptModuleRegistry$JavaScriptModuleInvocationHandler.invoke(JavaScriptModuleRegistry.java:94)
                                                       at java.lang.reflect.Proxy.invoke(Proxy.java:393)
                                                       at $Proxy4.receiveEvent(Unknown Source)
                                                       at com.dylanvann.fastimage.FastImageViewManager$1.onResourceReady(FastImageViewManager.java:104)
                                                       at com.dylanvann.fastimage.FastImageViewManager$1.onResourceReady(FastImageViewManager.java:62)
                                                       at com.bumptech.glide.request.GenericRequest.onResourceReady(GenericRequest.java:522)
                                                       at com.bumptech.glide.request.GenericRequest.onResourceReady(GenericRequest.java:507)
                                                       at com.bumptech.glide.load.engine.EngineJob.handleResultOnMainThread(EngineJob.java:158)
                                                       at com.bumptech.glide.load.engine.EngineJob.access$100(EngineJob.java:22)
                                                       at com.bumptech.glide.load.engine.EngineJob$MainThreadCallback.handleMessage(EngineJob.java:202)
                                                       at android.os.Handler.dispatchMessage(Handler.java:98)
                                                       at android.os.Looper.loop(Looper.java:148)
                                                       at android.app.ActivityThread.main(ActivityThread.java:5417)
                                                       at java.lang.reflect.Method.invoke(Native Method)
                                                       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
                                                       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

I created the PR #103 which has a more complete implementation of this feature, any comments are welcome.

@DylanVann Please merge this PR if it's OK. Thanks.

@ararog Are you still getting that error? It should have been fixed by this commit: https://github.com/DylanVann/react-native-fast-image/pull/107/commits/6c1d1f805b6cfff40c28c229a88037d05048faa2
I'll work on the iOS fix, then hopefully this will get merged in

Updated the PR with the fix for iOS

@jeremyclee yes, I'm still getting this error, with with this commit.

@jeremyclee on PR #140 I created another event called onReadDimensions, and of course, a single map instance, everything went well, but I believe your approach is more performant.

Merged. Thank you for working on this @jeremyclee

@DylanVann, Thanks for merge this awesome feature :+1:
@jeremyclee Thanks for this useful feature. :+1:
Guys can you add some example or doc about this? (or guide me here)
I don't know how we can get width and height for a <FastImage> I want to fit each image to device width (with auto height based on image dimension and ratio like what we see in Instagram app)

I checked again here and works fine, thanks!

@ararog can you post some snippet about how you use this?

`
_onLoadEnd = e => {
console.log(e.nativeEvent);
};
render() {
return ;
}
`

@ararog Thanks, but for me it's not working. It's just print a empty object for me ( {} ) :thinking:

There's some documentation on how it can be used here:
https://github.com/DylanVann/react-native-fast-image#onload-event--void
You should ideally know what the aspect ratio or dimensions of the displayed image are before showing the image. Then use an appropriate resize mode. This will prevent UI jank.
Also Instagram shows all images as square (if you know what's good for you you'll do the same 😆, so much pain doing variable aspect ratio UIs)
Let me know if it's still not working with the latest version v4.0.8.


Macbook M1 Building from source.

curl-impersonate/build/boringssl/crypto/../include/openssl/arm_arch.h:96:5: error: "unsupported ARM architecture"
error "unsupported ARM architecture"
Anyway we can successfully build this from mac m1?

I was able to run this on the server ubuntu without any problems with precompiled library and it worked just fine when executing PHP file, but however for local development. I was trying to use libcurl mac build but it somehow doesn't work when running it within apache2 env.

With #217 merged, curl-impersonate now builds fine on Mac (tested on Mac M2).
I don't know exactly what went wrong in your case; please share more details if this issue persists.


SQL Error in Labels Downloader: UNIQUE constraint failed

Describe the bug
I've been running scraper with the metadata update option and all posts to get all my folders up to date with the db's. However, on some accounts the labels scrape will succeed but when it tries to put them in the DB it fails, based on the debug logs. It will also terminate the download / metadata update loop, leading to a partially populated db with a lot of NULL fields, including Downloaded.
To Reproduce
ofscraper --log debug -mu --posts all --username userImScraping --action download
Expected behavior
It to run through all media successfully.
Screenshots/Logs
https://paste.passtheheadphones.me/?189aa025269ad731#FRLiqHT4CZtE5B4bSA5GVkCpm7mCrNMHLuGVPvuBrnPP
Desktop (please complete the following information):

OS: Windows 11
Browser: Chrome 121
Python 3.11.4

Just noticed this might be a duplicate of https://github.com/datawhores/OF-Scraper/issues/287. I'll leave this open in case it can help. The same error occurs on 3.7.1 and 3.8.0dev0.

I made a dupelabel branch
Unable to tell why this is happening currently

This issue is referenced in 287 so I close it
so things can be consolidated


Menu not responding after Data Import->select Owner

I found this issue long time ago (days, maybe weeks), only report now:

Menu not responding after Data Import -> choosed owner, please check the animgif

Are you sure that you have the latest version and everything is compiled?
You may reset your browsers cache also from time to time.
If you use Chrome I would suggest using this extension:

I can't reproduce on neither dev or live. If you try the live demo it's the same?

Usually I do (clear cache).

I did a new 2.3.4 install from scratch.
Now I cleared cache.
Still not working, same effect.
I use Brave browser (chromium under the hood).

Did you try the demo version? https://laravel-enso.com ?
However, I already tried Chrome, Safari and Firefox and all work ok.

Hmmm, interesting, on my instance, nor brave, nor chromium is working correctly, while in your demo both are working as excepted.
Apache cache or postgresql (I did not find exception or error on laravel debug console)? I check it.

No idea, what is the problem. I cleared everything in chromium/brave, on the server memcached etc. stop/start, emptied etc.
Same issue.

Ok, keep it open but low priority, later I will check it again.

Closing due to inactivity.


Make crow use trees for variables and datatypes

Linear lookups are too taxing on speed. Crow needs to use trees for faster lookups at the cost of using slightly more memory and taking slightly more time to add or remove variables (which in the long term shouldn't mean much since we look up variables more than we add or remove them.)

Structures should also internally be switched to using trees, or at least be sorted.

Crow no longer uses a variable list for variables, instead being put into closures. While this isn't as fast as a tree (this may be looked into in the future) it is fast enough for now. Structures will be switched to trees, however variables will not.


Add prestart tasks waiting for dependent services.

closes https://github.com/fredrikhgrelland/data-mesh/issues/33
This PR adds two consul-checks as prestart tasks. It utilize the consul docker image ( but it really could be any image containing jq ). and it works as follows:
task "waitfor-hive-metastore" {
      lifecycle {
        hook = "prestart"
      }
      driver = "docker"
      resources {
        memory = 32  #reduced memory footprint
      }

      template {
        destination = "tmp/service.json"
        # Render a template containing information on service as a json file
        data = <<EOH
          {{- service "hive-metastore" | toJSON -}}
        EOH
      }
      config {
        image = "consul:latest"  #Consul image contains jq.
        entrypoint = ["/bin/sh"] # Run a shell
        volumes = ["tmp/service.json:/local/service.json" ] #Mount the templated file into the container
        # run jq on file created by the template stanza
        # -e exits 0 if filter succedes. In this case "all service status is passing"
        args = ["-c", "jq </local/service.json -e '.[].Status|select(. == \"passing\")'"]
      }

Ref: https://github.com/fredrikhgrelland/data-mesh/issues/33


Mailcatcher 0.8.2 is not published to Docker Hub

The latest version on Docker Hub is 0.8.1. Could you update your published image with the latest release?
https://hub.docker.com/r/sj26/mailcatcher

Appreciate you maintaining a version for the community @spaquet! I prefer to use an official image for my use case though.
@sj26 any chance of updating this? Or is there a way I could help get it done?

MailCatcher v0.8.2 is now available on Docker Hub.
I'm interested in automating the building of images. They would need to be built based on the rubygem being pushed I guess? Although I do also upload the gem binary as github releases. Perhaps the Docker image could be built from the github release event and attached gem binary? Something to look at in a separate issue/PR.


Added support for Guzzle timeout to OllamaChat

Sometimes we need to tweak the timeouts of connecting to ollama but it's not possible ATM, this PR adds possibility to set timeout options conneting to ollama

tests fails only because of CVE-2024-51736 on symfony/process, unrelated to this PR

thanks @fballiano !


Composing of multi-variate time series supported

Main changes:

API wrapper for multivariate time series added (can be taken into account during API refactoring by @v1docq)
Metocean forcasting case simplified

The multivariate TS are represented as several datasources in multi-modal chain.

Hello @nicl-nno! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file fedot/core/data/data_split.py:

Line 168:9: E265 block comment should start with '# '
Line 170:9: E265 block comment should start with '# '
Line 239:33: W292 no newline at end of file


ArangoDB on Windows hanging...

Besides my main Linux installations of ArangoDB, I also occasionally use a local installation on my Windows 7 (x64). Mainly for running ArangoDB-PHP tests also on Windows.
So, I upgraded the Windows installation a few days ago, from 1.3.1(I think it was) to 1.4.0.
Today I was testing the ArangoDB-PHP Driver and the tests were suddenly failing (only locally on the windows machine). I closed the Batch window, and reopened it, and the tests were running successfully. I'd like to note here, that I hadn't turned off my machine since yesterday, so that batch window was open for at least 12 hours if I remember correctly.
I also noticed that the database files are not deleted from the data directory (The ones that are created and deleted through the ArangoDB-PHP test suite.)
I couldn't attach a zip file here, so I am sending the zipped data directory via mail to @jsteemann (Please also check Spam folder :smile: ) for analysis.

I had this hit me in 2.6.1 on windows 64. Was caused by having old deleted databases lying around. I hade to manually delete the the ones reported by the arandodb --upgrade.


feat: add prometheus recording rule to aggregate measurements

Added a rules file to create a recording rule that aggregates the metric sent by the host-metering client to indicate CPU hours on a per-hour resolution.
As part of the process changed the configuration of the local prometheus container to use the aggregated metric and added a makefile action to lint the prometheus configuration.

I think we could specify the image more specifically to avoid issues. The new make command worked just fine for me.

Specified the container registry too. Makes sense that we will pull from a specific registry.

Merging, since the PR is approved.


Updated godeps

Pointing to 1.8.0
@aledbf please double check at your convenience.

This change is

/lgtm

Thanks @aledbf.  Will run some manual tests before merging.

Coverage decreased (-0.02%) to 43.473% when pulling ebfaf1966f60332fedcd17f35ee7419f9fbd6113 on vendor-1.8 into 15e885fd4100d252207f41ff4d93cf9d6cab0e8f on master.


[FancyZones] Per-App: Lock Preferred Zone

Description of the new feature / enhancement
It would be nice to force specific apps to always open in a specified zone.
Scenario when this would be used?
Currently when apps can have multiple instances, the preferred zone gets overwritten by the state of the last-closed instance.

Open Notepad (instance 1) and snap to Zone 1
Open Notepad (instance 2), it auto-moves to Zone 1
Move instance 2 to Zone 2
Close instance 1
Close instance 2 (no instances open, Zone 2 is the last-used zone)
Open Notepad (instance 3), it auto-moves to Zone 2

It isn't a major issue in lightweight apps, since I can just bump them over quickly; but in heavier apps like Visual Studio it can get cumbersome when it snaps to the last-closed location instead of my primary zone/monitor. It can also come up in lightweight apps that have smaller windows like splash screens or launchers, causing them to sometimes get "lost in the noise" if they snap to a monitor I'm not expecting.
Example of new behavior:

Open Notepad (instance 1) and snap to  Zone 1
Open Notepad (instance 2), it auto-moves to Zone 1
Open FancyZones settings, and check the "Lock Zone" checkbox next to Notepad.exe (Zone 1 is now the preferred zone for Notepad)
Move instance 2 to Zone 2
Close instance 1
Close instance 2 (no instances open, Zone 2 is the last-used zone)
Open Notepad (instance 3), it auto-moves to Zone 1

Unchecking the "Lock Zone" checkbox would revert to normal behavior (the first example above).
Supporting information
I'm not comfortable enough with the codebase to do a pull request at this time; but:
It looks like this would basically add a boolean setting on the history data items which prevents updates to the zone-index-set in the app-zone-history.json file for a given application path; and a UI update to allow users to edit that setting (probably just a list of paths and check boxes, filtered by the current device ID?).
Then add a check for that boolean as a condition in AppZoneHistory::SetAppLastZones and AppZoneHistory::RemoveAppLastZone.

Does "Move newly created" not do this for you?

/needinfo

Multi-instance apps appear to set the "last known zone" to the last instance closed, so it doesn't match this particular pain point.
E.g. most days I might open Visual Studio in Zone 1.  Occasionally I might open a second instance of Visual Studio in Zone 2 (like when referencing another solution). When shutting down, if I accidentally close Zone 1's instance first, the next time I start up it will open in Zone 2 instead.
Easy to correct for in lightweight apps, just a pain point for heavy apps where the time to launch and manually move them can be longer (and annoying if the last-closed-zone happens to leave them blocking other apps in that zone while they launch).