How to use it?

No README? wtf ! I dowloaded it and I found that there's no readme in it .  Mr. Liu


[8.x] Use "sync" queue in Bus::dispatchNow()

This PR fixes two issues. First, it fixes an issue where job middleware are not run when doing something like ProcessPodcast::dispatchNow(). This can be fixed by dispatching this job through the sync queue instead of invoking it directly from the command bus.
The second problem it fixes is currently any jobs dispatched using dispatchNow do not have have access to the $this->job property, even though they use InteractsWithQueue. The $this->job variable will just be null. This is awkward because you may have a job that is sometimes dispatched using the normal dispatch method and sometimes using dispatchNow and the $this->job variable is only going to be available when dispatched using dispatch. So, you would have to put null checks within your job to work around this. This PR solves this by setting a SyncJob on the job if one is not already set.
Added a new dispatchDirectly method to execute a job directly within the command bus without using the sync queue or job middleware (basically the old dispatchNow behavior).
Alternative:
One alternative to this is to leave dispatchNow as it is and add a new dispatchSync method (or some similar name) that dispatches the job using the sync queue.

I've run into this myself with a project recently. Looks like a good change! 👍

I think updating the behavior here for dispatchNow definitely makes sense. At the moment, I can't see any way it'd break an app.

I know it's just semantics, but maybe a little more difference in method naming? So instead of dispatchDirectly you could emphasize the difference in behaviour more with for example dispatchRaw or dispatchOnly?

Decided to take a less breaking approach using a new dispatchSync method.

Hi @taylorotwell , great addition solves a problem I have in a app right now.
One question: as the dispacthNow will attach a SyncJob instance to the command, will the failed method be called when an exception is thrown inside the command?
That would simplify/unify a lot of jobs error handling strategies I current have between sync and queued jobs.
Thanks in advance.


Chrome dev tools cant parse resulting har file - change _initiator st…

…ructure

Hey @soulgalore, please consider this PR also for your chrome-har on which puppeteer-har depends. :)

This PR on chrome-har: https://github.com/sitespeedio/chrome-har/pull/44


Fix: vertical bar broken in chart/gear acitivity (issue: #1646)

Fixed the broken vertical line in Chart/Gears activities, ensuring its applicable on all plaforms (issue: #1646)
Steps:

Divided the toolbar in two parts left and right, used dislay: flex and justify-content: space-between to move both the part to ends of both side of screen. and used dislay: flex and align-item:centre to move hr to correct position.
Earlier the negative margin was causing the cross platform issue for hr. Removed the negative margin.

Chrome:

Firefox:

Electron.js app:

Ok  i will make the changes you mentioned. Thanks

Hey @llaske i have removed the comments and indentation in my latest commit please have a look

It works.
You're not supposed to change the library sugar-96dpi.css because it's a shared library copied in every activity.
BTW it could be acceptable if it's a minor change. But using flex in the toolbar a major change that will impact all activities. Plus split it in left/right has no logical reason.
Could you find a way to limit the impact to activity css only or to the hr element only?

I looked around a bit and found out that the issue with <hr/ > lines not being consistent was margin-bottom : -17px the negative margin is causing cross platform issue as different browser interpret negative margin differently (please correct me if i am wrong). Whiledisplay : flexproperty is consistent on all platforms.
I split right/left because on applying display flex the style class of pull-right was over written so to move the icons in group in 2 sides of screen i used justify-content : space-between

One thing that i can think of is that wrapping  tag with a  tag and apply flex property on parent  of
Is this fine ?

I understand your fix but I would like a fix where only the CSS .toolbar hr need to be updated.
Upgrading HTML and use a flex property will change too many files.

Hey @llaske .... i looked and tried out diffrent css properties for restricting the changes to <hr/> tag only but it seems pretty impossible without changing some properties of the parent tag.
As you mentioned we are not supposed to modify the sugar-96dpi.css one thing we can do is add the necessary changes in acitivity.css of the specific required files

@AdityaKrSingh26 that's fine if you can limit change to activity.css

@llaske please have a look at the latest commit.

It works but I'm not a big fan of using flex because it change the look and the behavior of the toolbar.
Why not replace the HR object by a DIV with a vertical border? It will avoid changing the toolbar itself.

It works but I'm not a big fan of using flex because it change the look and the behavior of the toolbar. Why not replace the HR object by a DIV with a vertical border? It will avoid changing the toolbar itself.

Ok i will make the changes

It works but I'm not a big fan of using flex because it change the look and the behavior of the toolbar. Why not replace the HR object by a DIV with a vertical border? It will avoid changing the toolbar itself.

Done. Please have a look

@llaske  Done :+1:
Firefox:

Chrome:

Electron:

Work also on Safari.
Good job, thanks.

Work also on Safari. Good job, thanks.

Finally. :sob:


WIP: Add testcontainers based integration test

TODO: war creation and debugging
closes #20

Hey @bsideup,
I have two things left I'm not quite sure how to solve, see the two TODOs in the PR.
The biggest problem is the lifecycle of the war file creation which is necessary for the test to run. Did you already do sth like this? Is this the wrong approach? Do you know tools, which can create a war at runtime?
Also, I didn't quite get debugging to work.

Hi @felixbarny,
Let me take a look at TODOs and the PR in general 👍

@felixbarny and don't forget to turn on Docker on Travis ;)
sudo: true
services:
  - docker

Codecov Report

Merging #57 into master will increase coverage by 0.11%.
The diff coverage is 80%.

@@             Coverage Diff             @@
##             master     #57      +/-   ##
===========================================
+ Coverage     75.28%   75.4%   +0.11%
- Complexity      545     547       +2
===========================================
  Files            71      71
  Lines          1922    1927       +5
  Branches        143     144       +1
===========================================
+ Hits           1447    1453       +6
  Misses          399     399
+ Partials         76      75       -1

Impacted Files
Coverage Δ
Complexity Δ

...a/co/elastic/apm/impl/ElasticApmTracerBuilder.java
84.61% <ø> (ø)
10 <0> (ø)
:arrow_down:

.../java/co/elastic/apm/report/ApmServerReporter.java
69.23% <ø> (ø)
11 <0> (ø)
:arrow_down:

...ain/java/co/elastic/apm/impl/ElasticApmTracer.java
79.03% <0%> (-0.65%)
33 <0> (ø)

...a/co/elastic/apm/report/ReporterConfiguration.java
100% <100%> (ø)
7 <0> (ø)
:arrow_down:

...a/co/elastic/apm/report/ReportingEventHandler.java
100% <100%> (+8.33%)
10 <7> (+2)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8936e3e...6f178e2. Read the comment docs.

I really don't get why tests are failing on Jenkins.
Instead of returning the Hello World jsp, Tomcat responds with a 404, even though the war seems to be deployed successfully.
11:17:00 [tomcat] 13-Apr-2018 09:17:00.322 INFO [main] org.apache.coyote.AbstractProtocol.init Initializing ProtocolHandler ["http-nio-8080"]
11:17:00 [tomcat] 13-Apr-2018 09:17:00.340 INFO [main] org.apache.tomcat.util.net.NioSelectorPool.getSharedSelector Using a shared selector for servlet write/read
11:17:00 [tomcat] 13-Apr-2018 09:17:00.350 INFO [main] org.apache.coyote.AbstractProtocol.init Initializing ProtocolHandler ["ajp-nio-8009"]
11:17:00 [tomcat] 13-Apr-2018 09:17:00.352 INFO [main] org.apache.tomcat.util.net.NioSelectorPool.getSharedSelector Using a shared selector for servlet write/read
11:17:00 [tomcat] 13-Apr-2018 09:17:00.354 INFO [main] org.apache.catalina.startup.Catalina.load Initialization processed in 717 ms
11:17:00 [tomcat] 13-Apr-2018 09:17:00.397 INFO [main] org.apache.catalina.core.StandardService.startInternal Starting service [Catalina]
11:17:00 [tomcat] 13-Apr-2018 09:17:00.397 INFO [main] org.apache.catalina.core.StandardEngine.startInternal Starting Servlet Engine: Apache Tomcat/8.5.30
11:17:00 [tomcat] 13-Apr-2018 09:17:00.414 INFO [localhost-startStop-1] org.apache.catalina.startup.HostConfig.deployDirectory Deploying web application directory [/usr/local/tomcat/webapps/ROOT.war]
11:17:00 09:17:00.828 [ForkJoinPool.commonPool-worker-1] INFO  docker[testcontainers/p7ack8jmx4ukkkwq] - Container testcontainers/p7ack8jmx4ukkkwq started

The relative referencing of ../simple-webapp/taget/ROOT.war when binding the war into the container does seem to work, as </var/lib/jenkins/workspace/elastic+apm-agent-java+pull-request+multijob-test/src/github.com/elastic/apm-agent-java/apm-agent-plugins/apm-servlet-plugin-it/simple-webapp-integration-test/../simple-webapp/target/ROOT.war does exist and the content is correct. But the symptoms are the same as deploying an empty directory.
The connection to the tomcat container doesn't seem to be the problem, otherwise tomcat wouldn't be able to respond with a 404.
The on init log message APM test application {} started in ApmTestHelperServlet does not appear in the logs, so there has to be a problem with the deployment. Investigating further...

@felixbarny are you running slave as a container? If so, did you configure the workspace and stuff? ( https://www.testcontainers.org/usage/inside_docker.html )
The most important part is  -v $PWD:$PWD -w $PWD which mounts cwd as a volume and sets working dir to cwd

@felixbarny or, actually, I see ✔ File should be mountable in the build's log, it means that Testcontainers checked the file mounting logic and it works, hence I recommend checking the test's code, could be unrelated to Testcontainers

I suspect something goes wrong when mounting the file.
I even downloaded the war from the workspace and executed locally with success. And assert on test startup that the war file exists and that it has content: https://github.com/elastic/apm-agent-java/pull/57/files#diff-86c386663b0d434b566084161d7f2a22R94.

are you running slave as a container?

I'm actually not sure if the jenkins slaves are bare metal, VMs or dockerized. I'll ask.

@felixbarny try adding a .waitingFor(Wait.forHttp("/someHTTP200endpointOfYourTestApp")) to your container, maybe just it takes longer to start an app on CI than locally?

Makes sense! Tomcat listening on a socket does not mean that the application has already been deployed!
Will have to wait until there are free build slaves..

Nope
2:25:07 10:25:07.063 [ForkJoinPool.commonPool-worker-1] ERROR docker[testcontainers/nu5blc23zgbzjttz] - Could not start container
12:25:07 org.testcontainers.containers.ContainerLaunchException: Timed out waiting for URL to be accessible (http://localhost:32771/apm/status should return HTTP 200)

The deployment of the war is also suspiciously fast:
12:24:08 [tomcat] 13-Apr-2018 10:24:08.520 INFO [localhost-startStop-1] org.apache.catalina.startup.HostConfig.deployDirectory Deployment of web application directory [/usr/local/tomcat/webapps/ROOT.war] has finished in [478] ms

Locally, it takes around 1.5 seconds.

Note that is also says

Deployment of web application directory  [/usr/local/tomcat/webapps/ROOT.war] has finished in [478] ms

When executing locally, it says Deployment of web application archive. So there must be something wrong either with the path or with the mounting.

@felixbarny out of curiosity - could you please try to deploy your app under a context (eg "app.war")?

Still no luck. The /usr/local/tomcat/webapps/app.war directory inside the docker container appears to be an empty folder:
total 8.0K
drwxr-xr-x 2 root root 4.0K Apr 13 12:05 .
 drwxr-x--- 8 root root 4.0K Apr 13 12:11 ..

@felixbarny there is a..... hm... questionable behaviour of Docker, when you mount non-existing file it mounts it as an empty folder.
One clue - does your CI environment runs as Docker Swarm?

Do you mean if the workers have docker swarm installed or are managed by docker swarm? As they are not docker containers I don't think they are managed by docker swarm.

@felixbarny more like if Swarm is initialized on the machines with agents...
Building remotely on slave-09c45e463891e2e0d (virtual x86_64 swarm ubuntu linux ubuntu-16.04) in workspace

This line (and swarm in it) attracted my attention. Swarm is a distributed scheduler of Docker containers and might not guarantee that the container will be scheduled on the same machine, this is why the mapping might fail.
Imaging running tests on machine A, but the container is started on machine B where /home/user/some/path/app.war does not exist

Good point. But then it's strange that copy-ing the war into the image does not work (https://github.com/elastic/apm-agent-java/pull/57/commits/206308574af6964ed526878033f97a360e224bf7).
@elasticdog could you comment on wether docker images executed in the CI environment may be scheduled to other machines via docker swarm? Is there anything special in our deployment which could cause mounting a file to fail?

Seems like testcontainers does not create a Dockerfile but uses the docker socket api. This is what is sent to the docker.sock, inclunding the bind:
{
  "Env": [
    "ELASTIC_APM_SERVER_URL=http://apm-server:1080",
    "ELASTIC_APM_IGNORE_URLS=/apm/*",
    "ELASTIC_APM_SERVICE_NAME=servlet-test-app"
  ],
  "Cmd": [],
  "Image": "testcontainers/jlsaqt1bpwtwpp31",
  "Volumes": {},
  "ExposedPorts": {
    "8080/tcp": {},
    "8000/tcp": {}
  },
  "HostConfig": {
    "Binds": [
      "/Users/felixbarnsteiner/projects/github/elastic/apm-agent-java/apm-agent-plugins/apm-servlet-plugin-it/simple-webapp-integration-test/../simple-webapp/target/ROOT.war:/usr/local/tomcat/webapps/ROOT.war:rw"
    ],
    "ExtraHosts": [],
    "Links": [],
    "NetworkMode": "bdd79fbf5e4f91806e7c8eff8a19ac1396dba10a8e9e1c8b5e4d96a98f6b4b14",
    "PortBindings": {},
    "PublishAllPorts": true,
    "VolumesFrom": []
  },
  "Labels": {
    "org.testcontainers": "true",
    "org.testcontainers.sessionId": "36cb2fb8-69e8-4e2e-b578-dda7c237ed5b"
  },
  "NetworkingConfig": {
    "EndpointsConfig": {
      "bdd79fbf5e4f91806e7c8eff8a19ac1396dba10a8e9e1c8b5e4d96a98f6b4b14": {
        "IPAMConfig": null,
        "Links": null,
        "Aliases": [
          "tc-LcXmZm2Q"
        ],
        "NetworkID": null,
        "EndpointID": null,
        "Gateway": null,
        "IPAddress": null,
        "IPPrefixLen": null,
        "IPv6Gateway": null,
        "GlobalIPv6Address": null,
        "GlobalIPv6PrefixLen": null,
        "MacAddress": null
      }
    }
  }
}

@elasticdog @felixbarny one can easily verify the environment in isolation by running something like:
docker run \
    -p 8080:8080 \
    -it --rm \
    -v /var/lib/jenkins/workspace/elastic+apm-agent-java+pull-request+multijob-test/src/github.com/elastic/apm-agent-java/apm-agent-plugins/apm-servlet-plugin-it/simple-webapp-integration-test/../simple-webapp/target/ROOT.war:/usr/local/tomcat/webapps/ROOT.war \
    tomcat:8

@elasticdog can you or someone from the infra team help to investigate on this?
Steps:

execute the build for this PR (which fails)
execute the docker run command in the comment above
try to access localhost:8080
expected: returns 200 and a Hello World! response
actual: when executing the build, a 404 is returned

Problem:
The ROOT.war is not properly mounted into the container. When examining contents in the container, the ROOT.war is not the expected file but an empty folder.

Not sure if my comment about the full path or not is related. However the path in the comment above doesn't seem to exist on the worker node.

docker run
-p 8080:8080
-it --rm
-v /var/lib/jenkins/workspace/elastic+apm-agent-java+pull-request+multijob-test/src/github.com/elastic/apm-agent-java/apm-agent-plugins/apm-servlet-plugin-it/simple-webapp-integration-test/../simple-webapp/target/ROOT.war:/usr/local/tomcat/webapps/ROOT.war
tomcat:8

The closest thing I can find is an empty directory at /var/lib/jenkins/workspace/elastic+apm-agent-java+pull-request+multijob-test/src/github.com/elastic/apm-agent-java/apm-agent-plugins/apm-servlet-plugin-it/simple-webapp/target/ROOT.war/

Jenkins test this please

@Crazybus It's supposed to reference the `ROOT.war in this directory: https://apm-ci.elastic.co/job/elastic+apm-agent-java+pull-request+multijob-test/ws/src/github.com/elastic/apm-agent-java/apm-agent-plugins/apm-servlet-plugin-it/simple-webapp/target/
I just triggered a build and the file does exist (until another build is started)

This is what it is actually trying to bind (via docker.sock):
"Binds":["/var/lib/jenkins/workspace/elastic apm-agent-java pull-request multijob-test/src/github.com/elastic/apm-agent-java/apm-agent-plugins/apm-servlet-plugin-it/simple-webapp-integration-test/../simple-webapp/target/ROOT.war:/usr/local/tomcat/webapps/ROOT.war:rw"]

Seems odd that the + signs are replaced by spaces.

@felix-lessoer I see the problem!
Oh jeez... + inside the path got replaced with   (space), there was a bug about it in docker-java.
https://github.com/docker-java/docker-java/commit/1952fa6fad94d7ff071e578f6c46e5bb81124c9a
Try Testcontainers 1.7.1, it should have docker-java v3.1.0-rc-2 where it was fixed

One of those problems which resolve themselves over time... But that's actually very good news, even if I have to wait for a new docker-java release.
Funny thing is that the new docker-java version does not log the docker.sock requests anymore, not even on TRACE.

Even with the fact that the problem is in docker-java and not in Testcontainers I still feel sorry about having you to debug the problem and me almost unable to help 😅
Actually, in Testcontainers 1.7.1 we replaced docker-java's NettyCmdFactory with our own which uses daemon threads instead (long awaited feature request from Spring guys), and removed the packages logger. While it might be helpful in some rare cases like yours, a lot of users (very unfortunately) doesn't know how to configure the logging and were complaining about HEX dumps in their tests, this is why we decided to drop the logger.
But now I'm thinking about adding a flag to turn it on (in Testcontainers' properties) for advanced users who want to debug the packets...

@felixbarny FYI it seems that you can easily workaround the docker-java's bug by doing something like the following:
File tempWarFile = new File(System.getProperty("user.home"), UUID.randomUUID() + ".war");
tempWarFile.deleteOnExit();
Files.copy(warFile, tempWarFile.toPath());
withFileSystemBind(tempWarFile.getAbsolutePath(), "/app.war")

Assuming that user's home directory will not have + in it :)

@felixbarny another option would be to start Tomcat and do:
tomcat.copyFileToContainer(MountableFile.forHostPath(warFilePath), "/usr/local/tomcat/webapps/ROOT.war")

@felixbarny snap! Apparently, there is also a bug in Testcontainers in MountableFileTest :(
I was able to reproduce it and going to fix it ASAP

I'm thinking about deploying the wars via the tomcat manager API (https://tomcat.apache.org/tomcat-9.0-doc/manager-howto.html#Deploy_A_New_Application_Archive_(WAR)_Remotely). That way I can reuse the container across different tests. But that's something for the future... @bsideup what do you think about that approach?

@felixbarny we had some major issues with such approach:

Tomcat's classloading leaks
java agent reporting traces of another app under the test
less isolated parallel tests

Also, Tomcat + deployment adds to the test execution time.
I would highly suggest you to use Jetty + jetty-servlet like here:
https://gist.github.com/bsideup/70091d2f762b8bc7d9255260c0b5ed15
You can build it as Java app with Maven as well, but it works much faster (Tomcat-on-classpath can also be used)

Thx for the suggestions. Whenever feasible, I'll test with a simple Groovy Jetty script. But part of what I want to verify with the integration tests is that the agent works properly with a specific application server in a specific version.
YEY, the tests are green :D


test_timeline_deletion_with_files_stuck_in_upload_queue flakiness

Rare:
AssertionError: assert not [
  (762, '2024-02-23T01:54:42.414003Z  WARN delete_timeline{tenant_id=faa1e715b82ea028c2ab77c827a4e253 shard_id=0000 timeline_id=f8b5e0a4e8c75657837989e9d700addb}: got not found err while removing timeline dir, proceeding anyway timeline_dir="/tmp/test_output/test_timeline_deletion_with_files_stuck_in_upload_queue[debug-pg14]-1/repo/pageserver_1/tenants/faa1e715b82ea028c2ab77c827a4e253/timelines/f8b5e0a4e8c75657837989e9d700addb" path="/tmp/test_output/test_timeline_deletion_with_files_stuck_in_upload_queue[debug-pg14]-1/repo/pageserver_1/tenants/faa1e715b82ea028c2ab77c827a4e253/timelines/f8b5e0a4e8c75657837989e9d700addb/000000000000000000000000000000000000-FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF__000000000171F4C1-000000000172BF51"\n'),
  (763, '2024-02-23T01:54:42.436209Z  WARN delete_timeline{tenant_id=faa1e715b82ea028c2ab77c827a4e253 shard_id=0000 timeline_id=f8b5e0a4e8c75657837989e9d700addb}: got not found err while removing timeline dir, proceeding anyway timeline_dir="/tmp/test_output/test_timeline_deletion_with_files_stuck_in_upload_queue[debug-pg14]-1/repo/pageserver_1/tenants/faa1e715b82ea028c2ab77c827a4e253/timelines/f8b5e0a4e8c75657837989e9d700addb" path="/tmp/test_output/test_timeline_deletion_with_files_stuck_in_upload_queue[debug-pg14]-1/repo/pageserver_1/tenants/faa1e715b82ea028c2ab77c827a4e253/timelines/f8b5e0a4e8c75657837989e9d700addb/000000000000000000000000000000000000-FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF__0000000001696321-00000000016A5D69"\n'),
  (764, '2024-02-23T01:54:42.445635Z  WARN delete_timeline{tenant_id=faa1e715b82ea028c2ab77c827a4e253 shard_id=0000 timeline_id=f8b5e0a4e8c75657837989e9d700addb}: got not found err while removing timeline dir, proceeding anyway timeline_dir="/tmp/test_output/test_timeline_deletion_with_files_stuck_in_upload_queue[debug-pg14]-1/repo/pageserver_1/tenants/faa1e715b82ea028c2ab77c827a4e253/timelines/f8b5e0a4e8c75657837989e9d700addb" path="/tmp/test_output/test_timeline_deletion_with_files_stuck_in_upload_queue[debug-pg14]-1/repo/pageserver_1/tenants/faa1e715b82ea028c2ab77c827a4e253/timelines/f8b5e0a4e8c75657837989e9d700addb/000000000000000000000000000000000000-FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF__000000000175F0B9-000000000176A2F1"\n')]

Unsure what that could be. Only example I've seen so far. Analysis in: https://github.com/neondatabase/neon/issues/6681#issuecomment-1961568118
Common:
AssertionError: assert not [
  (866, '2024-02-22T17:26:05.355705Z ERROR request{method=PUT path=/v1/tenant/01f38f79d16faf4f1caa45fe1fbed6da/timeline/b2d0a3673159e19a88e51f564cfac803/checkpoint request_id=3e7ffcba-0384-4464-878e-3593c876cb94}: Error processing HTTP request: InternalServerError(queue is in state Stopping\n')]

this sounds like an error message which was changed. Fixed in #6894.

Looks like a Stopping/Stopped string error, which I probably introduced.

The more rare case is a valid situation, which happens when the struct Layer drops happen at the same time as timeline deletion cleaning up local layer files -- perhaps now the walkdir actually has an upper hand because it is sync code, vs. struct Layer using spawn_blocking, but it is a race.
The individual layers have no knowledge of deletion happening and they were being kept alive by UploadTask entries in RemoteTimelineClient. I think the answer to this is to hold the gate a bit more, and be sure to hold the guard until the end of deletion.
Something to consider after #6028.

Next steps:

understand what is the single walrecord which bringing up a pg15 is

The rare case is likely handled by #7082.

A recent failure: https://neon-github-public-dev.s3.amazonaws.com/reports/pr-6407/8295888842/index.html#suites/837740b64a53e769572c4ed7b7a7eeeb/fbe892c34b63cdb

A recent failure

I think it was just being slow, pageserver logs do not appear to have any long stuckness..

This has no longer been flaky since the Timeline::gate usage introduction in #7082: last flaky was 2024-03-16. However that work was merged before, so unsure.


Adding file set for preserved tagging of facet and geometry resolution tolerances.

In order for tagged information to be preserved through file writing/loading, all of the entities created need to be gathered into a single MeshSet and written to file. Upon loading the file, this MeshSet will be recognized as the file set so that any information tagged on it can be retrieved.
This information is sought out by make_watertight.
These changes have been tested with the plugin (complied using gcc/g++ version < 5.0)  and files can now be made watertight.

@gonuke this look alright?

Thanks @Pshriwise


ARROW-4825: [Python] Destruct MemoryPool after its buffers.

This PR introduces Buffer.set_allocator method that is used to ensure that memory pool instance is destructed after garbage collecting all its buffers. Resolves ARROW 4825.

@wesm are you of the same opinion on this PR as https://github.com/apache/arrow/pull/3876?

Yes, I don't think this provides enough safety. If we are going to allow users to construct new MemoryPool* objects in Python then I think we need a facility (like an internal memory pool set) to ensure they don't get destructed prematurely


1.1.0 (ADAS) - FEATURE - enable security - WEB-Login screen popped up ..

the username/password could be in a config file or in mysql

Hi Varsha,
Are you suggesting to enable security on SSM UI?

Yes, to enable Kerberos on SSM

OK. Can you rephrase the title, to emphases that enable Kerberos support?  Also what's "the WEB-Login screen popped up" refer to?

Few weeks back when Kai was in SC, he mentioned about having a single user login using a web screen pop up when we open up SSM Gui. I dont' believe that is part of 1.1.0 branch and hence I mentioned the support for this. Again this was just a interim solution till Kerberos is officially supported


Add support for inner-class Visitor interfaces

on my wish list :)

The limitation is that you can't generate inner class from annotation processor. Visitors declared as inner-classes can be processed, but outer-level class will be generated for each inner-class visitor.
I have intentionally left this feature out as the connection between visitor-interface and value-class becomes too obscure... Do you think that it's worth it?

The use case is when you manually extends the generated value class.
In this case I would like that the Visitor interface be an inner class of the extending class.
An example of what I want:
public final class List<T> extends BaseList<T> {

   ...

   @GenerateValueClassForVisitor(resultVariableName = "R", valueClassName = "BaseList")
   public interface Cases<T, R> {
      R nil();
      R list(T head, List<T> tail);
  }

  public <R> R match(Cases<T, R> cases) {
      return super.accept(cases);
  }
  ...
}

Ideally I would also be able to customize the accept method name through the annotation.

You still can't do this because of circular dependencies.
See this commit.
There is working example as close to yours as I see possible:

BaseFancyListCases.java
FancyList.java

I see, yes I forgot the circular dependencies. Thank-you for spending the time to analyse this issue.
Maybe for this use case what I ultimately need is a generator of "typeclass" instances (of Show, Eq, Hash), which I could then use in my List implementation.

You can do what you want with open recursion. See:

BaseFancyListCases.java
FancyList.java

Yes this is what I want.
If BaseFancyListCases could be an inner (package-private) interface that would be perfect.

In my example BaseFancyListCases is package private interface. FancyList provides public inner interface Cases as part of it's public API to use in it's "match" method. It's full working example with current version of adt4j.
BaseFancyListCases can be innter-interface, but it's enclosing class should be some class other than FancyList. Otherwise this will create unresolvable for adt4j circular dependency.

@jbgi, does open-recursion version fits your needs?

yes. thank-you!

Inner classes should work now.

Works great!


⚠ TorrentLeech has degraded performance

In d067206, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 4957 ms

Resolved: TorrentLeech performance has improved in ebcd2bd after 6 minutes.


Add content to Hello World with Docker in Streaming Integrator Docs

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? Yes
Ran FindSecurityBugs plugin and verified report? Yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: RukshiW:x: nilminiwso2:x: sajithshnYou have signed the CLA already but the status is still pending? Let us recheck it.


how to start it and is auto trading ?

Hello , can i know how to unstall and start it , and can i use it for auto trading ? Thanks You!

The project has only been set up to training a model.  If you wanted to use it for trading you would have to implement the API and update functionality yourself. Would not recommend it


fix(libsinsp): removed libsinsp::events::set.data() writable method.

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area libsinsp
Does this PR require a change in the driver versions?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.11.0

/cc @incertum @Andreagit97 @jasondellaluce

This has been extracted by #889

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Use server.Options.Logger in server/http

Hi,
Just a little contrib to use server.Options.Logger in server/http instead of logger.DefaultLogger

Good addition, thanks!


Add an explicit "self" field in HalModelSerializer?

My workaround for #18 was to manually declare the self field on the serializer:
self = HalHyperlinkedIdentityField(
    view_name='event-detail',
)

I also wanted to display only that field so I set fields = ['self'].  Out of curiosity, I tried limiting another serializer to just self and got an error that it was not declared on the serializer.
Is there always a better way to get a link-only representation so fields = ['self'] is anti-pattern?  Or is this a reasonable thing to do in some circumstance that the HalModelSerializer should explicitly declare this field (technically URL_FIELD_NAME) so it can be included in fields?
FWIW due to error checking code, the HalModelSerializer would also need a Meta.fields that includes this value.

I closed #18 after realizing it was a drf limitation and not a drf-hal-json issue.  However, this proposal would provide a partial workaround for that issue.  By default, each item (if it uses a Hal... serializer) would include a _lists entry with at URL_FIELD_NAME field.  If you want just a list of links, you could use the self workaround.

I had followed the original drf-hal-json README and was missing the self setting.


[FLINK-11632] Add new config option for TaskManager automatic address binding

What is the purpose of the change
Introduce a new configuration option for TaskManager to explicitly control address bind policy. See [FLINK-11632].
Brief change log

flink-config.yaml is extended with a new optional config key taskmanager.host.bind-policy
the TM's automatic address binding mechanism is changed to not use heuristics by default. Instead a simpler hostname picking mechanism is used.
the old automatic address binding mechanism is kept available via explicit configuration taskmanager.host.bind-policy: auto-detect-hostname.
the TM's automatic binding mechanism is extended to be configured to use host's ip address, instead of hostname (for setups, where TMs are not reachable by DNS name, for example Kubernetes).

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: yes
The S3 file system connector: don't know

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs

@uce, @tillrohrmann thanks for feedback. I'll reopen an alternative PR targeting main branch.


Sync eng/common directory with azure-sdk-tools for PR 8396

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/8396  See eng/common workflow

/check-enforcer override


NuGet installed test adapters breaks NUnit on mono

If in project output folder presence any of "nunit.core.dll", "nunit.core.interfaces.dll" or "nunit.util.dll" (and they will if project has installed test adapter from NuGet), than attempt to run any test from that folder under mono will fail with InvalidCastException.

Can you give more precise instructions for reproducing this? That is, what steps do you take to run under mono?

Please look on example:
https://github.com/iskiselev/TravisExperiments/commit/3dd10829ba98beb459f13fce51a3393d969cd381
Here is sample of running this application on mono:
https://travis-ci.org/iskiselev/TravisExperiments/builds/76875707
Please look on next log lines:
[0K$ mono ./testrunner/NUnit.Runners.2.6.4/tools/nunit-console.exe ./bin/TravisExperiments.dll
WARNING: The runtime version supported by this application is unavailable.
Using default runtime: v4.0.30319
NUnit-Console version <IP_ADDRESS>50
Copyright (C) 2002-2012 Charlie Poole.

Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A. Vorontsov.

Copyright (C) 2000-2002 Philip Craig.

All Rights Reserved.

Runtime Environment -
   OS Version: Unix <IP_ADDRESS>
  CLR Version: 4.0.30319.17020 ( Mono 4.0 ( 4.0.3 (Stable <IP_ADDRESS>/d6946b4 Tue Aug  4 09:43:57 UTC 2015) ) )

ProcessModel: Default    DomainUsage: Single
Execution Runtime: mono-4.0
Unhandled Exception:
System.InvalidCastException: Cannot cast from source type to destination type.

Server stack trace:
  at (wrapper xdomain-dispatch) NUnit.Core.RemoteTestRunner:Load (object,byte[]&,byte[]&)

Exception rethrown at [0]:

  at (wrapper xdomain-invoke) NUnit.Core.RemoteTestRunner:Load (NUnit.Core.TestPackage)
  at NUnit.Util.TestDomain.Load (NUnit.Core.TestPackage package) [0x00000] in <filename unknown>:0
travis_time:end:02979cab:start=1440351695306066495,finish=1440351695623284465,duration=317217970

The command "mono ./testrunner/NUnit.Runners.2.6.4/tools/nunit-console.exe ./bin/TravisExperiments.dll" exited with 156.
travis_time:start:20c7823c
$ rm bin/nunit.core.dll
travis_time:end:20c7823c:start=1440351695626905941,finish=1440351695630482674,duration=3576733

The command "rm bin/nunit.core.dll" exited with 0.
travis_time:start:01d9aa98
$ rm bin/nunit.core.interfaces.dll
travis_time:end:01d9aa98:start=1440351695633992716,finish=1440351695637427931,duration=3435215

The command "rm bin/nunit.core.interfaces.dll" exited with 0.
travis_time:start:003828b3
$ rm bin/nunit.util.dll
travis_time:end:003828b3:start=1440351695640932866,finish=1440351695644434570,duration=3501704

The command "rm bin/nunit.util.dll" exited with 0.
travis_time:start:03d6d087
$ mono ./testrunner/NUnit.Runners.2.6.4/tools/nunit-console.exe ./bin/TravisExperiments.dll
WARNING: The runtime version supported by this application is unavailable.
Using default runtime: v4.0.30319
NUnit-Console version <IP_ADDRESS>50
Copyright (C) 2002-2012 Charlie Poole.

Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A. Vorontsov.

Copyright (C) 2000-2002 Philip Craig.

All Rights Reserved.

Runtime Environment -
   OS Version: Unix <IP_ADDRESS>
  CLR Version: 4.0.30319.17020 ( Mono 4.0 ( 4.0.3 (Stable <IP_ADDRESS>/d6946b4 Tue Aug  4 09:43:57 UTC 2015) ) )

ProcessModel: Default    DomainUsage: Single
Execution Runtime: mono-4.0
.Test executed

Tests run: 1, Errors: 0, Failures: 0, Inconclusive: 0, Time: 0.01374 seconds
  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0

There I've run tests from sample project using nunit-console.exe and it fails with System.InvalidCastException, after it I've deleted "nunit.core.dll", "nunit.core.interfaces.dll" or "nunit.util.dll" and run it again - and everything passed sucessful.
You can reproduce same on windows if you install mono, and then run any nunit test runner with mono.

There seems to be a combination of issues causing your problem.
NUnit 2.x uses the 2.0 runtime although it can (and does) run under higher runtimes. You are using Mono 4.0, which no longer supports the 2.0 runtime, so everything gets run under the higher level runtime. That's what's happening with you.
NUnit 2.x has always had an issue with it's internal assemblies being referenced by the tests. We actually expect those assemblies to be not found and their loading is handled by an AssemblyLoad event. I hadn't heard of a problem before simply because the assemblies are present, without being referenced. Check if you have references.
The Nuget installation creates references to the assemblies, and leaves them as CopyLocal = true. We then set CopyLocal=false in a powershell script. If you did the nuget installation on a Linux system, then the powershell script wouldn't run, so that may be a contributing factor. You should change CopyLocal.
So there seem to be two issues: 1) how are these assemblies ending up in your output directory and 2) exactly why that's a problem.

Thank you for clarification, looks like setting all referenced libraries as not for copy local resolve issue.
I've checked once more - if I install package under VS2015, it does not set CopyLocal=false.
It works good in VS2013.

According to http://blog.nuget.org/20150729/Introducing-nuget-uwp.html and https://docs.nuget.org/release-notes/nuget-3.1 support of install.ps1 is deprericated and was removed in NuGet 3.1.
Maybe it worse to ask question in NuGet/Home#627 how to support developer only libraries dependencies (copylocal=false) without powershell install script.

Thanks for asking them.

Tested it more - install.ps1 is called, but $package contains only two properties - Id and Version. There is no AssemblyReferences inside it, that's why script do nothing.

I had a conversation with the nuget team about this, and it is a bug in Nuget 3, see https://github.com/NuGet/Home/issues/1245

So let's go back to the original issue and see what remains.
How are the NUnit assemblies getting onto the Travis system?  Either one will be a problem when trying to build the project on a Linux system, if the assemblies end up in the output directory.

Really this issue will be resolved as soon as #78 will be resolved.
I have next scenario: project was created in Visual Studio 2015, after it NUnitTestAdapter packege was installed. Due to #78, nunit assemblies was not marked with CopyLocal=False.
After it project was saved and checked into repository. On Unix side, packages was restored (no script run at this point and it is right) and project built. On this step nunit assemblies were placed to output folder, as they were explicitly referenced by project with CopyLocal=True.
There is still small problem, that if somebody will install NUnitTestAdapter in non-Visual Studio environment (as install.ps1 will not run at all or will not have DTE), that user may have problems with running test case, but I don't think that anybody will install NUnitTestAdapter out of Visual Studio environment.
It still may worse to check why nunit fails if assemblies are in output folder, but you also could ignore it - as it is already incorrect behavior. Probably nunit test runners itself may do some more verbose output for this problem.
So, I think you could close this issue - thank you for your help.

I'd say it's not generally a good idea to check the adapter into the repo for any application that runs on non-windows systems. If you can find an alternative, I'd do that. One way would be to use the vsix.
In any case, I see it the same way and I'll close this issue.


refactor name scheme for External API

The current proposed name scheme for Disgo follows this pattern:
// Use resources to represent Discord objects in your application.
disgo.Resource<API Resources>

// Use events to represent Discord events in your application.
disgo.Event<API Events>

// Use the client to manage the bot's settings.
disgo.Client.Config.<Settings>

// Use requests to exchange data with Discord's REST API.
disgo.Request<Endpoints>

// Use sessions to handle events from Discord's WebSocket Sessions (Gateways).
disgo.Client.Session.Handlers.Add(<handler>)
disgo.Client.Session.Handlers.Remove(<handler>)

// Use the client to manage the optional cache.
disgo.Client.Cache.<Settings>
disgo.Client.Cache.<Requests>
disgo.Client.Cache.<...>

// Use the client's shard manager to handle sharding automatically or manually.
disgo.Client.Shard.<Settings>
disgo.Client.Shard.<map[Session][]map[Shard][]GuildIDs>

Here is an alternative proposed name scheme:
// Use resources to represent Discord objects in your application.
disgo.Res<API Resources>

// Use events to represent Discord events in your application.
disgo.Evt<API Events>

// Use the client to manage the bot's settings.
disgo.Client.Config.<Settings>

// Use requests to exchange data with Discord's REST API.
disgo.Req<Endpoints>

// Use sessions to handle events from Discord's WebSocket Sessions (Gateways).
disgo.Client.Session.Handlers.Add(<handler>)
disgo.Client.Session.Handlers.Remove(<handler>)

// Use the client to manage the optional cache.
disgo.Client.Cache.<Settings>
disgo.Client.Cache.<Requests>
disgo.Client.Cache.<...>

// Use the client's shard manager to handle sharding automatically or manually.
disgo.Client.Shard.<Settings>
disgo.Client.Shard.<map[Session][]map[Shard][]GuildIDs>

An example with the current name scheme.
request := disgo.RequestCreateApplicationCommand{
    Name: "main",
    Description: "A basic command",
}

An example with the alternative name scheme.
request := disgo.ReqCreateApplicationCommand{
    Name: "main",
    Description: "A basic command",
}

The first name scheme is more consistent, but requires slightly more effort. Both name schemes allow the user to use autocompletion effectively.
Implementation
There are two ways to implement the name scheme into the library:

Define the name scheme in the library itself (such that it is bundled with no change required).
Define the name scheme at bundle-time where the bundle file that contains all definitions (disgo.go) is refactored (via generator).

1 is easier in the long-term, but requires slightly more typing initially (if creating the parser doesn't require more effort).

I'm leaning towards the alternative name scheme; which will be applied once #3 is completed.


Q: Xamarin XAML to HTML

I'm loving the concepts of blazor that can be used to create mobile applications using the Xamarin native controls of the mobile platform; I was wondering if there was any view on using the same concept with the Xamarin XAML syntax rendering to HTML a true cross-platform-user-interface render anywhere ;)
great work guys

Hi everyone, it's been quite a while, so I'd like to share several updates that I think are relevant to this issue:

The announcement of .NET MAUI, Microsoft's official cross-platform UI system, planned for 2021. The plan is that Mobile Blazor Bindings will eventually move away from the Xamarin.Forms platform and move to .NET MAUI (which shares the same roots). You can follow .NET MAUI here: https://github.com/dotnet/maui.

If you're looking to build a cross-platform app that can also render to HTML, .NET MAUI would be the place to have that discussion. Mobile Blazor Binding is mostly focused on how you write the app, and not as much on where it runs (outside of having templates, and some platform-specific APIs).

New support for Hybrid Apps in Mobile Blazor Bindings Preview 4, announced today: https://devblogs.microsoft.com/aspnet/hybrid-blazor-apps-in-mobile-blazor-bindings-july-update/

If you're looking to build a cross-platform app using HTML as the UI rendering tech, please check out this preview! You can learn a bit more here: https://github.com/xamarin/MobileBlazorBindings/issues/121


Support uprobe and kprobe?

I'm new in BPF. Does this rbpf support kprobe and uprobe? If support uprobe, does it has context switch issues?

Hi, no it does not support kprobes or uprobes. This project is a Rust implementation of an eBPF runtime, but it does not include any attach point. To run a program, you need to explicitly call the library function, and to pass it the memory you want to use as a context.
This is different from what Linux does, for example. The kernel has an eBPF interpreter and a JIT-compilers, which rbpf tries to replicate; but the kernel also has access to all the memory of the system and can attach probes to kernel or user space function. You would have to implement something similar to handle the probes, likely as a kernel module to have access to kernel context, if you wanted to replicate kprobes/uprobes with rbpf.
Similarly, rbpf does not attach programs to network interfaces. You could write a program that receives packets from a socket and run an eBPF program on them, for example. But rbpf itself is just a library to run user programs on a memory area provided by the user.

Thanks a lot.


⚠ Bofrost.be has degraded performance

In e30e67c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1130 ms

Resolved: Bofrost.be performance has improved in 4d24605 after 6 minutes.


feat: Add file backend for audit logs

Adds a new audit log backend named file which simply writes log entries to a file (or stdout/stderr). That file can then be picked up by log aggregators for archival and querying.

Codecov Report

Merging #909 (5787032) into main (412a40d) will decrease coverage by 0.04%.
The diff coverage is 47.77%.

@@            Coverage Diff             @@
##             main     #909      +/-   ##
==========================================
- Coverage   54.56%   54.52%   -0.05%
==========================================
  Files         102      104       +2
  Lines       11120    11254     +134
==========================================
+ Hits         6068     6136      +68
- Misses       4541     4602      +61
- Partials      511      516       +5

Impacted Files
Coverage Δ

internal/audit/file/conf.go
0.00% <0.00%> (ø)

internal/server/server.go
54.54% <ø> (ø)

internal/audit/log.go
14.63% <3.57%> (-37.32%)
:arrow_down:

internal/audit/file/log.go
61.66% <61.66%> (ø)

internal/engine/engine.go
72.72% <0.00%> (+0.80%)
:arrow_up:

internal/storage/index/builder.go
84.35% <0.00%> (+5.02%)
:arrow_up:

internal/compile/errors.go
55.55% <0.00%> (+15.87%)
:arrow_up:


Orphaned embedded objects are not deleted when migrating from Object to Embedded Object

How frequently does the bug occur?
-- select --
Description
According this ticket migration from object to embedded should now be automatic in core and orphaned children should automatically be deleted: https://github.com/realm/realm-core/pull/5737
This doesn't seem to work in the Kotlin SDK. I have a unit test where I have an orphaned object and do an upgrade. Instead of deleting the orphaned object, the migration fails and the app crashes. I have no such issues on iOS
Stacktrace & log output
java.lang.IllegalStateException: [RLM_ERR_ILLEGAL_OPERATION]: Cannot convert 'MyEmbeddedObject:' to embedded: at least one object has no incoming links and would be deleted.
at io.realm.kotlin.internal.interop.CoreErrorConverter.asThrowable(CoreErrorConverter.kt:44)
at io.realm.kotlin.internal.interop.realmcJNI.open_realm_with_scheduler(Native Method)
at io.realm.kotlin.internal.interop.realmc.open_realm_with_scheduler(realmc.java:1757)
at io.realm.kotlin.internal.interop.RealmInterop.realm_open(RealmInterop.kt:194)
at io.realm.kotlin.internal.interop.RealmInterop.realm_open$default(RealmInterop.kt:183)
at io.realm.kotlin.internal.ConfigurationImpl.openRealm$suspendImpl(ConfigurationImpl.kt:110)
at io.realm.kotlin.internal.ConfigurationImpl.openRealm(Unknown Source:0)
at io.realm.kotlin.internal.RealmImpl$1.invokeSuspend(RealmImpl.kt:125)
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
at kotlinx.coroutines.EventLoopImplBase.processNextEvent(EventLoop.common.kt:284)
at kotlinx.coroutines.BlockingCoroutine.joinBlocking(Builders.kt:85)
at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking(Builders.kt:59)
at kotlinx.coroutines.BuildersKt.runBlocking(Unknown Source:1)
at io.realm.kotlin.internal.platform.CoroutineUtilsSharedJvmKt.runBlocking(CoroutineUtilsSharedJvm.kt:22)
at io.realm.kotlin.internal.platform.CoroutineUtilsSharedJvmKt.runBlocking$default(CoroutineUtilsSharedJvm.kt:21)
at io.realm.kotlin.internal.RealmImpl.<init>(RealmImpl.kt:107)
at io.realm.kotlin.internal.RealmImpl.<init>(Unknown Source:0)
at io.realm.kotlin.internal.RealmImpl$Companion.create$io_realm_kotlin_library(RealmImpl.kt:285)
at io.realm.kotlin.Realm$Companion.open(Realm.kt:82)
at sp.repcount.database.RealmFunctionsKt.getRealmInstance(RealmFunctions.kt:32)

Can you reproduce the bug?
Always
Reproduction Steps
No response
Version
1.10.1
What Atlas App Services are you using?
-- select --
Are you using encryption?
-- select --
Platform OS and version(s)
Android
Build environment
Android Studio version: ...
Android Build Tools version: ...
Gradle version: ...

I assume that this is exactly the same issue that used to be in the Java SDK: https://github.com/realm/realm-java/issues/7769 and was fixed by https://github.com/realm/realm-java/pull/7772

Hi @sipersso. Thanks for the report. It is a bit different in Kotlin as we do not yet support manual migration and can handle this on a per class basis. Only option in Kotlin is to set the default behavior on the configuration. We will see to expose that option on the configuration.

I don't understand. I did supply a migration block and was able to do the same workaround as I was previously using in java before it was handled automatically. What default behavior on the configuration are you talking about?

FIY: Same goes for when an object has multiple incoming links. This doesn't work either, but I guess it is the same issue?  Any updates on when this would be fixed?

@rorbech any info on when this might be released? I am currently blocked from releasing a big feature because of this. Would be nice to get a feel for how long it might take so I can adjust priorities.

I noticed that https://github.com/realm/realm-kotlin/pull/1490 is now merged. Any updates on the release? I can use the snapshot build for beta testing, but hesitate to use snapshots in production.


Product total inventory for all locale is showing wrong

Bug report
Title
Product total inventory for all locale is showing wrong
Issue Description
Product total inventory for all locale is showing wrong
Preconditions
1. framework Version- master

Steps to reproduce
1. step1- add a new locale
2. step2- create a product
3. step3- check the total qty for that particular product

Actual result

An inventory does not depend on the locale.


Upgrade kernel to 4.14.22

See https://github.com/agherzan/meta-raspberrypi/blob/f86f9f0013e11311c66fc640fbb43b399606ca2f/recipes-kernel/linux/linux-raspberrypi_4.14.bb, should be supported upstream.
Unsure of any additional changes need to be made. Just creating this pull request to start the conversation on the kernel upgrade.
Related: https://forums.resin.io/t/update-raspberry-pi-kernel/2843

@mbardelmeijer we currently use the rocko branch of the BSP and in there the kernel is at version 4.9.59 https://github.com/agherzan/meta-raspberrypi/tree/rocko/recipes-kernel/linux

If the BSP will not update the kernel in rocko, it is most likely that kernel 4.14 will be in the next release (sumo) which should happen sometimes in April.


[Documentation] Fix: Rename framworks.svg to frameworks.svg (#5403)

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR focuses on modifications to the SVG file located at apps/portal/public/icons/feature-cards/framworks.svg, likely to update the design or improve the icon representation.
Detailed summary

Updated the SVG file framworks.svg to reflect design changes.
Adjusted the dimensions and paths within the SVG for better rendering.
Enhanced visual elements to improve clarity and aesthetics.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Merge activity

Nov 13, 7:42 PM EST: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


Corrections to UNICODERANGE_DATA constant

The UNICODERANGE_DATA constant has 3 places where the structure isn't correct:

Bits 27 and 28 are grouped together and should be separate entries
Bit 50 is spread over two entries and should be a single entry
Bit 51 is spread over two entries and should be a single entry

The result is that tests like com.google.fonts/check/unicode_range_bits incorrectly report names of ranges beyond bit 27.
This PR fixes those three errors

thanks! I will review this now

ok. This is indeed a good fix. Thanks!


First review and testing of multi-tenant DCA

Assignment is to review and test the NF multi-tenant version of the DCA, in particular checking that NF-specific configurations are working as expected. Anything of interest will be documented here.

Able to login and see "Neurofibromatosis Demo" project - ✅
Able to select "Imaging Assay" template and other expected templates - ✅
Able to select "IHC Test" folder and other expected options - ✅
Able to generate Excel manifest and download - OK
Manifest looks like expected - no dropdowns, list options incompletely filled in ❌

Robert (Independent) Round 2 findings:

eTag and fileFormat shifted so that drop-down values are in eTag (specific to GSheets)

ANV Round 3 (10:45 AM Jul 11):

Stuck on "Getting Data" in dca-staging, try again later.

ANV Round 4 (11:30 AM Jul 11):

Go to Neurofibromatosis Demo project - OK
Choose "Sample Imaging Data"
Generate, update, and download manifest as .csv: NF-OSI - Imaging Assay - Sheet1.csv
Validate?


Remove static manual test pages

Description
All the pages were moved to a separate repo https://github.com/webrtc/test-pages and is served at https://webrtc.github.io/test-pages/.
Purpose
Remove redundant files and enforce the one app per repo principle.

Merging since the tests are already red and will be fixed by my other PR.


Fix to filtering tutorial.

There was a small bug in the filtering tutorial http://mne-tools.github.io/dev/auto_tutorials/plot_background_filtering.html#baseline-problems-or-solutions.
See if you agree.

LGTM

@Eric89GXL should probably take a look before merging

My PRs always seem to fail on circle for SUBJECTS_DIR not being set. I'm not sure why.

@jaeilepp it looks like this is very out of date, as the baseline stuff isn't even in there:
http://martinos.org/mne/dev/auto_tutorials/plot_background_filtering.html
Any idea why?

The link is for stable, which was updated on 2. of July. And this tutorial section is quite recent. Right?

No actually it is for dev, but it was updated in July. Seems like the updates on martinos aren't working. @agramfort

Circle failed here so I can't look at the result

Thanks @jaeilepp I looked at mne-tools.github.io and confirmed it's backward there


random seed

Hi, I would like to ask if there is a random seed designed in your code

In the current implementation fixing random seed is not used. You can add fixed seed using the code below
torch.manual_seed(seed)
torch.cuda.manual_seed(seed)
torch.backends.cudnn.deterministic = True

Thank you for your answer, I still have a question in the code “logistic_regression_evaluator” line 54-79, why is there a module "train model"?

It is an evaluation procedure for the pre-trained encoder, where a small linear classification model is trained on to of these representations, thus it requires training


Add Header and Body text for News page template

Changes in this PR
Add Header and Body html for News page template
This is taken from the frontend changes PR.
Next step is to add the latest news components

Would benefit from some pictures to try to get a picture of what's changed on the front-end. (This looks like a rearrangement of the body up the page mostly? Is there something more substantial I'm missing?)


[searchlight] Add spherical searchlight

@mjanderson09, the diamond shape in #221 is a spherical mask, right?

@mjanderson09, nevertheless I guess our users want an Euclidean-distance sphere as well, so I will leave this issue open.

@mjanderson09, _apply_mask_and_get_affinity in Nilearn nifti_spheres_masker is a bit unrelated because it uses real-world distance, not voxel distance, but I'll leave it here for reference:
https://github.com/nilearn/nilearn/blob/master/nilearn/input_data/nifti_spheres_masker.py#L22


Update Readme.md add platform variable to description

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


Google Collab Stable Diffusion

I have used colab stable diffusion for few hours couple weeks ago but after that its not letting me produce images. After couple of minutes it says connection errored out and in the colab it says pyngrok cannot found or something. Is this becuase i was a free user in collab? Also i can use this in google collab pro right?

free colab doesn't support sd anymore, I suggest runpod or paperspace that offer a larger variety of GPUs at cheaper price

so you would prefer the paparspace and runpod over google collab pro stable diffusion?

Yes, they offer more GPU choice and they are faster and cheaper
https://www.runpod.io/console/gpu-browse?template=runpod-stable-unified
https://console.paperspace.com/github/TheLastBen/PPS?machine=Free-GPU


diskinfo[0]: unbound variable

I was trying to get a list of devices every half a second, like the code below:
setInterval(function(){
  drivelist.list((error, drives) => {
    if (error) {
      throw error;
    }
    callback(drives);
  });
}, 500);

However, I was getting an error of unbound variable whenever I unplugged the usb flash drive:

It seems that this modification to the line 29 solved my problem:
#Check for diskinfo before continuing
if [ -z "${diskinfo-}" ]; then
  continue
fi
size=${diskinfo[0]}
...

Could you check that?

Thanks a lot for reporting this and investigating and potential solution. We're hitting this same issue on Etcher (https://github.com/resin-io/etcher/issues/1126#issuecomment-282083506), which is great. I'll push a fix today.

@jviotti great! I hope this will fix the issue :)

See https://github.com/resin-io-modules/drivelist/pull/148. I fixed for Linux and OS X, and working on the Windows one, which proved to be trickier.

@Edudjr I finally got the Windows VBScript working \o/ Feel free to review the PR. I'll publish a new version to npm once its merged. Thanks a lot!

@jviotti Awesome! Thanks for reviewing this issue so quickly :)


QueryDelegate now conforms to DistinctSelection interface

In Kotlin implementation of QueryDelegate function distinct() throws UnsupportedOperationException even though it is perfectly fine to perform distinct using delegation (similarly to all other implementations).
This PR follows the same convention as used throughout file to allow distinct to work.

Good catch thanks!


Benchmark result for an AMD 6800

I see the readme in this repo does provide the time/epoch for several GPU.
For future reference, here is my result on a Rx 6800, Rocm v5.2.3, Pytorch-Rocm v5.1.1
Time/Epoch:

Without mixed percision: 2m1s
With mixed percision:  1m7s

So apparently using mixed percision on Navi 21 does bring up certain improving in training time too.

Thanks for this @ffleader1, would you happen to have the accuracy scores too. I'll add it to the README with acknowledgement of course.

Thanks for this @ffleader1, would you happen to have the accuracy scores too. I'll add it to the README with acknowledgement of course.

Hmm
I did not run for 200 epochs. That would take quite a few hours, and I do need my PC for work. However, I do want to add some benchmark result. I will try to utilize my Pc when I am not actively using it. Will get back to you in a few days.

Did you use any special param tuning related to the learning rate?
Because after 200 epoch, without mixed percision, I got a 37% accuracy, which is horrendous, with the running command as you specified in the Read.me

Just the script I've put here, Maybe it need some changes due to how training in RX6800 libs are handled ?


Update scripts and README.md file to deal with Issue #1

Hi Sishuo,
As suggested, here you have a pull request. Below, you can find a list of the steps I followed:

Fork your repository.
Include the changes that we made to run bs_inBV in your ruby scripts.
Update the main README.md file to make it much easier to understand the following:

Installation procedure (i.e., gems, third-party-tools, changes to code required).
Usage + Example.
PAML usage when running MCMCtree with the approximate likelihood calculation.

Once you revise the proposed changes, if you think everything makes sense, you can merge the repos!
All the best,
Sandy

Dear Sandra,
Thanks so much! I will test it soon. meanwhile, i have created a v1.0 release for the previous version.
I think the following is why i thought only "colorize" and "parallel" need to be installed for ruby gems. i will add a testing script on top of your PR. In ruby 3.0+ some of the following have been installed by default (but except for "bio" and "bio-nwk"); I didn't realize the diff ruby versions issue until you pointed it out (not many people use ruby in bioinfo today). any Qs suggestions and Qs pls don't hesitate to let me know. Thanks again!

cheers,
sishuo


Few questions

O have a few questions about the project, some of them could even go on the readme to clarify things for others as well.
1 - How much of the code is sent to the server when genereating a doc? (Only the selected code, the entire file/class, the entire project, the chain of methods called ?)
2 - Are these information sen't to sever ONLY when the function "Write Docs" is called or there is some data being sent periodically?
3 - What is the API called to generate these docs, a open source one? A private one?

Hi @EverNife happy to answer these questions!

1 - How much of the code is sent to the server when genereating a doc? (Only the selected code, the entire file/class, the entire project, the chain of methods called ?)
The entire file
2 - Are these information sen't to sever ONLY when the function "Write Docs" is called or there is some data being sent periodically?
Only when "Write Docs" is called
3 - What is the API called to generate these docs, a open source one? A private one?
All of the functions that powers the servers you can find in the server/ folder or this repo

Thank You Very much for the clarification :D


Maybe AssetPack should not add its /asset route in the end

Today with 1.x AssetPack adds it /asset route to the end of all the existing routes.
Maybe this should be added at the time when you initiate asset->process()
Seems you can force it to add its route by issuing  asset->route() right after your asset->process()

I totally agree! I'm surprised this even works now.
Thank you 👍


Update libsndfile install instructions for Mac

I updated the README.md to recommend Homebrew for installing libsndfile because it's easier and has widespread developer adoption.

While this should be the norm, I have opted for compiling from source
because brew puts files in weird places instead of the usual /usr/local/,
and people were running into linking issues. Compiling from source puts
files in the right places.
If these problems can be smoothed out and documented, I'll be happy to make
the change.

Could you make this pull request with the dev branch and not master?

Could you link me to the issues people were having? I've experienced none so far.
From what I've read about and experienced with Homebrew it puts everything in /user/local. It put the libsndfile headers in /usr/local/include/.
Could you outline your git workflow in the "Contributing" section of the README?


Revisions not linking in comments

Oddly enough, it's working fine on my local server, but the live site is definitely not returning the links properly in the markdown.
My guess is that it's coming from the suggested changes link adapter but without access to the live server, I can't really tell.
Malachi and Obi originally identified this issue in genome/civic-client#582 and genome/civic-client#682
Markdown returned from the live server

And from my local server

This is super weird. Its working fine locally for me, @agraubert, and also in the console on the prod server. For some reason its not working correctly only in the production server environment. I've added a workaround that seems to have fixed it; but I'm still not sure what the issue actually was.


Missing Key, Value in Nuget.config on System.Data.SqlClient to Microsoft.Data.SqlClient porting cheat sheet

Describe the bug
Hello,
I'm following the cheat sheet to migrate from System.Data.SqlClient to Microsoft.Data.SqlClient, and after adding the required configuration and building I'm getting the following error messages:
Exception message:

Build FAILED.

C:\Program Files\dotnet\sdk\8.0.300\NuGet.targets(694,5): error : Unable to parse config file because: Missing required attribute 'key' in element 'add'.

And
C:\Program Files\dotnet\sdk\8.0.300\NuGet.targets(694,5): error : Unable to parse config file because: Missing required attribute 'value' in element 'add'.

Adding random key and value properties solved this issue. is there a specific key: value pair that needs to be used in this case?

@eilonik Please share the FULL config file and .csproj file. The configuration file changes are for .NET Framework projects, not .NET 8

Closing as stale


Maybe there should be an argument 'unique' in the function tf.where()

There IS the case where we wanna find the first position of the True element in a tensor.
suppose there is a tensor a like this:
[array([[[ True, False],
        [False, False]],

       [[False,  True],
        [ True, False]],

       [[False, False],
        [False,  True]],

       [[False, False],
        [ True, False]]], dtype=bool)

and u run tf.where(a), u will get:
 array([[0, 0, 0],
       [1, 0, 1],
       [1, 1, 0],
       [2, 1, 1],
       [3, 1, 0]])]

But we sometimes want a result like this:
 array([[0, 0, 0],
       [1, 0, 1],
       [2, 1, 1],
       [3, 1, 0]])]

which show us the index of first True element instead of all True elements.
I suppose the function tf.where() might as well add an argument unique , indicating whether u wanna show all true element positions or the first one. And the default value of unique should be False.
the prototype might be like this:
tf.where(input, unique=False, name=None)

and for the requirements shown above, i might run something like this:
tf.where(a, unique = True)

I don't think this is a very intuitive extension to the API. Why is it considering uniqueness only in some subset of the dimensions (i.e. 1, 2) but not in 0. I'd expect unique to do something like return
[[0,0,0]]. In any case, we are trying to adhere to NumPy as much as possible which does not have this.

I'm closing this issue due to inactivity. If new information is provided, I'll reopen this issue.


Add traceId header to the response

See https://w3c.github.io/trace-context/

Standard implementation will be available after

https://github.com/spring-projects/spring-boot/issues/36272
https://github.com/spring-projects/spring-framework/issues/30632


iBeacon not tracked by BLE Tracker

Home Assistant release (hass --version):
0.31.1
Python release (python3 --version):
Python 3.4.2
Component/platform:
Bluetooth LE Tacker
Description of problem:
HM-10 Bluetooth Module cannot be seen by the Bluetooth LE Tracker
Expected:
It should be seen by the Bluetooth LE Tracker
Problem-relevant configuration.yaml entries and steps to reproduce:
device_tracker:
  - platform: bluetooth_le_tracker

Additional info:
I was able to detect my iBeacon with the following script
https://github.com/switchdoclabs/iBeacon-Scanner-
I'm not sure if this is a small change to the code thats needed or a larger one, but I thought I'd bring it to attention.
I will be looking into this as well.

Have you found a solution for this?


Every model returns error "Bad CPU type in executable" on Mac M2 Max computer.

When running the v1.2.0 release for macOS, the following error occurs when trying to load every model:
Running command: /Applications/HARP.app/Contents/Resources/gradiojuce_client/gradiojuce_client --mode get_ctrls --url cwitkowitz/timbre-trap --output_path /Users/frank/Library/Caches/HARP/control_spec.json >> /Users/frank/Library/Caches/HARP/system_get_ctrls_log.txt 2>&1
sh: /Applications/HARP.app/Contents/Resources/gradiojuce_client/gradiojuce_client: Bad CPU type in executable

I was able to fix this by following these instructions, i.e. by installing rosetta manually.


Component not drawing over status bar anymore on iOS 13

This component is not working anymore on iOS 13, I am trying to find a solution for this but can't see anything ideal to fix it.

Yeah I have the same issue. I tried setting the .windowLevel on the internal notificationWindow instance that is used for the view -- to no avail.
It seems iOS 13 doesn't let you put any views or windows on top of the status bar!
I think this may be a bug in iOS 13.
I tried doing a quick "hide" of the status bar, then moving my app's window down 20 pix to "fake" it, but this breaks on iPad in windowed mode.
If anyone found a workaround, would love to hear it.

@cculianu Reported to Apple: https://forums.developer.apple.com/thread/124649?start=0&tstart=0

Ok, thanks. I also added a backing set of screenshots and info to the post.  Who knows -- maybe Apple will fix this.  I consider this a bug in iOS, TBH.
Other projects that try to draw to the status bar area are also affected.  Most of them implement a workaround by just avoiding the area now on iOS13.
Sad.

@cculianu Mind that your comment is not visible, you will need to remove the link from them or fake it. Apple is not approving any comments with link...

I put a link to the image...  Oh so I need to have the image url just be in the message but not as an <img= tag?
Why do they even enable HTML images then? LOL
It says "being moderated" -- I take it .. it will never get moderated?

Ok. Fixed. Thanks for the tip.

Yes it never gets moderated...


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v3.17 to before v4.19.196",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=da7141fb78db915680616e15677539fc8140cf53",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9436cdffaf5bc83f41e72dfbd4d9ec885ff7b4f2"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "da7141fb78db915680616e15677539fc8140cf53",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "9436cdffaf5bc83f41e72dfbd4d9ec885ff7b4f2",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "nilfs2: fix memory leak in nilfs_sysfs_delete_device_group\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.19.196 by commit 9436cdffaf5bc83f41e72dfbd4d9ec885ff7b4f2, it was introduced in version v3.17 by commit da7141fb78db915680616e15677539fc8140cf53. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001113


Added 3D elastic dry friction code and test

The docstring for the 3D friction is throwing an error. To build the docs locally see here: https://github.com/tmd-lab/tmdsimpy?tab=readme-ov-file#documentation
You may need to install additional requirements for the repo to build the docs since they have changed recently (see requirements files / install instructions).
Error I am getting:
WARNING: error while formatting signature for tmdsimpy.jax.nlforces.elastic_dry_fric_3d.ElasticDryFriction3D.init_history: Handler <function mangle_signature at 0x7b14a544d5a0> for event 'autodoc-process-signature' threw an exception (exception: Error parsing See Also entry 'This sets force history for evaluations with `force` method for static' in the docstring of init_history in /home/justin/Documents/Research/tmdsimpy/tmdsimpy/jax/nlforces/elastic_dry_fric_3d.py.)

Extension error (numpydoc.numpydoc):

The docstring for the 3D friction is throwing an error. To build the docs locally see here: https://github.com/tmd-lab/tmdsimpy?tab=readme-ov-file#documentation You may need to install additional requirements for the repo to build the docs since they have changed recently (see requirements files / install instructions).
Error I am getting:
WARNING: error while formatting signature for tmdsimpy.jax.nlforces.elastic_dry_fric_3d.ElasticDryFriction3D.init_history: Handler <function mangle_signature at 0x7b14a544d5a0> for event 'autodoc-process-signature' threw an exception (exception: Error parsing See Also entry 'This sets force history for evaluations with `force` method for static' in the docstring of init_history in /home/justin/Documents/Research/tmdsimpy/tmdsimpy/jax/nlforces/elastic_dry_fric_3d.py.)

Extension error (numpydoc.numpydoc):

This has been resolved with the new commits.

Verified that docs build without warnings and that all tests pass.
Pull request is being approved.


New JQtouch with scroll and tabbar

I tryed to make it work with the last JQtouch version but it doesn't
I like the new JQtouch and it will be interesting if your tabbar with iscroll could work

Sorry to post here, i wanted to send you a message directly but i can't,
I just work on a version with a tabbar and want to know if you want to participate, all icons are made with base64 images and mask, this way, we can change color with simple css ect. just tell me what you think. You can delete this post if you want.
http://itabbar.com/
http://www.itabbar.com/iphone-sim/iPhone-simulator.php


Clarify how to generate/modify mappings

I had a hard time understanding how to generate the mappings, now that I got it, here's what would have made it easier for me :)
I don't understand the 4., what does clone the current mapping mean?

Ping @mchlrch

I don't understand the 4., what does clone the current mapping mean?

Point 4 suggests to copy the mapping from a sample project and adapt it - assuming that this is easier than creating a mapping completly from scratch

I see, thanks! Since this section describes how to play with the examples, I changed it like this:
  3. In the Import wizard: Expand General and then click _Projects from Folder or Archive_ . Click Next . Click _Directory_ and select the cloned repository. Click Finish to add it to your Eclipse.
- 4. Open one of the samples like `employee-mapping` and try to clone the current mapping.
+ 4. Open one of the samples like `employee-mapping` and try to modify the current mapping.
  5. Xtext constantly watches the `.xrm` files in your Eclipse project and validates the syntax (ie. *on save*). When the syntax is valid, it automatically writes the R2RML files to `./src-gen`.


Shader compilation exceptions not holding messages

Describe the bug
The following exceptions (for example):
https://github.com/BabylonJS/BabylonNative/blob/432f0db8e0b0ddd98fb9c7744572f87c9c16528b/Plugins/NativeEngine/Source/ShaderCompilerD3D.cpp#L23-L26
https://github.com/BabylonJS/BabylonNative/blob/432f0db8e0b0ddd98fb9c7744572f87c9c16528b/Plugins/NativeEngine/Source/ShaderCompilerD3D.cpp#L93-L96
Don't hold their error messages (the "what" field) when being rethrown here:
https://github.com/BabylonJS/BabylonNative/blob/432f0db8e0b0ddd98fb9c7744572f87c9c16528b/Plugins/NativeEngine/Source/NativeEngine.cpp#L637-L644
Seems like this is just a case of memory getting freed, so should (hopefully) be a pretty straightforward issue to fix.
To Reproduce
I triggered the linker exception by doing something like:
shader.vertex.fx
uniform float _Radius_Top_Left_;

shader.fragment.fx
mediump uniform float _Radius_Top_Left_;

And I triggered the parser exception by trying to compile a shader missing a paren.
Expected behavior
The exception being thrown should hold the error message and show up in the javascript console. Right now the error message is blank:

Screenshots
The "what" of the error here is "":

But the info sink holds useful info:

Same here with the parser:

Other

Platform: Win32

Under the specified circumstances, the empty "what" being being bubbled up is expected from the current implementation because getDebugInfoLog() returns the empty string; the actual compilation error message can be retrieved by getInfoLog(). I'm not sure if we know of a circumstance where getDebugInfoLog() will contain information we need -- if so, we should add it back -- but assuming we're just grabbing the wrong log, I swapped them out in #772, which should address this issue.

@rgerd, can you confirm that this is fixed by #772 and, if so, close this issue?


Error when compiling NFT tutorial code

Taking the code from the “Implementing a basic NFT” section (https://book.getfoundry.sh/tutorials/solmate-nft#implement-a-basic-nft) in the NFT Tutorial, I get a compilation error:
 forge build
[⠒] Compiling...
[⠆] Compiling 5 files with 0.8.13
[⠰] Solc 0.8.13 finished in 23.37ms
Error:
Compiler run failed
error[6275]: ParserError: Source "lib/openzeppelin-contracts/contracts/contracts/utils/Strings.sol" not found: File not found. Searched the following locations: "PATH/TO/REPOBASE/foundry-playground".
 --> src/NFT.sol:5:1:
  |
5 | import "openzeppelin-contracts/contracts/utils/Strings.sol";
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

I installed the library and can find the following path: lib/openzeppelin-contracts/contracts/utils/Strings.sol.

Did you put the remappings in foundry.toml or ramappings.txt?

And how did you install the library? Through forge install or npm?

No, there was no remapping. So I just executed

forge re > remappings.txt

The error stays the same.

Did you put the remappings in foundry.toml or ramappings.txt?

And how did you install the library? Through forge install or npm?

Using forge install transmissions11/solmate Openzeppelin/openzeppelin-contracts.

btw: current code can be found here: https://github.com/thomasbach-dev/foundry-playground/tree/nft-with-solmate
I used nix to install foundry via https://github.com/shazow/foundry.nix

this looks like an issue with the remappings auto-detection that wrongly concatenates contracts twice.
you should be able to fix this by adding a oz remappings directly into foundry.toml

Nice, I was indeed able to get the build running by adding

remappings = ['openzeppelin-contracts/=lib/openzeppelin-contracts/']

to foundry.toml.


LargeBinary has a required field with a default value?

Describe the bug
LargeBinary has a required field, max_length but it has a default value. This means that IDE's can't help a user know that they forget a parameter.
class LargeBinary(ModelFieldFactory, bytes):
    """
    LargeBinary field factory that construct Field classes and populated their values.
    """

    _type = bytes
    _sample = "bytes"

    def __new__(  # type: ignore # noqa CFQ002
        cls, *, max_length: int = None, **kwargs: Any
    ) -> BaseField:  # type: ignore

To Reproduce
class MyModel(Model):
    class Meta:
        abstract = True
        metadata = metadata
        database = database

    data: bytes = LargeBinary()  # IDE cannot give warning here.

Expected behavior
I believe the class should be defined as:
class LargeBinary(ModelFieldFactory, bytes):
    """
    LargeBinary field factory that construct Field classes and populated their values.
    """

    _type = bytes
    _sample = "bytes"

    def __new__(  # type: ignore # noqa CFQ002
        cls, *, max_length: int, **kwargs: Any
    ) -> BaseField:  # type: ignore

An aside
It seems like not all databases require a max_length. Could another option be that the parameter is optional, and only used if the underlying database needs it (such as MySQL)?

Yes, the default value should be removed.
The problem is that at the point of validation the field is not yet assigned to the model hence it does not "know" what dialect is used in the model. And it would only fail during creation of db/table that can be harder to debug/find cause, and I want to fail early (zen of python :) ) The same applies to String, not all backends require length but it's mandatory in ormar.
Will fix that in next release.

Sounds good. This also led me to do some research and learn that SQLite does have a global max of 1GB by default for any BLOB field: https://www.sqlite.org/limits.html

closed in #195


Selection dialog should support some kind of search

Allow users to search by resource SysML type and title (or identifier if there is no title).
See: Updating generated dialogs
Or maybe allow OSLC query Where clause fragment int he Selection dialog search field.
Notes:
Selection Dialog sequence:

User does GET on selector path which calls ServiceProviderService1.ElementSelector (IoTPlatformService.RequirementAndChangeRequestAndResourceSelector)
no changes to this file

ElementSelector dispatches to /org/oasis/oslcop/sysml/elementselector.jsp (iotpselectiondialogselector.jsp) because the first time, terms will be null.
This JSP adds the selectType selection field and dispatches to the delegated-ui-typed.js file.
Also add an additional script to handle the selectType and adds the selected type to the selectionUri?type query parameter which is now used directly in the search

elementselector.jsp (iotpselectiondialogselector.jsp) displays type dropdown list, and allows the user to enter a Java regular expression to match on the dcterms:title of the resources with that type

elementselector.jsp (iotpselectiondialogselector.jsp) uses webapp/static/js/delegeted-ui-typed.js (delegatedUItyped.js) to get the terms search string and makes the second call too get the selection dialog

ServiceProviderService1.ElementSelector is called a second time, this time with terms set to the search string.

ElementSelector calls SysmlServerManager.ElementSelector (CE4IoTConnectorManager.RequirementAndChangeRequestAndResourceSelector) too get the resources that match the terms
ElementSelector needs to be updated to call queryElements
queryElements needs to match on the terms Java regular expression for the dcterms:title

ElementSelector then dispatches to /org/oasis/oslcop/sysml/elementselectorresults.jsp (iotpselectiondialogselectorresults.jsp) to display the results and allow the user to select the target resources for the link.

User then presses OK which dispatches back to the OSLC client application to actually create the links.

[ ] ElementSelector calls queryElements, but isn't providing the resourceType. Don't really need the type dropdown for now because there's no REST API query facility that can use it. Do this later, just match on the name for now. Deprecated by the new approach below.

[ ] Better idea: don't change the elementselector.jsp. Instead, allow the user to enter [type::]regex into the search field. Then fix queryElements to use the type and regex to match the elements.  This will be better because there are a lot of different types in SysML and a dropdown for them all would be impractical. type:: matches UML type syntax.

Note the regex should match element.toString() to match on the dcterm:title if available, otherwise on the dcterns:identifier.

This is now solved. I wanted to make a pull request, but ended up pushing to master anyway :-)


Not able to retrieve consumers

Summary
When a consumer is created with a user name as a guid the consumer is successfully created. After creation when you try to retrieve the consumer using the username, it says consumer not found.
Steps To Reproduce

Create a consumer with consumer name as a guid like 'ff305d54-85b4-341b-da2f-eb6b9e5460fc'.
REQUEST: curl -i -X POST  --url http://localhost:8001/consumers/  --data "username=ff305d54-85b4-341b-da2f-eb6b9e5460fc"

Try to retrieve the consumer .
REQUEST: curl http://localhost:8001/consumers/ff305d54-85b4-341b-da2f-eb6b9e5460fc

Expected Result: consumer details are returned

Actual Result: Got : {"message":"Not found"}

Additional Details & Logs
While retrieving consumer there is a validation "^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$" to select the data as username or id. If guid is not a valid consumer name, there should be a validation restricting the same.

Kong version ($ kong version)
Kong debug-level startup logs ($ kong start --vv)
Kong error logs (<KONG_PREFIX>/logs/error.log)
Kong configuration (registered APIs/Plugins & configuration file)
Operating System

this is by design, if it is a uuid, then it is considered the Kong ID, not the consumer name. An alternative might be to use custom_id.

Should we consider not allowing uuid for consumer name?


Changes on style and implementing extra div's

Hi Vedasree M,
Nice project! I noticed your todoapp could have a bit more margin between the elements and a delete button with fixed size.. I have made some changes, is that ok if I make a pull request to your project?  So you can analyze and see if changes are worth it.
Thank you!

Thank you for your feedback and suggestions! I appreciate your willingness to contribute to the project. Please go ahead and submit a pull request with your changes. I'll review them and get back to you.
Thank you!😊


Create overview image of architecture

https://development-1-5-0-dot-andersen-lab.appspot.com/data/release/latest