Generation is broken when all class names are not unique

I use a code generator to generate classes to represent my sql schema.
The generator generates code like this:
namespace dbo // schema name
{
    partial class Customers // table name
    {
           partial class RowData
           {
               // property for each column
           }

    }

    partial class Orders// table name
    {
           partial class RowData
           {
               // property for each column
           }

    }
}

In another file I put your attribute on my RowData types:
    [GenerateDataReaderMapper]
    public partial class RowData;

But since I have multiple types with the same name, your generator fails to do anything.

Same issue here. Would be great to be able to specify either a namespace or a custom extension method name. Preferably both.
Namespace
[GenerateDataReaderMapper(NameSpace = "MyProject.Customers")]
public sealed class Customer;

[GenerateDataReaderMapper(NameSpace = "MyProject.Orders")]
public sealed class Customer;

Method name
[GenerateDataReaderMapper(MethodName = "ToCustomer")]
public sealed class Customer;

...

reader.ToCustomer();

[GenerateDataReaderMapper(MethodName = "ToOrderCustomer")]
public sealed class Customer;

...

reader.ToOrderCustomer();


Child component props aren't updated after call to setState in async handler

I'm having trouble testing an event handler that does an async operation followed by a setState. Even when I wait for the setState to complete by awaiting a promise resolved from inside its callback, the component doesn't seem to fully re-render. What's weird is that I can confirm that the parent state is updated but props on a sub component that rely on that state are not. Even stranger, text that comes from those props inside the sub component is updated like normal.  I created a small script to reproduce the behavior. Note that after the onClick handler and setState finish this child component props are still { id: 'childDiv', counter: 1 }. Only after a forced update do they change.
import { mount } from 'enzyme';
import React, { Component } from 'react';
import Promise from 'bluebird';

class TestComponent extends Component {

  state = {
    counter: 1,
  };

  handleClick = async () => {
    await Promise.delay(100);
    return await new Promise((resolve) => {
      this.setState({ counter: 2 }, () => {
        resolve();
      });
    });
  };

  render() {
    return

      <TestSubComponent id={'childDiv'} counter={this.state.counter}/>

;
  }
}

class TestSubComponent extends Component {

  render() {
    return
{this.props.counter}
;
  }
}

describe('foo', () => {

  it('should foo', async () => {
    const component = mount(<TestComponent/>);

    // prints {"counter":1}
    console.log('=== parent state before click');
    console.log(JSON.stringify(component.state()));

    // prints { id: 'childDiv', counter: 1 }
    console.log('=== sub component props before click');
    console.log(component.find(TestSubComponent).props());

    // prints 1
    console.log('=== sub component props before click');
    console.log(component.find(TestSubComponent).text());

    await component.find('#parentDiv').props().onClick();

    // prints {"counter":2}
    console.log('=== parent state after click');
    console.log(JSON.stringify(component.state()));

    // prints { id: 'childDiv', counter: 1 }
    console.log('=== sub component props after click');
    console.log(component.find(TestSubComponent).props());

    // prints 2
    console.log('=== sub component text after click');
    console.log(component.find(TestSubComponent).text());

    component.update();

    // prints { id: 'childDiv', counter: 2 }
    console.log('=== sub component props after update');
    console.log(component.find(TestSubComponent).props());

  });

});

Your environment
API

[x] shallow
[x] mount
[ ] render

Version

library
version

enzyme
3.2.0

react
16.3.1

react-dom
16.3.1

react-test-renderer
16.4.0

adapter (below)
enzyme-adapter-react-16: 1.1.1

Can you try updating to the latest version of enzyme? we've fixed a lot of bugs since v3.2 (also, you'll need to ensure react-test-renderer is also on 16.3 - the minor version must always exactly match that of react and react-dom. You may also want to use the 16.3 adapter instead of the generic 16 adapter, altho both should work the same.

Thanks for the response! I tried upgrading to enzyme 3.7.0, using enzyme-adapter-react-16.3 and react-test-renderer 16.3.2. Unfortunately, I'm still getting the same test results. I'll update the versions in my original description as well.

@ljharb I would love to help out with Enzyme's open source, How may i help out with this issue

@k3ithl1m submitting a PR with test cases that cover the OP's issue would be great :-)

@ljharb Based on my understanding. Because the functions used like find, text, mount, state and props, are all in ReactWrapper. I will be updating ReactWrapper-spec.jsx with the examples that the OP gave. Correct me if I'm understanding it wrongly.

@ljharb did you mean fixing the bug where calling .props() will also print the right value? So for that case, it would print out
// prints { id: 'childDiv', counter: 1 }     console.log('=== sub component props after click');     console.log(component.find(TestSubComponent).props());
I apologize for taking what you said by PR with test cases literally as PR with only test cases. haha

I mean, also including the fix is even better, but the test cases are the most important part.

Okay. Ill make one PR with just the test and then try to work on the fix. Does that sound good?

Can I also install the bluebird module so that its similar to what the OP did? or should I write my own Promise.delay module

definitely not bluebird; something like function delay(ms) { return new Promise((resolve) => { setTimeout(resolve, ms); }); } is sufficient


Read-only auth should disable some editor features

Non-write should make all object static by mqtt token.
Doesn't make sense to enable editor boxes when the publish message will be declined.

Where possible this has been added in #47


Revert "[BEAM-5978] Changing parallelism for wordcount to 1"

Reverts apache/beam#7174

Run Portable_Python PreCommit

Run Portable_Python PreCommit

Run Python PreCommit

Run Portable_Python PreCommit


Identify .j2 extension as jinja2

Ansible uses .j2 as the de-facto extension for Jinja templates.  This change identifies .j2 files as jinja.

thanks!


Flask-Admin behind Nginx

Hello,
I have two flask apps running behind nginx. One flask app is running on port 5000 and the other (api) is running on 5001. I have two location blocks in nginx.conf. The api-app is served on server_name/api.
I can access Flask-Admin on "server_name/api/admin" but the problem is that Flask-Admin looks for static files at "server_name/admin/static", but they are available at "server_name/api/admin/static".
Is there a simple way to prefix the static_url_path with "api"?

I see now that my issue is much the same as #1837

That's how per-blueprint static urls are routed in Flask.
There are a few ways to fix it:

You can add an nginx rule to map /admin/static/ to a static directory where flask-admin is installed
Try flask-collectstatic (I never tried it myself) that would allow you to have a single folder with all collected static files to be shared through nginx

Also, there's a similar discussion: https://stackoverflow.com/questions/41853436/flask-raises-404-for-blueprint-static-files-when-using-blueprint-static-route

Thank you @mrjoes for pointing me in the right direction!
Adding another location block to nginx fixed things.
I would also like to give you thumbs up for not mocking a newbie asking simple/silly questions :+1:


Add PK type rails 6.x support

Bug reported in https://github.com/thoughtbot/clearance/issues/923 .
The issue is something to do with the generator options mapping changed on Rails 6.x that make the PK type global settings.

Thank you very much for the PR @max2320. I will work to get this merged soon, things have been busy recently.

Thank you very much for the PR @max2320. I will work to get this merged soon, things have been busy recently.

Np, glad to hear that you like it.

Np, glad to hear that you like it.


Unable to use feed from serial port (ttyS0)

Hi Mikey,
I get this error when trying to use the ttyS0 serial port for AIS data:
pi@SmoothIsland:~ $ sudo docker run --rm -i -t -p 8888:8000 -e DATA_SOURCES=ais-serial://dev/ttyS0?baud=38400 mik3y/airdash yarn run v1.22.5 $ WEBPACK_TARGET=prod node server/server.js info: Server starting ... info: config: {"listenPort":8000} info: Starting server ... Error: Failed to add data source ais-serial://dev/ttyS0?baud=38400: Error: Unsupported protocol "ais-serial:"

Hey @yanniedog! Okay, this will probably take us a few iterations, but I think we can get this solved.
Your command looks right, the issue here is the docker image has not been updated. I'm checking why, and it looks like this is because the automatic build failed - so there's no fresher one for you. Let me see if I can get this fixed, it may take about 20-30 minutes. Stand by!

Hey @yanniedog! Okay, this will probably take us a few iterations, but I think we can get this solved.
Your command looks right, the issue here is the docker image has not been updated. I'm checking why, and it looks like this is because the automatic build failed - so there's no fresher one for you. Let me see if I can get this fixed, it may take about 20-30 minutes. Stand by!

@yanniedog okay, please try updating the docker image, and running again.
You can update the image with this command:
docker pull mik3y/airdash

@yanniedog okay, please try updating the docker image, and running again.
You can update the image with this command:
docker pull mik3y/airdash

@mik3y thanks very much.

Hm! That error suggests you might already have it (or something) running on port 8888.
Try a docker ps to see what's running. You should then be able to use docker stop or docker kill to shut down an existing version of airdash, if one is running.

Hm! That error suggests you might already have it (or something) running on port 8888.
Try a docker ps to see what's running. You should then be able to use docker stop or docker kill to shut down an existing version of airdash, if one is running.

Thanks mik3y, that fixed it.
Now, Docker loads correctly, but unfortunately it still doesn't show ships on the map:
pi@SmoothIsland:~ $ sudo docker ps
CONTAINER ID   IMAGE          COMMAND                  CREATED        STATUS        PORTS                    NAMES
98ed63ea25c2   7819a6390adf   "docker-entrypoint.s…"   10 hours ago   Up 10 hours   <IP_ADDRESS>:8888->8000/tcp   awesome_payne

pi@SmoothIsland:~ $ sudo docker stop 98ed63ea25c2
98ed63ea25c2

pi@SmoothIsland:~ $ sudo docker run --rm -i -t -p 8888:8000 -e DATA_SOURCES=ais-serial:///dev/ttyS0 mik3y/airdash
yarn run v1.22.5
$ WEBPACK_TARGET=prod node server/server.js
info: Server starting ...
info: config: {"listenPort":8000}
info: Starting server ...
info: 200 | GET /api/entities duration=99 for=undefined agent=Mozilla/5.0 (X11; Linux armv7l) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.197 Safari/537.36
info: 200 | GET /api/entities duration=4 for=undefined agent=Mozilla/5.0 (X11; Linux armv7l) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.197 Safari/537.36
info: 200 | GET /api/entities duration=4 for=undefined agent=Mozilla/5.0 (X11; Linux armv7l) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.197 Safari/537.36

Thanks mik3y, that fixed it.
Now, Docker loads correctly, but unfortunately it still doesn't show ships on the map:
pi@SmoothIsland:~ $ sudo docker ps
CONTAINER ID   IMAGE          COMMAND                  CREATED        STATUS        PORTS                    NAMES
98ed63ea25c2   7819a6390adf   "docker-entrypoint.s…"    Sorry I missed your comment here.
Could you try enabling debug logs? I am not sure if it will add what we need.. but it could.
Should be something like adding the following to the docker run command:
-e DEBUG='airdash:*'

Whoops! Sorry I missed your comment here.
Could you try enabling debug logs? I am not sure if it will add what we need.. but it could.
Should be something like adding the following to the docker run command:
-e DEBUG='airdash:*'

Hey @yanniedog - I thought of another issue that could be affecting this. Do you run some other software that reads from the AIS device at port /dev/ttyS0? I believe only one program can read from that at a time, so if something other than airdash is running, that could be the problem.

Hey @yanniedog - I thought of another issue that could be affecting this. Do you run some other software that reads from the AIS device at port /dev/ttyS0? I believe only one program can read from that at a time, so if something other than airdash is running, that could be the problem.

The issue I'm really, really struggling with is the basic Docker / Docker-compose commands. I don't know how to do the fundamental basics of Docker, and don't have any idea what I'm doing when it comes to docker/docker-compose.  Sorry about that - I remember there is indeed a bit of a learning curve. And probably doubly so when you aren't sure whether the software itself is buggy (as could definitely be the case here)..
Would you be up for letting me ssh in and look things over? I can probably troubleshoot things quickly that way. No worries if not - I understand not wanting to trust a random internet stranger with computer access :-)


Update wsps.py

Resolve a bug in loading site_list

The original bug is that if there is only one site, list_site would be loaded as a string instead of a list.


[FEATURE REQUEST] Cloud deployment scripts for managing storage infra via CI/CD

Overview
Thanks for this package! It would be wonderful if one could deploy an arena-compatible S3 data lake via a set of terraform (or other cloud orchestration) scripts & Github Actions. Please close if this isn't the best place to raise this feature request.

Hi thanks for the request.
This is a topic that needs improvement for all of our components and already very high in our priority list.
For now I have moved the issue to the documentation repo as this is a general documentation related issue.
BR


Encode too ?

Question: this is a decoder only ?
no plans to also encode ?

Hey @gedw99 !
Yes I plan on doing encoder also. It is going to take me a long time though :)
If I can find some help it might go faster. I put my high level goals in #9


Bug Report: NameError: name 'TestElectraForSequenceClassification' is not defined

Introduction

paddle version: 2.2.0
macos
python3.9
ide: vscode
test plugin: Python Test Explorer for Visual Studio Code

Error Message
Failed to import test module: transformers.gpt.test_modeling
Traceback (most recent call last):
  File "/home/wj-Mcat/miniconda3/envs/paddle/lib/python3.9/unittest/loader.py", line 436, in _find_test_path
    module = self._get_module_from_name(name)
  File "/home/wj-Mcat/miniconda3/envs/paddle/lib/python3.9/unittest/loader.py", line 377, in _get_module_from_name
    __import__(name)
  File "/home/wj-Mcat/projects/paddle/PaddleNLP/tests/transformers/gpt/test_modeling.py", line 84, in <module>
    class TestGPTForTokenClassification(TestElectraForSequenceClassification):
NameError: name 'TestElectraForSequenceClassification' is not defined

Reproduce
This is the abvious error which does't need to reproduce it: there is no TestElectraForSequenceClassification importing in tests/transformers/gpt/test_modeling.py file.  So it throws the NameError issue in testing panel.
Although there is no error been thrown at runtime, but it's also a linting error which should be fixed. How do you think about it ?

@wj-Mcat Thanks for your feedbacks!

Resolve by #1306


Consider non-standard feature. Re-purpose useless %n to print a string then free it from the heap

When attempting to print strings generated by application specific functions, there are some difficulties...
For example, consider a function which takes an integer 'seconds' and describes the duration as xxDxxHxxMxxS.

char* period2txt(char* buf, int seconds);

If this function is used for multiple arguments to a single prnf(), a separate buffer must be provided for each call, eg.

char buf1[20];
char buf2[20];
prnf("Fred is %s, Bob is %s\n", period2txt(buf1, freds_age), period2txt(buf2, bobs_age));

The above is ugly and error prone. Accidentally put buf1 in both, and it breaks. Undersize the buffer and it breaks.
Proposed solution
The standard printf() provides %n, which is normally used to write the number of characters printed so far, to an integer pointed to by the argument. I cannot think of a practical application of this, and I don't know why it exists.
As %n expects a pointer (int*) as an argument, this could actually point to a string in a heap allocation. prnf() could then free the allocation after printing it.
If our function period2txt() makes a heap allocation when buf is passed as NULL, then we could do this:
prnf("Fred is %n, Bob is %n\n", (int*)period2txt(NULL, freds_age), (int*)period2txt(NULL, bobs_age));
Or, if period2txt is only to be used as a prnf argument, it could be of the form:

int* prnf_n_period(int seconds);

This would allow usage as simple as:

prnf("Fred is %n, Bob is %n\n", prnf_n_period(freds_age), prnf_n_period(bobs_age));

There are some benefits to this.

Mixing up %s and %n will give you a compilation warning, as %s expects (char*) and %n expects(int*)
Misuse of prnf_n_period() for other purposes, such as using it in an assignment to a char* will also cause a compilation warning due to pointer types.

Unfortunately the compilers format checking will complain if we try to specify width or precision for %n. So no alignment by prnf for %n strings would be possible.
If anyone is out there.. please share your thoughts on this.

As nobody is really following or watching, I just did it.

** BEWARE **
If you are writing your own variadic function which directly calls vsnprnf(NULL, 0, fmt, va) for the purpose of measuring the length of the output, and then vsprnf() to actually print the output, a double free will occur on any %n argument, as you are effectively printing twice.

I cannot think of a practical application of this, and I don't know why it exists.

Why can't you? Are you saying people should just always break their printf() calls into multiple calls and use the return value? You could make that claim, but then - why have printf() at all, when you could just have functions for each of the format specifiers supported?

prnf() could then free the allocation after printing it.

A bit convoluted, but I suppose it could be useful in the scenario you described.

prnf("Fred is %n, Bob is %n\n", (int*)period2txt(NULL, freds_age), (int*)period2txt(NULL, bobs_age));

No no no, very confusing, bad idea. And why must you cannibalize %n? There are plenty of unused letters you could take instead.

Or, if period2txt is only to be used as a prnf argument, it could be of the form:

int* prnf_n_period(int seconds);

This is is silly. And who's to say period2txt() isn't used elsewhere?
What you should probably do is have %S behave like %s, but free the argument after use. Simple and reasonable.

Why can't you? Are you saying people should just always break their printf() calls into multiple calls and use the return value?

Have you ever used %n? what for? If text alignment is needed, it's normally done to a fixed position using the width specifier.
Why would you want to know the position of some variable mid-point in the string?

why have printf() at all, when you could just have functions for each of the format specifiers supported?

I actually did this for years. It turns into a hell of a lot of function calls, and a lot of typing. Also there are different modes of output to accommodate. So I had ascii_int32() for writing to char buffers, and then ascii_fifo_int32() for writing to streams.
By using a function as an argument for printf() you can then use the same function with snprintf() fptrprnf() etc..

No no no, very confusing, bad idea. And why must you cannibalize %n? There are plenty of unused letters you could take instead.

Unused letters will produce the compilation warning "unknown type specifier %". I want to make use of the compilers format+argument checking. The only other type which takes a pointer is %p, but I may want to implement %p to display a pointer in the future.

More confusion... and who's to say period2txt() isn't used elsewhere?

Well the author of prnf_n_period() would document this as being specifically for the purpose of providing arguments to prnf().
If someone were to misuse it in an assignment to a char*, this would produce a compilation warning (due to different pointer types). If there already exists a period2txt(), then prnf_n_period() could be a simple wrapper to return the result as an int*
I agree it is confusing to see a function prototype returning an int* though, especially when it produces text.

What you should probably do is have %S behave like %s, but free the argument after use. Simple and reasonable.

This would have been a better option. But unfortunately %S has already been cannibalized. On AVR it means to read the string from program memory instead of ram.

Unused letters will produce the compilation warning "unknown type specifier %". I want to make use of the compilers format+argument checking. The only other type which takes a pointer is %p, but I may want to implement %p to display a pointer in the future.

Well, it shouldn't necessary print a warning; or one could disable the warning - but it's better to have it than to expose a printf()-like functions with different semantics for an existing specifier.

%S has already been cannibalized. On AVR it means to read the string from program memory instead of ram.

So you're already getting a warning about it?
Anyway, I still believe you should an unused specifier, not %n.

Thanks for your input Eyaloz. I understand the danger now.
On one hand, I have the irritation of warnings.
On the other hand, I have the danger of someone expecting %n to work like the printf %n.
I can document it. And they will have to read the documentation to enable the feature in the first place. But then a future programmer working on the project may think "Hey I wonder if this prnf thing has %n?", try it out, compile without warning, then crash.... It may be a painful thing to diagnose.
If I were to use %h instead of %n, I would still like to avoid using (char*) for the argument type. The reason being that it's too easy to accidentally use %s instead of %h, and then a memory leak will occur. I'm 100% certain that I will make this mistake myself.
Alternatively, if I do stick with %n, I think I should at least produce a compiler warning alerting the developer to the differences between prnf and printf. At least that way I can have a single warning, instead of a warning for every instance of %h


fix: Angle brackets no longer cause malformed html reports

PointlessTypeBound inspection has angle brackets in the explanation. The HTML report tries to pretty format the code and parse it back to XML element which has dangling angle brackets causing failure.
Fixes https://github.com/scapegoat-scala/sbt-scapegoat/issues/175

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (9ce6a8c) 86.75% compared to head (c9405cc) 88.08%.

Files
Patch %
Lines

...a/com/sksamuel/scapegoat/io/HtmlReportWriter.scala
66.66%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #815      +/-   ##
==========================================
+ Coverage   86.75%   88.08%   +1.33%
==========================================
  Files         142      142
  Lines        1578     1578
  Branches      274      278       +4
==========================================
+ Hits         1369     1390      +21
+ Misses        209      188      -21

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

@mccartney, would you release a new patch version please?

2.1.4 is on its way.


:memo: add navigation in core package

https://github.com/alakarteio/k-ramel/blob/doc/navigation/packages/k-ramel/doc/LISTENERS.md
The doc will be under this two navigation

Pull Request Test Coverage Report for Build 1517

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.218%

Totals

Change from base Build 1508:
0.0%

Covered Lines:
313

Relevant Lines:
322

💛  - Coveralls


Automate the release

sbt has good plugins for doing releases to Nexus - note ASF has its own Nexus instance, we don't use Sonatype. https://repository.apache.org/
also need to gpg sign and generate SHA256/SHA512 digests for the source and binary distributions (#78)

Since the core pekko build is really complicate, we could use https://github.com/apache/incubator-pekko/issues/92 as a test bed for creating an automated release (its going to be needed at some point anyways).
In regards to the plugins to automate a release, the most ideal would be to use sbt-ci-release however it does seem to be tied to sonatype, the nice thing about sbt-ci-release is it completely automates the process, a release is triggered just be pushing a git tag but this may be too modern for Apache and might go against some of their established processes. Another option is sbt-release which is a bit more manual/old-school however it allows you to customize the release process as you want and then you would just do an sbt release to trigger a release.

I use sbt-typelevel-ci-release - which appears to be a more active copy of sbt-ci-release.

Afaik its not that sbt-typelevel-ci-release is more maintained but rather it contains typelevel'isms related to their org's standards for releasing software.
In any case I was talking casually about these sbt release plugins to other Apache people and there appeared to be some hesitation. On the surface sbt-release seems to be closer to the Apache way of releasing things and most importantly isn't tied to sonatype

This PR is related to use sbt-ci-release to do release management https://github.com/apache/incubator-pekko/pull/129 . In summary it may be possible to use sbt-ci-release by just changing the sonatype host to repository.apache.org, hopefully the Apache Nexus repo is close enough to OS sonatype that they accept the same way of publishing.

we don't use Sonatype

Maybe we should clarify what a release is and what the desired outcome of doing a release is. How do artifacts end up on Maven Central if not going through Sonatype? Let's start listing all the steps necessary to do different kinds of releases (snapshots, RCs, and GA) and only then start about automating any of it. I imagine this is at least partly about what Apache considers a release? If yes, then we should clarify what it entails.
I would consider sbt-ci-release to be a tool to work with sonatype. If we don't go through sonatype (or a process very similar to sonatype with staging repos etc), we probably should not use it. Afaik, the main reason it was created was that sonatype (Nexus) needs some manual handholding to finally publish something to Maven Central.
AFAIK sbt-release is a plugin to organize a release-process that has multiple steps but is more abstract than publishing to a repository.
If this is just about pushing artifacts to a maven-compatible repository, we might not need any plugins for just that.

One of the nice things about using sbt-release instead of sbt-ci-release is that as you just pointed out, you can customize it to do whatever you want. For example even though we publish generated library jars to Apache's Nexus repo, this is not considered an official. An official release involves copying the sources over using rsync, so we can theoritically use sbt-publish-rsync or fork it for Apache's needs, and then we can can completely automate publishing both to Nexus and making an official Apache release.
For now I am just trying to get snapshots working, so I don't see a problem to see if we can adjust sbt-ci-release to work with Apache's Nexus repo however I am still personally leaning to using sbt-release for reasons stated earlier.

I created https://github.com/apache/incubator-pekko/issues/130 to try to collect an overview over all the steps that are needed. This ticket would be a subticket of the complete process.

@jrudolph repository.apache.org artifacts get synched to Maven Central - but I think only when you do full releases. Snapshots and staged (pre-release) artifacts are probably only accessible by adding an sbt resolver that has the repository.apache.org url set up.

@jrudolph repository.apache.org artifacts get synched to Maven Central - but I think only when you do full releases. Snapshots and staged (pre-release) artifacts are probably only accessible by adding an sbt resolver that has the repository.apache.org url set up.

True but its also nothing special, at least for sbt even the OS Sonatype snapshot repository needs to be manually added as a resolver (i.e. its not there by default).

Yes, and that's totally fine and also pretty much wanted (also according to https://infra.apache.org/release-distribution.html#unreleased which explicitly require "must not be distributed through channels which encourage use by anyone outside the project development community").


Git commit missing from contributor guide

In the 9th step of 'create and push a branch' here ,
https://docs.docker.com/project/set-up-git/ , while demosntrating git commit, it says
git -s -m "Making a dry run test."
which should be
git commit -s -m "Making a dry run test."
I would like to work on it, if triaged!

Looks like it was closed 2 days ago! Closing it!


Automated cherry pick of #12045 upstream release 1.0

To resolve issue #11997 in 1.0.2

@brendandburns @roberthbailey

LGTM.

Cross-link #12045.

LGTM


Create full-stack-developer.md

Add the full stack JD (with today's edits) to repo.

let

lgtm 🚀


itscybertech.com

Domain
itscybertech.com
Version
0.0.0
What browsers are you seeing the problem on?
Firefox
What OS are you seeing the problem on?
No response
(Optional) Anything else?
for example:
https://tinyurl.com/yk3yh6jg
https://tinyurl.com/yh7d6w43
https://tinyurl.com/ygt32pta
https://tinyurl.com/yzj58uv3
(by clicking on these links it is redirected to itscybertech.com)

Already bypassed.


[8.x] Adds type check for src/Illuminate/Auth/Middleware/Authenticate.php

Added type checks for method arguments where they're missing. Types for the parameters are taken from the method's doc.
This fixes the error produced when you add type check in your applications Authenticate middleware:
Declaration of App\Http\Middleware\Authenticate::redirectTo(Illuminate\Http\Request $request) should be compatible with Illuminate\Auth\Middleware\Authenticate::redirectTo($request)

Not doing this at the moment. Don't declare a type-hint when overwriting the method.


regex clean that parses time

What does this PR do?

What issue(s) does this change relate to?

Before submitting

[ ] Have you read the contributor guidelines?
[ ] Is this change a documentation change or typo fix? If so, skip the rest of this checklist.
[ ] Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
[ ] Did you update any related docs and document your change?
[ ] Did you update any related tests and add any new tests related to your change? (see testing)
[ ] Did you run the tests locally to make sure they pass?
[ ] Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

Cleaned regex that parses timestring in time.py. It will now support underscores (eg 4_000tok).

One general PR process thing, please include a complete description of the PR in the pr description, and a link to the JIRA item in the issues section of the pr description (e.g. "Closes [CO-XXXX](link to jira)").


 This seems wrong and I can't think of a reason why this would be desirable in the first place.
Darwin code uses the same clock family for both, so use CLOCK_MONOTONIC_COARSE on Linux as the approximate time source, and test for it during configure.

Amended; do we want to do the same in _dispatch_absolute_time() with a comment on top?

we probably should now that you point this out.

How does that look? Ship it?


Fix install instructions

Description
Our install docs were referring people to devtools/conda-envs/meta.yaml, which doesn't exist any more. We could either point them to psi4.yaml or basic.yaml. I think basic is better since it's more likely to be devs installing from source, and they'll know how to install psi4 afterwards if they need.
Todos
Notable points that this PR has either accomplished or will accomplish.

[x] Fixes #271

Status

[ ] Ready to go

The test failures are expected and are due to a different issue.


Release version 2.1.0

This is a component issue for 2.1.0.
Coming from opensearch-build#1818. Please follow the following checklist.
Please refer to the DATES in that post.
How to use this issue

This Component Release Issue
This issue captures the state of the OpenSearch release, on component/plugin level; its assignee is responsible for driving the release. Please contact them or @mention them on this issue for help.
Any release related work can be linked to this issue or added as comments to create visiblity into the release status.
Release Steps
There are several steps to the release process; these steps are completed as the whole component release and components that are behind present risk to the release. The component owner resolves the tasks in this issue and communicate with the overall release owner to make sure each component are moving along as expected.
Steps have completion dates for coordinating efforts between the components of a release; components can start as soon as they are ready far in advance of a future release. The most current set of dates is on the overall release issue linked at the top of this issue.
The Overall Release Issue
Linked at the top of this issue, the overall release issue captures the state of the entire OpenSearch release including references to this issue, the release owner which is the assignee is responsible for communicating the release status broadly.  Please contact them or @mention them on that issue for help.
What should I do if my plugin isn't making any changes?
If including changes in this release, increment the version on 2.0 branch to 2.1.0 for Min/Core, and <IP_ADDRESS> for components. Otherwise, keep the version number unchanged for both.

Preparation

[x] Assign this issue to a release owner.
[x] Finalize scope and feature set and update the Public Roadmap.
[x] All the tasks in this issue have been reviewed by the release owner.
[x] Create, update, triage and label all features and issues targeted for this release with v2.1.0.

CI/CD

[x] All code changes for 2.1.0 are complete.
[x] Ensure working and passing CI.
[x] Check that this repo is included in the distribution manifest.

Pre-Release

[x] Update to the 2.1.0 release branch in the distribution manifest.
[x] Increment the version on the parent branch to the next development iteration.
[x] Gather, review and publish release notes following the rules and back port it to the release branch.git-release-notes may be used to generate release notes from your commit history.
[x] Confirm that all changes for 2.1.0 have been merged.
[x] Add this repo to the manifest for the next developer iteration.

Release Testing

[ ] Find/fix bugs using latest tarball and docker image provided in parent release issue and update the release notes if necessary.
[ ] Code Complete: Test within the distribution, ensuring integration, backwards compatibility, and performance tests pass.
[ ] Sanity Testing: Sanity testing and fixing of critical issues found.
[ ] File issues for all intermittent test failures.

Release

[ ] Complete documentation.
[ ] Verify all issued labeled for this release are closed or labelled for the next release.

Post Release

[ ] Prepare for an eventual security fix development iteration by incrementing the version on the release branch to the next eventual patch version.
[ ] Add this repo to the manifest of the next eventual security patch version.
[ ] Suggest improvements to this template.
[ ] Conduct a retrospective, and publish its results.

Hi @amitgalitz ,
We found below issues tagged with v2.1.0 that were not closed or resolved in this release. Can you take a look and close the ones that are resolved or retag them with next version you see fit?
Thanks!
#518  Refactor AD code to improve testability                          bug, Priority-Medium, v2.1.0   about 2 months ago
#445  Checking feature queries as part of Validate API recommendation  enhancement, v2.1.0            about 1 month ago
#397  Deprecate the "Master" nomenclature                              enhancement, v2.1.0            about 1 month ago
#336  Support BWC using multiple older versions                        infra, Severity-Major, v2.1.0  about 2 months ago

@gaiksaya, thank you for pointing these out! Missed on our end, just updated everything.


Adds an abstraction layer to hide the underlying usage of Microsoft Data Encryption(MDE) library

This PR adds in an abstraction layer to hide the underlying dependency on Microsoft Data Encryption library. The abstraction is limited to only the classes which are currently exposed to public methods of client side encryption SDK.
Also added module-info for encryption  project
.NET PR - https://github.com/Azure/azure-cosmos-dotnet-v3/pull/2906

We have moved away from AAP EncryptionKeyStoreProvider  during client creation , and created our own EncryptionKeyWrapProvider.
This PR also introduced the wrapper implementation of AzureKeyVaultKeyStoreProvider in a cosmos lib wrapper as AzureKeyVaultKeyWrapProvider.

This PR is prerequisite for AAP shading jar PR https://github.com/Azure/azure-sdk-for-java/pull/26738

API changes have been detected in com.azure:azure-cosmos. You can review API changes here
API changes
+         exports com.azure.cosmos.implementation to com.azure.cosmos.encryption;
+         exports com.azure.cosmos.implementation.query to com.azure.cosmos.encryption;
+         exports com.azure.cosmos.implementation.patch to com.azure.cosmos.encryption;
+         exports com.azure.cosmos.implementation.batch to com.azure.cosmos.encryption;
+         exports com.azure.cosmos.implementation.caches to com.azure.cosmos.encryption;
+         exports com.azure.cosmos.implementation.apachecommons.lang to com.azure.cosmos.encryption;
+         exports com.azure.cosmos.implementation.apachecommons.lang.tuple to com.azure.cosmos.encryption;
+         exports com.azure.cosmos.implementation.guava25.base to com.azure.cosmos.encryption;
+         exports com.azure.cosmos.implementation.guava27 to com.azure.cosmos.encryption;

API changes have been detected in com.azure:azure-cosmos-encryption. You can review API changes here

API changes have been detected in com.azure:azure-cosmos. You can review API changes here

API changes have been detected in com.azure:azure-cosmos-encryption. You can review API changes here

API changes have been detected in com.azure:azure-cosmos. You can review API changes here

API changes have been detected in com.azure:azure-cosmos-encryption. You can review API changes here


Model hosting

Hello! Very interesting implementation. Do you have a trained model hosted available anywhere for downloading?
Thank you

Hi, thank you for your interest in ZETT! Currently, we don't have any published models online, but training a model takes less than 10 minutes for each split (based on TITAN RTX 24GB). Hope this helps!

This is the code for auto-generated templates by LLM, which was one of our experiments, but not included to the paper since the performance was not good enough. Maybe I should've deleted this part -- sorry for your confusion!


React v0.14

Use https://github.com/callemall/material-ui/pull/1733.

The docs site is still broken, we need to fix react-router
We have some warning to fix regarding deprectated usage

@oliviertassinari why is the build failing? Does this PR need more work before it can be merged?

@shaurya947 That an issue with npm. I still need to fix it.

@oliviertassinari okay. Holding off until issues are fixed.

@shaurya947 Found my issue, see https://github.com/karlbright/react-stub-context/pull/4.

Aah gotcha @oliviertassinari

What do you think about callemall forking react-stub-context and publishing it to npm so that we can move forward?

Or I could just clone it inside mui.

@shaurya947 This should be working now. I'm gonna see If I can fix the docs site. Looks like it's mainly fixing the breaking changes of react-router.

Do you want me to merge this and you can create a separate PR for the docs site? @oliviertassinari

@shaurya947 I have fixed the home screen of the docs, however, links are broken, etc, ...
See https://github.com/rackt/react-router/blob/master/UPGRADE_GUIDE.md for the migration.
I think that we can merge this one. Still more work is needed

Thanks @oliviertassinari! Looking forward to a fix with new react-router code :)

Perfect! I'll be making some pull requests once material-ui is compatiable w/ 0.14

Can help with react-router. I did the transition for my project some days ago. feel free to ping me on gitter. The upgrade doc is well done.

That would be much appreciated @cgestes :-)

@cgestes Any idea with the url is http://<IP_ADDRESS>:3000/#/?_k=5nnd1y when we go to http://<IP_ADDRESS>:3000?

I have the same :)
here are some info: https://github.com/rackt/react-router/issues/1967

Do we need all commits for a dev branch to pass all the tests like in master?
Maybe it would make sense to merge commits to react-0.14 branch if they look wanted and decent, so that we can try them easily, and provide correction easily.
And we take more care that everything is alright before merging to master?

@cgestes Here is the fix https://github.com/callemall/material-ui/pull/1766.

looks like commit enterred the branch already, forget the previous comment, I'am trying right now :)


{App Service}: Updating websites extension to the latest Python SDK version 0.48.0

Description
This PR update azure-websites extension to use the latest version of websites Python SDK 0.48.0
Testing Guide
Re-recorded all the tests
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

@qwordy do I need to wait for the following change to be merged https://github.com/Azure/azure-cli/pull/15508 for the build to pass?

/azp run

App Service


Focus trouble with negatives data-z property

Hello,
I saw troubles on form elements focuses when a step has a negative data-z property.
Even if there is only one slide.
have you already noticed something like that ?
Cordially,
Florian

Hi @florianharmel, @Earthwings, @scbritton, in an effort to clear up older issues/PRs we are pinging back to know if you are still tracking this request.
To give a little bit of context, recently a decision was made in the project to make the development more active and the first task is to clear up older issues like this one to see if the OP is still interested in keep it going.
Can anybody confirm if this is was indeed a browser bug and if the issue still exists? A simplified example would help a lot.

Closing due to the lack of feedback from the OP. If you want to work in the PR again, feel free to comment here to open up the discussion again


Fix Gen choose

For instances of generic interfaces, choose was throwing an ArrayStoreException because of different runtime types of the elements within the iterable. As suggested by Daniel this commit uses Iterator instead of Stream. This resolves the issue #1699.

Current coverage is 97.03% (diff: 100%)

Merging #1702 into master will increase coverage by 0.01%

@@             master      #1702   diff @@
==========================================
  Files            89         89
  Lines         11264      11264
  Methods           0          0
  Messages          0          0
  Branches       1878       1878
==========================================
+ Hits          10928      10930     +2
+ Misses          196        195     -1
+ Partials        140        139     -1

Powered by Codecov. Last update 2625b10...b296132

:+1: nice to know

Fixes #1699


Fix provisioned concurrency settings when in pay per request mode

Set to null when pay per request mode selected.
closes #10

:tada: This PR is included in version 0.3.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Automatic setting borderless_tables flag

Hi, I have a set of images that contains border and borderless tables. Actually, I am passing cropped table image region to img2table for extraction of data. But every time I have to set borderless_table flag true or false depending upon type of table. Is there any way to set borderless_tables flag True/False automatically?

I think that the effect of the parameter is not clear:

borderless_tables=False: only bordered tables are detected by the algorithm
borderless_tables=True: borderless tables are detected by the algorithm on top of bordered tables

So, I you set it to True all the time, the only difference would be that, when there are no borderless tables/only bordered tables in your document, it might detect some crap on top of your actual tables

Hi xavctn,
Thanks for your response. It's working. You can close the issue. 👍🏻

Hi xavctn,
Thanks for your response. It's working.


retire platform='gittip' for tips.json

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/7225428-retire-platform-gittip-for-tips-json?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

With the upgrade to Gratipay 2.0, we no longer have a concept of tips. This ticket is obsolete.


Proposed 0.32.0-rc1

[Current coverage][cc-pull] is 69.39%

Merging [#1739][cc-pull] into [develop][cc-base-branch] will decrease coverage by 0.32%

2 files (not in diff) in src/ripple/websocket were modified. more

Misses +7
Hits -7

3 files (not in diff) in src/ripple/protocol were modified. more

Misses -1
Hits +1

2 files (not in diff) in src/ripple/core/impl were modified. more

Misses -9
Hits +9

2 files (not in diff) in ...ipple/app/paths/impl were modified. more

Misses +11
Hits -11

3 files (not in diff) in ...ple/app/paths/cursor were modified. more

Misses +48
Hits -48

1 files (not in diff) in src/ripple/app/paths were modified. more

Misses +5
Hits -5

2 files (not in diff) in src/ripple/app were modified. more

Misses +1
Hits -1

3 files (not in diff) in src/ripple were modified. more

Misses +8
Hits -9

File ...paths/RippleCalc.cpp was modified. more

Misses +1
Partials 0
Hits -1

@@            develop      #1739   diff @@
==========================================
  Files           856        856
  Lines         62275      62543   +268
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          43421      43404    -17
- Misses        18854      19139   +285
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [bb61ad2...ce4c3a7][cc-compare]
[cc-base-branch]: https://codecov.io/gh/ripple/rippled/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/ripple/rippled/compare/bb61ad2afed200749454151ea8be647df84c782a...ce4c3a74976451356299cc5fe158e2bfda0f5e4c
[cc-pull]: https://codecov.io/gh/ripple/rippled/pull/1739?src=pr


[BUG] Conflicts with |center image extension from ITS Theme which is no longer working

Hey there,
after updating to the latest version of Image Converter, the |center image extension from ITS Theme's snippets (while using minimal theme) stopped working. Seems to affect Live Preview Mode only, tested in multiple Vaults.
Example:
![[image.svg|hmicro center]]
same unwanted result for:
![[image.webp|center]]
Centered images get aligned to the left suddenly. When deactivating Image Converter, everything seems fine again?!
Before I start searching for a fix, I quickly wanted to drop this here. If I find any more conflicts I would edit this post.
Greetings :)

Hey, many thanks for reporting!
I patched it just now, and while testing on ITS default Theme - the center positioning seems to work. But LEFT and RIGHT only works in Reading mode, is that's correct? I tried using the snippets you have linked, and those behave the same way. I tried disabling Image Converter plugin, but it seems to have no effect.
ITS snippets:
Center - works in live/reading mode
Left/right - only shows up in reading mode
Regardless, now, you will be able to use any custom snippet as long as you do not set alignment via the plugin.
When you set alignment via the plugin, plugins styling will come into effect and it will overwrite ITS snippet. For instance, if via the sippet you set it to |center and then via right-click you also set it to right - then precedence will be for the right alignment. Although, if you then disable right alignment via the right-click, it will then default back to using ITS snippet. Any resizing/alignment you do in Reading mode will show in Live preview, and vice-versa, any resizing/positioning you do in Live mode will show up in Reading mode too.
Basically, plugin won't affect any snippet or custom CSS unless you explicitly tell it to do so.
Although won't be out for quite some time just yet.

Many thanks for you detailed answer.
Strangely I did not set any alignment via Image Converter. Furthermore I noticed, on mobile there are some miss-alignments differing from the desktop version. For example one of the same two test images, both with a center alignment (snippet) was displayed left and one correctly was centered. I do not get it.
For me, there definitely was a difference when deactivating the plugin and still there is.
I am going to test it again, maybe I find a clue what could be the reason for this behavior - hopefully.

I created a new test vault with ITS Theme and Image Converter only and can reproduce it like this:

did not touched any settings
insert new image
add |center
swap between view modes (correctly centered in live preview, wrong alignment in reading mode)
deactivate plugin
see correct alignment in all view modes

Many thanks for you detailed answer.
Strangely I did not set any alignment via Image Converter. Furthermore I noticed, on mobile there are some miss-alignments differing from the desktop version. For example one of the same two test images, both with a center alignment (snippet) was displayed left and one correctly was centered. I do not get it. For me, there definitely was a difference when deactivating the plugin and still there is.
I am going to test it again, maybe I find a clue what could be the reason for this behavior - hopefully.

Sorry, I had to be more clear. I patched it, but have not released it yet. The demo video I have included is just to show that ITS theme and ITS snippets now work too.
I will update you when it comes out 🙏

Sorry, I had misunderstood that. Thank you for the clarification and your constant efforts with the plugin and the submitted issues. Have a great festive season! ⛄


Update docs to include ng-model

Updated the README to include information about ng-model as using it
over manually/programmatically trying to get the reCaptcha response is
easier and will lead to less issues.
This feature has been there for a while, but there are a ton of issues around people trying to get the reCaptcha response using getResponse(). getResponse() requires management of the widgetId in most cases (to be done right), as navigating a SPA generates a new widget for each time a route with a widget is loaded. ng-model does not require the developer to do anything special and works more "the angular way" ( :unamused: )

Thanks @TheSharpieOne !


dispatcher: Use Debug instead of Debugf

When the argument is a string returned by Error, we don't want to use Debugf, since if the error string happens to include % characters, this will cause a format string error.
Also prefer Debug over Debugf in cases where literal strings do not include any formatting (this part is purely cosmetic).

LGTM

LGTM


LAST nor returning LAST, using GROUP BY

I have the following query:
SELECT LAST("number") FROM "stats" WHERE "router" = "R1" AND "alarm" = "LinkDown" AND time >= now() - 7d GROUP BY "alarm", "object", "router", "severity"

And I'm getting two records, even though I was expecting only one (hence the LAST) ...
time                | Object | alarm    | router | severity | number
2019-04-07 19:05:10 | port A | LinkDown | R1     | deleted  | 2
2019-04-07 18:59:37 | port A | LinkDown | R1     | set      | 2

I was hoping to get the last one (2019-04-07 19:05:10) but I'm getting the last two ...
Why is that? What is it that I'm not understanding?
Thanks!

@laimaretto try removing the GROUP BY.


Refactor gulp build tasks

Refactor gulp build tasks to improve parity between development and production builds. Also isolate a point in the build process for a Babel transpilation step.
This PR consolidates several image processing and tasks that differ between development and production environments. Instead of having a separate task for each environment that differ only in output directories, a build-copy task will handle copying all assets to the correct build directory.
The new bundle task concatenates the application JS into a single bundle file along with the corresponding sourcemaps. This is where the transpilation step will be added in a follow up PR.
@miq-bot add_label euwe/no, refactoring
https://www.pivotaltracker.com/story/show/136032077

@AllenBW @chalettu thoughts?

So this PR looks like a good cleanup effort.  It seems to clarify and consolidate things and I am all for it.  Good work!

@miq-bot add_label euwe/no, refactoring


fix: prefix bug in MockDirectory.EnumerateDirectories (#815)

Fixes #815
System.IO.Directory.EnumerateDirectories(path) guarantees that returned paths are always prefixed with the value provided in the path parameter. However, MockDirectory.EnumerateDirectories always returned absolute paths, leading to an inconsistent behavior (e.g. when using relative paths).
We now check returned paths from MockDirectory.EnumerateDirectories and provide the proper prefix path.

Thanks for the fix @Phoenox, please have a look at the failing ubuntu and macOS tests :bow:


cmd + p crashing app

On a fresh install first time open, cmd + p bringing up the file explorer crashes the application. Sometimes it will stay open fine but crashes upon typing first letter. OSX 10.14.5

As I have experienced it too: it seems it tries to list too many items or one item is forbidden for Oni2. When I launch Oni2 directly in a directory (through the terminal) I don't run into any issue with cmd + p.

I will try and run through terminal and see if that fixes it!

(JIC) I give the path of the project folder as an argument to the oni2 command.

As I have experienced it too: it seems it tries to list too many items or one item is forbidden for Oni2. When I launch Oni2 directly in a directory (through the terminal) I don't run into any issue with cmd + p.

I expect this is the case, since MacOS is pretty protective over certain folders, so it wouldn't suprirse me if we run ripgrep over some places we shouldn't and hit some issues.

Looks like the same issue as #467

This should be fixed now on OSX with #467

I am still experiencing this with the last several nightly builds.

Thanks for the heads up, @jpinnix - will take a look.

This should be fixed now (no longer crashes in large directories).
The main fix for this was in #693 - which time-slices our processing of the results, so that we can stay on top of processing output from Ripgrep.
There are still improvements we can make to improve the speed of processing (important for large directories) - but the crash should no longer reproduce. I'll close this, but please let me know if you're still seeing crashes with Ripgrep on latest builds 👍

I believe this particular issue should be fixed now - this code-path:
Fontkit.FontKitRenderGlyphException("[ERROR]: Unable to render character at FT_Load_Char\n"):

is no longer code we run since switching to Skia. So hopefully this is fixed! But please feel free to open a new issue if this is still being hit.


using costura fody to embed these dlls?

do you think costura fody could be used to embed the 3rd party love dlls?

I can only seem to get it to work with love.dll
however maybe you can get it to work with the others

it might better to wrap the unmanaged c++ dlls into a managed C# one so that something like fody can be used with love2d.

this is the result of a discussion about how to embed native dlls into a managed library with a user called abnormal.
EmbeddedNative-Love2dCS.zip
It embeds the following..
love.dll
lua51.dll
mpg123.dll
OpenAL32.dll
SDL2.dll

good idea. i'm not use the costura, but i get the same effect. now it will release native library to temp folder.
11.0.37.

closed because it finished .


BUGFIX: Rename typoScriptPath to fusionPath in aspect class

The (internal) array key was renamed in Neos versions 3.1.2 and 3.0.4,
leading to an error because of an undefined index being used.

Closing in favour of #35


Link is not Working

Issue : Hovering over Contact Us Text ,their is nothing happens...
Solution: We should give a link (link Email id) when anyone hover over contact us...
https://user-images.githubusercontent.com/66238224/110579466-e3ab7000-818c-11eb-9669-644a2611f1c4.mp4

The Git hub link and all other social media links at the left side of Algo Phatoms website are not working. Please assign this issue to me as I'm the participant of GSSOC'21.

The Git hub link and all other social media links at the left side of Algo Phatoms website are not working. Please assign this issue to me as I'm the participant of GSSOC'21.

create a new issue and ask there @surbhi594

Issue : Hovering over Contact Us Text ,their is nothing happens...
Solution: We should give a link (link Email id) when anyone hover over contact us...
issue.mp4

The Contact Us over there is a heading and link is not needed over there.


remove  obsolete build tag

//go:build has been introduced in  go 1.18 and  the obsolete // +build lines  has been  removed  in modules declaring go 1.18 or later in their go.mod files.
i dont  think  it is  nessasary  that  prometheus support the  obsolete build tags(at 1.17 or lower)

Thanks!


Use 1.3.0 contract for all safes

Description
We should target 1.3.0 proxy factory and multisend contracts for all safes on all networks.
However, create new safes still only on 1.1.1 on mainnet and rinkeby.

Closes, because not needed. Safes created on 1.3.0 will target 1.3.0 contracts already.


quickfixj.server.configString not picked up

I have a SpringBoot application that depends on quickfixj-spring-boot-starter. The application fails to start up when using quickfixj.server.configString in yaml instead of quickfixj.server.config.
Here is my example comfig:
quickfixj:
  server:
    auto-startup: true
    log-factory: slf4j
    configString: |
      [default]
      FileStorePath=target/data/acceptor
      ConnectionType=acceptor
      StartTime=00:00:00
      EndTime=21:30:00
      ReconnectInterval=60
      FileLogPath=logs-server
      ResetOnLogon=Y
      ResetOnLogout=Y
      ResetOnDisconnect=Y
      BeginString=FIX.4.4
      HeartBtInt=60

      [session]
      SenderCompID=my-sender-comp-id
      TargetCompID=my-target-comp-id
      SocketAcceptPort=9877
      UseDataDictionary=Y
      DataDictionary=MY-FIX44.xml
      ResetOnLogon=Y
      ResetOnLogout=Y
      ResetOnDisconnect=Y

The actual error:
2022-02-18 11:32:46.530  WARN 3093460 --- [           main] ConfigServletWebServerApplicationContext : [] Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'serverAcceptor' defined in class path resource [io/allune/quickfixj/spring/boot/starter/autoconfigure/server/QuickFixJServerAutoConfiguration$SocketAcceptorConfiguration.class]: Unsatisfied dependency expressed through method 'serverAcceptor' parameter 2; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'serverSessionSettings' defined in class path resource [io/allune/quickfixj/spring/boot/starter/autoconfigure/server/QuickFixJServerAutoConfiguration.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [quickfix.SessionSettings]: Factory method 'serverSessionSettings' threw exception; nested exception is io.allune.quickfixj.spring.boot.starter.exception.SettingsNotFoundException: Settings file not found

Hi @MZlatev-MDigital,
I haven't had the chance to release this yet as I'm testing the new features. I'll try to do it today.
In the meantime you can clone the project, build it locally and use the snapshot version.
Thanks

Hi @MZlatev-MDigital,
I just release version 2.11.0 to Maven Central, which includes this feature (configString). Could you try with this version please?
Thanks
Ed

You're missing the classpath: prefix

Thanks for noting that. Previously (version 2.7.1) it was not needed.
I confirm that everything is working as expected. I am closing this ticket. This feature will ease running quickfixj in k8s.
Thanks for the support!


adapted example is not resolving deep sub-routes.

Version
5.1.2
Test Case
ive modified the example at https://reacttraining.com/react-router/web/example/nesting to include

a basename on the BrowserRouter
a route guarded component wrapper for the nesting example component
modification of the  {path} and {url} for the nesting example as it is now a child route

result: The Topics link has stopped working.
live example:
https://codesandbox.io/s/react-router-nesting-7b690
Steps to reproduce
## Expected Behavior

Clicking Topics link should load component

## Actual Behavior

Clicking Topics link gives a white screen and no error message

thanks

got it working with few tweaks. working sandbox available.


fix(deps): update icr.io/ext/sysdig/agent docker tag to v13.0.2

This PR contains the following updates:

Package
Update
Change

icr.io/ext/sysdig/agent
patch
13.0.1 -> 13.0.2

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 1.22.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to compile FaceAlignment and run fa_test.exe in windows.

@weixwx 您好，请到以下链接下载VS2013的工程，可以直接run
https://github.com/seetaface/SeetaFaceEngine/tree/master/FaceAlignment/examples

@seetaface 谢谢！


[1929], [1930] and [1931] - A couple of small provider-led snags

Context

https://trello.com/c/f5rQzGTW/1929-s-replace-review-and-submit-with-final-check
https://trello.com/c/NjL0oeqK/1930-s-check-h2-margins
https://trello.com/c/tiZZ3nUj/1931-s-change-page-title-to-record-submitted-on-trn-record-submission-page

Changes proposed in this pull request

A couple of snags from the provider-led demo

Guidance to review

 [S] Replace Review and Submit with Final Check

 [S] Check H2 margins

 [S] Change Page title to Record submitted on trn record submission page

All good! 🏄♂


Baidu Netdisk links  have expired

Hello,
Thank you very much for your work on Cascade-CLIP. It is excellent and has been very inspiring for me. However, the Baidu Netdisk links for the model weights of the PASCAL Context and COCO Stuff 164K datasets have expired. Could you please share them again?
Thank you!

Hello,
I'm glad to hear that you find Cascade-CLIP useful and inspiring!
I apologize for the inconvenience caused by the expired links. I have updated the model weights for the PASCAL Context and COCO Stuff 164K datasets, and you can now access them through the following new links.
Best regards!

Could you also provide google drive link the links provided are broken


Get configuration from environment variables as well as config.yaml

In order to support dockerizing Throat, add the ability for environment variables to supplement or override the default values or the values from config.yaml.
In the process I changed the Map class in app/config.py to gather config, default and environment variables together into one dictionary in the constructor.  I think this is an improvement over the previous version, where if you printed a Map you wouldn't see the defaults, but you could access them using __getattr__. Now what you see is what you get.
That didn't play well with db_connect in app.models.py which wants to change the config dictionary.  This appears to only be needed for the call to database_class so I dealt with it by modifying a duplicate copy of the dictionary.
Since underscores are used to separate the keys in the construction of environment variable names and are also present in some of the key names, there would be a bit of ambiguity if keys are someday nested more than two levels deep.  If that happens, this code would deal with it by throwing it all into the dictionary. So for example setting the environment variable FOO_BAR_BAZ=bop when cfg_defaults['foo']['bar'] is present would result in both config['foo']['bar']['baz'] == 'bop' and config['foo']['bar_baz'] == 'bop'.

Looks good to me. Thanks for your contribution


chore: update readme

Husky have updated how they execute the hooks.

https://github.com/prettier/pretty-quick#pre-commit-hook


Uses Version class to solve version mismatch

Solves https://github.com/jdhitsolutions/WingetTools/issues/20 but adds a performance penalty due to casting and comparison.

I originally had similar code using [version] but it fails when encountering version strings like <IP_ADDRESS>5.gabf054ab , v1.17.2, or 3.3.1+06924 which is why I reverted to plain text.

Thank you for the information. So, is it better to close the PR or to get some fancy Regex to cut the edge cases for non-standard versioning examples?

We can leave the PR pending for now. I'm really hoping that the Winget team can produce a high-quality PowerShell module that would make this module moot. I'm having discussions with a team leader, hoping to help guide them.


Changed version resolution from lte to lt

Seems that <= didn't work, but < does work.

I think this bug is papering over a different problem.  The specifier as it exists already should match 44:

I agree it doesn't, though:
http://cdn.polyfill.io/v2/polyfill.js?features=Array.prototype.find&ua=chrome/44
/* For detailed credits and licence information see http://github.com/financial-times/polyfill-service.
 *
 * UA detected: chrome/44.0.0
 * Features requested: Array.prototype.find
 *  */

So this needs a bit more investigation, probably starting here.

Would it be because Chrome has version numbers such as 44.0.2526.106, which is bigger than 44?

Looks like there was a bug in the version of the semver parser that we were using.  I updated it and it looks good now.

Should we add this as a test case?


Push: OnEntry / OnReceived not fired when running in background / or closed

Summary
Hi,
I am new to Shiny.
I am using Push for notifictions.
When the app is running in foreground, and a notification is received, it comes to OnReceived of the PushDelegate.
If the app is running in background or closed, Neither OnReceived or OnEntry are being called.
I followed all the steps in the guidelines.
API Changes
unknonw
Intended Use Case
Push notofications
Code of Conduct

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

This does not follow our guidelines for submitting issues


feat: add init and tags

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

13 of 52 (25.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.4%) to 10.119%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/config.go
0
1
0.0%

pkg/util/highlight.go
0
2
0.0%

cmd/install.go
2
7
28.57%

cmd/init.go
8
20
40.0%

pkg/ownbrew/ownbrew.go
0
19
0.0%

Totals

Change from base Build<PHONE_NUMBER>4:
1.4%

Covered Lines:
51

Relevant Lines:
504

💛  - Coveralls


feat(taskframework): taskframework integration with k8s client

What type of PR is this?

module-TaskFramework
type-feature
What this PR does / why we need it:
integration with k8s client for task running
Which issue(s) this PR fixes:

import io.kubernetes:java-client in maven pom
add K8sClient for  odc operation k8s cluster in different  k8s environment, PrimitivePodBasedK8sClient is the client for Primitive k8s cluster based on pod, PrimitiveJobBasedK8sClient is the client for Primitive k8s cluster based on k8s job,

Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:

Checkout a new feature branch from dev/4.2.x (eg. feat/424_task_framework), and the related PR firstly merge to this feature branch may be better?

Checkout a new feature branch from dev/4.2.x (eg. feat/424_task_framework), and the related PR firstly merge to this feature branch may be better?
add branch feat/424_taskframework for 424  task framework feature


Landscape Mode Issue

Hi ,
I hope you are doing well , your sample code is not working on landscape mode. showing a black screen.
I am Using Samsung S8 (OS 09)

On simulator or phone?

Im getting this issue as well. One plus 8 - android 10
I'd love to have a working version since the effect is so good

Comments following lines in SampleActivity is working for me
if (Build.VERSION.SDK_INT < 16) {
            getWindow().setFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN,
                    WindowManager.LayoutParams.FLAG_FULLSCREEN);
        } else {
            mPageFlipView.setSystemUiVisibility(
                    View.SYSTEM_UI_FLAG_FULLSCREEN |
                            View.SYSTEM_UI_FLAG_LAYOUT_STABLE |
                            View.SYSTEM_UI_FLAG_LAYOUT_HIDE_NAVIGATION |
                            View.SYSTEM_UI_FLAG_IMMERSIVE |
                            View.SYSTEM_UI_FLAG_HIDE_NAVIGATION);
        }


Corrected setUser TS interface to allow for null arguments

Goal
The setUser method in the notifier can take null as a parameter to clear the value from the user object. The current TypeScript interface only allows for string | undefined, which causes type errors when trying to pass null as a parameter.
BugSnag documentation mentions passing null to setUser as a possibility.
Design
The type definition has been updated to accurately reflect the capabilities od the notifier.
Changeset
The type definition for setUser's arguments has been extended from id?: string to id?: string | null.
Testing
The change was tested manually.

Highlighted in https://github.com/bugsnag/bugsnag-js/issues/1259#issuecomment-2465795944


chore(github): updates PR template

Apply https://github.com/mixmaxhq/mixmax/pull/480

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


parsing CSV file with 2 empty newlines at end fails

If there's two empty newlines at the end (so one fully empty line), our CSV parser chokes.
It's a bit difficult, as either we need to check for this in the first line counting pass (slows it down) or reduce the size of the final tensor afterwards. Need to think about the most elegant solution.
To reproduce just take a working CSV file and 2 newlines at the end and run.

This can partially be handled now by using the maxLines option added in #46. Still this particular case should be handled automatically.


Nanoseconds in relative time units causes error

Describe the bug
Conversion from relative time units containing nanoseconds causes an exception when attempting to convert in ERDDAP's internal routines.
To Reproduce
Steps to reproduce the behavior:

Add a dataset with time units attribute set to something with nanoseconds, e.g. nanoseconds since 1970-01-01T00:00:00+00:00.
Add dataset to datasets.xml to be inventoried by ERDDAP.
Dataset does not appear and a Java calendar exception is raised in the ERDDAP log file.

Excerpt from email correspondence where this issue was encountered:

Apparently ERDDAP doesn’t like something with a time related quantity:
While trying to load datasetID=dfo-hal1002-20020724T1839 (after 23 ms)
java.lang.RuntimeException: datasets.xml error on or before line #731958: ERROR in Calendar2.factorToGetSeconds: units="nanoseconds" is invalid.
at gov.noaa.pfel.erddap.dataset.EDD.fromXml(EDD.java:486)
at gov.noaa.pfel.erddap.LoadDatasets.run(LoadDatasets.java:364)
Caused by: java.lang.RuntimeException: ERROR in Calendar2.factorToGetSeconds: units="nanoseconds" is invalid.
at com.cohort.util.Test.error(Test.java:43)
at com.cohort.util.Calendar2.factorToGetSeconds(Calendar2.java:2725)
at com.cohort.util.Calendar2.getTimeBaseAndFactor(Calendar2.java:2550)
at com.cohort.util.Calendar2.getTimeBaseAndFactor(Calendar2.java:2518)
at gov.noaa.pfel.erddap.variable.EDVTimeStamp.<init>(EDVTimeStamp.java:186)
at gov.noaa.pfel.erddap.variable.EDVTime.<init>(EDVTime.java:27)
at gov.noaa.pfel.erddap.dataset.EDDTableFromFiles.<init>(EDDTableFromFiles.java:1778)
at gov.noaa.pfel.erddap.dataset.EDDTableFromNcFiles.<init>(EDDTableFromNcFiles.java:131)
at gov.noaa.pfel.erddap.dataset.EDDTableFromFiles.fromXml(EDDTableFromFiles.java:501)
at gov.noaa.pfel.erddap.dataset.EDD.fromXml(EDD.java:472)

After looking at ncdump output, it appears that time variable units are expressed as “nanoseconds since 1970-01-01 00:00:00” or something very similar.  I believe this is valid UDUnits and should be fine with CF as well, but ERDDAP doesn’t seem to like converting that unit with the module they have there.  If the previous sentence is correct, I’d argue that this is a bug/deficiency of ERDDAP.  If you’d like to get the files read in the meantime, I’d recommend converting over to at least seconds.

I have yet to encounter or test any datasets with even smaller time intervals such as picoseconds, but I imagine this issue may also exist with them.
Expected behavior
Relative time units expressed in nanoseconds ("nanoseconds since 1970-01-01T00:00:00+00:00") converts properly since it should be a valid UDUnits relative time.
Server

ERDDAP 2.23 Axiom Docker image

Desktop (please complete the following information):

Linux x86-64

Additional context
Add any other context about the problem here.

Milliseconds is the most fine grained time measurement currently supported in ERDDAP. https://github.com/ERDDAP/erddap/blob/main/WEB-INF/classes/com/cohort/util/Calendar2.java#L5741-L5770

This is surprising behavior because UDUNITS2 allows for all the way down to yocto (10^-24) prefixes on units: https://github.com/Unidata/UDUNITS-2/blob/c83da987387db1174cd2266b73dd5dd556f4476b/lib/udunits2-prefixes.xml#L74
While it's unlikely that the sort of data ERDDAP Is usually serving would have this fine of a time resolution, it is nonetheless valid in UDUNITS2 to express something like yoctoseconds since 2024-01-01+00:00.
An acceptable enough fix would be to parse that unit and switch/case a power of ten multiplier to convert the relative time prefix back to milli_(time interval)_ in the event that the Java calendar library threw an an exception.

The ERDDAP documentation for handling time units is here: https://erddap.github.io/setupDatasetsXml.html#timeUnits
It doesn't strictly follow UDUNITS2. ERDDAP doesn't use UDUNITS2 for time conversions. Adding additional time units (even just for time conversion) is possible but more work than a power of ten conversion in one spot. Additionally in several places the java.util.Date library is still used which only has millisecond precision. To properly support nanoseconds we'd need to update that code to a different library or verify it doesn't need the additional precision. Past nanosecond accuracy and standard java time libraries don't support that. Power of ten conversion only support might be possible, but I wonder how beneficial that is, especially if users don't realize they are loosing the precision they think they have.

Definitely the most efficient workaround here would be to convert the source data time variable to use millisecond or coarser precision. But, if that's not convenient/possible/etc, you could try doing that conversion in datasets.xml using a JEXL expression.
Example:
    <dataVariable>
        <!-- convert nanosecond time precision to seconds -->
        <sourceName>=row.columnLong("time")/1e+9</sourceName>
        <destinationName>time</destinationName>
        <dataType>long</dataType>
        <!-- sourceAttributes>
        </sourceAttributes -->
        <addAttributes>
            <att name="ioos_category">Time</att>
            <att name="long_name">Time</att>
            <att name="standard_name">time</att>
            <att name="units">seconds since 1970-01-01T00:00:00Z</att>
        </addAttributes>
    </dataVariable>

Worked for me when experimenting with the new parquet support, YMMV.

Wasn't aware of this bit of ERDDAP, but it looks like it's been around for quite some time.  Thanks for the info.


Remove the test helpers from the main package.

We don't want to import the testing library when you import "github.com/docker/libkv/store".
Signed-off-by: David Calavera<EMAIL_ADDRESS>

:heart_eyes:

Better! :dancer: thanks @calavera!
Sorry it needs a quick golint check and we are good to go :cry:

@abronan fixed, thanks for checking

LGTM


Moving Kamesh's Devops EAP Playbooks

Moving Kamesh's Devops EAP Playbooks to RH Consulting Ansible JBoss Middleware playbooks, first import

@kameshsampath as discussed in email, let's use separate repos for roles and playbooks. Let's use this repo only for playbooks. EAP role stuff can go to https://github.com/rhtconsulting/ansible-eap-role

Great will start to port roles to eap-role repo and keep playbooks to this repo.
Sent from Outlook Mobile
On Tue, Apr 5, 2016 at 10:28 AM -0700, "sherl0cks"<EMAIL_ADDRESS>wrote:
@kameshsampath as discussed in email, let's use separate repos for roles and playbooks. Let's use this repo only for playbooks. EAP role stuff can go to https://github.com/rhtconsulting/ansible-eap-role

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/rhtconsulting/ansible-middleware-playbooks/pull/1#issuecomment-205907692

are we just waiting for the the updated pull (only playbooks) so we can merge?

Albert,
Over next few days I will split the roles and playbooks into separate repo as discussed with @sherl0cks. And this repo will only have playbooks and eap-role repo will have all roles related to this.
Just wanted to split this into smaller commits as that will be easy for review and merge.
Sent from Outlook Mobile
From: Albert Wong
Sent: Thursday, 7 April, 1:07 AM
Subject: Re: [rhtconsulting/ansible-middleware-playbooks] Moving Kamesh's Devops EAP Playbooks (#1)
To: rhtconsulting/ansible-middleware-playbooks
Cc: Kamesh Sampath
are we just waiting for the the updated pull (only playbooks) so we can merge?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Closing this request


Failing tests on node 18.x

The provided tests in the example repo are failing on Node 18, due to a jasmine timeout error.
Steps to reproduce:

Checkout the repository
Setup node 18.x
Run tests -> npm run test

➜  parse-server-example git:(master) ✗ git --no-pager  log --oneline -n 1 && node -v && npm -v && npm run test
2d072a6 (HEAD -> master, origin/master, origin/HEAD) docs: update README (#419)
v18.12.1
8.19.2

><EMAIL_ADDRESS>test
> mongodb-runner start && jasmine

  ◝ Starting a MongoDB deployment to test against...DATABASE_URI not specified, falling back to localhost.
Started
warn: DeprecationWarning: The Parse Server option 'directAccess' default will change to 'true' in a future version. Additionally, the environment variable 'PARSE_SERVER_ENABLE_EXPERIMENTAL_DIRECT_ACCESS' will be deprecated and renamed to 'PARSE_SERVER_DIRECT_ACCESS' in a future version; it is currently possible to use either one.
warn: DeprecationWarning: The Parse Server option 'enforcePrivateUsers' default will change to 'true' in a future version.
..F..

Failures:
1) Parse Server example failing test
  Message:
    Error: Timeout - Async function did not complete within 5000ms (set by jasmine.DEFAULT_TIMEOUT_INTERVAL)
  Stack:
        at <Jasmine>
        at listOnTimeout (node:internal/timers:564:17)
        at process.processTimers (node:internal/timers:507:7)

Suite error: top suite
  Message:
    Error: Timeout - Async function did not complete within 5000ms (set by jasmine.DEFAULT_TIMEOUT_INTERVAL)
  Stack:
        at <Jasmine>
        at listOnTimeout (node:internal/timers:564:17)
        at process.processTimers (node:internal/timers:507:7)

5 specs, 2 failures
Finished in 11.087 seconds
➜  parse-server-example git:(master) ✗

Change the test command to;
NODE_OPTIONS=--dns-result-order=ipv4first jasmine

or change database uri to ipv6

Thanks for the reply. I update the test command so the the ipv4 is used and  test command executed successfully.
Based on the fix, would this be an issue with jasmine or a more general one related to how the dns resolvers are used by Node ?


fix: Group example

The Group example was showing multiple children in the group, contrary to the Note below.

Thanks!


Don't try lock the Lisp REPL if parse-js or read-js-number receive a NIL value

This pull request adds calls to check-type that ensure that the functions parse-js and read-js-number throw an error if they receive the NIL value.
Fixes #7

Wonderful. Merged.


fix(useMoralisFile): get moralisFile.ipfs()

name: "Pull request"
about: A new pull request
New Pull Request
Checklist

[ ] I am not disclosing a vulnerability.
[ ] My code is conform the code style
[ ] I have made corresponding changes to the documentation
[ ] I have updated Typescript definitions when needed

Issue Description

Related issue: #FILL_THIS_OUT
Solution Description

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[CI] Make mod tidy check work with Go modules

What does this PR do?

Makes the tidy check work with modules

Motivation

Keep all go.mod files synced
Test that the modules can be imported externally

Additional Notes

Not sure the dummy package test is actually needed; maybe go mod tidy suffices?

LGTM, maybe we should add a little bit of logging in check_mod_tidy to know what the task is doing (eg. what module it's currently working on, etc.)? Right now nothing gets logged.

I added the -e option so that we log the commands we are running and also printed when we are creating the main.go dummy file.


[release/1.5] cherry-pick: cri: add devices for privileged container

Cherry-pick PR #5939 from main to release/1.5

/test all

/retest

You need to rebase on the head of release/1.5 to pick up the change in test image reference required for tests to pass. Thanks!

still failed.


Difficult to find Netlify preview in Readme

It is hard to find the Netlify preview of Template Studio V2 in the readme for people who don't know the Netlify badge is clickable and that it brings the user to a page where they can click a link to go to the preview.

https://accordproject-studio.netlify.com/
Please add this link below the Netlify badge in the readme

@irmerk @Michael-Grover as the previous PR is closed, I have raised a similar PR to add the hyperlink for Template Studio V2 preview on Netlify. Please let me know if it can be merged.


[SMALLFIX] Removed explicit type argument in StorageTier

Removed an explicit type argument in the StorageTier class.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9860/
Test FAILed.

Jenkins, test this please.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9861/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, test this please.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9863/
Test FAILed.

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9866/
Test PASSed.

Merged build finished. Test PASSed.

LGTM


[Makefile] packit-service-worker is no longer based on packit image

closing per https://github.com/packit/packit/issues/762


ts-rest.com: fonts are broken in a couple of places

serif -> font-family="Virgil, Segoe UI Emoji"
This one should be easy, I guess the font just isn't linked properly / needs a better fallback

monospace -> style="font-family:'HackNerdFontComplete-Bold', 'Hack Nerd Font', monospace
This one is harder, embedded svg so the font should probs just be a path. not even God knows how to make svgs

It was easier to rebrand than deal with svgs lol


UNION across multiple joined relations leads to SQLParseException

CrateDB version
5.3.2
CrateDB setup information
3 nodes on k8s
Problem description
The query fails with:
SQLParseException[Cannot cast value `members` to type `integer`]

It seems like the order of the fields is getting mixed up.

There was/is a similar problem just recently: https://github.com/crate/crate/issues/13779
Steps to Reproduce
SELECT
  "s"."nspname" "table_schema"
, "c"."relname" "table_name"
, "a"."attname" "column_name"
, '1' "oid"
FROM (("pg_class" c
INNER JOIN "pg_namespace" s ON (("c"."relnamespace" = "s"."oid") AND (NOT ("s"."nspname" IN ('pg_catalog', 'information_schema')))))
INNER JOIN "pg_attribute" a ON ((("a"."attrelid" = "c"."oid") AND ("a"."attnum" > 0)) AND (NOT "a"."attisdropped")))
WHERE "c"."relkind" IN ('m', 'f', 'p')
 UNION  SELECT
  "table_schema"
, "table_name"
, "column_name"
, "pg_type"."oid" "oid"
FROM ("information_schema"."columns" cols
LEFT JOIN "pg_type" ON ("cols"."udt_name" = "pg_type"."typname"))
WHERE NOT ("table_schema" IN ('pg_catalog', 'information_schema'))
LIMIT 1000

Actual Result
error:
SQLParseException[Cannot cast value `members` to type `integer`]

It seems like the query plan randomly changes the order in the eval before the union. two examples:
LimitDistinct[100000::bigint;0 | [table_schema, table_name, column_name, oid]]
  └ Union[table_name, oid, column_name, table_schema]
    ├ Eval[relname AS table_name, '1' AS oid, attname AS column_name, nspname AS table_schema]
    │  └ HashJoin[(attrelid = oid)]
    │    ├ HashJoin[(relnamespace = oid)]
    │    │  ├ Rename[relname, relnamespace, oid] AS c
    │    │  │  └ Collect[pg_catalog.pg_class | [relname, relnamespace, oid] | (relkind = ANY(['m', 'f', 'p']))]
    │    │  └ Rename[nspname, oid] AS s
    │    │    └ Collect[pg_catalog.pg_namespace | [nspname, oid] | (NOT (nspname = ANY(['pg_catalog', 'information_schema'])))]
    │    └ Rename[attname, attrelid] AS a
    │      └ Collect[pg_catalog.pg_attribute | [attname, attrelid] | ((attnum > 0) AND (NOT attisdropped))]
    └ Eval[table_schema, table_name, column_name, oid AS oid]
      └ NestedLoopJoin[LEFT | (udt_name = typname)]
        ├ Rename[table_schema, table_name, column_name, udt_name] AS cols
        │  └ Collect[information_schema.columns | [table_schema, table_name, column_name, udt_name] | (NOT (table_schema = ANY(['pg_catalog', 'information_schema'])))]
        └ Collect[pg_catalog.pg_type | [oid, typname] | true]

LimitDistinct[100000::bigint;0 | [table_schema, table_name, column_name, oid]]
  └ Union[oid, table_schema, table_name, column_name]
    ├ Eval['1' AS oid, nspname AS table_schema, relname AS table_name, attname AS column_name]
    │  └ HashJoin[(attrelid = oid)]
    │    ├ HashJoin[(relnamespace = oid)]
    │    │  ├ Rename[relname, relnamespace, oid] AS c
    │    │  │  └ Collect[pg_catalog.pg_class | [relname, relnamespace, oid] | (relkind = ANY(['m', 'f', 'p']))]
    │    │  └ Rename[nspname, oid] AS s
    │    │    └ Collect[pg_catalog.pg_namespace | [nspname, oid] | (NOT (nspname = ANY(['pg_catalog', 'information_schema'])))]
    │    └ Rename[attname, attrelid] AS a
    │      └ Collect[pg_catalog.pg_attribute | [attname, attrelid] | ((attnum > 0) AND (NOT attisdropped))]
    └ Eval[table_schema, table_name, column_name, oid AS oid]
      └ NestedLoopJoin[LEFT | (udt_name = typname)]
        ├ Rename[table_schema, table_name, column_name, udt_name] AS cols
        │  └ Collect[information_schema.columns | [table_schema, table_name, column_name, udt_name] | (NOT (table_schema = ANY(['pg_catalog', 'information_schema'])))]
        └ Collect[pg_catalog.pg_type | [oid, typname] | true

⚠ sometime it actually spits out an executable query plan and the query succeeds.
Expected Result
Query returning x rows

Thx a lot for reporting this issue @proddata, The fix will be available with next hotfix release.


App Service Environment: making workerPools optional

This field is only required when provisioning an ASEv1 - it's optional for an ASEv2 - as such there's no need for it to be required
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

You don't have permission to trigger SDK Automation.
Please add yourself to Azure group from opensource portal if you are MSFT employee,
or please ask reviewer to add comment *** /openapibot sdkautomation ***.
Please ask<EMAIL_ADDRESS>(or NullMDR in github) for additional help.

And plus I suppose this PR should all get reviewed by  someone from the corresponding service team.

/azp run

Hi @tombuildsstuff I have emailed the corresponding service team to inform them to have a review on this PR. Once they approve, this PR is good to merge.

Can one of the admins verify this patch?

Hey @tombuildsstuff. Appreciate your effort in trying to make this better.  We actually need to change our source code to make this happen. I have made a note of removing this from the required properties and we will add it to our backlog.

Thanks @miwebst for the clarification. Hi @tombuildsstuff unfortuantely based on this I will close this PR then, hope you do not mind...


Poetry export creates a virtualenv

When running from outside of a virtualenv, poetry export will create a virtualenv before running the command.
This has two issues:

creating a virtual env is slow and makes the command leaves behind unnecessary resources
creating a virtual env prints Creating virtualenv ... to stdout, which means it not possible to use the command's stdout to put the output in a arbitrary location (see #77 )

On top of that creating the virtualenv is not needed for this command (I've tried with poetry config --local virtualenvs.create true and it works as expected)

The ExportCommand used to be a mere Command but now it is an InstallerCommand, which in turn is an EnvCommand.
I believe that it was made an InstallerCommand so that it had access to group dependency options, but it didn't really want to be an EnvCommand.

Using Poetry v1.2.0 this produces an error also when virtualenvs.create is disabled:
Building in Docker I received an error when running the following command:
RUN poetry export -f requirements.txt | pip install -r /dev/stdin
Error:
RUN poetry export -f requirements.txt | pip install -r /dev/stdin:
#0 26.79 ERROR: Invalid requirement: 'Skipping virtualenv creation, as specified in config file.' (from line 1 of /dev/stdin)

I am assuming that #106 fixes this as well

meanwhile send the output to file --output requirements.txt and use that, or (I haven't tested) --quiet might do the trick

As a workaround I did the following:
RUN poetry export -f requirements.txt --output requirements.txt
RUN pip install -r requirements.txt


Fix pypi installation EPW-template-file.epw

This pull request aims at making sure that the EPW-template-file.epw needed for epw conversion is well installed when downloading the packaging from pypi with pip.

Codecov Report

Merging #8 (db284b5) into main (4a08513) will increase coverage by 8.27%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   48.83%   57.10%   +8.27%
==========================================
  Files          11       11
  Lines         815      816       +1
  Branches      122      122
==========================================
+ Hits          398      466      +68
+ Misses        384      300      -84
- Partials       33       50      +17

Flag
Coverage Δ

macos-latest-3.8
57.10% <100.00%> (+8.27%)
:arrow_up:

macos-latest-3.9
57.10% <100.00%> (+8.27%)
:arrow_up:

ubuntu-latest-3.8
57.10% <100.00%> (+8.27%)
:arrow_up:

ubuntu-latest-3.9
57.10% <100.00%> (+8.27%)
:arrow_up:

windows-latest-3.8
57.00% <100.00%> (+8.29%)
:arrow_up:

windows-latest-3.9
?

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

Impacted Files
Coverage Δ

pygsod/epw_converter.py
62.50% <100.00%> (+31.46%)
:arrow_up:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Older version documentation missing

I have a project that uses version 16.2 of react, but when I click on the link on the document version history I get an error that the documentation is not found. We are not at a point to be able to upgrade since there are several dependencies that will need to be updated along with the latest version -- so having access to the documentation is very helpful.

Hey @cquick, thanks for filing an issue. This is a dupe of https://github.com/reactjs/reactjs.org/issues/3401 so I'm going to close this in lieu of that issue.

Hey @cquick, thanks for filing an issue. This is a dupe of https://github.com/reactjs/reactjs.org/issues/3401 so I'm going to close this in lieu of that issue.


Transfer Sample 04 terminating immediately after startup

Bug Report
Describe the Bug
A few seconds after completing the docker-compose, a failing http request in the the provider_1 container causes this container to terminate.
Expected Behavior
All containers should stay up and running.
Observed Behavior
A few seconds after completing the docker-compose, a failing http request in the the provider_1 container causes this container to terminate. The other containers are then terminating as well (like they should in such as case, since we are using the "--abort-on-container-exit" parameter).
Steps to Reproduce
Steps to reproduce the behavior:

Clone the Samples Project from Github.
Configure, build and run Sample No. 1 as described in the README. (Everything is working just fine here)
Download opentelemetry agent. Copy this file to the Sample04 root folder. (As instructed by the README)
Build Sample04 (of course using the given Gradle Wrapper)
Run docker-compose as specified in the README.

Context Information
Xubuntu 22.04
Docker version 24.0.4, build 3713ee1
docker-compose version 1.29.2, build unknown
docker-py version: 5.0.3
CPython version: 3.10.6
OpenSSL version: OpenSSL 3.0.2 15 Mar 2022
openjdk 17.0.7 2023-04-18
OpenJDK Runtime Environment (build 17.0.7+7-Ubuntu-0ubuntu122.04.2)
OpenJDK 64-Bit Server VM (build 17.0.7+7-Ubuntu-0ubuntu122.04.2, mixed mode, sharing)
Detailed Description
I have attached the logging output here:
log.txt
Possible Implementation
None that I know of.

It's actually the consumer that exits because of SEVERE : Error booting runtime: The following injected fields were not provided: Field "dataPlaneInstanceStore" of type [interface org.eclipse.edc.connector.dataplane.selector.spi.store.DataPlaneInstanceStore] required by org.eclipse.edc.connector.catalog.CatalogDefaultServicesExtension.
The provider timeouts/deadlines do not really cause something to terminate.
It seems the sample is not up to date; you can get it working by looking at transfer-01-file-transfer:

There is a implementation(libs.edc.data.plane.selector.core) dependency missing in the build.gradle, causing the error above.
After that, I still got an http error when posting the contract offer, replaced that with the one from sample transfer 01 (here and there you have to replace localhost with provider, similar for filetransfer.json), also replaced docker compose env vars with what's in the config.properties of transfer 01 (e.g. EDC_DSP_CALLBACK_ADDRESS instead of IDS_WEBHOOK_ADDRESS; these env vars get translated into config if I look at ConfigFactory.java); that seemed to get it working.

@matthiasdg would you mind provide a PR to fix the issue?

opened PR (can't login in newly created eclipse foundation account and fill out contributor agreement though, sent mail to webmaster)