[BUG] Package name is added to URL

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Create new recipe either through CLI or website
The output includes a link with a hard-coded package name

Expected behavior
The placeholder should not be substituted
Outputs
If applicable, add the output to help explain your problem.
Example recipe:
{% set name = "requests" %}
{% set version = "2.25.1" %}

package:
  name: {{ name|lower }}
  version: {{ version }}

source:
  url: https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/requests-{{ version }}.tar.gz
  sha256: 27973dd4a904a4f13b263a19c866c13b92a39ed1c964655f025f3f8d3d75b804

build:
  number: 0
  noarch: python
  script: {{ PYTHON }} -m pip install . -vv

requirements:
  host:
    - pip
    - python !=3.0,!=3.1,!=3.2,!=3.3,!=3.4
  run:
    - certifi >=2017.4.17
    - chardet >=3.0.2,<5
    - idna >=2.5,<3
    - python !=3.0,!=3.1,!=3.2,!=3.3,!=3.4
    - urllib3 >=1.21.1,<1.27

test:
  imports:
    - requests
  commands:
    - pip check
  requires:
    - pip

about:
  home: https://requests.readthedocs.io
  summary: Python HTTP for Humans.
  dev_url: https://github.com/psf/requests
  license: Apache-2.0
  license_file: LICENSE

extra:
  recipe-maintainers:
    - ADD_YOUR_GITHUB_ID_HERE

Here, the URL is url: https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/requests-{{ version }}.tar.gz with a hard-coded requests in it.
The example meta.yaml has the URL like so:  url: https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/{{ name }}-{{ version }}.tar.gz
The only thing where I found the URL with the placeholder to fail is if the package name is upper case but the user entered the name in lower case.
Then, the package name will be added in lowercase to the beginning of the YAML but that is not accepted in the URL.
Therefore, if that is changed, grayskull would need to make sure that the name is not taken directly form the user input but has its casing verified first.
For example, the package Janome generates this output, when entering the name in lowercase:
{% set name = "janome" %}
{% set version = "0.4.1" %}

package:
  name: {{ name|lower }}
  version: {{ version }}

source:
  url: https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/Janome-{{ version }}.tar.gz
  sha256: 6c2c38d894014d57cb3151265c11146506ead3b3bc290898adc33711711612de

build:
  number: 0
  noarch: python
  script: {{ PYTHON }} -m pip install . -vv

requirements:
  host:
    - pip
    - python
  run:
    - python

test:
  imports:
    - janome
    - janome.sysdic
  commands:
    - pip check
  requires:
    - pip

about:
  home: https://mocobeta.github.io/janome/en/
  summary: Japanese morphological analysis engine.
  license: AAL
  license_file: PLEASE_ADD_LICENSE_FILE

extra:
  recipe-maintainers:
    - ADD_YOUR_GITHUB_ID_HERE

When then substituting the URL as described above, the tests fail, because the name is in lower case.

I don't the placeholder would be the best solution here. I know that is one of the parts I would like to change on grayskull
Because you can ask pypi to get the URL as well, I believe that would be better and just replace the version by the placeholder

I don't the placeholder would be the best solution here. I know that is one of the parts I would like to change on grayskull
Because you can ask pypi to get the URL as well, I believe that would be better and just replace the version by the placeholder

That sounds good and yeah, I also disagree with the placeholder, just wasn't sure if that is documented behaviour or not. :)

That sounds good and yeah, I also disagree with the placeholder, just wasn't sure if that is documented behaviour or not. :)


what's the meaning about listening the finished event of response?

In Line 167#application.js, there have the code:
line162:    handleRequest(ctx, fnMiddleware) {
line163:      const res = ctx.res;
line164:        res.statusCode = 404;
line165:        const onerror = err => ctx.onerror(err);
line166:        const handleResponse = () => respond(ctx);
line167:        onFinished(res, onerror);
line168:        return fnMiddleware(ctx).then(handleResponse).catch(onerror);
line169:     }

onFinished(res, onerror) can add a listener on the error/finished event of res.
And then, the listener below in Line 107#context.js:
  onerror(err) {
    if (null == err) return;
    //... other codes
  }

But in finished event,  the null == err always true. So the code is always returned.It's make me confused.
So would you have a time can described the meaning ? Thanks very much!

@relax-code-relax-life you can check below test case:
https://github.com/koajs/koa/blob/master/test/application/index.js#L14
  it('should handle socket errors', done => {
    const app = new Koa();

    app.use((ctx, next) => {
      // triggers ctx.socket.writable == false
      ctx.socket.emit('error', new Error('boom'));
    });

    app.on('error', err => {
      assert.equal(err.message, 'boom');
      done();
    });

    request(app.callback())
      .get('/')
      .end(() => {});
  });

@nfer Thanks!!!!!


Unintended behaviour/error-catching bug when using workflows with IBM backend

Prerequisites
Before raising this issue, I have already checked that I am:

[x] running the latest version
[x] made sure that this issue has not already been filed

Describe the bug
Using workflows, if the QUBO problem to be compiled contains more qubits than the device chosen, running q.optimise() will just pass, without returning an error message.
To Reproduce
Steps to reproduce the behavior:

Choose an IBM device (eg: ibm_nairobi with 6 qubits)
Create a QUBO problem requiring more qubits than the device (eg : a 7-qubit MaxCut problem)
Run OQ as usual, and finally run q.optimise()
Cell passes, but it should have failed.

Expected behavior
An error message should be thrown to inform the user that the problem and device are incompatible.

PR 108 fixes this issue


Fix bug caused by messing nextProps and this.props

Changes Unknown when pulling 8944b8d1872e4901c8c51e7cb5983b05f7b7bf49 on qingweibinary:qingwei/fix into ** on binary-com:master**.


Question on addLocationNodeWithConfirmedLocation function

Just wondering, is there any chance or a scenario where the location of the node is confirmed but yet the location of the node is nil?
public func addLocationNodeWithConfirmedLocation(locationNode: LocationNode) {
        if locationNode.location == nil || locationNode.locationConfirmed == false {
            return
        }

I'm not sure - have you tested this to see @hazzaldo ?  FYI - I'm going to close this issue due to it's age.


Pagination count for first item is wrong if data is an empty array

mantine-react-table version
v1.0.0-beta.3
react & react-dom versions
v18.2.0
Describe the bug and the steps to reproduce it
If the table data is entry and the empty state text, the pagination shows the wrong counts:
Currently it shows 1-0 of 0
It should be 0-0 of 0 as in Material React Table from my point of view
The wrong place as far as i can spot: https://github.com/KevinVandy/mantine-react-table/blob/main/packages/mantine-react-table/src/toolbar/MRT_TablePagination.tsx#L97
Probably one should check if the data is empty and then only add +1
Minimal, Reproducible Example - (Optional, but Recommended)
{${firstRowIndex + data.length===0 ? 0 : 1}-${lastRowIndex} ${         localization.of       } ${totalRowCount}}
Screenshots or Videos (Optional)
Do you intend to try to help solve this bug with your own PR?
None
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

fixed in beta 4


Consider tagging releases

I notice that this repo has no tags. That means there are many different commits that correspond to a given release in PyPI. I recently wanted to know what releases might contain the fix for #147, so I went to the merged commit containing the fix (5def6648383e475f217ce92d2ae3986b572f5a6e), but it gave me no indication of what version(s) might contain that fix. After figuring out where the version is stored, I can see that the version at the time that commit was added was 1.8.0, implying that >1.8.0 would include the fix. Still, it would be nice if there were a way to see which commit specifically was the one that was released to PyPI for any given release. Is there any reason not to tag commits with the release numbers?

Old enough for a bump @asweigart

Running git blame on src/pyperclip/init.py to find commits matching particular releases is rather tedious, please tag releases to make things easier for downstream consumers. We at Arch Linux (and perhaps others) have to dig up the commit manually for every release when we want to update our package.

Finding what's changed between releases also requires this tedium.


Integrate backend for sort by likes

After the event likes backend is implemented , the sort by likes needs to be integrated with backend as well . Such that if we choose highest then events with the highest likes appear. I would like to work on it once the event likes feature is implemented.

make PR and as soon as possible and in initial rendering the events should be displayed by date which event is organized closest it should be displayed and have not any other constraints initially

hey kindly assign me this , I'll deliver asap !

@prisha-sh already assigned bro don't spam

@shreyamaheshwari1 i think the issue is solved

add one thing initial event rendering sort by recent event added and check the like thing if its not working properly then make changes there

Okay can it be assigned an appropriate level as well?

any updates ?

Hii @TejasNasre busy with some exams actually that's why it's getting delayed.

no worries

any updates @shreyamaheshwari1 if you are free please update if not i will assign to other


There are two ways I can think to generate this dataset:

Use a similar approach to the increasing cutoff experiment, where we just have a bunch of samples where we've changed the lower limit of detection, and therefore changed which features show up as missing.
For each sample, change the median abundance of the entire sample, effectively changing the limit of detection.

I think based on the way @hunter-moseley described it, and our discussion about the responses to reviewers, we want to go with option 1.

Variance of detection has to be across samples.  But need several groups:
No variance and then different levels of variance, likely set by a CV
estimate.
On Fri, Sep 6, 2024 at 9:02 AM Robert M Flight @.***>
wrote:

There are two ways I can think to generate this dataset:

Use a similar approach to the increasing cutoff experiment, where
we just have a bunch of samples where we've changed the lower limit of
detection, and therefore changed which features show up as missing.
For each sample, change the median abundance of the entire sample,
effectively changing the limit of detection.

I think based on the way @hunter-moseley
https://github.com/hunter-moseley described it, and our discussion
about the responses to reviewers, we want to go with option 1.
—
Reply to this email directly, view it on GitHub
https://github.com/MoseleyBioinformaticsLab/manuscript.ICIKendallTau/issues/31#issuecomment-2334002586,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEP7BZJMUXOJ6JFURYYRWLZVGRV3AVCNFSM6AAAAABNYRIX4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUGAYDENJYGY
.
You are receiving this because you were mentioned.Message ID:
<MoseleyBioinformaticsLab/manuscript.ICIKendallTau/issues/31/2334002586@
github.com>

--
Hunter Moseley, Ph.D. -- Univ. of Kentucky
Professor, Dept. of Molec. & Cell. Biochemistry / Markey Cancer Center
/ Institute for Biomedical Informatics / UK Superfund Research Center
Not just a scientist, but a fencer as well.
My foil is sharp, but my mind sharper still.
Email: @.*** (work)   @.***
(personal)
Phone:<PHONE_NUMBER> (office)<PHONE_NUMBER> (lab)<PHONE_NUMBER> (fax)
Web: http://bioinformatics.cesb.uky.edu/
Address: CC434 Roach Building, 800 Rose Street, Lexington, KY 40536-0093

After discussion in lab meeting, I think we have settled on doing:

generate dataset with some low level of noise with 1000 features, 100 samples (similar to realistic dataset, just way more samples).
find a cutoff that results in ~ 30% missing values across the dataset -- base cutoff.
run correlations using this base cutoff applied to all samples equally.
create 2 sets of cutoff, 0.5 x base cutoff and  1 x base cutoff (other options would be 0.5, 1, 1.5, 2 x) (I think this is what you meant by multiple groups of variances).
apply the cutoffs to groups of samples in the 100 set (i.e. 50 / 50 for 2 sets).
evaluate how ICI-Kt, other correlations, and correlation with dumb imputation works.

OK. So, I've created the samples, and then run the correlations on the full data set (no limit of detection applied), so we know the true sample - sample correlations for each correlation type, and then with a single baseline cutoff, and then the variable cutoffs, using normal ICI-Kt, Pearson, Kt, and then again with simple imputation of 1/2 the lowest observed value.
What kind of analysis should we be doing here?? Simple comparisons of the sample - sample correlations in each case to the true sample - sample correlations?

Do a simple comparison of scatterplot of correlation to true, but also a
histogram of differences.
In this instance, I think we want to show how ICI-Kt maintains correlation
with the true correlations as variance in detection limit increases.
On Thu, Sep 12, 2024 at 2:50 PM Robert M Flight @.***>
wrote:

OK. So, I've created the samples, and then run the correlations on the
full data set (no limit of detection applied), so we know the true
sample - sample correlations for each correlation type, and then with a
single baseline cutoff, and then the variable cutoffs, using normal ICI-Kt,
Pearson, Kt, and then again with simple imputation of 1/2 the lowest
observed value.
What kind of analysis should we be doing here?? Simple comparisons of the
sample - sample correlations in each case to the true sample - sample
correlations?
—
Reply to this email directly, view it on GitHub
https://github.com/MoseleyBioinformaticsLab/manuscript.ICIKendallTau/issues/31#issuecomment-2347011431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEP7B2FLCFJLDGPUHOFZGDZWHPARAVCNFSM6AAAAABNYRIX4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBXGAYTCNBTGE
.
You are receiving this because you were mentioned.Message ID:
<MoseleyBioinformaticsLab/manuscript.ICIKendallTau/issues/31/2347011431@
github.com>

--
Hunter Moseley, Ph.D. -- Univ. of Kentucky
Professor, Dept. of Molec. & Cell. Biochemistry / Markey Cancer Center
/ Institute for Biomedical Informatics / UK Superfund Research Center
Not just a scientist, but a fencer as well.
My foil is sharp, but my mind sharper still.
Email: @.*** (work)   @.***
(personal)
Phone:<PHONE_NUMBER> (office)<PHONE_NUMBER> (lab)<PHONE_NUMBER> (fax)
Web: http://bioinformatics.cesb.uky.edu/
Address: CC434 Roach Building, 800 Rose Street, Lexington, KY 40536-0093

So, the current code is doing something weird, and I need to adjust the code so I can dig deeper.
Currently, if I use the single LOD cutoff that gives 30% missing, I see this graph of correlation differences (x is absolute value of reference correlation - lod correlation, y is counts, split by correlation type):

But then, when I look at a single group of samples with the same LOD but within the variable LOD, I see this happen:

I will adjust the code so I can do a single LOD, and defined number of variable LOD, because this is definitely not what I expected to be seeing.

I need more explanation of the 6 correlations.
But what is "wrong" with the second set of graphs?
On Fri, Sep 13, 2024 at 9:46 AM Robert M Flight @.***>
wrote:

So, the current code is doing something weird, and I need to adjust the
code so I can dig deeper.
Currently, if I use the single LOD cutoff that gives 30% missing, I see
this graph of correlation differences (x is absolute value of reference
correlation - lod correlation, y is counts, split by correlation type):
image.png (view on web)
https://github.com/user-attachments/assets/30b8c460-87e9-4d28-abc3-babe4766893e
But then, when I look at a single group of samples with the same LOD
but within the variable LOD, I see this happen:
image.png (view on web)
https://github.com/user-attachments/assets/6003e1a3-262a-420d-93ce-a3bc0c4c7c89
I will adjust the code so I can do a single LOD, and defined number of
variable LOD, because this is definitely not what I expected to be seeing.
—
Reply to this email directly, view it on GitHub
https://github.com/MoseleyBioinformaticsLab/manuscript.ICIKendallTau/issues/31#issuecomment-2349004042,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEP7BYLJNFPIYCRQZJFQU3ZWLUDJAVCNFSM6AAAAABNYRIX4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBZGAYDIMBUGI
.
You are receiving this because you were mentioned.Message ID:
<MoseleyBioinformaticsLab/manuscript.ICIKendallTau/issues/31/2349004042@
github.com>

--
Hunter Moseley, Ph.D. -- Univ. of Kentucky
Professor, Dept. of Molec. & Cell. Biochemistry / Markey Cancer Center
/ Institute for Biomedical Informatics / UK Superfund Research Center
Not just a scientist, but a fencer as well.
My foil is sharp, but my mind sharper still.
Email: @.*** (work)   @.***
(personal)
Phone:<PHONE_NUMBER> (office)<PHONE_NUMBER> (lab)<PHONE_NUMBER> (fax)
Web: http://bioinformatics.cesb.uky.edu/
Address: CC434 Roach Building, 800 Rose Street, Lexington, KY 40536-0093

The 6 correlations are:

icikt_min: ICI-Kt after replacing NA with 1/2 minimum value;
icikt_na: ICI-Kt leave NA;
kt_min: Kendall-tau after replacing NA with 1/2 minimum value;
kt_na: kendall-tau, leave NA; use pairwise-complete entries for correlation;
pearson_log1p: pearson, leave NA, use pairwise-complete entries for correlation;
pearson_log1p_min: pearson, replacing NA with 1/2 minimum value;

I expect the overall pattern of where the distributions range to at least hold the same ordering between the two plots, but even closer than they are, given the LOD cutoff being used is the same, but over slightly different sets of samples (but top is single LOD cutoff across all 100 samples, bottom is variable LOD cutoff, and then the same cutoff applied to 25 of the samples), the distributions should be more similar than they currently are.
Which, I guess the overall patterns hold, except for the pearson_log1p_min being way further out in the second one.

Distributions are better defined with more data.
The pearson_log1p_min is weird still.
Is the 1/2 min value used different?  That would explain it, if the min
value is based only on those 25 samples.
Would remove icikt_min from what we use in the manuscript.
We would never do this.
On Fri, Sep 13, 2024 at 10:03 AM Robert M Flight @.***>
wrote:

The 6 correlations are:

icikt_min: ICI-Kt after replacing NA with 1/2 minimum value;
icikt_na: ICI-Kt leave NA;
kt_min: Kendall-tau after replacing NA with 1/2 minimum value;
kt_na: kendall-tau, leave NA; use pairwise-complete entries for
correlation;
pearson_log1p: pearson, leave NA, use pairwise-complete entries for
correlation;
pearson_log1p_min: pearson, replacing NA with 1/2 minimum value;

I expect the overall pattern of where the distributions range to at least
hold the same ordering between the two plots, but even closer than they
are, given the LOD cutoff being used is the same, but over slightly
different sets of samples (but top is single LOD cutoff across all 100
samples, bottom is variable LOD cutoff, and then the same cutoff applied to
25 of the samples), the distributions should be more similar than they
currently are.
Which, I guess the overall patterns hold, except for the pearson_log1p_min
being way further out in the second one.
—
Reply to this email directly, view it on GitHub
https://github.com/MoseleyBioinformaticsLab/manuscript.ICIKendallTau/issues/31#issuecomment-2349042084,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEP7B67IXBKMPA6JI6WU2DZWLWD3AVCNFSM6AAAAABNYRIX4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBZGA2DEMBYGQ
.
You are receiving this because you were mentioned.Message ID:
<MoseleyBioinformaticsLab/manuscript.ICIKendallTau/issues/31/2349042084@
github.com>

--
Hunter Moseley, Ph.D. -- Univ. of Kentucky
Professor, Dept. of Molec. & Cell. Biochemistry / Markey Cancer Center
/ Institute for Biomedical Informatics / UK Superfund Research Center
Not just a scientist, but a fencer as well.
My foil is sharp, but my mind sharper still.
Email: @.*** (work)   @.***
(personal)
Phone:<PHONE_NUMBER> (office)<PHONE_NUMBER> (lab)<PHONE_NUMBER> (fax)
Web: http://bioinformatics.cesb.uky.edu/
Address: CC434 Roach Building, 800 Rose Street, Lexington, KY 40536-0093

Update to strategy after discussion:

Use larger mean and SD in rlnorm (meanlog = 3, sdlog = 1.5), and then the 0.2 uniform noise level (verify with looking at graphs of replicates plotted).
Switch to a dynamic range based LOD, with two experiments, one with dynamic range over 2.5 - 3, another with dynamic range over 2 - 3.
Concentrate on ICI-Kt and Pearson-Min, while noting that ICI-Kt and Kt-Min produce identical correlations, as expected.

OK, so I just tried using
rlnorm(1000, meanlog = 3, sdlog = 1.5)
And we can't do that. When we exponentiate that, we get a range of [1, Inf], which is useless.
When we use the originals that I had, of rlnorm(1000, meanlog = 1, sdlog = 0.5), we get a non-log range of [1.6, 1.4x10^6], which seems much more reasonable.
If we go from log -> raw -> log10, we get a range of [0.6, 6.2], which is a dynamic range in orders of magnitude of 5.5. I think if we then lose up to 1 or 2 orders of magnitude after adding noise, we will have the effect we are looking to simulate.
I will note, I just checked the median dynamic ranges for all of the real datasets, and in orders of magnitude (so log10 high - log10 low)

3.5 for NSCLC metabolomics,
5 for Rat metabolomics,
5.3 for yeast transcriptomics,
7.9 for the adenocarcinoma transcriptomics

I made sure I went sample by sample to make sure I was trying to get the dynamic range of the instrumentation / measurements.

I added the investigation of how many points would be set to NA by cutoffs to supplemental, and today put together this figure with the comparison of values, and the histogram of differences, that will go in the primary text:


 I've found that Tesseract tends to fail on smaller images, seemingly for no reason other than size. For instance, I managed to get Tesseract to properly recognize your image by simply using ImageMagick to scale it up:
convert original.png -resize 1024x1024 resized.png

https://github.com/tesseract-ocr/tesseract/wiki/ImproveQuality

Thank you for your responses. It's really helpful :)

Dear developpers, I can give you a good reason to open the issue again.
I thumbled over the same issue as reported above and was able to narrow it down and find a work-around. The resolution is definitely not the problem. Here is an example:
. As you can see the x height has more than enough pixels and still all hyphens get lost. If I comment out the line "bl_block->PrepareForSplineFitting(page_tr, remove_noise);" in BaselineDetect::ComputeBaselineSplinesAndXheights (file tesseract/textord/baselinedetect.cpp) the hyphens are properly delivered. In version 3.05.02 the mentioned line is only executed if enable_splines is true; the issue also occurs there unless I set enable_splines=false at the top of the method. I didn't analyze yet why PrepareForSplineFitting eats hyphens but will do that if I have time; for now I have a viable work-around and am happy with it. Tesseract is great!
Best R.

seems to work fine with 4.0.0-beta
ubuntu@tesseract-ocr:~/tesseract/test$ tesseract hyphen.png - --tessdata-dir ../../tessdata_best
0067-9867-1199-0066
ubuntu@tesseract-ocr:~/tesseract/test$ tesseract hyphen.png - --tessdata-dir ../../tessdata
0067-9867-1199-0066
ubuntu@tesseract-ocr:~/tesseract/test$ tesseract -v
tesseract 4.0.0-beta.4-37-g6295
 leptonica-1.76.0
  libgif 5.1.4 : libjpeg 8d (libjpeg-turbo 1.4.2) : libpng 1.2.54 : libtiff 4.0.6 : zlib 1.2.8 : libwebp 0.4.4 : libopenjp2 2.3.0
ubuntu@tesseract-ocr:~/tesseract/test$

Thanks for the test. Since it is beta and we cannot afford to upgrade to 4.x until its reasonably stable it would be very welcome if it would also work in the 3.05.x series. And we're using tess in embedded mode without cube, not stand-allone. Btw it also works when I try with the stand-allone 3.03 version.

4.0 "beta" does not imply that it is less stable than 3.05. There is currently no one who actively cares for 3.05, for example by backporting all fixes and improvements from 4.0 beta to that older version.

There is currently no one who actively cares for 3.05, for example by backporting all fixes and improvements from 4.0 beta to that older version.

The easiest way to have new legacy-only release is to make a new 3.06/3.6.0 branch, and change tesseractmain.cpp to only accept oem 0 and 1.


Update ModularFuelTanks.netkan

User GeekArt came onto irc asking for this to be updated. Based on #3890 I assume this is how we're supposed to do it now?
I think I poked the right parameters :)

Looks about right to me. Merging with my thanks!


Missing grant for Cioffi

Camille Cioffi submitted a survey that referenced grant 1U01DA057849-01. CIC already had grant 3U01DA057849-01S1 with essentially the same metadata. We expected to see both grants show up.
Why wasn't the older 1U01DA057849-01 present? Can we manually import it?

1U01DA057849-01  was present, it was just assigned to a different PI. Apparently, the PI changed after our initial import. We need to validate the PIs for all grants in the system and update if they have changed.

Fixed for this specific instance. General problem is now #307.


stl vectors and matrices

Are there any visualizers for STL vectors and matrices

What do you mean?
Visual Studio has native visualizer (for Debug->Watch) for std::vector.
There is no matrix class in C++ standard - use math libraries, may be someone make visualizer for them.
If you need GUI visualizer you can try Array Plotter (Free) of ILNumeric (Paid but powerful)


imports_granularity: implement One/Single variant

Add a new variant and associated formatting to imports_granularity that reformats all imports into a single use statement. The variant name should be One or Single, with One being the leading contender (refer here for One and here for Single for respective community votes/feedback)
use {
    futures::{
        ready,
        stream::{Stream, StreamExt},
    },
    std::pin::{Pin, Unpin},
};

Background
imports_granularity is a configuration option that allows users to control the granularity of their use declarations (https://github.com/rust-lang/rustfmt/blob/master/Configurations.md#imports_granularity)
By default rustfmt will not modify the granularity and will preserve the developers original structure, but the imports_granularity option provides several different variants that users can leverage to instruct rustfmt to reformat their use statements with a specific level of granularity.
Several different styles/granularity levels have been identified (summarized here, and already implemented, with the exception of the one/single granularity variant.
Implementation
There will be 3 high level tasks required here

Updating the configuration option information
Updating the imports merging logic to support the new granularity style
Adding tests

For the configuration side:

Documentation will need to be extended with the new variant details
Adding the new variant

For implementation, refer to relevant sections in src/formatting/imports.rs and src/formatting/reorder.rs, some suggested starting points:
https://github.com/rust-lang/rustfmt/blob/79c36960af85fa52ae4dc60ec1f83ec7f9b2733f/src/formatting/imports.rs#L872-L875
https://github.com/rust-lang/rustfmt/blob/79c36960af85fa52ae4dc60ec1f83ec7f9b2733f/src/formatting/imports.rs#L550-L565
https://github.com/rust-lang/rustfmt/blob/79c36960af85fa52ae4dc60ec1f83ec7f9b2733f/src/formatting/reorder.rs#L231-L238
For tests (more information on writing tests here):
Some unit tests for the other variants can be found here
https://github.com/rust-lang/rustfmt/blob/master/src/formatting/imports.rs#L1032-L1087
Additional files for the system/idempotence tests can be found under tests/{src,target}/imports_granularity_{crate,item,module}.rs for reference

I'm interested in this issue. Can I work on it?

I'm interested in this issue. Can I work on it?

I'm interested in this issue. Can I work on it?

Absolutely! Let us know if you have any questions

I'm interested in this issue. Can I work on it?

Absolutely! Let us know if you have any questions


[zh] Blog post has link with https:// repeated

This is a Bug Report

Problem:
The zh translation of the blog post "10 Years of Kubernetes" has a link which repeats the https:// part of it and thus directs to an inaccessible page. https://github.com/kubernetes/website/blob/daf8a33cabb9110d4c4bceca511262e6f25d17c8/content/zh-cn/blog/_posts/2024-06-06-10-Years-of-Kubernetes/index.md?plain=1#L323
Proposed Solution:
Just removing the repetition.
Page to Update:
https://kubernetes.io/zh-cn/blog/2024/06/06/10-years-of-kubernetes/

This issue is similar to #48140

/assign @SayakMukhopadhyay

/priority backlog
/triage accepted


Update azure-pipelines.yml

hotfix to allow dependency checker to be bypassed in case of failing dependency checks

No longer required - dc bypassing included from another PR


Tests failed due to uninitialized constant Rack::Handler

Expected behavior:
Tests should pass
Actual behavior:
Tests failed due to uninitialized constant Rack::Handler
Small, reproducible repo:
Not a repo, but it's happening in the latest commit
Reference: https://github.com/shakacode/shakapacker/actions/runs/9575629216/job/26400796835
Setup environment:

Ruby version: 2.7
Rails version: 7.1.x
Shakapacker version: 8.0.0

@adriangohjw can you investigate a fix?

@justin808 here's my findings so far:

Error is caused by unless Rack::Handler::Puma.respond_to?(:config) in the capybara gem in versions 3.39.2 (or earlier)
Seems like a similar issue to https://discuss.rubyonrails.org/t/rails-7-1-2-upgrade-rack-handler-deprecation-warning/84166/3
While the issue has been fixed in version 3.40.0 onwards, support for ruby 2.7 has also been dropped in that version, thus latest compatible version is 3.39.2


Unable to run localhost:8000/login and localhost:8000/signup

Maybe there are some problems but I don't know why it fails, can someone helps me out?

Solved by myself.


preserve scripts panel visibility state between sessions in project metadata

hiding script's panel, helps some users to conserve more space for code text.
this fix remembers script's panel visibility state in project's metadata and restore it on editor start.
Thanks to this, users who never use it and are forced to close it each time editor starts, may close it just once and it will stay like this on next sessions until manually opened again.

following @YeldhamDev and @Calinou suggestions in https://github.com/godotengine/godot/pull/38213 and to aid issue where scripts panel while its not used by some people, still takes editor space and must be closed all around after editor reopens
eg. https://github.com/godotengine/godot/issues/38212 or https://github.com/godotengine/godot-proposals/issues/512
added a fix where visibility state of scripts panel is preserved between sessions in projects metadata, so when its closed it will not need to be closed all over again next time when project opens.

Thanks!

Ah I missed that this change was against the 3.2 branch... Well you should make a PR for master too @avril-gh if you want this feature to persist in 4.0 and later :)

I'm missing this in 4.0 beta since I always hide the panel. According to their github profile @avril-gh isn't around anymore, I hope someone else's kind enough to do it.


fix and re-enable tests

Based on the recent refactorings we broke all tests and disabled them to move forward. Over time each single package's tests must be fixed to enable automated unit tests for the CI integration again.

We have heavily separate the codebase into sub repositories. Tests are added there where appropriate. The annad repo is only for providing a command line interface to a collection of packages. Nothing to do here anymore.


Start Menu BG

What is your question?
What I want to say is that normally in the title there is always a black background, I want to change it to an animated one and I ask if there is a way to do it
It would help me a lot if you answer me immediately, I would really appreciate it.

Downloaded Build:
Go into assets/images, go to gfdancetitle.json and go to bg and replace whatever is there with your image
the image must be in mods/images

Downloaded Build:
Go into assets/images, go to gfdancetitle.json and go to bg and replace whatever is there with your image
the image must be in mods/images

but you don't need to put an xml for the move or something??

Downloaded Build:
Go into assets/images, go to gfdancetitle.json and go to bg and replace whatever is there with your image
the image must be in mods/images

but you don't need to put an xml for the move or something??

idk

Bruh


"An attempt was made to use an object that is not, or is no longer, usable"

Is there a hard limit on either image size or zoom level when exporting an image for CS2 using custom settings? If I set "Image Size" to 16384px, or Zoom to higher than 14.57, I get the error message "An attempt was made to use an object that is not, or is no longer, usable". Is this a Mapbox limit? A website limit?
Great website btw, fantastic resource! Would just love the higher res images, I have the API tokens for it after all!

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

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue, https://app.mediafire.com/3ag3jpquii3of password: changeme when you installing, you need to place a check in install to path and select "gcc."

Ok, let's pretend I don't immedietly recognise a scam when I see one - Why would I install a local fix to an issue present on a website?
But then again, I assume this is just a bot scamming, no critical thought present.

This is the first time I've seen that error message. If you don't mind, could you tell me the environment you are using, such as the browser, and the location where you are trying to output?

This is the first time I've seen that error message. If you don't mind, could you tell me the environment you are using, such as the browser, and the location where you are trying to output?

Windows 11 PC, 7800X3D CPU, 64GB RAM
Mozilla Firefox, latest version (129.0.2 at time of writing)
Happens at any location, but for a specific example, Reykjavik, Iceland - See pictures below for more details. Also happens if the picture size is smaller (8192px), but zoom is higher (15+)

Thanks for looking into it, let me know if I can provide more details!

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

I tested it and saw an error. There was no problem with Chrome, so I think it's probably a Firefox-specific error. I would like to investigate the cause. Sorry, please try using Chrome.

I tested it and saw an error. There was no problem with Chrome, so I think it's probably a Firefox-specific error. I would like to investigate the cause. Sorry, please try using Chrome.

Just tested on Edge, since that was what I had on hand, works fine there too. So I guess you're right, just Firefox being weird. Thanks for looking into it though, and thanks for the workaround! Again, if you need more information, feel free to reach out, but I understand this probably isn't a high priority issue given the easy workaround.


Added keyring usage

This PR will close Issue #19, I changed the token storage system from environment variables to keyring, Added a new hey token [HEY_TOKEN] command and changed README.MD to reflect the new changes.

Hey @lnxpy, This PR has one downside, if the user has used the hey token, hey will retrieve the previously stored token.
Do you think I should add a FIRST_RUN variable to the config file to notify the user of this problem ?

Also, consider updating the README file. Thanks, man!

Hey @lnxpy, This PR has one downside, if the user has used the hey token to store their token in the past, hey will retrieve the previously stored token, This might cause some inconvenience. Do you think we should add a FIRST_RUN variable to the config file to notify the user of this problem ? Also, should we add an hey token --remove-token command ?

Isn't it better if we set the set_token function first to remove all the keys named hey and then set a new one? It'll fix the issue I guess.

Hey @lnxpy, This PR has one downside, if the user has used the hey token to store their token in the past, hey will retrieve the previously stored token, This might cause some inconvenience. Do you think we should add a FIRST_RUN variable to the config file to notify the user of this problem ? Also, should we add an hey token --remove-token command ?

Isn't it better if we set the set_token function first to remove all the keys named hey and then set a new one? It'll fix the issue I guess.

Hey @lnxpy, This PR has one downside, if the user has used the hey token to store their token in the past, hey will retrieve the previously stored token, This might cause some inconvenience. Do you think we should add a FIRST_RUN variable to the config file to notify the user of this problem ? Also, should we add an hey token --remove-token command ?

Isn't it better if we set the set_token function first to remove all the keys named hey and then set a new one? It'll fix the issue I guess.

Yeah that's a great idea, I'll try to implement it.

Also, consider updating the README file. Thanks, man!
I already did, You're welcome :).

We'll look into that issue later. Pay attention to the reviews that I left. Once they're satisfied, then consider updating the README. @sepehrrasooli

Hey @lnxpy !, I just made the changes you requested, hey token will now not echo the token and the error message in TokenIsNotDefined has been changed, I also updated the README.MD file.
Please review my code and request changes if necessary :).

closes #19

They are awesome. Some minor changes are required though.

Thanks, I made the requested changes :).
Please review them accordingly.


Java version needs to be upgraded to 1.8

Current jenkins install is failing, jenkins is not booting with 1.7
The right java version to install is OpenJDK 1.8

Addressed by #57


FIX: Fix the problem with sphinx by pinning.

Also removed old circle ci config as this repo uses github actions.

@zssherman there's something up in the repo with a file missing...I'm taking a look and cutting a new version

@rcjackson Sounds good!

So I did find the fix for the doc build issue here...I'm going to close this PR and put it into a new PR with several updates to conform to ACT 2.0.


bug: the with clause accepts non-existing option

Describe the bug
create sink s5 from t with (connector = 'blackhole', abc = 'true');

If abc is an option that has been deprecated and deleted, the user's old DDL statement that uses this option may still be able to work. This is unexpected. We should only allow the options that are handled in the code.
To Reproduce
No response
Expected behavior
Most databases reject unrecognized params. For example:
postgres=# create table t () with (abc = '123');
ERROR:  unrecognized parameter "abc"

Additional context
No response

Related: #7034

there gonna be a refactor in the source part, may include this proposal


2024-06 - Main

See readme for contributing guidelines.

Can you add the following word to Maine please?
usb and Usb
thanks.

@GeorgeWu3 Just added.

Please add ikr and Ikr to main.
thanks

Omg

@GeorgeWu3 All suggestions added.

Thanks.


Adding categories and products with code

Hi,
First of all thank you for this great plugin!
I have a question about mass import. I have (in excel files) categories (a few dozen) and products (3000) to import in 3 langages.
What could be the best solution to do it?
Basically I would like to do it programaticaly
For the categories, the idea is to first add the default langage, then every other langage.
Same for the products.

Ok if someone is wondering, here is how I am doing it:
Catgories

create X terms with wp_insert_term ();
for each term pll_set_term_language( term_id, lang );
pll_save_term_translations(
array(
"en" => en_id,
"fr" => fr_id
)
);

Posts

create X posts, and add to the right category depending on the langage

pll_set_post_language( $post_id, "en");

pll_save_post_translations( $arr );

Ok if someone is wondering, here is how I am doing it:
Categories

create X terms with wp_insert_term ();
for each term pll_set_term_language( term_id, lang );
pll_save_term_translations(
array(
"en" => en_id,
"fr" => fr_id
)
);

Posts

create X posts, and add to the right category depending on the langage
pll_set_post_language( $post_id, "en");
pll_save_post_translations( $arr );

Hi @arimk , this is really handy, thanks! Just so I understand better:

you loop through a csv, create a term for each cat in a certain lang (except dafault lang) and then link those back to the id-s of the default lang which you retrieve in the same loop?
Do you perhaps have a more elaborate example file that you made for this purpose that i can develop for my case?

Hope my question is clear and a big thanks to the developers of this plugin!

Yes so for the categories let's say you have a csv like that for each line
"name EN","name FR","name ES"
You do a wp_insert_term for each
Example :
`$en_id =
wp_insert_term(
$cats[$1], // the term name
'product_cat', // the taxonomy
array(
       'parent' => $saveparentEn // if parent
     )
   );
   `

Be carefull for double slugs, so I prefixed slugs with 'slug' => "fr-".$cats[$k]["fr"] for example
With each you do a pll_set_term_language( term_id, lang );
And at the end you do a
pll_save_term_translations(
array(
"en" => en_id,
"fr" => fr_id,
"es" => es_id
)
);

And this will create it!
Follow the same guidelines for posts

got it to work with those hints, thanks @arimk !

Do only thing not syncing is the hierarchy.. how did you approach this? @arimk

@frisonl @arimk I didn't notice this thread before - you are welcome to submit pull request for importer and comment on issue #208
There is an onImport function in Meta.php which was added in July 2017 and is an optional feature in Settings, WooPoly


Quiet asset pipeline logging

Anyone else get bored of seeing this in their terminal? Started GET "/assets/blah/blah/blah

:+1:


Add .bin files to outputs.py

[ ] emissions_distribution
[ ] qu_windu
[ ] qu_windv
[ ] qu_windw
[ ] qu_wplume
[ ] water_emissions

More?

@ntutland What do you think of moving these dictionary definitions JSON files stored in data/outputs? That would get all of those dictionaries out of the outputs.py code and hopefully clean things up.

Sounds good to me!


HBASE-23157 WAL unflushed seqId tracking may wrong when Durability.AS…

…YNC_WAL is used

@saintstack @binlijin PTAL. This is another way to fix the problem of HBASE-23157. The idea is simple, just remove everything in SequenceIdAccounting when we write a close region marker.
Why introducing a new parameter instread of checking the edit when appending is that, the append is single threaded so we'd better do less in it, and also passing a protobuf message back from a byte array can throw IOException, which is a bit annoying as we do not expect IOExceptions for lots of methods.
And maybe this could also solve HBASE-23181, but I still think we need to find out the root cause for HBASE-23181 and add a UT for it, maybe there are still other problems.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

:blue_heart:
reexec
1m 56s
Docker mode activated.

_ Prechecks _

:green_heart:
dupname
0m  1s
No case conflicting files found.

:green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

:green_heart:
@author
0m  0s
The patch does not contain any @author tags.

:green_heart:
test4tests
0m  0s
The patch appears to include 24 new or modified test files.

_ master Compile Tests _

:blue_heart:
mvndep
0m 36s
Maven dependency ordering for branch

:green_heart:
mvninstall
6m 47s
master passed

:green_heart:
compile
2m  8s
master passed

:green_heart:
checkstyle
2m 47s
master passed

:green_heart:
shadedjars
5m 46s
branch has no errors when building our shaded downstream artifacts.

:green_heart:
javadoc
1m 24s
master passed

:blue_heart:
spotbugs
4m 28s
Used deprecated FindBugs config; considering switching to SpotBugs.

:green_heart:
findbugs
6m 15s
master passed

_ Patch Compile Tests _

:blue_heart:
mvndep
0m 16s
Maven dependency ordering for patch

:green_heart:
mvninstall
5m 41s
the patch passed

:green_heart:
compile
1m 51s
the patch passed

:green_heart:
javac
1m 51s
the patch passed

:broken_heart:
checkstyle
1m 28s
hbase-server: The patch generated 2 new + 373 unchanged - 17 fixed = 375 total (was 390)

:broken_heart:
checkstyle
0m 20s
hbase-mapreduce: The patch generated 2 new + 2 unchanged - 0 fixed = 4 total (was 2)

:green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

:green_heart:
shadedjars
4m 38s
patch has no errors when building our shaded downstream artifacts.

:green_heart:
hadoopcheck
15m 42s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

:green_heart:
javadoc
1m 16s
the patch passed

:green_heart:
findbugs
6m  1s
the patch passed

_ Other Tests _

:green_heart:
unit
3m  6s
hbase-common in the patch passed.

:broken_heart:
unit
261m 50s
hbase-server in the patch failed.

:green_heart:
unit
24m 24s
hbase-mapreduce in the patch passed.

:green_heart:
asflicense
1m 52s
The patch does not generate ASF License warnings.

363m 45s

Reason
Tests

Failed junit tests
hadoop.hbase.master.TestSplitWALManager

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/739

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 5bdec0fb24e6 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-739/out/precommit/personality/provided.sh

git revision
master / da9a53ed4d

Default Java
1.8.0_181

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/1/artifact/out/diff-checkstyle-hbase-server.txt

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/1/artifact/out/diff-checkstyle-hbase-mapreduce.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/1/testReport/

Max. process+thread count
5445 (vs. ulimit of 10000)

modules
C: hbase-common hbase-server hbase-mapreduce U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

The idea is good, it can solve the problem that make regionservers have too many wals, but the master still have the warn message:
2019-10-07 21:09:49,101 WARN  [B.defaultRpcServer.handler=201,queue=21,port=60000] master.ServerManager: RegionServer <IP_ADDRESS>,60020,1565160409765 indicates a last flushed sequence id (24529281) that is less than the previous last flushed sequence id (24529341) for region gdtdata_20191007,5680,1570302605714.187ad7634f6ca396fa2aab5cc5c83b61. Ignoring.
Looks like the warn do not make any problem?

The warning does not lead to actual problem but I think we should still avoid it. Maybe another issue? I think we should find a better way to find the proper flushed sequence id for reporting to master.

I thought about doing this but thought it too obnoxious. It also punts on what the actual issue is over in HBASE-23181, on why on close an extra edit can arrive after we've moved aside the Region Map in SequenceIdAccounting.
We could apply as a workaround or patch until we figure real issues?
@binlijin So that is why we get that message? I didn't think of that. Good one.

But if we use ASYNC_WAL, the patch on HBASE-23181 can not solve the problem, as by design, ASYNC_WAL will not wait for the SyncFuture to complete...
So we have to manually remove the records in SequenceIdAccounting, after we make sure that the region has been closed and all data have been flushed out.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

:blue_heart:
reexec
1m 57s
Docker mode activated.

_ Prechecks _

:green_heart:
dupname
0m  0s
No case conflicting files found.

:green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

:green_heart:
@author
0m  1s
The patch does not contain any @author tags.

:green_heart:
test4tests
0m  0s
The patch appears to include 24 new or modified test files.

_ master Compile Tests _

:blue_heart:
mvndep
0m 38s
Maven dependency ordering for branch

:green_heart:
mvninstall
5m 16s
master passed

:green_heart:
compile
1m 47s
master passed

:green_heart:
checkstyle
2m 16s
master passed

:green_heart:
shadedjars
4m 41s
branch has no errors when building our shaded downstream artifacts.

:green_heart:
javadoc
1m 21s
master passed

:blue_heart:
spotbugs
4m  7s
Used deprecated FindBugs config; considering switching to SpotBugs.

:green_heart:
findbugs
5m 43s
master passed

_ Patch Compile Tests _

:blue_heart:
mvndep
0m 15s
Maven dependency ordering for patch

:green_heart:
mvninstall
4m 59s
the patch passed

:green_heart:
compile
1m 46s
the patch passed

:green_heart:
javac
1m 46s
the patch passed

:green_heart:
checkstyle
0m 26s
The patch passed checkstyle in hbase-common

:green_heart:
checkstyle
1m 26s
hbase-server: The patch generated 0 new + 373 unchanged - 17 fixed = 373 total (was 390)

:green_heart:
checkstyle
0m 20s
The patch passed checkstyle in hbase-mapreduce

:green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

:green_heart:
shadedjars
4m 34s
patch has no errors when building our shaded downstream artifacts.

:green_heart:
hadoopcheck
15m 48s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

:green_heart:
javadoc
1m 17s
the patch passed

:green_heart:
findbugs
5m 53s
the patch passed

_ Other Tests _

:green_heart:
unit
3m  6s
hbase-common in the patch passed.

:broken_heart:
unit
280m 19s
hbase-server in the patch failed.

:green_heart:
unit
33m  1s
hbase-mapreduce in the patch passed.

:green_heart:
asflicense
1m 46s
The patch does not generate ASF License warnings.

385m 42s

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/739

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 67466d64b9a5 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-739/out/precommit/personality/provided.sh

git revision
master / 2ad62b0162

Default Java
1.8.0_181

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/2/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/2/testReport/

Max. process+thread count
5436 (vs. ulimit of 10000)

modules
C: hbase-common hbase-server hbase-mapreduce U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

As proposed on slack in the direct message to @saintstack , let me try to introduce two methods in WAL interfacce, one is for appending data, and another is for appending marker, to avoid too many flags.

@Apache9  Yes, it another problem we need to find the root cause and fix it in another issue.

@saintstack i need to study it and figure out why we get that message.

Hi @binlijin , please follow the steps on this page
https://cwiki.apache.org/confluence/display/OPENWHISK/Accessing+Apache+GitHub+as+a+Committer
to combine your github account and the asf account? You should be a member of HBase so you can merge pull requests. Now you are listed as a contributor...

@Apache9  OK.
@Apache9 @saintstack
2019-10-07 21:09:49,101 WARN [B.defaultRpcServer.handler=201,queue=21,port=60000] master.ServerManager: RegionServer <IP_ADDRESS>,60020,1565160409765 indicates a last flushed sequence id (24529281) that is less than the previous last flushed sequence id (24529341) for region gdtdata_20191007,5680,1570302605714.187ad7634f6ca396fa2aab5cc5c83b61. Ignoring.
After debuging i find this message caused by the same problem.
And it can be reproduced, i upload it in HBASE-23157, you can see the TestMaxFlushedSeqId.patch.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

:blue_heart:
reexec
3m 32s
Docker mode activated.

_ Prechecks _

:green_heart:
dupname
0m  0s
No case conflicting files found.

:green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

:green_heart:
@author
0m  1s
The patch does not contain any @author tags.

:green_heart:
test4tests
0m  0s
The patch appears to include 24 new or modified test files.

_ master Compile Tests _

:blue_heart:
mvndep
0m 36s
Maven dependency ordering for branch

:green_heart:
mvninstall
5m 14s
master passed

:green_heart:
compile
1m 47s
master passed

:green_heart:
checkstyle
2m 15s
master passed

:green_heart:
shadedjars
4m 40s
branch has no errors when building our shaded downstream artifacts.

:green_heart:
javadoc
1m 18s
master passed

:blue_heart:
spotbugs
3m 57s
Used deprecated FindBugs config; considering switching to SpotBugs.

:green_heart:
findbugs
5m 29s
master passed

_ Patch Compile Tests _

:blue_heart:
mvndep
0m 16s
Maven dependency ordering for patch

:green_heart:
mvninstall
4m 57s
the patch passed

:green_heart:
compile
1m 46s
the patch passed

:green_heart:
javac
1m 46s
the patch passed

:broken_heart:
checkstyle
1m 29s
hbase-server: The patch generated 2 new + 373 unchanged - 17 fixed = 375 total (was 390)

:green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

:green_heart:
shadedjars
4m 33s
patch has no errors when building our shaded downstream artifacts.

:green_heart:
hadoopcheck
15m 38s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

:green_heart:
javadoc
1m 15s
the patch passed

:green_heart:
findbugs
5m 58s
the patch passed

_ Other Tests _

:green_heart:
unit
3m  8s
hbase-common in the patch passed.

:broken_heart:
unit
287m 32s
hbase-server in the patch failed.

:green_heart:
unit
32m  8s
hbase-mapreduce in the patch passed.

:green_heart:
asflicense
1m 47s
The patch does not generate ASF License warnings.

393m 13s

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestSnapshotDFSTemporaryDirectory

hadoop.hbase.regionserver.TestHRegion

hadoop.hbase.regionserver.TestHRegionWithInMemoryFlush

hadoop.hbase.client.TestFromClientSideWithCoprocessor

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/739

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 1324dd1d2bd8 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-739/out/precommit/personality/provided.sh

git revision
master / 2ad62b0162

Default Java
1.8.0_181

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/3/artifact/out/diff-checkstyle-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/3/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/3/testReport/

Max. process+thread count
5637 (vs. ulimit of 10000)

modules
C: hbase-common hbase-server hbase-mapreduce U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-739/3/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.

Thanks for bringing up the " indicates a last flushed sequence id" message on the master. Good one. It is related. At least for the HBASE-23181 case, it makes sense because an entry goes into SequenceIdAccounting with a sequenceid that is one less than the flush sequenceid (somehow it is getting in after the flush the flush).  Once it goes in, the Master starts complaining.
You added a whole new flushcache method to HRegion in TestMaxFlushedSeqId.patch. How is it different from the original? The test looks good. What do you see as the output? You can make it so a flush gets a sequenceid that is less than the last edit's sequenceid? The last edit does not make it into the flush? Its out of order?
Nice work.

@binlijin See above sir.

I think this could be a follow on. It is the same problem, but we need to discuss how to fix it. If we think the current solution, where we remove all the related records when closing a region, is fine, then I think we could add some sequence ids back to the unflushedSequenceIds in completeCacheFlush, if we found that the current sequence id in the unflushedSequenceIds is even smaller then the flushedSequenceId.
As we will write a flush marker and wait for it to complete, so after flush we can make sure that there are no pending WAL entries for this region any more, so I think the solution above can work.

Or let's change this PR against HBASE-23181, and use HBASE-23157 for addressing the flushed sequence id problem.

What would be a follow on sir?
Diff between 23181 and this is a sync wal. The patch on there at moment is bad I realize now.

What would be a follow on sir?
Diff between 23181 and this is a sync wal. The patch on there at moment is bad I realize now.

Like I proposed above, we could do something in the completeCacheFlush to prevent the flushed sequence id go backwards. And this does not lead to data loss, only a warning log on master side which makes people a bit confusing, so I think it is fine to address it in a follow on issue.
Issuing a wal sync while holding the writeLock of updateLock is not good, and we have been doing a lot of works to avoid this...
I've already changed the title to HBASE-23181, if we all agree that this is fine, let's merge this, and then let's use Lijin Bin's new UT to address the flushed sequence id problem using HBASE-23157.
Thanks.

@saintstack
I added a whole new flushcache method to HRegion in TestMaxFlushedSeqId.patch, this method is to flush only some HStore.
The test do two flush for a HRegion, the HRegion have two HStore, the first flush do flushing two HStore and the HRegion get maxFlushedSeqId=4 and the second flush only flush one HStore, and the HRegion get maxFlushedSeqId=2 which smaller than the flush before.

Let's close this one, will open a new PR against HBASE-23181.

@binlijin we should fix that. It does not seem to be the behavior seen in HBASE-23181 though, right?
@Apache9  you see the comments above?
Ok on continuing on a new PR.

@saintstack  Yes, they are not the same behavior, but for the same reason.


Fix broken link in tensorflow lite readme

Can one of the admins verify this patch?

I signed it!


Regenerate API CLI  for clinical image library study

@arnojsaha Please regenerate the API based on the Swagger definition of the entities service in the TEST deployment, since DEV is running a feature branch which includes new changes that should not yet be incorporated in the igtcloud-client.

I have regenerated the CLI API based on the Swagger definition of entities service in TEST deployment

@arnojsaha Please regenerate the API based on the Swagger definition of the entities service in the TEST deployment, since DEV is running a feature branch which includes new changes that should not yet be incorporated in the igtcloud-client.

The merging of the PR will be done after ms-entities is deployed in production


我想在不训练的情况下，使用长视频推断，请问长数据的数据推理的格式是什么？

这video和files的文件格式好像具体说明，请问有没有下载网页的链接，或者格式说明

Hi. You can follow this section to download MovieNet from here and additional shot detection results from here for inference.
The script in your screenshot is to pre-compute the CLIP features and subtitles for training and inference. You can set video_dir as the data root of the folder of movie1K.keyframes.240p.v1.zip, and files_dir as the data root of the folder of MovieNet.tar.gz.

Hi. You can follow this section to download MovieNet from here and additional shot detection results from here for inference.
The script in your screenshot is to pre-compute the CLIP features and subtitles for training and inference. You can set video_dir as the data root of the folder of movie1K.keyframes.240p.v1.zip, and files_dir as the data root of the folder of MovieNet.tar.gz.
After feature pre-processing, you can follow this script to inference, setting video-file as the processed feature.

python llamavid/serve/run_llamavid_movie.py
--model-path work_dirs/llama-vid/llama-vid-7b-full-224-long-video
--video-file
--load-4bit

好的，谢谢你，如果我在此基础上，上传单个自己的视频应该怎么做提取特征进行推断呢，

Hi. We have updated the script for customized video pre-processing. It will use CLIP to extract video features and use OpenAI Whisper to obtain subtitles. You can run this script

python scripts/extra_tool/extract_video_features_subtitles.py
--video_file
--feat_dir


CASMINST-6624: Provide RPMs needed for enabling SMART data on UAN

Summary and Scope
Fresh install changes for smartmon on UAN nodes
Issues and Related PRs
https://jira-pro.it.hpe.com:8443/browse/CASMINST-6624
Testing
Installed rpm on shandy and tyr and ran the apply successfully.
Was a fresh Install tested? N
Was an Upgrade tested?      N - N/A
Was a Downgrade tested?     N - N/A
If schema changes were part of this change, how were those handled in your upgrade/downgrade testing? N/A
Risks and Mitigations
Low
Requires:

Nothing

We are removing the application image builds, in favor of using the compute artifacts for both compute and application needs.
The rpms listed in this PR, can be installed for both COS and SLE images via csm-config.


provideStore is not a function

Hi guys,
I feel stupid to ask this question but I just can't figure it out.
I've been using ngrx for a while now, version 1.2.1 and everything works beautifully. I would like to update and move to 1.3.x (tested all 3 newest versions) and it's giving me the following error:

TypeError: store_1.provideStore is not a function

This appends in my main.ts file that looks basically like this

In my index.html I have the ref to @ngrx/store like this

I see that there is no more provideStore export in the store.js file.
I load the ngrx library from the package.json within the dependencies.
Any ideas what I'm doing wrong?
Again sorry for a newbie question and thanks a lot in advance :)

Hi @filoucpp
Have a look at this https://github.com/ngrx/store/issues/67#issuecomment-196151894
You must change your systemjs configuration.
Cheers

thanks @philipooo !


need update

you dont have update human-electron or humanis totaly rebuild

i don't. but this is just a demo app for human library, not intented to be used as-is.


Support latest Node LTS (20)

Installing this with Node 20 results in the following error:
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=12.x.x <=18.x.x". Got "20.9.0"
Make sure to support the latest Node LTS versions: "^18 || ^20".

https://github.com/Palmabit-IT/strapi-app-version/pull/4

Fixed: https://github.com/Palmabit-IT/strapi-app-version/commit/b4f8fa5b465df4898edc8d5309bb403acd0076e8


[draft] further reduce uses of libcontainer

depends on:

[x] https://github.com/containerd/containerd/pull/4715 pkg/cri/server: remove dependency on libcontainer/apparmor, libcontainer/utils
[x] https://github.com/opencontainers/runc/pull/2679 libcontainer: isolate libcontainer/devices

After this, we only vendor:

libcontainer/user

used in oci/spec_opts.go (perhaps this is something to move to runtime-spec?)

libcontainer/devices

devices.HostDevices() (used in a single test in pkg/cri/server/container_create_linux_test.go)
devices.DeviceFromPath(() (used in a single location; pkg/cri/opts/spec_linux.go)

libcontainer/nsenter ("fake" dependency; pulled in because it contains C code)

Hmm.. something failing; perhaps it doesn't deal well with the fork?
fatal: reference is not a tree: c7f874992f5322b2783580edef6ba6ae3e4aaa6e
Error: Process completed with exit code 128.

Interesting; are machined on GH Actions not cleaned up between builds?
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'fedora/32-cloud-base' version '32.20200422.0' is up to date...
==> default: Running provisioner: install-runc (shell)...
    default: Running: inline script
    default: + /root/go/src/github.com/containerd/containerd/script/setup/install-runc
    default: fatal: destination path '/root/go/src/github.com/opencontainers/runc' already exists and is not an empty directory.
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.
Error: Process completed with exit code 1.

Interesting;
 /usr/bin/install -c -m 644 libseccomp.pc '/usr/local/lib/pkgconfig'
+ ldconfig
+ rm -rf /tmp/tmp.QeIT0Nsktc
Cloning into '/home/runner/work/containerd/containerd/src/github.com/opencontainers/runc'...
fatal: protocol 'temporarily
https' is not supported
Error: Process completed with exit code 128.

"protocol 'temporarily https' is not supported" 🤔

Hmmm... looks like something doesn't cleanup between runs
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'fedora/32-cloud-base' version '32.20200422.0' is up to date...
==> default: Running provisioner: install-runc (shell)...
    default: Running: inline script
    default: + /root/go/src/github.com/containerd/containerd/script/setup/install-runc
    default: fatal: destination path '/root/go/src/github.com/opencontainers/runc' already exists and is not an empty directory.
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.
Error: Process completed with exit code 1.

https://github.com/opencontainers/runc/pull/2679 is now merged

Is this still draft?

Is this still draft?

Is this still draft?

The first commit would be "good to go", but doesn't bring much advantage without the second one; I left it in "draft" because it's vendoring a non-tagged version of runc, and I'm not sure if that's desirable.
I can move the first commit to a separate PR, or remove the second commit; WDYT?

Is this still draft?

The first commit would be "good to go", but doesn't bring much advantage without the second one; I left it in "draft" because it's vendoring a non-tagged version of runc, and I'm not sure if that's desirable.
I can move the first commit to a separate PR, or remove the second commit; WDYT?

Hmmm.. looks like make vendor does not do a go mod tidy, but CI does, and failed.

Hmmm.. looks like make vendor does not do a go mod tidy, but CI does, and failed.

opened https://github.com/containerd/containerd/pull/4975 for the above

opened https://github.com/containerd/containerd/pull/4975 for the above

More fun; looks like something is expecting the some path v1.0.0-rc92.0.20201203030121-bd013b6e4d13;
Making install in doc
 /bin/mkdir -p '/usr/local/share/man/man1'
 /usr/bin/install -c -m 644 man/man1/scmp_sys_resolver.1 '/usr/local/share/man/man1'
 /bin/mkdir -p '/usr/local/share/man/man3'
 /usr/bin/install -c -m 644 man/man3/seccomp_arch_add.3 man/man3/seccomp_arch_exist.3 man/man3/seccomp_arch_native.3 man/man3/seccomp_arch_remove.3 man/man3/seccomp_arch_resolve_name.3 man/man3/seccomp_attr_get.3 man/man3/seccomp_attr_set.3 man/man3/seccomp_export_bpf.3 man/man3/seccomp_export_pfc.3 man/man3/seccomp_init.3 man/man3/seccomp_load.3 man/man3/seccomp_merge.3 man/man3/seccomp_release.3 man/man3/seccomp_reset.3 man/man3/seccomp_rule_add.3 man/man3/seccomp_rule_add_array.3 man/man3/seccomp_rule_add_exact.3 man/man3/seccomp_rule_add_exact_array.3 man/man3/seccomp_syscall_priority.3 man/man3/seccomp_syscall_resolve_name.3 man/man3/seccomp_syscall_resolve_name_arch.3 man/man3/seccomp_syscall_resolve_name_rewrite.3 man/man3/seccomp_syscall_resolve_num_arch.3 man/man3/seccomp_version.3 '/usr/local/share/man/man3'
 /bin/mkdir -p '/usr/local/lib/pkgconfig'
 /usr/bin/install -c -m 644 libseccomp.pc '/usr/local/lib/pkgconfig'
+ ldconfig
+ rm -rf /tmp/tmp.OsaU2C0NRz
error: pathspec 'v1.0.0-rc92.0.20201203030121-bd013b6e4d13' did not match any file(s) known to git
Error: Process completed with exit code 1.

More fun; looks like something is expecting the some path v1.0.0-rc92.0.20201203030121-bd013b6e4d13;
Making install in doc
 /bin/mkdir -p '/usr/local/share/man/man1'
 /usr/bin/install -c -m 644 man/man1/scmp_sys_resolver.1 '/usr/local/share/man/man1'
 /bin/mkdir -p '/usr/local/share/man/man3'
 /usr/bin/install -c -m 644 man/man3/seccomp_arch_add.3 man/man3/seccomp_arch_exist.3 man/man3/seccomp_arch_native.3 man/man3/seccomp_arch_remove.3 man/man3/seccomp_arch_resolve_name.3 man/man3/seccomp_attr_get.3 man/man3/seccomp_attr_set.3 man/man3/seccomp_export_bpf.3 man/man3/seccomp_export_pfc.3 man/man3/seccomp_init.3 man/man3/seccomp_load.3 man/man3/seccomp_merge.3 man/man3/seccomp_release.3 man/man3/seccomp_reset.3 man/man3/seccomp_rule_add.3 man/man3/seccomp_rule_add_array.3 man/man3/seccomp_rule_add_exact.3 man/man3/seccomp_rule_add_exact_array.3 man/man3/seccomp_syscall_priority.3 man/man3/seccomp_syscall_resolve_name.3 man/man3/seccomp_syscall_resolve_name_arch.3 man/man3/seccomp_syscall_resolve_name_rewrite.3 man/man3/seccomp_syscall_resolve_num_arch.3 man/man3/seccomp_version.3 '/usr/local/share/man/man3'
 /bin/mkdir -p '/usr/local/lib/pkgconfig'
 /usr/bin/install -c -m 644 libseccomp.pc '/usr/local/lib/pkgconfig'
+ ldconfig
+ rm -rf /tmp/tmp.OsaU2C0NRz
error: pathspec 'v1.0.0-rc92.0.20201203030121-bd013b6e4d13' did not match any file(s) known to git
Error: Process completed with exit code 1.

Probably https://github.com/containerd/containerd/blob/643bb9b66d55f1e0dcc4d112d9ad689fb34cb5d1/script/setup/install-runc#L24-L29
Which expects the reference in go.mod to be either a tag or a commit, but doesn't support the "pseudo versions"

Probably https://github.com/containerd/containerd/blob/643bb9b66d55f1e0dcc4d112d9ad689fb34cb5d1/script/setup/install-runc#L24-L29
Which expects the reference in go.mod to be either a tag or a commit, but doesn't support the "pseudo versions"

I think we should now decouple the runc binary installer from go.mod.
go.mod can have an untagged version of libcontainer.

I think we should now decouple the runc binary installer from go.mod.
go.mod can have an untagged version of libcontainer.

go.mod can have an untagged version of libcontainer.

Perhaps that's ok "between releases", but ideally we'd have a tagged version when doing an actual release. Especially with go modules having a strong preference for released versions (if a dependency has releases).

go.mod can have an untagged version of libcontainer.

Perhaps that's ok "between releases", but ideally we'd have a tagged version when doing an actual release. Especially with go modules having a strong preference for released versions (if a dependency has releases).

After this; dependency on runc vendoring is reduced to;
$ tree vendor/github.com/opencontainers/runc/

vendor/github.com/opencontainers/runc/
├── LICENSE
├── NOTICE
└── libcontainer
    ├── devices
    │   ├── device.go
    │   ├── device_unix.go
    │   ├── device_windows.go
    │   └── devices.go
    └── user
        ├── MAINTAINERS
        ├── lookup.go
        ├── lookup_unix.go
        ├── lookup_windows.go
        └── user.go

3 directories, 11 files

almost passed :( test timeout as it just didn't finish by 10m inside the Fedora-in-vagrant-on-macOS 🤓

Does it need a longer timeout? Don't think we can restart only a single job, correct?

/retest

LGTM
the failed CI job seems be because of a bad download
2021-02-04T17:00:59.5843050Z     default: Downloading: https://vagrantcloud.com/fedora/boxes/33-cloud-base/versions/33.20201019.0/providers/virtualbox.box
2021-02-04T17:00:59.6362490Z
2021-02-04T17:00:59.7170450Z Progress: 0% (Rate: 0*/s, Estimated time remaining: --:--:--)
2021-02-04T17:00:59.7172020Z Download redirected to host: download.fedoraproject.org
2021-02-04T17:00:59.7708170Z
2021-02-04T17:00:59.9694180Z Progress: 100% (Rate: 1766/s, Estimated time remaining: --:--:--)
2021-02-04T17:01:00.0522640Z Progress: 0% (Rate: 0/s, Estimated time remaining: --:--:--)
2021-02-04T17:01:00.0713810Z An error occurred while downloading the remote file. The error
2021-02-04T17:01:00.0714940Z message, if any, is reproduced below. Please fix this error and try
2021-02-04T17:01:00.0715840Z again.
2021-02-04T17:01:00.0716320Z
2021-02-04T17:01:00.0716840Z The requested URL returned error: 404 Not Found

ah, yes, the runc maintainers also ran into intermittent 404's for fedora; https://github.com/opencontainers/runc/pull/2787

opened https://github.com/containerd/containerd/pull/4999 to add the same hack


Bsweger/switch hubverse aws sync to rclone

Resolves https://github.com/Infectious-Disease-Modeling-Hubs/hubverse-cloud/issues/36
See individual commit messages for additional details:

bump actions/checkout version
swap the utility we use for S3 syncing

Output of an example run that shows syncing the contents of a new model-auxiliary folder but ignoring content that has already been synced to S3 (e.g., model-output)
Run hub_directories=(
2024/03/12 19:29:57 INFO  : data.csv: Copied (new)
2024/03/12 19:29:57 INFO  : data.txt: Copied (new)
2024/03/12 19:29:57 INFO  :     2.654 KiB / 2.654 KiB, 100%, 0 B/s, ETA -
2024/03/12 19:29:57 INFO  : There was nothing to transfer
2024/03/12 19:29:57 INFO  :           0 B / 0 B, -, 0 B/s, ETA -
2024/03/12 19:29:58 INFO  : There was nothing to transfer
2024/03/12 19:29:58 INFO  :           0 B / 0 B, -, 0 B/s, ETA -
2024/03/12 19:29:58 INFO  : There was nothing to transfer
2024/03/12 19:29:58 INFO  :           0 B / 0 B, -, 0 B/s, ETA -
2024/03/12 19:29:59 INFO  : There was nothing to transfer
2024/03/12 19:29:59 INFO  :           0 B / 0 B, -, 0 B/s, ETA -

Output of an example run that includes a file rename and a file delete.
2024/03/12 19:38:17 INFO  : cool-data.csv: Copied (new)
2024/03/12 19:38:17 INFO  : data.csv: Deleted
2024/03/12 19:38:17 INFO  : data.txt: Deleted
2024/03/12 19:38:17 INFO  :     2.630 KiB / 2.630 KiB, 100%, 0 B/s, ETA -
2024/03/12 19:38:17 INFO  : There was nothing to transfer
2024/03/12 19:38:17 INFO  :           0 B / 0 B, -, 0 B/s, ETA -
2024/03/12 19:38:17 INFO  : There was nothing to transfer
2024/03/12 19:38:17 INFO  :           0 B / 0 B, -, 0 B/s, ETA -
2024/03/12 19:38:18 INFO  : There was nothing to transfer
2024/03/12 19:38:18 INFO  :           0 B / 0 B, -, 0 B/s, ETA -
2024/03/12 19:38:18 INFO  : There was nothing to transfer
2024/03/12 19:38:18 INFO  :           0 B / 0 B, -, 0 B/s, ETA -

@LucieContamin Thanks for the 👀 , much appreciated. I'd love to make the cloud development more of a collaboration. Anna is off the project for a few weeks, and Matt is out this week, so if you wanna zoom on any of this, please let me know!


Location in publish Album.

we can't add location when posting an album.
is there any way to do that?
await ig.publish.album({
    items: arr,
    caption: 'Test', // nice caption (optional)
    // location: doesn't support
  });

This doesn't appear to be currently supported. The call to publish doesn't have enough parameters to handle passing in a location. Additionally the configureSidecar function doesn't have support for processing and configuring the location.

This is fixed in the latest version.


Accessibility Review

Website URL
https://jayantgoel001.github.io/
GitHub Repo URL
https://github.com/JayantGoel001/JayantGoel001.github.io

@emmalearnscode

Hello! Thank you so much for requesting a review. I am hoping to get some time to look at your site properly tomorrow.

Hello! Thank you so much for requesting a review. I am hoping to get some time to look at your site properly tomorrow.

Hi @emmalearnscode,
Ya sure No Prob😃😄


Check grouped buttons

Need styling
some comments are left

quick note:

can we have severe be the red-like color?
moderate yellow
blue-like color

Thoughts?

yes to more severe more red

something like gradient?

yes but diverging

Looks good for now, if we need to make tweaks once V1 is up and running we can always come back.
Merging changes

Amazing work @liadove !!!

Nice!!


How do we send this data to external cloud log

Thank you for this package. How accurate these data are and how to send them to a cloud server?

Sorry, you'll have to be a bit more specific as I don't understand your question


longest_repeating_subsequence.cpp

Issue Fixed : #495
Implemented the code of Longest repeating subsequence in C++
[x] - Code is well indented.
[x] - Code is well commented.
[x] - Code is not plagarized.
[x] - Code is placed in the proper folder.
[x] - The readme.md of the algorithm folder is updated with the code link.

Please review the code @tarun26091999 @plazzy99 @rudrakshi99 .

Hello @tarun26091999 can you tell me which conflict do I need to resolve?


Y-Axes labels are not checking for null or undefined

Returning null from the userCallback on the yAxes ticks is displaying a tick with the label "null".

Definitely a bug. Returning null or undefined should hide the grid line.

Fixed in #1645


Changing guardmode to home (value 1) doesn't trigger a push notification

When changing guard mode on station to 1 no push notification is generated. 0, 3 and 63 all generate push notifications. Don't know if it is guardModeChanged or currentModeChanged that prevents the call to emit of guard mode.
this.emit("guard mode", this, message.station_guard_mode, message.station_current_mode);
https://github.com/bropat/eufy-security-client/blob/master/src/http/station.ts#L288-L290
Log:
this.eufyStation.setGuardMode(63);
{"name":"eufyLog","hostname":"raspberrypi","pid":14352,"level":30,"msg":"Alarm mode for station T8010P2320281428 changed to: DISARMED","time":"2021-06-10T21:07:02.384Z","v":0}
{"name":"eufyLog","hostname":"raspberrypi","pid":14352,"level":30,"msg":"Received push notification for changing guard mode { guard_mode: 63, current_mode: 63, stationSN: 'T8010P2320281428' }","time":"2021-06-10T21:07:03.012Z","v":0}
this.eufyStation.setGuardMode(1);
{"name":"eufyLog","hostname":"raspberrypi","pid":14352,"level":30,"msg":"Alarm mode for station T8010P2320281428 changed to: HOME","time":"2021-06-10T21:07:03.726Z","v":0}
{"name":"eufyLog","hostname":"raspberrypi","pid":14352,"level":30,"msg":"Received push notification for changing guard mode { guard_mode: 1, current_mode: 1, stationSN: 'T8010P2320281428' }","time":"2021-06-10T21:07:04.091Z","v":0}
Version 0.8.3

@schliemann
I need the full log of the test to understand the cause, because the guard mode is changed not only by the push notification, but also via the P2P connection (fastest!) or HTTP. Let's say there are several ways to get a change (push, p2p, http) and only the first one leads to an emit.
There is another point in the code that performs the emit and that is here.

Ok. I understand. I'll try to add some more logging and do some more debugging.

@bropat Do you have any recommendations on how to retrieve the full log.  But couldn't get my fork up and running in the homebridge plugin.

@schliemann
Yes, you can simply pass to the constructor of the class EufySecurity a "log" object that supports the following methods:
trace: (_message?: any, ..._optionalParams: any[]) => {},
debug: (_message?: any, ..._optionalParams: any[]) => {},
info: (_message?: any, ..._optionalParams: any[]) => {},
warn: (_message?: any, ..._optionalParams: any[]) => {},
error: (_message?: any, ..._optionalParams: any[]) => {},

What I need then is that you repeat the error and send me the complete logs in debug level (you can find my mail here).

You can disregard the email I sent. I just realized the loggers loglevel was info. Might be missing entries. I'll send a new dump tonight.

I have sent a new email with the relevant log file attached.

Sorry, but I have very little time at the moment and therefore can not tell you when I can answer you. :(

It's perfectly fine. I understand.

Please test the new version 0.9.0.
Now there are 2 separate events "guard mode" and "current mode".

Works. Awesome job, @bropat 👏

Maybe closed this prematurely. Will do some more testing.

Any news about it?

@bropat I will try and re-test this.

@schliemann
Do you have any news on this?

@bropat I'm on vacation next week. Will try and verify by then.


feat(connector): add refineLater

Summary

Result

I created this branch as a stepping stone for https://github.com/algolia/instantsearch.js/pull/4446
and I guess we can use helper directly there.
So I'll close this. If we need this functionality somewhere else, then we can bring this back.


Module 3: Lab 12: Service Endpoints and Securing Storage

Module   3: Lab 12: Service Endpoints and Securing Storage
Ex 1   Task 3 Step 11: Need to click Add to add the second OutBound Rule.
Ex 1   Task 5 Step 2: "On the Virtual machines blade, click + Add." should   be "On the Virtual machines blade, click + Add, and then select Virtual   machine
Ex 1   Task 5 Step 3: "Windows Server 2019 Datacenter" should be   "	Windows Server 2019 Datacenter - Gen 1"
Ex 1   Task 5 Step 6: Boot diagnostics set to "Enable with managed storage   account (Recommended)"
Ex 1   Task 5 Step 8: "On the Virtual machines blade, click + Add." should   be "On the Virtual machines blade, click + Add, and then select Virtual   machine
Ex 1   Task 5 Step 9: "Windows Server 2019 Datacenter" should be   "	Windows Server 2019 Datacenter - Gen 1"
Ex 1   Task 5 Step 12: Boot diagnostics set to "Enable with managed storage   account (Recommended)"

updated


Could not functions start on node v4.x

Description
Initially my machine had Node v4.x, functions-emulator installed successfully but when I ran functions start it couldn't parse the start script it seems.
Once I installed node v6.x, I had to delete the previous folder in /usr/lib/node_modules/@google-cloud (not sure why NPM uninstall didn't work, but had to manually remove it) was able to get it functions start to run without errors.
Expected Behavior
functions-emulator shouldn't have been successfully installed if Node v6.x is required.
Actual Behavior
functions-emulator successfully installed but functions start couldn't start, had parse errors. (I think relating to ecmascript-6 style code? sorry I closed the terminal.)
Steps to reproduce

node --version should be v4.x
npm install -g @google-cloud/functions-emulator
functions start

Thanks!

The Emulator's package.json file contains the proper "engines" clause, i.e.
"engines": {
  "node": ">=6.9.1"
},

so npm should have printed a warning printed when the Emulator was installed with Node v4. According to npm "engines" docs:

Unless the user has set the engine-strict config flag, [the engines] field is advisory only and will produce warnings when your package is installed as a dependency.

So it sounds like if one wants the installation itself to fail when using an older Node.js version, one would need to set npm's engine-strict flag to true.


create rdma parent interface for vf device

Signed-off-by: Periyasamy Palanisamy<EMAIL_ADDRESS>

/cc @JanScheurich

@pperiyasamy Is this PR ready to merge?

yes @denis-tingaikin , this PR can go in now.


Time Series Graph with other non-main druid table column generates error

Im trying to create a Time Series V2 Graph on Superset based on a time column that is not the main that Druid works with. So I create a Dataset based on a Select with a CAST from a VARCHAR field to TIMESTAMP. The query works well, the Graph form seems to be working but when I try to update results, it appears an error.
How to reproduce the bug

Create a druid connection
Create a druid dataset from a table
Open SQLEditor
Create a query that uses other field (varchar field with timestamp) and cast it to timestamp
Run the query and check that works.
Press on Create Chart when query has finished
Check that no time field is set on Time Column, and the CASTed field is still a Varchar field.
Try to cast inside the CustomSQL tab in Time Column and CAST it again.
See error

Expected results
Superset should use the new Time Field for the ghrap.
Actual results
No time field is found despite having configured it. And If you try to force the cast inside the Time Column's CustomSQL it shows an error.
Screenshots

Environment
(please complete the following information):

browser type and version: Chrome 109.0.5414.120 (Build oficial) (64 bits)
superset version: superset version: 2.0.0
python version: python --version 3.8.12
druid version: 25.0.0

Hmm... as you can see by the abc label next to the __time column, Superset is not picking this up as a Temporal column. Can you try editing the dataset (via the three dots next to the table name in the left column) and explicitly marking that __time column as "temporal"? That might kick it into gear, but I'm not sure why it didn't pick that up automatically.

it worked well, thank you!


updated main.tf - with needs to generate values for public_ip_name an…

…d vpn_ip_configuration_name
Signed-off-by: sunmeher<EMAIL_ADDRESS>

@rich-ehrhardt we have integrate the VPN Gateway related subnet creation with this module. please validate and update.

@rich-ehrhardt updated the README.md file. please check and update. Thank you


pull request for e2e test at 01/31/2017 04:45:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


repository syncing and updating.

this will require some changes to the mPCC database.
One thought is to add a "Managed" boolean to the repository, this will override the Repo URL package to point to the mPCC, and enable the field(s) to take in the URL to sync from.  Jobservers will do the pull of the repository, sync to other jobservers, and the mPCC will do URL redirection to the jobserver(s) that host the repo.
An action/button/bool should also be added to perform a repo update from the source.  A crontab entry can be added to set this bit to enable scheduled updating.

repos that are managed will also need a "hosted_by" array field that links to jobservers

Added repo-update to #39  to template how it's done based on OS Type.

This is closed as of 244b21007f04e1ae92a4f85781596ac53c8cadf4


Erdtree Avatar in Caelid

I've done Caelid and I haven't met Erdtree Avatar listed in the Bosses section of the checklist. It is also not mentioned in the wiki for this area.
It might be that this boss was swapped for a second Putrid Avatar instead (issue #9)

On a related note, I think Isolated Divine Tower in Greyoll's Dragonbarrow was split in two: Divine Tower of Caelid: Center and Basement (wiki also has 11 locations listed plus Deep Siofra Well, which should be in Caelid)

So talking about this erdtree avatar issue specially:
This was once called Erdtree avatar but has since been renamed.
Wiki location still shows that here

Here is a youtube video with the previous name:
https://www.youtube.com/watch?v=7RqpYe852-A
Here is my current in game screenshot of the new name:

So I will update that

On a related note, I think Isolated Divine Tower in Greyoll's Dragonbarrow was split in two: Divine Tower of Caelid: Center and Basement (wiki also has 11 locations listed plus Deep Siofra Well, which should be in Caelid)

I don't believe this is related to this issue.
Isolated Divine Tower grace is still in the game https://eldenring.wiki.fextralife.com/Isolated+Divine+Tower
It is not really in caelid nor other places it's just in the middle of the map. Is it really the best to put it in Caelid graces? Probably not. I don't really have any good place to put it and got this data from the wiki itself. You can find the started discussion on which region is the best to place something in this issue here https://github.com/Gobluebro/Elden-Ring-Checklist/issues/8

Removed Erdtree avatar in this commit.
The correct name was already in caelid bosses listed as Putrid Avatar (West Caelid)


 I get to step 14.iv and I get the following errors (I've replace my actual domain here with "mysite"):
Any pointers I could try? Thanks.
/work # ansible-playbook -i inventory/hosts setup.yml --tags=install-all,ensure-matrix-users-created,start
PLAY [Set up a Matrix server] *************************************************************************************************************************************************************************************************************
TASK [Gathering Facts] ********************************************************************************************************************************************************************************************************************fatal: [matrix.mysite.site]: UNREACHABLE! => changed=false
msg: 'Failed to create temporary directory. In some cases, you may have been able to authenticate and did not have permissions on the target directory. Consider changing the remote tmp path in ansible.cfg to a path rooted in "/tmp",
for more error information use -vvv. Failed command was: ( umask 77 && mkdir -p "echo ~/.ansible/tmp"&& mkdir "echo ~/.ansible/tmp/ansible-tmp-1689276785.7256322-222-77612453783401" && echo ansible-tmp-1689276785.7256322-222-77612453783401="echo ~/.ansible/tmp/ansible-tmp-1689276785.7256322-222-77612453783401" ), exited with result 1'
unreachable: true
PLAY RECAP ********************************************************************************************************************************************************************************************************************************matrix.mysite.site       : ok=0    changed=0    unreachable=1    failed=0    skipped=0    rescued=0    ignored=0

You basically have the same issues as i have: https://github.com/beeper/self-host/issues/21
Nobody responding. I think the self-hosting project (read: support for a distributed environment) was born dead.

are you using DO droplet or not?

I am using a DO droplet.

I am not. I am running this on a physical server in my house.

Same issue, also receiving this error. Hosting locally, not digital ocean.

If you're hosting on the same machine as the domain is pointing to, add this into inventory/hosts
ansible_connection=local

From the self-hosted install, step 11, 'ansible_connection=local' is in 'inventory/hosts'. Also in 'inventory/hosts' is 'ansible_connection=community.docker.nsenter'. Could these be conflicting? I removed 'ansible_connection=community.docker.nsenter' from 'inventory/hosts' and got several different errors:
TASK [galaxy/geerlingguy.docker : Ensure handlers are notified now to avoid firewall conflicts.] ******************************************************************************************************************************************[WARNING]: flush_handlers task does not support when conditional
TASK [galaxy/com.devture.ansible.role.timesync : Ensure timesyncing daemon is started and autoruns] ***************************************************************************************************************************************fatal: [matrix.dzw6fjyd.site]: FAILED! => changed=false    msg: 'Could not find the requested service ntp: '

You need an NTP server somewhere to install this. This has nothing to do with local or remote SSH for ansible.
Guys, le's stick to the original issue which is a permissions issue inside the container.

Use the docker command provided in here opposed to the one in the readme

Can confirm. Running that docker command instead of the one in the readme at step 13 solves the permissions issue for me.

Probably. But do you really want to run a privileged container and give it access to the internet? Maybe i'm being paranoid here, but....

Can confirm. Running that docker command instead of the one in the readme at step 13 solves the permissions issue for me.

In addition to running the container as privileged, I also had to make sure that the SSH Key (id_rsa in the command above) was without a passphrase. From the ansible documentation:

Ansible does not expose a channel to allow communication between the user and the ssh process to accept a password manually to decrypt an ssh key when using the ssh connection plugin (which is the default). The use of ssh-agent is highly recommended.

For some reason ssh-agent was not working nicely with my Mac when loading the key into the docker container. Combining a privileged container with an unlocked passphrase is very scary from a perspective of security, but at least it seems to work...


feat: better readme docs

This PR adds readme documentation to auth, storage and network packages.
For details refer to issue #123
Type of Change

[ ] New feature
[ ] Bug fix
[x] API reference/documentation update
[ ] Other

Does this introduce a breaking change?
No
Are documentation updates required?
N/A
Testing information
Checklist

[x] Code is commented where needed
[x] Unit test coverage for new or modified code paths
[x] npm run test passes
[x] Changelog is updated
[x] Tag 1 of @yknl or @zone117x for review

👍🏼

@yknl @agraebe Let's merge this in? I'm going to include links to these new READMEs from the "Build apps" navigation with https://github.com/blockstack/docs/issues/926

@yknl @agraebe Let's merge this in? I'm going to include links to these new READMEs from the "Build apps" navigation with https://github.com/blockstack/docs/issues/926

@yknl please do the honors if you think this is good to go from your end as well

@yknl please do the honors if you think this is good to go from your end as well


Localized formatting

There should be a way to localize formatting for dates.
A few people have submitted PRs to my formatting library over the years. (I don't plan to update it to 0.19 - your token approach is better.) The results are here. You don't necessarily have to add all the localizations, but having a way for others to do them is useful. I think you hit upon a good pattern with the record of functions in the Relative module.

Hey Max,
I really like that idea!
It would be pretty silly to only support the english strings I've defined.
Let me update my DateFormat module!
Thanks,
Ryan

Awesome!
I literally had someone add a PR for Greek today.

Hey Max!
Just pushed up a 1.1.0 release to elm-package.
Check out this example if you want to see how it works:
https://github.com/ryannhg/date-format/blob/master/examples/FormatWithOptions.elm
Thanks for your suggestion, it greatly improved the overall flexibility of the package!
Thanks again,
Ryan


Delete unused CI configs

2 files

I will keep this files for now, maybe I active it...

Okay!
Remember: git it is unlimited 🗑


Refactor grouping and aggregation

PR for Issue 453
It might be easier to review commit by commit
I think that the main contribution is the methodDaru::DateFrame#group_by_and_aggregate, here are some benchmarks: (nowhere extensive)
1/ comparaison of groupping + aggregation - on my PR
require 'benchmark'

Benchmark.bm(8) do |b|
  b.report("gr + sum") { 1_000.times { spending_df.group_by(:year).sum } }
  b.report("gr + agg") { 1_000.times { spending_df.group_by(:year).aggregate(spending: :sum, nb_spending: :sum) } }
  b.report("gr_agg")   { 1_000.times { spending_df.group_by_and_aggregate(:year, spending: :sum, nb_spending: :sum) } }
end

#                user     system      total        real
# gr + sum   0.680000   0.000000   0.680000 (  0.673169)
# gr + agg   0.870000   0.000000   0.870000 (  0.871802)
# gr_agg     0.370000   0.000000   0.370000 (  0.370445)

2/ I end up refactoring the groupping method (will be a bit slower)
## test using `10_000.times { spending_df.group_by(:year).sum }` on first and last commit
#               user     system      total        real
# before    6.270000   0.000000   6.270000 (  6.302841)
# after     6.620000   0.000000   6.620000 (  6.620660)

=========================
Please, let me know if there is anything I can improve.

Thanks! And sorry for long wait.

There was a method access_row_tuples_by_indexs in dataframe.rb which has been recently removed from daru. This method has been used extensively in daru-view, so it is creating problems in the visualization with GoogleCharts and HighCharts when data is provided as Daru::DataFrame. @zverok @paisible-wanderer is there any alternative method provided or should we add it again?

@zverok no problem, I am happy that it was finally merged!


ephemeral instances should have their storage backend tuned for speed rather than reliability

It'd be nice for ephemeral instances to have their backing ZFS configured with sync=disabled.

This would potentially be handy to speed up CI runs using guest VMs.

I don't actually think that this is a good idea. We shouldn't assume that ephemeral means we're happy to lose data.
What ephemeral means is that when the instance shuts itself down or when the user stops it, that we want it to go away.
If the machine crashes for some reason, we will bring the ephemeral instances back online on boot, we won't delete them.
Making this change would therefore expose us to data loss.


[ci] Migrate to common CI script

Change .travis.yml to run common CI test using new common checks script in openwisp-utils. Also remove unnecessary script and update openwisp-utils.

Coverage decreased (-0.2%) to 98.68% when pulling 85510348e3cae4c6a5384f0ab88a21bd547861f6 on marfgold1:common-ci into 80a3cfb1e9997bca183106536a131eb5952da4ca on openwisp:master.