Expose net_ipv4_create, net_ipv6_create, and net_udp_create in standard header

Currently some of the modem drivers (drivers/modem/hl7800.c and drivers/modem/wncm14a2a.c) are calling net_ipv6_create, net_ipv4_create and net_udp_create which requires having:
zephyr_library_include_directories(${ZEPHYR_BASE}/subsys/net/ip) in CMakeLists.txt.  If these are suppose to be APIs that can be called on the networking subystem we should look to move them into include/net/ somewhere.

@jukkar was looking at various places we do zephyr_library_include_directories and if there is some implied cleanup that needs to happen.  This is one area related to networking interfaces and modem.  Don't really know much about it beyond that.

I would not want to create new APIs for these, as they are meant for internal use. Is there some issue if zephyr_library_include_directories is used in the driver?

I would like not to expose these APIs by putting them to include/net, as they are meant for internal use. Is there some issue if zephyr_library_include_directories is used in the driver?
Calling these private APIs from drivers is ok, but I would like to avoid exposing them to applications.

There's not a specific issue, just trying to ensure some level of encapsulation - even if its between a subsystem a driver.

@jukkar will close this out, we need a general solution/agreement on how subsystems export APIs to other parts of zephyr (not application).


Error /vendor/laravel/framework/src/Illuminate/Console/Scheduling/ScheduleListCommand.php

Laravel Version: 8.78.1

PHP Version: 8.0.8

Database Driver & Version: 10.5.13-MariaDB

Description:
Trying to run php artisan schedule:list and get an error that TimeZone is not correct.
I have to change this :
                (new CronExpression($event->expression))                             ->getNextRunDate(Carbon::now()->setTimezone($event->timezone))                             ->setTimezone($this->option('timezone', config('app.timezone')))                             ->format('Y-m-d H:i:s P'),
To this :
                (new CronExpression($event->expression))
                            ->getNextRunDate(Carbon::now()->setTimezone($event->timezone))
                            ->setTimezone(new DateTimeZone(config('app.timezone')))
                            ->format('Y-m-d H:i:s P'),

And works :
Steps To Reproduce:
Edit config/app.php , and change time zone to America/Argentina/Buenos_Aires

This is fixed in https://github.com/laravel/framework/pull/40304
Just wait for the next release!

This is fixed in #40304
Just wait for the next release!

Sorry , is that i had a challenge for a job interview and when i had this issue , i didn't  know what happend.
Thanks !


fix: hide in progress audit events from the audit event tables

Closes #364.
Changes

Hides in progress audit events by introducing an inProgress property to the AuditEvent schema type.

I also have migrations in progress - so we will clash - but best you merge this one in and then I will regenerate my migrations


After installing cilium 1.7.1 on minikube + cri-o, coredns never stops respawning

Bug report
Basically, coredns will terminate and start until the point of running. Then, the loop starts.
General Information

Cilium version (run cilium version): 1.7.1
Kernel version (run uname -a): Linux introdesk.g02.org 5.5.8-200.fc31.x86_64 #1 SMP Thu Mar 5 21:28:03 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Orchestration system version in use (e.g. kubectl version, Mesos, ...): Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.4", GitCommit:"8d8aa39598534325ad77120c120a22b3a990b5ea", GitTreeState:"clean", BuildDate:"2020-03-12T21:03:42Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}

Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.3", GitCommit:"06ad960bfd03b39c8310aaf92d1e7c12ce618213", GitTreeState:"clean", BuildDate:"2020-02-11T18:07:13Z", GoVersion:"go1.13.6", Compiler:"gc", Platform:"linux/amd64"}

Link to relevant artifacts (policies, deployments scripts, ...)

How to reproduce the issue

start minikube with cri-o: https://kubernetes.io/docs/setup/learning-environment/minikube/#starting-a-cluster-on-alternative-container-runtimes
install cilium:

@renich did you follow this tutorial to install cilium?
https://docs.cilium.io/en/v1.7/kubernetes/configuration/#crio

Not that one. Let me follow it and I'll report back in a moment.

OK, confirmed. It works fine. I suggest adding a note to this at: https://docs.cilium.io/en/v1.7/gettingstarted/minikube/. No way I could've known that.

BTW, thank you very much for the support.


[#time hours] only in non military time

Trying to use hours as follows. Seem to be getting non military time hours. Don't see a way to get military time or determine via an AM/PM boolean in the #time documentation.
search
  [#time hours minutes seconds]
  test = hours
bind @browser
  [#div text: "{{test}}"]

Oop, looks like hours-24 exists. looks like this is a problem with the documentation.

Closing this and openning https://github.com/witheve/docs/issues/10


How to do monitor for an event on more than one input?

How do you extend the gpioevents.rs example to handle multiple input pins using event handlers?
EDIT: To clarify, I don't really want to have to follow the poll methodology in monitor.rs as polling IO pins is inefficient and prone to missing changes in IO state.
-Andy.

Polling using your mechanism of choice is fine -- Providing a way to conveniently integrate this lib with tokio is not in the lib today but is something that probably should be added (probably behind a feature flag).
Events on the GPIO are queued using cdev so you will not miss them unless you really are not keeping up in your userspace application.  Each event comes with a timestamp for interrupt context which is what should be used if you care about when an event really happened.
If the lines are on the same gpiochip you can read multiple using a MultiLineHandle (see https://github.com/rust-embedded/gpio-cdev/blob/master/examples/multiread.rs).  The equivalent of events on this type of handle is not implemented at present although I think it should be possible (though I haven't tested this).

The equivalent of events on this type of handle is not implemented at present although I think it should be possible (though I haven't tested this).

Could you please provide a pointer to the docs of the C API that specify how to do it?
I am really interested in this and could provide a PR.
I would test it reading the two lines of a quadrature encoder (this is the use case I have).

I don't think #35 should have closed this issue.
While it's possible to monitor multiple lines for events, it's not nearly as ergonomic as the single line API.
I would personally love to see something like the Line::events() and LineEventHandle iterator implemented for Lines and (a new?) MultiLineEventHandle.


Fuzz testing

Hello,
I am looking into this library to use for serialization of data. I have used fuzz testing for testing similar libraries to find bugs. Based on the test case in cbor_test.c i wrote a simple fuzz testing suite. Using this i have detected some possible crashes. However, i don't know much of the API and how it is intended to be used. If i use it wring this might not be an issue.
Also, i noticed that if i try to encode the decoded data, i need a bigger buffer: cbor_libfuzzer.c#L33. Any ideas why?
  Multiple pull requests (#49, #25) have been created to address it, but none merged.


Lint suggestion: warn on multiple unqualified/exhaustive imports

Something that I encountered in PureScript that I really liked is that it will warn you if you have multiple "exhaustive" imports (im sure there is a better term for this, but I don't know it).
For example, you would get a warning for:
import Prelude
import Foo

but you could change it to:
import Prelude
import qualified Foo

or
import Prelude
import Foo (bar, baz)

This ends up making projects a lot nicer to work with, as its easy to see where various imports come from.

Thanks for the idea @joelmccracken - however, I have two concerns:

I don't like this style. It definitely isn't universal. So it could only ever be at most an opt-in default-off hint.
There's no easy way to suggest a replacement from a non-exhaustive style, since HLint doesn't have enough information.

While I'd probably take a simple patch, off-by-default hints don't tend to get much usage. You might be better off looking at Ghcide, the IDE, which has utilities to show imports with what comes from where, and to change them to be fully explicit, and could certainly gain the action to make an import explicit as a refactoring option. In that situation it has all the information and being an editor has a lot larger surface area to suggest/hint at more things.

Gotcha; I’ll do that. Thanks!


update sticky add custom-element

The extension 'amp-sticky-ad' is referenced in version '0.1' which is a deprecated version. Please use a more recent version of this extension. This may become an error in the future.

Hey @okabrionz , thanks a lot  👍


map_threshold doe not do two-tailed analyses

Currently  nistats.thresholding.map_threshold() only allows one-tailed test when thresholding stat maps. It would be useful to be able to do two-tailed tests.

I also think that map_threshold() function doesn't allow for cluster-level inference. Cluster threshold is only applied after voxel level inference, to filter out disconnected voxels. Cluster-level inference should be done other way around:

clusters are formed by applying cluster forming threshold
significance of clusters is is determined by comparing their size to cluster size threshold

Since it is way more popular approach to making inference on images. I can try to implement this and submit PR. But I would need authors suggestions if it should be implemented within map_threshold() function or separate function. Suggestion regarding function parameter and their names would also be useful to stay consistent with the project.

Note that significance requires you to have a null distribution for cluster size, that can only be obtained by permutation. Hence any type of cluster-level infrence should be done in non-pramaetric frameworks.
Note that #402 introduces another type of cluster inference based on "allin resolution inference approach".
Best,
B

De: "Devon Hjelm"<EMAIL_ADDRESS>À: "nistats"<EMAIL_ADDRESS>Cc: "Subscribed"<EMAIL_ADDRESS>Envoyé: Mercredi 16 Octobre 2019 14:58:14
Objet: Re: [nistats/nistats] map_threshold doe not do two-tailed analyses (#362)

I also think that map_threshold() function doesn't allow for cluster-level
inference. Cluster threshold is only applied after voxel level inference, to
filter out disconnected voxels. Cluster-level inference should be done other
way around:

1. clusters are formed by applying cluster forming threshold

significance of clusters is is determined by comparing their size to cluster
size threshold

Since it is way more popular approach to making inference on images. I can try
to implement this and submit PR. But I would need authors suggestions if it
should be implemented within map_threshold() function or separate function.
Suggestion regarding function parameter and their names would also be useful to
stay consistent with the project.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, [
https://github.com/nistats/nistats/issues/362?email_source=notifications&email_token=AABZHVVYGAZ6XM7AFTBNIUDQO4FWNA5CNFSM4H35LSK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBMMANA#issuecomment-542687284
| view it on GitHub ] , or [
https://github.com/notifications/unsubscribe-auth/AABZHVUBGIVVEYV37HSW3PTQO4FWNANCNFSM4H35LSKQ
| unsubscribe ] .


Enhancement: Add version maven jar plugin

Types of changes

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

And

[x] This Pull Request DOES NOT target the main branch on this repository (if so it must point to
a feature branch or to the release branch).
[x] I have read the CONTRIBUTING
document.
[ ] I have added/updated tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] There are no other open Pull Requests
for the same update/change.

Fixes

closes #116

Codecov Report

Merging #117 (ca6e84e) into release (f735cef) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             release     #117   +/-   ##
==========================================
  Coverage      98.79%   98.79%
  Complexity       174      174
==========================================
  Files              9        9
  Lines            993      993
  Branches          38       38
==========================================
  Hits             981      981
  Partials          12       12

Flag
Coverage Δ

unittests
98.79% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 e84cfed...ca6e84e. Read the comment docs.


Error in training on own data

I got the following error on trying to train with my own data. I followed the instructions in README.md .
Can you explain what is happening?
Traceback (most recent call last):
File "birdCLEF_train.py", line 791, in
loss = train_net(image_batch, target_batch, lr)
File "/usr/local/lib/python2.7/dist-packages/theano/compile/function_module.py", line 871, in call
storage_map=getattr(self.fn, 'storage_map', None))
File "/usr/local/lib/python2.7/dist-packages/theano/gof/link.py", line 314, in raise_with_op
reraise(exc_type, exc_value, exc_trace)
File "/usr/local/lib/python2.7/dist-packages/theano/compile/function_module.py", line 859, in call
outputs = self.fn()
MemoryError:
Apply node that caused the error: Elemwise{sqr,no_inplace}(Elemwise{Sub}[(0, 0)].0)
Toposort index: 156
Inputs types: [TensorType(float64, 4D)]
Inputs shapes: [(128, 128, 64, 128)]
Inputs strides: [(8388608, 65536, 1024, 8)]
Inputs values: ['not shown']
Outputs clients: [[Sum{axis=[0, 2, 3], acc_dtype=float64}(Elemwise{sqr,no_inplace}.0)]]

This is a memory error, typically raised when the GPU runs out of memory. There are basically two things you can do: Reduce the Batch Size or reduce the net complexity (less filters). You can also reduce the image resolution, but that requires more adjustments and might not be as applicable as the other two strategies.

I'm young researcher from srilanka. I got the following error on trying to train with my own data. I followed the instructions in README.md .
ERROR (theano.gof.opt): Optimization failure due to: local_abstractconv_check
ERROR (theano.gof.opt): node: AbstractConv2d{convdim=2, border_mode='half', subsample=(2, 2), filter_flip=True, imshp=(None, 1, 256, 512), kshp=(64, 1, 7, 7), filter_dilation=(1, 1), num_groups=1, unshared=False}(input, W)
ERROR (theano.gof.opt): TRACEBACK:
ERROR (theano.gof.opt): Traceback (most recent call last):
theano.gof.opt.LocalMetaOptimizerSkipAssertionError: AbstractConv2d Theano optimization failed: there is no implementation available supporting the requested options. Did you exclude both "conv_dnn" and "conv_gemm" from the optimizer? If on GPU, is cuDNN available and does the GPU support it? If on CPU, do you have a BLAS library installed Theano can link against? On the CPU we do not support float16.
Can you explain this issue.
project built in windows 10 64 bit on the CPU


Initial implementation of Store API

Hello!
As per #12 I created the basic implementation of the Store API.
Theres a single class method called app_details which accepts an appid and returns the details of the app.
Let me know if you need anything else on my end, e.g. refactoring/additional edge cases to test etc.

Created new const in Helpers module, STORE_API_BASE_URL
Created new Steam::Store module
Created Steam::Store. app_details which returns the details of a specific App by ID
Added test coverage for Steam::Store. app_details

Thanks!

@corykitchens some conflicts, sorry for the delay, if you can clean that up rq and change anything you'd like to be released in the next rev I'll probably package new ver tonight

Hey, @bhaberer thanks for the feedback. I've resolved the merge conflicts, verified tests passed, and pushed my changes up. Let me know if you need anything else on my end. 🙏


Some Rider painpoints, and other things

Hi, I have spent the weekend rewriting one of my work-projects to use TUnit instead of XUnit, and I have some experiences to share:
My setup:

JetBrains Rider 2024.2.5
Build #RD-242.22855.90, built on September 19, 2024
Runtime version: 21.0.3+13-b509.15 amd64 (JCEF 122.1.9)
Windows 11.0
.NET Core v8.0.4 x64 (Server GC)

Experiences:

Over-all, I love it, the time to run tests is phenomenal compared to XUnit, the "hook in at any level" is awesome
Double-clicking to navigate to test don't work, (right-click-> Jump to source and Show source, also don't work)
Messages written to test-output, TestContext.Current.GetDefaultLogger().LogInformation("LOG ME!"); and await TestContext.Current.OutputWriter.WriteLineAsync("OUTPUT ME!");, don't get shown in actual test output after test finishes in Rider. What  is outputted when running CLI. I Use the output of tests a lot
await Assert.That(_this).IsNotNull(); is a little verbose, but Assert.IsNotNull(_this);, or _this.Should().NotBeNull();, aren't that different
I miss fluent assertions, (saw your PR, so just an observation)

Suggestions:

Wrap Coverage and Reports in TUnit nugets, and leverage nuget to create the config needed with acceptable "defaults" for TUnit. Like you have all the modules for ModularPipelines, it would be nice to have defaults for e.g. DevOps

Thank you for making the effort creating this, and I hope you will continue. I'd offer to help, but for the next couple of months, I don't have the bandwidth 😞

Thanks very much! Appreciate it :)
I believe 2 and 3 are things for Jetbrains to fix.
Go to source works in Visual Studio, so the data is there, I just don't think Rider is using it.
And the test output window I think still hasn't been developed yet, as the testing platform is still very new. They recently added standard output and standard error values, which I have set. I think (but not 100%) that that should drive the test output window in IDEs but the IDE itself will have to look at that value and implement it.

@frankhaugen your experience sounds great. I'm also thinking about to switch to TUnit.
If you write an issue for Rider, let it know me here and I will support it. Thanks.

@frankhaugen @SaschaBa I've raised issues with Jetbrains:
Test output: https://youtrack.jetbrains.com/issue/RIDER-117847/Testing-Platform-Test-Output-not-showing-in-IDE
Jump to source: https://youtrack.jetbrains.com/issue/RIDER-117848/Testing-Platform-Navigate-to-test-does-not-work

@thomhurst wow, you are very fast. I voted both issues.

@frankhaugen out of curiosity what was the difference in speed between xUnit and TUnit for you?

@thomhurst wow, you are very fast. I voted both issues.

Looks like they've both been assigned so hopefully we'll see a fix soon!

:-) yes, and they got both a high priority

@frankhaugen out of curiosity what was the difference in speed between xUnit and TUnit for you?

So I did some really fiddly measuring and I got 13,5 % quicker over 50 runs. I feel it was quicker. I wouldn't count those numbers as "quality assured" but I ran devops pipline on the code before the rewrite and the one after 100 times, (50 of each type).
I feel on my computer its much quicker than 13%

@frankhaugen @SaschaBa I've raised issues with Jetbrains:
Test output: https://youtrack.jetbrains.com/issue/RIDER-117847/Testing-Platform-Test-Output-not-showing-in-IDE Jump to source: https://youtrack.jetbrains.com/issue/RIDER-117848/Testing-Platform-Navigate-to-test-does-not-work

Jetbrains have marked the issue as fixed.
Says it'll be available in the Next 2024.3 public build. 🤞

It's working now for me in 2024.3 EAP 4. So you can download that if you don't want to wait. Otherwise, it'll just be in the main 2024.3 release.
Because locations are working, you can execute the test using the green play button next to it like xUnit/NUnit now:

And the output window is now showing 😄

Going to close this for now.

@thomhurst Does it work for you for parameterized tests as well?


bad accessibility

i want to use the new windows termenal in my daly work but it doesn't work with my NVDA screen reader and it give me a bad accessibility errors inshort the terenal like plank page normaly i use git bash as a termenal but doesn't have that mutch feachers  <!--
🚨🚨🚨🚨🚨🚨🚨
I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:

If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce

Expected behavior

Actual behavior

You're gonna have to give us a few more details than that - what version of NVDA are you using? What exactly are you trying to do, what do you expect to happen, and what's actually happening? Having specific repro steps is the only way we can help investigate an issue. @carlos-zamora has done a ton of work in the last year to make sure the Terminal supports NVDA and Narrator, so if there's anything wrong, I'm sure he'd be happy to help fix it ☺

Hi @ahmedSamy31,
We've been working with NVDA. They added support for Windows Terminal in NVDA 2020.2 Beta 1. You can download the updated version at https://www.nvaccess.org/post/nvda-2020-2beta1/.
You might have to enable the following on NVDA:

Go to Preferences, then Settings
Select the "Advanced" category
Under "Microsoft UI Automation", enable "Use UI Automation to access the Windows Console when available".

Tagging @codeofdusk to verify if I'm missing anything.
Separately, @cinnamon-msft maybe we should add this to our docs page under troubleshooting?

Hi @ahmedSamy31,
We've been working with NVDA. They added support for Windows Terminal in NVDA 2020.2 Beta 1. You can download the updated version at https://www.nvaccess.org/post/nvda-2020-2beta1/.
You might have to enable the following on NVDA:

Go to Preferences, then Settings

Select the "Advanced" category

Under "Microsoft UI Automation", enable "Use UI Automation to access the Windows Console when available".

Tagging @codeofdusk to verify if I'm missing anything.
Separately, @cinnamon-msft maybe we should add this to our docs page under troubleshooting?

Nope, no need to enable UIA in console for Windows Terminal – it always uses UIA as the implementation is good. That setting is only for conhost.

i'm using NVDA2020.1 and i had this satings enabled  the problem is when i type any command NVDA can't read the outbot of this command like when i type "novd -v" to chick node js version it prented on the screen but NVDA doesn't read any thing and  i can't navigate the result or the oll window

เมื่อ ส. 20 มิ.ย. 2563 เวลา 02:16 ahmed samy alkhouly <
<EMAIL_ADDRESS>เขียนว่า:

i'm using NVDA2020.1 and i had this satings enabled the problem is when i
type any command NVDA can't read the outbot of this command like when i
type "novd -v" to chick node js version it prented on the screen but NVDA
doesn't read any thing and i can't navigate the result or the oll window
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/terminal/issues/6593#issuecomment-646824027,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP55P52AGYZ2MOYYMTJYYVLRXO2R7ANCNFSM4OCVXT7Q
.

@ahmedSamy31 Can you try NVDA 2020.2 Beta 1 instead of NVDA 2020.1?

i tried nvda2020.2 and it works realy thank you


aktualizr: Upgrade to 2020.10 release

aktualizr: Upgrade to 2020.9 release.
Signed-off-by: Anatoliy Odukha<EMAIL_ADDRESS>

I haven't touched master in a while, but if you want to update it, I would recommend forward-porting some of the recent changes that were merged directly to dunfell.

Specifically, I think this (and the other branches except dunfell) are missing the commits from https://github.com/advancedtelematic/meta-updater/pull/775.
Also, I don't see a PR for thud. I think we still have to support that for now, although maybe not forever.

Looks fine, but it's not great to use the merge commit; I prefer cherry-picking the individual commits from #775. Not a blocker, but it makes it a lot easier to compare what commits are where. The DCO check also isn't critical here, but normally I do rebase master --signoff to deal with that.

pushed cleaner version.

LGTM. It'd be good to have CI running smoothly again, though.

eh... I added ticket for this (OTA-5338)
Really configs are inconsistent and bitbake cannot build images. Probably #778 is responsible for the first error. Because other layers do not have gatesgarth.

I added ticket for this (OTA-5338)

Sounds good, but I don't think that's the right ticket.

Really configs are inconsistent and bitbake cannot build images. Probably #778 is responsible for the first error. Because other layers do not have gatesgarth.

Yeah, that's not helping. I think the RPi problems are still open issues here, too.


[HEM-607] Add suppert for modifying deployment strategy

Add support for changing the deployment strategy and its parameters.

Nice feature! 👍
For future reference, here's what this change would look like if we were using Helm charts: https://github.com/Vandebron/argocd/pull/101/files

Nice feature! 👍
For future reference, here's what this change would look like if we were using Helm charts: https://github.com/Vandebron/argocd/pull/101/files

So very similar except that we also have unit tests

Come on, at least try to be impartial 😅
The PR I linked to is roughly 20x smaller than this one, plus it's deployed to an actual Kubernetes cluster so we are certain it works in a real environment before it's even merged to the main branch. Opinions and preferences aside, the Helm-based change is objectively smaller and safer to merge.

In any case, since we're not yet discussing whether to adopt Helm or not, I suggest we pause this topic for now. My goal was just to leave a link here so we can compare real-life examples when we do pick up this topic again in the future 🙂

@nik-kashi standard k8s manifest snippets can be mixed into project.yml like this https://github.com/Vandebron/mpyl/pull/362. It reduces the amount of custom parameters you need to add, the schema is k8s own (so guaranteed to be correct) and people can consult the standard k8s documentation to find out what it does.

Come on, at least try to be impartial 😅
The PR I linked to is roughly 20x smaller than this one, plus it's deployed to an actual Kubernetes cluster so we are certain it works in a real environment before it's even merged to the main branch. Opinions and preferences aside, the Helm-based change is objectively smaller and safer to merge.
In any case, since we're not yet discussing whether to adopt Helm or not, I suggest we pause this topic for now. My goal was just to leave a link here so we can compare real-life examples when we do pick up this topic again in the future 🙂

I actually meant it in a positive way.. if you think away the test logic and add strategy_type to your pr, the approach is very similar.  🤷

@nik-kashi standard k8s manifest snippets can be mixed into project.yml like this #362. It reduces the amount of custom parameters you need to add, the schema is k8s own (so guaranteed to be correct) and people can consult the standard k8s documentation to find out what it does.

Thanks Sam
But what will happen if we want to modify only one field?
I believe it won't work with your implementation
https://github.com/Vandebron/mpyl/pull/362/files#diff-f2e93ed09e3ea127c21228d5487fed442fe1b3364117f6a7ec4705cd660a72a0R767-R769

@nik-kashi standard k8s manifest snippets can be mixed into project.yml like this #362. It reduces the amount of custom parameters you need to add, the schema is k8s own (so guaranteed to be correct) and people can consult the standard k8s documentation to find out what it does.

Thanks Sam But what will happen if we want to modify only one field? I believe it won't work with your implementation https://github.com/Vandebron/mpyl/pull/362/files#diff-f2e93ed09e3ea127c21228d5487fed442fe1b3364117f6a7ec4705cd660a72a0R767-R769

Ah yeah, correct. I forgot that. You need to merge the two dictionaries like:
merged_config = {**self.config_defaults.deployment_strategy, **(self.deployment_strategy or {})}
strategy = ChartBuilder._to_k8s_model(merged_config, V1DeploymentStrategy)

In favor of #362

Closed in favor of #362


audio control of parameters

Should happen at the Audio Graph level I guess?  That way the solution is available in Ruby, Max, AU, etc.
How can we do this?
Nils suggested that a slide~ like approach be applied to all parameters, but I think we can do this on an ad hoc basis if we can just control any parameter with a signal.
Contradicting myself: in order for the parameter changes to be in lock-step with the audio itself then we would have to do this as a part of the process message -- which implies that some of this is actually infrastructure required at the DSP level.
see more on the original redmine issue http://redmine.jamoma.org/issues/613

moving to Jamoma2


Fix CI for Cosmos V4

CI is broken for Cosmos V4 branch.
Here are the broken tests: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=123463&view=ms.vss-test-web.build-test-results-tab

This is done.


Automatically build deb package on a new version tag

In #796, I noticed that the deb package was not build since an automation was missing.
With this PR, I add the missing automation.
I tested the workflow in my repo...

when starting the workflow manually: https://github.com/MrSerth/pgcat/actions/runs/10737879151/job/29780286094
when drafting a new release: https://github.com/MrSerth/pgcat/actions/runs/10737835796/job/29780146212

Obviously, both workflows failed since I cannot upload to the APT repo. However, the version substitution for the workflow is working correctly (as shown when collapsing the first line of the "Build and release package" step).

Thank you!


Stencil if-tag doesn't support filters

With the template
{% if "FooBar"|contains: "Foo" %}
FooBar contains D
{% endif %}

I get the error:
'if' expression error: dangling token
However, when I replace the custom contains filter with a built-in operator the file generates normally.
{% if "FooBar" == "FooBar" %}
FooBar contains D
{% endif %}

Am I doing something wrong?

I am not so sure Stencil has a contains filter http://stencil.fuller.li/en/latest/builtins.html#built-in-filters But would love to be wrong :]

we have a custom filter, but looking at Stencil source I'm not sure if if tags support filters in current version? did you use them before @alisoftware @ilyapuchka ?

Yep, I'm also not sure if Stencil actually supports filters in if. Using {{ FooBar|country: "Foo" }}works fine though. Probably we need to file an issue in Stencil repo to improve that. /cc @kylef

I don't think so either.
One workaround would be to use the {% set %} not I use in SwiftGen (and soon available via SwiftGen/StencilSwiftKit onces we finish the Great Split of SwiftGen) to set it result of the filter into a temporary variable then do the if on it. Might be a workaround if SwiftGen/StencilSwiftKit comes before any PR on Stencil to support filters in if nodes.
(Of course in long term supporting filters in if expressions would indeed be better).

BTW @krzysztofzablocki even if SwiftGen's Great Split isn't finished yet, I think you may already start using https://github.com/SwiftGen/StencilSwiftKit as a pod dependency for Sourcery.
We still have to create an Xcode projet, add the unit tests from SwiftGen for those nodes, configure Travis, Write the documentation, etc… but the podspec already works, so even if it's not yet pushed to trunk you should totally be able to use pod 'StencilSwiftKit', :git => 'https://github.com/SwiftGen/StencilSwiftKit' already to give it a try :wink:

I didn't realise Stencil did not support custom filters in an if statement. In the Sourcery documentation, there's this line
{% if name|contains: "Foo" %} - check if name contains arbitrary substring
I guess that confused me.

yeah I think we made assumption it will work (it really should), we mostly use filters on for loops and in those it works :)

https://github.com/kylef/Stencil/issues/90


Cannot connect to peers behind NAT

Hi!
I've tried  to use your library  with different trackers and different torrent files but it doesn't work ,this is my logger output:
2012-11-12 23:26:06,023 [bt-announce(..616566)    ] INFO : Announcing STARTED to tracker with 0U/0D/6153490L bytes...
2012-11-12 23:26:06,043 [bt-announce(..616566)    ] INFO : Setting announce interval to 1755s per tracker request.
2012-11-12 23:26:06,043 [bt-announce(..616566)    ] INFO : Got 2 peer(s) in tracker response, initiating connections...
2012-11-12 23:26:06,052 [bt-connect-1             ] INFO : Connecting to peer://<IP_ADDRESS>:6882/? [Ci|Ci]...
2012-11-12 23:26:06,052 [bt-connect-2             ] INFO : Connecting to peer://<IP_ADDRESS>:51413/? [Ci|Ci]...
2012-11-12 23:26:06,054 [bt-connect-2             ] INFO : Could not connect to peer://<IP_ADDRESS>:51413/? [Ci|Ci]: Connection refused.
2012-11-12 23:26:06,054 [bt-connect-1             ] INFO : Could not connect to peer://<IP_ADDRESS>:6882/? [Ci|Ci]: Connection refused.
2012-11-12 23:26:06,789 [bt-client(..616566)      ] INFO : SHARING 0/376 pieces (0.00%) [0/0] with 0/0 peers at 0.00/0.00 kB/s.
2012-11-12 23:26:09,790 [bt-client(..616566)      ] INFO : SHARING 0/376 pieces (0.00%) [0/0] with 0/0 peers at 0.00/0.00 kB/s.
2012-11-12 23:26:12,792 [bt-client(..616566)      ] INFO : SHARING 0/376 pieces (0.00%) [0/0] with 0/0 peers at 0.00/0.00 kB/s.
2012-11-12 23:26:15,793 [bt-client(..616566)      ] INFO : SHARING 0/376 pieces (0.00%) [0/0] with 0/0 peers at 0.00/0.00 kB/s.
2012-11-12 23:26:18,795 [bt-client(..616566)      ] INFO : SHARING 0/376 pieces (0.00%) [0/0] with 0/0 peers at 0.00/0.00 kB/s.
And it prints this many many times
What's happening?Connections are always refused.I have downloaded files with number of peers in range from 50 to 150,but it's the same.
Thank you.

I connect to the internet using a router, so I am in a NAT.
I downloaded the highest seeder torrent from YIFY (10,000 seeders) and gave it to ttorrent client for download.  I was getting the same error/messages as the original poster.
However, the pieces started downloading slowly but surely.  After some one hour, I have had some 50 out of 450 pieces downloaded.
How do you explain this behavior?

Some of the seeders may be directly connected to the Internet, so you can connect to them even you are behind an NAT. (though these connections must be initiated by you since they cannot see you)
Once you tried to connect to them, the broker of the seed and the clients of seeders can handle the NAT issue for you.


9600 Baud rate for UART in STC15F104W

Hai,
I had seen your program, but i want to write the this program fr 9600 baud rate. how to do it?
Regards
Praveen M

You can try this. Replace the Delay2400 function with this:
void Delay9600(){	// 1 bit time for 9600 baud at 12 MHz (104 uSeconds)
__asm
nop
nop
push 0x30
MOV 0x30,#246
NEXT:
djnz 0x30,NEXT
pop 0x30
__endasm;
}
Then replace all the Delay2400() calls in the program with Delay9600()
I just used the delay tool in the STC-ISP program to generate the assembly code for 104 uSeconds at 12 MHz.  The format has to be changed a little for the assembler in SDCC. I compiled the code but didn't test it on the hardware. You may need to indent the code between the asm and endasm directives like the original Delay2400 routine. I typed it in the indentions  but they get removed in this comment.
Good luck.


[🐞] Lite components don't have key in types

Which component is affected?
Qwik Runtime
Describe the bug
When using a lite component typescript throws an error if trying to pass key when mapping over an array.
Reproduction
https://stackblitz.com/edit/qwik-starter-mwjway?file=src%2Froutes%2Findex.tsx
Steps to reproduce
Open stackblitz and view router/index.tsx in editor
System Info
 System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    @builder.io/qwik: latest => 0.23.0
    @builder.io/qwik-city: latest => 0.6.6
    undici: ^5.16.0 => 5.21.0
    vite: 4.2.1 => 4.2.1

Additional Information
I tried to make the change necessary for this but was having trouble about where to put the type and it was causing other build errors so opening an issue for someone more familiar with the jsx types to fix

@jweb89, perhaps this is not a bug. Lite components do not have a host element (whatever that is). So this might be by design.


Provide the ability to use some of @loader in the slim loader such as envs configuration

Modules that use slim loader like:
var loader = require("@loader");

var url = loader.serviceBaseURL + "/api/todos";

Should not throw. I think the way to approach this is that steal-tools can provide a shim module for @loader. It can then attach the envs configuration to that loader object so that if it looks like:
"envs": {
  "window-production": {
    "serviceBaseURL": "http://example.com"
  }
}

The pseudo-code for the loader module would be:
module.exports = {
  "serviceBaseURL": "http://example.com"
}

Issue moved to stealjs/steal-tools #721 via ZenHub


Log to stdout instead of stderr

Fixes #507
Signed-off-by: Andy Goldstein<EMAIL_ADDRESS>

Better test - ark server >ark.stdout 2>ark.stderr - make sure that everything goes to ark.stdout and that per-backup and per-restore logs continue to work.

Double checked with the separate redirects - backup and restore logs were properly populated. Nothing went to stderr during standard operation.
A message did come across stderr if I provided no credentials or incorrect ones:
x1c in /home/nrb/go/src/github.com/heptio/ark (git) master
% ./_output/bin/linux/amd64/ark server  >ark.stdout 2>ark.stderr

x1c in /home/nrb/go/src/github.com/heptio/ark (git) master
% tail ark.stderr
An error occurred: rpc error: code = Unknown desc = GOOGLE_APPLICATION_CREDENTIALS is undefined

This seems correct to me, as it's an error generated by the plugin, and prevents Ark from actually launching. It would still show up in GKE's logs, though.

Yes I think that's fine. It's an error that occurs during server startup.

LGTM.


Arrays on Sorted Union mismatch

The script has unite([1, 2, 3], [5, 2, 1, 4], [2, 1]);
while the test has unite([1, 3, 2], [5, 2, 1, 4], [2, 1])
This creates confusion for those using the tests to get an idea of what should be returned as the order does not match.

@Rafase282 Confirmed the issue you're pointing out exists though in this case, as I'm sure you discovered, it doesn't affect the answer/solution. I agree it should be "fixed" for clarity sake.

@catapixel Yes I figured it would not affect the answer, but while I was trying to figure it out I didn't notice the reason why my code would pass even thou the order was different. It makes the problem very confusing for those using the failed test before coding to get an extra hint.
The other issue was already solved which was checking for more than three arguments so I didn't add that one.

fixed


Contributing.md has broken links

I’d like to contribute to Chocolatey but some of the links in the Contributing.md are dead, so I thought I’d update it first. There's a link to the CLA, but it’s blank. Is the ChocolateyGUI CLA the same one as the Chocolatey CLA? I know they’re different projects.
dead CLA link: https://www.clahub.com/agreements/chocolatey/ChocolateyGUI
chocolatey CLA link: https://cla-assistant.io/chocolatey/choco
Also, there’s a link for the mailing list, but it says ‘Content unavailable.’ https://groups.google.com/access-error?continue=https://groups.google.com/g/chocolateygui
When I do a search for ChocolateyGUI I see ‘No groups matched.’
Is it possible that the google groups mailing list isn't the best way to communicate? Perhaps it should be updated to gitter?

@KrisBierma yes, Chocolatey GUI uses the same CLA as Chocolatey.  You can see it in action on this PR:
https://github.com/chocolatey/ChocolateyGUI/pull/700
Regarding the Google Group, it does look like it is still working, but you will need to be signed in to see it:
https://groups.google.com/g/chocolateygui
It isn't a particularly well used avenue for asking/answering questions though, so I am ok with pointing people in a different direction. While Gitter is good for quick questions, searching isn't great.  I think I would rather direct folks to ask questions via Discussions on the repository:
https://github.com/chocolatey/ChocolateyGUI/discussions
Thoughts?

@gep13 For the groups, I'm signed in with my personal email. I can search and see 'conversations matching query chocolateygui' but no 'groups.' If that's not really used, though, then I don't need to find it and this is just an fyi.
Since you're familiar with the project, I'll defer to you about the communication link. From your answer it sounds like the github discussions link is the best option.

Regarding the Google Group, it does look like it is still working, but you will need to be signed in to see it:

I wonder if there is some kind of permission issue (ie, the group is set as invite-only, private, etc).
Just tried the google group myself and seems I can not see it either (I never joined the group before).
I think it makes sense to change the link to direct to the discussion GitHub page instead though, due to the low traction in the google group.

@AdmiringWorm it could well be that something changed with regard to the permissions of the Google Group, but I would prefer that we switch to Discussions, rather then invest any time in figuring out what is going on there.
@KrisBierma would you be interested in raising a PR to update both of these URL's?

:tada: This issue has been resolved in version 0.20.0 :tada:
The release is available on:

GitHub Release
Chocolatey Package

Your GitReleaseManager bot :package::rocket:


Can't copy text

I cannot copy highlighted text with command c/strg c. The problem exist on mac and win 10.

+1

+1

+1


Bump argo worfklows to add e2e tasmin WorkflowTemplate

This only adds the e2e WorkflowTemplate for tasmin.

@brews you might have seen my slack message/gotten updates already, but in case -- we're running production tasmin workflows 😄
In order to do that, I simply took the e2e tasmax workflow template, copied it and replaced 'tasmax' by 'tasmin' where needed, since (1) the pipeline is variable-agnostic and the e2e-level parameters of tasmin should be the same as for tasmax (regridding type, etc) and (2) tasmin already has its own set of valid parameter files.
Let me know if you have any concern regarding these assumptions (1) and (2). Otherwise I think we're good.
So, in this PR, I am updating the kustomization so that argo gets this new e2e workflow template for tasmin that I added through this PR.
I thought this was an automatic argo CD process once a github merge happens, but... I don't see that template available in our argo interface and the version is still at 3.6. I could try to do that upgrade by hand, but I am not sure I have the argo CD credentials for that etc... So, I just used Workflow yaml files, and thought I'd wait for your input.
Thanks !

@emileten
In this case, the new tasmin workflow didn't sync up to the server because the new workflowtemplate needs to be referenced in workflows/templates/kustomization.yaml. I got this fixed in PR #509.
More details:
So, right now, ArgoCD is watching infrastructure/kubernetes-gcp/dc6-workflows/kustomization.yaml. On this line you can see that it's pointing to workflows/templates/kustomization.yaml. This workflows/templates/kustomization.yaml file needs to explicitly list each of the resources we want to get synced up to the cluster. So, because the workflowTemplate was missing from the list in PR #505, it didn't get automatically synced until we add it to the list in PR #509.
Point to this separate workflows/templates/kustomization.yaml file so that analysts don't need to tinker with deeper, technical files and settings in infrastructure/... to add, remove or modify common workflows. It's an attempt to separate responsibilities.


Support for panoptic segmentation on other datasets present in mmsegmentation

Describe the feature
I see that for panoptic segmentation the current models have  been trained on COCO.Are there any plans on introducing  models trained on different datasets (similar to the ones present in mmsegmentation)
Motivation
Currently all models are trained on the benchmark COCO dataset.As such applying those same models on other datasets like ADE (which has a vast vocabulary of common objects .for eg,it has 9000 images just corresponding to the sky label) may result in poor accuracy for panoptic segmentation purposes
A clear and concise description of the motivation of the feature.
Related resources
https://github.com/facebookresearch/Mask2Former/blob/main/configs/ade20k/panoptic-segmentation/swin/maskformer2_swin_large_IN21k_384_bs16_160k.yaml
Additional context
Add any other context or screenshots about the feature request here.
If you would like to implement the feature and create a PR, please leave a comment here and that would be much appreciated.

Hi, is there any update?


Initial support for image keyboard accepting only gif

@RocketChat/android
Should be really easy to add support for other mime types.
Closes #156

Awesome. I will check the APK

Great 👍
Please check 2 comments about the naming.
After that, let's merge this PR.

Glad you like it. xD
I'll make it show up in the app next.


the method for getting the cifar10 data in the setup_cifar.py was modified

Maybe there is something wrong happend on the method for getting the cifar10 data used in the setup_cifar.py file which make the model can't be convergent with the accuracy 80%.
So I changed the method whose result is identical to the result given in your paper.

Hmm, thanks for this. Not sure what's causing cifar to load incorrectly. Before merging this in, let me try to figure out what the problem is just so I know.

En, I'm Shiwei,Shen the author of AE-GAN.
Until now, I have done the experiments on the MNIST datasets and CIFAR10 datasets based on your model and code.  In the experiments,the no-targeted L2 attack was used.
Our AE-GAN can defense your attack with the accuray 0.9855 on MNIST and the accuray 0.7  on CIFAR10.
In addition, the accuray 0.9855 is better than the accuray 0.975 got by the another defense method adversarial training.
And now, I'm trying to defense your attack on the ImageNet datasets. But It's so slow to get the adversarial  examples.
Maybe you can give me some good suggests for our work.
Thanks for researching on the adversarial  attack with you.

I'll email you directly about the ImageNet performance. Long story short: it's slow, because ImageNet is big, but if you use only ~100 iterations then it'll still work very effectively.


Removes unused reveal-on-focus js

Removals

Removes reveal-on-focus.js, which does not appear to be in use.

Review

@KimberlyMunoz
@jimmynotjim
@sebworks

:+1:

Improves our coverage :+1:
26.51% ⟹ 26.55%

26.51% ⟹ 26.55%

LOL


Back button closes the activity in fragments

Checklist

[x] Bug Report - The issue you are reporting might be there already. Please search on the issues tracker before creating or reporting a issue.
[x] Before reporting, please ensure (if you can) that you have tested it with the latest (dev) version of the software.
[x] Don't forget to tag your issue, it makes it easier to discover.

Summary:
Back button closes the activity instead of going back to Home. Examples:

Search by barcode
Allergen alert
Preferences
Offline edit

Steps to reproduce:
Go to the section. Press back button.
Expected behavior:
Go to home
Observed behavior:
Activity closes

I will fix this issue .


[BC] hotfix to load/save OutMagFEMM

Thank you for fixing that :)
We still need to solve the main issue with parallelization, I will work on it this week normally.


What do you mean by "Custom-navigation provider"

What are trying to tell by "Custom-navigation provider" for SharePoint Online? It's not possible, doesn't it? You can't add custom SiteMapProvider in any way.

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

ID: fe686368-f607-9a7d-710b-d51c3f8f6c9a
Version Independent ID: 01b5902a-6e23-62d9-f0a3-7c1e86194ef1
Content: Navigation options for SharePoint Online
Content Source: Enterprise/navigation-options-for-sharepoint-online.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@VerdonTrigance  Thank you for the feedback. For a custom Navigation solution, you can refer to https://docs.microsoft.com/en-us/sharepoint/dev/solution-guidance/portal-navigation#using-a-custom-navigation-solution
Hope this helps!
Thanks
Sri

@msbemba  Thank you very much for the contribution and sharing this explanation. @VerdonTrigance Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


mkpr

Applied mkpr on package.json
sed s/"koa": ".*"/"koa": "^2.8.2"/

Coverage remained the same at 36.232% when pulling 773f66c3b241775be12edeb2603f47292d42454c on mkpr/1 into ba124dc972eebc23634910cd1eac6f5c1bd14c78 on master.


Moved from InferenceApi to InferenceClient to avoid deprecation warning for huggingface_hub

Updated huggingface_hub llm to use the new InferenceClient to avoid deprecation warning.
Resolves the issue: #10483
@baskaryan, @eyurtsev, @hwchase17, @rlancemartin.

Sorry if this is a dumb question, I am new to this. Is this update/fix on the newest version of langchain (0.0.303)? After updating I still get the inference api depreciation warning when running a huggingface_hub model:
\venv\Lib\site-packages\huggingface_hub\utils_deprecation.py:127: FutureWarning: 'init' (from 'huggingface_hub.inference_api') is deprecated and will be removed from version '0.19.0'. InferenceApi client is deprecated in favor of the more feature-complete InferenceClient. Check out this guide to learn how to convert your script to use it: https://huggingface.co/docs/huggingface_hub/guides/inference#legacy-inferenceapi-client.
warnings.warn(warning_message, FutureWarning)

@Leah-Mac This PR is still Work in Progress, once this will get merge then the warning will no longer be visible to you.

Any updates on this?


Mac Compatibity

Hi Adam, How to port this for the Mac app? Thanks

I think it'd need a fair bit of work to port to Mac use. The biggest change would be moving all UIBezierPath to NSBezierPath, ideally something like:
#if TARGET_OS_IPHONE
#define BezierPathClass UIBezierpath
#else
#define BezierPathClass NSBezierpath
#endif

and then replacing all UIBezierPath with BezierPathClass. The same would need to be done for PerformanceBezier too. That'd get a fair bit of the way there.


Cluster error

thanks for sharing， but when i compile there is an error！
in the cpp_test.cpp, in line 49, your code is
vector<vector > shifted_points = msp->cluster(points, kernel_bandwidth);
but your cluster funtion returns vector, so i changed to the following:
vector<vector > shifted_points = msp->cluster(points, kernel_bandwidth)[0] .support;
and i can change 0 to 1 also, the result is different, i think this is a big error

made some changes, give it a shot now

Going to close this, let me know if you still have an issue.


skip parameter to read_excel seems near random

Every time I use read_excel , the skip parameters is impossible to determine beforehand
Ie if I open the file in a spreadsheet application and check how many rows down before my data table starts, and then input that number into read_excel, it never gets right.
The process of determining the correct skip number always ends up as follows:

read_excel with what seems the right number
head( ) the data to see how wrong it got
Try to estimate new skip number from 2)
Repeat until it gets right.

This is a small small issue yes, but having happened to me ~1000 times it has reached "report-issue-on-github" level.
Also knowing that I have to check something by hand >1 times when it should be enough to check it only once, is surprisingly annoying.
I dont have any good test-data to supply, if its not as easy to reproduce as the frequency of occurence should suggest, ill prepare something.
T.

I have the same issue.  Sheet with 9 rows to skip before column headers, only reads correctly with skip=1.
Same issue appears when using RStudio Import Dataset.
The sheet also contains several charts.
readxl ver <IP_ADDRESS>00    (Excel 2013)

Hi, I have the same issue - regular excel file and I need to skip 22 rows and I have to enter 18 for some reason and there are no frozen headers in the excel file or anything.  What I did notice is that it is looking at how many rows have data in them in the first column at least.
Cheers and love the program so far and it is a lot faster than other methods.

Same issue using. I am pulling in all sheets in a .xlsx workbook; the sheets are similarly formatted, and the 9 rows before the data starts appear to be nearly identical on each sheet. For some sheets the skip parameter works as expected; for others it skips one line too many.
readxl ver 0.1.1

jamesui, I had noticed when I was working with it that it was not recognizing blank rows appropriately. It was a real issue but once I figured that out I was able to just count the filled rows and use that to import all of the sheets as they were the same. I am hoping that this is fixed soon especially as it is now an integral part of R studio. Cheers Bill

Is anyone in a position to share a problematic sheet as a concrete example?

Hi,
Here is an example of the importing error in readxl
Note the data begins on line 24 and if you use the first example of skipping 23 you read in no data but if you skip 18 which is the number of filled cells it works fine. I have not tried uploading an excel file here before so if it does not work let me know.
Cheers,
BIll
`# Load Libraries ----
library(tidyverse)
library(ggplot2)
library(dplyr)
library(tidyr)
library(readxl)
library(readr)
library(lubridate)
setting the working directory ----
set this by clicking session - set working directory
setwd("~/Desktop")
Does not work
bad_sonde.df <- read_excel("ltm.xlsx", skip = 23, col_names = TRUE)
Works
good_sonde.df <- read_excel("ltm.xlsx", skip = 18, col_names = TRUE)
`
ltm.xlsx

Somewhere I found an article/discussion of this and the error was known but not fixed yet.
Sorry i can't find it.

You can also try opening the file with the new R Studio data import function and it will do the same thing. Cheers, Bill

Thanks @wlperry. I see what you mean. This example will be helpful.

My experience is as follows:
If the Excel file is in the older .xls format then the value of skip=n must be the total number of rows before the data (or header row).
The same file in .xlsx format requires that skip=n only includes the non-blank rows.

sorrry.
it s really unclear.
what is skip.
i haven t mentioned . i work with a xlsx file , there are no blank inside.
i have test another R package : "xlsx".
after calling my xlsx with this package, dplyr::filter work.
2016-11-22 7:09 GMT+00:00 David Luckett<EMAIL_ADDRESS>
My experience is as follows:
If the Excel file is in the older .xls format then the value of skip=n
must be the total number of rows before the data (or header row).
The same file in .xlsx format requires that skip=n only includes the
non-blank rows.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hadley/readxl/issues/194#issuecomment-262164392, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC9GBSVwPTQMJPVd7gXetNVVxBztK403ks5rApUrgaJpZM4JZhXz
.

--
Fabien
Fabien Leboeuf
Research fellow ( University of salford)

phD Mechanics-Biomechanics (university of Poitiers, FRANCE)

College of Health & Social Care
University of Salford
Allerton Building
Frederick Road Campus
Salford, M6 6PU, UNITED KINGDOM

DavoOZ, That is also my experince with the skip option - it works with older xls and not xlsx files. I still cannot find a discussion that I saw on this a while ago. The problem is that if you have a directory of xlsx files that need to skip the first 20 rows before importing and sometimes there are differences in the data in the first 20 rows it causes a lot of issues. I still like the program and can work around this.

aaa34169 - skip is an option in the read_xl file that allows you to skip a set of rows in the excel file before it begins reading the data.
Yes there are a number of packages out there that will read in an excel file but the readxl seems to be very fast and Hadley is the author and uses a set of terminology that is consistent so what could be better!

Hi
I have ever seen readxlk is faster than other xls package. That why i chose
it.
Beside, I am a great fan of all Hadley packages.
So, for my issue, i need to use the skip option.
why i have to skip rows.
that s still unclear, sorry
2016-11-22 9:53 GMT+00:00 Bill<EMAIL_ADDRESS>
aaa34169 - skip is an option in the read_xl file that allows you to skip a
set of rows in the excel file before it begins reading the data.
Yes there are a number of packages out there that will read in an excel
file but the readxl seems to be very fast and Hadley is the author and uses
a set of terminology that is consistent so what could be better!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hadley/readxl/issues/194#issuecomment-262195585, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC9GBZ4SbKsQ3hoJdqwnm2GewY4x5THMks5rAruRgaJpZM4JZhXz
.

--
Fabien
Fabien Leboeuf
Research fellow ( University of salford)

phD Mechanics-Biomechanics (university of Poitiers, FRANCE)

College of Health & Social Care
University of Salford
Allerton Building
Frederick Road Campus
Salford, M6 6PU, UNITED KINGDOM

You may not need to skip rows. The excel file above shows an example where the top portion of the excel file has a bunch of metadata in it that is not in tidy data format and needs to be removed. What would be nice is if the instrument I am using put out a file with row 1 as column names and the rest was data but that is not the case for me. There are ways you can read in say row 15 and then rows 17 - whatever and rbind them. That is another way to do this.

dplyr::filter works fine with xls instead of xlsx.
2016-11-22 10:01 GMT+00:00 Bill<EMAIL_ADDRESS>
You may not need to skip rows. The excel file above shows an example where
the top portion of the excel file has a bunch of metadata in it that is not
in tidy data format and needs to be removed. What would be nice is if the
instrument I am using put out a file with row 1 as column names and the
rest was data but that is not the case for me. There are ways you can read
in say row 15 and then rows 17 - whatever and rbind them. That is another
way to do this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hadley/readxl/issues/194#issuecomment-262197620, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC9GBcWW3GYyw6SjEYa3cYAuRgzWbcz2ks5rAr1fgaJpZM4JZhXz
.

--
Fabien
Fabien Leboeuf
Research fellow ( University of salford)

phD Mechanics-Biomechanics (university of Poitiers, FRANCE)

College of Health & Social Care
University of Salford
Allerton Building
Frederick Road Campus
Salford, M6 6PU, UNITED KINGDOM

Not sure if this applies to anyone else's problem. Regarding the skip parameter for read_excel(), in reading .xlsx files. I found that counting the number of nonempty rows works for me. That is, if data column names start in row i and j out of the first (i - 1) rows are empty, then skip = (i - 1) - j

This is covered for xlsx under #224, which will soon be fixed. I am leaving #222 open to cover the xls case.


Peering Request

Full Name
Huge Networks
ASN
AS264409
PeeringDB Integration

[X] My PeeringDB Entries are current
[ ] Peer at ALL EXCHANGES (and skip the Peer Address(es) field below)

Peer Address(es)
Here are the requested IP information:
IPv4: <IP_ADDRESS>/32
IPv6: 2001:12f8:0:2::55:214/128
Multihop

[X] Multihop Peering Request

Thank you for your interest in RouteViews.
The addresses you provided are at IX.br in Rio where we already peer with you on <IP_ADDRESS>/2001:12f8:0:2::55:6 (not multihop as in the request).
Let me know if this is correct and I'll set up the other peerings as well.
Thanks!

Yes, that's correct, we have peering with you at IX.br in Rio, but at another location (PIX INTERNEXA).
This other one would be at PIX EQUINIX-RJ2.
Just to clarify, the new connection will also not be multihop, ok.
I'm available for any questions.
Regards,

Hello Philip.
Any update on the above request?
Regards,

Hello Philip,
I hope you're doing well.
I'm following up on my previous message regarding the peering request. Could you provide any updates or let me know if there are any additional steps needed from my side?
I appreciate your time and look forward to your response.
Best regards,

Hello Philip,
Any updates on the request above? I'm still waiting.
Best regards,

Hello Philip,
Any updates on the request above?
Best regards,

Hi
Sorry about the delay.  We'd be happy to set up a second session.  I notice that you are not announcing your full table to us in Rio.  Would that be possible?
BR Nina

Hello,
That's correct, we don't send the full table in these peerings, we only announce a few prefixes. So, in this case, we will follow the same pattern as the other session we already have.
I'll be waiting for the information about the new session.
Best regards,

We do prefer if you can send your full table.  Will the prefixes announced on this second session in Rio be different from the ones on first?  Sorry for all my questions, but if there is no new information from the new session, we prefer to keep just the one due to capacity restraints.

Hello,
I checked internally with the team and we're going to send full table in this new session, ok.
I'll be waiting for the information about the new session.
Best regards,

Hello.
Session has been configured with the information provided above. Awaiting configuration on your side.
Regards,

Our side is configured  and up.  We do not see the full table on tis new session, but the old one is now a full table.  Will you announce the full table on both?
route-views.rio.routeviews.org# sh bg su | include 264409
<IP_ADDRESS>       4     264409   5898241<PHONE_NUMBER>9125    0    0 01w2d15h       864802        0 Huge Networks -
<IP_ADDRESS>     4     264409       187        13 99319125    0    0 00:05:21          456        0 Huge Networks -
2001:12f8:0:2::55:6   4     264409  16913197<PHONE_NUMBER>2819    0    0 01w2d15h            1        0 Huge Networks -
2001:12f8:0:2::55:214 4     264409        14        13 89952819    0    0 00:05:21            0        0 Huge Networks -

Hello,
We made a change on our side and are now sending the full table in the session, as shown in the output below. Could you please validate and give us your feedback?
<IP_ADDRESS>
RouteViews
6447 23087 0 08d00h22m 0/0/974419 (IPv4)
378920 0
Best regards,

I see full routes on the v4 sessions, but still not on one  v6 session - it would be great if you could update that at your convinience, but let's close this ticket here
Thanks for your patience :)
route-views.rio.routeviews.org# sh bg su | include 264409
<IP_ADDRESS>       4     264409   8307534<PHONE_NUMBER>51796    0    0 03w1d23h       865734        0 Huge Networks -
<IP_ADDRESS>     4     264409   1331153     38338 120951796    0    0 01w6d07h       974774        0 Huge Networks -
2001:12f8:0:2::55:6   4     264409  16952826<PHONE_NUMBER>81208    0    0 03w1d23h            1        0 Huge Networks -
2001:12f8:0:2::55:214 4     264409    217575     38338 128781208    0    0 01w6d07h        77996        0 Huge Networks -
route-views.rio.routeviews.org#
Kind Regards
Nin


Added scientiac.space and map.scientiac.space to the list.

Websites : https://scientiac.space and https://map.scientiac.space
This PR is:

[x] Adding a new domain
[ ] Updating existing domain
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (darktheme.club site)
[ ] Other not listed

[x] The domain is in the correct alphabetical order
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ page, particularly the red section about inappropriate content, and I attest that this site is appropriate based on those criteria.

[x] Check to confirm

- domain: map.scientiac.space
  url: https://map.scientiac.space/
  method: javascript
  last_checked: 2023-09-11

and
- domain: scientiac.space
  url: https://scientiac.space/
  method: javascript
  last_checked: 2023-09-11

Thanks!


efficient? fast?

I am not sure if any profiling was done here but I wouldn't call this "efficient" or "fast" in terms of rendering.

I would agree, but it might be more productive to suggest a fix, or provide a PR.  As it stands this is just a rant.

@MetaFight a rant? that was barely a jab. Also to your other point, I would rather not spend my time working with a collaborator who plasters false adjectives all over their library in an attempt to gain stars, which I suppose is essentially analogous to karma on reddit, or at least equally irrelevant.


Area_Farm missing unit

https://datacommons.org/tools/statvar#Area_Farm
But https://datacommons.org/tools/statvar#Mean_Area_Farm has unit Acre

This has been updated with unit Acre: https://datacommons.org/tools/statvar#sv=Area_Farm


Footer: content overview link broken

In the footer, the "Content Overview" link doesn't point to anything, so it just reloads the current page. It should point instead to the Content Overview view.

this is not an issue; staging just hadn't been updated. Closing.


Better Performance

We currently run in (before) CoreCompile. But for that build step we are actually to slow it seems.
We need to increase the runtime of this tool significant. (ms instead of seconds)

We got faster (currently develop) but I would like to get even faster.


Documentation Optimization

Some documents(like https://github.com/matrixorigin/matrixone-operator/blob/main/docs/docker-compose.md) are out-of-date, pls check and fix them.

@sukki37 docker-compose will remove to matrixone repo


DropDown doesn't say that it needs localizations

if you use a Material DropDown without a MaterialApp, you get an exception because you don't have localizations, but it doesn't warn you in the build method. We should do the usual assert in the build method to require the presence of all our InheritedWidget dependencies.
In general we should audit all our widgets to make sure we are asserting for the right dependencies.

cc @HansMuller

Fixed by https://github.com/flutter/flutter/pull/20787


An error will occur

!pip install google-colab-selenium
import google_colab_selenium as gs
driver = gs.Chrome() # <----Error

Thanks for bringing this to my attention.
It looks like Selenium 4.20.0 was released today, which is causing the issue (since it still works on 4.19.0).
I'll see If I can find a fix.

Google-Colab-Selenium version 1.0.13 is out now, which adapts to the changes made in the new Selenium version.

Thank you!


tags with value 0 or False not pushed to InfluxDB

When I want to push tags with value equaling to 0 or False, the tag is not accepted.
After investigation, the line to blame is here:
https://github.com/influxdata/influxdb-python/blob/3d61f1f465c0fbfbe404c77a4728c5d7268aaf1e/influxdb/_dataframe_client.py#L374
I patched it and here is how I solved it:
Before:
tag_string = ''.join(                 [",{}={}".format(k, _escape_tag(v)) if v else ''                  for k, v in sorted(global_tags.items())]             )
After:
tag_string = ''.join(                 [",{}={}".format(k, _escape_tag(v)) if v not in [None, ''] else ''                  for k, v in sorted(global_tags.items())]             )
Furthermore, it would be nice to also send a tags even if the value is None. Here is my proposition:
tag_string = ''.join(                 [",{}={}".format(k, _escape_tag(v)) if v not in [None, ''] else ",{}={}".format(k, 'None')                  for k, v in sorted(global_tags.items())]             )

@michelfripiat thank you for the issue. Is this something you would consider contributing to the project?

For sure, what does that imply?

@michelfripiat all you need to do is open a pull request with the change you'd identified and someone will take a look.

Closed by #770


COOK-127 Update to HDP-UTILS <IP_ADDRESS>

This updates the version of the HDP-UTILS repository for all HDP
users.
Signed-off-by: Chris Gianelloni<EMAIL_ADDRESS>

test needs updated otherwise 👍 lgtm


Feature/bump sparqlify version

This PR bump the Sparqlify version to 0.8.4 and align ViewDefination with the latest Sparqlify changes.
Best

Please, @Aklakan double check if the changes made on SparqlifyUtils2 file are correct or not.
Best,

#57


No music controls on windows 10

when i install it and login on my account the extension no show music controls.

OS: windows 10 1809
Music Time for Spotify:  v2.0.5
vscode: v1.46

Thank you for the issue update. We'll look into this now and apologize for the inconvenience.

Sorry for the delay in getting back to you. The player controls should now display on windows vscode for version 2.0.6. Thank you for your patience.


provide error log by default

given error info by default to make admin/user easier to find
where's wrong settings

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
fixes #659
Special notes for your reviewer:
Release note:

/retest

/recheck

/retest

/retest

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.15

@jichenjc you need to rebase your PR, to avoid v1.12 failure(It will not run)

/test cloud-provider-openstack-acceptance-test-k8s-cinder

/test cloud-provider-openstack-acceptance-test-flexvolume-cinder

/test cloud-provider-openstack-acceptance-test-keystone-authentication-authorization

/test cloud-provider-openstack-acceptance-test-standalone-cinder

/lgtm

/lgtm cancel

@adisky this is such a simple patch, are you ok with us overriding the failed checks?

@hogepodge yes, lets merge this


Volume indexPrefixed broken

If you try to create a volume using indexPrefixed as an alternative to index, you get an error message that the parameter index is missing even though indexPrefixed is supposed to replace it.

This is a documentation error that is being corrected. The corrected documentation excerpt is:
To maintain VTK data compatibility, the `index` array may be specified with cell
sizes interleaved with vertex indices in the following format: $n, id_1, ...,
id_n, m, id_1, ..., id_m$. This alternative `index` array layout can be enabled
through the `indexPrefixed` flag (in which case, the `cell.type` parameter
should be omitted).

  -------------------  ------------------  --------  ---------------------------------------
  Type                 Name                Default   Description
  -------------------  ------------------  --------  ---------------------------------------
  uint32[] / uint64[]  index                         [data] array of indices (into the
                                                     vertex array(s)) that form cells

  bool                 indexPrefixed          false  indicates that the `index` array is
                                                     provided in a VTK-compatible format,
                                                     where the indices of each cell are
                                                     prefixed with the number of vertices

So please try setting indexPrefixed as a bool, and passing your data through index.


adds my details to index.html

@golfiti  thnx for contributing , try this year aswell with updated repo and make sure to follow me and star this repo so that you don't miss updates regarding the new changes and issues during Hacktoberfest 2020.


WIP: 3-cockpit-9: [no-test] Update translations from Fedora Zanata

Update translations from Fedora Zanata

[ ] po-refresh

po-refresh in progress on 4-cockpit-9.
Log: https://<IP_ADDRESS>:8493/logs/po-refresh-739-20190725-141616/

po-refresh in progress on 3-cockpit-9.
Log: https://<IP_ADDRESS>:8493/logs/po-refresh-739-20190725-141616/

po-refresh  done: https://github.com/cockpituous/welder-web/commits/po-refresh-20190725-142222

Codecov Report

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

@@           Coverage Diff           @@
##           master     #739   +/-   ##
=======================================
  Coverage   78.25%   78.25%
=======================================
  Files          66       66
  Lines        3343     3343
  Branches      703      703
=======================================
  Hits         2616     2616
  Misses        727      727

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 8cac69f...695e034. Read the comment docs.


Update README.md

Add 'to' before 'build your application'

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

@all-contributors please add @snehalbiju12 for docs


func-style rule

I have errors when i want to declare function with the following syntax
const shutdown = (code) => {}

To be valid, i should write
function shutdown(code) {}

@xavhan @gplancke @ArnaudSpanneut what is your opinion about this, should we move to the first syntax ?

The second one was enforced by the JPapa styleguide, the 1st one is useful for a functional style. I think that now, we know what we are doing and both can coexist.

👍


Feat(eos_designs): Add support for Terminattr token/cert based auth

Enhancement summary
Today we support only key-based auth for Terminattr towards on-prem.
For CVaaS we support token.
Since this will be breaking we cannot change the default behavior in 3.x, but we should definitely use token by default from AVD 4.x. For now we will need to have a knob.
Which component of AVD is impacted
eos_designs
Use case example
Newer releases of CVP will only support token.
Describe the solution you would like
N/A
Describe alternatives you have considered
No response
Additional context
No response
Contributing Guide

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

Raised by @adietrich-ussignal


Can't use latest react-dom

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: MacOS 10.14.4
Hyper.app version: Hyper 3.0.0-canary.8 (canary)
Link of a Gist with the contents of your .hyper.js:
Relevant information from devtools : bundle.js:1 Error: Minified React error #130; visit http://facebook.github.io/react/docs/error-decoder.html?invariant=130&args[]=object&args[]= for the full message or use the non-minified dev environment for full errors and additional helpful warnings. （ Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object. ）
The issue is reproducible in vanilla Hyper.app: I'm developing hyper-visual

Issue
After upgrade react-dom to latest, this error is thrown by hyper's react.
Please upgrade react-dom inside hyper, so plugin developers can use latest styled-components, so latest styled-icons. Thanks!

done


startPosition not working

The startPosition parameter is not working.
Somehow the javascript file from the documentation is different from the one installed using bower.
Diff here: https://www.diffchecker.com/v0k2fnlp

@rob-m-andrews can you answer this question, and-or straighten this out.

We'll this is not a question, is a 5 months old bug.

It looks like it's not actually a bug on develop, based on #182

@joeyjoejoejr I'm still seeing this bug on the version installed using bower.


add toTyped to Dataset syntax

fix https://github.com/adelbertc/frameless/issues/97

Current coverage is 90.04% (diff: 100%)

Merging #98 into master will increase coverage by 0.23%

@@             master        #98   diff @@
==========================================
  Files            25         25
  Lines           471        472     +1
  Methods         464        468     +4
  Messages          0          0
  Branches          7          4     -3
==========================================
+ Hits            423        425     +2
+ Misses           48         47     -1
  Partials          0          0

Powered by Codecov. Last update 51eae22...6fa7563

@OlivierBlanvillain @julien-truffaut are we waiting something for this PR? Should we merge?


Add references

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@9cef18f). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage        ?   45.77%
=======================================
  Files           ?       14
  Lines           ?      284
  Branches        ?        0
=======================================
  Hits            ?      130
  Misses          ?      154
  Partials        ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[kwokctl] Add podman as the kwokctl runtime

What would you like to be added?
https://github.com/containers/podman
Why is this needed?
Add a new runtime for kwokctl

/good-first-issue

/assign


Mandatory results fields

Hi AidStream,
There are a couple of fields in the Results element that have been set as mandatory fields on AidStream, which are not always required by the IATI Standard.
This is preventing some publishers from updating their data, as they are not able to save their data without filling in these fields.
The fields are:
Indicator Reference

Baseline Year and Value

Please can you have a look into this?
Many thanks,
Jahnvi

Hello Jahnvi,
Thank you for pointing out those issues to us. Those listed out have been managed in AidStream.
We have slight confusion regarding, Baseline > year attribute in following scenario:
If user enters any baseline attribute such as value,  we allow user to save as it is? Do we need to make user mandatory to fill up year attribute or not?
It will be of great help if this confusion could be cleared up as soon as possible.
Thanks in advance.
Kind Regards,
Karuna

Hi Karuna,
Thank you for working on these issues.
Regarding the Baseline > year attribute:
If a user enters any baseline attribute (such as value), the year attribute is then mandatory and users must fill it in.
Do let me know if any other clarifications are needed.
Best wishes,
Jahnvi

Hello Janvi,
We have made the changes in Result element as per the IATI standards.
Now Indicator Reference and Baseline attributes are made optional. But with the requirement of IATI standards such as @value of Baseline, Target, & Actual must be included for non-qualitative measures,  fields will get changed.
Thank you!

Thank you so much for fixing this Anjana! This is looking perfect.
Many thanks, Jahnvi


V3 precaching activate

R: @jeffposnick @philipwalton
Adds a cleanup step to precaching to remove cache entries and revision details that are now longer required.

Coverage decreased (-1.03%) to 94.898% when pulling d8568b393f83e44e5ef831bd95b29ce476987f31 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-1.03%) to 94.898% when pulling d8568b393f83e44e5ef831bd95b29ce476987f31 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-1.03%) to 94.898% when pulling d8568b393f83e44e5ef831bd95b29ce476987f31 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-1.03%) to 94.898% when pulling d8568b393f83e44e5ef831bd95b29ce476987f31 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-9.8%) to 86.085% when pulling eb38f4896b754eeed9a0942b029b1d6885baa316 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-9.8%) to 86.085% when pulling eb38f4896b754eeed9a0942b029b1d6885baa316 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-9.8%) to 86.085% when pulling eb38f4896b754eeed9a0942b029b1d6885baa316 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-1.03%) to 94.898% when pulling 06efe3137dc52f737ff9cc96ec9c4bc1199ce4c0 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-1.03%) to 94.898% when pulling 06efe3137dc52f737ff9cc96ec9c4bc1199ce4c0 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-1.03%) to 94.898% when pulling 06efe3137dc52f737ff9cc96ec9c4bc1199ce4c0 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.

Coverage decreased (-1.03%) to 94.898% when pulling 06efe3137dc52f737ff9cc96ec9c4bc1199ce4c0 on v3-precaching-activate into 0a6b15274e16eb25e0afa7e44538b98d108fb079 on v3.


Executing a Cmd with mandatory parameters defined but not provided in REPL window will get errors

In ISE:

In poshtools:

Technically related to adamdriscoll/poshtools/#120

Fixed in 68dc1bf


idea-info page ui updated

Issue No: 7
Improve the ui of idea info page #7
I have improved the UI of idea-info page with the theme of team info page

@Trisha-tech
nice....


Worker.py error every 30 seconds or so

Since some of the latest changes I'm getting the following error in my worker.log every 30 seconds or so
[2016-07-31 10:50:30,352][  worker-3][   ERROR][L 155] A wild exception appeared! Traceback (most recent call last):   File "worker.py", line 150, in run     self.main()   File "worker.py", line 192, in main     map_objects = response_dict['responses'].get('GET_MAP_OBJECTS', {}) TypeError: sequence index must be integer, not 'str'
Also, I'm noticing a much higher CPU usage than before. Anyone else noticing this?

Also, I'm noticing a much higher CPU usage than before. Anyone else noticing this?
Yep, I noticed the same. I was using 2 vCPU lunanode VPS, and the average usage when all 45 workers were working was around 1.50 ... after newest version, it jumped to ~5.00. So I had to decrease number of workers, to 30 to fit into max 2.0 average CPU usage :(

Looks like my CPU issues were related to some changes I had made in my fork. @amistad18 I could do you a deal on a VPS in Australia if you're interested?

@kravock Thanks for offer, but for me this is other side of the globe ... Poland here ;)
Anyway, I managed to lover the CPU usage by over a half, by disabling some unnecessary workers from the grid.


[BEAM-921] spark-runner: register sources and coders to serialize with java serializer

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @amitsela

Made changes discussed in the comments above.
R: @amitsela

Example failures are Dataflow and Apex related, and have nothing to do with this PR.
ResumeFromCheckpointStreamingTest is known to be flaky and should be handled separately (it's part of ongoing work on improving reading from UnboundedSource).

LGTM. I'll merge.


Fixes #12 : Prepare CloudStorage for Swift 6 Language mode

This PR fixes #12 by preparing CloudStorage for the Swift 6 language mode. It adds main actor isolation where necessary.
Possible backwards compatibility issues
Every class/struct that uses the CloudStorage property wrapper needs to be isolated to the MainActor. For iOS 18+, this is the standard for SwiftUI Views and issues with this change are minimal, so the impact of this PR should also have little impact on pre Swift 6 code.
I did a quick test and the functionality shouldn't be affected.

Thank you!


Improve build files

This is some lipstick. Mainly it improves readability for maintainers and output for users.

This change is

The move of things to generated folders introduces enough merge conflicts to warrant coming at this again on a new pull request.


Smiles file can't be opened

When you try to open files with extensions 'smi' and 'csmi' in Remote mode, molecules does not dispayed on canvas
Steps:

Choose 'Benzene' tool for drawing (at the left bottom)

Draw one 'Benzene' on canvas (one click on canvas)
Save to file
Reload page
Open saved file

ER:
'Benzene' did appear on canvas
AR:
'Benzene' didn't appear on canvas
The same flow for Standalone mode works correctly

Cannot reproduce. Please check if API_PATH is configured correctly.

Cannot reproduce. Please check if API_PATH is configured correctly.


Vanguard Xmas Swag

End of year tshirt gift for Vanguards

Any updates @aktdenis ?

@CaptainSplince I have couple of options ready. We'll have a discussion about swag on Wed at SIG-Design.


Message transport close takes a long time (5 seconds)

It appears to take 5 seconds for the message transport to close (asynchronously).  This means the server side lock manager doesn't notice the a departed connection for 5 seconds, and can end up blocking other connections from interacting with the system (because they are blocked on locks held by that client).
See: https://github.com/chrisdennis/galvan/tree/five-second-pause (BasicEntityInteractionIT)

When the client says goodbye to the server, it immediately closes its transport layer.  This causes problems on the server side.  Sometimes the goodbye is noticed first and proper closure on the server occur.  Sometimes the EOF from the socket close is noticed first and the server enters a reconnect cycle.  This reconnect cycle ends up in a delay on closure of the connection for a client that will never come back.  Will be fixed by sending the goodbye message and letting the server close the sockets.  The client side will close its sockets based the server side closure.


CORS Error

I was able to get around #3 with ng serve --host <IP_ADDRESS> --disableHostCheck true
Now I get:

Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://solid.community/.well-known/openid-configuration. (Reason: CORS header ‘Access-Control-Allow-Origin’ does not match ‘https://localhost:4200’).

Cannot reproduce personally, closing unless others raise the same issue


Charred Cherry doesn't sync with master

Isn't one reason, that we break the api? And the recent node does not work with the "old" wasm runtime?

Indeed, the reason is breaking changes in check_inherents and probably other functions.
 Useless("Verification failed: Error(CallResultDecode(\"BlockBuilder_check_inherents\"),

I'm not sure how this can be fixed in master, unless these breaking changes are reverted.

ok - i guess we could upgrade to the new API?

in any case, not a bug so closing.


Async Deletion

Polling the last operation after deletion:
When deletion has concluded, the expected response code is 410 (https://github.com/openservicebrokerapi/servicebroker/blob/v2.13/spec.md#asynchronous-operations). The only way to get LastOperation to return that response code is using brokerapi.NewFailureResponse. This workaround results in error logging, which isn't appropriate given this is actually a success condition.

This sounds like a bug in ODB. @deniseyu you may want to prioritise this story as part of the OSBAPI compliance track.
cc @kieron-pivotal

Hey @cholick
We've just taken a look at BrokerAPI to see how we could support this and, unfortunately, it doesn't look like there is a very elegant solution.
Returning a 200 to an async deprovision seems to be OK as per the spec, as a 200 response is defined as follows:
MUST be returned upon successful processing of this request. The expected response body is below.

The 410 response states Appropriate only for asynchronous delete operations., not that it MUST be used.
We agree that we don't really want to log a valid 410 response as an error, but changing the interface to allow custom response codes to be returned via BrokerAPI would be a breaking change that we would like to avoid making.
Since Cloud Controller is happy receiving a 200 response in this case today, how would you feel about just returning a 200?

The backing service thinks differently about deleted vs created. The operation query param is optional, so we can't be assured of differentiating between the platform asking "are you deleted yet?" vs "are you created yet?" Returning 410 lets us say "yup, this is gone" and the platform can bubble up failure because it was expecting the resource have successfully created, not gone away.
I didn't dig into the code, but 410 is a legit response in DELETE /v2/service_instances/:instance_id too; guessing that suffers a similar issue.

According to the spec, if a broker returns the operation string when starting an async operation, that query param MUST be provided to the last_operation endpoint.
410 is definitely a legit response, and 200 also seems legit in this scenario!

I see, thanks. That does give is enough context to fail when needed. 200 it is.


How to make this work in macvim?

I use macvim, which, from what I understand, is supposed to be very similar to vim. Almost everything about this tool works for me, except that when I hit control+F to create a file, Inkscape always seems to glitch out. In particular, when I hit control+F in insert mode, Inkscape will be launched, but then will quit itself before opening up anything. But I can still edit: When I hit control+F in command mode, I can select the file I want to edit, and the tool launches Inkscape just fine. I'm pretty new to all this, so I was wondering if anyone has advice on how to fix this. Thanks!

Try to run vim in your terminal. C-f would work. (it works for me at least)

Yes, vim in terminal works, but I was wondering about macvim in particular. I guess I could try neovim or something else though


pthread: add pthread_self/pthread_gettid_np function

Summary
explicitly defined functions can support assignment as function pointers
Impact
Testing
qemu x86_64

libs/libc/libc.csv, syscall/syscall.csv should also be updated appropriately

libs/libc/libc.csv, syscall/syscall.csv should also be updated appropriately

done.


add docs to deploy Chatwoot with Restack on kubernetes

Resolves #355
Adds docs along with screenshots to deploy Chatwoot on Restack.

@kuberaspeaking There is an error that is preventing the deploy preview, please have a look.


Thank you!

This is a great, and really powerful and featureful, theme to have as an open source project!

You are welcome, feel free to add a github Star, thanks!

Thanks indeed!


When the training will be stoped？

I run the training shell. And my loss down to 0.0. But when it will be stoped? Or Maybe Stop It by myself ???

@yanhuibin315 , Sorry for the delay. You can always specify the maximum number of training run you want. I am also printing the best validation rank and the epoch number when that best rank has been achieved.


Fix set default variant alert

I want to merge this change because...

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Translated strings are extracted.
[ ] Number of API calls is optimized.
[ ] The changes are tested.
[ ] Data-test are added for new elements.
[ ] Type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Test environment config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: SALEOR-1270 [Product] Missing save indication after selecting default variant


suggestion: free/unlocked in storage

suggestion: please can you add an option to make decorations free / unlocked in storage?
thank you!!!

Closing as added in 3bb39567a446360c6818d20be9b607b4002cd403


ripcord: init at 0.4.18

Motivation for this change

Ripcord is a desktop chat client for group-centric services like Slack and Discord. It provides a traditional compact desktop interface designed for power users. It's not built on top of web browser technology: it responds quickly to input, sips gently from computer resources, and gets out of your way. It does voice chat, too.

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @ldesgoui @MP2E @tadeokondrak

will come back to this in the next days

Hey @MDeltaX, do you think you'll get to this soon, and if not, do you mind if someone else pushes it along? Thanks for doing this work!

Does this need some Qt wrap hooks? I'm not able to run this on Wayland due to the usual Qt platform plugin err.

Christ this is old. Sorry this nobody ever bothered to merge this. My version is more up to date at this point anyway

Then let’s close in favor of the other PR.


Fix group expression not replace lowestCostExpressions in the group when group merge

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #4917
Problem Summary(Required) ：

When Group merge other goup,  the needReinsertedExpressions should consider the replacement of lowestCostExpressions in the Group, otherwise the optimizer will use wrong groupExpression when extractBestPlan

[FE PR Coverage check]
:heart_eyes: pass : 19 / 19 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/optimizer/Group.java
16
16
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/Memo.java
3
3
100.00%

[FE PR Coverage check]
:heart_eyes: pass : 19 / 19 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/optimizer/Group.java
16
16
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/Memo.java
3
3
100.00%

@Mergifyio backport branch-2.2


document need permission...

the document link in readme needs permission to view...

Just a problem of configuration in readthe docs.
Thanks for letting me know about this.


Enable scrollbars on the overflowing svg

Thank you for taking the time to report an issue with react-d3-tree!
Feel free to delete any questions that do not apply.
Are you reporting a bug, or opening a feature request?
bug/clarification
What is the actual behavior/output?
There are no scrollbars when the SVG is overflowing inside the container
What is the behavior/output you expect?
There should be scrollbars when the SVG is overflowing inside the container
Can you consistently reproduce the issue/create a reproduction case (e.g. on https://codesandbox.io)?
NA
What version of react-d3-tree are you using?
3.1.1
If react-d3-tree crashed with a traceback, please paste the full traceback below.
NA
Can someone help here please? TIA

@bkrem

@bkrem Can you please help, if you are available?

Hi,
Could you please share the relevant code (how you're rendering the tree, what container you've put it in), along with a screenshot of how your issue looks visually?
It's not clear to me from the description what the issue is.
Since the component effectively renders a canvas that can be panned/zoomed, there's no way to know when the element inside is effectively overflowing its container. So either the containing element is made larger or the tree's min/max/initial zoom level is adjusted
I'd also recommend checking out the svgClassName prop to see if you can customise CSS behaviour for your use case that way.

@bkrem Is there anyway we can achieve this as a new feature? Or can you guide me a hack to resolve this problem? This is a requirement for my app