Some examples crash when generating NuSMV / NuXMV: Likely class diagrams that have no state machines

The following UmpleOnline examples crash when generating NuXmv. Likely because there are no state machines. But the crash should not happen.
2DShapes, Accidents, Access Control, Accommodations
Likely most other class diagram examples have this problem
The problem should be solved. Also ideally the tests that generate Java from the examples should be enhanced to generate this output, so that we can detect cases that crash in the future.
Error on line 0 : Compiler Error (Generation). Could not generate NuSMV Please report an issue with this entire message and your input code to Umple developers. Stack Trace Follows. To locate the error in the Umple source, set GeneratedSourcePath to root directory where the generated Java is located: e.g. setenv GeneratedSourcePath ~/umple/cruise.umple/src-gen-umple/ Exception java.lang.ArrayIndexOutOfBoundsException (0) in at java.base/java.util.Arrays$ArrayList.get(Arrays.java:4350) at cruise.umple.nusmv.VariableSpecifier.printArray(VariableSpecifier.java:161) at cruise.umple.nusmv.NuSMVCoordinator.getEventAssignConstraint(NuSMVCoordinator.java:1017) at cruise.umple.nusmv.NuSMVCoordinator.generateParentModule(NuSMVCoordinator.java:107) at cruise.umple.nusmv.NuSMVCoordinator.generateModel(NuSMVCoordinator.java:300) at cruise.umple.nusmv.NuSMVCoordinator.generateModules(NuSMVCoordinator.java:563) at cruise.umple.compiler.NuSMVGenerator.generate(NuSMVGenerator.java:100) at cruise.umple.compiler.UmpleModel.generate(UmpleModel.java:1602) at cruise.umple.compiler.UmpleInternalParser.analyze(UmpleInternalParser.java:1133) at cruise.umple.compiler.UmpleModel.run(UmpleModel.java:1530) at cruise.umple.PlaygroundMain.processCommand(PlaygroundMain.java:747) at cruise.umple.PlaygroundMain.handleMessageFromClient(PlaygroundMain.java:459) at cruise.umple.AbstractServer.receiveMessageFromClient(AbstractServer.java:556) at cruise.umple.ConnectionToClient.run(ConnectionToClient.java:335) . More information (9200)

Will look at this during the weekend.
On Wed, Sep 12, 2018 at 9:36 AM TCLethbridge<EMAIL_ADDRESS>wrote:

The following UmpleOnline examples crash when generating NuXmv. Likely
because there are no state machines. But the crash should not happen.
2DShapes, Accidents, Access Control, Accommodations
Likely most other class diagram examples have this problem
The problem should be solved. Also ideally the tests that generate Java
from the examples should be enhanced to generate this output, so that we
can detect cases that crash in the future.
Error on line 0 : Compiler Error (Generation). Could not generate NuSMV Please report an issue with this entire message and your input code to Umple developers. Stack Trace Follows. To locate the error in the Umple source, set GeneratedSourcePath to root directory where the generated Java is located: e.g. setenv GeneratedSourcePath ~/umple/cruise.umple/src-gen-umple/ Exception java.lang.ArrayIndexOutOfBoundsException (0) in at java.base/java.util.Arrays$ArrayList.get(Arrays.java:4350) at cruise.umple.nusmv.VariableSpecifier.printArray(VariableSpecifier.java:161) at cruise.umple.nusmv.NuSMVCoordinator.getEventAssignConstraint(NuSMVCoordinator.java:1017) at cruise.umple.nusmv.NuSMVCoordinator.generateParentModule(NuSMVCoordinator.java:107) at cruise.umple.nusmv.NuSMVCoordinator.generateModel(NuSMVCoordinator.java:300) at cruise.umple.nusmv.NuSMVCoordinator.generateModules(NuSMVCoordinator.java:563) at cruise.umple.compiler.NuSMVGenerator.generate(NuSMVGenerator.java:100) at cruise.umple.compiler.UmpleModel.generate(UmpleModel.java:1602) at cruise.umple.compiler.UmpleInternalParser.analyze(UmpleInternalParser.java:1133) at cruise.umple.compiler.UmpleModel.run(UmpleModel.java:1530) at cruise.umple.PlaygroundMain.processCommand(PlaygroundMain.java:747) at cruise.umple.PlaygroundMain.handleMessageFromClient(PlaygroundMain.java:459) at cruise.umple.AbstractServer.receiveMessageFromClient(AbstractServer.java:556) at cruise.umple.ConnectionToClient.run(ConnectionToClient.java:335) . More information (9200)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/umple/umple/issues/1363, or mute the thread
https://github.com/notifications/unsubscribe-auth/AG0omJDy6RiediuSH6cEhl5NAoQVdqF9ks5uaQ3HgaJpZM4WlZfl
.

Thanks,
I might get a UCOSP student to volunteer though, as I am sure it is pretty trivial

On Sep 12, 2018, at 09:39, Opeyemi Adesina<EMAIL_ADDRESS>wrote:
Will look at this during the weekend.
On Wed, Sep 12, 2018 at 9:36 AM TCLethbridge<EMAIL_ADDRESS>wrote:

The following UmpleOnline examples crash when generating NuXmv. Likely
because there are no state machines. But the crash should not happen.
2DShapes, Accidents, Access Control, Accommodations
Likely most other class diagram examples have this problem
The problem should be solved. Also ideally the tests that generate Java
from the examples should be enhanced to generate this output, so that we
can detect cases that crash in the future.
Error on line 0 : Compiler Error (Generation). Could not generate NuSMV Please report an issue with this entire message and your input code to Umple developers. Stack Trace Follows. To locate the error in the Umple source, set GeneratedSourcePath to root directory where the generated Java is located: e.g. setenv GeneratedSourcePath ~/umple/cruise.umple/src-gen-umple/ Exception java.lang.ArrayIndexOutOfBoundsException (0) in at java.base/java.util.Arrays$ArrayList.get(Arrays.java:4350) at cruise.umple.nusmv.VariableSpecifier.printArray(VariableSpecifier.java:161) at cruise.umple.nusmv.NuSMVCoordinator.getEventAssignConstraint(NuSMVCoordinator.java:1017) at cruise.umple.nusmv.NuSMVCoordinator.generateParentModule(NuSMVCoordinator.java:107) at cruise.umple.nusmv.NuSMVCoordinator.generateModel(NuSMVCoordinator.java:300) at cruise.umple.nusmv.NuSMVCoordinator.generateModules(NuSMVCoordinator.java:563) at cruise.umple.compiler.NuSMVGenerator.generate(NuSMVGenerator.java:100) at cruise.umple.compiler.UmpleModel.generate(UmpleModel.java:1602) at cruise.umple.compiler.UmpleInternalParser.analyze(UmpleInternalParser.java:1133) at cruise.umple.compiler.UmpleModel.run(UmpleModel.java:1530) at cruise.umple.PlaygroundMain.processCommand(PlaygroundMain.java:747) at cruise.umple.PlaygroundMain.handleMessageFromClient(PlaygroundMain.java:459) at cruise.umple.AbstractServer.receiveMessageFromClient(AbstractServer.java:556) at cruise.umple.ConnectionToClient.run(ConnectionToClient.java:335) . More information (9200)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/umple/umple/issues/1363, or mute the thread
https://github.com/notifications/unsubscribe-auth/AG0omJDy6RiediuSH6cEhl5NAoQVdqF9ks5uaQ3HgaJpZM4WlZfl
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/umple/umple/issues/1363#issuecomment-420651547, or mute the thread https://github.com/notifications/unsubscribe-auth/AAdP_s-F-2i8cbzmBeA9gFJ95_AB3B93ks5uaQ6egaJpZM4WlZfl.

Timothy C. Lethbridge, PhD, P.Eng., I.S.P., CSDP
Professor of Software Engineering and Computer Science
Faculté de genie / Faculty of Engineering
Université d'Ottawa / University of Ottawa
Tel:<PHONE_NUMBER>x6685   Fax:<PHONE_NUMBER>    Mobile:<PHONE_NUMBER>
http://www.site.uottawa.ca/~tcl

@opeyemiAdesina   I'm not seeing an attached screenshot; however, have you tried doing a first-build? It would have been necessary after the recent release (needed after every release).

Here is the attached file. I did first and full builds yet got the error.
On Sun, Sep 16, 2018 at 11:02 AM TCLethbridge<EMAIL_ADDRESS>wrote:

@opeyemiAdesina https://github.com/opeyemiAdesina I'm not seeing an
attached screenshot; however, have you tried doing a first-build? It would
have been necessary after the recent release (needed after every release).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/umple/umple/issues/1363#issuecomment-421779086, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AG0omNTRt4e3tAA-fVEDvKWnsXt4gDc1ks5ubmf9gaJpZM4WlZfl
.

--
Opeyemi Adesina, PhD. | Instructional Support Coordinator
School of Computer Science, University of Guelph
Reynolds Building - Room 1112
474 Gordon Street, Guelph, ON N1G 1Y4

It’s generics related. I have resolved it but yet to get the pull request
past Travis CI. Will try soonest once I resolve the issue raised earlier
(in this thread).
On Sun, Sep 16, 2018 at 6:19 PM YiranShu<EMAIL_ADDRESS>wrote:

@opeyemiAdesina https://github.com/opeyemiAdesina I pulled the project
just now and also got an error when I was running first and full builds.
The error is related to JavaCodeGenValidator.java. Did you encounter the
same error?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/umple/umple/issues/1363#issuecomment-421843318, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AG0omB4eVzG7DcJMaZmshs61p1LX9pwVks5ubs5ZgaJpZM4WlZfl
.

--
Opeyemi Adesina, PhD. | Instructional Support Coordinator
School of Computer Science, University of Guelph
Reynolds Building - Room 1112
474 Gordon Street, Guelph, ON N1G 1Y4

This problem was solved by #1364


[Feature] Support ConvNeXt-V2.

Support ConvNeXt-V2
This PR gives an example of using ConvNeXt-V2 in mmdetection.
It requires the latest version of mmclassification-1.x.
Notice
The official finetuning setting of object detection is not released yet.
There are at least three settings unknown:

[ ] What is stochastic depth rate {0.1, 0.2, 0.3, 0.4} mean?
[ ] Does RPN use soft_nms?
[ ] What is sweep layer-wise lr decay?

Could you please provide your reproduction log or result?

Could you please provide your reproduction log or result?

I trained it last weekend. The box AP is aligned with the paper, but the mask AP has a gap of 0.2%.
here is the log:
20230113_110947.log

Could you please provide your reproduction log or result?

I trained it last weekend. The box AP is aligned with the paper, but the mask AP has a gap of 0.2%. here is the log: 20230113_110947.log

Thanks! I just noticed my problem, because I only reproduced the cascade mask rcnn convnext v1 base, the result of box AP 52.35 is my baseline. I keep the same config and change the backbone and retrained weight to v2, then I only get the box AP 52.48. Now I am curious whether v2 has little improvement on cascade mask rcnn.


A value for the 'policyAssignmentManagementGroupScope' parameter must be provided.

Just updated to release seven. Updated the policy.yml file to latest.
Now receiving message when running pipeline

can confirm the variable is set in the correct file

Can you share the content on templates/steps/assign-policy.yml ?

Update:  Troubleshooted with @KingBain and there was a missed merge for Custom Policy assignment step where this parameter is also required.  John, let me know once you've tested the pipeline run.

not able to run the pipeline yet, but I swear the merge only caught the one line
the two assigning spots

adding those two lines resolves the issue.


Count expressions doesn't work

SELECT COUNT(*) AS 'total' from TABLENAME
return null instead of the right value

Duplicate of #121. Thanks for the report!


[Question] reverse proxy web client

Hello, I have several Nav docker containers running inside a VM on Azure, and I'm trying (unsuccessfully) to set up Nginx as a reverse proxy to be able to open Nav web clients from the outside world using the hostname/public IP of the Azure VM.
I wonder if it requires any special configuration or if there is a guide on how to accomplish this not necessarily with Nginx?
I create containers using navcontainerhelper:
    New-NavContainer -accept_eula `
        -containerName $containerName ` // demo
        -auth Windows `
        -WebClientPort $webClientPort ` // 8080
        -credential $credential `
        -includeCSide `
        -licensefile $mylicense `
        -imageName $imageName `
        -updateHosts

my Nginx configuration is:
http {
    include       mime.types;
    default_type  application/octet-stream;
    sendfile        on;
    keepalive_timeout  65;
    server {
        listen 80;

        location /demo/ {
          proxy_pass http://demo:8080/nav;
          proxy_set_header Host $host;
          proxy_set_header X-Forwarded-For $remote_addr;
          proxy_set_header X-Forwarded-Proto $scheme;
        }
    }
}

The port 80 is open from Azure dashboard.
The only thing that happens with this set up is that when I open http://myAzureHostname/demo — it gets replaced with http://myAzureHostname/nav
Thanks

Containernames cannot be used as instance names - you would have to do this with different webclientports for the different containers and then add -additionalparameters @("--publish thatport")

Oh, so if I set WebClientPort to 8080, it's not exposed to the host but -additionalparameters @("--publish 8080:8080") will do that, so I can then proxy to it?

WebClientPort set to 8080 will make it accessible ON the host. the additional parameter will then publish that port so that it is accessible from outside the host. I do not know what nginx does and whether you can do what you want, but you also have to set --env publicdnsname= and based on that, a configuration setting called publicWebClientUrl is calculated by adding the /nav to that publicdnsname - you might have to modify that as well.
Let me know if you make it work - it might be something other people can use as well.

Thanks, Freddy, I'll try it tomorrow will get back to you.

@kostenko we do something similar and have implemented it by setting up the publicDnsName as Freddy mentioned and then in AdditionalSetup.ps1 create a new Web Server Instance with the name we want. That way we have the same web client name internally ("behind" nginx) and externally ("before" nginx) which afaik is necessary because the WebClient otherwise will always redirect you to /nav. We tinkered a lot with rewrite rules but this in the end was the only stable solution we could come up with and it actually is quite easily implemented.
To give you an example: We have a NAV container called "cronus" and through the AdditionalSetup.ps1 we create a WebClient instance called "cronus" inside the container as well. With that we can do a simple nginx config like this
    location /cronus/ {
        proxy_pass http://cronus/cronus/;
    }

For your reference, our AdditionalSetup.ps1 looks like this
$instance = $env:webInstanceName

if (!$restartingInstance -and ($instance -ne "")) {
    Write-Host "Creating NAV Web Server Instance $instance"
    New-NAVWebServerInstance -Server "localhost" `
                            -ClientServicesCredentialType $auth `
                            -ClientServicesPort 7046 `
                            -ServerInstance "NAV" `
                            -WebServerInstance $instance

    Remove-Item Env:\webInstanceName -Force
}

And when we run the container we give it an env param webInstanceName=cronus
If you happen to be at Directions next week, you might want to come by one of my Docker sessions where I'll either explain exactly that scenario directly as part of the session or we can have a discussion about it (and we need to discuss your avatar choice ;) )

In a very near future it will be possible to define the Web Client instance name (Part of Business Central branding), which I guess will make life easier for you.

Yes, then we could remove the custom AdditionalSetup script

Thanks, guys, I didn't have a chance to try @tfenster solution before I left for vacation but it looks pretty clear and is exactly what I was looking for. I'll close the ticket and will write back in 3 weeks.


Run cube animation at constant rate

If the cube is rotated at a fixed amount, the rotation will slow down if the frame rate drops for any reason. This implementation calculates the angle of rotation based on the amount of time passed since the previous frame, so the rate of rotation stays consistent.
I know it's not that big of a deal for the trivial example of a spinning cube, but it's useful boilerplate and a good example to set.

Performance (accd. to caniuse) not supported until iOS 9.2, Android 4.4.

@pindiespace Yes, it does say that. But I just tested it out on my xcode simulator with iOS 8.4, and performance.now() works just fine. But if you guys want, I'll rewrite it. I don't think a full polyfill is necessary in this case - I'll come up with something a little smaller.
BTW, @borismus et al: how far back do you all think we should support in terms of browser/OS/hardware versions? I'm constantly torn about this because on the one hand VR is really heavy stuff anyway, but on the other hand, broad accessibility is super important and I don't wanna make any BS excuses about it.

@brianchirls https://github.com/brianchirls  * I agree - a stub patch
so performance.now() doesn't crash should be enough.
====================

Here's my short list for graceful decay of the VR experience, with feature
detects:

Always polyfill Fullscreen API, if nonstandard, to a common
implementation. Browsers vary a lot on fullscreen, so IMHO it should be
done. Assume resulting Fullscreen API follows W3C version in later code.
This eliminates scattering vendor prefixes throughout otherwise vanilla
code.

Always polyfill Promise, if not supported, to a common implementation
(this would also allow using a Fetch API polyfill for device lists).

If TypedArray not supported, polyfill. This lets IE 9, 10 use the Three
Canvas Render to create a fallback image of the scene.

If tests for WebGL fail, fall back to the Three CanvasRenderer. But only
render a single frame, and put in a warning message. This covers IE9 and
IE10, and older smartphones.

If tests for Object.defineProperties fail, load an ES5 polyfill. Do this
whether or not we're actually going to support VR.

If QuerySelectorAll fails, polyfill. Do this whether or not we're
actually going to support VR so we can build a decent DOM-only warning.

If test for HTML5 Canvas fails, give up (no Flash equivalents anymore).
Rely on warning message in  markup.

If tests for AddEventListener or RequestAnimationFrame fail, give up.
Rely on warning message in  markup.

On Fri, Jan 22, 2016 at 5:44 AM, Brian Chirls<EMAIL_ADDRESS>wrote:

@pindiespace https://github.com/pindiespace Yes, it does say that. But
I just tested it out on my xcode simulator with iOS 8.4, and
performance.now() works just fine. But if you guys want, I'll rewrite it.
I don't think a full polyfill is necessary in this case - I'll come up with
something a little smaller.
BTW, @borismus https://github.com/borismus et al: how far back do you
all think we should support in terms of browser/OS/hardware versions? I'm
constantly torn about this because on the one hand VR is really heavy stuff
anyway, but on the other hand, broad accessibility is super important and I
don't wanna make any BS excuses about it.
—
Reply to this email directly or view it on GitHub
https://github.com/borismus/webvr-boilerplate/pull/93#issuecomment-173924661
.

--
=================
Dr. Pete Markiewicz
Email<EMAIL_ADDRESS>Sustainable virtual design blog:
http://sustainablevirtualdesign.wordpress.com
Sustainability template: http://greenboilerplate.com
*Teaching site: *http://plyojump.com http://www.plyojump.com/
*Buy my book! *- Millennials and the Popular Culture
Lifecourse: http://www.lifecourse.com/store/catalog/lca/mpc.html

AmazonAuthor Page:*
http://www.amazon.com/Pete-Markiewicz/e/B0069VZ7BG/ref=ntt_athr_dp_pel_pop_2
On that page, see my DARPA Grand Challenge 2005 entry video, "Team
Robomonster":
*Team Robomonster *(DARPA 2005 Grand Challenge):
http://robotsthatjump.wordpress.com/category/team-robomonster/
*Indiespace *(1st to sell downloaded music, March 1994):
http://www.indiespace.com. Also see http://archive.org under
"Kspace.com" 1996 for our original site.
============================================================
=================

@pindiespace
Okay, first of all, according to MDN:

The callback parameter is a DOMTimeStamp instead of a DOMHighResTimeStamp if the prefixed version of this method was used. DOMTimeStamp only has millisecond precision, but DOMHighResTimeStamp has a minimal precision of ten microseconds.  Furthermore, the zero time is different: DOMHighResTimeStamp has the same zero time as performance.now(), but DOMTimeStamp has the same zero time as Date.now().

... so I will fall back to Date.now() if performance is not available.
Second, wow that list is ambitious. I admire that level of discipline. But I don't think it's necessary to fall back to THREE.CanvasRenderer. It's a good size extra chunk of code to include for a fancy way to tell a very tiny percentage of the population that they can't really use your site. I think a DOM-based fallback would be sufficient at this point. And I'm not sure rendering such a warning requires the overhead for all those other polyfills either.

Well, the list is partly due to teaching a "web standards" class, so I have
to personally need to include lots of fallbacks to justify what I tell them
to do. Also, at my (!)design school we had IE 8 until 2 years ago, and IE 9
until this quarter.  For corporate environments still trying to standardize
on IE11 it makes sense to make the decay graceful. I put querySelectorAll
because it is used in Util.js, preventing a break.
I do think a fullscreen polyfill or wrapper is needed, since fullscreen
exists in different vendor prefixes on desktop browsers, and isn't there in
mobile browsers (the meaning of 'fullscreen' is not well-defined).
On Fri, Jan 22, 2016 at 11:40 AM, Brian Chirls<EMAIL_ADDRESS>wrote:

@pindiespace https://github.com/pindiespace
Okay, first of all, according to MDN:
The callback parameter is a DOMTimeStamp instead of a DOMHighResTimeStamp
if the prefixed version of this method was used. DOMTimeStamp only has
millisecond precision, but DOMHighResTimeStamp has a minimal precision of
ten microseconds. Furthermore, the zero time is different:
DOMHighResTimeStamp has the same zero time as performance.now(), but
DOMTimeStamp has the same zero time as Date.now().
... so I will fall back to Date.now() if performance is not available.
Second, wow that list is ambitious. I admire that level of discipline. But
I don't think it's necessary to fall back to THREE.CanvasRenderer. It's a
good size extra chunk of code to include for a fancy way to tell a very
tiny percentage of the population that they can't really use your site. I
think a DOM-based fallback would be sufficient at this point. And I'm not
sure rendering such a warning requires the overhead for all those other
polyfills either.
—
Reply to this email directly or view it on GitHub
https://github.com/borismus/webvr-boilerplate/pull/93#issuecomment-174022676
.

--
=================
Dr. Pete Markiewicz
Email<EMAIL_ADDRESS>Sustainable virtual design blog:
http://sustainablevirtualdesign.wordpress.com
Sustainability template: http://greenboilerplate.com
*Teaching site: *http://plyojump.com http://www.plyojump.com/
*Buy my book! *- Millennials and the Popular Culture
Lifecourse: http://www.lifecourse.com/store/catalog/lca/mpc.html

AmazonAuthor Page:*
http://www.amazon.com/Pete-Markiewicz/e/B0069VZ7BG/ref=ntt_athr_dp_pel_pop_2
On that page, see my DARPA Grand Challenge 2005 entry video, "Team
Robomonster":
*Team Robomonster *(DARPA 2005 Grand Challenge):
http://robotsthatjump.wordpress.com/category/team-robomonster/
*Indiespace *(1st to sell downloaded music, March 1994):
http://www.indiespace.com. Also see http://archive.org under
"Kspace.com" 1996 for our original site.
============================================================
=================

In terms of browser support, I'm targeting roughly Android K+ and iOS 8+ on any phone 2+ years old. Desktop fallbacks should work everywhere WebGL is available.

Looks like performance.now should work fine. I'll merge.

Looks like threejs polyfills performance.now for us: https://github.com/mrdoob/three.js/blob/317d6f66542ab63323c5dab6c7a4e9d08c07ca05/src/Three.js#L77

Oh yeah.
Honestly polyfills feel dirty. I prefer shims.
On Jan 22, 2016 6:08 PM, "Ash Connell"<EMAIL_ADDRESS>wrote:

Looks like threejs polyfills performance.now for us:
https://github.com/mrdoob/three.js/blob/317d6f66542ab63323c5dab6c7a4e9d08c07ca05/src/Three.js#L77
—
Reply to this email directly or view it on GitHub
https://github.com/borismus/webvr-boilerplate/pull/93#issuecomment-174082639
.


feat: add health-check as prepublish script

https://coveord.atlassian.net/browse/CDX-1362

Proposed changes
Run health-checks before publishing custom atomic component
Before merging

[ ] publish @coveo/atomic-component-health-check (#1190)

Testing

[ ] Unit Tests:

[ ] Functionnal Tests:

[ ] Manual Tests:

lgtm, no need for the before merging


Renaming of riak_sql_select record

This depends on a parallel commit in riak_ql

:+1: bb32683

@borshop merge


rxjs-es es5 code with es2015 modules

The .js files in rxjs-es contain es2015 code (let, const, arrow functions, etc). The es2015 module declarations are great for tree shaking but the code needs to be transpiled to es5 to be used by most browsers/apps.
My use case was tree shaking out unused rxjs stuff from an Angular 2 library (Ionic Framework).
I reached out to Ben Lesh and he asked me to create an issue and link to the twitter discussion.
Thanks so much,
Dan

This very much is related to any action we would take in #1671 as well but not sure if there was a final verdict from there.
That said, I believe the goal of rxjs-es was indeed to just be ES5.1 w/ ES6 modules, but it wasn't specifically stated AFAICT so this just wasn't thought about and not noticed because the angular peeps must have been running the code through a compiler that understood all ES6 features.

Thanks @jayphelps,
I will close this issue and post my comment there.
Thanks,
Dan

I think #1671 is talking about using rxjs to serve both CMD / UMD / ESM all at one place, while on the other hand, that rxjs-es could just change to provide ES5 w/ ESM in next tag publish, which we could start to use right away.
They couldn't do this right now because the building facility still using TypeScript 1.8, which has to be upgrade to TypeScript v2 in order to generate ES5 code w/ ESM, that's PR #1968 currently on hold with other matter depends on merging.
It'd be really helpful to have ES5 w/ ESM for Angular 2 users (or ES6 devs), because:
0. Angular 2 shipped by ES5 w/ ESM as default and its EventEmitter is extending Subject from rxjs
0. you couldn't get Angular 2 code running without first transpiling rxjs-es by Babel or Bublé etc., because ES6 Class couldn't subclassing by ES5 code
0. you can't therefor to use CMD bundle of rxjs because ESM is what Rollup alike needs for tree-shaking
Right now during development stage, it really helps to use ES6 code without transpiler since it's perfectly fine to run in most browser, so as an experimental you could:

have this repo cloned locally
change TypeScript version to v2
modify npm script compile_dist_es6 to -m es6 -t es5 --lib dom,es2017 and run
link dist folder as new rxjs-es into node_modules inside your project

With setup above, everything works great.
So before #1671 getting settled down, could we have rxjs-es to have ES5 w/ ESM first? (and reopen this issue along)

I agree with @imcotton.
Thanks,
Dan

I'm reopening this. I consider it a bug that it's not ES5 with ES2015 modules

@blesh,
I think it'd be ideal if rxjs-es was completely es2015, and then rxjs took advantage of the module or jsnext:main options to provide es5 code with es2015 import/exports.
That way, for all of the Angular projects that depend on rxjs set-up, they'll switch over automatically (more than likely) to use the es5 code with es2015 modules, and tree shaking will work.
Thanks,
Dan

I think it'd be ideal if rxjs-es was completely es2015, and then rxjs took advantage of the module or jsnext:main options to provide es5 code with es2015 import/exports. By keeping rxjs-es 100% es2015, it is easier to minify by using property renaming if someone was interested in going down that route.

I think the primary use-case for most people is having modules with es5 code for bunding and tree-shaking purposes. The minification angle is interesting, but seems edge-casey.

We decided collective to stop publishing rxjs-es for now, and revisit publishing ES modules and/or entire ES6 code for future releases. We want to unblock v5 release and it's easier to publish something later than to commit to something for 5.0.0 and then change our minds. My gut tells me at the very least we'll publish ES modules somehow in the coming months, at the very least so we don't keep asked about it and tooling that expects it "just works".

I believe issue itself is explained adequately, that we do no longer publish rxjs-es module issue itself does not exists anymore. As discussed in #2335 there are possibly lot of confusions of bringing es2015 modules package without finalized spec, while benefit is nearly non exist such as tree shaking's not working with Rx's codebase. This whole issue of publish native ES modules will be revisted with release planning once later.


Initial changes

To do : Need to add a make file to generate the libraries and generate the emulator binary.
This PR contains the modified files for emulator server.

PR needs a proper descriptive title

This is done in a different way with https://github.com/chipsalliance/caliptra-sw/pull/940


Need to find alternative to node-gyp!Failing million times for Windows x64 build npm install mongoose

I have tried to uninstall/install almost all the possible versions of Visual Studio/Visual C++/Windows sdk almost 12GB's of data but nothing seems to be working with node-gyp.By the amount of time I have wasted with this installation I could have actually wrote couple of native apps with ember or ruby on rails.Why this problem is not permanently fix ?Also the actual documentation has very less help regarding the build errors .

Please fix this for me.

Yeah.
It is a bit annoying to have to install VS just for it. I use it on dependency chain of my projects and I always get this error.

From the install instructions, the important bit is:

All Windows Versions
For 64-bit builds of node and native modules you will also need the Windows 7 64-bit SDK
You may need to run one of the following commands if your build complains about WindowsSDKDir >not being set, and you are sure you have already installed the SDK:
call "C:\Program Files\Microsoft SDKs\Windows\v7.1\bin\Setenv.cmd" /Release /x86
call "C:\Program Files\Microsoft SDKs\Windows\v7.1\bin\Setenv.cmd" /Release /x64

Closing, has been answered.


Support running validation every fixed number of examples

Is your feature request related to a problem? Please describe.
For IterableDataset, we may not know the length of the dataset in advance. Running validation every X examples would be helpful.

:+1: on this, I guess it should be easy to implement with a Hook

I'm able to do something similar that by setting up the trainer like this:
trainer = Trainer(
    ...,
    val_percent_check=kwargs["val_check_interval"],
    val_check_interval=kwargs["val_check_interval"],
)

If you set val_check_interval to (batch_size/len(train_dataset))*nb_batches validating every nb_batches works. Just be sure to make the DataLoaders you use shuffle the data, otherwise the same data will be used over and over again for validation.
Also the ModelCheckpoint-call back has to be adjusted:
checkpoint_callback = ModelCheckpoint(
    ...,
    period=kwargs["val_check_interval"],
)

Early-stopping seems not be influenced by the validation frequency.
Only caveat is that the validation batches are sampled now randomly from the validation dataset, and it's not guaranteed that all data is used after one epoch. Not too sure if that's an issue tho.


Match whole word when search by regex. Post keyword will be excluded by previous keyword when previous keyword is a sub-string of the post one

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
Version: 1.87.2 (Universal)
Commit: 863d2581ecda6849923a2118d93a088b0745d9d6
Date: 2024-03-08T15:21:31.043Z
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 23.4.0
OS Version: macOS Sonoma 14.4 (23E214)

Steps to Reproduce:

Example file to search:

#include <iostream>

void FooBar() { std::cout << "Hello there :)" << std::endl; }

int main() {
  FooBar();
  return 0;
}

Search by regex Foo|FooBar and select Match Whole Word and Use Regular Expression

Compare this result with grep

The 'match whole word' functionality is meant to match the whole input in the search bar as a single word in the text, therefore I wouldn't expect it to match "FooBar" to match in this case (although grep supports this this way). Internally, we simply surround the text by \b regex characters, so it ends up matching differently than you might expect.


[Bug]: Game name and play and have fun is not visible

Give a brief about the bug ✍
In light mode the game name and play and have fun is not visible.
What is the expected behavior? 🤔
Description
In order to make visible we need to change the color of both texts in light theme.
Provide step by step information reproduce the bug 📄
Description
on the light mode  the game is not visible.

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Ashasindhu !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹


Weird listing (out of order package?)

See attached screenshot, currently from: https://pub.dartlang.org/packages?page=4

The weird part is that the angular_components package is out of order. They should be listed in reverse of their last update times, yet, this package has an update time like a week back, without any new version published, listed around newer packages.
/ping @kevmoo @nshahan about the history of this package

I believe the explanation is that I have published the package more recently but they were alpha versions. It must be ordered by publish date (stable or otherwise) but the date is only showing the last stable version that was published.
For reference you can see the dates here on the tags I created when I publish:
https://github.com/dart-lang/angular_components/tags

Ah, yeah, the Versions tab doesn't show the new alpha version, it can be reached only from the direct link of the url. And that doesn't show the publish date either.
@kevmoo: this is clearly confusing. (A) we shall not update the updated timestamp when we have new developer versions or (B) update the formatted timestamp and list it on the versions page too.


Simplified version of the Forward-Forward algorithm using PyTorch

Let me know if the code doesnt work as expected.


Feature-Request: color the whole line

Hi,
currently it is possible to insert coloring of text or background via "<span>" tags.
I would like to have the possibility to set the background cell color for a whole line at once. It should be applied to the computed columns and to the "normal" columns at once.
UseCase:
I use kibana to show metrics, which are showing following fields for example:

kubernetes Node
namespace
number of ready pods
number of scheduled pods

Currently if the number of ready pods is smaller than the number of scheduled pods (target) I compute a status and set background color to green or red.
It would be really nice if I could color backround of all columns (computed and standard cols) by such condition.
Being able to color background of the whole cell and not just the text background would also be really nice.

@piellick
If you’re working on this feature, don’t hesitate to submit a PR!
I would be happy to merge it!

Sorry, I misunderstood your previous comment 😊
But for sure, this feature would be a great enhancement for numerous users, I think.

@piellick @0asp0
So you would like to add templates support for:

[ ] stock columns/metrics
[ ] table cell (not metric value)
[ ] table row

Good news: enhanced table v1.10.0 is out with both 'Rows computed CSS' feature and 'Cell computed CSS' computed column feature!
https://github.com/fbaligand/kibana-enhanced-table/releases/tag/v1.10.0


Add CI, linter and license checkers

Add initial CI to cover lint, license check, DLC
Enforce coding style
Update dependency and provide commands to fix code smell

You need an empty GHA file in default branch first, otherwise, you can't verify in PR.
Or you have to merge first and open another to test.

I will merge this one first, as some lint will fail for the OSPP student to fail. Thanks


Support TCP connections

I started porting the TCPClient, but kind of got stuck in the complexity of the issue.
Is this the correct direction / approach?

Where and how did you get stuck?
I personally would start by directly calling the socket API until I can stablish a connection to some server  while verifying that all the socket calls don't return errors and figuring out what arguments make sense to use with the socket API. Then I would continue to do some basic I/O through the socket. Only after I have understood how the socket API works would I try to create a Rust version of the TCPClient API. But just putting up a minimal connection example that directly uses the socket API would be a great start!

This is their library: https://github.com/spark/firmware/blob/develop/wiring/src/spark_wiring_tcpclient.cpp I wanted to port it to Rust (as directly as possible for now, without making it idiomatic).
That code looks pretty straightforward, the tricky part (for me) is the network / wifi stuff: https://github.com/spark/firmware/blob/develop/wiring/src/spark_wiring_wifi.cpp I can't find the place anymore, but somehow I found a rabbit hole that led to calls to the WICED library, which is pretty big. Maybe bindings to that could be generated with bindgen?

You could try to directly bind to their C++ firmware library for the TCPClient stuff. Bindgen supports C++ libraries but I haven't used it like that before so I can't provide further advice on that.

Ok, initial attempt with current bindgen:
$ cd particle-firmware
$ bindgen --use-core --ctypes-prefix ::ctypes firmware/wiring/inc/spark_wiring_tcpclient.h -- \
    -I firmware/system/inc/ -I firmware/hal/inc/ \
    -I firmware/services/inc/ -I firmware/hal/shared/ \
    -x c++ -std c++14
thread 'main' panicked at 'TranslationUnit::parse failed', /checkout/src/libcore/option.rs:794
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Any idea?

No idea. The error message is not particularly helpful. The documentation says that you will likely need to whitelist some std stuff to make this work but doesn't suggest what should be whitelisted (everything's whitelisted by default). You'll probably have more luck asking around on IRC or looking for a project that creates C++ bindings to see how it does it.


auth/token: add support for token file

reads the token from a file. works really nicely with the new vault agent functionality, so the vault agent manages the token and sidekick manages the secrets.

much appreciated @roboll


IE 7/8  - preview does not work

Great plugin but we need to to support IE 7/8 also :(  . I have read the instruction that it only supports ie9+
So is there a recommended fix, as of now the preview feature does not work , Is there an IE hack identified to make the position fix to make the preview work?

I know this is an old Issue, but I just wanted to say that MrBliz's solution worked perfectly and was very easy to implement :)  Thank you so much!


qdiskinfo: add desktop item

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The current version results in a desktop icon already.
result
├── bin
│   └── QDiskInfo
└── share
    ├── applications
    │   └── QDiskInfo.desktop
    └── icons
        └── hicolor
            └── scalable
                └── apps
                    └── QDiskInfo.svg

There is already a desktop file stored in the upstream repository at /dist/QDiskInfo.desktop

Ah, okay, closing this PR.


Abandoning 3D concept, switching to 2D

Whole engine is rewritten in 2D. Still errors with comunication and drawing world issuses.

Approve


Add CORS headers to media endpoint

This is better done in the Laravel app than in my nginx config

This has been done


[Request] Mobile support

Describe your idea.
This should mainly be done through geode for android, though it could also be a separate launcher. It should have the same menu, just with a button in the pause menu for opening, and a button for closing.
This is mostly a suggestion, feel free to deny this

This would require a new interface because the current buttons are too small for a mobile.


Remote boost from Mastodon throws error "acct is not an user"

💡 Summary
When trying to boost Mastodon's status from its original instance using misskey account, misskey throws "acct is not an user" error.
Following status has been used for testing:

The page below can be accessed via pressing the the boost button below the status:

🙂 Expected Behavior
When using other mastodon account<EMAIL_ADDRESS>in this case) it works fine, resolving the toot user interacted in that instance (twingyeo.kr):

☹ Actual Behavior
When using the misskey account<EMAIL_ADDRESS>it shows following error:

📌 Environment
Mastodon 3.0.1 -> Misskey 11.35.1

Probably 'Remote Boost' is not implemented yet
Mastodonの実装を見ると
webfingerの "rel":"http://ostatus.org/schema/1.0/subscribe" の href に
ユーザーじゃなくて投稿のURLを入れると取得して投稿を表示してブーストなど出来るみたい？だけど仕様が見つからないのでよくわからない

Related to #3899

Maybe related to #2217

This issue is not resolved. It still exists in 12.47.1

なおってないんじゃなくてv12で壊したかサポートされなくなった
Not a user と出る


[FAILED] Failed to start Load Kernel Modules.

Describe the bug
run the command kind create cluster --retain, and then get error message from container
[FAILED] Failed to start Load Kernel Modules.

To Reproduce

download kind
run kind create cluster --retain
kind returns error message ERROR: failed to create cluster: could not find a log line that matches "Reached target .*Multi-User System.*|detected cgroup v1"
check container log and get error message [FAILED] Failed to start Load Kernel Modules.

Expected behavior
it should work :)
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.2.0-rc1
Commit: c0b04d1ff14703d111dc51da00db885dda0b7f13 (v1.2.0-rc1)
System info:
macOS: 14.1.2 (23B92)
CPU: arm64, 12 cores
CPU model: Apple M2 Pro
Model: Mac14,12
Memory: 32 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2023-12-05T13-10-53.264887Z.zip

Screenshots and additional context (optional)
No response

Just noticed the same.  I believe I had it working a couple of weeks ago.  But I might be mistaken. 🤔
Mine in an attempt at trying Cilium:
---
kind: Cluster
apiVersion: kind.x-k8s.io/v1alpha4
nodes:
  - role: control-plane
  - role: control-plane
  - role: control-plane
  - role: worker
  - role: worker
  - role: worker
networking:
  disableDefaultCNI: true
  kubeProxyMode: "none"

Released in v1.2.0.


Round brackets pattern not working

It seems the round brackets pattern doesn't work as expected.
This doesn't work:
pretty-quick --staged --pattern "**/(apps|libs|tools)/**/*"
This does:
pretty-quick --staged --pattern "**/apps/**/*" --pattern "**/libs/**/*" --pattern "**/tools/**/*"
It seems that somehow round brackets aren't supported in this scenario.

Hello, were you able to solve this problem?

A minimal but runnable online reproduction is required.


fix(ui): AVRO Schema viewer

Using assets/tests/mock/schema_input.mock.json as schema input.
Before

After

please let me do some css fixes here…


Download Queue Reorder Broken

Steps to reproduce

Begin downloads, quantity requiring scrolling below screen size, more noticable with more downloads in the queue
Attempt to reorder a single download down the queue

Expected behavior
Dragging a chapter should reorder chapter downloads as positioned
Actual behavior
If you scroll down and begin to reorder, mihon will automatically jump to the top of the queue, item still floating. If you attempt to move the chapter down the queue, especially if scrolling down is needed, all chapters disappear, and reorder becomes close to impossible
Crash logs
No response
Mihon version
0.16.5
Android version
Android 14
Device
Google Pixel 8 Pro
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Screen recording of issue linked below
https://youtube.com/shorts/ZrJBm1wThBs?si=eBSTWmm152-s_2WH


Request for comment: Putting extensions in folders by author

The problem
The current organization of extensions is unsustainable considering the end goal is for everyone to submit every extension
When someone submits an extension called eg. encoding or networking, because we don't have folders, I set the bar very high for merging
Proposed solution
Extensions are instead added in folders, eg. fetch.js would now be GarboMuffin/fetch.js and accessible at https://extensions.turbowarp.org/GarboMuffin/fetch.js
This has benefits:

People can make their own extensions that use common names like "fetch" without polluting everything
Extension authors are given credit more visibly

Problems:

The URLs are ugly, but we generally want people to use the "Copy URL" or "Open Extension" buttons anyways so it's not that bad.

Eventually we may decide that some of these extensions are worthy of being accessible without a username. Using the build server and script, we can either:

Tell the build script to copy GarboMuffin/fetch.js to just fetch.js
Rename the extension to just fetch.js and tell the build server to copy it to GarboMuffin/fetch.js

Either way projects that use the old URL won't break.
The code for making folders work has already been written and merged.
Thoughts?

this is great!

snip
Eventually we may decide that some of these extensions are worthy of being accessible without a username. Using the build server and script, we can either:

Tell the build script to copy GarboMuffin/fetch.js to just fetch.js
Rename the extension to just fetch.js and tell the build server to copy it to GarboMuffin/fetch.js

snip

I think that the former would probably be better since they are still all in their own directories for development. Issue arises if the extension has multiple developers though - how is that treated? especially if they both already have their own extensions

I think that the former would probably be better since they are still all in their own directories for development. Issue arises if the extension has multiple developers though - how is that treated? especially if they both already have their own extensions

I believe contributions on someone else's extensions would be treated like regular pull requests but with an additional reviewer being the extension's author


phantom.args is deprecated and removed from PhantomJS 2.0

The phantom.args is deprecated. It's recommended to use system.args which has a script name as the first argument, and the rest is like phantom.args.
In order to support PhantomJS 2.0 we have to change this line to use system.args.

Could you send a PR please?

Created a pull-request for this - https://github.com/teampoltergeist/poltergeist/pull/567


allow signing other messages

other than known consensus transaction types, e.g. paratime / runtime transactions
fixes #109
preview for unrecognized consensus transaction:

preview for unrecognized signature context:

no translations yet, cc @lindalu1111
UI doesn't support variable length message--this bridge lock transaction used as an example of a runtime tx is a little too long
can't copy text, which prevents people from searching or using tools to analyze the context+message

It would be better if the transaction was shown as pretty printed JSON instead of hex-encoded CBOR. This currently makes it impossible to see what the body is.

I experimented with that briefly, but the byte arrays are ugly. in the
simplest implementation they get serialized as something like {"0": 123,
"1": 45, ...}.
I wonder if it also would choke on non-string dictionary keys.
we'd need to serialize to something slightly more sophisticated than JSON
per se.
other troubles: we don't know what the different byte arrays are in this
problem, because we're assuming that we don't know the structure of the
body. so even if we set up something nice to base64-encode byte arrays,
it's not a good experience for addresses, which usually we see in bech32
On Mon, Aug 30, 2021, 10:41 PM Jernej Kos @.***> wrote:

It would be better if the transaction was shown as pretty printed JSON
instead of hex-encoded CBOR. This currently makes it impossible to see what
the body is.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/oasisprotocol/oasis-wallet-ext/pull/110#issuecomment-908916049,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJDVILWXQOYPGN3VEHQZFJ3T7RTPBANCNFSM5DCZEEUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Disclaimer: This e-mail and any attachments may contain confidential
information. If you are not the intended recipient, any disclosure,
copying, distribution or use of any information contained herein is
strictly prohibited. If you have received this transmission in error,
please immediately notify the sender and destroy the original transmission
and any attachments without reading or saving.

Let's file a follow-up issue for nicer display


Issues when writing assemble into sub directory

Config:
options: {
    layout: 'page.hbs',
    layoutdir: 'source/configuration/layouts/',
    partials: 'source/configuration/partials/**/*.{hbs,css}',
    plugins: ['grunt-assemble-lunr'],
    lunr: {
      dataPath: 'destination/search.json'
    }
  },
  homepage: {
    options: {
      layout: 'homepage.hbs'
    },
    files: [
      {
        cwd: "source/content/_pages/",
        dest: "destination/",
        expand: true,
        src: 'index.hbs'
      }
    ]
  },
  pages: {
    files: [
      {
        cwd: "source/content/_pages/",
        dest: "destination/",
        expand: true,
        src: [ '**/*.hbs', '!index.hbs' ]
      }
    ]
  },
  blog: {
    options: {
      layout: 'blog.hbs'
    },
    files: [
      {
        cwd: "source/content/blog/",
        dest: "destination/blog/",
        expand: true,
        src: [ '**/*.hbs', '**/*.md' ]
      }
    ]
  }

Then I only publish the contents of the destination folder to the webserver, however all the lunr url's are destination/index.html and destination/blog/blogpost.html
An option to add cwd or the ability to remove the destination/ path would be very handy!

assemble: 0.4.42
grunt-assemble-lunr: 0.1.0


[PIO-34]Update templates.yaml

Request to merge templates build with new release version of predictionio 0.10.0-incubating.

Merged to livedoc. I checked the template by doing a clean build.


added clone to title to demo change

Why did the mushroom go to all the parties?
Because he was a real fungi!

lol A+


New placeholder management

Hy,
I couldn't figure out what the value parameter is used for. So I thought, why not use it for something I need. Maybe someone needs that too. Maybe I use the software all wrong...
The current behaviour is this:
$q = $this->qb->select()->setTable("someTable")->where()->equals("id", 12345)->end();
$s = $this->qb->write($q);
// $s looks like this: SELECT someTable.* from someTable WHERE id = :v1
$s = parent::prepare($s);
// Context: the surrounding class extends PDO
$s->bindValue(":id", $id);
$s->execute();
$r = $s->fetch();

New behaviour:
$q = $this->qb->select()->setTable("someTable")->where()->equals("id", "myID")->end();
$s = $this->qb->write($q);
// $s looks like this: SELECT someTable.* from someTable WHERE id = :myID
$s = parent::prepare($s);
// Context: the surrounding class extends PDO
$s->bindValue(":myID", $id);
$s->execute();
$r = $s->fetch();

Note how the placeholder changed from the generic :v1 to a useable (even if the query was generated dymically) :myID - if (for whatever reason) myID was used twice, the placeholder changes to :myID1 and so on.

Maybe I use the software all wrong... well, I did.
The tutorial was not quite clear about the ->getValues() thing in the WHERE-Section


Bitrate from an mp4

Is it possible to get the bitrate of an mp4 using the mp4 Parser library ?

I added an example:
https://github.com/sannies/mp4parser/commit/e9c8e69dd7156bc1870fdac559af7f9bfe03ff9a
https://github.com/sannies/mp4parser/blob/e9c8e69dd7156bc1870fdac559af7f9bfe03ff9a/examples/src/main/java/com/googlecode/mp4parser/BitRate.java


Fix using query expression within query action

Purpose

Fix compilation failure when using a query expression in the From-Clause of a query action

Fixes #40527
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@gimantha @pcnfernando FYI


Build the project with simple JavaScript files

What about people who doesn't know Java and how to build the project? I'm just trying to build it without minifying JavaScript to be able to somehow use only the player. It's a cool project but 90% of people using Three.js can't use Java.

I totally agree with you.
however sadly I'm really not good at Javascript and not able to build it.
but I'm planning to replace core parts with javascript.And share more documents for easy understand who's favorite programming-language.
This project use three.js-wrapper and I'm already start upgrade that.
so maybe this project woulb be update soon.


Creating a baseline for a multi-module project only contains issues from the last module that was run

Expected Behavior

Running detektBaseline should create a baseline file for all the issues in the multi-module project
I am currently trying to follow the instructions for a multi-module project to create a baseline file as described in: https://detekt.github.io/detekt/baseline.html#gradle
Observed Behavior

After running detektBaseLine, baseline.xml will only contain the issues from the last module that was run.
Steps to Reproduce

Multi-module project
Follow steps as described in: https://detekt.github.io/detekt/baseline.html#gradle

Context

baseline for an old multi-module project w/ 10+ modules

Your Environment

Version of detekt used: 1.11.1
Version of Gradle used (if applicable): 6.4.1
Operating System and version: Ubuntu 20.04
Link to your project (if it's a public repository):

Oh, we need to fix that documentation. Set baseline = file("baseline.xml") instead. Or any other relative path. This way the baseline is generated in each module.

I actually did that for the mean time and manually collated all the issues into one baseline file.
But I was expecting to have it ALL collated automatically into that one baseline file thru the detektProjectBaseline task provided in the documentation?

Alternatively you can centralize the baseline generation with the code below (put it in the top level build gradle kts and run with detektProjectBaseline)
val detektProjectBaseline by tasks.registering(io.gitlab.arturbosch.detekt.DetektCreateBaselineTask::class) {
    description = "Overrides current baseline."
    ignoreFailures.set(true)
    parallel.set(true)
    setSource(files(rootDir))
    config.setFrom(files("$rootDir/detekt.yml"))
    baseline.set(file("$rootDir/baseline.xml"))
    include("**/*.kt")
    include("**/*.kts")
    exclude("**/resources/**")
    exclude("**/build/**")
    exclude("**/buildSrc/**")
    exclude("**/test/**/*.kt")
}

I'm not sure I understand your suggestion @sanogueralorenzo. The snippet you provided seems to be the same as the one provided in https://detekt.github.io/detekt/baseline.html#gradle except for the following:
- buildUponDefaultConfig.set(true)
+ exclude("**/buildSrc/**")
+ exclude("**/test/**/*.kt")


Migrate to terraform 0.13

Terraform 0.13 is now available [1] and some changes may be needed on this project in order to support it. This issue intends to track which changes needs to be done.
Libvirt provider support:
https://github.com/dmacvicar/terraform-provider-libvirt/blob/master/docs/migration-13.md
[1] https://www.hashicorp.com/blog/announcing-hashicorp-terraform-0-13/

ping @juliogonzalez @moio

My devel machine is TW, and is in 0.13.3. Any progress here?

We didn't make any progress in this regard, and we cannot say when it will be added on the loop

Disclaimer: Quick & dirty / hacky Workaround, was fine for me to deploy a local Uyuni Dev Environment on my laptop with libvirt.
I'm using Terraform v0.14.5 on openSUSE Tumbleweed (20210212) and was overwriting the required_version parameter as well forced terraform to use the global plugin path.

Install terraform and the libvirt provider
sudo zypper in terraform terraform-provider-libvirt

Add terraform.rc file into sumaform root folder to define a local plugin path

terraform.rc
provider_installation {
  filesystem_mirror {
    path = "/usr/share/terraform/plugins"
  }
}

create / adjust main.tf based on your needs

Check were 'hashicorp/libvirt' will be required and place a override.tf in each location to use 'dmacvicar/libvirt' instead.
terraform providers

override.tf
terraform {
  required_version = ">= 0.14"
  required_providers {
    libvirt = {
      source  = "dmacvicar/libvirt"
      version = "0.6.3"
    }
  }
}

Lot of terraform version related errors will show up, add a override.tf file into each mentioned folder to get rid of them.
terraform init

terraform {
  required_version = ">= 0.14"
}

So far it seems like that the necessary sumaform adjustments might limited to handling the new registry namespace stuff accordingly.

Closing this since we already use version 1.0.10


fix: prevent scrolling while widget not expanded

Ticket: https://lifi.atlassian.net/browse/LF-9007

Found this while testing
Screen.Recording.2024-08-20.at.12.49.56.PM.mov

bug-not-reproducable-screen-capture.webm
Thanks Eugene! I believe this issue must have occured before we got to fix and merge into develop. I cannot reproduce it myself. Let me know if I am wrong and we still have the "flickering" or "jumping" of the widget.


Add Common transform which replaces smart quotes with ascii equivalent

Adding new 'Common transform' in response to #1676. This is a trivial change, my only concern is that the list of "common transforms" that people might want to have from this menu is essentially endless. I feel that adding this one is fine, but feel there is some need here to have something easily extensible that doesn't require adding code to add things to this list (maybe hooked into 'starred' transforms?)
@thadguidry @ettorerizza comments welcome

nb this replaces the following quotes:
“ “ U+201C (left curly quote) -> "
‘ ‘ U+2018 (left single curly quote) -> '
” ” U+201D (right curly quote) -> "
’ ’ U+2019 (right single curly quote) -> '
« « U+00AB (left angle) -> "
» » U+00BB (right angle) -> "
‹ ‹ U+2039 (left single angle) -> '
› › U+203A (right single angle) -> '
„ „ U+201E (bottom quote) -> "
‚ ‚ U+201A (single bottom quote) -> '


basic bundle support + minimal example

This library is exactly what I needed for my project (löve2d + non-blocking/threaded sockets) but it was missing bundle support. I thought I would take a crack at it and thankfully it didn't take too many lines to get it done. I followed the solution for bundles in https://github.com/mhroth/tinyosc and took advantage of as much of the code you already wrote
Main additions:
isBundle(udpM) - looks for the #bundle prefix
oscUnpackBundle - like oscUnpack but returns a table 1 or more tuples {oscADDR, oscTYPE, oscDATA}
I updated the oscdump.lua example to include bundle support

I've played with it, looks good.
My suggestion: the .isBundle() function should return the timetag if true, nil if false. Here's a possible version:
--check if packet is a bundle
function oLvosc.isBundle(udpM)
local iv = nil
local b = udpM:match("^#bundle")
	if b ~= nil then
		iv, _ = oLvpk.unpack("c8", string.sub(udpM, 9, 16))
	end
return iv
end

BTW, my .unpackTIME() function was working incorrectly, at least with liblo. It needed the unpack format strings to force big endian. ('>I>I', instead of 'II'). I didn't see that before, but I don't recall if timetag code was tested outside of Löve2d.
Extracting the 8 byte timetag directly in .isBundle() could be (should be?) moved to another function, as well, for reuse.
I expect to merge your pull fairly soon, and make those changes after. Having added this, I'll work on sending bundles...

Still will merge the pull shortly. I'll do that, then update the tag.
Afterwards, I have new functions that unpack nested bundles recursively (and some support functions for searching and flattening the nested results to simple table lists). After a bit of more testing, I'll update the project with the recursive functions and re-tag.
So your contributions will be integrated (and noted in the source). You really bumped me back into working on this!
Until then, here are the new base library functions:
--check if packet is a bundle
function oLvosc.isBundle(udpM)
local iv = nil
local b = udpM:match("^#bundle")
	if b ~= nil then
		iv, _ = oLvpk.unpack("c8", string.sub(udpM, 9, 16))
	end
return iv
end

--recursively unpack a bundle
--	returns a table of nested msgs and sub-bundles
--      Original (non-recursive) oscUnpackBundle() function by halee88 (https://github.com/halee88)
function oLvosc.oscUnpackBundle(packet, level)
local mpos = 17 -- skip '#bundle+timestamp'
local packet_size
local num_bytes = string.len(packet)
local parsed_messages = {}
	local tc, _ = oLvpk.unpack("c8", string.sub(packet, 9, 16))
	local tsec, tfrac = oLvosc.unpackTIME(tc)
	table.insert(parsed_messages, {'bun', level, tsec, tfrac})
	while mpos < num_bytes do
		packet_size = oLvpk.unpack(">i", packet:sub(mpos, mpos + 4))
		local bunElem = packet:sub(mpos + 4, mpos + 4 + packet_size)
		if oLvosc.isBundle(bunElem) then
			table.insert(parsed_messages, oLvosc.oscUnpackBundle(bunElem, level + 1))
		else
			local oscADDR, oscTYPE, oscDATA = oLvosc.oscUnpack(bunElem:sub(1, packet_size))
			table.insert(parsed_messages, {'tup', level, tc, oscADDR, oscTYPE, oscDATA})
		end
		mpos = mpos + 4 + packet_size
	end
	return parsed_messages
end

Bundles don't have names / labels, so I figure (other than searching the msg address) attaching the timetag and "level" (sub-bundle depth) to each msg is adequate for differentiating between other bundles...

Nice work! I didn't realize bundles could be recursive in OSC. Flattening them is, I think, the right call

The function that flattens the results table:
-- recursively flatten the nested tables in an unpacked bundle to a simple table list of the msgs
local function resultsToList(results)
	local rdata = {}
	if type(results) == 'table' then
		for _, elem in ipairs(results) do
			if type(elem) == 'table' then
				if elem[1] == 'tup'  then
					table.insert(rdata, elem)
				else
					local tmpdata = resultsToList(elem, rdata)
					for _,inlst in ipairs(tmpdata) do
						table.insert(rdata, inlst)
					end
				end
			end
		end
	end
	return(rdata)
end

It's kinda cool, as this function flattens both the raw results from oLvosc.oscUnpackBundle() and the results from my bundleSearch() function. It's pretty simple to call resultsToList() after unpacking a bundle, and it appeals to me to let the end user decide if they can use the nested table.
There's also a bundleView() function that prints the nested table to the shell. It's output looks like:
BUNDLE:  ( Lev: 1  Time: [0:1] )
-MSG: /messageInBundle/1  ss  ( Lev: 1  Time: [0:1] )
 "abcdefghijklmnopqrstuvwxyz" "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
-MSG: /messageInBundle/2  fssi  ( Lev: 1  Time: [0:1] )
 65.688003540039 "RASAWASAWWWWW hello weird world" "The bundle 2" 55411
-MSG: /messageInBundle/3/msg1/added/again  ss  ( Lev: 1  Time: [0:1] )
 "abcdefghijklmnopqrstuvwxyz" "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
    BUNDLE:  ( Lev: 2  Time: [10:4294967292] )
    -MSG: /nestedBundle/n1  fssi  ( Lev: 2  Time: [10:4294967292] )
     65.688003540039 "RASAWASAWWWWW hello weird world" "The bundle 2" 55411
        BUNDLE:  ( Lev: 3  Time: [4:1] )
        -MSG: /deepBundle/n2  ss  ( Lev: 3  Time: [4:1] )
         "abcdefghijklmnopqrstuvwxyz" "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
-MSG: /messageInBundle/4  ss  ( Lev: 1  Time: [0:1] )
 "abcdefghijklmnopqrstuvwxyz" "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
    BUNDLE:  ( Lev: 2  Time: [4:1] )
    -MSG: /deepBundle/n2  ss  ( Lev: 2  Time: [4:1] )
     "abcdefghijklmnopqrstuvwxyz" "ABCDEFGHIJKLMNOPQRSTUVWXYZ"

Of course you could easily do this with the level information in a flattened list... :-)
(I wrote a very short c program using the liblo library to send the nested bundles packets for the testing, so I know it's a valid packet.)
I can certainly add resultsToList() to the oLvosc library, rather than just as an external example function.

Code for creating valid bundle packets, for the oLvosc library. Not fully tested, but seems to be working OK...
-- ++++++++++++++++++++++++++++++++++++++++++++++++++++
-- BUNDLE send code
-- create a new bundle db structure
function oLvosc.newBundle(time)
	local bundle = {}
	bundle.ID = 'bun'
	bundle.time = time
	bundle.elemNum = 0
	bundle.size = 16 -- base size of a bundle without elems
	bundle.elems = {}
	return (bundle)
end

-- add a message to a bundle db structure
function oLvosc.addMsgToBundle(bundle, msgPack)
	local message = {}
	bundle.elemNum = bundle.elemNum + 1
	message.ID = 'msg'
	message.body = msgPack
	message.size = string.len(msgPack)
	bundle.size = bundle.size + message.size + 4 -- add new msgs size to bundle
	table.insert(bundle.elems, message)
end

-- add a populated bundle to an existing bundle in a db struture
--	No additional data can be added to this bundle after this operation
function oLvosc.addBundleToBundle(bundle, addBundle)
	bundle.elemNum = bundle.elemNum + 1
	bundle.size = bundle.size + addBundle.size + 4 -- add new bundle size to bundle
	table.insert(bundle.elems, addBundle)
end

-- create a bundle packet from a bundle db structure
--	call with the populated bundle db structure as the single arg:
--	    local bpacket = oLvosc.oscBundlePack(myBundle_db)
--	returns a transmissible osc packet
-- send packet with standard oLvosc.sendOSC()
function oLvosc.oscBundlePack(bun_db, level)
	local lpacket = ''
	if level == nil then level = 1 end

	if type(bun_db) == 'table' and bun_db.ID == 'bun' then
		local tsec, tfrac = oLvosc.unpackTIME(bun_db.time)
		lpacket = lpacket..'#bundle\0'..bun_db.time
		for  _,be in ipairs(bun_db.elems) do
			if be.ID == 'bun' then
				lpacket = lpacket..oLvpk.pack('string', '>i4',be.size)
				lpacket = lpacket..oLvosc.oscBundlePack(be, level + 1)
			elseif be.ID == 'msg' then
				lpacket = lpacket..oLvpk.pack('string', '>i4',be.size)..be.body
			end
		end
	end
	return(lpacket)
end


[Provider]: LaunchDarkly OpenFeature provider for the Server-Side SDK for Java

SDK
Java
Vendor/Tool
LaunchDarkly
Maintainer
Vendor
Artifacts
No artifact links.
Repository
https://github.com/launchdarkly/openfeature-java-server
Anything else?
Should have a logo from the dotnet and node provider.

Thanks, I'll knock this out tomorrow.

All good now, thanks @kinyoklion!
https://docs.openfeature.dev/docs/reference/technologies/server/java

@beeme1mr Thank you!


The latest task does not display app name, as shown below:

The latest task does not display app name, as shown below:

@gionlime Thanks for filing the issue, Please make sure you have provided title to your MaterialApp
MaterialApp(
      title: 'Your_App_Name',
      home: DashboardPage(),
    );

if the issue still persists then please share the output of flutter doctor -v.
Thanks


Server Aliases added to /etc/hosts

Hi,
When Shieldoo connects with the client app, I noticed /etc/hosts is updated with hostnames and IP addresses of servers according to their "Name" as set in General tab when creating a new server which is great. It automatically takes care of the manual step involved with having to maintain a hosts file and distribute it across multiple hosts.
In the last step of provisioning a full ZTNA setup for non-public services, DNS resolution is usually a big challenge.
To overcome complicated DNS setups and configuration, I manually added aliases to the /etc/hosts file but perhaps "aliases" can be an added feature.
Feature Request:
Adding custom aliases to servers that would also be added to the /etc/hosts entry for that server.
E.g. adding the following aliases to a Docker server called hq-docker1 would results in a /etc/hosts entry
<IP_ADDRESS> hq-docker1 dockernode1 bitwarden.mydomain.com portainer.mydomain.com

After connecting with the Shieldoo client, one would be able to navigate to "portainer.mydomain.com" in a browser and it would resolve to <IP_ADDRESS>.
This would overcome complications of trying to provision, manage and set up a split-DNS server to devices connected to the mesh network or having to manually manage a hosts file.
Alternatives

Manually creating the extra aliases in /etc/hosts file.
E.g.
<IP_ADDRESS> dockernode1 bitwarden.mydomain.com portainer.mydomain.com
It works but it must be manually managed.
Changing the server's name to the portainer.mydomain.com
This works, but is limited to a single FQDN

Additional context
In certain cases, some servers host multiple services which is accessible only via a FQDN and routed by a reverse proxy or something like Apache having multiple vhosts.
With multiple aliases added to /etc/hosts, it would take care of a complicated DNS setup to resolve DNS via Shieldoo when connected via the client and resolve DNS using the OS DNS server when not connected.

Interesting idea, we will put it to our roadmap.

Oh, wow awesome thank you!
Tested it now and it works perfectly!
Accessing non-publicly exposed docker containers behind a firewall via FQDN with zero extra DNS configuration server or client side


Revisit kvstore spec

MVP is to be able to record immutable data on the blockchain.
~No ACL.~
~No delete.~

After discussing with the product team, the user story will change from As a developer, I want to record arbitrary data on-chain so that it is **immutable**. to being able to modify and delete an entry if permitted.


Config entry options

Description:
Options support for configuration entries.
Uses same data entry flow as config entries.
How to identify if a configuration entry supports options flow:
A static method in config flow class called "async_get_options_flow" returns options flow handler.
Finalizing options flow triggers an update to configuration entry that will also trigger update listeners.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

@balloob @MartinHjelmare I'd say this is ready for a proper bashing 👍

See https://github.com/home-assistant/home-assistant/pull/18929#discussion_r258649333.


Have releay less logs in the travis output

I have to copy the GeoServer example data, anyone know a way to don't do that?

You could get the WAR with Maven, unzip and then copy the data dir. Maybe this old revision of the gradle script might help: https://github.com/mapfish/mapfish-print/blob/5282d68841ab730b8b0c829ce686f65bd5c55d36/examples/build.gradle#L30

Finally ready :-)
about 2000 log line instance of largely more than 10000 :-)

Great, thanks!


Create unit test for HttpHandler class

Please start to create unit test and test cases for the HttpHandler class.

Work in progress.

Almost done.


[improve](clickhouse jdbc) support clickhouse array type

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
In this PR, I match the array type of ClickHouse to the array type of Doris's jdbc external.
Thanks to @zhangstar333 for helping
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[x] Has document been added or modified
[ ] Does it need to update dependencies
[x] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.58 seconds
stream load tsv:          477 seconds loaded<PHONE_NUMBER>9 Bytes, about 149 MB/s
stream load json:         25 seconds loaded<PHONE_NUMBER> Bytes, about 89 MB/s
stream load orc:          76 seconds loaded<PHONE_NUMBER> Bytes, about 13 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230322032133_clickbench_pr_118347.html


Naked macro cause previous code doesn't compile: Error: unexpected token

main.cr:
A = {a: 1, b: 2, c: 3}

v = {
  {% for n, v in A %}
  {{n}}: {{v}},
  {% end %}
}
puts v

compile result:
In main.cr:1:9

 1 | A = {a: 1, b: 2, c: 3}
             ^
Error: unexpected token: 1
exit status 1

Crystal version:
 crystal -v
Crystal 0.35.1 [5999ae29b] (2020-06-19)

LLVM: 8.0.0
Default target: x86_64-unknown-linux-gnu

The reason is that when you do {{n}} the location of that node is preserved, so the error points to the a or 1 that you had in the node you copy information from.
Maybe this wasn't a good change after all.
In any case, macro expressions need to be valid complete expressions. In your case you need to write this:
A = {a: 1, b: 2, c: 3}

v =
  {% begin %}
  {
    {% for n, v in A %}
    {{n}}: {{v}},
    {% end %}
  }
  {% end %}

puts v

@asterite
Thank u so much.
I guess putting the original macro lines into a macro method makes those things becoming complete expressions so they compile, am I right?
A = {a: 1, b: 2, c: 3}

macro make_tuple
  {
    {% for n, v in A %}
    {{n}}: {{v}},
    {% end %}
  }
end

puts make_tuple


List when admin or ownership permissions required for different endpoints

Context
Currently, a number of endpoints, such as /agencies PUT, require ADMIN permissions, but do not mention this. This will likely confuse users, and should be adjusted.
Requirements

Update endpoint documentation so that it is made clear which endpoints require admin permissions.
A simple way to do this might be to automatically have "Requires Elevated Permissions" appended to the description of the documentation, when the authentication information includes an admin permission

Tests

Tests should not need modified

Docs

Documentation will be updated.

Open questions

None at this time.

@maxachis yes, automation for this sounds right to me! Keeping track of them all and remembering to update the docs is asking for trouble.


Fix handling of projective transforms

When transforming points with projective matrices,
we must divide the resulting vector by w, in order
to get the normalized result.
This commit fixes graphene_matrix_transform_point,
graphene_matrix_transform_point3d and
graphene_matrix_transform_bounds. I suspect that
some of the remaining transform functions need
similar fixes, but they are not used in GTK, so
I couldn't verify that.
Before:

After:

Notice how the bounding box for the projectively transformed button is much too small before, and just right after.

I think the proper fix is to do the division in graphene_simd4x4f_point3_mul

Here is a gtk-side fix for now: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/2514

Given things like https://gitlab.gnome.org/GNOME/gtk/-/commit/30ab8b3ecac15e22436f6c38c1149e54f42f0d50 it seems existing code relies on the current behaviour. So, is it safe to change this?


Querying for elements with 'position: fixed' stops working in 4.2

From<EMAIL_ADDRESS>on September 05, 2013 07:31:08
What steps will reproduce the problem? 1. Display WebView containing some header element fixed at the top of the view (position: fixed)
2. Call waitForWebElement(By.cssSelector(...)), getCurrentWebElements(By.cssSelector(...)), or clickOnWebElement(By.cssSelector(...)) with appropriate selector for header element What is the expected output? What do you see instead? The calls should succeed or return a WebElement (more than one, really... but saw the other issue). Instead, waitFor... returns false, click throws an assertionFailed, and getCurrentWeb... returns an empty list. What version of the product are you using? On what operating system? 4.2, Mac OS X 10.8, Android 4.1.2 Please provide any additional information below. In version 4.0, the getCurrent... call returns an element and the click call succeeds. the waitFor has no return value.
querying for other elements on the page that outside the 'position: fixed' header work as well. It seems to be exclusive to the fixed content insofar as my testing showed.
Original issue: http://code.google.com/p/robotium/issues/detail?id=512

From<EMAIL_ADDRESS>on September 05, 2013 20:55:27
Thanks for reporting this. Would it be possible for you to share the apk and test project?
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on September 08, 2013 22:14:36
Please test with Robotium 4.3.
Status: Verified

From<EMAIL_ADDRESS>on September 10, 2013 06:45:47
waitForWebElement(), clickOnWebElement(), and getCurrentWebElements() still don't seem to be working properly with the 'position: fixed' elements.... at least on Android 4.1.2. I haven't yet tried on other platforms.

From<EMAIL_ADDRESS>on September 24, 2013 09:06:45
Hi,
I left a comment on the google code page for this issue, but I guess that
doesn't alert you that I'm still having a problem with this one.
I also left a comment for issue #514 that was rejected. The click should
succeed once the web element scrolls into view, but it doesn't. That's the
bug. Otherwise, don't bother spending time scrolling.

From<EMAIL_ADDRESS>on September 24, 2013 23:03:28
Hi,
Would it be possible for you to send me an apk that exhibits this issue?
/Renas


Upload independent assets through transfer service

Describe the solution you'd like
We'd like to upload information which are associated with certain mice to code ocean, but aren't sessions in the way the the transfer service expects. Specifically, we have the anatomy information with all the CCF coordinates. This is usually done weeks after the ecephys and behavior is ready for upload. So we'd like to be able to generate a new asset that just contains this information for this mouse, separate from the ecephys session asset that would have been uploaded much earlier.
Describe alternatives you've considered
If its easy to just use the code ocean api to do this on my own, then perhaps this isn't needed. It's not clear to me what considerations should be made to determine if this is a good use case for the transfer service. It would be very nice to have this triggerable the same way though.

@saskiad @dbirman Saskia mentioned that we should do this by modifying the metadata, which can be done without creating a new data asset.

Also, it seems like CCF will soon be done entirely on code ocean, so we won't have this need much longer. Might be worth considering for other session-related data though.
How would the metadata be modified to contain other non-metadata?

I want to make sure I understand: the CCF information you are adding is CCF coordinates for the recorded units in a session? Or is it something else?
This needs to live within the same asset as the ecephys. I don't think it should be a separate asset, distinct from the ecephys data it's part of. That would be a new derived asset.
raw ecephys -> sorted ecephys -> sorted/aligned ecephys
Or is this something else altogether?

You're correct that this is the CCF coordinates for recorded units. The issue is that the OPT anatomy processing often isn't completed until several weeks after the ecephys is uploaded and processed. So we've been uploading a 'CCF' asset for the session separately.
We could reupload the information together with the CCF files as well I suppose. It would automatically rerun the spikesorting and eye tracking which might be redundant/wasteful (and create an asset that is mostly redundant)

@rcpeene this isn't a situation where the transfer service should be used for the reasons you mentioned. We should either create a derived asset in CO or push this to the metadata.
But I have a confusion here because I didn't think units were stored in the metadata. To quickly summarize, the situation is that after histology for a session is processed an alignment is run and the histology-aligned CCF coordinates are now available and we want to put this information with the sorting results. My confusion is that I didn't think unit locations were stored in the metadata at all.
@saskiad can you confirm what behavior here is correct? Should (1) a new derived asset be created with histology-aligned coordinates added as a file, or (2) we upload the coordinates via the aind-data-access-api and change something in the metadata.

You are correct that units are not stored in the metadata. This should result in a new derived asset that has the aligned coordinates.
It doesn't require re-running the spikesorting/eyetracking, it should only require adding the coordinates into the units table in the NWB file.

@rcpeene Does that all make sense? I'll close this issue with your confirmation.

Makes sense to me