ClassNotFoundException with Postgresql UUID datatype

Issue Overview
Running a Wildfly Swarm project to build a JaxRS API using Arquillian for automated testing.
Data layer is JPA entities.
Database is Postgresql
Making use of @UsingDataSet annotation with YAML files.
Expected Behaviour
When a datatype has an ID of type UUID, the following seed snippet will work.
some_entity:
  - id: "uuid'2aad615a-d8e1-11e2-b8ed-50e549c9b654'"
    other_attrs...

Current Behaviour
Exception thrown when it tries to seed.
Caused by: org.dbunit.dataset.datatype.TypeCastException: Unable to typecast value <uuid'2aad615a-d8e1-11e2-b8ed-50e549c9b654'> of type <java.lang.String> to uuid
	at org.dbunit.ext.postgresql.UuidType.getUUID(UuidType.java:88)
	at org.dbunit.ext.postgresql.UuidType.setSqlValue(UuidType.java:63)
	at org.dbunit.database.statement.SimplePreparedStatement.addValue(SimplePreparedStatement.java:73)
	at org.dbunit.database.statement.AutomaticPreparedBatchStatement.addValue(AutomaticPreparedBatchStatement.java:63)
	at org.dbunit.operation.AbstractBatchOperation.execute(AbstractBatchOperation.java:211)
	... 200 more
Caused by: java.lang.ClassNotFoundException: org.postgresql.util.PGobject from [Module "org.jboss.ironjacamar.jdbcadapters:main" from BootModuleLoader@7de62196 for finders [BootstrapClasspathModuleFinder, BootstrapModuleFinder(org.wildfly.swarm.bootstrap:main), ClasspathModuleFinder, ContainerModuleFinder(swarm.container:main), ApplicationModuleFinder(swarm.application:main), org.wildfly.swarm.bootstrap.modules.DynamicModuleFinder@163370c2]]
	at org.jboss.modules.ModuleClassLoader.findClass(ModuleClassLoader.java:198)
	at org.jboss.modules.ConcurrentClassLoader.performLoadClassUnchecked(ConcurrentClassLoader.java:363)
	at org.jboss.modules.ConcurrentClassLoader.performLoadClass(ConcurrentClassLoader.java:351)
	at org.jboss.modules.ConcurrentClassLoader.loadClass(ConcurrentClassLoader.java:93)
	at org.dbunit.dataset.datatype.AbstractDataType.loadClass(AbstractDataType.java:224)
	at org.dbunit.dataset.datatype.AbstractDataType.loadClass(AbstractDataType.java:213)
	at org.dbunit.ext.postgresql.UuidType.getUUID(UuidType.java:77)
	... 204 more

Steps to reproduce
Wildfly swarm project with usual datasource fractions and postgres
JPA entity using UUID datatype like:
    @Id
    @GeneratedValue
    @Column( columnDefinition = "uuid", updatable = false )
    private UUID id;

yaml seed file that specifies the ID
arquillian test that loads the seed using APE annotation
Additional Information
Have tried upgrading to the APE 2.x line, no change.
Have rules out Deployment and Shrinkwrap issues.  Outputting the libs shows the Postgres classes are all there, and running a test which doesn't seed, but checks for the libs, works.  See below.
The classpath issue only occurs during the APE seed stage.  Is there a way to configure it to find the postgres lib?
 @Test
    public void testLibs() {
        try {
            Class.forName("org.postgresql.Driver");
            Class.forName("org.postgresql.util.PGobject");
        } catch (ClassNotFoundException e) {
            System.err.println("Where is your PostgreSQL JDBC Driver? "
                    + "Include in your library path!");
            e.printStackTrace();
        }
    }

Wildfly swarm declarative move (UsingDataSet annotation) does not work with
incontainer tests. You neef to run them as client or standalone
El 17 jul. 2017 6:43 a. m., "Brad Murray"<EMAIL_ADDRESS>escribió:

Issue Overview
Running a Wildfly Swarm project to build a JaxRS API using Arquillian for
automated testing.
Data layer is JPA entities.
Database is Postgresql
Making use of @UsingDataSet annotation with YAML files.
Expected Behaviour
When a datatype has an ID of type UUID, the following seed snippet will
work.
some_entity:

id: "uuid'2aad615a-d8e1-11e2-b8ed-50e549c9b654'"
other_attrs...

Current Behaviour
Exception thrown when it tries to seed.
Caused by: org.dbunit.dataset.datatype.TypeCastException: Unable to typecast value <uuid'2aad615a-d8e1-11e2-b8ed-50e549c9b654'> of type <java.lang.String> to uuid
at org.dbunit.ext.postgresql.UuidType.getUUID(UuidType.java:88)
at org.dbunit.ext.postgresql.UuidType.setSqlValue(UuidType.java:63)
at org.dbunit.database.statement.SimplePreparedStatement.addValue(SimplePreparedStatement.java:73)
at org.dbunit.database.statement.AutomaticPreparedBatchStatement.addValue(AutomaticPreparedBatchStatement.java:63)
at org.dbunit.operation.AbstractBatchOperation.execute(AbstractBatchOperation.java:211)
... 200 more
Caused by: java.lang.ClassNotFoundException: org.postgresql.util.PGobject from [Module "org.jboss.ironjacamar.jdbcadapters:main" from BootModuleLoader@7de62196 for finders [BootstrapClasspathModuleFinder, BootstrapModuleFinder(org.wildfly.swarm.bootstrap:main), ClasspathModuleFinder, ContainerModuleFinder(swarm.container:main), ApplicationModuleFinder(swarm.application:main), org.wildfly.swarm.bootstrap.modules.DynamicModuleFinder@163370c2]]
at org.jboss.modules.ModuleClassLoader.findClass(ModuleClassLoader.java:198)
at org.jboss.modules.ConcurrentClassLoader.performLoadClassUnchecked(ConcurrentClassLoader.java:363)
at org.jboss.modules.ConcurrentClassLoader.performLoadClass(ConcurrentClassLoader.java:351)
at org.jboss.modules.ConcurrentClassLoader.loadClass(ConcurrentClassLoader.java:93)
at org.dbunit.dataset.datatype.AbstractDataType.loadClass(AbstractDataType.java:224)
at org.dbunit.dataset.datatype.AbstractDataType.loadClass(AbstractDataType.java:213)
at org.dbunit.ext.postgresql.UuidType.getUUID(UuidType.java:77)
... 204 more
Additional Information
Have tried upgrading to the APE 2.x line, no change.
Have rules out Deployment and Shrinkwrap issues. Outputting the libs shows
the Postgres classes are all there, and running a test which doesn't seed,
but checks for the libs, works. See below.
The classpath issue only occurs during the APE seed stage. Is there a way
to configure it to find the postgres lib?
@Test
public void testLibs() {
try {
Class.forName("org.postgresql.Driver");
Class.forName("org.postgresql.util.PGobject");
} catch (ClassNotFoundException e) {
System.err.println("Where is your PostgreSQL JDBC Driver? "
+ "Include in your library path!");
e.printStackTrace();
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/arquillian/arquillian-extension-persistence/issues/93,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABcmYSAan6hAAtRHZRXFbjBoMVPiWBxaks5sOuZ-gaJpZM4OZi2W
.

I'm confused.  I've been using these tests for weeks now without issue, and the problem has only arisen with the introduction of entities that use a UUID datatype, which requires a conversion to a native Postgres datatype.
I've spent a full day digging into the issue, trying different things, but currently have no reason to believe that the APE classpath issue is not the only thing preventing the functionality from working.  Why would running as a Client solve the classpath loading the postgres jar?
My understanding is that running as Client treats the system as a black box, so I also don't understand how it could manage the transaction rollbacks.

Ok then it means that arquillian swarm has fixed some of old problems with
APE so forget about my words
El 18 jul. 2017 1:03 a. m., "Brad Murray"<EMAIL_ADDRESS>escribió:

I'm confused. I've been using these tests for weeks now without issue, and
the problem has only arisen with the introduction of entities that use a
UUID datatype, which requires a conversion to a native Postgres datatype.
I've spent a full day digging into the issue, trying different things, but
currently have no reason to believe that the APE classpath issue is not the
only thing preventing the functionality from working. Why would running as
a Client solve the classpath loading the postgres jar?
My understanding is that running as Client treats the system as a black
box, so I also don't understand how it could manage the transaction
rollbacks.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/arquillian/arquillian-extension-persistence/issues/93#issuecomment-315909973,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABcmYc-BRKJykU1LE0X_7khVN2tMgVW9ks5sO-hHgaJpZM4OZi2W
.

Just an update on this, haven't fixed the problem, but tried going around it by just provisioning the test-data directory with junit.  That approach however also fails, because I run into transaction boundary problems with RestAssured, which is being used to integration test the API.
It seems I need some of the glue in APE in order for Arquillian, JPA and RestAssured to play nicely together, but without this classloader bug being fixed, I can't use PG datatypes.
I'm happy to work with you guys to try and reproduce the issue, will work on seeing if I can create a failing test simply within APE.
@bartoszmajsak any thoughts?

@wyaeld did you had any luck with this?

Also having the same problem. Would very much appreciate it if anyone has any ideas on solving this.

I know this is old, but i have had issues with jboss/wildfly/swarm and now Thorntail. Lucky for me when it came to Thorntail i knew from jboss/wildfly how i could fix it. This is what i was able to do to get postgresql uuid to work. There may be a better way.
Steps
Create new directory structure under src/main/resources/
as
modules/org/jboss/ironjacamar/jdbcadapters/main.
Create newt module.xml in the main directory with this content.
<?xml version="1.0" encoding="UTF-8"?>
<module name="org.jboss.ironjacamar.jdbcadapters" xmlns="urn:jboss:module:1.5">
   <properties>
        <property name="jboss.api" value="private"/>
    </properties>

    <resources>
        <resource-root path="ironjacamar-jdbc-1.4.9.Final.jar"/>
        <artifact name="org.jboss.ironjacamar:ironjacamar-jdbc:1.4.6.Final" />
    </resources>

    <dependencies>
        <module name="javax.api"/>
        <module name="javax.resource.api"/>
        <module name="javax.transaction.api"/>
        <module name="javax.validation.api"/>
        <module name="org.hibernate.validator"/>
        <module name="org.jboss.as.naming"/>
        <module name="org.jboss.as.transactions"/>
        <module name="org.jboss.jboss-transaction-spi"/>
        <module name="org.jboss.ironjacamar.api"/>
        <module name="org.jboss.ironjacamar.impl"/>
        <module name="org.jboss.logging"/>
        <module name="org.jboss.threads"/>
        <module name="javax.xml.stream.api"/>
        <module name="org.postgresql" />
</dependencies>
</module>

For the version of thorntail i am running "2.0.0.Final" this is the version of ironjacamar-jdbc it uses.
Now you should be able to run with no errors. I hope this helps someone.

I know this is old, but i have had issues with jboss/wildfly/swarm and now Thorntail. Lucky for me when it came to Thorntail i knew from jboss/wildfly how i could fix it. This is what i was able to do to get postgresql uuid to work. There may be a better way.
Steps
Create new directory structure under src/main/resources/
as
modules/org/jboss/ironjacamar/jdbcadapters/main.
Create newt module.xml in the main directory with this content.
<?xml version="1.0" encoding="UTF-8"?>
<module name="org.jboss.ironjacamar.jdbcadapters" xmlns="urn:jboss:module:1.5">
   <properties>
        <property name="jboss.api" value="private"/>
    </properties>

    <resources>
        <resource-root path="ironjacamar-jdbc-1.4.9.Final.jar"/>
        <artifact name="org.jboss.ironjacamar:ironjacamar-jdbc:1.4.6.Final" />
    </resources>

    <dependencies>
        <module name="javax.api"/>
        <module name="javax.resource.api"/>
        <module name="javax.transaction.api"/>
        <module name="javax.validation.api"/>
        <module name="org.hibernate.validator"/>
        <module name="org.jboss.as.naming"/>
        <module name="org.jboss.as.transactions"/>
        <module name="org.jboss.jboss-transaction-spi"/>
        <module name="org.jboss.ironjacamar.api"/>
        <module name="org.jboss.ironjacamar.impl"/>
        <module name="org.jboss.logging"/>
        <module name="org.jboss.threads"/>
        <module name="javax.xml.stream.api"/>
        <module name="org.postgresql" />
</dependencies>
</module>

Notice the last modules is org.postgresql.
For the version of thorntail i am running "2.0.0.Final" this is the version of ironjacamar-jdbc it uses.
Now you should be able to run with no errors. I hope this helps someone.

Thanks, mann-ed.
I met similar issue when testing with inet type as below. And it is fixed via adding module postgresql-jdbc jar to module.xml of org.jboss.ironjacamar.jdbcadapters.
org.jboss.arquillian.persistence.dbunit.exception.DBUnitDataSetHandlingException: Failed while seeding database.
Caused by: org.dbunit.dataset.datatype.TypeCastException: Unable to typecast value <<IP_ADDRESS>> of type <java.lang.String> to inet
Caused by: java.lang.ClassNotFoundException: org.postgresql.util.PGobject from [Module "org.jboss.ironjacamar.jdbcadapters:main" from local module loader @15d9bc04 (finder: local module finder @473b46c3 (roots: .....)


Proper catching of the pgoapi.exceptions.NotLoggedInException

Expected Behavior
If this error occures, attempt a reconnect (ideally with the last known coords)
Actual Behavior
Bot crashes (smart as I was I forgot to copy the stack trace...) will try to deliver
Steps to Reproduce
Unsure.. I just noticed that I've been disconnected after a while
Other Information
OS: Mac 10.11.5
Git Commit: 9ea015e1fd9c5d269276de8a8e822ee8eb0afa0f

@skartoffel Please update to the latest dev branch by running this command:
git reset --hard
git pull -f

Please upgrade to the latest dev branch by running this command:
git reset --hard
git pull -f

Then run this command:
pip install --upgrade -r requirements.txt
If issue persists, open a new issue.
/closed


General weather description

Retrieving weather description by location (e.g. "cloudy", "slightly raining", "foggy")

closing due to release 1.0


LuaJIT and DynASM updates from upstream

LuaJIT was updated to LuaJIT/LuaJIT:v2.1
DynASM was updated to capr/LuaJIT:v2.1-dynasm-luamode which is the new upstream[1] for DynASM-luamode

NOTE: you can pull this update directly from capr/snabbswitch:master which is the branch I made this PR from.
[1] the old upstream was luapower/dynasm:master but that has the "weird" luapower directory structure, plus now you can patch-in luamode over any up-to-date luajit fork by applying the patch at https://github.com/capr/LuaJIT/commit/403f386372adb9080594091f485b3441a6e8eda8.patch
with patch -p1 | curl <above url> from the LuaJIT root directory.

if merged, we should close https://github.com/SnabbCo/snabbswitch/pull/705.

Looks like the LuaJIT Makefile has a bug that it misses some dependencies that need to be rebuilt after a DynASM upgrade? (If I'm reading the CI error correctly.)

@eugeneia @capr: The "make clean" compile issue seems to be a bug in upstream LuaJIT. I filed an issue as luajit/luajit#126. I suggest we merge this code but also pull any fix that happens to materialize before the release.


pass notification object instead of self

This fixes bug #22.

oh, hah ok I do like this for the convenience method. thank you!


demo crashes when run from build folder

make demo works as does running ./bin/demo from the repo root directory.  However if I cd to the bin directory and ./demo the program crashes with the error:
OpenCV Error: Assertion failed (ssize.width > 0 && ssize.height > 0) in resize, file /home/jason/mysource/opencv/modules/imgproc/src/resize.cpp, line 4044 terminate called after throwing an instance of 'cv::Exception'   what():  /home/jason/mysource/opencv/modules/imgproc/src/resize.cpp:4044: error: (-215) ssize.width > 0 && ssize.height > 0 in function resize
Two small windows briefly appear but the larger demo one never does.  I will see  if I can figure anything out, but help would be great.  This is a great project by the way.

I assume the problem is this line:
auto image = cv::imread("res/demo.jpg");

Thank you for reporting! Fixed in #31


Bug: Lazybones is no longer available

Bug report
Lazybones is no longer possible to install. It is also not possible to use an already installed one.
To reproduce
$ sdk ls lazybones

Stop! lazybones is not a valid candidate.

$ sdk install lazybones

Stop! lazybones is not a valid candidate.

$ ls ~/.sdkman/candidates/lazybones/

0.8.3  current

$ ~/.sdkman/candidates/lazybones/current/bin/lazybones

Lazybones is a command-line based tool for creating basic software projects from templates.
...

System info
SDKMAN 5.11.6 on Linux

Hi @jest, please follow the contributing guidelines before opening a bug report here. This is not a bug. Please chat to us on Slack first.

@marc0der Well, I consider disappearing programs to be a bug of software manager.
Can you please point me to a commit or some announcement to watch for? I can find no mention in sdkman repos (GitHub search) or on a webpage on such a change.
I'm sorry, I will not create another Slack account just to ask a useless question - if it's not a bug then I see no prospect of a corresponding change in SDKMan and I just have to use other means to obtain lazybones.
Thank you for the otherwise great product, anyway!


Field names for the deletion count from the V 3’, 5’ and 3’ D, and 5’J

We are looking for field names for the deletion count from the V 3’, 5’ and 3’ D, and 5’J. This is related to issue #139 and alakazam issue 96. We would like to use names that are decided by the community and can be standard names. Let us know what your preference is, or if you have additional options.

Description
Option 1
Option 2

Number of 3' V germline nucleotides deleted
v_germline_deleted_3
d3v_length

Number of 5' D germline nucleotides deleted
d_germline_deleted_5
d5d_length

Number of 3' D germline nucleotides deleted
d_germline_deleted_3
d3d_length

Number # of 5' J germline nucleotides deleted
j_germline_deleted_5
d5j_length

Option 3: e3v_length, ...
Option 4: e3v_deletion, ...
"e" for excision/exonuclease.

If suggestions are possible:
nd_3v (number deleted 3' V)
nd_5d (number deleted 5' D)
nd_3d (number deleted 3' D)
nd_5j (number deleted 5' J)
I find length of deleted nt conceptually difficult.
On Tue, 22 Jun 2021, 18:26 Jason Vander Heiden, @.***>
wrote:

Option 3: e3v_length, ...
Option 4: e3v_deletion, ...
"e" for excision/exonuclease.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/airr-community/airr-standards/issues/536#issuecomment-866136357,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRD6RB2BJPM75FJCM65BRDTUC2UVANCNFSM47EAVPGQ
.

My two cents:

What ever the naming scheme is, we need 6 fields in total: 1x V, 2x D, 2x D2, 1x J
While I agree that @ssnn-airr's Option 2 is the most consistent with our current naming, I would find it confusing to have names like d3d2_length.
Do we agree that the values of these field have be integers >= 0?
Is the statement "length(np1) = np1_length = n1_length + p3v_length + p5d_length" always true, irrespective of the inferrence tool?

While I agree that @ssnn-airr's Option 2 is the most consistent with our current naming, I would find it confusing to have names like d3d2_length.

Yeah, that's kinda why I'm leaning towards something like "e", "ex", etc.
Just for references, the other detailed junction alignment fields are:

n1_length
n2_length
n3_length
p3v_length
p5d_length
p3d_length
p5d2_length
p3d2_length
p5j_length

Do we agree that the values of these field have be integers >= 0?

I do, if we include null.

Is the statement "length(np1) = np1_length = n1_length + p3v_length + p5d_length" always true, irrespective of the inferrence tool?

This seems right to me, for heavy chains at least; light being np1_length = n1_length + p3v_length + p5j_length.

Is there a way to "vote" here?
Options:

Option 1: v_germline_deleted_3
Option 2: d3v_length ("d" for deletion, confusing if 2 d_calls: d3d2_length)
Option 3: e3v_length ("e" for excision/exonuclease)
Option 4: e3v_deletion ("e" for excision/exonuclease)
Option 5: nd_3v ("nd" for number deleted)

I liked 1, but given that "Option 2 is the most consistent with our current naming"... my vote goes for Option 3 :)

While I agree that @ssnn-airr's Option 2 is the most consistent with our current naming, I would find it confusing to have names like d3d2_length.

It isn't necessary that we use prefixes and make the names short and obtuse. I wouldn't mind seeing some options that make the names more self-documenting.


Support PDF417 barcode images flipped along the horizontal axis

This situation is apparently common with ZA driver licenses.
Fixes #152
Needs an integration test.

Thanks. I will add a sample image to the test set.


[SMALLFIX] static import of TieredBlockStoreTest.java

static import of TieredBlockStoreTest.java

Merged build finished. Test PASSed.

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


error while using Socket(URL).connect()

Error :
Traceback (most recent call last):
  File "/usr/lib/python3.9/socket.py", line 953, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -2] Name or service not known

Screenshots
Here is my script that produces the error

System information

OS: Linux,Parrot os
Version of supabase-client    ==    0.2.5
Version of python == 3.9.2
version of realtime == 0.0.5

Additional context
Also i cant seem to install realtime version 0.1.1a0
ERROR: Could not find a version that satisfies the requirement realtime==0.1.1a0 (from versions: 0.0.2, 0.0.3, 0.0.4, 0.0.5)
ERROR: No matching distribution found for realtime==0.1.1a0`pip install realtime==0.1.1a0

here is the full error trackback:
Traceback (most recent call last):
  File "/home/scelester/Desktop/finland-backend/test.py", line 13, in <module>
    s.connect()
  File "/home/scelester/.virtualenvs/fin-land_rasp/lib/python3.9/site-packages/realtime/connection.py", line 90, in connect
    loop.run_until_complete(self._connect())
  File "/usr/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
    return future.result()
  File "/home/scelester/.virtualenvs/fin-land_rasp/lib/python3.9/site-packages/realtime/connection.py", line 94, in _connect
    ws_connection = await websockets.connect(self.url)
  File "/home/scelester/.virtualenvs/fin-land_rasp/lib/python3.9/site-packages/websockets/legacy/client.py", line 659, in __await_impl_timeout__
    return await asyncio.wait_for(self.__await_impl__(), self.open_timeout)
  File "/usr/lib/python3.9/asyncio/tasks.py", line 481, in wait_for
    return fut.result()
  File "/home/scelester/.virtualenvs/fin-land_rasp/lib/python3.9/site-packages/websockets/legacy/client.py", line 663, in __await_impl__
    _transport, _protocol = await self._create_connection()
  File "/usr/lib/python3.9/asyncio/base_events.py", line 1017, in create_connection
    infos = await self._ensure_resolved(
  File "/usr/lib/python3.9/asyncio/base_events.py", line 1396, in _ensure_resolved
    return await loop.getaddrinfo(host, port, family=family, type=type,
  File "/usr/lib/python3.9/asyncio/base_events.py", line 856, in getaddrinfo
    return await self.run_in_executor(
  File "/usr/lib/python3.9/concurrent/futures/thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/lib/python3.9/socket.py", line 953, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -2] Name or service not known

nevermind i just removed "&vsn.." from the url and it stated working.


[release-1.4] Use k8s.io/apimachinery yaml decoder in kubesource

This is an automated cherry-pick of #19045

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio_release-1.4
8eadf1103edab58b2a556e46ac1108cae91e2c0b
link
/test lint_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Sense is now free and open source.

updated the list

@bleskes  thank you! That's a challange: I want to leave both!


How long for forwarding?(224x224 image)

Hi, can you share your time for forwarding of a 224x224 image? And is it as fast as the speed reported in the mobilev2 paper.
I have train this model with caffe, but the speed is lower than mobilenetv1, also the paper.

@Bo331429856   yes, the speed is depended on the efficiency of the depthwise convolution layer.  The version of tensorflow Google used may do a lot of optimizations to speed up.

HI @suzhenghang :
How is your performance on your machine?
Would you please report your FPS ?


Use CommandLineParser instead of custom parsing args (WIP Early Feedback)

What I need help with
There are a large amount of changes, but only a few files I'd like reviewed for design:

CommandSettings.cs
CommandArgs.cs
AgentListener/Program.cs

Please don't bother with the other files for now.
Still to do:

Update unit tests (large task)
Verify Options on Remove
Dump args to log
General cleanup

Differences found so far:

The old parser allowed any flag or option, to be on any command (even if it is unused).  The new library requires flags and options to be attached to the verb.  I think this is a good change.
The old library would print unknown options to the screen and continue.  The new version has 2 options

Ignore unknown args (no error printed)
Don't allow unknown args
We should choose an approach here but I think 2 makes sense.  we'd have a nice error message saying why we failed.

It would be helpful to see an example of the old and new command lines next to each other for comparison.


Feature: Display conflicts modal when pasting on FTP locations

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #8609
Related #12228 (Files would still crash if the root / wasn't specified [e.g. ftp://localhost:21])

Changes

There was a duplicated class that was causing issues (FtpManager), so I left only one
I created IFtpStorageService which is essentially a copy of IStorageService. I might remove it and inject the service without its relative interface. @yaira2 what are your thoughts on this?

Limitations
~Pasting from FTP to FTP isn't showing the conlicts dialog (I'm working on this)~ Fixed
Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[x] Did you implement any design changes to an existing feature?

[x] Was this change approved?

[x] Are there any other steps that were used to validate these changes?

Open app and an FTP connection
Paste an item with a conflicting name

I created IFtpStorageService which is essentially a copy of IStorageService. I might remove it and inject the service without its relative interface. @yaira2 what are your thoughts on this?

Can you make FtpStorageService inherent from IStorageService?

I think it would cause issues once injected. We would have two different instances of the interface

What are the differences between the services?

What are the differences between the services?

There are no differences, but we would inject two IStorageService instances, Native and Ftp. I think this might cause some issues when retrieving them. I'll try and see what happens

I meant, why did you create IFtpStorageService?

I meant, why did you create IFtpStorageService?

Because I thought that if I have to retrieve it as IStorageService, but I have two instances of that (NativeStorageService and FtpStorageService), there would be issues since it may give me once Native and once Ftp
The two classes implement the interfaces but do not extend it

Maybe instead of duplicating IStorageService we could reuse it? I can think of two ways of doing it:

Use FtpStorageService in NativeStorageService and redirect calls to the former one when appropriate.
Manually inject (through the constructor) FtpStorageService instead of Ioc

I'm more into the second option because the first one would introduce more mess. The only difficulty with the second option is the injection itself -- the UI would need to inject appropriate implementation into the View Model
@ferrariofilippo

Perhaps you have something else on your mind? @ferrariofilippo

Not really, I'm not really an expert on this, that's why I used this bad "hack"

Manually inject (through the constructor) FtpStorageService instead of Ioc

I need FtpStorageService in a static method, how should I manage it?

You could inject it by passing it as a parameter however, then we'd have no guarantee that appropriate service was passed. Unless a major refactor in this area is done, that's going to be an issue. We could perhaps continue with your approach, but to bit reduce code duplication and increase reusability you could inherit from IStorageService
public interface IFtpStorageService : IStorageService


Inability to search by UUID

Environment

Python version: -
Nautobot version: 1.2.7

Steps to Reproduce

Use object UUID for search in Nautobot GUI search field

Expected Behavior
Object with given UUID should be found and returned in results.
Observed Behavior
Either non results or irrelevant IP address object is the only returned as a "matching" result.

This may be a duplicate of #923?

Yes, please.
Sorry for this duplicate. I somehow didn't notice it although I tried to search (honestly, only quickly) for such a topic.


backport: crl from main to release-1.3 branch

This PR backports changes related to crl cache from main to release-1.3 branch.
PRs that's backporting:
https://github.com/notaryproject/notation/pull/1076
https://github.com/notaryproject/notation/pull/1078
https://github.com/notaryproject/notation/pull/1079
This PR targets release-1.3 branch.

It looks like this PR includes #1076, #1078, and #1079.
For #1079, which is the test for the entire CRL feature and not limited to the cache, should we update the PR name to:

backport: CRL cache and CRL E2E test from main to release-1.3 branch

Also, please mention the PRs that we are backporting in the PR description. Thanks!

It looks like this PR includes #1076, #1078, and #1079.
For #1079, which is the test for the entire CRL feature and not limited to the cache, should we update the PR name to:

backport: CRL cache and CRL E2E test from main to release-1.3 branch

Also, please mention the PRs that we are backporting in the PR description. Thanks! @Two-Hearts

@JeyJeyGao Updated.

Is it expected that the test/e2e/go.mod bump is also included in this PR?

Is it expected that the test/e2e/go.mod bump is also included in this PR?

It was brought in by PR: https://github.com/notaryproject/notation/pull/1079.


Add Path enumeration Info KB entry.

why is this INFO ? shouldnt it be HIGH ? or atleast potentially if its flaky !

why do you think it should be high? this is literally path enumeration by query type. why would it be high?


Task 'aggregateJavadocs' not found in root project

I add below script to the build.gradle of my root android project and run command  ./gradlew aggregateJavadocs. got below errors. Can anyone provide some help?
Download https://jcenter.bintray.com/com/netflix/nebula/gradle-aggregate-javadocs-plugin/maven-metadata.xml
Download https://jcenter.bintray.com/com/netflix/nebula/gradle-aggregate-javadocs-plugin/2.2.1/gradle-aggregate-javadocs-plugin-2.2.1.pom
Download https://jcenter.bintray.com/com/netflix/nebula/gradle-aggregate-javadocs-plugin/2.2.1/gradle-aggregate-javadocs-plugin-2.2.1.jar
FAILURE: Build failed with an exception.

What went wrong:
Task 'aggregateJavadocs' not found in root project 'XXXXXXX'.

I too followed the same steps mentioned in the Readme, but faced the same issue.
Finally had to do this manually as specified in the below link:
https://stackoverflow.com/questions/29162820/how-to-create-javadoc-using-android-studio-without-r-and-buildconfig


Version 0.2.1 broken?

Hi,
after updating the plugin from 0.2.0 to 0.2.1 I get an error when clicking on the poll icon in the composer:
Error Invalid request
I can´ t see any error messages in the log though.

Update to 0.2.2 and try again

I have the same problem on 0.2.2 (just installed plugin on the latest nodebb version - 1.0.3)

Are you logged in? Where are you clicking on the icon? As in, what category? In a topic?
When you're in a spot where it doesn't work, can you execute the following code snippet in your browser JavaScript console and report back here with what is printed out:
require(['composer'], function(c) {console.log(composer.posts[composer.active].cid);});

Oh, sorry, works!
I have not noticed that the poll can be attached only to the first post.
The error is came up, when I trying add poll to the second post in the topic.
I think, will be good disable btn (disabled state and cursor style not-allowed), when somebody trying click on the btn in the second post.

Unfortunately that's a lot harder than it's worth, at the moment at least. I've since changed the button click flow though, so the error should at least be clearer.


On Ceiling Method Addition

Hi Guys,
I found out, there is a shortcut for controlling on floor but there is no shortcut for onCeiling control, i tought it might be useful to implement a same way for controlling if body collides ceiling or not.

Today i asked about how to use grunt and tested my code with
if(this.activeSprite.body.onFloor() | this.activeSprite.body.onCeiling() ){
        this.changeSprite("dead");
}

And its working. I tought if there is a function to control is body onFloor, there should be a function to control is body onCeiling.

Make sure the jsdoc is correct, including the method name.
Curious question: why not just using .blocked.up directly?

@pnstickne .blocked.down|up only works for TileMaps but not for other Arcade bodies - you'd have to check with touching or wasTouching, although these values change with every single collision if I remember correctly.

This may be old, but why not.


Update README.md with spelling fix

Apologies, I had already edited this by the time I saw this PR. Closing for now...


No intellisense on vanilla WSL Ubuntu setup

Fresh Windows setup (on Parallels, but that shouldn't matter)
Install Ubuntu LTS in WSL on Windows
By default, python3 exists...
Install VS Code Stable and then the Remote - WSL extension
In the WSL terminal, create a folder and then open code on it (code .)
Create a python file
Type in print and try to get Completions/IntelliSense
Result: No IntelliSense
I see this in the dev tools console:
workbench.main.js:238 [Extension Host] Error Python Extension: 2019-07-03 15:42:50: stderr jediProxy Error (stderr) Traceback (most recent call last):
  File "completion.py", line 645, in <module>
    import jedi
  File "lib/python/jedi/__init__.py", line 41, in <module>
    from jedi.api import Script, Interpreter, set_debug_function, \
  File "lib/python/jedi/api/__init__.py", line 27, in <module>
    from jedi.api.environment import InterpreterEnvironment
  File "lib/python/jedi/api/environment.py", line 14, in <module>
    from distutils.spawn import find_executable
ModuleNotFoundError: No module named 'distutils.spawn'

If I add "python.jediEnabled": false to my settings.json everything starts to work.
So out of the box, we don't get completions on WSL. Is the new language service going to be enabled by default soon?

ok if i install jedi i get IntelliSense.  Maybe this needs to go into the extension readme...

So this is Ubuntu 18.04? And are you using the system install of Python or did you explicitly request it be installed? I ask because distutils.spawn is in Python's stdlib, but Ubuntu/Debian are notorious for taking bits of the stdlib out (e.g. venv). And if installing pip pulls in distutils that would explain why this suddenly started working, not the install of Jedi.
P.S.: I strongly advise against globally installing into Python at all, but especially if you are using the system Python as you can very easily break your Linux installation that way.

Hi @chrisdias 👋 I could not replicate the issue, here's what I did:

installed Ubuntu 18.04 LTS on Windows
downloaded the "Remote Development" extension (on VS Code Insiders 1.37 commit bbfd5)
in the Ubuntu terminal, created a folder and then opened it with code-insiders .
created a python file, it then prompted me to install the Python extension, which I did
then it tried to install pylint, but I got an error message saying that pip didn't exist so I ran

sudo apt-get update, followed by
sudo apt-get install python3-pip (here I closed insiders).

⚠ as @brettcannon pointed out, this pulls distutils:

Setting up python3-distutils (3.6.8-1~18.04)

👉 reopened insiders and intellisense was working with Jedi enabled, I didn't have to install it.

@brettcannon It is Ubuntu 18.04 and it is the system install of Python, running in WSL.  So the image comes straight from the windows store.
@kimadeline I did not try to install pylint, but it looks like its the same result as brett calls out, installing the distutils causes it to work.  Again, I'm suggesting this is something in a readme, not a bug to be fixed.

@chrisdias , thanks for sharing about installing jedi! I'm new to wsl and it took me 2hours trying to debug why the intellisense wasn't working... After installing jedi via pip, it works!

This was fixed on the latest release. Please try it out and we'll reopen in case you still see this issue.
Thanks!


Add debugging information

Pull requests:

https://github.com/senzing-garage/docker-installer/pull/73


Change CLI parameters in qiita maintenance sysmessage

Currently we are providing a --time parameter, in seconds, and the developer needs to compute how many seconds are needed for the deadline.
Ideally we want to provide the time in a standard format, for example: MM/DD/YYYY-HH:MM:SS and put that time as the deadline.

After using this feature since it's creation I think changing this to hours vs. seconds will make the most sense. I normally set it up for 1-2 hours or 1-2 days. Thus, hours should suffice.


Allow all entity-containing items to be used with powered spawners and friends.

I believe items such as vanilla mob spawners, mob eggs, powered spawners, soul vials, golden lassos, cursed lassos, safari nets, broken spawners, etc should all be allowed in the soul binder and powered spawner and stuff.
Those are all entity-containing items. Some of them can be emptied, while others cannot. Some of them are consumed, while others are not. Some of them can be rebound, while others cannot.
Mob eggs in a soul binder should consume the mob egg. Soul vials in a soul binder should do what they already do. Safari nets should empty them. Vanilla mob spawners should never be consumed. Powered spawners and broken spawners should also never be consumed, thus allowing cloning of a spawner type.
This might require a new forge API, which for some reason we still don't have. We've had entity-containing items for years but we have yet to have an API for entity-containing items. Maybe this'll push for one.

Wrong tracker. Suggestions: http://ideas.theideawall.com/EnderIO

"Wrong tracker" says you.


Help people quickly build UnivariateFinite object

It seems that people are often getting tripped up by UnivariateFinite object. Typical situation:

User has some way of getting a vector of scores s
User assumes that given it's just a single vector of scores it's obvious it's a binary case
User gets annoyed because they can't just use auc or some similar measures.
User asks and we end up saying something like:

c = classes(categorical([:label0, :label1])[1])
y = [UnivariateFinite(c, [1-s[i], s[i]]) for i in s]

what would be good is to just have a simple way to do this directly something like
UnivariateFinite(s) # ==> does the above
UnivariateFinite(s, [:label_0, :label_1]) # ==> does the above with specific labels
UnivariateFinite(w) # ==> with w has c columns, do it with c labels
UnivariateFinite(w,  [:label_0, :label_1, :label_2]) # ==> same but specifying labels

Solving this would go along with solving https://github.com/alan-turing-institute/MLJ.jl/issues/511

I think this is a great idea. While not abolutely necessary, it would be less work overall to incorporate with this remedy to another issue.


verity-boot: append hash device to rootfs

Turned out that dev nodes for SCSI devices may not be
determenistic, where the hash device and rootfs may end
up appearing under /dev/sda and /dev/sdb respectively.
Instead of mounting a separate hash device, append the
verity Merkle tree to rootfs ext4 filesystem, similarly
to how it's done for layer VHDs and mount single VHD.
Remove redundant hash device code.
The default GuestStateFile filename was changed to kernel.vmgs.
Update the IVGM kernel init to reflect the changes.
The kernel command looks something like this:
 8250_core.nr_uarts=0 panic=-1 debug loglevel=7 root=/dev/dm-0 \
     dm-mod.create="dmverity,,,ro,0 173768 verity \
         1 /dev/sda /dev/sda 4096 4096 21721 21721 sha256 \
         42896a788a58da77b6acb8ddf53aa744bd269c19146cfdf48eb8fc5529a52e62 \
         a1c38923e44adffdd21f84e9185248c884fa28e767795d1025e5804e1c3df905" \
     init=/startup.sh

To break this down a little further:
  dm-mod.create="<name>,<uuid>,<minor>,<flags>,[table {verity_params}]"
  table="<start_sector> <num_sectors> <target_type> verity_params"
  verity_params="<version> <data_device> <hash_device> <data_block_size> \
                 <hash_block_size> <num_data_blocks> <hash_start_block> \
                 <algorithm> <root_digest> <salt> [<opt_params>]"

With the example above we get:
  name: "dmverity"
  uuid: ""
  minor: ""
  flags: "ro"
  table: 0 0 173768 verity <verity_params>
  verity_params:
    version: 1
    data_device: /dev/sda
    hash_device: /dev/sda
    data_block_size: 4096
    hash_block_size: 4096
    num_data_blocks: 21721
    hash_start_block: 21721
    algorithm: "sha256"
    root_digest: "42896a788a58da77b6acb8ddf53aa744bd269c19146cfdf48eb8fc5529a52e62"
    salt: "a1c38923e44adffdd21f84e9185248c884fa28e767795d1025e5804e1c3df905"

The support for booting non-SNP UVMs with dm-verity has also been added
as part of this PR. A new annotation can be used to pass the dm-mod.create
parameters to kernel. The assumption that the rootfs VHD will also have Merkle
tree appended after ext4 filesystem still holds. The new annotation is
"io.microsoft.virtualmachine.lcow.dmverity-create-args" and must be used
in conjunction with an existing "io.microsoft.virtualmachine.lcow.dmverity-mode"
annotation.
Add an internal "io.microsoft.virtualmachine.console.pipe" annotation, which
can be used to set the serial for the UVM for debugging purposes.
Note that dm-verity boot has a dependency on CONFIG_DM_INIT kernel config.
Signed-off-by: Maksim An<EMAIL_ADDRESS>

Clearly fabulous. Minor changes to some comments to clarify some things.

most of it was already in place, thanks to you/your team 😄


InvalidOperationException if language is Turkish (tr-TR)

An exception InvalidOperationException is thrown if the current language is Turkish (tr-TR). Additional information: No 'id' member found on type MyDataObject
I guess this is related to invalid use of string comparison or string modifications methods. Tested with the latest official release (Windows 10 UWP app). This issue breaks my app for any Turkish users.
Stack trace:
   at Microsoft.WindowsAzure.MobileServices.MobileServiceContractResolver.DetermineIdProperty(Type type, IEnumerable`1 properties)
   at Microsoft.WindowsAzure.MobileServices.MobileServiceContractResolver.CreateProperties(Type type, MemberSerialization memberSerialization)
   at Newtonsoft.Json.Serialization.DefaultContractResolver.CreateObjectContract(Type objectType)
   at Microsoft.WindowsAzure.MobileServices.MobileServiceContractResolver.CreateObjectContract(Type type)
   at Newtonsoft.Json.Serialization.DefaultContractResolver.CreateContract(Type objectType)
   at Microsoft.WindowsAzure.MobileServices.MobileServiceContractResolver.ResolveTableName(Type type)
   at Microsoft.WindowsAzure.MobileServices.MobileServiceClient.GetSyncTable[T]()

Possible fix in MobileServiceContractResolver.DetermineIdProperty:
JsonProperty[] idProperties = properties.Where(p => idPropertyNames.Contains(p.PropertyName) && !p.Ignored).ToArray();
=>
JsonProperty[] idProperties = properties.Where(p => idPropertyNames.Contains(p.PropertyName, StringComparison.OrdinalIgnoreCase) && !p.Ignored).ToArray();
In addition I would check all uses of Contains (or preferably all string comparisons / modifications).

Hi there, thank you for using Azure Mobile Apps. Please look at this section of the ReadMe that highlights the future of Azure Mobile Apps. Looks like this issue was opened long back, I'm closing this for now but feel free to reach out in case you have any questions. Thanks!


Add highest and lowest fares to each leg of an itinerary

Summary of changes
Asana Ticket: 🎫 Trip Planner | Compile fare information for an itinerary
Add highest and lowest fares to each leg of an itinerary. Not used anywhere yet, but will be in follow-up work.

Before getting review, please check the following:

[ ] Does frontend functionality render and work correctly in IE?
[ ] Have we load-tested any new pages or internal API endpoints that will receive significant traffic?
[ ] Are interactive elements accessible to screen readers?
[ ] Have you checked for tech debt you can address in the area you're working in?
[ ] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?
[ ] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages?

@alicia-mbta I've added a test and typespecs, and piped the data into the view for later use.
@thecristen pining you just in case you are interested in looking over this as well.


RELEASE

Coverage increased (+1.5%) to 90.046% when pulling 8cdaa722e034b181151976820412664d87ef9b76 on develop into c3d72da72f2c265e57ef9853e47a504f67083ae4 on master.


feat: Time limited Secrets. Can be used with aiven-kafka-topics debugging tool: Debuk

Names and appearance of structs could be discussed.
Feature will help developers debug a topic/application for an expiredAt timestamp. This do not have a limt.
This feature can be used with tool: https://github.com/nais/debuk.


Add numeric literal support to noDuplicateCase lint rule

Closes #544

Now that I think about it, this probably shouldn't get merged until after the PR that changes the formatted snapshot name. I should update this snapshot before it lands, so I'll convert this to a draft for now.

Thank you!


Update README.md

uses environment variables for happi steps

Can you test to see if the variable expands appropriately inside of the happi.ini file?

tried to use variable inside of ini, e.g.
path = %USERAPPDATA%\happi\happidb.json

I did not get it to work.  If you used a cli approach to write the ini, it could probably work

echo [DEFAULT] > %LOCALAPPDATA%\happi\happi\happi.ini
echo path = %LOCALAPPDATA%\happi\happi\happidb.json >> %LOCALAPPDATA%\happi\happi\happi.ini

I leave it to you which is better


Trading feature needs more delay

It often happens that when using any message button the input to open the chat is missed and it ends up opening any menu with hotkeys.
The solution is to add a delay between inputs, a 50-150 milliseconds delay was enough if I recall correct.

Have you tried just making the initial delay longer? Just making sure the
chat is open before you start sending keys might be enough, and just one
delay will not affect the time it takes to use it in any significant way.
El mié., 19 dic. 2018 a las 12:23, Tobias Hoffmann (<
<EMAIL_ADDRESS>escribió:

Yup, this is known and pretty frustrating. I'll try to find a better
solution than to add more delays though, it already has slight delays built
in.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/klayveR/xenontrade/issues/29#issuecomment-448561203,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB3iLmMr90m3j5AjLCeRXWDY48KOxVRIks5u6iHLgaJpZM4ZZ9_Q
.

--
Siete días siete Causas


Improve UX/UI of Account page in VENFT App

The whole UI of VENFT App needs refactoring to be much nicer and easier to understand :D so feel free to place some tips for UX/UI.
The Account Page is first what people discover after login. It is good to start here. The Account Page contains some advanced tools which are not necessary to be there (we can move them to tools page).
This page makes sense to refactor after:

#55
#57
#58

I think we did good move in the #101 in VEBlazor library. There is new page for account. It is little similar to old one, but much better responsive. In the transaction list it contains the possibility to add new transaction (classic, tokens, split, airdrop).
I will close this issue now. We need to add some specific things to the account page and it will be more clear to create new issues for the specific tasks.


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


3.4Rc4 - ${metadata.name} seems to always get the workflow name, not pod name

Checklist

[x] Double-checked my configuration.
[x] Tested using :latest images.

Summary
What happened/what you expected to happen?
In accordance with the documentation, I should be able to place ${metadata.name} into a custom pod link and it should output the pod name. Instead, it always outputs the workflow name.
This is a regression from 3.3.
I have tried with POD_NAMES set to v1 and this seems to resolve the issue.
What version are you running? 3.4 rc4

Message from the maintainers:
Impacted by this regression? Give it a 👍. We prioritise the issues with the most 👍.

@JPZ13 Would you like to look into this when you get a chance?

@JPZ13 Would you like to look into this when you get a chance?

Happy to. I'm a little tied up for the rest of the day, but @rohankmr414, could you take a look at this today and post any of your findings? I'll then take it from there
Also, @terrytangyuan what do we want the ideal behavior to be? The documentation says ${metadata.name}: Name of the current workflow / pod / event source / sensor but now that the workflow name and the pod name have diverged with pod names v2, it might make more sense to have the ${metadata.name} variable be the workflow name and add another variable to get the pod name. More than happy to go with whichever cc @tim-sendible

@terrytangyuan @JPZ13 is it blocker for 3.4 GA?

@sarabala1979 I personally think it makes more sense to have ${metadata.name} interpolate to the workflow name and wouldn't consider this a 3.4 GA release blocker, but I also want to hear @terrytangyuan and @tim-sendible's thoughts

Workflow name sounds more reasonable to me.

Still an issue.

I synced with @tim-sendible today - the ${metadata.name} field is working properly in the workflow-controller-configmap

I would suggest closing this issue

Agreed. From what I saw on @JPZ13's screen it all looks good now. Thanks.

Still an issue with 3.4.1! This used to work in previous versions. Please allow to reference the pod name in links

It almost works in 3.4.2: For this link:
http://grafana.home.local/explore?orgId=1&left=%7B%22datasource%22:%22Loki%22,%22queries%22:%5B%7B%22expr%22:%22%7Bcontainer%3D%5C%22main%5C%22,pod%3D%5C%22${metadata.name}%5C%22%7D%22,%22refId%22:%22A%22,%22editorMode%22:%22code%22,%22queryType%22:%22range%22%7D%5D,%22range%22:%7B%22from%22:%22now-3h%22,%22to%22:%22now%22%7D%7D
${metadata.name} correctly is resolved to a pod BUT %3D is not resolved to a = so loki gets this query:
{container%3D"main",pod%3D"backfill-worker-go62jrw-962151937"}
instead of this:
{container="main",pod="backfill-worker-go62jrw-962151937"}

@boniek83 - this is a different issue to the one I raised. I have covered it in #9935


Face mask does not work properly with "Traveling order"

Video
When attempting to edit the face mask of an LED pattern, it works correctly when using it normally. When enabling "Traveling order however, it masks the wrong faces. See video for example.

Closing this since the app is being re-written anyway


Query of missing property now returns None

Prior to this change, if a property did not exist in the current state, a query for it via "property path" syntax:
value = syc.setup.solution_control.time_step_size
would return an empty dictionary.
This is inconsistent with how System Coupling behaves and also with how existing but unset properties are represented in the state. In the case here, the property time_step_size could be missing entirely from the state either because solution_control is uninitialised or because other set-up renders time_step_size unavailable. (Either way, time_step_size is recognised as a potentially available variable - an exception would be thrown if an arbitrary property name were queried.)
The cause of the issue was the implementation delegating to the System Coupling GetState query which, although it mainly works for property paths, has the behaviour of returning an empty dictionary if the property does not exist in the state. Calling GetParameter has better behaviour. Given that we discriminate between object and property state in the calling code anyway this was a simple change.

@ochernuk this is the root cause of the Time/Iteration mixup in the charts, but I thought it better to pull it out to its own PR.


Output ports in GPU not working correctly

The ports on the GPU only support input, but do not support output.
Expected Behavior
When I write
mov port0, X
or
mov 63488, X
I expect this to reflect on the IOBus and there will be X at 0 cell
Current Behavior
For some reason this only works for input, i.e. mov X, port0 works as it should, but
mov port0, X does not work, or rather it works, and X really gets into cell 63488, but it is not accessible from the outside.
Possible Solution
I'm not sure, but in the code I have not seen any logic related to writing to IOBus, read only.
Steps to Reproduce
1.Enter the following code in the editor:
Main();
#include <drivers\drv_gl.txt>
void Main()
{
      glVertexMode( 1 );
      mov port0,220
      glClear( 70, 70, port0, 255 );
      glBindTexture( 'brick/brick_model' );
      glColor( 255, 255, 255, 255 );
      glRectWH( 128, 128, 256, 256 );
      glExit();
}

2.Install GPU somewhere
3.Install near Data port
4.Install a screen next to it
5.Connect screen to port0 of Data Port
6.Connect IOBus to Data Port
7.Refresh GPU
0 should still remain on screen
Context
I would like to send requests from the GPU to receive any data from the outside, because the internal memory of the GPU is limited, and for this IOBus and MemBus are provided there, but for some reason, they both work only for input.

I think this is intended since Gpu is clientside, but I'm not 100% familiar with it.

The GPU has code for sending writes to the server, but only for the membus (though there is no reason this couldn't be extended to support the IObus).

While the documentation doesn't say much about port I/O, the way the MemBus is described kinda implies that it's the only way to send data back to the server.
Chapter 7, section 1.3 (page 246) - Features

The memory area between addresses 65536 and 131071 (inclusive) is reserved as an
additional external memory bus. It is very slow, but it allows GPU to communicate with
other devices.

Chapter 7, section 3 (page 254) - Internal Registers

Memory offsets 63488..64511 are mapped to the IOBus (external ports). The memory
offsets 65536..131071 are mapped to the MemBus, allowing for access to external devices
from the GPU. There is support for both reading and writing this memory, although at
very low speed.

So the 63488 should work port output I presume?

Writing into membus not worked for me too
https://youtu.be/iRJKyNGDvKw

Writing to the port registers only overwrites the cached copy of the value previously read from the server. The documentation suggests that this might be intentional.

@Examnes

Reading into membus not worked for me too
https://youtu.be/iRJKyNGDvKw

The 64k memory model does not support MemBus I/O (on your version). Use 128k instead.


HDDS-6234. Repair containers affected by incorrect used bytes and block count.

What changes were proposed in this pull request?
Container Inspectors
This PR introduces the concept of container inspectors, which can be plugged in to container iteration on datanode startup to either log information about containers, or repair errors. Inspectors are developer/debug tools, and are hidden behind config flags so they do not slow down normal datanode startup. They are intended to find and fix container bugs like HDDS-5359. In cases like these, the actual on disk container needs to be checked and repaired beyond what SCM is capable of. If we ever suspect container bugs in the future and need to inspect/repair on disk data, new inspectors can be added to handle the task.
The KeyValueContainerMetadataInspector
This PR creates the KeyValueContainerMetadataInspector to identify and fix containers affected by HDDS-5359. It supports two modes: inspect and repair. In inspect mode, it will log metadata information about containers whose bytes used and block count totals do not align with individual RocksDB keys. By having a conclusive list of affected containers, we can examine their on disk state and confirm our understanding of the bugs. When the bugs are resolved, the inspector can be run in repair mode to fix the block count and used bytes totals in the containers. After this, SCM should properly identify empty containers and delete them.
Enabling
Passing the java system property -Dozone.datanode.container.metadata.inspector=inspect will cause the KeyValueContainerMetadataInspector to log metadata information about all containers on datanode startup whose aggregate values do not match the sum of their parts in the database. Changing inspect to repair will update these aggregate values to match the database.
Logging
The inspector will log its reports on the containers to a logger called ContainerMetadataInspectorReport. Incorrect containers will be logged at the ERROR level, and correct containers will be logged at the TRACE level. Each container's report will be a json object. Therefore, it may be desired to separate the inspector report from the main log by adding a log4j config like this:
log4j.appender.inspectorAppender=org.apache.log4j.FileAppender
log4j.appender.inspectorAppender.File=${hadoop.log.dir}/containerMetadataInspector.log
log4j.appender.inspectorAppender.layout=org.apache.log4j.PatternLayout
log4j.logger.ContainerMetadataInspectorReport=INFO,inspectorAppender

Note the use of an empty pattern layout in the appender, so that no log information is added to interfere with the json. If the inspector encounters any IO errors, these will be logged to the main log file to keep the json report in valid format.
Output in repair mode is the same as output in inspect mode, except that the repaired values will be set to true to indicate a repair was performed. Below is sample output for one container in repair mode:
{
  "containerID": 87310,
  "schemaVersion": "2",
  "containerState": "CLOSED",
  "currentDatanodeID": "69150d8a-8f25-4e94-b02c-41f65c1ac175",
  "originDatanodeID": "cba5d27d-e2aa-46b1-9e6c-e9c7e41fd4f0",
  "dBMetadata": {
    "#BLOCKCOUNT": -5,
    "#BYTESUSED": -51200,
    "#PENDINGDELETEBLOCKCOUNT": 0,
    "#delTX": 7843919,
    "#BCSID": 9303680
  },
  "aggregates": {
    "blockCount": 0,
    "usedBytes": 0,
    "pendingDeleteBlocks": 0
  },
  "chunksDirectory": {
    "path": "/data/1/hadoop-ozone/datanode/data8/hdds/CID-f72ad861-4e70-4fe5-982b-79ce831c6d31/current/containerDir170/87310/chunks",
    "present": false,
    "fileCount": 0
  },
  "correct": false,
  "errors": [
    {
      "property": "dBMetadata.#BLOCKCOUNT",
      "expected": 0,
      "actual": -5,
      "repaired": true
    },
    {
      "property": "dBMetadata.#BYTESUSED",
      "expected": 0,
      "actual": -51200,
      "repaired": true
    },
    {
      "property": "chunksDirectory.present",
      "expected": true,
      "actual": false,
      "repaired": true
    }
  ]
}

Operating on the results
Scanning multiple containers will append each json object to the log file. While the file as a whole may not be valid json since the container entries are not part of a list, it can still be processed by tools like jq. Below is an example jq command to get the container IDs of all containers that the inspector notes as having negative bytes used values:
jq '. as $parent | .errors[] | select(.property=="dBMetadata.#BYTESUSED" and .actual<0) | $parent.containerID' containerMetadataInspector.log
What is the link to the Apache JIRA
HDDS-6234
How was this patch tested?

[x] unit test added
[x] docker

Created a correct container and verified no errors were flagged.
Used ldb to modify the container's #BLOCKCOUNT key to 0.
Restarted DN in inspect mode and verified error was flagged.
Restarted DN in repair mode and verified error was fixed.

[x] live cluster affected by HDDS-5359

Thanks for the review @hanishakoneru. I have updated the patch to output json to a different logger. Please re-review.

Thanks @errose28 for working on this. The patch looks very good.
We should document the details on how to run this tool and what the output would look like. If not in the docs, let's have it in KeyValueContainerMetadataInspector's Javadoc at least. Can you please also include how to set the lo4j property with the custom layout pattern so that only the json output is logged to the file.

Thanks for the review @hanishakoneru. I will add documentation to the inspector class javadoc. I was thinking of adding docs to the ozone wiki as well, since that seems to be where we keep more developer specific documentation.

Thanks @errose28. +1 for merge pending CI.


Spurious dependency on utf8cpp imported target in assimp CMake imported target

Describe the bug
Since https://github.com/microsoft/vcpkg/pull/13521, the assimp CMake config file contain a transitive dependency to the utf8cpp imported target
Environment

OS: Windows
Compiler: Not relevant

To Reproduce
Steps to reproduce the behavior:

./vcpkg install assimp
Inspect the assimp cmake config file

Expected behavior
All the imported targets linked by assimp target should be properly found in the CMake config file with a find_dependency call.
Failure logs
-(please attached failure logs)
Additional context
See https://github.com/ignitionrobotics/ign-physics/issues/87#issuecomment-708549196 .

fyi @JackBoosY

@traversaro Yep, I missed find_dependency. And the utf8cpp is a header only library.


Fix shift amounts in structures.js.

As per the ECMA standard (Section 11.7.2), only the least significant five bits of the right operand of a shift operation are taken into account.

ping @xiemaisi

@sheerun Any ideas about this?

According to unzip source external flags are fetched with:
xattr = (unsigned)((G.crec.external_file_attributes >> 16) & 0xFFFF);

and then processed with:
switch ((unsigned)(xattr & UNX_IFMT)) {
    case (unsigned)UNX_IFDIR:   attribs[0] = 'd';  break;
    case (unsigned)UNX_IFREG:   attribs[0] = '-';  break;
    case (unsigned)UNX_IFLNK:   attribs[0] = 'l';  break;
    case (unsigned)UNX_IFBLK:   attribs[0] = 'b';  break;
    case (unsigned)UNX_IFCHR:   attribs[0] = 'c';  break;
    case (unsigned)UNX_IFIFO:   attribs[0] = 'p';  break;
    case (unsigned)UNX_IFSOCK:  attribs[0] = 's';  break;
    default:          attribs[0] = '?';  break;
}

where UNX_IFMT is 0170000 and e.g. UNX_IFDIR is 0040000.
This gives shift of 28 and mask of 0x0F for external flags. The mode is xattr with 0xFFF mask.
Plus even then ECMA says that shift by 60 is equal to shift by 28 (you can shift max by 31).
It all means it's totally correct, yet not really changing anyting commit.

@sheerun Ok, leaving it up to you to decide on merging or closing this PR.


Atmosphere: Globe doesn't render correctly when zooming into the ocean

When zooming into an area like the ocean far enough the 3d model will disappear and the screen will just show white. There are portions of the 3d tiles data that are below the WGS84 ellipsoid so I suspect this may be related to some assumptions made in the shader and this is caused by moving the camera below the ellipsoid surface?
Here's a series of screenshots from zooming in:

Ah, that does happen, and you’re correct. The issue is caused when the camera is located inside the ellipsoid.
This happens because the shader assumes the radius of the viewpoint is always larger than the atmosphere's bottom radius. It can be fixed by clamping it the bottom radius.
What I can’t fix is, because the scattered light is not occluded by scene objects, the workaround above results in something like this:

The horizon over the sea bottom would be correct if the scattered light were occluded by it.

Hmm - when you have a moment do you mind making a PR or something with your change? I'd like to take a look to understand what's happening. I wonder if shrinking the ellipsoid to it's always smaller than the terrain is an option?

Here is the updated viewpoint radius for the scattering. I only changed it for GetSkyRadianceToPoint, but we might need to rework GetSkyRadiance differently.
https://github.com/takram-design-engineering/three-geospatial/commit/7086d8e8994ef91120c26a4b9ef06f58304c708d

Added a workaround similar to the above in: https://github.com/takram-design-engineering/three-geospatial/commit/6981ac9ba05855307257476612f5bb5eb093faa1
Because this issue is emphasized by correcting altitude in the first place, I think adjusting the correction itself is a better way to handle it.

Looking at how Google Earth handles this (I guess they use a similar atmosphere model), there is a discontinuity, but they apply some scattering underwater.
https://github.com/user-attachments/assets/1e523096-9586-4924-b285-06e0624b0e4b

Looking at how Google Earth handles this (I guess they use a similar atmosphere model), there is a discontinuity, but it applies some scattering underwater.

Borrowing an idea from Google Earth, it’s a possible solution to add an additional radius for the lowest sea floor and switch it when the viewpoint is underwater.


Engine render() template should use renderTemplate() if passed template is a valid path

IMO it's totally legit that one want to pass a full path to Engine::render() and it should just work...
A simple is_file() check should do the trick while maintaing backward compatibility.

Fixed in v0.6.


⚠ File access has degraded performance

In 9380f95, File access (https://fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 12442 ms

Resolved: File access performance has improved in add897c after 7 minutes.


Resolves #280

Updated the documentation to add some additional examples and an overview of functionality

Hey @josegonzalez can you just have a quick check over this to see if you agree with me on "What the plugin will do" section.
Also, I'll let you merge this in as I'm assuming that readthedocs.org needs the Latex rebuilding once it's merged.

Coverage remained the same at 44.1% when pulling 780e2a275ca9ff37af887075c7f5131481ea40b3 on davidyell:issue-280 into 3b8807149625279c3abff9d53fe3750fd836d5e1 on josegonzalez:master.


Add new release labels

Apply new labels to release-index.json and release.json.
This PR will be merged on July 13th.
Context: https://github.com/dotnet/designs/pull/265
We cannot change the existing support-phase property, due to compatibility. It will go through the following progression:
preview -> go-live -> [lts | sts] -> maintenance -> eol
The new release-type property will be set as either lts or sts and will not change, even after EOL.
When release-type == support-phase, that means "active" of "full" support. All other states are self-descriptive. That's a bit wonky and not how we would have designed if we were starting from scratch, but we're not. This approach seems pretty workable.
The spec suggests a string array of labels. On further discussion, that doesn't seem necessary. This simpler approach seems better and more efficient for readers. The only label we're missing is latest. It doesn't seem warranted to add another property to account for that.
This change is a breaking change since it switches to sts instead of current. sts is "short-term support" while lts is "long-term support". This breaking change won't be observable until .NET 7 is GA and support-phase == sts instead of current.
I updated all the releases in release-index.json to make it internally consistent, but then only updated the in-support releases for release.json. That's definitely up for discussion. Seemed pragmatic.
@mairaw

@mairaw can we test this for dot.net to ensure this doesn’t break it?

We know that this change will break the website (mostly due to the same issue as https://github.com/dotnet/core/pull/7497). @mairaw would like to test this change, as you suggest, to see if there are other issues.
If we have other known contacts who use these files, now would be a good time to contact them.

With that change merged, we could even add the .NET 7 EOL dates back in.

This PR does that. We can wait to merge this PR or unrevert the other one. Might as well wait and then we can tell people about the changes all at once.

I'll leave that up to you. Whichever approach you prefer.

Are you planning on adding the release-type to older releases.json files, e.g. 1.0 or 2.2? If not, what should be the default value when reading these files for the property? We have use cases for processing these files and I want to make sure the releases API handles this in a consistent fashion. I could also consider an undefined enum.

I updated all the release objects in release-index.json. I didn't update the old release.json files in absence of a reason to do so. If you have a scenario where that would be helpful, I'm willing to do it. It won't take me very long. Should I?

I updated all the release objects in release-index.json. I didn't update the old release.json files in absence of a reason to do so. If you have a scenario where that would be helpful, I'm willing to do it. It won't take me very long. Should I?

That would be great, unless you have a suggestion for assigning a default value when the release-type is absent in the JSON - I'd be open to that as well.

Happy to deal with the merge conflicts. However, I think I promised that I wouldn't merge these files until 7/13 (or some similar date). I'd rather wait until July patch Tuesday. Fair?

Of course, no problem!

I just dealt with the merge conflicts since I also had to add the missing property to all other versions. And when would we add the changes to the support-phase property? I just need to account for the interim solution with the current values.

Thank @mairaw!

Do we have tests or any validation for install scripts?  There are args that expect Channel that will now be changing (https://docs.microsoft.com/en-us/dotnet/core/tools/dotnet-install-script#options)

Yes @richlander I'm thinking to make sure things like -Channel sts would be expected to work...I think it will work based on my 2m reading of the code, but might need to just expand test validation/coverage of adding the new labels.

Ah. I see now. I'm not sure there is a big scenario for that. I think the following scenarios make sense:

Latest release
Latest LTS
this x.y release

For example, what would sts return now? Seems like an anti-pattern. The logical labels should only exist if they are guaranteed to always return an in-support release. Fair?

I think today sts would error because it would try to construct an aka link that doesn't exist. https://github.com/dotnet/install-scripts/blob/main/src/dotnet-install.ps1#L907  Maybe a non-issue, but would be good to make sure things work as expected.  There is 'Current' in places in the code and tests...probably need expansion.

I believe that @MattGal and @mmitche were involved in setting up those feeds. Is "Current" the same as latest?
If so, we may want to undocument that and (compatibly) rename it to "Latest".

Tagging @bekir-ozturk also directly.

As I understand it, the label Current works as expected with install scripts (installs .NET 6.0) while it undesirably refers to 5.0 in VS.
@richlander already mentioned that this will be a breaking change, but I wanted to double check that we are on the same page about what exactly will be broken. So, here are some questions that popped into my head:

Does removal of Current label also mean the removal of Current channel? @mmitche @adiaaida
Current allows the users of install scripts to say "give me whatever is the latest GA version, whether it is LTS or not" which (warning: speculation ahead!) may be a common scenario for people wanting to stay up to date (end of speculation). If the channel Current will indeed be removed, can we say that "this is one of the breaking changes we are introducing"? What alternative can we offer for these users?
As we are updating install scripts documentation, do we want to add sts to the list of supported channels? Agreeing with @richlander above, it may be an anti-pattern since from the context of the scripts it will mean "give me the latest GA build, but make sure it is one of those that will expire soon".
I'm unsure whether the benefits of having an sts channel in scripts will worth the maintenance costs as well as the potential user confusion.

Does removal of Current label also mean the removal of Current channel? @mmitche @adiaaida

Current in terms of the concept that users can dotnet-install --version current (ie, the aka.ms links) seem like they would be unaffected by this change. Creating the current links is a manually run pipeline that we do once a release (more or less on GA release day).

I'm unsure whether the benefits of having an sts channel in scripts will worth the maintenance costs as well as the potential user confusion.

If we add sts to dotnet-install we will need to update a bunch of stuff in other places. Please ping me if we decide to do this.

Let's just stay as-is. I will write up some docs and probably a blog post on our updated approach. I will review it with you folks to ensure everything is correct.
We could change "Current" to "Latest" but that seems like a lot of churn for little benefit. I am NOT proposing that.

@jamshedd I resolved all the comments because changing those is no longer the plan of record. Please review again.

These changes should be included with all other releases.json changes for GA. We won't be able to handle different changes coming at different PRs for the website (or we'll need to skip some deployments).


how to set read_timeout? i experience it in my tornado project. but its not  available.

read_timeout = 1,
write_timeout= 1,
select expression： select sleep(10) from table_name limit 1;
thanks

Read_timeout and write_timeout are not supported. If read_timeout and write_timeout are supported for each read and write, IOLoop timeout needs to be registered. This is very important for performance, so if you need it, you can manually register a timeout in IOLoop to execute close(True) before executing. Close this connection and close this query. I think based on performance considerations, most of the reads and writes do not need to time out. It is a better solution for the application layer to decide whether to timeout.
Thank you.
不支持read_timeout和write_timeout，如果每次读写都支持read_timeout和write_timeout需要注册IOLoop timeout，这是非常影响性能的，所以如果有需要你可以手动在execute之前在IOLoop中注册一个timeout来执行close(True)关闭这个连接，从而关闭这次查询，我想基于性能考虑，大部分读写并不需要超时，由应用层决定是否超时是一个更好的方案。
谢谢。


remove redundant x-wasm-custom header

Fix #8
"x-wasm-custom: FOO" is from example and is no longer needed.

LGTM
søn. 8. mai 2022, 10:03 skrev Bryan Wu @.***>:

Fix #8 https://github.com/intel/modsecurity-wasm-filter/issues/8
"x-wasm-custom: FOO" is from example and is no longer needed.
You can view, comment on, or merge this pull request online at:
https://github.com/intel/modsecurity-wasm-filter/pull/9
Commit Summary

199b44c
https://github.com/intel/modsecurity-wasm-filter/pull/9/commits/199b44c064f7dd83f4ac469dd95f96063927383e
remove redundant x-wasm-custom header

File Changes
(1 file https://github.com/intel/modsecurity-wasm-filter/pull/9/files)

M wasmplugin/envoy-wasm-modsecurity.cc
https://github.com/intel/modsecurity-wasm-filter/pull/9/files#diff-a89ece57fb77f2d0742553f81948c458a6a11836b0667b556fff9ece71f0a58c
(1)

Patch Links:

https://github.com/intel/modsecurity-wasm-filter/pull/9.patch
https://github.com/intel/modsecurity-wasm-filter/pull/9.diff

—
Reply to this email directly, view it on GitHub
https://github.com/intel/modsecurity-wasm-filter/pull/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYAQ7EAZ57L4TYDFP7UDVI5YMDANCNFSM5VLSFTNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Fix footer anchor color

I was thinking about removing the footer all together. Facebook’s latest site https://bucklescript.github.io/ doesn’t have one and I like it better. Thoughts?


Privacy Policy + ToS

We should probably have a privacy policy page somewhere that details how we handle uploaded data, email addresses, etc.

If we have a privacy policy, then we will probably need terms of service as well. Use at your own risk, no guarantees, etc.

Privacy policy: http://pastebin.com/xEAd4njF
ToS: http://pastebin.com/2Tq8PA8N

An updated version of the documents is available here:

PP: https://docs.google.com/document/d/1PX4zSEk6s6K04ZYiR0M3TO3zm51Iyw96tDk4XeGTxIE/edit?usp=sharing
ToS: https://docs.google.com/document/d/1hP2npwBXtrTHx0Hp2Fj64jQwJ74Zk1IsVRg61_0kjIc/edit?usp=sharing

This shouldn't be too hard. If we stick a thing in the footer for them, and they can pop up in a modal.

I don't think a modal is strictly necessary, let's just have a dedicated page and prioritize other things.


add snippets

Tell me more Robert..what do these snippets do?  If there is a dbt page I should read, let me know.

Tell me more Robert..what do these snippets do? If there is a dbt page I should read, let me know.

These Snippets are more of a vs code function vs a dbt one. They help lazy coders like myself not have to remember or write out the exact config for an incremental model and other configs. It auto-fills for me.

Awesome..Thank you!


[BUG] Redscreen error when TouchRelease triggered on a component no longer in the tree

Unrecognized signal RESPONDER_RELEASE or state undefined for Touchable responder null
This issue is triggered by the following:

A Touch event on a component ( Keep Pressing  )
A state/props update removes the component from the tree and render is performed
A Touch Release event. ( When you release the press on the component no longer there )

version 0.18.1 of react-native

Just to confirm this is an issue on Android as well.

I'm having this same issue, @eadz!

Does my fix work you as well?
On Wed, 9 Mar 2016 05:51 Kane Rogers<EMAIL_ADDRESS>wrote:

I'm having this same issue, @eadz https://github.com/eadz!
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/react-native/issues/5604#issuecomment-194125381
.

I haven't yet had a chance to have a look - I'll check it out and let you
know!
On Wed, Mar 9, 2016 at 7:28 PM Eaden<EMAIL_ADDRESS>wrote:

Does my fix work you as well?
On Wed, 9 Mar 2016 05:51 Kane Rogers<EMAIL_ADDRESS>wrote:

I'm having this same issue, @eadz https://github.com/eadz!
—
Reply to this email directly or view it on GitHub
<
https://github.com/facebook/react-native/issues/5604#issuecomment-194125381
.

—
Reply to this email directly or view it on GitHub
https://github.com/facebook/react-native/issues/5604#issuecomment-194181208
.

Fix Merged.


Changing CSRF error status code

Shield is sending 403 HTTP status for mismatched CSFR token. As it's conflicting with cases that user is authenticated but does not have the right permissions, I suggest to choose 406 or 412 to avoid this conflict.
I also suggest to add an option in shield config, so it could be configured manually.
// shield.js
module.exports = {
    csrf: {
        enable: true,
        methods: ['POST', 'PUT', 'DELETE'],
        errorStatus: 406
  }
}

I believe, 403 is the perfect error code for CSRF failures. If you are looking to override for any specific reason, you can make use of exception handler to set a different status code.
The following code should do the job
  public handle (error, ctx) {
    if (error.code === 'E_BAD_CSRF_TOKEN') {
      error.status = 406
    }

    return super.handle(error, ctx)
  }

@thetutlage I don't think so about 403, but adding a config isn't much more straightforward and clear?

@amraei Sorry for the late reply. I just wanted to do my research before introducing a config value. I see that many framework including Django and Spring uses 403.
I agree that it can be debated further, but I suggest using the Exception handler to update to error status at the application level.

@thetutlage Thanks for replying. I just handled things using error.code. In my case, I have migrated an API server of a small app from Laravel to Adonisjs. So was looking for way to not change anything in front-end. However, did it.

Yeah, the goal is not to be Laravel compatible. Everyone draws inspiration from somewhere. I remember Taylor drew a lot from the .Net world.
I will close the issue, since there is no standard HTTP status code for CSRF errors. Many frameworks (as mentioned earlier) uses 403 and for now I would like to stick to it. :)


passthru in v2.2.0 does not consider users' passthru

When adding a custom passthru it will override the one created by mavenix, breaking mvnix-update. The user passthru should recursively extend the one set by mavenix.

Sorry @FRidh, I totally missed this. Remembered that you specifically used passthru from an earlier issue.
Was working with another project and did think about it there though :P


drive push doens't sync files

As the title says , whenever i clone the directory from my drive via '>>drive clone id-of-this-directory' , it gets downloaded normally .
however when i change the directory into it , and then make adjustment to any of the files contained within , the try pushing the new modifications , it doesn't make any change to the google drive directory ,it only outputs this message
checking for changes in 'text_files' .... Working directory is clean

Did you ever find a solution to this? I am having the same issue @z0xyz

@funtime1818  can't recall to be honest , but i'm sure that i've switched to gdrive back then .


Postgres loader configuration settings do not persist

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/879
Originally created by @dmor on 2019-08-14 17:15:08

Repro steps:

Make sure you are running Meltano 0.36
Select the Postgres loader and fill out the form in step 3 of the pipeline
Go back to the loader step and click on "Configure" and the fields are empty

Info I have from Derek:

ENV file still has this info (but it doesn't show up here)
If I change the fields here with new values, I don't overwrite the ENV (effectively I am stuck as a UI-only user)

View 3 previous comments from the original issue on GitLab


Cannot create ElasticSearch container.

Expected behavior
Creating an ElasticSearch instance.
Actual behavior
Getting the message: (HTTP code 404) unexpected - manifest for elasticsearch:latest not found
Information about the Issue
Using VB, OSX Mojave,
Docker version 18.03.1-ce, build 9ee9f40
Steps to reproduce the behavior

Open Kitematic
Click + New
Find official elasticsearch (elasticsearch:latest)
Click CREATE
You'll see the error.

Dupe of #4171.


cft_update.x needs to be compiled with big-endian read of diag file

in Applications/GMAO_Etc/GMAO_bias the program cft_update.x needs to be compiled with big_endian to be able to read conventional diag files.

The program runs if the environment variable FORT_CONVERT10 is set to 'big_endian' but not otherwise.
The GNUmakefile in CVS had "USER_FFLAGS = $(BIG_ENDIAN) $(DGEOS5)"

Oops. I guess I missed this. Well, first, it looks like DGEOS5 is unneeded as, well, nothing in that directory has #ifdef GEOS5. But the big-endian needs to be fixed.
If you'd like to test, the solution should be to add:
set_target_properties (convert_cft.x PROPERTIES COMPILE_FLAGS "${BIG_ENDIAN}")
set_target_properties (cft_update.x PROPERTIES COMPILE_FLAGS "${BIG_ENDIAN}")

to the CMakeLists.txt in Applications/GMAO_Etc/GMAO_bias

Thanks, Matt.  I looked for examples in other CMakeLists.txt files but the ones I saw changed compilation for all the executables.  The 'cft_update.x' is the only one that needs the 'big_endian', so your suggestion is right on track.  I will put that in and make a pull request.
We're not currently using this routine in the DAS.  It's being used in MERRA2, but in the current DAS we're now using a variational bias correction formulation.  So no wonder it hadn't been noticed until now.

@gmao-msienkie Ah okay. I just figured the CVS/GNUmake version was compiling everything with BIG_ENDIAN, so I decided to err on the side of caution.

OK.  I tried adding your suggestion but I don't think it worked.  So I decided to copy this from another CMakeLists.txt file
if (CMAKE_Fortran_COMPILER_ID MATCHES Intel)    string (REPLACE "${FPE0}" "${BIG_ENDIAN} ${FPE0}" CMAKE_Fortran_FLAGS_RELEASE ${CMAKE_Fortran_FLAGS_RELEASE}) endif ()
and was able to run the program.


UP-4852: Google Code Format as Java Linter

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines
[x] tests are included

Description of change

https://issues.jasig.org/browse/UP-4852
Checkstyle is flexible and powerful static analysis tool. :arrow_up:
However it lacks automatic fixing capabilities, and manual fixing of errors has been a blocker for wider adoption. :no_entry_sign: :cry:
PR #819 showed that the Google Java Format tool has the capability to automatically apply stylistic changes to code. :100:
Rather than requiring developers to manually format code, I purpose using the Google Java Format gradle plugin.
For developers this makes code styling as simple as ./gradlew googleJavaFormat.
And verification on CI would use ./gradlew check which will run ./gradlew verifyGoogleJavaFormat to ensure the developer has applied code styling to their commits. :white_check_mark:
:notebook: Maven based builds are currently not checked, they will be in the future as more of the build system migrates to Gradle.

Coverage remained the same at 22.88% when pulling 0280ac34b9faf51cf60654af987045ae22d9bebe on ChristianMurphy:UP-4852 into ce925e5ed28f05585fa3287b869f18bae151c405 on Jasig:master.

Coverage increased (+0.02%) to 22.896% when pulling 0280ac34b9faf51cf60654af987045ae22d9bebe on ChristianMurphy:UP-4852 into ce925e5ed28f05585fa3287b869f18bae151c405 on Jasig:master.


Update Aqua test keywords

The aqua test failure is covered by PR #2 and the Documentation doctest failure is unrelated. This should be good to merge.


Slack Channel Interactive Messages not working with Bot BUilder

Issue Description
I am sending the same JSON response to Slack Channel, Facebook Channel and Web Channel (Bot framework in built) to display buttons. It works fine on Facebook messenger and bot framework web chat. But it doesn't display the content as buttons in Slack Channel. Instead, it displays all the buttons with a preceding asterisk. Is slack channel supported with same interactive message components as other channels are currently supported?

Please share the .json you're sending.

@harshil94 Are you using .NET or Node.js SDK for BotBuilder? Please post your bot code and associated JSON response so we can better assist you.

Here I am creating a response object from the bot_object given by bot builder.
let response = new builder.Message(bot_object);
Then I am adding some data to display imBack Herocards on channel and the final json which I am sending in response is something like this.
{   "type":"message",   "agent":"botbuilder",   "source":"slack",   "address":     {       "id":"<channel id>",       "channelId":"slack",       "user":         {           "id":"<user id>",           "name":"<user name>"         },       "conversation":         {           "isGroup":false,           "id":"<conversation id>"         },       "bot":         {           "id":"B5E70L1SA:T5C069UQZ",           "name":"<slack bot name>"         },       "serviceUrl":"https://slack.botframework.com"     },   "text":"Hello! Will be glad to assist for any help required on below incidents.",   "suggestedActions":     {       "to":[],       "actions":         [           {             "type":"imBack",             "value":"Create Ticket",             "title":"Create Ticket"           },           {             "type":"imBack",             "value":"Check Status",             "title":"Check Status"           },           {             "type":"imBack",             "value":"Reopen",             "title":"Reopen"           }         ]     } }
Funny thing is, when I am sending the same response, it shows hero cards on facebook messanger as well as webchat interface of botframework. But it shows the response on slack channel as plain text. See the response on various channels as below:
Facebook messanger UI for Hero card:

Botframework webchat Interface UI for Hero card:

Slack UI for Hero card:

Any updates please?

@harshil94 The suggestedActions you have configured in your message object will be displayed differently based on the channel (aka Messenger vs Slack). What you are seeing is the expected behavior of Slack channel.
You can verify this by looking at the Channel Inspector, set on channel: Slack, feature: Suggested Actions

To send Slack-specific formatted messages, you need to put your custom message data in the channelData object of your message.
You can learn more about how to use the channelData under the section Create Full Fidelity Slack Messages:
https://docs.microsoft.com/en-us/bot-framework/dotnet/bot-builder-dotnet-channeldata#create-a-full-fidelity-slack-message


docs: final draft empty states pattern

Closes #333
Moving content into new branch for final polish before publishing.

@alisonjoseph any ideas why this deployment is failing?


wantsJson doesn't understand RESTful requests

Via http://jsonapi.org/format/

JSON API requires use of the JSON API media type (application/vnd.api+json) for exchanging data.

So it would be very welcome if wantsJson returned true for application/vnd.foo.v1+json instead of just application/json.
The dingo/api would also benefit from this (@jasonlewis).

Not a bug,

Use the accepts method we have.

That way you can reliably check if the client understands the content type.

@GrahamCampbell Why not use WantsJson? TO fixed it in any case https://github.com/laravel/framework/pull/10970

Should already be available in v5.1.25 based on @0fcceed

And that. ;P

I'm still against that change though, because wantsJson is incorrect if the client asked for application/foo+json and the server returns application/json.

That's why you should always use the accepts method to actually be sure.


Binary Config.xlsx - worst file format for merging

As a POC Config.xlsx was a very good concept. But for now, any binary file in development provides problems with merging and I with it is possible to switch from it to any plain text format as a set of files "csv","tsv" or single file "json" with a set of nodes.

I fully agree. The big downside is that end users from the workflow mostly familar with excel and dont even know what json is. It would be a step back in terms of accesablity.
I thought long about it and I think configurations should be integrated directly in the project.
End users could modify these settings with a user friendly and easy to use gui from the agent.
For unattended robots the orchestrator can manage these or even provide a form, similar to "kapplets" from kapow.

I fully agree. The big downside is that end users from the workflow mostly familar with excel and dont even know what json is. It would be a step back in terms of accesablity.
I thought long about it and I think configurations should be integrated directly in the project.
End users could modify these settings with a user friendly and easy to use gui from the agent.
For unattended robots the orchestrator can manage these or even provide a form, similar to "kapplets" from kapow.

Sad but true.  That is why I mentioned CSV which is not so familiar for non developers as xlsx but could bring the similar experience by using MS Excel.
In any case, thanks for the answer.


Use LinuxAdmin::NetworkInterface for network operations

Replaces the use of miqnet.sh for setting and getting network information.
Requires https://github.com/ManageIQ/linux_admin/pull/132, https://github.com/ManageIQ/manageiq-appliance-build/pull/66, and a new linux admin gem version

Requires https://github.com/ManageIQ/manageiq/pull/4707 for gemfile change.

:+1: LGTM  It's really nice to get away from using miqnet.


Error message is not useful to end-users

We need to alter the error message shown to end users. It's not useful to have a pop-up saying 'Error [Object object]'. We should change this to a jBox notification and have a specific cause of error that's simplified, if we may. As simple as; 'Gene not found' if specific to that. The program should resume as normal and inform the user to try an alternative search. Additionally, the message
should be both logged on the web server, and in the console for developers. This is a simple feature addition to KnetMaps.

Doesn't need to necessarily use jbox for this as jbox is quite dependency heavy. Could be a simple jquery popup with a simplified error message.

I'm closing this, cause it's superseded by #548.


Update Select.php , '()' missing

hello   i think u missed a '()'

@Quekie maybe not:
https://github.com/Quekie/laravel-admin/blob/b1e793a91ab21261e118df00281ab58fdfcb8d9d/src/Form/Field/Select.php#L245-L251

There is no problem here, this closure will eventually be called in the render method.


Python Smoke Test Pipeline

YAML file to setup the DevOps Pipeline to run the Dependency Smoke Test.

Can one of the admins verify this patch?


Add the toplevel column in powa_statements

...so that we don't have those duplicates inside our history

Running the regression tests locally leads to multiple errors.  I went ahead and added some basic CI to run the regression tests on all currently supported major versions to make sure this is detected earlier.

Thanks for the PR, merged!
Note that you can't add a column in the middle of a table in a version that we can upgrade to, as otherwise we could break things for people relying on the column order depending on whether they install from scratch or upgrade to that version.  And while changing that I realized that we actually relied on the column order in a few queries, so this would have been broken in case of an upgrade.
I fixed all of that in f6ec254e0177e013985288f377ebabe3dabf6f9f.


Right-click context menu in results pane

I am not getting the right-click context menu in the grid results for DBeaver after updating to 4.2.3.  It works in the "text" view of the results.

Apologies ... works now.

This is actually a bug. Context menu doesn't show in case of custom SQL queries.
Will be fixed as hotifx in 4.2.3, thanks for report.


Doc workflow improvements

Doc improvements - take a look at the commits for a breakdown of changes.
The biggest change you'll notice is including all doc pages here. This is better for tracking doc changes. Maybe we remove the generated folder prefix as well.
I cleaned up sphinx and latex problems - sometimes using a global search and replace, so there may be errors in that. I didn’t touch any of the main docs content here.

Oh, this isn’t pointing at your latest branch, so there may be conflicts with it.

@CamDavidsonPilon No worries. You can keep it pointed at master. I think these changes will be done before I finish 0.6.0. I can resolve the changes when I merge #81

I updated the logo on the subpages. Everything lgtm otherwise. Feel free to merge to master
Thanks again for all your help with ReadTheDocs!!