Remove more dead codes

What problem does this PR solve?
Issue Number: ref https://github.com/pingcap/tiflash/issues/6233
Problem Summary:
Remove more dead codes after https://github.com/pingcap/tiflash/pull/7853. Make the codebase more clear
What is changed and how it works?

Remove the param try_persist from RegionTable::tryWriteBlockByRegionAndFlush because all are called with try_persist=false
Remove RegionTable::writeBlockByRegionAndFlush because try_persist is always false and the function is the same as writeBlockByRegion
Move tryPersistRegion to RegionKVStoreTest to reduce the code complexity in KVStore

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests


Add Content-Type: application/json in curl examples

Related to slack thread (internal) cc @osanseviero @Narsil @julien-c
Models served on Inference API with TGI (e.g meta-llama/Llama-2-70B-chat-hf) requires 'Content-Type: application/json' to be set as header otherwise the server throws an error (Expected request with Content-Type: application/json). Let's add it to the cURL snippets to be consistent even if models served with transformers don't need it.
Another possibility would be to disable the check on TGI side.
EDIT: it's also possible to add the header only to the text-generation snippets. Seems a bit inconsistent to me but I can update the PR if needed.

would stick it just after the actual payload

Right, I just made the change.

Thanks for the review, I'm merging this :)


Rollups not running in unit test context?

I have some Apex code that relies on a DLRS rollup. In a unit test, when I query the parent record after an update, the rollup field value is null. However, when I edit it in the UI (not unit test context), it works fine!
Here are more specifics:

Child object field 1 is set by user or unit test
Child object field 2 is set by Apex based on value of field 1 (before update)
??? DLRS rolls up value of field 2 to Parent object field 1 - in a test context, this remains null.

At first I thought it might be due to order of execution or having multiple triggers, but when I set child object field 1 in the UI, the rollup works and I get a value in Parent object field 1. Any ideas?
(and yes I'm re-querying the parent object to get the supposedly-updated-by-dlrs field value)

Sounds like a bug, i'll take a look.

So there are a quite a few unit tests in the tool itself that do this, so i don't think its a problem in the tool as yet. Also there is no kind of condition logic in the tool around testing context.
I am wondering though, are you using custom metadata based rollups or custom object based ones? If its the later, your test will not see the rollup definition and thus even though the DLRS triggers are deployed the engine will not do anything. You can resolve this by moving to custom metadata, using SeeAllData annotation or creating the rollup definitions in your test (only for Custom object mode). My recommendation would be to use custom metadata based rollups, so you don't have to use SeeAllData or create rollup definitions in your test.
Hope this helps!

Ah, I see, I'm using a custom object rollup - it all makes sense now. Thanks for your help! I'll switch to a metadata rollup or create the rollup objects along with my other test data.


CI: Retry a couple of times when getting “null“ as cache key

In quite a few CI runs on trunk, obtaining the SHA of the tree of latest commit of the compiler fails, and so caching does something fairly unreliable. To avoid that, retry a couple of times to get that SHA1, waiting quite some time between retrials, to give github time to come back up
In the runs to try out that commit, I saw a couple of times “null”, and every time, asking a couple of times didn’t help, hence it will now the display the API answer to try and debug the issue further

OK, not particularly pretty - but quite understandable. Do we need to do something similar for the Windows CI?

For better or worse, we are not using a cache on windows builds, so no need for windows.
(It might be time to try and reconsider whether we could manage to get caching in working order there, but that’s off-topic here ;-)

Fair enough, I'll merge then :+1:


Issues when left alone for long periods

The chat seems so connect to the room multiple times when left alone. This results in the user's message being amplified multiple times. This also results in a memory leak according to the warnings. Still need more details of the behavior.
Possible reasons:

Heroku server sleeps after 30mins of inactivity
Web socket behavior with socket.io

Hi I am a user of HackerChat. Ths is still a big issue!! pls fix!!1 Thanks

@BenEmdon please donate to our patreon to meet our goal of $2000 a month to continue development!

I am do nt haeve much money, I am only a hacker


About the comments : the word "keyboard" is mistakenly used instead of "keyword"

Thank you, it is fixed now


First final draft of the sample application code

Added projects fisalert-service, fisuser-service, maingateway-service, node.js and setup files.
Signed-off-by: Abdul Hameed <<EMAIL_ADDRESS>>

@redhatHameed as far as I recall, waiting for you to temporarily remove the email action so it can work without email, to then accept and merge the pull request. Then to have you investigate and add back optional email functionality in a new PR.

@myeung18 Marco, this is now merged, will wait for your PR


Add propOr on Maybe

IMHO should be nice to have a propOr function to use in addition to prop on the Maybe module, where we can pass a default value when the prop is not there. Maybe composing 'default' and 'prop' together can be an idea. I can create a PR if you want.

propOr and propPathOr are not specifically in there because I assumed that people would want to keep things in the Maybe for further expansion. For example say I want a prop that needs to be a number, I can then do things like:
const chain = require('crocks/pointfree/chain')
const compose = require('crocks/helpers/compose')
const isNumber = require('crocks/predicates/isNumber')
const prop = require('crocks/Maybe/prop')
const safe = require('crocks/Maybe/safe')

const validNumber = key =>
  compose(chain(safe(isNumber)), prop(key))

So it made sense that if a user would want to "stay" in the vanilla JS they would either bring it in from a lib like ramda or tiny-func or something that stays in JS. If you think there is value in adding this to crocks then, heck yeah, submit a PR.
But I do not think Maybe is the place for it as it does not return a Maybe all the time. It will have a signature like propOr :: a -> String -> b -> c
// propOr :: a -> String -> b -> c
function propOr(def, key, data) {
  ...
}

So I am think this should live inside of helpers as it really has nothing to do with Maybe. When implementing, try to use the bare minimum of crocks functions as this will bloat stack traces on errors, and only use function inside of the core folder. Will keep some :eyes: out for a PR.
And if you want, might as well make propPathOr.

PR is merged.
Looks like this feller should be closed.


Support Project ID

Describe the change
Please provide a clear and concise description of the changes you're proposing. Explain what problem it solves or what feature it adds.
Provide OpenAI documentation link
https://platform.openai.com/docs/api-reference/organizations-and-projects-optional
Describe your solution
I've added an optional header, same as the Organization ID.
As @massenz mentioned here, while the Project ID it is optional, it would be great to match the official OpenAI client and have a double check for the project scope.
Tests
I've tested the changes manually.
Resolves https://github.com/sashabaranov/go-openai/issues/757

Thanks for the patch!
The change looks good to me ✅


support nested objects and arrays

this is to have parity with how we flatten objects and arrays on the server side.
this is in tandem with this PR
@f2prateek @sperand-io @radiofreejohn
I basically took the same node module i used (flat) and used it here (gave credit to source code of course) and added a polyfill for Object.keys because for some crazy reason I could not replicate the exact behavior by using for (var prop in obj).... method.

@f2prateek @sperand-io John from Heap gave me the green light on this PR. Do you guys mind merging after you take a look?
we're basically following suit with what we did with Kissmetrics irt flattening/stringifying

lgtm


[Question]

I'm just getting started with Cloudflare.
I have two DNS 'A' records set up in the Cloudflare dashboard - one for my root domain and a wildcard record for the subdomains.
Will your script work for this setup?
Would I have to run it twice, once for each record?  And if so, will it handle the wildcard record correctly?

If they fall over the same domain, what you can do is to just update 1 "A" record (preferably the main domain) then use CNAME for the wildcard and point to the "A" record. This way only 1 record you need to update frequently.

To add on to K0p1-Git's comment above, if you're utilizing the same IP address for the wildcard sub-domains as you are for your root domain, then you can simply update the root domain's A record with one script and utilize your running web service (i.e. nginx/apache) to redirect to the desired sub-domain or root domain depending on what domain is requested by a useragent. In addition to psycho-neon's comment, utilizing a CNAME for the wildcard sub-domains pointing to the root domain would also work, though would require running two scripts instead of one. In the future, multi-domains may be supported, though is not currently at this time.


Feat(web-twig): Link helper texts and validation texts to inputs to improve accessibility

⚠ WIP
Affected components:

[ ] Checkbox
[x] FieldGroup
[ ] FileUploaderInput
[x] Radio
[ ] Select
[ ] TextFieldBase

coverage: 96.46%. remained the same when pulling b1ac74668ce3a130aa0a4d96b70ce9fe83908a8a on feature/twig-link-helper-text-to-inputs into 0df0fbd63e6f6a2903b510e5a99685318d0b3110 on feature/twig-helper-text.

Rebased against current main.

Didn't we do this (partly) in https://github.com/lmc-eu/spirit-design-system/pull/1470?

Superseded by #1470.


Implementation of 1D Euler's PDEs system test

Here we can discuss the implementation of the 1D Euler's PDEs system test. This is a complex PDEs system (non linear, unstedy, hyperbolic, discontinuos solutions admitting...). It is a good candidate for the benchmark analysis.
I have already implemented it, but not uploaded due to some minor bugs. I should upload it soon in the next week.

Done in v0.0.4


[TimexExpression] TimexRangeResolver with only time candidate and a specific date range constraint does not resolve

Describe the bug

Use only a time candidate, for example "T09" ("9 o clock")
Use a date range constraint with specific dates , for example NextWeeksFromToday(3)
TimexRangeResolver.Evaluate() does not resolve into anything.
However, using a non-specific range constraint, for example "P21D" resolves into a result.
In the unit test below I was expecting "T09" to resolve to 2020-01-01 09.00 (and 21.00).

To Reproduce
Unit tests that demonstrates the bug - add these to TestTimexRangeResolve.cs
[TestMethod]
public void DataTypes_RangeResolve_time_with_date_constraints_should_resolve()
{
    var resolutions = TimexRangeResolver.Evaluate(
        new[] { "T09" },
        new[] { "(2020-01-01,2020-01-02,P1D)" });

    // Should resolve to 9am on Jan 1
    Assert.AreEqual(1, resolutions.Count);
}

[TestMethod]
public void DataTypes_RangeResolve_time_with_daterange_constraint_no_results()
{
    var candidates = new[] { "T06" };
    var constraints = new[]
    {
        // Does not resolve because we have a date range constraint with specific dates
        TimexCreator.NextWeeksFromToday(3),

        // Using "P21D" resolves correctly into a result (altough not with a date)
        // because we don't have a constraint with specific dates:
        // "P21D"
    };
    var resolutions = TimexRangeResolver.Evaluate(candidates, constraints);
    Assert.IsTrue(resolutions.Count > 0);
}

[TestMethod]
public void DataTypes_RangeResolve_time_with_datetime_constraints_should_return_dates()
{
    // Same problem even if we add times to the constraints
    var resolutions = TimexRangeResolver.Evaluate(
        new[] { "T09" },
        new[] { "(2020-01-01T00:00:00,2020-01-02T00:00:00,PT24H)" });

    Assert.AreEqual(1, resolutions.Count);
}

[TestMethod]
public void DataTypes_RangeResolve_switched_constraints()
{
    // Switching candidate and constraint resolves to "T09",
    // not to a datetime as one might expect.
    var resolutions = TimexRangeResolver.Evaluate(
        new[] { "(2020-01-01,2020-01-02,P1D)" },
        new[] { "T09" });

    Assert.AreEqual(1, resolutions.Count);

    // Fails. Dates are not included in resolutions
    Assert.AreEqual(1, resolutions.First().Month);
}

[TestMethod]
public void DataTypes_RangeResolve_time_workaround()
{
    // The workaround is to input all possible candidates at time 9.
    // Now all constraints works as expected, and we resolve to a datetime.
    var resolutions = TimexRangeResolver.Evaluate(
        new[] { "2020-01-01T09", "2020-01-02T09" },
        new[] { "(2020-01-01,2020-01-02,P1D)" });

    Assert.AreEqual(1, resolutions.Count);
    Assert.AreEqual(1, resolutions.First().Month);
}

Platform (please complete the following information):

Platform: .NET Core
Environment: nuget package
Version of package: v1.3.2 (27 feb 2020)

Fixed the bug and submitted a PR.
Added the unit tests above. All previous unit tests run successfully.


google/Web Fundamental COC(issue)

Welcome! Please use this template for reporting documentation issues on https://developers.google.com/web/.
For questions about Chrome DevTools, checkout the
google-chrome-devtools tag on StackOverflow.
For questions about Chrome, check the google-chrome
tag. If you think you've found a Chrome bug, check out our
feedback page for more details on how to file it.
Page Affected: https://developers.google.com/web/TODO
What needs to be done?
Please provide description of what's needs to be done here. Please be sure to include the link to the original page
and any supporting links/data.

[x] Appropriate type-something label has been added.
[x] Appropriate section-something label has been added.

[Welcome! Please use this template for reporting documentation issues on https://developers.google.com/web/.  For questions about Chrome DevTools, checkout the google-chrome-devtools tag on StackOverflow. For questions about Chrome, check the google-chrome tag. If you think you've found a Chrome bug, check out our  feedback page for more details on how to file it. Page Affected: https://developers.google.com/web/TODO What needs to be done? Please provide description of what's needs to be done here. Please be sure to include the link to the original page and any supporting links/data.  - [ ] Appropriate type-something label has been added. - [ ] Appropriate section-something label has been<EMAIL_ADDRESS>

Welcome! Please use this template for reporting documentation issues on https://developers.google.com/web/.
For questions about Chrome DevTools, checkout the
google-chrome-devtools tag on StackOverflow.
For questions about Chrome, check the google-chrome
tag. If you think you've found a Chrome bug, check out our
feedback page for more details on how to file it.
Page Affected: https://developers.google.com/web/TODO
What needs to be done?
Please provide description of what's needs to be done here. Please be sure to include the link to the original page
and any supporting links/data.

[ ] Appropriate type-something label has been added.
[ ] Appropriate section-something label has been added.


Shop with save system

Create a shop menu that allows the player to buy max health increases and starting weapons. Create a save system to store the upgrades and money gained that can be bought in the shop. Also allows runs to be saved midway

I don't think I'm going to be able to get this done.


Add type definitions

This adds TypeScript definitions to the package. I had to manually type the package to use it in my project, so I thought this would be a little nicer.
The index.d.ts file is generated automatically as part of the build process using tsc, a minimal tsconfig.json, and the JSDoc annotation that's already there — figured this would be easier to maintain than a manual typedef file, but I could be wrong. I'm happy to make any changes you see fit.

Hi @john-royal ,
thank you for your contribution! Adding TypeScript definitions to the package is great idea and will certainly make the package accessible to more broader scope.
Feel free to make any changes you see fit.


Update Rahul-Raghavan.md

Added working sessions

@DinisCruz & @torahulraghavan: Your PR contains a conflict, please resolve locally and push again so we can merge it.

ok I just fixed it and merge them


Feature: Have other linear interpolation for percentiles

xclim version: 0.28
Python version: 3.x
Operating System: Mac os

Description
Follow up to the discussion with @huard.
In other climate indices libraries such as climdex and icclim, the percentile interpolation method used is the 8th method of Hyndman and Fan. Both library have their own C/C++ implementation.
In Xclim, the default interpolation of numpy is used, which correspond to the 7th method of Hyndman and Fan.
However numpy does not implement the 8th method (see https://github.com/numpy/numpy/issues/10736#issuecomment-897851315).
It would be great if xclim:

Use the Scipy mquantiles method instead of np.percentile
Have a parameter to choose the interpolation in calendar.percentile_doy

As usual, I can work on this issue.

Question: there are many places in the xclim code where we use DataArray.quantile. Do we need to change this as well, or we are focussing here on percentile_doy only ?

Well I guess it depends.
The method 8 is particularly fit if you don't know the distribution function of your sample and your goal is to have an approximately unbiased median. It is advocated to be the best default function for quantile estimation.
However, if you know the distribution function of your sample you may have a better estimation with another linear interpolation method.
For example the method 9 of Hyndman&Fan is better fit, if the sample follow a normal distribution.
Plus, If you which to have an unbiased mean instead of an unbiased median, other methods could give better results. I found this on this paper, but it is not published in a scientific journal AFAIK.
I'm not qualified to know if it would be a good idea to use method 8 everywhere.
@pagecp Do you have a thought on this subject ?

@bzah Precipitation is clearly not a normal distribution, but temperature is much more... and the median is better than the mean for sure because it takes better into account the number of samples. So I would say method 9 for temperature may be better, and method 8 for precipitation....
But I am not an expert at all in statistics.

I spent some time jumping from library to library to see how others do this. It's a kind of a mess, so I will summarize here what I learned.
TL;DR
IMO, it is safe and best to use scipy::mquantiles with method 8 parameters on percentile_doy and only on percentile_doy.
On the method to use
As stated above, the method 8 of hyndman&fan should be used if we don't know the distribution function of the sample.
The method 9, should be used if we know we are working a normal distribution (such as temperatures).
For other distribution function I'm really not sure...
In the case of day of year sample, I don't think there is a recognizable distribution function even for temperatures. Instinctively I would think daily averaged temperature of a date is independent of the temperature in the previous year at the same date, but maybe I'm wrong.
In other cases, the best fitted method will depend on the data handled and if the distribution function is known.
About performances
In order to find a quantile we need to have a look at each value in the array and reorganize them.
We can either sort the whole array or partially sort it. The second option is what numpy does in ::percentiles which, I suppose, gives it such good performances on large arrays.
In the case of day of year values, it doesn't matter much because ::percentile_doy expect daily values or coarser, and even if the dataset span on a 100 years, the sort will be done a 100 doy values. In fact, it might even have better performances to use ::sort instead of ::partition on small arrays due to the possible overhead of numpy ::partition function (see here).
In other cases, it depends. On SciTools/Iris project for example, they have both numpy percentiles and scipy mquantiles and they let the user decide if they prioritize performances using numpy::percentiles or they prioritize correctness with scipy::mquantiles.
I don't like this approach because IMO it exposes too much of the internal mechanisms to the user.
About dask
To make things even simpler, Dask has its own implementation of quantiles.
This is probably necessary because to compute quantiles it may need to access different chunks.
For example with doy values, if the data is chunked on time, to computation of quantiles needs to access most (all?) of the chunks. However when chunked on lat/lon, we could have all doy values of interest per chunk.
A good explanation of this problem : https://github.com/SciTools/iris/pull/3901#issuecomment-797527498.
There is also some known issues with Dask implementation of quantiles: https://github.com/dask/dask/issues/6566
I'm still a big noob with Dask, so this is probably an issue that I don't fully understand.
About other methods
Because the fun never ends, there are other methods to compute quantiles.
One in particular is a "concurrent" to the 8th method as the best method when the distribution function is unknown.
This method is described in Harrell&Davis 1982.
It was known to Hyndman&Fan but not included in their paper from 1996 because it was not yet implemented in widely used packages.
Now scipy has an implementation of it under scipy.stats.mstats_extras::hdquantiles and is supposed to have very good performances (at least since this PR)
It may be particularly interesting on small samples if the distribution is not a normal distribution.
So it might be a good idea to use this one instead of method 8, but it's not clear to me what would be the side effects.
Final note
@huard I think it is best to see case by case if DataArray.quantile can be replaced by scipy::mquantiles, because:

The default of numpy is not that bad and is widely used.
When the backend is Dask it will call it's own implementation.
To choose the very best method to use, some analysis must be done on each case.

Thanks @bzah for the explanations.
I'm happy to let you decide on a course of action. I suggest we focus on percentile_doy for now and not try to tweak with the DataArray.quantile.

As a side note here to give some confidence in this code:
For rx90p, when bootstrapping the percentiles, it now gives results very similar to those of climdex.
Only a few points seem to be inaccurate and for some reasons mostly out of the bootstrap period.
Here the bootstrap was on 1990-2000.


We should use CDN in issue template JSFiddle

Types of changes

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

Related issue(s):

#110

Checklist:

[x] My code follows the code style of this project,
[ ] My change requires a change to the documentation.

The changes have been added under the original link


Remove tabs for disabled features

Setting ENABLE_QUERY, ENABLE_CATALOG to False should remove the Query and Catalogs tabs, respectively.
Presently, the Query and Catalogs tabs persist when their respective features are disabled, and accessing the tab results in a 'Forbidden' page. Experienced on Puppetboard v3.1.0

Hey, i provided a PR with changes that should fix it

@sebastianrakel @gdubicki thank you!


Node.js uses things from <uv.h> that aren't documented. Bug in Node.js or just lacking documentation?

Hi there,
For CloudABI I am adding a libuv compatible event loop directly to the C library. The official libuv can't be used here for a couple of reasons:

A fair number of interfaces provided by libuv (signal handling, global filesystem namespace, etc.) are simply not available on CloudABI. I don't want to turn the official libuv codebase into an #ifdef fest.
There is nothing available like kqueue, epoll, etc. I want libuv to be the core API against which event-driven software should be written; "libuv is my kqueue". At the same time I'd like to investigate whether I can implement this in a more async fashion, where operations can be enqueued within the kernel. We'll see.

Long story short, I'm trying to build Node.js against cloudlibc's of libuv. This is going pretty well so far, except that I run into various compilation errors for which the best course of action is debatable:

Uses of struct uv_*_s, as opposed to uv_*_t.
Uses of UV_ERRNO_MAP().
Relying on definitions that are available through <uv.h> implicitly, such as constants from <sys/socket.h>, <netinet/in.h>, <netdb.h>, etc.

My question is, are these things that Node.js should stop doing, or is it fair to say that that's simply part of what libuv offers? If it's the latter, do you want me to take a stab at documenting this?

Uses of UV_ERRNO_MAP().

Uses of struct uv_*_s, as opposed to uv_*_t.

The struct uv_loop_s forward declaration in node.h is to avoid pulling in uv.h for a single type (which is only used once and then only in an API intended for embedders.)
I found only one other place where a uv_*_s type was used and I filed a PR to fix that.

Relying on definitions that are available through <uv.h> implicitly, such as constants from <sys/socket.h>, <netinet/in.h>, <netdb.h>, etc.

Node.js should not be doing that, no. :-)

Hallo Ben,
Thanks for replying so quickly! I'll make sure to add UV_ERRNO_MAP() and struct uv_*_s to my implementation and send out a pull request for documenting those. At the same time, I'll send pull requests to Node.js for fixing some of those implicit header dependencies.

I also noticed that Node.js's src/stream_wrap.h:70 uses uv_pipe_t::ipc. Ben, do you think we should also start making this part of the documented API?

It's not considered private.  That said, there is no reason node.js couldn't just remember the type of handle and that's arguably a little cleaner.

It seems like this can be closed. There may be some things that can be addressed in Node.js, but there's no need for an issue here to remain open. Please reopen if I'm wrong.


Locking deck is not working

I tried locking my deck (jongmaihub deck), but when I am not logged in, I can still access (use flash card, fill in the blank, and also test mode). Maybe I misunderstood the purpose of your deck locking feature.
This is when I logged in with JongMaiHUB account.

This is when I logged out.

Now it just a mock-up. Work in progress. Thank you for reporting.


Use type-independent image functions

Greetings 👋!
Using imagecreatefromstring() instead of imagecreatefromjpeg() would prevent developers seeking to use this library from thinking about "but what if I want to use PNG or GIF images?"
Using imagecolorsforindex() instead of bit-shifting would reduce the mental overhead even further (just speaking from my own experience 🙈)
(Tested by adapting the tests to match the usage examples)
:octocat:

Thank you!


jn_zeros() hangs when trying to compute zeros for several bessel orders.

jn_zeros() is not able to compute the first 100 zeros of the following Bessel orders below 1000(I only tested it till 1000):
231,244,281,288,347,348,357,405,406,419,437,
505,506,507,570,582,591,643,644,655,658,679,
706,713,722,752,756,757,764,775,793,796,811,
820,840,855,875,886,916,942,948,966
Reproducing code example:
from scipy import special
special.jn_zeros(231,5)

Error message:

No error message! The command just hangs while using 100% CPU on one core.
Scipy/Numpy/Python version information:

1.4.1 1.18.2 sys.version_info(major=3, minor=8, micro=2, releaselevel='final', serial=0)

I have same problem.
>>> from scipy import special
>>> special.jn_zeros(1200,5)
array([1219.81758084, 1234.77622746, 1247.10400581, 1258.06004816,
       1268.12511012])
>>> special.jn_zeros(1250,5)

Here it is hanging =(
My scipy version
>>> import scipy
>>> print (scipy.__version__)
1.5.4

@Berxue and @pukhlyakova, thanks for reporting the problem.  This issue is a duplicate of https://github.com/scipy/scipy/issues/4690, so I'm closing this one.


Support 2 langages (fr, en)

Use Angular to load new text and replace live presentation

Close #1


BetterDiscord not installing

I've had multiple problems with it so I tried to reinstall both discord and betterdiscord but now it wont show at all.
Problem
Any suggestions?

Same here.


39 Add premerge ci tests

Fixes #39 .
Description
This PR is used to add some premerge ci tests for verifying changed bundles.
Status
Ready
Please ensure all the checkboxes:

[x] Codeformat tests passed locally by running ./runtests.sh --codeformat.

/build

/build

/build

/build

/build

/build


How to debug a Faster RCNN step by step？

I want to know the process of model forward and compute the loss,how can I get it by debugging?Which script should I debug?

You can try to use pdb https://docs.python.org/3/library/pdb.html or use an IDE to debug.

You can try to use pdb https://docs.python.org/3/library/pdb.html or use an IDE to debug.

so which script should I debug ?


Request: robust function to compute log(Phi(x))

In implementing a hierarchical tobit model I ran into some initialization problems that I finally tracked down to the following line:
lp__ <- lp__ + log(Phi(...))

where the quantity (...) can be large and negative. What happens is that Phi(...) rounds down to zero, so lp__ becomes negative infinity. You'll get this problem unless your initial parameter values are such as to make EVERY zero in the data sufficiently probable. To avoid the need to fine-tune the initial parameter values, it would be really nice if there were a robust function to compute log(Phi(x)), as log(Phi(x)) is representable as a double even when Phi(x) is not.
I ended up working around the problem by using the identity
log(Phi(-x)) = log(Phi(-x)/phi(x)) + log(phi(x))
    = log((1 - Phi(x)) / phi(x)) - 1/2 * (x^2 + log(2 * pi))

and then using the following Taylor series for (1 - Phi(x))/phi(x):
1/x - 1/x^3 + 3/x^5 - 15/x^7 + 105/x^9 + O(1/x^11).

I'm closing this issue: Bob's fix has made this more stable and we haven't seen it pop up recently.


uniform gap between the sections in the navigation bar

I would like to fix the spacing i.e move the sections towards left in  the navigation bar. Kindly assign this issue to me under GSSOC'23

share the page on which you would like to work , take screenshots of the issue.
Use the issue template to raise an issue.
Also, issue pertaining to navbar have been raised multiple times, do check existing issues first.

I would like to shift the sections in the navigation bar towards the left instead of being spread on the entire navigation bar.

Kindly assign me this issue under GSSOC'23

navbar on the homepage is updated, pl do a fresh fetch request to see the changes.


adding in summary on low_account_usage

Description
Adds in Summary to low_account_usage
Issues Resolved
[List any existing issues this PR resolves]
Contribution Check List

[ ] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] New functionality has been documented in CHANGELOG.MD

          1 Warning

:warning:
Should this include readme changes

  Generated by :no_entry_sign: Danger


Update quick-create-portal.md

 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.:x: nohaelanwer sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@nohaelanwer : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi, @nohaelanwer. It's unclear to me what you did, here.  The diff is registering the entire article as a change. Can you let me know what specific revisions you are suggesting?  Thanks!


Fix Column DB query flood

PR Description

Get rid of DasCustodySync.fillUp() calls flooding
Piggy tailing ConcurrentModificationException fix for the Sampler

Fixed Issue(s)
Partially address #108

LGTM Does it fix #67 btw?

Yeah, I hope so!


collision detection

Hello, thank you very much for the efforts! The video looks very cool! Just a question, how do you make collision detection and realize free collision? Because I don’t find the referenced code.
Thanks!

Collision detection is done using the rvo2_3d and PQP libraries which you can find under the libs directory.

Thank you!


[bug] simple query becomes extremely slow when nested

InfluxDB shell version: 1.3.0~n201703100800
I'm wanting to get a count of unique hosts and was hoping to do it by nesting a GROUP BY query with a select count.  It works but is typically 30x or more slower when nested.
Normal query runs under 2 seconds:
% time influx -database testdb -execute 'SELECT last("cpu_time") FROM "host_cpu" WHERE "group" =~ /^dev$/ AND "job" =~ /^test/ AND time > now() - 1h GROUP BY "host"' | tail

name: host_cpu
tags: host=host233
time       last
1489158241 3.4

name: host_cpu
tags: host=host234
time       last
1489158241 22.3

_real	0m1.400s
user	0m0.014s
sys	0m0.008s_
----       -----

Same query now wrapped takes 56 seconds:
% time influx -database testdb -execute 'SELECT count(last) FROM (SELECT last("cpu_time") FROM "host_cpu" WHERE "group" =~ /^dev$/ AND "job" =~ /^test/ AND time > now() - 1h GROUP BY "host")' | tail

name: host_cpu
time count
0    40

_real	0m56.615s
user	0m0.007s
sys	0m0.008s_
----       -----

Maybe I'm missing something?
Thanks,
Dave

@CarusDave thanks for the report. Can you provide us with information about how big the testdb database is? Do you know how many series, measurements and overall points there are?

Here's some database stats.
name: database
tags: database=testdb
numMeasurements numMeasurementsDropped numSeries numSeriesDropped

6               0                      13386006  2211227
name: write
pointReq  pointReqLocal req    subWriteDrop subWriteOk writeDrop writeError writeOk writeTimeout

303133578 303133578     130978 0            130978     0         0          126586  1987
name: httpd
tags: bind=:8086
authFail clientError pingReq pointsWrittenDropped pointsWrittenFail pointsWrittenOK queryReq queryReqDurationNs queryRespBytes req     reqActive reqDurationNs   serverError statusReq writeReq writeReqActive writeReqBytes writeReqDurationNs

0        51          914938  0                    9326212           278013215       65679<PHONE_NUMBER>81052    459533751      1052026 8<PHONE_NUMBER>13185 1986        0         71409    1<PHONE_NUMBER>44<PHONE_NUMBER>1745
name: runtime
Alloc       Frees        HeapAlloc   HeapIdle    HeapInUse   HeapObjects HeapReleased HeapSys      Lookups Mallocs      NumGC NumGoroutine PauseTotalNs Sys          TotalAlloc

62311512920<PHONE_NUMBER>18<PHONE_NUMBER>0<PHONE_NUMBER>0<PHONE_NUMBER>8 400858492<PHONE_NUMBER>2<PHONE_NUMBER>08 6694638<PHONE_NUMBER>10 4629<PHONE_NUMBER>35487<PHONE_NUMBER>40<PHONE_NUMBER>2424
name: queryExecutor
queriesActive queriesExecuted queriesFinished queryDurationNs

7             65667           65660<PHONE_NUMBER>36766
Thanks,
Dave

Slightly related to #3880 as that would make this query unnecessary, but we still need to look into the query performance anyway even if #3880 is implemented.

Is there any way we can have access to the data you're using? I tried using influx_stress to generate a simple database with the defaults and I'm not seeing the speed issue. I'm seeing the last() call alone taking longer (most likely due to memory allocations).
If the data is private, we can provide you with a place to send us your data so you don't have to post it publicly.

Thanks for looking into this. I was also thinking it was memory related partly due to the huge size of the memory index for this instance, though we are sitting at 40% system memory utilisation.
We know we are pushing the boundaries a bit with the numbers of tags, and a few of them could contain 0.5mil unique values within the retention policy we are using, but a requirement for our particular use case.
The data is private, but I can give you more details on this particular schema that might help you?
So in the meantime, I decided to troubleshoot a few things. Indeed, I found that by reducing the database size and hence the memory footprint (20x smaller), this query ran very much faster. Actually the nested count now runs slightly faster than the group select on it's own (maybe expected).
Here are the latest info on the smaller database (not sure how to get the nice table)
% time influx -database testdb2 -execute 'SELECT count(last) FROM (SELECT last("cpu_time") FROM "host_cpu" WHERE "group" =~ /^dev$/ AND "job" =~ /^test/ AND time > now() - 1h GROUP BY "host")' | tail
name: host_cpu
time count
0    40
real	0m0.117s
user	0m0.012s
sys	0m0.000s
name: database
tags: database=testdb2
numMeasurements numMeasurementsDropped numSeries numSeriesDropped

5               0                      807676    0
name: write
pointReq pointReqLocal req   subWriteDrop subWriteOk writeDrop writeError writeOk writeTimeout

15141795 15141795      12834 0            12834      0         0          12242   0
Thanks,
Dave


isLight Sermon Audio

I did manage to find at least one instance of a series having a different isLight value, which causes this issue:

I'm not sure if that value is being used on the site? If it is, this may be harder to fix. If it's not, when need to just go through and fix the values.

This looks to be an issue with data in EE. Maybe a default or something?
We are getting back lightswitch: light for a lot of old series, and lightswitch: dark for newer series.
Is it safe to assume that right now all our series have dark collection colors? I haven't seen ones that have a light collection color, but I know we have talked about ones that should be changed to a light color. If we can assume dark, then I can easily batch update all the series with some SQL.
@jbaxleyiii @jonhorton

per convo with @jbaxleyiii, we are going to go through and actually fix the colors and the switches.


fixed with of profile-list list-elements to cover 100% of the width

@guillemcordoba please check. Supposed to improve the UI of the profile-list element.
Before:

After:

Oh and there is also a change in the profile-prompt (removing justify-content: center). That's open for discussion but I think it's better to let this open to be defined in the parent because in case of using the slot in the profile-prompt things can get out of the viewport vertically in case of justify-content: center and overflow-y: scroll.

Second issue has been resolved by c36a7c041bd4289971ae47a3f0a836b8b3afea26


dragStart and dragMove triggered without dragging

Hi,
The events dragStart and dragMove are both triggered on mousedown without moving. I think at least dragMove should only be triggered when you're actually dragging/moving.
tns 2.9.3 / chrome+edge v98 / win10

+1


VlcControl cant set audio output device

Is there an options to set audio device to my prefered device?
If there is a documentation or sample program, i sos need that!
Thanks!

param as  stringcollection
param.Add("--aout=directsound")
param.Add("--directx-audio-device=" + DAudioList.Item(ComboBox3.SelectedIndex).deviceID)
// with debug.print it shows the right option in param
VlcControl1.SetMedia(New IO.FileInfo("c:\Documents\test.avi"), New String() {param(0), param(1)})
VlcControl1.Play()
if i put these two param directly to vlcmediaplayeroption property in design view then it works.
What i did wrong?

You need to pass the parameters as VlcMediaPlayerOptions (not media options), before the component has finished being initialized.


FCPListItem does not support imageUrl

Current implementation to set an image to a listItem is using an asset
    if image != nil {
      listItem.setImage(UIImage().fromFlutterAsset(name: image!))
    }

It should be possible to also use a HTTPs link to an image so we don't need to ship every image in assets

Hi @vanlooverenkoen I was able to load list items with image url by doing the following.

Open the flutter iOS project in XCODE

Open the PODS->Development Pods->flutter_carplay

Open the preceding .. unto the last directory, you will find flutter_carplay->Classes->Models -> Lists

Navigate to FCPListItem.

Find the codes:

if image != nil { ...
7.Replace with :
` if image != nil {
if image!.starts(with: "http"){
     let url = URL(string: image!)
     let stationImage = try? UIImage(withURL: url!)

     listItem.setImage(stationImage)
 }else{
     listItem.setImage(UIImage().fromFlutterAsset(name: image!))
 }

}`

Inside update, do the same.
This will intercept all urls beginning with http with custom loader and those beginning with any other format as asset images.
Thank you.

Yes indeed that is why a pull request implementing this feature a month ago. #24

flutter: Warning database has been locked for 0:00:10.000000. Make sure you always use the transaction object for database operations during a transaction  , After loading images from ulr i got this error and the mobile app ui freezes .


Improve Eligibility report v2

Background
Following November's Digital meeting, improvements to the Eligibility report, including Met/Not met flags, and several bug fixes were implemented. There has been further discussion with Rob, Dan and Matt highlighting the challenges of automating the creation of SCC Eligibility annexes due to requested format of information and the lack of consistency in eligibility requirements across multiple exercises. The objective with this ticket is to focus on the annexes required for Legal Exercises - in particular, statutory eligibility reqs (qualifications and PQE) and non-statutory reqs (PJE and RLoS.)
User Story
As an Admin, I would like to be able to input the eligibility recommendations content on the platform so that I can download the specified SCC-ready annexes for the SCC report.
Benefit(s) to user (if not already clear from User story)
Admins will be able to add recommendations for individual annexes rather than just an overall recommendation; this will support generation of the annexes according to the set template.
Feature(s) Description

[ ] Reconfigure eligibility report screen for legal exercises as in Miro board (covers additional points below too)
[ ] Add additional content from two fields on new application form
[ ] Add additional recommendation dropdowns
[ ] Add additional free text fields
[ ] If there is no eligibility criteria specified, e.g. for PJE, that section should not be included onscreen

Acceptance Criteria
It's done when:

Admins can make separate recommendations for Stat criteria, PJE and RLoS on Legal exercises
Admins can view candidate responses where they haven't met PJE or RLoS on Legal exercises
Admins can add separate comments about their separate recommendations for Stat criteria, PJE and RLoS on Legal exercises

Definition of Done

[ ] User stories/acceptance criteria met
[ ] Internal reviews passed (feedback actioned)
[ ] User testing passed
[ ] Relevant technical documentation updated
[ ] User guidance updated
[ ] Deployed and merged without errors

User Testing Steps
We are aiming to be able to create the content for Legal Statutory, PJE and RLoS SCC eligibility annexes on the platform. Test the showcased functionality by interacting with the page as you would when trying to conduct this activity.
Feature Champion
Arti Sherry

Hey team! Please add your planning poker estimate with Zenhub @drieJAC @HalcyonJAC @tomlovesgithub @warrensearle @KoWeiJAC

Please add your planning poker estimate with Zenhub @KoWeiJAC


Add necessary kubernetes template files for kubernetes-sigs move

This PR adds the necessary template files from https://github.com/kubernetes/kubernetes-template-project for the move to kubernetes-sigs.
There are probably a few areas that can be fleshed out more that haven't been yet. Here are some things in no particular order:

For the contributing.md it was left fairly vanilla from the template file, except I added a contributing a patch section taken from the cluster-api-provider-vsphere contributing.md.
For the Owners Alias, it currently only has @schrej, but we could add some/all of the VMware based contributors we have on our team (@tylerschultz @adobley @flawedmatrix and @christianang). Wasn't sure if it was better to do it before or after the move so I left it as is. Same with the Security Contacts file.
For the readme II added the Community, discussion, contribution, and support section, but it doesn't link to any specific cluster-api-ipam-provider-in-cluster specific slack or mailing list yet. Not sure what the process there is, but maybe that can be figured out later?

Relates to #110

We definitely need more than one reviewer and approver. I'd be happy to have all mentioned contributors as approvers, and probably @tylerschultz and @srm09 as approvers.
Before anyone can get added as approver or reviewer, they'll need to join the kubernetes-sigs org. We can potentially sort that out post-move if pre-move contributions to this repo count.

Let's get this in now though, since it's the last step before opening an issue to transfer the repository. We can update reviewers, approvers and security contacts later.


FIX - error.confirms.deal-not-found suggestion

To fix the "Deal not found" error below:
File "...\anaconda3\lib\site-packages\trading_ig\rest.py", line 222, in parse_response_with_exception
raise(Exception(response['errorCode']))
Exception: error.confirms.deal-not-found
I would recommend that you change the code at roughly line 442 of rest.py to use try and except like the following example. This enables the code to continue running despite not being able to confirm the deal - making it more robust.
if response.status_code == 200:
deal_reference = json.loads(response.text)['dealReference']
try:
deal_ref = self.fetch_deal_by_deal_reference(deal_reference)
return deal_ref
except:
print("NO DEAL FOUND")
#return self.fetch_deal_by_deal_reference(deal_reference)
else:
return response.text  # parse_response ?

Please create a pull request

@mrsocky32 Looks like something you should handle in your code. The library is raising an exception as it should; it is up to you to catch that exception and handle it gracefully in your program, and opt to continue execution if you choose to.

closing this, no activity


[getopt-win32] Allow use as a CMake package

Describe the pull request

What does your PR fix?
This PR allows using getopt-win32 as a package using find_package(unnoficial-getopt-win32 REQUIRED), and then using its target using target_link_libraries(main PRIVATE unofficial::getopt-win32::getopt), for example. It correctly selects the Release or Debug library, allowing developers not to bother with it, like previously with find_library.

Which triplets are supported/not supported?
As long as I know, yes.

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

sorry, wrong label.

No problem. Now I think it's everything ok =).

Nice! Do I need to do something else?

Is this added?  I just pulled master and while I was able to find the target, there's no include directories exported with it

Is this added? I just pulled master and while I was able to find the target, there's no include directories exported with it,
Here's the properties availible on the target:
unofficial::getopt-win32::getopt BINARY_DIR = C:/Users/User/Documents/GitRepositories/Repo/cmake-default-debug
unofficial::getopt-win32::getopt IMPORTED = TRUE
unofficial::getopt-win32::getopt IMPORTED_CONFIGURATIONS = DEBUG;RELEASE
unofficial::getopt-win32::getopt LOCATION = C:/Users/User/Documents/GitRepositories/Repo/cmake-default-debug/vcpkg_installed/x64-windows-static/debug/lib/getopt.lib
unofficial::getopt-win32::getopt LOCATION_Debug = C:/Users/User/Documents/GitRepositories/HiParTI/cmake-default-debug/vcpkg_installed/x64-windows-static/debug/lib/getopt.lib
unofficial::getopt-win32::getopt MACOSX_PACKAGE_LOCATION = C:/Users/User/Documents/GitRepositories/Repo/cmake-default-debug/vcpkg_installed/x64-windows-static/debug/lib/getopt.lib
unofficial::getopt-win32::getopt NAME = unofficial::getopt-win32::getopt
unofficial::getopt-win32::getopt SOURCE_DIR = C:/Users/User/Documents/GitRepositories/Repo
unofficial::getopt-win32::getopt TYPE = STATIC_LIBRARY
unofficial::getopt-win32::getopt VS_DEPLOYMENT_LOCATION = C:/Users/User/Documents/GitRepositories/Repo/cmake-default-debug/vcpkg_installed/x64-windows-static/debug/lib/getopt.lib

Now it's corrected in main branch.


Sorting on the result set is not happening with java code

Please read these guidelines before opening an issue.

Bug Description
We are using one index query to return a result, we are using "sort" key word to do sorting based on a timestamp field, and set limit to 1, this means we want to get the latest one record, but we found that the query is working fine in browser dashboard, but not working in java code with api. We have created related index in cloudant already, we are not sure the reason behind this.
1. Steps to reproduce and the simplest code sample possible to demonstrate the issue
Below is the sample code we are using:
String query = {"selector":{"requestMethod":"xxx","profileID":"xxx"},"sort":[ {"createdTS:string": "desc"} ],"limit":1}
List auditLogList = auditlogdb.findByIndex(query, AuditLogBean.class);
2. What you expected to happen
we expected the returned list has only one record, and the record is the latest one record
3. What actually happened
it is returning all record in a list, but no sorting happened on the records
Environment details

Version(s) that are affected by this issue.

2.9.0

Java version including vendor and platform

1.8 IBM JDK

Using the optional okhttp dependency.

Yes

I would recommend POSTing to the /_find endpoint with curl or similar using the contents of query. If the result is not what you are expecting, then your query is wrong.
We recommend the use of QueryBuilder (sample here which makes life easier since you don't have to worry about string escaping etc. There are options for sort, limit, etc.

Closing as stale, please reopen if you require further assistance.


Make  optional in

Submitted on Thursday, September 26, 2013 - 8:31am
Submitted values are:
Name:  Sibille - de Grimoüard Claire
Affiliation: "EAD" and "Authority data" expert groups of the French
Association of Standardization (AFNOR)
Does this represent an official comment from your affiliated group? Yes
Comment:
The <quantity> element should be made optional within <physdescstructured>
This change will create significant problems for us, as we frequently use the
equivalent of <unittype> with no related <quantity> given. For detailed
descriptions, we need to do a structured description, but we don’t have
necessarily enough time to count all the items. It is not satisfying to give
an imprecise text value like "multiple". And it is not logical to leave empty
a required element.
Example:
<physdescstructured physdescstructuredtype="materialtype" coverage="part"> <quantity>un certain nombre (!!!)</quantity> <unittype>photographies</unittype> </physdescstructured>
Moreover, the most important element for physical description in Beta is
<quantity> and not <physfacet> whereas in French libraries, <physfacet> is
very used to encode information about materials, techniques, scripts,
codocology, rule lines, marginalia, illustrations, decorations, seals,
stamps, bindings and containers.
Example:
<physdesc> <extent>24 p. et 4 p. de musique</extent> <dimensions unit="mm">245 × 160 mm</dimensions> <physfacet type="reliure">Reliure de soie moirée bleue aux initiales de   Pierre Lebrun</physfacet> <physfacet type="conditionnement">Dans un emboîtage de maroquin   lie-de-vin</physfacet> </physdesc>
The results of this submission may be viewed at:
http://www2.archivists.org/node/17190/submission/14514

Added examples. Closing


@OnTextChanged callback AFTER_TEXT_CHANGED execute twice

When I am using the following code and type a single character the log prints twice.
@OnTextChanged(value=R.id.EditText,callback = AFTER_TEXT_CHANGED)
    public void afterEditTextInput(Editable editable){
        Log.e("After Text", "demo log");
    }

We just pass the Android callback to your method...


set crowding penalty as a parameter

Summary:

What: Nextstrain recently did 2 changes related to crowding penalty:

There was a fix in the way it was applied. In below, blue samples are focal samples and we tried to pull green contextual samples based on their proximity to blue samples with varying ways to apply crowding penalty. In the following 3 settings, the green samples tend to land on the left-er side of the tree (aka more closely related to the blue samples).
Here is before the change (default crowding penalty 0.01):

Here is after the change (default 0.1):

Here is after the change and set it to 0:

So with the change, with default parameters, we are pulling samples that are more closely related to the focal samples while still reducing fully identical/stacking samples. Would be great to have this change in our workflow.

The crowding penalty was surfaced as a parameter in the main workflow here.

This PR will:

Consolidate the places we download ncov to only the Docker image
Pin the version in the Docker image to the current newest, and in the future we'll do a monthly update of the pin
Add crowding penalty as a parameter in our builds.yaml
Secretly tag along: add nano to tree docker

Demos:
Notes:
Checklist:

[ ] I merged latest <base branch>
[ ] I manually verified the change
[ ] I added labels to my PR
[ ] I tested in multiple browsers
[ ] I added relevant unit tests
[ ] I have notified others of changes they need to make locally (migrations, jobs, package updates, etc)

@jgadling please help! After cloning ncov in the Docker image, we also need to replace this line with         crowding = config["crowding"]["crowding_penalty"], (sorry indentation was eaten XD) Thanks!!

@jgadling please help! After cloning ncov in the Docker image, we also need to replace this line with  crowding = config["crowding"]["crowding_penalty"], (sorry indentation was eaten XD) I'll delete the current patch file if not recycled. Thanks!!

I added a commit to update this -- try it out?

I'll start from here on Monday :D


Join Wahoo?

We are still relatively new in town, and just getting started, but we are more stable than the other options, namely OMF. I think some of these functions could be turned into a handful of Wahoo plugins or maybe some could be even make it to core. We are always accepting feature request and feedback. I would love if you could give Wahoo a look.
Regards :smile:

Thanks for reaching me :smile:
I tried Wahoo for several minutes and it looks promising. I'll take a deeper look at it and try converting some functions here into Wahoo plugins.

@bucaran I've made a theme here: https://github.com/dorentus/theme-clownfish
Is there any chance it goes in to the wahoo theme gallery?

@dorentus Hi. It's been a while and a lot of things happened. Wahoo is still lurking around, but I am working now on a new system Fisherman, please check it out.
I'll fork your theme and send you a PR so you know how to make it work in Fisherman.


IFCWorker.js doesn't work for larger files on firefox

Trying to load larger files (more than 268mb) causes error:
UnknownError: IDBObjectStore.put: The serialized value is too large (size=374079546 bytes, max=267386880 bytes).
Is there some default for firefox capping it to 267386880 bytes and can we do anything about it? For other browsers it seems to work fine.

Yep, webworkers are not compatible with larger files unfortunately. We are working on a new loading system that is super fast (200mb ifc in 2 seconds), so the webworker approach won't be necessary anymore. For now, disabling webworkers is the only solution for that.


Module name parser incorrectly splitting on package names with '.'

Problem: npx force-resolution fails to execute as it splits the package name into a nested json node
To reproduce:

npm install {package name with . in the name} (e.g. twin.macro)
Add preinstall script: npx force-resolution
run npm i

Console output:
> npx force-resolutions

npx: installed 1 in 0.709s
Applying forced resolutions
resolve => 1.21.0
Finished applying forced resolutions
⸨ ░░░░░░░░░░░░░░░░░⸩ ⠋ loadIdealTree:loadAllDepsIntoIdealTree: sill install loadIdealTree
          NPM is trying to retrieve package "twin" with version "undefined"!
          Package location: undefined
          Package parse error: undefined
npm ERR! Cannot read property 'match' of undefined

A new version is been released recently check the readme to join the discord server if issues are still happening
Here is the new version: https://www.npmjs.com/package/force-resolutions?activeTab=readme


Function name

<gelman_plot> might be better named as <gelman_rubin_plot>, <psrf_plot>, <gr_plot>. Though I am usually first in last name, only keeping first author is not a good practice. I believe the standard name is Gelman Rubin diagnostics anyway.

renamed to psrf_plot
Thanks,
J.


Patch 1

Description
Adding myself as a developer to my zap-pipeline plugin. (Sorry about the last PR)
GitHub repository: https://github.com/jenkinsci/zap-pipeline-plugin
HOSTING issue: https://issues.jenkins-ci.org/projects/HOSTING/issues/HOSTING-646
Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[x] Add link to resolved HOSTING issue in description above

For a new permissions file only

[x] Make sure the file is created in permissions/ directory
[x] artifactId (pom.xml) is used for name (permissions YAML file).
[x] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[x] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Build failed; the context from the latest run is:
Expand to view
	at groovy.json.JsonBuilder.toString(JsonBuilder.java:347)
	at groovy.json.JsonBuilder.toPrettyString(JsonBuilder.java:361)
	at groovy.json.JsonBuilder$toPrettyString$0.call(Unknown Source)
	at io.jenkins.infra.repository_permissions_updater.ArtifactoryPermissionsUpdater$_generateApiPayloads_closure1.doCall(ArtifactoryPermissionsUpdater.groovy:193)
	at sun.reflect.GeneratedMethodAccessor8.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
	at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
	at org.codehaus.groovy.runtime.metaclass.ClosureMetaClass.invokeMethod(ClosureMetaClass.java:294)
	at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1024)
	at groovy.lang.Closure.call(Closure.java:414)
	at groovy.lang.Closure.call(Closure.java:430)
	at org.codehaus.groovy.runtime.ResourceGroovyMethods.eachFile(ResourceGroovyMethods.java:1070)
	at org.codehaus.groovy.runtime.ResourceGroovyMethods.eachFile(ResourceGroovyMethods.java:1088)
	at org.codehaus.groovy.runtime.dgm$936.invoke(Unknown Source)
	at org.codehaus.groovy.runtime.callsite.PojoMetaMethodSite$PojoMetaMethodSiteNoUnwrapNoCoerce.invoke(PojoMetaMethodSite.java:274)
	at org.codehaus.groovy.runtime.callsite.PojoMetaMethodSite.call(PojoMetaMethodSite.java:56)
	at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCall(CallSiteArray.java:48)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:113)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:125)
	at io.jenkins.infra.repository_permissions_updater.ArtifactoryPermissionsUpdater.generateApiPayloads(ArtifactoryPermissionsUpdater.groovy:134)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
	at org.codehaus.groovy.runtime.callsite.StaticMetaMethodSite$StaticMetaMethodSiteNoUnwrapNoCoerce.invoke(StaticMetaMethodSite.java:151)
	at org.codehaus.groovy.runtime.callsite.StaticMetaMethodSite.callStatic(StaticMetaMethodSite.java:102)
	at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCallStatic(CallSiteArray.java:56)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:194)
	at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:214)
	at io.jenkins.infra.repository_permissions_updater.ArtifactoryPermissionsUpdater.main(ArtifactoryPermissionsUpdater.groovy:317)
[Pipeline] stage (Archive)
Using the ‘stage’ step without a block argument is deprecated
Entering stage Archive
Proceeding
[Pipeline] archiveArtifacts
Archiving artifacts
[Pipeline] archiveArtifacts
Archiving artifacts
[Pipeline] }
[Pipeline] // node
[Pipeline] End of Pipeline

GitHub has been notified of this commit’s build result

ERROR: script returned exit code 1
Finished: FAILURE

Powered by the Comment Logger

Could you please merge this as I would like to release, thanks

Is there anything you would like me to change so this can be merged?

Nope this looked good AFAICT. I marked it approved so that another pair of eyes could double check and merge, but other reviewers are busy I think. Let's move forward.

Nope this looked good AFAICT. I marked it approved so that another pair of eyes could double check and merge, but other reviewers are busy I think. Let's move forward.


[$125] Web - Profile icon and wrench at the bottom have the same tooltips

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.36-0
Reproducible in staging?: y
Reproducible in production?: new feature
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Login in to New Expensify
Hover over Profile icon
Hover over wrench icon at the bottom
Note that the tooltips are the same

Expected Result:
The tooltips should differ, and describe the pages to which they lead.
Actual Result:
For both icons the tooltip is: "Settings". Even though one leads to Account Settings, and the other to Workspaces Settings.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/1ffaa1de-6381-4d44-a075-f35274c46da8
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011391fcb0d4fa84f7
Upwork Job ID:<PHONE_NUMBER>165429760
Last Price Increase: 2024-02-02

We think that this bug might be related to #wave8-collect-admins
CC @zanyrenney

Yes, it comes from the ideal nav PR. I don't think this is a blocker. Demoting and making it external so a contributor can work on it.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Profile icon and wrench at the bottom have the same tooltips.
What is the root cause of that problem?
We used the common.settings translation for the tooltipText prop in the PressableAvatarWithIndicator component.
https://github.com/Expensify/App/blob/3ab4e6e12d9ff38e69493321f0d31f032481b373/src/pages/home/sidebar/PressableAvatarWithIndicator.js#L59-L61
It is also used in the BottomTabBar component, thus will display identical text in the tooltip.
https://github.com/Expensify/App/blob/3ab4e6e12d9ff38e69493321f0d31f032481b373/src/libs/Navigation/AppNavigator/createCustomBottomTabNavigator/BottomTabBar.tsx#L66-L67
What changes do you think we should make in order to solve the problem?

Since the other components depend on the PressableAvatarWithIndicator component, the appropriate replacement common.settings would be  initialSettingsPage.account.
https://github.com/Expensify/App/blob/3ab4e6e12d9ff38e69493321f0d31f032481b373/src/languages/en.ts#L821-L822
tooltipText={translate('initialSettingsPage.account')}

What alternative solutions did you explore? (Optional)
If it is more appropriate to display Account Settings than Account, we can then use the initialSettingsPage.accountSettings translation.

@alitoshmatov this seems pretty simple. Can we get one of these moving?

I think @Amarparab2024 has the right solution, since we should replace that tooltip with Profile. Assigning to the job.

Looks like proposal is already selected without C+. I am assuming PR review by C+ is still required. Unassign me if review is not needed

Unfortunately not, and I think we should have changed the bounty for that issue too

@Amarparab2024 can you share your Upwork profile link please?

@alitoshmatov I need you to accept your offer: https://www.upwork.com/nx/wm/offer/28142751
@Amarparab2024 I need you to accept your offer: https://www.upwork.com/nx/wm/offer/100905317

Payment Summary:

@alitoshmatov paid $125 here - C+
@Amarparab2024 paid $125 here - Contributor

We do not need a Test for this.


Angular-CLI-Based-Template: Help link doesn't work

The Startup file in new template has below comment
         // To learn more about options for serving an Angular SPA from ASP.NET Core,
         // see https://go.microsoft.com/fwlink/?linkid=864501

But when we open this link it takes us to https://pastebin.com/raw/sZaduK8a which displays page showing below error:
This site can’t be reached

pastebin.com refused to connect.

https://docs.microsoft.com/en-us/aspnet/core/spa/index#installation
https://docs.microsoft.com/en-us/aspnet/core/spa/angular?tabs=visual-studio

Thanks - the link now redirects correctly.


LLAMA with OpenAI key?

The docs say it uses the opensource model and at the same time open_ai key is one of parameters... what model it is really using?
Also Gmail is not allowing user/password access... how did  you make it work?!
Very useful idea anyways...

Hello there,
Its the openai_key is for llama from https://deepinfra.com/, For Gmail you need to generate an app token as shown here . If you run it locally  the OPENAI_API_BASE will be localhost:someport  and the key can be empty.


Add codecov GHA

This PR adds codecov, via a GHA, back into the repo.

Do we want the to be mandatory checks?

Do we want the to be mandatory checks?

Do we want the to be mandatory status checks?

Meaning, with Bors?

Do we want the to be mandatory status checks?

Meaning, with Bors?

bors r+

Closes #263


Fall back to standard React handling of undefined elements

Follow up to the conversation on #591, specifically here
This removes our custom handling of undefined elements, allowing us to fall through to using whatever factory function is in scope to attempt (and fail) to create them, resulting in an error that is almost in line with what you'd expect from React:

(note the ForwardRef, which should actually be App)
From my testing this appears to give consistently the same error, but maybe @seanparsons you were trying something that I've missed?
Happy to discuss this approach further because I'm not attached to it, but it does feel closer to what we should be doing than the current custom logic.

I haven't tried it but this sounds like the good approach to me


[SPARK-20919][SS] Simplificaiton of CachedKafkaConsumer using guava cache.

What changes were proposed in this pull request?
On the lines of SPARK-19968, guava cache can be used to simplify the code in CachedKafkaConsumer as well. With an additional feature of automatic cleanup of a consumer unused for a configurable time.
How was this patch tested?
Existing tests.

Test build #77522 has started for PR 18143 at commit 4bd48aa.

Hi, @brkyvz as we discussed on the PR for CachedKafkaProducer that Guava cache can be used for consumer as well. May be you could take a look at this as well ?

Test build #77522 has finished for PR 18143 at commit 4bd48aa.

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

Merged build finished. Test PASSed.

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

Thanks @srowen for taking a look. I have tried to address your comments.

Test build #77532 has started for PR 18143 at commit 1e0ac23.

Test build #77532 has finished for PR 18143 at commit 1e0ac23.

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

Merged build finished. Test PASSed.

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

Generally looks ok to me, thanks.
2 questions -
Did you do any testing on workloads to see if performance stayed the same?
Is there a reason not to do the same thing to streaming/kafka010/CachedKafkaConsumer.scala for consistency?

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

Merged build finished. Test PASSed.

Test build #77640 has started for PR 18143 at commit dad946f.

Test build #77640 has finished for PR 18143 at commit dad946f.

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

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

Merged build finished. Test PASSed.

Sorry, noticed a couple more minor configuration related things.  Otherwise LGTM.

Thanks @ScrapCodes for this PR.  Feel free to push back if you think it's over-abstracting.

Thanks @koeninger for helping me.
Thanks @brkyvz, please take a look again and see if what I have done is along the lines the change you wanted.

Test build #77777 has started for PR 18143 at commit 288ab39.

Test build #77777 has finished for PR 18143 at commit 288ab39.

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

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

Merged build finished. Test PASSed.

@brkyvz I think consumers and producers have different cache strategy and seem sharing the same interface is weird. We can share the same producer in multiple tasks at the same time, but that's not allowed for consumers. The interface I'm expecting for consumers is:
// Create or fetch a consumer from cache if any. The cache doesn't store this consumer.
def borrowConsumer(groupId: ...): CachedKafkaConsumer = {...}

// Return a consumer to the cache.
def returnConsumer(consumer: CachedKafkaConsumer): Unit = {...}

@brkyvz and @zsxwing, do you think this object pool should be bounded ? Or they can become weak reference values, incase the object pool is unbounded ?

@ScrapCodes I think it should be bounded by spark.sql.kafkaConsumerCache.capacity.

Test build #79707 has started for PR 18143 at commit de733f1.

Test build #79707 has finished for PR 18143 at commit de733f1.

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

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

Merged build finished. Test FAILed.

Test build #79708 has started for PR 18143 at commit a0f1aa3.

Test build #79708 has finished for PR 18143 at commit a0f1aa3.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #79710 has started for PR 18143 at commit 5c105e5.

Test build #79710 has finished for PR 18143 at commit 5c105e5.

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

Merged build finished. Test FAILed.

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

Test build #79711 has started for PR 18143 at commit ea30495.

Test build #79711 has finished for PR 18143 at commit ea30495.

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

Merged build finished. Test PASSed.

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

Test build #79907 has started for PR 18143 at commit db86d97.

Test build #79908 has started for PR 18143 at commit 3c6f7c6.

@zsxwing, can you please take a look.

Test build #79907 has finished for PR 18143 at commit db86d97.

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

Merged build finished. Test PASSed.

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

Test build #79908 has finished for PR 18143 at commit 3c6f7c6.

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

Merged build finished. Test PASSed.

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

Test build #79909 has started for PR 18143 at commit 1c122ca.

Test build #79909 has finished for PR 18143 at commit 1c122ca.

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

Merged build finished. Test PASSed.

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

I am currently, trying to run some performance tests and see how this change impacts performance in any case. Meanwhile, if I could get an idea if things are moving in the right direction that would be kind.

@brkyvz and @zsxwing No comments on whether this will be useful or not, so far. Should I consider closing it ?

@ScrapCodes sorry for the delay. I think @tdas has fixed the issue. Please close the PR.


Add Response class

Coverage increased (+0.08%) to 99.383% when pulling e0a35a18bcd1ce191f830d33ec96618f4cfdb70e on response into 1a120dd94ac03eae5190fb309613af3341ec411e on master.

Coverage remained the same at 100.0% when pulling d56bbcfeaa9d1f11099d0b001f7560facbf79ee7 on response into dc9987286e6e6e2c477a0b137f247287a36b40a8 on master.

Coverage remained the same at 100.0% when pulling d56bbcfeaa9d1f11099d0b001f7560facbf79ee7 on response into dc9987286e6e6e2c477a0b137f247287a36b40a8 on master.


Submodule Linking with Halo

Signed-off-by: Mukul Malhotra<EMAIL_ADDRESS>Describe your changes in brief
Submodule Linking with Halo
Changes

[ ] Why is this change required? What problem does it solve?
[ ] If proposing a new change then please raise an issue first

How Has This Been Tested? (Optional)

[ ] Please describe in detail how you tested your changes.
[ ] Include details of your testing environment, and the tests you ran to
[ ] How your change affects other areas of the code, etc.

Screenshots (if appropriate)
Checklist

[x] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

PR#13 raised & merge in halo repo

@shailesh-vaidya, community build is good as tested as per,
[root@localhost cortx]# ls -ltr |grep cortx-halo
drwxr-xr-x.  5 root root   185 May 25 11:51 cortx-halo

[root@localhost cortx]# docker run --rm -v /mnt/cortx:/cortx-workspace ghcr.io/seagate/cortx-build:rockylinux-8.4 make checkout BRANCH=main

<output>
Switched to branch 'main'
Your branch is up to date with 'origin/main'.
/cortx-workspace /
Cloning into 'cortx-rgw'...
Updating files: 100% (11108/11108), done.
/

[root@localhost cortx]# cd /mnt/cortx/ && for component in cortx-motr cortx-hare cortx-rgw-integration cortx-manager cortx-utils cortx-ha cortx-rgw; do echo -e "\n==[ Checking Git Branch for $component ]=="; pushd $component; git status; popd ; done && cd -

<output>
==[ Checking Git Branch for cortx-rgw ]==
/mnt/cortx/cortx-rgw /mnt/cortx
# On branch main
nothing to commit, working directory clean
/mnt/cortx
/mnt/cortx/cortx-rgw


Decoding a class?

I noticed all the examples here use structs. Is the library able to decode classes along with separating the decoding logic from the model itself?

Answering my own question, this code works just fine:
class BaseObject {
    let id: Int

    required init(id: Int) {
        self.id = id
    }
}

extension BaseObject: Decodable {
    static func decode(_ json: Any) throws -> Self {
        return try self.init(id: json => "id")
    }
}

The only problem I see it's unable to define an extention for a subclass, as it's already conforming to the protocol:
class Object: BaseObject {
    var location: String

    required init(id: Int, location: String) {
        self.location = location
        super.init(id: id)
    }

    required init(id: Int) {
        fatalError("init(id:) has not been implemented")
    }
}

Is there a workaround?

You don't have to conform to Decodable, especially for classes which are better used for hierarchies, Decodable was designed around the Struct-as-data-container strategy.
Instead, have a constructor on your class that takes a JSON Any and uses Decodable to extract the values of the fields it wants.


Visual Bug(possible AC/LGC compatibility issue)

after buying walkie expansion and taking a walkie into your hand at first there is a small visual bug where walklie slot overlaps with your first item slot and is on top of it. then it goes back to the 5th slot as it should but then the walkie completely disappears from your inventory functionally till you teleport yourself and drop it. But while you are running or holding other things you can see your chaarcter model still holding the walkie. might be a compatibility issue with the mod that makes you use walkie without holding it/AC or late game company or something else idk

This was a general bug, and not related to compatibility with another mod. I believe this should be fixed now, but let me know though, thanks!


Remove extra package deps on mdx

This reverts #141. Package dependencies cannot be resolved to opam packages by dune as of today. This so-called fix introduces a bug and the package dependency on mdx should only be added when vendoring mdx itself, eg in a duniverse.
I'll re-enable that in the --duniverse-mode!

This what you would get if using current master without vendoring mdx:
$ dune runtest
File "infer-timestamp/dune.inc", line 1, characters 0-262:
1 | (alias
2 |  (name   runtest)
3 |  (deps   (:x README.md)
4 |          (package mdx)
5 |          (:y0 example.ml))
6 |  (action (progn
7 |            (run ocaml-mdx test --direction=infer-timestamp %{x})
8 |            (diff? %{y0} %{y0}.corrected)
9 |            (diff? %{x} %{x}.corrected))))
Error: No rule found for alias .mdx-files


commitable_code_suggestions property is ignored if git provider doesn't support gfm_markdown

The commitable_code_suggestions feature property is ignored for non github providers.
Looks like the gfm_markdown check in this if statement causes the statement to always be false thereby causing the code to enter the else case which publishes committable code suggestions.
There thus is no way to disable committable code suggestions which is problematic due to #1076

I don't understand this issue.
Bitbucket is the only git provider that does not provide basic support for graphical elements like collapsible (gfm markdown)
All other git provider, a.k.a github,gitlab, azure .... do support that.
Hence, we cannot present results in a comment-like manner in Bitbucket since they will take a ridiculous amount of space.
So we must publish them as commitable suggestions, one-by-one.
I don't like this mode. Suggestions in a single comment are dramatically better in my opinion. But we cant do that in Bitbucket, since its an inferior provider, not maintained or improved in the last ~10years.
If you are paying money for Bitbucket, I suggest you contact them, and demand they will add support for gfm. its extremely basic. We are not the first to want this.
Without minimal graphical support, we have no alternative but to publish only as commitable_code_suggestions
collapsible


Retitle README

The old title is carried over from the old repo. Renaming
the title to match the new repo name.
Signed-off-by: JBD<EMAIL_ADDRESS>

LGTM


When importing more than one .json, missing file extensions lead to silent error

Problem
I've been trying to import numerous JSON-files, which didn't had a proper filename extension. (They were named 'json-1' instead of 'json-1.json').
The import-process stopped on the page, where I had to select the root node: OpenRefine never stopped updating the preview. It seems, as if the manually assigned file extension was only used on the first json-file, but not on the remaining files to import. As the auto-detection failed there aswell, he stopped while updating the preview.
Workaround
Giving all the files a proper file extension has helped. But as this error is silent, it would be nice to fix it.
If necessary, I can provide you with the files I used for testing purposes.
Btw: Thank you for this great tool!

Thank for reporting this issue. Having the files will help to reproduce the issue.
@jackyq2015 looks like similar to #708

Sure. I can't attach it to this comment, but you can download the files from http://alexanderblohm.de/jnkdnfkjndfgn3iuu4u3sdf/json.zip. Will be available for at least a month or so.

thanks for the file. Did you see some error message at back end？
On Mon, Aug 10, 2015, 8:04 AM a-blohm<EMAIL_ADDRESS>wrote:

Sure. I can't attach it to this comment, but you can download the files
from http://alexanderblohm.de/jnkdnfkjndfgn3iuu4u3sdf/json.zip. Will be
available for at least a month or so.
—
Reply to this email directly or view it on GitHub
https://github.com/OpenRefine/OpenRefine/issues/1052#issuecomment-129423354
.

As far as I remember, there was some kind of Exception after some time.

A silent error sounds like a bug, not an enhancement request.


fix(map): Fix map marker click events.

Unquote the onClick function name to fix map marker click events.
Move useless scope.$apply.
Fixes #60.

This is deployed to production and seems to be working. There are some improvements that could be done to the maps in general, but those will need to be handled in a separate issue/PR.

:+1:


I thot that is custom control and I didn't found that in <AdnousUi.Controls:ToggleSwitch or AdonisUI:ToggleSwitch or even your documentation.

The ToggleSwitch is available as a style, see https://benruehl.github.io/adonis-ui/docs/guides/styles-and-templates/
You would do something like this:
<Button Style="{DynamicResource {x:Static adonisUi:Styles.ToggleSwitch }}"/>

@alexhelms Thanks for answering!
You're right, it's a style. Just a little addition: It targets toggle buttons so you can use it on ToggleButton or CheckBox for example.

Yep, that's it, thanks my dudes.
<ToggleButton Style="{DynamicResource {x:Static adonisUi:Styles.ToggleSwitch }}" /> work fine.


Some NTFS partition under GPT table not detected

At my macOS 10.14, when I run the "diskutil list"
I can see the two types of NTFS partition...
/dev/disk0 (internal, physical):
#:                       TYPE NAME                    SIZE       IDENTIFIER
0:      GUID_partition_scheme                        *500.1 GB   disk0
1:                        EFI EFI                     209.7 MB   disk0s1
2:                 Apple_APFS Container disk2         128.0 GB   disk0s2
3:       Microsoft Basic Data Win&Mac                 171.6 GB   disk0s3    >>    exFAT
4:       Microsoft Basic Data Windows                 199.9 GB   disk0s4    >>    under GPT table, NTFS and exFAT partition shows "Microsoft Basic Data"
/dev/disk1 (internal, physical):
#:                       TYPE NAME                    SIZE       IDENTIFIER
0:     FDisk_partition_scheme                        *320.1 GB   disk1
1:               Windows_NTFS Windows 10_old          142.4 GB   disk1s1
2:               Windows_NTFS Windows 7               42.9 GB    disk1s3
3:               Windows_NTFS App&Data2               134.7 GB   disk1s5    >>    Under MBR partition table, NTFS partition shows "Windows_NTFS"(Can detect)
/dev/disk2 (synthesized):
#:                       TYPE NAME                    SIZE       IDENTIFIER
0:      APFS Container Scheme -                      +128.0 GB   disk2
Physical Store disk0s2
1:                APFS Volume MacOS                   38.3 GB    disk2s1
2:                APFS Volume Preboot                 48.2 MB    disk2s2
3:                APFS Volume Recovery                507.4 MB   disk2s3
4:                APFS Volume VM                      5.4 GB     disk2s4
/dev/disk3 (external, physical):
#:                       TYPE NAME                    SIZE       IDENTIFIER
0:     FDisk_partition_scheme                        *1.0 TB     disk3
1:               Windows_NTFS BOB_DATA                1.0 TB     disk3s1
Maybe this situation, so when I use ezntfs all, can't remount some partition under GPT tables.
(Sorry for my bad English... Chinese is my main language, so I still work hard to learn English...)

Some different when I run these two command you mention...

diskutil info disk0s3
Device Identifier:         disk0s3
Device Node:               /dev/disk0s3
Whole:                     No
Part of Whole:             disk0
Volume Name:               Win&Mac
Mounted:                   Yes
Mount Point:               /Volumes/Win&Mac
Partition Type:            Microsoft Basic Data
File System Personality:   ExFAT
Type (Bundle):             exfat
Name (User Visible):       ExFAT
OS Can Be Installed:       No
Media Type:                Generic
Protocol:                  SATA
SMART Status:              Verified
Volume UUID:               275BA069-029B-397D-810F-E8FCEE3E4876
Disk / Partition UUID:     2D0566A2-3D64-47D7-AC5F-6AA979157EED
Partition Offset:<PHONE_NUMBER>24 Bytes (250673152 512-Byte-Device-Blocks)
Disk Size:                 171.6 GB<PHONE_NUMBER>28 Bytes) (exactly 335212544 512-Byte-Units)
Device Block Size:         512 Bytes
Volume Total Space:        171.6 GB<PHONE_NUMBER>72 Bytes) (exactly 335200256 512-Byte-Units)
Volume Used Space:         19.4 GB<PHONE_NUMBER>4 Bytes) (exactly 37892352 512-Byte-Units) (11.3%)
Volume Free Space:         152.2 GB<PHONE_NUMBER>48 Bytes) (exactly 297307904 512-Byte-Units) (88.7%)
Allocation Block Size:     131072 Bytes
Read-Only Media:           No
Read-Only Volume:          No
Device Location:           Internal
Removable Media:           Fixed
Solid State:               No
Hardware AES Support:      No

diskutil info disk0s4
Device Identifier:         disk0s4
Device Node:               /dev/disk0s4
Whole:                     No
Part of Whole:             disk0
Volume Name:               Windows
Mounted:                   Yes
Mount Point:               /Volumes/Windows
Partition Type:            Microsoft Basic Data
File System Personality:   NTFS
Type (Bundle):             ntfs
Name (User Visible):       Windows NT File System (NTFS)
OS Can Be Installed:       No
Media Type:                Generic
Protocol:                  SATA
SMART Status:              Verified
Volume UUID:               A43C1CEA-5659-4C65-A97D-D03187568D23
Disk / Partition UUID:     FDD88969-C2CF-483E-B36D-826ADD522C1E
Partition Offset:<PHONE_NUMBER>56 Bytes (586149888 512-Byte-Device-Blocks)
Disk Size:                 199.9 GB<PHONE_NUMBER>56 Bytes) (exactly 390361088 512-Byte-Units)
Device Block Size:         512 Bytes
Volume Total Space:        199.9 GB<PHONE_NUMBER>60 Bytes) (exactly 390361080 512-Byte-Units)
Volume Used Space:         30.8 GB<PHONE_NUMBER>4 Bytes) (exactly 60155072 512-Byte-Units) (15.4%)
Volume Free Space:         169.1 GB<PHONE_NUMBER>96 Bytes) (exactly 330206008 512-Byte-Units) (84.6%)
Allocation Block Size:     4096 Bytes
Read-Only Media:           No
Read-Only Volume:          Yes
Device Location:           Internal
Removable Media:           Fixed
Solid State:               No
Hardware AES Support:      No

Ok, It works!!!
Thank you very much.


Create an Empty SAPUI5 Project

Tutorial: https://developers.sap.com/tutorials/sapui5-101-create-project.html
Write here what you need help with ...
Step 4: Create a new run configuration
I get an error "There are no runnable resources."

Hi,
sorry for the late response. We updated the tutorial recently and now the run configuration is generated for you out of the box. I assume this resolves the issue. Can you please confirm?

I'll close this issue now but feel free to open it again if something remains unclear.


Sync Settings - unable to scroll

🐛 Describe the bug
Unable to scroll on Settings page:

🌴 Visual Studio Code Version : <IP_ADDRESS>
🌴 Code Settings Sync Version : 2.4.3
🌴 Standard or Insiders : ??
🌴 Portable or Installed : Installed in User scope
🌴 OSS or Official Build : Official
🌴 Operating System : Win 10 Enterprise
📰 To Reproduce
Steps to reproduce the behavior:

CTRL + SHIFT + P
Sync: Advanced Options
Sync: Open Settings
No Scroll bar.

💪 Expected behavior
I expect the settings page to scroll so I can get to all the settings.

I don't have a right column at all

Wanted to add that I'm having this issue too, but found that I can get to the lower fields by focusing the first field and TAB-switching between the fields. Similarly, once at the bottom, you must use TAB again to cycle to the top of the page.


Xcode 9.1 - Dead store errors

When analyzing my project in Xcode 9.1 using leveldb v1.20, I'm seeing the below issues:

I suspect something is wrong with the analysis.  E.g., the following code fragment is inconsistent with the offset_in_block complaint:
const size_t offset_in_block = initial_offset_ % kBlockSize;
uint64_t block_start_location = initial_offset_ - offset_in_block;
// Don't search a block if we'd be in the trailer
if (offset_in_block > kBlockSize - 6) {
block_start_location += kBlockSize;
}


hostmetrics preset filesystem config

So the current situation is:

I added filesystems in PR -> https://github.com/open-telemetry/opentelemetry-helm-charts/commit/d065b6b40ce1fde7d0245efadefe74066c572d5d
This tries to scrape metrics for all the fielsystems
This typically causes errors in the Collector.

Examples:
2022-12-02T14:22:24.979Z        error   scraperhelper/scrapercontroller.go:214  Error scraping metrics  {"kind": "receiver", "name": "hostmetrics", "pipeline": "metrics", "error": "failed to read usage at /boot: no such file or directory; failed to read usage at /var/lib/kubelet/pods/75ea3ea3-f784-423d-9635-25a9ef668858/volumes/kubernetes.io~local-volume-mongo: no such file or directory", "scraper": "filesystem"}

/var/lib/kubelet/plugins/kubernetes.io/csi/ebs.csi.aws.com/ebdcc357ad138b3ee228b233fcc7fe4fbe0393c49dc8dc420a248d8b1e41668e/globalmount: no such file or directory; failed to read usage at /var/lib/kubelet/pods/6b6567dc-6a31-4d07-b3ab-2e7dff54bfc1/volumes/kubernetes.io~csi/pvc-594d005e-632e-4b21-a099-c852a72e522a/mount: no such file or directory", "scraper": "filesystem"}

I know folks also have similiar errors scraping  /snap, /var/lib/kubelet/..
Good discussions:

https://github.com/open-telemetry/opentelemetry-helm-charts/pull/533#discussion_r1036149185
https://github.com/open-telemetry/opentelemetry-helm-charts/pull/496

What do we do?
Options:

Deprecate & revert filesystem scraper pr (No more out of space metrics/alerts :cry: )
FIgure out some default include list / exclude list that works?

@povilasv is the list proposed in https://github.com/open-telemetry/opentelemetry-helm-charts/pull/496 a good list?
exclude_mount_points:
  mount_points:
    - /snap/*
    - /boot/*
    - /var/lib/kubelet/pods/*
    - /var/lib/kubelet/plugins/*
  match_type: regexp

I agree that we should update the default config of the filesystem scraper to whatever makes the most sense for containers and don't throw any warnings.
Also we should adopt recently added feature https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/receiver/hostmetricsreceiver#collecting-host-metrics-from-inside-a-container-linux-only and remove the gopsutil env variables


AFX 1.5.4 - PlantUML diagrams wrong size.

Hi @rahmanusta,
A PlantUML diagram that I used successfully in an older version of AFX is rendered as a much larger image in AFX 1.5.4.
[plantuml, network-diagram, png]
----
@startdot
digraph G {
    node [shape=box]
	subgraph cluster_0 {
		node [style=filled];
		WIN_11 -> WIN_12 -> WIN_13;
		label = "WIN - Site 1";
		color = purple
	}

	subgraph cluster_1 {
		node [style=filled];
		WIN_21 -> WIN_22 -> WIN_23;
		label = "WIN - Site 2";
		color = purple
	}
	start -> WIN_11;
	start -> WIN_21;
	WIN_13 -> end;
	WIN_23 -> end;

    users [shape=Mdiamond]
    users -> start
    start [label="WIN_00", shape=invhouse]
	end [label="Peer-to-Peer\nReplication", shape=Msquare];
}
@enddot
----

appears as

when rendered using AFX 1.4.7, but appears much bigger when rendered using AFX 1.5.4 as shown below.

Do you know what could be causing this discrepancy?
Thanks,
Werner

It depends on image scale, zoom and it is dpi for plantuml. I had increased default dpi to get better quality in AsciidocFX.
So, You can give height to images or you can change dpi in Settings > Extension Settings in version 1.5.5

Settings > Extension Settings in version 1.5.5

Thanks. Workaround confirmed for 1.5.5 by setting DPI to 100.


Multiple ArgActionType attributes or multiple Types in the attribute

I would like to see support for the deferred action methods to be placed in multiple, separate classes which should promote better separation of concerns and cleaner code.
In other words doing this:
[ArgActionType(AActions)]
[ArgActionType(BActions)]
public class MainActions { ... }

public class AActions { ... }
public class BActions { ... }

Or allow for multiple types in a single attribute such as:
[ArgActionType(AActions, BActions)]
public class MainActions { ... }

public class AActions { ... }
public class BActions { ... }

Hi Adam
Could you provide some examples on how to use dependency injection in the overridden ArgActionResolver class, please?

Actually, I got swamped at work and didn't get a chance to try it out yet
(sigh). I will see if I can free up some of my time in the near future to
give it a go.
John
On Fri, Mar 27, 2015 at 3:26 AM, Roman<EMAIL_ADDRESS>wrote:

Hi Adam
Could you provide some examples on how to use dependency injection in the
overridden ArgActionResolver class, please?
—
Reply to this email directly or view it on GitHub
https://github.com/adamabdelhamed/PowerArgs/issues/57#issuecomment-86854453
.

Thanks John, I'm looking forward to hearing from you. :)


Update index.html

Sorry for all this mess with PR and commits. I am not very familiar with github control.

No worries ;)


Ruby 3.0 compatibility

Why was compatibility with Ruby 3.0 dropped in sidekiq-throttled 0.14.1 (74dbf05a5a962531c14e107a61d943daf337c8e2)? Are there plans to add it back?

Yeah we’ve been running sidekiq-throttled in production with Ruby 3 for a while and we can’t update this dependency now because of this change.

Yeah we’ve been running sidekiq-throttled in production with Ruby 3 for a while and we can’t update this dependency now because of this change.

@anark

This is blocking us from upgrading to Ruby 3.

Yes, me too.
I've created the patch and tested it on ybiquitous/sidekiq-throttled#1, but the test failed. I don't know the failure reason. 🤷🏼
Of course, I can open a pull request for this repo with the insufficient patch if it would help someone.

@anark Thanks! I've opened PR #102!

I'd love to give this gem a try but am using ruby 3. I'm also curious to know why ruby 3 was dropped on the last release.
I guess I'll fork it for now, until the PR is merged (hopefully soon), thanks!

Hey guys, any chance to get this Issue closed anytime soon? We want to update our project to ruby 3 and this is currently the only dependency that is not ready yet.

Unfortunately, the answer is no. 😢
You can see the CI failures on my fork PR ybiquitous/sidekiq-throttled#1.
(the CI on #102 is not approved...)

Our team is also blocked by this. I'm happy to help in any way to speed this up.

The fix was merged and shortly will do the release. Gonna merge in the last outstanding pr and will cut the release.


quick spelling fix

Saw a minor spelling typo, figured I'd do the quick fix.   Package looks cool, will keep in mind for future projects!

Thank you!